Add 3 more JSON schemas

Description
Adds three more JSON schemas, to be added with #364 :

Adds schemaCollisionMesh.json
Adds schemaPlayerSpawnConfig.json
Adds schemaSolNames.json

Notes

Many schema segments were derived off of #357 for consistency (thank you @Adrijaned)
dynamicObjects for collisionMeshes seems to always be blank/unused, so I put it in as blank in the schema accordingly. Is this property used somewhere that I missed?

Can one of the admins please verify this patch?

@GooeyHub : add to whitelist

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

@vampcat & @jellysnake either of you two want to sign off on and merge this, if it is ready?

@jellysnake no need to wait for older reviews that may have gone obsolete. If you're happy stuff you requested has been addressed and have tested feel free to override / merge locally and push (my usual approach).
Bit different of course if another review may have unanswered review comments / things pending but everything here looks addressed :-)

Okay, consider it merged.


problems when file is called main.ts

I've set it up in Windows and it works great, however when I created the files server/main.ts and client/main.ts, they got compiled (I could get compile errors) - but never executed, neither on client nor on the server.
Renaming any of the file from main.ts to anything else (for instance server.ts or index.ts) "fixed" the problem. It isn't really fixed, as I'd like to be able to create a file with the name main.ts. Maybe I can instead create main.something.ts - I'll check that later, but any idea why main.ts isn't working?

I've checked it and main.anything.ts is also not working

Thanks so much @pgrm . Was stuck on this for quite a bit.
And this has nothing to do with Windows btw. On Ubuntu, it seems to compile the main.ts file (I get compilation errors). But the code in it seems to be totally ignored. Changing the filename fixes it.

Seems to be not an issue any more.


Add golangci-lint workflow

Description
Add golangci-lint workflow

@orishoshan I'd love your take on this - the linter fails here with the following error:
level=error msg="Running error: 1 error occurred:\n\t* can't run linter goanalysis_metalinter: buildir: failed to load package pcap: could not load export data: no export data for \"github.com/google/gopacket/pcap\"\n\n"

I couldn't really figure out what's wrong with that package...

@amitlicht I think it's a wrapper for libpcap, which probably breaks the linter. Is it possible to add a skip rule for that package?


Used Statement in PostGISSchemaEditor._create_index_sql().

@charettes You introduced Statement and other DDL classes a long time ago, but the PostGISSchemaEditor was not touched.

Thanks!


Update README.md

sorry it was hurting my ocd so i decided to fix it
anyways, amazing video, very helpful!

Ty!!! I got ChatGPT to do this and it broke half-way and I was too hungry to fix it last night💀


Test installing on Synology

See how it works and if there's anything that can make it easier

This doesn't work because Docker on Synology doesn't support secrets. Looks like I need a new fallback feature...

Done and works with the latest build.

Done and works with the latest build.

How do you fixed this?

See the project wiki for information on how to install on a Synology or Unraid server.
If you want the code changes check out the pull request.


[pubsub] topic name does not match queue name with RabbitMQ

In what area(s)?

/area runtime

/area operator

/area placement

/area docs

/area test-and-release

What version of Dapr?

1.1.x

1.0.x
edge: output of git describe --dirty

Expected Behavior
I have a DAPR-subscriber and a non-DAPR publisher with RabbitMQ.
I have set the subscriber to subscribe to "test_queue". Then I have the publisher produce to "test_queue" with a direct implementation of RabbitMQ.
I expect the DAPR-subscriber would be able to receive such a message.
Actual Behavior
The DAPR-subscriber instead subscribes to "appname-test_queue" when I look on the RabbitMQ management UI. Setting the producer to produce to "appname-test_queue" is my current workaround
Having to add the app name in front of the queue on the publisher side is confusing. I couldn't find this behavior documented anywhere in the docs.
Steps to Reproduce the Problem
Using a pubsub.yaml
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: worker-pub-sub
spec:
  type: pubsub.rabbitmq
  version: v1
  metadata:
  - name: host
    value: "amqp://rabbitmq:5672"
  - name: durable
    value: "false"
  - name: deletedWhenUnused
    value: "false"
  - name: autoAck
    value: "true"
  - name: reconnectWait
    value: "1"
  - name: concurrency
    value: parallel
scopes:
  - appname

Next, following the official document to subscribe to raw events
https://docs.dapr.io/developing-applications/building-blocks/pubsub/pubsub-raw/#publishing-raw-messages
I implemented a grpc subscriber in Python as follows
from dapr.ext.grpc import App

app = App()

@app.subscribe(pubsub_name="worker-pub-sub", topic="test_queue")
def on_msg(*args):
  print("OK")

app.run(6002)

Then publishing a message to the same queue name: e.g.
import pika

with pika.BlockingConnection(pika.ConnectionParameters(host=RABBITMQ_HOST)) as conn:
  ch = conn.channel()
  ch.queue_declare("test_queue")
  ch.basic_publish("", ""test_queue", "test")

Release Note

RELEASE NOTE:

potential duplicate of https://github.com/dapr/dapr/issues/5178

For this use case I recommend you look at using the binding: https://docs.dapr.io/reference/components-reference/supported-bindings/rabbitmq/


we are implementing mathjax with visual math editor

Hi Fellows we are implementing mathjax with Visual Math Editor Here when we put this code in mathjax it is not rendered it into math functions.

You have not given enough information to identify the issue.  The page you link to seems to work for me.    And what browser and OS you are working with?
Because your are using a complex codebase, it may be difficult for us to diagnose any issues you are  experiencing.  If you could produce a minimal (non-)working example that illustrates the problem you are experiencing, that would be a big help.  Using jsFiddle or codebin are two popular options for presenting that.

Hi dpvc,
Thanks in advance,
we want to implement mathjax with quilljs with chemjs and other scripts. we currently open these scripts in iframe and where user will put equation and after that we put that equation on latex values on directly quilljs area but it not convert it them into latex equation. we are developing this for website we are using windows 7

While an overview is nice, it is not specific enough to diagnose your issue.  Without actual code details, there is not really much I can say about what is going wrong.
Make sure you have read the documentation about dynamo ic content and MathJax.  That gives the command you need to give when new math is added to the page.  Perhaps that is what you are looking for.


Add GCD and LCM utility functions to Python snippets

I added 2 code snippets, one for finding the greatest common divisor (GCD) of two numbers using the Euclidean algorithm and a second one for finding the least common multiple (LCM) of two  numbers.

GDC is already available as a function of the math package in python, so I think it should this function instead of calculating the result itself. For the LCM you can use x * y // gcd(x, y) which uses the gdc to calculate the lcm, it's a lot simpler.
Those a suggestion, feel free to ignore them if they dont fit what you want the snippets to look like

You are right, i totally forgot. Instead i made some string manipulation snippets I will show on a new pull request.

Already built-in functions from the math packages


The start position of dropdown looks weird if the last item is currently selected (if dropdownPosition is set)

In the image above, "angle" is the last item of a lengthy list of 2x items.
I believe the problem is due to the scrollTo  method of <ScrollView> in line 301 in dropdown/index.js.
I think to fix this we have to calculate the correct offset of the last few items? I haven't tried it yet.
Great library btw!

@longsangstan Can you try installing my fork and see if you still have this issue? npm install github:kodie/react-native-material-dropdown#master

Thanks for issue! Released fix in 0.10.0. Overscroll is now disabled for positive and negative dropdownPosition.


No Custom Title Bar

VSCode Version: 1.30.0
OS Version: Windows 10 version 1809

Steps to Reproduce:

Ensure "window.titleBarStyle" setting is either default, or set to "custom"

Does this issue occur when all extensions are disabled?: Yes
The custom title bar is no longer available in VS Code 1.30. Changing the setting has no effect.

/duplicate #63982


Completion does not work in VS Code

Seems like there are restrictions on the legal combinations of properties in CompletionItem.
Also, textEdit is now deprecated and I will from now on use range and insertText.
@mniewrzal @kaloyan-raev let me know if this will cause problems in Eclipse (Che)

@felixfbecker Can you point me to these changes in the protocol? I cannot find where it is stated that textEdit is deprecated. I also don't see the new range property in the CompletionItem. I am looking at version 3.0 of the protocol.

At the moment changes in CompletionItem are missing. Something goes wrong with latest commit.
https://github.com/Microsoft/language-server-protocol/issues/165

https://github.com/Microsoft/language-server-protocol/commit/b082a3bd761f7c50db0b538e954ac467e8aa5ba8 reverted the deprecation, very weird!

@felixfbecker Regarding your question. This change will affect other clients like Eclipse Che and LSP4E. Most probably they will need to adapt to use insertText and range instead of textEdit. But, if this is the intended direction of the protocol, then feel free to do it.

I agree with @kaloyan-raev. Main Java binding for LSP is still not updated with new CompletionItem but its just matter of time https://github.com/eclipse/lsp4j/issues/41

I am not using textEdit consistently atm either so if I want to clean it up I think the best would be to go with the new properties now. Just wanted to check if it would be a breaking change (major release), I think it would not because your clients should in theory fall back to inserting the label.

Not sure if my problem is directly related to this issue but here goes.
Auto completion are not automatically triggering for me. For e.g. if I have a class method with argument $options, when typing $op in the method body i would expect to get completion suggestion for $options, but i don't. I do get suggestions list upon using Ctrl + Space keys. Is this expected behavior?
I am using the language server release 4.2.2 and VS Code version 1.9.0-insider
Commit 4265f5e35cb3ea8ba7e0cf6b14ec606b6f8647ea

Not related, please open new issue

https://github.com/Microsoft/vscode-languageserver-node/issues/95


slide up and slide down in same div

Is there a way to use both slide up and down in the same div, according to the direction of the scroll?
Like, if im going to the bottom of the page divs slide up, and then i decide to go up again the divs went not visible state slide to down...

Hello @buutqn. Unfortunately it's not possible


Cyclic-1.10.2-1.5.3 cause Refined Storage Solderer stop working.

Please fix/check this bug. Version Cyclic-1.10.2-1.5.2  working fine. TQ

Damn. I released 1.5.2 to fix this bug so i must have made a mistake somewhere, ill check it out right away

I just tested it, and it seems ok for me.  I will list some of the mods installed
Cyclic-1.10.2-1.5.3
MCMultiPart-1.2.1-universal
refinedstorage-0.8.20
Forge version 18.1.2046
Now the solderer did not work until I added a Controller attached to it, totally regardless of power.  In fact i tested the config file and set
The energy used by Solderers [range: 0 ~<PHONE_NUMBER>, default: 3]
I:solderer=0

And even then, the solderer did not process at all UNTIL i added in the refined storage controller block.
The recipe i tested was silicon to printed silicon.
Please check your RF and your controller.  If it is still an issue, let me know what mods you have installed and what recipe you are testing and we can go from there

Thanks again for the report, but it works for me as noted so I will close this issue for now.  If it happens again please comment / reopen and post any details you have.  Its possible that some other 3rd mod is causing a problem for you, my tests were only with the two mods.

Please tell me how to smelt your Nether Ore?  I using Allthemods modpack. TQ
On Thu, Aug 25, 2016 at 12:52 AM, Sam Bassett<EMAIL_ADDRESS>wrote:

Closed #71 https://github.com/PrinceOfAmber/Cyclic/issues/71.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PrinceOfAmber/Cyclic/issues/71#event-766280454, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJ2w5_BRQhhJGN-0As70XO6u0WKXHJl0ks5qjHbDgaJpZM4JpOOF
.


saxon-he: 11.5 -> 12.4

Description of changes
Updates to the latest release of SaxonJ-HE.
Adds saxon_11-he and saxon_12-he attributes for specific versions, and sets saxon-he = saxon_12-he.
Adds an updateScript for the latest versions of Saxon-HE.
cc @avi-jois
Things done

[x] Built on platform: x86_64-linux
[x] Tested compilation of all packages that depend on this change - no reverse dependencies
[x] Tested basic functionality of  ./result/bin/saxon-he.
[x] 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes) N/A
[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 289490 run on x86_64-linux 1

5 packages built:

jing (jing-trang)
saxon
saxon-he (saxon_12-he)
saxon_11-he
xmloscopy

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4125

Result of nixpkgs-review pr 289490 run on x86_64-linux 1

5 packages built:

jing (jing-trang)
saxon
saxon-he (saxon_12-he)
saxon_11-he
xmloscopy


Add new git tag to fetch latest version for Swift Package Manager

I really love this framework but can not retrieve the latest version of it.
I think the problem is git tag version tagged at an old state.

Same, need tags

I'm having the same issue now in May 2020, the same as it was last year when I tried to use it.. Please fix this!

Hey use the master branch or latest branch in Swift package manager, it works fine

Hey use the master branch or latest branch in Swift package manager, it works fine

it doesn't work if you use prelude as dependencies for your own swift package.
I have error like this
because package  is required using a version-based requirement and it depends on unversion package swift-prelude and root depends on  1.0.0, version solving failed.

Sorry for the silence from maintenance :) We've always advertised Prelude an "alpha/experimental/unstable" set of tools that we apply to our own exploration and projects, and we really do recommend maintaining downstream forks of any code that is useful to you. We may cut more releases in the future, but would not recommend relying directly on this repo as a source of truth. Going to close this out, and will try to make the README more clear soon.


Allow use of Right Control for Find My Mouse activation

Description of the new feature / enhancement
Allow use of Right Control for Find My Mouse activation (in addition to the current Left Control and/or mouse shake)
Scenario when this would be used?
Lefties.  Our mouse is under our left hand, to the left of the keyboard and our right hand is free to press the activation button ... which is only on the WRONG side of the keyboard!
Supporting information
No response

Fair suggestion. Wanting to mark this as duplicate against 14651, because that is more inclusive.

Duplicate of #14651


Word ladder and word ladder II

Related Issue
Closes #685
Description
Added solution for leetcode problems 126 and 127
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[X] Documentation update
[ ] Other (specify): _______________

Checklist

[X] I have performed a self-review of my code.
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@Ajay-Dhangar the changes you suggested has been modified

@mahek0620, to calculate the score for your leaderboard, you'll need to add something to those files and then try it out at the bottom. Below is the code you need to include for your author profile in the documentation:
---

Authors:

{['github_username'].map(username => (
 <Author key={username} username={username} />
))}


[Feature][UI Next][V1.0.0-Alpha] Add the corresponding state for the …

Purpose of the pull request
This pull request  adds the corresponding state for the saving action.
Releated issues
close #8854

Please execute the rebase command local.

Codecov Report

Merging #8870 (69806c7) into dev (d3cbf8e) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8870      +/-   ##
============================================
- Coverage     45.35%   45.30%   -0.05%
+ Complexity     4084     4081       -3
============================================
  Files           702      702
  Lines         27169    27169
  Branches       2907     2907
============================================
- Hits          12322    12309      -13
- Misses        13686    13698      +12
- Partials       1161     1162       +1

Impacted Files
Coverage Δ

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (-8.04%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

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 d3cbf8e...69806c7. Read the comment docs.


[Modules] Update AKS Orchestrator version when upgrade.

Description
Problem:
When the AKS is deployed with a default or spezific Version and then redeploy with a newer Version,
the module updates only the Kubernetes control plane not the node pools.

As a best practice, you should upgrade all node pools in an AKS cluster to the same Kubernetes version.
Solution:
The Operational version will be set to the Kubernetes version if it's set in parameters.

Update API version

Pipeline references

Pipeline

Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] My corresponding pipelines / checks run clean and green without any errors or warnings
[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (readme)
[x] I did format my code

Great work!

I will fix the pipeline run, somthing with the new api version.

I take a deeper look into the deployment locally.
The deployment fails at the provisioning of the VMSS for nodePools.
in the dependencies we using a keyVault for diskEncryptionSet and this one needs Purge Protection activated.
Message

The key vault 'dep-jpe-kv-csmazure' used for disk encryption set 'dep-jpe-des-csmazure' must have purge protection enabled. Target: 'VM: '/subscriptions/b688a246-149f-4e7b-9ac4-15a993403b69/resourceGroups/ms.containerservice.managedclusters-csmazure-rg_aks_jpecsmazure001_nodes/providers/Microsoft.Compute/virtualMachineScaleSets/aks-systempool-16738708-vmss/virtualMachines/aks-systempool-16738708-vmss_9', disk: 'OSDisk''.

As a KeyVault with purge protection for the deploy and throw away approach is not posible due to the retaition time
schould we remove this from the module?


Added reference type constraints

Would it be better to have it check for default(TNotification)? I'm a little wary of adding more generic constraints.

I think that is 100% fair!

@jbogard  Just thinking, would default(TNotification) be correct? It means you just cant send an empty version of an INotification implementation. but null would still be allowed. If we constrain it to class it means that the null check would still make sense, and that INotification must be of a reference type. So basically don't send ints or structs. Would it make sense to actually keep the class constraint, but actually add the default(TNotification) check as well, and throw a different exception?

Just a second though, by checking default(TNotification) what if an implementations default value was actually something you did want to publish?
I think the class constraint ensuring its a reference type and not null might make the most sense?


chore(release): v0.12.0 🎉

:robot: I have created a release beep boop
0.12.0 (2024-09-03)
Features

config: enable choice of practices supplementally and exclusively (d141ec3)

Bug Fixes

practs: bump practs to latest best (50c4664)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/declapract/releases/tag/v0.12.0 :sunflower:


relevanceThreshold is not utilized preventing additional memory items

Describe the bug
ISemanticMemoryClientExtensions.SearchMemoryAsync does not utilize relevanceThreshold parameter provided. As a result, memory check for similar items always returns the first inserted item and subsequent items are not added to the memory.
To Reproduce
Steps to reproduce the behavior:
Chat using the UI and check for LongTermMemory and WorkingMemory. It does not accumulate.
Expected behavior
Working and Long term memory should accommodate as chat history expands.
Screenshots

Would you like me to generate a PR?

@habanoz Hi! Thanks for reporting this problem,
Yes, if you have code that fixes the issue, please send in a PR.

@habanoz Hi! Thanks for reporting this problem,
Yes, if you have code that fixes the issue, please send in a PR.

PR #920 is sent to fix the situation.

PR is merged. Closing...


Why is not there Japanese page?

Type of issue
Missing information
Description
Other sub pages and the parent page have Japanese translation version. But this page doesn't have it.
Page URL
https://learn.microsoft.com/ja-jp/dotnet/azure/sdk/authentication/
Content source URL
https://github.com/dotnet/docs/blob/main/docs/azure/sdk/authentication/index.md
Document Version Independent Id
0a5ffb08-9da7-6113-89c3-1da0219c881e
Article author
@alexwolfmsft
Metadata

ID: a1aa6e9f-0341-2089-70e6-c371fceeec00
Service: dotnet-azure

Thanks for reporting this issue. I've routed it internally to our content localization team.

Closing, as the page is now available in Japanese.


One or more compilation failures occurred: error CS0041: Unexpected error writing debug information -- 'The version of Windows PDB writer is older than required: diasymreader.dll

We are getting this error randomly in various environments.  There does not seem to be a repeatable pattern for when this error will be thrown and a simple bouncing of the IIS AppPool/.NET Core application resolves the issue.  The source of the error seems to be "Source: Microsoft.AspNetCore.Mvc.Razor".  We are currently not precompiling our razor views in our MSBuild process.  Looking at https://developercommunity.visualstudio.com/content/problem/196579/views-causing-pdb-writer-errors-when-built-on-vsts.html indicates that I am not the only one that has seen this.
I am running on premise builds on these machines:

Windows 2012 R2
.NET Core SDK Version 2.1.101
.NET Core Runtime Versions 1.0.4, 1.1.1, 2.0.0, 2.0.5, 2.0.6
.NET Core Framework Host Version 2.0.6
.NET Framework Version 4.7.1

The environment web servers are:

Windows 2012
.NET Core Runtime Versions 2.0.0, 2.0.5
.NET Core Framework Host version 2.0.5
.NET Framework Version 4.7.1

At this point I am not sure if this is a build issue or an environment issue.  What do I need to do to get the correct version of this file in place?  Or is there a better place for me to report this issue?
Thanks,
Rick

@NTaylorMullen / @rynowak / @pranavkm - any ideas on this?

Might be the same issue as - https://github.com/aspnet/Razor/issues/1955. We couldn't actually figure out why it was happening sporadically, but the guidance was to reference https://www.nuget.org/packages/Microsoft.DiaSymReader.Native/ in the application. We've addressed the package reference in 2.1.0.

@reicher001 feel free to reopen if you continue seeing the issue after you've added a reference to the DiaSymReader package.

@pranavkm - should this issue be closed?


MODULE SIM TO SHARE INTERNET?

Hello, I have visited the page very often, obtaining very good results and always taking my knowledge.
Now something occurred to me and I still do not find anything concrete. My question is the following ... Is it possible to use any of the SIM modules to use the data that has the microSIM, and that way to connect to the internet, just as we do when sharing the internet with the cell phone?
I think it should be activating this function using AT command but before moving something, I prefer to make sure, besides I do not know if I have to add the WIFIesp8266 to transmit this data. Thank you and a big greeting, even if I do it before, I will not hesitate to share it with the community.

Yes, there is, it's in PR #5, but I've been remiss in reviewing it, sadly...


Occurrences Highlight not visible in Bordo

There is a default setting in VS Code called Occurrences Highlight that will highlight all occurrences of a word when the caret is touching it. This highlighting is not visible in Noctis Bordo, but it works with all other themes.

Thanks for pointing this issue. I should be fixed now. Let me know if it's OK for you.
Cheers!

Yes, it looks good, cheers!


Fix/9404 transparent status bar

Description
Fixing transparent status bar issue on two remaining screens
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-9404

I think this would be the optimal way to also fix https://github.com/corona-warn-app/cwa-app-ios/issues/3728#issuecomment-1050160505

I think this would be the optimal possibility to also fix #3728 (comment)

This fix is only confined to the devices with iOS 12.5


Do not prompt to install Python when opening an existing notebook without Kernel

For #4965
If you have a notebook with metadata that has a language of python, but no kernel information, we still prompt to install Python.
Not sure how users can create such notebooks, in our extension users can create such notebooks because we always default language to python for blank notebooks.
This ensures we don't display prompts if there is no kernel information.

Codecov Report

Merging #5202 (f91a1ed) into main (d99989e) will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main   #5202   +/-   ##
=====================================
  Coverage     73%     73%
=====================================
  Files        402     402
  Lines      26403   26403
  Branches    3799    3799
=====================================
+ Hits       19432   19436    +4
  Misses      5403    5403
+ Partials    1568    1564    -4

Impacted Files
Coverage Δ

src/client/datascience/openNotebookBanner.ts
80% <ø> (ø)

src/client/datascience/utils.ts
62% <0%> (-3%)
:arrow_down:

...lient/datascience/jupyter/kernels/cellExecution.ts
77% <0%> (-1%)
:arrow_down:

...datascience/notebookStorage/nativeEditorStorage.ts
80% <0%> (-1%)
:arrow_down:

src/client/datascience/notebook/helpers/helpers.ts
76% <0%> (-1%)
:arrow_down:

src/client/datascience/jupyter/jupyterNotebook.ts
75% <0%> (-1%)
:arrow_down:

...active-common/intellisense/intellisenseProvider.ts
74% <0%> (-1%)
:arrow_down:

src/client/datascience/jupyter/kernels/kernel.ts
85% <0%> (+<1%)
:arrow_up:

...tascience/jupyter/liveshare/hostJupyterNotebook.ts
45% <0%> (+1%)
:arrow_up:

...ent/datascience/jupyter/kernels/kernelExecution.ts
75% <0%> (+1%)
:arrow_up:

... and 2 more


Alternative return types in auth function phpdocs

Tweaked phpdoc to "allow" alternative return types in overridden functions (e.g. SPAs wanting JSON instead of redirects)

At this point we might as well just put "mixed" on it.

Ah yes, good ol' mixed! They have a common base-type, if you go all the way down to \Symfony\Component\HttpFoundation\Response, but I thought in most of these cases the Laravel versions would be more explicit. Thanks for reviewing.


export to cvs: exception: undefined method `encoding' for nil:NilClass, when table (or scope) is empty

Hi,
I'm experiencing an issue with rails_admin 0.8.1, trying to export an empty table, instead of just creating an empty csv file, it throws this error:
undefined methodencoding' for nil:NilClass
Extracted source (around line #38):
def to_csv(options = {})
  # encoding shenanigans first
  @encoding_from =<EMAIL_ADDRESS>? 'UTF-8' : @abstract_model.encoding)
  @encoding_to = Encoding.find(options[:encoding_to].presence || @encoding_from)

  csv_string = generate_csv_string(options)

Rails.root: -hidden-
Application Trace | Framework Trace | Full Trace
vendor/bundle/ruby/2.2.0/gems/rails_admin-0.8.1/lib/rails_admin/support/csv_converter.rb:38:in to_csv' vendor/bundle/ruby/2.2.0/gems/rails_admin-0.8.1/lib/rails_admin/config/actions/index.rb:74:inblock (4 levels) in class:Index'
vendor/bundle/ruby/2.2.0/gems/actionpack-4.2.6/lib/action_controller/metal/mime_responds.rb:217:in call'
How could I avoid this error?
Thank you,
Gabriele

+1 - I am also experiencing this error on 0.7.0

Any update on this?

+1

+1

A similar issue (https://github.com/sferik/rails_admin/issues/2796) was resolved with https://github.com/sferik/rails_admin/commit/464440cbdea7147634f020cde21c4606d3629dc5 (part of releases since 1.1.1) - Is this still happening on recent releases?

This should not happen in the current codebase, I guess.


Refactor setup_environment_map function

Update() seems to work correctly without calling it explicitly. It's get called each time parameter changes and so on.. Which is what you expected it to do.

Looks good, thanks!


Feature request: tap-tap-drag to zoom in and out like google maps

I'm writing this one handed from the 545. I'd love to use my other hand but if I let go of the handhold I'll fall over onto someone next time we hit a bump.
And there in lies the issue: I want to be able to check the schedule at my connecting stop but can't easily move to it without zooming, which I can't do without letting go of my handhold.

This will be supported automatically when we build against the iOS 13 sdk


Check popover against DL and VRT

Check popover

[ ] matches DL
[x] check VRT passes
[x] check and update migration guides example

Issue with whitespace between examples in VRT


Cannot install nnsvs

I've tried to install it with the docs command (pip install -e ".[dev,lint,test]") with
Ubuntu 22.04
and
Python3.10,3.9,3.8 and 3.7
always show:
ERROR: Could not build wheels for nnmnkwii, pysptk, pysinsy, pyworld, which is required to install pyproject.toml-based projects

should be solved by the latest release


Fix favorites maestro test

Task/Issue URL: https://app.asana.com/0/1202552961248957/1204549050315628/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3175   👈

This comment was auto-generated by Graphite.


Fix /status route in server.py to include original model location

Proposed changes:

Adding in a new latest_model object in the /status response object that is basically the model.get_latest_model() so we actually can get the original model location.

Right now it only shows the model_file with the tmp location which I'm not sure if we want to just replace that or not with this option above?  The only reason I can see maybe keeping it around is for troubleshooting to know where the actual model is that we are using and not that we trained from.
Thoughts?
Status (please check what you already did):

[X] made PR ready for code review
[ ] added some tests for the functionality - waiting until I know if I'm removing the old object response or not yet.
[X] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

@wochinge can you review this for me this is from https://github.com/RasaHQ/rasa/issues/4586 cc @msamogh

 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.Brian Hopkins seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@btotharye Can you sign the CLA agreement?

@btotharye Can you sign the CLA agreement? Also, just noticed that you performed commits as another user, so you would have to rebase your commits and redo the commit with the same email as your GitHub account.

Hrm I just added my rasa email now to my account which I thought was what it was all under

ok so the CLA issue is now fixed

@wochinge I updated this to just replace the existing model_file like we discussed, please review and let me know, thanks


dart analysis server client couldn't create connection to server

dart analysis server client couldn't create connection to server
not sure I have steps to reproduce, it was working well until yesterday
Dart SDK version: 3.5.4 (stable) (Wed Oct 16 16:18:51 2024 +0000) on "macos_arm64"

I have logs here

Some snippets from the logs:
[16:27:53] [Analyzer] [Error] on_weather/weather/weather_sheet.dart), (uri: dart:ui/weather/weather/weather/weather/weather/weather/weather/weather/weather, path: /Users/user/DevTools/flutter/bin/cache/pkg/sky_engine/lib/ui/weather/weather/weather/weather/weather/weather/weather/weather/weather), (uri: dart:ui/weather/weather/weather/weather/weather/weather/weather/weather/r…
[16:27:53] [Analyzer] [Error] er/weather/weather/weather/weather/weather/weather_sheet.dart), (uri: dart:ui/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather, path: /Users/user/DevTools/flutter/bin/cache/pkg/sky_engine/lib/ui/weather/weather/weather/weather/weather/weather/weather/weather/weather/rec…
[16:27:53] [Analyzer] [Error] er/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather_sheet.dart, path: /Users/user/DevTools/flutter/bin/cache/pkg/sky_engine/lib/ui/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather/weather_sheet…

and
[16:27:53] [Analyzer] [Error] vices/available_declarations.dart:813:5)
#2      _File._fileForRelativeUri (package:analyzer/src/services/available_declarations.dart:1888:20)
#3      _File.refresh (package:analyzer/src/services/available_declarations.dart:1344:19)
#4      DeclarationsTracker._getFileByUri (package:analyzer/src/services/available_declarations.dart:814:10)
#5      _File._fileForRelativeUri (package:analyzer/src/services/available_declarations.dart:1888:20)
#6      _File.refresh (package:analyzer/src/services/ava…

@scheglov I feel like we've seen this error come up before, but I'm failing to find any other issues. The stack includes Declarations which I think are gone now, although I'm not sure whether there are other code paths that could call _File.refresh in the same way that could still trigger this.

Ah, found another issue:
https://github.com/dart-lang/sdk/issues/51455

The path keeps growing with the "components" section being repeated:
uri:: dart::_internal/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/ui_settings_security.dart

Deleting the C:\Users\USERNAME\AppData\Local\.dartServer\.analysis-driver folder resolved the issue for that user. @scheglov is there anything else to try (or try to extract from that folder) before testing if deleting it also solves the problem here?

Ah, found another issue:
#51455

The path keeps growing with the "components" section being repeated:
uri:: dart::_internal/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/components/ui_settings_security.dart

Deleting the C:\Users\USERNAME\AppData\Local\.dartServer\.analysis-driver folder resolved the issue for that user. @scheglov is there anything else to try (or try to extract from that folder) before testing if deleting it also solves the problem here?

Sadly I already deleted it, and this solved the issue. Although thing to note is that I am on Mac so folder is located at root of my user folder.
Few more things, looks like this path was cached because even after I switched to branch which didn't have the folder mentioned in logs - server still crashed with same error, this continued to happen even after I opened another project.
Ofc root cause should be identified and fixed, but in situation where server keeps failing would it make sense to delete the cached data automatically and assume it is corrupted...

The whole available_declarations.dart file was removed.
So, this should not cause issues in the next release.


I have a question during code review.

Hello. I'm studying about parameter reduction and newbie in deep learning stuffs.
I have a question about data_loader.py
I've ran main.py with config.json that included originally, and it works well i think.
but during data loading, there is only jpeg images and loaded only images without any kinds of class information.
So my question is,
How it(code) calculates predicted accuracy without image's original class information?
I can not find any kinds of loading class information in data_loader.py
plz help me guyz :)

I just stumbled into this project but I may be able to answer
If you look closely at data_loader, it actually loads both the image and the original class information. Maybe you didn’t notice because it’s encoded as an integer.

self.y_train = np.array([284, 264, 682, 2])

This number should map to your class name list.


boost.fusion and variadic template

This is no longer an issue in boost 1.67 but I think it is still worth reporting.
In earlier version of boost (like 1.64), boost.fusion added a workaround for MSVC which will use the non-variadic-template version. However, some tests in boost.hana don't work when variadic template isn't used.
For example, 'test\ext\boost\fusion\deque\auto_specs.hpp' needs to define something like
#define FUSION_MAX_VECTOR_SIZE 50
Otherwise, tests under 'test\ext\boost\fusion\deque\auto' will fail.
BTW, I think there are other issues in the non-variadic-template version because it is now almost dead code, so I didn't dig deeper into it. I put some of my findings here: https://github.com/xiangfan-ms/hana/blob/master/include/boost/hana/config.hpp (search for BOOST_HANA_WORKAROUND_MSVC_BOOST_FUSION_DEQUE_SIZE).

Thanks for the report. Hana doesn't officially support older versions of Boost (doing so is too much work because it requires systematically testing with all the supported versions of Boost). Because this is not officially supported, I won't move on this issue unless you tell me it is a significant problem on your side.

Maybe you want to update this in the make file if only the latest version of boost is officially supported / tested?
find_package(Boost 1.64)


GeoJson plugin

#37

create GeoJson plugin use for loading GeoJson file from url, assets or path.
add OnLoadingGeoJsonListener for handle preload, loaded and loadFailed event of GeoJson loading.
create onMarkerClickListener for handle marker click event and return JSONObject of point properties.

Pinging @tobrun and/or @cammace to help us review this PR. Great job @bkhezry and thank you very much for the contribution!

Thank you @zugaldia.
Please review this PR @tobrun

It would make more sense building it once and allowing to update the underlying sources directly on the plugin.

@tobrun
I will be adding setUrl, setAssetsName and setFilePath functions to GeoJsonPlugin. after creating it with GeoJsonPluginBuilder we can update GeoJson data with them.

almost of review have been fixed.

Why is getting a random material color part of this plugin?

with multiple Polygon or Polyline in GeoJson file, selecting random color may be useful.

This 50+ line method has multiple responsibilities and can be re factored into smaller classes that can be unit tested.

manual parsing GeoJson string replaced with SDK of Mapbox.
please review this PR again and write the review if necessary.
@tobrun @zugaldia

Hey @bkhezry, sorry this plugin is taking some time to review, we're sprinting to finalize v5.1.0 of the SDK and this is taking all our resources (release candidate launched last week https://github.com/mapbox/mapbox-gl-native/issues/9231 🎉 ). Thank you for your patience!

Hi @zugaldia,
What happened to review of this PR? 😊

@bkhezry My bad - holidays got us derailed O:-) We'll be looking into it next week.
@tobrun Anything else to add to your review?

@bkhezry changes looking great! will work on putting the finishing touches on this so we can get this merged. Great job! Thank you for the contribution.

@bkhezry things are looking good for a first release, can you resolve the conflicting files?

Hi @tobrun
The conflicting files have been resolved :-)

@bkhezry our CI build is failing on checkstyle:

[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:41: Line is longer than 120 characters (found 167). [LineLength]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:42: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:44: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:46: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:48: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:50: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:52: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:53: 'member def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:55: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:57: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:58: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:59: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:60: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:62: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:63: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:64: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:65: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:67: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:69: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:70: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:79: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:81: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:83: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:85: 'if' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:86: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:87: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:89: 'if rcurly' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:90: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:92: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:94: 'if' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:95: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:96: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:97: 'if rcurly' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:98: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:100: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:102: 'if' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:103: 'if' has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:105: 'if' child has incorrect indentation level 16, expected level should be 8. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:106: 'if' child has incorrect indentation level 16, expected level should be 8. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:107: 'if rcurly' has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:109: 'else' child has incorrect indentation level 16, expected level should be 8. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:110: 'else' child has incorrect indentation level 16, expected level should be 8. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:111: 'else rcurly' has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:112: 'if rcurly' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:113: 'else' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:114: 'else' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:115: 'else rcurly' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:117: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:119: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:120: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:124: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:131: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:132: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:133: 'if' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:134: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:135: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:136: 'if rcurly' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:137: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:139: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:141: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:142: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:143: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:145: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:147: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:148: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:149: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:151: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:153: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:154: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:155: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:157: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:159: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:160: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:161: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:163: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:165: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:166: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:167: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:169: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:171: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:172: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:173: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:175: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:177: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:178: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:179: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:181: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:182: Line is longer than 120 characters (found 121). [LineLength]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:183: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:184: 'if' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:185: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:186: 'if' child has incorrect indentation level 12, expected level should be 6. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:187: 'if rcurly' has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:188: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:190: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:192: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:193: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:195: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:198: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:200: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:202: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:203: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:205: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:207: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:208: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:209: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:211: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:213: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:214: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:214: Line is longer than 120 characters (found 123). [LineLength]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:215: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:216: 'method def rcurly' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:218: 'method def modifier' has incorrect indentation level 4, expected level should be 2. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:220: 'method def' child has incorrect indentation level 8, expected level should be 4. [Indentation]
[ant:checkstyle] [ERROR] /bitrise/src/plugins/app/src/main/java/com/mapbox/mapboxsdk/plugins/testapp/activity/GeoJsonActivity.java:221: 'method def rcurly'

Here is more information how to setup checkstyle cfr Mapbox checkstyle.

@tobrun checkstyle errors fixed.

CI isn't starting, going to create a new PR in https://github.com/mapbox/mapbox-plugins-android/pull/70, additionally will cleanup git history.
Thanks again for the contribution @bkhezry!


Lists the raw results of probes

Fixes #96
This PR adds a new view at /results which lists the probe results as seen below:

This is a good start I think. Thanks @dheerajodha. What I think could improve the outcome:

having the total number of plugins known in the result column (like 1800 / 1881 for the last-commit-date)
I wonder if there is a Collectors implementation which would simplify the if in the controller
I don't think having the transactional (which is good) method in the controller if ok. It might be better in the PluginService class.

I am just looking in more detail at some of the stats that @dheerajodha is pulling, and they arent making any sense? Specifically, deprecation, up for adoption, and size(?).

Sorry for this mess Jake, it's my fault :smile:
To answer your question,

"deprecation", "up-for-adoption", and "security" stats are not making sense because I blindly collected raw data for each probe based on the number of plugins which has a ResultStatus == "SUCCESS".

This is not right because for a probe, say, "deprecation", we want to see the stats for plugins that are deprecated. But in the code of this probe, we've given ResultStatus = "SUCCESS" for those plugins which are NOT deprecated. So, my code was capturing (and displaying) the opposite of the expected data.
Adrien suggested I create a custom mapping between a "probe" and "status" (for which we want to capture the data), so I just committed a fix for this based on his suggestions.
My code still needs to be reviewed, but you can see this mapping here. Notice that we've now passed "FAILURE" status to capture the expected data for these 3 probes.

regarding the "size": it was my dumb hack to display the total count of plugins on the UI (as you can see "/ 1881" in each row of the "Result" column).

I've fixed them now (I think), and the UI looks like this:

Also, we may want to get creative with the last commit stat, as the raw count doesn't really do us any good. Maybe show an average?

those are row data. Getting the last commit date is helpful while scoring but just its value has no meaning. I don't think we need to show that here.


Highlight Keyword "FIELDS" in grammar while creating Index, but "COLUMNS" is already highlighted

Describe the bug
Highlight Keyword "FIELDS" in grammar while creating Index, but "COLUMNS" is already highlighted
To Reproduce

Expected behavior
Keyword "FIELDS" must be highlighted
Environment (you can view the current version under Settings):

Operating System [e.g. Windows]: 11
Surrealist Version [e.g. 1.2.0]: 1.10
SurrealDB Version [execute "surreal version"]: 1.0.0

Fixed in the latest grammar


Elements are overridden after reload with vuex-persist

If you use vuex-orm with vuex-persist you onccur the problem, that, after a reload, vuex-orm starts to count at 1 again. So your persisted elements will be overridden.
Here you have an simple repo to check this out.
https://github.com/Maqsyo/vuex-orm-uid-bug

Ah... yea. I think this would happen. Hmmm, maybe we should store uid increment numbers to Vuex State as well, or either implement a real UID feature.

I think storing it in the state could be a really good quick fix. Thats how MySQL is doing it.

I tried two implementations yesterday, store method as @Maqsyo suggested and the UID refactoring as @kiaking suggested.
I found there are side-effects to both approaches but the store is more feasible especially since no additional logic is required when using third party persist tools.
UID’s are especially important when generating indexes so a lot of logic is required when refactoring. Where as we can simply use the mutate action to update the UID count to the store when a store is available.

@cuebit Thank you for trying out! Yea, I think storing inside Vuex State would work. There're lot of UID implementation out there, and if you really need UUID in client side, the work needed for the app would vary from user to user I guess. So keeping Vuex ORM default UID as simple as possible would be the way to go, I think 👍

My current quick-fix:
I'm using the current timestamp and concatenate it with an increment-value to reduce collisions.
import {Model} from '@vuex-orm/core';

let increment = 0;

export default class Post extends Model
{
    static entity = 'posts'

    static fields()
    {
        return {
            id: this.uid(() => `${new Date().getTime()}_${increment++}`),
            [...]
        };
    }
}

Ah yea, that’s smart 👍


Include system agent unit file in specfile

Goes together with https://build.opensuse.org/request/show/1112303


[Bug] Bring Back floatingCTA on Desktop

Now on stage the floatingCTA is not showing up when we scroll down on desktop homepage. This should bring it back.
There're also some little refactoring for readability.
Resolves: https://jira.corp.adobe.com/browse/MWPW-140911
Test URLs:

Before: https://stage--express--adobecom.hlx.page/express/
After: https://final-mobile-beta--express--adobecom.hlx.page/express/?martech=off

Codecov Report
Attention: 46 lines in your changes are missing coverage. Please review.

Comparison is base (78eeb34) 31.87% compared to head (e6c2590) 31.98%.

Files
Patch %
Lines

express/scripts/utils.js
0.00%
46 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #555      +/-   ##
==========================================
+ Coverage   31.87%   31.98%   +0.11%
==========================================
  Files         126      126
  Lines       24131    24102      -29
==========================================
+ Hits         7691     7709      +18
+ Misses      16440    16393      -47

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


UniqueID changes between tags fired on Container Loaded vs DataLayer Loaded

Hi Team,
I am using this template to generate a UniqueID to deduplicate Facebook Pixel vs Google Tag Manager Server Side Conversion API for each ecommerce message Facebook requires to track users on our website.  What I noticed in both Facebook Event Manager and Tag Assistant was the deduplication parameter i.e. the eventID generated by this template was different between Facebook Pixel and the GA4 message sent to the GTM server.
Tag Assistant showed the eventID between the FB Pageview and the GA4 Pageview had similar values however where not exactly the same generally the GA4 Pageview showed a longer string of numbers.  The other pertinent piece of information is that the FB Pixel is fired when the Container is loaded and the GA4 tag is fired when the dataLayer is loaded.
On reviewing your code and confirmed with testing is the following code adds the gtm:uniqueEventId to the UniqueID being generated.  In doing so the string produced by the template cannot be reliably used for deduplication of Facebook events.
return getBrowserId() + '_' + getPageLoadId() + getGtmUniqueEventId();
For the time being I have removed the "+ getGtmUniqueEventId()" for the template and the string produced is now exactly the same, as proven by testing.
Cheers,
Jim

Hi,
This variable generates a unique event id.
If you use trigger group or tag sequencing, this will generate different IDs for each sequence.
If you remove getGtmUniqueEventId, then this variable will become unique per page, not per event.
More info https://community.stape.io/t/the-curious-case-of-the-view-item-event-on-server-side-gtm/600/3


ModuleNotFoundError: No module named 'torch._six - Google Colab

"Traceback (most recent call last):
File "/content/ShapeConv/tools/test.py", line 7, in
from rgbd_seg.runners import TestRunner
File "/content/ShapeConv/tools/../rgbd_seg/runners/init.py", line 1, in
from .inference_runner import InferenceRunner
File "/content/ShapeConv/tools/../rgbd_seg/runners/inference_runner.py", line 3, in
from ..models import build_model
File "/content/ShapeConv/tools/../rgbd_seg/models/init.py", line 1, in
from .builder import build_model
File "/content/ShapeConv/tools/../rgbd_seg/models/builder.py", line 5, in
from rgbd_seg.models.decoders import build_brick
File "/content/ShapeConv/tools/../rgbd_seg/models/decoders/init.py", line 1, in
from .bricks import FusionBlock, JunctionBlock
File "/content/ShapeConv/tools/../rgbd_seg/models/decoders/bricks.py", line 8, in
from ..utils import ConvModule, build_module
File "/content/ShapeConv/tools/../rgbd_seg/models/utils/init.py", line 2, in
from .conv_module import ConvModule, ConvModules
File "/content/ShapeConv/tools/../rgbd_seg/models/utils/conv_module.py", line 10, in
from rgbd_seg.models.utils.shape_conv import ShapeConv2d
File "/content/ShapeConv/tools/../rgbd_seg/models/utils/shape_conv.py", line 9, in
from torch._six import container_abcs
ModuleNotFoundError: No module named 'torch._six'"

You can reduce the pytorch version to below 1.10

Thanks for reply.
Unfortunately on Google colab the earliest version we can download is 1.11.0


[Feature][Alert] enable users to customize email content

Search before asking

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

Description

when start  a process Instance。we can only send the default mail format.
we can use some placeholders to allow users to customize the email format. This feature does not involve the use of rich text
I can only complete the backend part for now

I want to make the red frame on the picture

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Good idea, we can provide some placeholder e.g. {projectName}, {WorkflowInstanceName}, {TaskInstanceName}...

Good idea, we can provide some placeholder e.g. {projectName}, {WorkflowInstanceName}, {TaskInstanceName}...

This is what we did in our secondary development。
However, I believe that each module does not have the same variable access, so we should provide the variables that the module can provide in each place where the content of the email is defined
To prevent spelling errors, you should do a select variable and then add operation in the front end


boards: arm64: phycore_am62x_a53: Update documentation

The latest Yocto release, PD23.1.0, has the CACHE_CMD included. Therefore, the default image can be used to boot Zephyr.

Why are you assuming that yocto is always used for this? Can you keep both sections?

The previous section was only added because this CONFIG/command was missing in the pre-built binary. We added it to our latest release. I don't see any reason to built u-boot manually.
Moreover, the previous documentation recommended to put the u-boot in the boot partition but not how. However, I could include the old section again and reference the pre-built u-boot only.

@gramsay0 FYI

@dnltz Thanks for updating this!
I gave it a quick test and I could boot Zephyr.
Although Linux would not boot, it seemed to hang after this:
[    1.583508] mmcblk0: mmc0:0001 S0J56X 14.8 GiB
[    1.590997] mmcblk0boot0: mmc0:0001 S0J56X partition 1 31.5 MiB
[    1.597131] mmcblk0boot1: mmc0:0001 S0J56X partition 2 31.5 MiB
[    1.603249] mmcblk0rpmb: mmc0:0001 S0J56X partition 3 4.00 MiB, chardev (237:0)
[    2.952766] sdhci-am654 fa20000.mmc: Power on failed
[    2.988404] mmc2: SDHCI controller on fa20000.mmc [fa20000.mmc] using ADMA 64-bit
[    2.998182] Waiting for root device /dev/mmcblk1p2...


Incorrect variable name in the error message pointing out old(..) for &mut argument preconditions

Very minor issue:
mod pervasive;
use builtin::*;
use builtin_macros::*;

verus! {

struct A { }

impl A {
    spec fn f(self) -> bool;
}

fn g(a: &mut A)
    requires a.f()
{ }

pub fn main() {}
}

error: in requires, use `old(a~2)` to refer to the pre-state of an &mut variable
  --> /st/verus/test.rs:14:14
   |
14 |     requires a.f()
   |              ^

The error message shouldn't use the name a~2 to refer to variable a.

fixed in 175b731a95d5544ef51dd34df0e082bfb31413b5


[WIP] Unified Data Sharing PD

Sorry if I'm being dense, but it seems to me that everything except

As a KBase developer, I want to make reference data publicly readable by KBase users, but distinct from public data shared by regular KBase users.

... is already implemented in the Narrative. Am I missing something?

I think @MrCreosote has a point in terms of PD. I think I know what you are trying to fix Matt but it is not clear from your PD what is not working and what demands fixing now.

Here is the directive from Adam's document on high priority items that this
was targeted for.  I left out implementation details based on the
understanding that those should not appear in a product description.
"Fix sharing. Sharing is a key concept in KBase and something we talk a lot
about as a differentiator. However, it is still cumbersome and difficult to
understand. There are several related tasks here that would greatly improve
our sharing experience. Extend user profile to support very simple groups.
Allow narratives to be shared by email address. Filter shared narratives by
user(s). Fix data sharing so that it does what people think it does."
On Tue, Mar 15, 2016 at 12:00 PM, MrCreosote<EMAIL_ADDRESS>wrote:

Sorry if I'm being dense, but it seems to me that everything except
As a KBase developer, I want to make reference data publicly readable by
KBase users, but distinct from public data shared by regular KBase users.
... is already implemented in the Narrative. Am I missing something?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/kbase/roadmap/pull/23#issuecomment-196973808

Right-- but there are a series of accessor methods, visualization, data browser proposals, group definition products, etc. that could impact all this. So there is a question of what the dependencies and scope is. Does this require a whole new design; complete rewiring of sharing; or a set of methods that allows us to dice and slice our shared data and display data selectors based on this. Would a lot be helped by really nice ways to find and group collaborators so you could more easi;y share to the group, etc.
I don't really know...just asking.

I guess the way I was interpreting these PDs based on the constraints (no implementation proposed) that really these are problems that should be tackled or new features added, and they are either selected as important and scheduled, either immediately or at some point in the future, or deselected as not important.
How the task realistically gets accomplished will have to be determined on the ground by the implementers, at least based on my understanding of how this should work.
For this particular PD, in the first sprint stakeholders (ideally users) would provide feedback to determine what the scope of the work should be.  I did not specifically mention groups for example, although that may in fact be one of the high priority items to deal with as indicated by the stakeholders.
If this PD is considered too vague, I might need some guidance on what is actually expected that would not include implementation details.  I'm happy to make revisions as needed.

I am not criticizing the level of detail particularly. I am saying that it overlaps in part with other products being proposed. Yes- these products to be tackling problems and adding features. And we are RIGHT NOW going through the prioritization and selection or rescheduling.
For this particular case we might want to think about the following:

We have a current sharing model. It is working kinda. There are some specific defects we need to address. What are they?
We have a proposal in another PD to institute a group management piece. How would this impact any model of sharing. This PD is in response both to internal thought and user feedback. So the question is does that PD owner think we have appropriate feedback to make this part of things. If so this might impact how you drive a proposed user study on sharing. It may be you want the entire PD to be deliver of a sharing model based on feedback. But the idea is you would design that in part based on knowledge of current limitations of sharing and in some of the related ambitions (such as groups).

Part of my point here is that there are some complexities that users will not appreciate until they really start working with the system. I'll consider a specific use case here.
I am running a large project called X with ten collaborating labs each of which has between 1 and 5 personnel in addition to the PI. We have point collaborations outside the project with people who help us, but those investigators have different privacy issues than within project. For example, all investigators within the project have agreed to share unpublished data freely. But our outside collaborator has not made such an agreement so data coming from their lab might only be shared with me (or another project X investigator).
My lab is involved in high throughput RNA-SEQ of plants and I am planning on creating a collection of data sets of the type (Condition X, Individual plant N, (Tissue M, RNA-Data)) implying I am measuring expression response of a species of plant over X conditions. In each condition, I am measuring gene expression in M different tissues according to some attached protocol. My collection is the set of data sets over X.
I want to be able to upload into KBase the reads in bulk from these experiments and share them with the project, and with these outside investigators. I might want to share individual subsets of these experiments (the datasets) with outside people. I'd like to make sure that permissions are not inherited by derived data products but I do want to make it easy to transfer those permissions as such. Also-- the individual datasets should likely have the same permissions as the collection, right?
Now let's say one of my outside collaborators is also doing RNA-SEQ of a similar style on a related plant. She has given me permission to perform a comparative analysis that might ultimately be shared with Project X but not yet. She shares her data with me. I create a collection containing a selected subset of datasets from my collection and from hers. Now I share this collection perhaps inadvertently with Project X. What should have happened? Well-- it could be that all the datasets in the new collection inherit the permissions from the new collection container; it could be they maintain their permissions from where I got 'em. If it is the latter then I shouldn't be able to share my new collection with Project X without a warning and a specific choice being made because datasets in the collection are really private to me and this outside investigator. (We might just say-- so what? That is the responsibility of me not to share data I promised to keep private-- but you can see how this could become confusing to a user... so giving a warning at least might be useful.) If it is the former then we have to keep track of a complex maze of permissions on references to data sets that seems hard to manage and confusing to a user.
There are other interesting issues here as well. What if I share DERIVED data from this set with the group. Again-- we could argue this is my responsibility. And it is. But sharing a Narrative might share ALL data equally (which it probably should) or if I make a new NArrative with just the derived data and share that-- then if Project X members reach back to figure out where the data set came from with provenance-- should they be able to know that while they may not have access to the input new collection that this collection was derived from Project X data (which they DO have access to) and a private collection?
In any case, my point is that the above would be hard for users who can barely use the interface as is to imagine yet and it is in part incumbent upon us to derive a reasonable model. Further, the above implies a series of operations on collections (upload, download, controlled subsetting, etc) that need to be operational and linked to any sharing system we derive. During sharing, we have to think about permission inheritance if there is any when data in brought together and the implications of that.
So it seems like among understanding well what we have, what we know we want to do, and what users experience to date has been we should be able to set up an example of what extensions and fixes to our sharing system should enable.

Currently, if Narratives and data (input or output of methods) share the same container (workspace), they have the same permissions, and there is no distinction for derived products.  There is no distinction between items in the container in terms of permissions.  This may or may not fit user expectations.
It sounds like you should be a stakeholder!
What you are describing are rules/policies that users could assign.  For instance, "alert me if my Narrative or data is shared with others".  My question though would be whether this generally meets user expectations, or if they expect stricter policy enforcement.  I think you are right that users have probably not encountered a complex sharing scenario yet in KBase.  Simulating a lab of researchers as a scenario for evaluation and testing would be a good way to identify specific problems, what the desired behavior should be, and then possible solutions.  I would be concerned about making sure we had a minimally viable product to deliver at the end of the quarter, so scoping appropriately is crucial.

I originally thought the purpose of this item was much more "simple" and it was just about having one service the knew how to talk to other KBase services to correctly share a workspace and all its related pieces (shock, awe, jobs, etc). So this would be able to replace NJP and handle manager and possibly some other things. There are lots of edge cases right now where we struggle to propagate the sharing correctly. This thing would be the uber sharer.

What you suggest Shane would be a great step in the right direction

So, what is the difference between adding a new sharing broker, and adding more metadata to the workspace object, which is the granularity of sharing in the system, to manipulate associated identifiers?


[Api-draft] add README.zh.md for new api

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

I will submit english version later.


How do you differentiate between an empty multi, and a multi which was blocked due to a watch?

Jedis 2.7.2
Server 2.8.19
How do you differentiate between an empty multi, and a multi which was blocked due to a watch?  The results from the transaction.exec() method are both null.  Is there something that can be surfaced to the caller to differentiate the two conditions?
When using the redis-cli the two use cases appear to have a different response.
<IP_ADDRESS>:6379> watch foo
OK
<IP_ADDRESS>:6379> multi
OK
<IP_ADDRESS>:6379> set foo transaction1
QUEUED

connection 2
<IP_ADDRESS>:6379> set foo nottransaction
OK

exec and check result on connection 1
<IP_ADDRESS>:6379> exec
(nil)
<IP_ADDRESS>:6379> get foo
"nottransaction"

However, when we execute an empty transaction, the response is slightly different:
<IP_ADDRESS>:6379> watch foo
OK
<IP_ADDRESS>:6379> multi
OK
<IP_ADDRESS>:6379> exec
(empty list or set)

Hopefully something obvious I'm missing.
Thanks!

:+1: wondering in order to implement optimistic locking :-)


Repository permissions for Netlify

Adding the repository to Netlify requires granting some permissions, described here. My attempts failed likely due to insufficient permissions to this repository - I don't have access to settings. If somebody can grant permissions same as to the BioC2022, I'll continue trying.

permissions should be allowed now.  Please try again
Lori Shepherd - Kern
Bioconductor Core Team
Roswell Park Comprehensive Cancer Center
Department of Biostatistics & Bioinformatics
Elm & Carlton Streets
Buffalo, New York 14263

From: Mikhail Dozmorov @.>
Sent: Thursday, December 8, 2022 9:15 PM
To: Bioconductor/BioC2023 @.>
Cc: Subscribed @.***>
Subject: [Bioconductor/BioC2023] Repository permissions for Netlify (Issue #1)
Adding the repository to Netlify requires granting some permissions, described herehttps://secure-web.cisco.com/1Wer2LKcDXMU-wWEpEV2c4kN5AuDPWelLyHiO30tLY2Na5b2aZbJG0ApsYQHUkJsgK2fi0_kVfm2TT9oXZaIGD9eF4oDmajpbGfqqYTApfH5p-hueR1pe6vEENR7l-VKacCDwcFhewoczaSrf5Lem4H_rmiVaeQpzgNRpq2wGn5KwKSyd1-C9WlLtmEGlRm4KNdSU9jaKmvIRqgiXJKT1T9AKNpqnbnRt3K583iHv-YZBRfXNSQ-iSn6-82LmSJQu-i2ekDEhwk1SypBGLwRo2meAdrV_TygSusgrhe4BbsSxuWbuzjwFTdUoVzDh5yV9/https%3A%2F%2Fdocs.netlify.com%2Fconfigure-builds%2Frepo-permissions-linking%2F. My attempts failed likely due to insufficient permissions to this repository - I don't have access to settings. If somebody can grant permissions same as to the BioC2022https://secure-web.cisco.com/1iLXsBNK5fsVEDOz_MB6yGJGsyqm1XVbn6j70XACsbcd_kz-PuyIRxTU3hlEpuiy0_weg8yKp-NPxIHk2Ox0rIqRfT2JYsKK2k8yCsxMoNGtPk9KPciOjcU2h36G3MrtLBrrJrc0pazltbEr7YFBsK5rknfVsBYKtXvYuQvZXALw7HCA1hTPBfBoXCQkafgUKiHp8ZbqGfiQDADo_9oF22ZOH8ChMhKS5OB2ck1mu-xdSAK_KHUIdl-ewu8uoWdR67oRGF9PbUd1-8kB5sdcXRCelXd0qjXvlxv0Vz1WedVwA1veDNcZxe3tmiH_SPLJA/https%3A%2F%2Fgithub.com%2FBioconductor%2FBioC2022, I'll continue trying.
—
Reply to this email directly, view it on GitHubhttps://secure-web.cisco.com/1ejD75GvWT3L-jQgszNDXINwZgUP2w6zurARbCop4P9JQ6ouf6Yeu2AbIOSWf0_xF7zwW8kU7glkn5sGeU8FLBkWMDihODjGMIHqvr6X7Uz3sjeJa_y1Fy5AOCJujhM-putHQK04VhjKwOxC4qa2mk9HwT3Vz4oKAIwnvl5aXmvrd_5JwGK5iUp33NAXwqxjGWzf5_jNEM-ybeg82EJS-JXAX1KN-NzoFkVyKMK8lGVS-TGW5ODhnafCoTjRI_hyNjjkeiB8LdBkecyXAJXjtlWVrmwNGRH1WjiwAKVWK2hkvs2g4imfJfswHzZgX3Xzm/https%3A%2F%2Fgithub.com%2FBioconductor%2FBioC2023%2Fissues%2F1, or unsubscribehttps://secure-web.cisco.com/12y1eQgnQYWaVgCyXISU4CDhQvi-9V6TJiwoRSJJOAt1C3NnhwsdmvTWjxMo-9wP1gI0aEwqJJD9QKRzlOmCfsHHYFP-BKG5ySKJ_7vnJm12WPQ9egiInV-ntWYaNv6EMteuUWgsIgejHB4ETHmQjv_NRhwEwvwMo6ZCRGjXQ-3mztP4obBdedZNGNGHg0ogKwgnqAp__pf1d9Wh_a8blmXI4REFNaPkQX5jynHHszAtWIIBHYT36nwRVrBEYnH1Em881wGLZd-uIZVxI-7tZ7CxiGxElgpB3AgBIhZjtMQ3uash6nFXb0k4lpTqCBwMQ/https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEO3MHD3HCAFTOJVXXRBCNDWMKI2LANCNFSM6AAAAAASY2BT6E.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
This email message may contain legally privileged and/or confidential information.  If you are not the intended recipient(s), or the employee or agent responsible for the delivery of this message to the intended recipient(s), you are hereby notified that any disclosure, copying, distribution, or use of this email message is prohibited.  If you have received this message in error, please notify the sender immediately by e-mail and delete this email message from your computer. Thank you.

Thanks, Lori, it works, https://bioc2023-test.netlify.app/. Now, we need the official bioc2023.bioconductor.org, but that's a separate issue.


software bug:close the node network, while create a cluster, after more then 20 minutes, the job pod is still running.

Describe the version
version about:

kubean v0.5.2

Describe the bug
create a cluster close the node network, after more then 20 minutes, the job pod is still running.
How To Reproduce

start a ops to create a one node cluster
close the node network: service network stop
check the node is not reachalbe
wait the ops related pod to be error
but i waited more than 20m, the pod is still running

Expected behavior
when the cluster node is not reachable for more than such as 20m, the ops related pod should can be stopped
Screenshots and log
when close the network, the log of the job:

after close the network ping the node:

the ops related pod is still running:

and the logs of the job is still stopped at :

The spec.activeDeadlineSeconds parameter can be set in the clusterOperation


Shared Dockerfile.dapper

In light of Shipyard supplying shared infrastructure, it should be easy
to shared Dockerfile.dapper as it's mostly a copy pasted file.
In the case it already exists it shoudn't be re-downloaded by consuming
projects, and they can remove it gradually.
Signed-off-by: Mike Kolesnik<EMAIL_ADDRESS>

Yes, let’s get rid of copy-pasta!