Change Charge Target and Target Time sensors to selects

Describe the feature
Would it be possible to change the intelligent tariff sensors for charge target and target time to selectors. It would be much easier to update these values if they were drop-down lists rather than having to type in values which are then validated (for example, it’s possible to set values that are invalid and rejected by Octopus).
Expected behaviour
Be able to select charge target and Target Time from a list
Use Case
Easy selection without need to type in numbers which are then required to be validated.
Confirmation

[X] By submitting this feature request, you agree that you have read the documentation and confirmed it does not already exist
[ ] I am willing/able to help contribute to the solution of this feature

Hello. I'm not on intelligent and the OE APIs only provide the values as numbers which is why they're exposed like they are. I'm also not sure if these accepted values differ per integration (e.g. Ford vs OHME). If you could provide the error that's raised, I can probably improve the sensors to not support invalid values. Alternatively, what values are provided as part of the app for you (and what intelligent integration are you with?)?

Thanks for quick response.I’m on Intelligent Octopus Go.For the target charge value, I can set any percentage, but if incorrect, the entry goes red, but no error. It is reflected in the Octopus App which then crashes if you try and edit the percentage. It should range between 10% and 100% in 5% increments.For the target time value, I get an error if set outside the permitted range. E.g setting 15:15 as target time results in:Failed to perform the action time/set_value. Errors in request (https://api.octopus.energy/v1/graphql/): [{'message': 'Serializer validation error.', 'locations': [{'line': 2, 'column': 3}], 'path': ['setVehicleChargePreferences'], 'extensions': {'errorType': 'VALIDATION', 'errorCode': 'KT-CT-4321', 'errorDescription': 'One or more serializers did not have valid input values.', 'validationErrors': [{'inputPath': ['input', 'nonFieldErrors'], 'message': '15:15:00 or 15:15:00 are outside the valid time range.'}]}}]Also, if I set the value to be something like 7:15, it is reflected in the Octopus App which crashes if I try and edit it. It should be in the range 04:00 to 11:00 in 30 minute increments.Regards,GrahamSent from my iPadOn 13 Nov 2024, at 17:00, David Kendall @.***> wrote:
Hello. I'm not on intelligent and the OE APIs only provide the values as numbers which is why they're exposed like they are. I'm also not sure if these accepted values differ per integration (e.g. Ford vs OHME). If you could provide the error that's raised, I can probably improve the sensors to not support invalid values. Alternatively, what values are provided as part of the app for you (and what intelligent integration are you with?)?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I’ve just come to ask for the same feature. I’ve recently installed HA to automate the control of my house battery since the scheduling in Intelligent Octopus Go became more dynamic and one thing has led to another and I’m now setting up a dashboard so I can devolve the nightly configuration of intelligent octopus go to the other 2 EV drivers in the house but to get them to use it I need to make it as easy as possible to set the time.
To add to Graham’s response I don’t think he mentioned the “Ready by time” has to be from 04:00 and 11:00 in 30 minute increments as he already mentioned.

Thank you for supplying the required restrictions. You're not the first people to request this change, especially as its exposed to more non-technical people. When the intelligent part was first added, the API didn't give any indication to any restrictions which is why it has been exposed the way it has.
In order to not create a breaking change in the integration, if this is introduced it will need to be a new sensor that is available along side the existing sensors.

New sensors would be great with the required restrictions. Ideally, these
sensors would be the same type as the megakid/ha_octopus_intelligent
sensors for the ready time and charge limit sensors i.e. selects.
On Sat, 23 Nov 2024, 09:39 David Kendall, @.***> wrote:

Thank you for supplying the required restrictions. You're not the first
people to request this change, especially as its exposed to more
non-technical people. When the intelligent part was first added, the API
didn't give any indication to any restrictions which is why it has been
exposed the way it has.
In order to not create a breaking change in the integration, if this is
introduced it will need to be a new sensor that is available along side the
existing sensors.
—
Reply to this email directly, view it on GitHub
https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/1079#issuecomment-2495418214,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDOUPSVO5CPZ5MW5QBTL4B32CBEMHAVCNFSM6AAAAABRU32FE6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJVGQYTQMRRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

This was kinda why I made the sensors as they are, as the API suggests it can be any value, and it might be that the values differ depending on the intelligent provider. There is already some validation on the values, which is why it goes red, but there could be more.
I'll probably concentrate on migrating the target time one first, as this seems like it's purposefully restrictive in the app across all providers.


Shape mismatch in visual encoder when using third_rgb_sensor view in evaluation

Habitat-Lab and Habitat-Sim versions
Habitat-Lab: master
Habitat-Sim: master
🐛 Bug
If we add third_rgb_sensor to the video_render_views, train an agent (using ddppo or ver) and try to evaluate it then we get a shape mismatch error.
Steps to Reproduce
Steps to reproduce the behavior:

Add third_rgb_sensor to the list of video_render_views
Train any skill using ddppo or ver (training it until getting the first checkpoint is enough)
python -u habitat-baselines/habitat_baselines/run.py --exp-config habitat-baselines/habitat_baselines/config/rearrange/ddppo_pick.yaml --run-type train
Evaluate such policy python -u habitat-baselines/habitat_baselines/run.py --exp-config habitat-baselines/habitat_baselines/config/rearrange/ddppo_pick.yaml --run-type eval

You should get an error like the following:
size mismatch for actor_critic.net.visual_encoder.running_mean_and_var._mean: copying a param with shape torch.Size([1, 1, 1, 1]) from checkpoint, the shape in current model is torch.Size([1, 4, 1, 1]).
size mismatch for actor_critic.net.visual_encoder.running_mean_and_var._var: copying a param with shape torch.Size([1, 1, 1, 1]) from checkpoint, the shape in current model is torch.Size([1, 4, 1, 1]).
size mismatch for actor_critic.net.visual_encoder.backbone.conv1.0.weight: copying a param with shape torch.Size([32, 1, 7, 7]) from checkpoint, the shape in current model is torch.Size([32, 4, 7, 7]).

Expected behavior
It should run as usual and show a video with an additional view of the agent, something like the following:

Possible solutions
We could filter the observational space that will be sent to the policy here
observation_space = self.obs_space
#observation_space = dict([(k,v) for k,v in observation_space.items() if k != 'robot_third_rgb'])
#observation_space = gym.spaces.Dict(observation_space)

We can also filter that observation before sending it to the visual encoder in the policy here

Solved in #959


chore: git subrepo pull (merge) noir

subrepo:
subdir:   "noir"
merged:   "52fc0778e"
upstream:
origin:   "https://github.com/noir-lang/noir"
branch:   "aztec-packages"
commit:   "52fc0778e"
git-subrepo:
version:  "0.4.6"
origin:   "???"
commit:   "???"
Please read contributing guidelines and remove this line.

Benchmark results
Metrics with a significant change:

note_history_trial_decrypting_time_in_ms (5): 263 (+139%)
note_trial_decrypting_time_in_ms (32): 67.2 (+88%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 9e6250aa and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,412
179,524
716,068

l1_rollup_calldata_gas
221,704
866,716
3,448,072

l1_rollup_execution_gas
314,191
984,456
3,667,729

l2_block_processing_time_in_ms
1,161 (-1%)
4,324 (-1%)
17,104 (-2%)

note_successful_decrypting_time_in_ms
263 (+1%)
912
3,310 (-5%)

note_trial_decrypting_time_in_ms
74.5 (+4%)
:warning: 67.2 (+88%)
149 (+10%)

l2_block_building_time_in_ms
16,301 (+3%)
64,602 (+3%)
259,279 (+3%)

l2_block_rollup_simulation_time_in_ms
11,774 (+3%)
46,584 (+3%)
187,634 (+4%)

l2_block_public_tx_process_time_in_ms
4,499 (+1%)
17,885 (+1%)
71,373 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,331
25,730 (+1%)

note_history_successful_decrypting_time_in_ms
2,116
4,253

note_history_trial_decrypting_time_in_ms
:warning: 263 (+139%)
179 (-2%)

node_database_size_in_bytes
17,297,488
33,202,256

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
220 (+3%)
44,777
26,073

private-kernel-ordering
178 (+4%)
44,393
14,809

base-rollup
1,250 (+4%)
128,850
881

root-rollup
81.7 (+3%)
4,088
677

private-kernel-inner
311 (+3%)
71,816
26,073

public-kernel-private-input
220 (+5%)
32,375
26,073

public-kernel-non-first-iteration
218 (+5%)
32,417
26,073

merge-rollup
7.62 (-1%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
2 leaves
8 leaves
16 leaves
32 leaves
128 leaves
64 leaves
512 leaves
1024 leaves
2048 leaves
8192 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (-1%)
11.8 (+11%)
12.6 (-1%)
17.0 (+1%)
22.8 (-1%)
63.5 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.9
17.5
23.0
31.6
47.0
143
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.590
0.659 (+11%)
0.537 (-1%)
0.526 (+1%)
0.479
0.439 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
81.0 (+9%)
47.2 (-1%)
238 (-1%)
460
895 (-1%)
3,531 (-1%)

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
N/A
N/A
N/A
159
96.0
543
1,055
2,079
8,223

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.501 (+9%)
0.486
0.435 (-1%)
0.431
0.427 (-1%)
0.426 (-1%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
103 (-1%)
56.4 (-1%)
348 (-2%)
686
1,355 (-1%)
5,376 (-2%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
N/A
N/A
N/A
197
104
691
1,363
2,707
10,771

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.490 (-1%)
0.495
0.474 (-1%)
0.474
0.472
0.470 (-1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
56.3
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
94.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
0.576
N/A
N/A
N/A
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,455
32,993 (+1%)

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
393 (+3%)
1,118 (+2%)

Metric
1 public data writes

tx_sequencer_processing_time_ms
564 (+3%)


Config updates

I'm fine with either this way or adding a cadence directory to the root of this repo.


Update Tool and API Versions

Bump FAR to the latests tool and API versions (e.g. K8s, OCP, Controller-gent, Kustomize, and Operator-Sdk).
The PR includes some fixes for writing/deleting the envtest target directory, fetching the machine client (for E2E) using OpenShift client-go, and removing kustomize v5 deprecation warnings.

Golang must be v1.20 prior to merging this PR.
/hold
Wait for #57 to be merged

E2E test will fail as it needs https://github.com/openshift/release/pull/40248 to be merged, so Golang version on E2E will be updated

/retest

/lgtm
/hold
Giving others chance to review

/lgtm
Giving a chance to get more feedbacks, feel free to unhold
/hold

I hope #20 will be merged soon, and I prefer to wait for this PR to be merged. Since this PR introduce huge bump (by time of versions, not necessarily it's true for functionality of FAR), thus I would prefer to wait for #20.

E2E PR was merged and this PR is ready for review

Ok, I will create PR for changing the OCP client-go and fetching machine client in E2E test and in another PR I will update and fix deprecation warnings due to Kustomize v5 changes.

migrate all of them to the new format.

When you say new format are you referring to Kubebuilder go/v3 to g0/v4?  I remembered that go/v4 was still as alpha, and looking again I see that it has already been stabilized ECOPROJECT-1315

there are also code changes, probably also worth their own PR?

I was thinking of doing another PR for them.

Ok, I will create PR for changing the OCP client-go and fetching machine client in E2E test

But looking again on the changes, they are quite minor IMHO, and due to bumping the K8s and OCP APIs I was aware of an error with vet under vendor due to the old github.com/openshift/machine-api-operator package.
Therefore, I would prefer to let the these code changes remain in this PR.
I can update the PR's title if you still see a need for that.
WDYT?

there are also code changes, probably also worth their own PR?

I was thinking of doing another PR for them.

Ok, I will create PR for changing the OCP client-go and fetching machine client in E2E test

But looking again on the changes, they are quite minor IMHO, and due to bumping the K8s and OCP APIs I was aware of an error with vet under vendor due to the old github.com/openshift/machine-api-operator package. Therefore, I would prefer to let the these code changes remain in this PR. I can update the PR's title if you still see a need for that. WDYT?

ack, wfm

Updating to kustomize v5, and removing new deprecation warnings will be done in a follow up PR when we change the project layout to go/v4 - ECOPROJECT-1315

/unhold


ClassControl SetValue Reflection Members should be Cached

This has been resolved on the Template repo. The changes will soon be merged into this repo.


workflow metadata query

Add support for workflow metadata query as described here and implemented in the Go SDK here.

Duplicate of https://github.com/temporalio/sdk-java/issues/335


[Question] Context Updating

Hi,
I'm really struggling with this issue:
I have an object saved in core data. I have a method which updates some properties of that object. I'm using that object's Managed Object Context from obj.managedObjectContext.
I change 2 properties: obj.pending = YES and obj.body = 'text'.
I then save the object back to Core Data as follows:
 if ([managedObjectContext save:&saveError]) {

            [persistenceController saveContextAndWait:YES completion:^(NSError *error) {
                 [OfflineManager postData];
            }];

        }

The postData method gets all objects from Core Data where the obj.pending == YES and posts the data. It's managedObjectContext is of type NSPrivateQueueConcurrencyType.
My issue is as follows: The obj.pending seems to be updating but the obj.body is not. If I add a breakpoint to the app before it posts and restart the app, then the obj.body is correctly updated.
If this is a context issue why are some of the object properties' changes being seen and other not?
I've been hacking away at this for hours, any help would be most grateful.
Thanks

Resolved.


i use .netcore3.0 and i ues aspnet-api-versioning 3.1.0  i met also this question like this :  An error occurred while starting the application. TypeLoadException: Method 'ApplyAsync' in type 'Microsoft.AspNetCore.Mvc.Routing.ApiVersionMatcherPolicy' from assembly 'Microsoft.AspNetCore.Mvc.Versioning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' does not have an implementation.

how i can done it?

You need to the preview version for now. It's the only .NET Core 3.0 compatible version. I'm working on getting the release build out. Expect it within the next week. There are no planned changes or known bug fixes, I've just been up to my eyeballs with my paying day job. ;)  Thanks for your patience.

You need to the preview version for now. It's the only .NET Core 3.0 compatible version. I'm working on getting the release build out. Expect it within the next week. There are no planned changes or known bug fixes, I've just been up to my eyeballs with my paying day job. ;) Thanks for your patience.

I see.  but i need which version?

The following package should work:
https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc.Versioning/4.0.0-preview8.19405.7

The following package should work:
https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc.Versioning/4.0.0-preview8.19405.7

The problem has been solved. very ths!!!

Keep an eye out for the official release. It is coming - soon. ;)

The following package should work:
https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc.Versioning/4.0.0-preview8.19405.7

Man, ur the best!!!

It would seem that this issue has been resolved. Feel free to come back if you have more questions. Thanks.


CORE-5451 Expose member lookup based on session key

Extends the membership group reader implementation to expose a lookup of member info based on session initiation key. This works in the same way as the ledger key lookup except uses a single key instead of a list. Key hash must be set in the member context for faster lookups. New lookup called lookupBySessionKey to avoid duplicate function signature.

@YashNabar FYI this is merged now so your PR for requesting member registration will need to include the session key hash.


Add a wiki: The Project Zomboid Wiki

An anonymous user has submitted a new wiki via our submission form.
Link to origin wiki: https://projectzomboid.fandom.com
Link to destination (independent) wiki: https://pzwiki.net

Added in v3.5.0, now available on both Chrome and Firefox.


[News] The last reply in the ‘View [number] replies’ list isn't disappeared after deleting

Environment:
macOS Sonoma Version 14.0
Google Chrome
Version 118.0.5993.70 (Official Build) (arm64)
Reproducible:
The bug is reproducible just for the last reply in the ‘View [number] replies’ list.
Build found:
19.10.23
Preconditions
At least a News is added;
A News includes a User's comment;
The replies are added to the comment.
Steps to reproduce

Open the site https://www.greencity.social/#/greenCity;
Logged-in as User;
Click on the 'Eco news' tab;
Click on the News that includes a comment;
Click on the ‘View [number] replies’ list for the comment;
Click on the 'Delete' icon under User's reply;
Click on 'Yes, delete' button;

Actual result
The reply is still present in the ‘View [number] replies’ list until the page is refreshed.
https://github.com/ita-social-projects/GreenCity/assets/138223368/8afa67b5-b43e-431b-99f4-a5b969dc819f
Expected result
The reply is deleted from the ‘View [number] replies’ list and from the total number of replies.
Postconditions
User story and test case links
E.g.: "User story #100
Test case"
Labels to be added
"Bug", Priority ("high"), Severity ("minor"), Type ("Functional").

Can not reproduce

You need to have one reply in the ‘View [number] replies’

Okey, I will fix it


Updated Carthage examples to use XCF

Updated Carthage examples:

Bumped Themis version to 0.13.9 that updates OpenSSL to 1.1.1k
Updated test projects to use themis.xcframework
Updated Carthage CI tests to build Themis XCFramework: added --use-xcframeworks flag

Note: Bitrise doesn't support the required Carthage version 0.38 to build XCF properly. They are going to update to 0.38 about in a week: https://discuss.bitrise.io/t/how-to-upgrade-carthage-to-version-0-38-0/16796. If we are not in hurry, this PR can wait for the Bitrise update.
Checklist

[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has proper documentation
[x] Example projects and code samples are up-to-date (in case of API changes)
[x] Changelog is updated (in case of notable or breaking changes)

If we are not in hurry, this PR can wait for the Bitrise update.

I suggest merge it, and then in a week, tweak Bitrise tests


Script Fails to Complete with Error - Health Checker

Provide Version Number
<IP_ADDRESS>8
Describe the issue
While attempting to run the health checker there is a error stating parameter name cannot be found.
On line 1254 the parameter -DisplayCustomTabNumber 2 is incorrectly entered as -DisplayCustomTabNumbermber 2

Changing the parameter to -DisplayCustomTabNumber 2 had saving then re running the script works successfully.

Will be in the next release of the script.
Resolved #989

@tOADPie issue is resolved with the latest release.


Functions as promises

Would it be possible to also support the async/await syntax if no callback is provided?
const data = await getCLS();

Thanks for the suggestion. That's actually how the API was originally designed, but it was converted in https://github.com/GoogleChrome/web-vitals/pull/3 to use callbacks instead because there were a few cases where the callback would need to be invoked more than once—which is a shame because the promise based approach is so much simpler!


Is subclass between WEMI and Endeavor enough?

Or do we need relationships between them?

A point of clarification (I hope):  subclass / superclass relates one class to another (like Endeavor and Work); a relationship relates one instance to another (like this-endeavor relates to that-work).
It's certainly possible to imagine sort of plausible relationships where the domain is Endeavor and the range is Work.  For example, "this-endeavor"  "this work" -- but I'm not sure why this belongs in OpenWEMI, at this level of abstraction.

Thanks, @lagbolt. Where I get doubts is when my brain sees WEMI as parts of a whole, which would make them parts of the Endeavor whole, not subclasses. In that view, it would not make sense for an Expression to be a type of Endeavor, but only for the expression of the work to be subclassed to Endeavor. In the classic FRBR view, WEMI is like a nested Russian doll, with E being inside W, M being inside E, etc. (The same works backward, IMEW). In fact, the relationship is that E expresses a W, meaning that in a sense E is E+W, not E alone.
I think that our documentation needs to state that other than the relationships we provide there are no inherent or assumed relationships between the entities beyond the ones we provide, and that no entities are required in a model using openWEMI. But I still feel brain pain when I try to think of it that way.

The group agrees that the appropriate relationship is "subclass" and that Endeavor is a kind of organizing "bucket".


Breadcrumb picker typed text foreground/background/border settings

Issue Type: Feature Request
While one can customize the appearance of the foreground and background of entries that appear in the breadcrumb picker, the box that pops up to show text as one starts typing cannot be similarly customized as far as I can tell. Typed text can sometimes be hard to read because of poor color contrast. I imagine new settings that are similar in effect to editorWidget.foreground, editorWidget.background and editorWidget.border for the Find/Replace widget. Alternatively, these elements could be mapped to existing foreground, background and border settings.

VS Code version: Code - Insiders 1.45.0-insider (a0fe37870c42e0704a492cdc6b8550b4cf23f63c, 2020-04-30T05:39:08.281Z)
OS version: Windows_NT x64 10.0.18362

Colors are:

listFilterWidget.background
listFilterWidget.outline
listFilterWidget.noMatchesOutline
list.filterMatchBackground
list.filterMatchBorder

Oh, awesome! Thanks, @joaomoreno !


Clone not possible in Windows (directory structure too long)

I saw this even when downloading the zip and trying to manually extract the contents.

Seems to be from this whopper of a directory structure:
aem-phonegap-starter-kit/content-dev/src/main/content/jcr_root/content/phonegap/app_name_placeholder/shell/_jcr_content/pge-app/app-content/phonegap/plugins/cordova-plugin-camera/tests/ios/CDVCameraTest/CDVCameraTest.xcodeproj/*

@timdonovanuk, the plugins have been removed from the repository. Can you confirm if you're still having this issue?


Add sveltekit plugin to worker.plugins Vite config

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Intent
In order for web workers to import variables from files dynamically created by SvelteKit (e.g. $env/..., $app/...), we need to define the sveltekit plugin in our vite worker.plugins config.
Resolves #10370

Is it worth adding a line to documentation/docs/10-getting-started/30-project-structure.md to explain this?

This isn't the right fix for the issue. Adding the entire sveltekit() plugin also kicks off it's sequential server/client/workers build step, and we don't need those for workers. What we need is a trimmed-down worker-specific plugin that supports the aliases and other core features. And perhaps sveltekit() can append that plugin to worker.plugins internally.

This isn't the right fix for the issue. Adding the entire sveltekit() plugin also kicks off it's sequential server/client/workers build step, and we don't need those for workers. What we need is a trimmed-down worker-specific plugin that supports the aliases and other core features. And perhaps sveltekit() can append that plugin to worker.plugins internally.

SGTM! Will close PR

Apologies for necro-ing the thread, but what did end up being the solution here? Was sveltekit() ever changed accordingly? Running into this trying to use $app/paths in a Web Worker...

I'm not sure if this has been fixed. Locally, I've overcome the issue by adding the below lines to my vite.config.js:
const config = {
	...
	worker: {
		plugins: [sveltekit()],
		format: 'es',
	},
	...
};

Yeah, I had that going for a little while, but IIRC it also outputs some warnings that made me nervous (and the comment that it wasn't the best way to do it, too).
I've got it working for my needs using a define in my vite.config.ts file:
import { defineConfig } from 'vite';

export default defineConfig({
    ...
    define: {
        '__APP_BASE_PATH__': process.argv.includes('dev') ? `''` : `'${deploymentPath}'`
    }
});

But turning Vite into a preprocessor feels like a bit of code smell. Would be nice if there was some clear supported way of accessing $app/paths but I get that it's complicated. 🫤


Add trybuild tests for event_enum! macro

Not sure if these are super helpful but might as well test what we can.

I think the first error message could be improved easily. The assert could have a message like 'well-known matrix events have to start with m.'.


Implement Grokfast into GradCache

I would like to implement the algorithm for grokfast, which is an exponentially weighted mean of past gradients added to the current gradients, with GradCache. I've been able to use it without GradCache, but I'm confused where it could be implemented with GradCache as I'm still learning the underlying mechanisms of GradCache. Any direction on how this might be done? Also curious if this would be an appropriate feature to this library

an exponentially weighted mean of past gradients added to the current gradients

shouldn't this be tracked in the optimizer (state) and applied as a gradient transformation?

I'm not entirely sure. Just spitballing but I think it would be implemented somewhere within these lines of codes: https://github.com/luyug/GradCache/blob/main/src/grad_cache/grad_cache.py#L193C9-L211
and use similar logic to the following:
def gradfilter_ema(
    m: nn.Module,
    grads: Optional[Dict[str, torch.Tensor]] = None,
    alpha: float = 0.99,
    lamb: float = 5.0,
) -> Dict[str, torch.Tensor]:
    if grads is None:
        grads = {n: p.grad.data.detach() for n, p in m.named_parameters() if p.requires_grad}

    for n, p in m.named_parameters():
        if p.requires_grad:
            grads[n] = grads[n] * alpha + p.grad.data.detach() * (1 - alpha)
            p.grad.data = p.grad.data + grads[n] * lamb

    return grads

I'll test this out later when I get a chance


Provide guidance links in mix tasks' Logger.error output

We just added excellent_migrations to a large repository with many newish Elixir developers contributing to it. I noticed that the current Logger.error output can be somewhat cryptic, and it isn't clear what the next steps are or where the errors originate.
This pull request adds links to the README's 'Checks' and 'Assuring Safety' section to the error output of the mix tasks.
I haven't yet determined how to do the same for the credo check, but I'll happily add it if this first PR is welcomed.
Before

After

Thanks for the PR and using the lib!
I like your solution, because it quickly adds value and I'll be happy to merge your PR after testing.
I was thinking about introducing something similar myself. My idea was to add README links to specific dangers that were detected. Additionally these links would point to README from a lib version that is used.
For example column_renamed check is triggered when using excellent_migrations 0.1.6 – and the message would contain this link: https://github.com/Artur-Sulej/excellent_migrations/blob/v0.1.6/README.md#renaming-a-column
With this approach users are pointed to the right place in docs and links would always work even if changed in newer versions.
After adjusting links to match check names, links could be built with this logic:
def build_link(current_version, anchor) do
  anchor =
    check
    |> Atom.to_astring()
    |> String.replace("_", "-")

<EMAIL_ADDRESS>end

Anyway, that could be done in the future.

After adjusting links to match check names, links could be built with this logic:
def build_link(current_version, anchor) do
  anchor =
    check
    |> Atom.to_astring()
    |> String.replace("_", "-")

<EMAIL_ADDRESS>end

I actually started down a path of trying to build out links and aborted due to not knowing if you'd be interested in a bigger, but I hadn't considered tying the links to the particular release tag, which would be 👨🏻🍳😘.
Happy to take on the subsequent change once this first one goes in.

@Artur-Sulej I'm curious how you're feeling about this. I just got around to auditing my dependencies on a side project, and realized my fork was behind origin. These warnings have aided me personally :)


Accommodate tunneling proxies that close the connection after an auth challenge.

A fix for #2426. This was mainly a rejiggering of the existing connection code.

LGTM
This is brilliant. Very nice.

@dave-r12 / @swankjesse : Many thanks to you guys  :satisfied:


letsencrypt: 0.0.0.dev20151123 -> 0.1.0

Not sure why travis fails here - seems unrelated. This PR works fine and was merged in 67fda15a8915118cf92818f33d517b47f442a671.


Plugin does't report absense of href attribute and empty href attributes

Basically the following fails:

but the following passes:

also ,

why ?
If the support for the above 2 are also added , it would be of great help , as currently i have a requirement of testing for all 3 scenarios above.
Thank you.

I'm not sure what options you're using, but I can provide some background. The first example will trigger the noEmptyFragments rule because it's exactly the scenario that rule detects (fragment is present but empty). The second example isn't recognized as a link at all (code looks for a+href, img+src, etc.). The third is recognized as a link, but has no fragment, so won't trigger noEmptyFragments. Because it points to the current page, it's valid by definition.
Hope this helps!

@DavidAnson quite insightful , thanks . would still be great if my use case could be accomodated simply because  and  is often intended to be  , its hard to be meticulous and type out   :) , but ofcourse as you've pointed out going by the definition , the plugins right in ignorning my other 2 use cases ! .. but i'd still love if a feature could be added.
Thanks for your time.

I've captured your request in #5, so am closing this issue as we agree things are working correctly. Thanks!


Continuous Song Cycling

When playing any station Pydora will continuously print song names out to the screen, 4 at a time, over and over. No music plays. I've only tried this on one computer: Mac running 10.12.3 and the latest version of Pydora.

This sounds like a duplicate of #8.
It may be that pydora is connecting successfully to the Pandora servers (the server always just gives the next four songs to be played), but then a failure occurs when mpg123 tries to access and stream those files.
Have you tested mpg123 with anything else to verify that it has been installed successfully?

This seems to be a somewhat recurring issue on certain versions of Mac OS (which I haven't been able to pin down precisely). mpg123 will hang but will continue to accept input from the player just never playing anything which will cause the playlist to cycle. I did some POC work to try to detect this but it was quite unreliable and I can no longer repro the issue. At the very least #8 would help to prevent blowing past the API limits so I'll probably implement that soon.
I've also got a private branch that's a work in progress to make the pydora output back-ends pluggable. I'm doing some testing with VLC in headless mode that looks promising but I don't really have an ETA on that yet.
I'm going to try to repro this specific issue again so will leave this ticket open in case I find something.

I've added more details to issue #52 and I think that will address the root cause of this issue as well. I'm going to close this one and track all work on issue #52 so follow that for updates.


Identify Majority-Minority Districts

Add an indication that a user has drawn a majority-minority district.

Duplicate of #426


MdAutocompleteTrigger writeValue is asynchronous

Bug, feature request, or proposal:
Request for change (RFC)
What is the expected behavior?
The MdAutocompleteTrigger writeValue ValueAccesor method should be synchronous.
What is the current behavior?
The MdAutocompleteTrigger writeValue ValueAccesor method is asynchronous.
What are the steps to reproduce?
Simply implement the autocomplete and try to detect a change being reflected within the view upon updating the formControl through setValue.
Doing this requires a setTimeout coupled with wrapping the body of the it statement with async(() => { ... }) when writing unit tests. If you wish to detect the change being relayed to the input value.
This is caused by the fact that (I assume) the writeValue wraps the _setTriggerValue method in a Promise.resolve(...): https://github.com/angular/material2/blob/master/src/lib/autocomplete/autocomplete-trigger.ts#L179
What is the use-case or motivation for changing an existing behavior?
The motivation for this RFC is the fact that it complicates testing. If their are valid reasons to make the autocomplete trigger asynchronous then that's fine, but the documentation should reflect this fact.
Which versions of Angular, Material, OS, browsers are affected?
2.0.0-beta.2

@kara why is this async?


Removing redundant renewal btn

Done for AGPUSH-1503

@thradec @lfryc mind to review/merge ?

@lfryc addressed it - and ys, got rid of the commented code :cactus:

@lfryc ok to merge ?

testing now...

@lfryc ok to merge ?

code-wise it looks fine, but I wasn't able to test it locally yet, the DB setup process is now too paintful


Missing top position options in propTypes

Setting element position to 'top-right-aligned', 'top-middle-aligned' or 'top-left-aligned' works but throws errors in console.

Added these positions to the list of allowed positions, it'll be part of the next release whenever that is.


Update index.html : help by ?

Lazy typist as I am, I've added ? to invoke help

valid one! thanks :D


getPcodeOps(Address refAddr) method returns NULL because hfunction is NULL

I have been using a small python script to access the pcodes of each function using Ghidra scripting engine. The script works fine for small binaries(I think) but doesn't work for large binaries. It is because the hfunction returns NULL resulting in error(Screenshot below).
Ghidra Github: https://github.com/NationalSecurityAgency/ghidra/blob/master/Ghidra/Features/Decompiler/ghidra_scripts/ShowCCallsScript.java

This error comes after the hfunction == None as given below:

However, this was not happening with small binaries. I have tested this is 750 KB binaries where it was working fine. Whereas for 1350 KB, I was getting the error. (I can be wrong).
The python script along with both kind of binaries are kept in the ghidra repo given below. The large file is the one getting error whereas the smaller one gives a proper output with the python script.
https://github.com/Yash0day/Ghidrapython
I have tried increasing the Decompiler Timeout from 30 to 240 and still having the error.

no, your high_func is None, if hFunction was null in the snippet above it would exit early.

Do you know why is that happening?
high = res.getHighFunction() from my piece of .py code gives NONE. Hence, getHighFunction() is faulty.


Example of spawning off-screen chunks

I'm prototyping a concept where the overmap is much larger than the visible map, and is made up of chunk types. Some distance from the map edge, I want to generate adjacent chunks off-screen, add them to the map, and potentially spawn in enemies/items/etc. I'd like to spawn chunks a bit away from the character, to allow for things like enemy troops to spawn a chunk or two away and advance on the player's position if they are detected. So it wouldn't be just-in-time, but maybe a screen or two of tiles.
Is there an example of this? I'm not thinking it has to be complex--just take the random world example and spawn in adjacent off-screen chunks.
In the meantime, I'm curious about how this currently works. Does autospawn just handle creating the chunks for me from a group of tiles, or does it detect when the camera is about to move off-screen and create a new chunk? I'm not sure whether the current chunk events are meant to tell me that I should run my map generators and create/add a chunk myself, or if it's up to me to detect when a new chunk is needed and do that myself.
Thanks.

There is a sort of example of this with the redo of random_dungeon.  I also just opened up the API on tilemap so that you can access to .events() which will allow you to see when chunks are created / despawned, allowing for stuff to be spawned in upon chunk creation if it needs to be.

There is a sort of example of this with the redo of random_dungeon.  I also just opened up the API on tilemap so that you can access to .events() which will allow you to see when chunks are created / despawned, allowing for stuff to be spawned in upon chunk creation if it needs to be.


Add URL support

See https://gist.github.com/egmontkob/eb114294efbcd5adb1944c9f3cb5feda
Prior art in Haskell: vty

Thank you. I have a pull request in progress to implement this.

@sofia-m-a, could you test my pull request? If you have access to a Unix-like operating system, that would be particularly helpful.

This feature is added to ansi-terminal-0.11.3, now on Hackage.

Sorry for the late reply, but it works well for me :) Thank you


no prerenderedAt value when static page generated using ISR

Environment

Operating System: Darwin
Node Version:     v18.18.0
Nuxt Version:     3.8.1
CLI Version:      3.9.1
Nitro Version:    2.7.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, routeRules
Runtime Modules:  -
Build Modules:    -

Reproduction
https://github.com/silverbackdan/nuxt-isr-reproduction
On Vercel
https://nuxt-isr-reproduction.vercel.app/
Refresh pages to see content served from the server.
Describe the bug
The prerenderedAt value should be set I think when the static version has been generated on the vercel side too? I think it'd give us a definitive value to check to see when the version being served was generated. Is this possible?
Additional context
No response
Logs
No response

Hi, I've pushed some updates - I've also noticed that the prerenderedAt value is only set and then retained depending on the very first load. When the page then changes, would it be reasonable to expect this value to then update as on the client-side the next page will always then be client-rendered, or do you think this should be the responsibility of the developer?

P.s. mind if I sneak a quick question in as well? On an ISR load, do I need to do something specific with a fetch to an API to still be carried out even on a static/isr page load?

With ISR, there is no prerendering as such. (Prerendering is a stage in the build process after the Vue app is built with vite, and before the nitro server is built.)
prerenderedAt is also only with respect to the first page.
Implementing a renderedAt option for all pages should be very possible in your project, but I'm not sure we'd want that in Nuxt in case it leaks too much information for users.

Ah ok, I see - I thought perhaps under the hood it was calling the same rendering process.
No problem if it isn't something that would be useful within the Nuxt core I'm happy to close and figure out my method for a renderedAt variable in context of my appllication
Thanks for your time.


Is the second argument (source) of applyChange superfluous?

From the interface point of view the applyChange function should not require the source object used while taking the diff. Therefore I found it confusing when I was trying it. I found that passing the second argument as empty object did not change the behavior. So when I looked into the code, the source argument is indeed not being used anywhere (except the check).
https://github.com/flitbit/diff/blob/master/index.js#L248-L272
Do you plan to remove it in future? It's important to know this because in my use case I compute the diff, save it to persistent storage and in future apply it on an object to get the original object. i.e. I don't have the source object in that scenario.
Thanks.

Indeed, source appears to be unused... I even recall using true as arg 2 a time or two... it smells, I know.
The intent, which is probably obsolete, was to provide another seam/callback by which the caller could do more sophisticated analysis of the change based on the origin — but that work never made it out of my brain cavity into the module.

still an issue.


RangeError: Maximum call stack size exceeded While setting min-max date

Hi,
I am facing Maximum call stack size exceeded issue while setting min-max date for date picker.
I am having two date picker (stare date, end date). When I am selecting start date I am setting mindate for end date picker and when I am selecting end date I am setting maxdate for start date picker.
Provided the sand box for the same.
https://codesandbox.io/s/lrnmx4qppl
Below I am providing the code for the same:
Flatpickr Component
import React, {Component} from 'react';
import Flatpickr from 'react-flatpickr';
import 'flatpickr/dist/flatpickr.min.css';
import 'flatpickr/dist/themes/light.css';

export default class DatePicker extends Component {
  constructor(props) {
    super(props);
    this.setDate = this.setDate.bind(this);
    this.setDatePickerProps = this.setDatePickerProps.bind(this);

    this.state = {
      selectValue: props.selectValue ? props.selectValue : "",
      options: props.options ? props.options : {},
      placeholder: props.placeholder ? props.placeholder : {}
    };
  }

  componentWillReceiveProps(newProps){
    this.setState({
      selectValue: newProps.selectValue ? newProps.selectValue : "",
      options: newProps.options ? newProps.options : {},
      placeholder: newProps.placeholder ? newProps.placeholder : {}
		});
  }

  setDatePickerProps(propKey, propValue){
    this.refs.refDatePicker.flatpickr.set(propKey, propValue);
  }

  clearDate(){
    this.refs.refDatePicker.flatpickr.clear();
  }

  setDate(newValue){
    this.setState({
			 selectValue: newValue ? newValue : ""
		});
    if(this.props.onChange){
      this.props.onChange(newValue);
    }
  }

  render(){
    return (
      <Flatpickr className="form-control clickable" ref="refDatePicker" placeholder={this.state.placeholder} options={this.state.options} value={this.state.selectValue} onValueUpdate={this.setDate} />
    );
  }
}

Below is the code for setting min and max date on parent component.
onFilterDateChange(str, date){
    if(str === "start"){
      this.refEndDatePicker.setDatePickerProps("minDate", date[0] ? date[0] : "");
      this.setState({
        objSearch: Object.assign({},this.state.objSearch,{ startDate: date[0] ? date[0] : ""})
      });
    }
    else if(str === "end"){
      this.refStartDatePicker.setDatePickerProps("maxDate", date[0] ? date[0] : "");
      this.setState({
        objSearch: Object.assign({},this.state.objSearch,{ endDate: date[0] ? date[0] : ""})
      });
    }
  }

When I select start date everything works fine but as soon as I select end date after that it shows this error "RangeError: Maximum call stack size exceeded"
Can anyone please help me out with this.
Your Environment
flatpickr version used: 3.6.4
Browser name and version: Chrome, Version 70.0.3538.110 (Official Build) (64-bit)
OS and version: Windows 10 Pro

I had this error too and for me the following was the problem:
The value is already set but in the first render it was null so flatpickr just used the current date which for me was BEFORE the minDate I set. That caused the component to not show any date and crash as soon as you try to change it.

In general when the value is before the minDate it crashes

@timojokinen Can you please let me know how did you fix this?

Solve validation in another way. I just removed the minDate property and added my own validation (for Example disabling submit button in a form or adding a red message below the component)


Training result is abnormal without pre-trained weights.

Hi,
Thanks for sharing your Yolo_v2 implementation using tf 2.0, first.
I trained this neural network after remarking "Load Pre-trained weights" code lines.
But both main and validation loss is not decreased at all. I got a loss graph looks like spike.
I think training is not proceeded correctly. Would you please give me an advice to solve this problem?
For example, I need to adjust hyper parameters, and so on.

Hi,
I tried to train the model without pre-trained weights. Train loss decreases but validation loss increases. I am not sure it'is possible to train yolo v2 on our own dataset without pre-trained weights. According to the original article, the author explains that the pre-trained weights come from a classification model trained on ImageNet.

Hi,
Thanks for your comment. I misunderstood code lines for loading pretrained weight (output for inference) that you have already been training with your data. I got it. I'll restore remarked code lines again and train it with custom data. By the way, you said train loss decreases. But I didn't see it. In my case, both train and validation loss is abnormal. I didn't understand your codes correctly. I will try again.

Hi, if your train loss increases or is unstable, you can try a lower learning rate.

OK! I will try. I learned many important points what I didn't know from
your code. Your code is very clean and nice. Thank you very much.
2020년 4월 25일 (토) 오전 4:20, jmpap<EMAIL_ADDRESS>작성:

Hi, if your train loss increases or is unstable, you can try a lower
learning rate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jmpap/YOLOV2-Tensorflow-2.0/issues/6#issuecomment-619196797,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG43MBN7PWKJBOJ7YW7FTZTROHRA5ANCNFSM4MN5DZIA
.

Thank your for your appreciation!


Sharepoint List folder Action not encoding URL Properly

Describe the Bug with repro steps
Previously the action would encode the URL with percent encoding (%252f, %2b) for the "File Identifier" param. Now the action does not seem to be doing this and it is resulting in 404 errors. I've worked around the issue and verified this is the problem by manually encoding the File Identifier with percent encoding and getting a successful run.
What type of Logic App Is this happening in?
Consumption (Portal)
Which operating system are you using?
MacOS
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Chrome
Additional context
No response

@okiejake - I am not able to repro this issue with List Folder action in LA consumption (east us)

We get this information from sharepoint and we use that value itself, there is no extra encoding done on this content by designer. if you still see this issue, please send me your network traces captured before you start browsing and select the folder up till then and share with me<EMAIL_ADDRESS>

@preetriti1

select folder by browser
switch to code make sure that the path is encoded
then try to modify the path by hand
then switch to code view and note the  encoding is not applied

expected result
the value should be encoded

@preetriti1

select folder by browser
switch to code make sure that the path is encoded
then try to modify the path by hand
then switch to code view and note the  encoding is not applied

expected result the value should be encoded

Thank you @mbarqawi, that is the issue we are facing exactly. In our case we can not select the correct folder in the browser tool. Once we go more than 3 folders deep the site crashes. As a solution we had manually updated the folder path using the editable field in the step but it gets rid of the encoding.

@okiejake @mbarqawi - The issue is what you see is just a displayName in folder picker... there is an actual value which your sharepoint list says it has that is the encoded path... client would not know how to encode it... so sharepoint needs to encode and send it back
if you are seeing the crash we will have to work on that bug, but doing the manual editing you take it away from UI and your edited value will be taken as source of truth.... the encodeUriCOmponent maybe can encode it during your runtime, but once user edits it we should not magically encode it to main user edited value

@okiejake - The designer treats all path parameters like file identifier here are always added a default encoding which API specifies, that is why we add the encodeURIComponent in designer since in runtime it will be used that way...
Anything else cannot be added without proper definition from API connector for specific parameter, that is how the design is. To cater to your ask I would suggest to open a ticket with Sharepoint connector and see what they suggest, this wont be just Logic Apps at this moment.
I will be closing this issue now.


Creating tileset assumption fix

Fixes #3404
I believe this fixes the issue, could you take a look before I write some tests.

The code looks mostly good. There may be an issue if the url is given as a directory without a trailing slash, but you can check for that in the tests.

Do you want to also address https://github.com/AnalyticalGraphicsInc/3d-tiles/pull/53#issue-126865190 in this PR?

Updated.

Updated.

If I don't check for the training slash and there already is one then I would have two slashes, wouldn't I?

appendForwardSlash handles that case.

Oh, gotcha. That's cool.

Updated.

I tweaked the test descriptions a little bit, but this is good to go. Merged.

Thanks Sean


Support MBTiles

Hello,
I wonder if this library support reading, writing and displaying MBTiles.
MBTiles files can be generated using Java software MOBAC (MOBile Atlas Creator by fetching various map source and storing maps offline.
MBTiles files can be displayed using Qt/C++ software GPXSee which can also handle WMS (Web Map Service) and WMTS (Web Map Tile Service).
Kind regards

Thanks for the details! Yeah, I think it is in scope for this package to support "tile sets" (of which the https://github.com/mapbox/mbtiles-spec is a very popular one).

https://github.com/mapbox/node-mbtiles/blob/master/lib/schema.sql provides a SQLite schema (create statements) for such a database.
You can find at:
https://github.com/perrygeo/python-mbtiles/blob/master/mbtiles.py
https://github.com/consbio/pymbtiles
some Python implementation


Channel Points - Message Highlights, Etc..

What should be added?

Channel Points - Message Highlights

In general, a message should be shown when channel points are used.
Regardless of what the reward is that is being redeemed.
Message Highlights was just a good example.
Why should it be added?

Because Channel Points are something that is used on nearly all channels.
A streamer should be able to see the rewards viewers redeemed with channel points.

Currently there is no visual indication at all in Chatterino when a user redeems channel points.
This makes Chatterino hard to keep using as a streamer since these indications are very valuable.
The Twitch API > Channel Points Event Message > Exists:
https://dev.twitch.tv/docs/pubsub#example-channel-points-event-message
Screen Shots for further explanation:
Stable Build:

Nightly Build:

i'm pretty sure this is resolved in latest nightlies

Indeed it does, although there is no message about the channel points in general.
Which means a message is highlighted, with no explanation

as to why.
It also does not add a message when other channel points are used either.

Probably fixed by https://github.com/Chatterino/chatterino2/pull/1809

Fixes in #1809
The only thing we do differently from Web Chat now really is the way we display "Highlight My Message" rewards - they're just highlighted with no header.


Fix PHP 7 compatibility

Wow @davidcochrum, this is an incredible PR!
I'm hesitant to split the cli into other libraries though (outside the command line cli) because then they could start diverging from each other.
As a php dev what are your thoughts?
Did you know you could also install via homebrew rather than using npx? https://www.dotenv.org/install/

I don't mind maintaining parity between the PHP and Node CLIs ... I don't anticipate them changing all that much anyhow ... I'm working on projects that don't have any other need for Node and my goal was to avoid installing it just for managing envs

I think I just realized that I don't need the CLI in any environment besides dev machines ... is that correct?

Hi @davidcochrum, sorry to do this to you but I am going to close this.
I made a slew of updates yesterday to fix this library up.
For any changes you would still like to add, could you re-add them under a separate PR. There are too many conflicts here to make it worthwhile. Sorry again for throwing out your hard work like this.

@davidcochrum I put together a Laravel guide here:
https://www.dotenv.org/docs/frameworks/laravel/heroku
But I also noticed you started building a Laravel package here:
https://gitlab.com/davidcochrum/dotenv-vault-laravel
Would you mind updating that to the latest phpdotenv-vault v0.2.x. Then I'd like to update our Laravel docs to use your plugin.
Also, would you consider hosting that code on GitHub instead of GitLab? We find the community can engage better over GitHub for PRs, comments, etc.

Updated and released v1.1.0
https://gitlab.com/davidcochrum/dotenv-vault-laravel/-/releases/v1.1.0
I have GitLab mirroring to GitHub so any community discussion/contribution will be synced

I'll work on updating our guides to use your Laravel plugin this evening!
Also could you take a look at this:
https://www.dotenv.org/blog/2023/11/07/phpdotenv-is-inconsistent-across-development-and-production.html
As a more experienced php dev than me, I'd like your take.

There's a whole paragraph in the phpdotenv README about thread safety with getenv()  and putenv()
In my experience, most devs don't interact with this library directly, but instead do so indirectly via their framework of choice. Laravel uses the library with safeLoad() and its own configured repositories.
Symfony uses its own library that behaves a bit differently and is completely independent from phpdotenv

Symfony's implementation looks more sane.


Added a new dataset

the data set is on Glaucoma right? if so would be great to have more info on what the file is in the file name.


Update to Chromium 98.0.4758.80

This PR updates Chromium to 98.0.4758.80.
Notable changes:

enable_nacl_nonsfi has been removed: (1273132)
block-trk-and-subdomains.patch has been updated due to the removal of DISALLOW_COPY_AND_ASSIGN 3174408
New additions to the safebrowsing patch due to [3306199], [3327674], [3254424]
A chunk of the fixes for building without reporting enabled have been implemented upstream 3306452
add-flag-to-hide-crashed-bubble.patch & google-api-warning.patch have been updated due to [3279686]
0006-modify-default-prefs.patch has been updated with the changes made in [3296854]
There were some code health changes that affected remove-unused-preferences-fields.patch, I updated those sections to remove the code rather than returning early.

Other notes:
The flag for the old settings page has been removed:  [3311012], [3320038]
The share pedal in the omnibox can be disabled in chrome://flags/ under "Desktop Sharing Hub in Omnibox" (#sharing-hub-desktop-omnibox) or by launching with --disable-features=SharingHubDesktopOmnibox

Fixes #1816
Blog post
Change log

Builds and runs on macOS x86_64 – Thank you.

@kramred Have you tried editing share_targets.asciipb locally and recompiling?

@kramred Have you tried editing share_targets.asciipb locally and recompiling?

Yes, I have now. When I replace all occurrences of "Facebook" with e.g. "noyb" and edit the url and icons in share_targets.asciipb and do a "dirty" build from that it seems to work:

Official Gentoo does it this way:
https://github.com/gentoo-mirror/gentoo/blob/481eead7384aa7c78229a85d2d7cf0065b0c7a8d/www-client/chromium/chromium-98.0.4758.80.ebuild#L829
	mv out/Release/chromedriver{.unstripped,} || die

🤷

Ah alright, well I'll just do it that way then

Any objections to merging this?

@Zoraver Are you planning to address your concerns in another PR after we merge this one? Alternatively you could simply write your comments here or in another improvement-issue.

Since nothing serious was found, I think, we are free to merge.

Just FYI: Since v98 cross compilation on macOS_x86-64 for macOS_arm64 kept failing with LLVM/Clang ≤ 13.0.1 due to the linker lld throwing errors. I had to compile/use a more recent version of lld to be able to build for macOS_arm64 on an macOS_x86-64 system (GItHub Actions build is still running but it got past the point where the linker error occurred)
So, if you're cross compiling and get errors due to the linker try a newer version of lld


Remove extra type casts in Unicode.swift

I believe this was done because the typechecker considers this a particularly complex expression.  Do you have numbers on whether the build time changed or is this is just no longer rejected for being too complex to solve.

Possibly related https://bugs.swift.org/browse/SR-139


Error at creation using lattice with positon at 0,0,0

When using the lattice command  with a BCC lattice or with a custom lattice with a particle position set to (0,0,0) exactly, the distribution of that particle to cells does not work and leads to crash.
In BCC it's the same because one of the particle is at (0,0,0) and the other at (0.5,0.5,0.5) in reduced position.
Creating a branch for that specific problem which needs to be solved at the exaNBody level.

Crash with exaStamp example in exaStamp/data/regression/lattice/generate_bulk_lattice.msp
The example works if shift: [1e-12,1e-12,1e-12] is appended to the bulk_lattice YAML block.


Added support for OAuth Authorizations API.

GitHub API docs: https://developer.github.com/v3/oauth_authorizations
Fixes #348.

See https://github.com/google/go-github/blob/master/github/authorizations.go.  Is this PR adding anything new?

err, sorry.  Didn't look at #348 closely :).  In any event, this should be added to the existing AuthorizationsService, not a new one.

d'oh! Totally missed the AuthorizationsService.
The List/Get/Delete Grant endpoints are new in this file. I'll go ahead and merge those into authorizations.go.

PTAL

LGTM.

Incorporated in https://github.com/google/go-github/commit/087a452730710d419f41a79043bc5e880f171e04


Enable usage of GCS URL as Module.Source

Add GCSBucket to list of available detectors and getters;
Rename GitSourceReader -> GoGetterSourceReader.
Update go-getter dependency v1.7.2 -> v1.7.3

https://developer.hashicorp.com/terraform/language/modules/sources#gcs-bucket
Usage:
source: gcs::[https://www.googleapis.com/storage/v1/<bucket>/<path_to_module_dir](https://www.googleapis.com/storage/v1/%3Cbucket%3E/%3Cpath_to_module_dir)>

Note:

Doesn't support package-notation (//) at the bucket level,
a dir in root of the bucket is highest possible level for a package.

Testing done:
$ gsutil ls gs://<bucket>/dir_1/dir_2
gs://<bucket>/dir_1/dir_2/
gs://<bucket>/dir_1/dir_2/custom-image/
gs://<bucket>/dir_1/dir_2/pre-existing-vpc/
gs://<bucket>/dir_1/dir_2/vpc/

---
blueprint_name: tst
vars:
  project_id:  <project>
  deployment_name: tst
  region: us-central1
deployment_groups:
- group: primary
  modules:
  - id: mod
    source: <source>

$ make && rm -rf tst && time ./ghpc create test.yaml

$ make && rm -rf tst && time ./ghpc create test.yaml

* `gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/vpc`](https://www.googleapis.com/storage/v1/%3Cbucket%3E/dir_1/dir_2/vpc%60) _no package_
```sh
...
real    0m1.536s
$ terraform -chdir=tst/primary init
...
Downloading gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/vpc](https://www.googleapis.com/storage/v1/%3Cbucket%3E/dir_1/dir_2/vpc) for mod...
- mod in .terraform/modules/mod
... # OK

gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/vpc/](https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/vpc/`) ends with slash

...
real    0m1.538s
...
$ terraform -chdir=tst/primary init
...
Downloading gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/vpc/](https://www.googleapis.com/storage/v1/%3Cbucket%3E/dir_1/dir_2/vpc/) for mod...
- mod in .terraform/modules/mod
... # OK

gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1//dir_2/vpc](https://www.googleapis.com/storage/v1/<bucket>/dir_1//dir_2/vpc`) package

...
real    0m2.112s
...
$ terraform -chdir=tst/primary init
...
Downloading gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1](https://www.googleapis.com/storage/v1/%3Cbucket%3E/dir_1) for mod...
- mod in .terraform/modules/mod/dir_2/vpc
... # OK
$ ls tst/primary/.terraform/modules/mod/dir_2/
pre-existing-vpc  vpc

gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1//dir_2/vpc/](https://www.googleapis.com/storage/v1/<bucket>/dir_1//dir_2/vpc/`) package, ends with slash
same as above

gcs::[https://www.googleapis.com/storage/v1/<bucket>//dir_1/dir_2/vpc](https://www.googleapis.com/storage/v1/<bucket>//dir_1/dir_2/vpc`) package separator after bucket name

Error: failed to get module at gcs::[https://www.googleapis.com/storage/v1/<bucket](https://www.googleapis.com/storage/v1/%3Cbucket)> to /tmp/get-module-655452250/mod: error downloading '[https://www.googleapis.com/storage/v1/<bucket](https://www.googleapis.com/storage/v1/%3Cbucket)>': URL is not a valid GCS URL
11:     source: gcs::[https://www.googleapis.com/storage/v1/<bucket>//dir_1/dir_2/vpc](https://www.googleapis.com/storage/v1/%3Cbucket%3E//dir_1/dir_2/vpc)

go-getter problem:
$ go-getter gcs::[https://www.googleapis.com/storage/v1/<bucket>//dir_1/dir_2/vpc](https://www.googleapis.com/storage/v1/%3Cbucket%3E//dir_1/dir_2/vpc) ~/tmp/ll
2023/10/09 23:42:36 Error downloading: URL is not a valid GCS URL

gcs::[https://www.googleapis.com/storage/v1/<bucket>/missing_dir](https://www.googleapis.com/storage/v1/<bucket>/missing_dir`) missing dir

Error: lstat /tmp/get-module-2223695200/mod: no such file or directory
12:     source: gcs::[https://www.googleapis.com/storage/v1/<bucket>/missing_dir](https://www.googleapis.com/storage/v1/%3Cbucket%3E/missing_dir)

This is a issue with<EMAIL_ADDRESS>the DirMode prevented raise of an error, after update to v1.7.3 WAI:
Error: failed to get module at gcs::[https://www.googleapis.com/storage/v1/<bucket>/missing_dir](https://www.googleapis.com/storage/v1/%3Cbucket%3E/missing_dir) to /tmp/get-module-3980916664/mod: storage: object doesn't exist
12:     source: gcs::[https://www.googleapis.com/storage/v1/<bucket>/missing_dir](https://www.googleapis.com/storage/v1/%3Cbucket%3E/missing_dir)

wut::kpss://wrong_protocol wrong protocol

Error: failed to get module at wut::kpss://wrong_protocol to /tmp/get-module-3977420508/mod: download not supported for scheme 'wut'
12:     source: wut::kpss://wrong_protocol
                ^
real    0m0.070s

---
blueprint_name: tst
vars:
  project_id:  <project>
  deployment_name: tst
deployment_groups:
- group: primary
  modules:
  - id: mod
    kind: packer
    source: <source>
    settings:
      subnetwork_name: any
      zone: danger

$ make && rm -rf tst && time ./ghpc create test.yaml

gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/custom-image](https://www.googleapis.com/storage/v1/<bucket>/dir_1/dir_2/custom-image`) no package

real    0m2.002s # OK
$ tree tst/primary/
tst/primary/
└── mod # + hcl files

gcs::[https://www.googleapis.com/storage/v1/<bucket>/dir_1//dir_2/custom-image](https://www.googleapis.com/storage/v1/<bucket>/dir_1//dir_2/custom-image`) package

real    0m4.980s # OK

$ tree tst/primary/mod/
tst/primary/mod/
└── dir_2
    ├── custom-image     # + hcl files
    ├── pre-existing-vpc # + tf files
    └── vpc              # + tf files

missing dir & wrong protocol yield the same result as Terraform test cases; WAI.

@mr0re1 can you add link to module docs in description?
https://developer.hashicorp.com/terraform/language/modules/sources#gcs-bucket
It's unclear to me if not supporting packages is a design decision or a limitation of Terraform support for GCS? I understood plan to be to pass through whatever Terraform could take (and reject GCS entirely for Packer modules)

@tpdownes

It's unclear to me if not supporting packages is a design decision or a limitation of Terraform support for GCS?

It's GoGetter limitation, I wasn't able to fetch GCS URL with // in it.

reject GCS entirely for Packer modules

Is it something we want to limit intentionally? Shall we add an extra restrictions if it already works "out of the box", though without support for //?

Hrm. What I may be seeing is extreme slowness when it encounters hidden directories with lots of files (e.g. .git .terraform). What should I see if I use // @mr0re1 ?

@tpdownes  I've addressed your comments, added a test-log to the description. Could you please take another look?

lstat error

Addressed

Doesn't support packages

Clarified that it doesn't support it at the bucket level, bucket itself can't be treated as a package, though directory within can;

Packer specificity

Haven't found any differences in Packer VS Terraform use cases.

Trailing slashes

Seem to work fine.

Extremely slow

Added a time, seems to be reasonable

Updated README


Add contributing section with steps on how to add a new resource

What problem are you facing?
Adding a new resource to tf based providers is pretty straightforward, which requires some minimal configuration. However, there is no document describing these steps.
How could Crossplane help solve your problem?
Add a contributing section to Readme with the required steps to add a new resource.

Closing via #82.


nested same component

Laravel Version: 5.4.x-dev
PHP Version: 7.1.0RC3
Database Driver & Version: none

Description:
When I try to nest the same component I've go an error
ErrorException in ManagesComponents.php line 79:
Undefined index: components.card (View: /path/to/laravel-54/resources/views/welcome.blade.php)

Steps To Reproduce:
@component('components.card')
	@slot('title')
		ma box 1
	@endslot

		@component('components.card')
			@slot('title')
				ma box 3
			@endslot

			plop 3
		@endcomponent

@endcomponent

Does this happen on 5.3.x too please?

Does clearing the view cache fix it?

If I'm not wrong, the "component system" is new to laravel 5.4, right ?
Anyway I just tested on Laravel 5.4.
I started a fresh install of laravel and tried the exemple above right away

How is your components card component file?

Component card

{{ $title }}

		{{ $slot }}

Ideally, I guess it should throw exception components.card already declared or is been in use.

This doesn't seem to happen in latest 5.4.

thank you @themsaid (if you are the savior)  it's working perfectly now !

Nope, I just pulled to test and found it working :D


fix demo thumbnails

as title says. this is temporary until the Ecosystem team rolls out the long-term plan for static content

lgtm


Support for Kubernetes 1.24

Describe the solution you'd like
The latest K8s release being 1.24.3 (https://kubernetes.io/releases/), we should support v1.24 minor series and update our e2e tests to be at par with the greatest minor release available.
Anything else you would like to add:
Update README and other relevant documents regarding K8s version support

Adding support for K8s v1.24.6


Does anyone is maintaining this repository?

If someone is maintaining this repo and if they have any documents for how this project started and which document is prepared and way to extend this demo it would be very helpful to understand and learn from all efforts done for this repository.
If we get some documents then it would be very helpful.

Closing all issues since this repo is being archived and no longer maintained.


Getting ground truth info from the simulator

As we talked in tasts-robots/upkie#104, could you give more context on how to obtain ground truth information from the simulator to extend Upkie's observation dictionary? Particularly the height of a torso-attached point. Thank you!

What differentiates simulation data from other sensory data is that we collect it inside the interface (as opposed to e.g. CPU temperature or joystick inputs which are read in the observer pipeline). This is not a blocker though, we can proceed similarly to the way IMU data is currently collected.
Sketch on how to proceed
I see mainly six steps:

In BulletInterface::cycle, call a new function to read extra data from the simulator:

  read_joint_sensors();
  read_imu_data(imu_data_, bullet_, robot_, imu_link_index_, params_.dt);
  read_extra_data(...);  // new

This data could have a custom ExtraData type similar to the ImuData implemented in Interface (for the pi3hat interface it would default to an empty class)
Add a latest_extra_ attribute to the Spine class in Spine.h:

  std::vector<actuation::moteus::ServoReply> latest_replies_;

  //! Latest IMU data. It is copied and thread-safe.
  actuation::ImuData latest_imu_data_;

  //! Latest extra (e.g. simulation) data.
  actuation::ExtraData latest_extra_data_;

Collect this new observation upon receiving a new actuation output, in Spine.cpp:

  // 3. Wait for the result of the last query and copy it
  if (actuation_output_.valid()) {
    // ...
    latest_imu_data_ = actuation_.imu_data();
    latest_extra_data_ = actuation_.extra_data(); // new
  }

Convert this output to a dictionary when cycling the spine, in Spine.cpp:

void Spine::cycle_actuation() {
  try {
    // 1. Observation
    Dictionary& observation = dict_("observation");
    observation::observe_time(observation);
    observation::observe_servos(observation, actuation_.servo_joint_map(),
                                latest_replies_);
    observation::observe_imu(observation, latest_imu_data_);
    observation::observe_extra(observation, latest_extra_data_);

With this pipeline, the spine can forward data computed within Bullet to the output observation dictionary.

This feature has been shipped in v2.5.0:

BulletInterface: Report the base state that can be used for training RL agents.

Here is an example reading the base height (the base frame is attached to Upkie's torso) and tuning some gain (using mpackview and valmix):

Hoping this helps!


Edge runtime experiment

Try edge runtime for base-org pages
fix datadog CSP issue:

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Port over Statistics extensions

Port over Numerics/Statistics extensions for f64 slice and iterable

Completed port of slice functionality (ArrayStatistics), closing this issue as the slice implementation is the only one that will be making it into 0.2. Will open up another issue for further implementations


More lazy fetching to avoid N+1 problems

Related Issue or Background Info
We've had some performance issues with N+1 querying. Trying to get ahead of those by a little.
Changes Proposed

lazy-fetch audit users for all the entities that have them

Friendly reminder to merge main to this PR, as new lint checks/configs were added by #885 and #995!

Already done elsewhere I think.


Update HWIOAuthExtension.php

Temporary workaround

Hey @cameronmurphy could you please give me write access to this bundle or if not then accept this pull request when you have some time? Many thanks.


Add more info to the metadata_measure docstring

Codecov Report

Merging #683 (91449c7) into dev (085af65) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #683   +/-   ##
=======================================
  Coverage   85.29%   85.29%
=======================================
  Files          39       39
  Lines        3414     3414
=======================================
  Hits         2912     2912
  Misses        502      502

Impacted Files
Coverage Δ

src/measures/meta_utilities.jl
96.66% <ø> (ø)

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 085af65...91449c7. Read the comment docs.

Closing in favour of https://github.com/JuliaAI/MLJBase.jl/pull/684 .


Flightgear

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Might tick of the requirements for https://github.com/NixOS/nixpkgs/pull/77799

@GrahamcOfBorg eval

@GrahamcOfBorg build flightgear

It won't build. Blacklisted because of size. Have to build locally to confirm.


Need to change path of ecs_file_sd.yml ?

Hi ,
I need to change default path of ecs_file_sd.yml , by default it set on root location i read docs and get to know that it can change by -config.write-to.
But i am getting error , could you please let me know the full docker command for the same.
It would be so helpful for me.

use: --config.write-to


There are no commands defined in the "ds" namespace

hi
I installed laradumps by composer on my laravel project and installed windows app on my system. then i ran  "php artisan ds:init" but this message back to me:
ERROR  There are no commands defined in the "ds" namespace.
how can I fix that? please guide me.
thanks.

Hello! Some things changed in version 2. Please see the upgrade guide
https://laradumps.dev/get-started/upgrade-guide.html

Hello there
Thanks for your email. I followed steps in version 2 but I installed
windows desktop v1.7.2 and that don't show any debug in its window. If I
have to installation the new version,please let me know. By the way, this
link for the new version returns back error 404.
https://github.com/laradumps/app/releases/download/v2.1.0/LaraDumps-Setup-2.1.1.exe
thanks.
On Sun, Jul 9, 2023 at 5:16 PM Luan Freitas @.***>
wrote:

Closed #119 https://github.com/laradumps/laradumps/issues/119 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/laradumps/laradumps/issues/119#event-9767262826, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIMPR5NQHKCRTNG6XI7GY23XPKYZ3ANCNFSM6AAAAAA2DN5PGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add trimming to rendered message and message template

Add separate functions to trim the rendered message and message template text when the corresponding database fields are too small for saving the data, implementing #209.
Add unit tests to check the trimming effect when called.
Update README.md with explanation about the trimming effect.

@ckadluba My apologies to the change of the quote in README.md file. Apperently JetBrains Rider didn't like the original quote and replaced it without me really knowing. I changed that to a straight single quote. That's OK for you?

There was a nagging feeling that the README was not correct, so I checked it again. I think that a forward tick was missing and I believe the text is properly rendered now.
At the same time I switched around the maximum length comparisons.

@studiopasokon thank you for the contribution. I think it looks very good. Only two more additions in the README I would suggest, then it can be merged from my point of view.

Here you go. I hope that I fixed the right things. If something is still not correct, just let me know.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.11 to before v5.4.176",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=c0cdc19f84a4712cf74888f83af286e3c2e14efd",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=da27b834c1e0222e149e06caddf7718478086d1b"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "c0cdc19f84a4712cf74888f83af286e3c2e14efd",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "da27b834c1e0222e149e06caddf7718478086d1b",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "rpmsg: char: Fix race between the release of rpmsg_ctrldev and cdev\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.176 by commit da27b834c1e0222e149e06caddf7718478086d1b, it was introduced in version v4.11 by commit c0cdc19f84a4712cf74888f83af286e3c2e14efd. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000154


BUG: Fix ConjugateGradientFRPR overrides PowellOptimizer, FRPROptimizer

There was a breaking change in the API of itk::PowellOptimizer and itk::FRPROptimizer, with ITK commit https://github.com/InsightSoftwareConsortium/ITK/commit/a7b0a734cf79ac82e70a61321b41ccfee4d5feec "ENH: Eliminates variable construct/destruct", Stephen R. Aylward, April 23, 2008: It added overloads of the virtual member functions, itk::PowellOptimizer::LineBracket, itk::PowellOptimizer::BracketedLineOptimize, and itk::FRPROptimizer::LineOptimize, which have an extra parameter, ParametersType & tempCoord. The old virtual member functions were then rewritten in terms of the corresponding new overloads.
As a consequence, it does not make sense anymore for elastix::ConjugateGradientFRPR to override the old virtual member functions of ITK. With this commit, it overrides the new overloads instead, with the extra ParametersType & tempCoord parameter.
This fix aims to restore the behavior of ConjugateGradientFRPR as it was intended. (ConjugateGradientFRPR was introduces with commit 1c64e2af588d0b996c4ddbe582d2a1c7a7c98dd0, Stefan (@stefanklein), 20 April 2006.)
The bug was found when trying to fix -Woverloaded-virtual warnings from GCC at https://my.cdash.org/index.php?project=elastix like:

itkFRPROptimizer.h:126:3: warning: 'virtual void itk::FRPROptimizer::LineOptimize(itk::FRPROptimizer::ParametersType*, itk::FRPROptimizer::ParametersType&, double*, itk::FRPROptimizer::ParametersType&)' was hidden

Follow-up to pull request #1118

@stefanklein Again (after pull request #1118) this looks like a major bug fix of ConjugateGradientFRPR (with the bug being introduced by an upgrade of ITK, back in 2008). It seems to me that this fix must be applied, in order to get the intended result from ConjugateGradientFRPR. But I guess this optimizer component is rarely used anyway, right?
You wrote about this component in the log message of commit 1c64e2af588d0b996c4ddbe582d2a1c7a7c98dd0 (20 April 2006):

Added the ConjugateGradientFRPR optimizer. It was already sitting in my elastix-tree for a while.This is a fully functional fletcher reeves conjugate gradient optimizer, using a dBrent exact line search, based on the description in Numerical Recipes in c++. It's not  a really good optimizer, but it works, and is known. The exact line search might proof useful somewhere/somewhen.

Anyway, I guess it's OK to merge the fix.


Added code for checking palindrome number.

Why:
Closes [Palindrome number #191]
What's being changed:
Added code for checking palindrome number.
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Hey @NeelPatel31
Thanks for your contributions to this project 😃
This is a really good PR 👍🏻
This PR will be reviewed and merged soon

Preview : https://codinasion--pr30-dev-6lg83kx6.web.app/programme/palindrome-number/
(expires Sat, 12 Mar 2022 01:05:26 GMT)

LIVE : https://codinasion.web.app/programme/palindrome-number/


correct spelling of interactions from 'interations'

Spelling mistake in title. Says Interations where should say Interactions

Affected area:

New Functionality      [ ]
Documentation          [X]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Demo                   [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
no

@tbertenshaw would you mind signing-off your commit as described in https://github.com/openservicemesh/osm/blob/main/CONTRIBUTING.md#sign-your-work. This PR is good to go otherwise.

don't know what went wrong then when i followed the steps. gonna reject this and redo.


PGP mails signed with wrong key

Expected behavior
When encrypting & signing a mail, K9 mail should use the key for the sender's mail (specified in "From:").
Actual behavior
K9mail may use the wrong key – without any user feedback.
Steps to reproduce

Add multiple private keys to OpenKeyChain.
Create new mail, change identity (sender) mail.
Mail may still be signed with the main PGP key for another identity/mail used.

Environment
K-9 Mail version: 5.208
Android version: LineageOS 14
Possible solution

short-term (bug fix): Get mail from senders field and use correct key (-> the purpose of this issue)
long-term: https://github.com/k9mail/k-9/issues/2510 and https://github.com/k9mail/k-9/issues/942

/cc @Valodim

Is this still an issue with the latest K-9 Mail version?

I have not noticed it again, but I also do not have multiple private keys anymore in OpenKeyChain.

Thanks. Chances are it has been fixed since then. If not, someone else will eventually run into it and open a new issue. Closing this one.


Hurtnotes not appearing

Describe your problem here.
I added the instakill notes into my game, and it works. But i try to change the texture of them and it stops working. I took the hurtnotes and changed how they look, put them back in, and now they're invisible. Help?

These are the hurtnotes im trying to use
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No response

I can't really help you with the information I have. Can you show the .xml file for your hurtnotes? I'm thinking it may be an error in the .xml file.

i figured it out, .xml file for some reason had no text when it was saying what the file is so it said:

and i just wrote down the name of the file.


[PLAT-11778] update documentation

Goal
To ensure the README matches the documentation, and is fully up to date with implementation details

Superseded by #429


Rand Aug

Hello @roatienza!
Thanks for this great repo!
I am trying to train using rand_aug but I am facing some issues. I get an error on blur.py when trying to convert from BGR to Grayscale. It seems the image has just one channel.
`error: Caught error in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/home/fmobrj/anaconda3/envs/vitstr/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 202, in _worker_loop
    data = fetcher.fetch(index)
  File "/home/fmobrj/anaconda3/envs/vitstr/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 47, in fetch
    return self.collate_fn(data)
  File "/media/hdd6tb/jupyter/notebooks/vitstr/deep-text-recognition-benchmark/dataset.py", line 500, in __call__
    image_tensors = [transform(image) for image in images]
  File "/media/hdd6tb/jupyter/notebooks/vitstr/deep-text-recognition-benchmark/dataset.py", line 500, in <listcomp>
    image_tensors = [transform(image) for image in images]
  File "/media/hdd6tb/jupyter/notebooks/vitstr/deep-text-recognition-benchmark/dataset.py", line 336, in __call__
    img = self.rand_aug(img)
  File "/media/hdd6tb/jupyter/notebooks/vitstr/deep-text-recognition-benchmark/dataset.py", line 357, in rand_aug
    img = op(img, mag=mag)
  File "/media/hdd6tb/jupyter/notebooks/vitstr/deep-text-recognition-benchmark/augmentation/blur.py", line 104, in __call__
    img = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)
cv2.error: OpenCV(3.4.18) /io/opencv/modules/imgproc/src/color.simd_helpers.hpp:88: error: (-2:Unspecified error) in function 'cv::impl::{anonymous}::CvtHelper<VScn, VDcn, VDepth, sizePolicy>::CvtHelper(cv::InputArray, cv::OutputArray, int) [with VScn = cv::impl::{anonymous}::Set<3, 4>; VDcn = cv::impl::{anonymous}::Set<3, 4>; VDepth = cv::impl::{anonymous}::Set<0, 2, 5>; cv::impl::{anonymous}::SizePolicy sizePolicy = cv::impl::<unnamed>::NONE; cv::InputArray = const cv::_InputArray&; cv::OutputArray = const cv::_OutputArray&]'
> Invalid number of channels in input image:
>     'VScn::contains(scn)'
> where
>     'scn' is 1`

Pls try it with a color image and see if there is still an error. AFAIK, the function was tested for color images only.


Test PR

Signed-off-by: Oleg S<EMAIL_ADDRESS>This is not a real PR, just testing GH actions.
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Functional tests have been added, if necessary.
[ ] E2E Workflow tests have been added, if necessary.

#2513

The original issue was fixed so closing this


Revised nfx3 parsing with additional extensions

Hi Mark
Many thanks for your work on the nfdump integration for Rust, which is very useful to us!
With this pull request, I propose a slight modification of the extension parsing for nfx V3 files, newly including the following extension types:

EXT_MAC_ADDR (MAC Addresses)
EXT_LAYER2 (Further layer2 information, e.g., for VLANs)
EXT_MPLS (MPLS Label Stack)
EXT_TUN_V4 / EXT_TUN_V6 (Extensions keeping information from outer IP header for tunneled packets)

Moreover, it would be useful to collect both IPv4 and IPv6 information for some extensions that were already implementend, namely  EXT_BGP_NEXT_HOP_V4|V6, EXT_IP_NEXT_HOP_V4|V6, and EXT_IP_RECEIVED_V4|V6.
Let me know what you think!
Best regards,
Simon from NetFabric.ai

What is the advantage of separating out the extensions into separate v4 and v6 counterparts? Do records exist where both exist?

I have no strong feelings on it at all. We at Merit don't actually use the V3 records at all in our data. I only included nfx3 because this was mostly an exercise for me to learn Rust. If you're actually using that code and this is better for you, I'm all for it.
Accepted.