Feat:Adding Scrollup buttons to all the pages

Is there an existing issue raised for this?

[X] I have searched the existing issues

Describe the feature
The scroll up button is applied only to 2 pages..but the remaining pages lack scroll up buttons
Add ScreenShots
Scrollup in home page:

There are no scrollup buttons in the other pages
Component
Adding to the remaining webpages
It should be implemented because
To make uniformity in the website
Would you like to work on this issue?
Yes
Terms & Conditions

[X] I agree to follow the Code of Conduct
[X] I'm a GSSOC'24 Ext contributor

@vishanurag assign me this task


Middleware that rewrites content seemingly can't change ResponseStream ContentLength

I have a Live Reload Middleware component that works fine in .NET Core 2.2 but has issues with truncated content in 3.0.
The middleware does a bunch of things, but the part that is problematic has to do with interaction with other content that in this case is output by the StaticFileMiddleware. Basically my middleware creates a new ResponseStream for all output, checks for an HTML response and if it finds it injects a block of script code into the HTML.
On the outbound pass (at the end of the pipeline) the middleware checks for HTML like this, and if it finds it changes the Response.ContentLength=null:
            // Make sure we force dynamic contentType since we're
            // rewriting the content - static content will set the header explicitly
            // and fail when it doesn't match
            if (_isHtmlResponse.Value)
                _context.Response.ContentLength = null;

The staticFile middleware has explicitly set the ContentLength and this code sets it to null to effectively let the Response calculate it. This has worked fine in .NET Core 2.x, but in 3.0 I see truncated content. Basically I see a small part of the injected content - basically the length of what otherwise would have been there in the original HTML document.
This bit of code should happen before any content has been written because it is part of the custom stream and fired before each write operation on the stream.
So the question are:

Should I be able to reset the Stream length dynamically using the code I have
If not what's the right way to Write a different ContentLength?
I've also tried explicitly writing out the new calculated content length but that too is ignored

It's possible that
If you want to take a closer look, the runnable sample project is on GitHub, here's the relevant code .
The request in question is the static HTML file link which should get intercepted and injects the script.

Upon further digging I actually see some nasty behavior here. It looks like ASP.NET is writing out the data and serving it to the browser correctly. The data is there, but the content length header is not updated when I rewrite it.
Here's a Fiddler trace:

The Content-Length is the original contentlength rather than the one I changed (should 1900+bytes). Even if I explicitly set the header like this:
if (_context.Response.ContentLength != null)
    _context.Response.ContentLength += WebsocketScriptInjectionHelper.GetWebSocketClientJavaScript(_context).Length;

I see the value going at 516 bytes, and coming out at 1900+ bytes, but the new value never shows up in the response.
Note again that no output has been sent yet when the content length is set.

Few questions:

What server are you using?
In 2.2 was the content length being set automatically? AFAIK we don't do none of the servers do that so maybe it was chunked?

It's Kestrel in dev mode. But happens also with IIS Express...
Same code runs in 2.2.
I think the problem is that in 3.0 there's an implicit FlushAsync() happening that's touching the response and basically makes it so headers can't be written anymore.
if (!isContentLengthSet && _context.Response.ContentLength != null)
{
    try
    {
        _context.Response.Headers.Remove("Content-Length");
        _context.Response.Headers.ContentLength +=
            WebsocketScriptInjectionHelper.GetWebSocketClientJavaScript(_context).Length;
        isContentLengthSet = true;
    }
    catch (Exception ex) //   <!---  Headers are read-only because content has been written
    {
    }
}

After some more poking around I also see that FlushAsync() is called before anything else.
Looking at the trace it comes from StartAsync() and I bet that causes the Response/Headers to get locked:

So I managed to now get this to work by adding this to FlushAsync():
public override Task FlushAsync(CancellationToken cancellationToken)
{
    if (!isContentLengthSet && IsHtmlResponse())
    {
        _context.Response.Headers.ContentLength = null;
        isContentLengthSet = true;
    }

    return _baseStream.FlushAsync(cancellationToken);
}

This seems to work now.
That's cool but seems a bit unwieldy. Why does FlushAsync() need to be called before any output is sent? And if that needs to happen to provide an 'initial' hook, maybe that shouldn't affect the Response.Header locking.

Just double checked: 2.x does not call FlushAsync() (or Flush()) on the start of the request and that probably accounts for why I was able to change the content-length in those versions. The FlushAsync() is definitely what's affecting the headers locking.

That looks like correct analysis. In 2.2 that would be the case during the first call to WriteAsync (first write will flush headers if the content length wasn't set). So it worked on 2.2 because that logic was OK before the first write but then would actually fail if you touched that property again.
Instead, you should handle OnStarting and do your header checks in there. That will make the code less brittle.
@Tratcher why did we add an explicit StartAsync before in SendFileAsync
PS: Replacing the body is more complicated than it seems. We made it a bit better in 3.0 by exposing  a new response feature that encompasses all of the possible ways to mutate the response body (see StreamResponseBodyFeature).

Response.OnStarting()  -  sigh - yeah that does look much easier to deal with assuming the headers are set at that point and can still be changed. I'll do that for an update.
Yeah creating a full stream just to replace the response is a pain especially for the content type scenario. I'll check out StreamResponseBodyFeature  - hmmm... got any links or examples? Search comes up with no hits.
Part of the problem is discovering all these myriads of APIs :-)

I do wonder if Response should throw

The alternative to OnStarting is to replace IHttpResponseBodyFeature. StreamResponseBodyFeature might be a good staring place. The response compression middleware is a good example: https://github.com/aspnet/AspNetCore/blob/2de6c732d1f410460a627cb75aca91ae2c741fd5/src/Middleware/ResponseCompression/src/ResponseCompressionMiddleware.cs#L54-L71
@davidfowl StreamResponseBodyFeature.SendFileAsync calls StartAsync so that you can derive and override just StartAsync for scenarios like this.

Setting any header after Start/Write/Flush should throw. We added some fast paths for ContentLength, I'll see if they bypass the HasStarted checks.

Testing on Kestrel, master branch.
context.Response.Headers["Foo"] = "bar"; throws InvalidOperationException: Headers are read-only, response has already started.
context.Response.ContentLength = null; doesn't throw when it should.

Confirmed this is only an issue with Kestrel. IIS in-proc and HttpSys both throw if you try to set Content-Length too late.


⚠ Bofrost.lu has degraded performance

In 2aa4ee2, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1337 ms

Resolved: Bofrost.lu performance has improved in 0475647 after 30 minutes.


Location sort no loader

Essentially, HereFunctionHandler exposes three major public methods, corresponding to allowing GPS/network use, forbidding GPS/network use, and refreshing the location. This allows the client to only have to worry about whether GPS/network use is permitted (basically corresponding to whether the activity is in focus).
The logistics of dealing with the location manager and deciding when a good enough location has been achieved are dealt with internally in the HereFunctionHandler class.
In EntitySelectActivity, there is a new boolean flag, containsHereFunction, which is set to true once the first here() function is evaluated. This boolean flag is saved in onDestroy, so that in a screen orientation change, we know that the activity already has a here function. If the activity already has a here function, we can avoid refreshing the location, and immediately re-enable GPS/network use.
There is also a new boolean flag in EntitySelectActivity called locationChangedWhileLoading, which is pretty self-explanatory.
I put in a ton of log statements that should be removed right before pulling. (Edit: we decided to leave them in for future dev work.)
Cross-requested with https://github.com/dimagi/javarosa/pull/231.
Release Note: Allow case lists with location data to be sorted by distance from your current location.

Looking good! Let me know when you finish addressing my comments and I can do another quick pass.

:tada: :satellite: :earth_americas: :earth_africa: :earth_asia:  :satellite: :tada:

@phillipm ^^ definitely trying to make sure you make the cut for the next emoji blog


Not showing templates from bundle version 1.3.0

The current version of bundles is 1.3.0 and should contain Durable Functions templates for Python. However, they're not showing for me. Do I need to clear a cache before they appear?

In the same project, func new shows the templates:

@anthonychu did you ever use FUNCTIONS_EXTENSIONBUNDLE_SOURCE_URI to test the staging templates in this instance of VS Code? If so, that might be messing with our caching logic. The staging feed was on 1.3.0 for several months and I'm pretty sure multiple changes were pushed to the same version. It's possible we cached the 1.3.0 templates the very first time you tested with that env var (before durable templates were added?) and we've never updated the cache since then because you're technically still on the same version.
As a workaround, you can temporarily switch to the staging template source (which is currently at 1.3.2) and then switch back to normal. This should delete the "old" 1.3.0 templates and get you back on track

Load VS Code with FUNCTIONS_EXTENSIONBUNDLE_SOURCE_URI set to https://functionscdnstaging.azureedge.net/public
Start the "Create Function..." flow just to make sure it updates the templates
Reload VS Code without the environment variable

I can investigate a fix on our side, but I don't think this will be a problem very often. External users probably won't be testing the staging template source, and hopefully the staging source won't be stuck on the same version for months like it was.

That worked! I don't think I ever set that variable on my local machine, but for a while I was running a custom build of the extension that was modified to bypass the cache and use the staging feed to test the Python Durable templates. That might have messed things up. 😬
Thanks for adding support for FUNCTIONS_EXTENSIONBUNDLE_SOURCE_URI BTW. Super useful. 👍

I'm also seeing that issue.

Never used that variable and I can reproduce. Can we reopen?
https://github.com/Azure/azure-functions-durable-python/issues/153

To be clear, it shows up in func new. Just not in vscode.

@MaximRouiller Anthony's situation was almost certainly unique to his machine's setup related to the staging feed. If you would like us to investigate your issue, please open a new issue with your own detailed repro steps


Add spellchecker extension to workspace file

Related ticket: https://jira.mitre.org/browse/REMS-526

FYI looks like Alex suggested this extension, and he also included an array of words for the spellchecker to ignore.
https://github.com/mcode/rems-setup/pull/7/files
Should we make a ticket to include those changes he suggested?


FP swizzles are broken on AVX with MSVC 2022

👋
It seems that AppVeyor/Azure Pipelines didn't catch a (possibly codegen?) error on swizzle when compiled for AVX under MSVC 2022 (19.32.31328.0).
(I'm working on adding GHA builds for Windows, as AppVeyor is simply too slow and also doesn't run on forks.)
[----------] 3 tests from swizzle_test/fallback_float, where TypeParam = class xsimd::batch<float,struct xsimd::avx>
[ RUN      ] swizzle_test/fallback_float.swizzle_reverse
swizzle tests
D:\a\xsimd\xsimd\test\test_batch_manip.cpp(165): error: Expected equality of these values:
  b_res
    Which is: { 0, 0, 0, 0, 7, 5, 3, 1 }
  b_exped
    Which is: { 15, 13, 11, 9, 7, 5, 3, 1 }
  while testing swizzle reverse test
[  FAILED  ] swizzle_test/fallback_float.swizzle_reverse, where TypeParam = class xsimd::batch<float,struct xsimd::avx> (1 ms)
[ RUN      ] swizzle_test/fallback_float.swizzle_fill
swizzle tests
D:\a\xsimd\xsimd\test\test_batch_manip.cpp(180): error: Expected equality of these values:
  b_res
    Which is: { 0, 0, 0, 0, 0, 0, 0, 0 }
  b_exped
    Which is: { 15, 15, 15, 15, 15, 15, 15, 15 }
  while testing swizzle fill test
[  FAILED  ] swizzle_test/fallback_float.swizzle_fill, where TypeParam = class xsimd::batch<float,struct xsimd::avx> (0 ms)
[ RUN      ] swizzle_test/fallback_float.swizzle_dup
swizzle tests
D:\a\xsimd\xsimd\test\test_batch_manip.cpp(195): error: Expected equality of these values:
  b_res
    Which is: { 1, 1, 5, 5, 0, 0, 0, 0 }
  b_exped
    Which is: { 1, 1, 5, 5, 9, 9, 13, 13 }
  while testing swizzle dup test
[  FAILED  ] swizzle_test/fallback_float.swizzle_dup, where TypeParam = class xsimd::batch<float,struct xsimd::avx> (0 ms)
[----------] 3 tests from swizzle_test/fallback_float (1 ms total)

Closing as upstream problem: https://developercommunity.visualstudio.com/t/MSVC-2019-and-2022-corrupt-the-upper-hal/10042321


Email csv not working

Email csv currently isn't working, saying there is nothing in the raw_data (list index out of range at 0) the problem appears to be coming from line 495-501 in data where no raw_data is added unless the amount of columns is 1?

my fault. should now work.

Did you push the changes? Not seeing them yet

Did you see it now?

Error has been fixed


chore: Release 0.9.1

:robot: I have created a release beep boop
0.9.1 (2023-07-21)
Features

add support for atomic memory opcodes (#232) (a7aa6e9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/noir-lang/acvm-backend-barretenberg/releases/tag/v0.9.1 :sunflower:


Rabbitmq Add create queue and binding routing

This a new option for the rabbitMQ scaler.
It will create the queue that the scaler is monitoring.
This is very useful for jobscaler. Since rabbitmq has a strong philosophy that code should just generate their queues and bindings topics.
It is a chicken/egg situation. The application (job) needs to be run for the queue to be created and the job will be only run if there is a message in the queue.
Like this, the scaler is creating the queue and proper bindings for the proper exchange.
Documentation: #620
Checklist

[ x]Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ x] Tests have been added
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ]x A PR is opened to update the documentation on (repo) (if applicable)
[ ] Changelog has been updated

Fixes #

-1 for this. Use an operator to create the queue.

Hi,
In this case, I agree @coderanger and @tomkerkhove. KEDA doesn't manage the resources, and opening this door, we could manage a lot of resources, losing the focus of KEDA.
I understand your point and your requirement about queue creation, but maybe the publisher could create the queue and your workload(ScaledJob) could be only a receiver. In that case you don't need to create the queue using KEDA because is the publisher who creates the queue

I agree with others on this one. I hope you understand @pascallap

I do understand.
I will do a simple post-install helm hook, to run once the job declared in the scaler.
This will cover the initial installation.
I will that a look at the existing operators for queue (and configurations) creation.
@JorTurFer the publisher shouldn't know the existence of the consumer. So, what you are proposing wouldn't be a valid solution.

@pascallap This is exactly the use case I wrote mine for btw, https://github.com/coderanger/rabbitmq-operator


Cannot run in heroku with SSL on

I am using standard Postgres in Heroku, which has SSL on by default. Whenever I try to run the migrations, I get hit with:
[ERROR] error: no pg_hba.conf entry for host "${ip}", user "${user}", database "${database}", SSL off
    at Connection.parseE (/app/node_modules/db-migrate-pg/node_modules/pg/lib/connection.js:539:11)
    at Connection.parseMessage (/app/node_modules/db-migrate-pg/node_modules/pg/lib/connection.js:366:17)
    at Socket.<anonymous> (/app/node_modules/db-migrate-pg/node_modules/pg/lib/connection.js:105:22)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at Socket.Readable.push (_stream_readable.js:134:10)
    at TCP.onread (net.js:563:20)

My migration command:
package.json:

    "migrate": "db-migrate --config migrations-config.js",
    "migrate-up": "npm run migrate up"

My migrations-config:
'use strict'
import {postgres} from 'config'
import {parse as urlParse} from 'url'

const parsed = urlParse(postgres.connectionUrl)
const auth = parsed.auth.split(':')
const user = auth[0]
const password = auth[1]

export default {
  driver: 'pg',
  user,
  password,
  host: parsed.hostname,
  database: parsed.pathname.substring(1),
  port: parsed.port,
  schema: '${schema}',
  ssl: (process.env.NODE_ENV === 'production')
}

and my config.js entry:
export const postgres = {
  connectionUrl: process.env.DATABASE_URL
}

If I change the ssl to be explicitly true via:
ssl: true
``` in the migrations config, the result is the same.

Any suggestions on how to fix this would be amazing!

Eventually I got this to work with:
module.exports = {
  dev: {
    driver: 'pg',
    url: postgres.connectionUrl,
    schema: 'flock_auth'
  },
  prod: {
    driver: 'pg',
    schema: 'flock_auth',
    url: postgres.connectionUrl,
    addIfNotExists: {
      sslmode: 'require'
    },
    overwrite: {
      native: true
    }
  }
}

and passing the env flag to the command line

@apolishch I know it's been a while, but did you fix this in a public repo? I ran into a similar problem today, though it looks like our setups differ. Would love to see the full repo.

For others coming here at a later time, this is what helped me.
I am using a database.json file as the default configuration. Turns out Heroku already offers a default JSON config for the database, so in the particular environment you are running the app in, all you need to add is this:
    "staging": {
      "overwrite": {
        "driver": "pg",
        "ssl": {
          "rejectUnauthorized": false
        }
      }
    }

rejectUnauthorized is the key element here.

Thanks @shto, you solved my problem!
Just to be overly clear for people who are bad at reading the docs (as myself). You have to pass in the explicit env you defined in your database.json when running the migration, e.g.:
"postinstall": "yarn db:migrate:up --env staging"

@shto @Alenius thanks for the help!


Confusing CLI output after CREATE statement

If you run this query on the example database in the documentation, you add 4 relationships (from Zhang to every other User node including from Zhang to Zhang). However the CLI output says 0 tuples.
kuzu> MATCH (a:User {name : "Zhang"}), (b:User) CREATE (a)-[:Follows {since :  2022}]->(b)
-
(0 tuples)

Then if you queried them, you can get those 4 records so only the output message is wrong.

This is expected. You get outputs only if the query ends with RETURN.


Update meta.yaml

Could you please merge.

@sannykr I can merge but I believe the packages won't be available. I just found out that something broke the package uploads. Keep an eye for the CIs on master to be certain. (I am working on it.)

I think it's merged now. Is it?


Fix default client env when var is set

This fixes an issue with https://github.com/dagger/dagger/pull/2122
Signed-off-by: Helder Correia
/cc @marcosnils @jpadams

Looks good in testing!
Here's one way I've used it with an extra conditional to get the behavior I want with a required, non-empty-string env var greeting for my hello action:
usage:
greeting="hello" image_name="alpine:3" dagger do hello --log-format=plain

package helloworld

import (
	"dagger.io/dagger"
	"dagger.io/dagger/core"
)

dagger.#Plan & {
	_alpine: core.#Pull & {
		source: client.env."image_name"
	}

	client: env: "image_name": string
	client: env: "greeting":   string | *""

	actions: {
		image_name: string

		_greeting: string
		if client.env.greeting != "" {
			_greeting: client.env.greeting
		}
		hello: core.#Exec & {
			input: _alpine.output
			args: ["echo", "\(_greeting), world! from \(_alpine.digest)"]
			always: true
		}
		bye: core.#Exec & {
			input: _alpine.output
			args: ["echo", "Goodbye, world! from \(_alpine.digest)"]
			always: true
		}
	}

}

nullblankenv ➤ greeting="" image_name="alpine:3" dagger do bye --log-format plain 2>&1 | grep sha256
1:14PM INF actions.bye | #3 0.059 Goodbye, world! from sha256:4edbd2beb5f78b1014028f4fbb99f3237d9561100b6881aabbf5acce2c4f9454

nullblankenv ➤ image_name="alpine:3" dagger do bye --log-format plain 2>&1 | grep sha256
1:14PM INF actions.bye | #3 0.060 Goodbye, world! from sha256:4edbd2beb5f78b1014028f4fbb99f3237d9561100b6881aabbf5acce2c4f9454

nullblankenv ➤ greeting="nonsense" image_name="alpine:3" dagger do bye --log-format plain 2>&1 | grep sha256
1:14PM INF actions.bye | #3 0.040 Goodbye, world! from sha256:4edbd2beb5f78b1014028f4fbb99f3237d9561100b6881aabbf5acce2c4f9454

nullblankenv ➤ greeting="Hello" image_name="alpine:3" dagger do hello --log-format plain 2>&1 | grep sha256
1:15PM INF actions.hello | #3 0.071 Hello, world! from sha256:4edbd2beb5f78b1014028f4fbb99f3237d9561100b6881aabbf5acce2c4f9454

nullblankenv ➤ greeting="Howdy" image_name="alpine:3" dagger do hello --log-format plain 2>&1 | grep sha256
1:15PM INF actions.hello | #3 0.068 Howdy, world! from sha256:4edbd2beb5f78b1014028f4fbb99f3237d9561100b6881aabbf5acce2c4f9454

nullblankenv ➤ greeting="" image_name="alpine:3" dagger do hello --log-format plain
1:16PM INF client.env | computing
1:16PM INF client.env | completed    duration=0s
1:16PM INF _alpine | computing
1:16PM INF _alpine | completed    duration=600ms
1:16PM INF actions.hello | computing
1:16PM ERR actions.hello | failed: actions.hello.args.1: invalid interpolation: non-concrete value string (type string):
    /Users/jeremyadams/src/isol/repro/yuvrajoss/nullblankenv/helloworld.cue:26:19
    duration=0s
1:16PM FTL system | failed to execute plan: task failed: actions.hello: actions.hello.args.1: invalid interpolation: non-concrete value string (type string):
    /Users/jeremyadams/src/isol/repro/yuvrajoss/nullblankenv/helloworld.cue:26:19

nullblankenv ➤ image_name="alpine:3" dagger do hello --log-format plain
1:16PM INF client.env | computing
1:16PM INF client.env | completed    duration=0s
1:16PM INF _alpine | computing
1:16PM INF _alpine | completed    duration=600ms
1:16PM INF actions.hello | computing
1:16PM ERR actions.hello | failed: actions.hello.args.1: invalid interpolation: non-concrete value string (type string):
    /Users/jeremyadams/src/isol/repro/yuvrajoss/nullblankenv/helloworld.cue:26:19
    duration=0s
1:16PM FTL system | failed to execute plan: task failed: actions.hello: actions.hello.args.1: invalid interpolation: non-concrete value string (type string):
    /Users/jeremyadams/src/isol/repro/yuvrajoss/nullblankenv/helloworld.cue:26:19

Sorry, I don't understand what you're trying to do. Do you want to fail executing the "hello" action in case the environment variable isn't set, or hide it?


Investigate "Kokoro - Test: Docker Image Validation" check failure

It's been failing for a while.

http://fusion2/ci/kokoro/prod%3Acloud-devrel%2Fclient-libraries%2Fjava%2Fjava-shared-config%2Fpresubmit%2Fgraalvm-docker/activity/cb7412fa-b235-4c81-9b6b-1482d4b1007e . It seems the Kokoro build configuration file is not there.
[17:08:06][ERROR] Failed to get build config
com.google.devtools.kokoro.config.ConfigException: Couldn't find build configuration file graalvm-docker. It must be a path relative to /tmp/workspace/workspace/cloud-devrel/client-libraries/java/java-shared-config/presubmit/graalvm-docker/src/github/java-shared-config/.kokoro/presubmit (the build directory) and it may omit the extension (.gcl and .cfg are the two supported extensions)

Thanks @suztomo! We've just merged the PR that adds a corresponding cfg file in github (https://github.com/googleapis/java-shared-config/pull/664/). Could we try rebasing the PR that is experiencing this issue with main?

As #664 is green, it's good.


Service: Record and Replay

This pull request defines a new service.

@pelikhan Please check the updated service define. Our current implementation is an MCU with only two gpio, one for record and one for replay. Cached time is handled by the protocol mcu.

We have a "sound player" service defined here: https://microsoft.github.io/jacdac-docs/services/soundplayer . Basically, a list of sound names and a way to play/cancel them. Is there too much overlap with this service?

@pelikhan Sorry that we didn't aware of this service and revert the last commit, just keep things simple.

@xmeow please take a look at the service to make sure it looks good for you


Pull replication in example project takes forever when I use my own database

Hello,
I have no experience with CDTDatastore or Cloudant in general and learning it by documentation and examples. Currently I am trying to figure out how to perform replications. So I opened the sample project and both push and pull replications in it worked as I expected. Then I tried to aim the project to my own database and found that push replications work pretty well, but pull replications now take forever to complete.
STR:

Clone CDTDatastore project.
pod install, open workspace in Xcode.
Run project, click sync button, make sure that new items are appeared.
Create new database named 'todo' in Cloudant dashboard.
Go to Permissions, generate API key.
Give _admin permissions to the generated API key.
Change username, password, db_name and urls in CDTTodoReplicator.
Run project, add several items, sync.
Make sure that new items are appeared in Cloudant dashboard.
Delete application, run project again.
Try to sync.
Result: there are no previously added items, pull replication callback never fired (unlike push callback).

Am I missing something? Maybe there should be additional setup in dashboard?

It seems the issue has appeared recently, in this commit. After checkout to previous commit pull replications are working fine.

Yeah, we've been trying to track down the exact problem with the code in the commit you mention. This bug is limited to CouchDB 2.0 and some Cloudant clusters running the next generation of our database core database software, known to us as DBNext.
There is a work around for now, if you can either create a new cloudant account on a cluster not running this software, or you can contact support via the dashboard and get them to move your account to one of the clusters not running the new version.
A list of clusters currently running DBNext

Porter
Sling

@romanzes Support have moved your account to Mead replications should start to work again with 1.1.0.

@romanzes just as an update we have a fix for this and should be able to release the fixed version of the library on Monday.

@romanzes sorry for the delay - we have fixed the _bulk_get issue in our latest release which is 1.1.1


Attributes does be set correctly when flat single property model

This is reported from Storage team.
The KeyPolicy only contain one property, so in the StorageAccountData it's flatted with KeyExpirationPeriodInDays, but first it becomes non-nullable and read-write which does not inherit the KeyPolicy attribute which is nullable and readonly.
Swagger is here: https://github.com/Azure/azure-rest-api-specs/blob/71240dcb17f8f66e3d69fae436ee09e7b93aa776/specification/storage/resource-manager/Microsoft.Storage/stable/2021-09-01/storage.json#L3599

The fact that Expiration has a get and a set is actually correct however as I was doing my testing around this I realized that in some cases where the outer property is Readonly and the Inner property is not we run into a potential null reference issue.
To address the original concern if we did not wrap this code would be valid
var data = GetStorageAccountFromAzure();
if (data.KeyPolicy is not null)
{
    var oldValue = data.KeyPolicy.KeyExpirationPeriodInDays;
    var newValue = oldValue + 1;
    data.KeyPolicy.KeyExpirationPeriodInDays = newValue;
}

However you cannot "create" a KeyPolicy on this data object its only returned from the service.
When we wrap the exact same behavior just without the extra layer in objects
var data = GetStorageAccountFromAzure();

var oldValue = data.KeyExpirationPeriodInDays;
var newValue = oldValue + 1;
data.KeyExpirationPeriodInDays = newValue;

The difference is there is no way to check if KeyPolicy was null and if it was then data.KeyExpirationPeriodInDays will result in a null reference exception.
I see two options in this case
We don't flatten when the outer property is readonly and the inner property is not.
This will result in roughly 30ish properties not being wrapped 2 of which are in the GA library Azure.ResourceManager.Resources which we can manage with EB never.  We can shrink this a bit by further looking at whether or not the object can be constructed with a null and if not then it would still be safe to flatten.
We add explicit exceptions to the setter and make the getter nullable
In the case of StorageAccountData and KeyPolicy it goes from
        public int KeyExpirationPeriodInDays
        {
            get => KeyPolicy.KeyExpirationPeriodInDays;
            set => KeyPolicy.KeyExpirationPeriodInDays = value;
        }

to
        public int? KeyExpirationPeriodInDays
        {
            get => KeyPolicy?.KeyExpirationPeriodInDays;
            set
            {
                if (KeyPolicy is null)
                    throw new InvalidOperationException("There is no KeyPolicy set for this StorageAccount");
                if (!value.HasValue)
                    throw new ArgumentNullException(nameof(value), "KeyExpirationPeriodInDays cannot be null");
                KeyPolicy.KeyExpirationPeriodInDays = value.Value;
            }
        }

The property type becomes nullable allowing us to return null if KeyPolicy is null.  The setter needs to provide valid errors in the cases where KeyPolicy is null instead of the current NullReferenceException


Unable to animate the Line element

Discussed in https://github.com/cduck/drawsvg/discussions/121

Originally posted by PieterMostert March  2, 2024
I've tried modify the example in the README to animate the endpoints of a line, but nothing happens:
import drawsvg as draw

d = draw.Drawing(400, 200, origin='center',
        animation_config=draw.types.SyncedAnimationConfig(
            # Animation configuration
            duration=8,  # Seconds
            show_playback_progress=True,
            show_playback_controls=True))

line = draw.Line(0, 0, 0, 200, stroke_width=4, stroke='red')  # Moving line
line.add_key_frame(0, sx=0, sy=0,  ex=200, ey=0)
line.add_key_frame(8, sx=0, sy=200,  ex=0, ey=0)
d.append(line)

d.display_inline()  # Display as interactive SVG

Am I doing something wrong, or is this a bug? Either way, I'd appreciate advice on how to get this to work.

The endpoints of the Line element cannot be animated because drawsvg.Line generates an SVG <path> tag instead of a <line> tag.  This could be solved by changing drawsvg.Line to use the <line> tag but this may have unintended effects on existing code using the Line element.
Workaround: Use this definition of Line in place of drawsvg.Line.
class Line(draw.DrawingBasicElement):
    '''A line element that uses the SVG <line> tag.

    The endpoints of this custom Line element can be animated.  This is a workaround because `drawsvg.Line`
    cannot be animated'''
    TAG_NAME = 'line'

    def __init__(self, x1, y1, x2, y2, **kwargs):
        super().__init__(x1=x1, y1=y1, x2=x2, y2=y2, **kwargs)


operator: add support for webhook sink raw mode

Current dependencies on/for this PR:

main

PR #175   👈

This comment was auto-generated by Graphite.


Add FittedOverflowBox

Description
This adds a new primitive widget called FittedOverflowBox, which is similar to SizedOverflowBox, except that it takes the size from the child widget, optionally only along one axis.
Tests

(no tests yet, hence the draft status)

We're going to do this differently.


bug: allow installation of any containerd version in case of docker

if the node we are running the installer is running with docker we can skip the containerd upgrade restrictions check, we can simply install it.

I have quickly test this in a single and in a multi node cluster. Upgrades from Docker are back up.  FOSSA is still failing but no new dependency was added by this PR.


fix(36102): fix extra new line in Organize Imports

Organize Imports was inserting extra newline.

Fixes #36102

@elibarzilay mind taking a look at this PR?

@andrewbranch is this related to #36688 ?

It is not.


Enhancement/81 html resource load

closes #81 :feelsgood: 🚑
So this PR the SUI switching from passing a dependency loaded index.html to sideloading (via ejs) two dependency partials at runtime. In the process it was necessary to yugely refactor server/.app.js as it was making a mess all over the place 🌋 😷 😲
I expect in this pr you'll notice no difference from master, except maybe a shorter app boot time and no more mess merge nonsense with index.html (hah because its gone, jk its still there, but changed).

gulp serve-dev should behave as expected
gulp build should behave as expected
app proxy with miq should behave as expected

SO @himdel @dtaylor113 @dclarizio @jeff-phillips-18 @AparnaKarve @chriskacerguis @chalettu
since all ya'll were mildly involved in this, one way or another gonna humbly demand some 👀 on the branch when you get a chance.  Yes yes review the code, but also if you can pop on the branch and confirm all works as expected.
🙏

@miq-bot add_label euwe/no, enhancement

(ps we might have to talk about code climate running elsint on things not the app)

@AllenBW well, let's start with the codeclimate fails :)

First pass, it looks good to me.  @himdel @martinpovolny thoughts?

@AllenBW gulp build doesn't create an index.html now - are you sure we want that?
AFAIK this won't work now, since we don't use node to serve it on the appliance yet, and I'm not sure that's something we should be doing in the production mode anyway.. I think the ejs -> html conversion should be a part of the build process too (why run EJS on every request when the file will never change..)

@himdel ooo goooooooood catch on that!  see i knew waiting for your 👀 would be worth it <3

No further complaints :+1: :shipit: :)

I pulled this branch, built, and ran. Everything worked as expected 👍

@jeff-phillips-18 very much so appreciated all the same!


Implement "Delete project" functionality

Extracted from https://github.com/Learn-by-doing/bitstarter/issues/7

mine

Addressing this in the updated PR https://github.com/Learn-by-doing/bitstarter/pull/83


Working w/ virtual ports

Hi!
I've some doubt about how to use this library. Currently I'm testing a simple project, which I'm gonna explain now.
I've got an android app working with your library. This app'd wait for some data, sent by serial port. That's okay.
I want my computer to emulate serial ports and send data using hyperterminal.
Well, I've downloaded some software from Eltima: Virtual Serial Port Driver  to create a couple of virtual serial ports, and Advanced Serial Port Terminal.
Everything works great creating a session port COM1 in terminal (to send text) and creating session port COM2 in terminal (to receive text).
Then, I left only one connection in eltima advanced serial port terminal and opened the android app... But within the app, I get the toast messasge "USB not supported".
Am I missing some configuration? What am I doing wrong?
Thank you for your support.

Hi! @russellhoff
Sorry for the delayed response. I have no idea what that didnt work. Did you solve it?

Hi! Don't worry. I left the implementation on Android and started an application on Spring Boot.
Thanks!


[Question] Sampling inside gunicorn gevent worker.

Hello,
Is this sampling strategy supposed to work in "gevent-driven" environment, like gunicorn gevent worker? I've tried it in our system, and looks like that after several minutes of sampling it just hangs somewhere in _sample, probably in while loop (other application code is working as expected).
Thanks.

Aw that's no good! We run the sampler in heavily gevent-dependent applications without any issues; I'm not sure offhand what the issue might be. What versions of gunicorn and gevent are you using? Do you have any other OS-signal-handling code in your application (in particular for SIGVTALRM)?

+1
can someone talk about how to integrate this in a gunicorn based deployment ?
currently we use cprofile and use it like this
      # Patch psycopg for gevent after we fork.
      psycogreen.gevent.patch_psycopg()
      worker.log.info('Made psycopg green: psycogreen.gevent.patch_psycopg')
      orig_init_process_ = worker.init_process
      def profiling_init_process(self):
            orig_init_process = orig_init_process_
            ofile = '%s%s' % (profiling_prefix, getpid())
            print 'Profiling worker %s, output file: %s' % (worker, ofile)
            cProfile.runctx('orig_init_process()', globals(), locals(), ofile)
      worker.init_process = profiling_init_process.__get__(worker)```

we then use runsnakerun to view it. would love to use nylas


[PR] Issue templates update

Description of change

Summary: Updated templates for issues & pr's

Codecov Report

Merging #32 into develop will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop     #32   +/-   ##
=======================================
  Coverage     4.37%   4.37%
=======================================
  Files           73      73
  Lines         1144    1144
  Branches        79      79
=======================================
  Hits            50      50
  Misses        1091    1091
  Partials         3       3

Continue to review full report at Codecov.

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


[DROOLS-380] Example 4.20 file logger newFileLogger

Ref. https://issues.jboss.org/browse/DROOLS-380
In the documentation [1] example 4.20 mention the following code snippet.
KieRuntimeLogger logger = KieServices.Factory.get().newFileLogger(ksession, "logdir/mylogfile");
However that doesn't compile. I believe is missing a .getLoggers() so the proper snippet, I believe, was intended as
KieRuntimeLogger logger = KieServices.Factory.get().getLoggers().newFileLogger(ksession, "logdir/mylogfile");
Ciao
MM
References:
[1] http://docs.jboss.org/drools/release/6.0.0.Final/drools-docs/html_single/#d0e1904

The documentation has been moved to https://github.com/droolsjbpm/droolsjbpm-knowledge. I fixed it there. Thanks!


HADOOP-19008. S3A: update aws-sdk version to 2.21.41

Description of PR
HADOOP-19008. S3A: update aws-sdk version to 2.21.41
How was this patch tested?

Did a run of pr #6277 with the version updated; errors there are unrelated to sdk updated
did a release build with cli tests, inc storediag, with new SDK and logging set to debug. all good. See output.txt

For code changes:

[X] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[X] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[X] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
16m 44s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+0 :ok:
shelldocs
0m  0s

Shelldocs was not available.

+1 :green_heart:
@author
0m  1s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 55s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
30m 42s

trunk passed

+1 :green_heart:
compile
16m  4s

trunk passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
14m 56s

trunk passed with JDK Private Build-1.8.0_392-8u392-ga-1~20.04-b08

+1 :green_heart:
mvnsite
18m 45s

trunk passed

+1 :green_heart:
javadoc
8m 21s

trunk passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
7m 34s

trunk passed with JDK Private Build-1.8.0_392-8u392-ga-1~20.04-b08

+1 :green_heart:
shadedclient
47m 35s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
1m  0s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
28m 31s

the patch passed

+1 :green_heart:
compile
15m 47s

the patch passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
15m 47s

the patch passed

+1 :green_heart:
compile
14m 41s

the patch passed with JDK Private Build-1.8.0_392-8u392-ga-1~20.04-b08

+1 :green_heart:
javac
14m 41s

the patch passed

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
13m 36s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
javadoc
8m 15s

the patch passed with JDK Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
7m 32s

the patch passed with JDK Private Build-1.8.0_392-8u392-ga-1~20.04-b08

+1 :green_heart:
shadedclient
48m 50s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
796m 12s
/patch-unit-root.txt
root in the patch passed.

+1 :green_heart:
asflicense
1m 28s

The patch does not generate ASF License warnings.

1084m 39s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestDirectoryScanner

hadoop.hdfs.TestRollingUpgrade

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6334/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6334

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs

uname
Linux d972ea4c3785 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 30cd0a52fa7ce7877e2ec1659d9d3838f31ea33b

Default Java
Private Build-1.8.0_392-8u392-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.21+9-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_392-8u392-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6334/1/testReport/

Max. process+thread count
3007 (vs. ulimit of 5500)

modules
C: hadoop-project . U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6334/1/console

versions
git=2.25.1 maven=3.6.3 shellcheck=0.7.0

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


java.lang.UnsupportedOperationException when updating entity properties

Here is the stack trace: Egeria Error Stack Trace.txt
If need full stack trace:
Full Stack Trace.txt
It seems likely that there is an attempt of using immutable structure from Clojure as mutable.
https://github.com/clojure/clojure/blob/abe19832c0294fec4c9c55430c9262c4b6d2f8b1/src/jvm/clojure/lang/APersistentVector.java#L247

😱 🙈 yikes -- schoolboy error on my part. Thank you for finding this!
I should have a fix in shortly, but just want to do a quick scan through the rest of the code to make sure I'm not making the same mistake elsewhere first...

Ok, this should be fixed in the latest snapshot now: https://oss.sonatype.org/content/repositories/snapshots/org/odpi/egeria/egeria-connector-xtdb/3.4-SNAPSHOT/egeria-connector-xtdb-3.4-20211208.123952-7-jar-with-dependencies.jar
Would you mind grabbing that (or rebuilding from the latest main branch of this connector repo), and re-testing?
I'll wait to ensure that this is resolved before creating the 3.4 release of the connector.

Thanks It works

For completeness, the error was that the connector code was doing an unchecked casting of a return value from XT into a List<String> (a mutable Java structure) while in fact what was being returned from XT was an IPersistentVector (an immutable Clojure structure).
The error was related to updating the list of users that maintains a given metadata instance, which occurs on basically every update operation. This list of users is only changed if there are new users contributing to the changes -- if the same user repeatedly creates / modifies metadata instances, the list is unchanged, and thus this error is not observed.
Of course, it is perfectly logical that multiple users can contribute updates to a given piece of metadata, and Egeria is designed to allow this -- as is the connector. The problem was that because the connector was treating an immutable Clojure structure as a mutable Java structure, when the .add() method is called (against an IPersistentVector) this throws the UnsupportedOperationException (since the object is immutable, it cannot be added to).
The connector code has now been updated to:

No longer incorrectly cast these return values, but instead to appropriately handle them as IPersistentVector structures (and only convert them to mutable Java structures when mutation is required)
Add explicit unit tests for different users doing creations and updates of the same objects (ensuring the .add() method is always called at some point), which was initially used to reproduce the error and now will be used to ensure no regression is introduced


Writing to read-only slave fails silently inside of MULTI block

Expected behavior
When writing to a read-only slave node outside of a transaction throws a JedisDataException.  I would expect the same behavior inside of a transaction, as seen in the command-line redis client:
<IP_ADDRESS>:7379> multi
OK
<IP_ADDRESS>:7379> set test test
(error) READONLY You can't write against a read only slave.
<IP_ADDRESS>:7379> exec
(empty list or set)

Actual behavior
Writing to a read-only slave as part of a transaction fails silently.
Steps to reproduce:
This fails to write anything without producing any errors:
Jedis j = new Jedis("read-only-slave");

Transaction t = j.multi();
t.set("test", "test");
List<Object> results = t.exec();

Redis / Jedis Configuration
Jedis version: 2.9.0
Redis version: 2.8.13, 3.2.8
Java version: 1.8.0_77

Works as expected
Check https://github.com/xetorthio/jedis/blob/1f1e23b26a4aa244ebfdbcbcdf489b630c841884/src/test/java/redis/clients/jedis/tests/commands/TransactionCommandsTest.java#L218-L232

I don't think your test is describing the same situation.  If create a transaction against a read-only database with a single write command and no reads, there is no error response.  This test fails:
  @Test
  public void testReadOnlyWrite() {
    Jedis j = new Jedis("localhost", 7379);

    Transaction t = j.multi();
    Response<String> resp = t.set("foo", "bar");
    List<Object> results = t.exec();

    assertEquals(results.size(), 1);
  }

@sheinbergon @HeartSaVioR can you give me a hand with this?. I'm super bust ATM :cry:

OK. so @marcosnils  @aniketschneider this is actually quite weird. I'm able to reproduce this issue, but only with a single write and a single read. There are several issues here

The exception exception error message is misleading/wrong. We should probably clarify it to make more sense.

The error handling logic is based on the fact the the set flag is not raised inside the Response class. That flag is only raised assuming some response is received. I think it makes sense, but it's worth discussing.

The problem is inside Queable , specifically in

 protected Response<?> generateResponse(Object data) {
    Response<?> response = pipelinedResponses.poll();
    if (response != null) {
      response.set(data);
    }
    return response;
  }

It tests if the whole object is null, where in-fact, we're facing a scenario where the response body/content is null. So the first fix ( and a one that would make this functionality work as expected )  would be to change  this test

The pipelineResponses field in this object contains only a single response object with its response/exception fields set to null. We of course expect to see the original ERROR message in  response. Sniffing the traffic via tcpdump, you see this :

01:46:06.021491 IP <IP_ADDRESS>.56202 > <IP_ADDRESS>.7379: Flags [P.], seq 16:69, ack 6, win 342, options [nop,nop,TS val 935806 ecr 935803], length 53
E..i<EMAIL_ADDRESS>..G~..G{*3
$3
SET
$3
noo
$3
bar
*2
$3
GET
$3
foo

01:46:06.021549 IP <IP_ADDRESS>.7379 > <IP_ADDRESS>.56202: Flags [P.], seq 6:69, ack 69, win 342, options [nop,nop,TS val 935806 ecr 935806], length 63
E..s..@.@.Qs...............y{.P)...V.g.....
..G~..G~-READONLY You can't write against a read only slave.
+QUEUED

01:46:06.021768 IP <IP_ADDRESS>.56202 > <IP_ADDRESS>.7379: Flags [P.], seq 69:83, ack 69, win 342, options [nop,nop,TS val 935806 ecr 935806], length 14
E..B..@.@.4.............{.P).......V.6.....
..G~..G~*1
$4
EXEC

So to make a long story short , The 'read-only' error takes place even before the exec and therefor not  caught by jedis, only some  weird null response is returned. That  means that it's not something jedis can cope on it's own and we should just stick with what I suggested in section 3.
Hope that's thorough enough :)

@marcosnils  I'll be happy to write a PR to fix this. Let me know.

@sheinbergon I appreciate you digging into this!  I am following along as best I can.
I'm not sure if this is helpful or not, but when following the execution of this test through the jedis code, I ended up here: https://github.com/xetorthio/jedis/blob/master/src/main/java/redis/clients/jedis/Transaction.java#L43
This getMany call does in fact return the relevant JedisDataExceptions caused by the READONLY status of the node, but they are being intentionally discarded.
From my naive perspective, if it's not possible to throw an exception or produce an error when the write command is initially issued (before the EXEC), I would like to see an exception here (before the EXEC is actually processed) if there are any errors in the list returned by getMany.  After all, if some of the commands have not succeeded, I don't want the rest of the transaction to be committed.  However, I don't know if this makes sense for other situations where an error might be included in that list.  After all, if this only comes up when a node is a readonly slave, then it doesn't matter if the transaction is committed or not because the transaction cannot make any changes to the data.

@aniketschneider Yes the point that this error takes place before the exec is what makes the handling more difficult, I came to the same conclusion. The fix suggested in paragraph 3 will eliminate this issue.
Regarding the 2nd  part of your comment, I'm not sure that's how transactions work. Conceptually, the whole point of a transaction is performing an "atomic" block - either everything succeeds or nothing happens. This is not a matter of "SDK implementation behavior", but rather how Redis itself behaves. So that covers.
In any case an internal Redis error is raised, the transaction shouldn't complete, and roll back ( if there's something to rollback at all). again, that's not up to the SDK :) so I believe that from that perspective, you're covered.
@marcosnils  what about commenting on my fix suggestion above?

@sheinbergon your proposed fix seems right, would really appreciate your help crafting a PR.

@marcosnils  my suggested fix was not good enough. It seems that there are some situations where Object data returned is expected to be null ( say some of the tests, or even in general - if eval was used inside multi-exec block). This makes testing against null response scenarios in unified fashion a bit irrelevant.
Zooming out a bit, discerning between scenarios where response is expected and those where the situation is opposite would be IMPOSSIBLE with the current state of the codebase.
The correct way to proceed from here is to try to catch the original error redis sends back when it detects a bad op ( write to read-only slave) prior to calling the exec.
By not discarding JedisDataExecptions that are returned from getMany and verifying to make none-of them exists. What's the original reason for discarding these errors ??

@marcosnils  any response ???

@marcosnils ???

@sheinbergon I've been super busy lately. I'll try to review this week. @HeartSaVioR can you help me out if you have some time?

@marcosnils  @HeartSaVioR  any response ???

Any more word on this?

I have the same issue with Jedis v2.7.3. Not sure if this issue has been fixed with newer versions. In Jedis v2.7.3, the transaction API only uses the reply of the last command in the transaction, that is, the "EXEC" command response. When the transaction gets sent to a slave Redis, despite commands between MULTI and EXEC fail, the reply of EXEC seems OK. So, we can't know if there's any error happens.


app crashing when no email returned in accesstoken by IDP (AzureAD)

I am trying hasura-auth with AzureAD, building webapp with hasura-auth-js SDK.
my app crashing after trying to login with AzureAD SSO. here is the logs
error in hasura-auth logs
auth          | {"time":"2024-04-24T02:53:43.499422588Z","level":"WARN","msg":"email didn't pass access control checks","trace":{"trace_id":"01d71ae9-7306-4a72-884e-78b4f6fbafde","span_id":"","parent_span_id":""},"request":{"client_ip":"<IP_ADDRESS>","method":"POST","url":"/v1/token"}}
auth          | {"time":"2024-04-24T02:53:43.501355046Z","level":"ERROR","msg":"call completed with errors","trace":{"trace_id":"01d71ae9-7306-4a72-884e-78b4f6fbafde","span_id":"","parent_span_id":""},"request":{"client_ip":"<IP_ADDRESS>","method":"POST","url":"/v1/token"},"response":{"status_code":200,"latency_time":2491666,"errors":["json: error calling MarshalJSON for type types.Email: email: failed to pass regex validation"]}}

webapp crash logs

accesstoken return for AzureAD  don't have email at right place

BUT this should not cause server crash...
### Tasks

Just a note, apparently Microsoft have different AD subscriptions, based on plan, expensive corporate plan adds email claim correctly but for low cost plan, email claim is added at upn and unique_name fields. (we have to upgrade to  expensive plan to customize claims)
Since  getting email claim consistently is crucial for hasura-auth/hasura-auth-js to work, we need custom configuration option for AzureAD provider to map email field OR automatically fallback to upn or  unique_name if email field is null or undefined.
Please let me know if I can contribute backward compatible  fix via PR

Please let me know if I can contribute backward compatible fix via PR

Of course, it will be more than appreciated/welcomed :)

@xmlking would you mind checking if the issue still persists in 0.32.0? While AzuredAD's email is probably not being populated yet we remove the requirement of having an email so the crash should be gone.

@xmlking would you mind checking if the issue still persists in 0.32.0? While AzuredAD's email is probably not being populated yet we remove the requirement of having an email so the crash should be gone.

Yes crash issue is fixed with recent changes
But I will be contributing fallback email mapping for azure AD when I find little time
Thanks


Deprecated module
https://forge.puppetlabs.com/modules/puppet/visualstudio/readme
Replacement module
No response
What authority do you have for making this request?
It's my module. (github username matches repo owner or contact info provided for verification), I am a Vox Pupuli PMC member
More information
Hi,
The module isn't maintained anymore and we archived the git repo. Please deprecate it on the forge.

@bastelfreak , It has been marked as deprecated now.


Add 'stats' tool.

Useful to answer questions like:

How many files are created as part of the build?
How many jobs can run concurrently?
What is the length of the longest chain of commands?
How many files are produced by my build?

This is useful when trying to improve the performance of a build system.
Maybe it could interest other people than me. The idea is to provide the basic statistics needed by more elaborated tools.
If it is considered for merging, I will write tests.

Thanks for the PR! It seems nice to have, but doesn't necessarily need to be part of upstream Ninja. Since there hasn't been any comments I'm closing this one, feel free to reopen.


[codelab] MDC-101 Web: Material Components (MDC) Basics (Web)

Just in the beginning when startig with mdc-101 I get

><EMAIL_ADDRESS>start
> webpack-dev-server --progress

sh: 1: webpack-dev-server: not found

Then I installed   sudo npm install webpack-dev-server -g and get
The CLI moved into a separate package: webpack-cli
Please install 'webpack-cli' in addition to webpack itself to use the CLI
-> When using npm: npm i -D webpack-cli
-> When using yarn: yarn add -D webpack-cli
internal/modules/cjs/loader.js:883
  throw err;
  ^

Error: Cannot find module 'webpack-cli/bin/config-yargs'
Require stack:
- /usr/local/lib/node_modules/webpack-dev-server/bin/webpack-dev-server.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:880:15)
    at Function.Module._load (internal/modules/cjs/loader.js:725:27)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/webpack-dev-server/bin/webpack-dev-server.js:65:1)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/usr/local/lib/node_modules/webpack-dev-server/bin/webpack-dev-server.js'
  ]
}

What should I do?

Notice: Error: Cannot find module 'webpack-cli/bin/config-yargs'
It requires webpack-cli to work.  Do: npm i -D webpack-cli


Size prediction mess and not remembering floating size

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit cc94123fa7d45e816327b4b8c8592589313f4718 dirty (renderer: minor fixes to transformations).
Date: Sun Mar 3 05:18:06 2024
Tag: v0.36.0-39-gcc94123f

flags: (if any)

System Information:
System name: Linux
Node name: Arch-ASUS
Release: 6.6.20-1-lts
Version: #1 SMP PREEMPT_DYNAMIC Sun, 03 Mar 2024 07:25:31 +0000

GPU information:
00:02.0 VGA compatible controller [0300]: Intel Corporation 4th Gen Core Processor Integrated Graphics Controller [8086:0416] (rev 06) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Regression
Description
Since f4f3aa2e505ca1b22688fb4a32f31c4bfb851dd0 things gone mad
how gtk and qt things looks on sway(-git)?:

launch app tiled
set float
change float size
close
it get saved

or set kinda rule to launch floating:

change float size
close
it get saved

next time you launch and toggle, or launch floated it got saved size
one drawback is that changing floating size not remembered when just toggling float/tiled, without closing app
as far as i can remember, Hyprland mostly never did making apps to save state, or maybe long time ago it did
but atleast before f4f3aa2e505ca1b22688fb4a32f31c4bfb851dd0 already saved sizes have been respected
for majority of gtk apps this sizes can be set/querried with gsettings or dconf
qt, as i understand stores in per-app config
what we have now:
for example i wish to launch Authenticator floating with rule.
okay, but as soon as it been launched tiled once, this settings got overwritten, if i togglefloating or launch again with rule, it get sized with this new weird predicted size
then i have to explicitly set it with gsettings, until next tiled launch
for qt apps thing might be more complicated as it might look like geometry=bytearray..., and it's not even possible to find all this config files with different naming schemas
or just launch sway and set it there, lul
sure i can use windowrules to set exact or atleast restrict max size for floating, but toggling float on tiled window not worky now
some apps i tested:

firefox (didn't found size settings)
Authenticator (gtk)
qBittorent (qt)
wireshark (qt)
Nautilus (gtk)
Telegram (qt)
easyeffects (gtk)

i understand that it might be apps' responsibility to correctly handle such thing, but still,
all of them save/restore sizes on sway
prior f4f3aa2e505ca1b22688fb4a32f31c4bfb851dd0 only Telegram saved floating size without any intervention
rest atleast respected it when togglefloating and float windowrule
exceptions are apps like Helvum (pipewire's native gtk patchbay) without any stored settings and always launching at fixed floating size, but toggling from tiled again gives predicted weirdo
the ideal thing would be sway's behaviour + respect floating size change when toggling float/tiled (like it works now, but until app is closed)
PS: and yeah, toggling tiled foot to float neither works, initially targeted by f4f3aa2e505ca1b22688fb4a32f31c4bfb851dd0
How to reproduce
above
Crash reports, logs, images, videos
No response

le pardon for such a long story, sometimes its hard to explain when there are no obvious conditions

remembering the size is the app's job. Hyprland seems to send the correct 0,0 configure whenever app is launching floating (aka. "do whatever") and sends a specific size when it's tiled to avoid a "flash" at the beginning.
Nautilus has this bug, but Dolphin doesn't, which further solidifies the case of this being a bug on the side of the apps.
windowrulev2 = size 600 400,floating:1 I guess

like okay.. but

togglefloating? chain size dispatcher or script? sure why not
sway is doing smth wrong and apps' bugs become features?

sway is doing smth wrong and apps' bugs become features?

yes, basically. Sway always sends the apps a "do whatever". I assume some apps take the last "don't do whatever" and forcefully remember that.


Decode more ISS values

Codecov Report

Merging #5 (90e02e1) into main (19278c3) will increase coverage by 3.25%.
The diff coverage is 89.38%.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   65.75%   69.00%   +3.25%
==========================================
  Files           8       12       +4
  Lines        1241     1410     +169
==========================================
+ Hits          816      973     +157
- Misses        425      437      +12

Impacted Files
Coverage Δ

src/bti.rs
0.00% <0.00%> (ø)

src/hvc.rs
0.00% <0.00%> (ø)

src/ld64b.rs
72.72% <72.72%> (ø)

src/lib.rs
50.80% <76.47%> (+0.49%)
:arrow_up:

src/sve.rs
100.00% <100.00%> (ø)

src/tests.rs
100.00% <100.00%> (ø)

src/common.rs
80.00% <0.00%> (+80.00%)
:arrow_up:

Continue to review full report at Codecov.

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


chore(*) upgrade to latest development version of Concerto with new packages

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Changes

Upgrade markdown transform to latest development version of concerto, with new package structure and metamodel-based parser

Coverage remained the same at 85.56% when pulling 80f43801e4a65019f56fd602f9adb0b1befb8911 on jeromesimeon:js-chore-latest-concerto-dev into 4f1201a8a808b03e50142ce74d8037e1e4667ab3 on accordproject:master.


python help for r.binary includes markdown

The help text that comes from help(r.binary) in the python driver includes markdown links that are nonfunctional, e.g.:

    * [coerce_to](/api/python/coerce_to/) can coerce `binary` objects to `string` types
    * [count](/api/python/count/) will return the number of bytes in the object
    * [slice](/api/python/slice/) will treat bytes like array indexes (i.e., `slice(10,20)` will return bytes 10–19)
    * [type_of](/api/python/type_of) returns `PTYPE`
    * [info](/api/python/info) will return information on a binary object.

We should either reformat this to include full links, or suppress the link parts.

There are a lot of places in the API documentation where this shorthand is being used. What happens in the other cases? Changing them from relative links to fully-qualified ones might be better handled by the Python help documentation build script.

Pinging @larkost on this, when you get a chance -- I think this might be something to change in the Python script. (I might be able to make that change, but not without querying you about whether that's a terrible idea first.)

Nope, that sounds like the correct approach given how we generate documentation for Python.

A change to the script which appears to work is in CR 2550.


Feedback extra info

Update eform to include extra information, such as, device platform and app version.
Update FeedbackView to send extra info
Look into ways to identify app version

https://github.com/rebeccahughes/react-native-device-info

Done in dev 132aadb

Add location (if available) and user preferences

Add location / GPS info is now handled under issue #179
Preferences to be added as a separate issue later if the need arises


python312Packages.mypy-boto3-*: updates

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 345867 run on x86_64-linux 1

28 packages built:

python311Packages.mypy-boto3-clouddirectory
python311Packages.mypy-boto3-clouddirectory.dist
python311Packages.mypy-boto3-codeartifact
python311Packages.mypy-boto3-codeartifact.dist
python311Packages.mypy-boto3-connect
python311Packages.mypy-boto3-connect.dist
python311Packages.mypy-boto3-pricing
python311Packages.mypy-boto3-pricing.dist
python311Packages.mypy-boto3-rds
python311Packages.mypy-boto3-rds.dist
python311Packages.mypy-boto3-resource-groups
python311Packages.mypy-boto3-resource-groups.dist
python311Packages.mypy-boto3-verifiedpermissions
python311Packages.mypy-boto3-verifiedpermissions.dist
python312Packages.mypy-boto3-clouddirectory
python312Packages.mypy-boto3-clouddirectory.dist
python312Packages.mypy-boto3-codeartifact
python312Packages.mypy-boto3-codeartifact.dist
python312Packages.mypy-boto3-connect
python312Packages.mypy-boto3-connect.dist
python312Packages.mypy-boto3-pricing
python312Packages.mypy-boto3-pricing.dist
python312Packages.mypy-boto3-rds
python312Packages.mypy-boto3-rds.dist
python312Packages.mypy-boto3-resource-groups
python312Packages.mypy-boto3-resource-groups.dist
python312Packages.mypy-boto3-verifiedpermissions
python312Packages.mypy-boto3-verifiedpermissions.dist


One additional step require for the step 6 of the Tutorial 5: Run the Employee Onboarding Sample Scenario

Hi,
While executing "Automate and Extend Employee Onboarding" starter scenario, we observed the below findings.
Tutorial 5: Run the Employee Onboarding Sample Scenario
Step 6: Continue with the workflow execution.
In the tutorial, it is written as:
3. In the SAP Fiori launchpad, open the Monitor Workflow - Workflow Instances tile, choose the filter button, add the Completed status.
Actual Result: When we choose the filter button, there are two options.

Now select the option “Status”. Then we will get the list to choose the “Completed” status.

Now add the “Completed” status.
Note: I have executed this test for AWS in trial account for US region.
Many thanks & regards,
Baskar

Tutorial updated. Please close the ticket


Patient-HSP schedule appointments and edit them-backend

HSP and Patient only modify patient. So we need to have one method that takes in Patient Object

finished


[FEATURE] Show only vulnerable dependencies by default

What are you trying to do?

From user testing, we heard that it would be nice to only show vulnerable dependencies found, which is similar to how bundler-audit and other tools function.

What feature or behavior is this required for?

This just makes the tool more like other unix tooling, giving only actionable information to someone

How could we solve this issue? (Not knowing is okay!)

It still makes sense to have a mode that outputs the non vulnerable dependencies, I would introduce a flag like --loud where you can get the full output!
In the other tools, we also grouped the dependencies such that it was:
Non Vulnerable Dependencies
....list
Vulnerable Dependencies
....list
SUMMARY TABLE
I think it makes sense to do the same thing here!

Anything else?

Have a blast! Hit me up if you need any questions answered!
cc @bhamail / @DarthHater / @brittanybelle / @gmohre

https://github.com/sonatype-nexus-community/chelsea/pull/33


HLIL should show a type cast when assigning to a different type

What is the feature you'd like to have?
When an assignment happens in HLIL, and the source and destination expressions have different types, a type cast should be displayed to indicate the difference in types.
Is your feature request related to a problem?
I want to know where type casts happen, so I can manually change the type of variables or structure members without having to go find the source type of the thing being assigned. Would also help with situations like #4582 by providing actionable diagnostic text to make it easier to work around deficiencies in analysis until they are improved.
Are any alternative solutions acceptable?
Uncertain, but likely not.
Additional Information:
Feature request by proxy :)

Perhaps we could do something a little nicer looking than C-style type casts as those would really harm readability. Perhaps something like := could represent an assignment with type cast and give you something to hover over to show what its type casting from/to

I'm not sure there's a more succinct way to indicate it than a C-style type cast. Also, given we have a display option to hide type casts, people will already have the option to turn them off if they don't care. The := idea, while interesting, probably wouldn't work for HLIL where casts can appear at arbitrary points in expressions. I'm not sure I'm a fan of requiring hover/mouse interaction either (also that would probably require much more difficult changes to the renderer).


.ascii list broken

It seems as though .ascii list is not working as documented. I receive the following when running a list command:
[ERROR] Invalid file name. (Use ".ascii list" for a list of valid file names.)
Seems to be true whether or not I generate the list file with the list creation script.

Are you naming the file list.txt or list? Not sure if this is actually an issue or not. Check your permissions, etc.

You use the ascii_list.py file to generate a list of all the files. This will make a list.txt file in the /data/ directory. You can upload that file to pastebin and then place the url that pastebin in your /data/ascii/ directory. This way when you do .ascii list, it will reply with a URL to all the files, instead of sending a line for EVERY ascii file.


⚠ Bofrost.at has degraded performance

In 2cf3a29, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1644 ms

Resolved: Bofrost.at performance has improved in 7bd6da7 after 9 minutes.


space is fake

the projection is inaccurate...please check out the Flat Earth TV Guide so to help you make a more accurate projection https://FlatEarthTVGuide.com help make it go viral even if to just get people to laugh at it

Thank you! I feel flatter already.


iOS Not Building On React Native 0.68

Followed all the setup steps and working find on debug builds and on the emulator but on release builds get the following errors on compilation:
▸ Compiling CrispChatSdk.swift

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:8:18: type 'CrispSDK' has no member 'setTokenID'

        CrispSDK.setTokenID(tokenID: id)
^

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:13:18: type 'CrispSDK' has no member 'user'

        CrispSDK.user.email = email
        ~~~~~~~~ ^~~~~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:18:18: type 'CrispSDK' has no member 'user'

        CrispSDK.user.nickname = nickname
        ~~~~~~~~ ^~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:22:18: type 'CrispSDK' has no member 'user'

        CrispSDK.user.phone = phone
        ~~~~~~~~ ^~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:27:18: type 'CrispSDK' has no member 'user'

        CrispSDK.user.avatar = URL(string: url)
        ~~~~~~~~ ^~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:32:18: type 'CrispSDK' has no member 'session'

        CrispSDK.session.segment = segment
        ~~~~~~~~ ^~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:37:18: type 'CrispSDK' has no member 'session'

        CrispSDK.session.setString(value, forKey: key)
        ~~~~~~~~ ^~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:42:18: type 'CrispSDK' has no member 'session'

        CrispSDK.session.setString(String(value), forKey: key)
        ~~~~~~~~ ^~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:47:18: type 'CrispSDK' has no member 'session'

        CrispSDK.session.setInt(value, forKey: key)
        ~~~~~~~~ ^~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:52:18: type 'CrispSDK' has no member 'session'

        CrispSDK.session.pushEvent(Crisp.SessionEvent(name: eventName, color: color))
        ~~~~~~~~ ^~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:52:54: argument passed to call that takes no arguments

        CrispSDK.session.pushEvent(Crisp.SessionEvent(name: eventName, color: color))
        ~~~~~~~~ ^~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:57:18: type 'CrispSDK' has no member 'session'

        CrispSDK.session.reset()
                                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

❌  /Users/ankur/Documents/Code/company/node_modules/react-native-crisp-chat-sdk/ios/CrispChatSdk.swift:63:77: 'ChatViewController' is unavailable: cannot find Swift declaration for this class

            UIApplication.shared.windows.first?.rootViewController?.present(ChatViewController(), animated: true)
        ~~~~~~~~ ^~~~~~~

▸ Compiling CrispChatSdk.swift

After further testing this error seems to have been resolved by removing armv7 from ARCHS  in buildSettings


Stop the animation if the value is beyond 1

Hi All,
Need help on how to stop the animation of the progress bar in the below scenario bar.animate(progressbarVal); // progressbarVal >1
and the text at the center of the progressbar should change if the value is >1
and my progressbar settings are
bar = new ProgressBar.Circle(progressbar_container, {
strokeWidth: 2,
easing: 'easeInOut',
duration: 1400,
color: colorVal,
trailColor: '#c0c0c0',
trailWidth: 2,
svgStyle: null,
step: function(state,circle){
circle.setText($scope.usageCalculation);
}
});
my intention is that color will be fixed/constant with no more animations if val keeps increasing beyond 1 but the circle.setText($scope.usageCalculation); keep updating the value even though its beyond 1

I Have fixed this issue.


[do not merge] debug CI for challenge tasks

Motivation and Context

How Has This Been Tested

Types of changes

Docs change / refactoring / dependency upgrade
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

This fixed the problem. I will close.


Possible to use TLBE for a replay?

Factorio does have the "enable replay" tickbox when starting a new game which will supposedly save everything that ever happened inside the savegame.
Now I wonder if it would be possible to use/setup TLBE while watching a replay.
When watching a replay I noticed that everything seems to happen exactly as it did in the original game
including which menus were opened at which time (inventory, production menus, etc).
Any idea if this would be possible at all?
I see two problems:

Does the replay even load if you didn't play that savegame with TLBE enabled?
If all the menu interactions are also replayed, how can you possibly setup TLBE?

It would be pretty cool if it were possible as you could easily make some taxing recordings (high resolution, multiple cameras, higher framerate) while running the replay in offline mode - possibly scripted?

Does the replay even load if you didn't play that savegame with TLBE enabled?

You cannot add new mods (or any other modification) on a replay, so even if TLBE would have been loaded, you cannot change camera settings (as the game is not able to figure out that this does not influence anything on the game play, has to do with consistency/predictability that is used by Factorio (multi player))

If all the menu interactions are also replayed, how can you possibly setup TLBE?

Exactly... :disappointed:
There are people who modify the control.lua of a game to take screenshot(s) afterwards, one of the devs even has a gist showing how it could be done.
Ofcourse these are pretty simple screenshots compared to what TBLE does... Maybe this could be used to (ab)use the mod to start taking screenshots? (Assuming the mod was loaded in the game already) Or copy&paste all mod code into the control.lua...?
TL;DR I am pretty sure the mod itself cannot be used (as a mod), but hacking control.lua (theoretically) would be able to.


Fix bug in enum tokenization

Let's wait. I'll do my best to fix names tomorrow, and then we can merge
#907.
On Tue, Jun 27, 2017 at 4:31 AM Ólafur Páll Geirsson <
<EMAIL_ADDRESS>wrote:

@olafurpg approved this pull request.
LGTM 👍 We may decide to wait until #907
https://github.com/scalameta/scalameta/pull/907 is merged to get this
in.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scalameta/scalameta/pull/974#pullrequestreview-46518870,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAlLgO5fsICKm5E1iI-RNlC2XrtUvzxSks5sIOgmgaJpZM4OGYkm
.


Design repository library

[Update 2/14/22]

ADR 10 shows the updated direction of this issue including further reading (design document, prototype implementation)
Comments in thread here below point to survey documents for existing repository tooling, based on which ADR 10 was developed

[Original issue description]
Identify what functionality from existing repository_tool/repository_lib/roledb/keydb modules should exist in a TUF repository library on top of a metadata model as sketched in #1112.
Start with API functionality.
Coordinate with #1134.

I think a good place to start with this would be looking at what scaffolding Warehouse ends up building on top of the metadata API.
https://github.com/pypa/warehouse/pull/7488
https://github.com/pypa/warehouse/pull/8955

See also a WIP addition to the specification describes the repository workflow for adding/updating targets which this library should implement: https://github.com/theupdateframework/specification/pull/153

See also a WIP addition to the specification describes the repository workflow for adding/updating targets which this library should implement: https://github.com/theupdateframework/specification/pull/153

This has landed and can be referenced here https://theupdateframework.github.io/specification/latest/#adding-updating-targets

Adding more relevant documents ...

TUF (Re-)Design Document / Section: Repo tool -- high-level directions, potential functionality
Repo Usage Scenarios -- details required inputs, e.g. knowledge about role metadata, for each scenario
Repo Tool Design -- basic high-level design with many open questions
Strawman repo design proposal -- bottom-up design, based on vmware-labs/repository-editor-for-tuf, seems very promising...

Next steps:

Revise open questions  and generalize problems
Revise strawman design for suitability to answer these questions
Revise strawman design / prototype implementation for remaining open questions
Write design document

Next steps:

Revise open questions  and generalize problems
Revise strawman design for suitability to answer these questions
Revise strawman design / prototype implementation for remaining open questions
Write design document

See updated issue description above for latest results...

Feature request:
Convenience interface that takes a path to local target file(s) and constructs targets metadata, computing length and hashes from the local file and normalizing the path to match the TARGETPATH specification (see #1019 for details and challenges)

I would like to work on this issue, where should I start ?

I would like to work on this issue, where should I start ?

Reading the resources referenced above in the issue description and brainstorming designs seems like the way to start.
But as @jku has mentioned on slack, this is likely a long term issue that is going to require experimentation and quite a bit of team work to solve.

Another implementation we could look to for inspiration is Rugged, which implements repository and a key manager classes on top of the low-level metadata API.
A key manager/keyring feels like a useful helper to provide.

I've decided to just do a PR of this:

a tiny repository abstraction
example implementation of that abstraction (a in-memory ephemeral repository a bit like the one in tests)
demo repository application using that in-memory implementation that simulates a live  repository by generating changes every few seconds

I have that 90% ready, will push the PR this week. Future steps would include:

example developer tool that is able to sign changes and submit them to the demo repository application. I have a rough idea, should not be too much work

I have no assumptions that the API would be final in this PR: as minimal as it is, new use cases seem to always bring new complications... So I suggest we

keep the repository abstraction as unstable API for now
build more implementations to gather data (potential ones at least RSTUF, repository-editor-for-tuf, repository-playground)


Allow setting search root

Often in a deep directory, I want to be able to cd and search to anywhere below $HOME.
I'd like to be able to do:
alias hcd='f() { SMARTCD_ROOT=$HOME; cd "$@"; unset SMARTCD_HOME; };f'

Would you consider a SMARTCD_ROOT variable from which the listing is generated?  I'd suggest a subshell for the generation to allow for user ^C or permission denied on the target directory.

Thank you! 💙


Fix map_blocks docs' formatting

Was looking funky. Not 100% sure this is correct but seems consistent with the others

Will merge this soon — speak now or revert later :)


Client part

/cc @inikulin
The client scripts and QUnit tests added. @inikulin please, see the structure and naming of the files, maybe we should make some changes.
Now linting is disabled for client scripts and tests because there are a lot of error (more than 6k), we'll fix it but it'll take some time.
Also, I've create a TODO list, what we should do next:

Linting (5500 errors in client scripts, 1200 errors in tests)
Task templates (run, run_iframe), move any logic from the templates to the scripts
Change some QUnit tests with functional tests
CI QUnit-testing and functional testing
use Hammerhead as a npm module
add a postfix to css rules via Hammerhead (move it from Gulpfile.js)
DXperience tests
Get rid of async

1)Let's move code from client/runner to client/test-run. NOTE: there are templates where already, use them.
2)Remove client/runner/api-v2
3)In all subpackages, move dependency wrappers to the deps dir, e.g. in ui move testcafe-core.js and hammerhead-api.js to the deps/testcafe-core.js and deps/hammerhead.js appropriately.
4)In ui move all cursor related files to the cursor dir and rename files (remove cursor-prefix)
5)Why ui/core.js contains just logic related to the progressPanel, shouldn't it be in the progress-panel.js?
In addition to the TODO list:
Get rid of jquery. And fill an issue for each item in the todo list, please.

We need a ES6 transition plan as well:
1)Constants (const keyword, new naming format)
2)Classes (use where possible, any object which has state now or init method or something like this should be turned into the classes and instantiated there necessary)
3)Arrows
4)Promises
More TODO:
Get rid of shared constants, use error types from https://github.com/superroma/testcafe-phoenix/blob/master/src/test-error/type.js, test run service message commands from https://github.com/superroma/testcafe-phoenix/blob/master/src/runner/test-run/command.js,
worker commands from https://github.com/superroma/testcafe-phoenix/blob/master/src/browser-connection/command.js
Please, turn all TODOs and ES6 plan into appropriate GH issues.

\r-

3)In all subpackages, move dependency wrappers to the deps dir, e.g. in ui move testcafe-core.js and hammerhead-api.js to the deps/testcafe-core.js and deps/hammerhead.js appropriately.
@inikulin jquery, async are deps too?

@AlexanderMoskovkin yes

FPR

Looking at this now I think we shouldn't move runner into test-run since they need end up in the different files. Really sorry for this.
\r-

no problem, I'll rollback this

FPR

lgtm

thanks


return current month' number of days if month value is lower than 1 or bigger than 12

May I know the reason why not accepting this pull request? Thank you.

I merged your more recent PR or have I got this wrong?

actually there are two PR's on same file... with different modifications.

Sorry about that, but I didn't want to do two different changes in the same pull request. I guessed I messed that up. Next time will be more careful.

actually I should have merged the first one first, doing it back to front has not caused conflicts, I'll correct that later or if you have time can you update the PR.


[6.0] tests: add a missing REQUIRES: executable_test in SILOptimizer/stack-promotion-crash.swift

Explanation: Fixes a test which was added in https://github.com/swiftlang/swift/pull/74982
Risk: Zero. Just fixes a test
Issue: rdar://131519168
Reviewer:  @shahmishal
Main branch PR: https://github.com/swiftlang/swift/pull/75169

@swift-ci test


Update platforms.md

QNX, BSD Unix, Fuchsia

Thanks again for helping keep the page up-to-date!
looks all good to me,


Get a 404 when hitting api endpoints defined in triggers

Describe the bug
When I go to the url of an API trigger, I get a 404 error.
To Reproduce
Steps to reproduce the behavior:

Create an api trigger
go to the url listed in the table

This needs to be a POST (https://github.com/saltcorn/saltcorn/blob/master/packages/server/routes/api.js#L289-L290), we should add that clarifier to the label in the admin UI

and how do you return a body?


Tube Creation Bug for exact 90° angles

Bug repro on playground:
Expected result: Tube with 90° angles
Current result: tube stops with spike and all following points of the tube are ignored

Changing the last point of the tube to slightly other positions in the end works, for example (0, 10, 0.1), but with exact values it fails. With a slightly changed last position it is weird as well: (0.1, 10, 0)

Pinging @jbousquie or @BabylonJSGuide

The tube is built on top the Path3D.
You need to add far more point in your path to get the correct final shape. This is a known (and expected) behavior.


NOISSUE: run func test to create fund only once

- What I did
- How I did it
- How to verify it
- Description for the changelog

Codecov Report

Merging #71 into master will decrease coverage by 0.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
- Coverage   15.30%   14.95%   -0.36%
==========================================
  Files           7        7
  Lines        1006     1070      +64
==========================================
+ Hits          154      160       +6
- Misses        844      897      +53
- Partials        8       13       +5

Impacted Files
Coverage Δ

...cation/builtin/contract/deposit/deposit.wrapper.go
0.00% <0.00%> (ø)

...et/application/builtin/contract/deposit/deposit.go
2.51% <0.00%> (+2.51%)
:arrow_up:

Continue to review full report at Codecov.

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