I have set it up, why can't I search for my content?

I have started the local dev, but the search found that I could not find the local content

Sorry, it's difficult to tell what went wrong without a reproduction example. Maybe you forgot to run build once to initially create pagefind index?

Sorry, it's difficult to tell what went wrong without a reproduction example. Maybe you forgot to run build once to initially create pagefind index?

I have found the issue. The search function works on the server but not locally. I tried it out. This function searches for all content in the articles. How can I modify it to only search for titles?

You still need to know about pagefind and read its documentation in order to use this integration. I'll close this for now since the issue is resolved.


Failure looking for non existant Microsoft.NET.Build.Extensions.Tasks.dll

[x] Include the log file upgrade-assistant.clef that is produced in the working directory

upgrade-assistant.zip
Describe the bug
workspace.OpenSolutionAsync is failing saying it could not load an assembly (because it doesn't exist):

Could not load file or assembly C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\Microsoft.NET.Build.Extensions\tools\net7.0\Microsoft.NET.Build.Extensions.Tasks.dll.

In my VS installation the Microsoft.NET.Build.Extensions\tools folder has no "net7.0" folder...?
But on my machine this assembly does live in:

C:\Program Files\dotnet\sdk\7.0.101\Microsoft\Microsoft.NET.Build.Extensions\tools\net7.0\Microsoft.NET.Build.Extensions.Tasks.dll

So perhaps there's a problem with assembly resolution paths somewhere? The var workspace = MSBuildWorkspace.Create(GlobalProperties) line has this in the GlobalProperties:

It seems to get further if I add C:\Program Files\dotnet\sdk\7.0.101\Microsoft\Microsoft.NET.Build.Extensions\tools\net7.0\ to the MsBuildExtensionsPath.
To Reproduce

git clone https://github.com/MoneyTools/MyMoney.Net.git
git checkout net7
git checkout 38dbc04123ac97ef7768ab94ee82868e084a72eb
cd MyMoney.Net\Source\WPF
upgrade-assistant analyze .\MyMoney.sln --verbose
and this project also fails to upgrade:
https://github.com/lovettchris/GraphViewer
Exceptions (if any)
17:47:52 DBG] [Failure] Problem loading file in MSBuild workspace Msbuild failed when processing the file 'D:\git\MoneyTools\MyMoney.Net\Source\WPF\MyMoney\MyMoney.csproj' with message: C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\Microsoft.NET.Build.Extensions\Microsoft.NET.Build.Extensions.ConflictResolution.targets: (30, 5): The "ResolvePackageFileConflicts" task could not be loaded from the assembly C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\Microsoft.NET.Build.Extensions\tools\net7.0\Microsoft.NET.Build.Extensions.Tasks.dll. Could not load file or assembly 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\Microsoft.NET.Build.Extensions\tools\net7.0\Microsoft.NET.Build.Extensions.Tasks.dll'. The system cannot find the path specified. Confirm that the  declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.
Further technical details

Windows version and bitness info (32-bit or 64-bit) Windows 11, 21H2, 64-bit

Include the output of dotnet --list-sdks
3.1.426 [C:\Program Files\dotnet\sdk]
6.0.404 [C:\Program Files\dotnet\sdk]
7.0.101 [C:\Program Files\dotnet\sdk]

Include the output of upgrade-assistant --version
0.4.355802+b2aeae2c0e41fbfed35df6ab2e88b82a0c11be2b

Fixed in new versions of UA CLI.


Add alert when delete address without transactions

Design: https://zpl.io/adYnBlp

implemented
https://github.com/BeamMW/android-wallet/issues/365


Inlining values from an enum-getter

summary
I'm currently working on a PR for AMP in which we'd like to use a @const to assist with inlining a variable. An issue has come up where we are required to use an enum-getter as opposed to directly using the enum, but this seems to deopt.
For example, this code inlines as expected:
/** @enum {string} */
const a = { cx1: 'hello', cx2: 'world'};
console.log(a.cx1, a.cx2); // compiles down to: console.log('hello', 'world');

Whereas when a getter is introduced, it stops inlining:
/** @enum {string} */
const a = { cx1: 'hello', cx2: 'world'};

/** @const @function():  typeof a */
const getA = () => a;

console.log(getA().cx1, getA().cx2); // compiles down to: console.log(a.cx1)

Is this an intentional deopt? Is there a workaround?

Can you share which flags your using to compile with? Thanks!

This is just a limitation.  "collapse properties" which is responsible for collapse global objects only runs once relatively early.  When it runs, the "getA" hasn't been inlined and object can't be collapsed because "getA" 'escapes' the object.  Once it was inlined it would be possible but it is too late.   If it were contained within function "collapse object literals", which is responsible for local values, might do it depending on the conditions.


Laravel 5.5 compatibility

Hi,
it is posible to use with Laravel 5.5? If not, are you going to do upgrade?
Thanks.

Not yet, feel free to submit a PR


[Devportal] APIs' details are not organized properly.

APIs' version and context are not placed in proper manner as follows.

Duplicate with https://github.com/wso2/product-apim/issues/6230


Search in record list ignores prefilter

When you have a record list with a prefilter set, you see a filtered list of records. But as soon as you use the search field on the right top, it does not take the prefilter in account.
Steps to reproduce:

Enter https://demo.crust.tech/crm/pages/74992049056645144/. This is the page "New Leads": a record list with the prefilter "Status = 'New'".
Insert the character "u" in the search input, top right.
-> More records appear, showing leads with different statuses (New, Converted,...)

What I expected is that the search allowed me to search the filtered record list only.

Looked at this. Currently on api.latest.crust.tech, this request returns records that are not filtered:
https://compose.api.latest.crust.tech/namespace/88714882739863655/module/69055788898975761/record/?filter=Status+%3D+'New'+OR+Status+%3D+'Assigned'+AND+FirstName+LIKE+'Peter%25'+OR+LastName+LIKE+'Peter%25'+OR+Email+LIKE+'Peter%25'+OR+Rating+LIKE+'Peter%25'+OR+Company+LIKE+'Peter%25'+OR+Country+LIKE+'Peter%25'+OR+LeadSource+LIKE+'Peter%25'+OR+Status+LIKE+'Peter%25'&page=0&perPage=20&sort=createdAt+DESC
Response includes records whose status is different from 'New' or 'Assigned'

I've added extra parenthesis around prefilter to prevent such issues in the future.


Prepare for security testing

Add additional throttling by IP for login actions
Add spiffy grey banner for pentest app role

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-2449

Review app deployed to https://teaching-vacancies-review-pr-3275.london.cloudapps.digital


feat: new home page with onboarding steps

Try out this version of the Hiro Wallet - download extension builds.

This PR implements new designs for the home page including implementation of the new onboarding steps banner.
*Note: I did end up using atomWithStorage and persisting it after sign out bc the UX was a bit odd w/out it. If people are ok with it, maybe we can refactor to use redux-persist after Kyran's work is merged?

cc/ @kyranjamie @beguene @He1DAr

@kyranjamie I refactored this on top of your work using redux. Take a look again when you get time.

This is great @fbwoolf!
I found a few things to review. Note that for the margins:
In the Next steps:

The images for illustrations look pixelated on retina screens. To solve this, we need to export any bitmap assets (PNGs, GIFS, etc) at 2x of the size they are displayed. Luckily, Figma allows this right in the export options.
[Only new tab version] Check the vertical spacing between the elements.  I've fixed the first step as an example with the pixel values that approximate to what Figma shows. Note that some spacings were 20px, which is not part of the design system scale.
[Only new tab version] Check the spacing for the whole section. Image attached.
[Only widget version] Check the spacing for the whole section. Image attached.
The close (x) button doesn't have the click affordance with the cursor like other buttons do, a.k.a. the Mickey mouse glove.
The close (x) button is not a 32x32 circle, it's slightly oval. It's very minor, but it caught my eye 👁

In the Balances empty state:

The "Fund your account" CTA should be styled as a subdued primary button and not a link

The images for illustrations look pixelated on retina screens. To solve this, we need to export any bitmap assets (PNGs, GIFS, etc) at 2x of the size they are displayed. Luckily, Figma allows this right in the export options.

These should already be fixed. I had accidentally exported them as svgs incorrectly which was causing the pixelated image. I did export them again as 2x PNG and swapped them out so hoping they are ok now.

[Only new tab version] Check the vertical spacing between the elements.  I've fixed the first step as an example with the pixel values that approximate to what Figma shows. Note that some spacings were 20px, which is not part of the design system scale.
[Only new tab version] Check the spacing for the whole section. Image attached.
[Only widget version] Check the spacing for the whole section. Image attached.

I will go back and check these and make adjustments.

The close (x) button doesn't have the click affordance with the cursor like other buttons do, a.k.a. the Mickey mouse glove.
The close (x) button is not a 32x32 circle, it's slightly oval. It's very minor, but it caught my eye 👁

Thx, I'll change the cursor. I noticed the oval too but tbh thought it was my eyes bc when I inspect it, it says 32x32. I'll look into what is causing this bc I also see it!

The "Fund your account" CTA should be styled as a subdued primary button and not a link

Funny, this changed on me during development. I'll get it matching. It def was a link at one point.

Moreover, as a took a deeper look I had topics to get the team’s opinions on:

The "Register a name" step seems more accessory to the main conversion flow, which is to fund and transact. I believe it should be moved to the last place.
Also, while testing the btc.us website it was failing to let me buy and address. Other users may have the same issue. We should check this before advertising it more prominently.
here are 3 repeated "Start" CTA's which look off to me. Not sure what we could change, but posing the question if anyone has ideas.
Persistence: We were talking with @fbwoolf Ӿ that now the dismissal will persist even if new Wallets are created on the same browser. So, creating new Wallets will not show it, as we assume it's the same user. Still, on principle, shouldn't we assume or consider each Wallet creation as a new "experience" that should display the onboarding? Maybe not a big deal, but wanted to point it out.

@fbwoolf as a reminder: are we adding tracking events for all interactions, including the close/dismiss?

@markmhx can we list out the events to track here?

@landitus  I think I have fixed the UI issues here, but still need to add the event tracking:

Let's trigger a new event called select_next_step for all four options and distinguish them with a step property with one of the following values:

back_up_secret_key
add_funds
register_username
explore_apps

The "Register a name" step seems more accessory to the main conversion flow, which is to fund and transact. I believe it should be moved to the last place.

+1

Also, while testing the btc.us website it was failing to let me buy and address. Other users may have the same issue. We should check this before advertising it more prominently.

Let's keep btc.us for this release since I don't think it's worth holding it up. But I agree that we should consider an alternative step here instead of buying a username with btc.us, not just because of the app's issues but because NFTs may actually be a more attractive option than names these days.
@landitus Want to create an issue to explore this change and work with @eugeniadigon on a possible illustration? We could link to https://stxnft.com/ perhaps.

There are 3 repeated "Start" CTA's which look off to me. Not sure what we could change, but posing the question if anyone has ideas.

These could be more specific to each task on hand:

View key
View purchase options
Find name
Find app

Persistence: We were talking with @fbwoolf Ӿ that now the dismissal will persist even if new Wallets are created on the same browser. So, creating new Wallets will not show it, as we assume it's the same user. Still, on principle, shouldn't we assume or consider each Wallet creation as a new "experience" that should display the onboarding? Maybe not a big deal, but wanted to point it out.

Yea I agree that ideally we should show this again for every new wallet within the same install. But this is a nice-to-have since these next steps are mainly intended to help first-time users reach an "aha moment" and less importantly will help existing users get back to these tasks.
Separately, it just occurred to me that we need to include and design (for legal reasons) a message that indicates that the user will be taken to a 3rd party site for both the register name and explore apps options. This would serve a similar purpose to what we have in the desktop wallet for stacking.club links.
@landitus could you design this for integrating into this particular release? I'd suggest checking out the desktop wallet with some STX to see what the disclaimer looks like there (I don't see it in Figma unfortunately).
LEGAL REVIEW

And when the user dismisses the steps, a new event called dismiss_next_steps.

Would this be better as, dismiss_onboarding_steps, what does _next_ refer to here when dismissing all of them together?

These event names treat the component in general as named "Next steps" (vs. "Onboarding steps") so dismiss_next_steps is akin to saying "dismiss component with next steps".
I'm not sure what we gain by alternatively calling it dismiss_onboarding_steps (consistency with code naming in general perhaps?), though it'd be functionally the same for my purposes if you want to go with that. We'd just have an inconsistency with select_next_step, so I'd suggest we call that one select_onboarding_step as well if you prefer "onboarding" to "next".

These event names treat the component in general as named "Next steps" (vs. "Onboarding steps") so dismiss_next_steps is akin to saying "dismiss component with next steps".
I'm not sure what we gain by alternatively calling it dismiss_onboarding_steps (consistency with code naming in general perhaps?), though it'd be functionally the same for my purposes if you want to go with that. We'd just have an inconsistency with select_next_step, so I'd suggest we call that one select_onboarding_step as well if you prefer "onboarding" to "next".

I can keep it as next but, yep, in code the component is referred to as OnboardingSteps. I didn't understand where next was coming from until you explained it, thanks. If it makes the most sense to you for event tracking as next, I'll definitely just use that (which is what I pushed up already).

@landitus just communicating here, pushed up new images so this is ready for design review again when you are ready.

@fbwoolf @markmhx I've added a disclaimer for external links as suggested, by using the tooltip and arrow icon where applicable. Also, I've mocked 2 options for the call-to-action links below each step:

Option A is less specific
Option B is more specific

Check them out: https://www.figma.com/proto/1hT5X43DR8KCuFFZEFlMV4/🛫-Onboarding?page-id=715%3A5567&node-id=715%3A5568&viewport=241%2C48%2C0.5&scaling=min-zoom&starting-point-node-id=715%3A5568&show-proto-sidebar=1

Note to self: I'd need to add the arrow icon to the popup/widget version as well.

@landitus currently our tooltip design in the wallet looks like this:

Would we want it different here or keep the same for now and look at redoing throughout the app separately?

Option A is less specific
Option B is more specific

I think maybe option B makes it more clear to the user that they will be leaving the wallet to go to another site?

Yes, I think option B, being more clear and specific (though slightly longer), will be beneficial.

Would we want it different here or keep the same for now and look at redoing throughout the app separately?

Yes, let's use the existing component so we can ship it with this PR.

Here's the widget size version. For the last step, I believe the icon will bleed to the next line (maybe not depending on browser rendering) but still we'd want to "tie" the icon to the last word. Otherwise, the icon may end up alone in the second line.

[Figma](https://www.figma.com/file/1hT5X43DR8KCuFFZEFlMV4/%F0%9F%9B%AB-Onboarding?node-id=715%3A5567)

@landitus I think I have fixed the UI issues here, but still need to add the event tracking:

Thank you @fbwoolf. All the UI fixes are correct! 🎉 While going through it again I found some new ones I overlooked in my first review:

(Both) Check spacing between button and text in the empty state
(Widget) Fix the margin-right and left of the steps to be 20px, so that it matches the rest of the elements. It's what we have right now all over the app. Eventually, we'll need to review this to be 24px, but 20 will do for now so it's consistent.

@landitus The spaces discussion came up in another PR. I wonder if you can sync with @fbwoolf to review what spaces are defined in the current system and as much as possible try to stick to them to avoid hardcoding margins.

See this file for existing spacing units https://github.com/hirosystems/ui/blob/main/packages/ui-theme/src/sizes.ts#L23-L32

Amended the latest commit to retrigger tests.

I want to get this merged asap for Ledger work, but once it's in let's give it another once over. Trying to fix the tests now and noticed that the home screen in full page mode isn't correctly responsive/has horizontal scroll

@landitus The spaces discussion came up in another PR. I wonder if you can sync with @fbwoolf to review what spaces are defined in the current system and as much as possible try to stick to them to avoid hardcoding margins.

Gotcha, sounds like a good idea to align on spacing!


maya2glTF.mel invalid encoding

The <FileUpdate> MSBuild task for updating the version in the mel script uses UTF-8 with BOM encoding when I believe Maya expects UTF8.
I see this error when attempting to load the MEL script:
source "C:/Users/robertlong/Documents/maya/2018/scripts/maya2glTF_UI.mel";
// Error: ï»¿proc string _quoted(string $text)
 //
// Error: "C:/Users/robertlong/Documents/maya/2018/scripts/maya2glTF_UI.mel" line 1.2: Syntax error //

I can't make a PR right now because of the LFS quota (see #31) but here's the fix.
In Maya2glTF.vcxproj change this:
<FileUpdate Files="$(ProjectDir)\maya\scripts\maya2glTF_UI.mel" Multiline="true" Singleline="false" Regex="\$maya2glTF_version\s*\=\s*"(.+)"" ReplacementText="$maya2glTF_version = "$(GIT_Tag) $(GIT_Hash)"" />

to:
 <FileUpdate Files="$(ProjectDir)\maya\scripts\maya2glTF_UI.mel" Multiline="true" Singleline="false" Regex="\$maya2glTF_version\s*\=\s*"(.+)"" ReplacementText="$maya2glTF_version = "$(GIT_Tag) $(GIT_Hash)"" Encoding="utf-8-without-bom" />

Great! This was really driving me mad, but I didn't find a workaround for it. Thanks for this.
BTW: I am going to transfer this repository to the WonderMediaProduction organisation. This started as a personal project, but my employer embraces Maya2glTF, and we will be using glTF as the main asset type in all our future productions. This should also solve the quota problems. However, I'm not sure what implications this has for you. Are you okay with this transfer? It will remain a public repository with the same license model.

Fixed by 9ce1b126e011c8c4eca5abfa8234e5a668934598.
Thanks @robertlong


added necessary step for getting correct css on calender

When I followed the steps without doing yard css:dev or the equivalent npm command the dropdown calender had no CSS applied, so I added this step to ensure that anyone that runs it locally won't be confused

Hey @luddens heads up I wasn't able to reproduce this. Running yarn build-dev should update the build artifacts - so running it should apply any updates to the SCSS in:
src/stylesheets/datepicker.scss

For CSS assets that will be used (dist/react-datepicker.css). Any direct edits to .css files will be overwritten when you compile SCSS, but running yarn build-dev should eliminate the need to run yard css:dev.
Any details you can provide about your workflow when you ran into this? It might also be an IDE thing. Which one are you using? There are some tools to just auto-compile SCSS on save (e.g., https://atom.io/packages/sass-autocompile).

@martijnrusschen I think this one can be closed out, unless @luddens I'm missing the mark here.

Okay. I'll upload a video showing me do the npm install, the npm run
build-dev and then npm run start to show the css malfunctioning.
Weird that I have to run css:dev too.
On Wed, Sep 14, 2022 at 4:22 PM Dan Mateer @.***> wrote:

@martijnrusschen https://github.com/martijnrusschen I think this one
can be closed out, unless @luddens https://github.com/luddens I'm
missing the mark here.
—
Reply to this email directly, view it on GitHub
https://github.com/Hacker0x01/react-datepicker/pull/3755#issuecomment-1247261809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIP6GX6VCPWNYB5FBADWHDV6IXXFANCNFSM6AAAAAAQJ6OO7U
.
You are receiving this because you were mentioned.Message ID:
@.***>

Okay. I'll upload a video showing me do the npm install, the npm run build-dev and then npm run start to show the css malfunctioning. Weird that I have to run css:dev too.

Ah, possible we're running npm run start here instead of yarn start?

I don't have TEE installed, so that errors out and the CSS never builds properly.

The same thing happens when I run yarn start or npm run start. Hmmm.
Interesting. I had to go to court today, spent half the day upstate so I
didn't get to make a video of the issue, yet. Basically every time I want
to display changes I have to do the css build.
On Wed, Sep 14, 2022 at 4:55 PM Dan Mateer @.***> wrote:

Okay. I'll upload a video showing me do the npm install, the npm run
build-dev and then npm run start to show the css malfunctioning. Weird that
I have to run css:dev too.
Ah, possible we're running npm run start here instead of yarn start?
—
Reply to this email directly, view it on GitHub
https://github.com/Hacker0x01/react-datepicker/pull/3755#issuecomment-1247292666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIP6GR5DNDUFOXY7BYTR5DV6I3UZANCNFSM6AAAAAAQJ6OO7U
.
You are receiving this because you were mentioned.Message ID:
@.***>


Suppress parking and bus_stop POIs until zoom 17

Parking comes in too early at z15 now, bus_stop too early at z16 now.
Potentially I can see including bus_stop earlier for TRANSIT layer only, but not for a general purpose reference map.

I also moved highway=bus_stop from z16 -> z17. Let me know if that was in error.

LGTM. Looks like most bus_stop are in highway, good catch.


Results re-run

With the new Haskell results, the Windows and Mac results could use updating. Cf. https://github.com/atemerev/skynet#results-on-my-shitty-macbook-12-2015-core-m-os-x
If you don't have time, I could replace them with timings on my Mac and Windows machine.

They probably all need rerun at some point. There are still a couple pull requests though.

I'm fine waiting, just want to get a clean span of them up, ideally with multiple runs, upper/lower bounds.

@bitemyapp As for Haskell, maybe Main.hs should benchmark all of them by default, and put a comment saying how to comment out tests.

@ChristopherKing42 I'd be opposed, too complex/noisy. Others are there only for people that want to toy around.

Dup of #39


Change functions and variables name to give the namespace (fix #12)

Need to be considered:

Variables in wpbcopy.sh and wpbpaste.sh such as $TRANS_URL and $CLIP_BODY left as is. Because I think they the files are not expected to be sourced  directly so they will be kept in the scope of the file, I think. Should we rename them, too?

yeah, those variables are under the child process. np


Word Filtering

Add filters for any text fields to prevent inappropriate language.

I would say content moderation is indeed difficult. Start with cleaning your data for SQL injection attacks before you move up to sentiment analysis.


Question: Access data currently used in result

Hello Nicolas,
First of all, thanks for creating such great tool! It is really cool ;-)
I have one question - is there a way to access data "currently" used in pivot calculations? Let say there is initial data set which have X columns and N rows. For that data I have defined 3 additional derived attributes, so there is N rows and X+3 columns in total.
End user creates pivot table based on that data and sets some filters up in the pivot. This action limits number of rows from initial N to M rows. I would like to reach that filtered "source" data; M rows of X+3 columns in order to use it in further processing (creating export file, display in table, other calculations).  Is is possible?
Regards.

Glad you like it!
This data is kind of an intermediary dataset which is hard to access but by wrapping the PivotData object you might be able to intercept it... Honestly it feels like a stretch, though.


problem with optgroup

I have a input with datalist, then contains groped options, and it's doesn't work
P.S. I am using Laravel blade templates for adding opt groups and options.
`

Type something (zero threshold)

@foreach($groupedItems as $organizationId => $items)

@foreach($items as $item)
{{$item->object_branch_name}}
@endforeach

@endforeach

`

yes currently option group are not supported (yet, maybe i will spend some time on it but i haven't seen the real need yet)
https://github.com/lekoala/bootstrap5-autocomplete/issues/7
it wouldn't be too hard to add, but i'm not sure it's really worth it. this is an autocomplete library, not a replacement for a select box. datalist, for examples, don't support optgroup either
in my other library
https://github.com/lekoala/bootstrap5-tags
i've added support for optgroups, it's more like a real select


How can I generate a model that does not include the aws Trait definition?

I want to use smithy to make a model like sdk json.
It looks like these JSONs don't contain aws Trait definitions, For example like aws.protocols#awsJson1_1 .
But It will be included in the model I generated.
How can I generate a model that does not include the Trait definition like sdk json?
model/main.smithy
namespace com.example

use aws.api#service

@service(sdkId: "example")
service Example {
    version: "2021-12-08",
    operations: [
        CreateExample,
    ],
}

operation CreateExample {
    input: CreateExampleRequest,
    output: CreateExampleResult,
}

structure CreateExampleRequest {
    Description: String,
}

structure CreateExampleResult {
    Description: String,
}

smithy-build.json
{
    "version": "1.0",
    "projections": {
        "example": {
            "transforms": [
                {
                   "name": "includeServices",
                    "args": {
                        "services": ["com.example#Example"]
                    }
               },
              {
                  "name": "removeUnusedShapes"
              }
            ]
        }
}

build.gradle.kts
plugins {
    id("software.amazon.smithy").version("0.5.3")
}

dependencies {
    implementation("software.amazon.smithy:smithy-model:1.13.1")
    implementation("software.amazon.smithy:smithy-aws-traits:1.13.1")
}

tasks["jar"].enabled = false

repositories {
    mavenLocal()
    mavenCentral()
}

You can use the removeTraitDefinitions transform in your projection to remove the trait definitions but leave the instances of traits intact on any shapes.

@kstich
Thank you for your help!!
I tried defining removeTraitDefinitions.
smithy-build.json
{
    "version": "1.0",
    "projections": {
        "example": {
            "transforms": [
                {
                   "name": "includeServices",
                    "args": {
                        "services": ["com.example#Example"]
                    }
                },
                {
                    "name": "removeUnusedShapes"
                },
                {
                    "name": "removeTraitDefinitions"
                }
            ]
        }
}

But Smithy returns a validation error.
[ERROR] com.example#Example: Unable to resolve trait `aws.api#service`. If this is a custom trait, then it must be defined before it can be used in a model.

How can this error be resolved?

Since you're building a projection, you'll need to declare the proper dependencies in buildscript. You can read more about configuring the dependencies for a projection build with the Gradle plugin here.

Thank you! 👍
I will read this document!


Update Docs

The guide currently online here: https://pantryparty.app/ are really out of date. Like, my app has something called Purchase factor In create a product. What even is that?

When you are on the purchase tab, tap the yellow bar at the top and you will seethe screens listed here: https://pantryparty.app/docs/basic_usage/product-creation.html
The term Purchase Factor is only visible on the Product Creation form and is equivalent to this step: https://pantryparty.app/docs/basic_usage/product-creation.html#step-5-quantity-conversion
Essentially for when you scan one item but add X to stock (think a pack of soda)


Scrolled element doesn't appear on refresh

Hi, great plugin! Using it at the moment for my project and definitely will continue using it in the future!
I stumbled on a little problem. When and element somewhere down the page is scrolled into view, it appears as it should, no problems so far, but when I refresh the page (the viewport is not reset at the top of the page), the element is in the viewport but it doesn't appear until I scroll a little.
Is there a solution to this problem?

Isotope has events - so - you can trigger onArrange when filtering.
http://isotope.metafizzy.co/events.html
function onArrange() {

$(".grid_item div").each(function() {
	initWowIsotope();
});

}

$grid.on( 'arrangeComplete', onArrange );

	function initWowIsotope() {
		//
		var wow = new WOW({
			boxClass: 'wow-iso',
			animateClass: 'animated',
			offset: 0,
			mobile: true, // trigger animations on mobile devices (default is true)
			live: true, // act on asynchronously loaded content (default is true)
			callback: null,
			scrollContainer: null
		});
		void wow.offsetWidth;
		wow.init();
	}

.wow-iso {display: hidden;}
You can also call it something like this:
function onArrange() {
   $(".grid_item div").addClass("wow fade-in-up-sm");
   $(".grid_item div").attr("style","visibility: visible; animation-name: fadeInUp;");
}


Clever package name?

MTENN is a great start, but I wonder if we can come up with a more clever name for this package.
Some ideas:

Structure Informed Deep Equivariant Compound Affinity Regressor (SIDECAR)
Neural Equivariant Graph Regression of Novel Ideations (NEGRONI)
Structure-based Learned Inference of Novel Graphs (SLING)

I was just about to raise this issue when I saw you've beaten me to it, @jchodera
I was gonna call it Structrual AnalyZer with Equivariance for Rapid Affinity Computation.
but on a second thought it's not half as elegant as SIDECAR.
Let's leave this to @kaminow to decide

I am about to cut a 0.1.0 release for @kaminow so that we can push ahead on some integration into asapdiscovery. @jchodera @yuanqing-wang @kaminow if you would like to change the name it would probably be best to do it before I make any tags.

From talking with @kaminow  and no immediate complaints I will close this issue and proceed to cutting a release. If people change their mind just re-open and I will sort it out.


Add openvino to requirements for samples

Describe what's required for deployment as well.
Ticket 155703

@ilya-lavrenov, can I merge this?


Controller

Used PID controller for the throttle and braking and the yaw controller for steering.

Sorry, wrong repo :)


app.nit: propagate events (such as a button press) to the parents of the control

UI events, such as a button press or a check box toggle, were only sent to the observers of a control, and the control itself. For this reason, every button was watched by the window or some other container.
Propagating the events to the parents (direct and indirect) reaches the window and all of its containers. This allows for more modular code and components, and it removes the need for manually declaring the window as an observer.

+1


More flexible disabling endpoint feature

This feature gives better flexibility to the RSTUF user to disable endpoints.
The user can disable specific endpoints and methods in a simple configuration using the RSTUF_DISABLE_ENDPOINTS.
It reduces the number of settings parameters (environment variables) RSTUF_BOOTSTRAP_NODE, and RSTUF_TOKEN_NODE, making the feature more flexible as we add more endpoints and methods to the RSTUF API.

Can you also update the docs?

@MVrachev it is ready now


Decide master VM size for our 1.0 performance goals

Several people have brought this up (including @fgrzadkowski and @yujuhong). We have various performance/scalability goals (e.g. #4521, #3954, #3952) but we haven't stated what VM size we will use for the master when making these measurements. (I'm assuming we can just decide a GCE VM size, rather than a node configuration for every cloud provider we support.)
We've been setting GOMAXPROCS in various places (#4605, #4675). I'm not sure we've been very systematic about it, but either way, or performance presumably depends on # cores of the master. I believe people have mostly been using 4-core masters for the performance measurements? (I can't remember if we've been measuring memory consumption vs. cluster size, but if not, we should to ensure the VM has enough memory for a 100 node cluster).
Unless someone objects, I'm going to assume we are going to use 4-core VM.
@filipg @wojtek-t @satnam6502 @roberthbailey @lavalamp @bgrant0607 @brendandburns

@erictune

Rather than having a fixed size master, perhaps we need to assume that the size of the master will be related to the size of the cluster or the total number of pods in the cluster? Perhaps an n1-standard-1 is fine for a four node cluster with 200 pods but we might need a n1-highmem-16 for a 500-node cluster with thousands of pods? Or perhaps the size of the master is more related to the activity (w.r.t. cluster management operations) rather than cluster size? A cluster with a lot of churn will put much more stress on the master than one which after configuration makes very rare adjustments?

I just meant what size will we use for the 100-node/30-pods-per-node goal for 1.0. I agree in general we need to scale the size of the master with the size of the cluster and the operation workload, but that's outside the scope of what I was intending for this issue.

2 suggestions:

Can we decided 1.0 goals for all vm sizes we offer? I'm fairly certain we won't scale linearly with resources on the master (this is the c10k problem)
Can we say a cluster with X homogenous nodes can support this QOS, instead of singling out the master?

We could point out that the master is probably a bottleneck if ther're nothing but no-op pause pods on a node. IMHO encouraging a heterogenous cluster will put-off more users than it wins over.

To be fair, GOMAXPROCS should only make a big difference if there's CPU bound parallelizable stuff--network bound stuff should be parallel anyway.
We do run multiple binaries on the master, though. 4 cores sounds like it should be more than enough.
@bprashanth what specifically won't scale?

I meant generally, it isn't safe to assume that the server on a machine will get twice as fast with double the resources.  People typically benchmark speed in terms of something like qps. If the master has 200 qps that each take 1s there will probably be 200 pending requests/concurrent connections. Most webservers create a thread to handle each connection and you'll hit a point where the os can't handle it, so performance falls off a cliff.
This is a hypothetical situation, anything in that software stack could crumble (specifically the parts we don't control, like etcd). It would be nice for users to not have to extrapolate our results to different instance-price points and figure this out the hard way.

cc @timothysc

We need data before we can make a decision: a plot of master cpu vs. number of nodes under a load scaled proportionally to the number of nodes
If a fairly beefy machine is required for 100 nodes but not for 10 nodes, we'll likely need to scale the master size with cluster size, for cost reasons

Most webservers create a thread to handle each connection and you'll hit a point where the os can't handle it,

We should measure. Go multiplexes goroutines over OS threads, so our go server won't have this problem (we can have related problems like running out of file handles). nginx I'm not sure about, but I'd like to get rid of that at some point anyway.

We need data before we can make a decision: a plot of master cpu vs. number of nodes under a load
scaled proportionally to the number of nodes

I think we need a realistic workload generator for that. I don't think anyone has started working on that yet as we are still benchmarking with fairly simple, unrepresentative workloads (which is totally the right thing to be doing right now).

@lavalamp I am working up a PR to remove nginx it from the GCE deployment (it's working locally, just need to clean up the salt stuff).

Nobody screamed, so I think we should say from now on all perf experiments for the 1.0 milestones (see first entry in this bug) should use n1-standard-4 GCE VMs. Does this sound reasonable?

SGTM

Regarding graphs etc I'm looking into cpu usage of the master. It seems this is currently the main bottleneck for API calls latency. It seems most of the time we spend it pkg/conversion/.*. I need to dig deeper to show some data.
From my experience, requirements for the master are much higher for larger clusters, even with no pods. Thus scaling master with scaling the cluster size will be definitely needed.
4 core master is a reachable goal for 100 node cluster.

@davidopp SGTM

@thockin said he observed conversion routines being called for every byte of some arrays/slices. Could be some low-hanging fruit there.

@fgrzadkowski on an n1-highmem with 16vcpus:
$ cat /proc/cpuinfo | grep "core id" | sort | tail -1
core id		: 7

I remain skeptical (in a healthy sense) that we can reliably service close to 1000qps [1Get/Put node update/2s+(1Get/Put pod status/10s)x30)x100+watch overhead] and 3 heavy weight master components on 4vcpus. Let the games begin :)
@timothysc out of curiosity what's the profile of your master?

@lavalamp has already started working on optimizing []byte conversion (see #6435)
@bprashanth, not saying vcpu is enough, but your calculation is slightly outdated. The node status will be (or has been?) changed to post every 10 seconds, and pod status is only sent on actual change.

I checked before saying :)
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubelet/kubelet.go#L86
though you're probably talking about https://github.com/GoogleCloudPlatform/kubernetes/issues/5953.  Node status only leads to 1qps*100 in that calculation though, and there's a whole lot of unaccounted watching overhead+queries from controller manager and scheduler I'm not accounting for.

I stand corrected about pod status if this is not the code sending it:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/kubelet/kubelet.go#L1631
But if I've read that right it should be once every 10s?

Hrm, I rescind that 6qps metric, we only send it every 10s on update so it would only really be a once every 10s if the pod is crashlooping

We've done what the title of this issue says, and I think we should move any followup discussion to one of other the dozen or so performance-related issues we have open. Closing.

@davidopp We are doing continuous etcd testing on GCE. Is it possible to tell me how many resource would you like to allocate to etcd? So we can help to run under that env and find problems early to ensure etcd can deliver what k8s needs for 1.0/

Hmm, that's an interesting question. I assume the question is for CPU only, not memory and disk. Maybe try 2 cores if you're running etcd by itself? It would be helpful if we knew during regular Kubernetes workloads what is the % of the CPU used by each of the components, so we could use that as a basline. (For example if etcd typically uses 50% of the CPU, then 2 cores would be reasonable.)
@wojtek-t @satnam6502

@xiang90
My current experiments show that currently huuuuge majority of CPU is consumed by Kubernetes apiserver (i.e. I didn't observe etcd as a bottleneck). However there are few parallel efforts to reduce amount of work apiserver is doing (see e.g. #6800 #6518 #6562) so hopefully this will be significantly improved.
But still - I think that 2 cores for etcd is too much (i.e. I mean that 2 cores for apiserver + scheduler + controllers is to low requirement for 100-node cluster).
Can we start we the assumption that etcd is allowed to use 1 core? Or is it unrealistic goal?
cc @fgrzadkowski

I completely agree with @wojtek-t. Based on CPU profiles etcd is currently using it should not be a bottleneck for a long time. One thing to consider when running etcd tests on GCE is using fast drives (either large PD or SSD PD). Currently k8s setup on GCE uses one of the smallest regular PD, which has insanely small IOPS (we use 10G disk; see https://cloud.google.com/compute/docs/disks/persistent-disks#pdperformance)

@xiang90 Does that answer your question? Sounds like you should use an n1-standard-1 and 10G PD.

@davidopp Yes. Thanks!


offboard position or attitude setpoints for fixedwing in mavsdk

Is fixedwing supports position and attitude setpoints in mavsdk ?

The question is whether it is supported in PX4. Let's see if @Jaeyoung-Lim knows.

@julianoes Explained in https://github.com/PX4/PX4-Autopilot/issues/18961 !

Thanks @Jaeyoung-Lim.


Pass the store through to the data view item component

Minor enhancement to pass the DataView store in the item component props (along with the item record). The need for this arose in a client app where my data view items have buttons to perform crud operations so they need access to the store.

My first comment is that this is a great opportunity for us to use the {@callback} notation in jsdoc.  We have used it a few times in the source already and are trying to roll it out.  Take a look at other examples like the Column API, or the StoreContextMenuItem API.
Regarding the actual signature, would it make sense to have the second argument be a 'metadata' object that contained the store and/or the DataView?  Might be a bit more flexible for adding additional secondary bits, and also seems similar to the approach that grid elementRenderer is taking.

Sorry for the delay here.  I discussed in detail with John and totally misunderstood this.  Didn't realize it is actually an elemFactory in the API, not an arbitrary callback.
We may have additional work around documenting the elemFactory type, but this seems appropriate to merge.


Restricting custom urls to certain users doesn't work

Not sure how long this has been busted for. It seems to be present in 6.4 and 6.5.

Add a new custom link on the Custom Links settings page, and restrict it to a specific user.
Login as a different user than the one you specified in the previous step. You will still be able to see the link on the left-hand side.

It was a permissions config problem


Addition of emailSending

I made this first-time program to send emails to bulk my typing their email address each time the loop executes. If you want I can work on this by reading you a CSV file containing all the email addresses of the receivers.

I made this first-time program to send emails to bulk my typing their email address each time the loop executes. If you want I can work on this by reading you a CSV file containing all the email addresses of the receivers.

Yes please do it with csv.

@Arghya-Banerjee ,what happened, why you closed the issue??

Was having a personal prob related to the code in the pr thus I closed it. But don't worry I am working on giving the script to send mail reading CSV.

I will open a new one when done including both the scripts


CLI update

This adds the cli/CLI.py wrapper, which allows the application to be run either with or without a GUI with equivalent features, discussed in #28.
The download function was moved from gui/GUI.py to the Downloader class to reduce code duplication. Additional work should be done to simplify argument passing to the Downloader class, but would be done in another PR.
To enable the CLI, the argument --nogui needs to be passed in. If it is not passed it, the GUI will be run and all other arguments will be ignored.

Thabk you for the changes. Just one quick question. Have you tested for any downloads?

I did not test every site, but tested a few (using One Piece as the show). I have not tested any requiring captchas yet.
Some sites worked and some did not work. I double checked the sites which didn't work with the current commit of dev and they did not work there either, so there's other issues that need to be resolved. I can open some more issues today for other problems I encountered while testing this PR.

Thanks for the update :D
PR seems to be good to be merged so far. I will do the final tests on it and will merge :)

I think this PR was meant to be merged but you forgot to?

Hi, @ArjixGamer I'm really sorry that I was not able to merge this. The reason is, I was working on a new features for the 2.0 release of monkey-dl. I was hoping to merge this PR once I was done with those changes to avoid any collisions with the work I've been doing (Since this PR has too many changes I need to try and sort out. Also I was reviewing the code changes of this PR as well). However, I got extremely busy with my work/university and actually had no time to invest on monkey-dl. I still have lot of issues I need to address here, and I'm aware that almost all the sites need to be fixed.
Hoping to at least do some quick fixes to those, but the thing is analyzing the changes and fixing the scrapes takes alot of time which is hard to find for me these days. Hopefully, I will manage to at least get a couple of sites working soon!


Waymo weather dataset

Could someone help me which waymo-dataset belongs to the adverse weather? I cannot find any data related to the rain, fog or snow weather while I have found the adverse weather simulation results (Lidar point cloud) in many publications citing the waymo dataset. It really confuses me. Thank you for your help.

Hello,
I assumed the dataset was recorded in different weather conditions like rainfall, sunny, snowfall, foggy.  Could you please direct me to the specific dataset folder which is captured during such adverse weather conditions or provide me with some papers which has some stats?
Awaiting your response!
Best,
Zilin

Hi, thanks for your interest in our dataset! We do not explicitly target different weather conditions in the main dataset. For rainy scenes, you can try the domain adaptation dataset.

Thank you for your reply. Could you please tell me how to convert the Waymo Open Dataset-format (tfrecords) data to the .bin format data which I can input to my augmented method? Thank you.

For example, I want to read the (x,y,z,reflectivity) from the tfrecords, is it possible?

Awaiting your response!

I'm not entirely sure what you're looking to do. .bin is a fairly generic format, could you provide some more specifics for what the target format is? For reading the data, we have some examples in our tutorial colab here: https://github.com/waymo-research/waymo-open-dataset/blob/master/tutorial/tutorial.ipynb.

where can i access this domain adapted dataset which contains rainy scenarios?


Use ManuallyDrop in union to express intent

core::mem::ManuallyDrop indicates that an entity will be dropped manually. This is useful in unions where the object of the right kind will be extracted in order to be dropped.
Fix #4.

Moot, closing.


feat: Adding allow inspecting inner fields of nw.Struct

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #
Closes #https://github.com/narwhals-dev/narwhals/issues/1107

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.

it's "just" typing

Typing can be nightmarish 🥹

wow, this is literal 🔥
well done, this pleases me to no end 🤗

Now, that's feedback.. Thanks! 😊


Support getFileBlockLocation in LocalCacheFileSystem

What changes are proposed in this pull request?
Delegate getFileBlockLocation to external file system in LocalCacheFileSystem.
Why are the changes needed?
Otherwise, LocalCacheFileSystem inherits the default behavior of org.apache.hadoop.fs.FileSystem which returns localhost only.
Does this PR introduce any user facing changes?
No.

Can you please describe what are the benefits if the client can get access to the block location info? Would that cause the client to directly talk to the datanode and bypass local cache?

@dbw9580 The compute framework can use it to schedule the split task to the node in the same block location.

@jiacheliu3 Thanks for your suggested comments, PTAL.

alluxio-bot, merge this please


cannot search for C#orf###

C open reading frames (Corfs) have lower case characters in their official gene symbol. str_to_upper(input$gene_symbol) breaks this.
I'll add an if_else to detect for this rare case.

Cannot search for a gene that's in the db

@dleehr and @johnbradley
I deleted the develop branch, as you suggested after the last merge. I also started a new branch for some validation work, EDA, etc.
Q: What is the best way for me to fix some of these minor issues (as above)? New branch for them? Push to master?

Q: What is the best way for me to fix some of these minor issues (as above)? New branch for them? Push to master?

@hirscheylab I prefer to always create a new branch and a PR for that branch. That way you can tie this issue to a PR. Then if the issue re-surfaces we can easily track down where the problem might reside.


[ML] tests: Add e2etests for index

Description
This pull request is a followup to #35251 that adds end to end tests for IndexOperations.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Add new prop tabBarElevation for the tabBar's elevation

Closes #276

Tabbars have style property where you can pass elevation. Why don't you like such approach?

Where / how? I don't see this.

https://github.com/skv-headless/react-native-scrollable-tab-view/blob/master/examples/FacebookTabsExample/SimpleExample.js#L10

Isn't that for the whole container ie including the content? In the case of
elevation the shadow appears on the bottom of the screen rather than below
the tabs.
On Wed, May 25, 2016, 09:05 Evgeniy Sokovikov<EMAIL_ADDRESS>wrote:

https://github.com/skv-headless/react-native-scrollable-tab-view/blob/master/examples/FacebookTabsExample/SimpleExample.js#L10
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/skv-headless/react-native-scrollable-tab-view/pull/277#issuecomment-221569495

Sorry. My fault. That should work
<ScrollableTabView renderTabBar={() => <ScrollableTabBar style={{elevation: 1}}/>}>

@cancan101 does my solution work for you?

Yes, that works.

cool


Remove intra-doc links

Any documentation links that were crate or path dependent have been updated
to use the relative paths of the generated Rustdoc output.
Signed-off-by: Kevin Leimkuhler<EMAIL_ADDRESS>

Hmm, my understanding is that intra-doc links should be preferred since relative links can sometimes break due to path differences between docs.rs and local docs — @carllerche may remember the specifics?

@kleimkuhler thanks for doing this, last issue for the alpha1 milestone.

@hawkw Intra-doc links are a nightly feature. cargo doc --open on master just had broken links in all those places. Only if a user knew to use cargo +nightly doc --open did they work properly.

The same issues exists on Tokio master as well right now. If docs are not compiled locally with nightly then most of the links do not work 😕

And to add in the fun the nightly compiler doesn't give any warnings that you're using a nightly only feature.


Implementation of solar retrieval seems to be incorrect

Hello @amotl,
first of all thank you for your great work.
After exploring dwdweather2, I noticed that solar_* returns always None.
However, e.g. for station ID 5856 (hourly resolution) there clearly is data available (see https://opendata.dwd.de/climate_environment/CDC/observations_germany/climate/hourly/solar/)
e.g:
query_hour = datetime(2019, 7, 17, 11)
result = dw.query(station_id=5856, timestamp=query_hour)
for item in result.items():
     print(item)

should return solar-values unequal None:
5856;2019071711:12;    1;   -999;   57.0;  321.0;  60;    28.00;2019071712:00;eor
Am I missing something here?
Kind regards

Dear @Nikolai10,
thanks for writing in. We are able to confirm your observations.
Invoking
dwdweather weather 5856 20190717T11 --resolution hourly

yields these log entries regarding acquisition of solar data
2019-11-04 15:23:09,194 [dwdweather.core     ] INFO   : Downloading "solar" data (ST)
2019-11-04 15:23:09,194 [dwdweather.client   ] INFO   : Requesting https://opendata.dwd.de/climate_environment/CDC/observations_germany/climate/hourly/solar
2019-11-04 15:23:09,223 [dwdweather.client   ] INFO   : Fetching resource https://opendata.dwd.de/climate_environment/CDC/observations_germany/climate/hourly/solar/stundenwerte_ST_05856_row.zip
2019-11-04 15:23:09,582 [dwdweather.client   ] INFO   : Reading from Zip: produkt_st_stunde_19970102_20190930_05856.txt
2019-11-04 15:23:09,625 [dwdweather.core     ] INFO   : Importing measurements for station "5856" and category "{'key': 'ST', 'name': 'solar'}"
2019-11-04 15:23:09,625 [dwdweather.core     ] INFO   : Importing "solar" data from "https://opendata.dwd.de/climate_environment/CDC/observations_germany/climate/hourly/solar/stundenwerte_ST_05856_row.zip/produkt_st_stunde_19970102_20190930_05856.txt"
 80%|█████████████████████████▌      | 159191/199323 [00:28<00:07, 5567.59it/s]

and finally delivers NULL values for solar as well:
    "solar_atmosphere": null,
    "solar_duration": null,
    "solar_end_of_interval": null,
    "solar_global": null,
    "solar_quality_level": null,
    "solar_sky": null,
    "solar_zenith": null,

We will have to look into that.
With kind regards,
Andreas.

After a short investigation, we see that a typical data line of air_temperature looks like:
5856;2018072003;    3;  16.2;  69.0

Respectively, that's a line for wind:
5856;2018081003;   10;   3.0; 240

However, a line for solar looks like:
5856;1997021322:20;    1;   -999;    0.0;    0.0;   0;   139.96;1997021323:00

or
5856;1997061306:06;    1;   -999;   54.0;   63.0;  12;    67.99;1997061307:00

So, we recognize suffixes on the timestamp like :20or :06 here, which most probably confuse the import routine import_measures_textfile [1] which is coming from the "old" dwdweather codebase and is reasonably convoluted that I didn't dare to touch it for a refactoring yet.
I will try to mitigate that by stripping away these suffixes altogether, if you or @wetterfrosch do not veto against it. Altogether, we are working on the "hourly" level here, right? So, I believe this should do no harm.
With kind regards,
Andreas.
[1] https://github.com/panodata/dwdweather2/blob/0.10.0/dwdweather/core.py#L402-L464

Dear @Nikolai10,
we just released dwdweather2 0.11.0 which might mitigate that problem.
Invoking
dwdweather weather 5856 20190717T11 --resolution hourly --categories solar

now yields data for the "solar" category.
    "solar_atmosphere": 60.0,
    "solar_duration": null,
    "solar_end_of_interval":<PHONE_NUMBER>00,
    "solar_global": 321.0,
    "solar_quality_level": 1,
    "solar_sky": 57.0,
    "solar_zenith": 28.0,

Please add the --reset-cache parameter in order to drop the SQL database beforehand and be aware that this will drop all data within that completely.
With kind regards,
Andreas.

@amotl: This has indeed resolved the issue. Thanks for your help


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

Codecov Report

Merging #131 (1359e70) into ci/dependency-updates (68b92ee) will not change coverage.
The diff coverage is n/a.

@@                  Coverage Diff                   @@
##           ci/dependency-updates     #131   +/-   ##
======================================================
  Coverage                  76.19%   76.19%
======================================================
  Files                         15       15
  Lines                        605      605
======================================================
  Hits                         461      461
  Misses                       144      144

Flag
Coverage Δ

linux
76.19% <ø> (ø)

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.


Job Quote hardcoded and through report selections

Dear team,
On the Job Card and Job List there is the possibility to preview and send the job quote.
Previewing the Job Quote is using the hardcoded reference to report "Job Quote".
The following actions are copied from the Job Card.
action("Report Job Quote")
{
    ApplicationArea = Suite;
    Caption = 'Preview Job Quote';
    Image = "Report";
    Promoted = true;
    PromotedCategory = Category8;
    ToolTip = 'Open the Job Quote report.';

    trigger OnAction()
    var
        Job: Record Job;
    begin
        Job.SetCurrentKey("No.");
        Job.SetFilter("No.", "No.");
        REPORT.Run(REPORT::"Job Quote", true, false, Job);
    end;
}

The functionality for sending job quotes uses codeunit Jobs-Send to handle the process.
This codeunit however uses the Report Selection that is setup for usage JQ, as can be seen in procedure SendProfile.
This report usage can however, to my knowledge, not be setup through a report selections page.
action("Send Job Quote")
{
    ApplicationArea = Suite;
    Caption = 'Send Job Quote';
    Image = SendTo;
    Promoted = true;
    PromotedCategory = Category8;
    ToolTip = 'Send the job quote to the customer. You can change the way that the document is sent in the window that appears.';

    trigger OnAction()
    begin
        CODEUNIT.Run(CODEUNIT::"Jobs-Send", Rec);
    end;
}

procedure SendProfile(var DocumentSendingProfile: Record "Document Sending Profile")
var
    ReportSelections: Record "Report Selections";
    ReportDistributionMgt: Codeunit "Report Distribution Management";
begin
    DocumentSendingProfile.Send(
        ReportSelections.Usage::JQ.AsInteger(), Rec, "No.", "Bill-to Customer No.",
        ReportDistributionMgt.GetFullDocumentTypeText(Rec), FieldNo("Bill-to Customer No."), FieldNo("No."));
end;

Can this be changed in a way that the same report (from report selections) is used on both occasions, to make it a preview of the actually sent report? Can you please also make this report usage updateable from a report selection page, so it can be set up?

@AlexanderYakunin: I currently checked the nextmajor symbols (20.0.31871). I see that on the Job Card it is indeed fixed as you noted. On the Job List the following code is still present, which according to me should also be changed accordingly:
action("Report Job Quote")
{
ApplicationArea = Jobs;
Caption = 'Preview Job Quote';
Image = "Report";
//The property 'PromotedCategory' can only be set if the property 'Promoted' is set to 'true'
//PromotedCategory = "Report";
ToolTip = 'Open the Job Quote report.';
trigger OnAction()
var
    Job: Record Job;
begin
    Job.SetCurrentKey("No.");
    Job.SetFilter("No.", "No.");
    REPORT.Run(REPORT::"Job Quote", true, false, Job);
end;

}
action("Send Job Quote")
{
ApplicationArea = Jobs;
Caption = 'Send Job Quote';
Image = SendTo;
//The property 'PromotedCategory' can only be set if the property 'Promoted' is set to 'true'
//PromotedCategory = "Report";
ToolTip = 'Send the job quote to the customer. You can change the way that the document is sent in the window that appears.';
trigger OnAction()
begin
    CODEUNIT.Run(CODEUNIT::"Jobs-Send", Rec);
end;

}
Additional question: When can we expect this modification in our production BC environments?

I see that in BC version 20.0.34635 mentioned inconsistency is solved.


How to generate a detection result figure like Figure 5 in the paper?

How to generate a detection result figure like Figure 5 in the paper?
Thanks!!!

Hi, for visualization, you can use the commend CUDA_VISIBLE_DEVICES="0,1,2,3,4,5,6,7" python eval.py -d="0" -n "dataset_name" -e="epoch_number" -p="visualize_savedir". The visualization result will be saved in the path specified by -p="visualize_savedir".


Misc

Added favicon
fixed a file upload issue
fixed spelling (attachement -> attachment)

Current coverage is 60.38% (diff: 100%)

Merging #508 into release will not change coverage

@@            release       #508   diff @@
==========================================
  Files            59         59
  Lines          2903       2903
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1753       1753
  Misses         1150       1150
  Partials          0          0

Powered by Codecov. Last update e386641...3a25d2e

Most immediately noticeable change:

@cjh1 some small changes ready to be taken a look at.
Should we still be merging into release?

@TristanWright I will take a look, if they are low risk target them a release.

Latest commit addresses #507 table list sorting, the changes in this PR are pretty surface level and don't take a ton of looking into like #497 i.e. no redux actions are involved changes are solely within individual components.

LGTM


Convert opentracing tags to zipkin tags.

This fixes #13.

Merged :)
Btw, you don't have to create a new PR every time you change a commit: just force push to your branch and github will update the PR.

:D, My first PR. But sorry, this PR fixes #8, not #13. Please reopen #13.

:D, My first PR

Congrats!
Btw, you might want to complete https://docs.google.com/forms/d/e/1FAIpQLSfacQHac-PYIDIrDOi_W2l7cWxBDRIVBs6xE5HnlojPvdhL9g/viewform :)

I submitted.


Click active tab-bar icon will reload current tab page

I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

For instance, if current active page is Schedule and the tab-bar shows the active schedule icon highlight.
Then if click the schedule icon again even it's already active, it will reload schedule page.
Expected behavior:

if the app's Ionic version is 2.x and angular 2.x , nothing would be happened after click an active tab-icon.

Thanks for the issue! Are you seeing this on the master branch of the conference app? I'm not able to reproduce this.

I think this is a bug with ionic and has been patched in the master branch. I also get this problem. Clicking the same icon multiple times reloads the page and eventually a black screen. Easy to duplicate on lower end devices.
https://github.com/driftyco/ionic/pull/11084/files/a16e076c80d065e4f36c81952fa4c9b9d685fa8c

@brandyscarney ,Not on the master branch. I just update project as https://github.com/driftyco/ionic/blob/master/CHANGELOG.md#steps-to-upgrade instructed.

@pixelone , Yes, problem solved after update to the latest dependencies.


python312Packages.snapcast: 2.3.3 -> 2.3.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.snapcast is: Control Snapcast, a multi-room synchronous audio solution
meta.homepage for python312Packages.snapcast is: https://github.com/happyleavesaoc/python-snapcast/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.4 with grep in /nix/store/4aa395nqhbii2k2bkidnfxmyfcx3lwzg-python3.12-snapcast-2.3.4
found 2.3.4 in filename of file in /nix/store/4aa395nqhbii2k2bkidnfxmyfcx3lwzg-python3.12-snapcast-2.3.4

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.snapcast
python311Packages.snapcast
python312Packages.snapcast

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4aa395nqhbii2k2bkidnfxmyfcx3lwzg-python3.12-snapcast-2.3.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.snapcast https://github.com/r-ryantm/nixpkgs/archive/0b376ff6e05a402fcad9a073725277784cc245e3.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0b376ff6e05a402fcad9a073725277784cc245e3#python312Packages.snapcast

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4aa395nqhbii2k2bkidnfxmyfcx3lwzg-python3.12-snapcast-2.3.4
ls -la /nix/store/4aa395nqhbii2k2bkidnfxmyfcx3lwzg-python3.12-snapcast-2.3.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.snapcast
python311Packages.snapcast
python311Packages.snapcast.dist
python312Packages.snapcast
python312Packages.snapcast.dist

Maintainer pings
cc @peterhoeg for testing.

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

5 packages built:

home-assistant-component-tests.snapcast
python311Packages.snapcast
python311Packages.snapcast.dist
python312Packages.snapcast
python312Packages.snapcast.dist


Support https for kubernetes-dashboard

What this PR does / why we need it:
For kubernetes v1.9, default dashboard was turn on with https: https://github.com/kubernetes/kubernetes/pull/53046
Thus default dashboard location will turn to:
http://localhost:8001/api/v1/namespaces/kube-system/services/https:kubernetes-dashboard:/proxy/
This will fail with tls: oversized record received with length 20527 if dashboard does not use https
Which issue this PR fixes
Related:

https://github.com/kubernetes/dashboard/wiki/FAQ#im-accessing-dashboard-over-https
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.9.md

Updated Dashboard add-on to version 1.8.0: The Dashboard add-on now deploys with https enabled. The Dashboard can be accessed via kubectl proxy at http://localhost:8001/api/v1/namespaces/kube-system/services/https:kubernetes-dashboard:/proxy/. The /ui redirect is deprecated and will be removed in 1.10.

Special notes for your reviewer:
The old kubernetesmasteraddons-kubernetes-dashboard-deployment.yaml was copied to kubernetesmasteraddons-kubernetes-dashboard-http-deployment.yaml.
https related changes were applied to kubernetesmasteraddons-kubernetes-dashboard-http-deployment.yaml directly.
Release note:
Support https kubernetes dashboard

One quick comment.  I'd like to see us deploy the new dashboard even for older 1.7/1.8 clusters assuming that works...

A detailed versioning matrix could be found here: https://github.com/kubernetes/dashboard/wiki/Compatibility-matrix
It mentions that 1.8 dashboard will only fully support 1.8+ cluster.
I've tested with acs-engine generate cluster of following versions to use the 1.8 dashboard with HTTPS:
v1.6.13
v1.7.10

http://localhost:8001/ui will redirect to http location:http://localhost:8001/api/v1/namespaces/kube-system/services/kubernetes-dashboard/proxy/
Manually navigating to http://localhost:8001/api/v1/namespaces/kube-system/services/kubernetes-dashboard/proxy/ will load the ui, but the page reports an error:
the server could not find the requested resource

v1.8.4

http://localhost:8001/ui will redirect to http location:http://localhost:8001/api/v1/namespaces/kube-system/services/kubernetes-dashboard/proxy/
Manually navigating to http://localhost:8001/api/v1/namespaces/kube-system/services/kubernetes-dashboard/proxy/ works

v1.9.0
v1.10.0-alpha.1

Works well


bazel installation error

I am using tx2 and getting into this error.
There is no file or directory availalbe
Can you please help?
/gcc --config=cuda
INFO: Found applicable config definition build:cuda in file /home/nvidia/JetsonTFBuild/tensorflow/tools/bazel.rc: --crosstool_top=@local_config_cuda//crosstool:toolchain --define=using_cuda=true --define=using_cuda_nvcc=true
ERROR: Config value opt is not defined in any .rc file
./tf_build.sh: line 29: bazel-bin/tensorflow/tools/pip_package/build_pip_package: No such file or directory
Thanks,
Shreyas

which method are you trying to install tensorflow ? also what version of Jetson are you running ? 4.2?


change "x" icon in clearable section

Is there any option to change "x" for clear selection
since i used non regular font, select2 "x" showing nothing
thanks
for now i do it like this :
$('.select2').on('select2:select', function (e) {
  $('.select2-selection__choice__remove').html('x');
});

Unfortunately no. And given the complexity around keeping things accessible, we probably aren't going to implement this.


It dosn't work on<EMAIL_ADDRESS>

I tried to modify it, but it didn't work

The sample seems to run fine with version 132.2. What error are you receiving?

I see, this issue has been confirmed. This was the first report, but #59 has a little more info.


Set various flags for environment creation

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #10   +/-   ##
======================================
  Coverage      94%     94%
======================================
  Files         171     171
  Lines       29198   29198
======================================
  Hits        27256   27256
  Misses       1942    1942

Continue to review full report at Codecov.

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


Add Battle Simulation

Impressive

here are the stats for the training camp, have no account that has the new sandstorm unlocked so can not help there

name
floor
class
level
health
xp
strength
dexterity
intelligence
constitution
luck

TrainingCamp
0
Mage
1
50
80
5
5
10
10
5

TrainingCamp
1
Mage
2
120
180
10
10
20
20
10

TrainingCamp
2
Scout
3
480
280
15
30
15
30
15

TrainingCamp
3
Warrior
4
1000
360
40
20
20
40
20

TrainingCamp
4
Warrior
5
1500
440
50
25
25
50
25

TrainingCamp
5
Mage
6
840
578
30
30
60
60
30

TrainingCamp
6
Scout
7
2240
716
35
70
35
70
35

TrainingCamp
7
Warrior
8
3600
881
80
40
40
80
40

TrainingCamp
8
Scout
9
3600
1071
40
80
40
90
45

TrainingCamp
9
Mage
10
2200
1287
40
40
80
100
50

I can help with all Dungeons except sandstorm.

Are you refering to this or this? I know they have a bunch of stuff figured out already, but I want to keep away from  copy-pasting code from other people. Especially damage calcs are often different/wrong between sf-tools/mfbot/private-servers and I don't want to get into "you stole my code" arguments later down the line, since none of them have any open source license.
If you have any good resource I am not aware of, feel free to share them

Are you refering to this or this? I know they have a bunch of stuff figured out already, but I want to keep away from copy-pasting code from other people. Especially damage calcs are often different/wrong between sf-tools/mfbot/private-servers and I don't want to get into "you stole my code" arguments later down the line, since none of them have any open source license.
If you have any good resource I am not aware of, feel free to share them

Can you please remove that comment? What you call "p server" is actually leaked playa code, so yea... The order of applying bonuses should be just about right ;) Not sure why one would consider that "stealing code", it is looking at the ground truth for an aspect, that would otherwise take unreasonable amounts of data collection and analysis. I am not talking about taking anything. There are just some aspects, that are not really observable without having a peek inside the black box.
That is by the way something I now know for a fact SFTools messed up, not sure if it was always broken or the discrepancy was introduced by an update on playas side.
On a side note I SFTools has a green light on embedding their compiled client. From what I can tell it's only used for authentication and data fetching though, not sure what the reason is he did't use more of the "real deal"-code, but again... I quickly loose patience when reading SFTools code.


Moderation commands

this pr adds moderation commands to androgee

This isn't quite what I had in mind for moderation with Androgee.
For user moderation I was thinking that -spray @xeon would do one of the two things:

Show a gif & an @ spraying xeon
If the calling user is a demigod, do 1 & add the muted role to them for a default of 60 seconds

Mee6 has robust user moderation tools and there's no reason to add redundant functionality with Androgee. Androgee's role should be community specific, for example, the -spray/-bonk meme is specific to our community when people become too rowdy.


Broken on latest master

On Julia 4a1350090b79bd667745c65a3d87cbc0a222f385
julia> @enter gcd(5, 20)
ERROR: type Array has no field args
 in prepare_locals(::LambdaInfo, ::Array{Any,1}) at /Users/viral/.julia/v0.5/ASTInterpreter/src/ASTInterpreter.jl:862
 in enter_call_expr(::Void, ::Expr) at /Users/viral/.julia/v0.5/ASTInterpreter/src/ASTInterpreter.jl:907
 in eval(::Module, ::Any) at ./boot.jl:237

@JeffBezanson has a branch which needs to be merged.

Fixed.


[MXNet] add disable static option in DJL to allow some model running

Description
fix #1731 .
This PR is aimed to allow user pass in disable static shape options.

Codecov Report

Merging #1735 (0c7f0b5) into master (bb5073f) will decrease coverage by 1.59%.
The diff coverage is 70.74%.

:exclamation: Current head 0c7f0b5 differs from pull request most recent head 577c40f. Consider uploading reports for the commit 577c40f to get more accurate results

@@             Coverage Diff              @@
##             master    #1735      +/-   ##
============================================
- Coverage     72.08%   70.49%   -1.60%
- Complexity     5126     5540     +414
============================================
  Files           473      525      +52
  Lines         21970    24632    +2662
  Branches       2351     2665     +314
============================================
+ Hits          15838    17364    +1526
- Misses         4925     5942    +1017
- Partials       1207     1326     +119

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
8.33% <0.00%> (-0.50%)
:arrow_down:

... and 407 more

Continue to review full report at Codecov.

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


Missing monorail.ipxe

I have noticed that in your 'example' deployment the vagrant image comes pre-loaded with the monorail.ipxe image. However in the on-tftp code repository there is no reference to this file other than the static/tftp directory where it is supposed to exist. Nor is there an NPM script to use in order to acquire it.
Where are we supposed to get the file from?

Evening @jimmypw,
monorail.ipxe is only one file needed - there's several. The files are available in binary form to download from bintray, and the Ansible role that we use in creating our example  Vagrant instance shows the details at https://github.com/RackHD/RackHD/blob/master/packer/ansible/roles/images/tasks/main.yml
The files themselves can be "rebuilt from source" following the instructions within the on-imagebuilder repo (https://github.com/RackHD/on-imagebuilder/) - we use TravisCI to build the binaries and host them on bintray for our example setup.


Added Latin Parser

Ancient Latin language Parser added.
This parser only supports numbers from 0 to 9999 due to limitations of
ancient Roman arithmetical prowess. The parser will give a Latin
language error messages if a number is outside of this range but still
inside the range of other parsers.

Very cool, I've never thought about adding this language! Great historical note you supplied for the negative numbers too.
Wouldn't the range be up to 999,999? e.g. 100,000 => 'centa milium'? Could you say 1 million by saying 'milia milium'?

This needs to be rebased now, if it stays stale I'm going to close it.


Multiple select not working

While working with the multiple select option on local machine, I was able to select multiple things, but once I did "grunt build" to get a prod ready version of my application, i suddenly started getting an error:

app/select2.png not found.

after this, I see that the multiple select dropdown is no more allowing me to select multiple options.
Before running "grunt build":

After running "grunt build", and running the dist version:

If you see the above images, the font awesome icons to select-all and deselect-all are also not there.

Solved this issue, When doing grunt build of the angular project, the generated script skips the part multiple=true instead it only has multiple. Also, there's an error with the CSS file path reference.
To solve, this either you can manually go-to the .JS file generated and add =true after multiple


options width and height

Hi, I try to resize my graph and follow the default example and adding
var options = {
width : 800,
height : 600
}
However, it has no effect. Tried on different browsers.
Could you tell me how to fix it?

Hej, please try
var options = {
  width: '800px',
  height: '600px'
};

These parameters should be strings.

Thanks Silsha. Got it. It works.
Also, I forgot to put 'options' inside Chartist.line ... sorry, rookie mistake.
Please close this one.

But now the chart is not responsive after we added height!

@sydorT If you want to keep the chart responsive, then you have to use the available aspect ratios. To use them, just add the correct class (e.g. .ct-major-tenth) to the chart container.
https://gionkunz.github.io/chartist-js/getting-started.html#create-your-first-chart


Don't return ServiceInstance in resolveClusterReferences and related methods

None of the methods is replacing the ServiceInstance, so there's no need
to return it (the returned value is always the same as the value passed
into the method).

/approve


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/668 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target.git

Docs Build status updates of commit 69c9e71:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-12-15-05-59-50 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-12-15-05-59-50

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Invalid call to 'GetValue' when Training Named Entity Recognition Data

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog): <IP_ADDRESS>5501
Visual Studio Version : 17.9.4

Describe the bug

On which step of the process did you run into an issue: Train
Clear description of the problem: When training data, the system processes the data, then throws an error "Invalid call to 'GetValue'"

To Reproduce
Steps to reproduce the behavior:

Select scenario
Select Environment (both CPU and GPU have error)
Add data
Train

Expected behavior
Training complete and moved to Evaluate section
Screenshots

Additional context
Error log:
at Microsoft.ML.DataViewSchema.Annotations.GetValue[TValue](String kind, TValue& value)
at Microsoft.ML.Data.SchemaAnnotationsExtensions.GetKeyValues[TValue](Column column, VBuffer1& keyValues)    at Microsoft.ML.TorchSharp.NasBert.NerTrainer.Trainer.GetRowCountAndSetLabelCount(IDataView input)    at Microsoft.ML.TorchSharp.TorchSharpBaseTrainer2.TrainerBase..ctor(TorchSharpBaseTrainer2 parent, IChannel ch, IDataView input, String modelUrl)    at Microsoft.ML.TorchSharp.NasBert.NasBertTrainer2.NasBertTrainerBase..ctor(TorchSharpBaseTrainer2 parent, IChannel ch, IDataView input, String modelUrl)    at Microsoft.ML.TorchSharp.NasBert.NerTrainer.CreateTrainer(TorchSharpBaseTrainer2 parent, IChannel ch, IDataView input)
at Microsoft.ML.TorchSharp.TorchSharpBaseTrainer2.Fit(IDataView input)    at Microsoft.ML.Data.EstimatorChain1.Fit(IDataView input)
at Microsoft.ML.AutoML.SweepablePipelineRunner.Run(TrialSettings settings)
at Microsoft.ML.AutoML.SweepablePipelineRunner.RunAsync(TrialSettings settings, CancellationToken ct)
at Microsoft.ML.AutoML.AutoMLExperiment.d__24.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.ML.ModelBuilder.AutoMLService.Experiments.NamedEntityRecognitionExperiment.d__13.MoveNext() in //src/Microsoft.ML.ModelBuilder.AutoMLService/Experiments/NamedEntityRecognitionExperiment.cs:line 133
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.ML.ModelBuilder.AutoMLEngine.d__21.MoveNext() in //src/Microsoft.ML.ModelBuilder.AutoMLService/AutoMLEngineService/AutoMLEngine.cs:line 199
Files:
NERKeyMap.txt
NER_Data.txt
Note: Data shortened to 1 training item for example, training data has 2000+ items over all, issue occurs on any number of items.

@zewditu Can you take a look


[Container Registry] Anonymous Access Client

#17755

Dependent on #18392

/azp run python - containerregistry - tests


Add support for ChildAggregates

The feature is still experimental, see here https://github.com/patchlevel/event-sourcing/pull/614

This is not needed anymore due to the implementation allows using the constructor.


compatibility process escape in string literal

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.21 seconds
load time: 435 seconds
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221115131343_clickbench_pr_46540.html


Add mixins folder

This moves several files to a new mixins/ folder. I was going to add
some new macros, per this comment, but thought it might clutter
things up a bit to put them in the root folder. I thought of adding a
macros/ folder, but decided to emulate the rubocop source
instead.

Before submitting the PR make sure the following are checked:

[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[ ] Added tests.
[ ] Updated documentation.
[ ] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[X] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

Hmm, is this failure related to something on master?

Not sure why the build is failing on this. It works for me locally.

Not sure why the build is failing on this. It works for me locally.

I think you’re blocked by #955.

#955 is merged. Please rebase. Sorry for the trouble.

Adjusted module namespace and rebased.
Let's wait for Github to catch up.

That's an option. Depends on how essential this is to move #863 forward.

Unfortunately, as you may have noticed, I haven't been able to be very consistent on following up with these PRs. I don't think #863 depends on this, since, if I remember correctly it was built on #934 instead. I was picturing adding some more macros and using those instead, but we could stick with #863 as is if you want to get that out sooner.

No rush, take your time and make any additions you see fit in #863.
We'll take care of this pull request later.
Even though RuboCop 1.0 and RuboCop RSpec 2.0 are around the corner, there's a fat list of leftovers there and something to work on here as well, so I guess it's not going to happen earlier than in 3-4 weeks.

I'll take care of rebasing this.
This is optional, but a nice cleanup nevertheless.

Superseded by #1056
Thanks, @mockdeep, this will become part of the 2.0 release!

@pirj thank you for rebasing!


Replace references to legacy tedge plugins with tedge-agent in docs

Is your documentation request related to a problem? Please describe.
After the tedge-agent was updated to provide both configuration management and log management capabilities as well, the corresponding docs need to be updated to replace the references to old tedge-configuration-plugin and tedge-log-plugin with tedge-agent as the daemon that provides these functionalities.
Describe the improvement you'd like
The following docs still have references to the old plugins:

getting-started.md
tedge-agent.md
operate/c8y/config_management_plugin.md
operate/c8y/tedge_log_management.md
domain-model.md
update_config_paths.md
DEBIAN_PACKAGE_HOSTING.md

The references to the legacy plugins have been removed from the above listed docs.

Looks OK


Integration Guide / NodeJS: example email with from<EMAIL_ADDRESS>always results in error

I am new to sendgrid and want to complete the integration guide to finish my account. I am using nodeJS, I start the "Integration Guide", the guide tells me to do the following:
const sgMail = require('@sendgrid/mail');
sgMail.setApiKey(process.env.SENDGRID_API_KEY);
const msg = {
  to<EMAIL_ADDRESS>  from<EMAIL_ADDRESS>  subject: 'Sending with Twilio SendGrid is Fun',
  text: 'and easy to do anywhere, even with Node.js',
  html: 'and easy to do anywhere, even with Node.js',
};
sgMail.send(msg);

Expected content
The code in the guide should work without errors, so I can complete the guide.
Actual content
ERROR: The from address does not match a verified Sender Identity. Mail cannot be sent until this error is resolved. Visit https://sendgrid.com/docs/for-developers/sending-email/sender-identity/ to see the Sender Identity requirements
Thus I can not complete the guide and "unlock" my account completely.
Please help!

Hello @Twisterking ,
<EMAIL_ADDRESS>is a placeholder value. You'll need to replace it with the email address you used when completing sender verification. If you still need to verify your email address, you can find instructions here: https://sendgrid.com/docs/for-developers/sending-email/sender-identity/
We hope this helps you troubleshoot, and thanks for getting started with SendGrid.
Best,
Twilio SendGrid Docs Team

We hope the information above solved your issue. We're closing this GitHub Issue for now. If you encounter more trouble related to the above issue, please reopen the Issue.
Thank you,
Twilio SendGrid Docs Team

Yes well it kinda helped. Thank you! :)
If you could tell your colleagues in the support team to finally answer my tickets (especially #4774650), that would be great.
There seems to be no way to reach your support team, which is really a bummer since I pay 90 USD per month for the Pro plan!

Sorry for the delay. I know our support team has been very busy. I have passed that ticket number to them.

Got a reply - finally! Thank you!


LDAP template fails to deploy

https://github.com/F5Networks/f5-appsvcs-templates/blob/master/templates/bigip-fast-templates/ldap.yaml
BIG-IP v16.1.0 build 0.0.19
f5-appsvcs-templates version 1.12.0, build 1
f5-appsvcs version 3.31.0, build 6
Please consider correcting these 2 issues with the LDAP template:

The "FAST- Generated Pool" option incorrectly offers default port "80" for the pool members instead of "389".
The "FAST-Generated Pool Monitor" option does not display the required "password" field, which prevents the deployment from succeeding. The resulting error info is as follows: "/test/test/test_monitor/passphrase/ciphertext: should NOT be shorter than 2 characters"

Thanks for reporting these issues. We are now tracking them internally with ID MYSTIQUE-456.

Closing.
The fix for this was included with Release 1.13.0.


Do the same audit that etl repo did to fininfra airflow

Per Jim, they are only using two operators: pythonOperator and financialETLOperator which is a subclass of bashOperator.
So the following change should cover the fin-infra use case.

cc @jimcchiang @chetcode

👀 @l201514

@jimcchiang do you need siqi @l201514 for further review or it is ok to merge?


near query parameters

When querying near leaving out the $minDistance parameters it fails.
near=12.1234,13.1345,2000
also when using a float for the $maxDistance it fails for me
near=12.1234,13.1345,2000.1,0

Thanks for the bug-report @joshkopecek. I have confirmed the bugs and I am working on a fix.

Both bugs are fixed in v4.0.2. Thanks for reporting @joshkopecek!


Send to... dialogue does not set schedule type

When selecting e.g. "send to text2img" or "send to img2img," schedule type is set to automatic rather than the schedule type from the sent image.
Is this intended? Am I missing a setting that would cause this?
Thank you so much for this extension. It's absolutely incredible. I still can't believe someone made one this complicated for sd webui. Seriously thanks! :)

Sorry, I don't know why either. The "Send To" feature of IIB is actually simulated through several hidden Gradio components. This part of the code hasn't been touched for a long time.
sd-webui-infinite-image-browsing:
https://github.com/zanllp/sd-webui-infinite-image-browsing/blob/6488c51f70da9dc205da903029454c65bf897ed5/scripts/iib_setup.py#L51-L64
When I implemented it, I referred to the source code of sd-webui:
https://github.com/AUTOMATIC1111/stable-diffusion-webui/blob/feee37d75f1b168768014e4634dcb156ee649c05/modules/ui.py#L875-L879

Thanks for the info! I'll check upstream.


[MINOR] Remove unuse module

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / 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:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3116 (2b8abbe) into master (7865da1) will decrease coverage by 42.67%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3116       +/-   ##
============================================
- Coverage     45.85%   3.17%   -42.68%
+ Complexity     5269      82     -5187
============================================
  Files           909     272      -637
  Lines         39336   10510    -28826
  Branches       4239    1073     -3166
============================================
- Hits          18036     334    -17702
+ Misses        19455   10150     -9305
+ Partials       1845      26     -1819

Flag
Coverage Δ

hudicli
?

hudiclient
0.00% <ø> (-30.45%)
:arrow_down:

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
6.72% <ø> (-45.01%)
:arrow_down:

huditimelineservice
?

hudiutilities
10.00% <ø> (-46.64%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Add Kong Gateway to Plugin Hub title

Summary
Add Kong Gateway to Plugin Hub title
Reason
Aghi request
Testing
/hub/

@mheap Are the Netlify checks expected to fail on this PR? I'm not sure what is causing them or how to fix.

It was a broken cache issue. I triggered a rebuild but the commit status didn't update (the comment did)