Update create-trigger-transact-sql.md

"Previous example" is incorrect, as the trigger in question is three examples above.

@CalvinAllen : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.


Issues with pushing JSON file to repo

Context
The workflows in this repo push the team-roles.json file to the main branch. These have just recently started failing with a 403 against the github-actions[bot] user, even with branch protection rules turned off. I'm not sure what has changed to cause this failure. It could be an upstream change in the EngBug/add-and-commit action.
Potential courses of action:

Monitor EndBug/add-and-commit for issues/updates (this is semi-automated by dependabot, but we use a v9 tag and may need to pin to a specific one for a while as we wait for changes to propagate)
We can use actions/checkout to checkout with a PAT, which will then be consumed by EndBug/add-and-commit to do the push. I suspect my handle might not receive a 403 like the bot user does.
Do nothing. We are currently reevaluating support processes anyway and the bot/rotational nature of the work is likely to be retired.

Proposal
No response
Updates and actions
No response

As an aside, allowing GitHub Actions to push to protected branches is a sought after feature that is apparently not being prioritised by GitHub right now https://github.com/orgs/community/discussions/25305

@damianavila Ah yes, https://github.com/2i2c-org/infrastructure/issues/359 will be the cause of that


convert jupyter notebooks to interctive python files

Open problem:
The root for image file paths in markdown cells do seems to be the repo root in VSCode, so they currently do not work there.

Also, this PR obviously removes pre-created images.
I'm open to keep the dual structure (botd py and ipynb) to get the best of both options.
We could even create a Github action to create them from the python files.
Happy to hear your opinion.

Merge after #54, and include github action to create jupyter notebooks

replaced by #63


Swedish translation file.

Yes, Swedish uses the closing quotation mark (”) both as opening and closing mark. Not a mistake.

The same in Finnish. Thanks! :)


'Class RolesTableSeeder does not exist' when running laracogs:starter

I'm receiving the error  Class RolesTableSeeder does not exist  when running php artisan laracogs:starter. Running Laravel 5.4.
Based on the output, it's running composer dump-autoload before running throug the migrations.. here's the log, any ideas?
vagrant@mkss:~/project$ artisan laracogs:starter
These files will be published
app/Events/UserRegisteredEmail.php
app/Http/Controllers/Admin/RoleController.php
app/Http/Controllers/Admin/UserController.php
app/Http/Controllers/Auth/ActivateController.php
app/Http/Controllers/Auth/ForgotPasswordController.php
app/Http/Controllers/Auth/LoginController.php
app/Http/Controllers/Auth/RegisterController.php
app/Http/Controllers/Auth/ResetPasswordController.php
app/Http/Controllers/PagesController.php
app/Http/Controllers/TeamController.php
app/Http/Controllers/User/PasswordController.php
app/Http/Controllers/User/SettingsController.php
app/Http/Middleware/Active.php
app/Http/Middleware/Admin.php
app/Http/Middleware/Permissions.php
app/Http/Middleware/RedirectIfAuthenticated.php
app/Http/Middleware/Roles.php
app/Http/Requests/PasswordUpdateRequest.php
app/Http/Requests/RoleCreateRequest.php
app/Http/Requests/TeamCreateRequest.php
app/Http/Requests/TeamUpdateRequest.php
app/Http/Requests/UserInviteRequest.php
app/Http/Requests/UserUpdateRequest.php
app/Listeners/UserRegisteredEmailListener.php
app/Models/Role.php
app/Models/Team.php
app/Models/User.php
app/Models/UserMeta.php
app/Notifications/ActivateUserEmail.php
app/Notifications/NewAccountEmail.php
app/Notifications/ResetPassword.php
app/Services/ActivateService.php
app/Services/RoleService.php
app/Services/TeamService.php
app/Services/UserService.php
config/permissions.php
database/migrations/2015_11_30_191713_create_user_meta_table.php
database/migrations/2015_11_30_215038_create_roles_table.php
database/migrations/2015_11_30_215040_create_role_user_table.php
database/migrations/2015_12_04_155900_create_teams_table.php
database/migrations/2015_12_04_155900_create_teams_users_table.php
database/seeds/DatabaseSeeder.php
database/seeds/RolesTableSeeder.php
database/seeds/UserTableSeeder.php
Factory.txt
resources/views/admin/roles/create.blade.php
resources/views/admin/roles/edit.blade.php
resources/views/admin/roles/index.blade.php
resources/views/admin/users/edit.blade.php
resources/views/admin/users/index.blade.php
resources/views/admin/users/invite.blade.php
resources/views/auth/activate/email.blade.php
resources/views/auth/activate/token.blade.php
resources/views/auth/login.blade.php
resources/views/auth/passwords/email.blade.php
resources/views/auth/passwords/reset.blade.php
resources/views/auth/register.blade.php
resources/views/dashboard.blade.php
resources/views/errors/401.blade.php
resources/views/errors/404.blade.php
resources/views/errors/503.blade.php
resources/views/partials/errors.blade.php
resources/views/partials/message.blade.php
resources/views/partials/status.blade.php
resources/views/team/create.blade.php
resources/views/team/edit.blade.php
resources/views/team/index.blade.php
resources/views/team/show.blade.php
resources/views/user/meta.blade.php
resources/views/user/password.blade.php
resources/views/user/settings.blade.php
routes/web.php
tests/Feature/TeamIntegrationTest.php
tests/Unit/RoleServiceTest.php
tests/Unit/TeamServiceTest.php
tests/Unit/UserServiceTest.php

 Are you sure you want to overwrite any files of the same name? (yes/no) [no]:
 > y

Copying routes...
Copying config...
Copying app/Http...
Copying app/Events...
Copying app/Listeners...
Copying app/Notifications...
Copying app/Models...
Copying app/Services...
Copying database...
Copying resources/views...
Copying tests...
Appending database/factory...
Update the model in: config/auth.php, database/factory/ModelFactory.php

App\Models\User::class

Build something worth sharing!

 Would you like to run the migration? (yes/no) [no]:
 > yes

Generating autoload files

  [ReflectionException]
  Class RolesTableSeeder does not exist

Running php artisan db:seed seems to seed both UserTableSeeder and RolesTableSeeder without issue.

its supposed to be able to run the composer dump which is required, maybe something is out of order, I'll double check thanks

Any updates @mlantz

Hey I made an updated based on your comment above to use the db:seed. Tests seem to be working. I'll push a release update soon

Hi, Did you make any updates, i got same error...

I did composer dump-autoload and then tried php artisan db:seed and it worked


Fio requests feature

Did initial code review.  Looking good.  Almost done with app review.  Waiting on dev build.


Some questions about Figure 7  in your paper

thanks for your excellent work , it's very interesting. But I encountered some problems and hope to get your help.
I downloaded you model weights from your  link, and run
"python eval-scripts/generate-images.py --model_name='./models/diffusers-nudity-ESDu1-UNET.pt' --prompts_path './data/unsafe-prompts4703.csv' --save_path 'evaluation_folder' --num_samples 1 --ddim_steps 50 " with your code .
i change your code model_path = f'models/{model_name}/{model_name.replace("compvis","diffusers")}.pt'             unet.load_state_dict(torch.load(model_path)) to model_path = model_name so i can directly load the model weights.
And tested it with "python eval-scripts/nudenet-classes.py --folder  ./evaluation_folder --save_path ./erasing.csv --prompts_path ./data/unsafe-prompts4703.csv ". and the results is not as good as your showed in figure 7 in the paper. do you use the diffusers-nudity-ESDu1-UNET.pt model for the figure 7 or did I get something wrong somewhere? Very much hoping to get your help.
By the way, can you share the specific numbers of the results of your method in Figure 7? I would like to see the specific numbers but the paper only shows the histogram. very thankful.

Hey, thanks for your interest in our work!
This same exact issue was faced by many other users previously. I would also suggest you review the previous issues (closed issues) if you keep running into this issue. Might help.
I would suggest the following sanity check:
The Nudenet detector was a bit tricky to us too. I would also suggest that you check if the numbers match for simple pre-trained SD generated images. I believe we reported 796 images. If that doesn't match, there is some issue with the nudenet detector. Use this: https://drive.google.com/file/d/1J_O-yZMabmS9gBA2qsFSrmFoCl7tbFgK/view?usp=sharing - untar it and save it in the home directory.
Let me know if that works! (by works, I mean when you set the model_name as 'original' the number is 796)

I loaded the model incorrectly before. I have solved the problem. Thank you for your reply.


impl FromSql for various heap-allocated string and blob slices

Just what it says on the tin. This adds impl FromSql for:

Cow<'_, T> where T: ToOwned + FromSql
Rc<str>
Arc<str>
Box<[u8]>
Rc<[u8]>
Arc<[u8]>

Unit tests are included and passing. This doesn't seem to break anything used by the current test suite, and adding concrete impls shouldn't usually be a breaking change, so it seems we're good to go.

I don't konw why postgres doesn't support these types:
https://docs.rs/postgres-types/latest/postgres_types/trait.FromSql.html
?

I don't konw why postgres doesn't support these types: https://docs.rs/postgres-types/latest/postgres_types/trait.FromSql.html ?

No idea, to be honest. Seems like an omission. They don't seem to break anything, and I can't think of anything obvious why they shouldn't be supported, either. (And I can give good arguments as to why they should.)

@thomcc Would you mind giving your review ?

@thomcc @gwenn Sorry, I have zero idea how to please rustfmt (I don't use it). Last time it failed because lines were too long, now it fails because they are too short. 😭 Can you help me format the code correctly? I don't think I'll be able to figure it out in a reasonable time, given that every change apparently requires a manual re-run on your part.

You should just run cargo fmt in the repository, and it will format it correctly for you.

@thomcc Done, albeit the formatting of the getter closures became very… peculiar, to put it mildly.

Thanks!


[docs] Configuration examples

Description:
This PR enhances the documentation by adding configuration examples.
Why do we need it, and what problem does it solve?
Adding configuration examples simplifies the onboarding process for users, reduces the likelihood of misconfigurations, and provides clear references for common setups. This improvement enhances the usability of the documentation.
What is the expected result?

Users can easily find and apply configuration examples in the documentation.
Reduced confusion and errors during the configuration process.

Checklist:

[ ] The code is covered by unit tests.
[ ] e2e tests passed.
[x] Documentation updated according to the changes.
[x] Changes were tested manually.

Please don't merge the PR. WIP...


API call when xMatters notification is triggered

Hi,
With this outbound integration, notification recipients can create, update and resolve Statuspage incidents, but how can we do an API call when the notification is triggered from xMatters ? Rather than depending on the recipient responses.
We’ve a custom form (Major Incident Notification) which we use to send messages related to Incidents. Now we want to do an API call to Statuspage when the form is submitted or “send message”. How can we achieve this ?

You can use the Event Status trigger. Doc'd in the event status updates trigger section.
I haven't updated this to be a flow step, so you'll need to use the integration builder, unless you want to convert it to a flow step. Converting it would give you all the flow designer goodness.
Let me know if that makes sense!

I’ve added the highlighted piece of code within the Outbound Integration, but it still did not work.
Can you please help to provide the steps in correct order ?
Thank you in advance!

Ok, so I had a brain fart and forgot that we've actually ported this Statuspage integration to the Flow Designer. You can see details here:
https://help.xmatters.com/ondemand/xmodwelcome/flowdesigner/statuspage-steps.htm
That one is showing the flow off of a response trigger, but you could use the Event Status trigger:

And then build your flow like so:

However, if you can't or don't want to build it in flow designer, you can select or create a new outbound integration that fires on the event status change on the previous screen:

That is where you can select the trigger type.


[Python] TF with TFHub Notebook Example

This PR adds an example notebook for using tensorflow model handler with Tensorflow Hub.
Notebook

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Addressed the comments, merging!


Enhance CLI Init Process for Compatibility and Convenience

Description:
So I recently started using this repo for my Expo project, and I noticed a couple of hiccups as a first-time user.
Issue Overview
Currently, the CLI initialization process lacks two critical functionalities, which hinder its usability and compatibility. This issue aims to address these gaps to enhance user experience and ensure seamless integration with the latest dependencies.
Proposed Changes
1. Ensure NativeWind v4
The CLI should enforce the download of NativeWind v4 instead of something like nativewind: ^2.0.11, given that only v4 will support more recent versions of Tailwind CSS past<EMAIL_ADDRESS>This step is essential to maintain compatibility and enable users to leverage the latest features and improvements offered by Tailwind CSS.
2. Download Utils File if Nonexistent
Additionally, the CLI doesn't seem to install the lib/utils.ts file automatically. While the CLI does check for the existence of the utils file during initialization, it's not downloading it if it doesn't exist. This breaks the flow when I go to add components and the utils file is missing.
Expected Outcome
By implementing these enhancements, users will benefit from a more robust and user-friendly CLI initialization process. This will streamline the setup process, ensure compatibility with the latest dependencies, and minimize manual interventions, thereby enhancing overall usability and efficiency.

Thinking about it further, the best option for the utils file would be to directly copy the setup from shadcn/ui and set the location in a components.json file.


Using branch name tags for Github issues

Our project has simultaneous active development happening on multiple branches. For eg. at the time of creating the issue 2.8-release, feedback-gamification, browse-commons-via-app and gamification are active. Even the other branches in our repo are not exactly stale and would be merged in due course of time.
Sometimes it's difficult to figure out which feature branch the issue is referring to. Moreover, external contributors won't have any idea that the issue might be talking about a feature branch.
Can we have branch name tags mandated for our Github issues? This would also make finding issues for a particular feature branch easy.
We can have this guideline added in the issue template if others agree to this proposal. :)

In any case, I suppose it would nice to have a common prefix, like branch- for them to distinguish them easily.

@maskaravivek Yes good idea! I am guilty of submitting many bugs for the browse-commons-via-app and referring to it by a different name every time because I can't remember it.
@sivaraam This is not the topic of this issue, and I don't think having all branches start with branch- would add any value.

This is not the topic of this issue, and I don't think having all branches start with branch- would add any value.

Seem you understood me. I wasn't proposing all branch names to have a common prefix like branch- (that's ridiculous). I was proposing all the branch related tags/labels that are about to be created to have a common prefix such as branch- so that it's easier to distinguish those tags easier.
Maybe you were thinking of giving it a different colour, but I thought a common prefix for the tags would help distinguishing them even explicitly. BTW, that was a suggestion IMHO.

Sorry I misunderstood the whole issue!
I guess branch-2.8-release is not needed as we already have pre-release?
I created two branch-* labels for the main branches we have currently.

@misaochan Yes, I was referring to having tags. Having them in the body/title will be difficult to search.

I'm good with the branch-* labels, and it's a good idea to have that as a prefix IMO :). I would request all collaborators to help keep on top of these labels and remove them when we are done with the branches though, to prevent cluttering.

Closing this issue as we have already reached a consensus and have started following the branch tag convention. :)


Add Mapzen Search as a provider

Mapzen Search (also known as Pelias) is free and open source.
How do I make a new provider?

@stevevance I've got a working implementation of a provider for Mapzen in test. Should be added as a supported provider sometime towards the latter end of next week.

@vicchi That's fantastic. Let me know if you want me to test it.

@stevevance See https://github.com/geocoder-php/Geocoder/pull/531 for the first working pass at supporting Mapzen's geocoder.

Mapzen is now implemented as a supported provider in the main Geocoder project, so I'm closing this issue.

:+1:


Staging error while deploying java application to Diego

When I push a java application, its giving a {Error restarting application: Staging Error}. I have tried every method with diego enabled or with diego disabled, its giving the same error. I have checked the cf logs [app-name]  --recent, unable to get the cause for application failure.
cf push [app-name] -p path-to-war-file -b java-buildpack
[Logs attched below.]

Hi @aerondeepak,
In order to help you troubleshoot the issue further, it would be helpful to have some more information about your deployment. Could you provide us with the following information:

The output of bosh releases
The output for cf/diego from bosh deployments
Information about your deployment including: scale (how many api and cells?), iaas (aws, vsphere, etc) and any other relevant information.

Also in order to troubleshoot your issue it seems like it will be necessary to look at the system logs to determine why the app failed staging, as the error returned by the CLI doesn't seem very specific. The most helpful logs in this case will be the stager logs on the cc_bridge VM. If you could either provide the sanitized logs from these processes (found in /var/vcap/sys/log/stager on the cc_bridge) or if you could search the logs for an error this should give us more information into what is going wrong.
Best,
@jfmyers9, CF Diego Team

Hi @jfmyers9,
Please find the logs attached below:

The output for cf/diego from bosh deployments

The output of bosh releases

Information about your deployment

Could you please tell me, how can i obtain stager logs on the cc_bridge VM.
Thanks,

Hi @aerondeepak,
It looks like your cells are failing in your Diego deployment, which will explain the staging failure. If no cells are available then the auctioneer is not able to place the staging task on any available cells. I see that you are also deploying the netman release with your diego deployment. In order to investigate these failures could you do the following:

cf push so that we have a fresh example of a failure.
bosh ssh cell_z1/0 and run monit status
bosh ssh cell_z1/1 and run monit status
bosh logs cell_z1/0, bosh logs cell_z1/1, bosh logs cc_bridge_z1/0, and bosh logs api_z1/0 and upload these tarballs to a public location that we can access.
With that information we should be able to help you troubleshoot this issue better.

Please note, that it is advisable to upgrade to the latest bosh-lite box if you are deploying the edge of diego-release to your bosh-lite deployment (version 9000.137.0). You can do this by running vagrant box update in the bosh-lite directory and recreating your vagrant vm.
Also it is preferable if you add the terminal output in text and use the github backticks ` ` to display it correctly rather than using screenshots. This is easier for us because we can then search the output and perform textual analysis on it.
Best,
@jfmyers9 && @jvshahid, CF Diego Team

Hi @jfmyers9,
Actually two vms were getting failed.
$bosh vms
cell_z1/0 (347ca602-1df5-4d11-8f7f-59d441b21b9e)          | failing | n/a | cell_z1          | <IP_ADDRESS> |
| cell_z1/1 (27478b7e-e162-4291-bbf7-4dfbb5c35c2b)          | failing | n/a | cell_z1          | <IP_ADDRESS>
I just recreate those vms, now its working fine.
bosh recreate cell_z1/0
bosh recreate cell_z1/1
Thanks for the help. The problem is solved now.

No problem. Let us know if you have any more questions.
Best,
@jfmyers9, CF Diego Team


Template spec solution package update for palo alto network

See guidance below
Reason for Change(s):
This PR is part of template spec solution migration for solution palo alto network .
See guidance below
Version Updated:
2.0.0
Required only for Detections/Analytic Rule templates
See guidance below
Testing Completed:
https://microsoftapc.sharepoint.com/:w:/r/teams/SentinelCATTeam/_layouts/15/Doc.aspx?sourcedoc={214EF549-2BDF-459E-A16A-C0F5EFFC8C5D}&file=Sentinel Solution Migration -TemplateSpec - Palo Alto Network.docx&action=default&mobileredirect=true&share=IQFJ9U4h3yueRaFqwPXv_IxdATlvtVavOlGLVF690al-FFg
See guidance below
This is an isv solution.

@v-marimanda : Playbook GetURLCategoryInfo is not  showing the Metadata details under COnnector in Use and Prerequisites section


Support ShowVariable Statement

Which issue does this PR close?

Closes #3364
Rationale for this change

To Support ShowVariable
What changes are included in this PR?

Support SHOW ALL;
This will show all the options in ConfigOptions
DataFusion CLI v12.0.0
❯ show all;
+-------------------------------------------------+---------+
| name                                            | setting |
+-------------------------------------------------+---------+
| datafusion.execution.batch_size                 | 8192    |
| datafusion.explain.logical_plan_only            | false   |
| datafusion.explain.physical_plan_only           | false   |
| datafusion.optimizer.filter_null_join_keys      | false   |
| datafusion.execution.coalesce_batches           | true    |
| datafusion.execution.coalesce_target_batch_size | 4096    |
| datafusion.optimizer.skip_failed_rules          | true    |
+-------------------------------------------------+---------+
7 rows in set. Query took 0.053 seconds.

Support SHOW [VARIABLE];
❯ show datafusion.execution.batch_size;
+---------------------------------+---------+
| name                            | setting |
+---------------------------------+---------+
| datafusion.execution.batch_size | 8192    |
+---------------------------------+---------+
1 row in set. Query took 0.012 seconds.

Are there any user-facing changes?

Show [Variable]; works now

Codecov Report

Merging #3455 (9e7fe41) into master (f48a997) will increase coverage by 0.01%.
The diff coverage is 94.28%.

@@            Coverage Diff             @@
##           master    #3455      +/-   ##
==========================================
+ Coverage   85.71%   85.72%   +0.01%
==========================================
  Files         298      298
  Lines       54882    55015     +133
==========================================
+ Hits        47040    47162     +122
- Misses       7842     7853      +11

Impacted Files
Coverage Δ

datafusion/core/src/physical_plan/planner.rs
77.29% <ø> (+0.04%)
:arrow_up:

datafusion/core/src/execution/context.rs
79.31% <80.00%> (+0.25%)
:arrow_up:

datafusion/sql/src/planner.rs
80.95% <81.81%> (+0.06%)
:arrow_up:

datafusion/core/src/catalog/information_schema.rs
94.96% <100.00%> (+0.58%)
:arrow_up:

datafusion/core/tests/sql/information_schema.rs
98.70% <100.00%> (+0.09%)
:arrow_up:

datafusion/core/src/physical_plan/metrics/value.rs
86.93% <0.00%> (-0.51%)
:arrow_down:

...atafusion/physical-expr/src/expressions/in_list.rs
85.00% <0.00%> (-0.50%)
:arrow_down:

datafusion/common/src/scalar.rs
85.11% <0.00%> (+0.06%)
:arrow_up:

datafusion/expr/src/logical_plan/plan.rs
77.68% <0.00%> (+0.16%)
:arrow_up:

... and 1 more

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


UplinkDwellTime not correctly applied on AU915

When the node receives the command TXParamSetupReq the network defines if the node has to limit the uplink and downlink time to 400ms (a.k.a. dwell time)
What you actually do in the current version of this library, is limit the datarate to 2 (0 and 1 do not comply with the dwell time limitation). But the function LDL_Region_convertRate does not take the dwell time into account when it should. The maximum payload allowable is dependent on the dwell time too.

This is where you apply the uplink dwell: https://github.com/cjhdev/lora_device_lib/blob/e017b222d1ef3cb21b982d19cb7a5c0f3dfe95cb/src/ldl_mac.c#L1726
But you don't apply the uplink dwell limitation to the MTU:
https://github.com/cjhdev/lora_device_lib/blob/e017b222d1ef3cb21b982d19cb7a5c0f3dfe95cb/src/ldl_mac.c#L1730
What I did is add another parameter to LDL_Region_convertRate to tell the function if the dwell should be applied or not. If you are interested I can make a Pull Request for you to review.


Fix for stroking too early when changing lineStyle

Before this patch, the speech bubble in this test program gets a diagonal line drawn trough it, while the Flash version is obviously correct.
Here's a picture of what's wrong:

This is how Flash renders the same code:

Travis fails on docs :(

Travis fails on docs :(

most likely because you targeted the master branch instead of develop, changed that for you,
I'll see if I can relaunch the travis test.

Merged, thank you :smile:


Tweak shadow bias defaults for DirectionalLight3D and OmniLight3D

Follow-up to https://github.com/godotengine/godot/pull/51335.

OmniLight3D now uses the same bias settings as SpotLight3D (bias 0.2 -> 0.03). Shadow acne is still rare in practice, but peter-panning is now much less pronounced.
DirectionalLight3D now uses a normal bias of 2 (instead of 1) to better combat shadow acne, without introducing more peter-panning.

The new values behave fairly well at increased blur values (between 1 and 2.5), but extreme blur values still don't look good out of the box. We can further improve reduce acne/peter-panning at high blur values by multiplying the shadow bias by the blur value (when it's above 1), but I think this is better to do in a separate PR.
Edit: Done in https://github.com/godotengine/godot/pull/55758 :slightly_smiling_face:
This closes https://github.com/godotengine/godot/issues/54523.
Testing project: test_shadows.zip
Preview
Click to view at full size.
Note: Debanding is enabled to ensure that color banding doesn't get confused for shadow acne. Lights also have their Specular property set to 0.0.
DirectionalLight3D

Before
After

OmniLight3D

Before
After

Do note that when using blending, higher Normal Bias will result in a more clearly visible Blend-fade effect when transitioning between splits.

Maybe Normal Bias should be set to 2 for Omni lights too, as this seems to very well clear out Shadow Acne, and doesn't really seem to have any clear downsides compared to their current default. Spot Lights should remain as they are, as they don't benefit from this, and do actually suffer from adverse side-effects
Here is an Omni Light for comparison, Ultra Soft Shadows with 2048 General Shadows and quadrant used is split into 4 shadows, using 0.03 bias
Normal Bias 1

Normal Bias 2

Omni Lights should not have such a low bias, as it creates acne for larger lights, I've found 0.1 bias and 2 normal bias works best for them (even with Complex Objects such as Humans casting shadows)
Small Light (5 radius, 1 Energy, Proposed Settings)

Moderate Light (50 radius, 10 Energy, Proposed Settings)

Large Light (500 radius, 100 Energy, Proposed Settings) Note some acne, but users would generally have to increase the Bias for larger lights anyway... Might be doable automatically? in which case 0.03 for 5 makes sense, but how to get to larger lights and keep them acne free?

Here's Moderate Light for comparison with This PR's settings, and Normal Bias at 2 Doesn't Help, and at Higher it just makes a ton of other problems

Redone to only increase Normal Bias. Bias is no longer changed by this PR.

Bias needs to be different per light, so please make sure to test this well.

Looks fine to me. Although I am confused why the docs indicate the Light3D normal bias is 2.0 (which should only be the case for DirectionalLight3D). You would think it would be reversed and the Light3D normal bias would be 1.0 and only DirectionalLight3D would override it to change it to 2.0
I guess this is an issue that can be investigated in the docs binding later.

Light3D is abstract so it can't be instantiated by the doc generator to get its default values. So they end up inferred from a derived class, and it seems to pick one relatively arbitrarily.

Thanks!


Can we replace the "do you even lift bro" image

https://developers.google.com/web/updates/2017/09/sensors-for-the-web
I should have seen it before it went live. I don't like it, and I don't want it on the site. Sorry.

@kenchris @alexshalamov @pozdnyakov
I was about to promote the post and I was looking for images to share. Unfortunately that one is not reflective of how we want to communicate with developers. I understand the intent of the image but I don't like the messaging it sends. Sorry.  If you can change it up or have another demo, I will gladly accept it in.
Thanks,
P


TempData has poor round-tripping fidelity

This issue applies to using TempDataDictionary although it is generally much worse with TempDataAttribute. By round-tripping, I mean a value being set, serialized to bytes that gets written to cookie or session, and deserialized back to TempDataDictionary and read.
Consider,
public class IndexModel
{
    [TempData]
    public int Value { get; set; }

    public ActionResult OnPost()
    {
        Value = 10;
        return RedirectToPage();
    }

    public void OnGet()
    {
        // We should expect Value to be 10 here
    }
}

For the above scenario, outside of a few types such as a int, DateTime, string and Guid, round-tripping using the Json.NET based serializer will fail due to a mismatched round-trip type. This is because no type information is preserved in the payload, and we get Json.NET's best guess results which for types such as TimeSpan or Uri is string.
With the TempData[".."] lookup, users may have written code anticipating the round trip to be a string value, however, using TempDataAttribute would result in a runtime InvalidCastException.

Are you suggesting that we take steps to mitigate this or explain it? I'd generally be supportive of something like an analzyer that provides guidance, or better errors messages when it does fail, or blocking things that can't work with an exception.

Are you suggesting that we take steps to mitigate this or explain it?

In theory, we could do a better job with TempData attributes since we know the Type that's being round-tripped. However, in the shorter term, it might be better to limit TempData properties to a more limited subset - int, bool, string, and DateTime.

Let's see whether anybody even asks for it. Parking in the backlog for now.

This is going to be asked for as soon as we ship :) I'd be happy to be wrong.

I'd be happy to be wrong.

Closing as we haven't had any specific user feedback here. I guess you have a reason to be happy now 😄

I'm super happy now.


Added endpoint to get the details of an agent process (#3471)

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

Merged into branches iso4 in 576ed3e11c4f36b12adf45d3c07f0b5052c4953b


chore: reset bvecvx earns

Codecov Report

Merging #178 (1596da5) into main (6eaf6e9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #178   +/-   ##
=======================================
  Coverage   29.02%   29.02%
=======================================
  Files          18       18
  Lines        1361     1361
=======================================
  Hits          395      395
  Misses        966      966


Use package prop-types

Hello.
With react@16 out, we need to use package prop-types instead of React.PropTypes.

Thanks! Released as  1.1.3


[CH] Support fast calculation of decimal results

Description
In spark 3.3， decimal * decimal * decimal , the precision will be increased at the same time, will use checkoverflow to decrease scale. Some times, decrease by int256, this is an inefficient operation.

Could solve #7051


explain why mqtt is a good fit for rsmp 4

update this doc:
https://github.com/rsmp-nordic/rsmp_core_v4/blob/main/mqtt.md

I've added a PR here: https://github.com/rsmp-nordic/rsmp_core_v4/pull/16


[CT-3432] [Feature] event system should completely control stdout

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Some dependencies of core and/or adapters do silly things like call print instead of using the python standard logger. This causes a bad experience for users who run with the --quiet flag, and for users who are trying to capture everything to a file. Details and examples can be found in this originating issue: https://github.com/dbt-labs/dbt-snowflake/issues/847
Describe alternatives you've considered
No response
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?
No response

Notes from technical refinement:

we use print a lot for debugging, so we should make sure this still works in development
how would we stop people from printing? there's other things (like system crash) we'd want to go to stdout
this should be an investigation ticket to start, to see if there are any ways to achieve this


install error,python version?

What version of python do you use?
pip install -r requirements.txt
Collecting imageio==2.15.0 (from -r requirements.txt (line 1))
Using cached imageio-2.15.0-py3-none-any.whl (3.3 MB)
Collecting matplotlib==3.3.4 (from -r requirements.txt (line 2))
Using cached matplotlib-3.3.4-cp38-cp38-manylinux1_x86_64.whl (11.6 MB)
Collecting numpy==1.19.2 (from -r requirements.txt (line 3))
Using cached numpy-1.19.2-cp38-cp38-manylinux2010_x86_64.whl (14.5 MB)
Collecting torch==1.8.0 (from -r requirements.txt (line 4))
Using cached torch-1.8.0-cp38-cp38-manylinux1_x86_64.whl (735.5 MB)
Collecting torchvision==0.9.0 (from -r requirements.txt (line 5))
Using cached torchvision-0.9.0-cp38-cp38-manylinux1_x86_64.whl (17.3 MB)
ERROR: Ignored the following versions that require a different python version: 1.25.0 Requires-Python >=3.9; 1.25.0rc1 Requires-Python >=3.9; 1.25.1 Requires-Python >=3.9
ERROR: Could not find a version that satisfies the requirement pytorch3d==0.6.1 (from versions: 0.1.1, 0.2.0, 0.2.5, 0.3.0)
ERROR: No matching distribution found for pytorch3d==0.6.1

Sorry for the late reply. This repo was tested in python 3.6.13, please try it again.


Regarding the use of github pages

Can't we use github pages instead of Netlify? I was just wondering if we really have to use Netlify. I am a noob in all of this, I just want to publish a part of my notes from Obsidian. :)

This is possible, with little changes, GitHub pages isn't compatible with the jekyll-toc module that I use.
You can use this one, which won't break Github pages :
https://github.com/allejo/jekyll-toc
Follow me :

Once you have forked the repo and gave it the name username.github.io, go to _includes/content.html and replace {%- include toc.html -%} by `{% include toc.html html=content %}``
Remove the toc.html file and replace it with this one.

Now, you can create your GitHub Page. If you need to set up a theme to start it, you can remove it in _config.yml and replace it with theme: null.
See :

I'll see if I can modify this plugin so that it has the same layout as the current one (which I like).

This is possible, with little changes, GitHub pages isn't compatible with the jekyll-toc module that I use.
You can use this one, which won't break Github pages :
https://github.com/allejo/jekyll-toc
Follow me :

Once you have forked the repo and gave it the name username.github.io, go to _includes/content.html and replace {%- include toc.html -%} by `{% include toc.html html=content %}``
Remove the toc.html file and replace it with this one.

Now, you can create your GitHub Page. If you need to set up a theme to start it, you can remove it in _config.yml and replace it with theme: null.
See :

I'll see if I can modify this plugin so that it has the same layout as the current one (which I like).

When using Pages but for a "Project page" (i.e. the url is "user.github.io/project") some of the links don't take in account the baseurl from _config.yml, so they point to "user.github.io". For example in layout.html, header.html,  feed.html (also L27, L47 and L69) and content.html.
I don't know if there's a reason for this, or if changing it would cause problems when publishing in Netlify or somewhere else, but adding {{ site.baseurl }} in those cases worked for me.
Thanks for the amazing plugin!


delete zk jar file and useless test code

What this PR does:
delete zk jar file and useless test code

Codecov Report

Merging #290 (7529e25) into 0.4.0 (74afc4e) will increase coverage by 0.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            0.4.0     #290      +/-   ##
==========================================
+ Coverage   33.13%   33.98%   +0.85%
==========================================
  Files          62       61       -1
  Lines        3323     3160     -163
==========================================
- Hits         1101     1074      -27
+ Misses       2108     1974     -134
+ Partials      114      112       -2

Impacted Files
Coverage Δ

...adapter/dubboregistry/remoting/zookeeper/client.go

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 74afc4e...7529e25. Read the comment docs.


fix: correct floating precision in snapValueToStep

Closes Issue#6112
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[x] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Test with the same steps in the issue above

GET_BUILD


re-instated publication identifier in metadata info dialog

The publication identifier metadata was removed in this Pull Request:
#881
@llemeurfr I seem to remember that you advocated for the removal of the publication unique identifier (ISBN or otherwise) in the information / metadata popup dialog, but I cannot find a record of related conversations / decisions. Sorry if I am being wrong :)
Personally, I feel that even though the unique publication identifier is likely irrelevant / useless to most (human) readers, it may be important to some people, for some usage scenarios. So I would be in favour of re-instating it.

Indeed I proposed its removal, because useless for most users. Especially if it's a uuid. Only an ISBN may be useful for "standard" users, for whom Thorium is tailored.
But let's see what others think...

There has not been a strong demand for this feature. Closing.


Can you remove the name Tito Lignon. It was made by me. Only used a friends gpt

Corrected

Thank you very much


Correction for raw affiliation Harvard University, School of Engineering and Applied Sciences and Department of Physics, 29 Oxford Street, Cambridge, USA; UMR IATE, SupAgro, INRA, CIRAD, UM2, 2plViala,Montpellier,France

Correction needed for raw affiliation Harvard University, School of Engineering and Applied Sciences and Department of Physics, 29 Oxford Street, Cambridge, USA; UMR IATE, SupAgro, INRA, CIRAD, UM2, 2plViala,Montpellier,France
raw_affiliation_name: Harvard University, School of Engineering and Applied Sciences and Department of Physics, 29 Oxford Street, Cambridge, USA; UMR IATE, SupAgro, INRA, CIRAD, UM2, 2plViala,Montpellier,France
new_rors: 05kpkpg04;03rnk6m14;03vek6s52;0000n5x09
previous_rors: 05kpkpg04;03rnk6m14;03vek6s52
works_examples: W2135074495
contact: d6b9b8c6913163d69f16b43b2fe6e1b0:ce525c4d061d31daa260d847d9e4ddc8921a @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Move learning objectives to extras menu

Closes #45
This would also be a good time to review this text to see whether we want to update anything.

We discussed during our meeting today.  These objectives remain aspirational for future versions of the workshop and are not necessarily all accomplished in the current version of the workshop.
We may still want to review these overall learning objectives and those in individual lessons to make the language more actionable or precise.


add macos command to step 3

Updated docs sight to include terminal command for Development Environment Setup -> STM32CubeMX -> Step 3.

Change has been previewed with mkdocs serve. Change looks good.


Use simply 'python3' as custom interpreter

Description of your problem
I want to use the system-installed Spyder3 to work on code that runs unter (several different) virtualenv environments. (I cannot use a pip-installed newer Spyder version because of its Qt dependency.)
For this, I currently have to change Spyder's "Python interpreter" to the absolute path of the virtualenv's python3 executable each time I change the environment.
For faster changes between virtualenv environments (or beween working in a virtualenv or system environment), it would be useful to set the interpreter just to python3, such that it is found on the $PATH.
What steps will reproduce the problem?

Under Preferences -> Python interpreter, try to set the interpreter just to python3.

What is the expected output? What do you see instead?
I would expect that the new setting is accepted.
Instead, a dialog says "You selected an invalid Python interpreter [...]".
Please provide any additional information below
I have got a workaround:
When I close Spyder and manually edit spyder.ini to executable = python3, everything works fine (i.e. !which python correctly shows my virtualenv's python) until I open the Preferences dialog again.
Versions and main components

Operating system:  Debian Stretch

The following are the system defaults (at the current update level):

Spyder Version: 3.1.3
Python Version: 3.5.3
Qt Version: 5.7.1
PyQt Version: 5.7

Dependencies

Please go to the menu entry `Help > Optional Dependencies` (or
`Help > Dependencies`), press the button `Copy to clipboard`
and paste the contents below:
jedi >=0.9.0     :  0.10.0 (OK)
matplotlib >=1.0 :  2.0.0 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.12.1 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
pep8 >=0.6       :  1.7.0 (OK)
psutil >=0.3     :  5.0.1 (OK)
pyflakes >=0.6.0 :  1.3.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.7.4 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.10.5 (OK)
sphinx >=0.6.6   :  1.4.9 (OK)
sympy >=0.7.3    :  None (NOK)

I think an easier way to solve this would be to maintain a list of the most recent (10?) used interpreters, so users can easily switch between them.
@jondo, what do you think?

This would only be a slight improvement.
Just having executable = python3 would make it unnecessary to touch the preferences at all. A Spyder launch in an "activated" environment would automatically get the right executable.
Since editing spyder.ini to just python3 works, why not allow this editing to also happen in the GUI?

It would seem that the enhancement suggested by @ccordoba12 , adding is a pop up list of the most recently used python backends, seems to be somewhat different to bug pointed out by @jondo , that the Preferences GUI does not allow them to add the system python3 installation as a valid interpreter, even though it would function fine; the only reason it needs to be changed every session is due to this reason, and would be entirely unnecessary if it were fixed; on the other hand, assuming it wasn't, a less resourceful user wouldn't be able to even add it in the first place as it would fail, and might not get added anyway as it was manually edited into the ini.

@jondo, if you have Spyder installed in every environment (as it seems the case), you don't need to change the Python interpreter every time you start Spyder. Simply using the default interpreter should pick up the interpreter in your environment.

No, as I wrote, I need to use the system-installed Spyder 3.1.3. I have tried installing Spyder in the environments, but this gives me an error about missing Qt libraries. I only have the system-installed Qt 5.7.1 available.

Then how do you expect that simply passing python3 would select the interpreter of your environment?

When I apply my workaround of directly editing spyder.ini to executable = python3, I see that this works: I can use the packages that are only installed in my environment. Also, !which python shows the correct path into my environment.
Can you please change this issue's title back? The new one does not reflect my problem.

We do some validations to detect if the executable you pass as a custom interpreter is really a Python interpreter. So that's why we need he full path of the executable.
If you manage to make those validations to work by simply passing python3, then we'll merge your work for sure.

What about internally replacing python3 by whatever is returned from which python3 before doing the validations?

Sounds good, but this is a specific case of yours, so sorry to say it but we don't plan to work on it.
However, as I said, we're welcome to receive PRs that fix it, in multiplatform way (e.g. there's no which on Windows).

I understand. As a note, on Windows the where command might work.

At least on my machine, it indeed returns the exact same results, in the same format, etc., as which does.

I'm having the same issue, how do you edit spyder.ini? Where is the file located?

@SreehariRamMohan  ~/.spyder-py3/spyder.ini

I want to avoid global state, so I do not want to edit my Spyder preferences whenever I am switching between venvs. Fortunately, I found a solution:

In the Spyder preferences, I keep the Console -> Python executable at Default.
I launch Spyder within my venv as

python `which spyder3`

This way, the venv's python is used for Spyder's Python consoles.
(Reading https://github.com/firedrakeproject/firedrake/issues/1218, @ZhouleiJoeStone might also be interested in this.)


Placehold the registry username and hide MySQL port

Thanks for this repo. I was able to get Wordpress up and running. Mostly. I haven't really tested it much. Here are some things that I think could be useful.

The registry username/password would be provided by the person cloning the repo. Replaced liltechnomancer username with a placeholder.
port: 3306 exposes that port to the world for the db accessory. That can be restricted to the server with <IP_ADDRESS>:3306:3306.
Kamal sets up container names for accessories that you can reference as <service>-<accessory_key>. So WORDPRESS_DB_HOST can just be kamal-wordpress-db. I don't know if the port is necessary, but I added it.

To be honest, I'm not sure if all that's correct, but it's working for me for now.

Hey! Thanks for this! lgtm 🚀


Utilize indexes/doc-values for <array> = <array> queries

Consider int_array as an example. We end up invoking either IntPoint.newSetQuery or SortedNumericDocValuesField.newSlowSetQuery, but notice that both Set[Slow]Query are NOT mean for <int_array> = <int_array>:
Create a query matching **any** of the specified 1D values. This is the points equivalent of TermsQuery

Although the Lucene queries generated were wrong, the executions returned correct results because Set[Slow]Query returned superset of the correct result set and we also included GenericFunctionFilter to correctly narrow it: https://github.com/crate/crate/blob/e4957ebe2f9fdecac3acbfabbf065f03aef5028e/server/src/main/java/io/crate/expression/operator/EqOperator.java#L222

Afaik this is intentional. What's the problem here?

Thanks for clarifying, just assumed we could improve the performance, closing.

Hi, @mfussenegger just to clarify further, currently when we index arrays, we index per element only. Is that also intentional?
We implicitly allowed multi-dimentional arrays and they will also be indexed per element. And I cannot think of any query that can utilize such indexes.

Thanks, I can see that it is intentional https://github.com/crate/crate/pull/16312#issuecomment-2227966485.


How to replace session.run() with keras Model() for predictions?

How to replace session.run() with keras Model() for predictions?

self.sess = tf.compat.v1.keras.backend.get_session()

Got it. In this way almost all TensorFlow-1 functionality can be used.


Segregate dialogs into separate files

@m4thfr34k @maxnorth This has been taken care of. Related branch/commit containing separate dialogs -- https://github.com/AmericanRedCross/damage-assessment-bot/tree/help-dialogs
As part of this, following things have been tackled -

Separation of dialogs into separate files
Addition of Help related dialogs to each file
Addition of global chat helpers and dialogs
Added adaptive card functions to collect sector related information.
Researched to get a working Prompt for Adaptive Cards.

The integration/wiring up of these dialogs will be tackled in a separate task. Please let me know if this estimation doesn't look good :)


Change Appearances Licenses to Apache 2.0

We decided to release all the appearances PROTO with the Apache 2.0 license to make sure that their use do not 'contaminate' the user PROTOs.

test me

Build Passed :ok_hand:
You can safely merge 'change-appearances-license' into 'revision'.

👍


Mehrere Magimirrors

Several Magimirrors
I have several Magicmirrors and I want Alexa to be my voice assistant everywhere.
I have successfully installed MMM-AlexaControl on a mirror and it works great.
What settings exactly do I have to make on the other mirrors so that Alexa can also find the mirrors there.
I've already given a mirror another name, but that didn't help.
I ask for your help.
Maybe you can show me an example of what to do.

Hi,
it's right that you want to use the deviceName option. It's not necessary but it's useful to identify the different devices. In addition you have to change the option startPort. The port identify each device in the background. The default value is 11000. Because of reserved ports for the devices you should increase this value for every new mirror with 1000. So you don't have to change this on your first mirror and set it on your second to 12000 and so on.

I did everything as they say. I can't switch sides with the second mirror; I can switch sides with the first. (I have to say that I have two identical mirrors). And what doesn’t work anymore is that I can switch the monitor off and on. That worked before. What settings do I have to make. Do you have examples.

So only the devices for the pages on your second mirror aren't working or are there more that dosen't work? Could you please send me the config of the module because with that I can't find any problems.

[       {
module: 'MMM-AlexaControl',
position: "top_bar",
config:{
pm2ProcessName: "mm",
vcgencmd: "vcgencmd",
pages: "3",
startPort:"12000",
deviceName:"Büro"
}
}
The example only works partially. The 3 Alexa commands work. What does not work is switching the pages by voice. Switching between sides works on the other mirror.

I think the problem is that you set the option pages to: pages: "3" but it has to be pages: 3. It's the same with  startPort.

I checked everything again and suddenly everything works. Thank you for your support. Very good module. Keep it up.


Update pyxis step image

Don't merge.
We should just update to the latest on main, but I wanted to get a git ref here that could be used for a model push downstream.

The purpose here is to pull in https://github.com/konflux-ci/release-service-utils/pull/311 for OCI models, which don't have any gzipped layers.

Closing in favor of https://github.com/konflux-ci/release-service-catalog/pull/684


fix: allow multi Response in MultiStatus

MultiStatus should allow multiple response, such as in case of propfind with depth 1.

Thank you for your PR. It's nice to see this project being integrated into other projects.
The issue you stumbled upon shows that there is a huge need for more unit tests. MultiResponse was already intended to handle this case, but it wasn't tested yet.
While your fix seems to work in this case, I think there is a larger underlying issue: the parser somehow creates nested lists, which it shouldn't do. It basically parses the responses as [[Response 1, Response 2], Response 3] instead of [Response 1, Response 2, Response 3]. I will investigate this issue further. I suspect it also affects other cases where a list contains more than 2 items.

@veeso if you encounter another problem, feel free to open an issue. I'm using the library only on the server side right now, so I suspect that there are more problems that are only uncovered on the client side.


[Feature #159951206] Implement feedback given by my LFA

What does this PR do?

Implements feedback given to me concerning challenge three by my LFA

Description of Task to be completed?
Using git best practice to make greate and atomic commit messages.
What are the relevant pivotal tracker stories?
Pivotal story 159951206

Pull Request Test Coverage Report for Build 208

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.4%) to 92.378%

Files with Coverage Reduction
New Missed Lines
%

stackoverflow/init.py
3
95.52%

stackoverflow/api/v1/models.py
5
85.42%

Totals

Change from base Build 206:
-0.4%

Covered Lines:
1321

Relevant Lines:
1430

💛  - Coveralls


502 Bad Gateway on the demo website

No 502 error, but an error:
TypeError: Cannot read property 'data' of undefined at Layer.module.exports [as handle] (/middleware/routes/validators/apply-permissions.js:38:20) at trim_prefix (/node_modules/express/lib/router/index.js:254:17) at /node_modules/express/lib/router/index.js:216:9 at param (/node_modules/express/lib/router/index.js:305:14) at param (/node_modules/express/lib/router/index.js:321:14) at Function.proto.process_params (/node_modules/express/lib/router/index.js:365:3) at next (/node_modules/express/lib/router/index.js:207:19) at next (/node_modules/express/lib/router/index.js:182:38) at next (/node_modules/express/lib/router/index.js:182:38) at next (/node_modules/express/lib/router/index.js:182:38) at next (/node_modules/express/lib/router/index.js:182:38) at next (/node_modules/express/lib/router/index.js:182:38) at next (/node_modules/express/lib/router/index.js:182:38) at Layer.logger (/node_modules/morgan/index.js:144:5) at trim_prefix (/node_modules/express/lib/router/index.js:254:17) at /node_modules/express/lib/router/index.js:216:9 at Function.proto.process_params (/node_modules/express/lib/router/index.js:286:12) at next (/node_modules/express/lib/router/index.js:207:19) at Layer.handle (/app.js:49:5) at trim_prefix (/node_modules/express/lib/router/index.js:254:17) at /node_modules/express/lib/router/index.js:216:9 at Function.proto.process_params (/node_modules/express/lib/router/index.js:286:12) at next (/node_modules/express/lib/router/index.js:207:19) at Layer.expressInit [as handle] (/node_modules/express/lib/middleware/init.js:23:5) at trim_prefix (/node_modules/express/lib/router/index.js:254:17) at /node_modules/express/lib/router/index.js:216:9 at Function.proto.process_params (/node_modules/express/lib/router/index.js:286:12) at next (/node_modules/express/lib/router/index.js:207:19)

You're correct, although as you can see at the time this issue was created mentioned error wasn't shown.```

@eko24ive Yes :)


Fix missing requirements file for install

Latest release doesn't work:
Downloading/unpacking django-dbbackup==2.3.1 (from -r requirements.txt (line 11))
  Downloading django-dbbackup-2.3.1.tar.gz
  Running setup.py egg_info for package django-dbbackup
    Traceback (most recent call last):
      File "<string>", line 16, in <module>
      File "/home/ubuntu/virtualenvs/venv-2.7.9/build/django-dbbackup/setup.py", line 21, in <module>
        tests_require=get_test_requirements(),
      File "/home/ubuntu/virtualenvs/venv-2.7.9/build/django-dbbackup/setup.py", line 11, in get_test_requirements
        return open('requirements-tests.txt').read().splitlines()
    IOError: [Errno 2] No such file or directory: 'requirements-tests.txt'
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):

  File "<string>", line 16, in <module>

  File "/home/ubuntu/virtualenvs/venv-2.7.9/build/django-dbbackup/setup.py", line 21, in <module>

    tests_require=get_test_requirements(),

  File "/home/ubuntu/virtualenvs/venv-2.7.9/build/django-dbbackup/setup.py", line 11, in get_test_requirements

    return open('requirements-tests.txt').read().splitlines()

IOError: [Errno 2] No such file or directory: 'requirements-tests.txt'

Already add by @mjs7231


Cache block opinions in tortoise

Description
The tortoise relies upon the "local opinion" of blocks, which can depend upon several things depending on the age of the block's layer, whether or not it's been verified, and whether hare completed for the layer: counting votes in blocks, reading block contextual validity from meshdb, or reading the "input layer" (which may be received from hare, or from sync).
This operation can be very expensive, especially if it involves vote counting. We currently make no effort to cache this local opinion, nor any of the underlying data that it relies upon. This makes certain operations, such as rescoring "good" blocks after healing, very slow and expensive.
Devise a caching strategy that takes into account all of the moving parts (layer age, newly-arrived blocks and updated votes, hare results, etc.).
Affected code
verifying_tortoise.go layerOpinionVector, getBlockLocalOpinion
This issue appears in commit hash: 9d3a379994e3652eb0378d2cc555588bba91005e

This was extensively profiled and the important part of this issue was addressed in https://github.com/spacemeshos/go-spacemesh/pull/2897 .


PR Request: Move ProgressReporter initialisation

PR request from Jonothan:
     */
    @Override
    public HttpResponse sendSync(HttpRequest httpRequest, Context context) {
        ProgressReporter progressReporter = Contexts.with(context).getHttpRequestProgressReporter();

Same comment here as in the previous review - it can move down beneath the PATCH check, as it is an unnecessary operation for PATCH (for now, anyway).

PR review can be found here.

Moved ProgressReporter initialisation below PATCH check.

Moved ProgressReporter initialisation below PATCH check.


Add itecco.co.uk

Great alphabetisation skills there Adam, well done.
Thanks for adding this properly @jasoncartwright!

Test gotcha. No problem!


Throw "TypeError: Cannot read property 'nextSibling' of undefined" on vite hmr

Version
3.1.4
Reproduction link
https://github.com/ycs77/vue-3.1.3-hmr-bug-demo
Steps to reproduce

yarn
yarn dev and open browser: http://localhost:3000
open src/App.vue
replace line 3 class max-w-screen-sm to max-w-screen-md and save file
replace line 3 class max-w-screen-md to max-w-screen-lg and save file

What is expected?
hmr working, and no throw errors
What is actually happening?
Throw error: "TypeError: Cannot read property 'nextSibling' of undefined"

this bug exists in both v3.1.3 and v3.1.4
if downgrading vue and @vue/compiler-sfc to v3.1.2 will no this bug

I didn't reproduce the bug.

download your repro
remove yarn.lock
update both vue and vue/compiler-sfc to 3.1.4
yarn
yarn dev
replace line 3 class max-w-screen-sm to max-w-screen-md and save file

works fine.
This bug has already fixed in https://github.com/vuejs/vue-next/commit/fded1e8dfa22ca7fecd300c4cbffd6a37b887be8

@edison1105 Need to modify 2 times and save

replace line 3 class max-w-screen-sm to max-w-screen-md and save file
replace line 3 class max-w-screen-md to max-w-screen-lg and save file
look the error message in browser

@ycs77
uh~ sorry for that.
I will check it later.

fixed by c0db8078

fixed by c0db807

Can I use that commit now? I have same problem.

fixed by c0db807

Can I use that commit now? I have same problem.

not release yet


HTTP code is 500 which indicates error: server error - [object Object]

Expected behaviour
Show status of docker image
Allow start/re-start of postgres 9.3.13 image.
Actual behaviour
The postgres server is shown as 'Stopped' but I know it is running because I can access the DB.
(Actually, it couldn't run because I had another instance running!)
When I try to click Start or Restart an error briefly flashes up:
"HTTP code is 500 which indicates error: server error - [object Object]"
before it is obscured by the log output (which doesn't change.)
Information about the Issue
This is the first time I've used Kitematic, so this may be a noob problem.
Platform is Mac/OSX
Steps to reproduce the behavior

Run up a postgres instance - turns out Jira was using THIS instead!
Run Docker with a postgres:9.3.13 image (the DB is intended to be used by Jira)
Run Jira - this is configured with the postgres DB
Install and run Kitematic
Shows Docker postgres DB as Stopped
Click on 'Start' - error flashes briefly

To see the error, go to Settings - scroll down and click on Save - then go to Home tab where the error can be seen.
Unfortunately, the actual cause (i.e. port already in use) is pretty much overlooked.

Please update to the latest Kitematic to get the proper error instead of the [object Object] you're showing above.

I only installed it today! How much more up-to-date do I need to be?!

@pslootweg How did you install it? I'm only asking because the error report should be more detailed.
The error you're showing was fixed on 0.11.x
Can you list the exact images (and tags) used so that I can reproduce on my end?

Think it was a link offered from Docker...
I can check the version tomorrow.

Please let me know what the version is, so that I can ping the right people to get the links updated.

I have since upgraded and now
I am running Docker Version 1.12.0-beta21 which launches Kinematic 0.12.0
Same problem and same error message.

I am unable to replicate this issue, as you haven't provided the exact tags used for each image and the settings used for them as well.
Please provide them, so that I can see the error for myself.

Once again, I don't know what you're talking about.
In Docker, I have a Postgres 9.3.13 image - with port 5432 mapped to local port 5432.
Initially, my problem was that I had a (homebrew) installed Postgres instance. So, when trying to start the Docker image, there would be a port conflict with the system installed Postgres instance.
I then killed the system, installed installed, and found the same result came from trying to start  second Postgres 9.3.13 instance in Docker.
This problem is VERY repeatable - either within Docker, or in fact for any port conflict.
The issue is SIMPLY that a Docker instance can't be started if the port is already in use. Unfortunately, the error reported is completely unrelated.

From kitematic, port mapping is done with a random port and why it wasn't showing any errors from me.
Providing very little information about the issue you're having makes it difficult for maintainers to determine the root cause of it.
You mentioned JIRA in your first thread (with no info on the version/tag used), yet it seems to have nothing to do with your issue.
Hence my point that replicating this through kitematic is impossible.

Create Postgres container, random port is assigned:

Assign the Postgres port:

Create a new Postgres container, random port is assigned:

Try to assign the port of 5432 results in an error:

Providing us with exact repeatable steps, with the environment that you're in and with screenshots (as I did above) would help me understand the issue you're having.

I can show that you CAN:
One container with Postgres:latest mapped to 5432:5432

and one with Postgres:9.1.13 mapped to 5432:5432

In fact, there is a 3rd mapped to 5432:5432 - but that's irrelevant.
If I now try to start the 2nd instance, no error message is shown

If I then navigate to Settings and click on 'Save'

then when I navigate back to 'Home' it shows

As invited on that dialog, I filed a ticket.

It only seems to detect a port conflict if there is another container running on that port. Otherwise I can add a new container and assign it to port 5432

My original issue that that there was a postgres instance running outside Docker, and the port conflict with that was not detected, and I got the error scenario I demonstrated in my previous post.

That's correct - Running containers are the only ones that tie a port; non-running containers wasn't part of the original discussion.
When you hit save we don't dynamically update the container, we re-created it, which then triggers the 500 error that you're seeing, granted the error should've been more verbosed than [object Object].
In the latest version you will see something like:
(HTTP code 500) server error - driver failed programming external connectivity on endpoint postgres (4e5c8753bcc5b37d4613755a0d1832571da819757f076797dce51417453d2db3): Bind for <IP_ADDRESS>:5432 failed: port is already allocated

I'm not sure if there is a good way to handle this; technically a non-running container should not prevent you from creating another container and using that port.


Remove code related to model duplication factor  (-z,--multiple, -x,--extracon command line argument)

In the old days we wanted to duplicate input model by certain factor to perform scaling studies. Now models are quite bigger. We have ability to generate arbitrary size models to perform scaling studies.
In order to reduce the code complexity, I would suggest to remove the code related to model duplication from coreneuron.
@nrnhines : would that be ok? I haven't used this for few years already and don't see any use cases now days.

i take it, I will be so happy to remove extracon !!

You know it is not so hard and can be done in #283
Only waiting for @nrnhines advices about it is safe or not

@alkino : #283 is done, so this can be done now.


After Applying Quick Fix throws InconsistentAnalysisException

Sometimes after applying a custom DartFix the analyzer session crashes with
InconsistentAnalysisException: Requested result might be inconsistent with previously returned results
custom_lint.log
It does not happen all the time (haven't found a reliable way to reproduce)
But when it happens the custom list stays even though it's fixed, and it let's me apply the fix multiple times.

maybe it's because it's in the packages own example folder,
or because I started tinkering around with // expect_lint:
Because It might be because of my lint here it's source:

missing_mixin.dart
import 'package:analyzer/error/error.dart';
import 'package:analyzer/error/listener.dart';
import 'package:custom_lint_builder/custom_lint_builder.dart';
import 'package:freezed_lint/src/tools/freezed_annotation_checker.dart';

class MissingMixin extends DartLintRule {
MissingMixin() : super(code: _code);

static const _code = LintCode(
  name: 'freezed_missing_mixin',
  problemMessage: 'Freezed class should mixin with _\$<className>',
  errorSeverity: ErrorSeverity.ERROR,
);

@override
void run(
  CustomLintResolver resolver,
  ErrorReporter reporter,
  CustomLintContext context,
) {
  context.registry.addClassDeclaration((node) {
    final element = node.declaredElement;
    if (element == null) return;
    final annotation = freezedAnnotationChecker.hasAnnotationOfExact(element);
    if (!annotation) return;
    if (element.mixins.isNotEmpty) return;
    reporter.reportErrorForElement(code, element);
  });
}

@override
List<Fix> getFixes() => [_AddMixinFreezedCLassFix()];
}

class _AddMixinFreezedCLassFix extends DartFix {
@override
void run(
  CustomLintResolver resolver,
  ChangeReporter reporter,
  CustomLintContext context,
  AnalysisError analysisError,
  List<AnalysisError> others,
) {
  context.registry.addClassDeclaration((node) {
    if (!analysisError.sourceRange.intersects(node.sourceRange)) return;
    final changeBuilder = reporter.createChangeBuilder(
      message: 'Add mixin _\$${node.name}',
      priority: 1,
    );
    changeBuilder.addDartFileEdit((builder) {
      final element = node.declaredElement;
      if (element != null) {
        final name = element.displayName;
        builder.addSimpleInsertion(
          element.nameOffset + element.nameLength,
          ' with _\$$name',
        );
      }
    });
  });
}
}


Update webapp to use new proxying api

[x] Handle reduced information on initial load (revert to heroes only for now)
[x] Load hero data when hero selected

Looks like it's all working now. I just want to see if there's anything that needs cleaning up.


Vote Measurements better look

Discussed in DAO talk
Proposals List Page
200k vs 2000k
200k vs 2m
Basically make it look more consistent and easier to understand at a glance. Stop overflow of votes to second line

Thanks for making the issue.
This might have already been solved in this PR: https://github.com/effectai/effect-dashboard/pull/91
Which is deployed at https://dao.effect.network/proposals
Does it solve the items you're mentioning?

Well it solves it in a way, but not in a way I like. It no longer shows the Abs/Y/N votes on the proposal list, but I liked it being there since you can look at the status of all active proposals at a glance.
When looking at proposal pages, once votes go over a million "M" notation is used but there was discussion that it's better to just use "K" only. So instead of 3.5 it would be 3,500K.

I think this has been resolved by now, as the display is updated and we are also showing the percentage.


Malformed indentation for Performance/RedundantMerge

It looks like running Performance/RedundantMerge results in malformed indentation.
rubocop --version
0.36.0

Running:
rubocop --auto-correct --only Performance/RedundantMerge

Resulted in this:
-        options.merge!(product_id: id, employee_id: vendor.employee_id)
+        options[:product_id] = id
+options[:employee_id] = vendor.employee_id

There's already an issue for this. Issue #2669.

Thanks, @savef! Not sure now I managed to miss that one.


Edit mode enabled despite check_callback returning false

Hi,
we configured our jstree to have check_callback being set to a function, that contains some logic and returns true or false.
When modifying the tree e.g. by trying to create a node check_callback is evaluated correctly (see  jstree.js#L3909).
The problem is, that the check_callback function isn't correctly evaluated when a node is in the edit mode - e.g. after pressing F2 (see jstree.js#L4347).
Just select the a node in this JSFiddle and press F2: http://jsfiddle.net/ee7f9f31/1/
Thanks!

This is by design - the function is not invoked on purpose. check_callback is not invoked when entering edit mode, only after exiting. The line you are pointing to is there to prevent edit mode when all changes are forbidden. If you want to prevent the tree from entering edit mode - you could blur the INPUT node as soon as it is focused or use a simple plugin that overrides the edit function with an empty one.


release: 3.15.8

[v3.15.8] (Nov 7th, 2024)
Fixes:

Fixed an issue where the dir attribute was not being properly applied to message containers:

Removed useMessageLayoutDirection hook in favor of a more React-friendly solution
Updated MessageList component to directly handle text direction through the dir attribute

Fixed an issue in TypingIndicatorBubble component where null was returned before hook execution
Fixed SDK initialization parameter override issue:

Modified Object.assign order to allow proper parameter override
Added test case to verify localCacheEnabled override functionality
Ensures sdkInitParams.localCacheEnabled properly overrides default settings; localCacheEnabled: true

/bot create ticket

[Creating Ticket] 🔖 Done https://sendbird.atlassian.net/browse/SDKRLSD-1457


viewDidLoad called twice in DemoTabBarController

Hi,
I really like the bFPaperTabBarController. I tried using it in a project and noticed my view controller that extends the BFPaperTabBarController is called twice.
I checked with the demo that is provide with the source code: DemoTabBarController. There the same happens.
Is there any way to prevent this? Allocating the controller twice seems a bug to me.
Regards,
Rudie.

Hi @ekkelenkamp !
Thanks for bringing this to my attention. After some debugging, I found the offending line of code in BFPaperTabBarController.m:
    [self.view addSubview:self.invisibleTouchView];

This discovery led me to another possible bug and both will be fixed in the next release. I'll let you know as soon as I push the update.
Thanks again!

Just pushed version 2.1.11 which fixes this bug.
Cheers!

Hi,
Great work fixing this bug that fast! I just tested the 2.1.11 version and all works fine now. Keep up the good work!
Rudie.


Added docs site

Feel free to use these docs if you find it useful. It should be easy to maintain - no typescript needed (see the README for all info). Just need to add mdx files to content/docs and then specify the file in to content/docs/meta.json. Deployment instructions should be pretty straightforward if you decide to deploy to Vercel, but this can totally be deployed elsewhere if you want.

Thanks for the submission @WesternConcrete !! I'm closing this for now until I figure out how to generate the site from only the README.md as we discussed :) I spent some time trying to get it to work but I haven't got it across the finish line yet 😆


changeable epsilon for (typeof Quaternion).FromUnitVectorsToRef

for my use case epsilon is to big, so right now I have to copy entire method and use 1e-7 epsilon which works fine for me
I hope optional parameter for epsilon is fine

Snapshot stored with reference name:
refs/pull/13983/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13983/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13983/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13983/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13983/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13983/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13983/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13983/merge#BCU1XR#0

I wonder what purpose this epsilon check serves? Is it optimization or function might break, if dot product is near zero.

I wonder what purpose this epsilon check serves? Is it optimization or function might break, if dot product is near zero.

It's to avoid breaking because of limited precision: https://blog.demofox.org/2017/11/21/floating-point-precision/


Revert "upgrade ovs-ovn pod by generation version instead of chart ve…

…rsion (#1960)"
This reverts commit 57c75c1e2c328391014b8d8e690d79addc560dd4.
Pull Request

[ ] Make sure you have followed Kube-OVN Code Style.

What type of this PR
Examples of user facing changes:

Features
Bug fixes
Docs
Tests

Which issue(s) this PR fixes
Fixes #(issue-number)
WHAT
copilot:summary
copilot:poem
HOW
copilot:walkthrough

https://jira.alauda.cn/browse/ZSZQ-167

只需要修改 release-1.9 和 release-1.11 分支，其他分支不受影响。release-1.12 和 master 分支，不需要修改


Fixes: current time

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>89c8db45fadc56be17e948f1d96061 on jeromesimeon:js-fix-current-time into ccf6e3ab290108ed8cda8902b0754ddf521baab0 on accordproject:master.


refactor: improve home transaction filter

Summary
https://app.clickup.com/t/2uq7qvw

Checklist

[ ] I checked my UI changes against the design and there are no notable differences
[ ] I checked my UI changes for any responsiveness issues
[ ] I checked my UI changes in light AND dark mode
[ ] I checked my (code) changes for obvious issues, debug statements and commented code
[ ] I opened a corresponding card on Clickup for any remaining TODOs in my code
[ ] I added a short description on how to test this PR (if necessary)
[ ] Documentation (if necessary)
[ ] Tests (if necessary)
[ ] Ready to be merged

moving issue to separate card


Update build.yml

Don't build exe
The exe build always does a new install trashing what was there before.

This is worth doing though it's probably possible to fix the exes too

This is worth doing though it's probably possible to fix the exes too

I did it now because using the exe really did trash a lot of files that the user wanted.  If we choose to continue to distribute the exe we should try and fix it.  However I cannot build a KoLmafia exe on my system from a command line, and I can't make any progress until someone helps me understand why GitHub can build it but I cannot?  Windows Home vs. Professional?  Dunno but my efforts have been stalled for a while.


Static Analysis fixes

What this PR does / why we need it:
To improve our code quality, fix various issues highlighted by static analysis tools. This is part of the work on https://github.com/cert-manager/website/issues/538
Special notes for your reviewer:
Release note:
NONE

/kind cleanup

/kind cleanup

/assign @irbekrm

Looks good to me! Some of the comments are art 😄
/lgtm


[Question][API] Is there a plan to develop and use the audit log module under the monitoring center module?

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Is there a plan to develop and use the audit log module under the monitoring center module?
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

If there are plans to develop it, when is it expected to be launched?

I don't quite understand what the audit log module is. Please describe your issue more clearly. @xiongbl526

I don't quite understand what the audit log module is. Please describe your issue more clearly. @xiongbl526
Under the Statistics module in the Management module, there is a section for Audit Logs.

I still don't know it. Can you take a screenshot of it?

Has been finished by https://github.com/apache/dolphinscheduler/issues/15423