Split up platform-specific controller logic

This is a rework of commits that didn't make the cut for #60.
This time the platform-specific logic is kept in the velero package instead of polluting the storage drivers.
There's also new tests that exercise the reconciler helper methods for all AWS and GCP regions.

Is the refactoring done in this PR final? I am adding support for azure in https://github.com/openshift/managed-velero-operator/pull/77 and I might have to refactor a lot of code if this gets merged.

@hamza3202 It's still under review.  @cblecker and I still need to go over it in detail.

Yeah, I heard about it from @cblecker. I am gonna keep my PR on hold till then but would be nice to get this merged soon.

@hamza3202 Sorry for the long delay, other priorities superseded this.
@cblecker and I finally had time to go through this in detail today.  He requested some structural changes which I'll try to complete in the next day or two.


relax avx512 threshold for zen5

zen5 has quite a low avx512 overhead and hence it might make sense to lower the minimum input size for this architecture.
http://www.numberworld.org/blogs/2024_8_7_zen5_avx512_teardown/

@ronag Can you be specific regarding the code you are thinking about?

@ronag I don't think we have an avx512 threshold in simdutf?
We exclude older Intel CPUs (e.g., Skylake X) because AVX-512 did poorly generally on these processors. Starting with Ice Lake, Rocket Lake, Zen 4 and Zen 5... AVX-512 is generally a win so we are not shy about using it.

Example: https://github.com/simdutf/simdutf/blob/eddf7e235cd722d73cb9cba82e30e4442f2a6f4e/src/westmere/implementation.cpp#L702

@ronag You are pointing at the westmere code which will only ever run on very old x64 processors. None of the two paths use AVX-512.
The relevant code for Zen 5 is the icelake kernel:
https://github.com/simdutf/simdutf/blob/eddf7e235cd722d73cb9cba82e30e4442f2a6f4e/src/icelake/implementation.cpp#L1289
There is still a branch (with long strings and short strings). But notice that both paths use AVX-512. Yet the algorithm differs. In one path, we process blocks of 256 bytes on the assumption that there are potentially several of time (so here, we assume 8 blocks of 256 bytes).
This could be tuned further, meaning that I just set the 2048 arbitrarily. We could adjust it. But it has to be at least 256.
Importantly, we always use AVX-512... except when the string is under 64 characters. For string under 64 characters, we just use a scalar routine. It would be easy to replace the scalar routine with a masked load...


fix: getTimeZoneOffset needs to return a double instead of integer at v8

I found that the return time of the datenum method in Chrome browser is incorrect, consider customizing a getTimeZoneOffset function to fix Date.prototype.getTimezoneOffset() return error value

I see, since v8 does not fix the problem, do we consider fixing the problem in the code.

@SheetJSDev, Would be nice if we get new release with latest commits. Waiting for once of the security fix ReDos in your package to be fixed. They say below that it is fixed in master branch but not yet released. Many thanks if you can do it.


Downward Compatibility

For rails5.2, ActiveRecord::Migrator#initialize only receive three arguments.
This gem support ruby2.6 in gemspec file, but numbered parameters is a new feature added in ruby 2.7

PS: English is not my native language; please excuse typing errors.

@ka8725 Can you help me with this PR? Thanks.


Tetr.io

Game Type
Web
Game Url
https://tetr.io
Comment
No response

I will add this in the next game update

k thanks!

FYI tetr.io is not trusted on school computers


Add ActionButtonIconsData for overriding action icons

This adds the ability to override the back button, close button, drawer button, and end drawer button icons. This also adds CloseButtonIcon, DraweButton, DrawerButtonIcon, EndDrawerButton, and EndDrawerButtonIcon.
Fixes https://github.com/flutter/flutter/issues/117578
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is [test-exempt].
[ ] All existing and new tests are passing.

Example Gist

Example for overriding action icons using ActionButtonIconsData in ThemeData
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

final hasCustomActionIconsNotifier = ValueNotifier(true);

class _EndDrawerButton extends StatelessWidget {
  const _EndDrawerButton({
    Key? key,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final localization = MaterialLocalizations.of(context);
    return Icon(
      Icons.more_horiz_rounded,
      semanticLabel: localization.openAppDrawerTooltip,
    );
  }
}

class _DrawerButton extends StatelessWidget {
  const _DrawerButton({
    Key? key,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final localization = MaterialLocalizations.of(context);
    return Icon(
      Icons.segment_rounded,
      semanticLabel: localization.openAppDrawerTooltip,
    );
  }
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return AnimatedBuilder(
      animation: hasCustomActionIconsNotifier,
      builder: (context, _) {
        final isCustomActionIconsEnabled = hasCustomActionIconsNotifier.value;
        return MaterialApp(
          title: 'Flutter Demo',
          debugShowCheckedModeBanner: false,
          theme: ThemeData(
            useMaterial3: true,
            actionButtonIcons: isCustomActionIconsEnabled
                ? ActionButtonIconsData(
                    backButtonIconBuilder: (context) {
                      return const Icon(Icons.arrow_back_ios_new_rounded);
                    },
                    drawerButtonIconBuilder: (context) {
                      return const _DrawerButton();
                    },
                    endDrawerButtonIconBuilder: (context) {
                      return const _EndDrawerButton();
                    },
                  )
                : null,
          ),
          home: const MyHomePage(title: 'Flutter Demo Home Page'),
        );
      },
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(title),
      ),
      drawer: const Drawer(),
      body: Center(
        child: Row(
          mainAxisAlignment: MainAxisAlignment.center,
          children: [
            Flexible(
              child: ConstrainedBox(
                constraints: const BoxConstraints(
                  maxWidth: 600,
                ),
                child: Column(
                  mainAxisAlignment: MainAxisAlignment.center,
                  children: <Widget>[
                    AnimatedBuilder(
                      animation: hasCustomActionIconsNotifier,
                      builder: (context, _) {
                        return SwitchListTile(
                          value: hasCustomActionIconsNotifier.value,
                          onChanged: (value) {
                            hasCustomActionIconsNotifier.value = value;
                          },
                          title: const Text('Enable custom action icons'),
                        );
                      },
                    ),
                    ListTile(
                      onTap: () {
                        Navigator.of(context).push(
                          MaterialPageRoute(builder: (context) {
                            return const MySecondPage();
                          }),
                        );
                      },
                      title: const Text('Next page'),
                      trailing: const Icon(Icons.forward),
                    ),
                  ],
                ),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

class MySecondPage extends StatelessWidget {
  const MySecondPage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Second page'),
      ),
      endDrawer: const Drawer(),
    );
  }
}

Screenshots
Customized

Default

This isn't entirely conventional but it's close. I would have expected an ActionIconTheme and ActionIconThemeData. If you moved all of the action buttons and icons to action_buttons.dart you could probably factor out some common implementation into private classes.
CC @darrenaustin @QuncCccccc @guidezpl @esouthren

@HansMuller, I've refactored the code as per your suggestions. If it looks good then I can write tests for them.

This appears to be headed in a good direction. Two things:

Please fix up the analyzer errors
I've assigned https://github.com/flutter/flutter/issues/107646 to you because I'm hoping that we can resolve that issue here. If that doesn't seem possible, let me know.

@HansMuller I've added tests for the action buttons, action icon theme, and an example (with test) in examples/api.

Screenshots of the example

I've fixed everything but 3 tests are still failing 🙁

Thank you very much @HansMuller  for helping, and reviewing the PR! I'm very grateful. What are the next steps here for me?

"Google testing" failed because existing applications defined a widget called DrawerButton. We (Google) will need to deal with that here, before this PR can land.

Merging with Google Testing failing. @HansMuller has an internal patch ready to be applied when this lands.

The internal patch is cl/511506840


Add dgeorgiev23's slide

Add a second slide to the original presentation.

Add second slide to the original presentation.

Add content to second slide.

Add content to second slide.


change(scan): Use the on-disk database for keys and results

Motivation
This PR replaces the in-memory database with a database on disk.
Depends-On: #8031
Close #8019
The remaining checklist items in that ticket actually belong in other tickets.
PR Author Checklist
Check before marking the PR as ready for review:

[x] Will the PR name make sense to users?
[x] Does the PR have a priority label?
[x] Have you added or updated tests?
[x] Is the documentation up to date?

For significant changes:

[ ] Is there a summary in the CHANGELOG?
[ ] Can these changes be split into multiple PRs?

If a checkbox isn't relevant to the PR, mark it as done.
Specifications
The design is in ticket #8019.
Complex Code or Requirements
The "get all keys" code is a bit messy, we can fix it up after the MVP when we have a column family just for keys.
Solution

Implement database serialization for keys and results
Add key/birthday read and write methods
Add scanning result read and write methods
Spawn blocking database functions on a blocking async task

Testing
I updated the existing tests, and marked the functions that need tests with a TODO. We can do those tests in the PRs where we actually use those functions, because we might change how those functions work when we start using them.
Review
This is a routine scanner database change.
Reviewer Checklist
Check before approving the PR:

[ ] Does the PR scope match the ticket?
[ ] Are there enough tests to make sure it works? Do the tests cover the PR motivation?
[ ] Are all the PR blockers dealt with?
PR blockers can be dealt with in new tickets or PRs.

And check the PR Author checklist is complete.
Follow Up Work

#8021

And then:

#8022
#7813

After i approved https://github.com/ZcashFoundation/zebra/pull/8031 and it gor merged the automatic rebase into main didnt worked very well. The commits were repeated and there were conflicts. I cherry picked the relevant commits and force pushed.

Refresh didn't worked so i sent a small doc change to trigger the CI again. https://github.com/ZcashFoundation/zebra/pull/8036/commits/d216d03e781ede2baf4281f17fb352ddae7ddecd


Remove password protection

Hi, I'm just looking for general guideness to remove password protection. Can you please point me in the right direction and let me know if there is something I should consider about it?
So far I understant I will loose the hability to have playlist.
I don't need account functionality.

User accounts (and with them, passwords) are a critical part of how Koel works, so the short answer is: you can't. If you must though, a hack like an auto-submitting the login form may work.


Auto Refresh Issue

Hi,
I am new to the concept of SSE and hence please excuse my naiveness.  I have used your package and it works like a charm.  Here is the issue I am facing and would like some guidance.
I am creating a stock market watchlist in my app where the server is repeatedly downloading the ticker price data from the database and sending it to all the client using your package.  The front end uses ReactJS which is somewhat irrelevant in this context
client.js
	componentDidMount(){
	 if (!!window.EventSource) {
		 this.source = new EventSource('http://localhost:3000/channel/liveFeed?');
		 this.source.addEventListener("watchlist", function(e) {
			 var { feed } = JSON.parse(e.data);
			 var quotefeed = feed.map((each,idx)=>{
																							 return(
			{
                                                             ticker:each['Symbol'],
							     time:each['Time of Last Trade'],
							     open:each['Opening Price'],
							     high:each['High Price'],
							     low:each['Low Price'],
							     close:each['Close Price'],
							     volume:each['Volume'],
							     lastClose:each['Closing Price Previous Day'],
							     change:parseInt(each['Close Price']) - parseInt(each['Closing Price Previous Day']),
							     percentChange:Math.round((parseInt(each['Close Price']) - parseInt(each['Closing Price Previous Day']))/parseInt(each['Closing Price Previous Day'])*100)/100
}
																							 )
																						 })
			 this.setState({livePriceFeed:quotefeed});
		 }.bind(this), false);

		 this.source.addEventListener("open", function(e) {
			 console.log("OPENED");
		 }.bind(this), false);

		 this.source.addEventListener("error", function(e) {
			 console.log("ERROR");
			 if (e.readyState == EventSource.CLOSED) {
				 console.log("CLOSED");
			 }
		 }.bind(this), false);
	 }
 }

And server-side uses your example using express with small tweak to the data-provider.js
const priceFeeder = () => {
  logger.info('I m database');
  pool.getConnection((err,connection)=>{
    if (err) {
      logger.info('error when connecting to db:', err);
      connection.end();
    };
    connection.query('SELECT * FROM DataLake', function(err, rows, fields)
    {
      if (err) {
        logger.info('error when connecting to db:', err);
        connection.end();
      };
      logger.info("Rows:" + rows.length);
      emitter.emit('liveFeed', {
            id: (new Date()).toISOString(),
            event: 'watchlist',
            feed: rows
        });
      connection.release();
    });
  })
}

setTimeout(priceFeeder, 5000);
setInterval(priceFeeder, 60000);

channels.js
const liveFeedChannel = new SseChannel({
    retryTimeout: 250,
    cors: { origins: ['*'] },
    pingInterval: 60 * 1000,
    jsonEncode: true,
    historySize: 5
});

dataProvider.on('liveFeed', function(randomData) {
    liveFeedChannel.send({ event:randomData.event,data:randomData,id:randomData.id});
});

As you can see, the idea is to first trigger  priceFeeder 5000ms after the server starts and then periodically after every 1 minute.  This works per expectation but the issue is with the client side.
When the client starts and establishes the SSE connection using the url 'http://localhost:3000/channel/liveFeed?', it starts getting the feed from the server only after the first data refresh on the server side post client launch.  So, for ~ 1minute, it shows no data and then starts refreshing as usual.
Now if I change the url to 'http://localhost:3000/channel/liveFeed?lastEventId=0',  the client gets all the data from the start the moment the client launches (which is great) but it does not refresh every time new data is sent by the server (auto refresh does not work anymore).
The client should see some data the moment it launches and then get refreshed every 1 minute when new data is sent by the server through SSE.  This is the base case and I do have some small complexities which I can resolve once this base setup is up and running.
Please help me if this is something can be achieved using your package/
Regards
SD

Should I create first eventsource with lastEventID = 0 (to get the initial dump) and on receipt of the message close it.  And then again establish a second eventsource without the lastEventID to get the usual periodic refresh?

I'm kind of swamped right now, so I don't have time to dive into this fully, but if you're using IDs, they should be numeric.  Either use a counter for the IDs, or use Date.now() instead of (new Date()).toISOString()

This doesn't seem like a bug, but an implementation problem. If you have a more specific bug to report, please comment. Closing.


data: post.slug breaks meta layer on index collection page

Steps to reproduce:
collections:
  /blog/:
     permalink: ..
     data: post.known

Serve /blog/. See error in stdout
MESSAGE: Cannot read property 'name' of null

TypeError: Cannot read property 'name' of null
    at getPostSchema (/ghost/ghost-workspace2/core/server/data/meta/schema.js:82:61)

@kirrg001 wasn't able to reproduce this one. Used following configuration for collections:
collections:
  /blog/:
     permalink: /blog/{slug}/
     data: post.unknown

Both http://ghost.local/blog/ and http://ghost.local/blog/cant-find-me returned regular 404 with no additional errors in the output. Maybe there are some additional steps I'm missing?

Did you try a known post slug?

Tried following. Published post renders normally: http://ghost.local/just-a-page/ , and when going to http://ghost.local/blog/just-a-page/ it renders a 404 like above.

collections:
  /blog/:
     permalink: ..
     data: post.known

Serve /blog/ reproduces the problem.

What do you have in permalink where .. is?

Updated

Are there any other configurations in the routes.yml you are using? Can't get this error happening on my machine. Created a post with known tag and collections config like above and it works when opening  http://ghost.local/blog/known/ and 404 is rendered when visiting http://ghost.local/blog/.
Is the theme you are testing on v0.1 by any chance?


translate form generic data into japanese

WIP:

[x] home
[ ] getStarted
[ ] api
[ ] advanced
[x] builder
[ ] faq
[x] nav
[x] generic

thank you @kotarella1110 for your effort today 👍


Write manual test xases for "MichiganPage".

Please write a manual test case for the following pages manual test which involves breaking down the functionality of the webpage into specific steps that can be tested.
MichiganPage: https://prayersconnect.com/mosques/search/Queens%252C%2520New%2520York%252C%2520United%2520States/40.7282239%2C-73.7948516/40.7282239%2C-73.7948516
Please use this Google doc for your test cases.
https://docs.google.com/spreadsheets/d/1O1NJRmobfNfk6cGgc5BWHCMhRSjF6vJK3C-AljYxgHo/edit?usp=sharing

https://docs.google.com/spreadsheets/d/1O1NJRmobfNfk6cGgc5BWHCMhRSjF6vJK3C-AljYxgHo/edit#gid=125654671

Thank you.


feat(pixels): add my new pixel (12, 7)

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

Love learning this.


Specify local file with `vcpkg create'

Hi,
Is it possible to provide a local file to the vcpkg create command and then possibly change the target URL in the future when I upload the project to a global URL?
Thanks

We don't currently support local files as arguments to vcpkg create, however you could edit the resulting portfile to directly unpack a local archive:
vcpkg_extract_source_archive(C:/local/path/to/project.zip)

Note that the extract source archive command does invalidation based on the name of the file, so if you're updating the file in place you probably want to also purge the sources on every run.
# Do this before vcpkg_extract_source_archive
file(REMOVE_RECURSE ${CURRENT_BUILDTREES_DIR}/src)

When you upload your project to a global URL, you can simply add a vcpkg_download_distfile or vcpkg_from_github directive above the extract step.

This issue hasn’t been updated in a year; if it is still an issue, please reopen.

Hi, Is there any update on this? its been more  than 3 years now.
Having this capability would be greatly appreciated

Hi, Is there any update on this? its been more  than 3 years now.
Having this capability would be greatly appreciated


Fix disabling of incremental scala compilation

When disabling incremental compilation by setting force to true, the
analysis file was still created and loaded, causing Zinc to believe it
was running incrementally.
However the compilation output was effectively deleted. This resulted in
only producing output for changed types.
With the changes in this commit, the analysis information is ignored,
discarded if existing, and not preserved when incremental compilation is
disabled.
~This PR lacks an integration test - adding it in parallel of pushing the code changes through CI. The fix has been confirmed on reproducers.~
Tests added

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


Fix env variables for frontend

Noticed errors on the https://ui.dmphub.uc3dev.cdlib.net/login page. The requests were going to the wrong url
The variables in ecs-frontend config were erroneously defined to include = signs, and it's throwing off the url, so that it is coming through as https://ui.dmphub.uc3dev.cdlib.net/=%22https://ui.dmphub.uc3dev.cdlib.net%22/apollo-csrf in the request. Will remove the = sign and the double quotes.
Currently, the ecs-frontend template includes several environment variables:
            - Name: 'NEXT_PUBLIC_BASE_URL'
              Value: !Ref NextPublicBaseUrl
            - Name: 'NEXT_PUBLIC_SERVER_ENDPOINT'
              Value: !Ref NextPublicServerEndpoint
            - Name: 'NEXT_PUBLIC_GRAPHQL_SERVER_ENDPOINT'
              Value: !Ref NextPublicGraphqlServerEndpoint

Currently, the env variables in my local .env.local file, are defined as:
NEXT_PUBLIC_BASE_URL="http://localhost:3000"
NEXT_PUBLIC_SERVER_ENDPOINT="http://localhost:4000"
NEXT_PUBLIC_GRAPHQL_SERVER_ENDPOINT="http://localhost:4000/graphql"

Migrated changes up using sceptre commands after PR approval


Update fieldset component caller()

Controlling whitespace when fieldset is consumed in other components has become very tricky.
Removing trim and and indent will allow exact positioning to be handled where fieldset is called from.
This will fix whitespace in https://github.com/alphagov/govuk-frontend/pull/388
but break it in date-input, which is getting fixed here

Would be good to understand this better, can we go through it on Monday? I thought indentation would be fine as long as the block passed by the caller isn't indented…


Config structure unable to accept maps

Summary:
The config section under the Stack CRD needs to be able to accept a map. E.g.:
apiVersion: pulumi.com/v1alpha1
kind: Stack
metadata:
  name: stack-name
spec:
  config:
    aws:region: us-east-2
    aws:assumeRole:
      roleArn: arn:aws:iam::xxxxxxxxx:instance-profile/nodes.xw-cluster-1.k8s.local

In the scenario where we would like to specify an AssumeRole using a roleArn, we would run pulumi config set --path aws:assumeRole.roleArn arn:aws:iam::xxxxxxxxx:role/nodes.xw-cluster-1.k8s.local and it would automatically generate the config in Pulumi..yaml. This auto-generated config does not work when it  is placed under a Stack yaml file.
error validating data: ValidationError(Stack.spec.config.aws:assumeRole): invalid type for com.pulumi.v1alpha1.Stack.spec.config: got "map", expected "string"

Steps to reproduce

git clone https://github.com/pulumi/pulumi-kubernetes-operator.git
cd pulumi-kubernetes-operator
git fetch --all --tags
git checkout tags/v0.0.17 -b v0.0.17
kubectl apply -f deploy/crds/
kubectl apply -f deploy/yaml
create a Stack yaml (named stack.yaml) with the config shown in the above
create github repo with pulumi project
point Stack yaml to github repo
kubectl apply -f stack.yaml
Expected:
Be able to configure an AssumeRole for pulumi POD to utilize the IAM role on the host
Actual:
Failing because config only accepts strings and not maps

Thanks for opening this @xwang0818 - are you able to use 0.0.18 (released yesterday) to use the checked-in Pulumi stack configuration to work around this?

@viveklak I upgraded my pulumi operator to the latest release v0.0.19. However, this version still does not support the stack config described above.

@viveklak I upgraded my pulumi operator to the latest release v0.0.19. However, this version still does not support the stack config described above.

I haven't had a chance to investigate the issue around the richer stack config. My specific suggestion was to check-in the stack config file (Pulumi.<stack>.yaml) to source control with the desired assume role config and have it be picked up instead of specifying it in the stack CR.  v0.0.18 and up include the fix in https://github.com/pulumi/pulumi-kubernetes-operator/pull/187 which should let you load stack yaml files. I was curious if that was a feasible workaround for you for the moment?

Let's close this as a duplicate of: https://github.com/pulumi/pulumi-kubernetes-operator/issues/258


docker-compose ?

Do you have a working sample for a docker-compose.yml ?
I tried this and it just errored out with a pastebin command line help output.
version: '3.2'

services:
  pastebin:
    image: mkaczanowski/pastebin:latest
    container_name: pastebin
    ports:
      - 7780:8000

└─> docker-compose up
Pulling pastebin (mkaczanowski/pastebin:latest)...
latest: Pulling from mkaczanowski/pastebin
69692152171a: Pull complete
66e198a52f6b: Pull complete
Digest: sha256:17c72bcb06c8a29c209a2ca068083c2c374883ecddca5079dbdfd27b893146ce
Status: Downloaded newer image for mkaczanowski/pastebin:latest
Creating pastebin ... done
Attaching to pastebin
pastebin    | pastebin 0.1.3
pastebin    | Simple, standalone and fast pastebin service.
pastebin    |
pastebin    | USAGE:
pastebin    |     pastebin [FLAGS] [OPTIONS]
pastebin    |
pastebin    | FLAGS:
pastebin    |     -h, --help               Prints help information
pastebin    |         --ui-line-numbers    Display line numbers
pastebin    |     -V, --version            Prints version information
pastebin    |
pastebin    | OPTIONS:
pastebin    |         --address <address>                       IP address or host to listen on [default: localhost]
pastebin    |         --db <db-path>                            Database file path [default: ./pastebin.db]
pastebin    |         --environment <environment>               Rocket server environment [default: production]
pastebin    |         --keep-alive <keep-alive>                 Keep-alive timeout in seconds [default: 5]
pastebin    |         --log <log>                               Max log level [default: normal]
pastebin    |         --plugins <plugins>...
pastebin    |             Enable additional functionalities (ie. prism, mermaid) [default: prism]
pastebin    |
pastebin    |         --port <port>                             Port number to listen on [default: 8000]
pastebin    |         --slug-charset <slug-charset>
pastebin    |             Character set (expressed as rust compatible regex) to use for generating the URL slug [default: [A-Za-z0-
pastebin    |             9_-]]
pastebin    |         --slug-len <slug-len>                     Length of URL slug [default: 21]
pastebin    |         --tls-certs <tls-certs>                   Path to certificate chain in PEM format
pastebin    |         --tls-key <tls-key>                       Path to private key for tls-certs in PEM format
pastebin    |         --ttl <ttl>                               Time to live for entries, by default kept forever [default: 0]
pastebin    |         --ui-expiry-times <ui-expiry-times>...
pastebin    |             List of paste expiry times redered in the UI dropdown selector [default: 5 minutes, 10 minutes, 1 hour, 1
pastebin    |             day, 1 week, 1 month, 1 year, Never]
pastebin    |         --uri <uri>                               Override default URI
pastebin    |         --uri-prefix <uri-prefix>                 Prefix appended to the URI (ie. '/pastebin') [default: ]
pastebin    |         --workers <workers>                       Number of concurrent thread workers [default: 0]
pastebin exited with code 0

please see: https://github.com/mkaczanowski/pastebin/pull/26


How to stop suggestions from hiding Svelte component props?

Hi,
I am not sure if this is a bug, but intellisense suggestions for properties on a svelte component are completely hidden by windi class names, which shouldn't even be an option - unless its attributify suggestions.
If this is intended, is there a way to disable it?
Thanks,
D

can you please add screenshots or a quick screencast?

Sure, see below - cursor is in the space where the arrow is pointing

Just pushed a fix. please try after it gets update in the marketplace and let me know

Awesome, thanks for the quick response this was driving me insane.

Thanks, the classes are gone, but the attributify options still show up above the props (it is disabled in my windi.config).  Thinking about it, they probably should not be showing in component prop options either as they are not props of the component and would have no effect.

for attributify there is a extension setting
https://github.com/windicss/windicss-intellisense/blob/main/package.json#L170
https://github.com/windicss/windicss-intellisense/blob/main/package.json#L175

Thanks, that works for components, but now also disables it on native elements (not that I am using attributify so I'm OK with this) - Others might not be, so perhaps worth re-opening.


Zephyr does not boot with CONFIG_PM=y

I am running Zephyr v2.7.2 on the nucleo_l552ze_q board.
Not long after enabling CONFIG_PM, I produced a build that simply does not boot.  Where there is normally a banner announcing the Zephyr version, there is nothing, and the system has clearly halted (my normally blinking LED is dark).
Oddly, arbitrary changes to the code/configuration make the problem disappear and reappear.  For example, putting some code in a loop; adding logging; returning early from main; enabling CONFIG_ASSERT or CONFIG_DEBUG -- any of these might cause the program to boot again, or to stop booting.
Importantly, it is consistent:  With a particular code and configuration, it always either boots or does not.  Better than an intermittent bug, but still very troubling 🙂
I normally run MCUBoot but I can reproduce the problem without it.
I did not see the problem before introducing power management, so I strongly suspect it is related.
I have produced a minimal configuration that reproduces the problem.  Remove/modify any part of it and the problem goes away:
CMakeLists.txt:
cmake_minimum_required(VERSION 3.20.0)

set(BOARD nucleo_l552ze_q)
set(DTC_OVERLAY_FILE board.overlay)

find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
project(ulmo)

target_sources(app PRIVATE
	src/main.c
)

board.overlay:
&cpu0 {
	cpu-power-states = <&stop0 &stop1 &stop2>;
};

&rcc {
	apb1-prescaler = <2>;
	apb2-prescaler = <2>;
};

&lptim1 {
	status = "okay";
};

&lpuart1 {
	pinctrl-0 = <&lpuart1_tx_pa2 &lpuart1_rx_pa3 &lpuart1_rts_pb1 &lpuart1_cts_pb13>;
	current-speed = <115200>;
	status = "okay";

	gsm: gsm-modem {
		compatible = "zephyr,gsm-ppp";
		label = "gsm_ppp";
	};
};

&usart2 {
	pinctrl-0 = <&usart2_tx_pd5 &usart2_rx_pd6>;
	current-speed = <115200>;
	status = "okay";
};

&usart3 {
	pinctrl-0 = <&usart3_tx_pd8 &usart3_rx_pd9 &usart3_rts_pd12 &usart3_cts_pd11>;
	current-speed = <19200>;
	status = "okay";
};

&uart4 {
	pinctrl-0 = <&uart4_tx_pc10 &uart4_rx_pc11>;
	current-speed = <9600>;
	status = "okay";
};

/ {
	power-states {
		stop0: state0 {
			compatible = "zephyr,power-state";
			power-state-name = "suspend-to-idle";
			substate-id = <1>;
			min-residency-us = <100>;
		};
		stop1: state1 {
			compatible = "zephyr,power-state";
			power-state-name = "suspend-to-idle";
			substate-id = <2>;
			min-residency-us = <500>;
		};
		stop2: state2 {
			compatible = "zephyr,power-state";
			power-state-name = "suspend-to-idle";
			substate-id = <3>;
			min-residency-us = <900>;
		};
	};

	chosen {
		zephyr,console = &usart2;
		zephyr,shell-uart = &usart2;
		zephyr,gsm-ppp = &lpuart1;
	};
};

prj.conf:
CONFIG_PM=y
CONFIG_PM_DEVICE=y

CONFIG_MODEM=y
CONFIG_MODEM_GSM_PPP=y
CONFIG_GSM_PPP_AUTOSTART=n

CONFIG_NETWORKING=y
CONFIG_NET_DRIVERS=y
CONFIG_NET_PPP=y
CONFIG_NET_L2_PPP=y

CONFIG_LOG=y

src/main.c:
void main() {}

Update:  The behavior is not as consistent as I previously thought (perhaps I didn't do many repetitions before).  I now see the behavior changing after reset, for a particular firmware image.  Sometimes it boots, sometimes not.
I should say that the impact is considerably higher now, if we can't be sure after a single test whether the program will boot or not.

@erwango I'm able to reproduce it on v2.7.2 and v2.7.1, but not on v2.7.0.  It's hard to be totally sure about v2.7.0, as the same program doesn't always show the same behavior between versions (e.g. my minimal reproducer did not have the issue (but I didn't have any trouble finding a variant that did)).  But I did try several variants and they all booted on v2.7.0.
I'll revert to v2.7.0 for now and I'll keep you posted if the problem shows up again.
Let me know if there are any specific revisions you'd like me to test.

I spent a couple tedious hours trying to narrow down which revision could have caused this bug.  I would take these results with a small grain of salt because it has proven difficult to find code/configuration which provokes the bug.  That is, even though I wasn't able to provoke the bug in a particular revision, it may still be that it exists there.
That said, my testing indicates that the bug might have been introduced in c65645cc39e4a3513ac754733266766459b3cef0.  Does this seem with the realm of possibility, @erwango ?
Here are all of the revisions I tested (YES means I saw the bug, NO means no):
v2.7.1 - YES
5c08f183f2bcd428df3c42f76c5b130f771cf101 - YES
525c112ac3195661583f618b0e63275b64faf274 - YES
edd1612388faf0524ac0c41aac9826ec32fbd760 - NO
7e2e9d612db6045bab0a47b5d4cd225e2febdc41 - YES
7bdf17756c815e7d4d24cd2b39b5bcb9d9c854c9 - YES
053dc7572062845255aeb3551a1e904b8bf9fc09 - YES
b7954c9991d753177ad5a4ae984d2ae1a5d37a12 - YES
f5d4fb40b510d86996e2f7bf2acd682f059120e1 - YES
6a851f1f485a59b7bc50a82e8d2d1d9d6e9435ed - YES
68a6a3e5c3d44be1309dd2297326533b8b93c8fd - YES
dff331a40c507ce111501a45164542445ec283d7 - YES
c65645cc39e4a3513ac754733266766459b3cef0 - YES
cef9cbeb608117b8babfc4aea026d9c1805ab382 - NO
v2.7.0 - NO
It would be prudent to check some even earlier revisions (pre-2.7.0), but I don't have any more time for it today.

@erwango Can you think of a better way to diagnose the problem than mere trial?

It would be prudent to check some even earlier revisions (pre-2.7.0), but I don't have any more time for it today.

Thanks for this work. Given that the SHA1 you pointed doesn't have impact on binary and it is so close to V2.7.0, I'm pretty sure this is not a regression but rather a bug that could be silent under specific conditions.
i'll try to reproduce on my end and keep you posted.

@gordonklaus Unfortunately, I'm not able to reproduce on my end.
The only difference with your set up is that I kept console on lpuart1. can you check if this is a real reproduction condition ?

@erwango I'm also not able to reproduce it when I move the console back to lpuart1 (I tried many configurations, but I suppose it's still possible I didn't search long enough).  Do you think this could somehow be related?
I was able to reproduce it on another identical Nucleo board, FWIW.

@erwango I'm not sure how to explain that c65645cc39e4a3513ac754733266766459b3cef0 should have no effect on the binary, yet I saw different behavior.
I diffed the binaries before and after that change and they only differ in the boot banner (which include the git revision) and in two 16-bit words that are swapped at lower address:
4902c4902
< 00013250: ff7f 0600 0100 ff7f ff7f ff7f ff7f ff7f  ................
---
> 00013250: ff7f 0100 0600 ff7f ff7f ff7f ff7f ff7f  ................
5318,5319c5318,5319
< 00014c50: 322e 372e 302d 352d 6763 3635 3634 3563  2.7.0-5-gc65645c
< 00014c60: 6333 3965 3400 2a2a 2a20 426f 6f74 696e  c39e4.*** Bootin
---
> 00014c50: 322e 372e 302d 342d 6763 6566 3963 6265  2.7.0-4-gcef9cbe
> 00014c60: 6236 3038 3100 2a2a 2a20 426f 6f74 696e  b6081.*** Bootin

But neither of these binaries reproduces the problem (and I can't find a configuration that does again at that revision) :(


Crash even  there is an observable Finally

Hello,
I am learning Rx following http://introtorx.com/Content/v1.0.10621.0/11_AdvancedErrorHandling.html
When I try this code below from the tutorial, I found the bug is not fixed. Rx.net version is 3.1.1
var source = new Subject();
var result = source.Finally(() => Console.WriteLine("Finally"));
result.Subscribe(
Console.WriteLine,
//Console.WriteLine,
() => Console.WriteLine("Completed"));
source.OnNext(1);
source.OnNext(2);
source.OnNext(3);
//Brings the app down. Finally action is not called.
source.OnError(new Exception("Fail"));

That's because you are not observing the OnError-notification (you commented it out). The overload of Subscribe you use will just rethrow the exception, thus bringing the app down. Either do something with the exception (log it or whatever) or use Catch on source. Note that Finally will not supress the exception (like in a C#-finally-Block).


Running single tests that are in an ignored category is not possible anymore

This is a regression. I have this problem since I've updated to R# 9.2 and resharper-xunit 2.2.8.
When running tests explicitly (e.g. by using the action icon next to a test method), the test should be run regardless of any ignore settings.  Since the update, they are ignored, however.

Can you remember when it last worked? I've just tried 9.2 + 9.1, and both will treat this test as ignored.

I think I was running R# 9.1.2 when it last worked, but I don't remember the version of the xunit extension. If I remember incorrectly, then it was R# 9.0 and not lower, that's for sure.

Thanks. I'm confused by this one - I don't know how it's ever worked :)
@bradwilson is there anything in the API that would allow a runner to force a skipped test to run?

I see :-)
Here is some more background information: At the beginning of the project, I tried to get the "run only when explicitly started" behavior by just adding Skip to the [Fact]. That didn't work, the test was never run, even when started directly. I then found that I could ignore whole categories (e.g. [Trait("Category", "NoAutoRun")] ) by specifying the category name in the Resharper options under Tools > Unit Testing > Skip tests from categories. With this approach, the test was run when started directly, but skipped when running a bunch of them, e.g. with Run test in solution.
So that was the path we took, but it stopped working with the update to R# 9.2.

And by the way, we've been using xunit.net 1.9.2 since quite some time and haven't changed anything on that end.

Aha! I can recreate that. And sadly, it's a regression in ReSharper, and affects nunit, too. Here's the ticket: RSRP-447562 for you to track and vote. I'm going to close this issue, as there's not much I can do here.


Include TestAsset info in the IR logs

I added the main bits (Input Curie, Output Curie, Expected Output) as tags to each test, plus you have the Test Case and Test Asset numbers to go and find the less important bits in the Tests repo.


NodeId should not be part of the configuration file.

The NodeId should be a command line argument like port number.
With the current scheme one is forced to have one node configuration file per node in a topology file.

@karknu, I think the node Id should be inferred from the PBFT leader credentials, not specified at all.

PBFT indeed doesn't need it anymore (https://github.com/input-output-hk/ouroboros-network/pull/1192).

If we need anything like this at all, it's only to allow sharing a network configuration description between multiple nodes. It has no connection whatsoever to any notion of node id from consensus (since there is no such notion).

The topology part of de-NodeId-ification is being handled in https://github.com/input-output-hk/cardano-node/pull/318


Add a molecule testing for aplly cp

SUMMARY

Note: maybe this could be added to the default scenario to avoid extending again the molecule test run duration...
ISSUE TYPE

Feature Idea

It's covered by the JWS specific molecule scenario that I'm adding, so I'll close this one out.


Updates executablePath return type

This updates the executablePath return type when the /tmp directory already exists. Instead of returning a string, it will return a promise (to behave more consistently).

Sorry for sitting on this for so long @dehli, it crossed my mind!
Your last commit rejecting with a string instead of returning null would make the headful use case buggy (if you pass null, puppeteer should fallback to the default bundled Chromium path).
Regardless, thank you for bringing this up - I'll be pushing your initial proposal soon. :+1:


failed to invoke procedure embedding.deepgl

using version 1.0.2 and running this cypher code:
CALL embedding.deepgl("Address","PAYS", {
nodeFeatures: ['pagerank'],
iterations: 2
})
This error occurs >>
Neo.ClientError.Procedure.ProcedureCallFailed: Failed to invoke procedure embedding.deepgl: Caused by: java.lang.NoSuchMethodError: org.neo4j.graphalgo.core.IdMap.buildMappedIds()V
Not sure what it is!? Config file is updated for the unrestricted algo.,  embedding.
I tried the latest verions 1.0.3 and get a completely different error:
Neo.ClientError.Procedure.ProcedureCallFailed: Failed to invoke procedure embedding.deepgl: Caused by: java.lang.UnsatisfiedLinkError: no openblas in java.library.path
I am using neo4j version 3.5.5
happy to hear anyones ideas on what i might be doing wrong here
Many thanks!

Hi @AdamT23 & @mneedham, any update?
I have the exact problem when invoking embedding.deepgl
Neo.ClientError.Procedure.ProcedureCallFailed: Failed to invoke procedure embedding.deepgl: Caused by: java.lang.NoSuchMethodError: org.neo4j.graphalgo.core.IdMap.buildMappedIds()V
Instead embedding.deepwalk is working fine.
Thanks!

Im using neo4j 3.5.8 comunity , Docker and ml-models 1.0.3 - pre-built
Im getting errors using  embedding.* - where as CALL algo.pageRank('Host', 'CONNECTION') works.
It seems like a dependency issue
Do we have to compile ml-models ourselves -- @DagobertoJoseHerreraMurillo  since deepwalk is working for you. Can u describe your setup ?
CALL embedding.deepgl("Host","CONNECTION", {
nodeFeatures: ['pagerank'],
iterations: 2
})
Error
Neo.ClientError.Procedure.ProcedureCallFailed
Neo.ClientError.Procedure.ProcedureCallFailed: Failed to invoke procedure embedding.deepgl: Caused by: java.lang.NoSuchMethodError: org.neo4j.graphalgo.core.heavyweight.HeavyGraph.degree(ILorg/neo4j/graphdb/Direction;)
CALL embedding.deepWalk()
Neo.ClientError.Procedure.ProcedureCallFailed: Failed to invoke procedure embedding.deepWalk: Caused by: java.lang.NoSuchMethodError: org.neo4j.graphalgo.api.Graph.forEachNode(Ljava/util/function/IntPredicate;)V

Same issue with the Neo4j v. 3.5.13.


Link to Docs in help section goes to the GitHub page instead of the docs

The documentation button leads to : https://github.com/excalidraw/excalidraw#documentation
It could be : https://docs-excalidraw.vercel.app.  ?

closed via #6654


more details on setting up catalog

I just went through the quickstart - and ended up being confused when it came to the catalogs.  The details are not sufficient in the quickstart.  I ended up following https://learn.microsoft.com/en-us/azure/deployment-environments/how-to-configure-catalog and https://learn.microsoft.com/en-us/azure/deployment-environments/configure-catalog-item and figured it out
I would recommend some steps be added to either

add some folders/files manually with the yaml files OR
2)clone the sample repository (https://github.com/Azure/deployment-environments) and
then a link to create the PAT..
Then specify the values that needed to be added

side note - right now when adding a catalog it wants a PAT even for a public repo (if you don't need a PAT it would be easy to simply point to a public repo)
finally when done it would be great to show where this appears in the UI so you can find the resource group and test it

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 6ba8591f-2bd6-d95e-4618-d70c0138c657
Version Independent ID: 567f158a-2470-70d2-d46f-8bde5649e75c
Content: Configure the Azure Deployment Environments service
Content Source: articles/deployment-environments/quickstart-create-and-configure-devcenter.md
Service: deployment-environments
GitHub Login: @anandmeg
Microsoft Alias: meghaanand

@ahmedsza
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@anandmeg
Could you Please review this and update as appropriate.

#assign: @RoseHJM

@ahmedsza Thank you for taking the time to share your ideas and feedback. Your request will take some time and additional resources to complete. We've added this to our internal backlog, which we'll attend to as soon as we can.

@ahmedsza Thank you again for sharing your ideas and feedback. We're happy to report that we've updated Quickstart: Create and configure a dev center. We'll close this issue now, but we will continue to review and refine this documentation as new features are added and workflows are simplified.

#please-close


Cannot declare unique symbol type in jsdoc in js file

Bug Report
🔎 Search Terms
jsdoc, unique symbol
🕗 Version & Regression Information
Any version.

This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________

⏯ Playground Link
Playground link with relevant code
💻 Code
/** @type {unique symbol} */
const a = Symbol();

🙁 Actual behavior
a is typed unique symbol in ts file but symbol in js file.
🙂 Expected behavior
a is typed unique symbol in both ts file and js file.

Hi,
Can i have a try on this issue?

I don't know if it's relevant to post it here, but I still have this issue (I opened an issue in vscode repo, may be it is more appropriate here ?)
https://github.com/microsoft/vscode/issues/200489

The difference is only in quickinfo display. I filed a new bug: #56737


Bulk updates affect only a cell

Nice tool, Jeff!
However I've and issue on the bulk updating. That is, editing more than one cell at once (i.e. just drag and dropping or batch deleting). For some reason, the update only appears only to affect to the first row clicked, and not the others.
Some code reproducing the issue are here:
server.R http://pastebin.com/YfjmR35w
ui.R http://pastebin.com/wcQAUH2q
Here the snapshot of a two-cell edit that results in only one update. The first table is an editable htable whereas the second just renders the same table. It is worth noting they differ. A sessionInfo() is added. The code skeleton is the 02-matrix example kindly provided along with the installation.

Thanks,
John

Did you find a solution? I have the same issue


BUG : LOGIN PAGE

Describe the bug

the 2nd "LOGIN"  button in the home page

is totally messed up
Expected behavior

I want if any contributor can fix this bug & make the login page as it was previously or either you can make this login page from scratch . Give me your ideas in this issue's comments.
Screenshots (Mandatory)

Additional Context (Mandatory )
It is an open issue . Whoever will comment first with the best idea , I will assign it to HIM/HER
Are you contributing under any open-source program ?

GSSOC'24

Please assign this issue to me under gssoc, i will fix the bug and will modify the login page accordingly.

Please assign me with this issue I will fix the bug

I  can fix this bug....
the idea is firstly to remove the second option of login and secondly making the to look good as the footer and navbar both are creating the mess on this page by adjusting navbar and footer also fixing the  login section can make this page to look beautiful also I will try to add animations to this page . I request @apu52 to assign this issue to me.
thank you!!

@DiwareNamrata23 I think you have got the best idea. so I have assigned you this issue. try to complete this issue within 2-3 days


Add passes to lower AIE Device code to LLVM.

This does not get us all the way to binary, but the sample lowers to LLVM successfully.
Next step is to generate the actual binaries.

Qq: can this be fed to peano only or chess as well?

I hope both. I havent gotten there yet. Thats the next thing to do.

Qq: can this be fed to peano only or chess as well?

We can feed to both,  Mainly we target Peano, and insert workarounds for Chess.  The Chess workarounds are:

IR textual format workarounds, due to different LLVM versions
Intrinsic differences, so we link with a small library to redirect intrinsics to what Chess expects.


Samsung devices on Android 9 do not show a proper date/time keyboard - most noticeable with the missing / character needed for proper date input.

https://github.com/flutter/flutter/blob/c5f8edd9eeabafc0458c98e51433ba8d44d5c997/packages/flutter/lib/src/material/input_date_picker_form_field.dart#L246
See https://github.com/material-components/material-components-android/issues/682 for similar issue and fix in android components.

Hi @GenuineRex
Is it specific to Android 9? Have you experienced it on others Android versions on Samsung devices?
this comment says there are versions who have this issue
What Samsung device are you using for testing on Android?
Can you please provide your flutter doctor -v your flutter run --verbose and a complete reproducible minimal code sample
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

This is still the case.  My Galaxy Tab A shows a numeric only keyboard when I select datetime as the keyboard type.  So no ':' to separate hours from minutes.
Flutter stable channel, updated to the latest release.

@mbalmer
This issue is closed, please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.


Npm audit needs to be run

I'm getting a bunch of things that need to be fixed:
67 vulnerabilities (10 low, 26 moderate, 29 high, 2 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

Seems like we could have a bot that run npm audit fix all the time.

Fixed


fix(android): add missing function declaration setAutoLogAppEventsEnabled and setAdvertiserIDCollectionEnabled

As mentioned in #155, these two functions were missing for android
closes #155

Thank you for your contribution!


Typo fix

    <xs:documentation><!-- _locID_text="VersionSufix" _locComment="" -->Suffix that can indicate additional information at the end of the verion (i.e. beta)</xs:documentation>

Typo here as well

Refers to: src/MSBuild/Microsoft.Build.CommonTypes.xsd:1804 in c2aac57. [](commit_id = c2aac578afb659b6e83a2ec5d632b506e67e6efa, deletion_comment = False)

:shipit:

@RaulPerez1 we changed the description for VersionSuffix and VersionPrefix. Still look good to you?

Looks good to me.

 What about now? :)

Thumbs up!


Fix db password

Sync expects database username and password to be 'odk'.
Before this change, I could not log in to the sync endpoint server.

Closed as I realised db-bootstrap adds the user to the database


Get instance to test ChatBot

Description:
Sanjeewa wanted me to sort out an instance to test the ChatBot that will be used with our WSO2 documentation.
Related Issues:
https://github.com/wso2/docs-apim/issues/4279

Had a meeting with Lakindu and Sanjeewa to get the requirements of the instance.

Provided the size and other details of the OpenStack instance to Maheshika via email so that she can create an instance and provide access for Aarthif Nawaz to access it.

Checked with Aarthif Nawaz to make sure that he can access the created instance.

Reopening this task as the OpenStack instance got terminated yesterday and another needs to be created.

Sent a request on the following email address making a request in this regard.
Email: OpenStack instance needed to test documentation related Chat Bot

Maheshika created an instance with me as the owner for this purpose.


Adds name property to Tab interface.

Fixes:  #1041.
Changes proposed in this pull request:

adds name?: string to Tab itnerface

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thx for your PR @MattAndDev


Draft next Newsletter

promote website redesign
NDoCH
mention Civic Camp items to come
First Responders Program

Depending on timing we could include call for speakers.
On Mon, Jul 23, 2018 at 12:30 PM, Janel<EMAIL_ADDRESS>wrote:

promote website redesign
NDoCH
mention Civic Camp items to come
First Responders Program

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086, or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6Ixv3Tf1UWjSDScxDYyMx3q2mqs0hmks5uJfokgaJpZM4VbTw2
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose

I was aiming for Friday (7/27) but could wait till Monday or Tuesday next
week.
On Mon, Jul 23, 2018 at 12:37 PM, zachambrose<EMAIL_ADDRESS>wrote:

Depending on timing we could include call for speakers.
On Mon, Jul 23, 2018 at 12:30 PM, Janel<EMAIL_ADDRESS>wrote:

promote website redesign
NDoCH
mention Civic Camp items to come
First Responders Program

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/
AQ6Ixv3Tf1UWjSDScxDYyMx3q2mqs0hmks5uJfokgaJpZM4VbTw2>
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086#issuecomment-407121542,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK4rpkCSA81tgZ79d84WwSWQrJlckD51ks5uJfvWgaJpZM4VbTw2
.

When are you thinking for the one after that?
On Mon, Jul 23, 2018 at 1:06 PM, Janel<EMAIL_ADDRESS>wrote:

I was aiming for Friday (7/27) but could wait till Monday or Tuesday next
week.
On Mon, Jul 23, 2018 at 12:37 PM, zachambrose<EMAIL_ADDRESS>wrote:

Depending on timing we could include call for speakers.
On Mon, Jul 23, 2018 at 12:30 PM, Janel<EMAIL_ADDRESS>wrote:

promote website redesign
NDoCH
mention Civic Camp items to come
First Responders Program

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/
AQ6Ixv3Tf1UWjSDScxDYyMx3q2mqs0hmks5uJfokgaJpZM4VbTw2>
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ncopenpass/NCOpenPass/issues/
1086#issuecomment-407121542>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AK4rpkCSA81tgZ79d84WwSWQrJlckD51ks5uJfvWgaJpZM4VbTw2>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086#issuecomment-407130789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6IxnFxcAzBvp27aSISFEYDt6vCQ3EVks5uJgKXgaJpZM4VbTw2
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose

Timing is flexible - if we need to push on attendees for NDoCH before
August 11.
We probably should be a little more consistent on the timing of the
newsletters as we move into the "busy" time for the next couple of months.
Every two weeks would probably be a good timing come August - thoughts?
On Mon, Jul 23, 2018 at 1:07 PM, zachambrose<EMAIL_ADDRESS>wrote:

When are you thinking for the one after that?
On Mon, Jul 23, 2018 at 1:06 PM, Janel<EMAIL_ADDRESS>wrote:

I was aiming for Friday (7/27) but could wait till Monday or Tuesday next
week.
On Mon, Jul 23, 2018 at 12:37 PM, zachambrose<EMAIL_ADDRESS>wrote:

Depending on timing we could include call for speakers.
On Mon, Jul 23, 2018 at 12:30 PM, Janel<EMAIL_ADDRESS>wrote:

promote website redesign
NDoCH
mention Civic Camp items to come
First Responders Program

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/
AQ6Ixv3Tf1UWjSDScxDYyMx3q2mqs0hmks5uJfokgaJpZM4VbTw2>
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ncopenpass/NCOpenPass/issues/
1086#issuecomment-407121542>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AK4rpkCSA81tgZ79d84WwSWQrJlckD51ks5uJfvWgaJpZM4VbTw2>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/ncopenpass/NCOpenPass/issues/
1086#issuecomment-407130789>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AQ6IxnFxcAzBvp27aSISFEYDt6vCQ3EVks5uJgKXgaJpZM4VbTw2>
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1086#issuecomment-407131187,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK4rpmV9r6QJQCT-LkYIGFzMeAy3ny8iks5uJgLmgaJpZM4VbTw2
.

I'd rather hold off and have a newsletter more focused on call for speakers


Make Protoc plugin JAR self-executable

Currently, the Spine Protoc plugin consists of 3 artifacts:

the main artifact-a JAR with the plugin;
a .sh launcher;
a .bat launcher.

It works this way because it is impossible to make Protobuf compiler (aka Protoc) run a Java archive with java -jar <...>.jar. Instead, it runs the launchers, which run the JAR itself.
What we should do instead is one of the following:

Make the JAR executable by prepending the launcher script to its binary representation. This way is robust, but looks not quite obvious.
Make the launchers download the plugin on their own instead of relying on Gradle to do so. This way is more comprehensible than the 1st one, but may impact the build performance.

In one of the latest updates, Gradle Protbuf plugin included the feature to launch JARs instead of just executables. We should use it.


Support for "search/issues" API from GitHub

Description
Is search/issues API from GitHub supported? If not, and plans/ETA to implement it?

Background/Details
Trying to use @semantic-release/github in my Gitea repo. It works if I disable successComment and  failComment as these steps seems to perform a search that is not compatible with Gitea.
My test .releaserc looks something like this:
{
  "branches": ["main"],
  "repositoryUrl": "https://xxx/my-org/test.git",
  "plugins": [
    "@semantic-release/commit-analyzer",
    "@semantic-release/release-notes-generator",
    "@semantic-release/changelog",
    [
      "@semantic-release/git",
      {
        "assets": ["CHANGELOG.md", "package.json", "package-lock.json"],
        "message": "chore(release): ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}"
      }
    ],
    [
      "@semantic-release/github",
      {
        "githubUrl": "https://xxx",
        "githubApiPathPrefix": "/api/v1",
        "successComment": false,
        "failTitle": false,
        "failComment": false
      }
    ]
  ]
}

If I do not set successComment and failComment to false, when Gitea action runs, I will get:
[semantic-release] › ✘ An error occurred while running semantic-release: RequestError [HttpError]: 404 page not found

# for the following URL (i decoded it for ease of reading)

https://xxx/api/v1/search/issues?q=repo:my-org/test+type:pr+is:merged+b2c210e22ae555c72b558ac1cad301a72b7da678+1b9064ab221042e2308b0914f2970853f4aa3c17+4dfea9afa9eaf49184166b689a7f2fd514e97ade+829bd9c0a62c9af1fb8140c244834d96d8fd762d+9753873f431dbc31ad121a9427803c6f3078cde8

# it looks like Gitea does not support the API search format above

I believe on of the @semantic-release/github step involves search for issues in the repository.
Other that this issue, it looks like we can directly use @semantic-release/github to perform semantic release. I am aware of saitho/semantic-release-gitea but looks like no longer maintain so I did not want to use that.
Gitea Version
1.21.11
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Self hosted, docker based installation.
Database
MySQL/MariaDB

If you want a new API, please check whether that API has/has not implemented. And if not, it should be a proposal but not a bug.

My apologies. I should not have mark it as a bug. Looking at the API docs, looks like this has not been implemented.
I will recreate it as a proposal instead. Thanks for quick response.


Handmade color palettes throw an error with unquoted keys

Problem Summary:
When using wrapper mixins (like dark-theme()) and passing a handmade palette with unquoted keys, it causes a compilation error. However, if the keys in the palette are quoted, the error is resolved, and the mixins work as expected.
STEPS TO REPRODUCE

Create a handmade palette

$handmade-palette: (
    primary: (
        50: #e6eff8,
        '50-contrast': black,
        100: #bfd7f2,
        '100-contrast': black
    ),
    secondary: (
        50: #fef7e2,
        '50-contrast': black,
        100: #fdeab7,
        '100-contrast': black
    )
    ...
);

Use the palette in a wrapper mixin like dark-heme():

@include dark-theme(
    $palette: $handmade-palette
);

Compilation error occurs.

BEHAVIOR:
Unquoted keys in palettes cause compilation errors.
Quoting keys resolves the issue.
EXPECTED BEHAVIOR:
The wrapper mixins (or theme()) should handle both quoted and unquoted keys in handmade palettes, OR
The documentation should explicitly state that palette keys must be quoted to avoid errors.

Seems this is only true for the gray color variants. Likely due to the fact that the key is interpreted as color instead of string.


Update project_report_plugin.adoc

There is a typo in project_report_plugin.adoc under the Tasks subsection.
Before Change:
dependencyReport — HtmlDependencyReportTask
After Change:
htmlDependencyReport — HtmlDependencyReportTask
Context

Correcting the typo will ensure that the doc does not seem confusing to the users.

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Thanks for the contribution @akmalick. Could you please sign off your commit so we can accept it?

All checks seem to have passed. I was able to get over the issue; thanks to that StackOverflow user.
@eriwen Thank you so much for your comment; it was a great reminder and pulled me out of procrastination.

Thanks @akmalick

Let me thank you, please @big-guy . You all are doing a great job at Gradle. The dev community loves all of you.


remove extra spaces in doc comments

Coverage remained the same at 100.0% when pulling 2a608cd8cd5ca714331d77e5187131f22060174c on option-docs-tweak into ac32adaa3474fca3df227c5e94cb234afab7359a on master.


[Docs] Tutorial hotfix

Description
This PR attempts to make minor adjustments to parts of the tutorial documentation with prim prefix still being used.

Looks good!

Obligatory comment: nice PR #.


[iam-role-for-service-accounts-eks] cert-manager policy

Is your request related to a new offering from AWS?
No
Is your request related to a problem? Please describe.
Support for IAM role for cert-manager https://cert-manager.io/docs/configuration/acme/dns01/route53/#set-up-an-iam-role
Describe the solution you'd like.
I can provide a pull request, based on external-dns policy with minimal changes
Describe alternatives you've considered.
Making custom resources
Additional context
Basically just creating issue to check if it's OK to provide pull request for cert-manager support

This issue has been resolved in version 4.20.0 :tada:


Automated cherry pick of #5378: fix: Add 'scope=system' and 'system' for Caches when querying from Keystone.

Cherry pick of #5378 on release/2.13.
#5378: fix: Add 'scope=system' and 'system' for Caches when querying from Keystone.

/lgtm

/approve


MrackConfig: improve the approach to be object based and easy to extend

base on PR: https://github.com/neoave/mrack/pull/36
This commit implements a MrackConfig which allows to define options
and arguments otherwise needed in a configuration file and thus
save people's time with writing them in CLI.
Config file looks like:
[mrack]
mrackdb = config/mrackdb.yaml
provisioning-config = config/provisioning-config.yaml
metadata = config/metadata.yaml

Where each value is optional and existance of the config is also
optional.
Mrack looks for the config on these locations:

user provided path via -c or --mrack-config global option
~./mrack.conf
~/.mrack/mrack.conf
/etc/mrack/mrack.conf

It also changes CLI of commands:

metadata is no longer an argument but an option: -m or --metadata
provisioning config shortcut changes to global -p

With these changes it is possible to define a config file to allow
just typing:
$ mrack up
$ mrack list
$ mrack ssh
$ mrack output
$ mrack destroy

The MrackConfig class is also extensible for new use cases.
Signed-off-by: Petr Vobornik<EMAIL_ADDRESS>

Works for me @pvoborni thanks for the patch and improvement :)


SIL: Remove uncurryLevel parameter from type lowering

There was only one place where we lowered AST types with a non-zero
uncurry level, in SILGenApply.cpp. Add a new overload of the
getSILFunctionType() method that takes an uncurry level. All the
other methods no longer have to thread it through.

@swift-ci Please smoke test

+1!

@swift-ci Please smoke test

@swift-ci Please smoke test


Can't install package from private git repo which has dependencies from another private git repo

maxblock@mbp ~ $ pipx install --spec<EMAIL_ADDRESS>privcli
  Could not find a version that satisfies the requirement privlib==0.0.5 (from privcli) (from versions: )
No matching distribution found for privlib==0.0.5 (from privcli)
'/Users/maxblock/.local/pipx/venvs/privcli/bin/python -m pip install<EMAIL_ADDRESS>-q' failed

But this private cli can be installed via pip directly with --process-dependency-links flag:
pip install<EMAIL_ADDRESS>--process-dependency-links
Is it possible to add --process-dependency-links flag for pipx?

Sorry, I haven't noticed --pip-args flag.
So, the problem can be solved like this:
pipx install --pip-args="--process-dependency-links" ...


refresh failed

Now I'm using microk8s v1.18.4 on Ubuntu 18.04.1 and snap is tries to refresh it to v1.18.6, but it always fails at the same step.
I attach the log file.
refresh-failed.log
The error message in the log file is in Japanese.

ln: シンボリックリンク '/var/snap/microk8s/1551/opt/cni/bin/bridge' の作成に失敗しました: ファイルが存在します

It means

ln: Failed to create symbolic link '/var/snap/microk8s/1551/opt/cni/bin/bridge'.: File exists.

Are there any workarounds?

Hi @shojiro000 what channel are you currently on? Can you share the output of snap list microk8s or the tarball produced by microk8s.inspect? Could you also share the output of snap changes microk8s? I need to understand what is the refresh path that is failing. Thank you.

Hi @ktsakalozos . Thank you for your response.
Here are some snap commands and outputs.

root@oven:~# snap list microk8s
Name      Version  Rev   Tracking       Publisher   Notes
microk8s  v1.18.4  1503  latest/stable  canonical✓  classic
root@oven:~# snap changes microk8s
ID   Status  Spawn                   Ready                   Summary
175  Error   yesterday at 22:33 JST  yesterday at 22:41 JST  Auto-refresh snap "microk8s"
176  Error   today at 02:08 JST      today at 02:23 JST      Auto-refresh snap "microk8s"
177  Error   today at 10:48 JST      today at 10:54 JST      Auto-refresh snap "microk8s"
root@oven:~#
I don't think that you need to see the output of microk8s.inspect but if you do I'll share it.
Thank you.

What can you tell me about the deployment you have? How old is it? Where you always following the latest/stable channel? What addons do you have enabled? Any special configuration you have?
The opt/cni/bin is where kubernetes finds the CNI utils. The path to the cni utils used to be /snap/microk8s/current/opt/cni/bin/. With the multus support we moved the path to the cni utils in the writeable location /var/snap/microk8s/current/opt/cni/bin/. In doing so we create symlinks from the old location to the new one. The error message you see is that the creation of the symlink is failing because a file with the same name already exists. In [1] we
create the new CNI location and properly symlink the utils. The CNI utils path is referenced in containerd configuration /var/snap/microk8s/current/args/containerd-template.toml and kubelet /var/snap/microk8s/current/args/kubelet.
An obvious fix for the issue you report is to check that k8s is not using the /var/snap/microk8s/current/opt/cni/bin/, remove it (take a backup just in case) and try to refresh the snap.
I would like however to understand how the deployment could have reached this state. Could it be a half finished refresh? What do you see with:
sudo ls -l /var/snap/microk8s/current/opt/cni/bin/

What are the contents of /var/snap/microk8s/current/args/containerd-template.toml and /var/snap/microk8s/current/args/kubelet?
[1] https://github.com/ubuntu/microk8s/blob/master/snap/hooks/configure#L299

Hi @ktsakalozos . Thank you for your information.
I'm using microk8s since April. Always using latest/stable channel.
Here attachments are the file list of directories and some files you requested.
containerd-template.toml.txt
kubelet.txt
ls-snap-microk8s-current-opt-cni-bin.txt
ls-var-snap-microk8s-current-opt-cni-bin.txt
microk8s.status.log
I'll try to refresh it after remove /var/snap/microk8s/current/opt/cni/bin/.
Thank you.

Hi. @ktsakalozos .
Your workaround has worked.
If you need more information, I'll share them.
Thank you.

If I recall correctly before the merge of multus (and switch to default CNI bin to var with links), if calico was enabled it just copied all the CNI bin files over to /var/snap/microk8s/current/opt/cni/bin/ and never really cleaned them up.  Shouldn't be possible now, but for anyone who had enabled calico in the past (which I think came via HA?) they might need to manually clean them up before upgrading.

You are right @apnar. However, I do not think we ever exposed calico in latest/stable.
@shojiro000 is it possible you temporarily refreshed to latest/edge/ha-preview or tried to enable HA in any other way?


nullish coalescing support

Any plans for built-in nullish coalesing support?  When ?? is added as a binary op similar to ||, the right side evaluates regardless of the nullishness of the left side.
@6utt3rfly tagging given your outstanding jsep plugins, and the possibility that you may have thoughts on this.

@tvkit - is your comment more about jse-eval? JSEP parses statements into an AST, but doesn't evaluate the code. I could add the newer operators into the JSEP plugins, however

@6utt3rfly that would be great, thanks! Question: would the right side only evaluate when the left side is nullish?


Kf 1105 gh75 fix crd upgrade

fix: added upgrade handling
https://github.com/canonical/training-operator/issues/75
Summary of changes:
- Added upgrade event handler.
- Modified K8S resources installation to use forced conflict resolution
  in upgrade path.
- Updated requirements to bring in latest chisme (need to do forced
  server-side apply).
- Added remove and upgrade integration test that reproduced the issue.
- Modified method naming to be in-line with other charms (unification).

Many changes to requirements* are due to update chisme. Please review in time. This solution blocks another upgrade issue.
NOTE: CI might fail due to microk8s issue.

Merged into track/1.6
https://github.com/canonical/training-operator/issues/75


Separate docker container?

Since this is already a docker container, could you make some installation instructions for directly on Docker, without HA?

This container can't run outside of HA.


Lookup table BGR or RGB?

Hi, I'm trying to use the software only for calibration (both radiometric and vignetting) and export the result to somewhere else without using this software's io system. For this purpose, I saved the 'coefficients_' variable as a csv file. Here, I'm a bit confused whether the values of the variable is ordered RGB or BGR. For example, if I have camera response function lookup table that starts with

0, 0, 0, 0.044090159, 0.040110908, 0.040918224 ...

How is this ordered? And does the same thing hold for non-parametric vignetting lookup table?
Thank you in advance!

The order of color channels in the calibration files (both CRF and vignetting) is exactly the same as in input images coming from the cameras. The OpenNI2 and RealSense grabbers implemented in this repository output BGR images, so this is the order you have.


Update Get-CsUserCallForwardingSettings.md

First Draft version for released feature in SFB Server 2019 CU1 version

Docs Build status updates of commit 3b225d7:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsUserCallForwardingSettings.md
:x:Error

Details

:x:Error

Details

skype/skype-ps/skype/Get-CsUserCallForwardingSettings.md

[Error]

Failed to parse skype-ps/skype/Get-CsUserCallForwardingSettings.md due to :57:(616) '###
'
 Unexpected header name #, please use PlatyPS to do content validation

Line 138: [Error] W:\k002-s\.optemp\packages\opbuild.scripts.1.78.0\tools\opbuild\docset.prebuild.ps1 failed to execute

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Schema checks failing. Please resolve.


Robotic Sounding Voice Problem

Hi guys!
I'm trying to get the perfect chipmunk voice - like the one in the TLC.mp3 Novocaine preview - I'm reading / playing back from a prerecorded file.
But all I get is a robotic voice with weird / almost no pitch alteration.
Does anyone have a good example how to achieve a good chipmunk voice?
Thank you very much!

Anyone? Please

Hey, the purpose of Novocaine is to work with raw audio. Unfortunately, producing audio effects, like robot voices, is out of the scope of this project. If you are looking to do time- or frequency-stretching like making chipmunk voices on a live production system, I would recommend this: http://www.zynaptiq.com/ztx/. I have used it in a previous gig. I'm not sure how much the license costs anymore.
I'm going to close this issue, because this isn't about a core feature of Novocaine.


Raise click ceiling

Allow click 8. This would help me out of some dependency hell, so if you wouldn't mind spinning a release for this, I'd appreciate it; let me know if I can help!
I tried each of the top-level commands and nothing is obviously broken.

I've got to fix #80 and #83, which I'll do this week, and with that version shipping I'll update to click 8, it looks like they didn't make any breaking changes that aws-sso-util should hit (I wish there was a good way to test code that uses Identity Center APIs, but there really isn't).

Thank you!


copilot suggestions are seemingly unaware of sql syntax of editor

I have MySQL, PGSQL, Kusto & Oracle extensions installed. Now, if I open new query window (default is set to MSSQL language) & try to use the hint I end up getting PGSQL syntax:

it seems like copilot doesn't know which SQL syntax the current editor is set to

copilot is flat out making suggestions to me using tables and columns that don't even exist


Risk Calculation needs Clarification and Summary

There are various risk values of some form. There is no central place to look them up.
Use-case: I got a warning about a "low risk" encounter. I want to know what kind of encounter counts as "low risk".
Existing important Risk Values:

Risk Level
Total Risk Score
Combined Risk Score

Risk Level
Missing Summary
This is the user facing warning when an exposure happened.

possible values: "low risk" or "high risk"
based on the "combined risk score" for the overall summary (?)
based on the "total risk score" for individual encounters (?)
mapped according to the following parameters: https://github.com/corona-warn-app/cwa-server/blob/master/services/distribution/src/main/resources/master-config/risk-score-classification.yaml
covering the last 14 days
examples

[low risk, minimum requirement]
[high risk, minimum requirement]

Clarification Needed:

The app displays two values, one total risk as a title, one as "n exposure(s) with low/high risk". Is the latter based on the "total risk score"?

Combined Risk Score
Missing Summary
Total risk deducted from all past encounters with known covid positive persons.

Based on Total Risk Scores
possible value: ?

Total Risk Score
Missing Summary
Risk of infection for a past single encounter with a known covid positive person.

possible value: 0-4096
based on

Transmission risk level of person tested positive (https://github.com/corona-warn-app/cwa-documentation/blob/master/cwa-risk-assessment.md)
context of the encounter (proximity, duration, when; https://github.com/corona-warn-app/cwa-documentation/blob/master/solution_architecture.md#risk-score-calculation)

Clarification Needed:
Dismissal of short or distant encounters
According to the transmission risk level calculation short or distant encounters are dismissed. The exposure config  however still has bins for these values. It is not clear what a brief and/or distant encounter with a known covid positive person will result in.
Possible would be:

No exposure warning: the encounter was dismissed
Risk Level "Low risk": the encounter was noted and likely rated with a Combined Risk Score of 0, which maps to "low risk"

What is the value of the minimalRiskScore?
According to the docs there is another threshold at play. I can't see what it's set to or if it remains unset. Here the relevant paragraph:

Google/Apple framework allows to set a minimalRiskScore to exclude exposure incidents with scores lower than the value of this property. In the current version of the API the time spent within the ranges of attenuation buckets are accumulated over all exposure incidents during one matching session. Since the number of requests is currently limited, it is not possible to get these values for each day and each exposure incident separately. While by default there is no minimum value set, this value is being configured accordingly, so that presumably irrelevant exposure incidents are excluded.

Internal Tracking ID: EXPOSUREAPP-3495

Hey @cougarten ,
Thanks for the input. I have forwarded your feedback to the dev team by creating an internal Jira ticket (EXPOSUREAPP-3495). They will now investigate how the documentation can be reformulated or extended to make it more understandable. We will inform you about updates on the progress of the rewrite here in this issue.
Regards,
CH

Corona-Warn-App Open Source Team

The value of the minimum risk score is 11 as linked at the end of CWA Risk Assessment document.
Maybe the following blog post can help answer some of your questions?

@cougarten
https://www.coronawarn.app/en/blog/2020-12-30-cwa-behind-the-scenes/ and
https://www.coronawarn.app/en/blog/2020-12-17-risk-calculation-exposure-notification-framework-2-0/
provide a lot of updated information about risk calculation. Does this answer your question now?

@cougarten
https://www.coronawarn.app/en/blog/2020-12-30-cwa-behind-the-scenes/ and
https://www.coronawarn.app/en/blog/2020-12-17-risk-calculation-exposure-notification-framework-2-0/
provide a lot of updated information about risk calculation. Does this answer your question now?

The documentation has been updated.
See https://github.com/corona-warn-app/cwa-documentation/blob/main/solution_architecture.md#mobile-applications for current information.
The document https://github.com/corona-warn-app/cwa-documentation/blob/main/cwa-risk-assessment.md is now deprecated.

Is there an update available here? I don't think @cougarten's questions were answered sufficiently.


Pandas Series output objects name

Is your feature request related to a problem? Please describe.
When a function node receives a pandas Series as input and outputs a new pandas Series derived from it, the Series keeps the name of the input object.
For example:
timestamp = pd.Series(name="event_timestamp", dtype=datetime64[ns])

def driver_id() -> pd.Series:
  return pd.Series(name="driver_id", dtype=int)

def day_of_week(timestamp: pd.Series) -> pd.Series:
  return timestamp.dt.day_of_week

def count_per_day_of_week(driver_id: pd.Series, day_of_week: pd.Series) -> pd.Series:
  df = pd.concat([driver_id, day_of_week], axis=1)
  return df.groupby(["driver_id", "day_of_week"]).count().reset_index(drop=True)

Here, the code will fail because the groupby won't recognize the name day_of_week. The name associated with day_of_week remains timestamp, i.e., the name associated with the input of the function day_of_week().
Describe the solution you'd like
The above example is fixed by wrapping the result of day_of_week like this:
def day_of_week(timestamp: pd.Series) -> pd.Series:
  return pd.Series(timestamp.dt.day_of_week, name="day_of_week")

It would be great if this could be handled automatically, or at least catch this type of error to provide a relevant message.

Yeah my worry with enforcing naming is that we mutate things.
Instead I think the check that the name matches the function name isn't a bad one. Otherwise the clearer thing to do is what you describe I think -- rather than get the framework to enforce it.

Agreed that having the pandas default is better than enforcing another arbitrary default. Thinking about it, it is the kind of check that should be part of a unit test and not necessarily data validation because it is a property of the code logic that should be invariant to data.
Closing the issue!


allow version 1.x of gen_stage

Apologies - intended to make this PR against my fork while I test some stuff! Doesn't look like much has changed in gen_stage between v0.14.3 and v1.0.0 so fingers crossed this will just work!

Apologies - intended to make this PR against my fork while I test some stuff! Doesn't look like much has changed in gen_stage between v0.14.3 and v1.0.0 so fingers crossed this will just work!

Yea, thanks for taking that on. Happy to merge and release a new version.

Yea, thanks for taking that on. Happy to merge and release a new version.

All good @blatyo - thanks for getting this adapter together in the first place! Top stuff 😄
Have now tested this locally against gen_stage v1.0.0 and it appears to be working without hitch 👍

All good @blatyo - thanks for getting this adapter together in the first place! Top stuff 😄
Have now tested this locally against gen_stage v1.0.0 and it appears to be working without hitch 👍

Ooops, just realised dependabot had this covered already - closing in favour of #39

Ooops, just realised dependabot had this covered already - closing in favour of #39

Support for 1.X was added to the newest version

Support for 1.X was added to the newest version

Wonderful, thanks @blatyo :)

Wonderful, thanks @blatyo :)


Client fails to setup due to race during docker node startup

When deploy is invoked with like ./parity-deploy.sh --config aura -n 1 -r nightly --min-gas-price 0 --enable-client --reserved-only, client node is created.
Unfortunately the following scenario is possible: https://gist.github.com/0x7CFE/df64afad8244e74156a772a6b2df28f0
tl;dr: Client fails to start due to failed name resolution (related to https://github.com/paritytech/parity/issues/6907). Looks like this happens because of a race between nodes. If host1 starts before client all is working as expected. However, client may try to resolve host1 before latter obtains an IP address.
In order to fix that, we need either fix the order in which nodes start or assign static addresses to nodes.

Found the best solution so far and it is: https://docs.docker.com/compose/startup-order/
So, client node should depend on all hosts. That would fix the race.

Unfortunately that wasn't enough. On a more complex case other than 1 client 1 authority, the problem remains. That's because each authority node is configured so that other authorities are set as reserved nodes. So host1 depends on host2 and vice versa.
This results in the following dead loop:
$ sudo docker-compose up
Creating network "paritydeploy_default" with the default driver
Creating host2
Creating host1
Creating client
Attaching to host1, host2, client
host1     | Loading config file from /parity/authority.toml
host2     | Loading config file from /parity/authority.toml
host2     | Invalid node address format given for a boot node: enode://be0df3e855e5995a57ecf1e863aba96f0e29f075a9cbb359456b045b67d895c87c4935599306b797ef2b172d2b6556d0c57f60dcc1b5f0b4080bb74ebb2c32e2@host1:30303
host1     | Invalid node address format given for a boot node: enode://ba1d0e6d920b7e219f13ef540319c361abbd9c26e5487ff35f6a019cd8dd2d6d62642b55a06472a766f12705de971a1bb40d5f643d0f38e58f7960f79fe13ef4@host2:30303
client    | Loading config file from /parity/client.toml
host1 exited with code 1
host2 exited with code 1
client    | Invalid node address format given for a boot node: enode://be0df3e855e5995a57ecf1e863aba96f0e29f075a9cbb359456b045b67d895c87c4935599306b797ef2b172d2b6556d0c57f60dcc1b5f0b4080bb74ebb2c32e2@host1:30303
client exited with code 1

We need either to solve nameserver issues, retry resolution after some time or bring all hosts up and then actually start the business logic.

I believe a fix was introduced for this. Can you please retest.

@0x7CFE is this still any issue?

@ddorgan, sorry for the late response. I haven't used docker recently. Probably would be better to ask someone, who is working on it now.

@0x7CFE Thanks! Closing issue.


cloudstack: If no volumes are found return an empty list

Otherwise we got this:

Traceback (most recent call last):
  File "auroracompute.py", line 16, in
    print(conn.list_volumes())
  File "/home/wido/repos/libcloud/libcloud/compute/drivers/cloudstack.py", line 2172, in list_volumes
    for vol in volumes['volume']:
KeyError: 'volume'

Simply return a empty list if no volume is found

@runseb Can you take a look?

@Kami Yes, that's a nice syntax :) Fixed!

Merged, thanks :)


light mode colors

Light mode colors for ui

Thanks for the PR! Code looks good - I'll test it out tomorrow then merge it in.

Ok great.


With Xcode 12 not working

build successfully but error showing info.plist not found.

Had the same issue, upgraded cocoapods, clean project, clean Derived data and worked again

https://khushwanttanwar.medium.com/xcode-12-compilation-errors-while-running-with-ios-14-simulators-5731c91326e9
checkout this post @bhaveshCog0919 @gitpushdev


Attempt at basic NonEmptyLists

Overrides many methods in ::[A] and List[A] to keep type information when doing size-preserving transformations
Add a NonEmptyList factory, which cannot conform to IterableFactories[NonEmptyList] because it is impossible to construct a Builder[A, NonEmptyList[A]], as that trait requires a clear() method, returning the underlying collection to an empty state

Really hopeful that some form of NonEmpty collections can make it into the next redesign in one way or an other :)

I fear this would negatively impact type inference and/or give confusing error messages. Also, why should non-empty lists be treated specially? Why not also have lists of length greater to some constant, or lists of length equal to some constant? It seems non-empty lists scratch only the surface here.

@odersky you have a good argument, just a couple points:

I do think the distinction between (length == 0) and (length > 0) is not really arbitrary. There are many methods on List[A] that for example throw exceptions when called on Nil:

head
tail
last
init
min / minBy / max / maxBy

which means that when using List in practice, code is often obfuscated in .headOption.map..., pattern matches, or Try(list.max).toOption, and I think you end up with lots of error handling code because of the lack of NonEmptyList. Just my opinion though

Why not also have lists of length greater to some constant, or lists of length equal to some constant?
That would be cool! Sort of like an HList maybe?

case class :: [+A, +Tail <: List[A]](x: A, private[collection] var next: Tail @uncheckedVariance)

val atLeastTwo: Int :: Int :: List[Int] = 1 :: 2 :: 3 :: Nil

Cheers!

@odersky allow me a rethorical question: why treat lists of 0..1 entries specially?
both Option and non-empty collections are things that occur quite frequently in day to day programming, or at least for me they do. weren't there places in scala macros where the AST used (A,List[A]) instead?

The difference is that Option is a solid abstraction. A non-empty list abstraction is destroyed the moment you take tail because that loses the knowledge of nonemptyness.

i don't see how this would be "not solid" - of course taking the tail of a non-empty collection is partial, just like taking the head of a regular (possibly-empty) collection.
same as List has headOption, NonEmptyList would have tailOption - or tailList if you want.

The original proposal was to retrofit non-emptyness on existing lists, which IMO will never work. Exploring a separate abstraction for non-empty lists with all the necessary methods - sure, why not? But such an abstraction should be developed and widely user-tested before we would consider putting this in the strawman.

I think it is valuable to have a trial implementation in the collections strawman as a branch because it's another kind of use-case that is reasonably common (there are a number of NonEmptyList-like implementations for the current Scala collections).  If the strawman does not support it as well as the current collections do, we may wish to think if we can tweak the design to support it better even if we don't include it.  Because there are several battle-tested NonEmptyLists out there (in scalaz, cats, and scalautils (except that actually goes out to two+ elements with "many"), among others).
I agree that we likely don't want it as part of regular List because with List it is convenient for type assignment that x :: Nil has type List not type ::, and since this is how it was done historically, changing that would cause a lot of hassles.  But making NonEmptyLists a lot harder to implement would also cause hassles.

It is "the point" (or rather, the copoint :wink:) of NonEmptyList to be "destroyed the moment you take head"; it's a Comonad. Extending the NonEmptyList context to another function that requires it is as easy as nel.extend(f) (i.e., nel.tails.map(f)).
However :: does not at all make for a readable NonEmptyList type name.


Error with IPNS paths with ending slash

Hi!
I found a little Bug in this implementation when the URL have a trailing slash
The script enters in this IF and treats the answer like a directory and not a file. I tested locking it as always false and it runs OK.
https://github.com/RangerMauve/js-ipfs-fetch/blob/631127a8568aecab8f497c4cf6149162b87ebf8f/index.js#L129

The trailing slash usually means that it's a directory. 🤔 From there it will look for an index.html or similar to resolve to a file if it exists.
Could you point to an example URL with the trailing slash as well as which software it works in?


Errors w/ Linux binaries

Running with bash via Ubuntu 16.04 subsystem for Windows.
Followed instructions here: https://github.com/Azure/azure-functions-docker-python-sample/wiki/Getting-started-using-Functions-Core-Tools.
Downloaded linux zip file from https://ci.appveyor.com/project/appsvc/azure-functions-cli/build/<IP_ADDRESS>4/artifacts
Upon using binaries to run func init . --worker-runtime python (or any other call to func), I get the following errors:
Failed to load X{z, error: libunwind.so.8: cannot open shared object file: No such file or directory
Failed to bind to CoreCLR at '/mnt/c/Users/Administrator/dev/soh-local/func_venv_bash/linux_functions/libcoreclr.so'
Fixed by running apt-get install libunwind8

@ahmelsayed

This issue was moved to Azure/azure-functions-core-tools#468


Go/src/cmd/compile/internal/types/type.go line 1029 typo/error!

https://github.com/golang/go/blob/51ac0f0f4cb432204dee3d434335fd1e61ca8446/src/cmd/compile/internal/types/type.go#L1029
xname := t.Extra.(string)

should ACTUALLY BE:
xname := x.Extra.(string)

Thank you @PraxisProject for this report and welcome to the Go project! Great catch with a code audit, we’ve had that in for 3 years. If you’ve got time could you please send a patch and you’ll become a Go contributor too.
/cc @josharian @randall77

Heyo. Submitted PR.


[Bug]: Infinite ReAct loop when setting max_iterations as 1

Bug Description
When I set max_iterations to 1, my ReAct Agent stuck in infinite loop.
Version
0.11.14
Steps to Reproduce
Set query that agent can't find information from tool.
agent = ReActAgent.from_tools(
    tools,
    llm=llm,
    max_iterations=1,
    verbose=True,
)
agent.astream_chat(query)

Relevant Logs/Tracbacks
No response

_get_response function in ReActAgentWorker is checking iterations. In current situation, But if current_reasoning starts from 2 and max_iterations is 1, then ReAct agent will be stuck in infinite loop.
So my suggestion is to modify operator == to > like below
def _get_response(
        self,
        current_reasoning: List[BaseReasoningStep],
        sources: List[ToolOutput],
    ) -> AgentChatResponse:
        """Get response from reasoning steps."""
        if len(current_reasoning) == 0:
            raise ValueError("No reasoning steps were taken.")
        elif len(current_reasoning) > self._max_iterations:
            raise ValueError("Reached max iterations.")