NewtonSoft is not able to serialize integers 0008 and 0009 when passed in this format.

Our request -
{
"TransactionDate": "2017-04-09T18:45:15.0000000-05:00",
"Channel": "CONSUMER EXCHANGE",
"Note": "Perform Exit Routing",
"PartnerTrackingReference": "PTRF124402086333",
"PlantCode": "4407",
"InventoryAdjustmentLineItems": [
{
"LineItemNumber": 1,
"TransactionType": "Inventory Mvt",
"BusinessCode": "INV_MVT",
"Material": "5HX-00001",
"SerialNumbers": [
"12345"
],
"Quantity": 1,
"StorageLocation": "0010",
"DestinationStorageLocation" : "9020",
"ReasonCode": 0008,
"RecieveSKU": ""
}
The issue -
Not working, Newtonsoft is not able to identify 0008 as number 8, same with the case reason code 0009.
Note that this works fine when we pass 0000 to 0007, the problem only happens when 8 and 9 are involved. 0010 is processed as '8'.
Example of error that we are seeing -
When we try to deserialize explicitly with Newtonsoft, noticing below error and stack trace.
“Input string '0008' is not a valid number. Path 'InventoryAdjustmentLineItems[0].ReasonCode', line 19, position 24.”
Stack trace for above error.
at Newtonsoft.Json.JsonTextReader.ParseNumber(ReadType readType)
at Newtonsoft.Json.JsonTextReader.ParseValue()
at Newtonsoft.Json.JsonTextReader.Read()
at Newtonsoft.Json.JsonWriter.WriteToken(JsonReader reader, Boolean writeChildren, Boolean writeDateConstructorAsDate, Boolean writeComments)
at Newtonsoft.Json.Linq.JTokenWriter.WriteToken(JsonReader reader, Boolean writeChildren, Boolean writeDateConstructorAsDate, Boolean writeComments)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateJObject(JsonReader reader)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
at Newtonsoft.Json.JsonConvert.DeserializeObject(String value)
at ConsoleApp1.Program.Main(String[] args) in C:\VSTSWorkspace\1DSC\InventoryAdjustment\Source\Inventory\ConsoleApp1\Program.cs:line 39

I believe Json.NET treats integral numbers with leading zeroes as octal instead of decimal.

Yes, Json.NET, as with some other JSON parsers, is liberal in that it allows leading zeros before numbers that are all in the range [0-7], and treats them as octal, to match the sort of results one would get using eval in javascript.
The JSON standard bans leading zeros in any case other than the integer zero or where zero is immediately followed by a decimal point.

Correct.
Either trim leading zeros or surround the value with quotes to make it a string.


split windows PR tests up to avoid timeouts

For #10293

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #10438 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10438      +/-   ##
==========================================
+ Coverage   60.94%   60.94%   +<.01%
==========================================
  Files         578      578
  Lines       31122    31190      +68
  Branches     4418     4429      +11
==========================================
+ Hits        18966    19008      +42
- Misses      11194    11216      +22
- Partials      962      966       +4

Impacted Files
Coverage Δ

src/datascience-ui/react-common/arePathsSame.ts
75% <0%> (-12.5%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/activation/node/manager.ts
28.3% <0%> (-1.7%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
:arrow_down:

src/client/common/experimentGroups.ts
100% <0%> (ø)
:arrow_up:

src/client/telemetry/constants.ts
100% <0%> (ø)
:arrow_up:

src/client/activation/languageServer/manager.ts
83.6% <0%> (+0.84%)
:arrow_up:

src/client/activation/languageClientMiddleware.ts
33.59% <0%> (+25.48%)
:arrow_up:

Continue to review full report at Codecov.

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


[Feature][Request] No border radius

Hi,
Congrats for the elegant framework.
I have this project where i don't need rounded corners, so i made a class that sets the border radius to 0 in all of it's children.
Wouldn't be better if this was included in the framework ?

You can set default border radius across all UI with one global variable.
Add this to  /src/site/global/site.variables
@defaultBorderRadius: 0em;

Yes,
But sometimes within the same project, we need some items with rounded corners and others without rounded corners.
Having this class built in is a little nice feature.

I dont think this makes sense to add for everyone. You can also specify border radius on a UI level.

I'm running into the same issue.  I think a good example is a header in a phone application.  If it's black you see small corners of white due to the rounding, whereas elsewhere (buttons for example) the rounding is fine.  In my case, I somehow need to fix the header.


Updates to serviceinfo and replace sdo_sys with fdo_sys

Replaced the existing sdo_sys Owner ServiceInfo module implementation
to fdo_sys.
Updated devmod:modules to align with FDO spec and moved it to
fdo_types.c
Added support for configurable MTU by allowing the user to decide the
maximum ServiceInfo size that the device can support, by updating a file
data/max_serviceinfo_sz. This inturn, decides the size of the buffer
being allocated for protocol-specific messages.
Added a flag to track  TO2.DeviceServiceInfo.IsMoreServiceInfo that is
used to check in TO2.OwnerServiceInfo for empty ServiceInfo scenario.
Fixed an issue where the ServiceInfo module iteration was done on the
primary pointer leading to 'module not found' error at certain
situations.

Signed-off-by: Chandrakar, Prateek<EMAIL_ADDRESS>

Updated as per review comments in commit 6a8b5f05d217b37b655b0c6616c9fd9edc25ea6e


Need to add button in every datarow to work with datarow and datacell

Please Describe The Problem To Be Solved
(Replace This Text: Please present a concise description of the problem to be addressed by this feature request. Please be clear what parts of the problem are considered to be in-scope and out-of-scope.)
(Optional): Suggest A Solution
(Replace This Text: A concise description of your preferred solution. Things to address include:

Details of the technical implementation, e.g. Flutter widget solving the problem.
Tradeoffs made in design decisions
Caveats and considerations for the future

If there are multiple solutions, please present each one separately. Save comparisons for the very end.)

can you please Describe The Problem To Be Solved?


Allow customisation of popover background

When the popup for definition appears it gets mixed with the text behind it like this

It only appears properly if the place behind it has a picture or is empty and works normally

This makes it difficult to use when studying because it is always overlapping
But it is still a great plugin.

This is likely due to the theme you're using, which set the background color to one that is transparent. I guess I can add an option to easily customise the background colour of the popover, but in the meantime, you can solve this by amending the "definition-popover" CSS class.

I use the default theme as of now, what value should I give the
definition-popover class (I'm not very good with css)
On Thu, Jul 18, 2024, 9:23 PM dominiclet @.***> wrote:

This is likely due to the theme you're using, which set the background
color to one that is transparent. I guess I can add an option to easily
customise the background colour of the popover, but in the meantime, you
can solve this by amending the "definition-popover" CSS class.
—
Reply to this email directly, view it on GitHub
https://github.com/dominiclet/obsidian-note-definitions/issues/67#issuecomment-2236914835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APR7SOG7FHBJQSIP6GY66T3ZM7QH5AVCNFSM6AAAAABLAVFXFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZWHEYTIOBTGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

From the root of your vault, you should see a hidden folder .obsidian. Enter the folder .obsidian/plugins/note-definitions, where you will see the files related to this plugin. You should see a css file styles.css among them. Open it and find the .definition-popover class. You want to overwrite the background-color property, which currently should hold a value of var(--background-secondary). Replace this with any colour you want. For more info on the values you can set you can refer to this https://developer.mozilla.org/en-US/docs/Web/CSS/background-color

Have you restarted obsidian after changing it?

I did not, but after restarting the plugin stopped working. It isn't underlining the text and clicking also does not work.

hmm, I'm not sure why that is happening, but for now just do a clean delete and reinstall of the plugin I guess. I'll add a customisation option for the color so you won't have to mess around with the css.

I've added a customisation option for the popover background colour in 0.24.0, hope it helps.


[gateway] Add authentication methods beside NTLM

Currently gateway authentication is NTLM only.
Provide additional methods such as Negociate and Digest
TODO:

[ ] Make Negociate and Nla opaque
[ ] Abstract the direct use of rdpTransport to allow a different transport layer (http for gateway)

this was implemented with https://github.com/FreeRDP/FreeRDP/pull/7934


maui-check can't update after visual studio 2022 preview 4.0 installed and then VS produces empty maui solutions

Hello,
here the error in maui-check:

.NET SDK - Workloads (6.0.100-rc.1.21458.32) Checkup...

android-aot (Microsoft.NET.Sdk.Android.Manifest-6.0.100 : 31.0.100-rc.1.12) installed.
ios (Microsoft.NET.Sdk.iOS.Manifest-6.0.100 : 15.0.100-rc.1.1534) installed.
maccatalyst (Microsoft.NET.Sdk.MacCatalyst.Manifest-6.0.100 : 15.0.100-rc.1.1534) installed.
× tvos (Microsoft.NET.Sdk.tvOS.Manifest-6.0.100 : 15.0.100-rc.1.1534) not installed.
× macos (Microsoft.NET.Sdk.macOS.Manifest-6.0.100 : 12.0.100-rc.1.1534) not installed.
maui (Microsoft.NET.Sdk.Maui.Manifest-6.0.100 : 6.0.100-rc.1.1608) installed.
wasm-tools (microsoft.net.workload.mono.toolchain.manifest-6.0.100 : 6.0.0-rc.1.21451.13) installed.
microsoft-net-sdk-emscripten (microsoft.net.workload.emscripten.manifest-6.0.100 : 6.0.0-rc.1.21430.3) installed.
¤ Recommendation: Install or Update SDK Workloads
! Attempt to fix? [y/n] (y): y
» Attempting to fix: .NET SDK - Workloads (6.0.100-rc.1.21458.32)
SHELL: C:\Program Files\dotnet\dotnet.exe workload update --no-cache --disable-parallel --from-rollback-file "C:\Users\Erik\AppData\Local\Temp\maui-check-e0c352a5\workload.json" --source "https://api.nuget.org/v3/index.json" --source "https://pkgs.dev.azure.com/dnceng/public/_packaging/6.0.100-rc.1.21458.32-shipping/nuget/v3/index.json"

Updated advertising manifest microsoft.net.sdk.android.
Updated advertising manifest microsoft.net.sdk.ios.
Updated advertising manifest microsoft.net.sdk.maccatalyst.
Updated advertising manifest microsoft.net.sdk.maui.
Updated advertising manifest microsoft.net.workload.emscripten.
Updated advertising manifest microsoft.net.workload.mono.toolchain.
Workload update failed: Invalid rollback definition. The manifest IDs in rollback definition C:\Users\Erik\AppData\Local\Temp\maui-check-e0c352a5\workload.json do not match installed manifest IDs.
I tried all workaround I could find in the previous issues but none work.
Now whaen I try to create a .NET maui app in VS 2022 I only get an empty solution without any error.

Now whaen I try to create a .NET maui app in VS 2022 I only get an empty solution without any error.

When you create MAUI solution uncheck "place solution in same folder"

This might help: https://github.com/dotnet/maui/wiki/Installing-.NET-6

This might help: https://github.com/dotnet/maui/wiki/Installing-.NET-6

I think on windows by now you can use visual studio 2022 preview to install Maui and .net 6. But you have to uninstall previous installations with the powershell script from the maui check repository.

I think on windows by now you can use visual studio 2022 preview to install Maui and .net 6. But you have to uninstall previous installations with the powershell script from the maui check repository.
all was removed, reinstalled. All steps from your previos post was made - result you already saw.


⚠ Bofrost.es has degraded performance

In 78883b8, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1505 ms

Resolved: Bofrost.es performance has improved in a6aaaaa after 6 minutes.


[BUG] AI (Internal): 72 tracking

Description/Screenshot
AI (Internal): 72 is being treated as a critical error. I don't think userAgent tracking should be defined as this. There is no way to disable this from being logged without disabling all telemetry.
Steps to Reproduce
https://github.com/microsoft/ApplicationInsights-JS/blob/0c76d710a0cd465f0b8b5e143250898122511874/extensions/applicationinsights-properties-js/src/PropertiesPlugin.ts#L97-#L100
Expected behavior
This should only go to AI if loggingLevelTelemetry is set to 2 (track everything)
Additional context
Others also having an issue

Thanks for creating this Issue, as we had not been made aware of this being a general issue from the community.
We will review and determine a method (config) for disabling this internal message.
This message should not be incurring any charges on your account, but I understanding that it may be frustrating being present in your logs.
Some background
This was created as an internal message so that we have an insight into the browser versions that consumers of the SDK use. Which is how we have been identifying that we can now (as part of the upcoming v3 (beta)) drop ES3 support as the level of usage is low enough to leave those users on v2 while still providing more updated language usage in v3.
This specific message is also gated on when a new ai_user cookie is created which has a lifespan of 1 year, so the frequency of this message (assuming cookies are enabled) is limited to 1 request per user per year, so that it did not overload anyones data store.
This is classified as an internal message so that we have some limited access to view a percentage of this anonymous information, we don't have access to any other events flowing through the system.
And all events sent to your connection string / instrumentation key are always sent and present in your events stream / tables.

@MSNev - Thank you for the update on this. Would it be possible to configure any of the AI Internal messages? We are also seeing the following (assume there is no charge for this neither) - AI (Internal): 55 message:"Maximum ajax per page view limit reached, ajax monitoring is paused until the next trackPageView(). In order to increase the limit set the maxAjaxCallsPerView configuration parameter." props:"true"

Well, depending on the number of ajax (XHR/fetch) requests you are making you may want to increase the config value, because if your getting this message then you are loosing Ajax events.
Another option came to mind this morning (after reading your latest comment), because all events (including these) pass the standard pipeline you could provide a telemetry initializer to intercept the event and when you detect this type of event simply return false and the event won't be sent

ie. you could do this now without waiting for us to add config 😄

@siyuniu-ms This is related to the internal task
Task 24666531: [AI] Disable the UserAgent AI Internal by default


Added personal author file

Signed-off-by: Sebastian Ingino<EMAIL_ADDRESS>Description
Adds the personal author file for myself
Issues Resolved
None
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Also, make sure all commits are signed by following the instructions in https://github.com/opensearch-project/project-website/pull/2144/checks?check_run_id=17027539111

Hey can you fix your commits with the following?

In your local branch, run: git rebase HEAD~2 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin feat/profile

Fixed both issues, thank you! My top jazz pianists would probably be Bill Evans and John Lewis, but I'm also a huge fan of Gerald Clayton's recent work!


Toil script as a program generates assertion

Take and example Toil program from the documentation, turn it into a python program `tryIt', with
#!/usr/bin/env python

as the first line.
 ./tryIt
Traceback (most recent call last):
  File "./tryIt", line 10, in <module>
    j1 = Job.wrapJobFn(helloWorld, "first")
  File "/cluster/home/markd/opt/centos6.0/x86_64/lib/python2.7/site-packages/toil/job.py", line 238, in wrapJobFn
    return JobFunctionWrappingJob(fn, *args, **kwargs)
  File "/cluster/home/markd/opt/centos6.0/x86_64/lib/python2.7/site-packages/toil/job.py", line 1350, in __init__
    self.userFunctionModule = ModuleDescriptor.forModule(userFunction.__module__).globalize()
  File "/cluster/home/markd/opt/centos6.0/x86_64/lib/python2.7/site-packages/toil/resource.py", line 331, in forModule
    assert extension in ('.py', '.pyc')
AssertionError

Close was erratic.
We need to change the assertion into a RuntimeException indicating a user error.

Would be obsoleted by #858.


Exception: java.lang.UnsupportedOperationException: Method isValid in class org.neo4j.jdbc.Connection is not yet implemented.

Getting this exception when using Driver with Scalikejdbc.

Hi Rohan, I confirm you the method is still not yet implemented as many other ones regarding meta-data, because we don't know exactly which ones are used by every tool and implementing all of them was too hard for Rel. 1.0.
We're going to prepare a patch for you asap.
Cheers,
Lorenzo

This should be fixed for BOLT module with c7acc09
A new snapshot version jar is available.
Can you please try it out and give us a feedback?

Confirmed this resolved the issue for me, see https://github.com/neo4j-contrib/neo4j-jdbc/issues/65#issuecomment-234482085.

Closed,
thank you for your help in report this issue, feel free to open other issues if needed.
Cheers


Create unit tests for a decoration mapping bug

Hello,
Our team has come across a bug with the DecorationSet map function. I've created a test suite where some of the tests do fail. I wasn't sure how to best integrate this with the existing tests, but I at least wanted to bring this to your attention in hopes that the bug could be fixed.
Our scenario is we are trying to insert multiple nodes throughout the doc in a single transaction. Then we undo the transaction. On undo, we notice the resulting decoration positions are off.
The tests I created mimick this scenario by doing the following:

Start with the original doc beforeDoc
Apply a mapping actionMapping, representing our transaction for inserting nodes
Apply a mapping undoMapping, representing our transaction for undoing the previous action

Of the four test cases I created, only "test case 2" passes all the tests. What surprises me is that "test case 1" and "test case 2" have the same decoration positions but the difference is the placements of the inserted nodes in between decorations.

Why are you creating the mappings manually, rather than deriving them from steps? That makes it rather hard to verify they are correct.

Why are you creating the mappings manually, rather than deriving them from steps? That makes it rather hard to verify they are correct.

ah, i created the mapping objects based on what I had observed while I was debugging.
I updated the PR to use a Transform instead. Would this be better?


Some Babel errors are not caught by eslint plugin

Example:
ERROR in ./app/scripts/mobilePages/BookPage/index.jsx
Module build failed: SyntaxError: ......./BookPage/index.jsx: Line 20: Namespace tags are not supported. ReactJSX is not XML.
  18 |                     <button type="button" className="btn btn-default" data-toggle="collapse" data-target="#bs-example-navbar-collapse-2">
  19 |                       Toggle navigation
> 20 |                       <svg className="icon-three-bars"><use xlink:href="#icon-three-bars" /></svg>
     |                                                             ^
  21 |                     </button>
  22 |                     <button type="button" className="btn btn-default" data-toggle="collapse" data-target="#bs-example-navbar-collapse-1">
  23 |                       <svg className="icon-search"><use xlink:href="#icon-search" /></svg>

In this case, I should have used xlinkHref.
I suppose these errors should be reported by the plugin too.

:+1: Thanks!


[ENH] test_update_predict_predicted_index should have no gaps between data passed in fit and the consecutive update steps.

Is your feature request related to a problem? Please describe.
The forecasting test test_update_predict_predicted_index does the following:

splits a time series y in y_train and y_test.
fits a forecasting algorithm using y_train
calls update_predict on the fitted forecaster using y_test and a SlidingWindowSplitter.

The problem with the test is that the first non-trivial update call uses data that are more than one time step ahead than the data passed in fit. As a result when we try to reconcile y_train and the subset of y_test from the splitter the resulting time series has gaps and frequency None. For some reason this was not causing any forecasting algorithm to break apart from BaggingForecaster in #2248 and in only 2 test cases (see CI log).
The following code reproduces the behaviour of the test:
from sktime.utils._testing.series import _make_series
from sktime.forecasting.model_selection import (
    SlidingWindowSplitter,
    temporal_train_test_split,
)

n_columns = 1
fh_int_oos = [2, 5]
window_length = 1
step_length = 5

y = _make_series(n_columns=n_columns, all_positive=True, index_type="datetime")
y_train, y_test = temporal_train_test_split(y)
cv = SlidingWindowSplitter(
    fh_int_oos,
    window_length=window_length,
    step_length=step_length,
    start_with_window=False,
)

print(f"max date seen during fit: {y_train.index.max()}")
counter = 0
for train_idx, test_idx in cv.split(y_test):
    print(f"split {counter} min date seen during update: {y_test.iloc[train_idx].index.min()}")
    counter += 1

# Prints:
# max date seen during fit: 2000-02-06 00:00:00
# split 0 min date seen during update: NaT
# split 1 min date seen during update: 2000-02-11 00:00:00

Describe the solution you'd like
Having a complete time series is a fair requirement from a forecasting algorithm, the test should reflect this.
Using the ExpandingWindowSlicer instead of the SlidingWindowSplitter will fix this.
Describe alternatives you've considered
Not implementing BaggingForecaster
⚠ Disclaimer ⚠
There's a good chance that my conclusion that the test needs some changes is wrong and there's a valid reason for it being that way.

FYI - @khrapovs and @fkiraly

How odd. It looks like you found out why we have problems with test_update_predict_predicted_index repeatedly.
Well, this is the third reason, I think I removed two other reasons here https://github.com/alan-turing-institute/sktime/issues/2555 and here https://github.com/alan-turing-institute/sktime/pull/2466 ...
This test is cursed.
I think there are two topics here: a design question, and pragmatic next steps.
Regarding pragmatic next steps: I think your PR is perfectly appropriate as a fix, until we have answered the design questions. Since it moves intention and logic back to the case sktime supports and which is widely tested, that of regular time stamp indices (I intentionally don't say "continuous" since this term has multiple, reserved meanings).
Regarding design questions: having read this three times, I am not sure whether this is:

a problem with the test logic, e.g., we do not need/want to test that esimators are fine with irregular data
a problem with the test implementation, i.e., whether the intention was to have what you changed it to
a problem with the framework, i.e., that it does not mitigate the impact when data becomes irregular like this

This seems to boil down to some design questions related to irregular data:

should all forecaster estimators be fine with irregular data? E.g., if received in update like this.
if forecasters are not fine with irregular data, but receive a time stamp "on the grid", should the "updating memory" functionality of the framework perhaps fill in NAs? Then, forecasters which can deal with NAs but not irregular data would still be ok. We have an encapsulated entry point for the memory update since this PR https://github.com/alan-turing-institute/sktime/pull/2412, so this could be addressed there.
if some forecasters can deal with irregular data while others cannot, should we introduce a tag for it, and have conditional behaviour and testing?

From answers to the above, we can derive:

framework implementation and logic implementation targets
an answer to "what the test_update_predict_predicted_index test should do
whether we need additional tests related to irregular data

I think this is deep and consequential enough to be an agenda point at the next regular dev meeting?

I think this is deep and consequential enough to be an agenda point at the next regular dev meeting?
I agree! I personally like the tag option for irregular time series.

Kudos to that. I am working on irregular time series. For now only in splitters though.


Revert "fix(config): revert configurable fields in values"

This reverts commit e1030290b695e1e386feb814d8d91785b2df8df8.
Signed-off-by: Allen Leigh<EMAIL_ADDRESS>
Description:
Revert adding back the configurable fields in values, due to breaking e2e tests.
https://github.com/openservicemesh/osm/runs/2521101517

Affected area:

Install                [x]
CLI Tool               [x]

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

Can we fix the failing test instead?
[Tier 2][Bucket 3] Tests traffic via port exclusion
/home/runner/work/osm/osm/tests/framework/common.go:136
  Test global port exclusion
  /home/runner/work/osm/osm/tests/e2e/e2e_port_exclusion_test.go:19
    Tests HTTP traffic to external server via global port exclusion [It]
    /home/runner/work/osm/osm/tests/e2e/e2e_port_exclusion_test.go:33

    Unexpected error:
        <*errors.errorString | 0xc00096eb30>: {
            s: "UpdateOSMConfig(): MeshConfig.config.openservicemesh.io \"osm-mesh-config\" is invalid: spec.traffic.outboundPortExclusionList: Invalid value: \"string\": spec.traffic.outboundPortExclusionList in body must be of type integer: \"string\"",
        }
        UpdateOSMConfig(): MeshConfig.config.openservicemesh.io "osm-mesh-config" is invalid: spec.traffic.outboundPortExclusionList: Invalid value: "string": spec.traffic.outboundPortExclusionList in body must be of type integer: "string"
    occurred

    /home/runner/work/osm/osm/tests/e2e/e2e_port_exclusion_test.go:70

Closing the PR, as the e2e test is fixed in #3306


Adding  square brackets in the comment make it failed

Adding 方括号(square brackets) in the comment in a .clang-format file will make the file checking failed. follow is the example:
#  说明：对齐连续的位域分隔符(:)
# 类型：AlignConsecutiveStyle
# 取值: [ None, Consecutive, AcrossEmptyLines, AcrossComments, AcrossEmptyLinesAndComments ]
AlignConsecutiveBitFields: None
It make  the color of the file in vscode lost.  When I remove the [ ] from
[ None, Consecutive, AcrossEmptyLines, AcrossComments, AcrossEmptyLinesAndComments ],
the color come back well.
detail
release
2015/11/28 01:59:09
last update
2019/1/22 20:24:27
indentification
xaver.clang-format
vscode version:
版本: 1.70.2 (system setup)
提交: e4503b30fc78200f846c62cf8091b76ff5547662
日期: 2022-08-16T05:35:13.448Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041

I get the answer: chose language mode from lni to yaml. It do well now.


Add more context to the default Platform#reportFailure

Currently, it just prints the cause.toString. We could add some more context (e.g. "An unhandled failure occured:") and maybe assist the user in checking what went wrong.
(I'd love to know myself what to do to get rid of these! cc @jdegoes )

I think recently the Cause has been imlroved to show stack trace...if not, we should add that. The message printed should be enough to diagnose the problem.
Also I changed to print to std err by default.

When exactly does this get called? On all defects? Would be good to have that printed as well

It will be called on all unhandled typed errors, and all defects.


Add gofumpt formatter for go

gofumpt enforces a stricter format than gofmt.

Would you please also mention it in the readme? Thanks!

Thank you!


KNOX-2509 - Added OpenAPI UI (v3.36.2) for documenting Knox's REST API endpoints and models

What changes were proposed in this pull request?
In this change, I'm integrating Open API UI version 3.36.2 into the Knox Home page, under home/swagger. The UI itself is downloaded and unpacked as I needed to modify the out-of-the-box index.html.
I annotated many of the resources in the relevant classes, but that work is not yet finished (it'll come later in a different PR). I just wanted to make sure the UI itself, its build integration, and the basic annotations are merged into Apache.
The location of the Swagger browser is available here (assuming a fresh Knox install): https://localhost:8443/home/swagger/
How was this patch tested?
Ran a full build including tests, static code analyzing tools.
Manually tested the UI itself.

Thanks, @risdenk !


No Horizontal Scroll in Basic Script Editor (MacOS)

Describe the bug
There does not appear to be any horizontal scrolling available in the built-in script editor when multiple files are open in the editor. The only workaround is to resize the window/pane.
To Reproduce

Open Script Manager
Right-click a script
2a. Edit with basic editor
Find line of code longer than the width of the editor pane
Attempt to view the full line without having to resize the window.

Expected behavior
I expected a scroll bar.
Screenshots

**Attachment
N/A
Environment (please complete the following information):

OS: MacOS 10.14.3 (Mojave - Dark mode)
Java Version: 11.0.2
Ghidra Version: 9.0

Additional context
N/A

It's working for me with an Apple Mighty Mouse.  What type of device are you using?

If you have only one tab open for that window (I see you have 2 in that screenshot), does it change anything?

Yep, it appears the issue is linked to having multiple files open in the editor.

Could you try switching your Look and feel to something else?
(Main window -> Edit -> Tool Options -> Tool -> Swing Look and feel, then restart Ghidra)

Closing, no response.


update labels

Updating the labels for our grafana alerts - per https://github.com/dotnet/dnceng/issues/652

That issue also mentions updating the "Compliance" tag, are you covering that here too?

Looking - thanks for the reminder


Ship guns

About the PR
Solves #8565.
it's a draft. shoo.
Screenshots

Changelog

:cl:

add: Adds ship guns.

Closing until I get motivation to work on it again.


Fix to request.tpl

Hi @moszeed
This fixes an attribute from being duplicated and allowing us to get a response without errors.

sry, thats not working with the test, maybe you can look into to it again ?


Refactor componentWillReceiveProps

Refactored componentWillReceiveProps to work with componentDidUpdate
Fixes #1084 and #1245

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

Hey y'all when will this be merged?

Hello, any update on this issue? Thanks in advance

I also would like this to be merged. Can we get an update? Seeing an issue where the FlexibleXYPlot is not resizing properly when paired with Suspense because componentWillReceiveProps fires before the display: none !important; is removed from the suspended component, which means it sets the width and height to 0.

👍

Sorry all for the terrible response time.
As you may have been aware, development on react-vis was stalled for some time.
We are getting back into it however, as we are focusing on modernizing it a bit and putting a focus on performance.
I've rebased and fixed some tests over this this PR (that is based on this one) https://github.com/uber/react-vis/pull/1343

@Xiot Is there any update on when this can be merged? This issue is becoming more and more pressing for us! Thanks!


images uploading

Hello guys, wtf with images in this wysiwyg implementation ??? 👎

Check this out:
https://github.com/summernote/summernote-rails/wiki/Image-File-Upload-to-Server


Add Codecoverage

Codecov Report

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

@@          Coverage Diff           @@
##             master    #1   +/-   ##
======================================
  Coverage          ?   75%
======================================
  Files             ?     3
  Lines             ?   168
  Branches          ?    32
======================================
  Hits              ?   126
  Misses            ?    35
  Partials          ?     7

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 7e09d72...0067345. Read the comment docs.


Collection log showing incorrect completed count

Checklist

[X] I'm reporting a problem with Dink
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
My collection log count is not up to date with the game, see the screenshot below:

But the in-game notification said this:

Now I have to say, that I think it somehow resetted some col logs slot that I had in the game itself, because I can see on screenshots that I already had the 53 before that you see in the screenshot some hours earlier, the Dink notifications for these were also incorrectly, 51-54 in game, 55-58 in the Discord notifications.

Do you guys have any clue what is going on? Thanks in advance!
Screenshots
No response
Runelite Logs
client.log
client_2024-07-27.0.log
Runelite Version
1.10.35

Can you upload the client.log that includes yesterdays date in the file name, once it hits midnight local time RuneLite starts a new log file

Can you upload the client.log that includes yesterdays date in the file name, once it hits midnight local time RuneLite starts a new log file

I updated my start post with the log file.

seems to be a jagex bug rather than a dink issue (i've confirmed the varp id we utilize to obtain that data is still accurate)
i'd recommend restarting your client and playing with the Character Summary tab selected

unable to replicate

I guess it was a temp bug on my account, works fine now, thanks.

if this occurs again, can you try testing with the collection log popup disabled in the game settings (so there's only the chat message for the clog)

It happened again, I tried removing the collection log plugin when it happened the first time, but it doesnt help.
As you can see it shows 3 different things, my chat shows it being 78, Dink reports it being 80, and my actual collection log is at 77.. So I'm not sure if it's even a Dink problem, seems like a bug in my account. But I'll try your suggestion.

strange - yeah it's either a jagex bug or you have some misbehaving hub plugin (you can run runelite in safe mode to eliminate this possibility)

I removed like 8 plugins from the plug-in hub that I suspect are faulty and/or that I rarely use, I'd rather not play in safe-mode because I'm reliant on some and it happens only once in a week I think. I'm curious what will happen now. Although it has nothing to do with Dink, thanks for thinking along, its appreciated.


teleport: 11.3.4 -> 12.0.2, reintroduce teleport_11

Description of changes
New major version.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Please see the updated PR.

Added teleport_11 alongside the updated default teleport (12)
Added package option to the module
Updated release notes to recommend settings this option to upgrade to v11.

Updated teleport_11 from 11.3.4 to 11.3.5
Replaced ymatsiuk as the maintainer for teleport tests, per request.

@justinas this package fails to compile on darwin, it looks like the issue mentioned here: https://github.com/NixOS/nixpkgs/pull/214425#issuecomment-1416721340
Here's my build output:
$ yarn workspace @gravitational/teleport build --output-path=../../../webassets/teleport/app
$ g-build --config webpack.prod.config.js --progress --bail --output-path=../../../webassets/teleport/app
assets by status 3.92 KiB [cached] 4 assets2mginugin
Entrypoint app = runtime~app.d96ed60b58b56df22960.js app.33babda0435fe6278445.js
runtime modules 2.5 KiB 3 modules
./src/boot.tsx 1.22 KiB [built] [code generated] [1 error]

ERROR in ./src/boot.tsx 41:2
Module parse failed: Unexpected token (41:2)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
| ReactDOM.render(
>   <Teleport history={history.original()} ctx={teleportContext} />,
|   document.getElementById('app')
| );

webpack 5.73.0 compiled with 1 error in 2552 ms
✨  Done in 26.90s.
buildPhase completed in 9 minutes 18 seconds
installing
cp: cannot stat 'webassets/.': No such file or directory
error: builder for '/nix/store/q1pp1875j0gcjcgsri45ksplg5ll8srx-teleport-webassets-12.0.2.drv' failed with exit code 1
error: 1 dependencies of derivation '/nix/store/qhiav5bc3crjra9b8a0vvnlnsi09d3mn-teleport-12.0.2.drv' failed to build
error: program '/nix/store/45ws0jiwkilqcg6mwnpp5rz5danvm0ll-nix-2.11.0/bin/nix-shell' failed with exit code 100

Hi @takeda ,
I'm terribly sorry about this! The upstream fix for the web assets made it into 11.3.4, but not 12.0.2 (which we packaged up here) - it was only made available in 12.0.3 onwards.
In any case, I've opened a PR to fix: https://github.com/NixOS/nixpkgs/pull/219971 .


Cannot run android web tests on selenoid

After many tests and debug, I run my tests with this command on Java:
mvn verify -Dwebdriver.driver=remote -Dwebdriver.remote.url="http://$remoteAddress:4444/wd/hub" -Dwebdriver.remote.driver=appium -Dserenity.driver.capabilities="browserName:android" -Dappium.browserName=android
It seemd that the selenoid android starting succesfull but keep giving me this:
error: no devices/emulators found
My config for browsers.json is
{     "android": {         "default": "6.0",         "versions": {             "6.0": {                 "image": "selenoid/android:6.0",                 "port": "4444",                 "path": "/wd/hub"             }         }     } }
Do you see any issue in my configs? What am i doing wrong ? :/

Are you running this one hardware server or a VM with nested virtualization enabled?

I am running it on a cloud web server machine. So I need to enable nested virtualization ?

@xMagiciaNx correct.

I can't understand the issue really....
Here is what the coommand looks like:
mvn verify -Dwebdriver.driver=remote -Dwebdriver.remote.url="http://<ip>:4444/wd/hub" -Dwebdriver.remote.driver=appium -Dserenity.driver.capabilities="browserName:android" -Dappium.browserName=android
Here is what I get from my VM:
2020/08/26 10:15:13 [-] [NEW_REQUEST] [unknown] [<IP_ADDRESS>] 2020/08/26 10:15:13 [-] [NEW_REQUEST_ACCEPTED] [unknown] [<IP_ADDRESS>] 2020/08/26 10:15:13 [628] [LOCATING_SERVICE] [android] [] 2020/08/26 10:15:13 [-] [DEFAULT_VERSION] [Using default version: 6.0] 2020/08/26 10:15:13 [628] [USING_DOCKER] [android] [6.0] 2020/08/26 10:15:13 [628] [CREATING_CONTAINER] [selenoid/android:6.0] 2020/08/26 10:15:13 [628] [STARTING_CONTAINER] [selenoid/android:6.0] [8be47cd418f66d4693f9881a4c041c3cf90881e129e4ce1534283fac72c0d026] 2020/08/26 10:15:14 [628] [CONTAINER_STARTED] [selenoid/android:6.0] [8be47cd418f66d4693f9881a4c041c3cf90881e129e4ce1534283fac72c0d026] [0.46s] 2020/08/26 10:15:54 [628] [SERVICE_STARTED] [selenoid/android:6.0] [8be47cd418f66d4693f9881a4c041c3cf90881e129e4ce1534283fac72c0d026] [40.44s] 2020/08/26 10:15:54 [628] [PROXY_TO] [8be47cd418f66d4693f9881a4c041c3cf90881e129e4ce1534283fac72c0d026] [http://<IP_ADDRESS>:4444/wd/hub] 2020/08/26 10:15:54 [628] [SESSION_ATTEMPTED] [http://<IP_ADDRESS>:4444/wd/hub] [1] 2020/08/26 10:15:54 [628] [SESSION_FAILED] [http://<IP_ADDRESS>:4444/wd/hub] [500 Internal Server Error] 2020/08/26 10:15:54 [628] [REMOVING_CONTAINER] [8be47cd418f66d4693f9881a4c041c3cf90881e129e4ce1534283fac72c0d026] 2020/08/26 10:15:58 [628] [CONTAINER_REMOVED] [8be47cd418f66d4693f9881a4c041c3cf90881e129e4ce1534283fac72c0d026]
And the Java stacktrace:
The desired capabilities must include either an app, appPackage or browserName
The think is that I actually pass this parameter. I testing this for weeks and no solution found yet :(

@xMagiciaNx try to use deviceName = android instead of browserName = android.

Oh, wow now this it may lead somewhere, i get
Original error: URI and package arguments are required
Android actually is starting it seems, i get all the ADB logs.

@xMagiciaNx should specify app capability with an URL to tested application APK.

The only thing I need is a browser to test a web app. Why I should put APKs ?

@xMagiciaNx these images selenoid/android:6.0 are for application testing and don't have Chrome inside. For Chrome mobile use these ones: https://aerokube.com/selenoid/latest/#_chrome_mobile

Ty for your support but the issues is still there.
ADB is starting normally but I still getting the following error:
URI and package arguments are required
My browsers.json:
{ "chrome-mobile": {         "default": "mobile-79.0",         "versions": {             "mobile-79.0": {                 "image": "selenoid/chrome-mobile:79.0",                 "port": "4444",                 "path": "/wd/hub"             }         }     } }
Running command:
mvn verify -Dwebdriver.driver=remote -Dwebdriver.remote.url="http://$remoteAddress:4444/wd/hub" -Dserenity.driver.capabilities="browserName:chrome-mobile"
Some of the error i get from ADB:
Error1:
2020-08-26 18:34:52:014 - [debug] [W3C (2807d632)] Encountered internal error running command: Error: URI and package arguments are required 2020-08-26 18:34:52:014 - [debug] [W3C (2807d632)]     at ADB.startUri (/opt/node_modules/appium/node_modules/appium-adb/lib/tools/apk-utils.js:59:11) 2020-08-26 18:34:52:015 - [debug] [W3C (2807d632)]     at AndroidUiautomator2Driver.commands.setUrl (/opt/node_modules/appium/node_modules/appium-uiautomator2-driver/lib/commands/general.js:146:18) 2020-08-26 18:34:52:015 - [debug] [W3C (2807d632)]     at commandExecutor (/opt/node_modules/appium/node_modules/appium-base-driver/lib/basedriver/driver.js:330:9) 2020-08-26 18:34:52:016 - [debug] [W3C (2807d632)]     at /opt/node_modules/appium/node_modules/async-lock/lib/index.js:125:12 2020-08-26 18:34:52:016 - [debug] [W3C (2807d632)]     at AsyncLock._promiseTry (/opt/node_modules/appium/node_modules/async-lock/lib/index.js:249:31) 2020-08-26 18:34:52:017 - [debug] [W3C (2807d632)]     at exec (/opt/node_modules/appium/node_modules/async-lock/lib/index.js:124:9) 2020-08-26 18:34:52:017 - [debug] [W3C (2807d632)]     at AsyncLock.acquire (/opt/node_modules/appium/node_modules/async-lock/lib/index.js:140:3) 2020-08-26 18:34:52:017 - [debug] [W3C (2807d632)]     at AndroidUiautomator2Driver.executeCommand (/opt/node_modules/appium/node_modules/appium-base-driver/lib/basedriver/driver.js:343:39) 2020-08-26 18:34:52:018 - [debug] [W3C (2807d632)]     at AppiumDriver.executeCommand (/opt/node_modules/appium/lib/appium.js:510:36) 2020-08-26 18:34:52:018 - [debug] [W3C (2807d632)]     at process._tickCallback (internal/process/next_tick.js:68:7)
Error2:
2020-08-26 18:34:52:376 - [debug] [W3C (2807d632)] Encountered internal error running command: NotImplementedError: Method is not implemented 2020-08-26 18:34:52:378 - [debug] [W3C (2807d632)]     at AndroidUiautomator2Driver.execute (/opt/node_modules/appium/node_modules/appium-android-driver/lib/commands/execute.js:14:11) 2020-08-26 18:34:52:378 - [debug] [W3C (2807d632)]     at commandExecutor (/opt/node_modules/appium/node_modules/appium-base-driver/lib/basedriver/driver.js:330:9) 2020-08-26 18:34:52:378 - [debug] [W3C (2807d632)]     at /opt/node_modules/appium/node_modules/async-lock/lib/index.js:125:12 2020-08-26 18:34:52:378 - [debug] [W3C (2807d632)]     at AsyncLock._promiseTry (/opt/node_modules/appium/node_modules/async-lock/lib/index.js:249:31) 2020-08-26 18:34:52:379 - [debug] [W3C (2807d632)]     at exec (/opt/node_modules/appium/node_modules/async-lock/lib/index.js:124:9) 2020-08-26 18:34:52:379 - [debug] [W3C (2807d632)]     at AsyncLock.acquire (/opt/node_modules/appium/node_modules/async-lock/lib/index.js:140:3) 2020-08-26 18:34:52:379 - [debug] [W3C (2807d632)]     at AndroidUiautomator2Driver.executeCommand (/opt/node_modules/appium/node_modules/appium-base-driver/lib/basedriver/driver.js:343:39) 2020-08-26 18:34:52:379 - [debug] [W3C (2807d632)]     at AppiumDriver.executeCommand (/opt/node_modules/appium/lib/appium.js:510:36) 2020-08-26 18:34:52:379 - [debug] [W3C (2807d632)]     at process._tickCallback (internal/process/next_tick.js:68:7)
This really drives me crazy. I almost search the whole WWW

@xMagiciaNx browserName should be chrome and version can be anything, e.g. mobile-79.0.

Nope, this not working again :/ .
Again it asks for URI and packageName.
Should i try different image ?

@xMagiciaNx what's the reason of setting browser name two times? Removing this should help: -Dserenity.driver.capabilities=\"browserName:android\"


Override components hidden when iframed

From @louh on November 11, 2015 20:53
Search and locate-me are automatically hidden when we're in an iframe. Sometimes this is not desirable, e.g. when embedded into a blog post or page where the demo requires interaction. (See Indy's "Where the streets have no name.") Let's preserve the current assumption for the drop-in use case, but provide a way to override this behavior. (Idea: option { forceDisplay: true })
Copied from original issue: mapzen/ui#29

cc @meetar Is this the thing you were planning to work on for @IndyHurt ? I think I might have forgotten this issue was opened.

From @meetar on April 28, 2016 14:23
Yep! I was looking at this yesterday, trying to figure out how to pass
options to the other components. I assume I'll need to export them as
self-instantiating modules, the way the bug is now?
On Wed, Apr 27, 2016 at 5:55 PM, Lou Huang<EMAIL_ADDRESS>wrote:

cc @meetar https://github.com/meetar Is this the thing you were
planning to work on for @IndyHurt https://github.com/IndyHurt ? I think
I might have forgotten this issue was opened.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mapzen/ui/issues/29#issuecomment-215241937

In progress!


Call to a member function applyCalledMethods() on null

Describe the bug
I get this error when I set 'send_requests_to_ray' or 'send_views_to_ray' to 'true'.
http://flareapp.io/share/dPb4ADL5
Versions
1.12.0

This has been fixed in the latest release.


expose sjcl

Coverage decreased (-0.11%) to 93.74% when pulling 0cbf3d9891c64e9891aeb4e6bd9aba07bafac0d4 on matiu:sjcl into eb62309a7bd89a2751ad789423e1a492caf3c7d5 on bitpay:master.


Can I handle the transaction?

Hi
I am using the DeleteFromQuery to execute the batch delete, I want to delete data from several tables and I want to handle the transaction to commit when completed or rollback when exceptions are met.
So Do we have any transaction managed?

Hello @rex-pj ,
Yes, the transaction started from the Context should be handled by the DeleteFromQuery method.
Best Regards,
Jon

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

Hi Jonathan, I'm assuming the Delete() and DeleteAsync() extensions are also compatible with transactions for bulk delete ops?

Unlike we are missing something, yes.
If a transaction is started within Entity Framework, the code should use it.

@JonathanMagnan
As I test in my application, the data is deleted after the DeleteFromQuery() is executed, and no ways to rollback.
For example:
dbContext.Articles.DeleteFromQuery(); // Deleted
dbContext.Users.DeleteFromQuery(); // Deleted
dbContext.SaveChanges(); // No meaning
So the data is deleted after DeleteFromQuery() and even the SaveChanges() is no meaning, I also test with a transaction like BeginTransaction() then use Rollback() but the data still not rollback().

Btw, I can handle the transaction by using dbContext.BeginTransaction() and I can Commit and Rollback also, my problem is solved, thanks for your help.

I think issues like ignoring DbContext.SaveChanges() and respecting DbContext.Database.BeginTransaction() must be included in the docs.


The column action doesn't work if a closure is used as parameter

Package
filament/filament
Package Version
v2.17.40
Laravel Version
v10.11.0
Livewire Version
v2.12.3
PHP Version
PHP 8.2.4
Problem description
The modal is not displayed if a closure is used as a parameter in action().
Tables\Columns\TextColumn::make('date_of_birth')
    ->action(function (People $record): Tables\Actions\Action {
        return Tables\Actions\Action::make('dateOfBirthAction')
            ->form([
                TextInput::make('date_of_birth')
                     ->default($record->date_of_birth)
            ]);
    })

Expected behavior
As in the following example, a modal should open.
Tables\Columns\TextColumn::make('name')
    ->action(
        Tables\Actions\Action::make('nameAction')
            ->form(function (People $record): array {
                return [
                    TextInput::make('name')
                        ->default($record->name)
                 ];
        })
    ),

Steps to reproduce

Clone my the repository.
https://github.com/plusquamperfekt/filament-table-column-action

Install composer packages
composer install

Run vite
npm install && npm run dev

Reproduction repository
https://github.com/plusquamperfekt/filament-table-column-action
Relevant log output
No response

Thanks, but this is deliberate. If a closure is passed, we assume this is a function to run one-time when the column is clicked. Kinda shorthand for if you dont need to open a modal, just run a function

@plusquamperfekt I'd be interested to know if you ever worked around this and what your solution looked like.


PushPageModel: The application may be doing too much work on its main thread.

I have defined the following Command:
OpenNotificationPageCommand = new Command(async () =>
{
    // Push modal page
   await CoreMethods.PushPageModel<NotificationsPageModel>(null, true, false);
});

I have overrided the public async Task PushPage(..) method of IFreshNavigationService in my custom NavigationService as described in this tutorial.
When I running the app in Android and the previous PushPageModel is called, the console show me this message:
[Choreographer] Skipped 36 frames!  The application may be doing too much work on its main thread.
Indeed the application take a while before show the NotificationsPage.
Specifically that in app there are only 3 Pages with their PageModels and no background activities (connection to servers, calling to API,..) are executed so the app runs only UI operations. Furthermore I perceive this behavior only in Android (with real device and emulator), no warnings in iOS are shown. There is a way to fix?
Thanks

This is not a issue with FreshMvvm just Xamarin.Forms. Wouldn't you agree?


Peek/goto definition stop working after upgrading to 1.5

VSCode Version: 1.5.1
OS Version:Ubuntu 14.04

Steps to Reproduce:

Right click on a function and choose goto definition from the menu.
Error shows that the file cannot be found.
The file exists on disk. Using find feature can find the file.

@harrywangwork Which language and extensions are you using?

I was using C/C++.
It worked fine with previous version.

Trying to switch to 32-bit to see if there is luck.

The Peek/Goto definition features are implemented by extension. Please log this issue in the repo for the C/C++ extension you are using. They will be able to find the root cause faster and re-open this issue here if there is any problem with the VS Code APIs

sounds good.

Issue resolved after updating C++ extension. Thanks for the tip.


http://fhirtest.uhn.ca/base/Patient/_search?name=George

This 500's against this server, dstu1, and dstu2
Courtesy of Jenni Syed

Issue looks like it was this: https://java.net/jira/browse/GLASSFISH-20840
I've patched GF and restarted and it seems to be ok now. Thanks!!


Dictionaries in GADTs + Refinements to the solver

Internally, amc now supports constructors with vectors of arguments.
The surface language only exposes this by letting us put dictionaries
inside GADTs, like God intended:
type dict 'c = Dict : 'c => dict 'c
type 'p :- 'q = Sub of ('p => unit -> dict 'q)
(* Entailment - morphisms in the category Con *)

dict is a newtype with the interesting property that dict 'c ~ c;
Matching on it will reveal the evidence for 'c that was hidden away.
This lets us talk about the category of constraints, Con, inside the
language, which revealed a treasure trove of weird edge cases in the
constraint solver which I have spent the last of my sanity fixing.
For instance, Con is cartesian: You can ignore information. This
didn't work until 5144968, but it does now, so all good.
let weaken1 : forall 'a 'b. ('a * 'b) :- 'a = Sub (fun _ -> Dict)
let weaken2 : forall 'a 'b. ('a * 'b) :- 'b = Sub (fun _ -> Dict)

You're right, it is impossible.


feat: Initial backend code to get list of installed mods

For now simply parses enabledmods.json.
Note that this file will not be up-to-date if the user just installed a mod but hasn't launched Northstar yet.

All the CSS changes are essentially to get scrolling working in mods view without scrolling the whole window.

So this would be ready to merge (apart from merge conflict) except that for some reason https://github.com/GeckoEidechse/FlightCore/blob/2ad6aa1006efa5cb7fa2ff5de177833eea30c4ff/src-vue/src/views/ModsView.vue#L27-L46 is called already on application launch and not when opening the view.
As such it has a race condition with finding install path which it tends to win and as such then throws an error cause this.$store.state.game_path is null at that time.
My assumption was that code in mounted() would only be run when actually opening the view unless I'm mistaking something here...
Any ideas @Alystrasz? ^^

My assumption was that code in mounted() would only be run when actually opening the view unless I'm mistaking something here...

Since our component views are tabs, they're all mounted at application start; even if they're not visible, they still are built.
This might show there's a design flaw in the app; why shouldn't we able to call Rust methods without arguments at all? Backend is supposed to have all information (game_path, install_type...)

This might show there's a design flaw in the app; why shouldn't we able to call Rust methods without arguments at all? Backend is supposed to have all information already (game_path, install_type...)

Cause the information is stored in frontend not backend. The design is for frontend to handle UI and storing info and backend functions are called to execute various tasks.
Keeping information in frontend makes sense cause it means we don't need any global data store or perform synchronisation/locking of variables in the backend. All threads in backend can therefore be completely independent, meaning there's no need for locking at all.
In theory I could change get_installed_mods() to re-discover game path if no info was passed but that breaks when the game install cannot be auto-discovered, e.g. when installed with Lutris on Linux... ._.

Keeping information in frontend makes sense cause it means we don't need any global data store or perform synchronisation/locking of variables in the backend. All threads in backend can therefore be completely independent, meaning there's no need for locking at all.

This however leads to inconsistencies between Rust state and UI state, which I foresee a lot of bugs will come from :smiling_face_with_tear:

This however leads to inconsistencies between Rust state and UI state, which I foresee a lot of bugs will come from

Well good news, there's absolutely zero state in Rust currently :P

Well good news, there's absolutely zero state in Rust currently :P

The issue here is currently :)
I think I'd prefer a stupid UI which only displays data and have logic written in Rust, compared to an UI that consistently have to call backend with same arguments

I don't think that's feasible. The UI needs to have some state e.g. for displaying the current tab, or what to show on the omni-button.
Having the state stored in frontend means we're getting this for free. The alternative would be to refactor the whole codebase to emit events on state changes and adding various forms of reference counting and locking for global variables (or using a shared object controlled by some thread). I think we can both agree on that storing state in both front and backend is bad due to needing to sync etc. Furthermore we would also have to redo persistence cause that is currently done via frontend plugin.
So given that currently frontend is already the one holding the state while backend is stateless, I think it's easier to just adjust this PR to load mods when switching to Mods tab (or when game install has been detected) :P

So given that currently frontend is already the one holding the state while backend is stateless, I think it's easier to just adjust this PR to load mods when switching to Mods tab (or when game install has been detected) :P

It's easier to just adjust this PR, of course :)
You could listen to tab-click events (https://element-plus.org/en-US/component/tabs.html#tabs-events), and load your content only when mods view is displayed.

Closing as implemented with #27 and #28


MIP after label creation

Does the MIP still work for you in workflows that contained the labeling function? I ran the 9x8 example. It seems to have saved the labels. But when I try to open the MIP, the napari plugin fails.
It's a bit suspicious that the MIP appears to have an extra pyramid layer. Maybe the MIP assumed the labels folder was another pyramid layer?

Just removing folder 5 from the MIP folder doesn't help though. Maybe the Zarr structure replication needs to start handling labels? We can also do MIPs on labels, that is an approximation of the 2D segmentation, though not always the best (people may want to run specific 2D models) => not sure whether we urgently need the ability to do MIPs on labels. It's a nice to have, but priority would be that labels don't interfere with MIPs :)

The bug is here: https://github.com/fractal-analytics-platform/fractal/blob/main/fractal/tasks/replicate_zarr_structure_mip.py#L47.
(to be fixed a bit later)

In principle this is fixed with https://github.com/fractal-analytics-platform/fractal/commit/ddb5c124d3a93652de984d59e5d8438109f41263. I tried on a small example and it produced a reasonable MIP.
@jluethi: if you have an example where it works, we can close this issue.

Yes, that fixed it! Thanks a lot for the fast bug fix @tcompa ! 👏🏻
Here is an MIP of the 2x2 with labeling as part of the workflow, works again like before :)


Replace Should.js with Chai

Should.js hasn't been updated on npm since 2018, and its repo is archived. We had to move from Should.js to Chai in Frontend (getodk/central-frontend#1015). There are a lot of similarities between the two, but Chai is better documented. There are also some differences, and I think trying to remember those differences while switching between Backend and Frontend will end up causing friction. This issue is to replace Should.js with Chai in Backend. We should time work on this issue such that no large Backend work is in progress.

We had to move from Should.js to Chai in Frontend

Is there any background on what forced the change?

Would it be acceptable to replace should incrementally?  E.g. in different test suites individually?

We had to move from Should.js to Chai in Frontend

Is there any background on what forced the change?

I'm working on replacing Vue CLI with Vite (getodk/central-frontend#671). As part of that, I'm replacing Karma with Vitest (specifically, Vitest's browser mode). However, Should.js doesn't play well with Vitest: I see an error when I try to use it. I think it's because Should.js doesn't play well with ES Modules.
It actually didn't take too long to replace Should.js with Chai in Frontend, which is part of what made me think that we might as well make the switch in Backend as well. I like that Chai is much better documented and that it's maintained.

Would it be acceptable to replace should incrementally? E.g. in different test suites individually?

I think it'd be reasonable for migration tests to start using Chai right away. If anything, it'd be one fewer instance of Should.js for us to convert later. 😅
Chai has three options for assertion style: assert, expect, and should. I adopted the should style in Frontend, since that allowed me to minimize the number of changes. However, like Should.js, the should style of Chai extends Object.prototype, so I don't think that Should.js and Chai can coexist in tests that are run together. It's not like one integration test could use Should.js while another uses Chai. But since migration tests are separate, I think it would work for them to use Chai from the start.


Some updates to the Windows Insider Program documentation

This PR adds:

Removes all "en-us" locale from links except for those that really need as it the links won't work without it

This PR fixes:

Before:
After:

Before:
After:

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

I will fix the wip/at-home/Whats-new-wip-at-home-20h1.md file conflict when I have time :)

This PR can be merged now :) cc @bleblancMSFT @lizap


Add i686-unknown-linux-musl cross-std builder

This should probably use the linux builder with the old build system as we would want to run a full make check like we do with the x86_64-bit musl support.
cc https://github.com/rust-lang/rust-buildbot/issues/78
cc @japaric

Ok, this has a similar problem as the i586-unknown-linux-gnu builder. We can either (a) build and test this target like x86_64-unknown-linux-musl, or (b) only build it like we do for other cross targets. In either case we cannot build in the standard linux-cross image because there's no 32-bit linker/gcc there (the gcc-multilib package conflicts with all the cross-gcc packages).
So either we need to take the route of (a) which seems a bit heavyweight, or we need to figure out how to get a 32-bit compiler in the linux-cross image.

take the route of (a) which seems a bit heavyweight

What makes you say that? Because building std for i686-musl would require running the full test suite?

or we need to figure out how to get a 32-bit compiler in the linux-cross image.

We can use crosstool-ng to build a toolchain (+ libc/libgcc/etc) for i586 and another one for i686. Why two of them? Because we had problems with using a lower bound compiler for two slightly similar targets: arm-gnueabihf and armv7-gnueabihf. Let me know if you want to go down this route, and I'll look into the crosstool-ng configuration.

Yeah it's true we could just not run tests, from a convenience point of view it'd be easier to run in the normal linux-cross image as that's where all other cross compilations happen. Otherwise we'd need to set up a new builder and configure it correctly.
That seems albeit easier than setting up a whole new gcc toolchain for two targets though, so that's perhaps just the best option. Setting up a dedicated builder to cross to 32-bit and maybe smoke test musl at least.

Working on this over at https://github.com/alexcrichton/rust-buildbot/tree/more-cross-builders

Ok, I've made a PR: https://github.com/rust-lang/rust-buildbot/pull/95
We'll need to add Cargo + rustup builds for i686-unknown-freebsd as well eventually to push that compiler over the finish line.

Done!


Feature/rag evaluation framework

Context

We need to be able to compare different configurations of our RAG pipeline to see whether we are making improvements (either to earlier configuration versions, or to generic non-Redbox approaches).
Changes proposed in this pull request

Added a notebook that can be used to create an evaluation dataset locally, using the RAGAS framework

Added a notebook listing and linking to Core API promtps that can be tweaked to change performance of Redbox RAG chat

Added a notebook for completing the evaludation dataset by adding actual_output from running the Redbox RAG locally. This notebook then runs evaluation metrics for retrieval and generation evaluation using the DeepEval framework.

Guidance to review

Check if the notebooks run on your local machine
Relevant links
Things to check

[ ] I have added any new ENV vars in all deployed environments
[ ] I have tested any code added or changed
[ ] I have run integration tests

evaluation_dataset_generation.ipynb
When I brought in two PDFs we used to build the original Redbox, Langchain chunking just put the whole (short-ish) doc in a single chunk. Is this enough of a problem to bring in the core-API chunking method now?

This is not enough of an issue to address now. In establishing a base evaluation dataset, we just want input questions and expected outputs. The chunking strategy used to get these is not important. Certainly something we can look into in future thought.

Deleted the old notebooks that were holding code as we set up a clean evaluation framework, which is the purpose of this PR.  Suggest adding:
import os
import boto3
from botocore.exceptions import ClientError
from dotenv import load_dotenv, find_dotenv

load_dotenv(find_dotenv())

client = boto3.client(
    "s3",
    aws_access_key_id=os.getenv("AWS_ACCESS_KEY"),
    aws_secret_access_key=os.getenv("AWS_SECRET_KEY"),
    endpoint_url=f"http://localhost:{os.getenv('MINIO_PORT')}",
)

try:
    client.create_bucket(
        Bucket=os.getenv("BUCKET_NAME"),
        CreateBucketConfiguration={"LocationConstraint": os.getenv("AWS_REGION")},
    )
except ClientError as e:
    if e.response["Error"]["Code"] != "BucketAlreadyOwnedByYou":
        raise e

Or we could add a make backend that does it for them.
@wpfl-dbt is this now covered in the eval_backend command you added to Makefile?


setup.py: move mysqlclient/coverage to optional deps

Use extras_require feature from setuptools:
Coverage is now an extra, and can be automatically installed by:

pip install teuthology[coverage]
pip install git+https://github.com/ceph/teuthology.git#egg=teuthology\[coverage\]
pip install git+https://github.com/ceph/teuthology/archive/wip-minimal-reqs.zip#egg=teuthology\[coverage\]

In requirements.txt the proper syntax is:
teuthology[coverage]==<version>
BTW, motivation for this change comes from https://github.com/ceph/ceph/pull/29504#discussion_r316768010, where we have to install teuthology deps to run dashboard tests.
Signed-off-by: Ernesto Puerta<EMAIL_ADDRESS>

Why this is required?

BTW, motivation for this change comes from ceph/ceph#29504 (comment), where we have to install teuthology deps to run dashboard tests.

retest this please

Why this is required?

BTW, motivation for this change comes from ceph/ceph#29504 (comment), where we have to install teuthology deps to run dashboard tests.

I kind of assume that by quoting things that way you already got the answer you were looking for. But just to shed more light into this: that mysqlclient dependency, which is not strictly required to run teuthology (only for coverage measurement), cannot be fully satisfied by pip. It needs Python & MySQL devel packages installed from the distro package manager.

@kshtsk i think this makes sense. as mysqlclient is not a necessarily a runtime or even build-time dependency in all cases. so probably we can just make it optional.

@kshtsk i think this makes sense. as mysqlclient is not a necessarily a runtime or even build-time dependency in all cases. so probably we can just make it optional.

I like this idea.
@gregsfortytwo  @zmc do you know the cases we where teuthology-coverage is used?

Since coverage may require there should bootstrap script fixed as well to use extra.

But bootstrap script is using requirements.txt (which still includes mysqlclient) for the dependencies, and as I haven't touched that, it should work fine.

Since coverage may require there should bootstrap script fixed as well to use extra.

But bootstrap script is using requirements.txt (which still includes mysqlclient) for the dependencies, and as I haven't touched that, it should work fine.

bootstrap uses ./virtualenv/bin/python setup.py develop

Since coverage may require there should bootstrap script fixed as well to use extra.

But bootstrap script is using requirements.txt (which still includes mysqlclient) for the dependencies, and as I haven't touched that, it should work fine.

bootstrap uses ./virtualenv/bin/python setup.py develop

Which does nothing, regarding dependency install. And a few lines before:
https://github.com/ceph/teuthology/blob/42ebe0d580e9669c38aaeabf768cc1fef31de16f/bootstrap#L139-L140
And inside that:
https://github.com/ceph/teuthology/blob/42ebe0d580e9669c38aaeabf768cc1fef31de16f/requirements.txt#L58

Since coverage may require there should bootstrap script fixed as well to use extra.

But bootstrap script is using requirements.txt (which still includes mysqlclient) for the dependencies, and as I haven't touched that, it should work fine.

bootstrap uses ./virtualenv/bin/python setup.py develop

Which does nothing, regarding dependency install. And a few lines before:
https://github.com/ceph/teuthology/blob/42ebe0d580e9669c38aaeabf768cc1fef31de16f/bootstrap#L139-L140
And inside that:
https://github.com/ceph/teuthology/blob/42ebe0d580e9669c38aaeabf768cc1fef31de16f/requirements.txt#L58

Actually it does, look carefully, it creates binary scripts.

I tried to refactor bootstrap/tox.ini to use pip install -e .\[coverage], but it required more and more changes and caused complex dependency issues (as this requires sitepackages too).
Reverted to the original proposal.

going to run deployment test

Thanks, @kshtsk !


Add support for EDGE_ALL of margin / padding / borderWidth

See the SVG file changes between the two commits. (here)

Good catch! I think we're also completely missing borderWidth. Should we do that here?

@jacobp100 Yeah it is also EDGE_ALL for node.setBorder, so I think it can be in this PR.

👍

Thanks!

Shipped as 0.0.24


Yeomon CRUD Module Sub-Generator breaks project

yo meanjs
grunt
localhost:3000 works perfectly as expected
yo meanjs:crud-module myModule
Nothing loads on localhost:3000

In which version are you experiencing this problem?

I assume "latest stable", because that's what the docs say.

the Official Yo Generator, which generates the latest stable copy of the MEAN.JS boilerplate

Also
$ yo --version
1.3.3

This is probably obvious, but I should also mention that the yo meanjs:crud-module <module-name> step is part of the "Getting Started" guide.

I will check this for 0.3.x. I'm working on the generator right now for 0.4.x.

You can verify the version of the generator with npm view generator-meanjs version. Latest seems to be 0.1.12. Just to make sure you have the latest version.

Yup, mine is 0.1.12.

Works for me w/ 0.1.12. Are there any errors when running Grunt after yo meanjs:crud-module myModule - either in the console or your browser? The page will be blank if bower install failed to install Angular for some reason.

the crud-module generator works for me. can you check your javascript browser console for any errors?

Here is the error in the javascript browser console:
Uncaught Error: Menu does not exists menus.client.service.js:40validateMenuExistance menus.client.service.js:40addMenuItem menus.client.service.js:84(anonymous function) mymodules.client.config.js:7invoke angular.js:3965(anonymous function) angular.js:3811forEach angular.js:325createInjector angular.js:3811doBootstrap angular.js:1444bootstrap angular.js:1459(anonymous function) application.js:19trigger angular.js:2573(anonymous function) angular.js:2853forEach angular.js:325eventHandler

This is right after running this command:
➜  advancement git:(master) ✗ yo meanjs:crud-module myModule
? Which supplemental folders would you like to include in your angular module?
? Would you like to add the CRUD module links to a menu? Yes
? What is your menu identifier(Leave it empty and press ENTER for the default "topbar" menu)? myModule
   create app/controllers/mymodules.server.controller.js
   create app/models/mymodule.server.model.js
   create app/routes/mymodules.server.routes.js
   create app/tests/mymodule.server.model.test.js
   create app/tests/mymodule.server.routes.test.js
   create public/modules/mymodules/config/mymodules.client.routes.js
   create public/modules/mymodules/controllers/mymodules.client.controller.js
   create public/modules/mymodules/services/mymodules.client.service.js
   create public/modules/mymodules/tests/mymodules.client.controller.test.js
   create public/modules/mymodules/config/mymodules.client.config.js
   create public/modules/mymodules/views/create-mymodule.client.view.html
   create public/modules/mymodules/views/edit-mymodule.client.view.html
   create public/modules/mymodules/views/list-mymodules.client.view.html
   create public/modules/mymodules/views/view-mymodule.client.view.html
   create public/modules/mymodules/mymodules.client.module.js

No errors from grunt:
  advancement git:(master) ✗ grunt
Running "jshint:all" (jshint) task
>> 66 files lint free.

Running "csslint:all" (csslint) task
>> 2 files lint free.

Running "concurrent:default" (concurrent) task
Running "nodemon:dev" (nodemon) task
Running "watch" task
Waiting...
[nodemon] v1.2.1
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: app/views/**/*.* gruntfile.js server.js config/**/*.js app/**/*.js
[nodemon] starting `node --debug server.js`
debugger listening on port 5858
NODE_ENV is not defined!  However, it doesn't have an articles property for the built-in articles module either.

In firefox I also get an error about livereload:

Seems like you set an invalid menu (if you haven't modified anything); ? What is your menu identifier(Leave it empty and press ENTER for the default "topbar" menu)? myModule. Instead of myModule try with the default and see if that helps.

Note that the menu ids are in no way related to you app names.

@coryfklein, @nanuuki is correct. The app is boilerplated with only one pre-existing menu. Use "topbar"

That resolved the problem.
I read What is your menu identifier as asking me what I wanted the item to be named in the top menu bar. My bad.

No problem. Glad it's resolved.

I just made the exact same mistake. Maybe the wording in the generator prompt should be changed in the future?

made the exact same mistake

@GPnow Maybe submit a pull request?

using "yo meanjs" version 1.4.7, creating crud module still doesn't show up on topbar - would appreciate any suggestion? - thanks
i tried "topbar" as suggested by earlier comments, regenerated all files, restarted grunt but doesn't seem to be working
bash-3.2# yo meanjs:crud-module superheroes
Which supplemental folders would you like to include in your angular module? css, img, directives, filters
What is your menu identifier(Leave it empty and press ENTER for the default "topbar" menu)? topbar
create app/controllers/superheroes.server.controller.js
create app/models/superhero.server.model.js
create app/routes/superheroes.server.routes.js
create app/tests/superhero.server.model.test.js
create app/tests/superhero.server.routes.test.js
create public/modules/superheroes/config/superheroes.client.routes.js
create public/modules/superheroes/controllers/superheroes.client.controller.js
create public/modules/superheroes/services/superheroes.client.service.js
create public/modules/superheroes/tests/superheroes.client.controller.test.js
create public/modules/superheroes/config/superheroes.client.config.js
create public/modules/superheroes/views/create-superhero.client.view.html
create public/modules/superheroes/views/edit-superhero.client.view.html
create public/modules/superheroes/views/list-superheroes.client.view.html
create public/modules/superheroes/views/view-superhero.client.view.html
create public/modules/superheroes/superheroes.client.module.js


An error occurs when executing "Edit Scale Rule Setting..." command from command palette

OS: Win10/M1
Build Version: 20220520.7
Regression: Not a Regression
Repro Steps:

F1 -> Azure Container Apps: Edit Scale Rule Setting...
Check whether there is no error.

Expect:
There is no error when executing "Edit Scale Rule Setting..." command from command palette.
Actual:
An error occurs when executing "Edit Scale Rule Setting..." command from command palette.

More Info:
This issue does not reproduce when executing "Edit Scale Rule Setting..." from context menu.

Verified this issue on extension build 20220525.1, it has been fixed.


fix clippy lints

Closes #427.
This PR fixes or allows all errors reported by rust-clippy master. No lints are allowed globally, but we might want to consider it (float_cmp and blacklisted_name seem particularly spurious).
Unfortunately, there is not really a way to "future-proof" this right now. We will have to just run clippy periodically and check the output (unless we want to add it to our CI, but that might be more trouble that it's worth).

Looks great to me, thanks for splitting these all up. :)


Separation of Concern and Shared Component Updates

First I implemented the same folder structure as MRTK:
SceneUnderstanding

Core
-- Editor
-- NavMesh
-- StandardAssets
-- Understanding
Examples
-- NavMesh
-- Placement
-- Simple
-- StandardAssets
-- Understanding

We should now be able to produce a Unity Asset Package from the SceneUnderstanding folder. The only thing missing would be the NuGet packages.
Next I removed the word "Sample" from all scene names. Since all scenes are under the "Examples" tree, it's inferred that they are samples.
Think of the folder structure under Core and Examples as representing "Features". For example we have an Understanding feature, a NavMesh feature and so on. All Scripts / Resources / etc. that will be required by all developers to use a feature go in Core/Feature. Any Resources / Scripts / etc. that are only used to demonstrate the feature go in Examples/Feature.
Any Resources / Scripts, etc. that are shared across all features go in the "StandardAssets" folder. "StandardAssets" is a MRTK nomenclature, but I've seen it used in other Unity projects as well. There are both Core/StandardAssets and Examples/StandardAssets.
There was a scene called "Simple". But because "Simple" isn't a feature, I renamed this to "Understanding". Understanding is really the main feature this scene is demonstrating.
It may seem a bit excessive to have a "Scenes" folder under a Feature since each feature currently only has one scene. However, in the future we may want more than one scene do demonstrate a given feature. For example, NavMesh may have a Simple demo and an Advanced one. Or perhaps maybe someday a touch version instead of HMD. Looking at MRTK we can see a total of seven scenes to demonstrate the Hand Tracking feature alone.
There were a number of folders in the root that had mainly to do with the fact that this Unity project can be deployed as an app itself. For example, the Resources folder had UWPImages which are used for the splash screen, tiles, etc. These assets were moved under a SceneUnderstandingApp folder. This will hopefully more clearly separate these assets for deployment rather than being part of the Core or Examples.
If I could not readily determine whether something was Core or Example, generally I put it under Examples. This is especially true for things named "Legacy".
I turned the menu into a prefab and improved how it was being shown and hidden. It now uses TextMeshPro and no longer depends on a MeshRenderer being in the root.
I converted SolarGridPattern from TGA to PNG. This shrunk from 16 MB to 38k with no perceptible loss.
I updated ObjectPlacerManager to have an inspector slot for material. This way the material isn't loaded by name or a hard-coded path.
Several classes (including SceneUnderstandingMangaer) were using Resource.Load to load materials and other assets if the user didn't specify them. However, MRTK avoids using Resource.Load. I'm not positive why, but educated guesses are to avoid having multiple Resource search paths as well as to avoid name clashes. Instead, if default values are desired then prefabs are generally used. (See a list of UX prefabs here.) Using a prefab also allows default values to be changed in one place and have that change apply to all scenes (unless overridden). I created a prefab for SceneUnderstandingMangaer and updated the sample scenes to use it.
InputManager was updated to be reusable across scenes. InputManager has an option to include the default commands (both speech and keyboard key) and it's easy to add new ones. Note how much simpler ObjectPlacer and PathFindingController are now.

Updated the Menu to be shared across scenes. The menu dynamically updates to show whichever commands are registered in the InputManager, including ones that are only for the current scene.

I show phrases and keys at all times now since HoloLens may have a keyboard and spoken phrases do work in the Editor. We can change this behavior back if you prefer. I do disable commands in the editor that only make sense on the device.

Tagging @jorgonz and @SzymonSPS for review.


possible API change to cve list

During cve list, it is no longer guaranteed that cve['requested_by']['user'] is true. If a user is unknown, it is not being set. This appears to be due to change in the CVE Services API which occurred since Jan 11 2024.
To fix this I'm using the following in cve_list():
    for cve in cves:
        user = "pre_idr_user"
        if 'user' in cve['requested_by']:
            user = cve['requested_by']['user']
        lines.append(
            (
                cve["cve_id"],
                cve["state"],
                cve["owning_cna"],
                f"{user} ({cve['requested_by']['cna']})",
                human_ts(cve["reserved"]),
            )
        )

What is odd, is that old CVEs which lack a user can still be called with cve show $CVE_ID, even though print_cve_id() calls cve['requested_by']['user'].

I can confirm the same issue.  pip3.11 install cvelib on RHEL-8.

Related issue in cve-services: https://github.com/CVEProject/cve-services/issues/1176


字体文件加载失败

downloadable font: download failed (font-family: "element-icons" style:normal weight:400 stretch:100 src index:0): status=2152398866 source: chrome-extension://ckkcogedanhgeifldcjgnodgdcfhiben/fonts/element-icons.woff

svg文件问题

Hi, This problem has been resolved?  Can you provide solution?


ci(changesets): versioning packages - v0.92.2 @ master

Summary
In this release, we:

Upgraded forc to v0.62.0

Features

#2755 - Add testing to create-fuels, by @danielbate

Chores

#2579 - Integrate launchTestNode into current test suites, by @maschad
#2794 - Upgrading forc to 0.62.0, by @arboleya

Docs

#2772 - Add more comments around the create-fuels template app, by @Dhaiwat10

@FuelLabs/sdk-ts Whad do you think about changing the title of the changesets PR?
From:

ci(changesets): versioning packages - v0.92.2 @ master

To:

ci: release v0.92.2 @ master


Not render aggregation_column for category if aggregation is count.

This PR fixes #9638.

could you take a look @javierarce? 🙏

👍  A couple of super minor things.


Permission verification was not conducted when deleting the S3 bucket

I've found that users with write permissions can delete S3 buckets.
Is it a good idea to change Write permissions to Admin permissions?
This way, there would be fewer risky operations.

#5312

If you want to limit privileges only for administrators, then it’s better to do it here
https://github.com/seaweedfs/seaweedfs/blob/6181aa7594a7f8459910807e581d84adb2ab44c6/weed/s3api/s3api_server.go#L281

If you want to limit privileges only for administrators, then it’s better to do it here
https://github.com/seaweedfs/seaweedfs/blob/6181aa7594a7f8459910807e581d84adb2ab44c6/weed/s3api/s3api_server.go#L281

Thanks for your reply.
I saw this code through your guidance and modified my PR.
Do you have time to have a look?


fix(Tabs): use ::after pseudo-element for tab stripe

Replaced iui-tab-stripe with ::after pseudo-element applied on ul.iui-tabs.
Using a pseudo-element ties the positioning to the main element, and fixes random bugs that break stripe positioning in scenarios like scrolling (e.g. iTwin/iTwinUI-react#421).
Before:

After:

The stripe animation doesn't appear to be working for me.  I'm using the latest version of Brave.

The stripe animation doesn't appear to be working for me. I'm using the [latest version of Brave]

Fixed


bug in is_isomorphous cell comparison

The method documentation says:
https://github.com/rs-station/reciprocalspaceship/blob/ceae60e293bfdb3e969d0e3e2b53fa3a2b9e34f9/reciprocalspaceship/dataset.py#L1191-L1192
But the code looks for an absolute change:
https://github.com/rs-station/reciprocalspaceship/blob/ceae60e293bfdb3e969d0e3e2b53fa3a2b9e34f9/reciprocalspaceship/dataset.py#L1217
By default this will return False if the cells differ by more than 5 Å or 5 degrees in any case.
I'd suggest we change this to make it relative, as the documentation indicates.

Can confirm. This is a bug. I like your proposal.

what's a reasonable default for the cell_threshold?  i think 0.5% is in the right ballpark. @tjlane

fixed in #279


error failed to apply the ignorePolicy

trivvy-operator logs following error messages:
filter error: filtering error: unable to filter vulnerabilities: failed to apply the policy: unable to prepare for eval: 1 error occurred: trivy.rego:0: rego_parse_error: empty module
I expect ignoPolicicy config from values file apply via configMap on scan jobs. It did work before.
Scan jobs pods being OOMKilled.
Environment:

Trivy-Operator version: 0.19.3
trivy-operator helm chart version: 0.21.3
Kubernetes version: v1.28.5
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): AKSUbuntu-2204

@KateFiroozi have you updated resources

@KateFiroozi have you updated resources

Yes, this issue we had to solve as well:
  resources:
    requests:
      cpu: 500m
      memory: 1Gi
      # ephemeralStorage: "2Gi"
    limits:
      cpu: ""
      memory: 1Gi
      # ephemeralStorage: "2Gi"

we have also managed to solve ignorePolicy with this workaround in values.yaml (although it should have not be necessary):
  # ignorePolicy: |
  #   # applies to all other workloads
   ignorePolicy.kube-system.azure-.: |
     package trivy

     import data.lib.trivy

     default ignore = true

I guess so, but can then docs be updated to have proper instructions?

@KateFiroozi sure, would you like to raise a PR ?


create method of procedurally generating quests for import

This is never going to happen. All we can really do is better plan our quests and use copy and paste to o the heavy lifting for us. Alas, this is a heavily manual process and the build in editor is probably the best and most efficient UI


循环Fix any errors below, if possible.

Issue
选择一个文件，输入“分析功能，画出plantuml”
能正确输出plantuml，但会出现："Fix any errors below, if possible."
然后一直循环重复

Version and model info
Aider v0.39.0
Model: deepseek/deepseek-coder with whole edit format
Git repo: .git with 101 files
Repo-map: disabled

Sorry, I'm not sure I understand the issue you are reporting?

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Code review

Hello,
I checked this component and tested it with Intel 8080 Parallel 16bit interface. It is working but I have some notes for code:

Why do you use malloc and color data recalculation in panel_ili9488_draw_bitmap? It is wasting of memory, it is working properly, when changed init to { LCD_CMD_COLMOD, { ILI9488_COLOR_MODE_16BIT }, 1 } and put whole color data esp_lcd_panel_io_tx_color(io, LCD_CMD_RAMWR, color_data, color_data_len * 2);
Use this ili9488->bits_per_pixel / 8 instead of the constant color size. You can initialize right by this and do not need allocate another buffer (again).

I didn't checked all code line by line but these things are important to change. Please, could you fix it. Or should I make PR?
If you have any questions, feel free to contact me.

@espzav the 16 bit color mode does not work with SPI :( so I guess that's the reason.
Here's my past attempt https://github.com/lvgl/lvgl_esp32_drivers/pull/103

@espzav the 16 bit color mode (RGB565) does not work with SPI :( so I guess that's the reason.
Here's my past attempt lvgl/lvgl_esp32_drivers#103

Oh, thank you for clarification. Then, there should be any condition for using with parallel without wasting memory.

@tore-espressif Please pull the latest master and give it a test as I've addressed your feedback and it should work correctly now for both with no need to modify the ili9488 code.

@atanisoft I saw the readme, please, could we add "Intel 8080" into the Communication interface? I will test the changes on parallel LCD, tomorrow.

@espzav I've added that and a note on the color depth mode requirements for SPI vs Intel 8080 on master.

@atanisoft Thank you very much!

@atanisoft I tested parallel interface on HW now. It is working properly. You can upload it into Component manager as version 1.0.5.

Thanks @espzav for testing on HW. Do you have any recommendations on HW that supports the parallel IO modes?  I'm currently looking at a few bare displays using a 40-pin FPC and using an adapter to convert to breadboard while developing a possible hardware product.

@atanisoft Do you think recomendation with ILI9488 controller or any parallel controller? It depends on size and application. All supported and tested LCD displays are here: https://github.com/espressif/esp-bsp/blob/master/LCD.md Parallel are ESP32-S2-HMI-DevKit-1 and Waveshare 7" LCD. Both are 800x480, the physical size is different.