declaration: Namespace which exports class can't be merged with variable

TypeScript Version: 2.2.0-dev.20170117
I try to write declaration for got module.
Code
// Use interface, because `got` function has `delete` property.
interface GotApi {
    (): any;
    delete(): any;
}

declare var got: GotApi;

declare namespace got {
    export class HTTPError extends Error {}
}

export = got;

Expected behavior:
No errors. HttpError class is available using got exported variable:
import * as got from 'got';

// let error = ...
if (error instanceof got.HTTPError) {
}

Actual behavior:
I get compiler errors for declaration file:
index.d.ts(6,13): error TS2300: Duplicate identifier 'got'.
index.d.ts(8,19): error TS2300: Duplicate identifier 'got'.

Used tsconfig.json:
{
    "compilerOptions": {
        "module": "commonjs",
        "moduleResolution": "node",
        "target": "es2015",
        "noImplicitAny": true,
        "strictNullChecks": true,
        "outDir": "build"
    }
}

Related issue: https://github.com/Microsoft/TypeScript/issues/1784

Note that the GotApi interface is not exported

Note that the GotApi interface is not exported

How interface should be exported? This code (similar to request declaration) also isn't compiled with the same error:
declare namespace got {
    export interface GotApi {
        (): any;
        delete(): any;
    }

    export class HTTPError {}
}

declare var got: got.GotApi;
export = got;

If remove class export from namespace, compilation is completed:
interface GotApi {
    (): any;
    delete(): any;
}

declare var got: GotApi;

declare namespace got {
    // export class HTTPError extends Error {} <--- Comment this line
}

export = got;

Ideally this would be rewitten as function+namespace but it can not, https://github.com/Microsoft/TypeScript/issues/1784 (and https://github.com/Microsoft/TypeScript/issues/9846) track that.
Instantiated namespaces (i.e. ones that has a value in them) can not merge with variables, since one of them will override the other at run-time.
you can write your declaration as such:
declare class HTTPError { }

interface GotApi {
    (): any;
    delete(): any;
    HTTPError: typeof HTTPError;
}

declare var got: GotApi;
export = got;

Instantiated namespaces (i.e. ones that has a value in them) can not merge with variables, since one of them will override the other at run-time.

As I understand from documentation about declaration merging function and variable are value creating declarations, i.e.
interface GotApi {
    (): any;
}
declare var got: GotApi;
declare namespace got {
    export class HTTPError {}
}
export = got;

and
interface GotApi {
    (): any;
}
declare var got: GotApi;
declare namespace got {
    export class HTTPError {}
}
export = got;

both fails to compile. But function declaration compiles:
declare function got(): any;
declare namespace got {
    export class HTTPError {}
}
export = got;

Why this happen?
Also, I propose to make error message is more detailed, because current message error TS2300: Duplicate identifier 'got'. is cryptic at first glance. For example, `namespace and variable declaration can't be merged'.

var got: ()=> any is treated as a variable, and not as a function declaration. the reason is a variable can be assigned to later on where as a function can not.

@mhegazy Thanks for explanation.

@mhegazy So why is it then not allowed to merge a declare const with a declare namespace? AFAIK, const is also not assignable more than once.

Merging consts with namespaces really makes sense. Please consider reopening this issue. It's not a duplicate #1784.
For example, let's take TinyMCE writing proper type definitions for which require this feature. Namespaces are really convenient for describing types for this library because all the classes from the API are attached to the tinymce namespace object or to its subnamespaces. In the API, there are normal classes like tinymce.Editor and so called 'static classes' like tinymce.EditorManager which are just a type (interface) and an object value of this type attached to the namespace:
declare namespace tinymce {
  // normal class
  class Editor {
    show(): void;
    // ...
  }
  // "static class"
  interface EditorManager {
    activeEditor: Editor;
    // ...
  }
  const EditorManager: EditorManager;
  // ...
}

Nothing unusual so far, but there is a plot twist. The namespace object tinymce itself implements the EditorManager interface. It could be easily solved by merging a const with the namespace:
declare const tinymce: tinymce.EditorManager;

But unfortunately this isn't allowed.


Track stats for people engaging with get.bacalhau.org (version, dates, IPs through a system we own)

Tracked through the gated downloads epic. https://github.com/orgs/bacalhau-project/projects/17?pane=issue&itemId=41107355


feat #7 A new class for the adhoc coverage tool

The class will hold public arrays for the tool to use.
Fixes #7

Variable name "m" might be a bit bad :disappointed:


Support Composer Installs

With Drupal 8 leveraging composer more and more, it would be great to be able to specify sites to be installable via composer as well (currently DrupalVM appears to support just drush make?).

@nateswart - So, as someone who has never installed Drupal using Composer before, but has installed plenty of other PHP apps, what would this process even look like. AFAICT, there's still an open debate over whether Drupal.org will support Packagist or build/host it's own Drupal-specific packagist.

Shameless Plug: https://github.com/drupal-composer/drupal-project provides a good template if you want to user composer with D8.

Thanks @webflo - we're actually already using that. Nice work!
@geerlingguy The more I think about it, the less I think this is required. Having composer installed on the VM by default is enough. Closing.

@nateswart is there any documentation on how to use a composer workflow instead of drush make in a VM build? There isn't anything immediately available in the docs.

@emarchak - Please follow the discussion in #506; we'll be switching Drupal VM to use Composer by default soon (hopefully!), and there may be some fruitful discussion in that issue.


Question on FullTextSearch with field containing URL in F#

Hey there,
First off, just wanted to apologize in advance if I'm completely missing something here as I'm not super familiar with how Postgres' built-in text search functions work under the hood.
I'm currently messing around with using Marten in an F# project and I had a quick question on an issue I'm running into. I'm currently trying to use full text searching with an F# record representing an aggregate with this structure:
[<CLIMutable>]
type MealAggregate =
    { Id: Guid
      Name: string
      Protein: string option
      Source: string option
      Created: DateTime
      Updated: DateTime option
      Deleted: DateTime option }

Everything standard with regards to writing and retrieving the data is working as expected. I'm using inline projections and the row is being written correctly to mt_doc_mealaggregate (FSharp.SystemTextJson is being used for serialization of F# types).
My problem is with searching against one specific field of the record. My search functionality is currently written as such:
let private getNonDeletedItemsQuery<'T> (session: IQuerySession) =
    query {
        for aggregate in session.Query<'T>() do
            where (aggregate.MatchesSql("data->> 'Deleted' IS NULL"))
            select aggregate
    }

let search<'T> (documentStore: DocumentStore) (text: string option) =
    let session = documentStore.QuerySession()

    let queryable = getNonDeletedItemsQuery<'T> session

    let result =
        match text with
        | Some parameter ->
            query {
                for aggregate in queryable do
                    where (aggregate.WebStyleSearch(parameter))
                    select aggregate
            }
        | None -> queryable

    task {
        let! aggregates = result.ToListAsync()
        return aggregates |> Seq.toList
    }

As you can see, I'm trying to run a WebStyleSearch on the aggregate record. My problem is that virtually every field of the aggregate is searchable except for the Source field which specifically contains URL's.
Here's an example of a stored projection:
{
    "Id": "402680f5-b29c-4136-be1b-22866a3e4314",
    "Name": "Greek Chicken",
    "Source": "https://example-recipes.com/greek-chicken",
    "Created": "2024-08-15T19:04:51.009561Z",
    "Deleted": null,
    "Protein": "Chicken",
    "Updated": "2024-08-15T19:06:19.331963Z"
}

So in the example above, searching for "greek" would return the record, while searching for "recipes" would not. I thought it might be a problem with F# Option types, but the Protein field is also an option and searching on that works fine as well.
I wanted to check if I was potentially missing some additional configuration or if this might be a limitation of Postgres that I'm not familiar with?
Also, in case it's helpful, here's the creation of my DocumentStore:
DocumentStore.For(fun options ->
        options.Connection(connectionString)
        options.AutoCreateSchemaObjects <- AutoCreate.All

        // Projections
        options.Projections.Snapshot<MealAggregate> SnapshotLifecycle.Inline |> ignore

        // Schema
        options.Schema.For<MealAggregate>().FullTextIndex() |> ignore

        let serializer = Services.SystemTextJsonSerializer()
        serializer.Configure(fun options -> options.Converters.Add(JsonFSharpConverter()))
        options.Serializer serializer)

Thanks in advance!

If your using web style search then I would suggest having a look at websearch_to_tsquery and running your search text through that to see what the result vectors look like that are being searched for.  Additionally running your source values (the url for example) through to_tsvector and seeing how that is translated.  I'm pretty
Under the hood the full text index in postgres tokenises the indexed value.  It does break apart urls, but I don't believe it does so in a way that would support your search use case e.g.
select to_tsvector('english', 'https://example-recipes.com/greek-chicken');

Will tokenise as:
'/greek-chicken':3 'example-recipes.com':2 'example-recipes.com/greek-chicken':1

None of those will match on a search term of recipe (or even using a 'starts with' lexeme with prefix matching wont work, because none of the tokens start with the word recipe.
In this instance you might want to change your approach to combine a free-text search with a like %term% type query over just the data->>Source property (though depending on how much data you have that will could be quite slow as it's likely to require a full table scan).  Alternatively you could tweak the raw DocumentConfig of the indexes configuration in Marten to manipulate the data->>'Source' property being indexed to break up the url in a way that better fits your search use case (e.g. maybe replacing all non-letter characters with spaces via a regex).

@bittercoder Thanks a ton for the response.
With regards to your last suggestion of tweaking the index for the data->>'Source' property, are you referring to something along the lines of using a Calculated Index or adding this transformation elsewhere?
For what it's worth, I tried adding a calculated index to that field just to test it out:
let getSourceIndex = FunctionAs.LinqExpression<MealAggregate, obj> (fun (aggregate: MealAggregate) ->
    match aggregate.Source with
    | Some source ->
        Regex.Replace(source, @"[\W_]+", " ") |> box
    | None -> "" |> box)

options.Schema.For<MealAggregate>().Index(getSourceIndex, (fun x -> x.Name <- "test_index")) |> ignore

(The compiler wasn't playing nicely with automatically converting the F# function to a LINQ expression so I had to do it)
The index creation succeeded, but resulted in the following:
CREATE INDEX IF NOT EXISTS test_index
    ON public.mt_doc_mealaggregate USING btree
    ("((data -> 'Source'::text) ->> 'Value'::text)" COLLATE pg_catalog."default" ASC NULLS LAST)
    TABLESPACE pg_default;

which doesn't look like it's completing the Regex transformation I need. I don't see any examples for creating indexes in the docs outside of relatively simple property selection (_.Schema.For<User>().Index(x => x.UserName);), so I assume I'm just trying to do something unsupported.
Either way, greatly appreciate your help.

I was more meaning changing the how the fulltext index statement is generated, here's a rough C# example that should hopefully help:
    public static void ConfigureMarten(DocumentMapping<Contact> mapping)
    {
        Expression<Func<MealAggregate, object>>[] expressions =
            [x => x.Id!, x => x.Name!, x => x.Protein!, x => x.Source!, x => x.Created!, x => x.Updated!, x=>x.Deleted!];

        var members = expressions
            .Select(FindMembers.Determine).ToArray();

        var regexCleaningDocumentConfig = StringExtensions.Join(members.Select(x => $"REGEXP_REPLACE(COALESCE((data ->> '{x[0].Name}'),''), '[\\W_]+', ' ')"), " || ' ' || ");

        var index = mapping.FullTextIndex("english", expressions);
        var documentConfig = $"({regexCleaningDocumentConfig})";

        // handle most values being nullable
        index.DocumentConfig = regexCleaningDocumentConfig;
    }

I haven't tested the above code, so there may well be problems - but hopefully this is enough to help you find a potential solution.


introduce jenkins v2 image

@gabemontero ptal.  rhel image won't merge/pass tests until the new plugin is rpm-packaged and the jenkins 2.7 package exists.
otherwise things went pretty smoothly.

You are already responding :-) but to summarize, a couple of minor comment suggestions and questions, along with the "should we use symlinks for files which seemed the same between v1 and v2?".
Otherwise, totally cool that things went smoothly - IGTM

OK ... symlink question resolved (we can always change it later if updating 2 versions gets too onerous and we satisfy our Docker build concerns).
LGTM

my usual process for managing the version diffs is to run a diff between the "1" and "2" directories and make sure the only deltas are ones that make sense.  It is definitely not free, but so far it's been a manageable process and really it's one you need to do anyway since there could be files that have small differences and thus can't be symlinked anyway, but still need to stay largely the same.

[test]

[test]

[test]

[test]

[test]

[test]


Add support for profile option

The status script could only be used with default AWS CLI credentials
because the profile option was missing.

Declares profile_opt variable
Use profile_opt variable in describe-stack-events call

This would be nice to get merged, i have done the same thing already 👍


CLI argument parsing IndexError with =

I typed = instead of - and the CLI crashed showing a stacktrace.
$ az storage blob list =c ''
string index out of range
Traceback (most recent call last):
  File "/Users/debekoe/Repos/derekbekoe-azure-cli/src/azure-cli/azure/cli/main.py", line 34, in main
    cmd_result = APPLICATION.execute(args)
  File "/Users/debekoe/Repos/derekbekoe-azure-cli/src/azure-cli-core/azure/cli/core/application.py", line 99, in execute
    if noun[0] == '-':
IndexError: string index out of range

The problem here is actually not with the = sign. It is trying to find a '-' in ''.


[otbn] Design for bronze

This is the OTBN design for bronze with a couple of commits pulled from master; the only "new" commit is "[otbn] Insert dummy gate count for bronze".
I cannot double-check if DC keeps the random gate generator, I'd appreciate if @msfschaffner or @eunchan could double-check that.
I also didn't hook up the RNG distribution network yet; AES doesn't add that connection yet either (it seems), let me know if there's something I should do. (Tracked in #2638.)

Merging into bronze to avoid more rebases as generated files change; we can fix follow-ups in bronze directly.


split doesn't work with default profile set to a contributed one

set default profile to a contributed one (js debug, for ex)
run the split terminal command via the command palette
it's not split

same issue if you use the keybinding to split the terminal (ctrl shift 5)


Mm bugfix

Corrected a Runtime Error (ReferenceError) encountered when running yarn test per example (from https://docs.near.org/docs/tutorials/contracts/cross-contract-calls)
Issue appear to be a stray line left in the file to variable that isn't defined or used elsewhere.
Also adjusted package.json to add a postbuild step to crate the default directory that near deploy expects (./out)  copy the contract.wasm into it as main.asm (default name) {similar to what I've seen in other examples}

Accepted

Reopening after cleaning my comments/description

Fixes issue Yarn jest fails to run with a Reference Error #201


Example of a Pangeo-Forge-type workflow using Apache Beam

This doesn't use the Pangeo-Forge library (yet), but may be a useful for
design discussions.
When I run this on Google's internal infrastructure, it processes a 1 TB
dataset from hundreds of netCDF files into Zarr (all already in
distributed storage) in 20 minutes or so. Presumably one could achieve
similar performance with Cloud Dataflow.
Compared to the existing NetCDFtoZarrMultiVarSequentialRecipe:

My equivalent of prepare_target (write_zarr_metadata) is performed
in parallel, on different workers. This is quite helpful for datasets
with moderately large numbers of variables and/or large (multi-GB)
input files. Otherwise waiting on downloading all of the source files
to the same machine to figure out the metadata becomes the slowest
step. This would probably require some modifications to the
"PipelineExecutor" model, to allow for collecting distributed inputs
for a downstream task.
I expand the time-dimension using xarray.concat rather than writing
and then modifying a Zarr array. This lets me write the metadata with
consolidated=True, which removes my need for anything like
finalize_target.
Because I create the time-variable in the destination Zarr file before
writing chunks and use xarray's indexing to lookup the right region,
I can handle variable length inputs (e.g., data stored in one file
per real-world month or year, which can have variable length).
By the looks of it, the existing recipes were designed to handle the
case of many input files -> one zarr chunk. My recipe was designed for
the opposite (one file -> many chunks). I think the general template
you have works for both, but the names iter_chunks and store_chunk
may be a little confusing because a "pangeo forge chunk" may
correspond to one or more "zarr chunks".

Thanks so much for this Stephan! Super interesting and useful.
I have a big deadline this week, but I plan to dig into this next week and see how we can incorporate some of your insights.

Now that we have the beam executor, I'm going to close this for the sake of cleaning up the list of pending PRs. Thanks a lot Stephan for getting the discussion started.


Implement progress indicator in speed read

https://github.com/michigan-com/nova/commit/8b9affa126d81d2cdd369cd709c5df095fc37340

https://github.com/michigan-com/nova/commit/c56b4fabc899e23c37ff68f47e133534ce0c8976


[BUG] type become "unknown" whenever i use charkaStyles or styles prop

Description
When i use the chakraStyles property or the styles property, the inferred type throughout the whole component becomes unknown.
I would have expected this to not make any difference to the inferred type.
Works fine without:

Unknown type:

chakraStyles:

chakra-react-select Version
3.3.3
TypeScript?

[X] Yes I use TypeScript

Operating System

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS/iPadOS
[ ] Android

Hmm I'll have to look into why this is happening specifically when you use chakraStyles (or styles). However, it may be something to do with the inner workings of react-select and not something I can change.
However, its relatively easy to fix by passing in the structure of your option object as a generic into your Select component.  Here's an example of how you could do that:
import type { OptionBase } from "chakra-react-select";

// If you use other fields like isDisabled, isFixed, colorScheme, or variant
// extend it from the type OptionBase for convenience
interface Option extends OptionBase {
  label: string;
  value: number;
}

// otherwise you can declare the interface on its own
interface Option {
  label: string;
  value: number;
}

const MySelect = () => (
  <Select<Option> // <-- Pass a generic into a React component like this
    chakrastyles={chakraStyles}
    size="sm"
    defaultvalue={{ label: client?.clientName, value: client?.clientId }}
    options={clients.map((c) => ({
      label: c.name,
      value: c.id,
    }))}
    onChange={(e) => {
      e?.value && dispatch(AssumeclientAsync(e.value));
    }}
  />
);

You can see in this example that passing in the generics to the Select alleviates this issue if you hover over the onChange function: https://codesandbox.io/s/chakra-react-select-styled-like-a-default-chakra-select-qwq3o?file=/app.tsx

If i define the charkaStyles inline like in your linked example it appears to work fine.
The problem only happens if i define the charkaStyles object separately using the ChakraStylesConfig type.
But I'll just do it inline, thats good enough :)

Thanks for your help.

Ah yes, ok that would make sense.  I think I know what the issue is then.
Both the ChakraStylesConfig type and the Select component can take the optional Option generic to inform the inner workings of react-select what to expect for each option object.  When you define the chakraStyles prop outside of the component, it has no way of inferring the interface structure of your options, so then passing it into the component causes the Select to be confused about the shape of that object and make it infer unknown instead.
Overall I still don't fully understand the inner workings of react-select enough to know exactly why this happens, as it is quite a complex package, but if you want to read more about how react-select uses TypeScript, check this page of the docs out: https://react-select.com/typescript

On a side note, I just tested this with the original React Select package, with the styles prop instead of chakraStyles and got the same result: https://codesandbox.io/s/chakra-react-select-react-select-inferred-types-x677s8?file=/app.tsx
So its good to know this is not something unique to my chakraStyles prop.  This also means there's nothing I can really do to fix this issue.

Thank you for taking the time to investigate, and so quickly as well 👍


[Bugfix][Kernel] Implement acquire/release polyfill for Pascal

#8558 introduces the use of acquire/release instructions, which are not supported on architectures below sm_70. This leads to build errors when compiling for older architectures:
ptxas /tmp/tmpxft_0000060f_00000000-13_custom_all_reduce.compute_60.ptx, line 338; error   : Feature '.release' requires .target sm_70 or higher
ptxas /tmp/tmpxft_0000060f_00000000-13_custom_all_reduce.compute_60.ptx, line 343; error   : Feature '.acquire' requires .target sm_70 or higher
...

(https://github.com/sasha0552/pascal-pkgs-ci/actions/runs/11005971990/job/30578026588)
This PR attempts to fix these build errors by implementing a polyfill of these instructions.
The polyfill was adapted from this code:
https://github.com/NVIDIA/nccl/blob/2ea4ee94bfb04c886c79ccae60ac9961000fdee2/src/device/op128.h#L304-L312
https://github.com/NVIDIA/nccl/blob/2ea4ee94bfb04c886c79ccae60ac9961000fdee2/src/device/op128.h#L324-L330
This PR hasn't been tested yet; I'll mark PR as ready for review when I test it.

 I'm not aware that Pascal is still a supported hardware architecture for this project.

actually we don't support pascal officially, but it's good to fix it if there's pr for it. I will hand it over to @hanzhi713  for review, and merge after @hanzhi713 's approval.


chore(deps): Update dependencies (TT-1473)

For information, the test that is changed was not running. Saw a warning and discovered it was not running at all since it did not have a verify/verifycomplete. Added the verify, it failed, so fixed it!


The issues in date_from_string and datetime_from_string functions.

Hi,
We realized  that an inccorect date entries such as "3023-01-01" breaks the code and not just for this specific document but also all the remaining documents in that search dialog. We simply added a try catch statement and returned a None value on error. By doing so we easly identify those currepted documents and inform the owner so those can be fixed. Right now we are using this forked version of your library but we will be glad if you can address this issue so we can keep using the library.
Thank you in advance
Altar

Thank you for raising this issue and for your contribution. I will upload the new version to PyPI shortly.


Allow step implementations to return Result

Allows step implementations to optionally return a Result. This would be a nice-to-have feature, whose main purpose is to allow writing more concise implementations using the ? operator.

FCM
Allow Step functions to return Result (#151)


boards: spresense: Add CONFIG_USERMAIN_STACKSIZE=3072 to wifi/defconfig

Summary

I noticed that stack for the init task was filled almost 100%
This commit fixes this issue

Impact

Affects spresense:wifi configuration only

Testing

Tested with renew/mount/wget/nxplayer commands

@jerpelea


Prebuilt artifacts for linux-arm64 contain an x86-64 frontend_server_aot.dart.snapshot

Steps to reproduce

try using meta-flutter to build Flutter apps for a linux arm64 target on an arm64 host
at some point, flutter-sdk_git.bb downloads https://storage.googleapis.com/flutter_infra_release/releases/stable/linux/flutter_linux_3.24.2-stable.tar.xz (but the same appies to https://storage.googleapis.com/flutter_infra_release/flutter/a6bd3f1de158bb61090e0c8053df93a10cb548e1/linux-arm64/artifacts.zip)

Actual results
inside the archive, bin/cache/artifacts/engine/linux-x64/frontend_server_aot.dart.snapshot is in the wrong architecture:
file flutter/bin/cache/artifacts/engine/linux-x64/frontend_server_aot.dart.snapshot
flutter/bin/cache/artifacts/engine/linux-x64/frontend_server_aot.dart.snapshot: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, BuildID[md5/uuid]=7d7673e2df1d68aa1222110aef3343b6, with debug_info, not stripped

This prevents building flutter apps on the host.
The archive for 3.24.3 has the same problem.
Logs
No response
Flutter Doctor output

Doctor output
[!] Flutter (Channel stable, 3.24.2, on Ubuntu 22.04.5 LTS 5.15.0-122-generic, locale en_US.UTF-8)
    • Flutter version 3.24.2 on channel stable at
      /home/simone/yocto/kirkstone/build/tmp/sysroots-components/aarch64/flutter-sdk-native/usr/share/flutter/sdk
    ! The flutter binary is not on your path. Consider adding
      /home/simone/yocto/kirkstone/build/tmp/sysroots-components/aarch64/flutter-sdk-native/usr/share/flutter/sdk/bin to your path.
    ! The dart binary is not on your path. Consider adding
      /home/simone/yocto/kirkstone/build/tmp/sysroots-components/aarch64/flutter-sdk-native/usr/share/flutter/sdk/bin to your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cf269e36d (5 weeks ago), 2024-09-03 14:30:00 -0700
    • Engine revision a6bd3f1de1
    • Dart version 3.5.2
    • DevTools version 2.37.2
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and
      upgrades.

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/to/linux-android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✗] Chrome - develop for the web (Cannot find Chrome executable at google-chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/to/linux-android-setup for detailed instructions).

[✓] Connected device (1 available)
    • Linux (desktop) • linux • linux-arm64 • Ubuntu 22.04.5 LTS 5.15.0-122-generic

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 4 categories.

</details>

@cbracken any ideas on this?

Re:
file flutter/bin/cache/artifacts/engine/linux-x64/frontend_server_aot.dart.snapshot
flutter/bin/cache/artifacts/engine/linux-x64/frontend_server_aot.dart.snapshot: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, BuildID[md5/uuid]=7d7673e2df1d68aa1222110aef3343b6, with debug_info, not stripped

It looks like you're looking in the linux-x64 artifact cache and finding x86_64 binaries, which seems expected. What are you seeing under the linux-arm64 directory?

https://storage.googleapis.com/flutter_infra_release/flutter/a6bd3f1de158bb61090e0c8053df93a10cb548e1/linux-arm64/artifacts.zip contains frontend_server_aot.dart.snapshot: ELF 64-bit LSB shared object, x86-64. Isn't it supposed to be arm64?
As to https://storage.googleapis.com/flutter_infra_release/releases/stable/linux/flutter_linux_3.24.2-stable.tar.xz, you're right: I was looking in the wrong place. There's no linux-arm64 directory there, so I looked into the only linux*64 directory without paying too much attention :)

https://storage.googleapis.com/flutter_infra_release/flutter/a6bd3f1de158bb61090e0c8053df93a10cb548e1/linux-arm64/artifacts.zip contains frontend_server_aot.dart.snapshot: ELF 64-bit LSB shared object, x86-64. Isn't it supposed to be arm64?

Yeah that one definitely looks wrong to me, unless arm64 is only supported as a cross-compile target. That's not how I remember things though. @christopherfujino @HidenoriMatsubayashi I think both of you did most of the work on arm64 support, do you recall what the expected binaries are here?

I'm guessing we're copying the prebuilt: https://github.com/flutter/engine/blob/main/flutter_frontend_server/BUILD.gn#L10

And it looks like the host_prebuilt_dart_sdk var that we copy the snapshot from is defined in:
https://github.com/flutter/engine/blob/main/common/config.gni#L141
# When building 32-bit Android development artifacts for Windows host (like
# gen_snapshot), the host_cpu is set to x86. However, the correct prebuilt
# Dart SDK to use during this build is still the 64-bit one.
_host_cpu = host_cpu
if (host_os == "win" && host_cpu == "x86") {
  _host_cpu = "x64"
}

_target_prebuilt_config = "$_target_os_name-$target_cpu"
_host_prebuilt_config = "$_host_os_name-$_host_cpu"

target_prebuilts_path = "//flutter/prebuilts/$_target_prebuilt_config"
host_prebuilts_path = "//flutter/prebuilts/$_host_prebuilt_config"

if (flutter_prebuilt_dart_sdk) {
  target_prebuilt_dart_sdk = "$target_prebuilts_path/dart-sdk"
  host_prebuilt_dart_sdk = "$host_prebuilts_path/dart-sdk"

Is dependent on the host_cpu var. Per the docs, that is defined as: https://gn.googlesource.com/gn/+/main/docs/reference.md#var_host_cpu

Oh, but we're cross compiling these artifacts from x64, so I guess that's why the host_cpu var is x64?
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux Production Engine Drone/963998/infra

My guess is that in this BUILD file: https://github.com/flutter/engine/blob/main/flutter_frontend_server/BUILD.gn#L10 we should instead by using the target_prebuilt_dart_sdk var, defined here: https://github.com/flutter/engine/blob/main/common/config.gni#L140


Form Components should NOT update state

The flow that Mortar tries to encourage is:

The page renders the mortar component
Changes to the form are captured in the FormStore
The component that is rendering the mortar component listens to changes on the FormStore
Changes are grabbed from the FormStore and handled by the parent component
State is only ever updated in the change events via this.setState()

However, there is a severe oversight in that currently, we're not doing anything to stop the inputs in the rendered <input> elements from trickling back up and editing state without this.setState().
This is a big issue, because it leads to out of sync data which undermines any security that the data being edited / sent to the API is what you expect.  Also, it can easily lead to the data displayed and the data in the store being out of sync.
The first priority is to stop all events from bubbling up and setting state.  Mortar components should only edit the FormStore, which the developer can use to update the data in state and displayed on the page.
Second, we should evaluate Immutable JS and potentially implement it in all of Mortar's stores.
Requirements

[x] ALL Form Components stop default event propagation
[ ] Implement Immutable JS in all of Mortar's stores (Optional)

The initial bit of work has been done on this.
The issue was actually in the FormStore.  It was not an issue of needing to preventDefault but rather, how we were registering the data to the form.  https://github.com/fuzz-productions/Mortar-JS/blob/dev/lib/stores/FormStore.js#L101
We were passing a reference to the data of the Form into the FormStore.  So, changes were directly changing the data in state for the parent component.  Now, we're cloning the data, so it's no longer passing a reference around.
A consequence of this is that now all form components can be responsible for their own rendering by requesting just their data from the FormStore.  More on this when I update the wiki entries.

@walterbm
Form now passes down the Form Value for each individual component directly into the component via this.props
The FileInput doesn't get this.props, but probably should.


Was working before, not anymore

What is wrong with the code? It's not responding to the !ping command.
https://github.com/Hamsterland/Himari

Hey.
I can't spot any errors, but I know that there are some bugs in my code.
I will do a clean rewrite of it, but I can't tell you when I will release it. Maybe this week, maybe next one.
I will take a deeper look into your specific situation tomorrow.

Thanks for looking into it. It was working before but I changed nothing and it's not responding. The PingHandler does work but not any commands. Thanks.

I know solution. I've done some of researching with log package... and It seems I found error!

And my bot now work with this (prefix - !):
!obj
! obj
! obj

I can provide code if you need it @Lukaesebrot @Hamsterland.
Or I just can create pull request with fixes.

@FedorLap2006 sorry I don't quite understand what you're saying. Which of those two arguments should work?

Hmm, they all work, but... It seems commands handler allows incorrect (non-existing) commands to run. I don't know why but I'm doing my research.

I don't see you using this library in your commands at all. You're utilizing dgrouter instead. I don't believe he can help you with that as it is not his library.

@Soul85 I switched since, couldn't wait forever. I'm just gonna close the issue. Thanks anyway everyone.


List of supported versions combinations for running Camel-K with Knative

Is there online doc (link?) with that information?
In particular what versions of Knative are suported by  Camel K 1.0 GA? Knative Eventing APIs are changing fast? Supported 0.14+?

Related to https://github.com/apache/camel-k/issues/1336

Glad that you bring this up. Current Camel-K version is not compatible with Knative Eventing v1 CRDs. When we try to deploy one of the knative examples, camel-k stuck on deployment phase:
Warning  IntegrationError             2m34s (x18 over 14m)  camel-k-integration-controller      Cannot reconcile Integration notifier: error executing post actions: error during replace resource: could not create or replace resource notifications-notifier: no matches for kind "Subscription" in version "messaging.knative.dev/v1alpha1"

Hi @nicolaferraro, while this compatibility issue is not fixed, how do we integrate with knative?

Relates #3306


examples/amg88xx: add example for amg88xx driver

Summary
Simple example for the amg88xx driver
Impact
Testing

@LuchianMihai did you have time to convert the driver to video/camera interface? Case you don't, maybe we should merge it as normal driver, because is better to have a driver anyway because it could become the base for other people to modify it

Hi @acassis,
I had to deliver the project I was working on so right now I do not have any sensor board to work on, I've designed an shield similar to this one. Right now is in transit.
Also I still have a single sensor (hopefully working) that I will solder on the board.
So for the moment the development is blocked. Maybe an better idea would be to close as is, and continue when I have needed hardware.

@LuchianMihai instead of closing it you can just convert to draft, it way it is easier to track later

Close, as in merge my PR

Hi @acassis,
Can you point out what went wrong here?

Hi @acassis, Can you point out what went wrong here?

look like it's the temp network error:
====================================================================================
Configuration/Tool: bl602evb/wifi
2024-08-29 15:16:22
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:02 --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:03 --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:04 --:--:--     0
  0     0    0     0    0     0      0      0 --:--:--  0:00:04 --:--:--     0

100 12993    0 12993    0     0   2524      0 --:--:--  0:00:05 --:--:--  2524
100  496k    0  496k    0     0  93110      0 --:--:--  0:00:05 --:--:-- 1560k
curl: (16) Error in the HTTP2 framing layer
make[1]: *** [v1.9.1-ble-test.zip] Error 16
make[1]: Target `context' not remade because of errors.
make: *** [arch/risc-v/src/.context] Error 2
make: Target `all' not remade because of errors.
/Users/runner/work/nuttx-apps/nuttx-apps/sources/nuttx/tools/testbuild.sh: line 378: /Users/runner/work/nuttx-apps/nuttx-apps/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize bl602evb/wifi


Problems with plotting the results

I am going through the examples in live plotting tutorial (https://blocks.readthedocs.org/en/latest/plotting.html) with bokeh-0.9.0 (newer versions of bokeh yield an error already reported in #15), but I am not getting any figures in the end.
I use bokeh-server and localhost:5006
The MainLoop have created a new document in Bokeh Documents called 'Plotting examples'.
It also looks there is some communication between main_loop.run() and the server (see the excerpt below).
However I don't see any plots neither during training nor when it is finished.
I tried this under Safari and Iceweasel.
Excerpt:
INFO:tornado.access:200 POST /bokeh/bb/36e95f94-0c8e-462e-9d0e-8a193e05f510/bulkupsert (<IP_ADDRESS>) 19.91ms
INFO:tornado.access:200 POST /bokeh/bb/36e95f94-0c8e-462e-9d0e-8a193e05f510/bulkupsert (<IP_ADDRESS>) 28.04ms
INFO:tornado.access:200 POST /bokeh/bb/36e95f94-0c8e-462e-9d0e-8a193e05f510/bulkupsert (<IP_ADDRESS>) 40.00ms

Make sure, that you installed bokeh correctly.
Sometimes there's some issue with CSS styles in bokeh, I had to remove .collapse{..} entity using a debugging console.


feat: add alert rules for mso

This commit adds initial rule file for monitoring-stack-operator
Signed-off-by: Jayapriya Pai<EMAIL_ADDRESS>Description:
Adding this initial PR to add alert rules. I have added only one rule in this commit, I will add more once its clear if this is how we want.

@fpetkovski @sthaha PTAL

This looks good so far. Do we want to monitor additional metrics, e.g. queue depth?

This looks good so far. Do we want to monitor additional metrics, e.g. queue depth?

Ya I will add other rules as well, created initial PR with only one just to get an idea. Also since we are not using jsonnet how do we want to include other alerts like monitoring prometheus-operator?

Yes, that's a good point. We should include PO alerts as well.

@fpetkovski I added couple of alerts as well, PTAL. I can create separate PR for Prometheus Operator

Nice patch @slashpai ! Looks good overall. Some minor corrections

Thanks Sunil. I updated with changes, PTAL

@fpetkovski are we good to merge? or anymore change is required :)


JDK SUpport

is there a plan to support JDK9 and higher? Java 8 is about 6 years old and is missing some features.

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

ID: ef3d7e8a-f011-dca1-8834-7d99f934448a
Version Independent ID: 0571dc14-76a6-1d73-bcb7-08e84d256744
Content: Use Java and Gradle to publish a function to Azure
Content Source: articles/azure-functions/functions-create-first-java-gradle.md
Service: azure-functions
GitHub Login: @KarlErickson
Microsoft Alias: karler

@KlemensM, thank you for reaching out. We are looking into this and would get back to you soon on this thread.

@KlemensM We are currently working on this and targeting a preview version to be out by mid-year if not sooner. But these time lines may vary.
For now I will proceed with closure of this and If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Migrate to get metric data api calls

Anything in specific that I could help with on this PR?

@bheight-Zymergen - Currently not. Just moved in a new flat and do not have much time to tackle this bigger topic. If you are interested in doing it, i can throw my stuff away and review / merge your PR.

As it is a blocker for me currently I'll check out your current branch and see what I can do.

Closed in favour of new pr.


fix: update monaco config even in invalid spec

Description

update monaco config even in invalid spec
switch to old json schemas due to problem that monaco cannot fetch schemas from https://asyncapi.com/definitions...

Related issue(s)
Resolves https://github.com/asyncapi/studio/issues/252

@mcturco Could you look at it?

@mcturco Yeah, you're right person for this, I mean for approval 😄 but if you want, you can make review of changes. I know that you probably don't understand the source code of that project but these changes are very needed.

@fmvilas Could you check again?


The -d option does not appear to be optional.

Steps to reproduce:
I ran the following command:
./seed-my-ontology-repo.pl -u openrif -t "Contributor Role Ontology" cro -f
Result:
The script exited with an error.  See attached log file.  Running the script with the -f flag does not work, either.
seedlog.txt
Expected Result:
Given that the -d flag is described as an option in the help, I expected it to be optional.

Ah, that's an oversight.
As a workaround try -d ro or a small ontology, and then you can simply choose to ignore the dependency stuff ad remove the import

turned out to be an easy fix


Cannot give value to the component

                   <vue-tel-input @onInput="onInput" v-bind:phone="newContact.phoneNumber" name="phone number" v-validate="'required'" data-vv-delay="1000"></vue-tel-input>
I am trying to load data to the input field, but I cannot find how to do it.

I've just added the possibility to pass v-model to the component.
You can use it as <vue-tel-input v-model="phone"></vue-tel-input>.
The phone property will be a string, contains phone number with the country code.

Thank you for responding.
I downloaded the latest version, I see you added props, but I had to add this piece of code for passing value to the phone number input field.
created() {     if(this.value) {       this.phone = this.value     }   }

Cheers. would you like to open a PR?

I created pull request #8


Make it possible to extract Path from Request.pathInfo

Sometimes it's useful to extract the Path directly from the request and work on that.
When using http4s-directives, we never want to handle Methods using pattern matching, since we care about correct HTTP response codes.
This makes foot-shooting possible, but that was possible anyway.

When thinking about it, this will not handle Path.Root, so never mind.


Fixes for #2080 and #2081

Tested changes for #2080 and #2081 by publishing to npm and installing/running the package locally (specifically with --sass-binary-path CLI flag).
Fix for #2081 would be really nice. I work at a big corporation, and we have firewalls that prevent easy direct access to downloading sass binaries (which causes us some headaches). Being able to point to a compiled binary, using a relative path, would help us a lot.
Thank you!

Summary:

removed ES6 find() function, replaced with ES5-compliant solution
made code changes to adhere to coding style
made suggested cross-platform update

Please review. Thank you!

A few note. Removing support for the current --foo bar style is a breaking change. To get this change over the line we'd need to support both the old and new style. This can be done by asserting whether the matched value contains =.

However that won't work, as all custom CL arguments must have '--' prefixed.

I'm not sure I follow why this wont work given we're finding the index in the arg list the number of args beforehand shouldn't matter.
npm install node-sass -- --foo bar

All custom CLI arguments must have '--' prefixed. So, this will fail:
npm install node-sass -- --foo bar

But this would work:
npm install node-sass -- --foo --bar

When you consider passing a binary path, we would need to strip the -- from the beginning of the path.
npm install node-sass -- --sass-binary-path --./node_binary/binding.node

However, that's weird. I'd rather align with industry standard of arg=val, like so:
npm install node-sass -- --sass-binary-path=./node_binary/binding.node

Summary:

modified extensions.getBinaryPath() to return an absolute path from a relative path if possible

it does this by attempting to resolve() the relative path against the current working directory (as you suggested earlier), but traverses up the directory tree trying to resolve() the path. This is how we get around the arbitrarily-nested node_modules directory problem

undid changes to binding.js since they were no longer necessary

Please review. Thanks!

@wittibs is getting the absolute path really necessary?

@saper Yes - it is critical to ensuring relative binary paths will work for any level of depth of nested node_modules sub-directories.
For example: if I provide a binary path ./my-node-binding/binding.node (relative to my project directory), I should expect the binary path to be resolved correctly regardless of node-sass directory depth. See sample file structure below (taken from @xzyfer 's comment from outdated code changes above):
project
|-index.html
|-index.js
|-my-node-binding
  |-binding.node
|-node_modules
<EMAIL_ADDRESS>    |-lib
      |-binding.js
  |-gulp-sass
    |-node_modules
<EMAIL_ADDRESS>        |-lib
          |-binding.js
|-package.json
|-package-lock.json

Thx @wittibs for this work.
@zxyger can we merge this ASAP ?  We need this feature for our deployment pipeline to work :)

Hey @xzyfer just following up - have you had a chance to review this solution? If so, any feedback or thoughts?
Let me know if there's anything I can do to help expedite the PR. Thanks!

I am going to give it a try - how can Inreproduce the problem using Win Server 2016?

@saper try to install node-sass package with npmrc variable sass_binary_path set to a relative path. (#2081)
Try using any sass binary CLI argument during install or runtime (#2080)

Interesting problem when trying to use absolute paths: #2149

I'm considering closing this PR because:

The fix for #2081 is non-deterministic, which is buggy in nature
I don't think #2081 can be solved without some hackish fix. I think it should be the consuming application's responsibility to provide an absolute path for sass_binary_path.

Let me know what you all think. At the very least, I can:

keep the fix for #2080
provide error message for when relative path is provided to sass_binary_path

Thoughts?

I've tried this today and it seems to work:

><EMAIL_ADDRESS>install /tmp/node/node_modules/node-sass
> node scripts/install.js

node-sass build Binary found at /tmp/binding.node

><EMAIL_ADDRESS>postinstall /tmp/node/node_modules/node-sass
> node scripts/build.js

Binary found at /tmp/binding.node
Testing binary
Binary is fine

I think something strange happened to npm since I believe the space separated form worked as well.
Can you try without -- and with = separator?

I tried that and... it worked! Although I have no idea why .... I don't think I fully understand how CLI arguments get parsed & passed to node apps. :/
Well this has been a good learning experience for me. Thank you for your time.

@saper @wittibs Running into the same issue, but that fix has thrown me off completely. Can you tell me what I'm doing wrong?
$ node
><EMAIL_ADDRESS>install /tmp/lib/node-sass/linux-x64-51_binding.node
SyntaxError: Invalid or unexpected token

<EMAIL_ADDRESS>install /tmp/lib/node-sass/linux-x64-51_binding.node
zsh: command not found<EMAIL_ADDRESS>


Alpine container using OpenRC

Hi Jess,
I wanted to create a PR for this Alpine container using OpenRC into your dockerfiles but I don' t have permission to do it.
If you think it's a good one (https://github.com/danilofalcao/dockerfiles/tree/master/alpine) just let me know.
❤
-dfs

ah thanks! I try to keep this repo only to dockerfiles I use to make sure I catch breaks, but thanks so much :)


missing authentication issue.

I tried to run the already available template using below link.
https://console.aws.amazon.com/cloudformation/home?region=us-east-1#/stacks/new?templateURL=https:%2F%2Fs3.amazonaws.com%2Fsolutions-reference%2Faws-mlops-framework%2Flatest%2Faws-mlops-framework.template
I am referring to below solution
https://aws.amazon.com/solutions/implementations/aws-mlops-framework/?did=sl_card&trk=sl_card
Once I run the cloudformation stack , the creation is complete , i am provided a link to access my model, on clicking the link I get a missing authentication issue.
is it something to do with the models which are created using the template or something else??

AWS MLOps Framework provisions a pipeline based on the inputs received from either an API call or a Git repository.
I would like to use a sample model to understand the working of project. How can I do it??
What should I be specifying in the codecommit repo ??
what files should my model contain?? ( I am aware it requires dockerfile and files for building Amazon SageMaker compatible Docker image but I need to know specifically what all files and if preferrable an example would be helpful )
Further more if there is any documentation on what all to upload and where to upload it could be helpful.

AWS MLOps Framework provisions a pipeline based on the inputs received from either an API call or a Git repository.
I would like to use a sample model to understand the working of project. How can I do it??
What should I be specifying in the codecommit repo ??
what files should my model contain?? ( I am aware it requires dockerfile and files for building Amazon SageMaker compatible Docker image but I need to know specifically what all files and if preferrable an example would be helpful )
Further more if there is any documentation on what all to upload and where to upload it could be helpful.

Hi @AditModi, thank you for trying our solution.
The API call body, in the example you provided is missing "model_artifact_location". For example, here is the API body to provision a Sagemaker Inference Endpoint using Sagemaker Builtin algorithms (e.g. linear-learner):
{
    "pipeline_type": "byom",
    "custom_model_container": "",
    "model_framework": "linear-learner",
    "model_framework_version": "1",
    "model_name": "myrealtimemodel",
    "model_artifact_location": "model.tar.gz",
    "training_data": "training/data.csv",
    "inference_instance": "ml.m5.large",
    "inference_type": "realtime",
    "batch_inference_data": ""
}

In this example, you need to upload your trained model (e.g. using linear-learner builtin algorithm) to the root of the assets S3 bucket created by the pipeline. A link to the assets S3 bucket can be found in the outputs of the Cloud formation template.
Notice here the "custom_model_container": "" is left empty because we are using Sagemaker Builtin algorithm.
A second example to provision a Sagemaker Inference Endpoint using your own custom algorithm :
{
   "pipeline_type": "byom",
	"custom_model_container": "custom/custom_image.zip",
	"model_framework": "",
	"model_framework_version": "",
	"model_name": "mycustomrealtimemodel",
	"model_artifact_location": "model.tar.gz",
	"training_data": "training/my_data.csv",
	"inference_instance": "ml.m5.large",
	"inference_type": "realtime",
	"batch_inference_data": ""
}

Here the "custom_model_container": "custom/custom_image.zip" is specified, which refers to the location of your custom algorithm docker image assets (as required by Sagemaker) in the assets S3 bucket. The "model_framework": "" and "model_framework_version": "" are left empty because we are using custom algorithm.

Hi @AditModi, thank you for trying our solution.
The API call body, in the example you provided is missing "model_artifact_location". For example, here is the API body to provision a Sagemaker Inference Endpoint using Sagemaker Builtin algorithms (e.g. linear-learner):
{
    "pipeline_type": "byom",
    "custom_model_container": "",
    "model_framework": "linear-learner",
    "model_framework_version": "1",
    "model_name": "myrealtimemodel",
    "model_artifact_location": "model.tar.gz",
    "training_data": "training/data.csv",
    "inference_instance": "ml.m5.large",
    "inference_type": "realtime",
    "batch_inference_data": ""
}

In this example, you need to upload your trained model (e.g. using linear-learner builtin algorithm) to the root of the assets S3 bucket created by the pipeline. A link to the assets S3 bucket can be found in the outputs of the Cloud formation template.
Notice here the "custom_model_container": "" is left empty because we are using Sagemaker Builtin algorithm.
A second example to provision a Sagemaker Inference Endpoint using your own custom algorithm :
{
   "pipeline_type": "byom",
	"custom_model_container": "custom/custom_image.zip",
	"model_framework": "",
	"model_framework_version": "",
	"model_name": "mycustomrealtimemodel",
	"model_artifact_location": "model.tar.gz",
	"training_data": "training/my_data.csv",
	"inference_instance": "ml.m5.large",
	"inference_type": "realtime",
	"batch_inference_data": ""
}

Here the "custom_model_container": "custom/custom_image.zip" is specified, which refers to the location of your custom algorithm docker image assets (as required by Sagemaker) in the assets S3 bucket. The "model_framework": "" and "model_framework_version": "" are left empty because we are using custom algorithm.

hi @tabdunabi ,
you mentioned "you need to upload your trained model (e.g. using linear-learner builtin algorithm) to the root of the assets S3 bucket created by the pipeline. " I am unclear as to what should my training model (e.g. using linear-learner builtin algorithm) contain ( what are the different files the trained model should comprise of ) . what is the file structure that is being stored to the root of the assets s3 bucket.
Giving an example of the trained model that i can upload to assets s3 bucket could be really helpful.
Also regarding the custom algorithm, the custom_model_container refers to the location of your custom algorithm docker image assets (as required by Sagemaker) in the assets S3 bucket. ( is it possible to list all the files that the zip file is expected to have ie dockerfile etc )
It could be really helpful if you could provide a sample zip containing all the necessary files ( as a sample example to refer to ) and file structure in which the model is zipped and uploaded to s3 bucket.

hi @tabdunabi ,
you mentioned "you need to upload your trained model (e.g. using linear-learner builtin algorithm) to the root of the assets S3 bucket created by the pipeline. " I am unclear as to what should my training model (e.g. using linear-learner builtin algorithm) contain ( what are the different files the trained model should comprise of ) . what is the file structure that is being stored to the root of the assets s3 bucket.
Giving an example of the trained model that i can upload to assets s3 bucket could be really helpful.
Also regarding the custom algorithm, the custom_model_container refers to the location of your custom algorithm docker image assets (as required by Sagemaker) in the assets S3 bucket. ( is it possible to list all the files that the zip file is expected to have ie dockerfile etc )
It could be really helpful if you could provide a sample zip containing all the necessary files ( as a sample example to refer to ) and file structure in which the model is zipped and uploaded to s3 bucket.

@AditModi , the solution uses models trained using Sagemaker. you should know how to develop ML models using Sagemaker. Please see Sagemaker developer guide and examples. When using Sagemaker, the trained model is saved in *.tar.gz format.
Regarding what should the custom container zip file include, please see examples provided in this page. Follow the links for the different examples to see detailed notebooks.

@AditModi , the solution uses models trained using Sagemaker. you should know how to develop ML models using Sagemaker. Please see Sagemaker developer guide and examples. When using Sagemaker, the trained model is saved in *.tar.gz format.
Regarding what should the custom container zip file include, please see examples provided in this page. Follow the links for the different examples to see detailed notebooks.

@tabdunabi ,
I trained a model using amazon sagemaker and using the api call provisioned the pipeline . the pipeline would fetch the artifacts I created earlier. everything works for a model trained using sagemaker.
Currently I have a different use-case. I would like to build/train a model ( elsewhere not aws sagemaker) and deploy model using the provision pipeline feature. Do I require to use the custom container ( bring your own model feature ) if so, then how to achieve that.
I saw the  bring your own model example using scikit-learn example but I am still unable to grasp what all should the zip file include . take an example say the scikit_learn_bring_your_own_model should contain the zip of the container folder ???

@tabdunabi ,
I trained a model using amazon sagemaker and using the api call provisioned the pipeline . the pipeline would fetch the artifacts I created earlier. everything works for a model trained using sagemaker.
Currently I have a different use-case. I would like to build/train a model ( elsewhere not aws sagemaker) and deploy model using the provision pipeline feature. Do I require to use the custom container ( bring your own model feature ) if so, then how to achieve that.
I saw the  bring your own model example using scikit-learn example but I am still unable to grasp what all should the zip file include . take an example say the scikit_learn_bring_your_own_model should contain the zip of the container folder ???

Hi @AditModi,
Currently, the solution supports models trained using Amazon SageMaker builtin algorithms, or Bring Your Own Model (BYOM) feature via a custom docker image, and supporting files,  that is compatible with Amazon SageMaker Endpoint.
In the example bring your own model example using scikit-learn, the custom/custom_image.zip in "custom_model_container": "custom_image.zip" should contain the contents of the container folder.

Hi @AditModi,
Currently, the solution supports models trained using Amazon SageMaker builtin algorithms, or Bring Your Own Model (BYOM) feature via a custom docker image, and supporting files,  that is compatible with Amazon SageMaker Endpoint.
In the example bring your own model example using scikit-learn, the custom/custom_image.zip in "custom_model_container": "custom_image.zip" should contain the contents of the container folder.

In the example bring your own model example using scikit-learn, the custom/custom_image.zip in "custom_model_container": "custom_image.zip" should contain the contents of the container folder. I followed the same approach as mentioned by you.
Issues faced:
the Code-Deploy step fails.
In the cloudwatch logs it is mentioned that An error occurred (ValidationException) when calling the DescribeModel operation: Could not find model "arn:aws:sagemaker:us-east-1:account-arn:model/decision-tree-model".
What is causing this issue??
A second example to provision a Sagemaker Inference Endpoint using your own custom algorithm :
{
"pipeline_type": "byom",
"custom_model_container": "custom/custom_image.zip",
"model_framework": "",
"model_framework_version": "",
"model_name": "mycustomrealtimemodel",
"model_artifact_location": "model.tar.gz",
"training_data": "training/my_data.csv",
"inference_instance": "ml.m5.large",
"inference_type": "realtime",
"batch_inference_data": ""
}
Here the "custom_model_container": "custom/custom_image.zip" is specified, which refers to the location of your custom algorithm docker image assets (as required by Sagemaker) in the assets S3 bucket. The "model_framework": "" and "model_framework_version": "" are left empty because we are using custom algorithm.
I am specifying the provision pipeline as shown earlier.
How to ensure that a sample-application work for our use-case.

In the example bring your own model example using scikit-learn, the custom/custom_image.zip in "custom_model_container": "custom_image.zip" should contain the contents of the container folder. I followed the same approach as mentioned by you.
Issues faced:
the Code-Deploy step fails.
In the cloudwatch logs it is mentioned that An error occurred (ValidationException) when calling the DescribeModel operation: Could not find model "arn:aws:sagemaker:us-east-1:account-arn:model/decision-tree-model".
What is causing this issue??
A second example to provision a Sagemaker Inference Endpoint using your own custom algorithm :
{
"pipeline_type": "byom",
"custom_model_container": "custom/custom_image.zip",
"model_framework": "",
"model_framework_version": "",
"model_name": "mycustomrealtimemodel",
"model_artifact_location": "model.tar.gz",
"training_data": "training/my_data.csv",
"inference_instance": "ml.m5.large",
"inference_type": "realtime",
"batch_inference_data": ""
}
Here the "custom_model_container": "custom/custom_image.zip" is specified, which refers to the location of your custom algorithm docker image assets (as required by Sagemaker) in the assets S3 bucket. The "model_framework": "" and "model_framework_version": "" are left empty because we are using custom algorithm.
I am specifying the provision pipeline as shown earlier.
How to ensure that a sample-application work for our use-case.

Hi @AditModi , there is an issue with the mentioned SageMaker example. If look at this issue I created yesterday, you will find the easy fix for the Dockerfile.
Please apply the same fix in your Dockerfile, re-zip the folder, upload again to S3, and retry.
Note: when zipping the custom_image folder use something like the following.
cd custome_image
zip -r ../custom_image.zip *

Hi @AditModi , there is an issue with the mentioned SageMaker example. If look at this issue I created yesterday, you will find the easy fix for the Dockerfile.
Please apply the same fix in your Dockerfile, re-zip the folder, upload again to S3, and retry.
Note: when zipping the custom_image folder use something like the following.
cd custome_image
zip -r ../custom_image.zip *

Hi @AditModi, following up on this issue that you mentioned on SageMaker example, the amazon-sagemaker-examples issue was addressed and merged on Feb 16.
Thank you for your inquiry. Please let us know if you need additional assistance here. Closing this issue for now.


Update README

How to access configuration in custom commands
How to add commands to sections
History of dev.py

Closes #19
@rgommers I would appreciate it if you could help me correct and/or expand the history section as necessary.

Thanks, @rgommers, for the review! I've made the updates suggested.


[dv/alert_esc_agent] Temp fix to support pwrmgr with multi-reset

This PR temp fix alert firing issue in pwrmgr where cip_base_vseq and alert_esc_agent have different clk and reset.
The better way to solve this is documented in issue #16374.
Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

@matutem this should fix your pwrmgr alert response issue.

I understand this is a temporary way to handle this unusual condition, where pwrmgr's clk/rst_n pair do not behave like most (all?) other units. Notice pwrmgr also needs to specialize the handling of the cip_scoreboard's reset_asserted() and reset() for a similar reason. Perhaps your eventual fix will also deal with that?
LGTM

I have not thought too much about that tbh. But the fix will separate alert response from cip_lib. Maybe we can discuss more after M2 in terms of how to fix the rest of cip resets.


UX | Design UX for 'Generated dialog', allowing users to capture more than one piece of user input at the time (aka form)

For task-oriented dialogs such as ordering food, a table or tickets, the bot typically needs information from the user. Sometimes users provide all of the needed information at once (I want a double tall soy latte with Ice) and sometimes the bot needs to ask the user to ask additional questions to complete the order or request.
How can we build on the current composer UX and SendMessage/Prompt experience to define more than one user input/question?
See also :
https://www.figma.com/file/HLtKaKHctLldivPnzQ7BqQOI/Composer-MI-lotsotings?node-id=1873%3A158

This has shipped with form dialogs


WIP: add sdc_overload_method decorator

We will need to add inline='always' (and possible other attributes) to all our @overload_methods after numba @overload_method refactoring merge. Add our own attribute wrapper so it can be made from one place.

Could you please do the same for @overload_attribute with replacing existing attributes and introduce sdc_overload without replacing existing overloads?

Hello @Hardcode84! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file sdc/utils.py:

Line 575:1: E302 expected 2 blank lines, found 1


Bug: Git Submenu is empty for non GitHub Repos

Description
The Git Submenu is empty for non GitHub Repos:

Steps To Reproduce

Clone a Repo from GitHub
Clone a Repo from e.g. GitLab
Go into the GitHub Repo and check the Git Submenu in the Contextmenu
Go into the GitLab Repo and  check the Git Submenu in the Contextmenu

Requirements
Git Submenu shouldn't be empty for non GitHub Repos
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3447
Log File
debug.log

Thanks for the feedback!
We currently support only GitHub. I'm not sure why parent menu is visible to begin with for non-GitHub repos, but we'll track this issue.

Agreed, the menu option shouldn't be visible in the first place.


More robust server checks

Now that we have subnamespaces, we can't have missing namespaces in the
middle of a tree, only at the root. Simplify the authz checks to reflect
this, and better handle the two reasons why the namespace might be
missing (HNC is starting up vs the namespace is truly orphaned).
Tested: manually confirmed that we can fix namespaces with the
CritParentMissing condition, either by making it a root or recreating
the missing parent. Also manually confirmed that we can't change the
properties of a namespace with a critical ancestor. Added logs and
verified that we're checking authz on the correct namespaces in multiple
cases. Also added new tests to confirm all of this automatically.

/hold
DO NOT SUBMIT until after #702
/cc @yiqigao217
/cc @rjbez17

/hold cancel
/assign @yiqigao217
/assign @rjbez17

This is ready for another review - thanks!
/cc @yiqigao217
/cc @rjbez17

Failure appears to be an unrelated flake; will file PR to fix the flake
/retest

/cc @sophieliu15
/cc @adrianchung

/lgtm
/approve
/hold
Since this change is large, I feel safer to have another eye on this.

/lgtm
/approve
/hold cancel


wasmgen continued

We're finally moving to webpack 4 on Yoroi and I need to update the wasm bindings to get it working.
I rebased the wallet-wasm2 onto master (since it's way out of date) and then tried to make the wasm generation work but there have been quite a few changes in the rust-cardano library since the wallet-wasm2 change so there's a bunch of compiler errors that need to be fixed.
I'd appreciate help fixing these if you have time 🙏
error[E0624]: method `from_slice` is private
  --> src/lib.rs:59:35
   |
59 |         let xprv = handle_result!(hdwallet::XPrv::from_slice(bytes));
   |                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0624]: method `from_slice` is private
  --> src/lib.rs:69:36
   |
69 |         let xprv  = handle_result!(hdwallet::XPrv::from_slice(&bytes));
   |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0061]: this function takes 2 parameters but 1 parameter was supplied
   --> src/lib.rs:131:21
    |
131 |         let attrs = address::Attributes::new_bootstrap_era(None);
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected 2 parameters

error[E0061]: this function takes 2 parameters but 1 parameter was supplied
   --> src/lib.rs:139:21
    |
139 |         let attrs = address::Attributes::new_bootstrap_era(Some(payload.0.clone()));
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected 2 parameters

error[E0599]: no method named `to_bytes` found for type `cardano::address::ExtendedAddr` in the current scope
   --> src/lib.rs:190:38
    |
190 |         util::base58::encode(&self.0.to_bytes())
    |                                      ^^^^^^^^

error[E0277]: the trait bound `cardano::address::ExtendedAddr: cbor_event::Deserialize` is not satisfied
   --> src/lib.rs:195:74
    |
195 |         let addr  = handle_result!(cbor_event::de::RawCbor::from(&bytes).deserialize());
    |                                                                          ^^^^^^^^^^^ the trait `cbor_event::Deserialize` is not implemented for `cardano::address::ExtendedAddr`

error[E0277]: the trait bound `cardano::address::ExtendedAddr: cbor_event::Deserialize` is not satisfied
   --> src/lib.rs:200:73
    |
200 |         let addr  = handle_result!(cbor_event::de::RawCbor::from(bytes).deserialize());
    |                                                                         ^^^^^^^^^^^ the trait `cbor_event::Deserialize` is not implemented for `cardano::address::ExtendedAddr`

error[E0599]: no method named `is_some` found for type `std::result::Result<cardano::hdpayload::Path, cardano::hdpayload::Error>` in the current scope
   --> src/lib.rs:263:39
    |
263 |             self.key.decrypt_path(dp).is_some()
    |                                       ^^^^^^^

don't use wallet-wasm2, the be new bindgen generated API is in cardano-wallet directory on master branch


Svelte / HIghlight

Description

Neovim version
NVIM v0.7.2
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by builduser
Nvim-lspconfig version
No response
Operating system and version
5.18.16-1-MANJARO #1 SMP PREEMPT_DYNAMIC Wed Aug 3 11:18:52 UTC 2022 x86_64 GNU/Linux
Affected language servers
svelte-language-server
Steps to reproduce
.
Actual behavior
.
Expected behavior
No response
Minimal config
.

LSP log
NVIM v0.7.2 Build type: Release LuaJIT 2.1.0-beta3 Compiled by builduser.

What do you expect to happen? Highlighting is not related in any way with this repo. Highlighting via LSP might be enabled in future (pre-work for this to work here: https://github.com/neovim/neovim/pull/15723), but at the moment Neovim has no support for semantic highlighting.
You can use a Vim plugin or nvim-treesitter (https://github.com/nvim-treesitter/nvim-treesitter/) for highlighting svelte

nvim-treesitter is already install and configured, that s reason i post here to understand where is the problem

lsp config file

@kamalkech and have you installed svelte, html and JS parsers? :TSInstall svelte javascript typescript html
In any way, this is not related to lsp-config

@ThatsNoMoon i am sorry for that because i am new user of neovim and thank for ur answer ;)

svelte-lsp provides semantic highlights, if you get it working it looks just like VSCode's highlights. Better in my opinion.


Update install JDK script an instructions to make it easier to use for local development

Resolves https://github.com/aspnet/AspNetCore-Internal/issues/2163
Like the PR description say...the goal here is to make acquiring the JDK much easier for our contributors and devs.
cc @Tratcher

I’m ok with whatever you do here so long as things keep working and that it doesn’t become harder to enable E2E tests for Mac and Linux in the future.


Also update progress_bar in training_epoch_end

🚀 Feature
PR 1357 implements training_epoch_end to log metrics. The comments in the issue suggest that it should behave like validation_epoch_end, but the PR only replicates the callbacks and metric logging. This feature would add updates to the progress bar as well.
Motivation
Motivation is for the same usecase as having a progress bar.
While tensorboard or any other logger is a great resource to have. It is painful to set up in environments like slurm where you can only submit jobs and possibly cannot forward ports or set up the logging.
Pitch
The changes should only entail adding an update to tqdm here

I will look into this after #1450 is done :)


command "payto" error

version: 3.1.2
The error messages:

Server-side exception: <Fault -32603: Server error: File "/root/electron-cash/lib/transaction.py", line 493, in from_io | AssertionError
>
Traceback (most recent call last):
File "/root/electron-cash/packages/jsonrpclib/SimpleJSONRPCServer.py", line 376, in _dispatch
return func(*params)
File "/root/electron-cash/lib/daemon.py", line 279, in run_cmdline
result = func(*args, **kwargs)
File "/root/electron-cash/lib/commands.py", line 88, in func_wrapper
return func(*args, **kwargs)
File "/root/electron-cash/lib/commands.py", line 441, in payto
tx = self._mktx([(destination, amount)], tx_fee, change_addr, domain, nocheck, unsigned, password, locktime)
File "/root/electron-cash/lib/commands.py", line 428, in _mktx
tx = self.wallet.make_unsigned_transaction(coins, final_outputs, self.config, fee, change_addr)
File "/root/electron-cash/lib/wallet.py", line 927, in make_unsigned_transaction
fee_estimator, self.dust_threshold())
File "/root/electron-cash/lib/coinchooser.py", line 180, in make_tx
tx = Transaction.from_io([], outputs)
File "/root/electron-cash/lib/transaction.py", line 493, in from_io
for output in outputs)

The bug is in _mktx method of Commands class in commands.py file. I fixed the bug in my computer by changing the line
final_outputs.append((TYPE_ADDRESS, address, amount))
to
final_outputs.append((TYPE_ADDRESS, Address.from_string(address), amount))
This is another one of the "Address" class related bugs when using command.

This seems like a duplicate of #489, have you tried running it from git head?

Yes. It's duplicate. Thank you.

No problem :)


Compatibility with segment replication

Summary
With 2.9.0 release, there are lot of enhancements going in for segment replication[1][2] feature (went GA in 2.7.0), we need to ensure different plugins are compatible with current state of this feature. Previously, we ran tests on plugin repos to verify this compatibility but want plugin owners to be aware of these changes so that required updates (if any) can be made
What changed
1. Refresh policy behavior

RefreshPolicy.IMMEDIATE will only refresh primary shards but not replica shards immediately. Instead post refresh, primary will start  a round of segment replication to update the replica shard copies leading to eventual consistency.
RefreshPolicy.WAIT_UNTIL ensures the indexing operation is searchable in your cluster i.e. RAW (Read after write guarantee). With segment replication, this guarantee is not promised due to delay in replica shared updates from asynchronous background refreshes.

2. Refresh lag on replicas
With segment replication, there is inherent delay in documents to be searchable on replica shard copies. This is due to the fact that replica shard copies over data (segment) files from primary. Thus, compared to document replication, there will be on average increase in amount of time the replica shards are consistent with primaries.
3. System/hidden indices support
With https://github.com/opensearch-project/OpenSearch/pull/8200, system and hidden indices are now supported with SEGMENT replication strategy. We need to ensure there are no bottlenecks which prevents system/hidden indices with segment replication.
Next steps
With segment replication strong reads are not guaranteed. Thus, if the plugin needs strong reads guarantees specially as alternative to change in behavior of refresh policy and lag on replicas (point 1 and 2 above), we need to update search requests to target primary shard only. With https://github.com/opensearch-project/OpenSearch/pull/7375, core now supports primary shards only based search. Please follow documentation for examples and details
Open questions
In case of any questions or issues, please post it in core issue
Reference
[1] Design
[2] Documentation

Request owners to add v2.9.0 label on this issue.

Hi Plugin Owners,
Gentle reminder to look into this issue as code freeze date for 2.9.0 release is near i.e. July 11th.

Gentle reminder for plugin owners as we are past code freeze date for 2.9.0 release

2.9 released, we don't see issue for this. Close now

@ylwu-amzn : Thanks for working on this issue. I just wanted to call out that please do verify that for strong reads (if any), your plugin relies only on get/mget APIs. I am asking as it is still possible to have strong reads guarantees via write paths by using IMMEDIATE/ WAIT_UNTIL refresh policy which ensures replica shard copies are refreshed with the indexing request, ensuring any follow up data retrieval request receives latest data. With SEGMENT both IMMEDIATE/WAIT_UNTIL does not guarantee replica shard refreshes, so for strong reads, any follow up data retrieval query (other than get/mget APIs), client would need to provide either _primary - hits primary shard only but provides strong consistency or _primary_first - hits primary first (as name suggest), if primary not available request is routed to replica copies resulting in better availability (though data when hitting replica could be stale).


Suggestion: random day in the future or past

It would be useful to have two subclasses:

Random::Day::InTheFuture
Random::Day::InThePast

Which act like Random::Day, but will always provide a day in the future/past.

@tobyink ok, interesting. Thank you for the idea.

@tobyink Implemented in 0.13


Google Places Autocomplete does not work on mobile

It's not possible to select an google places autocomplete entry on mobile.
I'm using the ngAutocomplete directive.
It does work on desktop:

And this happens on mobile:

Here an example plunker:
http://plnkr.co/edit/UNQCivHSRMEofAP944RJ?p=preview
You can test this behavior on mobile and also with the chrome dev tools device emulation (Version 37.0.2062.3 dev).
Here a corresponding forum thread:
http://forum.ionicframework.com/t/google-maps-address-autocomplete-inside-slide-box/5278/8

@mhartington the ion-google-place directive does not seem to work. However, @gsabater 's slolution only works when you relode content in Safari Developer tools.
It seems that the directive does not see the places dynamic div when you navigate to the view, but when you refresh while being on the veiw the directive adds needed class:
before refresh:

after page refresh the directive adds needed class:

Also i have noticed that if you go back and forward google just appends multiple divs:

Additi

@quipu @mhartington  it seems that ion-google-place does not work with current ionic-beta14.
What version of ionic you are using ion-google-plase directive with?

Did you use it with simple way?.Test it with simple javascript.Here is a link how to use it.
http://www.w3docs.com/learn-javascript/places-autocomplete.html

Hey guys! Have someone came up with solution for @raywu issue? I'm facing the same one right now..

As @ph88lee said, I just added the relevant code to a controller function that triggered using ng-focus and it worked well like so:
$scope.disableTap = function(){
    container = document.getElementsByClassName('pac-container');
    // disable ionic data tab
    angular.element(container).attr('data-tap-disabled', 'true');
    // leave input field if google-address-entry is selected
    angular.element(container).on("click", function(){
        document.getElementById('your_input_id').blur();
    });
  }

@TillaTheHun0 Hey can you paste the exact way you did it? What code did you move into a controller?

@jason-engage I took the relevant code and placed it into a function like above. So in my controller:
.controller('MyCtrl', function($scope) {
  $scope.disableTap = function(){
    container = document.getElementsByClassName('pac-container');
    // disable ionic data tab
    angular.element(container).attr('data-tap-disabled', 'true');
    // leave input field if google-address-entry is selected
    angular.element(container).on("click", function(){
        document.getElementById('searchBar').blur();
    });
  };
})

I then used the ng-focus directive to execute the expression on the desired element like so:
<input id="searchBar" type="text" placeholder="Search" ng-focus="disableTap()">

This basically fires an event when the element gains focus, switches the containers 'data-tap-disabled' attribute to true. When you click the container, it removes focus, thus emulating user interaction. Hope this helps.

@TillaTheHun0 Thanks a lot, only this method worked for me

I'm sorry, where is the pac-container in the HTML. I basically have a HTML like this and this code does not work -
<ion-content scroll="false">

           <input type="text" id="searchBar" ng-focus="disableTap()">

</ion-content>

This method does not work.

@sidchilling pac-container is the class used to create the element for the dropdown when using Google Maps Autocomplete control. Since it is generated dynamically via the autocomplete element you setup, you don't specify it anywhere.
It's because that element is generated dynamically that this thread exists, b/c we have no control over its attributes when our views are initially rendered and when our controllers are setup in our app.
your html above should look something like this:
<ion-content scroll="false">
    <input type="text" id="searchBar" ng-focus="disableTap()">

</ion-content>

then in your controller you would have this:
var input = document.getElementById('searchBar');
var options = { //options for autocomplete object
    types: ['geocode']
 };
 autocomplete = new google.maps.places.Autocomplete(input, options);

you can then have your css to make your pac-container look pretty. (copy pasta from maps API):
#pac-input {
  background-color: #fff;
  font-family: Roboto;
  font-size: 15px;
  font-weight: 300;
  margin-left: 12px;
  padding: 0 11px 0 13px;
  text-overflow: ellipsis;
  width: 400px;
}

#pac-input:focus {
  border-color: #4d90fe;
  margin-left: -1px;
  padding-left: 14px;  /* Regular padding-left + 1. */
  width: 401px;
}

.pac-container {
  font-family: Roboto;
}

Hope this helps.

@TillaTheHun0 Thanks for your reply. Yes, I figured that out.
This still doesn't work for me. I get the autocomplete options but I can't click on one to select it. When I try the map which is behind gets clicked instead and the autocomplete suggestions goes away. Nothing in the search box and I don't get the places_changed event.
Here is my code -
HTML -
<ion-view view-title="Select a Hotel" cache-view="false" can-swipe-back="false">

	<ion-nav-bar class="bar-positive">

Select a Hotel

		<ion-nav-buttons side="right">
			<button class="button button-icon icon ion-close-round" ng-click="exit()"></button>
		</ion-nav-buttons>
	</ion-nav-bar>

	<ion-content scroll="false">

            <input id="search-input" type="text" style="height: 50px; width: 100%; padding-left: 10px; padding-right: 10px;" placeholder="Search a Hotel" ng-focus="disableTap()">

	</ion-content>

</ion-view>

Javascript (generated from coffee script) - [Partial]
$scope.disableTap = function() {
      var container;
      container = document.getElementsByClassName('pac-container');
      angular.element(container).attr('data-tap-disabled', 'true');
      return angular.element(container).on('click', function() {
        return document.getElementById('search-input').blur();
      });
    };
    $scope.initSearchBox = function() {
      var autocomplete, bookingType, input, searchBoxDiv;
      bookingType = window.getBookingType();
      if (bookingType === 'search') {
        input = document.getElementById('search-input');
        autocomplete = new google.maps.places.Autocomplete(input);
        return google.maps.event.addListener(autocomplete, 'place_changed', function() {
          var location, place;
          alert("Coming inside place changed");
          place = autocomplete.getPlace();
          if (place.geometry) {
            location = {
              latLng: {
                lat: place.geometry.location.lat(),
                lng: place.geometry.location.lng()
              }
            };
            return $scope.locationSuccess(location);
          }
        });
      } else if (bookingType === 'nearby') {
        searchBoxDiv = document.getElementById('searchBox');
        return searchBoxDiv.style.display = 'none';
      }
    };
    $scope.showMap = function() {
      var divEl;
      divEl = document.getElementById('map-canvas');
      $scope.c.map = plugin.google.maps.Map.getMap(divEl);
      return $scope.c.map.addEventListener(plugin.google.maps.event.MAP_READY, $scope.onMapReady);
    };

Do you see anything that I am missing? I am trying to get this to work for hours now with no luck :(

@sidchilling do you have the data-tap-disabled attribute set to  true for your map div?

@TillaTheHun0
you the man! Thanks you :)

@ph88lee
Thanks mate, that worked.

i wanted to add backdrop on ng-focus and release on ng-blur but i dont want backdrop on input element how can i do it , what z-index should i set

So anyone has a solution for this at this point? Meaning one that can work on mobile instead of just desktop. Also use Google Places instead of Geocoder.

@quanghoc https://github.com/driftyco/ionic/issues/1798#issuecomment-95943955 this method worked for me, its already present in the comments above

As @wallstudios , this http://plnkr.co/edit/SjSDzb70NcXImWk9n5lX?p=preview does not work on iOS mobile device. I tried triggering tap disable when click on the input, still same problem.
I am using latest Ionic 1.4.5

Never mind, I missed this post TillaTheHun0 commented on Apr 5

I've been trying to solve this issue by following the suggestions on the tread, @TillaTheHun0  the pac-container doesn't exist when the ng-focus event fires, any suggestions?

If pac-container doesn't exist, your Google Places API didn't get init in the first place. It should show the suggestion list first.

@alanquigley the pac-container should be generated by the Google Places API when the user starts to type in a place. If it is not present, then check to see if you have the Places API implemented.
https://developers.google.com/maps/documentation/javascript/places

The idea is to wait for the generated .pac-container to become available then attach the data-tap-disabled attribute to it as well as closing the selection dropdown after the user has tapped a choice.
Setting a fixed timeout is unreliable because your timeout may expire before .pac-container is generated. Setting repeated timeout or interval kept crashing my app on android. I finally found a solution that can detect node insertion using CSS keyframe animation. It's been working well for me on Android. If you try this on iOS, please tell me if it works.
//
// CSS - define a keyframe animation called 'nodeInserted'
//
@keyframes nodeInserted {
	from { opacity: 0.99; }
	to { opacity: 1; }
}
div.pac-container {
  animation-duration: 0.001s;
  animation-name: nodeInserted;
}

//
// Jquery - listen for 'nodeInserted'
//
$(document).one('animationstart', 'div.pac-container', function(event){
   if(event.originalEvent.animationName === 'nodeInserted'){
   	$(event.target).attr('data-tap-disabled', true).click(function(){
   		$('#myAutocompleteInputElem').blur();
   	});
   }
});

I've also created a gist

It still doesn't work for me on Iphone only. The dropdown for selection is closed when I chose one of the suggestions but then the places_changed event is not triggered. Has anyone found a solution to this?

@mohsinali1317 sounds like data-tap-disabled isn't being set to true on the pac-container, which is that dropdown. Have you tried my suggestion above?

@TillaTheHun0 I found that the issue is with foundations frameworks fastclick JS. It conflicts with the auto complete on touch devices.

TillaTheHun0 solution is working perfectly but while using a modal view it does indeed add data-tap-disabled true to pac-container but the options are still unclickable.
dash.html :
<ion-view view-title="Dashboard">
  <ion-content class="padding">

Open

  </ion-content>
</ion-view>

controller.js :
$ionicModal.fromTemplateUrl('templates/localisation-modal.html', {
        scope: $scope,
        animation: 'slide-in-up'
 }).then(function(modal){
        $scope.modal = modal
 });

// Fixe for ngAutocomplete
$scope.disableTap = function(){
    container = document.getElementsByClassName('pac-container');
    // disable ionic data tab
    angular.element(container).attr('data-tap-disabled', 'true');
    // leave input field if google-address-entry is selected
    angular.element(container).on("click", function(){
        document.getElementById('autocomplete').blur();
    });
}

@brieuc28 I think I need more information. Can you put together an example that reproduces the issue?

I'm locking this issue. This is a conversation that should take place in the forum, rather than github issues. Thank you.


Feature/roe 1730 trust involved show bos

JIRA link
https://companieshouse.atlassian.net/browse/ROE-1730
Change description
Work checklist

[ ] Tests added where applicable
[ ] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

SonarQube Quality Gate failures and merge conflicts need resolving


Added the allowed values to $select, $expand and $filter queries unde…

this is related to github issue 9130

/azp run automation - sdk

Can one of the admins verify this patch?

Would it be too much for a fix of issue #9130 which is only complaining that the examples are not correct by changing those parameters from string to enum?
And this would introduce a lot of breaking changes in the downstream SDKs (parameter type changed from string to an enum type).
Furthermore, I suppose the service may support more filter in the future, if we convert these into enums, when the service supports more filters in the future, you will have to add more entries for the enum and introduce breaking changes to the swagger in the meantime.
Could it just work to just modify the corresponding examples but leave the swaggers unchanged?

As we need to mention the allowed values for the clarification of the customers, can we include them in the description without changing the type of the parameters?
@avirishuv Could you please clarify?

The intent of this proposed swagger update is simply to explain to customers how can they use the parameters $expand, $select, $filter, and what are allowed values for these.
No changes needed on the type of parameters itself, but we just need to clearly document the usage & allowed values. Hope that helps, @srcharug

Thanks for the clarification @avirishuv . Then, I will just make the changes to the description which does not effect the test cases/examples.

/azp run automation - sdk

/azp run

/azp run

Hi @srcharug could you please also go through those examples?
For instance this one, the value of "$expand": "virtualMachines\\$ref" seems it does not make much sense.

Hi @ArcturusZhang, when we query for instance view of VMs in VMSS, we can apply combinations of $select and $expand fields to get different views of VMInstances.
For example, if $select=instanceView&$expand=instanceView we get instance view of VMs,
if $expand=instanceView we get full view of VMs.

Hi @srcharug I am suggesting we should also use "service-acceptable" parameters in the corresponding examples in addition to this change, for instance this example, and actually this was the file the original issue was complaining about.
And the same situation may happen to the examples of other api-versions, please change them too.

/azp run automation - sdk

I made the suggested changes. I was not able to understand that when you mentioned about the examples earlier


Improve tabs horizontal scrollbar size/contrast

The file tab horizontal scrollbar is tiny and low contrast. This makes it cumbersome to drag it with the mouse or to track it visually while using the scroll wheel.

Similar requests have already been reported but their focus quickly changed to other areas of the file tabs UI. There wasn't any discussion about the scrollbar and now those threads are locked (ex: #51546 #7987).

/duplicate of #32192


Bug

Bug description
My problem is bot doesn't play songs when bot is restarted and on 24/7
Step to reproduce
...
Operating system
Linux
Nodejs version
V20.10.0
Errors / Logs
No response

24/7 in lunox only make the bot keep connected to voice channel when the bot got stop/disconnected/restarted.


feat: make lagrange_basis_unnormalized + folders in uni-stark pub

Making these methods public makes it easier for projects trying to use individual components in uni-stark.

Sounds good, feel free to merge @dlubarov


API documentation missing

I would like to implement #1 but don't know in which format the endpoints expect their input. I tried to guess it by looking at the source code but failed with various attempts, I always get Error parsing Request Body: Error parsing User to struct. (Needless to say that that's a very inconvenient method to get this information anyway.) Can you please add documentation for the API? Sample snippets would be enough.

There is documentation in the Open API format here
You can explore it more easily on a site like this
Are there examples missing?

Oh, I didn't know that file yet. That's a big improvement already! 👍
What's missing is how to supply the username + Fachschaft. E.g. for me is it "christoph" or "fsgi.christoph" or "cn=christoph,o=fsgi" (*) or do I have to supply another property called fs or noch was ganz anderes?
(*) this seemed the most logical variant for me because /api/users/list supplies the names in this format, but it didn't work when I tried.

What's missing is how to supply the username + Fachschaft

Why do you want to supply the Fachschaft? Does it not work when just specifying username + group as described in the docs?

Does it not work when just specifying username + group as described in the docs?

No, just supplying the username didn't work.
But now I found the error: My payload JSON had the property named groupname, not group. After fixing that it works fine with just the username 👍 I never looked for the mistake there because the error message was nevertheless: "Error parsing Request Body: Error parsing User to struct" 🤔

Why do you want to supply the Fachschaft?

To me it seemed logical that I had to supply the Fachschaft because the removeUser and changePassword endpoints require it too. Why is it needed there but not here?

when just specifying username + group as described in the docs

No, just supplying the username didn't work.

:laughing: so not as described in the docs

I never looked for the mistake there because the error message was nevertheless: "Error parsing Request Body: Error parsing User to struct"

Which is the correct error, since the User that has been parsed from the request failed validation as it misses a required field ("group") (Further details below).

[...] because the removeUser and changePassword endpoints require it too

This stems from the fact that the backend differs between "GroupUser" which is a combination of username and group association and "User", which has a password and fachschaft.
At the moment of programming this seemed to be the best option, as we can do proper validation now when initializing the structs. If all properties would be in a single struct, they would often be nil and validation becomes more difficult (what properties are needed for this specific request? does password need to be set now? Or is it a groupAdd request that does not have a password? )
Will close this issue as it has been solved

😆 so not as described in the docs

Yes, I admit that 😇
but...

Which is the correct error

I very much contest this - a user and a group are two different things; I wouldn't expect the latter to be a property of the former.

Are there examples missing?

I came across one more when I saw sha512 in the frontend code again: The docs don't state in which way the password should be submitted (the user could expect cleartext or some other hash method just as well).

Merged with https://github.com/fs-geofs/UserManager/pull/10