Undocumented required configuration "API_EXTERNAL_URL"

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
It seems like using GoTrue with the minimal configuration is not possible. While performing e.g. a request to login a user the following message is logged:
{"component":"api","level":"warning","method":"POST","msg":"parse \"\": empty url","path":"/token","referer":"$REFERRER","remote_addr":"$IP","request_url":"","time":"2023-07-21T09:31:17Z","timestamp":"2023-07-21T09:31:17Z"} and the request returns with a http 500.
The reason for this behaviour is that API_EXTERNAL_URL is not set but seems to be necessary (although it is allowed to be empty according to internal/conf/configuration.go )
Furthermore the readme does not refer to this parameter at all or mark this parameter as required.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Clone the repository
Rename example.docker.env to .env.docker
Run docker-compose -f docker-compose-dev.yml up and wait for the application to start
Perform a request to POST http://localhost:9999/token?grant_type=password with any email and password as payload
See error

Expected behavior

There is no error, as API_EXTERNAL_URL is a required configuration (thus also it would need to be checked why this might be empty in tests...)
API_EXTERNAL_URL is referenced in the documentation
API_EXTERNAL_URL is part of all example configurations

System information

OS: Windows WSL2 (Ubuntu 22.04.2)
Version of gotrue: 2.84.0

On a similar note there are more things which I noticed while setting Gotrue up.

Additionally to the parameters which are described in the readme as required for E-Mail signup at least the parameter GOTRUE_URLPATHS_CONFIRMATION needs to be set. (Most likely also *_INVITE , *_RECOVERY and *_EMAIL_CHANGE)
The parameters mentioned in the point above currently default to "/" which does not lead to a working setup. For Gotrue as standalone application the default seems to be "/verify" which does not seem to be mentioned anywhere.
The parameters under the mailer configuration do not use the GOTRUE_* prefix in the readme. I didn't check whether this prefix is required here, but at least in the example files the prefix is set. IMO the consistency could be improved here to avoid errors in the future.

I guess these points depend on whether Gotrue is planned to be used as standalone application without the full supabase stack. If that's the case give me a sign and I might attempt to improve the points mentioned in this issue :)

Hey @szczepad, thanks for pointing out these issues! wrt to your question:

Gotrue is planned to be used as standalone application without the full supabase stack

Yes! gotrue can be used as a standalone application and you can self-host it without supabase. We'd welcome any help with improving the docs around this.

Hey @kangmingtay !
Thanks for the clarification. I've created a PR for the mentioned topics. Feel free to comment. :)
However I didn't add the GOTRUE_ prefix to the values in the documentation as those do not seem to be set explicitely as envvars in configuration.go and I'm not sure whether this prefix might break settings in this scenario as there seems to be some implicit stuff going on in the envconfig lib. (See https://github.com/kelseyhightower/envconfig/issues/148 )


[HOLD][$250] Migrate index.native.js to function component

Class Component Migration
Filenames

[ ] src/components/PDFView/index.native.js

type: React.Component
has state values: true
has refs: false
has context: false
lifecycle methods: componentDidUpdate

Task

We currently have some class components in our codebase that we would like to refactor to a function component.
Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
If you need additional guidance, please ask in #expensify-open-source
Test for any regressions and verify that there are no breaking changes

⚠ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

Heads up! The pricing for this issue has been adjusted based on the scope of the work and the fact that no complex bug investigations or proposal are required.

I'd love to work on this.

I have this queued to work on, not overdue.

I can work on this issue.

I'm able to work on this if needed!

Hey team! I'm working on this one internally.

Getting close to working on this one, just a few competing priorities to deal with first!

I am available to work on this.

Should have a PR in soon!

Had to switch my PRs due to a bit of a weird old commit, here's the PR https://github.com/Expensify/App/pull/24615

Added internal label to be clear that I'm working on it. Sorry @himanshuragi456!

That would be awesome thanks @0xmiroslav, I'll request your review when it's ready :)

I'm close to having the PR ready! Been doing some testing this week to ensure I understand what's happening in componentDidMount, will hopefully have the PR ready for review soon.

Same as above, have been OOO.

Having trouble with both native VMs for testing:
Android: https://expensify.slack.com/archives/C3BGN04K1/p1695353158552769
iPhone: https://expensify.slack.com/archives/C049HHMV9SM/p1694494427011429?thread_ts=1693989321.525899&cid=C049HHMV9SM

Been OOO, iPhone VM is fixed but android is still not working, trying to get testing videos.

Latest update here https://github.com/Expensify/App/pull/24615#issuecomment-1765591377

Latest update here https://github.com/Expensify/App/pull/24615#issuecomment-1782304251

Latest update here: https://github.com/Expensify/App/pull/24615#issuecomment-1797941582

Looks like this should be finished before the EOY

@Santhosh-Sellavel reassigning, please take over as C+. If you don't have bandwidth, unassign yourself. Thanks

This is sitting with @MonilBhavsar for final review, so it's possible no more c+ action is needed.


Video conversion and synchronizion issue

For the command:ffmpeg -y -i %s -qscale:v 2 -async 1 -r 25 %s
you set the fps to 25. But what if the original video's fps is not 25? Does that influence synchronization?

ffmpeg should resample the video to minimize the synchronization error.


Bug: easy to trigger a 500 when resetting project

Our reset project dialog makes it easy to double click buttons that trigger network requests.
E.g. "Reset" can take a while and if you click it again while you're waiting it triggers a 500. (and the trace doesn't have any useful info 😭)

Well, I can't figure out why the trace I first got in prod doesn't have any exception details on it:

While the one I generate locally does:


Support attribute references of positional/keyword arguments with string formatting

from loguru import logger

class Foo:
  bar = "baz"

foo = Foo()

# Both the following are valid syntax, but mypy flags them
logger.debug('The bar is "{0.bar}"', foo)  # note: Expected 0 but found 1 positional arguments for log message
logger.debug('The bar is "{my_foo.bar}"', my_foo=foo)  # error: my_foo.bar keyword argument is missing

❯ pip freeze | grep loguru
loguru==0.5.2
loguru-mypy==0.0.2

(IMHO this is more a bug since it's flagging valid syntax)

HI @ThibaultLemaire . I am deeply sorry but I don't think there will be much time for me to take a look. Actually I don't have time even to finish PR that is already here and another code sits in my desk. Dang, it was even 7 days already since reporting this issue.
If you have an idea how to fix that PR are welcome ;)

I'll take a look at the code and see what I can do. It should be the opportunity to see how a mypy plugin is made, maybe I'll learn a trick or two to enhance mypy checks (I'm thinking mainly of type-checking pymongo/motor queries, as that's something I've been meaning to do for a long time).
I've updated the description with loguru and loguru-mypy versions

Okay, so it looks like you've got some custom code there to support loguru's syntax, but mypy already has the capability to type .format(), like so:
class Foo:
    bar = "baz"

foo = Foo()

'The bar is "{0.bar}"'.format(foo)  # typechecks
'The bar is "{0.bar}"'.format("not foo")  # error: "str" has no attribute "bar"
'The bar is "{my_foo.bar}"'.format(my_foo=foo)  # typechecks
'The bar is "{my_foo.bar}"'.format(my_foo="not foo")  # error: "str" has no attribute "bar"

So I wonder if we could leverage mypy directly to typecheck something like
logger.debug('The bar is "{my_foo.bar}"', my_foo=foo)

as if it was written like this
logger.debug('The bar is "{my_foo.bar}"'.format(my_foo=foo))

Progress report:
After wiring up a debugger to mypy I was able to analyse the way it checks calls to .format. The call stack looks roughly like (Most recent call last):
mypy.checkexpr.ExpressionChecker.visit_call_expr
mypy.checkexpr.ExpressionChecker.check_str_format_call
mypy.checkstrformat.StringFormatterChecker.check_str_format_call

Where that last check_str_format_call is the one I'm really interested in, the one that should do all the type checking for us.
Now all I need is a way to call that method with all the necessary arguments as mypy is a very tightly coupled mess.
Hopefully though, the ctx.api that is passed to the plugin seems to have quite a lot of methods available, among which is visit_call_expr, so that looks like a promising line of investigation

Progress report:
So after further investigation, ctx.api is an instance of mypy.checker.TypeChecker that's why it has all those nice methods.
Now, I've been able to create a mypy.nodes.CallExpr that looks enough like that of a .format call (I think) and I was able to call ctx.api.check_str_format_call without error, but I don't get the corresponding mypy warnings...
I'll have to trace and debug mypy with the loguru plugin to see exactly what's going wrong.

There we go, I've got some sprout of working code, continuing technical discussion over at PR #43

Hey @kornicameister when do you think this will be released? I'd like to use it at work.

@ThibaultLemaire released with 0.0.3 but there are some build errors.I think I have introduced some time ago.

Ah that's too bad, don't worry though, this is low priority for me.

Not for me  ;-)
I have some time off to play around here, so I will release things soon I believe.

@ThibaultLemaire  released @ https://pypi.org/project/loguru-mypy/0.0.3

Thank you very much, I added loguru-mypy to our project on a feature branch and confirmed it's working in the scenario that initally prompted me to open this issue. :+1:
However, I'm hitting two new scenarii where the plugin fails to correctly validate the code :slightly_frowning_face:
The first, I had anticipated when working on my PR:
from loguru import logger

logger.configure(
    handlers=[
        dict(
            sink=sys.stderr,
            format="Site {extra[site]} broadcasting: {message}",
            backtrace=False,
            diagnose=False,
        ),
    ],
    extra={"site": "unknown"},
)

site = 19
scp = 682
# We're passing the `site` number for local formatting of this message, but we're also passing the `scp` number as `extra` for the formatter of the handler.
logger.warning("SCP-{} containment breach detected", scp, site=site)
# error: Not all arguments converted during string formatting
# But this is easily circumvented:
logger.bind(site=site).warning("SCP-{} containment breach detected", scp)

The workaround isn't too bad, and fixing the issue - I think - would prove a tad more complicated than my previous PR (see #48 ).
But the second issue is a real blocker: essentially it's the 3rd exemple of #49 as reported by @Delgan
from loguru import logger

foo = "bar"
logger.info(foo)  # error: INTERNAL ERROR

So I think I'm going to start working on fix for that, as I think it shouldn't be too complicated.

Thank you very much, I added loguru-mypy to our project on a feature branch and confirmed it's working in the scenario that initally prompted me to open this issue. :+1:
However, I'm hitting two new scenarii where the plugin fails to correctly validate the code :slightly_frowning_face:
The first, I had anticipated when working on my PR:
from loguru import logger

logger.configure(
    handlers=[
        dict(
            sink=sys.stderr,
            format="Site {extra[site]} broadcasting: {message}",
            backtrace=False,
            diagnose=False,
        ),
    ],
    extra={"site": "unknown"},
)

site = 19
scp = 682
# We're passing the `site` number for local formatting of this message, but we're also passing the `scp` number as `extra` for the formatter of the handler.
logger.warning("SCP-{} containment breach detected", scp, site=site)
# error: Not all arguments converted during string formatting
# But this is easily circumvented:
logger.bind(site=site).warning("SCP-{} containment breach detected", scp)

The workaround isn't too bad, and fixing the issue - I think - would prove a tad more complicated than my previous PR (see #48 ).
But the second issue is a real blocker: essentially it's the 3rd exemple of #49 as reported by @Delgan
from loguru import logger

foo = "bar"
logger.info(foo)  # error: INTERNAL ERROR

So I think I'm going to start working on fix for that, as I think it shouldn't be too complicated.


Change SETUP_APP_VERISON(TBB to set(TBB_VERSION)

Fixes CMake warnings when using newer cmake.
Also modified bv_tbb to remove SETUP_APP_VERSION, as it's not necessary.
Resolves #5292
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
~- [ ] I have updated the release notes~
~- [ ] I have made corresponding changes to the documentation~
~- [ ] I have added debugging support to my changes~
~- [ ] I have added tests that prove my fix is effective or that my feature works~
~- [ ] New and existing unit tests pass locally with my changes~
~- [ ] I have added any new baselines to the repo~
[X] I have assigned reviewers (see VisIt's PR procedures for more information).

Just ran into this myself so glad you are fixing. Thanks 🥇

Just ran into this myself so glad you are fixing. Thanks 🥇


Bypass URI.encode on params

I have a route that requires test=a,b but when passed as a param to Req it converts that into test=a%2Cb which is wrong.
Current work around is
  defp put_param_unencoded(req) do
    update_in(req.url.query, fn
      nil -> "test=a,b"
      query -> query <> "&" <> "test=a,b"
    end)
  end

  Req.new(...)
  |> put_param_unencoded()

Thank you for the report. Do you know what the spec says about encoding this character and/or how other popular HTTP clients treat it? In other words, do you have a proposal how this should work in Req?

Actually I think I am being very very stupid. Thanks for sanity checking me.


Import error

Describe the bug
Cannot import my project into bolt again
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-djxlhl
Steps to reproduce
Open in bolt when on the stackblitz editor
Expected behavior
The import works
Screen Recording / Screenshot
I'm importing your StackBlitz project into Bolt. This may take a moment as I set everything up. Once it's ready, you'll be able to explore and interact with your code.
Importing Stackblitz Project
sb1-djxlhl
Unexpected token 'U', "Unexpected"... is not valid JSON
An unknown error occured while running an action.
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (X11; CrOS x86_64 14541.0.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 68933827,
  "usedJSHeapSize": 60646379,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = patinas
Chat ID = 06abf40d06f2

Additional context
Error:
An unknown error occured while running an action.

@patinas Thanks for the report and apologies for the frustrating experience! The fix is explained in #1471


Fix: OpenSSL 1.1.0

OpenSSL 1.1.0 renamed some libcrypto symbols (eg: sk_* to OPENSSL_sk_*), and dropped support for DSS. I chose to drop DSS, too, for the principle of least suprise —a binary built against OpenSSL 1.0 would fail to run if OpenSSL is upgraded to 1.1.
This allows to build the Crystal test suite against OpenSSL 1.1.0.
closes #4204

Looks like there are two specs to remove for OpenSSL::HMAC


Remove unnecessary brackets

Hi,
just another small linting PR 😁

Nice, thanks @zntfdr!
Sent with GitHawk


Hide sources from binary

All the js sources are to be read from binary even with a notepad. Any recommended way to hide them?

Hi!
May I ask you how did you "read" them?

From the README:

scripts is a glob or list of globs. Files specified as scripts will be compiled using v8::ScriptCompiler and placed into executable without sources. They must conform JS standards of those Node.js versions you target (see Targets), i.e. be already transpiled.

So you have to configure your pkg correctly. Classic RTFM.


Creating a prodcution build will fail.

While its great to see it works well in dev mode. Building a production build with yarn build --realease should work too.

@puttyman have you able to solve this issue?

@yutingzhao1991 any help will be appriciated. we are stuck here.

I am in the same boat.

Is it feasible to merge the latest version from react-firebase-starter into your repo (I'm using it in a several projects without issues)? Alternatively, would you paste an error message that you get here?


[Wait for #2536][application] add generate_multiple_tokens for llm

Added generate_multiple_tokens function for first generation on llm.
This function takes one logits and generates multiple output tokens.
To meet the purpose of the target application,
even if input are multiple logits,
only the first logits is used to generate multiple output tokens.
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2540. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2540-202404051420140.12845396995544-9b084590641606c09502ef79096695d67cd0c0db/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2540-202404051521080.72869992256165-23e5df0a40b07abae40e9bd2133689a0a97f7c3c/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2540-202404051613580.66992688179016-23e5df0a40b07abae40e9bd2133689a0a97f7c3c/.


Update tutorial.md

Description of the change:
Fixes a typo
Motivation for the change:
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

looks like you need to sign your commit (with the -s option)


Fix #84 Wrong MCC+MNC encoding

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

[X] I have read the CONTRIBUTING document.
[X] Each of my commits messages include Signed-off-by: Author Name<EMAIL_ADDRESS>to accept the DCO.

Hi @rjaksa,
Your bug should be fixed by this PR. Can you try PacketRusher again with this fix / using branch fixmnc.
Thanks a lot,
Valentin

The code change works for the gNB. But there is an informative printf regarding the PLMNs Identities supported by AMF that shows the MNC incorrectly. It lists the 3-digit MNC with the digits in encoded order, instead of the "human readable" format. E.g. MNC 123 is printed as "231".
E.g. for MCC=987, MNC=123, the printf looks like:

INFO[0000] [GNB][AMF] PLMNs Identities Supported by AMF -- mcc: 987 mnc:231

Thank you @rjaksa, it's fixed!


Remove jsconfig.json lightbulb hint

Issue #15205
Bug
Typescript now infers the js project if no jsconfig.json is found. This means that the light bulb hint about creating a jsconfig.json file is not required.
Fix
Remove the jsconfig.json light bulb logic.
closes #15205

LGTM. Thanks. @joaomoreno :dancing_men:


[fix][cms]destroy_all: add condition for forms _menu.html.erb

Coverage remained the same at 84.617% when pulling c3667acb6a4822b8755ae4dbd5bcba5f6e1223b8 on fix-rspec_for_destroy_all into 195d4b770dbd2fddd4c6ecf4ee242f36a4c3fa82 on master.


format 'umd' instead of es6?

I was curious about the format: 'umd' in rollup config (and the babelrc: false when you have an existing .babelrc in the project). I was interested because I wanted to use the package in my own library using rollup. Maybe I am misunderstanding something. I thought that typically a umd release would be transpiled to pkg.main (/dist/index.js) and an es6 module (also transpiled) would be in pkg.module (maybe /dist/index.es.js or something).

As a reference, here's a project's config from a project of the rollup creator himself: https://github.com/Rich-Harris/magic-string/blob/master/rollup.config.js

The package.json for this plugin has a "main" field that points to the ES5 distributable (dist/index.js) and a "module" field that points to src/index.js. Since the ES6 source is a single file, with no other dependencies (except Vue, which is just a peer dependency), there isn't anything else to bundle and so no output target for it in my rollup config.
I use rollup in my own build workflow, and since rollup understands the module field in package.json, there's no other configuration required to bundle the plugin into the larger codebase.
The .babelrc in the project is there for the sake of Jest, which needs it for its own transpile step before running tests. I'm using the "env" preset there and in rollup.config, but with different settings, so for rollup I ignore .babelrc and just specify the preset options in the plugin.

Ahh. Gotcha. Also, you could use Webpack 2/3 at your app level as it will also automatically grab your ES6 module (defined in your package.json module field). Thanks for clarification!


Suggestions for region statistics ?

Dear @lassoan I need a little push in the right direction.
There is good overall progress with both Segmenter and Analyzer, but I am missing the possibility to calculate the extent of  "infiltrated" and "collapsed" regions of the lungs dorsally versus ventrally (and in apical, middle, and lower third). All this would be an awesome parameter for the need of putting patients supine vs. prone during ventilation.
My first idea was to cut the lung masks into the respective pieces with the "surface cut" method.
I already realized the calculation of the centroids (thanks for your code in the repository) in the right and left lung masks. Works great. I could easily define the plane for a mask "surface cut" from here.
However, if I do this, I will be missing out on the few millimeters of "cut out" lung mask during the final Lung CT Analyzer processing.
Alternatively, I could divide the result segments of Lung CT Analyzer into pieces, but I dont want to loose "material" of the segments.
As an example I would need:
Infiltrated left lung
Infiltrated left upper third
Infiltrated left middle third
Infiltrated left lower third
Infiltrated left ventral half
Infiltrated left dorsal half
.
.
Do you have any suggestions here?
Maybe also a topic for Discourse if you think so.

You can create mask segments (left,  left upper third, left middle third, left lower third, left ventral half, left dorsal half) and either combine with segments (infiltrated, etc) using Logical operators effect.
You can also experiment with Segment cross-section area module, which provides you a curve of how the area of a cross-section of a segment changes along a chosen axis. By dividing area of infiltration by total area, you could get percentage of infiltration along AP, IS, LR axes. You could get volumes in a region (e.g., upper third) by summing the values in a range and multiplying it with voxel height.


Advanced meal assist

Dev pathway for ama happens here.

This seems to be main place for AMA feature to live, and was able to catch up to dev via https://github.com/openaps/oref0/pull/119, https://github.com/openaps/oref0/pull/121.  This should co-ordinate with https://github.com/openaps/oref0/issues/114

Checks are now passing, so this is a candidate for merge if no one flags any issues.  I believe @cjo20 said things looked good for him.  Anything else we need to do before merging to dev?

Yup, things looked good for me when I upgraded. I'm not sure if the reordering of the yargs stuff in oref0-get-profile has introduced any upgrade problems, but it should be minor

Yeah, I'm not too worried about anything that only affects people who've added a feature in the last week or two.  :-)

One last call for any objections to merging AMA to dev.

Left a few notes regarding tips on maintaining positional arguments vs option switches over the long haul.  Overall I'm excited to get this into dev and subsequently do a release.

Sounds like we should merge to dev and then do some additional yargs conversion (in addition to the stuff that has recently been added) before releasing to master. If no objections, I'll do the merge to dev today.

I'd be ok requiring the --auto-sens, --meal, and other options, maybe the merge to dev is the time to do that, I updated my rigs to use the options and it's not a big deal.


Poor Text Placement

Before creating an issue, go through all open/closed issues to avoid getting your issue/suggestion marked as duplicate##
Issue and Steps to Reproduce:
The text is not padded in mobile view. I would like to add some css and bootstrap to make it responsive. Kindly assign this task to me under GSSOC'23
Screenshots:

Expected:
Actual:
Additional Details:

please provide the page where the issue was found.

please provide the page where the issue was found.

Saw it in rejenuate section but now I see its updated. however the image is coming out of the container in mobile view.

I would like to fix it under gssoc'23, please assign me this issue.

Hi, @rishitachail are you still working on this issue, if so please what are issues are resolved and what are issues are still remaining.!


Any plans to load data from remote sources?

The usage appears limited to

Right-click on a .csv or .tsv file and choose View in SandDance.

I work primarily with databases, and I'm sure I'm not alone. It takes time to query data from a database and save the data in local files. It'd be useful if SandDance did this work for me.
Are there any plans to address this use case?
Are there any alternatives you can suggest? Maybe this can be accomplished with another, existing VS Code extension and I just don't know where to look?

Azure Data Studio is designed to accomplish this 🤓 and we have a SandDance extension for it.

Also, you can enter a URL in the UI at https://sanddance.js.org/app/
Click the "Sample Data" button to drop-down other data file choices, then choose URL.

FYI The other server will need to accept CORS.


PyKOB/MKOB/MRT Issues found during testing of version 1.3.1/4.2.1/1.3.1 for release.

This single issue will be used to collect small issues found during the (hopefully) final testing before release.
Comments will be added as small issues are found. If any major issues are found, a separate issue will be created to track it. At release, any issues that remain unresolved will be moved into individual issues, so they can be tracked.

Computer (synth) sounder occasionally stops sounding open/close circuit, though it does sound the text.
3/13/2024 SB

If you have an interface configured, and you don't have it available when you start, it displays the message letting you know, but then the keyboard sender window won't let you type anything. You can fix it by going to the icon/task bar and click on the icon to reselect the window (clicking on the window itself doesn't fix it).
3/13/2024 ES

MKOB: Tools > Show Packets not working.
3/13/2024 SB

The red connected notification doesn't show when the MKOB is set to autoconnect.
It does connect OK but it doesn't turn red.
If I disconnect and reconnect it shows but not on startup.
3/18/2024 KJ

The simulated sounder is whacky it makes a horrible buzzing instead of either the tone or clicks.
MRT is fine though.
3/18/2024 KJ

RE No Network Crash
I just tried MKOB without a network in my office and got the same behaviour.
The GUI stops responding before I can get access to anything.
I will try it on another computer and let you know.

Just tried with brand new install of bot Python and the current files.
Same behaviour, with no internet connection MKOB stops responding and crashes.
Ciao
KJ

Interesting using the current files (with the sound and connection indicator fixes) the no Internet problem seems to be fixed.
I have attached the detailed log for sending FRED, once while connected and once disconnected.
Hopefully that will be useful.
CWindowsSystem32cmd.exe.txt


Ergonomics issue with 0.16

Hi,
while upgrading a project from rust_postgres 0.15 to 0.17 and r2d2_postgres 0.14 to 0.16, we have some troubles with the new definition of PostgresConnectionManager<T> that contains a new generic parameter <T>.
The problem is that the r2d2 pool returns a PooledConnection that now has the very same generic parameter <T> that, as a consequence, has to be redeclared in every place where the connection is used.
For example, our project structure follows the Domain Driven Design principle, then:

we have a set of Repositories that use a PooledConnection, so they all have to declare the <T> generic param
we have a set of Services that use the Repositories, so all Services need to declare <T> too
we have a set of Controllers that use the Services, so all Controllers need to declare <T>
and so on...

In the end, this <T> generic is declared literally everywhere.
In addition, the rust_postgres Client has no generic parameters at all so it is not clear why the PooledConnection requires it.
I don't know if it is feasible, but the issue could be fixed by changing the signature in the r2d2 pool from:
    pub fn get(&self) -> Result<PooledConnection<M>, Error>;

to
    pub fn get(&self) -> Result<PooledConnection<M::Connection>, Error>;

Or, maybe, the <T> param can be boxed in the PostgresConnectionManager struct:
#[derive(Debug)]
pub struct PostgresConnectionManager {
    config: Config,
    tls_connector: Box<MakeTlsConnect<Socket, TlsConnect=Send, Stream=Send, ... >>,
}

Do the Repositories explicitly need a PooledConnection or can they just work with a &mut Client?
Changing the type of PooledConnection would need to happen on r2d2, not r2d2-postgres. You could also make a type alias for the pooled connection type to avoid having to spell the full type out everywhere.

@sfackler
It's a good proposal and, indeed, it solves the issue with the repositories. Nevertheless, we still have the issue with the services that own a copy of the r2d2 pool.
As a temporary workaround, we created our own implementation of the ManageConnection that needs no generic parameters:
use r2d2::ManageConnection;
use postgres::{Config, Socket, Client};
use postgres::tls::MakeTlsConnect;
use tokio_postgres::Error;

pub struct PostgresConnectionManager {
    config: Config,
    tls_connector: Box<dyn Fn(&Config) -> Result<Client, Error> + Send + Sync>,
}

impl PostgresConnectionManager {
    pub fn new(config: Config, tls_connector: Box<dyn Fn(&Config) -> Result<Client, Error> + Send + Sync>) -> PostgresConnectionManager {
        PostgresConnectionManager {
            config,
            tls_connector,
        }
    }
}

impl ManageConnection for PostgresConnectionManager {
    type Connection = Client;
    type Error = Error;

    fn connect(&self) -> Result<Client, Error> {
        (self.tls_connector)(&self.config)
    }

    fn is_valid(&self, client: &mut Client) -> Result<(), Error> {
        client.simple_query("").map(|_| ())
    }

    fn has_broken(&self, client: &mut Client) -> bool {
        client.is_closed()
    }
}

#[cfg(test)]
mod test {

    use super::*;
    use tokio_postgres::NoTls;

    #[test]
    fn new_connection() {

        let tls = NoTls;
        let manager = PostgresConnectionManager::new(
            "host=localhost user=postgres".parse().unwrap(),
            Box::new(move |config| config.connect(tls.clone())),
        );

        let _pool = r2d2::Pool::new(manager).unwrap();
    }

}


Fix valueChanged event firing twice

Please take a moment to fill out the following:
Fixes issue #21
Changes Proposed in this pull request:

Value and ValueFormatted are updating with backing fields now.

PR Checklist

[ ] Has tests (if omitted, state reason in the description)
[x] Rebased on top of a master at the time of PR
[x] Consolidate commits as makes sense

Looks good.


I just stumbled upon this project

First of all thanks for raising awareness of projects that abuse open-source term!
I've made similar project on a more FOSS-respecting platform: https://codeberg.org/illiliti/openwashing. However I'll deprecate it in favor of yours in order to join efforts. I didn't plan to hunt for these incidents anyway(I only listed those that I knew of already), so joining efforts would make sense for me. Do you support my proposal?
Regardless, there is a case that you didn't document yet: https://codeberg.org/illiliti/openwashing#skiff. I can create a separate issue if you want.

Sure! Happy to join efforts, as long as you're happy with the scope/direction/methods I've set-out, although happy to take feedback on that.

Great! I've archived my repo in favor of yours.

I've just opened https://github.com/ssddanbrown/Open-Source-Confusion-Cases/issues/22 to track Skiff, thanks for letting us know of that case!

I'll follow up, thanks!


cocomac: Watch vuoncay.vn

cocomac requests the watch of the watch_keyword vuoncay\.vn. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

vuoncay\.vn has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Sorry, here is the one TP: https://metasmoke.erwaysoftware.com/post/377770

Approved by Makyen in Charcoal HQ


Add build and deploy script to travis

Codecov Report

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

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   37.73%   37.73%
=======================================
  Files          20       20
  Lines        3230     3230
=======================================
  Hits         1219     1219
  Misses       1873     1873
  Partials      138      138

Continue to review full report at Codecov.

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


chore(main): release 0.11.0

:robot: I have created a release beep boop
0.11.0 (2023-02-20)
Features

admin: defines error codes of several RPCs in the admin server (f5ed66f), closes #312
metarepos: add grpc error codes to the metadata repository service (2903f8c)
metarepos: fix grpc error code (ce5feb3)
storagenode: add gRPC error codes to the admin service (80cd082), closes #312
storagenode: add gRPC error codes to the log server (5e813fc), closes #312
varlogcli: do not deny filtered logsteam (26fec68)
varlogcli: select log stream within AllowedLogStreams (26627a0)

This PR was generated with Release Please. See documentation.

Codecov Report
Base: 63.38% // Head: 63.26% // Decreases project coverage by -0.13% :warning:

Coverage data is based on head (2c28090) compared to base (67f0c8d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #321      +/-   ##
==========================================
- Coverage   63.38%   63.26%   -0.13%
==========================================
  Files         130      130
  Lines       17456    17456
==========================================
- Hits        11065    11044      -21
- Misses       5845     5862      +17
- Partials      546      550       +4

Impacted Files
Coverage Δ

internal/storagenode/logstream/committer.go
78.62% <0.00%> (-2.42%)
:arrow_down:

internal/storagenode/logstream/executor.go
79.67% <0.00%> (-1.82%)
:arrow_down:

...metarepos/dummy_storagenode_client_factory_impl.go
64.02% <0.00%> (-1.22%)
:arrow_down:

internal/metarepos/storage.go
80.01% <0.00%> (-1.10%)
:arrow_down:

internal/metarepos/report_collector.go
77.85% <0.00%> (+0.85%)
:arrow_up:

pkg/util/runner/first/first.go
93.75% <0.00%> (+3.12%)
:arrow_up:

internal/storagenode/replication_server.go
81.35% <0.00%> (+5.93%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

:robot: Release is at https://github.com/kakao/varlog/releases/tag/v0.11.0 :sunflower:


Problems with gRPC/Protobuf on Windows

Issue Description
I'm trying to create a project using Protobuf and gRPC, but Omnisharp keeps failing.
Steps to Reproduce
The project I have issues with is a lot bigger, but I have tried to create a minimal example.

Create a new project with the following layout:

.
├── test.csproj
├── grpc
     └── test.proto

where test.csproj looks like this:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFrameworks>netcoreapp3.0</TargetFrameworks>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Grpc" Version="2.24.0"/>
    <PackageReference Include="Grpc.Tools" Version="2.24.0"/>
    <Protobuf Include="grpc/test.proto" ProtoRoot="grpc"/>
  </ItemGroup>
</Project>

and test.proto like this (though I don't think this matters):
syntax = "proto3";
package test;

message Test {
  int32 testy = 1;
}

Press "Restore" when the prompt in the lower right corner says there are unresolved dependencies.

Wait for a few seconds.

Expected Behavior
The project is loaded with no errors.
Actual Behavior
A prompt appears in the lower right corner saying

Some projects have trouble loading. Please review the output for more details."

See the Logs section for more logs
Logs
OmniSharp log

Starting OmniSharp server at 2019-10-03, 18:36:30
    Target: c:\Users\martint\test

OmniSharp server started.
    Path: C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\OmniSharp.exe
    PID: 3916

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on Windows 6.2.9200.0 (x64)
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 2 MSBuild instance(s)
            1: Visual Studio Professional 2017 15.9.28307.518 - "C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin"
            2: StandAlone 16.0 - "C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\.msbuild\Current\Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        MSBUILD_EXE_PATH environment variable set to 'C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\.msbuild\Current\Bin\MSBuild.exe'
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: StandAlone 16.0 - "C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\.msbuild\Current\Bin"
            CscToolExe = csc.exe
            CscToolPath = C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\.msbuild\Current\Bin\Roslyn
            MSBuildExtensionsPath = C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\.msbuild
            MSBuildToolsPath = C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.3\.omnisharp\1.34.5-beta.24\.msbuild\Current\Bin
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in 'c:\Users\martint\test'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.MSBuild.ProjectSystem
        No solution files found in 'c:\Users\martint\test'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in 'c:\Users\martint\test'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: c:\Users\martint\test\test.csproj
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.EditorConfigWorkspaceOptionsProvider, Order: 200
[info]: OmniSharp.WorkspaceInitializer
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location 'c:\Users\martint\test' on host 21208.
[info]: OmniSharp.MSBuild.ProjectManager
        Successfully loaded project file 'c:\Users\martint\test\test.csproj'.
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: test
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: c:\Users\martint\test\test.csproj
[fail]: OmniSharp.MSBuild.ProjectLoader
        The specified task executable location "\windows_x64\protoc.exe" is invalid.
[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file 'c:\Users\martint\test\test.csproj'.
c:\Users\martint\test\test.csproj
C:\Users\martint\.nuget\packages\grpc.tools\2.24.0\build\_protobuf\Google.Protobuf.Tools.targets(264,5): Error: The specified task executable location "\windows_x64\protoc.exe" is invalid.

[info]: OmniSharp.MSBuild.ProjectManager
        Update project: test

C# log

Getting latest OmniSharp version information
Downloading package 'Latest OmniSharp Version Information' (1 KB).................... Done!

In case you don't, try installing Visual Studio 2019 build tools.
You can find it here: https://visualstudio.microsoft.com//downloads/

Do you have Visual Studio 2019 installed on your machine?
In case you don't, try installing Visual Studio 2019 build tools.
You can find it here: https://visualstudio.microsoft.com//downloads/

I tried installing VS2019, including the build tools, but the issue remains. This is my new OmniSharp log:

Starting OmniSharp server at 2019-10-04, 09:22:04
    Target: c:\Users\martint\test

OmniSharp server started.
    Path: C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.4\.omnisharp\1.34.5-beta.24\OmniSharp.exe
    PID: 11968

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on Windows 6.2.9200.0 (x64)
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 3 MSBuild instance(s)
            1: Visual Studio Professional 2017 15.9.28307.518 - "C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin"
            2: Visual Studio Professional 2019 16.3.29326.143 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Current\Bin"
            3: StandAlone 16.0 - "C:\Users\martint\.vscode\extensions\ms-vscode.csharp-1.21.4\.omnisharp\1.34.5-beta.24\.msbuild\Current\Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: Visual Studio Professional 2019 16.3.29326.143 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\Current\Bin"
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in 'c:\Users\martint\test'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.MSBuild.ProjectSystem
        No solution files found in 'c:\Users\martint\test'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in 'c:\Users\martint\test'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: c:\Users\martint\test\test.csproj
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.EditorConfigWorkspaceOptionsProvider, Order: 200
[info]: OmniSharp.WorkspaceInitializer
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location 'c:\Users\martint\test' on host 4340.
[info]: OmniSharp.MSBuild.ProjectManager
        Successfully loaded project file 'c:\Users\martint\test\test.csproj'.
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: test
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'c:\Users\martint\test\test.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: c:\Users\martint\test\test.csproj
[fail]: OmniSharp.MSBuild.ProjectLoader
        The specified task executable location "\windows_x64\protoc.exe" is invalid.
[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file 'c:\Users\martint\test\test.csproj'.
c:\Users\martint\test\test.csproj
C:\Users\martint\.nuget\packages\grpc.tools\2.24.0\build\_protobuf\Google.Protobuf.Tools.targets(264,5): Error: The specified task executable location "\windows_x64\protoc.exe" is invalid.

[info]: OmniSharp.MSBuild.ProjectManager
        Update project: test

The C# log is identical to the one before.

I think the line

C:\Users\martint.nuget\packages\grpc.tools\2.24.0\build_protobuf\Google.Protobuf.Tools.targets(264,5): Error: The specified task executable location "\windows_x64\protoc.exe" is invalid.

is the relevant one. Looks like the protobuf targets are trying to concatenate a path with a msbuild variable that's not populated. Can you look at that file and share that line?

These are the lines which start with 264:
    <ProtoCompile Condition=" '@(_Protobuf_OutOfDateProto)' != '' "
      ToolExe="$(Protobuf_ProtocFullPath)"
      Generator="$(Protobuf_Generator)"
      Protobuf="%(_Protobuf_OutOfDateProto.Source)"
      ProtoPath="%(_Protobuf_OutOfDateProto.AdditionalImportDirs);$(Protobuf_StandardImportsPath);%(_Protobuf_OutOfDateProto.ProtoRoot)"
      ProtoDepDir="$(Protobuf_DepFilesPath)"
      OutputDir="%(_Protobuf_OutOfDateProto.OutputDir)"
      OutputOptions="%(_Protobuf_OutOfDateProto._OutputOptions)"
      GrpcPluginExe="%(_Protobuf_OutOfDateProto.GrpcPluginExe)"
      GrpcOutputDir="%(_Protobuf_OutOfDateProto.GrpcOutputDir)"
      GrpcOutputOptions="%(_Protobuf_OutOfDateProto._GrpcOutputOptions)"
    >
      <Output TaskParameter="GeneratedFiles" ItemName="_Protobuf_GeneratedFiles"/>
    </ProtoCompile>

I think what the error message refers to is the ToolExe, which is defined like this on line 69 and forward:
    <PropertyGroup>
      <!-- First try environment variable. -->
      <Protobuf_ToolsOs>$(PROTOBUF_TOOLS_OS)</Protobuf_ToolsOs>
      <Protobuf_ToolsCpu>$(PROTOBUF_TOOLS_CPU)</Protobuf_ToolsCpu>
      <Protobuf_ProtocFullPath>$(PROTOBUF_PROTOC)</Protobuf_ProtocFullPath>

      <!-- Next try OS and CPU resolved by ProtoToolsPlatform. -->
      <Protobuf_ToolsOs Condition=" '$(Protobuf_ToolsOs)' == '' ">$(_Protobuf_ToolsOs)</Protobuf_ToolsOs>
      <Protobuf_ToolsCpu Condition=" '$(Protobuf_ToolsCpu)' == '' ">$(_Protobuf_ToolsCpu)</Protobuf_ToolsCpu>
      <Protobuf_ProtocFullPath Condition=" '$(Protobuf_ProtocFullPath)' == '' and '$(Protobuf_ToolsOs)' == 'windows' "
           >$(Protobuf_PackagedToolsPath)\$(Protobuf_ToolsOs)_$(Protobuf_ToolsCpu)\protoc.exe</Protobuf_ProtocFullPath>
      <Protobuf_ProtocFullPath Condition=" '$(Protobuf_ProtocFullPath)' == '' "
           >$(Protobuf_PackagedToolsPath)/$(Protobuf_ToolsOs)_$(Protobuf_ToolsCpu)/protoc</Protobuf_ProtocFullPath>
    </PropertyGroup>

So it seems like the problem is that the Protobuf_PackagedToolsPath gets set to an empty string. It is defined in the corresponding props file: Google.Protobuf.Tools.props in the same folder as the targets file. Line 11 looks like this:
    <Protobuf_PackagedToolsPath>$( [System.IO.Path]::GetFullPath($(MSBuildThisFileDirectory)../../tools) )</Protobuf_PackagedToolsPath>

Not sure how to continue debugging here. I have very little experience with MSBuild files.
I'm attaching the two files for reference (with .txt appended)
Google.Protobuf.Tools.targets.txt
Google.Protobuf.Tools.props.txt

Still having problems with this, but I have found a workaround that seems to help
<Protobuf Condition="! $(MSBuildExtensionsPath.Contains('omnisharp'))" ...>

After some testing we (me and @koliyo) have found that the Google.Protobuf.Tools.targets file seems to be parsed before Google.Protobuf.Tools.props, and that therefore the properties defined in .props is never used by .targets.
The way we found it is by adding an <Error Message="Error from .targets"/> tag in the targets file under the Protobuf_ResolvePlatform target, and some garbage (e.g. fdjkfsjsfljksdf) somewhere in the props file. OmniSharp then shows the targets file error before the props file error.
The workaround suggested by @koliyo only works when using the StandAlone MSBuild instance. When Visual Studio is installed that one will be used instead, and MSBuildExtensionsPath will not contain the string "omnisharp". A more robust (maybe) workaround we have found is to add these lines to omnisharp.json:
  "msbuild": {
    "Configuration": "omnisharp"
  }

and then this condition to the csproj file:
<Protobuf Condition="'$(Configuration)' != 'omnisharp'" ...>

Labeling as a feature request. The underlying project system used by OmniSharp will need some (perhaps significant) work to properly handle this kind of scenario like the VS project system does.

A lot has changed with O# project loading since this issue was opened. If you are still seeing this problem, please open a new issue at https://github.com/OmniSharp/omnisharp-roslyn/issues


GitHub Tab - Cannot add organization repository to teams tab

When adding a Github Tab to a teams page, users can only see their private github repos.
Steps:

Create a github user that has both private and organization repositories
Go to Microsoft Teams and add a tab for Github
Login with your github user

Result: User only has access to personal repositories to select for the team tab
Expected Result: User has access to both personal and organizational repositories
Note: The azure app service continuous deployment pipeline has options for organizational repositories. Seems like an oversight for this to be missing from Teams.

@seannybgoode - thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@officedocsbot close


Kudu Diagnostic Console does not show temp files that are created from the web app

According to documentation, the web app is allowed to write to %TEMP% folder. If this is done, any files written will not show up in the Kudu Debug console file browser. I verified the temp file gets written correctly and can be read back by the IIS serving the request. However, Kudu does not show these files, leaving the user puzzled. Could this be due to a file ownership / ACL problem?

It's by design, but you can change this using WEBSITE_DISABLE_SCM_SEPARATION. See wiki for details.


browser.querySelector problem

I'm not quite sure what to make of this issue, but here it is... for some reason certain selectors don't work when run through Zombie even though the element(s) they would resolve do in fact exist. These same selectors work on the same markup in a number of different scenarios:

Loading the raw html in the browser
Using jsdom directly to access the DOM

However, when running the same markup through zombie, the selector fails.
I've tried my best to summarize the issue in the following gist: https://gist.github.com/944655
It covers both of the above scenarios where it succeeds as well as the Zombie case where it fails. Also, please note that even though I'm using jQuery in the gist, that does not seem to be related to the problem, as just doing document.getElementById("loginForm") works for both the raw html and jsdom methods, but fails for Zombie.
Please let me know if this needs further clarification.
Thanks!
Joshua

Same problem here :S
browser.html() has the content.
browser.querySelector('whatever') undefined
browser.document.getElementById('whatever') null
somehow the same works elsewhere, but not here.

rata0071,
I have the same problem as you here.In my recent testings i was testing wih various different browser option params likw waitfor and runscript. By default without any config i didnt fetched any img tags in the nodelist, however changing the time and script made it get 13.
Mabey it not that easy at all as just saying it should work on every DOM tree, espacially if that tree is being grenerated by AJAX calls etc,
Are you in control of the client-side of the scrapped site ? In that case some client side code might help/assist the zombie request but if a public url w/o control i fear there is no easy solution on always getting the "same" and expected results.


feat(payments_v2): payment intent diesel and domain models changes v2

Type of Change

[x] Refactoring

Description

This PR refactors the codebase to include all the payment related operations under the v1 feature flag.
This also includes changes to the payment intent diesel and domain models to add new fields that are relevant to the v2 version of application.
Additional Changes

[ ] This PR modifies the API contract
[x] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code

All the comments will be addressed in the upcoming PR. We can get this PR merged as there are some dependencies


Add some plugins

Add the 3 plugins I have created so far:
• osx-eject
• hash
• lipsum

Looks awesome 👍


Changed str_to_u32 to remove unsafe code. Need to verify Native Endia…

Hi just a minor detail, I saw some unsafe code that might have impacted an issue that I found,
it doesnt fix the problem but you might be interested in incorporating it, assuming its ok.
Also changed the tests to be optimized as the issue  I have only is present in release builds.
All tests pass ...

Yes, it's strange how something is crashing in release builds. Thanks for finding this pattern to get rid of this transmute. I think I do things wrong with endianess between intel and arm and rgb and bgr pixel formats.


Ability to search by Thalukk or Village for camps

Subject of the issue
The current camp search has the option to search by District alone. When number of camps are large (eg Ernakulam with ~150), navigating pages is difficult especially with no sort option as well.
Since our data entry has Thalukk and Village, can we add a filter for atleast one of these fields
Your environment
Windows 7, Chrome
Expected behaviour
When a district is selected a filter to choose Thalukk should appear. It should be a drop down of unique thalukks from all camps in that district. The search results should then be limited to that Thalukk.
Something like below

Actual behaviour
Only search by District is available currently

The thaluk based search is experiencing some issues, I am looking into it

@vigneshhari Need help with this one?

The exact taluk is not being entered which causes a lot of inconsistencies, We have to first maintain a fixed list of taluks before implementing the search, I'm sorry I didn't reply earlier.

@vigneshhari  - Giju and Naveen may be able to provide us with the fixed list that was prepared last year.

List is available as a python list in one of the merges. Only thing is
we'll have to find a way to deal with the existing taluks with different
spellings
On Mon, 19 Aug, 2019, 3:40 PM Bobinson K B<EMAIL_ADDRESS>wrote:

@vigneshhari https://github.com/vigneshhari - Giju and Naveen may be
able to provide us with the fixed list that was prepared last year.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IEEEKeralaSection/rescuekerala/issues/1029?email_source=notifications&email_token=AC6LOEE2Z5SWZMDIXTOAMLLQFJWQFA5CNFSM4IKYMJGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4SMNFA#issuecomment-522503828,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC6LOECWQIKF2W3BPU3WICTQFJWQFANCNFSM4IKYMJGA
.


check api key is valid

Now is checked in CML,  should be done here instead.
Once done CML code has to be removed

solving #65 this is not needed


hwmv2: atmel: The SOC_FAMILY_SAM do not follow _ suffix

Describe the bug
When porting the Atmel SoC's was detected at #68838 that SOC_FAMILY_SAM does not have the vendor as part of the suffix. On the PR was agreed that this could be made after the collab-hwm branch be merged, see _https://github.com/zephyrproject-rtos/zephyr/pull/68838#discussion_r1489378425_
Expected behavior
Tree wide/hal upgrade from SOC_FAMILY_SAM/SOC_FAMILY_SAM0 to SOC_FAMILY_ATMEL_SAM/SOC_FAMILY_ATMEL_SAM0 to align with new HWMv2 model.
Impact
No impact is expected.
Additional context
This issue was created to allow easy track of the change.

Closed by #69377


Single reference is generated as "multiple" (array)

A referenced field that is not set as "multiple" has the type generated as an array.
Example from the demo data (from the starters): a header object has a reference to a page - page_reference. It looks like this
  {
      "data_type": "reference",
      "display_name": "Page Reference",
      "reference_to": [
          "page"
      ],
      "field_metadata": {
          "ref_multiple": false,
          "ref_multiple_content_types": true,
          "description": "Refers a linked entity of type page",
          "instruction": "Refers a linked entity of type page"
      },
      "uid": "page_reference",
      "non_localizable": false,
      "mandatory": false,
      "multiple": false,
      "unique": false
  }

However, the generated type looks like this:
    navigation_menu?: [
        {
            /** Label */
            label?: string;
            /** Page Reference */
            page_reference?: Page[]; // <--- should not be an array unless "multiple" is set to "true"
        },
    ]

I tried changing it to multiple and back to single, but nothing changes with regards to the generated type.
Version:
❯ csdx plugins
contentstack-cli-tsgen 2.1.6

Hi @Cry0nicS,
We are looking into this and we will get back to you soon with updates.
Thanks!

Hi @Cry0nicS , We have handled this now. Install the latest TSGEN v2.1.8 plugin to get the fix.

This change causes some new TS issues in our existing code base. For example, I have a Content Model with two reference fields. One is for a single reference, one is for multiple references. This now with v.2.1.8 generates the following types:
export interface TestSg {
  title: string;
  referenceSingle?: Banner;
  referenceMultiple?: Banner[];
}

However, both types are actually returned as arrays from the API:
{
    "reference_multiple": [
        {
            "uid": "blt5eb659f9251c84f6",
            "_content_type_uid": "banner"
        },
        {
            "uid": "bltac62e4a814af4dcb",
            "_content_type_uid": "banner"
        }
    ],
    "reference_single": [
        {
            "uid": "blt5eb659f9251c84f6",
            "_content_type_uid": "banner"
        }
    ],
    "title": "Test TSGen"
}

IMO this fix for v2.1.8 needs a rollback to have the API output match the types again.


Bump manifest version

I'm not exactly sure why, I went down to re-generating ksonnet.4 but for some reason withReadOnlyRootFilesystem is not available anymore. In order to move forward I removed it for now.
@paulfantom @squat @metalmatze @s-urbaniak

that's a bummer that this setting disappeared :thinking: just one nit regarding readOnlyRootFilesystem else :+1:


KSC CommNet Origin range is low (2Gm)

Right now KSC (CommNet origin) connection range is 2Gm, while in KSP1 with DSN level 2 it was 50G.
We should upgrade it or provide a way to "upgrade" it.

Now it's configurable in settngs


Add a “Related” block to posts

This follows the examples at https://gohugo.io/content-management/related/ to add a "related content"  block to posts (see https://honeypot.net/2024/05/06/demand-that-car.html for an example). Hugo's own examples use a post's keywords, date, and tags to find related content. This adds categories into the mix to support the micro.blog way of doing things.
To enable, edit config.json and rename the relatedCHANGEME key to related.

I'm going to make this a plugin instead.


Geo-Classification offloads too much to FRA1

Our Geo-Classify Functions give FRA1 the highest share of traffic, by quite a bit, but AMS3 has less than a fifth of that. Considering they are so close to each other, I think we should re-balance that.
It's probably like that because of the $germanic variable containing too many top level countries. Perhaps we should get rid of that and $vikings and just have $isles, randomly selecting the rest.

Location
Downloads

fra1
26.50%

sfo1
21.17%

sgp1
20.02%

nyc3
19.51%

ams3
9.38%

lon1
3.43%

Note: Line 97 has a spelling mistake.

Based on this months statistics, even FRA1 can withstand 10x the current load. Based on the last release spike, that's about what we'll be putting down, so evening this out isn't a release blocker, and doesn't have any release data anyway.

I don't think this is critical right now, but it's probably not really wishlist either.

Location
Downloads

fra1
29.31%

sgp1
19.63%

sfo1
18.94%

nyc3
18.79%

ams3
9.99%

lon1
3.35%


Getting error while training the SeqGAN model

Hi,
I got the following error when training the SeqGAN model.
epoch:13
[1.1808003]
[1.1286035]
[1.1499095]
epoch:14
Traceback (most recent call last):
  File "main.py", line 85, in <module>
    parse_cmd(sys.argv[1:])
  File "main.py", line 69, in parse_cmd
    gan.train_oracle()
  File "path/to/Texygen/models/seqgan/Seqgan.py", line 131, in train_oracle
    self.train_discriminator()
  File "path/to/Texygen/models/seqgan/Seqgan.py", line 52, in train_discriminator
    self.dis_data_loader.load_train_data(self.oracle_file, self.generator_file)
  File "path/to/Texygen/models/seqgan/SeqganDataLoader.py", line 70, in load_train_data
    self.labels = np.concatenate([positive_labels, negative_labels], 0)
ValueError: all the input arrays must have same number of dimensions

The command I ran is
python main.py -g seqgan

Do you have any idea how it happens or how to fix it?
Thanks in advance.

Hi, are you running the original Texygen with Python3? Seems your generator fails to output text, could you please show me your 'save/generator.txt'?

Thanks! It works perfectly now. I think there was some i/o issue last time I ran it.


Update build.boot in boot-setup

Added start-repl to :refer attribute of adzerk.boot-cljs-repl entry in require form of build.boot so client browser repl can start as per  README.md.

Thanks!


Up version to D2010

I am running this on D2010 and the shorter version runs fine on it.
The {ELSE} actually creates a memory leak which I have not been able to track down:
The block is currently used for an object of class: DUnitX.Windows.Console$ActRec

I no longer have D2010 installed to test this - I will merge but may switch back if others report issues with this change.


Create a timeline

Add a list of all your activities, achievements, work experience. Add as much info as you can remember. In case you don't have working experience tell about your pet-projects or student projects. Don't be shy :)

It all started since 2016, to become a Frontend Developer.
2016 - Started working with WordPress for a new startup, mostly with premium themes.
2017 - Started learning about HTML and CSS on Udemy and YouTube.
2018 - Started learning about JavaScript, GIT, CSS frameworks, and similar stuff on Udemy, Treehouse, Frontend Masters.
2019 - Started learning a little bit about React in Udemy.
2020 - Left the start up and started a new path as a freelancer, managing sales, marketing and development in pandemic times.
2021 - Couldn't handle the pandemic, so I decided to get a new job on a startup with around 12 employees.
2022 (half) - Got a new job on another startup, having two jobs at a time.
2023 - Decided to retire from the older job and stay with the most recent one.


AI section not working

{"error":"Failed to detect contract type"}
Request URL:
http://localhost:8080/contracts/detect-type
Request Method:
POST
Status Code:
500 Internal Server Error
Remote Address:
[::1]:8080
Referrer Policy:
strict-origin-when-cross-origin
access-control-allow-credentials:
true
access-control-allow-origin:
http://localhost:3003
connection:
keep-alive
content-length:
42
content-security-policy:
default-src 'self';base-uri 'self';font-src 'self' https: data:;form-action 'self';frame-ancestors 'self';img-src 'self' data:;object-src 'none';script-src 'self';script-src-attr 'none';style-src 'self' https: 'unsafe-inline';upgrade-insecure-requests
content-type:
application/json; charset=utf-8
cross-origin-opener-policy:
same-origin
cross-origin-resource-policy:
same-origin
date:
Sun, 20 Oct 2024 09:40:16 GMT
etag:
W/"2a-XF+hW11LwHDUPPb07tahbLHgRMQ"
keep-alive:
timeout=5
origin-agent-cluster:
?1
referrer-policy:
no-referrer
strict-transport-security:
max-age=31536000; includeSubDomains
vary:
Origin
x-content-type-options:
nosniff
x-dns-prefetch-control:
off
x-download-options:
noopen
x-frame-options:
SAMEORIGIN
x-permitted-cross-domain-policies:
none
x-xss-protection:
0
accept:
application/json, text/plain, /
accept-encoding:
gzip, deflate, br, zstd
accept-language:
en-US,en;q=0.9
connection:
keep-alive
content-length:
49856
content-type:
multipart/form-data; boundary=----WebKitFormBoundarytVwLtlQKp8psqbyQ
cookie:
__stripe_mid=28621ecd-59ae-4fd6-b0e0-9b1cc0fa9bacbe4662; connect.sid=s%3A1JFOkMw6fADvNE2Qe-YjJGEBJC-8jvRE.Lcb1KHjAbsLkEgPTXLm%2BbvwAGaAmMd9BS66vcLVxFPs
host:
localhost:8080
origin:
http://localhost:3003
referer:
http://localhost:3003/
sec-ch-ua:
"Google Chrome";v="129", "Not=A?Brand";v="8", "Chromium";v="129"
sec-ch-ua-mobile:
?0
sec-ch-ua-platform:
"macOS"
sec-fetch-dest:
empty
sec-fetch-mode:
cors
sec-fetch-site:
same-site
user-agent:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

message me on discord, cause without seeing the actually log error in ur backend server cant know whats the issue is other than the 500 which shows its server errorr:  https://discord.com/invite/cMNSUQvYRt


Template review feedback

Thanks for submitting a template to the gallery, there were a few issues in your template that needed to be resolved ahead of approving the template for the gallery.

encodeUriComponent should be used on user provided values that are then used as parameters in your url. These methods take in a single text argument and return a URI Encoded string. You should use this method on measurementId when generating the url
'https://www.googletagmanager.com/gtag/js?id='+measurementId

I did it

I did it

The link https://https//support.google.com/analytics/answer/9267735 provided in "Tag Configuration" is not reachable. Please resolve the issue.

We see error when we try to run the code. Please resolve the issue.

I fixed the issues.

https://https//support.google.com/analytics/answer/9267735 to https://support.google.com/analytics/answer/9267735
now it works.
I appended parameter initializing code upper 60 line when user doesn't put any parameter in this template.

Please check this template again.
Thanks.


set RUST_LOG for aardvark and keep aardvark stderr open with debug log level

set RUST_LOG for aardvark
keep aardvark stderr open with debug log level
It is important to read stderr output from aardvark to debug things like
panics. Since keeping the stderr open can cause EPIPE when aardvark tries
to write to it and the user closed the original stderr, this is only done
for debug/trace level.

/lgtm


Bump proto

Pull Request Test Coverage Report for Build 49

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.295%

Totals

Change from base Build 47:
0.0%

Covered Lines:
113

Relevant Lines:
139

💛  - Coveralls


Rust multithreaded tests

Since we've removed the logger (which did some non-thread-safe global operations) it's possible to run the rust tests in parallel. I also added a relwithdebinfo build profile for use in CI after noticing that the tests on the macos runner were running significantly slower than in release mode (on Linux its not as big of a difference, ~4s in release mode vs ~15s in debug mode, but on macos it was ~4s in release mode and ~600s in debug mode).
I also noticed that we weren't explicitly setting the build parallelism for the bundled cmake build in a few places (and auto-detection doesn't seem to be working on macos at least, so that was slowing down some builds as they were running single-threaded).

I think the multithreaded part is running into process limits for virtual memory, making mmap fail.


Change ExtendedRequest message

In order to implement RFC3062 from a server perspective, one needs to access the raw buffer of ExtendedRequest.value instead of the string representation.
This PR changes:

ExtendedRequest.value getter is changed to still always return a string
ExtendedRequest.valueBuffer getter is added to return the original raw buffer (or a buffer of the set string, but this is only for client side perspective)
ExtendedRequest.requestValue now is a buffer when retrieved, instead of a string

Why does ExtendedRequest.requestValue need to be a Buffer?

In rfc2251, ExtendedRequest.requestValue is specified as follow:
     ExtendedRequest ::= [APPLICATION 23] SEQUENCE {
                requestName      [0] LDAPOID,
                requestValue     [1] OCTET STRING OPTIONAL }

But in rfc3062, the request is specified as follow:
passwdModifyOID OBJECT IDENTIFIER ::= <IP_ADDRESS>.4.1.4<IP_ADDRESS>

PasswdModifyRequestValue ::= SEQUENCE {
     userIdentity    [0]  OCTET STRING OPTIONAL
     oldPasswd       [1]  OCTET STRING OPTIONAL
     newPasswd       [2]  OCTET STRING OPTIONAL }

2.1 A Password Modify request is an ExtendedRequest with the requestName
field containing passwdModifyOID OID and optionally provides a
requestValue field.  If the requestValue field is provided, it SHALL
contain a PasswdModifyRequestValue with one or more fields present.

The requestValue is changed from OCTET STRING to a SEQUENCE. In order to parse this correctly, one needs access to the buffer. To be flexible for other extensions, the functionality is not implemented in the extended request file. An example implementation could be:
var asn1 = require('asn1');
var BerReader = asn1.BerReader;

// buffer is an instance of Buffer
function passwdModifyRequestValue(buffer) {
  var requestValue = {
    userIdentity: null,
    oldPasswd: null,
    newPasswd: null,
  };

  if(! Buffer.isBuffer(buffer))
    return requestValue;

  var ber = new BerReader(buffer);

  if(ber.peek() == 0x30) {
    // Sequence
    ber.readSequence(0x30);

    if(ber.peek() == 0x80) {
      requestValue.userIdentity = ber.readString(0x80);
    }

    if(ber.peek() == 0x81) {
      requestValue.oldPasswd = ber.readString(0x81);
    }

    if(ber.peek() == 0x82) {
      requestValue.newPasswd = ber.readString(0x82);
    }
  }

  return requestValue;
}

var server = ldap.createServer();

server.exop('<IP_ADDRESS>.4.1.4<IP_ADDRESS>', function(req, res, next) {
  var parsedValue = passwordModifyRequestValue(req.valueBuffer);

  // change the password parsedValue.newPasswd in the backend

  res.end();
  return next();
});

So for a PasswdModifyRequestValue would there be an issue with converting it to a string for requestValue then back to a buffer with valueBuffer?
I'm trying to keep this backwards compatible with ldapjs and just be an updated drop in for anyone that might use ldapjs.

I changed it so the original ExtendedRequest.requestValue's behaviour is unchanged.
A new property, requestValueBuffer is used to read the string from the message. This is converted to a string, the same way BerReader does this, and stored in the original requestValue.
A new getter and setter valueBuffer is added, which returns the requestValueBuffer and sets the buffer to value which in turn sets it to requestValue.
In the constructor, requestValueBuffer is set to requestValue as a buffer, or and empty buffer if it is not defined.

Added some extra tests, also to test the setters

🎉 Thanks for working through all the changes. I'll release a new version soon.

Thanks for working through all the changes. I'll release a new version soon.

Had to figure it out for my own setup, could as well share it with the rest of the world.


AssetWritterInput Code=-11800 "The operation could not be completed

Im trying to substitute the CMSamplebuffer that comes out of the native camera in the method
- (void) captureOutput:(AVCaptureOutput *)captureOutput didOutputSampleBuffer:(CMSampleBufferRef)sampleBuffer fromConnection:(AVCaptureConnection *)connection
with my own generated CMSampleBufferRef that I build out of a GPUImage output. But I get an error when the kickflip AssetWriterInput tries to appendBuffer
writer error Error Domain=AVFoundationErrorDomain Code=-11800 "The operation could not be completed" UserInfo={NSUnderlyingError=0x133a94f60 {Error Domain=NSOSStatusErrorDomain Code=-12780 "(null)"}, NSLocalizedFailureReason=An unknown error occurred (-12780), NSLocalizedDescription=The operation could not be completed}
//THS IS MY CODE
broadcastOutputTarget = [[GPUImageRawDataOutput alloc] initWithImageSize:CGSizeMake(1920, 1080) resultsInBGRAFormat:YES];
        [filterChain.output addTarget:broadcastOutputTarget];
        self.streamImage = [[UIImage alloc] init];
        __block ToyViewController *safeSelf = self;

        __weak GPUImageRawDataOutput *weakRawOutput = broadcastOutputTarget;
        [broadcastOutputTarget setNewFrameAvailableBlock:^{
            GLubyte *outputBytes = [weakRawOutput rawBytesForImage];
            NSInteger bytesPerRow = [weakRawOutput bytesPerRowInOutput];

            CVPixelBufferRef pixelBuffer = NULL;
            OSStatus result = CVPixelBufferCreateWithBytes(kCFAllocatorDefault,
                                                           1920,
                                                           1080,
                                                           kCVPixelFormatType_32BGRA,
                                                           outputBytes,
                                                           bytesPerRow, nil, nil, nil,
                                                           &pixelBuffer);

            CMVideoFormatDescriptionRef videoInfo = NULL;
            result = CMVideoFormatDescriptionCreateForImageBuffer(NULL, pixelBuffer, &videoInfo);

            CMSampleTimingInfo timingInfo = kCMTimingInfoInvalid;
            timingInfo.duration = frameDuration;
            timingInfo.presentationTimeStamp = nextPTS;

            //NSLog(@"TIME: %f", CMTimeGetSeconds(timingInfo.presentationTimeStamp));

            CMSampleBufferRef sampleBuffer = NULL;
            result = CMSampleBufferCreateForImageBuffer(kCFAllocatorDefault,
                                                        pixelBuffer,
                                                        true, NULL, NULL,
                                                        videoInfo,
                                                        &timingInfo,
                                                        &sampleBuffer);
            NSLog(@"BUFFER : %@", sampleBuffer);
            //THIS IS WHERE I SEND THE BUFFER TO THE ENCODER
           ** [safeSelf.broadCastRecorder captureGenerateVideoOutput:sampleBuffer];**
            //Increment presentation time
            nextPTS = CMTimeAdd(frameDuration, nextPTS);
            // release the copy of the sample buffer we made
            CFRelease(sampleBuffer);
}];

And my custom captureGenerateVideoOutput:CMSampleBufferRef method that I integrated in my Recorder class. that is pretty much a clone of the KFRecorder.h/m
-(void) captureGenerateVideoOutput:(CMSampleBufferRef)sampleBuffer{
    if (!_hasScreenshot) {
        UIImage *image = [self imageFromSampleBuffer:sampleBuffer];
        NSString *path = [self.hlsWriter.directoryPath<EMAIL_ADDRESS>        NSData *imageData = UIImageJPEGRepresentation(image, 0.7);
        [imageData writeToFile:path atomically:NO];
        _hasScreenshot = YES;
    }
    if(_h264Encoder)
        [_h264Encoder encodeSampleBuffer:sampleBuffer];

}

Did you ever figure this one out? I am frustrated over having the same problem.

Dont use their service, not worth it. You should try Red5Pro its way more supported and with more features.
On Feb 8, 2017, 1:53 PM -0800, jonasandero<EMAIL_ADDRESS>wrote:

Did you ever figure this one out? I am frustrated over having the same problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/Kickflip/kickflip-ios-sdk/issues/43#issuecomment-278473695), or mute the thread (https://github.com/notifications/unsubscribe-auth/AA8vweMVp7DLS-a3CximRr1KFj_IUw6mks5rajlPgaJpZM4GTbaD).


Siri "wasn't able to find anything like that this time"

Pre Steps
activate Siri, say command

Expected Behavior

Execute command
Current Behavior

Siri responds with "Sorry, I wasn't able to find anything like that this time"
Steps to Reproduce (for bugs)

Versions

Homebridge: homebridge --version
Node:       node --version
OS:

Configuration

Homebridge Log / Command Output

I've had homebridge for several months, maybe a year, and it used to work well with Siri.  Now it only works in the Home app or from the swipe-up UI.  Nothing I say to Siri ever works with my homebridge and this has been happening for the last couple of months.  I see my cachedAccessories file hasn't been modified since October, would it be safe to rename or delete that and restart homebridge?  Or do you have any other ideas why Siri won't control anything and yet everything works as expected when controlling on screen?

Well, are you using a HomePod?

no, the only apple device I have is an iPhone

Some more info on this issue.  I added a new device to homebridge this week, a wifi thermostat (homebridge-tcc)  Siri works with the new device, but nothing else.  Everything else works perfectly in the Home app.
Would emptying the persist directory possibly help?

I was having this problem this morning.  It was working yesterday.  None of the usual fixes for things not responding was working (i.e. turn airplane mode on/off, restart the phone, restart the RPi, restart the homebridge server).  Just like you, the buttons in the Home app worked, but Siri kept giving that response, which is not one I'd specifically seen before.
It was affecting both my and my wife's (who was invited to the home/room) iPhone 6s.
What finally worked was this (I don't know if all these steps are necessary):

Stop homebridge pm2 stop homebridge
Remove the persist directory (I moved it just in case: cd .homebridge;mv persist ..)
Start homebridge pm2 start homebridge
In the home app, hard-press an accessory, tap details, tap bridge, tap "Remove bridge from home"
Re-scan the QR code that's in the log by tailing the log (tail -n 100 /home/pi/.pm2/logs/homebridge-out.log) and tapping "Add Accessory" in the home app
Turn airplane mode on and back off

It then took me a few tries, but everything started working again via Siri.
The only thing that had changed between today and yesterday was I had added a breadboard to my RPi and installed some stuff to be able to control some 433Hhz RF transmitter/receiver chips.  That involved a few installs.  I had also changed the noobs setting to boot into Raspbian and created a couple aliases in my .profile.  Among other things, I had run an RF sniffer program and a program I wrote to control an LED.  The breadboard and the RF receiver are still connected.  I was about to start working on the transmitter when I discovered Siri wasn't working.

Thanks @hepcat72 I'll give that a try.  I have tried deleting/moving the persist directory.  But I'll try the steps you listed.

Incidentally, I don't know if you use pm2 to start/stop your homebridge server.  My guess is that those steps aren't necessary, but it will put the QR code at the bottom of your log.  However you usually start/stop homebridge is how you should do it.

And I deleted my saved persist directory out of my home dir once I confirmed it was working again.

Did that work for you? I just added a plugin called homebridge-commander and the one outlet I set it up to control refuses to work with Siri.  It works great in the Home app, but I still get the error you're getting.  I performed my trick above a few time and although it worked with my previous issue, it doesn't seem to fix this issue.

I have a similar issue but it’s with the homebridge-openhab plugin that I’ve been using for years.

I have also had this issue. However I have had interesting results using the room name + device name PLUS descriptor (such as kitchen can lights vs kitchen can) in my Siri voice commands.
I have also gotten lights turned off, albeit all of them by referring to the room name, as in kitchen lights off.
Seems like the requirements to identify a device have been tightened up. It would be nice to know exactly how though.
B

On Apr 5, 2019, at 4:29 PM, gijoecool<EMAIL_ADDRESS>wrote:
I have a similar issue but it’s with the homebridge-openhab plugin that I’ve been using for years.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I also just started having this issue.  Nothing has changed with my setup.  I can control everything within the Home app but Siri "wasn't able to find anything like that at this time" every time I ask her to do something.  Interestingly, if I say "turn on the office" it will turn on my office light, but "turn on the office light" doesn't work.  Seem's like this is a bug on Apple's end.

I've tried deleting accessories and persist directories and that didn't resolve the issue.  I then also deleted the HomeBridge bridge from the Home app and changed the username and pin values in the config before adding it again.  Siri still is not controlling most of my devices.  Maybe I'll try add scenes since @joe248 said that worked for him.

I also just started having this issue.  Nothing has changed with my setup.  I can control everything within the Home app but Siri "wasn't able to find anything like that at this time" every time I ask her to do something.  Interestingly, if I say "turn on the office" it will turn on my office light, but "turn on the office light" doesn't work.  Seem's like this is a bug on Apple's end.

Yes it seems to be an Apple thing. Normally I was able to say 'Turn on speakers', but now I've to say: 'Turn on the speakers switch'. Also with light switches, previously I could say 'Turn on ', now I've got to say 'Turn on  light'. Maybe I can make shortcuts with siri shortcuts to make things simple again :P

Just wanted to update that this issue started happening again for me about a week ago.  Not sure what to do because constantly deleting my persist and cachedAccessories and re-adding everything isn't viable.

@joe248 I just started having the same issue, but the fix came in @unconnected's solution. I have a cmd switch to turn on my receiver. I used to be able to say "Turn on the Airplay Input" but now I have to say "Turn on the Airplay Input switch". This only seems to be affecting my Homebridge devices, as I can still say "turn on the China Cabinet" which is a native Homekit switch and it finds it fine.
If I had to guess I'd say this isn't Homebridge specifically but all hubs since I found other people complaining of the same issue with Hue and other hubs. Siri is now requiring more information to find the specific device.
The weirdest thing to me is that Siri is clearly recognizing to device because it is capitalizing the name (i.e. it displays my speech as Airplay Input or Floods Toggle instead of airplay input or floods toggle). Yet it still won't work with out saying "switch" or "lock" or whatever device I'm trying to control. Really frustrating since this is a big step backwards in language understanding capabilities, even if it is (for me at least) only an issue for things through Homebridge.

@joe248 I just started having the same issue, but the fix came in @unconnected's solution. I have a cmd switch to turn on my receiver. I used to be able to say "Turn on the Airplay Input" but now I have to say "Turn on the Airplay Input switch". This only seems to be affecting my Homebridge devices, as I can still say "turn on the China Cabinet" which is a native Homekit switch and it finds it fine.
If I had to guess I'd say this isn't Homebridge specifically but all hubs since I found other people complaining of the same issue with Hue and other hubs. Siri is now requiring more information to find the specific device.
The weirdest thing to me is that Siri is clearly recognizing to device because it is capitalizing the name (i.e. it displays my speech as Airplay Input or Floods Toggle instead of airplay input or floods toggle). Yet it still won't work with out saying "switch" or "lock" or whatever device I'm trying to control. Really frustrating since this is a big step backwards in language understanding capabilities, even if it is (for me at least) only an issue for things through Homebridge.

I have an iPhone 4s that I keep plugged in and treat it like a smart speaker.  I noticed that I can tell it "turn on (or off) the lights", but my iPhone 6s wants me to say "turn on (or off) the lights outlet".  However, one time, I forgot and left off the "outlet" on the command to the 6s... and it worked!  Subsequent tries did not work - I had to specify "outlet".
Lately, Siri has been complaining that she tried but couldn't execute my command.  Sometimes the device does turn on or off.  Sometimes, it doesn't.  In both cases, Siri doesn't think it worked.  Sometimes, I can issue the command a second time and it will work.  Other times, it doesn't matter how many times I issue the command... it never works!  But in those cases, I can go to the home app and use the button and that always seems to work.
I think the problem may be Siri.  Siri recently forgot who my dad was and then when I fixed that, I would say "call my dad mobile" (as I always do), and Siri would ask "Which phone number for...".  Then, when I tapped "mobile", she would ask me again in a never ending loop...  I have a screen recording of that behavior.
So in my opinion, Siri has Alzheimer's.  She's pretty fragile.

I've started to have the same issue since iOS 12.2.
Before it would understand the abreviated comand frase example "Turn off the FAN" but now I have to specify the exact device name and the room were it's located "turn off the HRV FAN in the basement" and I only have that one FAN

I agree - it's very weird that Siri clearly recognizes my devices (because she capitalizes them as you mentioned) but she refuses to control them without using very precise language.  I also agree that the problem is most likely Siri, however, it's also strange that nuking my Homebridge / Homekit setup and starting from scratch does allow things to work the way they used to for some brief period of time.

Just ran into this issue, I can control most things just fine with "Turn on/off X" but my new lock will only work if I saw "Lock/Unlock Front Room Front Door" but if I just say "Lock/Unlock Front Door" it will do the "wasn't able to find anything like that this time". Very frustrating, I really don't want to re-setup all my devices in all my rooms but I might try that tomorrow.

I found this solution and it seems to work for me: https://discussions.apple.com/thread/250239027

@gijoecool thanks for posting this.  I just renamed one of my switches, used Siri to control the switch under the new name, then renamed it back.  The problem appears to be fixed now for all of my devices, at least for the time being.

@gijoecool Interesting find! I was able to rename mine to “Entrance Door” (from Front Door) and Siri started working but as soon as I renamed it back to Front Door I had the same issue. It makes me assume this is a “Siri” issue not a “HomeBridge” issue. That said I hope this thread stays open so we can discuss fixes here (much better than Apple Forums).
Also in case it helps anyone I just uses scenes called “Lock Door” and “Unlock Door” as a work around for right now (my door is the only thing experiencing this issue). This is far from an ideal solution but I thought I’d throw it out there.

For me, the problem comes and goes over time. I discovered this by accident by forgetting to add “outlet”. I agree that this seems to be an issue with Siri/iOS.

Yeah, this is definitely a Siri issue. I switched away from Homebridge with hopes of resolving this issue, but it’s followed me to Home Assistant. Hopefully it gets fixed in iOS 13.

Odd that Siri understands what I say according to the speech to text on screen, even showing device names as they are in "Home," and was previously able to process the commands, but seemingly out of the blue stopped.

FYI, I’ve been getting this error a lot lately. This morning, I changed the room name that a switch was assigned to and it started being able to control the “space heater” whereas before, I had to say “space heater switch”.

Perhaps we should find a forum where we can talk about the issues with Siri?  This morning, Siri couldn’t find anything like my garage door. I made a screen recording of it in action. Eventually, I removed the garage door from the “room” it was in (named “back yard”) and created a copy of the room. When I put it in the new room, Siri worked.
https://www.dropbox.com/s/cswd4sq1tapg0ma/IMG_2633.TRIM.MOV?dl=0

I'm simply using wemo switch on homekit (no homebridge at all) yet. Which leads me to believe it is not a homebridge issue at all.  It happened to me the same way, just out of no where, I was not able to use siri, homekit app still worked fine though.
I think it started roughly after a device updated its firmware.
I simply re-named the device to something else, and it worked.  Then re-named it back and it was working fine after that.

I also just started having this issue. Nothing has changed with my setup. I can control everything within the Home app but Siri "wasn't able to find anything like that at this time" every time I ask her to do something. Interestingly, if I say "turn on the office" it will turn on my office light, but "turn on the office light" doesn't work. Seem's like this is a bug on Apple's end.
Edit: I just deleted my persist directory and cachedAccessories file. I then re-added Homebridge and all 30 of my accessories, which was a pain, but Siri control is working correctly now.

This "Siri not understanding some of my homekit accessories" is happening for me on all my IOS devices (iPhones and iPads) but not on the macOS (Siri on my iMac understands my command perfectly) - This is strange as I use the same apple ID across all my Apple devices and the Home app on all devices works fines as well.
I'm reluctant to delete my persist directory and cachedAccessories file as I have over 100 accessories across multiple Homebridges. Re-adding all my Homebridge accessories will be a real pain :-(
Anyway, since the issue does not occur on the macOS maybe it's an iOS bug and probably not related at all with homebridge

I restarted my Apple TV 4 (which is configured as the Homekit hub for remote access) and it fixed the issue. :-)
I would never have guessed that it was the Apple TV all along.
This is really strange since this issue was occurring on my local network and only on iOS devices.
Anyway, I'm happy to say that the solution was very simple after all
Go figure ;-)
P.S. I guess this is an Apple Homekit bug and not a Homebridge issue

Something that might be helpful to others with this issue: I had to delete my entire house and re-create it in order for Siri to start to recognize my accessories again.

Try renaming the devices in Homekit to something different. Verify that it works again and name it back. Worked for me.

Sure, that can work once, but it will start happening again at some point.

Try renaming the devices in Homekit to something different. Verify that it works again and name it back. Worked for me.

Renaming the whole Home also fixes the issue, and you don't need to rename every single device.

Sounds like a HomeKit Naming issue, closing issue.


Spades: Have bot bid Nil if it would have bid 1 with 2-card pass

Why bot bid one on 2 card pass? If you only got one then bid nil and pass it

We had another customer report an issue that appears to be due to this behavior over the weekend.

Just noticed this is the same as #39. Moving comments there so we can properly track how much feedback we're getting on this (lack of) behavior.


No related songs found

The radio feature stopped working for me. When the last song in the playlist is done playing, or I click the next song button, nothing happens. I looked at the console and I did get an uncaught error:
Uncaught Error: No related songs found

Followed by the JSON object of the sounds in the queue. It doesn't matter if I have 1 song in the queue or many songs, I still do not get any related songs.
Any idea if this is a bug on my end (maybe too many Youtube requests?) or a bug somewhere else in the code?

YouTube has presumably revoked Streamus' API key as they noted they
would at 5PM on Friday for non-compliance issues.
On 7/17/15, Ivan Montiel<EMAIL_ADDRESS>wrote:

The radio feature stopped working for me. When the last song in the playlist
is done playing, or I click the next song button, nothing happens. I looked
at the console and I did get an uncaught error:
Uncaught Error: No related songs found

Followed by the JSON object of the sounds in the queue. It doesn't matter if
I have 1 song in the queue or many songs, I still do not get any related
songs.
Any idea if this is a bug on my end (maybe too many Youtube requests?) or a
bug somewhere else in the code?

Reply to this email directly or view it on GitHub:
https://github.com/MeoMix/StreamusChromeExtension/issues/612

Oh no! So we can only play songs that we have already added to our playlists? It looks like we can't search either. Bummer. I guess I can use developer mode + my own API key? Did you ever find out if playing the Youtube video in the tab view would suffice?

I suspect it will suffice, but YouTube has ceased communicating with me so I'm just going to have to upload it and find out.

I feel for you, all of that work separating and decoding the music from the video is going to be for naught. Good luck, hopefully I'll have time this weekend to delve into your code.