fix deployment for cloudscale-metrics-collector

Summary
Adds missing update for new configuration options since #22 got merged.
Checklist

[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Update tests.
[x] Link this PR to related issues.

LGTM now I think. You might want to enable multi instancing support if we need to instance this exporter for every zone. See: https://syn.tools/commodore/reference/architecture.html#_component_instantiation

this is enabled in the exoscale metrics collector but I don't really understand it tbh. Probably because I'm not familiar enough with commodore and our deployment setup. Not sure if @Kidswiss knows more if we need this or not?

@mweibel This change to the metrics collector is so, that each instance will only handle exactly one cluster where the cloudscale appcat services are deployed.
We currently have two clusters where this is the case, appuio cloudscale and appuio exoscale. So in order to get the metrics of all the clusters, we would need two instances of this deployed on the appuio cloudscale cluster (where the database currently lives).
I know that's a rather suboptimal approach, but once we have an actual API for the billing, we can then deploy the collectors to each cluster.

added multi instance - hope that's done the right way?

ah wait, nvm - still need another change.

ok all good now.

after discussion with @ZuGao we decided to keep image v0.4.1 as the version to deploy for the image and first test the multi instance deployment. This also ensures when merging this and updating it in the catalog, we can test first multi instance deployment and only afterwards the k8s API changes.


Provide error code/exit code if data set is not cmor compliant, etc.

Backend should provide error information/code on exit, why it failed.
Examples:

File not found
File not readable
File empty
File not cmor compliant

The stack trace you get when the program stops because it encounters an error should provide most of this information. If you're on Python 3 or only using a single process that is, Python 2 will only give you the stack trace of the main process.
I agree that it would be good to work on making nicer error messages at some point, but it will be a lot of work if not impossible to cover all possible errors.

I think the most important errors with a need for specification are those, a user can easily correct. And these usually are file related. All the mentioned errors need to be checked at some point and are necessary/required for some projects when ESMValTool should be automatized.
E.g. If ESMValTool cannot read due to missing cmor standards, data should be cmorized.
It is redundant to check within the automation process, if ESMValTool needs to check either way.

I agree with Bouwe that it would be a nice feature, but at the moment we should focus on other issues.
I'm adding this to the goals for the beta release of v2.0.

Closing since already covered by #873


Concurrency control (MVTO) discussion

In trying to improve the performance of the CC system, I've identified several design decisions that I'd like to discuss here and possibly change in the future.

We maintain and rely on a doubly-linked version chain. This is at odds with both the MVCC paper:

Since it is not possible to maintain a latch-free doubly linked list, the version chain only points in one direction.

...and our own wiki entry on our CC protocol:

The last meta-data field is the 64-bit pointer that stores the address of the neighboring (previous or next) version (if any) of a particular version.

We add READs to a TransactionContext's RWSet. One of #1402's changes tries to optimize around this, but really it shouldn't be done in the first place. I understand the need to add READ_OWNs since we need to release the write lock on a tuple at transaction end, but a normal READ shouldn't be there.

Our tuple header is "full" at 64 bytes. We could always make the reserved field larger if we needed to, but spilling such a heavily-accessed structure beyond a single cache line sounds bad to me. Removing the doubly-linked version chain would buy us some space, as would exploring atomic operations on timestamps to remove the SpinLatch. I'm not convinced the latter is possible, but I am experimenting with 128-bit CAS implementations.

Reusing owned tuple slots for updates and deletes seems like a premature optimization that sacrifices correctness. As currently implemented, if we repeatedly update a tuple within a single transaction and then abort, we do not have enough information to reconstruct the old versions that need to be pruned from the indexes. If we stick with reusing tuple slots, then it seems like we need more info added to the TransactionContext, like index write sets (or wait for logging).

Also related to reusing owned tuple slots, I can generate an assertion failure in the master branch (debug mode) with the following SQL:

CREATE TABLE test(a INT PRIMARY KEY, b INT);
BEGIN;
INSERT INTO test VALUES (3, 30);
UPDATE test SET a=5, b=40;

Assertion failed: ((GetLastReaderCommitId(tile_group_header, old_location.offset) == current_txn->GetCommitId())), function PerformDelete, file .../peloton/src/concurrency/timestamp_ordering_transaction_manager.cpp, line 525.

This is similar to #1336 and #1337.

We don’t set the last-reader timestamp of a new version to the writer's timestamp. Instead it is 0. This goes against all of the MVTO references I've found, and may be a contributor to issue 5 listed above.

Hybrid index scan is fiddling with CC stuff. This strikes me as legacy code from before we had a proper CC system since it's from 2016. Based on Coveralls it doesn't seem like we exercise this codepath anyway, but anyone outside of the CC system changing tuple headers jumps out at me as concerning.

I'd like to have a discussion on these topics and if necessary come to a conclusion on how to address them.
@yingjunwu @gandeevan

Hi @mbutrovich, nice thought! Most of the problems you mentioned above have been discussed before. You can find my answers below.

We do use doubly-linked version chain. Ideally, a single-direction version chain, which was described in the paper, is enough. However, during the implementation, we found that we have to add the reverse-direction pointers to guarantee the correctness of our latch-free implementation. You may find the reason in my code comments. (I also need to recall why I did this :-))

Yes we only need to track write sets when using MVTO. See my comments in #1402

I remembered that I tried to replace spinlatch with a single CAS operation by compacting the to-be-updated data fields, but the performance didn't change too much. You could try to optimize it, but I am not sure whether it really helps. Also I am not sure how you wanna implement 128-bit CAS. As a main-memory DBMS, I think it would be more interesting to think about whether we can save memory as much as possible.

We never delete older tuple versions when updating the tuple, so I don't think there's any issue if we want to reconstruct the old versions when confronting transaction aborts. A tuple will not be directly removed from index if tuple deletion is performed within a transaction. But I do remember there's any issue with index. Please check my code comments or previous issues. One interesting thought about multi-update to a single tuple is that how to keep track of the updated fields. I think you may want to talk to the logging team about this, as they want to do delta logging.

5 & 6. I thought this was a solved problem, and I am not sure why it happens again. May need to double check.

The hybrid_index_scan code is not well maintained -- I haven't touched that code for a long time and I didn't really know when to use it. Also, this code should be replaced by the new LLVM code.

I am open to discuss any of these issues, and please let me know if there's anything I can help with. Hope my answers help :-)


Gem dependency problem - Rails 3 / Bundler support?

I'm running into some dependency issues while using Bundler 1.0.10 and Ruby 1.8.7 in a Rails 3.0.3 project:
First off, Bundler can't detect the dependency that oauth2_provider has on Validatable.  If you add that to the Gemfile, you then run into this issue:
/usr/lib/ruby/gems/1.8/gems/oauth2_provider-0.2.0/lib/oauth2/provider/model_base.rb:13: uninitialized constant Oauth2::Provider::ModelBase::Validatable (NameError)

You can get past this by adding "require 'validatable'" to the top of that file, but then you run into the next issue:
/usr/lib/ruby/gems/1.8/gems/oauth2_provider-0.2.0/app/controllers/oauth2/provider/oauth_authorize_controller.rb:6: uninitialized constant ApplicationController (NameError)

... and so on.

I had a similar problem with some other gem, your ** workaround ** is cool, thanks!


Binance OmiseGo throws PricingDataNotLoadedError

Dear Catalyst Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: Windows 10
Python Version: 3.6.4
How did you install Catalyst: conda

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue
I successfully run

catalyst ingest-exchange -x binance -f minute

catalyst ingest-exchange -x binance -f minute -i omg_btc

I basically try to fetch the price history for every coin on Binance, by using data.history.

What happened instead?
All the coins succeed except OmiseGo which is failing on the Error PricingDataNotLoadedError

Thanks 😄

try upgrading to 0.5.17

Hi @eranhirs ,
Does the issue occur after trying to ingest again using catalyst ingest-exchange -x binance -f minute -i omg_btc?
If so, on what dates you are running your algorithm and what dates have missing information?
Thanks!

Hi,
Same problem, and yes, the problem persists after trying to reingest just omg_btc.
Best,
Samuel

@sam31415 and @eranhirs ,
Perhaps the first ingestion of the data was corrupted.
If you don’t have too much data ingested already, I would try to workaround this by clearing the exchange data using catalyst clean-exchange -x binance and re-ingesting from scratch the omg_btc minute data.

Thanks, but I already tried to clean and reingest all the data (I need the data for all pairs, not only omg_btc). As far as I can tell, it is only the data for omg_btc that is missing: forbidding the algorithm to query it allows the backtest to go through.
Also, when ingesting I get repeteadly the error described in #405.

Just tried twice with both 0.5.15 and then re-installing catalyst, cleaning exchange data and using 0.5.18

Binance OmiseGO still doesn't have any data compared to other coins
All coins don't have binance data after 2018.06.26


Yuzu Early Access v2.0.4 not available!

I am trying to download the latest version of the Yuzu Early Access launcher from this repository but whenever I download v2.0.4 after opening it I see that it is v2.0.3. I think you have accidentally uploaded the previous version instead of v2.0.4. Please update it soon!

same issue here

You don't to download it from here. If you have an older version it will update itself.
What is happening with you is. The 2.0.4 you downloaded runs the previously installed 2.0.3.
And if 2.0.3 is saying that it can't connect to internet. This is because it is unable to download metadata from the previous source. In this case, please right click the launcher shortcut and open containing folder. Then delete and replace the old "yuzu Early Access Launcher.exe" with the new one and rename the new one same as the old one.


Fixing plugin issues

Asking permission to read external storage
Fixing callback

Thanks for this!

Thanks for the plugin :)


Recreate the redwood-stripe app from scratch

This issue use the information in Reseting the database? to successfully resolve the issue Something changed in RedwoodJS, since you updated the code with my initial set of issues

In order to facilitate a clean regeneration of the app, use the following steps in the given order

Delete redwood-app folder
Run yarn create redwood-app --ts ./redwood-stripe and yarn rw setup ui tailwindcss
Edit the just generated prisma.schema to set the provider to be postgres, and define the rest of required data
Created the .env file with all needed information
Run yarn rw prisma migrate reset - resulting with

Running Prisma CLI...
$ yarn prisma migrate reset --schema /Users/nik/dev/work/redwood/rw-community/stripe/my-generalui/redwood-stripe/api/db/schema.prisma

Environment variables loaded from .env
Prisma schema loaded from api/db/schema.prisma
Datasource "db": PostgreSQL database "redwoodstripe", schema "public" at "localhost:5432"

✔ Are you sure you want to reset your database? All data will be lost. … yes

Database reset successful

✔ Generated Prisma Client (3.15.2 | library) to ./node_modules/@pr
isma/client in 98ms

Running seed command `yarn rw exec seed` ...
[11:41:47] Generating Prisma client [started]
[11:41:47] Generating Prisma client [completed]
[11:41:47] Running script [started]

Using the default './scripts/seed.{js,ts}' template
Edit the file to add seed data

[11:41:48] Running script [completed]

🌱  The seed command has been executed.

Note that resetting the database using RedwoodJS generator has all kind of side effects, including running the seed command.
Run the following few remaining commands:

yarn rw prisma migrate dev
yarn rw generate page home
edit the file routes.tsx by adding the line <Route path="/" page={HomePage} name="home" /> as the first route
yarn rw dev

that last command creates the following output:
gen | Generating TypeScript definitions and GraphQL schemas...
gen | 19 files generated
api | Building... Took 521 ms
api | Debugger listening on ws://<IP_ADDRESS>:18911/ad374dda-cfa5-4613-9c97-d902c4fbc0d2
api | For help, see: https://nodejs.org/en/docs/inspector
api | Starting API Server...
api | Loading server config from /Users/nik/dev/work/redwood/rw-community/stripe/my-generalui/redwood-stripe/api/server.config.js
api |
api | Importing Server Functions...
api | /graphql 119 ms
api | /auth 1 ms
api | ...Done importing in 122 ms
api | Took 176 ms
api | API listening on http://localhost:8911/
api | GraphQL endpoint at /graphql
api | 12:13:32 🌲 Server listening at http://[::]:8911
web | assets by path static/js/*.js 4.62 MiB
web |   asset static/js/app.bundle.js 2.84 MiB [emitted] (name: app) 1 related asset
web |   asset static/js/src_pages_LoginPage_LoginPage_tsx.chunk.js 450 KiB [emitted] 1 related asset
web |   asset static/js/src_pages_SignupPage_SignupPage_tsx.chunk.js 449 KiB [emitted] 1 related asset
web |   asset static/js/src_pages_ResetPasswordPage_ResetPasswordPage_tsx.chunk.js 441 KiB [emitted] 1 related asset
web |   asset static/js/src_pages_ForgotPasswordPage_ForgotPasswordPage_tsx.chunk.js 420 KiB [emitted] 1 related asset
web |   asset static/js/runtime-app.bundle.js 51 KiB [emitted] (name: runtime-app) 1 related asset
web |   asset static/js/src_pages_HomePage_HomePage_tsx.chunk.js 3.62 KiB [emitted] 1 related asset
web |   asset static/js/src_pages_NotFoundPage_NotFoundPage_tsx.chunk.js 3.49 KiB [emitted] 1 related asset
web | asset README.md 1.9 KiB [emitted] [from: public/README.md] [copied]
web | asset favicon.png 1.7 KiB [emitted] [from: public/favicon.png] [copied]
web | asset index.html 483 bytes [emitted]
web | asset robots.txt 24 bytes [emitted] [from: public/robots.txt] [copied]
web | Entrypoint app 2.89 MiB (2.84 MiB) = static/js/runtime-app.bundle.js 51 KiB static/js/app.bundle.js 2.84 MiB 2 auxiliary assets
web | orphan modules 411 KiB [orphan] 116 modules
web | runtime modules 34.3 KiB 19 modules
web | modules by path ../node_modules/ 2.56 MiB 930 modules
web | modules by path ./src/ 115 KiB
web |   modules by path ./src/pages/ 87 KiB
web |     ./src/pages/FatalErrorPage/FatalErrorPage.tsx 3.17 KiB [built] [code generated]
web |     ./src/pages/ForgotPasswordPage/ForgotPasswordPage.tsx 18.1 KiB [built] [code generated]
web |     + 5 modules
web |   modules by path ./src/*.css 23.4 KiB
web |     ./src/scaffold.css 2.39 KiB [built] [code generated]
web |     ./src/index.css 2.38 KiB [built] [code generated]
web |     + 2 modules
web |   modules by path ./src/*.tsx 4.88 KiB
web |     ./src/App.tsx 1.8 KiB [built] [code generated]
web |     ./src/Routes.tsx 3.08 KiB [built] [code generated]
web | webpack 5.73.0 compiled successfully in 4924 ms
api | 12:13:34 🌲 incoming request GET xxx /auth?method=getToken
api | 12:13:34 🌲 request completed 21ms

At this point the basic Redwood app, ready to start using Stripe is created.
Note that before being able to run the seed-stripe-subscriptions.ts, the previously created products:

need to be manually deleted via Stripe dashboard.

I have added a reference to this in the readme. Note that you can also go to the developer page on stripe and click "Delete all test data..."


Entity/Link/Action attribute manipulation

Does anyone have a use-case where they would like to be able to change the .href of an entity, link, or action?
For instance if there happens to be out of band information in the API and you need to something on to the url? Like let's say the action's .href is http://example.com/api/messages. But for some reason you need to edit the .href to look like http://example.com/api/example/message.

Only use case I have is to add query parameters.
On Wed, Feb 11, 2015 at 7:57 AM, Chason Choate<EMAIL_ADDRESS>wrote:

Does anyone have a use-case where they would like to be able to change the
.href of an entity, link, or action?
For instance if there happens to be out of band information in the API and
you need to something on to the url? Like let's say the action's .href is
http://example.com/api/messages. But for some reason you need to edit the
.href to look like http://example.com/api/example/message.
—
Reply to this email directly or view it on GitHub
https://github.com/cha55son/siren_client/issues/3.

If you need to add additional query parameters you can do that by supplying the extra params to the action.where(param1: 'test', param2: 'test2').


Multiple draggables within a sortable context

I haven't been able to find a word on this in the dox, so please point me in the right direction, if this has been addressed. I need to implement a multiple items selection feature  which would allow them to be dragged within their parent (sortable container) and have non-selected siblings sort themselves as if the selected draggables were a single draggable item. My current setup looks like this:

Would this be possible?

I am not a maintainer, but as I understood, you have to implement this manually.
So you have to render appropriate overlay, and track selection to handle that in onDragEnd event, and move items appropriately.

I am not a maintainer, but as I understood, you have to implement this manually. So you have to render appropriate overlay, and track selection to handle that in onDragEnd event, and move items appropriately.

Thanks a lot for the hint... I was hoping there would be some interface already available in DND Kit since there is an example here


Slack - User context tool gets called when we are chatting with an agent about things unrelated to slack.

Steps to reproduce the problem:

Create an agent with Slack and github tool bundles.
Chat with the agent, about anything unrelated to Slack

Notice that Slack User context tool gets called.
I see Slack User context tool  getting called even when I chat about github tool.

Note - This issue is also seen when creating an agent with just Slack  tool as well or an agent with slack and any other tool bundle.
I have seen this issue fixed as part of validating - https://github.com/otto8-ai/otto8/issues/492. But seeing it happen again.

Merged.

This issue is not seen anymore when testing with latest version of the tool.


Properties rules in json

Add all test properties description in JSON file like it is done in spring
https://github.com/spring-projects/spring-boot/blob/master/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/resources/META-INF/additional-spring-configuration-metadata.json

https://epam.sharepoint.com/:w:/r/sites/JDIDarkTeam/_layouts/15/doc.aspx?sourcedoc={96c6370f-2b6d-41c9-af5e-c8a395326ccf}&action=edit&cid=d4308b07-7eb3-41ae-9499-765eacb997ec
list of  test properties

Correct English text (replace "next" with "following", check plural)
Move file to JDI Dark core from example resources

Done by Julia


Add terminal-2 icon

Preview

terminal-square?

terminal-square?

What about terminal-app?

What about terminal-app?

any updates on this PR?


[Bug]The stacking context of multiple components when they being used one after other on canvas

Description The stacking context of multiple components on Appsmith editor.
[What happened]
Steps to reproduce the behaviour:
1.Go to the canvas
2.Drag and drop the widget
3.add the data source
4.Try to bing the query which has the issues
5.Now open the debugger
6.verify the display

Important Details

Version: [Cloud ]
OS: [MacOSX]
Browser [ safari]
Environment [release]

@riodeuno @somangshu Should we work on this before we are done with the docking of the property pane? If not, I say we just bring the property pane in front of the other things over here but do not let it go over nor behind the entity explorer or the navbar.


Create new Dirmon Entry from an existing instance process

I've created the new about Create new Dirmon Entry from an existing instance. Next step to create a Jira ticket for addressing this process

I'm going to explain what I've done:
First idea: when the copy.html.erb calls the existing create method, the Copy process works as expected with a route patch :create (It creates a new DE from an existing DE with pending status), but the Edit process doesn't work.
I got the below message without the route patch :creat

I got the below message when I tried to update a DE

Second idea: when the copy.html.erb calls the new replicate method the Copy and Edit process work as expected with a route patch :replicate.
I got the message when I set up a route post :replicate this why I didn't use the post route

I've read this about how works Patch routes https://weblog.rubyonrails.org/2012/2/26/edge-rails-patch-is-the-new-primary-http-method-for-updates/


fluentd-gcp-v2.0 does not start on deployment.

Error: DaemonSet has no nodes selected

H01 has the same error in both global and local.


miss route headers in UAS BYE request

I found that there is no route headers in UAS BYE request.
It was removed in this commit, is this a bug or is there some other reason?

yeah I see bug. Thanks for reporting

@hateeyan Check the latest main branch

@emiago Thanks, it works fine!


Camdozaal v1.1 - Migrate Camdozaal plugin from old repo to new one.

You should really just not change the plugin's name, leave it in the camdozaal file

You should really just not change the plugin's name, leave it in the camdozaal file

I would like to go forward with the name change.

I too would prefer if it wasn't renamed. There is no reason to rename it and it just causes headaches

I've decided to disallow renames like this since it is entirely pointless. Do not rename the file in this repo

As a sign of protest I have decided to request disabling this plugin


Weird Segmentation Fault

It appears the segmentation fault while commenting out line 19, printf("%d", numbers[i];. Then the rest of the program wouldn't work as expected.

Turns out should be more careful while using malloc.


Review text color in themes for CommandBarFlyout example

Is this a totally new sample, an addition to an existing sample page, or a modification of an existing sample page?

[ ] New sample
[ ] Addition to existing sample
[X] Modification to existing sample

If this is an addition/modification to an existing sample page, which one?
CommandBarFlyout
Describe the sample
When you toggle theme to 'light mode', and click the 3-dot option on the flyout: the 'Resize' and 'Move' options are hard to read.
Is your sample request related to a problem? Please describe it

Not a problem necessarily, perhaps from an accessibility perspective? Just hard to read the options.
Additional context

This was addressed at some point:


Doesn't respect --target-binary-version flag.

We generate our versionName and versionCode values dynamically, so with code-push in the past we've had to manually specify the target binary version using the flag --targetBinaryVersion.
First off, with appcenter that flag is now listed as --target-binary-version. That's fine.
However, even if you specify the --target-binary-version or -t flag when using appcenter codepush release-react, it still complains about not being able to find the version and actually says you should use the --targetBinaryVersion flag:
Detecting android app version:

Error: Command 'codepush release-react -a joshuapinter/CNTRAL-Mobile --deployment-name Staging --entry-file index.android.js --gradle-file ./cntral/build.gradle -t 6.53.0' failed with exception "The "android.defaultConfig.versionName" property value in "./cntral/build.gradle" is not a valid string. If this is expected, consider using the --targetBinaryVersion option to specify the value manually."

Neither of those flags work, tho.

@joshuapinter thanks so much for pointing at this, appreciate it!
Here is PR that should fix it: https://github.com/Microsoft/appcenter-cli/pull/314

Fixed in release 1.0.4.


settings.php file not shared between replicas, causes havoc

Since the default settings.php that Drupal ships with doesn't use environment variables, Drupal gets... funky when you have multiple containers running and only one of them has the installed settings.php file.
In other words... you really need to build Drupal outside the running container then drop it into the container with a flexible/generic settings.php.
And apparently geerlingguy/drupal isn't currently doing that.
Will need to figure this out at some point :)
Related: https://github.com/geerlingguy/raspberry-pi-dramble/issues/111

For now going to bump replicas back down to 1 for demo purposes.


Candlestick shape customization

I'm interested in implementing candlestick enhanced customization:
1. Rounded candlestick body.
2. ctx.lineCap = 'round' candlestick shadow.
Attaching image from my svg-chart project:

Hi,
thanks interesting beautiful suggestion.

Rounded candlestick body.

It might affect performance, because instead of fillRect to draw candlestick we need to use path (lineTo, moveTo, arc, etc) to draw one candle. I haven't number right now, but we need to really worry about that here. We can try to do it with noting that rounded candlesticks might reduce fps/ux.

ctx.lineCap = 'round' candlestick shadow.

It works for stroking, but currently (and will be in the future I believe) wicks drawn with fillRect due 1) performance and 2) pixel-perfect. I don't think that we'll decide to use stroking instead, so we have only 1 solution here - use paths... That's all is a bit ambiguous (at least for me). Yeah, we need to measure perf before make any change here.

Thank you for detailed response.

I'm going to close the issue. If you'll have any question or updates - feel free to re-open it.


Set badge icon when app is closed

Hi,
Is there a way to set badge icon when app is closed? Event onNotificationReceived is triggering only when app is in foreground or background, but I need to set number of notifications on icon when user receive notification and his app is closed.
Thank you.

I'm having the same issue, does anyone have an idea on this ?

I am not receiving push notifications when the app is killed. Anyone know why? It works correctly when the app is in the foreground and background.

try adding the following in AppDelegate.m

(void)userNotificationCenter:(UNUserNotificationCenter *)center willPresentNotification:(UNNotification *)notification withCompletionHandler:(void (^)(UNNotificationPresentationOptions options))completionHandler {
completionHandler(UNNotificationPresentationOptionBadge | UNNotificationPresentationOptionSound | UNNotificationPresentationOptionAlert);
}

I found a way to do this. Appending a field "badge" in notification payload while sending the push from backend (in my case it was sent from a cloud function).
var payload = {       notification: {         title: valueObject.user.name,         body: valueObject.text,         sound: "default",        badge:'10',       },       data: {         title: title,         message: message       },     };


[FR] - Sensible, step-by-step install and config documentation

After attempts on mac and ubuntu and running into errors at every step, I humbly ask you to produce a document with clear steps to install and run a node.
I.e. what's a genesis file, config.yaml, cabal vs nix, /.state db blah blah i'm frustrated
Internal/External
Internal   if an IOHK staff member.
External   otherwise.
Area
Token Locking    Related to Token Locking (Allegra).
Native Tokens    Related to Native Tokens (Mary).
Other   Any other topic (Delegation, Ranking, ...).
Describe the feature you'd like
A clear and detailed description of what you are suggesting.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context / screenshots
Add any other context or screenshots about the feature request here.

Please specifically describe the errors you are running in to.

Closing this. If this is still relevant please reopen.


feature: support for k8s controller for autoscaler and ksctl agent

Tasks description
Issues
Completed Issue(s)

https://github.com/ksctl/enhancements/pull/1
#251

Related Issue(s)

#301

Solution
Sub-Tasks

[ ] milestones or sub-task which needs to be coverted

Note to reviewers

[ ] Ran Tests locally
[ ] Checked Contribution's guidelines

@prateek041 you can close the PR #301 and use this branch instead

this will be helpful
https://protobuf.dev/programming-guides/proto3/

For trying out the server you can use grpcui

@prateek041 we will be requiring a new ksctl storageProvider for storing in the configmap and secrets
so that its very consistent when we switch between 2 providers when moving from local -> k8s for other operations

TODO 📓

[ ] need to provide proper versioning for the ksctl controllers
[ ] assess any missing part
[ ] Improving the codebase
[ ] Come up with the automation for the releases of the new controllers
[ ] Fix the Issue and PR linking as we are not going to add feature for the ksctl autoscaler in this PR cycle


Err: Couldn't download certificate, got this instead: {"There is no certificate with ID &#x27 ...

Issue Description
When performing 'sigh' command I get an error. This didn't happened prior to the upgrade to latest version. (1.66.0)
Error:
download_certificate': [!] Couldn't download certificate, got this instead: {"responseId"=>"9ff3e459-9ebc-42f3-a149-b53575d2508d", "resultCode"=>7100, "resultString"=>"Certificate 'VGFB8JESL7' not found", "userString"=>"There is no certificate with ID 'VGFB8JESL7' on this team.", "creationTimestamp"=>"2016-03-04T20:47:05Z", "protocolVersion"=>"QH65B2", "userLocale"=>"en_US"

I am assuming that the : **' ** characters in this output is a verbose character issue in how it translate to the debug output, but if not, that might be the issue. The only change from working environment to a none-working environment was the latest upgrade of fastlane.
Complete output when running fastlane, including the stack trace and command used
This output happens weather i run : fastlane beta    or    sigh  directly from the command line in the project folder.
+-------------------------------------+-----------------+
|                Summary for sigh 1.4.0                 |
+-------------------------------------+-----------------+
| adhoc                               | false           |
| skip_install                        | false           |
| development                         | false           |
| force                               | false           |
| app_identifier                      | xxxxx         |
| username                            | apps@xxxx         |
| ignore_profiles_with_different_name | false           |
| skip_fetch_profiles                 | false           |
| skip_certificate_verification       | false           |
+-------------------------------------+-----------------+

[14:47:00]: Starting login with user 'xxx@xxxx'
[14:47:03]: Successfully logged in
[14:47:03]: Fetching profiles...
-------------------------------------------------------------------------------------------
😨  An error occured. Please enable crash reports using `fastlane enable_crash_reporting`.
👍  This makes resolving issues much easier and helps improve fastlane.
🔒  The reports will be stored securely on getsentry.com.
🙊  More information about privacy: https://github.com/fastlane/fastlane/releases/tag/1.33.3
-------------------------------------------------------------------------------------------
/Library/Ruby/Gems/2.0.0/gems/spaceship-0.22.0/lib/spaceship/portal/portal_client.rb:311:in `download_certificate': [!] Couldn't download certificate, got this instead: {"responseId"=>"9ff3e459-9ebc-42f3-a149-b53575d2508d", "resultCode"=>7100, "resultString"=>"Certificate 'VGFB8JESL7' not found", "userString"=>"There is no certificate with ID 'VGFB8JESL7' on this team.", "creationTimestamp"=>"2016-03-04T20:47:05Z", "protocolVersion"=>"QH65B2", "userLocale"=>"en_US", "requestUrl"=>"https://developer.apple.com/services-account/QH65B2/account/ios/certificate/downloadCertificateContent.action", "httpCode"=>200} (Spaceship::Client::UnexpectedResponse)
	from /Library/Ruby/Gems/2.0.0/gems/spaceship-0.22.0/lib/spaceship/portal/certificate.rb:309:in `download_raw'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:87:in `block (2 levels) in fetch_profiles'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:85:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:85:in `block in fetch_profiles'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:82:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:82:in `find_all'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:82:in `fetch_profiles'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/runner.rb:20:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/lib/sigh/manager.rb:7:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/bin/sigh:35:in `block (2 levels) in run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:153:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/runner.rb:428:in `run_active_command'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane_core-0.37.0/lib/fastlane_core/ui/fastlane_runner.rb:23:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/delegates.rb:15:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/bin/sigh:91:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.4.0/bin/sigh:109:in `<top (required)>'
	from /usr/local/bin/sigh:23:in `load'
	from /usr/local/bin/sigh:23:in `<main>'

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Fastfile:
# =======================[SETUP]=======================
fastlane_version "1.48.0"
default_platform :ios

platform :ios do
# =======================[BEFORE ALL]=======================
  before_all do
    ENV["SLACK_URL"] = "https://hooks.slack.com/services/xxxxxx"

    ensure_git_status_clean
 end

  desc "Runs all the tests"
  lane :test do
    snapshot
  end

# ======================= ======================= =========
# ACTIONS
# ======================= ======================= =========

# =======================[BETA]=======================
  desc "Submit a new Beta Build to Apple TestFlight"
  lane :beta do
    slack(
        message: "[Fastlane] Starting GIT Activities (BuildBumps/Commit/Push).",
        payload: {
            'At :' => Time.new.to_s
        })

    ensure_git_branch(
      branch: 'develop'
    )

    git_pull
    increment_build_number

    commit_version_bump(
      message: 'minor [CI] Build Bump'
    )

    push_to_git_remote

  slack(
      message: "[Fastlane] Starting Signing of provisioning profiles.",
      success: true,
    payload: {
        'At :' => Time.new.to_s
    })

    sigh

    slack(
        message: "[Fastlane] Building the App (XCODE)",
        success: true,
    payload: {
        'At :' => Time.new.to_s
    })

    gym(scheme: "xxxxx"
        ) # Build your app - more options available

    slack(
        message: "[Fastlane] Delivering app to Testflight servers.",
        success: true,
    payload: {
        'At :' => Time.new.to_s
    })

    pilot(distribute_external: false)

    slack(
        message: "[Fastlane] Cleaning GIT Repo.",
        success: true,
    payload: {
        'At :' => Time.new.to_s
    })

    reset_git_repo

  end

  # ======================= ======================= =========
  # GENERAL HANDLERS
  # ======================= ======================= =========
  # =======================[AFTER ALL]=======================
  after_all do |lane|
    # This block is called, only if the executed lane was successful

     slack(
       message: "Successfully deployed new App Update to TestFlight",
       success: true
     )
  end

  error do |lane, exception|
     slack(
       message: exception.message,
       success: false
     )
  end
end

# More information about multiple platforms in fastlane: https://github.com/KrauseFx/fastlane/blob/master/docs/Platforms.md
# All available actions: https://github.com/KrauseFx/fastlane/blob/master/docs/Actions.md

Environment
fastlane version (fastlane -v): 1.66.0
Do you use bundler, rbenv or rvm? Nope

I'm seeing this exact issue. I have enabled crash reporting on fastlane and ran sigh and this is the ID it gave me for the crash report: ae9ab8c36b43bcd2a80816bbf92b0d93 Might it have something to do with the new Apple Dev Portal?

I'm hitting this same issue too.

I have tried reinstalling and have the same
Issue on two computers.

same here, fastlane (1.66.0), sigh (1.4.0), spaceship (0.22.0) - both on local machine and CI

Same here (fastlane 1.66.0 | sigh 1.4.0 | cert 1.4.0 | 0.22.0)

I confirm this issue. Downgrading to older versions of fastlane, sigh and spaceship didn't solve the problem

I believe the issue is with sigh using spaceships to download and/or find a certificate on apple server. Downgrading does not work because Apple changed the iTunes connect portal. And using the latest version does not work either because of this bug.

:+1:

That the same error I get when I try to update devices in existing provision profile
Sent from my iPhone5

On Mar 7, 2016, at 8:24 AM, Stephan Heilner<EMAIL_ADDRESS>wrote:
—
Reply to this email directly or view it on GitHub.

Can anyone comment if this got fixed on the latest release?
I am unable to access my server at this moment.

After much debate, I cleared all my provisioning profiles from the local computer and the developer portal. Started from scratch and it started to work.
So in my case this was fixed by replacing all my certificates. Can't tell which one was the culprit.

In my case, I finally got it working by installing the new Apple WWDR root certificate. The one I've be using for years expired at the end of February. Installing the new root certs solved the problem.

So I am going to close the issue then.
Thanks everyone!

cleared everything with match nuke; then generated new certificates with match; and then sigh worked just fine


Configure notifications for cron triggered workflows

Configure sending out notifications to multiple collaborators for cron triggered workflows on GHA.
Currently only the latest user who pushed onto the repository gets a notification, please see here.

This also affects workflows run on schedule events for message_ix and eventually other repos, so any fix can also be applied there.


[Select] Select content position in wrong location

Bug report
Current Behavior
When a Select is within an element with position: absolute, clicking on the trigger opens the content in a completely different position.
Expected behavior
The content should open right above the trigger.
Reproducible example
CodeSandbox
Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-select
0.1.2-rc.50

React
n/a

Browser
Chrome

Assistive tech

Node
n/a

npm/yarn

Operating System

Hey @stevenfabre, it doesn't look like the demo is working, maybe forgot to save the app file?

https://user-images.githubusercontent.com/2289736/177185487-5557f245-e507-432d-af02-b22bea18a4c5.mp4
Hi @benoitgrelard, I may have not save the CodeSandbox properly. Here is what I'm seeing in that updated
CodeSandbox.
Are you able to reproduce?

Ah, I just realised the problem, you are using the latest release candidate, but are not adding the new Portal part wrapping Content. If you add it, it should work fine. I'll close the issue, but let me know if you have other problems.

I am running into this same issue when trying to render a Select inside of a Dialog.

If I wrap the Select  Content inside of a portal - the content is not visible at all
If I omit the Select Portal - the content is visible but the position is in correct

Are you able to replicate this in a sandbox?

Are you able to replicate this in a sandbox?

I've determined this was due to other styles being applied via tailwind/daisyui


ResNet based SN-DCGAN model for training ILRSVRC2012 dataset?

Will you release the code of ResNet based SN-DCGAN model for training ILRSVRC2012 dataset? I'm curious about the details of this network architecture. Thanks.

Hi, thanks for your attention.
We will release the source code for reproducing the results after the notification of the decision (and maybe we may create another repository for the open source code).

Hi,
we have uploaded the code of ResNet for ImageNet: https://github.com/pfnet-research/sngan_projection.
Please look at the code if you are interested in.
Thanks,


PowerShell on Target Machine: Special Characters in password

I am running into an issue where my admin password contain special characters.  As a result, when executing the script, the authentication fails.
password - %D0`H\vW'RUc?buZ5
While I could, and probably will, in the short term, update the password to remove special characters, I wanted to bring this issue to your attention.  With password policies becoming more robust, it is only a matter of time, if not already, before this becomes an issue for others.
Error log excerpt:
2016-11-03T15:38:46.6387093Z Deployment status for machine 'ServerRemoved:5985' : 'Failed'
2016-11-03T15:38:46.6543337Z ##[debug]System.Exception: AuthorizationManager check failed.
2016-11-03T15:38:46.6699590Z ##[error]AuthorizationManager check failed.

Which task are you using?

@goofy78270
Fixing this is already in our backlog and we are picking it up in the coming weeks.

What task?

@goofy78270
This is fixed in "PowerShell on Target Machines" task. The password you provided works fine with task version 1.0.43.
Which version were you facing this issue with? Or is it some other task?

1.0.31 is the version we are currently using

@goofy78270 Nothing changed in the task between these versions related to this. Please provide me below details for reproducing the issue

TFS version
OS version of the agent
OS version of the target machine
PS version on the agent
PS version on the target machine

Please try creating a remote PS Session from your agent box to target box just to confirm the setup is good, and task is failing to create remote session. You can use below script
$cimSessionOption = New-CimSessionOption -Protocol Wsman; $cimSessionOption.CertCACheck = $false; $cimSessionOption.UseSsl = $false

$securePassword = ConvertTo-SecureString "%D0`H\vW'RUc?buZ5" -AsPlainText -Force

$cred = New-Object System.Management.Automation.PSCredential('domain\username', $securePassword)

$cimSession = New-CimSession -ComputerName ServerRemoved -SessionOption $cimSessionOption -Credential $cred -Port 5985

Still not working...

TFS version = 2015 - Version 14.102.25423.0
OS version of the agent = 2012 R2
OS version of the target machine = 2012 R2
PS version on the agent = 4.0
PS version on the target machine = 4.0

Using the same code as above, but my credentials, I am able to login without issue.

NOTES:
The trouble account and myself are both members of the admin group on the target server.
Restriction policy is set to unrestricted.
The credential is accurate as I am able to login to the server using the credentials for the troubled account.
UPDATE:  the actual password is %D0`H\vW'RUc?buZ5.  I must have removed a \ in testing various things...but even the correct login does not work for the remote session.

Using the following appears to work in the test listed above.
$securePassword = ConvertTo-SecureString '%D0`H\vW''RUc?buZ5' -AsPlainText -Force
note that I changed the double quotes to a single quote and escaped the single quote within the password

Sorry, issue not closed

@goofy78270
I tried with the same setup and things were working for me with 1.0.31 version of task too.
And, during build you were getting a different error than simple Access Denied.
Can we have a call where we can access your setup and debug? Please mail us at RM_Customer_Queries at Microsoft dot com.
Meanwhile please try again with below script:
When credential is hardcoded - the tick character (`) is treated as an escape character and script fails. Below version of the script doesn't hardcode the password - it takes input during execution
$cimSessionOption = New-CimSessionOption -Protocol Wsman; $cimSessionOption.CertCACheck = $false; $cimSessionOption.UseSsl = $false

$cred = Get-Credential

$cimSession = New-CimSession -ComputerName ServerRemoved -SessionOption $cimSessionOption -Credential $cred -Port 5985

By manually entering credentials during the test run, it works without issue, similar to changing the double quotes to a single quote and escaping the single quote as mentioned above.

I am unsure if it matters, but the password does not appear to be posting correctly.  The are supposed to be 2 \ in the password not just one - %D0`H\\vW'RUc?buZ5
maybe this double \ has something to do with the issue

Here is some further information:
I used the following to create a test to compare the results and while the password appears the same in both test, the results are different for the three test.  After this testing, I am unsure if this is script related or just a bug in powershell as the password shows correctly in all 3 instances, but the first fails for some reason

@goofy78270
Thanks for the info. I am not sure of the issue in 3 tests, but your target machine's WinRM settings are correctly set - the above script worked with Get-Credential.
But somehow the PowerShell on Target Machines task is failing. In the very first message I noticed that the error is not simple Access Denied. It is Authorization Manager check failed. We have to understand more - It will be helpful if we can set-up a 30min session where we can debug this together. (RM_Customer_Queries at Microsoft dot com)

I am out of office for 2 weeks. including @chshrikh and @rajatagrawal-dev
@chshrikh
At first it seemed like the special character issue. But now, I strongly believe it is not. If it is, it should have thrown Access Denied error. Not Authorization Manager check failed. We should check if it is the script issue.

Create this password as a non-secret variable in build. And try to print this in the PS on Target Machine task. Check if it is received correctly.
Use a hello-world script to see if this is a script issue

@goofy78270  please let us know if you are still blocked on this

I am unsure where the issue lies, but we have moved forward by simply removing special characters from our passwords.  Everything seems to work great now.  If there is a need for us to look into this again, I will reopen this case.
Thanks for all your help and sorry for running in circles.

https://github.com/PowerShell/PowerShell/issues/4024

Hi Team, I am working on a PS script to install VSTS agents on my Azure VMs through an Azure DevOps Release Pipeline.
Here in the below script, I am passing a Service Account Password, which is set by my customers. Below script fails whenever Password consist of special characters like

")';:(

Code:
 .\config.cmd --deploymentgroup --deploymentgroupname DeployGroupSQL --agent $env:COMPUTERNAME
--runasservice
--windowsLogonAccount $(ServiceAccountUser)
--windowsLogonPassword "$(Service_Account_Password)" --work '_work'
--adddeploymentgrouptags --deploymentgrouptags $SQL
--url 'https://dev.azure.com/campusmgmt-product-development/'
--projectname 'Student' --auth PAT --token u4bq6j******************************bhhyq;

I have tried putting whole password into double quotes as well as single quotes but of no help.
This Service Account Password is a Pipeline variable and hence needs to be passed like $(Service_Account_Password).
Can you please suggest on how can I deal with this situation. I have already tried putting this as a here-string, Secure-String, or script parameter but has not proved of any help.
Hoping to hear back soon as this is urgent for my project release.

Feel its my luck day today. After banging my head for past 8 hours on this I could finally work this through using Stop-Script character in PowerShell '--%'
Solution I tried:
.\config.cmd --deploymentgroup --deploymentgroupname DeployGroupCTS --agent $env:COMPUTERNAME --runasservice --work '_work' --adddeploymentgrouptags --deploymentgrouptags $CTS --url 'https://dev.azure.com/campusmgmt-product-development/' --projectname 'Student' --auth PAT --token u4bq6j************************************hhyq --windowsLogonAccount $(ServiceAccountUser) --windowsLogonPassword --% "$(Service_Account_Password)" %;%
Ref: https://stackoverflow.com/questions/18923315/using-in-powershell
Hope this might helps others.

Thanks @praharshp it was about 4 hours into it when I found your post. For me it worked with a combination of what you posted and the stackoverflow link you provided here is my solution (it just a powershell script in a local machine):
First I assigned the password to an environment variable
$env:password =$password
Then I use the %password% syntax with the '--%' in front of it:
& ".\config.cmd" --unattended --url "http://MyServer/Tfs"  --auth "Integrated" --pool $PoolName --agent $agentName --runAsAutoLogon --windowsLogonAccount $user --windowsLogonPassword --% "%password%"
I hope this helps others as well.


[FEATURE]

Description
The script.js file is currently exposing the firebase API key.
I think it would be a good practice to secure our key( a free one too) by creating a dotenv file in .gitignore to secure the keys to prevent any possible security issue ( bots may scan for API keys and access/use resources)
Screenshots
No response
Additional information
No response

I would like to work on it if you think the issue is valid

Correct indeed! sure you can work on it in the coding period :)

@ansharora28 are you still working on this issue?

@kshitij01042002 Hi,yes I'm working on it.

@ansharora28  have you created .env file it? if yes please raise a PR soo that I can document it for issue #113

@kshitij01042002 I am facing some some problems after moving the keys to the dotenv file. I will take a look at it again and ask the mentor to unassign me if I'm not able to work it out.
You can proceed with the issue then

Okay Works!

@narayan954 I was not able to figure out the problem with my code. Hence,I would like to explore other issues. You can assign it to @kshitij01042002

@narayan954 can you assign me this under codepeak?

@narayan954 while creating .env file and accessing it. The application is crashing which seems to be a common problem with no solid solution.Soo I suggest rather than creating a dotenv file we should document how to integrate app with firebase and modify firebase.js

@narayan954 while creating .env file and accessing it. The application is crashing which seems to be a common problem with no solid solution.Soo I suggest rather than creating a dotenv file we should document how to integrate app with firebase and modify firebase.js

Okay, we can add instruction to generate own API key as you're saying for now. This issue remains open till a solution is found I guess.

Hey, @narayan954 please assign this issue to me

Hey, @narayan954 please assign this issue to me

under iwoc?

Yess

Great! go ahead :)

@narayan954 I have added all the env variables and they are coming correctly as I console. logged them. But firebase is giving me an error saying "invalid API key"

@narayan954 I have added all the env variables and they are coming correctly as I console. logged them. But firebase is giving me an error saying "invalid API key"


Make compatible with polars 0.28

This should make pyo3-polars compatible with polars-0.28.

Thanks Marius. We don't have any CI setup here (yet), so I will trust you here. :)


Fix formatting typo & case-sensitive image paths

Windows 10 S Pro header didn't have a space before ##
URLs of some images were in the wrong case, breaking GitHub preview.

@lizap @AndreaBarr - PR is ready for final review. This is just for typo/cosmetic issue.
Thank you.


rm req for ords download; non-root for OL8

@rhondaday, can you give this change a once-over and approve if good?


Local ORMs

Hey, maintainer of http://aphrodite.sh here.

Trong ORM is a JavaScript library for building reactive SQLite queries for web, mobile and desktop

Looks like we're thinking along the same lines :)
Lmk if you'd like to bounce any ideas around when it comes to local ORMs.
On my end I've been wondering if native IndexDB support is worth it or to only support AbsurdSQL / sql.js.
Also, looks like AbsurdSQL isn't very well maintained anymore. The sql.js author is also looking for a new maintainer. Maybe those are things we could jointly work on / take over together.
Last thing -- have you seen or tried to work with https://github.com/rhashimoto/wa-sqlite/blob/master/README.md ?

Hi @tantaman !
I happy to see you here 🙂
I checked your project — looks very cool!
Great idea, I like how it allows to generate strict typings for your queries, looks like prisma 🙂. But as for me I want to make deep integration to the SQLite, without any other DBs support. To use all the power of the SQLite(which is really poerwfull!).

On my end I've been wondering if native IndexDB support is worth it or to only support AbsurdSQL / sql.js.

I play around with IndexedDB a lot, and I can say it doesn't worth. I used Dexie, PouchDB — they all sucks on large datasets, and lacks support of, for example, full-text search. SQLite just far away in terms of performance and utility.
And one other thing, IndexedDB has a lot of bugs that varies from browser to browser. Absurd SQL make more simple read/write(as I understand it read/write with blocks of data) and absurd SQL breaks rarely (Acutally you could find all the bugs at Dexie.JS repo at issues)
I was using Dexie in https://github.com/quolpr/harika for ~4-5 month, and switched to Absurd SQL once it was released, and already use it for ~5 months. It works very stable and very performant.
One downside of AbsurdSQL — no iframe and other things support due to COOP https://github.com/jlongster/absurd-sql#requirements . But one of the workaround is to try service worker workaround — https://github.com/jlongster/absurd-sql/issues/17
I was speaking with author of AbsurdSQL. He is pretty busy, but I hope he will give me a maintain rights. Actually he wanted to give me them ~6 month ago 😅 I would be happy to maintain it together.
As for https://github.com/rhashimoto/wa-sqlite — looks cool! But the license is GPL, but I want to have MIT for my project.

But as for me I want to make deep integration to the SQLite, without any other DBs support.

Yeah, there is a really good case to only support SQLite, some of that case being made here: https://changelog.com/posts/sqlite-web-renaissance
I think it's a solid decision.

@tantaman thanks, cool article 👍

@tantaman there are some insides about Riffle https://www.youtube.com/watch?v=BmgL1SRfUyc, maybe you will be interested 🙂

Btw, do you want to make trong-orm as a query engine? It already supports expo, electron, vite, CRA. I'm also working on doc https://trong-orm.netlify.app/ .
You can you it without any react integrations, it is totally framework-agnostic. To use it, you just need:
import sqlWasmUrl from "@trong-orm/sql.js/dist/sql-wasm.wasm?url";

const db = await initDbClient({
  dbName: "helloWorld",
  dbBackend: absurdWebBackend({
    wasmUrl: sqlWasmUrl,
  }),
  plugins: [
    // migrationsPlugin({ migrations: [] }), // uncomment if you need migrations support
    // reactiveQueriesPlugin(), uncomment if you need to support listenQuery https://trong-orm.netlify.app/listen-queries ( https://trong-orm.netlify.app/how-reactivity-works )
  ],
});

await runQuery(db, sql`SELECT 1`); // or you can use query-builder

// or transaction

await transaction(db, () => {
  await runQuery(db, sql`DELETE * FROM ${table('notes')}`);
})

// You can subscribe to queries (only if reactiveQueriesPlugin used), and reactivity will work in multi-tab

const notesTable = sql.table("notes");

listenQueries(db, [select().from(notesTable)]).subscribe((res) => {
  console.log("Queries result: ", res);
});

It also supports middlewares on query run, and some events on transaction run. Actually reactiveQueriesPlugin ( https://github.com/trong-orm/trong-orm/blob/main/packages/reactive-queries-plugin/src/reactivePlugin.ts#L24 ) uses transaction events and query middlewares.
Let me know what do you think 🙂

And you can check all examples:
With CRA: https://github.com/trong-orm/trong-cra-example
With Vite + React: https://github.com/trong-orm/trong-orm/tree/main/packages/vite-react-example
With Expo (native+web): https://github.com/trong-orm/trong-expo-example
With Electron: https://github.com/trong-orm/trong-electron-better-sqlite3-example

@tantaman hey! I built the vFS for wa-sqlite with the performance that absurd-sql has (or almost the same, sometime 10% difference). You can run benchmark here, and see the VFS implementation here. The code is pretty messy, but it does the job. I plan to refactor it.

So, who will be here that will make MIT version of wa-sqlite? 😅

Awesome. Do you see any advantages of wa-sqlite over absurd-sql?

No need to use COOP. With COOP you are not able to use iframes from the other domain (like youtube.com). Also, I noticed that wa-sqlite version starts faster (especially on big data sets) — but I didn't measure it.

Awesome. Do you see any advantages of wa-sqlite over absurd-sql?

@tantaman Absurd is MIT licensed and WASQL is GPL V3. Big difference depending on your goals.


/etc/monit/conf.d/kube-proxy on master seems to be outdated

root@kubernetes-master:/home/biskup# cat /etc/monit/conf.d/kube-proxy
check process kube-proxy with pidfile /var/run/kube-proxy.pid
group kube-proxy
start program = "/etc/init.d/kube-proxy start"
stop program = "/etc/init.d/kube-proxy stop"
if does not exist then restart
if failed
  host <IP_ADDRESS>
  port 10249
  protocol HTTP
  request "/healthz"
then restart

root@kubernetes-master:/home/biskup# ls /etc/init.d/kube-proxy
ls: cannot access /etc/init.d/kube-proxy: No such file or directory

The file is created from ./cluster/saltbase/salt/monit/kube-proxy

I will take a look at this quickly.


Consider using data URI scheme instead of credits_en.html

Description
Spun off from #777.
Currently about_dialog.cc tries to open credits_en.html from SystemUtil::GetDocumentDirectory() as follows.
https://github.com/google/mozc/blob/b6aa2a31255f83b248de93eee99d07ecee795799/src/gui/about_dialog/about_dialog.cc#L77-L79
However it means that each installer (including the one made by Linux distribution package maintainers) needs to correctly install credits_en.html into the target system, which is fragile and hard to maintain as we've seen in #777.  Let's see if we can use data URI scheme to embed the data into mozc_tool executable or not.
Steps to reproduce

Open mozc_tool
Click 'open source software'

Expected behavior
See the opensource license information without relying on credits_en.html.
Version or commit-id
b6aa2a31255f83b248de93eee99d07ecee795799
Environment

OS: all desktop

It turns out that this is not possible at least in Windows due to the size limit of CreateProcessW.

The maximum length of this string is 32,767 characters, including the Unicode terminating null character.

Unfortunately our credits_en.html is already too long to be passed with CreateProcessW. So closing as "not planned" for now.
For future reference, we could still embed credits_en.html into mozc_tool by showing it with either  QTextDocument::setHtml() (or converting it to markdown then showing it with QTextDocument::setMarkdown()).


Add security audit concept doc

Initial drop of security auditing concept doc. Will require screenshots and we can elaborate more on other methods as well such as dotnet list package --vulnerable and NuGet.org features if desired.

Should be good to review and ship


Bump dcos-ui for 1.9.1

High Level Description

Can't delete CMD in UI or JSON editor
Correct pod definition gives error
Environment vars for Pods are broken
Volumes size should say GiB not MiB
Zeppelin package is gone from Universe in UI
Incorrect Resource Reporting
Update help text and tooltips in create flow
"EDIT" button in services UI doesn't do anything when clicked

Related Issues

DCOS-14894
DCOS-14842
DCOS-11781
DCOS-14854
DCOS-15137
DCOS-13077
DCOS-15014
DCOS-14890

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: unit, integration
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@orlandohohmeier @MatApple could you please review


🐛(skin) fix education terms link

Purpose
The full target URL of the target page should contain a trailing slash.
Proposal
Fix terms URL :sweat_smile:

don't forget the changelog :-D

don't forget the changelog smile

I don't need to, it's a fix for the previous un-released change :sweat_smile:


Discussions

What is your question?


Enable SSH forwarding of the API server

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Proxy settings as --chromium-args not working

I want to use a proxy server for browservice (in particular, to use it in conjunction with Protoweb). This makes for a retro-cool web browsing experience on my vintage PC collection!
Supplying the proxy server as a chromium argument when starting browservice doesn't seem to have any effect. Browservice starts but it still takes me to the non-proxied URL, This may be user/syntax error so happy to be corrected!
I am currently using the following command line on Windows 11 (v <IP_ADDRESS>):
browservice.exe --show-soft-navigation-buttons=YES --vice-opt-http-listen-addr=<IP_ADDRESS>:8080 --chromium-args=proxy-server("http=wayback.protoweb.org:7851")
Any help appreciated. To use protoweb with browservice I have to set the proxy settings on my host machine or use a VM, both of which are quite big workarounds!

The following is working on Linux:
browservice-v<IP_ADDRESS>-x86_64.AppDir/AppRun --vice-opt-http-listen-addr=<IP_ADDRESS>:8080 --show-soft-navigation-buttons=YES --chromium-args=proxy-server="http://wayback.protoweb.org:7851" --start-page=http://www.yahoo.com/


vector.im/{beta,staging,develop} and riot.im/{app,staging,develop} permalinks should work as well as matrix.to ones

It looks like we might have stopped picking up non-matrix.to permalinks in the app?

Weird
https://vector.im/develop/#/room/#irc_#matrix:openmarket.com/$147489683046681qGbJa:matrix.org
https://riot.im/beta/#/room/#irc_#matrix:openmarket.com/$147489683046681qGbJa:matrix.org
https://vector.im/beta/#/room/#irc_#matrix:openmarket.com/$147489683046681qGbJa:matrix.org
https://riot.im/staging/#/room/#irc_#matrix:openmarket.com/$147489683046681qGbJa:matrix.org
https://vector.im/staging/#/room/#irc_#matrix:openmarket.com/$147489683046681qGbJa:matrix.org


Add Item from ctr VS. Add item by method

Hello, if i try write this code:

I have empty listView. But if i write this code:

Then already ok, What I'm doing wrong?
I use retrofit+your library

Hi,
EasyAdapter has several constructors but there isn't one to add a single item. The one you are using in the first case is:
/**
     * Constructs and EasyAdapter with a Context, an {@link ItemViewHolder} class and a generic listener.
     *
     * @param context             a valid Context
     * @param itemViewHolderClass your {@link ItemViewHolder} implementation class
     * @param listener            a generic object that you can access from your {@link ItemViewHolder} by calling
     *                            {@link ItemViewHolder#getListener()}, This can be used to pass a listener to the view holder that then you
     *                            can cast and use as a callback.
     */
    public EasyAdapter(Context context, Class<? extends ItemViewHolder> itemViewHolderClass, Object listener) {
        super(context, itemViewHolderClass, listener);
        mListItems = new ArrayList<>();
    }

As you can see, the third parameter is a generic object that is intended to pass a listener/callback to your view holder, so the third parameter is NOT a data item for the adapter and that's why your code is not working. If you want, you can pass a list of data items in the constructor but not a single one.


Cannot change search text without switching pages

The Issue
Something must have broken with the way we handle search text after searching in a recent version. You can select, but it won't overwrite it until you switch pages / then retype.
System Configuration

Device:
Android version:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of the user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

Fixed. Creating a new build.


CLOUD-55500 fix gcp disk type

@doktoric

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2314/


[BUG] Python bindings will not compile under Windows

Describe the bug
When using Microsoft's "vcpkg" package manager to build and install OpenImageIO with Python bindings, a CMake error is produced, and the product will not build.
To Reproduce
Steps to reproduce the behavior:

Install Microsoft's package manager:
git clone https://github.com/microsoft/vcpkg
Bootstrap the vcpkg source clone:
.\vcpkg\bootstrap-vcpkg.bat
Install OpenImageIO with python bindings:
.\vcpkg.exe install openimageio[pybind11]
Then THIS happens (reproduce the exact error message if you can)
Error: building package openimageio:x86-windows failed with: BUILD_FAILED

Expected behavior
Package builds and installs successfully.
Evidence
The PowerShell window indicates that additional error messages are available:
See logs for more information:     C:\Dev\vcpkg\buildtrees\openimageio\install-x86-windows-debug-out.log
Examining this log reveals the following error:
C:\Dev\vcpkg\buildtrees\openimageio\src\a25d056cb0-36fa00c6d2.clean\src\python\py_oiio.cpp(77): error C3861: 'ssize_t': identifier not found
If we are going to mess with the source and then attempt to reinstall, we should use the editable flag, like so:
.\vcpkg.exe install openimageio[pybind11] --editable
Reexamining the build log for the error points us to the following source file:
C:\Dev\vcpkg\buildtrees\openimageio\src\a25d056cb0-36fa00c6d2\src\python\py_oiio.cpp
Open this file, and insert the following code, starting at line 8:
#if defined(_MSC_VER) #include <BaseTsd.h> typedef SSIZE_T ssize_t; #endif
Save the file, and re-run the install:
.\vcpkg.exe install openimageio[pybind11] --editable
The package will now build and install without error.
It should be noted that I am by no means a qualified Windows developer - I'm pretty sure that ssize_t is a POSIX type, which is why that hack worked, but I'm not comfortable doing a pull request and making changes. However, I hope this bug report makes it easier for someone who is qualified to patch the code.
Platform information:

OIIO branch/version: master/<IP_ADDRESS>
OS: Microsoft Windows 10 Professional - Version 20H2 (OS Build 19042.1288)
C++ compiler: cl.exe - Microsoft C/C++ Optimizing Compiler Version 19.29.30136 for x86
Any non-default build flags when you build OIIO:

To fix that I modify header with:
using ssize_t = Py_ssize_t;

Is this still a problem?


First steps for espresso recipes

What new feature would you like to see?
Very nice project,
I would like to add some recipes for Quantum Espresso, especially for other executable ph.x, q2r.x, matdyn.x, bands.x and neb.x. I quickly created a quaacc.recipes.espresso.core module (have a look on my fork on the espresso branch)
I tried with covalent and everything seems to work fine, on my local machine anyway. Before going further, I am not sure what is the correct way to deal with parameters, pseudopotentials etc... should it be passed into each function call? should the pp_path be set using the ESPRESSO_PSEUDO env variable, etc...
Some guidances for 'best practices' would be highly appreciated!

Hi @tomdemeyere:
Thank you for the kind words and reaching out!

I would like to add some recipes for Quantum Espresso, especially for other executable ph.x, q2r.x, matdyn.x, bands.x and neb.x. I quickly created a quaacc.recipes.espresso.core module (have a look on my fork on the espresso branch)

This would be great! I would be more than happy to accept a PR of this type. I've outlined some ideas below, but I'm happy to help out and provide feedback as needed. Feel free to open a draft PR at anytime to request input.
Settings
Let's start with the easiest question first: environment variables and global settings.
In general, any environment variables or parameters that you don't want the user to need to set every single time they run a calculation should be defined in quacc.settings. You can see an example for VASP here. This makes it possible for the user to have a configuration file where all their settings are defined, as described in more detail here.
In the case of Espresso, you have a few options:

You could define a quacc setting ESPRESSO_PSEUDO that, if not None, would set os.environ["ESPRESSO_PSEUDO"] = ESPRESSO_PSEUDO.
You could define a setting QE_PSEUDO_DIR in quacc.settings that the user can specify. You can then import this global variable as from quacc import SETTINGS and then call pseudo_dir=SETTINGS.QE_PSEUDO_DIR in each recipe, although this might not be the best from a developer standpoint.

The first route is what I would recommend.
Execution
For the executable commands like the mpirun kind of stuff in the profile, you can copy what was done for ORCA.
Parameters
As you correctly identified, you can set reasonable defaults as calc_defaults, like you did for static_job. These can be overridden or added to with the any user-supplied **kwargs. You can also make other dedicated args or kwargs to static_job as you feel necessary. For instance, a variable that might make sense as a dedicated kwarg is the functional.
For something like the pseudopotentials, you have many possible options. My personal recommendation would be to define a pseudopotentials kwarg in static_job. If set to a dict, the recipe will use that (via pseudopotentials=pseudopotentials in the calculator call).
Since this may be unwieldy to have to do every time someone wants to run a calculation, one option might be to mimic what is done for VASP. I have made a bunch of YAML files containing "preset" lists of pseudopotentials, e.g. shown here. You could make a similar one for the recommended library of pseudopotentials for QE and then have a routine that: 1) Reads the YAML file. 2) Given an Atoms object, creates a dictionary for the pseudopotentials variable based on the atoms present and the entries in the YAML. Then, if the pseudopotentials kwarg to the recipe is of type str | Path, it can read that file and generate pseudopotentials from it for the calculator. This is similar to the preset kwarg I used in the VASP recipes.


Bump msgs, transport, and fuel-tools version in Harmonic

🎉 New feature
Summary
Bumping to use the new msgs, transport, and fuel-tools versions in Harmonic: msgs10, transport13, fuel-tools9
Depends on:
https://github.com/gazebosim/gz-fuel-tools/pull/326
Part of https://github.com/gazebo-tooling/release-tools/issues/878
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@osrf-jenkins run tests please

Fixed deprecation warnings to get ubuntu CI green in 2d87eda

CI is working!


Outdated links

Hi, i found an outdated link in courses/free-courses-id.md, specifically in

Belajar TypeScript Untuk Pemula - Eko Kurniawan Khannedy, Programmer Zaman Now

PR #10774 raised, thanks for reporting

Closing this issue as the pull request is merged and the problem is fixed.


Revise  to allow the encoding of a display label

Neither @n nor @label can be used on <verse> to encode a label, such as "1." that precedes the syllabic content of <verse>.   @label isn't acceptable for this because it's not supposed to be used for document content.
Adding  to the content of <verse> appears to be a workable solution --
<content>
  <rng:zeroOrMore>
    <rng:ref name="label"/>
  </rng:zeroOrMore>
  <rng:zeroOrMore>
    <rng:choice>
      <rng:ref name="dir"/>
      <rng:ref name="dynam"/>
      <rng:ref name="tempo"/>
      <rng:ref name="space"/>
    </rng:choice>
  </rng:zeroOrMore>
  <rng:oneOrMore>
    <rng:choice>
      <rng:ref name="volta"/>
      <rng:ref name="model.sylLike"/>
      <rng:ref name="model.appLike"/>
    </rng:choice>
  </rng:oneOrMore>
  <rng:zeroOrMore>
    <rng:ref name="model.lbLike"/>
  </rng:zeroOrMore>
</content>

When necessary, the @source attribute can be used to indicate the particular source(s) for each label.
Doing this will allow @n to continue to be used to indicate verses with a different order than encoding order.

With  available, there should be <labelAbbr>, too. When having multiple verses, some publishers print the verse number repeatedly on every new system.

@rettinghaus, can you provide an example please?

If <labelAbbr> will always be the same as , then it would not be necessary to have different from .  But if <labelAbbr> is not allowed, then the number-at-every-break style would have to be input externally in the graphical music rendering process, such as an option to verovio.  Allowing <labelAbbr> would function as a sort of option to display such numbers, in a similar manner as instrument abbreviations are implemented in verovio (if there is an abbreviation, then display it).  So in this case, if there is a <labelAbbr> then display it; otherwise, the default style of just numbers at the start of the lyrics would be displayed.
Here is a case that @rettinghaus described:
http://kern.humdrum.org/cgi-bin/ksdata?file=erk001.krn&l=users/craig/songs/erk/liederschatz/band1&format=pdf

There are also cases where the verse numbers change in the middle of a piece, which I would expect  implementations would be able to handle.

Thanks @craigsapp, this is exactly what I meant. So <labelAbbr> is, like for <staffDef>, an instruction for repetition, as I argued before in #361:

<labelAbbr> is fine, but I was wondering if not <labelRpt> would fit better into naming scheme. In the end it is a container for repeating the label. Could be less confusing if anybody asks why  goes over <labelAbbr> on the first line …

Sorry, I still don't get your point.  If you want to provide another label, repeated or not, use the  element at the appropriate place.  The  element simply divides the content of <verse> into a label and the syllables that follow --

  <measure>
    <staff>
      <layer>
        <note pname="f" oct="4" dur="8">
          <verse>
            1.
            <syl>A</syl>
          </verse>
          <verse>
            2.
            <syl>E</syl>
          </verse>
          <verse>
            3.
            <syl>I</syl>
          </verse>
        </note>
      </layer>
    </staff>
  </measure>

  <measure>
    <staff>
      <layer>
        <note pname="g" oct="4" dur="8">
          <verse>
            1.
            <syl>klopft</syl>
          </verse>
          <verse>
            2.
            <syl>Hol</syl>
          </verse>
          <verse>
            3.
            <syl>mir</syl>
          </verse>
        </note>
      </layer>
    </staff>
  </measure>

@pe-ro : with dynamic rendering you do not necessary know where the new system will occur. If I understand correctly the idea is to supplied the abbreviation <labelAbbr> at the same place where  is given and the understanding of it should be to use this abbreviation of the label for its repetition at the beginning of a system.

@lpugin Yes, exactly. This was the idea.

Ok, now I understand.  You're expecting this --
<measure>
  <staff>
    <layer>
      <note pname="f" oct="4" dur="8">
        <verse>
          verse 1:
          <labelAbbr>1.</labelAbbr>
          <syl>A</syl>
        </verse>
        <verse>
          verse 2:
           <labelAbbr>2.</labelAbbr>
          <syl>E</syl>
        </verse>
        <verse>
          verse 3:
          <labelAbbr>3.</labelAbbr>
          <syl>I</syl>
        </verse>
      </note>
    </layer>
  </staff>
</measure>

when the verse numbering system changes.  And in the case where it doesn't change, <labelAbbr> wouldn't be present, right?  The idea is that the <labelAbbr> element would affect the verse numbering at the next system break, wherever/whenever that might happen, correct?
It's not likely to be a problem, but, of course, this markup could occur in any measure, not just the first, leading to more data than necessary.  Just calling attention to the possibility.

@pe-ro It could and should occur in any measure.
Many editors combined refrains in print, so that not only the label itself would change, but indeed the "repeated" label would, too (that is, if it should be repeated on system starts).


id field in the downloaded model

The downloaded model has an id field (which contains the id for bioimage.io).
However, this is currently not part of the spec, so the model will fail validation due to that.
There are two ways to fix this:

Move the id field to config
Add id as optional field to the spec

@oeway I would prefer the first solution, but if your code hinges on this in many places we can also add it to the spec.

cc @k-dominik @FynnBe

Hi, I think we can go for option 1, but that will require some more time to fix and test as the id is an important key for the website.
I suggest that we fix it from the packager for now.

Fixed with https://github.com/imjoy-team/bioimage-io-models/commit/1a71de33101132ff02e5da21cd0613b89bcbf0c5


cmake: emu: nsim: fix nsim props and args name

use this change to fix https://github.com/zephyrproject-rtos/zephyr/issues/38019
nsim_sem_mpu_stack_guard board use nism_sem board's config file
Signed-off-by: Jingru Wang<EMAIL_ADDRESS>

Isn't it simpler to create 'nsim_sem_mpu_stack_guard.[args|props] files instead of the extra cmake code for handling this?

Hmm, is the commit message still actual?

Before nsim_sem_mpu_stack_guard board uses nism_sem board's config files, so we had to use the extra cmake code for handling it.

We don't do workarounds in cmake code now (the only change in cmake is just a cleanup as I can see)


GridGlyphs.woff 404 in release mode.

Hello,
I've encountered the following issue with the MVC5.Grid pack. All went well with installing and using it. However when publishing a website in release mode or testing a website in release mode, a 505 error occurs when the browser tries to get
GridGlyphs.woff from the /MvcGrid/Fonts/GridGlyphs.woff path, to solve this I've changed
src: url('../MvcGrid/Fonts/GridGlyphs.woff') format('woff');
to src: url('/Content/MvcGrid/Fonts/GridGlyphs.woff') format('woff'); I've found no other workaround for it. All mvc grid files have been added to a styles bundle as per the following code:
bundles.Add(new StyleBundle("~/Content/gridmvc").Include(
"~/Content/MvcGrid/mvc-grid.css"
));
Am I missing something ?
Thank you.

Once you bundle your files these relative urls starts looking for font files from the new bundled file location, so it should be on the same level as mvc grid content. I'm not sure how to make this more reliable, so I just used what other using. I think that setting it to /content/.. does not work with apps published under a non empty domain path, but I might be wrong.

I forgot to mention, I've tested this by publishing it to a testing web server, this is an MVC 5 project, I haven't tested the MVC6 grid library however.
The browser usually points to http://address.testing.net/MvcGrid/Fonts/GridGlyphs.woff instead of http://address.testing.net/Content/MvcGrid/Fonts/GridGlyphs.woff . All other libraries are loading fine.

Yes, because your bundle is probably included from /Content/Bundle.css and not from /Content/AnyFolderName/Bundle.css, so relative url doesn't work.
As for /Content/MvcGrid/mvc-grid.css, try deploying to root address.testing.net/path/, and your include will most likely fail too, because it will try to include address.testing.net/Content/MvcGrid/mvc-grid.css, that's why I'm suggesting using different bundle folder.


[Java Client] Fix wrong behavior of deduplication for key based batching

Motivation
Currently message deduplication doesn't work well for key based
batching. First, the key based batch container doesn't update the
lastSequenceIdPushed. So a batch could contain both duplicated and not
duplicated messages. Second, when createOpSendMsgs is called, the
OpSendMsg objects are sorted by the lowest sequence ids, and the
highest sequence id is not set. If a batch contains sequence id 0,1,2,
then the message with sequence id 1 or 2 won't be dropped.
Modifications

Refactor the key based batch container that the
BatchMessageContainerImpl is reused instead of maintaining a
KeyedBatch class.
When createOpSendMsgs is called, clear the highest sequence id field
and configure the sequence id field with the highest sequence id to fix
the second issue described before.
Add testKeyBasedBatchingOrder to show and verify the current
behavior.
Add test for key based batching into
testProducerDeduplicationWithDiscontinuousSequenceId to verify
lastSlastSequenceIdPushed is updated correctly.

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] no-need-doc
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-added
(Docs have been already added)

Though this PR changed the current behavior that the OpSendMsgs are now sorted by the highest sequence id instead of the lowest sequence id. I think it should be a bug fix rather than a feature. So I added the labels for older branches. If anyone worried about this behavior change, feel free to point it out.

Hi @BewareMyPower. I don't think we've solved the problem.

The original implementation, you think there might be this duplication:

// batch 1th
send [seq_1, seq_3]
// batch 2th
send [seq_2, seq_3]

So the message which is sequence_3 was repeated.

I don't know whether I have correctly understood your meaning. If there is any misunderstanding, thank you for telling me.

First, both cases should not be valid. User should be responsible to make the sequence id unique.
What this PR wants to solve is the resend case of an existing message. For example, 4 messages might be split into two groups in key based batching:

A: 0, 3 (i.e. messages whose keys are all "A", and the sequence ids are 0 and 3)
B: 1, 2

If the client didn't receive the ack receipt of batch A, it might resend batch A (msg-0 and msg-3). There are two possible cases:

batch A has been persisted -> broker should not accept the resend
batch A has not been persisted -> broker should accept the resend

Assuming msg-0 and msg-3 are sent individually (without batch).
For the 1st case, we can verify it easily with the unit test.
    public void test() throws Exception {
        final String topic = "persistent://my-property/my-ns/test";
        admin.namespaces().setDeduplicationStatus("my-property/my-ns", true);

        final Producer<String> producer = pulsarClient.newProducer(Schema.STRING)
                .topic(topic)
                .batcherBuilder(BatcherBuilder.KEY_BASED)
                .create();
        final List<TypedMessageBuilder<String>> messagesToSend = new ArrayList<>();
        messagesToSend.add(producer.newMessage().value("msg-0").key("A").sequenceId(0));
        messagesToSend.add(producer.newMessage().value("msg-1").key("B").sequenceId(1));
        messagesToSend.add(producer.newMessage().value("msg-2").key("B").sequenceId(2));
        messagesToSend.add(producer.newMessage().value("msg-3").key("A").sequenceId(3));
        final List<CompletableFuture<MessageId>> futures = new ArrayList<>();
        messagesToSend.forEach(msg -> futures.add(msg.sendAsync()));
        producer.flush(); // flush the messages in the batch container
        final List<MessageId> ids = futures.stream().map(CompletableFuture::join).collect(Collectors.toList());
        // Resend messages in batch A
        ids.add(messagesToSend.get(0).send());
        ids.add(messagesToSend.get(3).send());
        // TODO: print the ids
    }

Before this patch
After this patch

msg-0
rejected
rejected

msg-3
accepted
rejected

Before this patch, the resend of msg-3 is accepted, this behavior must be wrong. Because msg-0 and msg-3 have already been persisted.
For the 2nd case, we can modify the test to:
        final List<CompletableFuture<MessageId>> futures = new ArrayList<>();
        // Assume msg-0 and msg-3 are lost
        futures.add(messagesToSend.get(1).sendAsync());
        futures.add(messagesToSend.get(2).sendAsync());
        producer.flush(); // flush the messages in the batch container

Before this patch
After this patch

msg-0
rejected
rejected

msg-3
accepted
accepted

The behaviors are the same. Though before this patch, msg-3 is accepted because 3 > 1, while after this patch, msg-3 is accepted because 3 > 2.
It's a pity to see msg-0 is rejected as the duplicated message. It's true that this PR cannot solve this problem as well.
However, if we resent msg-0 and msg-3 as a batch,
        // Resend messages in batch A
        futures.add(messagesToSend.get(0).sendAsync());
        futures.add(messagesToSend.get(3).sendAsync());
        final List<MessageId> newIds = futures.stream().map(CompletableFuture::join).collect(Collectors.toList());

Before this patch
After this patch

msg-0
rejected
Never Completed

msg-3
rejected
Never Completed

After this patch, the futures of sendAsync never completed. It might be another bug to fix.

2022-05-03T04:17:20,988+0800 [main] WARN  org.apache.pulsar.client.impl.ProducerImpl - Message with sequence id 0 is definitely a duplicate

In short, this PR solves the problem that duplicated messages might be accepted even if the previous messages were persisted successfully.

With the default batching builder, the highest sequence id is updated in broker.
(Before this patch) With the key based batching builder, the lowest sequence id is updated in broker.

To make it more clear and simple, https://github.com/apache/pulsar/pull/5491 introduced the highest sequence id to MessageMetadata so that broker could update the latest sequence id to the sequence id of the sequence id of the latest message in the batch. However, the current implementation of the key based batching never tells the broker the sequence id of the latest message in the batch.

/pulsarbot run-failure-checks

messagesToSend.add(producer.newMessage().value("msg-0").key("A").sequenceId(0));
messagesToSend.add(producer.newMessage().value("msg-1").key("B").sequenceId(1));
messagesToSend.add(producer.newMessage().value("msg-2").key("B").sequenceId(2));
messagesToSend.add(producer.newMessage().value("msg-3").key("A").sequenceId(3));

What is the expected behavior for the above case?
The deduplication depends on the monotonically increasing sequence ID, but the key-based
batcher will break the rule.

messagesToSend.add(producer.newMessage().value("msg-0").key("A").sequenceId(0));
messagesToSend.add(producer.newMessage().value("msg-1").key("B").sequenceId(1));
messagesToSend.add(producer.newMessage().value("msg-2").key("B").sequenceId(2));
messagesToSend.add(producer.newMessage().value("msg-3").key("A").sequenceId(3));

What is the expected behavior for the above case?
The deduplication depends on the monotonically increasing sequence ID, but the key-based batcher will break the rule.

Before this patch
After this patch

message order
A-0, A-3, B-1, B-2
B-1, B-2, A-0, A-3

highest sequence id pushed
1
3

I've explained in details in the previous comment. It's true that even after this patch, message deduplication cannot achieve effectively-once, instead, it's at-most once. But it can avoid the duplicated messages. It doesn't solve the problem fundamentally, but it make it better.
I noted that when #4435 added the key based batch container, it also sorted the batches by sequence id.
https://github.com/apache/pulsar/blob/a1fb200ff707e9855efb563a27a894664a59c58b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/BatchMessageKeyBasedContainer.java#L154-L156
At that time, #5491 was not pushed and there is no highest sequence id field in MessageMetadata. In #5491, it looks like the key based batch container was forgotten so that there were no changes.
Even without considering the enhancement (maybe meaningless at this time) to the deduplication, the refactor of this PR that reuses the BatchMessageContainerImpl rather than the KeyedBatch could make the code easier to maintain. For example, when currentTxnidMostBits and currentTxnidLeastBits were added in #8415, these two fields must be added to both default and key based containers.
BTW, maybe in future, we can support deduplication by keys and Key_Shared mode can also support effectively-once as well.

The initial purpose of this PR was that I'm wondering why adding a KeyedBatch class in BatchMessageKeyBasedContainer rather than reusing the BatchMessageContainerImpl. Then I found the difference that  BatchMessageKeyBasedContainer

doesn't update the ProducerImpl#lastSequenceIdPushed
sort the batches by sequence id, not highest sequence id

So during the refactoring, I tend to adopt the more correct behavior rather than keeping the previous behavior (might not be on purpose, but a mistake).

Oh, I see. I thought the PR was for support deduplication for key-based batching before.

@codelipenghui All comments are addressed, PTAL again.

@hangc0276 Could you also take a look?

I accidentally found (triggered) a bug of GitHub that I merged a PR twice because of the network issue.:sweat_smile:

This PR is still in an Open state, However, please don't merge this PR again.


Add flags to the runner script.

The runner script should match the following usage:
./run.sh [run | install]

Don't forget to update the readme


[best practises] falcor-router and update a document in our model

Hello,
Quick question about falcor-router. For example I am updating an object in my mongo's collection with a .call as below:
  {
  route: 'articles.update',
  call: async (callPath, args) =>
    {
      let updatedArticle = args[0];
      let articleID = String(updatedArticle._id);
      let article = new Article(updatedArticle);
      article.isNew = false;

      return article.save(function (err, data) {
        if (err) {
          console.info("ERROR", err);
          return err;
        }
      }).then ((res) => {
        return {
          path: ["articlesById", articleID],
          value: updatedArticle
        };
      });
    }
  },

... as you can find above, the only thing that I return is that:
        return {
          path: ["articlesById", articleID],
          value: updatedArticle
        };

My question: is it the only thing, that I need to return in that route in order to inform the falcor's model on front-end that an article with the articleID has been changed? Is it enough to have properly updated $model on the front-end? :-)
Looking forward to your feedback and any improvements suggestions.
Regards,
Kamil

👍 I'm bumping into the same architectural question. Did you make any progress?

If your articlesById[articleId] route is not an atom, returning a whole object isn't the way to go because call is expected to return an array of path values. You'd probably need to do one of two things:
Invalidate the local version:
return [
  { path: [ 'articlesById', articleId ], $invalidated: true },
];

Or return all changed props (anything from the get route):
return [
  pathValue( [ 'articlesById', articleId, 'field1' ], article.field1 ),
  pathValue( [ 'articlesById', articleId, 'field2' ], article.field2 ),
  pathValue( [ 'articlesById', articleId, 'field3' ], article.field3 ),
];

@eddieajau looks like return [   { path: [ 'articlesById', articleId ], $invalidated: true }, ];

@przeor Certainly. Invalidating the path will require another round trip to the server when that data is needed, as it is invalidated on the client-side cache. This is desirable in many cases to prevent overfetching, but not in all. Also make sure you get acquainted with the refPaths and thisPaths when calling functions, too.

I am not sure what you mean by:
Also make sure you get acquainted with the refPaths and thisPaths

Could you expand on that @joshdmiller ?

They are explained in the docs at the link I sent, but they are the third and fourth arguments to the call method and control what data is returned at the caller's request. One important goal with falcor is to be able to fetch as little data as needed to accomplish a job, so you can specify on a call what should get returned.

I'm currently performing issue triage as we get ready to perform a proper release, and closing/tagging as I go.
Thanks @joshdmiller for the comprehensive answers. Closing.


MissingAuthUrlException (Missing auth url)

Expected Behavior
Page gets redirected correctly to a welcom page  after clicking the "install app" button
i.e.

Current Behavior
Getting a MissingAuthUrlException after clicking the install app button
After clicking the install app button

I got the following error

My dev environment infomation

Laravel Framework 8.83.25
PHP: 7.4.16
Node: v14.17.0
Composer: 2.0.12
OS: Win10
XAMPP: v.3.3.0
ngrok version 3.1.0

Steps to Reproduce
I followed the installation instructions on this repository to install the package and followed the steps of this tutorial
https://www.youtube.com/watch?v=YepqHFryFN0
Tried solutions but not worked for me.
https://github.com/osiset/laravel-shopify/issues/926
https://github.com/osiset/laravel-shopify/issues/908
Any suggestions on getting rid of this problem are welcome.
Thanks a lot.

I am facing the same issue


Show a remote runtime node slightly differently

Currently they look identical to "normal" nodes. Things to maybe visualize:

connected / ready-state.
type of runtime
the address

This is a special noflo-runtime feature. I don't think it makes sense to implement here.
However, it would be useful to know process activation/deactivation status, for example with MsgFlo.

When we can have multiple runtimes with protocol forwarding it will come up again. But we can deal with it then