LPGMA:Is there fine tuning code?

I read the LPGMA paper.
Thanks for the good research.
However, I immediately tried to use the code in the paper to actually use it, but I cannot find the fine tuning code.
In the paper, it was written as follows: Is the fine tuning code not published?
If not, could you please publish it or tell me how you fine tunig it?
Thank you.
"We also report the result of the model that is then fine-tuned on the training
set of ICDAR2013, labeled by †. The results of DeepDeSRT on SciTSR come
from [2]. The results are demonstrated in Table 1, from which we can see that the
proposed LGP-TabNet vastly surpasses previous advances on these three benchmarks by 4.4%, 3.5%, 2.5%, respectively. "

We have only published  the model trained on PubTabNet, you may use the same code the train the model on ICDAR2013 or other datasets.

Thank you.
So I have one more question, could you please tell me how you set up the config for that fine tuning?
Especially about lr, momentum and weight_decay for optimizer.
The epoch was mentioned as 25 in the paper, but there was no mention of anything else.
I am currently doing fine tuning with my own data, but I am having trouble just keeping the settings in lpgma_base.py and adding the publicly available maskrcnn-lgpma-pub-e12-pub.pth to load_from.
The accuracy of the detection has dropped drastically from what it was before fine tuning.

Some of the parameters you mentioned are actually un likely to have a big impact on the performance of the model. It is usually caused by a possible problem with the data label or target generation. In addition, if you evaluate the detection indicators, the lgpma training target is the enlarged aligned boxes, which cannot be evalated with the original text bounding boxes. It is better to directly see the visualization result to judge whether the model is convergence, or in accordance with the evaluation of the table structure.

Also, for fine tuning, I set "load_from=" in lgpma_base.py to the publicly available maskrcnn-lgpma-pub-e12-pub.pth, is that correct?
Not setting it to "resume_from".

The quatity of IC13 images is very small, so the model should be converged (or even over-fitted) easily. You may train the dataset for a larger epoch to see whether it can be overfitted on the training set.

Also, for fine tuning, I set "load_from=" in lgpma_base.py to the publicly available maskrcnn-lgpma-pub-e12-pub.pth, is that correct? Not setting it to "resume_from".

yes

Thanks for answering my question.
I just don't see any convergence when I set the EPOCH to 100. ......
Is the quality of the data bad ......
I'll try a few more things.


raja-perf ramble implementation

@pearce8 The benchmark works on all three configs/machines and can be merged into main


Debugging problems

Even with a very basic source, we can clearly see problems with debugging (mapping lines in Scala sources):

during the function tracing, the cursor is switching to the beginning of the function
Step Over often moves the cursor into the called function

This behavior makes tracing of complicated functions very difficult.
Repro:
In the following source, set the breakpoint to the first line of function DebugSample,main, launch it and try to use Step Over to trace the function. See where the cursor will appear.
import scala.scalajs.js.JSApp
import scala.util.Random

object DebugSample extends JSApp {
  def main(): Unit = {
    val angle = Random.nextDouble() * 90.0
    val rad = angle.toRadians
    println(s"Random angle $rad radians")

    val angle2 = Random.nextDouble() * 45.0
    val rad2 = angle2.toRadians
    println(s"Random angle $rad2 radians")
  }
}

When using step-by-step execution of Scala.js, it is recommended to disable the optimizer with:
scalaJSOptimizerOptions ~= { _.withDisableOptimizer(true) }

I have tried this option. While it prevents stepping into the functions, still there is the issue of the current execution point jumping between real current location and the beginning of the main function on almost each step.

The screenshots are with IntelliJ IDE, when debugging with the Chrome Dev tools the results are even worse, it is not possible even to place breakpoints on most lines and stepping through is very erratic.
Are there any tools for inspecting or visualizing or otherwise debugging source maps? The behaviour seems surprisingly annoying for such small code sample.


[SPARK-50204][SQL] Factor out HiveTableRelation read path resolution

What changes were proposed in this pull request?
Factor out HiveTableRelation resolution into a separate RelationConversions method to reuse it in the single-pass Analyzer.
Why are the changes needed?
Context: https://issues.apache.org/jira/browse/SPARK-49834
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
copilot.vim.

+1, LGTM. Merging to master.
Thank you, @vladimirg-db.


💡 [REQUEST] - Automatically generate YAML schema

Implementation PR
No response
Reference Issues
No response
Summary
We have created a YAML schema for TTPForge that lives in docs (https://github.com/facebookincubator/ForgeArmory/pull/77). The first iteration was created manually, which adds overhead to maintaining it.
To combat this, we should automatically generate the schema from the TTPForge source code.
Basic Example
See schema in https://github.com/facebookincubator/ForgeArmory/pull/77
Drawbacks

Will take some work

Unresolved questions
No response

https://github.com/facebookincubator/TTPForge/pull/482


Quick open project

Description
Big update for Quick Open Project extension by @zerowidth

Added IDE detection for each project
Added support for some of JetBrains IDEs (Webstorm and PyCharm for now)
Added support for Git Fork
Added recursive search
Added ignored paths

Type of change

Bug fix / improvement for somebody else's extension

Screencast
https://user-images.githubusercontent.com/21295738/138677472-6f5c7a1f-ff8d-4bff-ba19-51963a58cc8d.mov

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I checked that extension's icon has size of 512x512

Hey @Loskir, just wanted to double check if you plan to address suggestions by @zerowidth? Let me know if you have any questions.

@zerowidth thanks for review!

It feels too presumptive
I agree with that. I turned out to be very hard to make a script that accurately predicts appropriate IDE for a project. I'm thinking about detecting all installed IDEs and maybe using a dedicated frecency instance for choosing an IDE for every project. This would solve presumptions for VSCode and JetBrains IDEs and also allow users to more precisely adjust tools for each project.
If this looks like a good idea, the easiest way to do this would be to disable project type detection entirely and just let users train their frecencies for their needs. After that we may add project type detection back but as a recommendation system, not as a filter for IDEs. Though it might require some complex algorithms and I'm not really qualified in that field.

do you usually have project{x, y z} dirs alongside a subprojects/{a,b,c} container?
Yes, I have ~/Projects directory which contains both ~/Projects/some-small-project and ~/Projects/some-big-project/repo-{a,b,c}. I made the extension detect ~/Projects/some-small-project and ~/Projects/some-big-project/repo-{a,b,c}, but not the ~/Projects/some-big-project/ itself.
But it might actually have been an overkill. As there are not so many some-big-projects, I could just specify ~/Projects, ~/Projects/big-project-{a,b,c} as scanning path and add ~/Projects/big-project-{a,b,c} to ignored dirs.
I don't really know how to make it work the best so I'd like to hear some suggestions.

are there other apps that should be added here?
Yeah, It would be nice to add more IDEs, terminals, git tools, maybe something else.

Would it be possible to split this up into smaller PRs?
I'll try to isolate smaller changes into separate PRs

Unfortunately, I don't have much spare time to work on this project, but i'll try my best. Anyway, all help and suggestions are welcome!

@Loskir @zerowidth What's up with this PR, since this one is still awaiting something.
@Loskir Can you please resolve the conflicts so we could simply merge one?

We can close this for now, but will consider these requests/patches for future updates.


Remove Configure section from Application Insights Performance Counters page

The Configure section is obsolete because the Application Insights Status Monitor is not required to collect performance counters.
CC: @iusafaro

@olegsych,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@olegsych : Thanks for your contribution to the Azure documentation! The author, @CFreemanwa, has been notified to review your proposed change.

#sign-off


update context in i18n/ja.toml

update context in i18n/ja.toml
see #20009

/language ja

/cc @kubernetes/sig-docs-ja-reviews

/lgtm
@sftim can you approve this? I don’t think I have the permission here

@inductor I recommend you / other people add a “/approve” anyway, for the record, and then it's obvious why someone else is echoing it to keep Prow happy.
/approve


Editor hint in new file can't be reached by tabbing

Version: 1.60.0-insider
Commit: e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff
Date: 2021-09-01T10:39:32.559Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 18.7.0
Steps to Reproduce:
Open VS Code and create a new untitled file. Tab over to try to click on the "Select a language" action and the hint disappears. Can we make this action keyboard focus-able?
cc @isidorn

Originally I did not make those links clickable because all those actins can be accessed via the command palette, however I do agree that they should be clickable.
Ideally the first tab would focus onto the Select a language and the next on the don't show again.
Or even better the first tab would focus on the whole thing - which would read the whole message, and left right or subsequent tabs would navigate to the links.
@TylerLeonhardt what do you think? Also assigning to you since you now own the untitled hint.
Assigning to September as I think this is minor and we can easily fix.

Hey , since it is a good first issue, can I work on this ?  , if no, will delete comment, sorry for any inconvenience caused @isidorn

@san-coding feel free to provide a PR which tackles this issue. Thank you

I think this makes sense!

Hello, I tried working on this issue, and as you can see, I can now tab to focus the two links.
But when I activate the link by keyboard (by pressing space or enter), it will write the pressed key.
I implemented the link activation with a listener on the HTML node:
const languageOnKeyPress = async (e: KeyboardEvent) => {
	if (e.code === 'Space' || e.code === 'Enter') {
		e.stopPropagation();
		// Need to focus editor before so current editor becomes active and the command is properly executed
		this.editor.focus();
		await this.commandService.executeCommand(ChangeModeAction.ID, { from: 'hint' });
	} else {
		this.editor.removeContentWidget(this);
	}
	this.editor.focus();
};

How can I prevent the editor to write when I activate the link?
And I don't really know how to test this feature. Should I instantiate a UntitledTextEditorHintContentWidget and trigger the events? If so, how?
Thank you

I've been working on this issue for a bit now. Will send a PR soon!

fyi @ulugbekna

@isidorn I tried to reproduce this issue, I found that even when workbench.editor.empty.hint is set to text in the below-mentioned VS Code Version. The hint was not visible. I tried to dig in for the reason and found that in #213165 we have updated from const shouldRenderDefaultHint = model?.uri.scheme === Schemas.untitled && languageId === PLAINTEXT_LANGUAGE_ID && !inlineChatProviders.length; to const shouldRenderDefaultHint = model?.uri.scheme === Schemas.untitled && languageId === PLAINTEXT_LANGUAGE_ID && hasEditorAgents; and hasEditorAgents is false for the below-mentioned version. (Ref: https://github.com/microsoft/vscode/pull/213165/files#diff-f8e886d59b8516de7fea9105924026f872bf576f42bc20cff450073df79337f7L149).
Is it an expected flow? Would love to fix this if this is not expected.
Version: 1.90.0
Commit: 89de5a8d4d6205e5b11647eb6a74844ca23d2573
Date: 2024-06-04T19:34:48.028Z
Electron: 29.4.0
ElectronBuildId: 9593362
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Darwin arm64 23.5.0

Assigning first to @joyceerhl. Feel free to assign back to me if you would like me to investigate.

@manav014 the issue you describe is fixed in Insiders https://github.com/microsoft/vscode/issues/214679

Hey @joyceerhl Thanks for confirming. I can take this forward, would be great if you could assign this to me.

Thanks for assigning this @joyceerhl !
I tried to work on this.
I am planning to introduce an onKeyDown  event that will trigger once user press any key in editor. If it is tab then select the tab indexed anchor tag for select a language, fill with template and don't show in the order.
But, I Found that there is no class name for anchor tags for select a language, fill with template and don't show. In order to assign them a tabIndex and focus them onKeyDown I will need a class name . Shall I introduce a new class name to these anchor tags?
Or if this approach doesn't seems feasible. Do you have any idea in mind about how can I approach the solution to this?

@joyceerhl I have created a PR that will fix this issue. Would be great if you can review the PR.

Hey @joyceerhl did you get a chance to look into this?


Add android documentation on README

+1

PR welcomed for the Doc 👍

I'd happily help with a PR but with no initial pointers I'm struggling to get it working on Android...
I come from an primary iOS background so there must be something I'm missing in my Android knowledge.
I've gotten as far as adding the following the the apps main AppManifest file under a ".MainActvity" activity:
            <intent-filter android:priority="100">
                <action android:name="android.intent.action.OPEN_DOCUMENT" />
                <category android:name="android.intent.category.DEFAULT" />
                <category android:name="android.intent.category.OPENABLE" />
                <data android:mimeType="*/*" />
            </intent-filter>

However I'm seeing the following error:
"No Activity found to handle Intent { act=android.intent.action.OPEN_DOCUMENT cat=[android.intent.category.OPENABLE] typ=public.content }"
Any ideas?
Thanks!

Screenshot:

@lprhodes Did the latest readme edit help you from merging https://github.com/Elyx0/react-native-document-picker/pull/37 ?

@sibelius Hello, I'm the problem, in Adnroid equipment APP pop up after you this mistake after he collapses, excuse me, can you solve it?

@Elyx0 Hello, first of all thank you for your patience see my question!Thank you!The same problem I met the original poster here, do you have a solution?

@lprhodes Hello, I'm the problem, in Adnroid equipment APP pop up after you this mistake after he collapses, excuse me, can you solve it?

@lizhiren2016 I deleted your multiple posts, you don't have to mention us one by one you could group them. @sibelius would surely know about it since it used to work and I didn't take part in the Android port.

@Elyx0 Ok, I'm sorry!Get you into trouble

@lprhodes I'm having the same issue selecting documents from Android. It says: No activity found to handle intent (act=android.intent.action.OPEN_DOCUMENT...). Did you solve this? I don't know how to fix it. Thank you.

I'll be trying again in the next few hours. I'll let you know how it goes
On 30 May 2017, 8:53 AM +1000, Miguel Alejandro Bolivar Portilla<EMAIL_ADDRESS>wrote:

@lprhodes I'm having the same issue selecting documents from Android. It says: No activity found to handle intent (act=android.intent.action.OPEN_DOCUMENT...). Did you solve this? I don't know how to fix it. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Elyx0 Unfortunately it's not - the documentation is pretty much what react-native link react-native-document-picker does.
@Darking360 Did you find a solution? Looks like I'll be shipping without it on Android

@lprhodes Maybe anything from https://github.com/Elyx0/react-native-document-picker/issues/41#issuecomment-305476646 ?

That helps thanks; changing to conditional file types based on platform

@lprhodes https://github.com/Elyx0/react-native-document-picker/issues/41#issuecomment-305579345 ! 🙏

@lprhodes I'm going to give it a try to the solution proposed by @Elyx0. I'll keep you updated. Maybe tomorrow I'll write about it, I live in Venezuela and the internet connection doesn't help at all. Thank you in advance 👍


WIP: add retry to getComposeStatus function

add retry to getComposeStatus function-
avoid a case when worker stopped responding on create new image
related to EdgeTHEEDGE-2419
Signed-off-by: mgold1234<EMAIL_ADDRESS>Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I run make pre-commit to check fmt/vet/lint/test-no-fdo

/retest

open new one


Attribute incorrectly added for opening JSX tag

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.63.0
OS Version: Ubuntu 20.04.1

Steps to Reproduce:

Create a functional React component
Inside the element's return create a React Fragment
Inside the fragment's scope type "<" and start typing a html element name until intellisense suggests some. Type tab or enter. Instead of "button" for example, it will place "button={}"

@armanio123 @RyanCavanaugh This looks like a potential candidate for the TS recovery build. Even when you set "javascript.preferences.jsxAttributeCompletionStyle": "none", the = still ends up being inserted

@mjbvz good catch, this is annoying as heck

namespace JSX {
    export interface IntrinsicElements {
        button: any;
        div: any;
    }
}
function fn() {
    return <>
        <butto
    </>;
}

Bring up completion and tab-complete at butto, the new text is
    <button={}

when it should be (as was in 4.4)
    <button

Even when you set "javascript.preferences.jsxAttributeCompletionStyle": "none", the = still ends up being inserted

I can't reproduce that bit; are you in a tsx, which instead works on "typescript.preferences.jsxAttributeCompletionStyle" (note the "typescript")? The expression does check that explicitly.
In any case, I've sent #47096; what was happening was that in this broken AST, it looks like an attribute, when in fact it's the tag name. 4.4 didn't have JSX attribute snippets, so never had to care about this detail.

Thanks for taking a look @jakebailey! The VS Code recovery build will go our Tuesday. Would it be possible to get this into a published TS 4.5.4 by then?

Yes, I think we can do a cherry-pick/build.


Hide the remove icon of organization SSO authentication option when the app is marked as B2B app.

Describe the issue:
Hide the remove icon of organization SSO authentication option when the app is marked as B2B app

The service provider object will have a property to find whether the app is a b2b app or not. That propert is isAppShared as shown below SS.

The response of this request /t/<tenant-domain>/api/server/v1/applications/<app-id>

Duplicate of - https://github.com/wso2/product-is/issues/17609


Fix controller log floods

Resolves the problem of controllers flooding their logs with data related to missing images. The image stream update events were firing when no update was being performed. Its possible this could lead to latent gaps in events, but on balance, this is the big win.
Tested with OSE 3.3-3.4 and OCP 3.5-3.6 using Hub 4.2.1

Merge to allow nightly cycle


Plan / Follow-up Tab: Existing Research Outputs are lost when adding a new Research Output

Please complete the following fields as applicable:
What version of the DMPRoadmap code are you running? (e.g. v2.2.0)
v4.0.7
Expected behaviour:
Plan / Follow-up Tab: The list of Research Outputs should be modifiable.
Actual behaviour:
Plan / Follow-up Tab: Existing Research Outputs are lost when adding a new Research Output
Steps to reproduce:

Save some Research Outputs on the Follow-up tab.
Return to the tab
Add new Research Outputs and Save
Note the changes to the list

Noting here that the items being lost are 'related identifier' not 'research outputs'

I tested this and can now add new Research Outputs to an existing list under the Follow-Up tab. The additional outputs are added and do not remove any existing outputs.

I tested this just now, and when I added a new article to our research outputs and it did stick without wiping out the others! :)
However :( there are also now a number of un-related research outputs listed that I did not put there (though the unrelated research outputs do not appear in our final DMP: https://doi.org/10.48321/D1CW23

very strange. Thanks for bringing this to our attention @Br-Johnson.

This has been patched @Br-Johnson and we've run a script that re-fetched the citation information and attached it to the correct DOIs.
Please have a look at your DMP and let me know if anything still looks incorrect.


Parallels installation on OS X

We need Parallels to be installed on OS X.
More info about Parallels:
https://www.parallels.com/products/desktop/download/

Successfully installed Parallels:

Fixed in master

Tested in prod.


Fix possible targetkv name mangling

This fixes a bug which occurs when fs_kv_path is a substring of filepath.
I learnt this the hard way:
fs_kv_path=conf
filepath=conf/myapp/application.conf
in consul I wondered why I'm having a key of conf/myapp/application.

in 1.0.5 thanks!

in 1.0.5 thanks!


Golomb sequence in C

💥 Proposal
Golomb sequence in C
Have you read the Contributing Guidelines on Pull Requests?
YES

/assign


Compatibility with Serenity 0.11

Hi, I've been trying to make the most basic voice example work in serenity, however ive ran into a block when implementing it into my own project as it doesn't seem to support serenity 0.10.
The example uses .register_songbird() on the ClientBuilder instance to register songbird, however that does not seem to be present when using Serenity 0.11. It however works when downgraded to serenity 0.10 (with the same code), but unfortunately that does not support Gateway Intents which I rely on.
Can the examples be updated with working code for the latest version of serenity, or is there any workaround that allows me to use songbird with Serenity 0.11?
Thanks!

Turns out, the latest stable version of songbird only works with up to serenity v0.10, even though the latest stable version of serenity is 0.11. You have to use the next branch of songbird for serenity >= v0.11 compatibility

Solved by v0.3.


{racket,racket-minimal}: 8.8 -> 8.9

Description of changes
https://github.com/racket/racket/releases/tag/v8.9

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build racket
@GrahamcOfBorg build racket-minimal


Clean Add Org backend DB

Overview
Production backend has test data within it and it must be cleared to ensure data integrity. Test records typically include the word test within them
Action Items

[ ] Access production django backend and clear records that may appear to be tests

Resources/Instructions

@ladissi Could you clarify if this is the same as issue #941?


fix mongodb composite types example schema

Changes

Change type of price from Decimal (unsupported) to Float in the example schema.

What issue does this fix?
The MongoDB connector does not support the Decimal type. So this example would previous raise an error. Reference

Thanks @TasinIshmam , good catch, our docs clearly state that we don't currently support Decimal. And it sounds like we want to go with Float instead of string.. @matthewmueller can you give this a quick review, and then let's get it merged?

Using String is our official suggestion: https://github.com/prisma/prisma/issues/12637#issue-1190313305 (also linked from the reference)

Thanks @janpio , I saw in the comment that string was the initital proposal, but further down in thread it seemed as though Float is more widely used...?

I vote we leave it as Float for now, til we hear from Matt. Go ahead and merge it if you are still awake, @Tasin... or I can in a bit if I see you haven't yet. I just want to get rid of the Decimal` asap. ;-)

From a technical perspective it is dangerous to represent prices as Floats as doing math with them introduces nice bugs that make you loose money. Random Google result on the topic: https://dzone.com/articles/never-use-float-and-double-for-monetary-calculatio


Tst branch

Coverage decreased (-0.8%) to 84.0% when pulling 602b3fa6c9ab3b53fe618b0a3f099d4eddd121f3 on tst_branch into 6c02aa0a24c5e158c081e318ae7ba080879bdfd6 on dev.


No definition for GetPlatformPointer

Hi
From the code example:  this.cloudSession.Session = aRSession.subsystem.nativePtr.GetPlatformPointer();
I get the error that GetPlatformPointer(); doesn't exist. Unity2019.2
What should it be?

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

ID: 47ed5367-ee97-31a2-78ff-dd98e5c23d24
Version Independent ID: 2e975197-ca07-351a-f538-372500c0cc8f
Content: Create & locate anchors in Unity - Azure Spatial Anchors
Content Source: articles/spatial-anchors/how-tos/create-locate-anchors-unity.md
Service: azure-spatial-anchors
GitHub Login: @RamonArguelles
Microsoft Alias: rgarcia

@jelmerS2 Thank you for your query , our team will look into it and get back to you at the earliest.

Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @jelmerS2, the definition for GetPlatformPointer is here, https://github.com/Azure/azure-spatial-anchors-samples/blob/master/Unity/Assets/AzureSpatialAnchors.SDK/Scripts/SpatialAnchorExtensions.cs#L49, it's an extension method in SpatialAnchorExtensions.

Any update on this. I am facing the same issue in UNity 2019.4.15.

Any update on this. I am facing the same issue in UNity 2019.4.15.

@RamonArguelles fyi. Thanks

@RamonArguelles fyi. Thanks

We have opened an internal ticket (#31481757) to make sure that we are more explicit about that function in this docs page. @asergaz please assign to me.

We have opened an internal ticket (#31481757) to make sure that we are more explicit about that function in this docs page. @asergaz please assign to me.

any updates on this? There is still an example in the documentation which does not work.
https://docs.microsoft.com/en-us/azure/spatial-anchors/how-tos/create-locate-anchors-unity
Set up the session
#if UNITY_ANDROID || UNITY_IOS
    this.cloudSession.Session = aRSession.subsystem.nativePtr.GetPlatformPointer();
#elif UNITY_WSA || WINDOWS_UWP
    // No need to set a native session pointer for HoloLens.
#else
    throw new NotSupportedException("The platform is not supported.");
#endif

    this.cloudSession.Start();

GetPlatformPointer() is not defined !

Hello - can you please reassign this to Pamistel? thank you!

I realize this issue is very old now - but wanted to leave a note here that we updated our tutorials in May - please let us know if these are helfpul, thank you
Tutorial: Create a new HoloLens Unity app - Azure Spatial Anchors | Microsoft Docs
microsoft/OpenXR-Unity-MixedReality-Samples: Sample Unity projects to demo how to use OpenXR plugin in Unity for HoloLens 2 and Mixed Reality headsets (github.com)


FT.Aggregate with APPLY - text fields data is returing in lowercase if we do the multi column sort

Thanks for reporting an issue in NRedisStack! Please update the appropriate text below, as much data as possible really helps!
NRedisStack Version:0.12.0
Redis Stack Version:
Description:
I am facing the issue with multi column sort - the text fields data am getting in lowercase, i need this values what i stored. the values will be stored as in capital / pascalcase / camelcase but all values am getting in the lowercase. I have did some observation on this please look into this
FT.Aggreagate & FT.Search Commands
FT.Search -
Its support the single column sorting
If I have decimal fields & its type as numeric with set sortable true then,  sorting for this column not doing perfect.
Example - FT.Search indexname "*" SORTBY sales_ty ASC;
FT.Search command not support for multi column sorting
2. FT.AGGREGATE with LOAD
Its support the multi column sort for text & numeric fields is working as expected
but if I have the field like (sales - 23435.3446 ) in that case its not working as expected
Example - FT.AGGREGATE indexname * LOAD * SORTBY 6 @manufacturer_desc DESC @units_ty DESC @sortassist DESC LIMIT 0 40;
3.  FT.AGGREGATE with APPLY
Its support the multi column sort for text & numeric fields is working as expected.
For numeric field like (2364 or 3435.345), its also working as expected
But issue is for string/text sortable fields - where its return the data in lowercase
Example - FT.AGGREGATE indexname * SORTBY 6 @manufacturer_desc DESC @units_ty DESC @sortassist DESC APPLY @manufacturer_id AS manufacturer_id APPLY @manufacturer_desc AS manufacturer_desc APPLY @week_id AS week_id APPLY   @week_enddate AS week_enddate APPLY @units_ly AS units_ly APPLY @fiscal_week_desc AS fiscal_week_desc APPLY @units_ty AS units_ty APPLY @sortassist AS sortassist LIMIT 0 40
Stored Result

Sorting Result

Please help on this ASAP

Hi slorello89, thanks for help!
If I use this command ( FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * ) by taking LOAD after the sortby its looks good up to page no 3 but after for page no 4, 5, 6 its returning the previous page records. am taking **load *** becuase for payload, properties are dyanamically created for every scehma.
Examples - scenarion 1

FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * limit 1 10  -  page 1 => looks good
FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * limit 11 10 - page 2 => looks good
FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * limit 21 10 - page 3 => looks good
FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * limit 31 10 - page 4 => looks good
FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * limit 41 10 - page 5 => returning the page 4 records
FT.AGGREGATE test * SORTBY 2 @name ASC LOAD * limit 51 10 - page 6 => returning the page 4 records

Examples - scenarion 2
FT.AGGREGATE dev1481pl_36aedac5cd0e4f0d777499bff2bc812ef8ad4f6854ac4fc490c710ddcccdee33 *  SORTBY 4 @textFields DESC @sortassist ASC LOAD 6 text1 text2 numeric1 numeric2 text 3 numeric4 LIMIT 1 10
for this example - its showing the correct records but the text fields showing in lowercase again.
Please help on this.

Hi @balasahebr92 - I'm not able to reproduce either of these issues on my end. Not scenario 1 where you get duplicate pages, not scenario 2 where the UNF does not seem to apply.
For scenario 1, I'd ensure that the index is in the state you think it is before querying it.
For scenario 2 my shot in the dark guess is that you did not drop and recreate the index between runs
I created a gist with some example code so you can see what I'm talking about (this uses UNF and works).

Hi slorello89, thanks for help!
Still am not clear the thing
With this below query its working fine for me but as per your suggestion need to use the loadAll after the sortby
FT.AGGREGATE indexname* LOAD * SORTBY 4 @column1 ASC @column2 DESC LIMIT 1 10
Example - lets I have the 100K+ records and I need to retrieve the data by page size 10 every time but sometime like ( 20, 50, 100 pagesize) so what type of query you suggest for best performance point of view
Query 1
FT.AGGREGATE indexname* LOAD * SORTBY 4 @column1 DESC @column2 DESC  LIMIT 1 10
Query 2
FT.AGGREGATE indexname* LOAD 7 column1 column2 column3 column4 column5 SORTBY 4 @column1 DESC @column2 DESC  LIMIT 1 10
Query 3
FT.AGGREGATE indexname* SORTBY 4 @column1 DESC @column2 DESC  LOAD *  LIMIT 1 10
Query 4
FT.AGGREGATE indexname*  SORTBY 4  @column1 DESC @column2 DESC LOAD 7 column1 column2 column3 column4 column5  LIMIT 1 10

Hi slorello89
As per above scenarios am using the Query 2 it's working for me as per my expectation but if I use the load after the SORTBY - for text fields its return the data in lowercase even if I declare this text column as UNF while creating the index.
thank you very much!

Hi @balasahebr92 - as I mentioned, you need to use the LOAD first to override the normalization.  Otherwise  it will not override what's already in the schema.

Hi slorello89
Yeah, its looks good now.
thanks for help. am closing this issue now from my side


New alignment option: "exact" without broadcasting OR Turn off automatic broadcasting

Is your feature request related to a problem?
If we have two objects with dims x and x1, then xr.align(..., join="exact") will pass because these dimensions are broadcastable.
I'd like a stricter option (join="strict"?) that disallows broadcasting.
Describe the solution you'd like
xr.align(
    xr.DataArray([1], dims="x"),
    xr.DataArray([1], dims="x1"),
    join="strict",
)

would raise an error.
It'd be nice to have this as a built-in option so we can use
with xr.set_options(arithmetic_join="strict"):
    ...

Describe alternatives you've considered
An alternative would be to allow control over automatic broadcasting through the set_options context manager., but that seems like it would be more complicated to implement.
Additional context
This turns up in staggered grid calculations with xgcm where it is easy to mistakenly construct very high-dimensional arrays because of automatic broadcasting.

I'm curious to see if in that specific case using a custom index for staggered grid coordinates wouldn't work as an alternative solution. The alignment rules are pretty strict (same index type + same sequence of coordinate names & dims). Not 100% sure if that applies in the xgcm case, but in theory it would raise an error regardless of the join method if you try to align objects that have broadcastable coordinates with incompatible indexes.

This is on my to-do list to think about really hard :sweat_smile:
I think one problem would be that sometimes a different grid position implies a different length coordinate array, e.g. "outer" vs "center" is larger by one element.

Basically I want the following to raise an error because I've picked the wrong variable with dimension x1 instead of dimension x by mistake
xr.DataArray([1], dims="x") / xr.DataArray([1], dims="x1")

I think we could do it with a "staggered grid index", but it seems useful in general

Ah ok I see, thanks! Yes I agree this may be useful.

@dcherian — I'm discussing this with @etienneschalk over at #8698
Can we clarify your proposal? Is it:

All dims must match — i.e. ds1.dims == ds2.dims?
or something more muted, like not broadcasting a dimension of size 1 to the same-named dimension of size n?

The first seems very strict

Thoughts from the meeting:
"join" is about indexes, and alignment and broadcasting should be orthogonal ops.
So instead of a new join option, we'll add an option to control the broadcasting. So something like:

align(..., join='exact', broadcast=False) # new kwarg broadcast
And a new global option arithmetic_broadcast to complement arithmetic_join . This will control automatic broadcasting in binary ops.

Hello @dcherian
I adapted changes made in #8698 with the added clear separation between the join and broadcast keyword parameters.
So while the form (API) changed, I did not change the current substance, and it is still not entirely clear for me, what is expected to implement this issue successfully. I wrote a small text following the discussions with @max-sixty about the changes I currently made, and my current understanding: https://github.com/pydata/xarray/pull/8698#issuecomment-1936982530 ; is this aligned with what you would expect? (pun intended... 😅)
The key test you can have a look for in the PR, testing what you described in your comment:

@etienneschalk I sincerely apologized. I  deeply misunderstood what align was doing when I wrote this issue. align isn't actually broadcasting, it is simply checking indexes (as it should).
Given the discussion in the meeting, I think the simplest solution is to raise an error here if arithmetic_broadcast is False:
https://github.com/pydata/xarray/blob/ff0d056ec9e99dece0202d8d73c1cb8c6c20b2a1/xarray/core/variable.py#L2265-L2268
This should be a significantly smaller and easier PR.
Again, apologies for leading you down the wrong path.

No problem, even if the PR is not solving what was expected, it was the opportunity for me to dig into the xarray internal logic which is valuable learning!
I can try to implement your suggestion, reusing the existing 'arithmetic_broadcast' flag


Made a pypi package

Hey, wanted to thank you for your work on this plugin. I found myself needing a python package that did the same conversions and none seemed to exist that sufficed (or could handle CONTANT_CASE to dash-case conversion).
So I put it up on Pypi here: https://pypi.python.org/pypi/case_conversion/1.0.1
Here's the github repo: https://github.com/AlejandroFrias/case-conversion
I changed the names so it looked more like type casting, added some aliases for dashcase that are common and added some thorough testing for sanity check. Otherwise, it's a relatively straight forward port of the non-Sublime aspects of this plugin. You could even factor out that stuff now and import this...
I mostly wanted to let you know I did this in case you had any issues with how I gave you credit. If you want more or less credit, just let me know (but I did use your copy right and mentioned all the contributors in the README.

@AlejandroFrias sounds great to me! Thanks a bunch.


Fixed cmdline_force_redraw hack for path completion

This fixes #188. The hack was doing stuff to make sure the screen is redrawn when typing a substitution command, but s/ also matches every path with a directory with an s at the end. For the user, it looks like every path with an s at the end is automatically accepted (see the bug report). I Changed the hack to first check for an s/ and if there is one, check that the current command is not a command that could use path completion.
A better way would probably be to check if the completion pop-up is currently open, but I don't know how to do that and this works.

Thank you for the report! I've changed it to check for vim.fn.pumvisible() instead. Seems to work fine with that. Thanks again!


Config change support refactoring

Fixes #579
Fixes #1183
Fixes #578

Follow up: https://github.com/oracle/helidon/issues/1420


Link logo to homepage '/'

Addressed by https://github.com/submariner-io/submariner-website/pull/113


Implement BRS for Azure_BackupVault_DP_Enable_Soft_Delete_Trial

Make sure Always-On or Control ID related texts are same across.

Update access related message "To enable always-on ........."

Check Always-On casing as per documentation

Make sure Always-On or Control ID related texts are same across.

Updated accordingly

Update access related message "To enable always-on ........."

Updated message accordingly

Check Always-On casing and "-" addition uniform as per documentation

As per documentation
https://learn.microsoft.com/en-us/rest/api/dataprotection/backup-vaults/get-in-subscription?view=rest-dataprotection-2024-04-01&tabs=HTTP#softdeletestate
'AlwaysOn' is used inside a variable, and that variable is utilized elsewhere.


Update Hub CI so data pipeline test only runs when /meltano files are changed

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2978
Originally created by @tayloramurphy on 2021-10-07 21:39:32

Noticed in https://gitlab.com/meltano/hub/-/merge_requests/116 the new job which can slow the pipelines down.

View 1 previous comment from the original issue on GitLab


issues using agent and key

Running atom 0.152.0 and remote-edit 1.6.0 on Ubuntu 14.04 I can't seem to connect using the agent or key choices. Password works.
I did some investigation and the error always is 'Timed out while waiting for handshake' around line 232. The value of self._state seems to always be "initexchg". I also dumped the opts.agent and it points to the right socket for the SSH agent.
Using the agent with the normal openssh cli tools works as expected and so do a couple of GUI tools I tried.

Are you still experiencing this issue? I just tried key and agent auth on a fresh install of debian testing and it worked great.

yeah, I just tried 1.6.2 as well and I continue to see the timeout error

I just tried again using 0.177.0 and I'm still seeing the handshake error. I moved my entire .atom dir out of the way and then installed just remote-edit. Then I got the following strack trace:
TypeError: undefined is not a function
  at /home/matthias/.atom/packages/remote-edit/node_modules/ssh2/lib/client.js:136:12

It's smelling like something about my SSH is not working ...

I found this bug which might be related.


Supports for IPv6

Maybe related to: https://github.com/beakerbrowser/beaker/issues/1246

From my observations, IPv6 addresses seem to be ignored completely while discovering peers, which seems to be a huge downside as IPv4 is in many cases blocked by NAT while IPv6 aren't. Essentially, with IPv6 there's no need for UPnP whatsoever.

This is where ipv6 support would need to be added. https://github.com/hyperswarm/dht/issues/1


Error on Heroku

Item could not be created:
We couldn't deploy your app because the source code violates the Salesforce Acceptable Use and External-Facing Services Policy.

me too

Same

All deployment errors has been solved  in my repo
Deploy code server on heroku

It can be mining so heroku ban it

Heroku deployment errors has been solved in my repo Deploy code server on heroku

This is working with heroku 😃


removed repetitive buttons

Deleted the bar on top.
Deleted profile on the left.
Added Lineups section for cs2 and val + icons


Windows Defender tags file as Trojan:Win32/Wacatac.B!ml

Just downloaded the zip and got a detection soon after. I'm not sure if this is useful, first time actually using github, but hopefully this helps.
containerfile: C:\Users\Arslan\Downloads\TF2CC_win.zip

file: C:\Users\Arslan\Downloads\TF2CC_win.zip->TF2CC/TF2CC.exe

webfile: C:\Users\Arslan\Downloads\TF2CC_win.zip|https://objects.githubusercontent.com/github-production-release-asset-2e65be/812575773/4917c59e-6218-4f69-9404-ed8aaf587712?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=releaseassetproduction%2F20240614%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240614T110137Z&X-Amz-Expires=300&X-Amz-Signature=f396908bb9f843b910a8d056bee3528821a246b505c5096616113d84ba77eedb&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=812575773&response-content-disposition=attachment%3B%20filename%3DTF2CC_win.zip&response-content-type=application%2Foctet-stream|pid:15484,ProcessStart:133628365012645505

To say it nicely, windows defender will flag any executable that doesn't come from either a trusted source or is singed with a certificate. To get one you need to pay a lot of money to micorsoft for one and let the review it. I am nog going to do that ofc. This stackoverflow awnser explains how and what I would need to do: https://stackoverflow.com/questions/48946680/how-to-avoid-the-windows-defender-smartscreen-prevented-an-unrecognized-app-fro/66582477#66582477
You should be able to trust the program in your antivirus software (windows defender will just do a popup). If you do not trust my program for whatever reason, that's fine. A solution to that would be to install python from https://www.python.org/downloads/release/python-3124/ and install the dependencies with pip3. Then you can just run the source code on it's own. Tldr: I can't do much about that due to windows being annoying.


Update OID

@brandonbk @B77Mills I think this is correct, but I don't know how to 'test' this
https://trello.com/c/wSdJsr7E/218-incorrect-oid-in-olytics-script

@jwade1327 This doesn't appear to be the correct value, even with the incorrect value (which appears to have been copied from another tenant's config) it still processes up until trying to check the site domain in Omeda.
DEV:

PROD:


fix: nonContentRoutes diff path

fixed "diff" path in nonContentRoutes.
If we had a path like "/test/test-a/difference" where the last part of url contains the word 'diff' it was considered as non-content route and it was wrong

📚 Documentation preview 📚: https://volto--6102.org.readthedocs.build/

@giuliaghisini I fixed the changelog, although it seems that there is one test failing, could you please take a look? Thx!!


Accessing reference to stack navigator within static header without 'this' Component?

What's the best way to do this? You can see my ugly static reference here:
export default class TestComponent extends Component {
  static ourself = null;
  constructor() {
    super();
    ourself = this;
  }

  static navigationOptions = {
    title: 'Header',
    header: ({ state, setParams }) => {
      let right = (
        <Button
          title="Header Button Test"
          onPress={() => ourself.props.navigation.navigate('OtherScreen')}
        />
      );
      return { right };
    },
  };
}

I'm also trying to the same thing, help!

@tonydiepenbrock - navigation is the argument that is passed to the header function:
// how you're defining the function
header: ({ state, setParams }) => {
  // ...
}

// is equivalent to
header: (navigation) => {
  let state = navigation.state
  let setParams = navigation.setParams
  // ...
}

So you can just do:
header: ({ state, setParams, navigate }) => {
  let right = (
    <Button
      title="Header Button Test"
      onPress={() => navigate('OtherScreen')}
    />
  );
  return { right };
},

Someone wanna send a PR removing the destructuring syntax from the docs?

That's a great idea, it's not really clear in an "untyped" language like JS that this tuple object is actually the navigation. Thanks!

Thanks, @jeffcarbs.


Net::LDAP Search works but login fails

I've been stuck at this for about 15 hours. It's on the verge of throwing my laptop. The weird issue is, the LDAP search/listing works PERFECTLY. It binds with the username and password I've given and shows all the users who have that attribute (eg, CN, UID, sAMAaccountName). However when I try to login, it shows:

This is how I listed all the users,

ANY help would be immensely appreciated!

Have you changed the default authentication key in config/initializers/devise.rb?  It should look like this:
Devise.setup do |config|
....
config.authentication_keys = [:username]
....
end

In your screen shots it looks like you are using a username but the default for this gem is :email.


'list' object has no attribute 'add_new'

I am using a custom uploader for the blocks and I give a parent page after determining a collection view page is the object to add a task, event, or etc to.
new_page = parent_page.collection.add_row(title=subject) #this gets created
uploadCustom(markdown_to_upload, new_page) #this sometimes does work but sometimes doesn't because many users are accessing and uploading things to their notion from my app so I don't always know what is getting uploaded.

I receive this whenever editing a collection view and attempt to add a new task. I'm happy to elaborate on the issue if this is not enough info! (This is my first issue made ever on github 😅)
in uploadCustom
uploadBlock(blockDescriptor, notionPage, mdFile, imagePathFunc)
File "/python/lib/python3.6/site-packages/md2notion/upload.py", line 72, in uploadBlock
newBlock = blockParent.children.add_new(blockClass, **blockDescriptor)
AttributeError: 'list' object has no attribute 'add_new'

Any suggestions?

Hmm, a full stack trace would help
Is the first snippet how you're actually calling it? I'm not sure if I've ever used a CollectionRowBlock in there (return of add_row). But .children is on ever Block so I'm not quite sure why it'd be a list and not a Children object

Ill get back in a week or two with the trace.... cheers for the fast response!

Here's the traceback! So the goal is to search for the parent collection or block, and if it is a collection, check if any pages or blocks under it exist, and if not, then in this code create a page under a collection_view_page and upload the contents wanted to said page in markdown.
Traceback:
Uploading TextBlock, 1/11 (9.1%) 'list' object has no attribute 'add_new'
Traceback (most recent call last):
  File "/workspace/notionsnag.py", line 1080, in notion_snag
    upload4Hitsnag(body, new_page)
  File "/workspace/notionsnag.py", line 230, in upload4Hitsnag
    uploadBlock(blockDescriptor, notionPage, mdFile, imagePathFunc)
  File "/workspace/.heroku/python/lib/python3.6/site-packages/md2notion/upload.py", line 72, in uploadBlock
     newBlock = blockParent.children.add_new(blockClass, **blockDescriptor)
  AttributeError: 'list' object has no attribute 'add_new'

I am using a custom function to add to a file that I have already
#Had to create a custom upload function due to mdFile needing to not be a file path but rather actual text search
def upload4Hitsnag(mdFile, notionPage, imagePathFunc=None, notionPyRendererCls=NotionPyRenderer):
	"""
	Uploads a single markdown file at mdFilePath to Notion.so as a child of
	notionPage.
	@param {file} mdFile The file handle to a markdown file
	@param {NotionBlock} notionPage The Notion.so block to add the markdown to
	@param {callable|None) [imagePathFunc=None] Function taking image source and mdFilePath
	to transform the relative image paths by if necessary (useful if your images are stored in weird
	locations relative to your md file. Should return a pathlib.Path
	@param {NotionPyRenderer} notionPyRendererCls Class inheritting from the renderer
	incase you want to render the Markdown => Notion.so differently
	"""
	# Convert the Markdown file
	rendered = convert(mdFile, notionPyRendererCls)

	# Upload all the blocks
	for idx, blockDescriptor in enumerate(rendered):
		pct = (idx+1)/len(rendered) * 100
		print(f"\rUploading {blockDescriptor['type'].__name__}, {idx+1}/{len(rendered)} ({pct:.1f}%)", end='')
		uploadBlock(blockDescriptor, notionPage, mdFile, imagePathFunc)

It is called here:
elif parent_block_type == "collection_view_page": #get correct collection type to know row can be added
    new_page = parent_page.collection.add_row(title=subject)
    upload4Hitsnag(body, new_page)

I hope this helps more, would be happy to elaborate again! Cheers for maintaining this package too btw!

Just saw the potential for rewrite and I think its a good idea for the future! At the moment Im in a bit of a pinch to get a new deploy out and would love to get this error fixed or figure out the bug. I understand this is open source and you are very busy but any feedback or potential solution would be awesome. Cheers!


[FAB-17374] chaincode service integration test

Type of change
-Test update
Description
Adds integration test to fabric using fabric-chaincode-go change from
https://github.com/hyperledger/fabric-chaincode-go/pull/19.
The test can be followed via integration/e2e/chaincode_service_test.go
which was adapted from the "basic solo.." integration test but adds
support for

controlling starting the chaincode
generating connection.json

The goal is to provide a basis for creating other chaincode service
tests in future.
Signed-off-by: muralisr<EMAIL_ADDRESS>

@muralisrini, it's been a couple of weeks. Any idea when you might come back to this?
Thanks.

It would be really nice to get this work done, as we march closer to a new LTS release, having the integration test would be great as people start to migrate from 1.4 LTS to 2.x LTS and start using new features, so we can make sure it functions as expected moving forward.

/azp run

I'm closing this as it's seen no activity in months


Searching DataObjects

Considering the current PRs in progress and such, I'm just going to leave this patch here for now. It may require a little tweaking to match the latest version, however it should mostly be good to go.
This covers off issue #22 in regards to the ClassNameHierarchy_ms being applied incorrectly, and also covers any related problems with ParentsHierarchy_ms restriction (where your data object isn't actually under a site tree element).
diff --git solr/code/pages/SolrSearchPage.php solr/code/pages/SolrSearchPage.php
index ed9942d..aade653 100644
--- solr/code/pages/SolrSearchPage.php
+++ solr/code/pages/SolrSearchPage.php
@@ -409,14 +409,27 @@ class SolrSearchPage extends Page {
 		$offset = isset($_GET['start']) ? $_GET['start'] : 0;
 		$limit = isset($_GET['limit']) ? $_GET['limit'] : ($this->ResultsPerPage ? $this->ResultsPerPage : 10);

-		if (count($types)) {
+		// Apply any hierarchy filters.
+
+		if(count($types)) {
 			$sortBy = $this->solrSearchService->getSortFieldName($sortBy, $types);
-			$builder->addFilter('ClassNameHierarchy_ms', implode(' OR ', $types));
-		}
-
-		if ($this->SearchTrees()->count()) {
-			$parents = $this->SearchTrees()->column('ID');
-			$builder->addFilter('ParentsHierarchy_ms', implode(' OR ', $parents));
+			$hierarchyTypes = array();
+			$parents = $this->SearchTrees()->count() ? implode(' OR ParentsHierarchy_ms:', $this->SearchTrees()->column('ID')) : null;
+			foreach($types as $type) {
+
+				// Search against site tree elements with parent hierarchy restriction.
+
+				if($parents && (ClassInfo::baseDataClass($type) === 'SiteTree')) {
+					$hierarchyTypes[] = "{$type} AND ParentsHierarchy_ms:{$parents})";
+				}
+
+				// Search against other data objects without parent hierarchy restriction.
+
+				else {
+					$hierarchyTypes[] = "{$type})";
+				}
+			}
+			$builder->addFilter('(ClassNameHierarchy_ms', implode(' OR (ClassNameHierarchy_ms:', $hierarchyTypes));
 		}

 		if (!$sortBy) {

This is now pushed up as a pull request, so I'll close this.
https://github.com/nyeholt/silverstripe-solr/pull/34


References to imported module not the same in lerna monorepo

Do you want to request a feature or report a bug?
bug
What is the current behavior?
When running a jest test in a lerna monorepo,  a common module imported in two subpackages is not the same object reference. This causes instanceof and strict equalities to fail.
This does not happen using simple file modules.
If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
https://github.com/claudiorodriguez/jest-error
What is the expected behavior?
Reference to imported module should be the same, instanceof operation should return true in jest as it does when running through babel-node
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.
See repo example for package versions. Using node 8.8.1, npm 5.5.1, Mac OS X 10.11.6

This is fixed in jest@test. I have to admit I'm not really sure when it was fixed...
$ npmt

><EMAIL_ADDRESS>test /Users/simen/Development/jest-error
> jest

 PASS  ./test.spec.js
  ✓ is instance on jest (5ms)

  console.log packages/main/index.js:7
    is instance on main:  true

  console.log packages/check-instance/index.js:4
    is instance in checkInstance:  true

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        1.043s
Ran all test suites.
$ npmE jest --version
v21.3.0-beta.8


./fgfa_rfcn/core/tester.py import dill

importerror: no mudule named dill

annotate (#import dill) helps.

Hello, have you solved the problem?


Add utterances support to nextra-theme-blog

https://utteranc.es/ is a popular comments "plugin" that manages comment data in GitHub repo. Would be great to have it built-in to nextra-theme-blog, or allow users to easily add it

Yeah I'm kinda focused on the docs theme at the moment, but it would be great to have a "footer"/"postFooter" option for the blog theme (so you can put any component there), as well as a dedicated section in the docs for disqus/utterances/analytics etc.

@shuding Actually I tried putting the utterances script in footer, such as

            Comments

However, it always takes a refresh to be able to see the widget, and when that happens I find the page is built from dev console. I am not familiar with how the SSR work, therefore not able to figure out how.
Any advice? @shuding

Did you try this? That dangerouslySetInnerHTML isn't necessary.

  Comments


fix typos

This fixes primarily double slash XML C# comments.
Pay special attention to whether the changes to the following files are acceptable.
src/System.Management.Automation/engine/parser/Compiler.cs
src/System.Management.Automation/engine/runtime/Binding/Binders.cs
src/System.Management.Automation/security/CatalogHelper.cs
src/System.Management.Automation/engine/ComInterop/IDispatchComObject.cs
src/Microsoft.Management.UI.Internal/HelpWindow/ParagraphBuilder.cs

Per request I can scan files with extensions other than .cs and .resx for typographical errors.

@iSazonov
Is there a basis to rerun the 2 failing checks?

will do a soft reset

my soft reset shouldn't result in the check fails

Reopen to restart CIs.

@spaette Thanks for your contribution!

to be precise these typo corrected strings weren't found in comments
AmbiguosAuthentication
IsHomogenousArray
sucessfulDecompression
and there were some typo fixes of strings which had appeared enclosed in double quotes
also hopefully some textual changes for the purpose of readability


Missing level control cluster

Problem
Missing level control cluster
Proposed Solution
Implement it

Not needed given import issues filed


Document class contextType as the primary consuming mechanism

https://github.com/facebook/react/pull/13728
This documents the Class.contextType API.
The intention here is to document this as the primary way to consume a Context from a class, and since classes are still the mainstream component API, it's the primary way to consume a Context period.
The Consumer API is documented as the way to consume a Context from a function component.
As a result I could remove the following examples:
Accessing Context in Lifecycle Methods - Accessing context in life-cycles is now easy in since the primary way makes that the default. Technically this is still relevant for multiple context in a single class, but it's such an edge case that I feel like it belongs on Stack Overflow or some recipes section. Not the primary documentation.
Forwarding Refs to Context Consumers - While this is still something you might want to do, this is not different than forwarding refs in general. It used to be that this pattern was unique in that when you converted from a class to use Consumer + forwardRef, you would get into this awkward situation. However, if the primary API is to use .contextType on a class, then this situation doesn't happen so doesn't seem to add the confusion.
Consuming Context with a HOC - This pattern is possibly still relevant but it's only relevant because the render prop syntax is kind of a mess when you compose many of them. However, it looks like we're not going to be recommending this pattern and instead look to make it easier to consume context in other ways like the unstable_read proposal. Since this pattern isn't critical even before that's stable I think we should just remove it.

I think the HOC examples should be kept around. I'm not sure that consuming just a single context from a component is actually that common. The new API works well as a 1-1 replacement for some usages of legacy context, but most apps I've seen use a combination of HOCs and render props to inject data providers and there are usually more than one (for example react-intl injectIntl + Redux connect, plus some app-specific login context, navigation context, etc.).

You can still use render prop API for cases when you need that. This just attempts to cover the 80% case.


test(mark as unread): latest post should appear unread after marking the channel as unread in LHS via menu option or shortcut key

Summary

Adding new test cases for mark channel as unread
Ticket Link

Also, we're following conventional commits - https://www.conventionalcommits.org/en/v1.0.0/. Since we're doing "squash and merge", the commit message takes the PR title. With that, I'm recommending the following format for test cases PR:
test(feature): general description of tests

And so, please change to something like:
test(mark as unread): latest post should appear unread after marking the channel as unread in RHS via menu option or shortcut key

Thanks @saturninoabril
I've made all the changes as requested. Please take another look 🙂

Thanks @jgilliam17! LGTM.

I'm merging this now.


[BUG] Answer Section rerenders and show loader when Related Section is streaming and showing loader

Is there an existing issue for this?

[X] I have searched the existing issues

Vercel Runtime Logs

[x] I have checked the Vercel Runtime Logs for errors (if applicable)

Current Behavior
On the Search page when Answer Section has finished loading and the Related Section is streaming it also rerenders and show loader on Answer section.
Expected Behavior
Only the current streaming section should show the loader.
Steps To Reproduce

Write a prompt, Submit and go to Search page.
Wait for the Search Related section to start loading
You will see the Answer Section also shows the loader.

Environment
- OS: Mac
- Browser: Chrome

Anything else?

Please let me know if anyone is facing or had faced this issue and what was the solution to it.
I already looked into the AnswerSection and SearchRelated sections component and other required components but didn't found the issue. Streaming code base Seems confusing.

The re-rendering issue is related to the current implementation of AI SDK RSC. We are planning a migration: https://github.com/miurla/morphic/issues/368
This issue should be resolved once the migration is completed.


Feat: Svelte version

It would be amazing to have this UI library for Svelte as well. Would you consider it in the future?

Might be of interest: https://github.com/builderio/mitosis

Write components once, run everywhere. Compiles to Vue, React, Solid, Angular, Svelte, and more.


fix nested expression

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：
Fixes #4422
Problem Summary(Required) ：
ProjectNode couldn't found the input column. we need decode it before project node

[FE PR Coverage check]
:heart_eyes: pass : 6 / 6 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/optimizer/rewrite/AddDecodeNodeForDictStringRule.java
6
6
100.00%

[FE PR Coverage check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/sql/optimizer/rewrite/AddDecodeNodeForDictStringRule.java
7
7
100.00%

@Mergifyio backport branch-2.2

@Mergifyio backport branch-2.1

@Mergifyio backport branch-2.0


Rollout Controller will panic if the MutatingWebhookConfiguration and ValidatingWebhookConfiguration are not found

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

create an new kind cluster
run rollout controller on it

2. What did you expect to see? (Required)
3. What did you see instead (Required)
It panics
4. What is your KusionStack components version? (Required)
latest on main

I believe it is because the current webhook initialization process treats the secret as the Single Source of Truth (SSOT) and requires the ValidatingWebhookConfiguration to be pre-configured in the cluster.
I suggest changing the approach by considering the local certificate directory as the Single Source of Truth (SSOT) for TLS, instead of relying solely on a secret. Furthermore, we can make the WebhookConfiguration optional in the current webhook initialization process.


Gardening: Adjust piracy/exe verification message

We haven't had a true pirate to make fun of in Discord for a very long time, but altered/corrupt EXE files show up fairly frequently.  Adjust the message to better reflect this.

Thank you!


changed k8s api endpoint

Description
Changed k8s api endpoint
Other changes
changed build command in reserve-site yarn file
Tested

Deployed a testnet
Did a yarn build

Related issues
N/A
Backwards compatibility
Yes.

✅  on reserve build change


AKS Base 16.04

the below table shows that 1604 AKS Base image is available for AKS-Engine 0.60.1

On the other hand, it is mentioned that 1604 is deprecated

do we have discrepancy in this documentation?

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

ID: e69382e4-c307-4dfe-9885-92d2436ef858
Version Independent ID: 135ea653-550b-039d-121c-95f2e7c93f10
Content: Release notes for Azure Kubernetes Service (AKS) engine on Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/kubernetes-aks-engine-release-notes.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@HassanAlmanasrah
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mattbriggs
Can you please check and add your comments on this doc update request as applicable.

@HassanAlmanasrah Thanks for catching this. updating...
#in-progress

updated.
#please-close


Review new SBOM page

Review done as a separate commit. A bit of cleanup was done, though no major changes to content required.
Rendered locally, content appeared to be as intended. Link-check failure is due to an external issue.
This relates to doc PR #419
Signed-off-by: Katrina Prosise<EMAIL_ADDRESS>Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[x] Run spelling and grammar check, preferably with linter.
[] Avoid changing any header associated with a link/reference.
[x] Step through instructions (or ask someone to do so).
[x] Review for wordiness
[] Match tone and style of page/section.
[x] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Rather than getting permission to push to @doanac fork, pushed to a new branch of the main repo. @angolini I believe most of your comments were already addressed, but feel free to give feedback.  If all looks good, I can just copy/paste this as a review of the original pull request

Docs for 94d8184ed6263f565d273ae893939c97d0620b16 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1692/docs/artifacts/html/index.html

another thing I just thought about - we may want to include a disclaimer stating SBOM generation isn't fool-proof and can miss certain types of packages. I'm checking with legal and will get back to you.

@doanac Yep. I'm somewhat familiar with this sort of issue. I can write up a short statement in the mean time.

As I also posted in slack, perhaps a disclaimer saying something like “Foundries.io does not provide legal advice of license compliance.
It is recommended that SBOM results be verified.
Foundries.io is not liable for any legal repercussions resulting from failure to verify SBOM results and to comply with licensing.”

Hi. I just read the docs, looks good but I think we are missing talking about the UI functionalities.
IMHO, what we did in the UI is really really cool, so why not explain that even before fioctl?

@kprosise what do you think about creating a chapter about the UI and mention that we did it to facilitate customers' dally use?

@munoz0raul We have been discussing just that, and I agree, the web UI looks amazing. However there's some decisions that need to be made first, and the plan (to the best of my knowledge) is to merge it as it is, and then add the web UI info in a different commit. I may get started on it now though! Let us chat tomorrow.

Added additional reviewers, largely to help keep everyone informed. After the build checks, should we go ahead and merge, adding web UI details later? We could likely squeeze it for this release, especially if the above screenshots still mirror the current state of the web UI.

Link-check  failure can be ignored, as it has already be fixed in main.

i don't think the UI has changed or will change. So feel free to include a screenshot if you'd like. Maybe in a new PR since this has grown over time.

@doanac I'll go ahead and merge, and then open up a Jira issue to add web UI details.


Changing role during unit test does not work

Hello Martin,
I want to test that the ui shows only the elements which are available based on the user role.
The logout methode from the AAppTest.class removes a role/user from the
SecurityContextHolder/SecurityAuthenticatedUser, but that does not affect how karibu interacts with the "ui".
This leads to the issue that in a test in which data gets prepared by an admin user,
the "not admin" user still sees all the elements he normally should not be able to see.
The work around is to trigger additional navigation. Of course I would like to avoid that.
Is there a way to logout in a "stronger" sense?
@Test
    void an_employee_can_only_view_data() {
        //given
        tester.asAdmin();
        login(tester.getLogin(), tester.getPW(), tester.getRoles());

        var threat = createThreatInPool();
        logout();

        //when
        tester.asEmployee();
        login(tester.getLogin(), tester.getPW(), tester.getRoles());

        //this two navigations are only needed so that the ui gets "reopend" and the user role reevaluated
        // which leads to a check of the user role
        tester.navigate.toSafeguardTable();
        tester.navigate.toThreatTable();

        //then
        checkThatDataCanOnlyBeViewed(threat);
    }

I use the following dependencies:

org.springframework.boot
spring-boot-starter-parent
2.7.7

<vaadin.version>23.2.2</vaadin.version>
<karibu.version>1.3.24</karibu.version>
karibu-testing-v10-spring
karibu-testing-v23
The logout methode from AppTest class
    protected void logout() {
        SecurityContextHolder.getContext().setAuthentication(null);
        if (VaadinServletRequest.getCurrent() != null) {
            final MockRequest request = (MockRequest) VaadinServletRequest.getCurrent().getRequest();
            request.setUserPrincipalInt(null);
            request.setUserInRole((principal, role) -> false);
        }
    }

I think the problem is that the UI doesn't know that the user has been logged out and needs to be redrawn. You can reload a page to achieve that.

Thanks Martin,
while that was clear, it was not clear to me how to reload the page.
This can be done with: UI.getCurrent().getPage().reload();
It works now!


CSS rework

Update CSS from camelCase to kebab-case, full BEM to be done (possibly before merge)
Partially solves #71

These changes don't appear to have stuck around, this PR only has the logo changes in it


Add support for combined_field query

This adds support for the CombinedFieldQuery from the Lucene sandbox.
The supported syntax is as follows:
{
  "combined_field": {
    "query" : "quick brown fox", // required
    "fields" : [ // if no fields specified then matches nothing
      "a_text_field", // must be text field, else will be ignored
      "a_text_field_with_weight^5"
    ],
    "analyzer" : "custom_analyzer", // optional
    "zero_terms_query" : "none" //optional
  }
}

If no analyzer is specified, terms are derived from the union of terms from all fields' analyzers. The behavior of zero_terms_query is like for multi_match.
Fixes:

https://github.com/opensearch-project/OpenSearch/issues/3996

Signed-off-by: Michael Froh<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Discarding this pull request. The approach I took isn't correct, I think.
If / when I come back to this, I'll try reusing more stuff from MultMatchQueryBuilder.


Add arm builds

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@vgvassilev This PR should add arm builds to the conda-forge package

@vgvassilev do you also approve this PR? If yes, can it be merged please?


Whitelist for Mendix Dedicated IPs

Added a section on maintaining a whitelist of outgoing IP addresses for Mendix Dedicated Cloud

Release  delayed until at least 30 April 2020

Not released on 30 April due to dependencies.
Planned for 1 May.


parameterize unit test for hide_files

Signed-off-by: Robert Cerven<EMAIL_ADDRESS>

@rcerven needs rebase

rebased

rn not updated: just refactor of tests


Allow switch to be text-aligned properly

thank you! I agree, it makes sense to do this to match the normal checkbox


WIP: identity patching

This is a WIP draft of a COB that can handle patching an identity by gathering signatures from delegates to replace a previous document with a proposed document.

Ahh interesting! You don't think it makes sense to use the Patch COB for this?

Ahh interesting! You don't think it makes sense to use the Patch COB for this?

There were a couple of reasons I thought the Patch COB wasn't quite right for this, but maybe it could be reworked. It was a useful exercise for me to work through this at least, anyway :) So...
The main goal of this new COB is to gather the Signatures so that we can make a valid Identity commit with the latest Doc. The Patch does not actually have Signatures for Action::Review -- which incidentally might be a good thing for verifiable reviews. Even if it did, we would probably want the Signature over the latest Oid in the case of code review, but we want the Signature over the Doc, in our new case -- that, however, could be fixed by changing the signing to use the Oid instead, which is actually what Link was doing.
The other thing that made me avoid it is Action::Merge. As I said above, the aim was to gather signatures but I was aiming to avoid committing the final result until all of the signatures are gathered. The merge action seems to allow doing that beforehand. The way I was thinking about this was that Identity would have another method, something like:
pub fn merge(&self, proposal: Proposal) -> Result<Identity, Error> {
  /* 1. Checks that proposal.previous.current === self.current
     2. Checks that proposal.passes_quorum()
     3. Commit proposal.proposed
     4. Create the new Identity
  */
}

Thoughts? :)

I see! I need to think about this, but there is one benefit in using the patch cob, which is that UIs we build would be re-usable quite easily. Perhaps it's worth thinking about an "identity" mode for patches, though I also don't want to have a bunch of if-statements everywhere, so needs some good design. I think by parametrizing one or two things, we should be able to make the patch cob work in this scenario. Eg. maybe the merge rule is customizable using some trait or closure; and perhaps the signing function, ie. what is signed and how the signatures are stored is also customizable.

Great, yes there's value in just sketching this out before we commit to a design.
Where are we signing raw bytes? I'm confused :D

Where are we signing raw bytes? I'm confused :D

Specifically for signing the Doc we use the bytes instead of the revision's oid:
https://github.com/radicle-dev/heartwood/blob/e49226c50023e012657c022009d58489054d6f99/radicle/src/identity/doc.rs#L210-L215

Ah in the doc, yes. We can definitely sign the blob oid instead.

I wonder though, whether we sign the bytes or oid though, it's still not a valid commit signature :thinking: -- which means we're forced to add the sig to the trailer.. Would it make sense to create a commit signature instead?

I wonder though, whether we sign the bytes or oid though, it's still not a valid commit signature thinking -- which means we're forced to add the sig to the trailer.. Would it make sense to create a commit signature instead?

The Oid is the blob's Oid though not the commit :) But also realised that it doesn't make any sense for my point about Patch above :facepalm:

Yeah, I know it's the blob's that's why I'm suggesting if the commit should be signed instead to be more git native..

Yeah, I know it's the blob's that's why I'm suggesting if the commit should be signed instead to be more git native..

I'm not sure I follow :) The signature needs to be included in the identity's commit, no? So what would be the point of signing the commit?

Commit signatures are included in commits via the gpgsig header :)

Commit signatures are included in commits via the gpgsig header :)

Ya, but I believe that changes the SHA of the commit. I tested that quickly using the git-commit tests

Ok, yeah that makes sense since the sha-1 is just taken on the entire commit payload. This does mean though that we can't use commit header signatures (gpgsig) for identity docs, as stated in the spec? Because those sign the entire commit. It means we're forced to using trailer signatures, which is what we currently do.
If we used commit header signatures, it would "work", but would be an abuse of git, because they wouldn't actually be signing the commit. So this means RIP#2 needs to be updated to use trailer sigs, right?

Ok, yeah that makes sense since the sha-1 is just taken on the entire commit payload. This does mean though that we can't use commit header signatures (gpgsig) for identity docs, as stated in the spec? Because those sign the entire commit. It means we're forced to using trailer signatures, which is what we currently do.
If we used commit header signatures, it would "work", but would be an abuse of git, because they wouldn't actually be signing the commit. So this means RIP#2 needs to be updated to use trailer sigs, right?

Nowhere in our code are we signing the commit though. The signature is currently on the Doc payload. That's why I've gone down this route of "gathering signatures" via the Proposal COB, and then committing the final result with the signatures attached.
I'm not sure if there's any benefit to signing the commit itself, but perhaps I'm missing something.

Added a CLI subcommand for interacting with proposals. Here's a taster:

There's something strange going on with the TextBox and the ansii colouring in the diff output.

Nowhere in our code are we signing the commit though. The signature is currently on the Doc payload. That's why I've gone down this route of "gathering signatures" via the Proposal COB, and then committing the final result with the signatures attached.

Yes, that's correct, it's just the RIP that needs changing to match what's in the code. Now if we want to sign the git-object-hash of the blob instead for some reason, we can make that change as well.

Looks cool, the UI; I'd anyway move away from TextBox as I have done with the inspect command, because they don't really add anything but make parsing the output really annoying, and break if there's line-wrapping involved..

Fair enough! Is there/Will there be a way of separating out the parts of the output? I liked how the boxes separate out the diff, acceptances, rejections, and quorum :)

Added publishing to the CLI command :) Ignore the first list output :sweat_smile:

Fair enough! Is there/Will there be a way of separating out the parts of the output? I liked how the boxes separate out the diff, acceptances, rejections, and quorum :)

Yeah good question. There's no out-of-the-box way, but you can use headers like I use in patch, eg. YOU PROPOSED, with a blank line around it. There's the terminal::io::headline function which does that and bolds..

Btw I'm still of the opinion that this should be part of the patch command, as it's essentially identical, and will need all the same sub-commands. Only if we think it's sufficiently different should it merit its  own subcommand and crdt..

Fair enough! Is there/Will there be a way of separating out the parts of the output? I liked how the boxes separate out the diff, acceptances, rejections, and quorum :)

Yeah good question. There's no out-of-the-box way, but you can use headers like I use in patch, eg. YOU PROPOSED, with a blank line around it. There's the terminal::io::headline function which does that and bolds..

Oh sweet, I'll take a look :ok_hand:

Btw I'm still of the opinion that this should be part of the patch command, as it's essentially identical, and will need all the same sub-commands. Only if we think it's sufficiently different should it merit its own subcommand and crdt..

I can try playing around with that today idea today but I'm sceptical. I think they look the same but have different outcomes. We'll see :)

Btw I'm still of the opinion that this should be part of the patch command, as it's essentially identical, and will need all the same sub-commands. Only if we think it's sufficiently different should it merit its own subcommand and crdt..

I can try playing around with that today idea today but I'm sceptical. I think they look the same but have different outcomes. We'll see :)

The more I dig into this, trying to abstract out the parts that are different are pain points. I get into cases where I need some existential loading of an identity patch versus a code patch. That doesn't even get as far as changing the logic for Patch creation in the CLI. Whereas the proposal code I've written so far is lightweight and self-contained.
I'm not sure what you had in mind for using Patch, so maybe I might have gone off-piste and got lost under an avalanche of generic type trickery.
@cloudhead how do you feel about proceeding? Because I'm not sure putting more time into this is worth it without some alignment on the approach going forward :)

Ok, I'm fine with proceeding with a separate CRDT / CLI if you think doing otherwise would add too much complexity to Patch. I honestly haven't thought it through deeply, but it seemed like very similar workflows. Perhaps there are other points of convergence at a higher level, eg. if we implement code review for patches, we can re-use those UI components for identity change reviews, while keeping the backends different. I'm more interested in offering similar UX for both patch and identity than to have them implemented with the same datatypes. I wouldn't want to re-implement all our reviewing tools for both, basically.
So I'd say let's proceed with your original approach for now. I would however move the commands to rad id *, as rad proposal and rad patch are too similar.
Perhaps we could have something like rad id edit be the equivalent of rad patch open.

Sorry about the delay. Worked on the UI and have some screenshots:

I'll clean up and push what I have :)

I'm happy with where this is at. I'm going to write some test examples for the CLI and then should be good to go :+1:


succinct sklearn rubicon pipeline example

Is your feature request related to a problem? Please describe
There should be an sklearn integration example notebook that succinctly demonstrates the overall functionality
Describe the solution you'd like
Create a notebook example that demonstrates how useful the sklearn pipeline integration can be

See docs


Duplicate tag ":UltiSnipsAddFiletypes"

Thanks for a great plugin and all your work in keeping the issues down to zero! This is remarkable for a plugin of this complexity!
When updating my plugins just now, I get:
[neobundle] Error generating helptags:
[neobundle] Vim(helptags):E154: Duplicate tag ":UltiSnipsAddFiletypes" in file /home/ravi/.local/share/vim/bundle/.neobundle/doc/youcompleteme.txt
[neobundle] Update done: (2016/09/28 21:24:43)

Can NeoBundle handle it if there is a duplicate tag used in two different plugins?

I should mention that I'm also using Unite, should that make any difference.

Can NeoBundle handle it if there is a duplicate tag used in two different plugins?

The warning is Vim feature.
The plugin authors must not define duplicated tag.
I think YouCompleteMe or UltiSnips tag is wrong.


InvalidArgumentError:  Incompatible shapes: [2] vs. [8]

I encounter the error(bath size Multiplied by 4) when I try to train but when I put batch_size=1 there is no error.
basic_model = models.buildin_models("mobilenet", dropout=0, emb_shape=512, output_layer='E')
tt = Train(data_path, save_path='TT_mobilenet_base_emb512_dr0_bs400.h5', eval_paths=eval_paths,
basic_model=basic_model, model=None, lr_base=0.1, lr_decay=0.1, lr_decay_steps=[20, 30],
batch_size=2, random_status=0)
InvalidArgumentError:  Incompatible shapes: [2] vs. [8]
[[node Equal (defined at :316) ]] [Op:__inference_train_function_20481]
Function call stack:
train_function

I didn't meet that issue, the place where batch_size may multiply with 4 is the triplet case, but it's batch_size = batch_size // image_per_class * image_per_class. Can you share the full log?

I didn't meet that issue, the place where batch_size may multiply with 4 is the triplet case, but it's batch_size = batch_size // image_per_class * image_per_class. Can you share the full log?

/thanks for your reply. I solved it. was related to the size of my inputs. my other question is about to overfit or underfit, how can I check them? otherwise, how can I achieve validation loss?
because I train my own dataset without lfw format(.bin), I don't use evals.py and eval callbacks and I just test with eval_folder

/thanks for your reply. I solved it. was related to the size of my inputs. my other question is about to overfit or underfit, how can I check them? otherwise, how can I achieve validation loss?
because I train my own dataset without lfw format(.bin), I don't use evals.py and eval callbacks and I just test with eval_folder

I once thought about that, but it's not a common case. As the common used lfw / agedb_30 is created for that job, so we can compare with others implementation. The reason why they are common is that their test cases are more general.
You may create your own bin file that defines which two images are same, which are not. I think is a better way using your own evaluating data in training process.
Another way is define your own eval_callback replacing the __eval_func__ part in evals.py.

I once thought about that, but it's not a common case. As the common used lfw / agedb_30 is created for that job, so we can compare with others implementation. The reason why they are common is that their test cases are more general.
You may create your own bin file that defines which two images are same, which are not. I think is a better way using your own evaluating data in training process.
Another way is define your own eval_callback replacing the __eval_func__ part in evals.py.

thanks for your reply. I tried it and it needs to generate pairs.txt, how can I be sure that I generate it correctly? can I calculate issame_list myself to check the result of the pair.txt?(for a small dataset)

thanks for your reply. I tried it and it needs to generate pairs.txt, how can I be sure that I generate it correctly? can I calculate issame_list myself to check the result of the pair.txt?(for a small dataset)

I' m not familiar with how the pair.txt is generated, but the final bin file just need two part, [image_data] and [is_same_list]
If we have folder:
.               # dataset folder
├── 1           # person folder
│   ├── 10.jpg # face image
│   └── 11.jpg # face image
├── 2           # person folder
│   ├── 20.jpg # face image
│   └── 21.jpg # face image

Then the bin file can be:
image_data = [10_data, 11_data, 20_data, 21_data, 10_data, 20_data, 10_data, 21_data, 11_data, 20_data, 11_data, 21_data]
is_same_list = [1, 1, 0, 0, 0, 0]

And yes, you may check the is_same_list, or it will be too large. It maybe a random pick, or calculating the distance and pick the hard ones.

I' m not familiar with how the pair.txt is generated, but the final bin file just need two part, [image_data] and [is_same_list]
If we have folder:
.               # dataset folder
├── 1           # person folder
│   ├── 10.jpg # face image
│   └── 11.jpg # face image
├── 2           # person folder
│   ├── 20.jpg # face image
│   └── 21.jpg # face image

Then the bin file can be:
image_data = [10_data, 11_data, 20_data, 21_data, 10_data, 20_data, 10_data, 21_data, 11_data, 20_data, 11_data, 21_data]
is_same_list = [1, 1, 0, 0, 0, 0]

And yes, you may check the is_same_list, or it will be too large. It maybe a random pick, or calculating the distance and pick the hard ones.

thanks a lot, which accuracy do you recommend (PCA=False or True)? and what about calculating tpr or far?

thanks a lot, which accuracy do you recommend (PCA=False or True)? and what about calculating tpr or far?

The PCA accuracy is actually calculating the tpr and far, then calculates accuracy according to them.
acc = float(tp + tn) / dist.size

My eval_callback is calculating accuracy by pick each thresh value in far ∈ [0 ~ 0.1], and pick the max accuracy value.
t_steps = int(0.1 * ff.shape[0])
acc_count = np.array([(tt > vv).sum() + (ff <= vv).sum() for vv in ff[-t_steps:]])
acc_max_indx = np.argmax(acc_count)

If set PCA_acc=True, then both these two accuracy values is calculated, but they are independent.
My recommendation is... False, because it's faster. But still it's your choice.

The PCA accuracy is actually calculating the tpr and far, then calculates accuracy according to them.
acc = float(tp + tn) / dist.size

My eval_callback is calculating accuracy by pick each thresh value in far ∈ [0 ~ 0.1], and pick the max accuracy value.
t_steps = int(0.1 * ff.shape[0])
acc_count = np.array([(tt > vv).sum() + (ff <= vv).sum() for vv in ff[-t_steps:]])
acc_max_indx = np.argmax(acc_count)

If set PCA_acc=True, then both these two accuracy values is calculated, but they are independent.
My recommendation is... False, because it's faster. But still it's your choice.


Add HasData seeding example for many-to-many relationship

When using many-to-many with generated join entity, the exception we throw is not great, e.g.:
The seed entity for entity type 'Tag' cannot be added because it has the navigation 'Posts' set. To seed relationships,  add the entity seed to
'PostTag (Dictionary<string, object>)' and specify the foreign key values {'TagsTagId'}.

all these are generated by EF so it might be nice to show people how to do this (or maybe throw better exception for m2m with generated join entity?)

            modelBuilder.Entity<Post>()
                .HasMany(p => p.Tags)
                .WithMany(t => t.Posts)
                .UsingEntity<Dictionary<string, object>>(
                    "PostTag",
                    r => r.HasOne<Tag>().WithMany().HasForeignKey("TagId"),
                    l => l.HasOne<Post>().WithMany().HasForeignKey("PostId"),
                    je =>
                    {
                        je.HasKey("PostId", "TagId");
                        je.HasData(
                            new { PostId = 1, TagId = "general" },
                            new { PostId = 1, TagId = "informative" },
                            new { PostId = 2, TagId = "classic" },
                            new { PostId = 3, TagId = "opinion" },
                            new { PostId = 4, TagId = "opinion" },
                            new { PostId = 4, TagId = "informative" });
                    });

Is it possible to specify the seed data for a generated join entity separately without needing to define a CLR type for it?
I seed all my data separately from where I define my entity configurations. I've got an entity configuration class like this, defining a many-to-many relationship between ProductConsistencyRule and ProductMinMixRange via a generated join table called ProductMinMixRangeAssociation:
public class ProductConsistencyRuleConfiguration : DomainEntityConfiguration<ProductConsistencyRule>
{
    public const string ProductMinMixRangeJoinTable = "ProductMinMixRangeAssociation";

    protected override void ConfigureCore(EntityTypeBuilder<ProductConsistencyRule> builder)
    {
        builder.ToTable(nameof(ProductConsistencyRule));

        builder.Ignore(x => x.ProductMinMixRanges);
        builder.Property(x => x.CommissionBonus).HasColumnType("decimal(18,2)");

        builder
            .HasMany(pcr => pcr.ProductMinMixRanges)
            .WithMany("ProductConsistencyRules") // Workaround until https://github.com/dotnet/efcore/issues/3864 resolved
            .UsingEntity(j => j.ToTable(ProductMinMixRangeJoinTable));
    }
}

Then elsewhere I try to seed this join table using its name:
modelBuilder.Entity(ProductConsistencyRuleConfiguration.ProductMinMixRangeJoinTable)
    .HasData(productMinMixRangeAssociations);

Then the first time my DbContext is instantiated, I get this exception:

System.InvalidOperationException: 'The entity type 'ProductMinMixRangeAssociation' is in shadow state. A valid model requires all entity types to have corresponding CLR type.'


Fixing non NOSASL use of beeline

a bit of refactoring
closes #567

@mistercrunch is this entirely in line with #567? Maybe pull authMechanism and use it to set auth when available? That would allow auth=maprsasl as mentioned in the issue thread.

@bolkedebruin , here's a second take on it

@mistercrunch seems good to me!

oddly enough in my environment ... I am passing user, schema, and extra parameters to beeline.
{"use_beeline": true, "auth":"plain", "hive_cli_params": "-i /home/airflow/hiveconf.hql"}
The connection string is actually causing the trailing params/arguments to fail.
(-bash: -n: command not found)
enclosing the connection string in quotes has resolved the issue for me.
Would adding:
jdbc_url = '"{jdbc_url}"'.format(jdbc_url=jdbc_url)
following:
jdbc_url = jdbc_url.format(**locals())
be acceptable to you ? It could also be done in a single statement ... but would look a bit odd.

Moving this to a new issue.