Minimum Viable Schema Protocol

TL;DR
I want to propose a "minimum viable protocol" for libraries to accept schemas as part of their API without depending directly on Valibot (or another library such as Zod)
// Protocol:
export let parseSymbol: unique symbol = Symbol.for("https://github.com/fabian-hiller/symbol-parse")
/** @throws {unknown} If the input is rejected. */
export type Parse<T> = (input: unknown) => T
export type Schema<T> = { [parseSymbol]: Parse<T> }

// Example library:
export async function fetchJson<T>(endpoint: string, schema: Schema<T>): Promise<T> {
  let response = await fetch(endpoint)
  let json = await response.json()
  try {
    // Call `parse()` on some unknown data and get the schema's output type
    return schema[parseSymbol](json)
  } catch (error: unknown) {
    // Catch whatever error was thrown by the schema
    throw new Error("Unexpected data", { cause: error })
  }
}

Background
I'm building a library that accept a Valibot schema's as part of its API.
import { sql } from "lib"
import * as v from "valibot"

function getUserAge(userId: string): number {
  return sql`select age from user where id = ${userId}`
    .pluck(v.number())
    .one()
}

For the purposes of this library, I don't really need to do much of anything with Valibot itself. I'm not creating my own schemas and I just want to give users nice typings and convenient APIs when using their own schemas.
To make this work, I just need to be able to call Valibot's parse(schema, input) method.
Problem
It would be nice if I didn't have to pull in Valibot as a dependency just for its most basic functionality. It would then mean I also have to keep it up to date for major versions.
It would also be nice if I could just say that I accept several different schema libraries.
Solution
If Valibot could commit to supporting (across major versions of Valibot) a well-specified, minimum viable protocol similar to that of Promises .then, Iterators [Symbol.iterator], or Observables [Symbol.observable], then library authors could depend just on the protocol and not need to pull in Valibot as a dependency.
It would also make these libraries generic across any other schema library that wanted to implement the protocol.
There could be a package similar to https://github.com/benlesh/symbol-observable that just exposes a shared symbol:
export let parseSymbol: unique symbol = Symbol.for("https://github.com/fabian-hiller/symbol-parse")
/** @throws {unknown} If the input is rejected. */
export type Parse<T> = (input: unknown) => T
export type Schema<T> = { [parseSymbol]: Parse<T> }

Then in libraries they can just make use of it in their types:
import { parseSymbol, Schema } from "symbol-parse"

function parse<T>(schema: Schema<T>, input: unknown): T {
  return schema[parseSymbol](input)
}

Why "minimum viable" protocol?
Asking lots of developers to coordinate around a shared protocol can be like herding cats. A larger footprint becomes harder to come to consensus on and asks more of everyone who interacts with it.
For this purpose I suggest:

No separate Input or Output types

I don't think this would be particularly useful for libraries that want to be generic over multiple schema libraries/versions.

No Issues or other error/rejection types

If libraries really want to introspect the issues types, they should just depend on a schema library directly

No safeParse() or SafeParseResult

If Issues isn't typed, there really isn't much of a purpose for this at all. Library authors can just use try-catch (if they even need to) and that helps reduces the scope of this protocol.

In general, I welcome such ideas. However, the main problem for Valibot will be that following a protocol will unnecessarily increase the bundle size for all users not using that protocol.
As a workaround for now. Valibot provides a parser and safeParser method that returns a function. If your users wrap their schemas in parser or  safeParser, you can run them directly without adding Valibot as a dependency. Another workaround that many form libraries use is adapters and resolvers.
import * as v from 'valibot';

const parseNumber = v.parser(v.number());

const number1 = parseNumber(123); // returns 123
const number2 = parseNumber('123'); // throws error

I suggest you also have a look at TypeSchema, Standard Schema and this discussion.

I would be surprised if this had much of a real world impact, it's not like Valibot gets used without parse()/safeParse() (which are called by parser()/safeParser()) anyways which already includes the code for global config and such. If you abstracted it away to a function call in all of the schema factory functions, it's not much more code than the parse functions that every user has to include anyways.
export function string() {
  return defineSchema({ ... })
}
// or
export let string = defineSchema(() => {
  // ...
})

You honestly might want to do something like that with these factories anyways because you could introduce some caching which could improve the performance of Valibot a lot.

It makes a difference because parse needs to import ValiError and if people only use safeParse this code will never be used. It is true that the real world impact may be small, but it still feels wrong to me because it goes against the philosophy of our API design and implementation. If all the other libraries follow such a specification, Valibot will probably follow too, but Valibot is the wrong library to start such an initiative.

You honestly might want to do something like that with these factories anyways because you could introduce some caching which could improve the performance of Valibot a lot.

Create idea! I will investigate this as part of #572.

If such a proposal instead was an equivalent of safeParse() and specified a return value of:
type Issue = { path?: PropertyKey[], message: string }
type Result<T> =
  | { ok: true, result: T, issues: void }
  | { ok: false, result: void, issues: Issue[] }

export let parseSymbol: unique symbol = Symbol.for("parse")
/** @throws {unknown} If the input is rejected. */
export type Parse<T> = (input: unknown) => Result<T>
export type Schema<T> = { [parseSymbol]: Parse<T> }

Would that be more acceptable?

Yes, but supporting this format will probably add additional code besides safeParse, which will increase the size of the bundle even more. I support your idea, and we should discuss it as a community, but Valibot will probably adopt it later than other libraries due to our focus on bundle size and modularity.

Yeah, I'm discussing it with you now to understand what version of this protocol you'd accept. It's possible that it could include things that reduce your implementation even further:

Allow additional properties on Result and Issue

Simplifies the implementation so you don't need to map your names of properties to the spec's, you could just use the same names and return the full object you need.

Don't require an ok/success/valid parameter on Result ({ value: T, issues?: Issue })

You do this in Valibot by mapping success: !dataset.issues
Consumers of the protocol could do the same

Together this would mean the implementation of this spec is no more than:
let decorate = schema => {
  schema[parseSymbol] = input => {
    return schema._run(input, getGlobalConfig())
  }
}

// or minified (maybe code-golf-able further in context)

let d=s=>s[p]=i=>s._run(i,c())

Or going even further, if you wanted to be able to just replace _run with a symbol (or whatever the spec wants to use for its property:

Allow additional parameters to the parse() method

This would allow you to accept config as an optional second argument and even call getGlobalConfig()

That would make this the full change in Valibot:
  export function number(
    message?: ErrorMessage<NumberIssue>
  ): NumberSchema<ErrorMessage<NumberIssue> | undefined> {
    return {
      kind: 'schema',
      type: 'number',
      reference: number,
      expects: 'number',
      async: false,
      message,
-     _run(dataset, config) {
+     [parseSymbol](dataset, config = getConfig()) {
        if (typeof dataset.value === 'number' && !isNaN(dataset.value)) {
          dataset.typed = true;
        } else {
          _addIssue(this, 'type', dataset, config);
        }
        return dataset as Dataset<number, NumberIssue>;
      },
    };
  }

Which minified is a negligible amount of bytes other than getConfig() which always has to be used in Valibot since its included in all of the parse methods.

Wrote a proposal over here: https://github.com/standard-schema/standard-schema/issues/3

Thank you! The [parseSymbol](dataset, config = getConfig()) ... change looks much better for Valibot. One thing to note is that dataset contains more info then the raw input. So I am not sure if this will work the same way with other schema libraries.

Wrote a proposal over here: https://github.com/standard-schema/standard-schema/issues/3

Thanks! I will have a look at it.

I think I can close this issue. Further discussion should take place in the Standard Schema repo: https://github.com/standard-schema/standard-schema


Clarification on Configuration with ECS

Trying to add the OTEL Collector and Emitter as sidecar containers to our Java app to be able to do tracing in AWS X-Ray using the instructions here. I wasn't sure what the difference between the default config and task metrics config was, so I went with the default config. Now, In X-Ray, I see an application called "SampleServer" running on port 8000. There was nothing in X-Ray before, so I assume it's picking our application up, but our application runs on a different port.
How can we specify specific configuration for our app using the sidecar container? My understanding was that we would not need to modify our application in any way for this to work, but that may be wrong.

I wasn't sure what the difference between the default config and task metrics config was
The default config is to setup/enable the application metrics and x-ray tracing in the collector for your application services.  And task metrics is to monitor the infrastructure metrics of your ECS task like network, memory metrics, etc.

Our sample setup has also provide a sample-app example running together with OTel Collector in ECS Task, that's why you see SampleServer running on port 8000. You can replace your own application docker image there and
The sample app code is here.
https://github.com/JohnWu20/aws-xray-sdk-go-sample
You can also reference our Java Sample app here which emits Metric and traces data.


introspect fetch queries before they are run

I'm authoring a library that makes json-api compatible apis using Bookshelf. At the moment, this requires some gross contortions to unnest any queries that include relations. You can see an example app here. I'd like to implement a way to get a handle on all the query builders that would be used when calling fetch({withRelated:[...]}), that way I can call them myself in endpoints-source-bookshelf, thus avoiding the un-nesting step. I'm happy to do all the legwork for the implementation (it's high time I started contributing directly to this project), I just want to makes sure you're down with the idea, and to hear your thoughts on what you think the API for this should look like.
buildFetch({withRelated:[...]}?
fetch{{withRelated:[...], preflight: true})?
Something else?

I guess you could already do something like this with a single model using related, but you can't for collections.

annnd nevermind again, i guess i can use load.

...a

The project leadership of Bookshelf recently changed. In an effort to advance the project we close all issues older than one year.
If you think this issue needs to be re-evaluated please post a comment on why this is still important and we will re-open it.
We also started an open discussion about the future of Bookshelf.js here https://github.com/bookshelf/bookshelf/issues/1600. Feel free to drop by and give us your opinion.
Let's make Bookshelf great again


Create BaseYii stub

@AdeAttwood I take that code from package https://github.com/proget-hq/phpstan-yii2

This code is good as is, just needs some tests. @WinterSilence would you like me to finish this PR off and push the changes to WinterSilence/PsalmPluginYii2

@AdeAttwood  don't understand you

@AdeAttwood don't know how test stubs, but if is work fine in phpstan plugin it's must work fine in psalm plugin too

but if is work fine in phpstan plugin it's must work fine in psalm plugin too

Unfortunately to often this is not the case. I will get some test done and get this merged in the next couple of days.

@WinterSilence thanks for this, sorry for the delay it has been put in with #9 with the tests. If you want to see how the tests are done for the stubs, then you can see them here


Unescaped meta tag content breaking the HTML

Issue moved from MicrosoftDocs/feedback#2829

Please respond to @aptwebapps.

From @aptwebapps on Sunday, June 7, 2020 11:47:49 AM
Describe the bug
The description meta tag's content is not escaped and is creating invalid HTML
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/dotnet/api/xamarin.forms.bindableobject?view=xamarin-forms
See meta content along with garbage tag close displayed at top of page

Expected behavior
Meta tag content should not be HTML
Screenshots

Desktop (please complete the following information):

OS: MacOS
Browser: Firefox
Version: 77

Smartphone (please complete the following information):
Untested
Additional context
None

Issue moved from MicrosoftDocs/feedback#2829

Please respond to @welcome[bot].

From @welcome[bot] on Sunday, June 7, 2020 11:47:50 AM
Thank you for creating the issue! One of our team members will get back to you shortly with additional information. If this is a product issue, please close this and contact the particular product's support instead (see https://support.microsoft.com/allproducts for the list of support websites).

reported also here:
https://github.com/xamarin/Xamarin.Forms-api-docs/issues/197
https://github.com/xamarin/Xamarin.Forms-api-docs/issues/199

Duplicate of #197


cached_query_torsion_drive_results fails for multi-dimensional grid ids

If a TorsionDriveResultCollection contains TorsionDrives with multi-dimensional grid indices, cached_query_torsion_drive_results will fail here with ValueError: could not convert string to float.
The TorsionDrive grid indices are stored as stringified tuples such as '(-165)'. For multi-dimensional TorsionDrives, the grid indices look like '(-165, -165)' so that float(x[1:-1]) fails. I think something like
        grid_ids.sort(key=lambda s: tuple(float(x) for x in s[1:-1].split(", ")))
will work for both one-dimensional and multi-dimensional grid indices.
I have this patch in a new branch that I'm testing now. Any ideas for a solution that handles this problem more elegantly?

Thanks @chapincavender! Can you make a PR with your change? I think your approach is close to what I might do, but I'd probably go with:
grid_ids.sort(key=lambda s: tuple(float(x) for x in s.strip('()').split(',')))

This avoids any assumptions on slicing.

Fixed in #202. It looks like the grid indices are stringified lists instead of stringified tuples as I stated above, so I went with split("[]") instead of splicing.

Closed via #202 and released in version 0.3.2


Resolve Ctrl+Shift+V default shortcut conflicts

Just extra information: the Ctrl+Shift+V was previously set by default for Close All Windows, and it was conflicting with the new Clipboard history feature. Since Ctrl+Shift+V is standard in other IDEs for the clipboard history we made the same here and redefined Close All Windows to Ctrl+Alt+W.


Application on join

What is your username and discriminator?
8-Ball#1862
Why did you join Discord Linux?
Cause I love linux and want to discuss tech stuff on them with other people!
Where did you hear about Discord Linux?
From google. I've been searching for much time to find you!
What is the passphrase that was assigned to you by the bot?
sudo -i trial-turkish-thong-ruthless

From Ethan Weegee :
The passphrase you have entered is not correct.  Please check the message that mentions you in the welcome channel to find your passphrase.

My apologies for that!
sudo -i liver-gainful-bottom-setting
I believe this is the one!

From Ethan Weegee :
Looks good.  I've given you a temporary role so that you can enter the passphrase in the welcome channel.


Add jagadeeshi2i to Kubeflow org

Hi All, I'm adding myself (jagadeeshi2i) to the kubeflow org in this PR.
I've mostly been involved in KFServing torchserve integration, and I'm likely going to be focusing on contributions to KFServing.
Contributions:
kubeflow/kfserving#1156, kubeflow/kfserving#1161, kubeflow/kfserving#1269, kubeflow/kfserving#1268, kubeflow/kfserving#1265, kubeflow/kfserving#1264, kubeflow/kfserving#1263, kubeflow/kfserving#1252, kubeflow/kfserving#1228, kubeflow/kfserving#1220, kubeflow/kfserving#1219, kubeflow/kfserving#1202, kubeflow/kfserving#1186, kubeflow/kfserving#1185, kubeflow/kfserving#1182
Sponsors:
@yuzisun

Output of test_org_yaml.py:
======================================================================= test session starts =======================================================================
platform linux -- Python 3.8.5, pytest-6.0.2, py-1.10.0, pluggy-0.13.1
rootdir: /home/ubuntu/Repositories/kubeflow/internal-acls/github-orgs
plugins: tornasync-0.6.0.post2, xdist-2.2.0, forked-1.3.0, rerunfailures-9.1.1
collected 1 item

test_org_yaml.py .                                                                                [100%]

======================================================================== 1 passed in 0.27s ========================================================================

welcome @jagadeeshi2i ! Jagadeesh has been heavily contributing to the KFS/TorchServe integration, please help add him to the kubeflow org.
/assign @Bobgy

welcome @jagadeeshi2i ! Jagadeesh has been heavily contributing to the KFS/TorchServe integration, please help add him to the kubeflow org.
/assign @Bobgy

/lgtm
/approve
Welcome to Kubeflow!

/lgtm
/approve
Welcome to Kubeflow!


Console Logs of Scrolling

console.log($(window).scrollTop());  is present in app.js which logs out numbers according to our position in the window during scrolling.
Was it supposed to be there?

I believe that was used for the purpose of debugging. Feel free to raise a PR and remove it

I believe that was used for the purpose of debugging. Feel free to raise a PR and remove it

#912 FIXED.


chore(payment): INT-6554 Bump to ^1.288.0

What? INT-6554
Bump to 1.288.0
Why?
Update code with this approved PR https://github.com/bigcommerce/checkout-sdk-js/pull/1588
Testing / Proof
...
bigcommerce/checkout @bigcommerce/apex-integrations

Hey @roberbnd there is a new bump for SDK 1.288.1, can you please update this PR with the version. Thanks

Already released as part of a later bump.


[WIP Tracker] Update to 1.19

Description
Slowly working on porting to 1.19. Will not build let alone run yet.
Waiting for us to be able to run on latest forge with registrate before merge.
Fixes #59
Todo

[x] Buildable
[x] Runnable
[ ] Fix issues
[ ] Tidy the new code
[ ] Review old practice, make sure we're following standards.
[x] Wait for PR's to be merged
[x] Fix merge
[x] Upgrade to latest forge
[x] Wait for registrate
[ ] Wait for registrate to update past fluid rework
[x] New codesplitting technique

Issues:

[x] Energy and progress tooltips don't display numbers
[ ] Fix door datagen MinecraftForge/MinecraftForge#8687
[x] Check datagen is fixed in EIODecor once we bump past forge 41.0.41

Note: this PR got big, so some of the 1.19 porting stuff will continue in future PRs now. Just waiting for the Registrate PR then we'll merge.
Checklist:

[x] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Gonna go for merging this soon, waiting for the build to hopefully pass first


Remove texture reference completely

Related to #7232. This PR completely removes any APIs related to texture reference, which has been marked as deprecated since forever (?) and is finally removed in CUDA 12+.

/test mini

Sorry, I had wrong understanding. The texture reference APIs from CUDA runtime are removed in CUDA 12.0, but those from CUDA driver (that we use) are intact. Nevertheless, I still think it's good to be slated for removal in order to be in preparation for a future CUDA change and suppress build-time warnings. Functionality-wise, texture (and surface) objects are the way to go.

btw @kmaehashi @emcastillo this is technically a breaking change, but I seriously doubt there's any user... Just FYI for planning

yeah, we are not very sure about what to do with this and wanted to discuss it with you

That's what I figured 🙂 My justification is that there's no public project using get_texref(), which is the last API in the pipeline for utilizing texture references, so I assume the impact is very small:
https://github.com/search?q=cupy+get_texref+-org%3Acupy+language%3APython&type=code&l=Python
The other justification is we've documented it since the beginning that TextureReference is deprecated and subject to removal. Since all capabilities are also coveted by TextureObject and the TextureReference API was designed to mimic that of TextureObject (on purpose), I think even if there's user code broken it is at-most a one-line fix.


Type hints

Should we (gradually) introduce type annotations? In recent python versions you can do:
def sum(a: int, b:int): -> int
    return a + b

and a tool such as mypy or pylance will check your code for violations while you're typing, often
pointing out bugs based on expectations of what another component will do.
My proposal is that from henceforth, contributors who want to, may add such annotations.

This would mean dropping support for python 3.6 which will be EOL in 6 months: https://endoflife.date/python

All in favour of type annotations here. Are there any important libraries that are still 3.6 only?

That's a good question. From what I can find; numpy, scipy, matplotlib, scikit-learn and scikit-image all follow NEP 29, a policy which requires them to support recent python versions and drop support for versions older than 42 months.
For our depencies that leaves joblib, h5py (python versions not listed but uses 3.7 for their CI) and tqdm, which all seem to support 3.7.

Also just noticed that numpy added type annotations very recently in release 1.20, February 2021. Very useful for us given number of numpy arrays going around.


Redeploying only hello function fails with an error

I'm testing that plugin and I have issues just following the tutorial. The first issue I met is regarding deploying to a namespace different from default https://github.com/serverless/serverless-kubeless/issues/188#issuecomment-605328537.
The other issue I have is redeploying the function hello only. When I try I get the following
serverless deploy function -f hello
Serverless: Packaging function: hello...
Serverless: Excluding development dependencies...
Serverless: Redeploying hello...

  Type Error ---------------------------------------------

  TypeError: Cannot read property 'artifact' of undefined
      at KubelessDeployFunction.getPkg (/Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/serverless-kubeless/deploy/kubelessDeploy.js:85:37)
      at /Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/serverless-kubeless/deploy/kubelessDeploy.js:111:28
      at /Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/lodash/lodash.js:4905:15
      at baseForOwn (/Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/lodash/lodash.js:2990:24)
      at /Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/lodash/lodash.js:4874:18
      at Function.forEach (/Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/lodash/lodash.js:9342:14)
      at /Users/cscetbon/src/git/serverless-kubeless/examples/get-python/node_modules/serverless-kubeless/deploy/kubelessDeploy.js:110:11
      at processTicksAndRejections (internal/process/task_queues.js:97:5)

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information ---------------------------
     Operating System:          darwin
     Node Version:              13.8.0
     Framework Version:         1.67.0
     Plugin Version:            3.6.1
     SDK Version:               2.3.0
     Components Version:        2.22.3

The issue from line https://github.com/serverless/serverless-kubeless/blob/master/deploy/kubelessDeploy.js#L85 where description.package is undefined. I'm using the default serverless.yaml provided
I'm using version 0.9.2. I've been looking at that plugin a year ago and thought it was mature, but just following the tutorial fails..

I've met the problem as you too.

Hi @cscetbon , @donhui , replace python2.7 with python3.7. Runtime python2.7 does not exists any more.

Hi @sepetrov , I've replaced python2.7 with python3.7. But the problem still exist.

Hi @cscetbon , @donhui , replace python2.7 with python3.7. Runtime python2.7 does not exist any more.

Hi @sepetrov  Can we create the serverless-kubeless template with python2.7 runtime and then edit it to python3.7?
or Is there any other workaround by which we can use python3 instead of python2 for serverless-kubeless?
Any help/references would be appreciated.
Thanks

Hi @cscetbon , @donhui , replace python2.7 with python3.7. Runtime python2.7 does not exist any more.

Hi @sepetrov  Can we create the serverless-kubeless template with python2.7 runtime and then edit it to python3.7?
or Is there any other workaround by which we can use python3 instead of python2 for serverless-kubeless?
Any help/references would be appreciated.
Thanks


Is variable allow arithmetic operation?

Hi,
Is it possible to do arithmetic operations for variables defined in the model?
Thanks,
Hang

I'm not really sure what you mean. Do you mean forming expressions from variables and then using them to define constraints? If yes, then no that's not supported directly in russcip, but you could use russcip through the good_lp crate with the scip feature enabled to get these ergonomic model building capabilities.

will look into good_lp crate carefully. Thanks!


flutter vscode devcontainer hot-restart issue

I have been trying to perform a hot-reload for my flutter-web project.
Been struggling to get hot restart functionality enabled when running via viscose dev-containers. I tried:
Having the flutter run -d at the entry point.
Executing the debug via the vs-code debugger.
Neither of them solves the hot-restart issue on the browser on which the initial page is rendered after building from the inside container. Please help how to achieve what I am trying. TIA.

Hi @knightswatch3, thanks for filing the issue.
To determine the cause of the issue belongs to VSCode or Flutter, please provide the information indicated below:

The detailed reproduction steps
The output of flutter doctor -v
VSCode version
Error logs from the console if any

Thank you!

@huycozy Here is the requested information:
[root@5cd7c82ba1de flutter-setup]# flutter --version

┌─────────────────────────────────────────────────────────┐
│ A new version of Flutter is available!                  │
│                                                         │
│ To update to the latest version, run "flutter upgrade". │
└─────────────────────────────────────────────────────────┘
Flutter 3.4.0-19.0.pre.296 • channel master •
https://github.com/flutter/flutter.git
Framework • revision bf5d608788 (6 days ago) • 2022-09-15 20:59:35 -0400
Engine • revision d12fe86ba2
Tools • Dart 2.19.0 (build 2.19.0-204.0.dev) • DevTools 2.17.0
[root@5cd7c82ba1de flutter-setup]# flutter doctor -v
[✓] Flutter (Channel master, 3.4.0-19.0.pre.296, on CentOS Linux 8
    5.10.124-linuxkit, locale en_US.UTF-8)
    • Flutter version 3.4.0-19.0.pre.296 on channel master at /flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision bf5d608788 (6 days ago), 2022-09-15 20:59:35 -0400
    • Engine revision d12fe86ba2
    • Dart version 2.19.0 (build 2.19.0-204.0.dev)
    • DevTools version 2.17.0

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /usr/bin/firefox

[✓] Linux toolchain - develop for Linux desktop
    • clang version 12.0.1 (Red Hat 12.0.1-4.module_el8.5.0+1025+93159d6c)
    • cmake version 3.20.2
    • ninja version 1.8.2
    • pkg-config version 1.4.2

[!] Flutter IDE Support (No supported IDEs installed)
    • IntelliJ - https://www.jetbrains.com/idea/
    • Android Studio - https://developer.android.com/studio/
    • VS Code - https://code.visualstudio.com/

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-arm64    • CentOS Linux 8
      5.10.124-linuxkit
    • Chrome (web)    • chrome • web-javascript • Running without a11y
      support!
      Mozilla Firefox 91.4.0esr

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.```

Here is my viscose version:

Version: 1.71.0 (Universal)
Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Date: 2022-09-01T07:25:38.437Z (2 wks ago)
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.6.0
Sandboxed: No```

Hi @knightswatch3, thanks for your response.
But I see there's an open issue addressing the case you described #89721 that proposes an official support Flutter Devcontainer for VSCode. Please also see https://github.com/microsoft/vscode-dev-containers/issues/162. That implies that it is not yet fully supported to handle other aspect issues.
Closing this issue as a duplicate, so please follow up on it for further updates. You can leave there your comment and react an upvote to the original comment will help to raise the issue. Thank you!


Could allow multiple target-text columns

Got this idea because during developing the interact command I realized we could use a multi-select.  This would avoid people having to do repeated runs for one datafile.

This is great!  But I think it would also involve modifying Output so users new which columns their results corresponded to.  So it isn't a super simple task.

This has been determined to be out of scope and is being closed.
If you wish to run multiple columns I suggest simply running the tool twice in a bash script and copying each output file with an appropriate name. Then you can combine the files yourself 😃


[bug]: Incorrectly shown forwarded payments amount

Background
Occationally, forwarded payments are shown with incorrect amount. Unclear when it exactly happens. Observed with different peers, but not consistenly. For some forwards wrong, for others right. It seems to be forward specific.
I guess it is related to formatting of amounts.
Example forward for a channel with zero base fee and 400 ppm : lncli fwdinghistory:
...
        {
            "timestamp":  "1706844342",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "937",
            "amt_out":  "937",
            "fee":  "0",
            "fee_msat":  "374",
            "amt_in_msat":  "937467",
            "amt_out_msat":  "937093",
            "timestamp_ns":  "1706844342485766825",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
...

In Thunderhub and Raspiblitz the forward is correctly shown with amount 937 sats and fee 374 msats.
The same forward is shown incorrectly in lndg (see last row of screenshot, other rows are forwards of the same in/out-peers with unchanged same PPM):

Environment

version of lndg:  v1.8.0
version of lnd:  0.17.3-beta
which operating system (uname -a on *Nix): Raspiblitz Linux raspberrypi 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr  3 17:24:16 BST 2023 aarch64 GNU/Linux

Steps to reproduce
Unclear.
Expected behaviour
Amounts shown as within lncli.
Actual behaviour
Amounts sometimes shown wrong.

The last row here appears to match what you have shown from lncli, the ones above seem odd to me, what do these other records look like from lncli?

Hi @cryptosharks131,
Thank you for checking the issue. Have a closer look, the five "Payments Routed" shown in the screenshot have all the same in-peer and out-peer with unchanged=same fee 400ppm.

1.209 for fee earned 0,483 sats
1.254 for fee earned 0,501 sats
1.370 for fee earned 0,548 sats
1.133 for fee earned 0,453 sats
937.467 for fee earned 0,374 sats

The last row as shown by lncli output is a payment with 937,467 sats but is shown as 937.467 sats.
BR

I do see that, it is odd.  I was hoping to get an example of both a record that looked okay and one that was wrong.
Another good comparison would be seeing what is in the LNDg api data that is feeding the page, you can browse the api data from this url: /api/forwards/
I have not been able to replicate this but I suspect this is some sort of browser language issue when the page is rendering.

Hi @cryptosharks131,
Here a lncli fwdinghistory output for a couple of forwarding records, some of them ok, some of them shown wrong:
{
    "forwarding_events":  [
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "1063",
            "amt_out":  "1063",
            "fee":  "0",
            "fee_msat":  "425",
            "amt_in_msat":  "1063531",
            "amt_out_msat":  "1063106",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "996",
            "amt_out":  "996",
            "fee":  "0",
            "fee_msat":  "398",
            "amt_in_msat":  "996506",
            "amt_out_msat":  "996108",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "1205",
            "amt_out":  "1205",
            "fee":  "0",
            "fee_msat":  "482",
            "amt_in_msat":  "1205614",
            "amt_out_msat":  "1205132",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "1458",
            "amt_out":  "1458",
            "fee":  "0",
            "fee_msat":  "583",
            "amt_in_msat":  "1458742",
            "amt_out_msat":  "1458159",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "1171",
            "amt_out":  "1171",
            "fee":  "0",
            "fee_msat":  "468",
            "amt_in_msat":  "1171585",
            "amt_out_msat":  "1171117",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "874",
            "amt_out":  "874",
            "fee":  "0",
            "fee_msat":  "349",
            "amt_in_msat":  "874436",
            "amt_out_msat":  "874087",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        },
        {
            "timestamp":  "<timestamp>",
            "chan_id_in":  "<in-channel-id>",
            "chan_id_out":  "<out-channel-id>",
            "amt_in":  "880",
            "amt_out":  "880",
            "fee":  "0",
            "fee_msat":  "352",
            "amt_in_msat":  "880440",
            "amt_out_msat":  "880088",
            "timestamp_ns":  "<timestamp>",
            "peer_alias_in":  "<in-peer>",
            "peer_alias_out":  "<out-peer>"
        }
    ],
    "last_offset_index":  7
}

Here the corresponding UI-appearance (within Firefox, default language German)

BR

Thanks for the details! Could you also confirm what these records look like from your LNDg api data? This will help rule out any issue from the backend loading the data and it should match what you see in the lncli responses.
You can filter the LNDg api data from /api/forwards/ or open browser dev tools (F12) on the lndg page you took a screenshot, refresh the page, open the network tab and look for the forwards api call that was made. This will have the records that got loaded onto this page in the response data.
These are the calculation done on the front-end, I don't see anything in particular that might cause the issue
https://github.com/cryptosharks131/lndg/blob/master/gui/templates/channel.html#L523-L525
https://github.com/cryptosharks131/lndg/blob/master/gui/templates/base.html#L20-L32
Does changing your browser language to English give accurate results (although they may not be in the preferred language format)?

Hi @cryptosharks131,
I can confirm that LNDg api data from /api/forwards match those from lncli fwdinghistory. Changing the browser language to English does not give accurate results. Tried it with Chrome/English. Also no accurate result.
BR

I am seeing this same bug on LNDg v1.8.0. I just had three payments routed via the same channel, each with the reported Fees Earned off by a factor of 1000, like so:
In: 885.277 Out: 885.008 Fees Earned: 0.269 PPM Earned: 304
The PPM is reported correctly, but the fees earned should be 269, not 0.269.
It's the first time I've noticed this bug, but I saw it three times in a row.


ridgeback_desktop: 0.1.1-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository ridgeback_desktop to 0.1.1-1:

upstream repository: https://github.com/ridgeback/ridgeback_desktop
release repository: https://github.com/clearpath-gbp/ridgeback_desktop-release.git
distro file: melodic/distribution.yaml
bloom version: 0.8.0
previous version for package: null

ridgeback_desktop

No changes

ridgeback_viz
* Updated laser scan topic.
* Contributors: Tony Baltovski

Holding for melodic sync


Out of Stock Configurable

When I set a configurable product to out of stock, I like for the options and add to cart to disappear, while the 'Notify me when this is in stock' message appears. But I also like for the simple products in the dropdown to show when out of stock for other products.
Is this not possible? If I set 'Show out of stock configurations' to Yes, out of stock simple products in the dropdowns will appear as intended. But entire configurable products that are of stock will still show the dropdown options.
If I set 'Show out of stock configurations' to No, the inverse happens where my configurable products are now showing out of stock, but out of stock simple products in dropdowns disappear.

@chugginn This is right now a limitation of the extension. Actually I think it's a limitation of Magento. I didn't do anything special to show the out of stock configurations. I only set a [flag (provided by Magento) to true](https://github.com/tzyganu/Switcher/blob/master/app/code/community/Easylife/Switcher/Model/Observe[r.php#L58].
I will close this for now because I don't plan to "fix" it, since is't a core "feature".


Allow custom location of environment file

Currently the environment file .merlinenv has to be located in the user's home directory. This is somewhat inconvenient in some cases.
The current PR adds support for using an optional environment variable MERLIN_ENV_PATH pointing to the location of the .merlinenv file and fails back to the user's home if not present.

Hello @eddienko! Thanks for opening this PR.

In the file merlin/__init__.py:

Line 10:81: E501 line too long (93 > 80 characters)

Hello @eddienko! Thanks for opening this PR.

In the file merlin/__init__.py:

Line 10:81: E501 line too long (93 > 80 characters)


Help changing load order

Hello! I am using the osu cat for other games, and i was wondering if it was possible to change the load order for smoke and up, so that up isnt on top of smoke? its causing a double image the way i have it set up,,

Can you check the latest release and see if it works?


[DOCU-1837] Fix broken links

Summary

Fixing various broken links in the docs
Based on the report, adding redirects for common errors and broken links that live outside the doc site
Enabling the link checker to test /hub/ and /community/ links - most of the reported broken links are for /hub/, and for whatever reason, we have that disabled.

Ran the link checker locally after enabling it on /hub/, caught a bunch more broken links, fixed those

Note to reviewers: the jekyll redirect_from plugin does not work on plugin index pages, and should never be used in plugins (it shouldn't really be used anywhere, but that's a bigger/separate issue).
Reason
Marketing team gave us a broken link report.
Testing
Spot-check some link fixes and redirects, eg:
https://deploy-preview-3225--kongdocs.netlify.app/hub/kong-inc/basic-authentication - redirects to basic-auth
https://deploy-preview-3225--kongdocs.netlify.app/hub/kong-inc/ldap-authentication - redirects to ldap-auth
https://deploy-preview-3225--kongdocs.netlify.app/admin-api/ - redirects to gateway OSS Admin API doc

Note to reviewers: the jekyll redirect_from plugin does not work on plugin index pages, and should never be used in plugins (it shouldn't really be used anywhere, but that's a bigger/separate issue).

This is a great opportunity to open an Issue to rm the plugin.

Note to reviewers: the jekyll redirect_from plugin does not work on plugin index pages, and should never be used in plugins (it shouldn't really be used anywhere, but that's a bigger/separate issue).

This is a great opportunity to open an Issue to rm the plugin.

Issue or ticket? This is unfortunately a pretty big undertaking, as you'd have to:

Remove the redirect_from value from all files that use it (there's a lot).
Fix all links that were being redirected (for whatever reason, maintainers used to just set up redirects and never update links, so we have a ton of legacy links floating around).
Figure out which redirects we do still need to keep and add them to app/_redirects.
Uninstall plugin.

And yes, I've thought this through before, why do you ask? 😄

I get a 404 for this page: https://deploy-preview-3225--kongdocs.netlify.app/hub/kong-inc/ldap-authentication

That would be because I pasted the wrong one 🤦♀  It's actually https://deploy-preview-3225--kongdocs.netlify.app/hub/kong-inc/ldap-authentication-advanced that redirects to ldap-auth-advanced. But I'll add a redirect for that one as well, as people tend to try the full plugin name as the link.


[question] Does V extension implies F and D?

I cannot locate any mention of it, but Spike code suggests these two are mandatory; riscv/isa_parser.cc:
    switch (*p) {
      case 'p': extension_table[EXT_ZBPBO] = true;
                extension_table[EXT_ZPN] = true;
                extension_table[EXT_ZPSFOPERAND] = true;
                extension_table[EXT_ZMMUL] = true; break;
      case 'v': // even rv32iv implies double float
      case 'q': extension_table['D'] = true;
                // Fall through
      case 'd': extension_table['F'] = true;
    }

Yes, V implies F and D. V requires Zve64d, and that in turn requires D.

Thank you!

This is documented in "18. Standard Vector Extensions" section of the vector spec.
The V extension depends upon the F and D extensions...

@topperc I overlooked that... sorry for the buzz.


Bug: Priority of imports .json is higher ranked as .ts

+1
If I turn on resolveJsonModule and put a MatchmakingConfig.ts and a MatchmakingConfig.json in the same path, a nasty unexpected behavior awaits:
// this results in MatchmakingConfig.json being imported, not MatchmakingConfig.ts!
import MatchmakingConfig from "./path/to/MatchmakingConfig";

Allowing .ts suffix will let us specify .ts files specifically.
Originally posted by @tonygiang in https://github.com/microsoft/TypeScript/issues/37582#issuecomment-846950192
Solution
Priority needs to get changed anyway .ts needs to get resolved with a higher priority then .json

@weswigham any thoughts?

OP, for now, you can just use the .js extension in your import, it'll refer to the TS source; you don't need ts extension imports.
But I'm actually surprised that .json is a higher priority extension to try than ts source - that seems... Off? Moreover, I can't reproduce that locally? When I write imports they correctly prefer the .ts file to the .json one.

Originally posted by @tonygiang in https://github.com/microsoft/TypeScript/issues/37582#issuecomment-846950192 can you please publish a reproducer repo anything that we can run that shows that failure?

Yes, here it is: ts-sandbox.zip
And here is a screenshot showing the direct contradiction between VS Code's suggestion icon and the actual output:

Tested on Node 16.14.0 LTS.

this tripped me off a bit, I thought the node.js extension-less file resolution was preferring json over js.
it seems the bug is actually in ts-node. when you compile the repro with npx tsc, you can run the transpiled code with node index.js and you'll see type: "Module"
now, if you run ts-node after transpiling again, it'll correctly output type: Module as well. it's possible it runs the .js script the second time around. (haven't looked at the docs if this is intended)

Yeah... That's a ts-node issue, not us, from what I can see. We resolve to the .js file first. I'm not even sure if ts-node fully supports module: nodenext yet.

@weswigham i know they are still working on it https://github.com/TypeStrong/ts-node/pull/1361

I see the real issue now. Times like this makes me kind of want TypeScript to ship with a standard JIT engine so that issues don't get fragmented to different stacks. Deno runtime has a similar issue also stemming from running untranspiled .ts files directly. Instead of importing the .json file, Deno just finds nothing and throws a "Module not found" error. This could have been 1 issue on 1 standard runtime instead of 2.

I see the real issue now. Times like this makes me kind of want TypeScript to ship with a standard JIT engine so that issues don't get fragmented to different stacks.

not sure what that would solve. typescript is providing the type-checker and transpiler, ts-node is taking that and hooks it into the require hook to transpile on-the-fly. I assume the bug stems from leaving the node.js resolve algorithm in order (.js -> .json -> .node(?)) and picks up the .ts extension afterwards. just a bad guess.

There was apparently an Import Assertions proposal submitted to TC39 that is at Stage 3 at the time of writing which resolves this issue for good with an additional assert { type: "javascript" } or assert { type: "json" } statement. So now I'm not sure what direction the TypeScript project is going to resolve this issue now.?

https://devblogs.microsoft.com/typescript/announcing-typescript-4-5/#import-assertions
keep in mind typescript is not down-leveling this syntax to a commonjs equivalent, although I'm thinking it could (or even should, at least for commonjs), meaning you can use it only with a module target.
node.js has support now as well, and I'm assuming deno will too:
https://nodejs.org/dist/latest-v17.x/docs/api/esm.html#import-assertions

It has to do with tooling consistency. Currently, we have VS Code and TypeScript both developed by the same company with the expectation that .ts files are transpiled before execution. ts-node and Deno were created with the expectation to run untranspiled .ts files directly and created 2 issues on 2 different 3rd-party projects and produced behaviors inconsistent with what will be suggested by VS Code.
If a JIT engine was developed by the same company too, the best case scenario is that the developers walk into either VS Code or the JIT Engine with the other tool in mind and align expectation so that this would never become an issue. Maybe the JIT Engine will force a pre-transpile if it detects a .json file with the same name with a .ts file, or maybe VS Code will simply give you a warning in that case and now you can rename some files and get no more runtime surprise. The worst case scenario is that it became 1 issue instead of 2.

@tonygiang there is also ongoing work on typed json as far as i am aware of your jit arrgument is valid and the conclusion that the other projects are breaking behavior is also correct.
i think still that using rollup as loader would solve the issues at present i can workaround any issues like that via my rollup + plugin typescript setup but i was not able to isolate that into a useable generic product at the current point in time as there are many moving parts.
i do not remember what deno did use to run code but it was a relativ simple zero configuration bundler that they use to run the code. So deno has no own code to run code they are creating bundlers it was parcel
so anything that is valid for parcel is valid for deno at the end deno works with v8 so the only supported module system is ESM with importMaps and absolute or relativ pathes.

lets close this as it is not a typescript bug. but in the moduleResolution documentation on typescriptlang should maybe be a example in what order json gets loaded when it gets loaded.


🍒 Cherry pick PR #6332 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/6332 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.1.15-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @AndrewGable in version: 1.1.15-15 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Does this not work in safari anyone?

Hi!
I could be doing something wrong but I've tried using both your website and mine in safari and it doesn't work.
Is there something i need to add or does it not work in safari for now?
Thanks!

 Sorry about that! Thanks for the reply


New 3D/2D interface model

@fvanderlinden , @dzimmer : Please check the functionality of the new 3D/2D interface before merging this pull request.

There are some issues to reduce DAE index when the multibody frame of the 3D/2D interface rotates.

Issue with rotating multibody frame of the 3D/2D interface resolved


back Error

http://www.photoswipe.com/latest/examples/05-jquery-mobile-ajax.html
in this link, when I open a photo (fullscreen)
and I click in browser backbutton
dont close !!
error in Chrome e IOS safari !!!
Somebody can help me  ???

confirmed, is there any fix for this? back button in the ajax example also doesn't work on the iphone.

this is a temporary fix, I added rel="external" in the a tag that links to the gallery page.
now jquery doesn't add the back button anymore to the gallery page when data-add-back-btn is set, so I added it manually:
Back


feat(monthly report): much updated monthly report for all backers

including general stats across all collectives backed

Coverage decreased (-0.05%) to 71.979% when pulling fb714598cc97a37267a48a9bbc902d6e789c7c69 on fix1050 into 8180ebb1d7bc0c007d743345b51137e294c6e5a3 on master.


HV-2025 Remove OSGi testing and support

https://hibernate.atlassian.net/browse/HV-2020
https://hibernate.atlassian.net/browse/HV-2025
Opening this as a draft for now. It

removes the felix plugin that was generating the bundle info for OSGi
removes most of the mentions of OSGi from the docs
removes direct dependency on Expressly as without OSGi support we can skip the step of trying to load the EL factory from the EL impl classloader.
removes the OSGi directory with tests and karaf features.
removes the corresponding relocation pom.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on licensing, please check here.

Thanks for taking a look! 🎉 🏆


an error appears. download folder is not accessible.

how to fix this?

Alright, which one of you did this to SoniTranslate? 'Cause I didn't do nothing wrong to it at HuggingFace and I'm the only one who's been patient. Tell me, who, did, this, to, this, app?


Remove pass-through activator for pages

This change removes a 'pass through' IRazorPageActivator that we were
using to avoid the default one initializing the Page twice.
The fix is to add an adapter so that the IRazorPage that the RazorView has
isn't the real page, it's the adapter. The adapter doesn't have anything
interesting to activate :).

/cc @pranavkm

:shipit: as long as things compile.

/cc @pranavkm the activator wasn't handling cases where you don't have a model property. I added support for that and cleaned up the tests, which were kinda gross. Take a look if you want, or just treat it as FYI.


Switch from buildkite to GitHub actions.

We now have the ability to connect to Packit directly from GitHub actions, which removes the need to use our internal Buildkite infrastructure.
The new workflow is simpler by not having to deal with Docker, and should also be easier to debug and work with. It uses the new mrc-ide/orderly-action to handle the initialization and authentication to Packit.

This works similarly to the existing buildkite pipeline: it reads the output of db_extraction from Packit and runs the rest of the workflow. The packets it produces are pushed back to the Packit server.
Unfortunately the analysis based on the db_extraction packet that is hosted on Packit is broken and the workflow fails to complete. I tried to re-run the db_extraction task locally to regenerate that packet but ran into encoding issues when reading from the Access database.


The minus symbol in MathJax.tex2mml result is different with the symbol in a MathJax rich text editor.

Issue Summary
The minus symbol in MathJax.tex2mml result is different with the symbol in a MathJax rich text editor.
Steps to Reproduce:
When I use MathJax.tex2mml('16-9='), I get <math xmlns="http://www.w3.org/1998/Math/MathML" display="block">\n  <mn>16</mn>\n  <mo>−</mo>\n  <mn>9</mn>\n  <mo>=</mo>\n</math>.
I noticed that the minus symbol is Hex code −. In my application, the next step is put the mml in a tinymce editor, then user can edit, and finally search the mml in a database.
If I add a minus symbol using the tinymce editor, I get -, which is also the symbol - , rather than −.
Is there any config or option or parser to get - as minus symbol in MathJax.tex2mml('16-9=')? Or can I have a map about this?
Technical details:

MathJax2 and MathJax3
Browser: Chrome 29.0.1547.57

Supporting information:
I use mathtype editor in wiris demo, then I input minus symbol by click the button. The symbol is - (-).

As you point out, the minus sign is U+2212, and that is why MathJax TeX input produces that in its MathML output.  The remapping of U+002D to U+2212 is handled through a TeX character map, and you can modify it using the following configuration in v3:
MathJax = {
  startup: {
    ready() {
      const {MapHandler} = MathJax._.input.tex.MapHandler;
      MapHandler.getMap("remap").map.delete('-');
      MathJax.startup.defaultReady();
    }
  }
}

The output processors also convert U+002D to U+2212 when used as the only character inside an <mo>, so the typeset output will still be the same.


Chat icons

To go along with #4, I suggest the following warning icons. These should be displayed in the rightmost edge of the chat textbox, to prevent being too intrusive.

Green ✅

The mod has been discovered on the server
The server is using 1.18.2 or below
No detected reportable messages (so using different channel)

Yellow ℹ

The mod is working client-side only

Red ⚠

Player overrid the warning of #4 and messages are reportable

these should be displayed in the config menu when on  a server

these should be displayed in the config menu when on a server

Nope, the mod doesn't support any kind of config menu right now.
I'll go with original suggestion.

these should be displayed in the config menu when on a server

Nope, the mod doesn't support any kind of config menu right now. I'll go with original suggestion.

Well then it should go into the pause menu! It would still be too invasive for the chatbox, unless the messages are reportable

To be clear, I did only mean the box where you write your messages/commands in, not the individual messages themselves. So just one icon in the corner of the screen, while user is typing anyway.

So far my implementation looks like this

Looks great, except that last tooltip is a bit too complex. Try shortening the sentences a bit to make the message clearer.

these indicators must be able to be disabled

these indicators must be able to be disabled

There will be a config option for that.

Was added in 1.19.1-pre2-v1.3.0.


Feature: Implement storybook

This is a WIP issue still...
It may be worthwhile to consider Storybook here as well since we use it for jambonz-ui already and could leverage most of the configuration implemented there.

https://storybook.js.org/docs/react/get-started/introduction
https://github.com/jambonz/jambonz-ui/tree/main/.storybook
https://github.com/jambonz/jambonz-ui/tree/main/stories

ok, but I really dont want to invest any time into new architectural stuff until we implement the multi-user support, as this is really needed.  Its on me to do a bit of design for it, but would rather not launch any other major initiatives till that is done

ok, but I really dont want to invest any time into new architectural stuff until we implement the multi-user support, as this is really needed. Its on me to do a bit of design for it, but would rather not launch any other major initiatives till that is done

Sounds good. This stuff isn’t meant to be done right now. I’m just logging the information here to keep track of it. If there’s a better place you’d prefer to keep track of this type of stuff we can move it there. I think the testing is important (much more so than this) but that is also something that doesn’t have to be worked on immediately and can be accomplished over time as time permits. We could use a new issue prefix or label called “Backlog” or something to be more clear about it.


Facebook and Google login authorization does not work on my phone

As the title，Facebook and Google login button does not respond，but it works on simulator

Content blocking is enabled on my mobile browser， Just turn off the content blocker in the browser


[direct-core] Properly handle onError return value in inmem commit lo…

…g reader.

This only handles false return value. I don't think we properly handle true return value. Same problem remains in batch reader as well.


Use the correct element for package descriptions

[ ] Make sure the descriptions are correct for the project
[ ] Make sure the license url is valid
[ ] Make sure the project url is correct

Checked all
Explicit - Grpc, TensorFlow and OpenCensus - OK

/azp run


[Bitwarden Vault] Extension encountered an error

Extension
https://www.raycast.com/jomifepe/bitwarden
Description
Error:
Error: Command failed with Unknown system error -86: /Users/raphael/Library/Application Support/com.raycast.macos/extensions/5fdb86d3-dfcc-45a7-8969-8113b51434ab/bw-2023.8.2 lock
spawn Unknown system error -86

ChildProcess.spawn:child_process:413:11
Object.spawn:node:child_process:757:9
je:search.js:47:4352

Steps To Reproduce

in any macOS version running the latest version of raycast
run any bitwarden command

Current Behaviour
No response
Expected Behaviour
No response

Hello @raphtlw, sorry for the inconvenience, the extension recently stopped requiring the installation of the CLI and that may be the cause of the issue.
Can you tell me if you see any message saying "Initializing Bitwarden CLI" once you launch a command, before the error?

Hello @raphtlw, sorry for the inconvenience, the extension recently stopped requiring the installation of the CLI and that may be the cause of the issue.
Can you tell me if you see any message saying "Initializing Bitwarden CLI" once you launch a command, before the error?

Hi, the issue was caused because I hadn't installed rosetta on my m1 mac. While I no longer need the support, I did not see any message coming up in the raycast window when I tried to run Bitwarden. Is the node rpc interface just not aarch64?
Thanks for the help,
Raphael


Implement Filter::mergeWith for BytesRange and BytesValue

Filter::mergeWith API is used to merge filters pushed down from the join to topN operators into collocated table scan. Currently this API is implemented for numeric types. This task is to implement the API for strings.
https://github.com/facebookincubator/velox/pull/119 PR introduced Filter::mergeWith() API with implementation for numeric types.
https://github.com/facebookincubator/velox/pull/103 introduced filter pushdown that uses Filter::mergeWith() API.
CC: @pedroerp @aditi-pandit @majetideepak

@mbasmanova Could you please assign this issue to me?

I think this issue can close because it has finished in https://github.com/facebookincubator/velox/pull/297.


Add Bluebox-ng install script for Kali 2 GNU/Linux

This script updated for Kali 2 GNU/Linux reference to this issue https://github.com/jesusprubio/bluebox-ng/issues/57. Already tested in Kali 2 GNU/Linux with bluebox-ng source code from https://github.com/jesusprubio/bluebox-ng

Cool! Thanks!!! :)

Ey dude, could you please also to remove the old one and update the link into README? :)
But wait, I'm giving you permissions in the repo, this way you can avoid the PRs. Welcome!! BTW, I've sent you an invitation to a Slack group in which we're speaking about the whole thing. Channel #bluebox-ng. See you there!

I'm use kali2.sh instead of kali.sh because i don't tested in old Kali GNU/Linux. But, if you are want to target Kali 2 GNU/Linux then i will update.
Thanks for inviting me to your project and Slack group.

Yes, IMHO we can drop it, it's time to update. If we both agree of course. :)


Fix --source flag

Seems like we were mainly ignoring the source flag from the start. We had a nice value in the struct to store the source coming from the config or flag but were setting the spec flags directly instead. The config should have gone into the config directly and then the spec Imagesrc is generated from that.
This should fix the takeover install and probably other use cases that we have not hit before as the --source flag seems to be rarely used, mostly it defaulted to the media dir which is the main deployment path (iso,netboot or usb)
Related to: https://github.com/kairos-io/kairos/issues/1528

A test is broken. Also I wonder if we could add/change some test in the kairos repository to catch this.

A test is broken. Also I wonder if we could add/change some test in the kairos repository to catch this.

we should probably do a takeover install, but seems a bit complex. Boot with some random livecd, install docker, do takeover install, reboot and acceptance tests
maybe this has fixed the loop devices installation......which also stopped working around the same time...
that could be easier to test

lol, this actually fixes the install to loop devices, so this is even better....we can create the raw images with kairos now...
qemu-img create test.img +20G
kairos-agent manual-install --device /dev/loop0 --source oci:quay.io/kairos/opensuse:leap-15.5-core-amd64-generic-v2.5.0 config.yaml

blkid of the loop devices after:
/dev/loop0p5: LABEL="COS_PERSISTENT" UUID="7c98afd2-435c-4b3b-81f4-ea5757749270" BLOCK_SIZE="4096" TYPE="ext4" PARTLABEL="persistent" PARTUUID="9324d87d-83f1-4563-a21e-14965d680040"
/dev/loop0p3: LABEL="COS_RECOVERY" UUID="dd9e6401-4ffd-491e-9d01-efdf825a0cef" BLOCK_SIZE="4096" TYPE="ext4" PARTLABEL="recovery" PARTUUID="8d076e21-5203-4e12-86aa-319c1bbe3720"
/dev/loop0p1: SEC_TYPE="msdos" LABEL_FATBOOT="COS_GRUB" LABEL="COS_GRUB" UUID="3251-13C3" BLOCK_SIZE="512" TYPE="vfat" PARTLABEL="efi" PARTUUID="67a09af7-8919-40c1-b336-c0b2705149c1"
/dev/loop0p4: LABEL="COS_STATE" UUID="5907cbdf-22e9-420b-aa57-67c30bbcbe65" BLOCK_SIZE="4096" TYPE="ext4" PARTLABEL="state" PARTUUID="bfb89772-da52-44f9-81e7-9a8a2c039e39"
/dev/loop0p2: LABEL="COS_OEM" UUID="057f74e4-8304-4f35-83bd-5577003d2a3a" BLOCK_SIZE="1024" TYPE="ext4" PARTLABEL="oem" PARTUUID="7fc730c7-dfeb-4bb6-95f5-0b3db0b02a29"

so now the test.img is ready to be burn to a disk and booted...

qemu-img create test.img +20G
kairos-agent manual-install --device /dev/loop0 --source oci:quay.io/kairos/opensuse:leap-15.5-core-amd64-generic-v2.5.0 config.yaml

Some steps are missing here, no? How does the second command know about the test.img file created by the command above? Shouldn't we first mount the img?
If this is working, does it mean we can get rid of these 2 scripts?

https://github.com/kairos-io/osbuilder/blob/master/tools-image/build-arm-image.sh
https://github.com/kairos-io/osbuilder/blob/master/tools-image/prepare_arm_images.sh

qemu-img create test.img +20G
kairos-agent manual-install --device /dev/loop0 --source oci:quay.io/kairos/opensuse:leap-15.5-core-amd64-generic-v2.5.0 config.yaml

Some steps are missing here, no? How does the second command know about the test.img file created by the command above? Shouldn't we first mount the img?
If this is working, does it mean we can get rid of these 2 scripts?

https://github.com/kairos-io/osbuilder/blob/master/tools-image/build-arm-image.sh
https://github.com/kairos-io/osbuilder/blob/master/tools-image/prepare_arm_images.sh

Yes, sorry, mounting the img as a loop device is missing.
Potentially yes, we may be able to remove those scripts to generate raw images for arm64. RPI still won't work as it would need extra packages but we can probably work something out to generate RPI images directly


CNRB0017804 returns 404 not found

The API returns 404 for CNRB0017804 which means that the code is likely invalid.
However, it is available in the List of NEFT enabled bank Branches (Bank-wise Indian Financial System Code) updated as on January 31, 2021 dataset on RBI website
https://m.rbi.org.in/Scripts/bs_viewcontent.aspx?Id=2009
It is also available on the RBI IFSC/MICR search interface
https://www.rbi.org.in/Scripts/IFSCMICRDetails.aspx

Now it is working fine.


PENV-272: replace sleep with wait for records count in db

- What I did

replace sleep with wait for records count in db
small review fixes from https://github.com/insolar/block-explorer/pull/21

(note that receive_records_test.go was renamed, it's not a new file)
- How I did it
- How to verify it
- Description for the changelog

Codecov Report

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

@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   61.19%   61.19%
=======================================
  Files          29       29
  Lines        1201     1201
=======================================
  Hits          735      735
  Misses        400      400
  Partials       66       66

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 e2752a6...58f0db9. Read the comment docs.


Rework get_environment (the command 'execute sensor detail' is not pr…

get_envrironment should be reworked since the command "execute sensor detail" is not present in every device. I have replaced it by the command "execute sensor list"

That makes sense! Thank you again!


dist folder not getting generated properly using gulp build

@DimaSavchenko I tried creating dist folder using gulp. When I open dist folder only fonts folder is getting generated.

I was able to solve it by changing the version of gulp-jscs.


Is it alive ?

Is this project  supposed to be alive ?

Hello, Sure, this project is alive! Any contribution is welcome!


Add message collection window, and synchronize image and layers using header timestamp if available

#7 alternative but with improvement implementation
Closes: #5

Before changes, there is a clear lag between the image and the topic (ball detection) as seen in the video here: https://www.youtube.com/watch?v=BYN1tczU7xc
With the new changes in this branch, the timestamp is respected to get synchronized images, if the header timestamp is available (https://www.youtube.com/watch?v=1OHLSWl-JzM)


Fix searchType timing bug

This bug crashes the extension. The error log is as follows and only happens sometimes.
Uncaught TypeError: Cannot read properties of null (reading 'searchType')
    at Input (contentScript.js:270:69)
    at renderWithHooks (contentScript.js:37216:18)
    at mountIndeterminateComponent (contentScript.js:40980:13)
    at beginWork (contentScript.js:42493:16)
    at beginWork$1 (contentScript.js:48332:14)
    at performUnitOfWork (contentScript.js:47466:12)
    at workLoopSync (contentScript.js:47372:5)
    at renderRootSync (contentScript.js:47340:7)
    at recoverFromConcurrentError (contentScript.js:46756:20)
    at performSyncWorkOnRoot (contentScript.js:47002:20)

Leaving this issue open incase #22 is not a permanent fix.


maxcut notebook is broke

cells don't execute

Hi @ampolloreno can you tell us what cells dont execute and what specific error message you are receiving.


Request for adding new face CNNs to wiki page

added PAMs, ResFace-101 and Resnet-101 for regressing 3DMMs

Wow this is nice - thanks so much!

@Yangqing Thank you so much for adding this!
I just figured out that the caffe icon is not visible properly here Can you replace it with the correct link below? (I don't want to open another pull request).
https://caffe2.ai/static/images/download-c1.png
Another possibility is that I get access to write the page, then I won't bother you anymore.
Thank you!


Drop support for Node.js 10.x

Node.js 10.x is end-of-life since 2021-04-01, so new major versions of flora can drop support for Node.js < 12.

Fixed in 6152bc4


Application successfully tested. Promoting to Production.

Changes made during testing:
GameLogic.cs
Line 101 - PlaceShip method

SpotLetter = row.ToUpper(),
Need to convert user input to upper case for cleaner display to the console

Line 36 - IdentifyShotResult method

ship.Status = GridSpotStatus.Sunk;
Ships were not changing status to sunk. Fixed the functionality.

UIDisplay.cs
Line 28 - AskForShot method

Console.WriteLine();

Added a writeline for cleaner display

Console.Write($"{player.UsersName}, please enter your shot selection: ");

Enhanced method to pass in player name, so that each player knows whose turn it is.

UILogic.cs
Line 13 - DisplayShotGrid method

Console.Write($" {gridSpot.SpotLetter}{gridSpot.SpotNumber} ");
Added various spacings on lines 27, 31, 35, and 39. Results in cleaner display for user.
Also added two Console WriteLines on 43 and 44 for better spacing.

Line 47 - RecordPlayerShot method

(row, column) = GameLogic.SplitShotIntoRowAndColumn(shot);

isValidShot = GameLogic.ValidateShot(activePlayer, row, column);

Surrounded these two method calls in a try-catch block for better exception handling.

Application will no longer crash for bad user input on ship selection for firing on.

Line 80 - DisplayShotResults method

New method to tell user if their shot is a hit or miss. Results in better communication for the user.

Line 109 - PlaceShips method

Added null and empty validation check
Added try-catch block for better exception handling.

Edge cases fully tested. Approved, and promoting to production. Good job!


Why do you require write access to my public repos just so I can submit feedback on the docs pages?

Not that I think there is anything there I care that you read, but I do not see why you need to write to my repo for any reason.
To give a bit of context, I was looking for an MSBuild xsd at its docs page and was presented with this choice

Since I was not finding the xsd file I was looking for, I clicked on the "Sign in to give documentation feedback" and was presented with this...

... and I ain't clickin' that.

@ghogen Can you follow up with the folks that implemented this feedback system to see if we really need all that access?


ops_test.model.deploy ignores pinned revisions in a bundle

When using ops_test.model.deploy to deploy a bundle, it seems to ignore pinned charm revisions.
Steps to reproduce
The following template named bundle.yaml.j2:
{%- set testing = testing is defined and testing.casefold() in ["1", "yes", "true"] %}
bundle: kubernetes
name: identity-platform
website: https://github.com/canonical/iam-bundle
issues: https://github.com/canonical/iam-bundle/issues
applications:
  hydra:
    charm: hydra
    channel: {{ channel|default('edge', true) }}
    scale: 1
    series: jammy
    trust: true
  kratos:
    charm: kratos
    channel: {{ channel|default('edge', true) }}
    scale: 1
    series: jammy
    trust: true
  kratos-external-idp-integrator:
    charm: kratos-external-idp-integrator
    channel: {{ channel|default('edge', true) }}
    scale: 1
    series: jammy
  identity-platform-login-ui-operator:
    charm: identity-platform-login-ui-operator
    channel: {{ channel|default('edge', true) }}
    scale: 1
    series: jammy
    trust: true
  postgresql-k8s:
    charm: postgresql-k8s
    channel: 14/stable
    series: jammy
    scale: 1
    trust: true
  tls-certificates-operator:
    charm: tls-certificates-operator
    channel: stable
    scale: 1
    {%- if testing %}
    options:
      ca-common-name: demo.ca.local
      generate-self-signed-certificates: true
    {%- endif %}
  traefik-admin:
    charm: traefik-k8s
    revision: 136
    channel: edge
    series: focal
    scale: 1
    trust: true
  traefik-public:
    charm: traefik-k8s
    revision: 136
    channel: edge
    series: focal
    scale: 1
    trust: true
relations:
  - [hydra:pg-database, postgresql-k8s:database]
  - [kratos:pg-database, postgresql-k8s:database]
  - [kratos:endpoint-info, hydra:endpoint-info]
  - [kratos-external-idp-integrator:kratos-external-idp, kratos:kratos-external-idp]
  - [hydra:admin-ingress, traefik-admin:ingress]
  - [hydra:public-ingress, traefik-public:ingress]
  - [kratos:admin-ingress, traefik-admin:ingress]
  - [kratos:public-ingress, traefik-public:ingress]
  - [identity-platform-login-ui-operator:ingress, traefik-public:ingress]
  - [identity-platform-login-ui-operator:endpoint-info, hydra:endpoint-info]
  - [identity-platform-login-ui-operator:ui-endpoint-info, hydra:ui-endpoint-info]
  - [identity-platform-login-ui-operator:ui-endpoint-info, kratos:ui-endpoint-info]
  - [identity-platform-login-ui-operator:kratos-endpoint-info, kratos:kratos-endpoint-info]
  - [traefik-admin:certificates, tls-certificates-operator:certificates]
  - [traefik-public:certificates, tls-certificates-operator:certificates]

is rendered correctly with ops_test.render_bundle (it includes the revision 136 in traefik-k8s), but when deployed in an integration test using ops_test.model.deploy, the most current revision 141 is deployed instead.
Example CI
Workaround
Deploy the bundle with ops_test.run("juju", "deploy", rendered_bundle, "--trust")

@natalian98 thanks for raising this issue.  ops_test.model is a Model object from python-libjuju
I believe there are numerous issues against deploying in python-juju regarding this
You might wish to open this issue there


Disk IO wrong values

Hello, I have an issue with some Disk IO graphs. To be specific: values of md0 are swapped. At the moment I'm syncing to this array and it should display write not read(see screenshot).

Thanks for pointing this out!
You're absolutely right. I accidentally swapped the read/write labels for extra disk charts during localization updates.
I'll release a patch soon which will correct this and fix the labels for historical data as well.

Hah don't worry, sh** happens. Interesting is that additional disks not in array seems fine.

This should be fixed now in 0.7.3 :+1:


Revert "Trim GlanceCRD and optimize extraMounts struct usage"

Reverts openstack-k8s-operators/glance-operator#181
We still don't have a way to notify the update to the sub-resources if something changes in the top-level CR (we can't trigger the reconcile on the sub-resource if something different than "Spec" changes.
This change will be re-proposed if a clean way will be found.

I think one idea might be to store a hash of the ExtraMount in the Glance.Status.  Then the GlanceAPI watches Glance and its reconcile loop will be trigger in the event that Glance.ExtraMount changes (because the Glance controller will update the Glance status with the new hash).  At least...I think when a controller watches a resource, it will be triggered by status updates and not just for spec updates.

I think one idea might be to store a hash of the ExtraMount in the Glance.Status. Then the GlanceAPI watches Glance and its reconcile loop will be triggered in the event that Glance.ExtraMount changes (because the Glance controller will update the Glance status with the new hash). At least...I think when a controller watches a resource, it will be triggered by status updates and not just for spec updates.

Or is the problem that the Glance controller doesn't wake when the Glance resource's ExtraMounts is updated?

I think one idea might be to store a hash of the ExtraMount in the Glance.Status. Then the GlanceAPI watches Glance and its reconcile loop will be triggered in the event that Glance.ExtraMount changes (because the Glance controller will update the Glance status with the new hash). At least...I think when a controller watches a resource, it will be triggered by status updates and not just for spec updates.

Or is the problem that the Glance controller doesn't wake when the Glance resource's ExtraMounts is updated?

So the Glance controller reconcile loop is triggered when ExtraMount(s) change, the problem I see here is being notified in the glanceAPI controller and trigger the reconcile loop due to this event.
I already added the hash with something like:
+       var ev []storage.VolMounts
+       for _, vm := range instance.Spec.ExtraMounts {
+               ev = append(ev, vm.VolMounts...)
+       }
+       h, err := storage.Hash(ev)
+       configMapVars["ExtraMounts"] = env.SetValue(h)

and I see this hash being update if I stop my local operator and make run it again: actually stopping and starting triggers the GlanceAPI controller and I'm able to update both internal and external deployments.
I'm trying to solve the problem of being notified by playing a bit w/ SetupWithManager in the glanceapi_controller:
+       extraVols := func(o client.Object) []reconcile.Request {
+               result := []reconcile.Request{}
+               // Get Glance Top level CR and fetch ExtraVolumes if defined
+               glances := &glancev1.GlanceList{}
+               listOpts := []client.ListOption{
+                       client.InNamespace(o.GetNamespace()),
+               }
+               if err := r.Client.List(context.Background(), glances, listOpts...); err != nil {
+                       r.Log.Error(err, "Unable to retrieve API CRs %v")
+                       return nil
+               }
+
+               for _, cr := range glances.Items {
+                       if cr.Kind == "Glance" {
+                               name := client.ObjectKey{
+                                       Namespace: o.GetNamespace(),
+                                       Name:      cr.Name,
+                               }
+                               result = append(result, reconcile.Request{NamespacedName: name})
+                       }
+               }
+               if len(result) > 0 {
+                       r.Log.Info("Reconciling Request!")
+                       return result
+               }
+               return nil
+       }
        return ctrl.NewControllerManagedBy(mgr).
                For(&glancev1.GlanceAPI{}).
                Owns(&keystonev1.KeystoneEndpoint{}).
@@ -192,6 +210,9 @@ func (r *GlanceAPIReconciler) SetupWithManager(mgr ctrl.Manager) error {
                Owns(&corev1.ConfigMap{}).
                Owns(&appsv1.Deployment{}).
                Owns(&routev1.Route{}).
+               // Watch for Glance{} ExtraVolumes
+               Watches(&source.Kind{Type: &glancev1.Glance{}},
+                       handler.EnqueueRequestsFromMapFunc(extraVols)).
                Complete(r)
 }

with something like the above, when glance CR is modified, I should always trigger a reconcile loop (then the idea is to reduce the scope to the input hash that changes), but so far I only get:
2023-03-21T14:25:39+01:00       INFO    controllers.GlanceAPI   CR KIND Glance
2023-03-21T14:25:39+01:00       INFO    controllers.GlanceAPI   FOUND glance
2023-03-21T14:25:39+01:00       INFO    controllers.GlanceAPI   Reconciling Request
2023-03-21T14:25:39+01:00       INFO    controllers.GlanceAPI   CR KIND Glance
2023-03-21T14:25:39+01:00       INFO    controllers.GlanceAPI   FOUND glance
2023-03-21T14:25:39+01:00       INFO    controllers.GlanceAPI   Reconciling Request
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciling Service 'glance'
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      (TOP_LEVEL) EXTRAMOUNTS (CONFIGMAP): 0x16be600
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciling Service 'glance' init
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciled Service 'glance' init successfully
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciling Service 'glance' update
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciled Service 'glance' update successfully
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciling Service 'glance' upgrade
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciled Service 'glance' upgrade successfully
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Deployment glance successfully reconciled - operation: updated
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Deployment glance successfully reconciled - operation: updated
2023-03-21T14:25:39+01:00       INFO    controllers.Glance      Reconciled Service 'glance' successfully

I'm able to see the top-level CR being changed, I think I'm able to enqueue a reconciliation request, but I don't see it triggered in the underlying controller!


Create new package terra-form-textarea

Issue Description
Terra Form textarea needs it's own package so that non-passive changes to other form components won't cause several major version bumps to terra-form.
Issue Type

[ ] New Feature
[X] Enhancement
[ ] Bug
[ ] Other

PR
PR being validated #982

PR being validated #982

This has been released in terra-form-textarea 1.0.0


Picker modal modified like Action sheet

We have to close picker modal manually by invoking javascript or adding class to html. But it is easy to close action sheets just by clicking anywhere.  How to implement this to picker modal?

No, picker modal is not intended to be used like this, so by default it is not implemented. But you can add some kind of such snippet:
$$('html').on('click', function (e) {
    if ($$(e.target).is('.picker-modal') || $$(e.target).parents('.picker-modal').length > 0) {
        myApp.closeModal('.picker-modal');
    }
});


Add ability to show image on index

Thanks for your PR, looks great!
Could you please add a tiny bit of documentation for the image_on_index option in the appropriate section of the README?
Then I’ll merge and cut a new release of the gem 💎.

@michelegera added some info to the readme and rebased/squashed it up for you. LMK if you need anything else.
One possible enhancement would be to reference a thumb version of the image, if available, to save on load time for the index. But I think that's could also be added as an enhancement after merging this in.

@michelegera should be all set.

@jondkinney thanks for your work on this! I’m going to merge and release a new version of the gem.
Definitely agree with you on the thumb to display on index for increased performance. Would you mind submitting a PR for that, when you have time?


Schema gets updated with every message produce

I have notices an issue with produce with configuration with schema and schema serializer. For some reason the defaults (which can be null) are removed from from the server schema as soon as I run the tests.
It seems the goavro lib that is being used does this as part of serialization. The problem is, this schema later is used as in a post request to fitch the schema id from the server. This post request update the server schema in this case. I could not find a way that we provide the schema id to bypass this logic/behavior.

Any update on this?

Hey @saad1200 @ssam5532!
Can any of you provide an example script (and/or configuration) you use, so I can test it?

@mostafa Any produce code example from the repo can be used to reproduce this. It has been few months now, but I think it is an issue with goavro lib.
we noticed that xk6-kafka is posting a new schema when called with produceWithConfiguration. Looks like  it’s because they’re stripping out some of the java specific types and before posting the schema to the registry (which I guess they need to do, to get an ID).
https://github.com/mostafa/xk6-kafka/blob/57dfc78d0b9c85557519772c3bce7e5f17ed3b30/schemaRegistry.go#L59
If you look at line 54 and 56,  goavro’s codec is turning the schema into and object, then making it into a string again on line 56.

@ssam5532 our work around is to setup wiremock and use it as schema registry proxy. It is another dependency and far from ideal solution but allowed us to continue using K6.

@saad1200
I suppose this can be fixed by trying to fetch the schema first before trying to post one. We can also use something like srclient and have it handle everything, but it's an extra dependency for just two HTTP requests.

@saad1200 @ssam5532 I'd be happy to have your feedback on #52.

@saad1200 @ssam5532
The PR (#52) is merged now, and you can test it to see if it fixes the issue or not. I haven't released a version yet, so you need to build the latest main according to xk6 build instructions to test it. Let me know if you face any issues along the way!
Also, based on my tests, the schema doesn't get updated.

@mostafa sorry for the delay. I will block some time next weekend to test the changes.

@saad1200  , can you please help me here. , my schema also get updated with each sent message on topic , how you retrieve the schema first and then send same ?

@kusumkappdirect
If you are using latest release and still having this issue all you need to do is to setup wiremock on the dev server or locally, then in the k6 setup method get the schema from the schema registry and stub the methods to return that schema using wiremock. Pass wiremock address instead of the schema registry to the k6 extension produce method.
I will charge my code with you soon


InlineChunkHtmlPlugin works with empty publicPath

The InlineChunkHtmlPlugin plugin didn't work when the publicPath variable is empty or unset (e.g. when I want to use relative URLs).
Before:

If publicPath was unset or wasn't a string, webpack would throw an error
If publicPath was an empty string, the plugin would fail to inline a chunk

Now: the plugin behaves as expected even with empty or unset publicPath.

I have signed the CLA.

n.b. for other reviewers, this is not a bug. It only affects users who eject from Create React App.

 It just happens to affect people who eject from CRA (like my case). Nonetheless it's not less of a bug :)

@Timer any update on this? When can it be merged?

Ping :)

Thanks!


Node 16 Deprecated

Upon using this action, the summary section of the workflow always provides the following annotation:

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Is there any timeline to upgrading from Node 16?

hey @IamGreut thanks for bringing this to our attention!
I just updated the action to use node 20 in v1.1.1


feat(rust): Move all Rust dependencies into the workspace

Motivation
For fast builds, it helps to have all dependencies defined in the workspace.  This way, dependencies can be pulled and built regardless of which packages we have.
Changes

Move all Rust dependencies into the workspace.
Add a test to ensure that all the rust dependencies are in the workspace.

Tests
See CI.  Builds should complete as usual.

The Docker image builds the Wasm which is then released. Adding linter and formatter checks within the container might make sense to have the same tooling do everything?
I'm also not familiar with caching on GitHub Actions, but is it maybe easier (and more performant) to cache the resulting images rather than caching the filesystem of the actions?
Really, I'm just stricly thinking out at loud. Not that much an idea.


Implement cron trigger

This pull request implement CronTemplate API to create diagnosis periodically at fixed times by Trigger.

LGTM


Bing maps won't load

Describe the bug
The request returns an error:

The request was forbidden.  Your credentials may be denied or suspended.

To Reproduce
Steps to reproduce the behavior:

Go to https://openlayers.org/en/latest/examples/bing-maps.html
See error

i can't reproduce an error。

I also don't get an error anymore.

We're periodically hitting a daily limit.  The examples aren't currently working.
curl "https://dev.virtualearth.net/REST/v1/Imagery/Metadata/RoadOnDemand?uriScheme=https&include=ImageryProviders&key=$KEY&c=en-us&jsonp=olc_3"

olc_3({"authenticationResultCode":"DeniedCredentials","brandLogoUri":"http:\/\/dev.virtualearth.net\/Branding\/logo_powered_by.png","copyright":"Copyright © 2021 Microsoft and its suppliers. All rights reserved. This API cannot be accessed and the content and any results may not be used, reproduced or transmitted in any manner without express written permission from Microsoft Corporation.","errorDetails":["The request was forbidden.  Your credentials may be denied or suspended."],"resourceSets":[],"statusCode":403,"statusDescription":"Forbidden","traceId":"e146307576e642f18ed61165ac98124a|CO00004BCF|<IP_ADDRESS>"})

I'm going to overwrite the key used in the hosted examples while awaiting a review on #12964.


No Retries

I tried to follow the advanced example to setup rascal. However it seems, the failed messages are being directly sent to dead letter without retries.
Tried to follow the example to see if if I missed anything, but doesn't appear so.
    recovery: {
      deferred_retry: [{
        strategy: 'forward',
        attempts: 10,
        publication: 'retry_in_1m',
        xDeathFix: true // See https://github.com/rabbitmq/rabbitmq-server/issues/161
      }, {
        strategy: 'nack'
      }],

      dead_letter: [{
        strategy: 'republish',
        immediateNack: true
      }]
    },

    redeliveries: {
      counters: {
        shared: {
          size: 10,
          type: isMaster ? 'inMemory' : 'inMemoryCluster'
        }
      }
    },

        // Setup subscriptions
        subscriptions: {
          consumer: {
            queue: [`${(process.env.NAMESPACE || '').replace(/^jx-/, '').replace(/[^a-zA-Z0-9]/g, '_')}_${process.env.APP_NAME}`],
            redeliveries: {
              limit: 10,
              counter: 'shared'
            }
          },
        },

Hi @heroic ,
Can you share the code you are using to ackOrNack with the error please?
Also can you set the DEBUG environment variable to rascal:SubscriberError and share the output.

Hi @cressie176
Logs:
a9102727-cf5c-4641-bea6-cbde38c662c6 Error Processing
  rascal:SubscriberError Handling subscriber error for message: a9102727-cf5c-4641-bea6-cbde38c662c6 +0ms
  rascal:SubscriberError Attempting to recover message: a9102727-cf5c-4641-bea6-cbde38c662c6 using strategy: fallback-nack +3ms
  rascal:SubscriberError Message: a9102727-cf5c-4641-bea6-cbde38c662c6 was recovered using stragegy: fallback-nack +8ms

Code:
  broker.subscribe('consumer').then(subscription => subscription.on('message', async (message, content, ackOrNack) => {
      console.log(message.properties.messageId, 'Got data', content);
      return handler(content.name, content.args).then(result => {
        if (result === true) {
          console.log(message.properties.messageId, 'Finished Processing', result);
          return ackOrNack()
        } else {
          throw new Error(result)
        }
      }).catch((e) => {
        console.log(message.properties.messageId, 'Error Processing', e.message);
        return ackOrNack(e)
      });
    }).on('error', console.error))

Hi @heroic, you need to specify the recovery strategy when rejecting a message. Because you aren't doing this, rascal is using the default "fallback" recovery strategy, which is a nack. See here

aah ok! I assumed it will follow the recovery strategies as in the configuration in the order they were added. got it!

I agree it's odd that they are not really part of the rascal configuration. I could consider overloading ackOrNack so it accepted a recovery strategy profile name, which it attempted to find in config.
ackOrNack(err, 'dead_letter');