More spelling fixes

Followup to #16095.

Merged, thanks @andersk!
I wish there were tooling that could help avoid introduction of this sort of thing; we have a lot of non-native English speakers contributing.


[SPARK-12317][SQL] Support units (m,k,g) in SQLConf

This PR is continue from previous closed PR 10314.
In this PR, SHUFFLE_TARGET_POSTSHUFFLE_INPUT_SIZE will be taken memory string conventions as input.
For example, the user can now specify 10g for SHUFFLE_TARGET_POSTSHUFFLE_INPUT_SIZE in SQLConf file.
@marmbrus @srowen : Can you help review this code changes ? Thanks.

Can one of the admins verify this patch?

Jenkins, ok to test.

Test build #2349 has started for PR 10629 at commit a37a05e.

Test build #2349 has finished for PR 10629 at commit a37a05e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

I'm going to merge this and fix the indentation myself. Thanks.

@rxin I am so sorry that I didn't reply earlier. The code passed the style check, I copied the code from the existing codes, and I thought the indentation 2 is fine. So I am not sure how to make changes. But appreciated your help. Next time, I will raise the questions more quickly.


Remove userID after deprovisioning

Description
Changes proposed in this pull request:

Removed userID after deprovisioning

Related issue(s)

/retest

/retest

/retest


merge from DataLinkDC/dlink

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

thanks


Wait async & add mirror for python

Add mirror for python (from ali) so that people in China can download it fast.

This is a great change! Thank you @taooceros


Alert dialog to match the maximum of the width of its content

The problem here is the modifier.width(IntrinsicSize.Min) we pass to AlertDialogContent, asking it to size itself to the minimum width it can. This causes (at least) two problems:

The dialog is trying to be as narrow as possible, not using the available horizontal space.
Using Modifier.widthIn on the AlertDialog doesn't do anything, but it seems like a reasonable way to control the width of a dialog on the desktop.

Ideally, we shouldn't be trying to control the width at all (which would also get rid of the crash when you put lazy content in an AlertDialog), but the problem with that is with the fillMaxWidth() here:
        buttons = {
            // TODO: move the modifiers to FlowRow when it supports a modifier parameter
            Box(Modifier.fillMaxWidth().padding(all = 8.dp)) {
                AlertDialogFlowRow(
                    mainAxisSpacing = 8.dp,
                    crossAxisSpacing = 12.dp
                ) {
                    dismissButton?.invoke()
                    confirmButton()
                }
            }
        },

Without controlling the width outside of that, it will cause the dialog to always take the maximum available horizontal space.
But without the fillMaxWidth it's hard to position the AlertDialogFlowRow correctly because we can neither pass a modifier to AlertDialogFlowRow nor control its alignment inside the Column in AlertDialogContent.
Note that all the discussion below applies to the case when the user doesn't specify a width on the AlertDialog.
There are two possible solutions I see:

Use IntrinsicSize.Max instead of IntrinsicSize.Min
Try writing a custom layout to wrap AlertDialogFlowRow in, and don't specify any width on the AlertDialogContent.

Pros of 1:
The dialog would be sized to the content's maximum preferred width, but not wider. With the 2nd solution, it would always take the width of the entire window.
Pros of 2:
Using lazy content would not crash (although we said using lazy content in an AlertDialog is not a valid use-case). With the 1st solution, it would continue crashing.
Note that both solutions would change the existing (I would argue wrong) behavior which, if there are buttons present and the user didn't specify a width, would use the width of the buttons for the size of the dialog (because their minimum width is larger than the minimum width of the text). So before, it would look like this:

and after the change:

But note that this is only if the user doesn't specify a width (or widthIn) himself.
Proposed Changes
This PR goes with the simpler solution of using IntrinsicSize.Max. I think the benefit of sizing to the preferred content width is more important that allowing lazy content.
Testing
Test: Manual testing with code like this:
@OptIn(ExperimentalMaterialApi::class)
fun main() = singleWindowApplication(
    state = WindowState(width = 500.dp, height = 400.dp),
) {
    MaterialTheme{
        AlertDialog(
            modifier = Modifier.widthIn(max = 400.dp).padding(horizontal = 30.dp),
            onDismissRequest = { },
            title = { Text("This is the title") },
            text = { Text("This is the text This is the text This is the text") },
//            dismissButton = { Button(onClick = {}){ Text("Dismiss")} },
//            confirmButton = { Button(onClick = {}){ Text("Confirm")} },
            confirmButton = {}
        )
    }
}

Issues Fixed
Fixes: https://github.com/JetBrains/compose-multiplatform/issues/2836

This PR is a lot of words for a 2-character change in the source :-)

Maybe dialogs could have a default width set, and the developer can override that default if it doesn't work?

@m-sasha Are you trying to run ./compose/scripts/testDesktop to check backward compatibility?

@m-sasha Are you trying to run ./compose/scripts/testDesktop to check backward compatibility?

I haven't, but I will now.

Maybe dialogs could have a default width set, and the developer can override that default if it doesn't work?

Maybe, but that requires a decision, and decisions are hard 😉

Added default padding of 24.dp to AlertDialog.


Remove placeholders

Bug Report
Description
The contrast of the color of placeholders with their background does not meet the minimum color contrast ratio of 4.5:1. Darkening the placeholder color increases the risk of making it appear as actual entered value. For this reason, placeholders in terra-date-input, terra-date-picker, terra-time-input should be removed and replaced with help text.
See this comment on https://github.com/cerner/terra-core/issues/2992 for guidance.
Additional Context / Screenshots

Expected Behavior

Possible Solution

Environment

Component Name and Version:
Browser Name and Version:
Node/npm Version: [e.g. Node 8/npm 5]
Webpack Version:
Operating System and version (desktop or mobile):

@ Mentions

The alteration direction below is copied from https://github.com/cerner/terra-core/issues/2992#issuecomment-651135365
Alteration directions for:

terra-date-input
terra-date-input/lib/DateInputField
terra-date-picker
terra-date-picker/lib/DatePickerField
terra-time-input
terra-date-time-picker

[ ] The strings (the localized letter abbreviation date/time format masks, e.g. ) that are provided to the text inputs and the placeholder prop/html attributes need to be removed. The Date-Input component should remove "Month" as the placeholder text in the select and the text "- Select -" should be used instead to match the form-select components.
[ ] For each of the base components (non-Field compositions), a new element that matches the styling of the help text prop in terra-form-field needs to be added, and localized letter abbreviation date/time format masks originally put in the placeholder (MM/DD/YYYY, hh:mm:ss, Month DD, YYYY)  is now given to the help text placement
[ ] For the component that do not have Field compositions (Time-Input and Date-Time-Picker) need to add Field compositions. All the Field compositions need to remove the help text of the format letter abbreviation masks in the stand-alone/imported inputs, and need to supply the text to the actual help prop.  Format notes: If no additional help text is provided, the help text is only the provided format mask e.g. MM/DD/YYYY. If additional help text is provided, the format mask should be supplied first at (MM/DD/YYYY) followed by a space, followed by the text given to the help prop.
[ ] Remove any css styles and variables for the placeholder in the input elements (exception for Date-Input which will keep the "- Select -" placeholder in the dropdown select).
[ ] Add major version Upgrade Guide that includes these comments:
TODO add the language needed for the accessibility and human factors guidance switching from consumers providing placeholders to providing more detailed help text with Field compositions.


Question about RISC-V designs

Hello！ I recently read your paper, you used 6 open source RISC-V designs to generate data. I want to know what open source data you used, can you give me a download link?

Sure, it is pulpino.
There are originally 3 designs, pulpino configured with RISCY core, pulpino configured with RISCY core and floating point extension, pulpino configured with zero-riscy core. And we implement a hierarchical flow on these 3 designs to transform some sub modules into IPs and regard them as another 3 designs. So that adds up to 6 designs.

Ok, thank you. When you carry out logical synthesis, do you directly use the rtl code in the RTL folder in the GitHub warehouse of pulpino, or do you need to download the whole pulpino project and carry out some operations before logical synthesis? I am not familiar with pulpino at present. I hope you can give me more instructions and suggestions.

You need to run ./update-ips.py first to get all ips.
Then the rtl files are in /rtl and /ips.
This issue pulp-platform/pulpino#114 may contains useful information.

Hello, thank you very much for your work. At present, I am also trying to carry out logical synthesis of RISC-V design. I have read the issue you mentioned, and it seems that some replacement operations such as sp_ram_bank are still needed. But I don't quite understand how to operate it, because I just learned logic synthesis and am not familiar with it. Could you please tell me some details about your logical synthesis of the original design? Is it better to have a detailed process? In particular, I am curious about how to set the constraints required by logical synthesis. Are there relevant instructions in pulpino? Or if it is convenient for you, could you please provide the scripts used in the synthesis.


ASMX support

Summary
Support ASMX web services in .NET 6+
Motivation and goals
We have a ASMX web service that a third party is using. This project, which was previously unknown to us, now requires a bug fix, but it depends on other libraries that we've already upgraded for .NET 6. Rolling back would be challenging at this point. Some options appear to be

Upgrade ASMX project to use .NET 6
Rewrite the API layer and convince the 3rd party to change their client
Rewrite the API layer but have it work identically to the ASMX
Copy the old code dependencies, before they were updated to .NET Core, and have the ASMX use that (resulting in some code duplication).

Risks / unknowns
I know very little about ASMX services, so I can't provide more info. I'm just trying to get out of a jam.

Supporting a whole framework like ASMX web services is out of scope, unfortunately. The System.Web adapters are more targeted at making sure specific APIs from System.Web work in ASP.NET Core and .NET Standard contexts.
In this case, if you need to upgrade an ASMX web service to .NET 6, you might want to consider using CoreWCF, though that's still a non-trivial change. System.Web adapters could be used if, in your service's logic, you need to use specific System.Web APIs that weren't previously available on .NET 6/7. This would be option 1 or 3 from your list.
Given that this service is apparently not updated very often, you may also want to consider staying on .NET Framework (which is still supported) and just using old versions of dependencies if possible or duplicating some code as described in your option 4.


Windows: Daemon broken on master

Signed-off-by: John Howard<EMAIL_ADDRESS>A recent change to use pkg\idtools causes a chown to be on the startup path of the daemon. Chown is not supported on Windows, hence the daemon would not start.

I dunno, maybe in that case makes sense to create file with _unsupported. Because that package really won't be used on windows.

I'll change it at call site instead

@lk4d4 - Think this is a better fix. I think ultimately, we'll probably want to factor out in some way the entire package from the Windows daemon, but that's a lot more invasive and for another day.

LGTM

LGTM if janky's happy

thanks @jhowardmsft ; I can figure out the longer-term option to factor out use of idtools on Windows


repo: Reorg to remove Vulkan-Headers submodule

Remove Vulkan-Headers submodule
Add CMake code to add VULKAN_HEADERS_INSTALL_DIR and
VULKAN_LOADER_INSTALL_DIR to search when locating
Vulkan headers and loader

(LOADER_REPO_ROOT still being honored for now)

Yes, it actually needs to set both HEADERS and LOADER.  I'll update it.  Thanks.


Add Microsoft.ACE.OLEDB.12.0

Tool name
Microsoft.ACE.OLEDB.12.0  Microsoft Office 12.0 Access Database Engine OLE DB Provider
Tool license
Microsoft
Add or update?

[X] Add
[ ] Update

Desired version
12.0
Approximate size
No response
Brief description of tool
As of Aug 2021 Windows-2019 image doesn't have Microsoft.ACE.OLEDB.12.0  Microsoft Office 12.0 Access Database Engine OLE DB Provider. Appreciate if you can update this Microsoft hosted agent
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[ ] Windows Server 2016
[X] Windows Server 2019

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No

Hello @sbollap1,
We will consider a possibility to add the tool.

@sbollap1 could you try to install the package using the following snippet? Does it work for you?
Install-Binary -Url "https://download.microsoft.com/download/2/4/3/24375141-E08D-4803-AB0E-10F2E3A07AAA/AccessDatabaseEngine_X64.exe" -Name "AccessDatabaseEngine_X64.exe" -ArgumentList ("/quiet", "/norestart")
(New-Object system.data.oledb.oledbenumerator).GetElements() | select SOURCES_NAME, SOURCES_DESCRIPTION

I'm going to close the issue.
Please feel free to contact us if you have any concerns.
Thank you!


Hi , my question is how to scale font ? 怎么样缩放字体

To scale everything, an option might be to expose and adjust HtmlView.pixelScale. Another option might be to enable overriding the default style sheet.
For scaling individual text, use CSS styling.


Changes for Azure Test Plan Task

Making changes to create test run with test plan information and populating manual test point information for associated test case results parsed from test result files
Testing Screenshots:

Task :

Test Plan Run Tab:

Test Tab

Test Plan Execute Tab

Test Point Pane

Test Point Execution History

Feature Flag Implementation has introduced duplicate code. Once the feature flag is rollout, the function signature would be changed and code will be cleaned up.

lgtm

Can we check what is the test run name created in case of vstest?

Can we check what is the test run name created in case of vstest?
It is different in this experience, since the test run create in vstest uses release everytime and this task would be available as part of build as well.
But, run title is something we pass as property to command. Task code has to take care of sending the correct run title. Incase not provided, we are setting it as TestRunner_TestResults_BuildNumber.


[embeddingapi][XWALK-4167] Add embeddingapi usecase for setDownloadLi…

…stener
-add embeddingapi usecase for XWalkView setDownloadListener
-override onDownloadStart to realize your own download
BUG=https://crosswalk-project.org/jira/browse/XWALK-4167
Impacted tests(approved): new 1, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 1, fail 0, block 0

[XWALK-4167] ??


fix(semantic): params in export default (function() {}) flagged as SymbolFlags::Export

The code changes in builder.rs optimize the export flag check in the SemanticBuilder implementation. Instead of setting the export flag to true for all cases except ExportDefaultDeclarationKind::ClassDeclaration(ref class), the flag is now set to true only for ExportDefaultDeclarationKind::ClassDeclaration and false for all other cases. This change improves the efficiency of the export flag check.

#4480  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite

How about this case?
export default function foo() {
}

will foo still be marked as exported?

Yeah, this is breaking default exports for interfaces an enums
export default interface Foo {}

Yeah, this is breaking default exports for interfaces.
export default interface Foo {}

Ah, Thanks you found this. I also noticed we even don't have Foo symbol. 😅


Issue running demo

I am running this repo without any code changes. After clicking the "Embed Report" I get this error:

is there a workaround for this?

We're closing GitHub issues on 01/01/2025.
Please continue to share your feedback and get support through these channels:

Feature Requests: Submit your ideas and suggestions to the Fabric Ideas Portal, where you can also vote on ideas from other developers.

Bug Reports and Technical Assistance: Visit the Fabric Developer Community Forum. Our team and community experts are ready to assist you.

Additional Support: Contact your account manager or reach out to the Fabric Support Team.

We highly value your contributions and look forward to collaborating with you to enhance our offerings based on your feedback.
Thank you,
The Power BI Embedded Team


Support for other Android manufacturers

According to stats in multiple websites, for example https://gs.statcounter.com/vendor-market-share/mobile and https://www.counterpointresearch.com/insights/global-smartphone-share/ There's other Android manufacturers that could be included like: Xiaomi, Oppo, and Vivo.
Any plans to support some of those?

At the same time, the Nexus device should probably be removed. It returns super old versions of Android.

OK, noted.


Illegal instruction exception inside nlp_create() on multiple platforms

There have been reports of recent freedv-gui builds crashing on startup inside the code brought in by #43. Example crash dump from macOS (posted to the freetel-codec2 mailing list):
Process:               FreeDV [1050]
Path:                  /Applications/FreeDV.app/Contents/MacOS/FreeDV
Identifier:            org.freedv.freedv
Version:               ??? (1.8.5)
Code Type:             X86-64 (Native)
Parent Process:        ??? [1]
Responsible:           FreeDV [1050]
User ID:               501

Date/Time:             2022-11-15 16:57:48.889 +1100
OS Version:            Mac OS X 10.15.3 (19D76)
Report Version:        12
Anonymous UUID:        E215C155-F950-4ED2-AF3A-7CB392630AD3

Time Awake Since Boot: 31000 seconds

System Integrity Protection: enabled

Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BAD_INSTRUCTION (SIGILL)
Exception Codes:       0x0000000000000001, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Illegal instruction: 4
Termination Reason:    Namespace SIGNAL, Code 0x4
Terminating Process:   exc handler [1050]

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   liblpcnetfreedv.0.4.dylib     	0x000000010512c2ce __codec2__nlp_create + 254
1   liblpcnetfreedv.0.4.dylib     	0x000000010511d16c codec2_pitch_create + 108
2   liblpcnetfreedv.0.4.dylib     	0x0000000105121b6f lpcnet_dump_create + 159
3   liblpcnetfreedv.0.4.dylib     	0x00000001051239e1 lpcnet_freedv_create + 33
4   libcodec2.1.0.dylib           	0x0000000104fc78d7 freedv_2020x_open + 1095
5   libcodec2.1.0.dylib           	0x0000000104fc1775 freedv_open_advanced + 101

and one from Linux at https://github.com/drowe67/freedv-gui/issues/292. At least with the latter, the workaround seems to be for the user to build freedv-gui/codec2/LPCNet themselves using the provided build scripts, so the issue seems to be build related and may be able to be mitigated with CMake changes.
Unfortunately I'm not able to duplicate this locally thus far and attempted debugging with libasan instrumentation doesn't seem to make it any easier to do so, either. Help along those lines would be good.

The crash seems to be happening here per the previous stack traces:
    for(i=0; i<m/DEC; i++) {
	nlp->w[i] = 0.5 - 0.5*cosf(2*PI*i/(m/DEC-1));
    }

I don't think there's any sort of buffer overflow happening here but I could be missing something. Per the mentioned PR we did change at least one definition from COMP to float, but it doesn't look like struct NLP was changed.

Yes it's a strange one.  That code hasn't changed in years but I guess you never know with C.  If the compiler builds it OK, it's hard to see how it could be an illegal instruction.  Perhaps the memory containing this code is being overwritten somehow.

I disassembled the code around that area and it seems to be using AVX2 instructions:
    0x103d712f3 <+291>: vbroadcastsd 0x4074(%rip), %ymm1       ; pitch_gain_cb + 560
    0x103d712fc <+300>: vmovaps %ymm1, 0x80(%rsp)
    0x103d71305 <+309>: vbroadcastsd 0x3992(%rip), %ymm1       ; eband5ms + 64
    0x103d7130e <+318>: vmovaps %ymm1, 0x60(%rsp)
    0x103d71314 <+324>: vbroadcastss 0x358f(%rip), %xmm1
    0x103d7131d <+333>: vmovapd %xmm1, 0x50(%rsp)
    0x103d71323 <+339>: nopw   %cs:(%rax,%rax)
    0x103d7132d <+349>: nopl   (%rax)
    0x103d71330 <+352>: vmovdqa %xmm0, 0x40(%rsp)
->  0x103d71336 <+358>: vcvtdq2pd 0x40(%rsp), %ymm0
    0x103d7133c <+364>: vmulpd 0x80(%rsp), %ymm0, %ymm0
    0x103d71345 <+373>: vdivpd 0xa0(%rsp), %ymm0, %ymm0
    0x103d7134e <+382>: vcvtpd2ps %ymm0, %xmm0
    0x103d71352 <+386>: vmovapd %xmm0, 0x20(%rsp)
    0x103d71358 <+392>: vzeroupper
    0x103d7135b <+395>: callq  0x103d746e4               ; symbol stub for: cosf

Considering that thus far, people have been able to work around the issue by building freedv-gui themselves, I wonder if we need to disable AVX2 in LPCNet for the binaries that we put out? I don't know how much of a performance hit that'd be, though.


watcher comparison

will you be comparing watcher in etcd and zk ? in a large cluster

Yeah have plans to do it, but haven't figured out the best way to measure it.

Duplicate with https://github.com/coreos/dbtester/issues/20


Possible algorithm suggestion

I just saw part of the video on twitch. Since i am not familiar with processing, i am using this issue to give a possible solution.
First i will assume that correct maths are not necessary to avoid overcomplexity. Given that, i will use this math.stackexchange answer as a base.
From it

we can derive a new equation for R = ± sqrt((L t)/π + r^2) and t!=0 for the external radius.
Second we understand that in a tape recorder, it is the reading head that drives the tape

Given that, we could have:

constant max tape length: L
variable tape length of the left supply reel : L1
variable tape length of the left supply reel : L2
constant tape thickness: th
conversion equation: R = ± sqrt((L th)/π + r^2)
the head: loop/stream

returning a certain amount of length by time
or just directly changing L1 and L2

left supply reel:

taking the stream of length and using the conversion equation to create new circles of stroke th
or just do it directly depending on L1

right take-up reel: subtracting the stream of length and doing the reverse from the supply reel.

The exact code will depend highly on the capabilities of the processing language...

Oh, this is such a great reference thank you!

This is a cool algorithm and can be implemented with not too much difficulty; I will create an alternate Version, with this algorithm in my free time.


Add arg configuration to proxyAdmin and allow name to act as artifact

At the moment [owner] is passed as the constructor argument to every custom AdminContract in the proxy option. This is not desireable since a lot of ProxyAdmin contracts will take different constructor arguments. Specifically, for my use case the OpenZeppelin ProxyAdmin has no constructor at all.
This change allows you to specify what arguments get passed to the AdminContract.
This change also adds a fall back to the artifact being the name if no artifactName is specified. So here, because no artifactName is specified, both the name and artifact is "ProxyAdmin".
For example:
    await deploy('Foo', {
        from: deployer,
        proxy: {
            proxyContract: "TransparentUpgradeableProxy",
            viaAdminContract: {
                name: "ProxyAdmin", // Use "ProxyAdmin" as the artifact as well as the name
                args: [], // Allows you to pass in custom arguments for ProxyAdmin constructor here
            },
        },
    });

Just a side note, I think the naming could be improved too. viaAdminContract is a little verbose. Just adminContract would be more consistent imo.


ImportError: No module named 'skopt'

I am using keras method for BatchAI training for my python script. Python script requires 'skopt' to be imported in that.
I am getting error message saying,
/anaconda/envs/py35/lib/python3.5/site-packages/h5py/init.py:36: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
Using TensorFlow backend.
Traceback (most recent call last):
File "/mnt/batch/tasks/shared/LS_root/mounts/afs/KerasSamples/cnn_for_Azure.py", line 27, in
import skopt
ImportError: No module named 'skopt'

You need to create your own docker image which includes skopt or setup it using jobpreparation task (see here https://github.com/Azure/BatchAI/blob/master/documentation/using-azure-cli-20.md#job-preparation-task or take a look at horovod recipe) which uses it.
e.g
"jobPreparation": {
"commandLine": "pip install scikit-optimize"
},

I'm having this same error but I think it is because my path to the package skopt is wrong. I have installed scikit-optimizer through pip and conda in my virtual environment but I am unable to load the package.


hierarchical directed layout: edge crossing

The edge crossing bug seems to be known but it still would be nice if it gets fixed or at least if one could toggle if he want "optimazation but crossing" or "no crossing but no optimization" mentioned here.
Maybe this is to some degree related to https://github.com/visjs/vis-network/issues/83.

Is it always possible to organize a hierarchy without any edges crossing? I think there will be cases where it's simply impossible in 2D space.

@Thomaash I do agree, in case of cross references between nodes, edges will have to cross each other. So there is not always a possibilty to organize a hierarchy without any edges crossing.
In some cases crossing can be avoided if "no edge crossing" is prioritised over "keeping the configured space between nodes".  (first graph of this fiddle shows what i mean http://jsfiddle.net/huwp23bz/)
This fiddle http://jsfiddle.net/m56qtkcu/ also shows "edge with node crossing" where on the top graph, node 3 overlaps with edge from 1 to  5.

I try to render a tree with hierarchical directed layout, but the edges get crossed. Would it help if it was possible to specify in the options that the graph is a tree (ie. no cross references), so that the algorithm could take a hint, while laying out the nodes?

Hi @UnicornDeluxe and @dawidchodura,
I was thinking about this and I think we should add a new option which could look like this:
interface HorizontalLayout {
  algorithm?:
    // the way it is now and would be the default
    | 'none'
    // fast O(N) and perfect for trees, will fail on multiple paths to the same node
    | 'tree'
    // some heuristics to get a reasonably good layout for a generic graph
    // no idea how to implement this ATM, maybe not a part of the first PR
    | 'best-effort'
  propNameOrGetter?:
    // name of the property that will be used for ordering the nodes
    // would default to id (just to make it stable as ids generally don't change)
    | string
    // node comparator (like Array.sort callback for given nodes descendants)
    | ((a: NodeOptions, b: NodeOptions) => number)
}

Do you think this would satisfy your needs?
Do you have any objections or improvements?
PS: If you can come up with better more descriptive names for the properties or string literals I'm all ears.
PPS: @mojoaxel do you want to join this discussion?

Looks good to me.
Concerning the "best-effort" algorithm, I guess the Reduction of the Number of Edge Crossings part of the Sugiyama-Algorithm can help.

@UnicornDeluxe were you able to achieve no edge crossing? i looked into Sugiyama, and Dagre layout implements it so i am using dagre to calculate coordinates for nodes and edges. With visjs i can set the nodes at those coordinates determined by dagre but no way to tell vsjs edges point coordinates

@deepanshu11madan no, I wasn't able to achieve no edge crossing. What do you mean by

no way to tell vsjs edges point coordinates

?
Can you share a screenshot of the resulting graph?

So dagre is a layered graph layout algorithm based on sugiyama, it calculates coordinates for nodes and control points for edges. With Visjs I can place the nodes on the coordinates generated by dagre, which works perfectly, but the edges are rendered by Visjs and they cross and overlap. I have the control points for edges generated by the dagre algorithm, but no way to tell visjs to use those array of points to draw edges instead. Does that make sense?
I do have a enhancement issue created which Thomaash plans to work on at some point. Meanwhile I was wondering if you were able to figure the edge crossing issue.

That does indeed make sense.
Have you tried placing 'fake nodes' at the mentioned control points for the edges to force visjs to draw the edges as you want?
So for example:

Dagre gives you the coordinates of nodes a, b, c and d and a control point e for the edge from b to d you can simply set a node at e and split the edge from b to d into b to e and e to d. Make the node e invisible and you kind of told vis to use this specific point to draw the edge from b to d.
Which is what you/we want. Does this work?

yeah actually that's I am doing right now, but i have edge properities like  dragging edges, edge, arrows, labels which doesn't work well, will need to dirty hacks and the corners are also not very smooth, for example , in the image below, do you know if it can be made smooth somehow, the connection point of dummy node

you can try to reduce the size of dummy nodes, but otherwise I have no idea

Wondering if any progress has been made on this issue. I am running into edge crossing problems when using vis to visualize WordNet networks. I don't have much new information to add to this issue, but could provide some further examples of networks where edge crossing occurs even though the graph can be laid out without edge crossing. Would that be helpful?

I've been utilizing this example https://visjs.github.io/vis-network/examples/network/other/configuration.html to easily recreate this issue. I've tried playing with all of the options to no avail. To add some visuals to this issue, here is an image of the result when to arrows are added, layout is set to hierarchal, sortMethod is directed, and shakeTowards: roots

By moving the nodes around manually was able to produce the ideal solution, but it'd be amazing if there was a way to achieve something similar to this without the manual process of moving nodes around

Hi @Thomaash, I was wondering if you have made any progress concerning this issue 🙃

Hi @whme, unfortunately no.

The problem is, that edge pulling force is less than node repulsion. What if node repulsion is disabled till the network is stable for the first time and edges have minimal length and then node repulsion kicks in and seperates the nodes from eachother?

@Xerusial can you elaborate on this? How did you determine that that was the problem? Do you have some sense of what would be required to code up the solution you propose?

Hi @Xerusial,
your solution makes no sense to me but, as always, that doesn't mean it's wrong (I don't understand everything). Could you open a PR so that we can discuss the code? I don't have the time to fix this myself right now but I will definitely find some time to review and hopefully merge a solution.
Thanks.

Hi
I'm struggling with this as well.
Is there a way to influence the ordering on the cross axis? Assuming I have a "UD" hierarchical layout, I could solve my "edges crossing" problem by specifying not only the level (Y-Axis) but also the cross-axis-level (X-Axis) (or vice verca for LR/RL direction)

Havinf the same issue...with sortMethod: "directed", edges get crossed on initial render. Manually, I'm able to create a graph with 0 edge crosses while preventing the hierarchy

@Thomaash
I'm dealing with this too.
I think @Xerusial is basically pointing out that the spring force can't overcome the repulsive force to un-cross the nodes in a hierarchical layout. Is it possible for nodes to "pull past" each other in a hierarchical layout if the spring force of the "parent" nodes is greater than the repulsive force between children?

@Thomaash

Is it always possible to organize a hierarchy without any edges crossing? I think there will be cases where it's simply impossible in 2D space.

You can test if the graph is planar. If so, it is possible to draw it without edge crossings (but I don't know if a hierarchical layout is always possible in planar cases).
The dot layout engine of Graphviz minimizes edge crossings very well. I think they use Sugiyama algorithm. I used it to get the coordinates of the nodes because in viz.js (JavaScript version of Graphviz created with emscripten) it's not possible to move the nodes manually. Cytoscapejs also use dagre's layout.

Subscribing to this as it can truly break the reading experience.
This 3 nodes graph is rendered flat.

https://jsfiddle.net/uadc4n05/


fix(react-simulator-renderer): detached node has children

detached node has children will return false, causing memory leaks.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhangxingxing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update workflows concurrency

Task/Issue URL: https://app.asana.com/0/488551667048375/1203827189225739/f
Description
This PR updates the workflow concurrencies
Steps to test this PR
N/A

Current dependencies on/for this PR:

develop

PR #2768   👈

This comment was auto-generated by Graphite.


<EMAIL_ADDRESS>hash check failed

Hmm, it was updated 23 days ago, so it surely isn't the cache. Regardless, follow the instructions in the README. :smile:

Hmm, it was updated 23 days ago, so it surely isn't the cache. Regardless, follow the instructions in the README. :smile:


Question: disabling remote context loading

While it is clearly part of the JSON-LD specification to fetch remote context documents (either via "@context": "https://example.com/context.json" or via @import within an existing @context), is there a mechanism for disabling this feature? It seems that JsonLdOptions.setDocumentLoader(null) may work for that.
The use case I have in mind is an LDP server that accepts JSON-LD documents but which (for security reasons) does not wish to fetch arbitrary remote resources.

that's an interesting question. Thanks. Setting document loader to null is tricky, it causes JsonLdError with error code LOADING_DOCUMENT_FAILED. To avoid that you can implement something like dummy/mock loader that always returns an empty JsonObject but in this case you might not get the right result after processing because of the missing context.  You can also pre-process a document with a loader that just collects required imports and fetch the imports outside LDP later.
Let me know if there is a solution that works for you. A dry-run mode collecting all linked documents to fetch seems to be doable as a feature.

In this case, I don't mind handling an exception (either way, I would want the response to be mapped to some sort of 4xx error to an LDP client), so the LOADING_DOCUMENT_FAILED error would be entirely workable. Basically, for uploaded JSON-LD documents, I want to put the responsibility on a client to embed the required context.
Thanks for the response

This suggestion works for me. Thanks again

You are welcome. Just an idea. A server might require JSON-LD in an expanded form, that solves the issue with loading remote documents on server side and makes the contract between a server and a client more transparent + you don't need to have JSON-LD processor on a server but client.


Error in assigning the value from a action which returns a tuple

In a scenario like  below:
var [data, i] = check self.socket->read(remaining);

"read" is an action there, and this gives the following error:
error: lafernando/socketsample:0.1.1::socketsample_connector.bal:52:35: action invocation as an expression not allowed here

The only way to workaround this is by doing something similar to the following:
var x = check self.socket->read(remaining);
var [data, i] = x;

Fixed with #19203


Use tighter formulation for energy storage assets

Description
Implement early work from Maaike's research to improve the energy storage constraints:
https://www.overleaf.com/project/6454fde7fdf7f1c73ceeb2a0
Sub issues

[ ] Write the equations in this issue using the Tulipa names for variables and constraints (translate Maaike's formulation into Tulipa)
[ ] Create the new parameters and update the input files with the new data
[ ] Create the new variables and constraints. Modify existing ones, including the objective function (if needed)
[ ] Update the output solution (if needed)
[ ] Update the documentation (concepts and tutorial/how-to)

Methods:

[ ] No binary (with/without inflows current formulation) -> suitable for hydro in cascade, and PHS is feasible charge/discharge at the same time
[ ] Binary (German's formulation covers cases with/without inflows). It implies that no spillage (as a discharge) can happen simultaneously as charging.


Collaborators cannot see LUIS application. LUIS contributors are managed in the Azure portal for the authoring resource, using the Access control (IAM) page.

LUIS App ID: e5697074-1ad5-4f10-8f1b-ee178550de68
Azure Subscription ID: 0d7e8268-8913-4a0e-bbb7-40e102be9ced
Content: Collaborate with other contributors on LUIS apps in Azure
Content Source: articles/cognitive-services/LUIS/luis-how-to-collaborate.md
Service: cognitive-services

@hshahbaz Could you mention the document you are referring to for us to assist you better ?

@hshahbaz Can you please add more details about your LUIS application. Can you please confirm Is you app migrated to azure authoring resource or not migrated yet.
How do I give collaborators access to LUIS with Azure Active Directory (Azure AD) or Role-based access control (RBAC)?
See Azure Active Directory resources and Azure Active Directory tenant user to learn how to give collaborators access.
Please follow the troubleshooting for frequently asked questions.
https://docs.microsoft.com/en-us/azure/cognitive-services/luis/troubleshooting#collaborating-and-contributing

@hshahbaz We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@ram-msft  sorry for the late reply. The app is migrated to Azure. I used the following to give access to LUIS: https://drive.google.com/open?id=1I6IZxeyp_9hL5DdQwodJ_Z0F4W5YOsLl
The user has Contribute access for LUIS authoring app but when the user logs in to LUIS he is not able to see the LUIS app.

@hshahbaz  we would recommend to raise a Azure support desk ticket from Help+Support blade from Azure portal. This will help you to share the details securely and work with an engineer who can provide more insights about the issue that if it can be replicated.


[mygui] Update to 3.4.2

updated to 3.4.2
removed ports/mygui/Use-vcpkg-sdl2.patch because upstream now performs similar step to link SDL2
removed  fix-osx.patch since it was accepted to the upstream repo
fixed wasm32-emscripten build

while 3.4.3 is already out, it's quite different in some parts. I'm working on a project that doesn't compile with 3.4.3. I guess we should accept 3.4.2 first.

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review".

note for myself: I should try linking opengl here: https://github.com/MyGUI/mygui/blob/a81a88693fa25710a85a5cd64b2481071bf87f30/Common/CMakeLists.txt
find_package(OpenGL REQUIRED)
target_link_libraries(main PRIVATE OpenGL::GL)

@MonicaLiu0311 go

An error occurred while testing features on x64-windows-static:
./vcpkg install mygui[*]:x64-windows-static

...

-- Configuring x64-windows-static
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
    Command failed: "C:/Program Files/Microsoft Visual Studio/2022/Enterprise/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" -v
    Working Directory: G:/mygui/buildtrees/mygui/x64-windows-static-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      G:\mygui\buildtrees\mygui\config-x64-windows-static-dbg-CMakeCache.txt.log
      G:\mygui\buildtrees\mygui\config-x64-windows-static-rel-CMakeCache.txt.log
      G:\mygui\buildtrees\mygui\config-x64-windows-static-out.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake:252 (vcpkg_execute_required_process)
  ports/mygui/portfile.cmake:31 (vcpkg_cmake_configure)
  scripts/ports.cmake:147 (include)

G:\mygui\buildtrees\mygui\install-x64-windows-dbg-out.log:
-- Configuring MYGUI 3.4.2
-- Found Freetype: optimized;G:/mygui/installed/x64-windows-static/lib/freetype.lib;debug;G:/mygui/installed/x64-windows-static/debug/lib/freetyped.lib (found version "2.12.1")
-- Found ZLIB: optimized;G:/mygui/installed/x64-windows-static/lib/zlib.lib;debug;G:/mygui/installed/x64-windows-static/debug/lib/zlibd.lib (found version "1.3.0")
-- Found BZip2: optimized;G:/mygui/installed/x64-windows-static/lib/bz2.lib;debug;G:/mygui/installed/x64-windows-static/debug/lib/bz2d.lib (found version "1.0.8")
-- Looking for BZ2_bzCompressInit
-- Looking for BZ2_bzCompressInit - found
-- Found PNG: optimized;G:/mygui/installed/x64-windows-static/lib/libpng16.lib;debug;G:/mygui/installed/x64-windows-static/debug/lib/libpng16d.lib (found version "1.6.39")
-- Could NOT find SDL2 (missing: SDL2_LIBRARY) (found version "2.28.3")
-- Could NOT find SDL2_image (missing: SDL2_IMAGE_LIBRARIES) (found version "2.6.3")
CMake Deprecation Warning at CMake/Dependencies.cmake:56 (cmake_policy):
  The OLD behavior for policy CMP0072 will be removed from a future version
  of CMake.

  The cmake-policies(7) manual explains that the OLD behaviors of all
  policies are deprecated and that a policy should be set to OLD only under
  specific short-term circumstances.  Projects should be ported to the NEW
  behavior and not rely on setting a policy to OLD.
Call Stack (most recent call first):
  CMakeLists.txt:189 (include)

-- Found OpenGL: opengl32  found components: OpenGL
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
--
-----------------------------------------------------------------------------
-- The following external packages were located on your system.
-- This installation will have the extra features provided by these packages.
+ freetype
+ opengl
-----------------------------------------------------------------------------
-- The following OPTIONAL packages could NOT be located on your system.
-- Consider installing them to enable more features from this software.
+ Doxygen: Tool for building API documentation <http://doxygen.org>
-----------------------------------------------------------------------------

-- Found OpenGL: opengl32
-- Configuring done (5.2s)
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
SDL2_IMAGE_LIBRARY (ADVANCED)
    linked by target "EditorFramework" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/EditorFramework
    linked by target "LayoutEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/LayoutEditor
    linked by target "SkinEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/SkinEditor
    linked by target "ImageEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/ImageEditor
    linked by target "FontEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/FontEditor
SDL2_LIBRARY (ADVANCED)
    linked by target "EditorFramework" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/EditorFramework
    linked by target "LayoutEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/LayoutEditor
    linked by target "SkinEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/SkinEditor
    linked by target "ImageEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/ImageEditor
    linked by target "FontEditor" in directory G:/mygui/buildtrees/mygui/src/MyGUI3.4.2-2a0c5de7a4.clean/Tools/FontEditor

-- Generating done (0.2s)
CMake Warning:
  Manually-specified variables were not used by the project:

    FETCHCONTENT_FULLY_DISCONNECTED
    _VCPKG_ROOT_DIR

CMake Generate step failed.  Build files cannot be regenerated correctly.

@MonicaLiu0311 fixed

All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static


(T4) Shielded transfers with Concordium Client over a longer period

confirmed:T4
address:3oEn5U41hJ4zg4AHiH6XjrLaYnvkGWZDg9vwMCksEUQA1GmXUK

confirmed:T4
address:3oEn5U41hJ4zg4AHiH6XjrLaYnvkGWZDg9vwMCksEUQA1GmXUK


Feature addition- Automated Portfolio Management Tool

I propose to implement an automated portfolio management tool that helps users manage their investments more effectively within the NexTrade platform.
Key Features:
Portfolio Tracking: Allow users to view their stock holdings, current market value, and performance over time in a comprehensive dashboard.
Automated Rebalancing: Provide users with an option for automated rebalancing based on predefined criteria (e.g., risk tolerance, target asset allocation). The system can suggest trades to keep their portfolio aligned with their goals.
Market Insights and Alerts: Integrate market analysis tools that provide insights, trends, and alerts for significant market movements or changes in stock performance, helping users make informed trading decisions.
Risk Assessment: Offer risk assessment tools that evaluate users' portfolios and suggest diversification strategies to mitigate risks based on their investment goals.
Educational Resources: Include resources, tutorials, or articles that educate users on portfolio management strategies and how to leverage the tool effectively.
Benefits:
Empowers users to make data-driven decisions while simplifying the management of their investments.
Increases user engagement by providing additional value beyond basic trading functionalities.
Attracts both novice and experienced investors by offering tailored support for various investment strategies.
This feature would enhance the overall functionality of the NexTrade platform, making it more appealing to a wider audience while providing tools for better investment management.

@Akshat111111 can you please add the necessary labels to avoid any complications in my leaderboard score.
Thank you

@Akshat111111 heyy???

Yeah, I will update all soon

Please do not forget to provide me with the necessary labels after merging my pr

yeah


Destructive Chaos Testing: Integration tests

Overview
Cover the following heal scenarios with integration tests:

Local NSMgr(r) +

none - https://github.com/networkservicemesh/deployments-k8s/pull/1789
Local Endpoint(d) - moved to https://github.com/networkservicemesh/deployments-k8s/issues/1928
Remote NSMgr(~r~/d) - moved to https://github.com/networkservicemesh/deployments-k8s/issues/1928
Remote Forwarder(d) - moved to https://github.com/networkservicemesh/deployments-k8s/issues/1928
Remote Endpoint(d) - moved to https://github.com/networkservicemesh/deployments-k8s/issues/1928

Local Forwarder(d) - https://github.com/networkservicemesh/deployments-k8s/pull/1789
Local Endpoint(~r~/d) - https://github.com/networkservicemesh/deployments-k8s/pull/1789
Remote NSMgr(~r~/d) +

none - https://github.com/networkservicemesh/deployments-k8s/pull/1789
Remote Endpoint(d) - moved to https://github.com/networkservicemesh/deployments-k8s/issues/1928

Remote Forwarder(d) - https://github.com/networkservicemesh/deployments-k8s/pull/1789
Remote Endpoint(~r~/d) - https://github.com/networkservicemesh/deployments-k8s/pull/1789
Registry(r) - https://github.com/networkservicemesh/deployments-k8s/pull/1789

Blockers

sandbox testing - https://github.com/networkservicemesh/sdk/issues/898

Single death cases implemented in #1789.
Multi death cases moved to #1928.


TypeError: can't pickle _thread.RLock objects

Traceback (most recent call last):
File "train.py", line 118, in
main()
File "train.py", line 115, in main
train_detector(model, datasets, cfg, distributed=distributed, validate=args.validate, logger=logger,)
File "/home/muyangren907/cv/SE-SSD/det3d/torchie/apis/train_sessd.py", line 299, in train_detector
model_ema = copy.deepcopy(model)
File "/usr/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/usr/lib/python3.6/copy.py", line 280, in _reconstruct
state = deepcopy(state, memo)
File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/usr/lib/python3.6/copy.py", line 240, in _deepcopy_dict
y[deepcopy(key, memo)] = deepcopy(value, memo)
File "/usr/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/usr/lib/python3.6/copy.py", line 306, in _reconstruct
value = deepcopy(value, memo)
File "/usr/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/usr/lib/python3.6/copy.py", line 280, in _reconstruct
state = deepcopy(state, memo)
File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/usr/lib/python3.6/copy.py", line 240, in _deepcopy_dict
y[deepcopy(key, memo)] = deepcopy(value, memo)
File "/usr/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/usr/lib/python3.6/copy.py", line 280, in _reconstruct
state = deepcopy(state, memo)
File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/usr/lib/python3.6/copy.py", line 240, in _deepcopy_dict
y[deepcopy(key, memo)] = deepcopy(value, memo)
File "/usr/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/usr/lib/python3.6/copy.py", line 280, in _reconstruct
state = deepcopy(state, memo)
File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/usr/lib/python3.6/copy.py", line 240, in _deepcopy_dict
y[deepcopy(key, memo)] = deepcopy(value, memo)
File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/usr/lib/python3.6/copy.py", line 215, in _deepcopy_list
append(deepcopy(a, memo))
File "/usr/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/usr/lib/python3.6/copy.py", line 280, in _reconstruct
state = deepcopy(state, memo)
File "/usr/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/usr/lib/python3.6/copy.py", line 240, in _deepcopy_dict
y[deepcopy(key, memo)] = deepcopy(value, memo)
File "/usr/lib/python3.6/copy.py", line 169, in deepcopy
rv = reductor(4)
TypeError: can't pickle _thread.RLock objects

I same problem :
CUDA Version 10.0.130
cudnn : 7501
pytorch : 1.1.0
python: 3.6.13

请问问题有解决吗，

I met the same problem. Anyone solved?


It doesn't do nothing

I makethe first lines:
api = DerivAPI(app_id=app_id)
print (api)

Get the object:
<deriv_api.deriv_api.DerivAPI object at 0x0000022AF7504280>
And nothing more happens!
I am using simple_bot4.py from samples folder

Sure, I know it's an object
I solved it doing a downgrade to python 3.9.13


Cells do not deselect when scroll view delegate is set for ListStackedSectionController adapter

Hi, I am using a ListStackedSectionController. One of the section controllers has multiple items. These can be selected, but there should only be one selected at a time (the others should deselect on new item selection). This works fine, until I set adapter.scrollViewDelegate = self. For some reason, it makes the collection view act as if allowsMultipleSelection is set to true. Even if I explicitly set false to allowsMultipleSelection it still allows multiple selection of items within the section controller. This has got to be a bug?

@yusuftor sounds really bizarre. Could you send us a sample app reproducing the bug? Then I can figure out what’s going on.
Sent with GitHawk

I think it'll be quicker if I just tell you how to reproduce it using the IGListKitExamples app, it's quite an easy one to reproduce:

In StackedViewController add adapter.scrollViewDelegate = self to viewDidLoad()

Add the scrollViewDelegate extension:

extension StackedViewController : UIScrollViewDelegate {
    func scrollViewDidScroll(_ scrollView: UIScrollView) {}
}

In DisplaySectionController add this code:

override func didSelectItem(at index: Int) {
        print("select index", index)
    }

override func didDeselectItem(at index: Int) {
    print("deselect index", index)
}

Now run the app and go to Stacked Section Controllers and click on some of the items like the ones called "Section 0, Cell 0" etc. You'll see that didSelectItem is always called but didDeselectItem isn't. Now remove the line adapter.scrollViewDelegate = self and try it again. You should see didDeselectItem starts to work! Hope this is an easy fix!

No worries, your wish is my command! I have set it up as stated above IGListKit-master-2.zip

Ok I can reproduce the behavior. Should note that it happens with any section controller, not just stacked. You can also repro this by setting the collectionViewDelegate.
This is a real bug! My bug in fact, introduced in #853 😅
The proxy checks for didSelect Selectors but not deselect. Will get a PR up with this.

Hello, is there any timeline for 3.3.0?

@ajanuar not too long! Wrapping up a few more PRs and internal projects first.

Hey, is this solved? I am still facing this.


Value of constant PHI is wrong

The value of phi should be about 1.61803399. The value in Constants.java is about 2.11803399.

Corrected the calculation and added an extra unit test (phi == 1 + 1/phi).


[bug]: Can't run web UI, model is a directory?

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
No response
What happened?
I'm trying to run the web interface via Google Colab.
After entering
/content# cd InvokeAI
/content/InvokeAI# python scripts/invoke.py --web

I get the error you can see in the screenshot. It looks like it copies the model to the wrong place and then fails to find it. It did say the path was correct at the step where you enter the path.
Screenshots

Additional context
No response
Contact Details
No response

CLI is supported and that is what is in the instructions.
Our UI is requires you to run an HTTP/websocket server; it is against the TOS to run a server on Google Colab.

Ah, I see. Thanks for the explanation.


[Fix #11191] Add new Lint/NumericOperationWithConstantResult cop

Fixes #11191.
Modifications were made to Lint/BinaryOperatorWithIdenticalOperands in order to stop it from acting on arithmetic operations, as to avoid doubling up.
At any rate, Lint/BinaryOperatorWithIdenticalOperands would benefit from a rework at some point to support autocorrect, so limit the extent of that task anyways.

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good to me overall, but I left a couple of small remarks.

Thanks for the review! I've addressed both of your comments.


Java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.

Make sure to check the demo app(s) for sample usage
Make sure to check the existing issues in this repository
If the demo apps cannot help and there is no issue for your problem, tell us about it
Please, ensure your title is less than 63 characters long and starts with a capital
letter.
Trying to connect to some endpoints with SSL results in this error:
java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.

Apparently this error can happen when SSL certs are configured incorrectly, but I am very confident mine are correct, and it does the same thing with google.com which I think we can assume is correct.
Which platform(s) does your issue occur on?

iOS/Android/Both Android
iOS/Android versions 11
emulator or device. What type of device? Both

Please, provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it) 8.0.1
Cross-platform modules: (check the 'version' attribute in the
node_modules/tns-core-modules/package.json file in your project) 6.5.27
Runtime(s): (look for the "tns-android" and "tns-ios" properties in the package.json file of your project) "@nativescript/android": "8.0.0"
Plugin(s): (look for the version numbers in the package.json file of your
project and paste your dependencies and devDependencies here)

default dependencies from a new project with ns create
Please, tell us how to recreate the issue in as much detail as possible.
Describe the steps to reproduce it.
Try to connect to an SSE endpoint with SSL/https. Some endpoints work fine and some don't. I can't find a correlation between any certificate configuration or TLS version/cipher. Google.com causes the error, Yahoo.com works as expected (Not event stream error)
.
Is there any code involved?

provide a code example to recreate the problem
(EVEN BETTER) provide a .zip with application or refer to a repository with application where the problem is reproducible.

// this throws the error
let sse = new SSE('https://google.com');

Here is a sample project that shows the problem:
ns-sse-sample2.zip

unzip
npm install
change the "events" property from "protected" to "public" in node_modules/nativescript-sse/sse.common.d.ts to avoid compiler errors
ns run android

This is caused by the underlying native library (https://github.com/tylerjroach/eventsource-android) obtaining the certificate from the IP address and not the domain name.
Putting https://<IP address for google.com> in your browser will result in a placeholder certificate containing the message "No SNI provided; please fix your client." This does not happen for yahoo.com.


🐞 [BUG] - Easy Navigation

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
Currently, The index section for every category doesn't work. The Redirection links which were made for easy navigation doesn't link well with the website.

Expected behavior
The Table of content section for every category should navigate to their corresponding section. Currently it contains broken links that should be change to working link that will provide user easy navigation.
Steps to reproduce
No response
Additional Context
No response

I can dot it please assign it to me. I have more than 2 years of experience.


feat(db): add search to reth db list command

Adds search to reth db list with --search.

--search 0xff will search for [0xff]
--search ff will search for [0x66, 0x66]
search is done only over RawKey<T> and RawValue<T>, and so, it might miss certain rows on compressed tables

Adds --count to just count the number of results without actually decoding/decompressing anything

good to merge @joshieDo ?


Add support for JWT authentication against SalesForce

Followed the provided authentication mechanism to now work with JWT and key.
Added tests for the new auth method.
Fixed existing failing tests.

@g8rswimmer Hey, I cannot seem to add you as a reviewer to this so I figured I should just mention you here. Cheers!

@g8rswimmer I've addressed your comments. Feel free to, please, review :) Cheers!

@g8rswimmer Hey! Would you mind smashing a release for this please?


Question about macro variable in nested code

Hi,
I run the code using call execute. The 1st code run occurs errors, because  dset_id variable has the value 0. But, 2nd run has no issue.
How can I fix it?
Thank you in advace.
MinJi

Hi, which program are you running and can you show the log, with mprint, mlogic, symbolgen.

Hi, which program are you running and can you show the log, with mprint, mlogic, symbolgen.

Hi, when I ran write_datasetjson.sas and execute  %write_json_metadata_array(work.column_metadata), I got the error.

dset_id is created opening the dataset. Without a full log it is hard for me to know what is going on.

Hi, I resolved the error. Thanks.


Remove case for old daemons in integration testing

Now that Docker versions 1.9-1.11 have been removed from the CI, this special case is no longer needed.
Signed-off-by: Nishant Totla<EMAIL_ADDRESS>

Test failure related to https://github.com/docker/swarm/pull/2765


styles not building Invalid option: --sourcemap=none

styles weren't building and upon digging it turned out to be https://github.com/sindresorhus/gulp-ruby-sass/issues/192
There is a simple fix suggested (worked for me) for it which can be committed to even this repo. Though it requires to execute bundle install which I'm not sure if it is OK to add.

Thanks for your this issue, we will look into it!

Hi @ismusidhu. This option was initially inserted to avoid a bug in the gulp-ruby-sass package as described here https://github.com/mwaylabs/generator-m/blob/master/app/templates/gulp_tasks/injecting.js#L44. Obviously this workaround causes problems with some versions of Sass.
Which version of Sass are you using? In case you're not sure how to retrieve the version: run sass -v in the terminal.

I saw the related issue. libsass might help. I saw one other generator asking me to choose between these two. I think i was gulp-angular. You night want to have a look at it.

Sorry tapped wrong button by mistake.

We're probably going to go to with the libsass/nodesass/gulp-sass stack in the next release. We're not going to offer both because:

we want to get rid of the ruby dependency to reduce the number of hurdles and thus make it easier to use for beginners. See https://github.com/mwaylabs/generator-m/issues/3.
we won't be able to update to gulp-ruby-sass v1.0.x because their new implementation doesn't go well with our project structure. See https://github.com/mwaylabs/generator-m/issues/199.

I'm gonna close this issue because it's covered in the two I mentioned. Is that ok for you?


Python: Unable to debug from remote

Product and Version: VSCode 1.39.2
OS Version: Client - macOS 10.15, Server/Host - Windows Server 2016 version 1607
Live Share Extension Version: 1.0.1123
Target Platform or Language: Python
Steps to Reproduce / Scenario:

Start a python project
Start live share in said project
Join client into live share
Note that selected interpreter is set to 'none'
Set interpreter in client
Interpreter will be set to 'none' in server instead.

The same happens when you change the interpreter in the server. You cannot debug from the client in any case. If you don't have interpreter set, it will return an error saying you need to select interpreter. If you select interpreter and try to debug, the server will respond with "Request DebuggerHostService.launchDebugSession failed with message: Failed to launch debug configuration:Python: Current File"
Logs_client.zip
Logs_server.zip

Closing this as resolved, since we haven't heard reports of this issue for a while. Please re-open it if you're continuing to encounter this bug. Thanks!


Rook fixes

What this PR does / why we need it:
The rook-ceph-toolsbox image change and CrashLoops if created with the wrong Deployment. Pulling the latest version.
Also, the deploy script is not idempotent and may fail with:
error: 'owner' already has a value (operator), and --overwrite is false

Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #
N/A
Public facing documentation PR (if applicable)

N/A
Special notes for reviewer:
Checklist:

[X] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[X] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] requires draining and/or replacing nodes.
[ ] will break the cluster.
I.e. full cluster migration is required.

@Xartos I addressed all your comments, specifically:

made container image stable
added resource requests
removed comment after namespace

I also added a comment to clearly document origin of file and what changes we made. PTAL.


Add a SKIPPED result status for stages

A bit tricky what's going on here.
First Jenkins on its own does not differentiate in the UI a failed stage in a pipeline from a stage that has been skipped because of an earlier failure.
For example in this job, only the first stage failed but all the others are marked as failed:
While that's how jobs status are stored by Jenkins, BlueOcean is able to mark those other jobs as "skipped", see:

Currently skipped jobs are marked as failed in Datadog, this PR introduces a new tag value "result:SKIPPED" for those jobs

Fix for https://github.com/jenkinsci/datadog-plugin/issues/146

This is great.  Thanks.

Test failure is a windows flake on the jenkins-owned CI. Confirmed working as expected in a lab.


Fix parsing of equations

Some {{Formel|....}} aren't parsed properly. This bug seem to be present for equations inside a box like in the code
{{:Mathe für Nicht-Freaks: Vorlage:Definition
   |definition={{Formel|<math>x^2</math>}}
}}

fixed in c29fc3f032ba13eec38bdfb165277c9b710a1c03.


Add test to check kubevirt disk perms

Signed-off-by: Kedar Bidarkar<EMAIL_ADDRESS>
What this PR does / why we need it: Add test to check kubevirt disk perms
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer: Add test to check kubevirt disk perms
Release note:

NONE

/retest

/approve
Thanks! Once someone adds lgtm and the tests pass, it will be merged.


Release Version 2.5.0

Release Version 2.5.0
This is a component issue for 2.5.0.
Coming from opensearch-build#2908. Please follow the following checklist.
Please refer to the DATES / CAMPAIGNS in that post.
How to use this issue

This Component Release Issue
This issue captures the state of the OpenSearch release, on component/plugin level; its assignee is responsible for driving the release. Please contact them or @mention them on this issue for help.
Any release related work can be linked to this issue or added as comments to create visiblity into the release status.
Release Steps
There are several steps to the release process; these steps are completed as the whole component release and components that are behind present risk to the release. The component owner resolves the tasks in this issue and communicate with the overall release owner to make sure each component are moving along as expected.
Steps have completion dates for coordinating efforts between the components of a release; components can start as soon as they are ready far in advance of a future release. The most current set of dates is on the overall release issue linked at the top of this issue.
The Overall Release Issue
Linked at the top of this issue, the overall release issue captures the state of the entire OpenSearch release including references to this issue, the release owner which is the assignee is responsible for communicating the release status broadly.  Please contact them or @mention them on that issue for help.
What should I do if my plugin isn't making any changes?
If including changes in this release, increment the version on 2.5 branch to 2.5.0 for Min/Core, and <IP_ADDRESS> for components. Otherwise, keep the version number unchanged for both.

Preparation

[x] Assign this issue to a release owner.
[x] Finalize scope and feature set and update the Public Roadmap.
[x] All the tasks in this issue have been reviewed by the release owner.
[x] Create, update, triage and label all features and issues targeted for this release with v2.5.0.
[x] Cut 2.5 branch

CI/CD

[x] All code changes for 2.5.0 are complete.
[x] Ensure working and passing CI.
[x] Check that this repo is included in the distribution manifest.

Pre-Release

[x] Update to the 2.5.0 release branch in the distribution manifest.
[x] Increment the version on the parent branch to the next development iteration.
[x] Gather, review and publish release notes following the rules and back port it to the release branch.git-release-notes may be used to generate release notes from your commit history.
[x] Confirm that all changes for 2.5.0 have been merged.
[ ] Add this repo to the manifest for the next developer iteration.

Release Testing

[ ] Find/fix bugs using latest tarball and docker image provided in parent release issue and update the release notes if necessary.
[ ] Code Complete: Test within the distribution, ensuring integration, backwards compatibility, and performance tests pass.
[ ] Sanity Testing: Sanity testing and fixing of critical issues found.
[ ] File issues for all intermittent test failures.

Release

[ ] Complete documentation.
[ ] Verify all issued labeled for this release are closed or labelled for the next release.

Post Release

[ ] Prepare for an eventual security fix development iteration by incrementing the version on the release branch to the next eventual patch version.
[ ] Add this repo to the manifest of the next eventual security patch version.
[ ] Suggest improvements to this template.
[ ] Conduct a retrospective, and publish its results.

Performance Results
Run on sift 1M data set
Query latency

Ingest took time

Conclusions
It appears faiss/nmslib query latencies have increased. Will investigate to see if this is noise or indicative of something more.
Raw Results
lucene-hnsw
{
  "metadata": {
    "test_name": "index-workflow",
    "test_id": "Index workflow",
    "date": "01/14/2023 00:38:56",
    "python_version": "3.8.15 (default, Nov  4 2022, 20:59:55) \n[GCC 11.2.0]",
    "os_version": "Linux-5.10.147-133.644.amzn2.x86_64-x86_64-with-glibc2.17",
    "processor": "x86_64, 36 cores",
    "memory": "388161536 (used) /<PHONE_NUMBER>4 (available) /<PHONE_NUMBER>0 (total)"
  },
  "results": {
    "test_took":<PHONE_NUMBER>7269997,
    "delete_index_took_total":<PHONE_NUMBER>999747,
    "create_index_took_total":<PHONE_NUMBER>9992457,
    "ingest_took_total":<PHONE_NUMBER>7500006,
    "refresh_index_store_kb_total": 2399878.1208984377,
    "refresh_index_took_total":<PHONE_NUMBER>999972,
    "query_took_total": 89300.5,
    "query_took_p50": 9.0,
    "query_took_p90": 10.3,
    "query_took_p99": 11.3,
    "query_memory_kb_total": 0.0,
    "query_recall@K_total": 0.9996663,
    "query_recall@1_total": 1.0
  }
}

nmslib-hnsw
{
  "metadata": {
    "test_name": "index-workflow",
    "test_id": "Index workflow",
    "date": "01/17/2023 18:21:49",
    "python_version": "3.8.15 (default, Nov  4 2022, 20:59:55) \n[GCC 11.2.0]",
    "os_version": "Linux-5.10.147-133.644.amzn2.x86_64-x86_64-with-glibc2.17",
    "processor": "x86_64, 36 cores",
    "memory": "518090752 (used) /<PHONE_NUMBER>6 (available) /<PHONE_NUMBER>0 (total)"
  },
  "results": {
    "test_took":<PHONE_NUMBER>4252457,
    "delete_index_took_total":<PHONE_NUMBER>02231,
    "create_index_took_total":<PHONE_NUMBER>1274978,
    "ingest_took_total": 57379.42679281114,
    "refresh_index_store_kb_total": 2974269.7612304688,
    "refresh_index_took_total": 14803.044791298453,
    "query_took_total": 59873.5,
    "query_took_p50": 6.0,
    "query_took_p90": 6.7,
    "query_took_p99": 8.0,
    "query_memory_kb_total": 1305907.3,
    "query_recall@K_total": 0.9997376000000001,
    "query_recall@1_total": 1.0
  }
}

faiss-ivf
{
  "metadata": {
    "test_name": "index-workflow",
    "test_id": "index workflow",
    "date": "01/17/2023 19:32:11",
    "python_version": "3.8.15 (default, Nov  4 2022, 20:59:55) \n[GCC 11.2.0]",
    "os_version": "Linux-5.10.147-133.644.amzn2.x86_64-x86_64-with-glibc2.17",
    "processor": "x86_64, 36 cores",
    "memory": "519663616 (used) /<PHONE_NUMBER>8 (available) /<PHONE_NUMBER>0 (total)"
  },
  "results": {
    "test_took":<PHONE_NUMBER>0917127,
    "delete_model_took_total": 76.34547710767947,
    "delete_index_took_total":<PHONE_NUMBER>958299,
    "train_model_took_total": 1565.2957786049228,
    "create_index_took_total":<PHONE_NUMBER>710211,
    "ingest_took_total": 55251.11392940162,
    "refresh_index_store_kb_total": 2474974.2994140624,
    "refresh_index_took_total":<PHONE_NUMBER>901834,
    "query_took_total": 89507.2,
    "query_took_p50": 8.9,
    "query_took_p90": 9.2,
    "query_took_p99": 12.6,
    "query_memory_kb_total": 1074782.6,
    "query_recall@K_total": 0.9471395000000001,
    "query_recall@1_total": 1.0
  }
}

faiss-ivfpq
{
  "metadata": {
    "test_name": "index-workflow",
    "test_id": "index workflow",
    "date": "01/17/2023 21:57:20",
    "python_version": "3.8.15 (default, Nov  4 2022, 20:59:55) \n[GCC 11.2.0]",
    "os_version": "Linux-5.10.147-133.644.amzn2.x86_64-x86_64-with-glibc2.17",
    "processor": "x86_64, 36 cores",
    "memory": "526671872 (used) /<PHONE_NUMBER>6 (available) /<PHONE_NUMBER>0 (total)"
  },
  "results": {
    "test_took":<PHONE_NUMBER>7730983,
    "delete_model_took_total": 74.28844840615056,
    "delete_index_took_total":<PHONE_NUMBER>997673,
    "train_model_took_total": 54554.71392819891,
    "create_index_took_total":<PHONE_NUMBER>0031134,
    "ingest_took_total": 54378.95677300403,
    "refresh_index_store_kb_total": 1723524.0901367187,
    "refresh_index_took_total": 1824.3272777006496,
    "query_took_total": 85335.7,
    "query_took_p50": 8.3,
    "query_took_p90": 9.7,
    "query_took_p99": 12.2,
    "query_memory_kb_total": 146438.9,
    "query_recall@K_total": 0.613657,
    "query_recall@1_total": 0.9942999999999997
  }
}

After some more tests, I do not believe this is a true degradation. The source of the discrepancy is that the 2.5 cluster produces more segments than 2.4 cluster with a refresh interval of 1s.
To prove this, I ran the indexing workload and tracked how many segments are produced for nmslib-hnsw and faiss-ivfpq:
| 2.4 | 2.5
-- | -- | --
nmslib hnsw segment count - test 1 | 243 | 295
nmslib hnsw segment count - test 2 | 247 | 267
nmslib hnsw segment count - test 3 | 237 | 284
nmslib hnsw segment count - avg | 242.33333 | 282
faiss ivfpq segment count - test 1 | 368 | 446
faiss ivfpq segment count - test 2 | 406 | 489
faiss ivfpq segment count - test 3 | 384 | 459
faiss ivfpq segment count - avg | 386 | 464.66667
From here, I ran a set of experiments where the refresh interval is set to 30s. This produces the following graph:

Its unclear at the moment what is causing more segments to be created with 2.5 than 2.4. Potentially, segment creation is faster in 2.5, allowing the 1s refresh interval to be followed closer than 2.4 (i.e. faster indexing prevents segment creation log jam). Additionally, for future releases, we need to figure out how to prevent discrepancies based on segment count in our performance tests. We can take this as an AI.

for future releases, we need to figure out how to prevent discrepancies based on segment count in our performance tests. We can take this as an AI.

Can we add force merge segments steps in the perf tool and put a fixed value for segment count to remove this discrepency?


Customizable Usernote Categories

Instead of the default categories such as "Spam Watch", "Spam Warning", etc. allow the names of the categories to be set in the subreddit's toolbox configuration.

This has been requested before.  It's on our radar, but unlikely to happen anytime soon.
The biggest issue is the note dialogue.  How does it work if a sub has 20?  The second is color selection.

I don't think he's requesting to make a neverending list of new categories. The request is to be able to define existing categories.

Concatenate.

Cleanup, cleanup...


Render Tuple and ValueTuple using ITuple

When .NET Standard includes the ITuple interface, we should use that.
ITuple also has the benefit of supporting more than 8 elements, which is currently not supported.
(Continuation of #36.)

Closing this. I don't think it relevant to target.NET Standard 2.1.

.NET Framework will never be modified to support later versions of .NET Standard -- https://devblogs.microsoft.com/dotnet/update-on-net-standard-adoption/.


fix: https://github.com/ApeWorX/ape-vyper/issues/17 line 95

fixed as per https://github.com/ApeWorX/ape-vyper/issues/17#issuecomment-876689353 & tested working locally

@fubuloubu do I need to add that contract in the test_integration.py as well?

@fubuloubu do I need to add that contract in the test_integration.py as well?

yup, although it should still just compile and trip the bug as well.
looks like we are seeing the issue when it's unspecified of which version to use!

to recreate this error I needed to uninstall my vyper versions
https://github.com/ApeWorX/ape-vyper/pull/19/checks?check_run_id=3025282666#step:5:56


No events propagate properly to LUISlider, LUICheckbox, etc.

Most widgets in Elements.py seem to have broken amist recent changes to Panda3D; No events arrive to them.
main.py in /Builtin can be ran to quickly notice this.
I tried to tweak LUISlider back to work, but I just can't figure what I'm doing wrong with it -- LUIButton, LUISpriteButton, LUILabel, still work OK.

This is because the controls have not been reworked yet, so they do not work with the new event system.
With the new system, only solid elements recieve and trigger events. You can make an element solid by specifying it in the constructor (the solid parameter).
The controls with demos have been reworked yet, I did not have time to rework the other controls yet :) It takes some time because I also clean up and comment the source code when I rework a control.
Am 04.03.2015 16:33 schrieb tzaeru notifications@github.com:Most widgets in Elements.py seem to have broken amist recent changes to Panda3D; No events arrive to them.
main.py in /Builtin can be ran to quickly notice this.
I tried to tweak LUISlider back to work, but I just can't figure what I'm doing wrong with it -- LUIButton, LUISpriteButton, LUILabel, still work OK.
—Reply to this email directly or view it on GitHub.

Oh, it's the solid part. =D
Thanks. I was trying -everything- to fix a few widgets! Got Slider to work now, I'll prolly push on the weekend some changes..
In other news -- I did somekinda hierarchical tree-list widget. The basics work ok, still need to add automatic sorting and somekinda feature to automatically show an optional cross / minus sign when a branch is open/closed. The underlaying LUI parent-child and layout thingy are pretty great and made it an easy widget to add.


Invalid rotation... again

Hey. Got "Invalid rotation" for MCH and some other classes again. Why is that? I waited 2 days, but its still not fixed. i am on 2.8.0.

Hey. Got "Invalid rotation" for MCH and some other classes again. Why is that? I waited 2 days, but its still not fixed. i am on 2.8.0.

--
2.8.0 is currently bugged and the author is away on a trip. You can fix the issue locally for now by reverting to a previous version of the plugin from the releases page. In case you're unsure how to do this, I'll give you a brief outline:
-> disable the rotations, and auto-update rotations within the plugin while in-game, then quit the game
-> download the releases and rotations files from here and here.
-> Go to C:/Users/[Your Username Here]/AppData/Roaming/XIVLauncher/installedPlugins/RotationSolver and copy/paste in the RotationSolver files from the first link.
-> Now go to C:/Users/[Your Username Here]/AppData/Roaming/XIVLauncher/pluginConfigs/RotationSolver/Rotations and copy/paste in the rotations files from the second link.
You can now load up the game and carry on playing with RotationSolver until a fix comes out. If you run into issues while trying to do this, I suggest you join the project discord server here and you can receive help there; or if you would just like to stay up to date with the project.
I hope that this helps, and I hope you have a good day :)


Allow multiple attribute values for a single attribute name

Resolves #23

Nice, this is exactly what I came here looking for...
Going to check out this PR and give it a go :)

Sorry it took me so long to merge this, I missed it somehow


Build and publish image for test

Build docker image with web server with postgresql database and redis cache for tests.

https://github.com/AinaArd/redis-simple-app


CelebA setting

Hello, thank you for your generous sharing.
I did not find anything about CelebA in the code. Hope to confirm your experimental settings. Taking heavy_makeup as the target and gender as bias as an example, the number of samples of the four combinations of (heavy_makeup, gender)  in the valid set is 3667, 8449, 7742, 9. Does the article say that taking the same number of samples is to select only 9 samples in each case?
I use ResNet18 to train the baseline model to classify heavy_makeup. The correct classification numbers for the four combinations of (heavy_makeup, gender) are 2829/3667, 8444/8449, 6998/7742, 4/9. The result is significantly higher than the baseline in the article. So I want to confirm if I have misunderstood the description of the experimental setup in the article.

Hello. Yes you are correct, taking the smaller class size (9 for heavy makeup and 182 for blond_hair) for every attribute pair is how we interpreted the original setup as well (which is taken from https://arxiv.org/pdf/2007.02561.pdf, including the baseline results)

Thank you so much for the quick and kind reply.
I checked the code of LfF. It seems that they calculate Acc. separately for each combination, and then take the take the average. In principle, it is consistent with the practice you described.
Whether such a setting will over amplify the impact of the combination with the least number of samples, because it seems to be much lower than other combinations. I'd like to hear your thoughts on this issue.

Yes, I do agree that such a small size is not ideal and can lead to a greater variability. You could definitely use the full valid set to obtain more precise results, or even build a new test (however you'd need to re-test all of the techniques)


Implement support for the delay_template in scripts

Description:
In #16401, I described a use case for wanting to create a delay with a dynamic amount of milliseconds. @balloob suggested that using the dict approach to building a delay would make sense. Unfortunately, the delay does not allow a template dictionary to be used.
I've implemented support for a delay_template in scripts (and I believe it works in automations as well).
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
I'll make the documentation changes once this approach is validated.
Simple example for configuration.yaml (if applicable):
input_number:
 random_color_loop_transition_speed:
    name: Transition Time
    initial: 1.5
    min: 1
    max: 10
    step: 0.5

script:

  random_color_loop:
    sequence:
      - service: script.turn_off
        entity_id: script.random_color_loop_control
      - delay_template:
          milliseconds: "{{ (((states('input_number.random_color_loop_transition_speed') | float) / 2) * 1000) | int }}"
      - service: script.random_color_loop_control

  random_color_loop_control:
    sequence:
      - service: script.turn_off
        entity_id: script.random_color_loop
      - delay_template:
          milliseconds: "{{ (((states('input_number.random_color_loop_transition_speed') | float) / 2) * 1000) | int }}"
      - service: script.random_color_loop

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Delay already supports templates https://www.home-assistant.io/docs/scripts/#delay

Just a note, delaying for milliseconds is not going to work like you expect because it is only checked once per second whether your delay has expired.
If you need millisecond delays, helping with home-assistant/architecture#50 is probably a productive use of time.


try adding UTF-8 BOM on sha miss

fix https://github.com/microsoft/pxt-arcade/issues/1999
This seems to behave fine to me? Causes the project to load properly (both the one from the issue and the other one I ran into today), and doesn't show the files in the diff. I suppose we'd need to check for utf16 big / little endian BOM as well if we wanted to be fully thorough, but notepad produces UTF-8 + BOM which I believe was the issue here / before as well?

Right, I was thinking about adding the BOM but wasn't certain how the rest of the app would handle it; it doesn't show in the diffs on our end as we just compare against the text, though when the user makes the first commit after pulling will clear all the BOMs / list them as changed in the log if that's a concern (Potentially for merge conflicts?):
https://github.com/jwunderl/test-bom-clone/commit/e78f50db29e26d2c27e1d36ae0c1057d5d8f74c4

As long as our merge algo is smart about those BOM changes we're fine. Did you try this any this? This is a cool fix.

Okay, tested out a few merges and looks like we handle everything nicely that I saw: https://github.com/jwunderl/test-bom-clone
As far as I can tell, any place that we merge things we will already be comparing with the text on the tree, which wouldn't have the bom anyways; so as is, we'll be handling any merge with a bom by just stripping it. This seems fine / like the right behavior to make sure everything works in the editor to me~


Failed tests

Hi,
Here are some logs after executing flutter test .\test\test_main.dart
  Expected: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, ...]
    Actual: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, ...]
     Which: at location [2868] is <13> instead of <9>
  blend mode BlendMode.dst compact true File: 'test/reference_images/blend/BlendMode.dst.png' differ

  package:matcher                                     expect
  package:flutter_test/src/widget_tester.dart 458:18  expect
  test\test_main.dart 193:5                           checkRendered

And
  Expected: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, ...]
    Actual: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, ...]
     Which: at location [14896] is <203> instead of <202>
  AVD => .si: File: 'test/old_avd_tests/avd\issue_13_forgiving_1_avd.xml' and File: 'test/reference_images/avd/issue_13_forgiving_1_avd.png' differ

  package:matcher                                     expect
  package:flutter_test/src/widget_tester.dart 458:18  expect
  test\test_main.dart 193:5                           checkRendered

See the new https://github.com/zathras/jovial_svg/blob/main/test/README.md -- the regression tests aren't expected to work on other platforms, but they're still useful, as outlined in the document.


Add Eigenmode option

Close #2043

I am confused. How is it a fix to say I can't use assign_symmetry with Eigenmodes? Can I create my modal as modal, implement the boundary conditions, and then switch to Eogenmode?

Two things.

You can have single iitem in the ID list if you enclose it in brackets:
aedtapp.assign_symmetry([face_id1], symmetry_name=sym_name1, is_perfect_e=False)

Open the design as modal, then, after the boundaries are set, switch to Eigenmode:
pyaedt.Hfss(solution_type="Eigenmode")

Hi @jpotter,
The problem is fixed but it was merged into the main code, which is not the same as the one released on pip.
Basically, when you pip install pyaedt, you are installing the latest version released:
https://github.com/pyansys/pyaedt/releases/tag/v0.6.30
If you want to test a new feature/fixed bug, you need to clone the repository instead of using pip install. By the way, if you update the pyaedt version right now (v0.6.30) it should work with Eigenmode. Please try.

Samuel,
Are we talking about a reinstall using a batch file?
I'm not sure I know what cloning the repository
means. Meanwhile, I have figured out a work
around. I create the model in Modal. Then I add
the boundary conditions and switch it to Eigenmode.
Thanks for your help in figuring this out. It was pretty frustrating.
Best regards,
Jim
At 10:50 AM 11/28/2022, you wrote:

Hi @.***,
The problem is fixed but it was merged into the
main code, which is not the same as the one released on pip.
Basically, when you pip install pyaedt, you are
installing the latest version released:
https://github.com/pyansys/pyaedt/releases/tag/v0.6.30https://github.com/pyansys/pyaedt/releases/tag/v0.6.30
If you want to test a new feature/fixed bug, you
need to clone the repository instead of using
pip install. By the way, if you update the
pyaedt version right now (v0.6.30) it should work with Eigenmode. Please try.

Reply to this email directly,
https://github.com/pyansys/pyaedt/pull/2047#issuecomment-1329508797view
it on GitHub, or
https://github.com/notifications/unsubscribe-auth/AKX7QMHSNGKJSX44K723GDTWKTWHBANCNFSM6AAAAAASNGUYWIunsubscribe.
You are receiving this because you
commented.Message ID: @.***>

James M. Potter, PhD, President
JP Accelerator Works, Inc.
2245 47th Street
Los Alamos, NM 87544
TEL:<PHONE_NUMBER>

After the update:

Jim

Hi @jpotter0 ,
Can you try the following command in a console:
pip install pyaedt==0.6.30
Thanks

Thanks, Maxime,
Windows 10 appears to have installed v0.6.30, but pyCharm still shows v0.6.28?
How do I tell if it really got innstalled and if it did is their something I need to do to get pyCharm to recognize it?

When I reran the pip install I get an error message:
ERROR: Could not find a version that satisfies the requirement pyaest==0.6.30 (from versions: none)
ERROR: No matching distribution found for pyaest==0.6.30
I am using PowerShell as an Administrator.
The first time I did a pip install it seemed to run and end normally. Maybe that means it recognizes that it's alreay installed??
Jim

I am trying to incorporate an HFSS script into pyaedt:
oDesign = oProject.SetActiveDesign("ihdtl12")
oModule = oDesign.GetModule("BoundarySetup")
oModule.AssignFiniteCond(
[
"NAME:FiniteCond1",
"Faces:="               , [26,27,29,30,31,32,33,220,221,222,223,254,255,256,257],
"UseMaterial:="         , False,
"Conductivity:="        , "58000000",
"Permeability:="        , "1",
"UseThickness:="        , False,
"Roughness:="           , "0um",
"InfGroundPlane:="      , False,
"IsTwoSided:="          , False,
"IsInternal:="          , True
])
I have added oDesign = aedtapp.odesign
I tried to add oModule = aedtapp.omodule , but that didn't work and don't know what to do with the GetModule("BoundarySetup").
is it possible to run this bit of code in pyaedt. If so, how do I set it up?
Thanks, Jim

I have given up on trying to use the conductivity boundary conditions and tried to use "assign_coating". There are several problems. It doesn't accept a list of faces it wants objects. The trouble is it applies a conductive coating to everything, including previously defined boundaries. Also, I get an error message that coating cannot be used with Eigenmode solutions. This is incompatible with my application. As a workaround, I can select faces by name and remove the ones that have H boundary conditions, but the whole object is to automate the processing of several hundred cavities with differing dimensions.
What can I do? I seem to have hit a roadblock trying to use the HFSS script. I guess I can create a separate script to apply conductivity boundary to the desired faces in every cell. Each cell is in a separate Design.
Does anyone have a different solution?

Can I run an IronPython script from pyaedt or do I have to run it from within HFSS?