🔧 vue 3 composition create pages

part of #4750
#6392

⚠ need to test /massmint page
edit:
massmint tabs doesn't work on beta

BSX:
Massmint:
https://basilisk.subscan.io/tx/0x7995a248086df736647cacad700991957c58fad546775592a16eb6a019e3c9d4
NFTs listed properly
redirection after minting working

RMRK2:
creating a collection redirects to the NFT tab properly
massmint:

RMRK2
Massmint: ✔
mint: https://kusama.subscan.io/tx/0x54ef873876b94b72e1c73940bf579f9344065c58bf49d3bbf794a9ad1ab0e69e
listing: https://kusama.subscan.io/tx/0x6917fe3ba84973f209e9b714350268858ddc59ae82b5ae0be2bb4f505b734061
obs: asked for 2 transactions
Collection: https://kusama.subscan.io/tx/0xceada1d1fb90ef8c9ddc7f2b403883feb4e812ef5ed633cf9ff24639a673ef40
NFT: https://kusama.subscan.io/tx/0x71c325b27f413fe520e5e8348eab00280713f8f3120c5c3239396c4f5efeefec
https://kusama.subscan.io/tx/0x84622531c1750d5306e5598e3d15c420395fa0c4ef8ed47d2e1f4a53fbf0a7f5
RMRK1
BSX
SNEK
STMN
will be updating as i go

Hello, just so you know I didn't change anything related to minting process.
Only thing changed are pages redirection & tab related with url (/:prefix/create/?tab=NFT & /:prefix/create/?tab=Collection)

Looks oki,
let's fix the redirect and happy to proceed

Wait does the redirection work for you? When I tested it on my side it was working as excepted (tab relates with URL path)

Wait does the redirection work for you? When I tested it on my side it was working as excepted (tab relates with URL path)

Oh, i got it now, yes , the tab redirection works properly for all of them. When testing i was testing both the tab redirection and the automatic redirection that happens after i mint something (this is what is not working)

Wait does the redirection work for you? When I tested it on my side it was working as excepted (tab relates with URL path)

Oh, i got it now, yes , the tab redirection works properly for all of them. When testing i was testing both the tab redirection and the automatic redirection that happens after i mint something (this is what is not working)
I may have got confused because i saw a timeout function

Alright I'll be checking this next week

Wait does the redirection work for you? When I tested it on my side it was working as excepted (tab relates with URL path)

Oh, i got it now, yes , the tab redirection works properly for all of them. When testing i was testing both the tab redirection and the automatic redirection that happens after i mint something (this is what is not working)
I may have got confused because i saw a timeout function

Alright I'll be checking this next week

Thank you for clarifying it for me


Problem with EM-Websocket running in a thread under JRuby

I'm working on some code that runs EM-Websocket in a thread.  It works fine under MRI but fails under JRuby.  I've created a reproducer here with more details in the README but I will give a brief outline here:
I'm using EM-Websocket to implement LiveReload functionality for the Jekyll static site generator.  The goal is for users to run jekyll serve --livereload and Jekyll will refresh a connected browser as it regenerates pages based on changes the user makes to the underlying files, in effect giving the user a real-time preview of how their changes will appear.
To accomplish this, I start an EventMachine reactor in a separate thread that is responsible for sending the LiveReload messages over the websockets connection.  When Jekyll regenerates a file, I've inserted a hook that prompts the reactor to send a reload message to the browser.
Under JRuby, the reload messages never actually seem to get sent.  I've looked at packet captures (also provided in the reproducer repo referenced earlier) under both the MRI and JRuby and the packets with the reload message never actually hit the wire under JRuby (although the initial LiveReload handshake does get made).
This may be a problem with EventMachine itself; I'm not enough of an expert to tell, but I thought I would start by reporting the issue here since my use case is websockets specific.
Any assistance or insight you could provide would be greatly appreciated.

Have you tried instrumenting the code to see if send_data is being called? I have a hunch that there is something in the guts of EventMachine.. or we have some callback exploding & being swallowed.

Yeah, it looks like send_data in connection.rb in EventMachine just invokes EventMachine::send_data @signature, data, size and then... nothing.  So this does appear to be a problem in EventMachine itself.  Thanks for the suggestion on instrumenting the code.

D'oh.  Looks like someone else already hit this issue!  Sorry for the spurious issue report!


Follow Up PR for Scala Native Example Code

Follw up for #3647

CC @sideeffffect @lolgab as folks who know scala-native and gave feedback on the original PR


Add auto merge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/ryvencore-feedstock#2.


Extensions: flicker when opening first time

Steps to Reproduce:

start vscode
click on extensions view

=> 🐛 list flickers

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Alias for '/' not worked in nested routes

Version
2.5.3
Reproduction link
https://jsfiddle.net/x4storm/qmjw452y/
Steps to reproduce
click / and /bar in Route Alias section, it will same result
click / and /home/foo in Route nested Alias section, the / alias not work
What is expected?
nested route alias for / just like the flat alias.
What is actually happening?
alias for / not worked in nested routes

This is fixed by 8e30c34


v4 errorClass vs. errorMessage

Hi, after update to v4 I wanted to accomplish the same logging result but it is quite "hacky". How can we set errorClass and groupingHash from notify method?
Our v3 implementation was:
bugsnag.notify(errorClass, errorMessage, data, 'error');
Expected behavior
There should be probably some parameter in the notify method or as a field in opts to set errorClass and groupingHash.
Version
v4.0.0
Additional information
What I do currently is that I set the errorMessage even as an empty string (first param of notify method) and in beforeSend() I modify errorClass and add a groupingHash if provided in data.
notify (consoleSeverity: BugsnagSeverity, errorName: string, data: object, message?: string): void {
    this.bugsnagClient.notify(
        message || '',
        {
            beforeSend: (report: Report): void => {
                report.errorClass = errorName;
                if (report.metaData.hasOwnProperty('groupingHash')) {
                    report.groupingHash = (<any> report.metaData).groupingHash;
                }
            },
            metaData: {...data},
            severity: consoleSeverity,
        },
    );
}

You can do bugsnagClient.notify({ errorClass: 'Error', errorMessage: 'your message here' }).
groupingHash can only be set via beforeSend callback though.

ohh, thx
I looked to the implementation of normaliseError() and it would be nice then to extend a ts definition from error: any in:
public notify(error: any, opts?: INotifyOpts): boolean;

to something like:
export interface ErrorObject1 {
    name: string;
    message: string;
}
export interface ErrorObject2 {
    errorClass: string;
    errorMessage: string;
}
export type ErrorType = string | number | boolean | ErrorObject1 | ErrorObject2 | any;
...
public notify(error: ErrorType, opts?: INotifyOpts): boolean;

It could be still improved as there are 2 "glitches":

any has to stay to handle isError()
There should be actually 4 IErrorObject* interfaces to handle all combinations possible in the code, but it should be fine to handle the inputs reasonably

Do you think it would be ok to create a PR for it?

My thoughts around this are…

since any will always be allowed (it needs to be, for the reason you mentioned and things like the angular error handler, my understanding is that the addition of the other types here are to provide editor support and helping suggest the correct usage?
if so, I think we should have the following definition: ErrorType can be

an instance of Error
{ name: string; message: string; }
{ errorClass: string; errorMessage: string; }
or any

I want the types to help users make the correct choice, and these are the most correct ways to use notify()… passing primatives is supported but it's not as useful so I want to guard against that if possible. What do you think?
If you don't see any problem with that, I'd gladly welcome a PR 👍 Glad to have another perspective on the Types also. Thanks

@bengourley I created a PR, let's move the discussion there if necessary. I totally agree with encouraging to use only recommended objects (therefore omit scalar values).

Just so I'm clear, the reason for using any is so you can do runtime rejection of objects not listing in this library?
function isError(value) {
  switch (Object.prototype.toString.call(value)) {
    case '[object Error]': return true;
    case '[object Exception]': return true;
    case '[object DOMException]': return true;
    default: return value instanceof Error;
  }
}

If so, as one of the primary points of typescript is to avoid runtime errors, this choice seems backwards to me. You give up compile time type checking for runtime type checking with soft failure.


Error: process.nextTick is not a function

I am getting Error: process.nextTick is not a function
when I use observer, please help me

process.nextTick() is a node.js API call:  https://nodejs.org/api/process.html#process_process_nexttick_callback_arg
Are you trying to use this under node.js or somewhere else (inside a browser??)  This is specifically a node.js package.

Thanks for reply, may be wrong place to raise this issue. I am trying to connect react-native using ddp-client and I get this error. please suggest how I can fix this.

I'm having the same error when trying to connect.

Are you running in node.js or a browser?

If someone gets on this issue when searching, the solution is to add the polyfill specified on minimongo-cache


node: Cannot use Socket because it doesn't match net$Socket

I have an code snippet that looks (basically) like this:
export default function handleSocket(request: IncomingMessage, response: ServerResponse): void {
  /* … */
  return listenToSocket(request.socket);
}

function listenToSocket(socket: Socket): void { /* … */ }

flow is returning an error on return listenToSocket(request.socket); saying:

request.socket (Cannot call listenToSocket with request.socket bound to socket because net$Socket [1] is incompatible with Socket [2].)

However in the type definitions Socket is only an extension of net$Socket
https://github.com/facebook/flow/blob/ac184467b87d2726c193aea332be9695ce3d5185/lib/node.js#L1402.
I feel like this message is an error, but I am not sure how to circumvent it.
Any help would be great!

I see that IncomingMessage has socket member that points to net$Socket
https://github.com/facebook/flow/blob/ac184467b87d2726c193aea332be9695ce3d5185/lib/node.js#L1222
This could be the source, but I can understand why you are referencing the private definition.


remove calls to vasprintf

What problem is this PR intended to solve?
#2481 proposed cleaning up our local implementation of vasprintf and using it on all platforms; however this PR takes an alternative approach of removing calls to vasprintf in favor of using rb_vsprintf since we need to create Ruby String objects from all the errors being handled this way.
Some nice side effects:

TruffleRuby should like this better, since Sulong was missing functionality around varargs
we no longer have to do platform-specific detection of vasprintf at compile time

Note also that this PR is based on #2480, so that should be considered before this one.
Have you included adequate test coverage?
Yes! Some of the error handling code did not have good, or any, coverage. Test coverage has been added in this PR.
Does this change affect the behavior of either the C or the Java implementations?
No behavioral changes are being introduced.
cc @Garfield96 @stevecheckoway

Thanks for asking! This looks all great! No RB_GC_GUARD necessary.
There's a single remaining stdc free() call: https://github.com/sparklemotion/nokogiri/blob/1b7f25917e157a59656010e012c14f20630e2c49/ext/nokogiri/gumbo.c#L276
It should be gumbo_free() instead. Or even better gumbo_alloc() and siblings could be converted to ruby_xmalloc equally.

@larskanis I saw that free call, but left it for now because gumbo_alloc and gumbo_free aren't public.
I agree it would be nice to add a feature to libgumbo to allow us to set the alloc/free functions (like xmlMemSetup does for libxml2), so I've created #2490 to drive that discussion.

At the moment, Gumbo isn't tied to ruby in any way, I think it makes sense to keep it that way, absent a compelling reason.
Gumbo originally had a way of supporting other allocators, but it made the code worse and was removed. It makes sense to me to keep Gumbo's "public" API returning memory from the standard allocator. I don't think any of the utility functions in util.h should be part of Gumbo's public API.
I'd be in favor of removing gumbo_free() entirely and just replacing it with calls to free().

Ah, OK, @stevecheckoway I just saw your update and will defer to your decision if you feel strongly. Go ahead and close #2490 if you wish.

I've rebased after merging #2480 and so this changeset should be more readable now.

I also kicked off the truffle CI jobs on this branch because I think this PR may break some of the patches being applied in https://github.com/oracle/truffleruby/blob/master/lib/truffle/truffle/patches/nokogiri_patches.rb
cc @eregon @aardvark179

@flavorjones Those runs failed, any idea of the issue?
For the first 2 I can't find the error, maybe Expected '| ' but got ?
For the 3rd job it's:
 /__w/nokogiri/nokogiri/ext/nokogiri/xml_xpath_context.c:301:63: warning: function 'xpath_generic_exception_handler' declared 'noreturn' should not return [-Winvalid-noreturn]
va_list args; rb_raise(rb_eRuntimeError, "%s", "Exception:"); return;
                                                              ^
/__w/nokogiri/nokogiri/ext/nokogiri/xml_xpath_context.c:344:27: error: use of undeclared identifier 'errors'
xmlSetStructuredErrorFunc(errors, Nokogiri_error_array_pusher);
                          ^
1 warning and 1 error generated.

It fails the same way on master (https://github.com/sparklemotion/nokogiri/runs/5800516006?check_suite_focus=true)

@eregon some of the TR patches are no longer applying, let's discuss at https://github.com/sparklemotion/nokogiri/issues/2491


chore(batcher): use thin block info to create block info

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2238  👈 (View in Graphite)
#2237
#2231
#2230
main

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

crates/starknet_batcher/src/batcher.rs line 428 at r5 (raw file):
Previously, ArniStarkware (Arnon Hod) wrote…
I like it here. Each function called validate_object is, by nature, very confusing. I want to see what we are validating as clearly as possible.

A better way is to create 2 internal functions with clear naming and just calling them

crates/sequencing/papyrus_consensus_orchestrator/src/sequencer_consensus_context.rs line 56 at r8 (raw file):
Previously, alonh5 (Alon Haramati) wrote…
Add TODO to remove?

Done.

crates/starknet_batcher/src/block_builder.rs line 221 at r5 (raw file):
Previously, alonh5 (Alon Haramati) wrote…
Okay, I didn't fully understand, but it's not important for now. I'll add a task to Monday to reconsider it

I think it should not even be on Monday.

Merge activity

Dec 5, 4:36 AM EST: A user started a stack merge that includes this pull request via Graphite.


login page

As a user who wants to log in
GIVEN go to the home page and click on the login button to Log in.
WHEN the user write email and password in the box and click on button login
THEN see the dashboard with the user name

I am on the dashboard but I can't see my name


Update Composer dependencies (2022-01-13-00-09)

Visual regression test passed!
View the visual regression test report


bevy_master vs bevy_main

The readme states:

The branch bevy_main is tracking bevy/main.

...but there is no bevy_main branch.  Perhaps the bevy_master branch was meant to be renamed?

Yeah, sorry for the confusion. I will delete the bevy_master branch since it was already merged in #6.

Thanks for explaining!  Do you have plans for making a release to target Bevy 0.5.0?

The latest release targets Bevy 0.5.0. Is it not working for you?

Oh! No, I just jumped to the conclusion that bevy_kira_audio 0.4.0 was for Bevy 0.4.0.  I'll go give it a try right now!

Works great! I opened a PR for a minor issue I hit: https://github.com/NiklasEi/bevy_kira_audio/pull/8


Additional configuration to ignore games from different accounts

Hi, it seems that the Steam option "Ignore installed games from other accounts" doesn't work on my computer. My partner and I use the same computer: we have two Windows accounts linked to two different Steam accounts with two Playnite instances. We have the option "Private Account" selected with our own API key and "Ignore installed games from other accounts" but still Playnite imports games from the other account. I noticed that by removing "Import installed games", it doesn't happen anymore but unfortunately Playnite doesn't have a clue about what games are installed. Not sure if important but we use the shared library functionality because from time to time we try games before buying them but I think from the Steam API Playnite should be able to identify the owned games. Please let me know what info I can provide because I'm not really sure. I have generated the diagnostic ID.
Diagnostics ID
875c401c-5f63-4b71-9eef-d3200209d298

You have to actually set those additional accounts that should be used to detect installed games from extra accounts unde "Additional accounts" tab.
We can't automatically ignore installed games that are not found on your linked account because there are some cases where Steam API we use won't report games as part of your library, which could lead to incorrectly ignoring games. Therefore you need to set what extra account should be used to detect games that are not from your main account.

Hi, I'm sorry but I have the opposite problem, the games from the other account are imported regardless of the "Ignore installed games from other accounts". I do not want them in my account, I only want my games in my account. I don't see an option "exclude games from this account" in the "additional accounts" tab you suggested. Thank you.

I see, configuration you describe is currently not supported.

The one workaround would be letting those games to import and then remove then and add them to the ignore list.


replace hyphen with underscore in DB_NAME

Hi , this is a fix for replace hyphen with underscore in DB_NAME .
lambo my-project

the DB_NAME should be my_project because mysql doesn't accept hyphen in database name

Thanks @ahmedash95, but MySQL does accept hyphens in the database name; it just doesn't accept them in column names.

@mattstauffer really ! , oh sorry i just try it and didn't work for me with many tries and search. sorry again :)


[Feature]: metrics and more logs for barman

Is there an existing issue already for this feature request/idea?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new feature request to be evaluated.

What problem is this feature going to solve? Why should it be added?
There is no metrics for barman, which can be used for alarming.
I found "cnpg_pg_stat_archiver_*"
but from the numbers these seem to be focused on WAL archiving (continuous backup).
I didn't find any barman related metrics like "time_since_last_barman_backup" , where I could base an alarm on.
This would help to find ScheduledBackups which are stuck in "running" state and pile up. Seems like there is no
timeout for ScheduledBackups I guess.
Describe the solution you'd like
E. g. metrics like:
cnpg_pg_stat_barman_seconds_since_last_archival
and
cnpg_pg_stat_barman_seconds_since_last_failure
Describe alternatives you've considered
I already have alarming in elastic based on log entries like "backup failed".
But this solution doesn't cover all cases and also has downsides. Take this particular case: When a new backup job finds the same job already running you can find the log entry: "Backup is already running on". But often backup jobs run nightly, and therefore the first time you get notified about this is during on-call duty. It would be beneficial to check on your backups based on a metric which provides the time of the last successful barman backup since I can check on them based on the time frame I expect them to be finished  - and also last failed barman backup would definitely help.
Additional context
Right now, I have two clusters, which
have multiple ScheduledBackup jobs in "running" state. Logging shows no errors.
Only "Backup is already running on" is logged.
For one cluster I deleted the "running" jobs and started a backup task manually. This
finished successfully. The following ScheduledBackup was also ok.
(Maybe a timeout is also an option. But I don't have details about whats going wrong so... )
Backport?
Yes
Are you willing to actively contribute to this feature?
No
Code of Conduct

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

Hi @quercus-carsten
This information it's already available here you have the documentation https://cloudnative-pg.io/documentation/current/monitoring/#predefined-set-of-metrics
timestamps indicating last failed and last available backup, as well as the first point of recoverability for the cluster

Closing now, feel free to reopen if needed
Best regards!


Increase GPU Utilization?

This is my first time training on this GPU and using neon. I just started running some examples and I noticed that my GPU is only peaking about 31% during training and stays more around 22%. Can I increase the GPU usage somehow in order to train faster?

This will really depend on the model that you are training. If the model has small layers (feature map size and/or filter size), the GPU kernels will execute very fast and the python code in neon becomes the limiting factor. More computationally intensive models should be able load the GPU at nearly 100%. For example, you should see higher utilization in general for models training on imagenet data than cifar or mnist data. You can also try using more feature maps for each layer to get higher utilization, although whether this is worthwhile will depend on if that improves your model.

Well, for instance, I'm planning to do a lot of work with RNNs. The first example I ran was the Shakespeare text example. I ran python text_generation_lstm.py -b gpu . I was watching a movie at the same time so prior to running it my GPU was at around 15%. While training, it stayed around 25% and took about 1 minute 5 seconds be epoch. I know could blow through the training if I could utilize most my card.

Right so for that network, the hidden size is 512 and your batch size is probably set to 128. The LSTM layers are basically doing a for loop over time steps and evaluating several matrix multiplies at each step. In this case we are bound both by available parallelism and kernel launch overhead. In order to be compute bound in the GPU kernels and fully utilize available FLOPs, our matrix multiply kernels have a tile (CUDA block) size of at least 128x32. So at best you are getting 16 blocks, which may not fill your GPU depending on which one you have (for example Titan X has 24 SMs, each of which could concurrently execute 2 blocks of this kernel). The second problem here is kernel launch overhead. This matrix multiply kernel will probably complete in less than a millisecond, which may be shorter than the time it take the python code to launch the next kernel, leaving your GPU under-utilized. We have some special RNN kernels available to help mitigate this launch overhead, but they currently only support specific cases of RNN and BiRNN layers.

Stewart, it should be pretty trivial to integrate these new gemm kernels into neon:
https://github.com/openai/openai-gemm
The 32x32 tiles really help utilization and are at least as fast as the 128x32 kernels.  The python code also memoizes most of the launch overhead.  Those kernels also allow arbitrary mixed precision (though that python code doesn't expose it to limit kernel permutations for libs that statically load them)

Thanks for sharing Scott. I'll take a look at these.

I do have some changes worked out to integrate these GEMM kernels into neon, but need to figure out how to install the code from the openai-gemm repository with neon. Temporarily using a setup.py in my local.
Unfortunately Tom this won't really give you much improvement on that network. I did some profiling and the launch overhead is significant as I expected. However, most of the overhead appears to be coming from the elementwise optree processing (see image below). In the future, our graph based framework should help with this. We have a preview of that available now if you are interested, but it doesn't support these types of recurrent networks yet: https://github.com/NervanaSystems/ngraph

The GEMM kernel, where the majority of compute is happening, is only running about 7% of the time. The other time appears to be spent mostly in the python code involved in calling the elementwise kernels.


pgsoduim interaction breaking schema introspection on supabase

Sorry if this issue belongs elsewhere. Feel free to redirect me to the right repo. Thank you!
Describe the bug
A clear and concise description of what the bug is.
Unlike on the self-hosted Supabase setup via Docker, the GraphiQL editor on the hosted app.supabase.com throws errors that disable the docs feature but do allow queries to be executed.
To Reproduce
Steps to reproduce the behavior:

Create a new project on app.supabase.com
Create a table using the SQL Editor page
Navigate to the API Docs -> GraphiQL page

Expected behavior
Steps to reproduce the behavior:
To be able to browse the docs for my GraphQL API.
Screenshots
If applicable, add screenshots to help explain your problem.

Versions:

pg_graphql v.1.2.2 (as returned by: select * from pg_available_extensions where name = 'pg_graphql';)
PostgreSQL 15.1 (Ubuntu 15.1-1.pgdg20.04+1) (as returned by select version();)

Additional context

If it's helpful for an admin to pull up the account/project that shows this issue, the following url should identify it in the dashboard: https://app.supabase.com/project/bagsxvvlcxltwrhrvctu/api/graphiql
The following is the error trace from the first screenshot:

{
  "errors": [
    {
      "message": "Names must only contain [_a-zA-Z0-9] but \"aead-ietf\" does not.",
      "extensions": {},
      "stack": "u@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:36586\ns@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:85601\na@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:85893\n67950/ie/this._values<@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:95816\nie@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:95660\n80707/h/h</<@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:133539\n80707/h/h</<@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:133703\n80707/h/h<@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:134131\nr@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:45036\nh@https://app.supabase.com/_next/static/chunks/4231-d56b093093772f41.js:1:132725\n71254/_v/E</<@https://app.supabase.com/_next/static/chunks/966b35fd-3515f03fdeac5969.js:1:288080\n"
    }
  ]
}

hi @ian-s-mcb
I just responded to the support issue you opened. Copying the same information here in case others hit the same issue

The issue you're experiencing is https://github.com/supabase/pg_graphql/issues/172
It occurs when the name of a Postgres enum or one of its variants does not meet there requirements for GraphQL name. Those requirements are [_A-z0-9]+. In your case it looks like you have some disallowed characters in the key_type postgres enum.
There is a draft PR https://github.com/supabase/pg_graphql/pull/376 that will add the ability to override enum variant names for GraphQL but in the immediate term the solution would be to change the enum variants to valid GraphQL names [_A-z0-9]+

in your case you could resolve by changing aead-ietf to aead_ietf or AEAD_IETF to match GraphQL conventions
note that the API will continue to resolve queries when you hit this issue, only introspection rendering is effected
thanks

dupe of #172

@olirice without any changes to my tables I started experience this issue this week with the exact same name: "aead-ietf"

@olirice that enum wasn't created by me. Are you familiar with pgsodium? I ran the following SQL to list all my enums and that uncooperative enum is associated with pgsodium. Does this warrant an issue on supabase/supabase?
select n.nspname as enum_schema,
       t.typname as enum_name,
       e.enumlabel as enum_value
from pg_type t
   join pg_enum e on t.oid = e.enumtypid
   join pg_catalog.pg_namespace n ON n.oid = t.typnamespace;

enum_schema
enum_name
enum_value

pgsodium
key_status
default

pgsodium
key_status
valid

pgsodium
key_status
invalid

pgsodium
key_status
expired

pgsodium
key_type
aead-ietf

pgsodium
key_type
aead-det

pgsodium
key_type
hmacsha512

pgsodium
key_type
hmacsha256

pgsodium
key_type
auth

pgsodium
key_type
shorthash

pgsodium
key_type
generichash

pgsodium
key_type
kdf

pgsodium
key_type
secretbox

pgsodium
key_type
secretstream

pgsodium
key_type
stream_xchacha20

auth
aal_level
aal1

auth
aal_level
aal2

auth
aal_level
aal3

auth
code_challenge_method
s256

auth
code_challenge_method
plain

auth
factor_status
unverified

auth
factor_status
verified

auth
factor_type
totp

auth
factor_type
webauthn

Same problem here

thanks all, we're investigating now
cc @michelp

okay a fix is expected to merge this afternoon
through the usual channels it may take a few days for it to roll out to all projects
If you need a solution sooner than that:

if you're not using pgsodium or vault you can drop those extensions to get schema introspection working now
if you respond with your project_ref/s I'll update the instances as soon as the v1.2.3 pg_graphql release is cut


gulp-typescript make .d.ts file with references and without interfaces

Expected behavior:
I expect a declaration file with interfaces and classes like:
export interface IAnimal {
    name: string;
    move(steps: number): boolean;
}
export declare class Dog implements IAnimal {
    name: string;
    move(steps: number): boolean;
}

Actual behavior:
But I obtain this:
import * as interf from '../interfaces/animal';
export declare class Dog implements interf.IAnimal {
    name: string;
    move(steps: number): boolean;
}

gulpfile.js
gulp.task('compile', function () {
	var tsProjectDts = ts.createProject('tsconfig.json', { sortOutput: true });
	var tsResult = gulp.src('src/**.ts')
		.pipe(ts(tsProjectDts));
	tsResult.dts
		// .pipe(concat('index.d.ts'))
		.pipe(gulp.dest('dist'));
});

tsconfig.json
{
    "compilerOptions": {
        "removeComments": false,
        "noLib": false,
        "emitDecoratorMetadata": true,
        "noImplicitAny": false,
        "module": "commonjs",
        "target": "ES5",
        "experimentalDecorators": true,
        "sourceMap": false,
        "declaration": true,
        "noResolve": false,
        "moduleResolution":"node",
        "outDir": ".tmp"
    },
    "filesGob": [
        "src/**/*.ts",
        "!typings/**",
        "!node_modules/**"
    ]
}

All of this on a simple example
You can check all of this here gulp-typescript-example-declaration
I trying to make a library with respective .d.ts file. #393  #385

This is the correct behavior of TypeScript, and I cannot change that in gulp-typescript. Though the generated declaration file can be used when you publish it on NPM, you can set the typings property of the package.json file. If you can't find it, I can post the link later on.

Thanks @ivogabe, I don't have problem with NPM, just with generated .d.ts file.
It would be better if the import is eliminated and replaced by the code of imported file. Any way for do this?

No, that's not possible with TypeScript and thus also not in gulp-typescript.


Validator: WR and RD float on reset

https://github.com/andreas-jonsson/virtualxt/blob/feecf2b4f30b516d9bee9797c356f28ec51d2084/tools/validator/pi8088/pi8088.c#L175
This assert fails on my particular 8088, as it seems both RD and WR read active-low until the first ALE.  Might be different on different revisions? My chip has a copyright date of 78 & 81

I have two 80C88's from 78 that both work. However... I did find a V20 that shows this behavior. :)
It is fixed in cff9edd3ca9e77141a7fe2985a31fb35f726af2c.
Thanks for reporting!


feat: a superficialchange to trigger a release

How to review this PR
Leave this text block for the reviewer

Check component hierarchy is followed correctly
Check the design Heuristics have been followed
Check naming conventions have been applied
Check for these gotchyas:

Missing exports for Oak components
Accidental export of Internal components
Snapshots of unexpected components have been modified
Circular dependencies
Code duplication (via not using base components)
Non-functional storybook for this or related components
Sensitve files changed eg. atoms, or style tokens
Relative imports
Default exports

Add your PR description below
Link to the design doc
A link to the component in the deployment preview
Testing instructions
ACs

Theres no detail in this pr?

Theres no detail in this pr?

Theres no code.

Theres no detail in this pr?

Only the final commit is used to decide whether to release and in the previous PR it was docs . So no release was made.


Clean up after test run? / If tests pass, and are immediately rerun, should pass again.

The tests also create an uploads directory in the root higlass-server directory. Can this be placed in /tmp and removed when the tests finished?
Same for db_test.sqlite3.

I am not sure about this. I think we want to be sure tests can be run repeatedly, and if the test fails it can be good to hold on to artifacts from the past run... I guess I don't see cleaning up from a successful run as so important: It's more important to be able to rerun reliably after failed tests.
I might be looking at this the wrong way.

@pkerpedjiev : This is the issue you noted today. There are several nice-to-haves, but we can't get them all at the same time:

(a) Leave the instance up at the end of every test run, to help with follow-up debugging.
(b) Do not automatically wipe state when tests start: that could be dangerous.
(c) Let the names and ports used be stable, so we don't need to look them up each time.

For higlass-docker, we have a+b, but the ports change, and we accumulate instances: It has worked for me, but might not be for the best going forward. Here, currently, we have something like a+b+c, so reruns start with a dirty state and fail.
Is there an obvious right answer?

In my opinion, the right answer would be 'none of the above'. The tests should start from a clean state and finish with a clean slate. At least that should be the goal.
I just ran into an issue that illustrates why this is important:

Changed higlass_server/settings.py to accept an extra environment variable
Want to run ./test.sh but it has already start python manage.py --port 6000 so I need to manually kill it and restart is so that it picks up the environment variables.

So ideally, the tests should create an instance, run the tests, bring down the instance and remove the database. The goal being that each time the tests are run, they can be run independently and repeatedly.

I should add that python manage.py test xxx automatically creates a test database which it automatically removes at the end of the test run so maybe there's some way to use that rather than starting one externally.

That all makes sense. The tests started as something bolted on after the fact, assuming that something else does the provisioning, and then testing that it works. A part of that needs to be inverted. Thank you for pressing this.


.Net 3.5

Is it possible to use it with .NET 3.5?
If not, do I need change a lot?

No.
You need to convert your app


Adds capitalize option to word and syllable

Allow capitalize to be passed as an option to word or syllable methods.

Looks great, thanks!


Fix Windows CI

@rgieseke can you make appveyor point at this new url?

I'd suggest you create a new account as openscm or we do the tests using GitHub Actions which also supports Windows!

Beauty will do!

Which option?

github actions

Almost fixed by #293. Remaining problem is getting the notebooks passing on windows. Any interest @rgieseke or @lewisjared ?

If that's more of a problem of running/testing notebooks on Windows under CI i probably wouldn't spend too much time on it ... If there are changes in notebook output they are hopefully already caught by the Linux tests ...

I had one failing in #293 and obviously don't have a windows machine so couldn't easily work out what was going on. In an ideal world we'd run both but not a problem I can easily solve right now.

There are trade-offs with testing everything everywhere ... time to finish CI, energy usage, what do we want to check with testing notebooks? They shouldn't get out-of-sync with the code is probably most important. If there were a mis-match with Windows output only, then this should be covered in regular tests i think.

Beauty, will close then and we can open again in future if needed


added repo support

fixed path

You need to update previous commit to have only one commit with your changes


Just Watch support?

Would there be any interest in integrating this into Just Watch? I frequently use that site to determine if I should request a movie or not and this seems like the perfect integration for that.

Hi @milogert, thank you for your suggestion ! I've added support for JustWatch in version v1.4.2 that should be available in the next few hours. Let me know if it suits your needs !

Perfect! That's great. Thanks for the addition

I don't see any button in Just Watch single movies? Random example.

I don't see any button in Just Watch single movies? Random example.

I do see it in the page you linked:

Are you sure that the extension is enabled ? If yes, do you anything in the browser console ?


[3.x] Make several actions in the Inspector dock more obvious

Backport of https://github.com/godotengine/godot/pull/48742.

Thanks!


Infinity loading of edit page with a lot of has many associations

Is there a way to split edit fields into two ajax tabs? I know about grouping, but it renders all fields on the page anyway.
As I understand, problem comes from _form_nested_many.html.haml file (it take a lot of time for render)

Are you using deeply nesting accepts_nested_attributes_fors?
If so, this might help:
https://github.com/sferik/rails_admin/wiki/Associations-basics#nested-form


Search Provider : Yelp Search

Implemented Yelp Search Provider
Related Issue(s)

Associated issue https://github.com/swirlai/swirl-search/issues/917
Type of Change

[ ] Bug fix or other non-breaking change that addresses an issue
[x] New Feature / Enhancement (non-breaking change that add or improves functionality)
[ ] New Feature (breaking change that is not backwards compatible and/or alters current functionality)
[ ] Documentation (change to product documentation or README.md only)

And hello again, @Harshil0512 ! Thansk for this one as well...similar changes if you could please?

Retarget this file to land in SearchProviders/untested on the develop branch
Could you add some additional result_mappings?  We'd like to see at least mappings for Swirl's supported fields (title, body, author, date_published, and url) plus
Also, beware of the spaces in your result_mappings line
Do you need the LenLimitingResultProcessor based on the size of the response body from the source?

Much appreciated!

Hello @erikspears, Thank you for your response. I've performed the changes can you please have a look into it? I've removed LenLimitingResultProcessor as I'm limiting the response by query string only

Hello @erikspears ,
Thank you for your response,
I did the mappings you've asked can you please validate it
Thanks again


Dropping files into Kap results in unexpected behavior

I'm new to using Kap, and I thought that maybe dragging an already exported GIF back into Kap would allow me to edit it (size, FPS, etc.). Looks like that's not possible, but it also does some strange things...

It just pops the image into the Kap window and then there's no way to quit/reset Kap without going into Activity Monitor and forcing the process to quit. Right-clicking on the menubar icon no longer displays a menu.
Just to see what else happens... 😈

Dragging a zip file into Kap prompts you to save the zip file
Dragging a folder into Kap results in the menubar icon not displaying anything when clicked

Not many people should come across this issue, but perhaps it should be prevented from breaking the app with no way to reset it.
Great app though 😀

Oops, ha. 😅

I'm new to using Kap, and I though that maybe dragging an already exported GIF back into Kap would allow me to edit it (size, FPS, etc.).

That's an excellent idea though!

@skllcrn @scferg https://github.com/wulkano/kap/pull/100 fixes this. We might later revisit with the "drop to edit" idea! 👍


Couldn't update the settings. Check your connection

After I choose the "device" and login there's a progressdialog with "Linking your ... account".. then I got an error: "Couldn't update the settings. Check your connection",why?

Can you try a few times and see if it works after another check?
On Thu, Aug 17, 2017, 11:26 PM pantongren<EMAIL_ADDRESS>wrote:

After I choose the "device" and login there's a progressdialog with
"Linking your ... account".. then I got an error: "Couldn't update the
settings. Check your connection",why?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/actions-on-google/actionssdk-smart-home-nodejs/issues/26,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADI58749xoDY9WWuoS-K3sImxDXcYun5ks5sZQRbgaJpZM4O7Drv
.

@Fleker I have tried many times,
I was in accordance with the README step by step .When I  copy the url "https://xxxxx..ngrok.io/" ,open it in a browser and clicked on the submit button to login and then it jumped to a blank page, nothing at all. Next,I try it in Google Home App ,and it shows"Couldn't update the settings. Check your connection".

Is your local server getting the authentication request and the SYNC event?
On Thu, Aug 17, 2017, 11:54 PM pantongren<EMAIL_ADDRESS>wrote:

@Fleker https://github.com/fleker I have tried many times,
I was in accordance with the README step by step .When I copy the url "
https://xxxxx..ngrok.io/" ,open it in a browser and clicked on the submit
button to login and then it jumped to a blank page, nothing at all. Next,I
try it in Google Home App ,and it shows"Couldn't update the settings. Check
your connection".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/actions-on-google/actionssdk-smart-home-nodejs/issues/26#issuecomment-323252727,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADI589o9k-UW_VA48iV7CzgG-7ktP9Orks5sZQrigaJpZM4O7Drv
.

@Fleker ,yes,here is my log:
getUser rick
logging in  { uid: '1234',
name: 'rick',
password: 'oldman',
tokens: [ 'psokmCxKjfhk7qHLeYd1' ] }
login successful  rick
authCode successful  cuvp84qffp4w84s04cs8w0o8o
POST /login 302 1.865 ms - 1478
/token query {}
/token body { grant_type: 'authorization_code',
code: 'cuvp84qffp4w84s04cs8w0o8o',
redirect_uri: 'https://oauth-redirect.googleusercontent.com/r/smartliving-8f0d
9',
client_id: 'ZxjqWpsYj3',
client_secret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
getClient ZxjqWpsYj3, hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv
return getClient { clientId: 'ZxjqWpsYj3',
clientSecret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
client { clientId: 'ZxjqWpsYj3',
clientSecret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
handleAuthCode {}
getClient ZxjqWpsYj3, hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv
return getClient { clientId: 'ZxjqWpsYj3',
clientSecret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
getAccessToken =  { uid: '1234',
accessToken: 'psokmCxKjfhk7qHLeYd1',
refreshToken: 'psokmCxKjfhk7qHLeYd1',
userId: '1234' }
return getAccessToken =  { token_type: 'bearer',
access_token: 'psokmCxKjfhk7qHLeYd1',
refresh_token: 'psokmCxKjfhk7qHLeYd1' }
respond success { token_type: 'bearer',
access_token: 'psokmCxKjfhk7qHLeYd1',
refresh_token: 'psokmCxKjfhk7qHLeYd1' }
POST /token 200 5.177 ms - 100
post /ha { 'content-type': 'application/json',
'google-assistant-api-version': 'v1',
authorization: 'Bearer psokmCxKjfhk7qHLeYd1',
host: '6fe99024.ngrok.io',
'content-length': '79',
'user-agent': 'Mozilla/5.0 (compatible; Google-Cloud-Functions/2.1; +http://ww
w.google.com/bot.html)',
'accept-encoding': 'gzip,deflate,br',
'x-forwarded-proto': 'https',
'x-forwarded-for': '<IP_ADDRESS>' }
post /ha { inputs: [ { intent: 'action.devices.SYNC' } ],
requestId: '6345068453055187986' }
post /ha SYNC
query { uid: '1234',
auth: 'psokmCxKjfhk7qHLeYd1',
requestId: '6345068453055187986' }
POST /ha 200 1.806 ms - 146

@Fleker   What's wrong with my log, sir?

Good evening, I've been out of office.
Your logs suggest no issue. Your server is getting all the events. Your
phone and the home app dont appear to be getting the response, either due
to a problem in the app or some sort of timeout in the app. I'm unsure of a
good way to fix at the moment. I'm going to raise the question internally
in the morning.
Have a good night!

@Fleker +1 for me. I also get this error, after trying to link the account and it seems the browser cant recall the app.

@marziman Have you solved this problem?

@Fleker thank you sir, waiting for  your good news.

+1 I'm having the exact same issue.

@marziman Hi,
when I'm using Google Home App on iOS, I'm able to display login form within my system when using Home Control -> Add devices.
Then after successful login I get "Something went wrong. Try Again"
No SYNC call received in API.
Do you have the same problem?

@pantongren , @mwidlok ,
I get the error like the first comment: "Couldn't update the settings. Check your connection"
and the linking wont happen, so there is no SYNC sent.
So I think this is a Google platform issue and must be either fixed in app or platform.
I am waiting for feedback from @Fleker . I am sure he is busy checking that issue.
So lets wait for his feedback.
BR

@marziman I solved this issue yesterday, we have to give a correct-format value to the constant "Data" in the datastore.js file, The original value is empty. If you run the program before you assign a vaule to the constant, the program will reports an exception and then you will get the message"Couldn't update the settings.  Check your connection" on your phone.

@Fleker Any updates on this issue?
Our client are going to Berlin IFA show next week, to present their Smart Home system. He wants to show Google Home assistant features but at the moment this is impossible.

@pantongren,
not sure if I understand it correctly. I use a real smarthome backend with oAuth capabilities and not the mock implementation here with datastore.js.
Can you explain how you fixed it in detail? Or did you just found a quick fix for your problem..

@marziman  I  didn't use a real smarthome backend with oAuth capabilities. So, I'm not sure whether it is authentication or SYNC leads to your “Couldn't update the settings. Check your connection”，You can check your server output logs after you click on the login button. if your server get the request of SYNC, I think it is the problem with your response data。The reason I failed to link my account most likely was that my server did not reply the correct-format  data  to the phone after SYNC request.  The correct-format response data like here:  {
"id": "g:3",
"type": "action.devices.types.SWITCH",
"traits": [
"action.devices.traits.OnOff"
],
"name": {
"defaultNames": [
"HAM Group"
],
"name": "Park Lights"
},
"willReportState": false,
"deviceInfo": {
"manufacturer": "HAM Systems",
"model": "HAM Group",
"hwVersion": "1.0"
}
},

@Fleker I also have this issue.I try to add devices through the android phone, most of the cases have failed, only a few success. From the log analysis, my server did not receive the sync call. I can be sure that my server is working well and I try to use the postman to sending sync, query and execute requests that all get the right response.
Here is my log:
login successful  rick
authCode successful  81bvf0thxsc000000000000000
GET /oauth?response_type=code&client_id=ZxjqWpsYj3&redirect_uri=https://oauth-re
direct.googleusercontent.com/r/test-smart-home&scope=email&state=CrwCQUZEXzV0bTR
TR09NQXhXcWtTc185OW9pNGFKRXpjX2xmRHVubzdBLTJ4dXJhdXVRUWs0MkVSMGVuLTd4Y181dmpDWll
sdG9MZmVVVjd0cm1lamtaY1BId2lrd3YwSWQ0X3Y0Yk4yNmpCVFJDNWJ3Y1B6V0ViLXJTYUo5dTY4NVN
YR0JJc09oMHB3ZlFuVWI4M2VhMHUyMXQxY3NfY2Y3Skx6b3VIWS1Nb3pEVXRzcG5JQkx4YVVFSnJZNWt
FQXFkY0VVX20tdlM1YnBVR1YyS01Xbk45UzdxM044MDdTRXpqTkV2TXJJdzV2VlNkTDRvZEtRN2s4YVN
iVWxEcGVyOVBoQU4tREpJRlFaTUVXWFR0UmdYSXNPWWhfYU1WQno3ZHBBSjQ3bjU2WjRzTnJpd3otaFF
KcE1ScnlfVBIcYXNzaXN0YW50LWhhbmRvZmY6Ly9jb21wbGV0ZSI-aHR0cHM6Ly9vYXV0aC1yZWRpcmV
jdC5nb29nbGV1c2VyY29udGVudC5jb20vci90ZXN0LXNtYXJ0LWhvbWUqFXJpbmVoYWlkdm9uQGdtYWl
sLmNvbTITdGVzdC1zbWFydC1ob21lX2Rldg 302 5613.128 ms - 1474
/token query {}
/token body { grant_type: 'authorization_code',
code: '81bvf0thxsc000000000000000',
redirect_uri: 'https://oauth-redirect.googleusercontent.com/r/test-smart-home'
,
client_id: 'ZxjqWpsYj3',
client_secret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
getClient ZxjqWpsYj3, hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv
return getClient { clientId: 'ZxjqWpsYj3',
clientSecret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
client { clientId: 'ZxjqWpsYj3',
clientSecret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
handleAuthCode {}
getClient ZxjqWpsYj3, hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv
return getClient { clientId: 'ZxjqWpsYj3',
clientSecret: 'hIMH3uWlMVrqa7FAbKLBoNUMCyLCtv' }
getAccessToken =  { uid: '1234',
accessToken: 'psokmCxKjfhk7qHLeYd1',
refreshToken: 'psokmCxKjfhk7qHLeYd1',
userId: '1234' }
return getAccessToken =  { token_type: 'bearer',
access_token: 'psokmCxKjfhk7qHLeYd1',
refresh_token: 'psokmCxKjfhk7qHLeYd1' }
respond success { token_type: 'bearer',
access_token: 'psokmCxKjfhk7qHLeYd1',
refresh_token: 'psokmCxKjfhk7qHLeYd1' }
POST /token 200 26.913 ms - 100

@rinehaidvon can you post or email me your project-id and the account you used to create this project?

@Fleker
i have already reply you, please check your email, thank you.

Following the tutorial, it lead us to the same issue than @rinehaidvon, Oauth calls seem to be ok, no call to SYNC... Also the same "Couldn't update the setting. Check your connection" message in the mobile app after sign in.
Also tried to swith to our corporate sign in pages: everything seemed to be ok but led to the same result...
Don't known if it's relevant: from the mobile, just after sign in, I can see a "[Upload] response code: 500" in the mobile logs using logcat...

@BenjaminBertran that may actually be pretty relevant. Can you obtain a more in-depth bug report for your device, then email me the zip? Are there any other logs in the app that indicate what stage the error is occurring?

@Fleker is there any update about this issue?

Sorry, the issue seems to be affecting a select number of accounts but we
don't have an update right now.
On Sun, Aug 27, 2017, 7:46 PM rinehaidvon<EMAIL_ADDRESS>wrote:

@Fleker https://github.com/fleker is there any update about this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/actions-on-google/actionssdk-smart-home-nodejs/issues/26#issuecomment-325247624,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADI5818S_yLeCpwogv3JZ2AWJFoT2ygWks5scioSgaJpZM4O7Drv
.

If you check the ADB logs when you link your account, do you see a 500 error when you see the sync failure in the app and a stacktrace?
W/S3NetworkUtils: [Upload] response code: 500

I'm using iOS app. What is the way to debug it?
Pozdrawiam,
Maciej Widłok.
2017-08-28 19:59 GMT+02:00 Nick<EMAIL_ADDRESS>
If you check the ADB logs when you link your account, do you see a 500
error when you see the sync failure in the app and a stacktrace?
W/S3NetworkUtils: [Upload] response code: 500
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/actions-on-google/actionssdk-smart-home-nodejs/issues/26#issuecomment-325428661,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADB8YVKCFcB60nI6cwc5YmZuR-xDGN5nks5scwAWgaJpZM4O7Drv
.

I don't know how to get logs from iOS. 🤔
This might only be available to check on Android right now.

@Fleker Could you be so kind and tell does submitting app to review and then publishing, will allow to test the app without any problems? My customer is going to IFA Exhibition in Berlin this week and wanted to show Google Home assistant features working with their smart home system.

I can't guarantee that it would be working if it's published, if it passes review. We can discuss over email some potential workarounds for your deadline.

@Fleker Thx, right now I'm trying to setup our google home api in production env. Tomorrow when all will be set will email you to discuss workarounds.

@AlexSneedMiller on Android? Can you grab a bug report after doing this in both cases?

Having the exact same issuse with the excat same timeline. It does not seem to be a linking issuse as I got from the logcat and also from the oauth playground everything with my oauth implementation (that works also for other clients) works fine.
Nevertheless I got no sync request and real devices and the simulator keeps stating that "The settings cannot be updated. Please check your network connection". I use the exact same user (owner) for the setup as for the test.
I posted my problem on stackoverflow, google plus and opened a support ticket. Hope that this one is more promising.
Please find below my complete logcat output.
log.txt

I have the exact same problem with the exact same timeline. It does not seem to be a oAuth problem. The oAuth implementation is working for other smart home assistents and also for the google oAuth playground. I grepped from logcat the following lines:

08-28 21:27:38.114: I/ActivityManager(1496): START u0 {dat=assistant-handoff://complete/?result_code=SUCCESS&result_message=Accounts+now+linked. cmp=com.google.android.googlequicksearchbox/com.google.android.apps.gsa.assistant.handoff.BrowserControlActivity} from uid 10030

Which basically indicates that the oAuth is succesfully completed and the google assistent has to do the next step.
I posted that problem on stackoverflow, google plus and opend also a support ticket. No reponses except other people stating that they have exactly the same problem.
I added the complete logcat output. Any help is highly appreciated!!
log.txt

@dbrammertz  Have you implemented Implicit flow or Authorization code flow? We had problems with first one, but Authorization code works as it should.

We are using the Authorization Code flow. Works perfectly on https://developers.google.com/oauthplayground/, but fails for google action. Unfortunately there is no hint what is wrong. On our server I can see that a valid access_code is generated:

2017-08-28 21:27:38  INFO  [  /  ] LoggingFilter:132 -Request: request id=173; method=POST; content type=application/x-www-form-urlencoded; uri=/dcloud-main-webapp/oauth/token; payload=
2017-08-28 21:27:38 INFO  [  /  ] LoggingFilter:149 -Response: request id=173; payload={"access_token":"af3df1bb-0a8a-4533-8dfa-af6694dedfe0","token_type":"bearer","refresh_token":"692ee03c-475c-4c5b-a606-fb847be9386a","expires_in":1199,"scope":"auth"}

On the mobile device I can see via logcat that the access_code was accepted:

08-28 21:27:38.114: I/ActivityManager(1496): START u0 {dat=assistant-handoff://complete/?result_code=SUCCESS&result_message=Accounts+now+linked. cmp=com.google.android.googlequicksearchbox/com.google.android.apps.gsa.assistant.handoff.BrowserControlActivity} from uid 10030

But still the account linking is failing on the device:

Same for me, there must be an issue with the app. I always get this message as reported before.

Yes it does not appear to be an issue with auth. Something unusual seems to happen that prevents the SYNC from being sent to your smart home app. Still investigating.

@Fleker ,I tried "turn on light" on web simulator and it
correctly responded. But  when I closed and reopened the web simulator to test it ,it didn't work, just replied"I'm sorry, there was an error and I'm unable to control your home device", and it would continue this reply unless I unlink and relink my Accont. why? Is there anything expired after I close the web simulator?

@pantongren it may be an OAuth issue that happens when you start a new session.

@Fleker  thanks,
Is there any solution?

@pantongren not sure what a good solution may be, but this sounds like something you should put into a separate issue. Are you only using the web simulator? How are you doing OAuth? Detail your answers in that new thread.

@Fleker   I tried to reset my Google Home Device a few days ago,but  failed to set it up during the last step of configuration. So, I can only use the web simulator or Allo App. I  have tried "turn on light" with Allo App, it replied "I can't quite control that yet". So, it works only by the web simulator.  I'm just testing this given sample which contains the whole OAuth process. I have done no modifacation with the sample OAuth,thanks.

@pantongren please place further discussion of your issue in #22 as it's unrelated to this.

my smart home  have the same issue   after token exchange. When trying to add devices with my test application,
I do get the error message that the "Cloudn't update setting. Check your connection".I tried many times.why?who can help me.

my smart home have the same issue after token exchange. When trying to add devices with my test application,
I do get the error message that the "Cloudn't update setting. Check your connection".I tried many times.why?who can help me. @pantongren @Fleker @marziman @bartvollebregt @mwidlok

@xj13567180468 you can email your project id and email to me.

We just pushed out a server-side update to the Smart Home servers. I have successfully SYNC'd my devices with a test account that was affected. Can others check and confirm that they are unblocked?

we had email to you project id.

@Fleker we are unable to SYNC test devices

@Fleker It now works for me!! Thx

I am closing this issue since it appears like the server fix has unblocked developers from getting a SYNC command through the Home app, allowing them to correctly execute commands. Thank you for your patience, and I'm excited to see what you make.

We just try to Google application tried to link accounts and SYNC however, the problem is still there and Route for SYNC didn't get triggered.

the problem is still there and Route for SYNC didn't get triggered.we still had this problem。

@Fleker it worked for me and I was able to SYNC and EXEC.
Thanks to you and the team..I am sure you guys had alot stress with this topic.
BR Mehmet

i  still have the  smae error ,why?

dear @marziman ,how did you resolve it, i am very Ardent desire your advice.

my project id is  dev-xingtel

authorization code type author have got it,thanks, all my accounts is ok.

I have my own OAuth2 server which worked for Alexa, but I still have this issue with Google, what is the solution exactly?  are we saying it only works for google's sample OAuth2 implementation?

authorization code type author have got it,is your implicit autho type?

Hi - just wanted to say this isn't working for me either - can see 11-06 19:07:38.171  7315 11191 I ActivityManager: START u0 {dat=assistant-handoff://complete/?result_code=SUCCESS&result_message=Accounts+now+linked. cmp=com.google.android.googlequicksearchbox/com.google.android.apps.gsa.assistant.handoff.BrowserControlActivity} from uid 10024 pid 1184 on display 0 - but seeing same behaviour as @dbrammertz

Let's use issue #58 to continue discussing the issue

Hi
Its not working for me either, looks like it is not able to sync to /smarthome
keep getting error "POST /smarthome 500 9.289 ms - 1456"
I did enter https://xxxxxxxx.ngrok.io/smarthome to webhook URL in "Smart Home Action" panel
And on google assistant app it links succesfully but throws an error "Something went wrong. Try again"
Attached is log file
gaerror.txt
Just wanted to confirm :- We have to use same client id and secret id as mentioned in config-provider.js? and only change the API key correct?


Implement fuzzy search tests

We need test for fuzzy search ! :)

Implemented first tests

test tests against implementation

updated 9428140b1ae0e7f8bc5cf0a8308b172543c5aa62


Coerce + sexpr roundtrip of regex pattern fails

Repro:
(rewrite-clj.node/sexpr (rewrite-clj.node/coerce "\\s+"))
==> ExceptionInfo Unsupported escape character: \s.  clojure.core/ex-info (core.clj:4617)

Imo this should return the exact same string that went into coerce.
PR upcoming.

This behaviour comes from #176.
But maybe #176 wasn't the right choice. I'll drink more coffee and refresh may brain.

I'm poking around and looking at what rewrite-clj v0 did vs what v1 does.
I've increased to scope to look at string coercion where strings contain newlines.
v0 strings
In v0 strings were parsed to StringNodes but always coerced to TokenNodes.
The :multi-line aspect of StringNode refers to strings in source code appearing over multiple lines.
Strings like this:
"foo
  bar
   baz"

(both v0 and v1 normalize newline variants to LF for such strings, but we'll not cover that issue here).
v1 strings
In v1, I must have thought coercing strings to TokenNodes was odd, so I coerced them to StringNodes.
My changes:

considered a coerced "foo\n\bar\nbaz" to be :multi-line, this is plainly incorrect.
the StringNode assumes strings are from the parser, which escapes strings while reading them.
My coercion did not escape. This was partly fixed in #176.

Option 1: Coerce strings to TokenNodes
This restores v0 behaviour and might be appropriate.
The question of why we aren't coercing strings to StringNodes might confuse future devs/me, but perhaps a well-placed comment would help here.
Option 2: Coerce strings to existing StringNode appropriately
The StringNode is expecting escaped strings so conform to this.
This would include #217 but:

perhaps instead use Clojure's char-escape-string with string/escape instead of my invention.
and also address the inappropriate :multi-line-ing (i.e. don't do that), coerced strings will never convert to :multi-line StringNodes.

Option 3: Coerce strings to a modified StringNode
It is basically saying, ok StringNode, for coerce support we don't need this escaping stuff, so I'll just have you save the raw string too.
This is pretty much the strategy TokenNode takes.
This is #214.
But this PR does not yet handle our '"hey " man"' coercion test.
Proposal
I'm leaning toward option 1.
Both StringNode and TokenNode return a tag of :token, so we aren't breaking any public interface.

Note we started with Option 1, but @borkdude discovered usability issues with this approach in his inlined version of rewrite-clj in clj-kondo. See https://github.com/clj-kondo/clj-kondo/issues/2026 for the symptom.
So we landed on Option 2, but with a better implementation than proposed above.
Instead of mimicking the parser we reuse it.


Script to acknowledge "I moved it"

When a VM is removed from one VM and then registered in VCenter 6.7, there's a prompt when starting asking whether the VM was copied or moved -- is there a pyvmomi way of acknowledging that in advance? (That is, something that can be executed after it's been registered but before attempting to start it?)

https://github.com/vmware/pyvmomi-community-samples/blob/master/samples/virtual_machine_power_cycle_and_question.py


Enable discussions on this repo to reduce clutter in issues?

A lot of "issues" here would be more fitting in "Q&A" questions in github discussions.
Would it be possible to enable discussions here?

@pke just enabled, thanks for the suggestion!

Thanks @whitedogg13!


Universal Format Selection As Language

I have several scenarios where I have multiple language types in a single file such as plain text or markdown where I would like to format just a selected code block. For example, I might be in markdown and want to just change the code block section using the json or xml formatter. Today, I have to change the entire language to that language of json or xml, use the format select and switch it back to markdown. I would love to be able to tell it to format selection as a specific language format.

VSCode Version: 1.9.1
OS Version: OS X El Capitan

Steps to Reproduce:

Create a markdown file.
Create a code section using

{"languages":["javascript","java", "sql", "xml"]}

Select the line just containing the json code and try to format it.

The markdown extension has to either embed a JSON formatter or call the JSON extension with a virtual document.

This is related to #639 and seems like a duplicate of #14447. We don't have a good story for managing mixed language files and I don't think that the markdown extension should be in the business of handling json formatting specifically. @aeschli Has there been any further discussion of supporting this type of mixed language scenario?

My comment https://github.com/Microsoft/vscode/issues/14447#issuecomment-262736975 is still my thinking after my experience on HTML.

I don't feel like my request is the same as #14447. I am not asking to format the whole file and determine and format different languages appropriately. I know that is a difficult problem. I am just asking if a manual selection is made that a user can format just the selection as the format they want. A particular use case, if an author is writing a programming book in markdown and they have code examples in Java, JavaScript, HTML, etc they can manually format as the desired selection rather than changing the entire file format to format the selection and switch back to markdown.

closing as dupe of #27073


Reduce default values of the quest file

To reduce the quest file size, omit default values.

Renamed NBTReplaceUtil -> NBTUtil
Added "reduce" flag parameter to writeToNBT()s to specify whether to reduce or not. (Don't reduce on nbt editor.)
Added IPropertyReducible to omit default values.

I'll fix backward compatibility

Please fix the merge conflict


Mixer: Re-evaluate limit enforcement

Reported by @kd0aij. At thrust close to 1 the actuator outputs seem to dip to min.

Unable to reproduce the problem in manual attitude control mode:

commit 59ebc99,  branch limitAutoRPrates

I'm not surprised that you can't, because for the saturation to occur you would need to set rates instantaneously, which is is almost impossible by just twiddling an RC stick.

Just rechecked the logs: same mode in both, manual attitude control

@kd0aij Can you share a link to the log.

first log: https://copy.com/dZ6jHciEUuhOULZo
second log: https://copy.com/ekAYIlWe29NdGPpN

BTW, I've accidentally caused integer overflows in MatrixPilot, which is mostly implemented on DSP hardware; results are usually quite exciting for a very short time :)

@tumbili The proposed approach is along the lines of this algorithm. You need to implement it partially at least in the controller to lock integrals from building up.
CASE A: Slew rate limiting: Last attitude setpoint and current attitude setpoint differ by more than e.g. 30 degrees OR the requested angular rate is relatively high AND the mixer reports hitting a limit:

Limit demanded angular rate to a fixed threshold in controller (e.g. 200 dps)
Limit demanded attitude to an attitude max. 15 deg or so from current attitude
Lock all integral gains

This forces the system into a slew rate limiting mode and ensures that higher-level controllers do not build up integrals.
CASE B: Flip-prevention: Current attitude and setpoint differ by e.g. more than 20 degrees (NOTE: The limit in degrees here has to be higher than in slew rate limit mode so that slew rate limiting always triggers first). This mode might loose or gain altitude.

If hitting high limit: Reduce throttle to 60-80% of the demanded value to allow higher spread
If hitting low limit: Increase throttle to 120-160% of the current value

This should allow the system to recover if it was close to flipping.

Fixed in master.


Add support for multilabel classification

multilabel classification adds two new fields: predictions and confidences. Add support for these fields.
mutlilabel classification doesn't return prediction and confidence fields. Mark these fields as optional.
prediction and confidence fields will eventually be removed. Add "to be deprecated" comments.

Important note: this PR does not introduce any breaking change.

@sanderland I implemented all your requested changes 👍. For the tests it is not possible to use a real co.classify() call as it would require to finetune a model first (which takes ~30min). Instead, I mocked the _request() call and tried all the possible format outputs.


Update Antagonists Guidebook Sections

Flesh out the Rat King and Space Dragon subsections of Minor Antagonists section in the guidebook. Rewrite some of the rest of the Antagonist entries in the guidebook, mostly to fix some minor grammatical mistakes from the original entries and to bring content up to date with PRs changing the function of the Antags since the original entries.

About the PR

I rewrote and expanded the guidebook entries for Rat King and Space Dragon. I plan to go through the guidebook and improve entries I see as lacking important information and detail as a series of PRs during the freeze. I'm open to feedback to improve my writing style, add information I've missed, or make things more clear and concise.
EDIT: I have now touched up most of the other antag entries as well, mostly to fix some minor mistakes and to bring the Guidebook up to speed with the current state of the game.
Why / Balance

The sections on the Rat King and Space Dragon were a bit lackluster. Especially the Space Dragon was missing any mention of the recently added Fire Breath attack (https://github.com/space-wizards/space-station-14/pull/26746) and important information concerning Carp Rifts, especially the fact that the Space Dragon simply disappears if five minutes pass since the last carp rift (unless you have all three). "I finally get to play this cool ghost role! Wait.. where'd they go?" Isn't fun and should merit at least a mention in the guidebook.

Media

Rat King and Space Dragon

Nuclear Operatives

Revolutionaries

Space Ninja
(Note the two instances of 'Threat' are hyperlinks back to the MinorAntagonist entry, which contains both Revenant and Space Dragon.)

Traitors

Breaking changes

I wrote numerical values for the dragon's period to open a Carp Rift and similar values and didn't reference the DragonComponent file. If these numbers were tweaked and this guidebook entry not updated, the guidebook would become inaccurate. If there's a way to dynamically generate these numbers please let me know and I'll implement that otherwise these seems like an acceptable risk (other areas of the guidebook have become stale over time and needed to be updated, hence PRs such as this one.)

I think it would be good to include that the space dragon is immune to the fire from the dragon breath (or just flames in general? never actually tested it.)

I thought about this but was concerned about bloat (too much text = hard to read) and I was similarly unsure. I may add it in after I test to confirm Dragon is actually immune.

 (A little more than 10:1 Brute:Burn) I think I'll just leave the entry as is since you're not immune just very resistant to it and I don't know a way to convey that concisely in a way that's not super confusing.

Actually did the det lighter get removed as a steal objective? I was writing that section from the code-base on objectives but maybe that one was somewhere else... It was kinda easy tho sometimes if the det isn't there yet and the det room is mapped in maint or something you could basically just walk up, take it, and leave.

This is very nitpicky, but the nukie introduction audio that you have as text at the top of their page is clearly saying “Operatives at standby”, not “Autotell on standby”

This is very nitpicky, but the nukie introduction audio that you have as text at the top of their page is clearly saying “Operatives at standby”, not “Autotell on standby” Also, I’m pretty sure that steal objective was not removed.

Nitpicky is fine, trust me a lot of these changes are fixing missing commas and stuff. This was an error from the original entry and it got through because it wasn't formatting or grammar but I listened to the SFX (/Resources/Audio/Ambience/Antag/nukeops_start.ogg) and agree that's definitely Operatives and not Autotell On, so I've updated it for correctness.
Thank you for catching it for me :+1:

Oh no the wiki has the same typo ( https://wiki.spacestation14.io/wiki/Antagonist )

Also, I’m pretty sure that steal objective was not removed.
I've just looked into this, lighter was never a traitor objective it's one of the many things that Thief can want so there aren't objectives missing from the list of possible objectives for traitor in guidebook


Added option for static-IP configuration

Search was failing to discover WeMo devices on my network (IoT devices are on a separate subnet).  I've added the ability to specify an IP (and optional port) for a switch when configuring an accessory which works around the problem for me.

Great, thanks! I was having the same issue and this works perfectly. Hopefully this will get merged soon.

I was looking for this too, can this get merged?


Support cleaning mirrord-agent pods

Using mirrord 3.0.10-alpha
mirrord exec results in an orphan mirrord-agent pods with status NotReady
I think you should document this behavior, and add a way to remove the pods using mirrord CLI.

Hey @itaischwartz ! Thanks for opening the issue. This sounds like a bug. Did mirrord work alright?

It didn't
Maybe it was something on my side (We're using istio service-mesh)
I was using the following command:
mirrord exec --pod-name backend -n apps --impersonated-container-name
backend ./bin/server :80
בתאריך יום א׳, 2 באוק׳ 2022 ב-21:28 מאת Aviram Hassan <
@.***>:

Hey @itaischwartz https://github.com/itaischwartz ! Thanks for opening
the issue. This sounds like a bug. Did mirrord work alright?
—
Reply to this email directly, view it on GitHub
https://github.com/metalbear-co/mirrord/issues/482#issuecomment-1264705334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLEM6OPSRHOXVIAVD3MX7TWBHH6TANCNFSM6AAAAAAQ2ZY4JQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Opened relevant issue - https://github.com/metalbear-co/mirrord/issues/485
Regarding the cleaning of the pods - we shouldn't have any lingering pods after you're done using, so I don't think we want to add such feature (as it might imply to users that we mirrord persists longer than promised). Closing this and let's track this in the new issue.


Add JSON schemas for basic G-Cloud 9 content

These are generated from the basic G9 content in this pull-request: https://github.com/alphagov/digitalmarketplace-frameworks/pull/362
(Which should become content version 5.10.0 once merged).

LGTM 👍


Feature: Annotated prototypes

User need:
I need to understand the context of this prototype so that I take away from it what was intended.
In the future we'll want to use prototypes as examples of multi-page design patterns. It's going to really help if we can add notes to those examples. Some kind of dismissible annotation layer - notes column down the right hand side etc. We tried something similar before with the Exemplar Screenshots.

What kind of user is the I in this need?

Good question.
How about a designer looking at a complex design pattern - for example, a multi-page eligibility checker. We're trying to communicate some of the subtleties of branching and conditional flow, but we want to do it whilst they're clicking through the example prototype.
Possibly also a designer who wants to give some rationale to the design decisions in their prototype. Ideally they'd be able to do this verbally in person, but it's not always possible.
We also know that designers often copy stuff from other people's prototypes without understanding the context. We want people to share their work though - annotations gives us a way to do this.

Following review the Prototype Team are closing this ticket. If the original author wishes to raise this issue again please feel free.


Failed setup, will retry: object of type 'NoneType' has no len()

System Health details
System Information

version
core-2024.11.1

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.4

os_name
Linux

os_version
6.6.54-haos

arch
x86_64

timezone
Asia/Jerusalem

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

HACS Data
ok

GitHub API Calls Remaining
4993

Installed Version
2.0.1

Stage
running

Available Repositories
1471

Downloaded Repositories
63

AccuWeather

can_reach_server
ok

remaining_requests
32

Home Assistant Cloud

logged_in
true

subscription_expiration
December 1, 2024 at 02:00

relayer_connected
true

relayer_region
eu-central-1

remote_enabled
true

remote_connected
true

alexa_enabled
false

google_enabled
true

cloud_ice_servers_enabled
true

remote_server
eu-central-1-22.ui.nabu.casa

certificate_status
ready

instance_id
84707140cb5942839bb5e56ccf2a189a

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 13.2

update_channel
stable

supervisor_version
supervisor-2024.11.2

agent_version
1.6.0

docker_version
27.2.0

disk_total
30.8 GB

disk_used
23.4 GB

healthy
true

supported
true

host_connectivity
true

supervisor_connectivity
true

ntp_synchronized
true

virtualization
kvm

board
ova

supervisor_api
ok

version_api
ok

Dashboards

dashboards
4

resources
29

views
26

mode
storage

MiWifi

version
3.0.0

<IP_ADDRESS> (xiaomi.router.ra69)
ok

Recorder

oldest_recorder_run
November 10, 2024 at 19:03

current_recorder_run
November 12, 2024 at 19:24

estimated_db_size
171.94 MiB

database_engine
sqlite

database_version
3.45.3

Xiaomi Miot Auto

component_version
0.7.22

can_reach_server
ok

can_reach_spec
ok

logged_accounts
1

total_devices
19

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue

Reproduction steps
1.--------------
2.
3.
...
Debug logs
--------------

Diagnostics dump
No response

Please check for update and verify with latest version


add key test

I know this is merged now, but could you add some verification that this works well under the tests/ folder?

I know this is merged now, but could you add some verification that this works well under the tests/ folder?

:wave: @alrolorojas sorry yeah i'm still getting used to developing using pre-commit hooks. I'm actually still working on this but even though this is merged existing processes using these hooks won't get updated unless i post a new a release.
when i clean all this up today i'll make sure everything is tested and documented.


client crash

hey we found another bug with your mod!
client crashes when using  malissis door factory and block mixer when colored lights are installed
http://pastebin.com/bXzuNiMb

Fixed with https://github.com/CptSpaceToaster/CptsModdingLight/commit/3d465c34748daa145593ce5b5b6a5ffcecd4122a


GEODE-4098, add mutex timeout

GEODE-4098, add mutex timeout  to avoid stuck in client

I would also like to see a test for this change. Is there an issue you can reproduce in a test to exercise this change?

Also, the Travis build should succeed - it has a compilation error....

@pivotal-jbarrett Thanks, for review, passed now with the the changes
The issue( lock in client)  reproduced in geode 1.2.0 release ,  under high load from multiple clients, possibly related to buckets moving, rebalancing at the time, with  single-hop enabled for the relevant   region
gemfire::ReadGuard::ReadGuard(ACE_RW_Thread_Mutex&)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ReadWriteLock.hpp:33
gemfire::ClientMetadataService::isBucketMarkedForTimeout(char const*, int)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ClientMetadataService.cpp:1817
6) gemfire::ThinClientPoolDM::getConnectionFromQueueW(GfErrType*, std::set<gemfire::ServerLocation, std::lessgemfire::ServerLocation, std::allocatorgemfire::ServerLocation >&, bool, gemfire::TcrMessage&, signed char&, bool&, bool&, gemfire::SharedPtrgemfire::BucketServerLocation const&)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ThinClientPoolDM.hpp:247
7) gemfire::ThinClientPoolDM::sendSyncRequest(gemfire::TcrMessage&, gemfire::TcrMessage&, bool, bool, gemfire::SharedPtrgemfire::BucketServerLocation const&)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ThinClientPoolDM.cpp:1400
8) gemfire::ThinClientPoolDM::sendSyncRequest(gemfire::TcrMessage&, gemfire::TcrMessage&, bool, bool)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ThinClientPoolDM.cpp:1328
9) gemfire::ThinClientRegion::executeFunction(char const*, gemfire::SharedPtrgemfire::Serializable const&, gemfire::SharedPtrgemfire::CacheableVector, unsigned char, gemfire::SharedPtrgemfire::ResultCollector, int, unsigned int)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ThinClientRegion.cpp:2955
10) gemfire::ExecutionImpl::execute(char const*, bool, unsigned int, bool, bool, bool)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ExecutionImpl.cpp:226
11) gemfire::ExecutionImpl::execute(char const*, bool, unsigned int, bool, bool)/jphome/jp/ccjp/mb_ccjp/workspace/GEODE_Maven_Build/geode-client-native/src/cppcache/impl/ExecutionImpl.cpp:83
The issue do not seems to be reproducing with the fix

@igorbarc would you please create a test that your fix may be validated against as @pivotal-jbarrett requested, Thanks in advance EB

@igorbarc What is the expected behavior of these functions should they return without updating the member data? Under what circumstances is this mutex held indefinitely that making it timed is corrected? Strikes me that this fix is hiding the real issue.
Can you please also correct formatting.

I believe the GEODE-4051 (uncaught timeout exception) was possible root cause server side  of indefinitely held mutex
With above fix , we see  bucket not available timeouts responses  , instead of deadlocked threads in this flow
Will update formatting shortly

Fixed formatting with clang-format

@pivotal-jbarrett @mhansonp
Thanks for review. Made requested changes

@pivotal-jbarrett Thanks, made requested changes

Closing for no activity. Please reopen if you want to follow up on this.


Make it work with the CLI/deps.edn

Adds deps.edn file.
(! 700)-> clojure -X:test

Running tests in #{"test"}

Testing spock.logic-test

Ran 2 tests containing 6 assertions.
0 failures, 0 errors.

Ran 0 tests containing 0 assertions in 0 msecs
0 failures, 0 errors.
(! 701)-> clj
Clojure 1.10.3
user=> (require '[spock.logic :as spock])
nil
user=> (spock/solve nil '(= [_ _ n] [1 2]) {})
()
user=> ^D

Ok, I just changed the code to always treat keywords and symbols as the same, so the example code now should be:
(spock/solve nil '(= [:_ :_ :n] [1 2]) {})

Besides that, sounds good to me :)


Resolve symlinks when checking EC service versions

Fixes #127  - This PR introduces the following to ec instances:

Any symlink is now considered to have had changes if the target has had changes
A performance improvement (Roughly 2s down to 1s when using bl01c) by reducing calls to git. This is a nice reduction in latency which is good for autocompletion
svc_graph is renamed to version_map because its more representative

@gilesknap I did a re-run of the last working action for this repo which now fails. Does https://github.com/epics-containers/edge-containers-cli/actions/runs/8469131442/job/24112327403#step:7:541 make any sence to you?

@gilesknap I did a re-run of the last scuessful workflow run for this repo and it now fails. Does https://github.com/epics-containers/edge-containers-cli/actions/runs/8469131442/job/24112327403#step:7:541 make any sence to you? I believe this issue is making my tests fail (they pass if I run them locally)

Maybe - I got the tests to run anyway. I'm thinking that the globbing of files in the config folder has non-deterministic order.

This looks good. How much has it slowed down completion?

This looks good. How much has it slowed down completion?

As I said in the initial comment, this PR actually results in a performance improvement compared to what we have. The reason is because we were doing a git diff num service x num tags times. Now we only it num tags times. The simlinks dont seem to add much overhead.

ec instances
Time to do 10

Current
19.8s

This PR
12.2s

This PR + Skip symlinks
10.8s

*I use time for run in {1..10}; do ec instances bl01c-di-dcam-01; done

Although looking at it now, I think I just want to consider the edge cases a bit more. Also maybe its also a bit long as a single function

While you are thinking about that. What do you think about having some completion cache in a file and keeping it for a little longer? I find auto completion a little too sluggish.

While you are thinking about that. What do you think about having some completion cache in a file and keeping it for a little longer? I find auto completion a little too sluggish.
This will only get worse once we use these repos in anger and have many tags

We already do do this: https://github.com/epics-containers/edge-containers-cli/blob/7282c71f20f679490a6f3a907f603994fe5f410b/src/edge_containers_cli/globals.py#L33  I initially thought this was too short, but it also means that you wont be waiting for changes.
I believe the git clone has the largest overhead. If I do time for run in {1..10}; do git clone https://gitlab.diamond.ac.uk/controls/containers/beamline/bl01c.git; rm -rf bl01c; done - It takes 7.4 seconds. However time for run in {1..10}; do git pull; done takes only 1.8 seconds. So maybe we should check if there is already a repo present in temp and if so do a git pull rather then make sure we have the latest?

Maybe an interesting comparison is actually to run time for run in {1..10}; do ec --version; done as a baseline

Command
Time to do 10

ec instances Current
19.8s

ec instances This PR
12.2s

ec instances This PR + Skip symlinks
10.8s

ec --version
5.6s

which I believe stands to show that chasing further improvements will give diminishing returns versus the limitations of the project/framework as it currently stands

Yeah sure. Re the timings: where is your venv? Local or remote filesystem?


Update KRisIO -- skip list collection where applicable

Hi @ursjoss !
Thanks for the great tool that make my daily work much easier 😄
I have couple of ideas for small improvements and would like to introduce them to you in separate PRs.
Let me know what you think.
Changes:

do not collect flow to list on exporting, just write on every line emitted
readLines returns a list so for huge files we will be pushing a lot of objects to the heap which are not easily collected by GC

Hi @pawelpasterz
Thanks a lot for your PR and your interest and appreciation of KRis. Your PR looks good on first sight. I will try to find the time to give it some more thought in the next days, and will hopefully merge it soon and release a new version.
Cheers, Urs

Thanks a lot @pawelpasterz

@pawelpasterz Thanks for your contribution. I have released KRis-0.4.5. It should be available once maven finishes the synchronization.


Check expected log message when testing hello_lambda example

Currently, runningmake test-examples will:

Build the hello_lambda example
Deploy it into localstack
Send a test message to the lambda
Print out the log messages produced by the lambda

This is a convenient way to manually check whether the lambda is working as expected, but an automated test it does not quite make. Let's turn this into an actual automated test by reading the resulting log lines and asserting that they contain the expected value.
Exactly how we do this is a bit open-ended. We could use a bash or python helper, we could use jq to search for the expected message, or something else entirely.

@TanyaSrinidhi as discussed earlier today, I thought this would be a good onboarding bug. Take the time to checkout the repo and poke around, try running various targets from the Makefile, and generally get a feel for how we structure things. Key points include:

The make- and docker-compose-driven workflows, based on the 3-musketeers pattern
The use of localstack for local testing
And of course, the presence of a bunch of Rust code :-)


[AdvancedColorPicker] Added the component

@rmarinho I updated it. If the property is set to null, the the color values will all be 0 => black


Center image and reset pan when resetting the zoom level

Is there a way to center an image vertically and horizontally?
Also I think it would be good to reset the pan to center when resetting the zoom via zoom tool. The zoom should also prevent having a negative zoom, cause I honestly think there is no use case for zooming it out too much since the library already rescale the image to fit it in the view port.

there's not a way to center the image yet, but I think it's a good idea.


Error: spawn sqlite3 ENOENT when creating db

Hi,
I've installed dblite on my raspberry pi and when I'm trying to instantiate a db
var dblite = require('dblite');
var db_path = __dirname+'/db/db.sqlite';
db = dblite(db_path, '-header');

and run node app.js, I'm having this error
Error: spawn sqlite3 ENOENT
The db exists and is readable and the path is correct.
On OSX the error doesn't appear.
Thanks!
\m

do you have sqlite3 installed ?
$ which sqlite3
/usr/bin/sqlite3

if your output is different, please specify the right path for the executable as explained in the Readme.
example
var dblite = require('dblite');
dblite.bin = "/usr/local/bin/sqlite3";

var db_path = __dirname+'/db/db.sqlite';
var db = dblite(db_path, '-header');

no reply over almost a year.


Currently, when trying to run tests on Windows via the action, test_impl.exe exits with a 1 status code and nothing at all is logged to indicate why this might be. Yet this works without any issues on my own system. Oh well, no automated tests on Windows. (Or on MacOS, which also failed for bizarre reasons.)