Adding alpha var to RenderAlphaOnSurface

It seems useful to be able to control the alpha setting on RenderAlphaOnSurface.  This is so we can do fade outs etc unless you can think of a better way.  I keep changing this file so figured I would ask if can just get it added.  Thanks.
public float Alpha = 1;
in lateupdate
     _mpb.SetFloat("_Alpha", Alpha);

Oh also maybe a helper function someplace to get height at position.  I keep adding this to OceanRenderer as well.  Again you might have a better idea but use this a fair amount.
public float GetWaterHeightAtPosition(Vector3 position)
{
float height = 0;
var r = transform.lossyScale.magnitude;
var collProvider = OceanRenderer.Instance.CollisionProvider;
var rect = new Rect(new Vector2(position.x - r, position.z - r), 2f * r * Vector2.one);
if (collProvider.GetSamplingData(ref rect, 2f * r, _samplingData))
{
var pos = position;
if (OceanRenderer.Instance != null)
{
collProvider.SampleHeight(ref pos, _samplingData, out height);
collProvider.ReturnSamplingData(_samplingData);
}
}
return height;
}

Hey, I'm unsure about the former because then it will be driving from code a parameter on the material that an artist may want to tweak, it will be confusing in this case if the code stomps over it. An option would be then to hide it from the material gui to avoid this confusion, but im not sure this is my intention. I'd be inclined to leave it as it is and suggest that if you want to drive it from code, drive it directly to the material from the code that cares about it. An (inefficient) example:
MyScriptThatWantsTheAlphaChanged.cs:
...
theAlphaObject.GetComponent().material.SetFloat("_Alplha", whatever);

No problem.  Close it out.  You're right was not really thinking about it
clearly
On Wed, Feb 6, 2019, 2:23 PM Huw Bowles<EMAIL_ADDRESS>wrote:

Hey, I'm unsure about the former because then it will be driving from code
a parameter on the material that an artist may want to tweak, it will be
confusing in this case if the code stomps over it. An option would be then
to hide it from the material gui to avoid this confusion, but im not sure
this is my intention. I'd be inclined to leave it as it is and suggest that
if you want to drive it from code, drive it directly to the material from
the code that cares about it. An (inefficient) example:
MyScriptThatWantsTheAlphaChanged.cs:
...
theAlphaObject.GetComponent().material.SetFloat("_Alplha", whatever);
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/huwb/crest-oceanrender/issues/180#issuecomment-461211984,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoDjoV2InLWjIs8mDe9g0KVrpsNhgFuGks5vK1XpgaJpZM4afvVI
.

The latter is intentional because GetSamplingData() is a nontrivial operation (in particular if using displacement textures, in which case the code must seek out the lod index that provides the desired data), so it is supposed to make calling this function a conscious decision (and if possible call it once for multiple queries). However if there are lots of disparate single queries i can see how it would be tedious. I could make a function like this and give it a name and comment that implies its for single queries only and can be expensive, would that make sense?
(Minor request - in the future could we not mix issues together like this as it makes my OCD flare up :), and it will make it hard for me to organise and refer back to discussions which i frequently do.)

Sure all that makes sense. Again I think I was not thinking about the
expense so I need to double check my code and see if I really should be
doing this, this way, or should try and combine these.  Would be bad based
on what you said to do a lot of these septate calls be batching them so I
can see why you did not create a function for this
On Wed, Feb 6, 2019, 2:28 PM Huw Bowles<EMAIL_ADDRESS>wrote:

The latter is intentional because GetSamplingData() is a nontrivial
operation (in particular if using displacement textures, in which case the
code must seek out the lod index that provides the desired data), so it is
supposed to make calling this function a conscious decision (and if
possible call it once for multiple queries). However if there are lots of
disparate single queries i can see how it would be tedious. I could make a
function like this and give it a name and comment that implies its for
single queries only and can be expensive, would that make sense?
(Minor request - in the future could we not mix issues together like this
as it makes my OCD flare up :), and it will make it hard for me to organise
and refer back to discussions which i frequently do.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/huwb/crest-oceanrender/issues/180#issuecomment-461213593,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoDjofnnQYGVOcXPmu9DKx9Z4qO_ekxqks5vK1cSgaJpZM4afvVI
.

Well, I'll close but feel free to post back here after you think about it. I'm not against making that change i mentioned if you think its helpful.

Thanks.  No problem.  New information or was not thinking about it correct
On Wed, Feb 6, 2019, 2:35 PM Huw Bowles<EMAIL_ADDRESS>wrote:

Well, I'll close but feel free to post back here after you think about it.
I'm not against making that change i mentioned if you think its helpful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/huwb/crest-oceanrender/issues/180#issuecomment-461215621,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoDjoZ32jvvE_uZwPm2vun7sGi51PPcjks5vK1i0gaJpZM4afvVI
.


Cannot find make on Strawberry

https://github.com/tobyink/p5-acme-githubactions-testing-tobyink/runs/1417446078

It looks that make command was found, but make couldn't call CreateProcess.
process_begin: CreateProcess(NULL, rem, ...) failed.
make (e=2): The system cannot find the file specified.

mingw32-make: *** [Makefile:333: blibdirs] Error 2

Please add --debug option, and show me the result.

make --debug is here:
https://github.com/tobyink/p5-acme-githubactions-testing-tobyink/runs/1427051830
The distribution can be found in this artifact:
https://github.com/tobyink/p5-acme-githubactions-testing-tobyink/suites/1531534311/artifacts/27132643
(Under the build directory!)

I found it!
You should set MAKE environment.
For historical reasons, there are multiple implementations of make command on Windows.
ExtUtils::MakeMaker uses the make used to build Perl by default.
And the make to build Strawberry Perl v5.24 is dmake.
https://github.com/shogo82148/github-action-test/runs/1437185226?check_suite_focus=true#step:3:16
name: CI
on:
  push:

jobs:
  printInputs:
    strategy:
      fail-fast: false
      matrix:
        os: [ubuntu-latest, windows-latest, macos-latest]
    runs-on: ${{ matrix.os }}
    steps:
      - uses: shogo82148/actions-setup-perl@v1
        with:
          perl-version: "5.24"
          distribution: strawberry
      - run: perl -MConfig -E 'say $Config{make}'

If you want to use GNU make, you need to set MAKE environment to overwrite the default.

perl -MConfig -e'exec $Config{make}' seems to work nicely. Thanks!


Remove the @team notification on completed brews.

The @ team notifications are causing two issues:

Coffee vultures are waiting for unclaimed spots without using the dibs systems
Too many notifications. For each brew there are at least 2 @ team notifications.

:+1:


why bullets is not a object like costs?

why bullets is not a object like costs?
as:
{
  "bullets": [
    {
      "name": "storage",
      "unit": "TB",
      "value": 5
    },
    {
      "name": "concurrent connections",
      "unit": "conunt",
      "value": 40
    },
    {
      "name": "Shared fake server",
      "unit": "conunt",
      "value": 0
    }
  ]
}

Its just an array of strings. Since its part of an example, that we don't really explain (or provide a schema for), people aren't really meant to examine it that closely :-)


[acr] Fix misspelling of 'repository' in acr subcommands help

Related command
az acr
Description
There's a typo and I don't like typos
Testing Guide
N/A
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

acr


when will be the next release version?

Would like to understand when will be the next release? I see the current version is in JDK 1.7, any plans to upgrade to 1.8 or more latest JDK version in next release?

The next release date is not defined for now. You can use v5.2.0 with Java 8 currently. The plan is to upgrade to Java 8 (or may be 11, a LTS version) in the next major version v6.

Currently, the plan is to release:

v5.3 by the end of June
v6.0 by the end of December

NB: 5.3 will be the last feature version of 5.x line. I will still maintain this line for one year with only bug fixes if any.


Move providers to their own repositories

One of the weaknesses of our providers today is that they have to live in our arcctl repository. It was always the intent that we allow the community to contribute providers, but requiring us to approve PRs into the main repo goes against that desire for self-service.
What we'd like to do is strip the providers out into their own repositories. This requires us to change the contract between the providers and the main CLI. Our intent is to mirror Terraform plugin behavior by having the providers become their own processes that communicate with the main CLI via gRPC. The main CLI will still be responsible for pulling down and starting the binaries for each provider, but will do so as-needed by the requests to update datacenters and environments.

Providers have been deprecated. Closing.


[Feature Request] Add ability to have a main configuration file name with .cjs extension

Hello, could you please add ability to have a main configuration file name with .cjs extension?
You have stated in README this:
Configuration via:
- .markdownlint-cli2.jsonc
- .markdownlint-cli2.yaml
- .markdownlint-cli2.js
- .markdownlint.jsonc or .markdownlint.json
- .markdownlint.yaml or .markdownlint.yml
- .markdownlint.js

and it could look like this:
Configuration via:
- .markdownlint-cli2.jsonc
- .markdownlint-cli2.yaml
- .markdownlint-cli2.js or .markdownlint-cli2.cjs
- .markdownlint.jsonc or .markdownlint.json
- .markdownlint.yaml or .markdownlint.yml
- .markdownlint.js or .markdownlint.cjs

It's important because of new type option in package.json.
Also as a bonus, it would be fine to add .json and .yml extensions too to cli2, at least .json because .jsonc is not such a standard yet.

As you can see from the list of supported file names, I am trying to avoid including redundant file names with the CLI2 set of files. (Learning from my "mistake".) Having extra names is more confusing for users, adds complexity to the code, and creates more opportunity for mistakes. As I understand, CJS is used to refer to "Common JS" which is already a format that the JS type allows. As such, I am not inclined to make this change.

Yes, type can be set to 'commonjs', but when I set it to 'module' then all .js files are considered as ESM. But .cjs will guarantees that file will be run as CommonJS regardless of module type. And if I am not wrong, then .markdownlint.js has to be of CommonJS type.

I'm not sure the behavior of the package.json type field is significant here? If this configuration file exists, it is probably not used by the package as a script. Unless the presence of the file is interfering with something like ESLint?
I'm also not sure this file needs to be CommonJS? That may be true, but I need to try it. If it's possible to allow both kinds, I would be less inclined to make the extension explicit.

Looks like you're right about this file needing to be CommonJS: https://nodejs.org/api/modules.html#the-mjs-extension
So my thinking at this point is that I may make a breaking change such that the extension for this file must be CJS.


Old build on ooni.org

The build available on the homepage still points to 3.0.2 for Windows and Mac
https://ooni.org/install/all
It would also be advantageous to all users if the site displayed the current version info.

Thanks for noticing this!
I have merged the PR which should lead to the updated version being served: https://github.com/ooni/ooni.org/commit/c7d0a1beaffc02455cfe5fb6d913d91243f9bf15.

It would also be advantageous to all users if the site displayed the current version info.

This is a great suggestion! I filed an issue for it here: https://github.com/ooni/ooni.org/issues/602.


Format Document causes a jump in the window viewport

Whenever I call LSP: Format Document, the cursor is moved to the last line of the file, and the viewport is moved up (but not all the way to the top, the top most visible line becomes the second line in the document).
I was considering using this command combined with the sublime-hooks plugin to add auto-formatting functionality for my Golang files, but with the current behaviour, that's pretty much undoable due to the distraction of having your cursor + viewport move every time you save.

Closing this, as I believe this is actually related to https://github.com/sourcegraph/go-langserver/pull/83.

The described behaviour (erase and rewrite entire document) is/was similar to python language server, but I've only ever manually invoked the format call. Thanks for reporting, please re-open if there is something LSP should do to handle this better.


XML sitemaps keep showing beta URLs

https://elixirschool.com/sitemap_dark_mode.xml is including beta URLs:
<urlset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://www.sitemaps.org/schemas/sitemap/0.9" xsi:schemaLocation="http://www.sitemaps.org/schemas/sitemap/0.9 http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd">
  <url>
    <loc>https://beta.elixirschool.com/blog?ui=dark</loc>
  </url>
  <url>
    <loc>https://beta.elixirschool.com/privacy?ui=dark</loc>
  </url>

Similar for https://elixirschool.com/sitemap.xml - in this case, it's fine when I browse it (I'm in Spain), but Rocket Validator is also getting beta URLs (the scraper is in US).
Rocket Validator can't scrape the site, because the URLs are not in the same subdomain.

Fixed! Thanks! 🚀

@kinson is the MVP!

@jaimeiniesta good catch 😅 on that one


iPhone 8 not found

Hello,
Since last update my IPhone 8 is no longer found. iPhone SE still works.

It could be that the Device ID of the iPhone 8 has changed, perhaps following an iOS update.
Use the "Locate My Devices (Input)" node to get a new list of all devices for your AppleID account.
Then check the iPhone 8 to see if it is different to what you were previously using.

I wrote a comment at the end of Issue 14 https://github.com/PfisterDaniel/node-red-contrib-apple-find-me/issues/14
This may also help when things that were previously working, stop working.

@RaoulSargent: Great, thx for your support :-D. That's the right answer

I close this Issue.... If you have more problems please create a new Issue or reopen this...


Fix code scanning alert - Remote property injection

Tracking issue for:

[x] https://github.com/webrtc-org/make/security/code-scanning/3

https://gist.github.com/gilbertalgordo


🚀 [Feature] On Register: subscribe to Newsletter (Clicktip)

:rocket: Feature

On Register: subscribe to Newsletter (Clicktip)

User registers
Hook is sent to Clicktip (server) to register his EMail with the newsletter

Furthermore we need an options in the user profile in the client to unsubscribe/resubscribe

Where can we find the corresponding api documentation for the clicktip service? @BerndHueckstaedt
Is there an implementation already for this? @einhornimmond
How do we want to handle the tunneling of the clicktip api stuff? Community server? Node server? Login server? @einhornimmond
Design & Implement UI @ogerly
Implement API connection Frontend <-> Backend @Mogge

Here is the API doc for klicktipp:
https://support.klicktipp.com/article/393-api-function-overview
It exist a php-Wrapper for it so I think the best would be implement this functionality in community-server.
But this is a feature which is used only by us. It dosn't belong to gradido, so better I implement it such that it is easy to remove later.

First step is done with #241

It seems with api key user can only subscribed, unsubscribed and manuel tags can be removed.
So if we like to save the status if user has subscribed to newsletter or not, we must save it in our db.

It is essential that the user selects the language on register so we send him newsletter in the proper language. It will not be obvious that the user can change his Newsletter language Settings in the wallet and he will not be informed to do so in the newsletter. Therefore it is very important that he gets the right language on the first time.
Furthermore: On register the language menu is coupled with UI, later on it is not. The user can change the UI language separately from the newsletter language.
Furthermore: we should consider to switch the default language to German since that is the main audience at this moment.


make test dump log debug level

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
133493f3220638cf92351f9a67de643911340f7e
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
133493f3220638cf92351f9a67de643911340f7e
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Wiki lists "array2D" type instead of "string" for snap_from_csv() function

As per title, snap_from_csv() lists the arguments as array2D, [cellDelimiter], [stringDelimiter] in the preview, whereas it should be string, [cellDelimiter], [stringDelimiter].

Thanks for reporting this. I'm in the middle of refactoring the library so I will address this at the same time as pushing out a new version.


Allow for more than Top5 users in replies

Hi, @gedankenstuecke I would like to work on this. I understand that we want to allow the visualization of replies from 5 to more -- should we allow the user to select the number of usernames he/she wants to visualize?

@minervapanda yes, given the current setup this would mean having to process the data once for each number of users because so far the underlying data is only stored for the top5 users (see https://github.com/gedankenstuecke/twitter-analyser/blob/master/tweet_display/analyse_data.py#L101)
Maybe we could store the Top 10 users instead and then just give a dropdown menu on the website so that people can filter down if they want to select fewer/more users (up to 10)?

Exactly what I had thought. Sounds good -- I will make a drop down for users to filter keeping the max. limit to 10.

A checkbox with explicit ranking mentioned on its side to select users from Top 10 might be even more user-friendly.

Yes, that sounds good too but the simpler version would be enough for a start 👍

@minervapanda filtering for the number of users to show  👍


Feature Request: I would like to create "My Orders" screen and the "Edit Profile" section in Profile.

My Orders: I would like to create the "My Orders" screen for this app, where users will be redirected to their "Orders" section after checking out. All their purchased items will be visible on that screen, along with the date and time of checkout.
Edit Profile: Additionally, I would like to include the option for users to add and edit their profile picture, name, and email.
@yuvrajsinghgmx Please assign me this issue . I would really love to work on it...

@yuvrajsinghgmx  Update 1: I have completed the Edit Profile Section till now
-Validation Checks on Empty user name and wrong email syntax.
-Saves the user's information and persists it.
-Can view and change profile picture .
https://github.com/user-attachments/assets/70c74707-3972-49c6-84f7-a89d6845ef9d

@yuvrajsinghgmx I will start with  the "My Orders" section tomorrow.I'll keep you informed


Any EncryptionMode in FluentFTP?

Client Computer OS: Windows
FluentFTP Version: 45.0.4
Framework: .NET 6
I need to apply encryption mode in FluentFTP's ftp.
The syntax "EncryptionMode" don't exist in .net 6.
var conn = new FtpClient();
conn.EncryptionMode = FtpEncryptionMode.Explicit;

How should I apply EncryptionMode in FluentFTP's ftp in .net6?
Thank you?

Config.EncryptionMode
https://github.com/robinrodricks/FluentFTP/wiki/FTP-Connection#how-do-i-connect-with-ssltls--how-do-i-use-ftps


v1.0.1

This repository contains GitHub workflows for managing releases automatically.
Please name this issue what you would like the new release to be called. E.x. "v1.0.0"
The following slash commands are available to use in comments on this issue:

Command
Description
Parameters

/release
Creates a new released based on the current state of the master branch
frontend=<version>backend=<version>gitapi=<version>dispatcher=<version>agnosticv=<version>anarchy=<version>poolboy=<version>

/promote
Promotes the created release to the next environment
N/A

/cancel
Cancels the current release (closes this issue)
N/A

/release frontend=v1.0.1 backend=v1.0.1 gitapi=v1.0.1 dispatcher=v1.0.6

/promote

/promote

Release v1.0.1 completed!!


fix: [core] Respect webpack config stats in static build

Issue:
The stats webpack config options have no effect on the output.
I think this touches on one part of #5250, though it looks like that issue might not be completely resolved by this PR, as that issue also mentions things about hot module reloading behavior its documentation.
What I did
I updated build-static.js to make sure that we're passing the stats config to any method that takes it.
Note: I did not update dev-server.js to explicitly pass the stats config property. We will instead continue to rely on it being overridden by an optional config.devServer.stats.
How to test

Is this testable with Jest or Chromatic screenshots? No
Does this need a new example in the kitchen sink apps? No
Does this need an update to the documentation? I don't think so. This brings the codebase into line with how we'd expect it to work given the current documentation.

cc @ndelangen


Assignment-3

screenshot
drift:

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
58

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
E (at least 1 Blocker Vulnerability)Note: A being best, E being worst

vulnerabilities
2


Purchasing zuut.com domain?

Hey! Sorry about the weird (non-)issue here. I found your Github profile from the Contact page of your blog, zuut.com/contact, and I wasn't sure how else to get a hold of you 🙃
I am interested in purchasing the zuut.com domain. Is this something you would be open to? Thanks for reading!

Sorry, its not for sale. Cheers


Enhancement/move to thriftpy2

This PR fixes #64 and #68.

Bump? It looks like the tests are setup to run an obsolete Python version (3.5).

If @jcrobak answer/review this PR I can work on changing everything to another Python version.

Finally getting around to fixing the build, and I'm incorporating this fix. Thanks!
See: https://github.com/jcrobak/parquet-python/pull/70


Created Upper_triangular_matrix.java

Submission Checklist

[x] Your pull request targets the master branch of the repository.
[x] You have only one commit (if not, squash them into one commit).
[x] You have read the Contributing guidelines and your changes follow them.

Type of Change

[ ] Bug fixes
[x] New implementation

PR Description

Implemented logic for upper triangular matrix identification using Java.
Issue #[Add issue number here.]

Implementation successful


Windows clients don't flush DNS on connect

Suggest to add 'push "register-dns"' to server conf to force Windows clients to flush DNS on connect.

Require additional config: https://stackoverflow.com/a/47415482/6945353
So no. Thanks for the issue though.


Wrong functionType is determined for the FunctionInvocationWrapper

We have a bean like this:
@Bean
Scheduler reactiveScheduler() {
	return Schedulers.elastic();
}

where a target instance is a Supplier, according Schedulers.CachedScheduler extension. And this one is registered in the FunctionCatalog as a Supplier.
But at the same time it looks like functionType for the FunctionInvocationWrapper is determined from the bean definition type.
And in this case we have a plain:
public interface Scheduler extends Disposable {

so, that Disposable is set into the functionType, which leads to an exception like this when we perform function lookup:
Caused by: java.lang.IllegalArgumentException: Must be one of Supplier, Function, Consumer or FunctionRegistration. Was interface reactor.core.Disposable
    at org.springframework.util.Assert.isTrue(Assert.java:118)
    at org.springframework.cloud.function.context.catalog.FunctionTypeUtils.assertSupportedTypes(FunctionTypeUtils.java:390)
    at org.springframework.cloud.function.context.catalog.FunctionTypeUtils.getInputType(FunctionTypeUtils.java:176)
    at org.springframework.cloud.stream.function.FunctionConfiguration$FunctionChannelBindingInitializer.bindSimpleFunctions(FunctionConfiguration.java:377)
    at org.springframework.cloud.stream.function.FunctionConfiguration$FunctionChannelBindingInitializer.bindOrComposeSimpleFunctions(FunctionConfiguration.java:353)
    at org.springframework.cloud.stream.function.FunctionConfiguration$FunctionChannelBindingInitializer.lambda$afterPropertiesSet$0(FunctionConfiguration.java:261)
    at java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948)
    at java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:580)
    at org.springframework.cloud.stream.function.FunctionConfiguration$FunctionChannelBindingInitializer.afterPropertiesSet(FunctionConfiguration.java:251)
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeInitMethods(AbstractAutowireCapableBeanFactory.java:1863)
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1800)
    ... 64 common frames omitted

For me it sounds like we really need to check a bean definition type (not the target object) before registering into the FunctionCatalog.
Also would be great to improve that exception message to let us know which bean is guilty.

Related bug in Spring Cloud Stream, for posterity: spring-cloud/spring-cloud-stream#1794

I've been reading the documentation updates in Spring Cloud Stream. The project is moving away from being a light Spring Integration input/output router, and instead towards enabling pure microservices/functions, which got me to wonder if we are no longer using it philosophically correctly.
Instead of trying to figure out how to exclude the binder infrastructure bean that happens to look like a functional interface from being discovered, it may be preferable to explicitly mark it as "those are not the functional beans you are looking for".
Would Spring Cloud Function or Stream consider adding an annotation like @NonFunctional that would explicitly exclude a  bean from being considered for the function registry?

@elefeint @artembilan
Elena
There are several issues at play here, so I decided to do a little writeup that I hope to turn into a blog. But first let me give you the short one:
The core issue is that in s-c-function we determine function type based on the signatures of the method, class etc., otherwise we’re giving you an option to configure all of it yourself via FunctionRegistration class. In your case the signature of the factory method in question is not a function type (not Supplier, Function or Consumer), however the instance returned by this factory method and the resulting bean is a function (Supplier) and that is the problem which we did not  account for in s-c-function. So regardless of everything else, that is the root of the problem and it needs to be addressed.
Now the long one. . .
Interesting perception - "being a light Spring Integration input/output router. . .”, but I have to disagree, since it never has been about Spring Integration(SI) and/or router. In fact this statement shows part of the problem where SI (framework of choice to support some of the internal requirements of spring-cloud-stream(SCSt), was somehow perceived to be the core of spring-cloud-stream in such way that many perceive SCSt to be an extension to SI. It is not. It has always been about pure microservices. Just read the first sentence of our front page "Spring Cloud Stream is a framework for building highly scalable event-driven microservices connected with shared messaging systems."
If you abstract yourself far enough from knowing the internals of SCSt, you quickly realise that it is really a binding and activation framework. It binds a piece of code provided by the user to broker destinations supported by the provided binder and activates such code according to binder implementation (e.g., message arrival etc). That is pretty much it.
To function or not to function
Let's look at the following two code snippets
Annotation-based:
@SpringBootApplication
@EnableBinding(Processor.class)
public class SampleApplication  {
    @StreamListener(Processor.INPUT)
    @SendTo(Processor.OUTPUT)
    public String uppercase(String value) {
        return value.toUpperCase();
    }
}

Function based:
@SpringBootApplication
public class SampleApplication  {
    @Bean
    public Function<String, String> uppercase() {
        return value -> value.toUpperCase();
    }
}

Both are valid and fully functioning SCSt applications. Both are doing the same thing and both will produce the same result except that in the annotation-based example user has to be aware of SCSt abstractions (i.e., messaging, channels, bindings etc.) while the code has nothing to do with any of these abstractions. That raises a question why? Spring has always been about “you worry about functional requirements and we take care of non-functional (boilerplate)”. So, in he context of SCSt as a framework and its core goals of "binding and activation" we quickly realised that these abstractions are boilerplate and should not be leaked into the user’s code especially in the form of annotations as they contribute to binary dependency of such code on SCSt for no valid reason.
Also, given that the basis for most new frameworks within spring portfolio is spring-boot, think about the spring-boot's core message  - dependency (e.g., JAR) includes auto-configuration which is effectively an opinion on how we (spring) believe things should be, while giving you a way to opt out. So, in this context why do you need to provide so many instructions especially thru annotations (EnableBinding, Processor, StreamListener etc) where one can easily extract/derive the same information (in the context of SCSt ) by simply following some convention. For example, Function bean in the context of SCSt is a Processor. We know that Processor has only one input destination and one output and we know their names so why do we need to explicitly state the known and the obvious? And so on. . . Also keep in mind that while deriving all of that we are still preserving the use of consumer/producer properties and all other configuration options. They still apply here allowing you to configure and reconfigure the same things as you would with the StreamListener.
So with that what I am also saying is that we are starting on our slow journey of moving away from annotation-based programming model and into a more spring-boot aligned opinionated model of clearly documented and intuitive convention with limited out-of-the-box configuration required from the user.
So finally to your case where something in your configuration (outside of your control) fits the category of function, but that is/was not your intention. Valid, interesting and we definitely need to provide a path to opt-out and that is what we’re working on, but I hope I was able to convince you that while you are correct in requesting such option, annotation would not be the right approach here, so I am considering a property.
Please feel free to provide any feedback
Cheers
Oleg

That's a good blog post as is!
I don't need much convincing that less boilerplate is good -- I was looking at Spring Cloud GCP stream binder sample, and half of it consists of annotations that will go away in the new world.

So the fix is in and I believe it actually addresses the issue. Basically with this fix during the default discovery process (process where function definition is not provided) we no longer treat instance of Function, Supplier or Consumer as functional, if its declared type is not actually Function, Supplier or Consumer (see example below):
@EnableAutoConfiguration
public static class SCF_GH_409ConfigurationAsSupplier {

	@Bean
	public Serializable blah() {
		return new Foo();
	}

        private static class Foo implements Supplier<Object>, Serializable {
		. . .
	}
}

So for the case in SCSt where it was firs discovered it will no longer be discovered in the first place.

Also, the above solution is valid IMHO for another reason. If we can't properly discover the actual type of the function (it's input/output argument types) we can't perform any type conversion effectively pushing a problem down he stack instead of addressing it.

Excellent! I backed out the spring.cloud.stream.function.definition workaround, and our build is healthy.
Thank you for your help!


feat: add concurrency option to run and exec

To speed things up, add a concurrency option to aegir run and
aegir exec - this will run commands up to the cocurrency limit
while still taking monorepo sibling dependencies into account.

Not ready for merge yet - this still needs everyMonorepoProject to work out batches.
E.g. when project A has dependents B and C, and C has dependent D, the batches would be:
[A]
[B, C]
[D]

[B, C] should not start to execute until [A] is finished, [D] should not start until [B, C] is finished.
Any circular dependencies should cause an error to be thrown, as now.

@achingbrain take a look at this implementation.
It tracks the number of outstanding dependencies for each project, uses that to determine the next batch.

This takes js-libp2p build time from 1m58s to 41s on my local machine

Great work on this PR, my total time is more than 50% faster
non-parallelized  2:24.22
parallelized 1:11.28

Nice, down to about 30s now.


Create and document community compensation structure

@gusaus, are there other similar community groups that have a compensation structure we can learn from?

@schalkneethling Looks like I haven't been able to circle back to provide detail and show examples. There are some good examples and discussions in this 'pinned' thread in #opensource. The best, most thorough, transparent example I've seen is actually OFC https://blog.opencollective.com/team-set-salaries-ocf/
Considering https://opencollective.foundation/ is not only a key player, but the fiscal host for OpenProducer and Drupal Contribution Group, we're already in this together and the barrier to collaboration and support on their end has been nobody beyond me interested/able to represent the community.
It sounds crazy, but it's kinda true.
The good thing is the Open Startup discussion started by Ruth (an absolute innovator in the Open Collective space), could provide a clear entry for working out the details if our collectives were involved.


Add WASM for better performance.

On hold for now. Turns out the slow part is the canvas api which i would have to use anyways.
I will need to think about this a bit more.


changes six to be from a dependency and not django's

fixes #45

>pip freeze
asgiref==3.2.3
coverage==4.5.4
defusedxml==0.6.0
Django==3.0
djangorestframework==3.10.3
flake8==3.3.0
mccabe==0.6.1
py==1.8.0
pycodestyle==2.3.1
pyflakes==1.5.0
pytest==3.1.3
pytest-cov==2.5.1
pytest-django==3.1.2
pytz==2019.3
six==1.13.0
sqlparse==0.3.0

I've updated tox.ini on this PR to add in the new versions, but I haven't ever configured anything through the tox files, so it might need further adjustment. Just FYI, I ran the test suit with python 3.7 and it worked fine...

Closed by #53


Add Refresh Button

Reload data on page by clicking the refresh button.

I didnt realize that you added refresh on a 60 second interval. Added a button to do it manually anyway

I hope it's calling the same function. >_>

it is

https://github.com/schauhan19/Spectatr/blob/master/app/App.js#L54


rocket page

as per now rocket button on nav bar is not redirecting to any page i would like to make a page and ridirect that button there

I am interested in issue but I don't understand what you want changes.

I am interested in issue but I don't understand what you want changes.

You are participant or mentor

participant

@nitishapiplani the page for that is yet to be added. Closing this.


how do I get an API-Token?

I tried to apply for one twice.
I stated that i wanted to use it to display the stats of my r6 clan.
They never replied.
Idk if they are even handing out keys anymore.

There's currently a backup on key requests. I believe Nader is the only one that can hand out keys, hence the delay. Sorry for the inconvenience :/


"Save", "Save and close", "Save as" won't work with most Gelbooru images

I'm encountering a problem when browsing Gelbooru, where most images won't work with the save buttons in the image window, like "Save" and "Save and Close".
Here's an example of an image that works for me:
http://gelbooru.com/index.php?page=post&s=view&id=3079478
But this one doesn't:
http://gelbooru.com/index.php?page=post&s=view&id=3071898
This might be the the same as Falord's issue #487 but I haven't tried other sites.

What do you mean don't work?
The directory is created but the image not saved, as in issue #487?
If it is a new installation (not an update): prior to this happening, did you even once open the options window and click the "Ok" button? (even without changing anything, it seems some default values are not properly set in the last version)
Could you upload your main.log file of after this happens? I can't reproduce this issue.

The directory is created, but the image is not saved. I have opened the options window since the installation. When I try and save something, Grabber indicates that it works - e.g. I'll get the Saved! message, and Save and Close will close the window - but I'll check the directory and it's just not there.
Here's my main.log.
http://pastebin.com/jRN04ysU
All I did was open Grabber; search for, open, and save the image that works fine; then search for, open, and (try to) save the image that doesn't work.


[Help needed] TCP server - data not in readable format

Ran tcp sever in OSX  (using netcat command, nc -l 31006), started receiving data however its not in readable format.  Can you please help with a suggestion/snippet to receive data in the format you had shown.
TCP SEVER OUTPUT:  @	G?@?hsAbN>????>??Q??????
                                x??׀L@0?u@??DAff>??? >?(Q????}??H?ԱL@&??@???A	K?>??? >?(Q????}??H?ԱL????@??`A?>À??>??Q?????p?@??K??׮@?-A
                                                         ?w>??>??Q???????????L@G?@?A1'>?h???>Q????׀??ـ??ЀL@	X@?"?A

Have you tried the pyserver.py script? I used it to receive data.

It worked with pyserver.py script.  Will try to get it worked on ESP8266.   Thanks.


provide network layout before training

https://groups.google.com/forum/#!topic/neon-users/f0YyWD2yLaE
I called just before mlp.fit() in examples/mnist_mlp.py
print mlp
it errors out:
...
"../neon/neon/layers/layer.py", line 771, in str
self.name, self.nin, self.nout)
AttributeError: 'Linear' object has no attribute 'nin'
call after mlp.fit() works.
But I want to exam the network before the training start to watch for any unexpected numbers; after training it's too late (also considering training typically takes long time).

as in Torch7: print network layout, and each layer's shape
nn.Sequential {
[input -> (1) -> (2) -> (3) -> (4) -> (5) -> (6) -> (7) -> (8) -> (9) -> (10) -> (11) -> output]
(1): nn.SpatialConvolutionMM(1 -> 32, 5x5)
(2): nn.Tanh
(3): nn.SpatialMaxPooling(3,3,3,3)
(4): nn.SpatialConvolutionMM(32 -> 64, 5x5)
(5): nn.Tanh
(6): nn.SpatialMaxPooling(2,2,2,2)
(7): nn.Reshape(256)
(8): nn.Linear(256 -> 200)
(9): nn.Tanh
(10): nn.Linear(200 -> 10)
(11): nn.LogSoftMax
}

Hello,
In that case, the -v verbose flag should meet your needs, as it prints the network layout right before training. Alternatively, you can first initialize the model by calling mlp.initialize(train_set, cost), which will configure all the layer shapes. Then, you can inspect the network via print mlp.
Hanlin

Hi @mingwugmail - I'm going to close this issue. Please let us know if you have any problems with the suggestions Hanlin outlined!


Raspberry pi2 not working for awhile. Power light solid red

Similar/same as this issue.
Pulled out SD card and stick it into another Ubuntu computer.

Connected monitor and saw something is wrong. At the end of the output the green light turns off while red is still solid:
blk_update_request: I/O error, dev mmcblk0, sector 4587523
mmc0: timeout waiting for hadware interrupt.
:
end kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(179,2)

Took away the SD card and looked into it
$ sudo fdisk -l /dev/mmcblk0p1
Disk /dev/mmcblk0p1: 63 MiB, 66060288 bytes, 129024 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disklabel type: dos
Disk identifier: 0x00000000

https://raspberrypi.stackexchange.com/a/40855/58336

Found another mount point on the same SD card. Again, not sure what's wrong.
$ sudo fdisk -l /dev/mmcblk0p2
Disk /dev/mmcblk0p2: 7.4 GiB,<PHONE_NUMBER> bytes, 15433728 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes

One thing to check for the particular SD card: whether it wore out. Ref. wear levelling.

Should open a new ticket for OpenHAB installation etc.


Fixing couple of layout issues with Settings page

Converted Language and Keyboard layout to combox boxes
Minor layout changes for network and bluetooth settings page.

Hi @rachitb777, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

looks good! thx Rachit.


pip install eagle: python3 or python2?

I've got logs below while installing eagle:
$ pip install eagle
Collecting eagle
  Using cached eagle-0.2.tar.gz (417 kB)
    ERROR: Command errored out with exit status 1:
     command: /home/andy/anaconda3/envs/pytorch/bin/python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-vw3svb83/eagle_ca7b4cba24b942c19a8c35b503fdfc19/setup.py'"'"'; __file__='"'"'/tmp/pip-install-vw3svb83/eagle_ca7b4cba24b942c19a8c35b503fdfc19/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-_uca8aqj
         cwd: /tmp/pip-install-vw3svb83/eagle_ca7b4cba24b942c19a8c35b503fdfc19/
    Complete output (8 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-vw3svb83/eagle_ca7b4cba24b942c19a8c35b503fdfc19/setup.py", line 3, in <module>
        import ez_setup
      File "/tmp/pip-install-vw3svb83/eagle_ca7b4cba24b942c19a8c35b503fdfc19/ez_setup/__init__.py", line 176
        print "Setuptools version",version,"or greater has been installed."
                                 ^
    SyntaxError: Missing parentheses in call to 'print'. Did you mean print("Setuptools version",version,"or greater has been installed.")?
    ----------------------------------------
WARNING: Discarding https://files.pythonhosted.org/packages/dd/5c/4318a37b78339ab35e8340d51ddcb76f8005261dffcd5d1e9ab7cec0b5cd/eagle-0.2.tar.gz#sha256=7af5dffd0f1f2b5c46dcc19f5b5a47e96dc3ce387f1de05199df249e8d2bed59 (from https://pypi.org/simple/eagle/). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement eagle (from versions: 0.2)
ERROR: No matching distribution found for eagle

It shows that there's a print syntax error in init.py line 176
The syntax of print  "Setuptools version",version,"or greater has been installed." is compatible for python2 but not python3.
Please help to download the package.

git clone https://github.com/SamsungLabs/eagle

an eagle folder created, then
pip install eagle

doesn't work, still show the same log like above.

pip install eagle-main.zip
Successfully installed Eagle-1.1.0 astunparse-1.6.3 cachetools-4.2.4 charset-normalizer-2.0.7 clang-5.0 flatbuffers-1.12 gast-0.4.0 google-auth-2.3.2 google-auth-oauthlib-0.4.6 idna-3.3 keras-2.6.0 oauthlib-3.1.1 pyasn1-0.4.8 pyasn1-modules-0.2.8 pyyaml-6.0 requests-2.26.0 requests-oauthlib-1.3.0 rsa-4.7.2 six-1.15.0 tensorboard-2.7.0 tensorboard-data-server-0.6.1 tensorboard-plugin-wit-1.8.0 tensorflow-2.6.0 tensorflow-estimator-2.7.0 tqdm-4.62.3 typing-extensions-<IP_ADDRESS> urllib3-1.26.7 wrapt-1.12.1
It works but the default Tensorflow version is the latest version instead of you requirement(>=1.13,<2)
Will have to uninstall Tensorflow and reinstall.

Yes, sorry for that. We actually have a fix for TF>2 waiting to be pushed to this repo. Maybe some time soon I'll find some time to do that.
Anyway, since the problem seem to have been resolved I'm closing the issue. Feel free to reopen if needed.


Invalid set of options values are 5, 10, 20.

How can disable checking for the number of rows in a table?
Warning: Material-UI: You have provided an out-of-range value 19 for the select component.
Consider providing a value that matches one of the available options or ''.
The available values are 5, 10, 20.

@Kuzmich100kM could you update the issue to follow our issue template? What's the issue? How do we reproduce?

I need the table to take up all the available height, so it can have any number of rows. This throws an error in the console. How can I disable checking for a multiplicity of 5?

Can you provide the code for rendering this component? It would help us easily debug what is the issue. You can start by editing this codesandbox link: https://codesandbox.io/s/t0xjx

I created an example, but now I noticed that I am using a "material-table" library.
Sorry, if I was wrong with the topic :-)
https://codesandbox.io/s/blissful-hamilton-9mcd2?file=/demo.js

I see, feel free to use the same codesandbox to open issue on their github repository then :) Closing this one


Automated on/off boarding emails for users

The idea is that managing people in PEAKS should notify a department's existing on/off boarding process, to avoid duplicate work.
Simple version: An admin setting with an optional "Person Notification Email".  If it exists, whenever a person is created or deleted from a team, we send an email to that email address with the person's info.  i.e., "John Smith was created in PEAKS team XYZ.  Details: ...".   Admins would put their service-now email most likely and it would kick off their individual on/off boarding processes.
Complex version: Same as above but we add a toggle so the person creator can determine if they want the onboarding email sent.

So... to do this:

[ ] Add a column to the team table to hold the email
[ ] Add a page to edit the email value
[ ] Create a new table to hold the notification. Possibly expand the Notification table
[ ] Create a new email template
[ ] Create a new email service
[ ] Update and deploy email job
[ ] Everywhere a person can be added, deleted, or activated, call a new service to write the pending notification. Note!!! None of the imports are doing this at the moment

Is this going to work at all???? People already need to have a Kerberos to be added to PEAKS. For transfers from one department to another, that's fine, but on-boarding a brand new employee can not begin in PEAKS. On the other end, I'm not sure deleting/marking inactive is the first thing someone wants to do, so probably not going to be were they kick off that process with SSO.

I think this is all done. Just testing tonight to make sure I deployed the job.

Looked in the DB. Is done


encoding_rs is broken

https://github.com/rust-lang/packed_simd/issues/359
https://github.com/rust-lang/packed_simd/issues/360

encoding_rs 0.8.34 addresses this.


Make kMaxFramesToCapture constant passed into AGCT a configurable profiler parameter

Hi,
I've been using honest profiler pretty intensively for the past few weeks and I must say it's a pretty awesome tool, great job! I'm using the FlameGraphDumperApplication to generate a log file that can be fed to FlameGraph.pl, which generates the nice searchable SVGs for me. However, I noticed something weird, like in the picture below:

You can see that I have calls to DaemonCommandExecution.proceed, and those are visible at different "heights" in the graph. But this is just plain wrong. They should be all found at the same level, which tells be that the call stack was truncated.
At first, I blamed the export tool and tried to debug it, but I can see that if it was parsing the file wrong, it would probably end up throwing an exception. Nevertheless, I have attached a file that is the actual result of a profile session, and causes the problem.
hp.log.zip
It seems to happen very frequently when stack frames are big, which is, for various reasons, unfortunately my case. For simple cases I can "reconstruct" the missing parts of stack frames, but it's time consuming and error prone.
Any idea what could be going on?

(To be clear, the issue is visible on the left and right of this picture, where those bars actually "belong" to the middle)

We currently have a default value to the max number of frames in the stack which is used in the call to AGCT here:
	  (*asgct)(&trace, kMaxFramesToCapture, context);

The value is defined here:
// Things that should probably be user-configurable
// Maximum number of frames to store from the stack traces sampled.
static const int kMaxFramesToCapture = 128;

As the comment suggests, it should probably be configurable...

Awesome, that was it! I rebuilt the agent and now I have full call stacks, thanks!

Cool, I'll re-purpose this ticket as a reminder to fix this

@melix thanks for reporting, and if you have the time and initiative to crack this issue in a PR that would be awesome :-)

I can change a hardcoded constant in a C file. Going beyond is going to involve a bit more, since I haven't touched such code for what, 15 years? :D

Hi @nitsanw, @melix
I currently have the same issue with truncated traces, so I decided to make kMaxFramesToCapture configurable parameter. I've implemented it naively and tested locally, but not sure if it's ready for PR since C++ is not my strong suit. Could you please take a brief look into my changes https://github.com/RichardWarburton/honest-profiler/compare/master...ikavalio:master just to make sure I'm heading in the right direction.

Closing, fixed by #151


Replacing robotics specialization with NwU's "Modern Robotics" from UPenn's "Robotics"

I was reading Hackers News yesterday. A link to a free textbook called Modern Robotics was posted. Someone in the comments mentioned a corresponding Coursera specialization from Northwestern University. The currently suggested specialization is from the University of Pennsylvania. It seems to me that the specialization from Northwestern University contains courses that are more contemporary and higher rated than the specialization from the University of Pennsylvania. I propose that the specialization from Northwestern University be recommended instead of the one from the University of Pennsylvania.
I haven't completed either specialization and I haven't read the textbook. I can provide a more informed suggestion once I have.
OSSU is the best source I've found for self-studying CS. Thanks to everyone who has contributed.

Looking forward to the answer on this.  I'm still working my way through the Core CS section but when I do get to the Advanced Applications robotics is likely to be the one (or one of the ones) that I go with.


Change Request: Guidelines for C99 - design.md

Naming conventions.

Can we use single leading underscore to indicate non-public API?
Missing: Do use upper-case for macros, constants, and enumerators.
The way that we organize the variables inside of the structures shall NOT be mandatory. Other factors may be more relevant than the type or the alphabetic order.
No pascal-casing in the enum. Let’s be consistent with snake-casing.

Client interface.

The az_shortname_create_client() function predict that this function will allocate and return the control block of the client. Instead, it shall receive the memory already allocated and just initialize it. This function shall return a result instead of a pointer.
The API should be symmetric, it means that if we have create, we should have destroy. For open, close, and for init, deinit.
What is the difference between begin function and async?
Everything particular to HTTP should not be in the guidelines.
We cannot use libuv.

Pagination.

Why we have it in the guidelines?

Error handling.

Missing az_result format.

Memory management.

The az_iot_create_client example is not correct, it should return az_result, and the name should init because we are not creating the client.
Missing a better definition of the memory ownership, for example, if a function needs to keep a provided buffer for more time that its execution, how the function inform the caller that it is time to release the memory?

Object model.

Init function shall return az_result.
internal is a reserved word. We are using double definition of the structure, one private with the parameters, and one public with a dummy size.
The example az_catherding_herb_init_with_cats is not correct.

Missing test guidelines.

The guidelines shall make it clear that no code shall be pushed without UT and E2E test, no exceptions.
What is the minimum code coverage or any other parameter that we can reinforce that no code is pushed without test.
To be able to write define the UT, all components shall have a list of requirements.

Missing sample guidelines.

All functionalities shall have samples to easily show developers how to use it.

Missing code review guidelines.

The guidelines shall predict the minimum number of reviews over any code before go to master.

Internal is a reserved word by whom? I'm not seeing it in the C spec

#721 for changing internal

In the case of the catherding_herd_init functions they can not fail, and so don't return az_result.

#722  for returning az_result instead of az_error


Remove need for HDF5-1.6 API being defined

There are a few HDF5 api version-related symbols or requirements in the current netCDF source code and build system.  There is an explicit setting of the H5_USE_16_API define which triggers some defines in the hdf5 H5Version.h include file.  There is also a check for the H5Literate_Macro and if it isn't found, then a warning about building hdf5 with the --with-default-api-version=v18 define.
There is also a requirement that the HDF5 version be at least 1.8.10 which means that unless hdf5 is built with deprecated symbols deleted, the symbols from v16 and v18 exist -- typically by appending a 1 to the function name.
This PR removes these checks for a specific API version and explicitly calls out all deprecated functions that are being used by appending the 1 to the function name.
Later, we can modify the calling arguments wehre needed to be able to convert from the deprecated function to the currently recommended function (some of which are already being used (for example there is a mix of H5Gopen1 and H5Gopen2 function calls throughout the code base.
The motivation for this change is to make it easier to use netCDF with installed versions of HDF5 which may not have been built with the v18 API.  There is also a need to be able to compile with recent development versions of HDF5 which need some patching to enable v16 and v18 api macros.  One other potential issue is that I'm not sure how safe it is to redefine an HDF5 define (H5_USE_16_API) which was not defined during the build; probably safe, but just seems dangerous.

This has been tested with HDF5-1.10.7, HDF5-1.12.0, and the HDF5 development version from the hpc-io github repository for testing the use of VOL [async, cache, provenance] with netCDF/HDF5.

sorry for the churn.  I guess I missed a couple in my 1.8.21 builds...

Thanks, and no worry! It is passing all the tests, which looks great, but given the number of files it touches I'm just reviewing it now and should get it merged (I suspect) in fairly short order. Thanks, as always!

Thank you @gsjaardema as always!!

You're welcome.  Glad I can contribute


[Ember-Migration] Created Scroll To Top Button

Resolves Issue
Closes #334
What is the change?

Created scroll to top button

Is Development Tested?

[x] Yes

Before / After Change Screenshots
Web View

Mobile View

Note for the Reviewer

CI is failing due to package-lock.json being not found as we are using yarn.lock here. Therefore, need to update the Workflows or create a new workflow. Issue for the same - #327
Tests will be written in a PR. It is being tracked in the issue #281

LGTM slightly_smiling_face👍🏻

Thanks @heyrandhir  😄

Good work on adding this Rohan as this is a great UX improvement task. This will help users a lot.

Thank you so much @harshith-venkatesh


Support Asynchronous client

Futures and other related crates have been making great progress and we should support an asynchronous client along with the synchronous one (The current is synchronous).

Async requests has been fully integrated. Closing


Workaround to allow forking after grpc_init()

Note: This is NOT a proper fix but rather a quick&dirty workaround to allow process forking after grpc_init() has been called. Hopefully there will be a proper fix soon.
The current problem with GRPC as described in many places (e.g. #17695, #13412, #11814) is that once grpc_init() has been called in a process, then this process should not be forked. There is a somewhat partial forking support but only if no connection has been established yet.
In my case I have an extension for PHP which is running under Apache. When Apache runs in prefork mode, it dynamically forks child processes to handle the incoming load. This means I need to be able to initialize a new GRPC instance in forked processes even when the parent process already has an active GRPC connection. Since this use case is not yet supported I made a workaround.
Essentially what this workaround does is to reset the internal GRPC state just enough so that a new and clean GRPC instance can be created in a forked child process.
My code change is based on the 1.24.3 release and can be found here.
There are several functions that are called internally to collect active sockets, pthread_once states, mutexes, and conditional variables:
void grpc_add_socket_fd(int fd);
void grpc_add_once_init(pthread_once_t *once);
void grpc_add_mutex(pthread_mutex_t *mutex);
void grpc_remove_mutex(pthread_mutex_t *mutex);
void grpc_add_cond(pthread_cond_t *cond);
void grpc_remove_cond(pthread_cond_t *cond);

Then there are new globals which hold a pointer to critical internal GRPC states:
void **fork_g_event_engine;
void **fork_g_resolver_registry_state;
...

From the user perspective, all that is needed is to call the newly added function grpc_clean_after_fork() before initializing GRPC. Be sure to call it ONLY ONCE per process.
grpc_clean_after_fork() closes all inherited sockets, resets mutexes, conditional variables, pthread_once states, and other internal states that are left-overs from the parent process. Afterwards GRPC will initialize properly in every forked child.
I'm just sharing my findings here in case somebody else is in a similar situation and wants to give it a try. However, while this workaround works for me, it might not work in other cases.

Coming from #13412, I assume you have tried to enable the following php.ini settings?
grpc.enable_fork_support = 1
grpc.poll_strategy = epoll1

@stanley-cheung thanks for your suggestion but I'm not using the GRPC PHP extension. I have my own PHP extension that starts a long-running thread under the webserver process (e.g. Apache). This thread maintains a GRPC connection that lives beyond any PHP script execution.
The forking is done by Apache, not by PHP. So unfortunately the php.ini settings won't help.

It sounds like you are using PHP but not using gRPC PHP, so I'm not very familiar with your specific use case - but gRPC Python and PHP clients support fork after connections have been established and have done so since v1.15 (for Python). See the release notes and this comment broadly describing the API from core's perspective that allows this to work: the child process after fork is responsible for triggering grpc_shutdown in core, and the specifics of how to accomplish that in core differ by language (gRPC Python includes its own fork handlers to make this automatic for users). It sounds roughly similar to the approach you describe in your code, with the benefit that it doesn't require hacking gRPC core to add (much) additional global state beyond a boolean that says if a fork has occurred.
It looks like https://github.com/grpc/grpc/blob/master/doc/fork_support.md is in desperate need of an update, but there are numerous now-closed issues centered around this issue before it was fixed in v1.15: please let me know if you have any other questions about this and I will try to get a proper accounting of the current state of fork support into that doc.

@ericgribkoff that's correct. I'm using PHP but not gRPC PHP. To be more specific, I have a shared library that communicates with a server via gRPC and that library is used by a PHP extension. The same library is also used by other languages, so it's not really PHP-specific.
I had tried using grpc_shutdown in the child process but encountered problems with it, since it seems to expect certain gRPC threads to be running which isn't the case after a fork. Basically these parts had issues and caused a crash:

Executor::SetThreading
stop_threads
grpc_timer_manager_shutdown

I read that fork handlers won't run if multiple threads are calling into gRPC. I do have two stubs in two threads sharing one channel, so maybe that's a reason too why grpc_shutdown doesn't work in my case.
Another point that comes to mind is, what if a fork happens while a thread is calling into gRPC? Mutexes can be in a locked state, so destroying them, like it's done in grpc_shutdown will result in undefined behavior.

@dschnabel If you are trying to write a gRPC C++ program that supports fork, you will have to obey the same contract as our gRPC Python and gRPC PHP libraries require: namely, no threads can be calling into gRPC core code when the fork occurs, and all gRPC core owned resources must be closed in the child process after fork before the child re-initializes gRPC. This is non-trivial but handled transparently for users by our PHP and Python layers: if you are attempting to re-create the same functionally in your own C++ code I would strongly suggest instead re-architecting things so that your C++ process does not fork. C++ does not have the same constraints that compel PHP/Python users to fork. If your library is not PHP-specific as you comment, you may want to consider moving it out of the PHP process entirely.
Below I discuss more details about the internal fork support in core, but please bear in mind that this is a complicated topic and is a core API intended for gRPC language implementers, not user applications, to consider, and was very specific to PHP and Python and may in some regards not even be compatible with our C++ wrapping layer.
The approach you mention in your original comment:

grpc_clean_after_fork() closes all inherited sockets, resets mutexes, conditional variables, pthread_once states, and other internal states that are left-overs from the parent process.

Does not scale in a development sense, as internal state in the gRPC library is (a) vast and (b) constantly changing.

I had tried using grpc_shutdown in the child process but encountered problems with it, since it seems to expect certain gRPC threads to be running which isn't the case after a fork

You can't just directly call grpc_shutdown in the child process, as the core API is rather more complicated than that. grpc_init/grpc_shutdown are basically increment/decrement functions that only have effect when the count goes from 0 to 1 or from 1 to 0 - the first case triggers gRPC core initialization, and the last cleans everything up but is dependent on all references to core-allocated objects, such as channels, having actually been properly released by the user application. gRPC PHP, and Python layers manage this complexity automatically for the user.

Another point that comes to mind is, what if a fork happens while a thread is calling into gRPC?

Fork + multiple threads do not mix well. We avoid this in core by having pthread_atfork handlers that shutdown all core-created threads before the fork occur. As you point out, if multiple user threads are calling into gRPC at the time of fork then our handlers will detect this and not run - there is nothing gRPC core can do to make user threads safe during a fork, with multiple failure conditions potentially arising in gRPC core's own state. But in general it's a bad idea for any application to call fork if it knows it's running multiple threads (there are some caveats to this for Python but the gRPC Python layer handles these itself).
Keep in mind that all of the core fork-specific behaviors required the GRPC_POSIX_FORK_ALLOW_PTHREAD_ATFORK to be enabled at compile time, and are controlled further at runtime by the GRPC_ENABLE_FORK_SUPPORT environment variable.

I appreciate the insights you are giving into the gRPC core. These make sense but only allow fork support to a certain extent. Based on your post I think the main constraint right now is that an application needs to know in the parent process when a fork is called. Only then can it prevent calls into gRPC until forking is done and the child process can do a proper cleanup using the mechanism you described.
In my case, however, I have no way of knowing in the parent if a fork is about to happen. I can only indirectly determine this in the child by noticing that the PID has changed. But then it's already too late since the fork has happened. And this is not a matter of re-achitecting my application.
I will try to describe in more detail my use case as I think it's not that uncommon:

My shared library (let's call it profiling.so) is written in C++ and offers an API to applications to collect/process/send profiling events and metrics.
during initialization profiling.so creates a gRPC channel to an endpoint and two worker threads that process incoming events and metrics. These worker threads each have their own stub on the gRPC channel.
One application that uses profiling.so is a PHP extention (let's call it php-prof.so). In MINIT it initializes profiling.so so that the channel and worker threads are created once. In MSHUTDOWN it destroys the channel and worker threads.
In RINIT/RSHUTDOWN (PHP request cycle) php-prof.so creates events and metrics and puts them on a queue which gets processed and sent off by the two worker threads started by profiling.so. This happens asynchronously meaning that the PHP request cycle is not affected by the gRPC calls happening in the worker threads.

So far, so good. No forking is involved up to this point. Forking comes into play next:

PHP runs in Apache as a webserver module.
Apache is configured to run in prefork mode. In this mode, the Apache parent process forks and manages Apache child processes depending on the request load that's coming in.
Once forked, there's only one thread running (the apache main thread) in the child process. The first time RINIT is called, php-prof.so detects that the PID has changed and re-initializes profiling.so.
After re-initialization this child process has a new gRPC channel and again two worker threads.

To summarize my setup:
Apache -> PHP module -> PHP extension (php-prof.so) -> C++ library (profiling.so)
Forking happens in Apache and is completely transparent to PHP. Hence profiling.so cannot prevent calls into gRPC until forking is done.
I hope it becomes clear now that it's not a matter of re-achitecting my application. I need to be able to clean up gRPC given any internal gRPC state as Apache can fork at any given time. The workaround I provided in the original post seems to be the solution to this problem. I know it's hacky but wanted to share it for people who face similar dilemmas as I do.

Thanks for the greater detail about your architecture with Apache -> PHP extension -> C++ library -> gRPC. It is indeed (very) difficult to combine fork and multithreading - so difficult that I struggled to find any recommendations for doing so when working on this support for gRPC Python, versus many many recommendations to always avoid this situation. For gRPC PHP and Python it was decided that it was, unfortunately, untenable to disallow fork for our users due to constraints with the language, and so rather a lot of effort was put into making this work.
If you can't change your library structure to move the C++ library out of the PHP process (which you probably can't - in general, something like gRPC would allow the PHP process to communicate with the C++ "library" running in a separate process, but your profiling.so name suggests that latency/etc may preclude this) then you essentially have to reimplement our PHP/Python fork handlers in your C++ library, following the contract with core that your fork handlers will release all core resources in the child post-fork. How feasible this is really depends on how your C++ library is using gRPC: you can see how I did it for gRPC Python in https://github.com/grpc/grpc/pull/16264. It's not trivial, but I would argue that this approach (while functionally ~equivalent to the patch you posted originally) is significantly more maintainable as gRPC development continues to move forward. That said, the API with core is really intended for gRPC language implementers and not consumers of the C++ surface layer, so there may be significant obstacles with integrating this into your library.
(If exploring this approach, also keep in mind that core must be compiled with the GRPC_POSIX_FORK_ALLOW_PTHREAD_ATFORK macro defined and running with the GRPC_ENABLE_FORK_SUPPORT environment variable set.)

Forking happens in Apache and is completely transparent to PHP. Hence profiling.so cannot prevent calls into gRPC until forking is done.

This, fortunately, should not actually be the case. profiling.so must register its own fork handlers, which will allow it to pause calls into gRPC prior to the fork, and properly resume/clean-up post-fork in the parent and child. This is how gRPC Python does things (via Cython) in the PR linked above, and how PHP also handles cleanup.

This, fortunately, should not actually be the case. profiling.so must register its own fork handlers, which will allow it to pause calls into gRPC prior to the fork, and properly resume/clean-up post-fork in the parent and child.

I see that in the PR #16264 you used pthread_atfork() to register your own fork handlers. I had briefly looked into using that as well but didn't persue it further because I thought it was a dead end. But if that's what you used in python and php, maybe that's the way to go. I will give it a try. Thanks for this great tip @ericgribkoff !

Just wanted to note that we've made great experiences with the original workaround I posted bove. Our application which uses the patched version of gRPC has been deployed on over 16,000 hosts, many of them running in a forked environment. And we've heard of no issues so far.
Since the suggested method of using fork handlers did not work for me, consider my workaround here as an alternative way of allowing fork after grpc_init().


Download artist profile description and their twitter url to a text file

Is it possible to store the artist's profile description and the url to their twitter account within a text file?

That might fall out of the scope of the program. I don't think the API provides that information and would require an html scrape. There are also artists who, while they have a twitter account, may not link them with their pixiv account.
Storing metadata into a text file, however, I feel would be a useful feature.


feat: Add additional fields to UIOptions

Description
Added additionalFields - an array of additionalField's - to the UI Options. Each additionalField requires a name and a field. The developer can also add the label, description, helpLink, and required properties to configure the label; otherwise, they can choose to label the component themselves.
interface AdditionalField {
  name: string;
  field: FieldWidget;
  label?: string | false;
  description?: string;
  helpLink?: string;
  required?: boolean;
}

Order
Additional fields can be included in the UI Option order override by adding the additional field's name to the order array. Note, the additional field's name can not exist as a property name in the schema.
const uiOptions = {
  additionalFields: [{ name: 'additionalField', field: () =>
Additional Field
, label: 'Additional Field Label' }],
  order: ['schemaField1', 'schemaField2', '*', 'additionalField']
}

Field sets
Additional fields can also be included in field sets by adding the additional field's name to the fields array in the field set.
const uiOptions = {
  additionalFields: [{ name: 'additionalField', field: () =>
Additional Field
, label: 'Additional Field Label' }],
  fieldset: [{ titile: 'field set 1', fields: ['schema field 1', 'additionalField', '*' }]
}

Task Item
Closes #4208

Coverage increased (+0.08%) to 55.685% when pulling c70e88aebdf41c223ccbde88872602ff4ec67a8b on tdurnford:feature/additional-fields into d361ddbb2c266a1a076cd45506388837be66ad33 on microsoft:main.

This isn't directly related to this PR, but I'm curious how fieldsets and order work together?

This isn't directly related to this PR, but I'm curious how fieldsets and order work together?

Currently, the the ui order take priority over the order provided in the fieldset. Not sure if I like that behavior. Thoughts?


EDU-1430 Multitrack on mobile

Closes https://educandu.atlassian.net/browse/EDU-1430

While testing I realized that on mobile (small screen, not all touch devices) we do hide the volume control(s) due to lack of space probably. I think that's what they might have meant in the ticket 🤔

While testing I realized that on mobile (small screen, not all touch devices) we do hide the volume control(s) due to lack of space probably. I think that's what they might have meant in the ticket 🤔
Otherwise the timeline works super smoothly! 😎 👏

Scratch that about the volume, all good!


Update TableTestBase tests to run with formats v1 and v2

This is a work in progress and includes commits from #913. Once that's merged, I'll rebase.

Let me go through this now.

+1, ran the tests locally.


License

Please add license file to enable documentation:
https://pkg.go.dev/github.com/shellfly/algo/algs4

workaround:
https://godocs.io/github.com/shellfly/algo

Thanks I just added the license and it works now. The documentations are not well written for this repo.😷

thank you!


help with no pdf creation

Hi,
I'm completely new to grav, and I'm having trouble get your plugin to work.
I installed plugin via gpm, dependencies (amd64 one) via composer, copied user/plugins/snappygrav/snappygrav.yaml into user/config/plugins/snappygrav.yaml and set correct path to wk_path. I have also set Absolute URLs to true.
I'm using Learn2 theme (added to clean install, no skeleton), and I've replaced code in github_link.html.twig with your code. PDF icon appears, but clicking it complains about non-existing templates dir - user/plugins/snappygrav/templates. I've created the dir, but clicking PDF icon just brings me back to homepage.
Any idea where to start troubleshooting? Thank you in advance.

Hi,
I too am new to GRAV, about four months ...
I built the plugin SnappyGrav for use with Blog Site skeleton & Antimatter theme and I realize now not to have written it in the README, I apologize for that ...
But I really like the idea that you had to use it with RTFM Site skeleton & Learn2 theme.
I recreated on my PC that you are getting the error and I hope to resolve it as soon as possible.
Meanwhile, thanks to you.

Hi,
I made the appropriate changes to correct the error returned by you.
You can find the new version, to download, to the following address: https://github.com/iusvar/grav-plugin-snappygrav/archive/master.zip
Now it should work, I close this issue.
Thanks for your feedback.


Importing projects from older spaces to Integration Studio 8.0.0

When exporting a project created by an older Integration Studio version to Integration Studio 8.0.0, it is necessary to create a new project first.
Need to add this content to 'Importing a project' page.

This has been labelled as a bug because this is important information that is being missed


Object oriented interface for Unirest (e.g. new UnirestClient()...)

From #138
It would be useful if Unirest could provide an object oriented interface for those who wish to use it e.g. new UnirestClient() or similar.
Then one could create this object and shut it down, in isolation from other or future Unirest clients.
Unirest.XX static methods could still exist for those wishing to use them, using a UnirestClient a singleton. The Options class would also need to be pulled out of static-land.
My use case is a long-running application, and does not use Unirest all the time, so it's threads can be shut down in between.
This would also solve #127
Side-Note: If this feature is implemented, I also think that the async methods should not be able to be accessed via Unirest's static classes - they should use this via an Object, so it is more obvious that they are also responsible to shut the object down. This however would break backward compatibility, so IMHO the feature should be deprecated and removed in a future version.

Closed as part of the initial re-design / refactoring


Fix (?) PropType for node.type.

Issue: Console errors about node.type expecting an object and receiving a function.
What I did
Updated the propTypes for Props to expect node.type as a node, not an object.
Unsure whether this handles all cases; a maintainer should verify.

Thank you for this PR @adamdonahue 🙇

My pleasure!
Sent from my iPhone

On Jun 9, 2017, at 5:48 PM, Norbert de Langen<EMAIL_ADDRESS>wrote:
Thank you for this PR @adamdonahue 🙇
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@thomasbertet Good catch.  I figured I hadn't tested all the code paths.


flux: remove references to flux-hwloc

Problem: The flux-hwloc(1) command is being removed by
flux-framework/flux-core#4262.
Replace uses of flux-hwloc with flux resource list.

Agreed. @ryanday36 should have the final approval on this one. Thanks!


Remove unused variable

Testing helm lint

@alex-slynko I'm Heather with Linux Foundation support and I'm looking into your EasyCLA issue. We've made an update and I'd like you to try your authorization again. Please choose "CFF Migration" as your company and it should recognize now that you've been whitelisted. Let me know here if it doesn't work. thanks!

It works now. Thanks!


[Tableau] - default db name cannot be an empty string

So far a default database name in an empty string. Tableau requires non-empty db names to populate UI. We should pick some non-empty, default db name and implement support for that.

Moreover we should extra support following related queries:
DROP TABLE `#Tableau_01_sid_00000001_1_Connect`
SELECT CONNECTION_ID()
show variables like 'collation_server'
SELECT `table_name`, `column_name`\nFROM `information_schema`.`columns`\nWHERE `data_type`='enum' AND `table_schema`=''
SHOW DATABASES LIKE '%'


SignService server: error while signing "The application has failed to start because its side-by-side configuration is incorrect"

While trying to update our SignService instance to the most recent master branch code per https://github.com/dotnet/SignService/issues/60#issuecomment-489245289, ran into the following error running the latest code:
The application has failed to start because its side-by-side configuration is incorrect. Please see the application event log or use the command-line sxstrace.exe tool for more detailSystem.ComponentModel.Win32Exception (14001): The application has failed to start because its side-by-side configuration is incorrect. Please see the application event log or use the command-line sxstrace.exe tool for more detailat SignService.Utils.Kernel32.ActivationContext..ctor(String assemblyName) in d:\a\1\s\src\SignService\Utils\Kernel32.cs:line 77at SignService.AzureSignToolSignService.RunSignTool(AuthenticodeKeyVaultSigner signer, String file, String description, String descriptionUrl) in d:\a\1\s\src\SignService\SigningTools\AzureSignToolSignService.cs:line 105
I deployed to our Azure Website that we've been using to host the SignService instance using an artifact deployment via the https://github.com/dotnet/SignService/blob/master/azure-pipelines.server.yml file in this repository.
Is this a configuration issue with my specific deployment? Or an issue with the latest code?

Are you hosting this on App Services or a VM? How are you building it? I have 1.1.15 running without incident so far on App Services.

App services - built using your pipelines script on Azure pipelines. Deploying the artifact created from that script directly to the App Service
Sent from my iPhone

On May 4, 2019, at 1:33 PM, Oren Novotny<EMAIL_ADDRESS>wrote:
Are you hosting this on App Services or a VM? How are you building it? I have 1.1.15 running without incident so far on App Services.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That's quite unusual, I woudln't expect that. Can you try to redeploy? Also, verify that in the deployment zip, that the various dll/manifest files are in the tools dir

Will do - I’ll check
Sent from my iPhone

On May 4, 2019, at 2:15 PM, Oren Novotny<EMAIL_ADDRESS>wrote:
That's quite unusual, I woudln't expect that. Can you try to redeploy? Also, verify that in the deployment zip, that the various dll/manifest files are in the tools dir
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

x86

x64

Does this look right? Based on the contents of the .zip

Additional issue at startup:
2019-05-04 22:39:09.099 +00:00 [Critical] Microsoft.AspNetCore.Hosting.Internal.WebHost: Hosting startup assembly exceptionSystem.InvalidOperationException: Startup assembly StartupBootstrapper failed to execute. See the inner exception for more details. ---> System.IO.FileNotFoundException: Could not load file or assembly 'StartupBootstrapper, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.at System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codeBase, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, IntPtr ptrLoadContextBinder)at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, IntPtr ptrLoadContextBinder)at System.Reflection.Assembly.Load(AssemblyName assemblyRef)at Microsoft.AspNetCore.Hosting.WebHostBuilder.BuildCommonServices(AggregateException& hostingStartupErrors)--- End of inner exception stack trace ---

Wonder if the issue is that Environment.GetEnvironmentVariable("REGION_NAME")) isn't returning a value for some reason.

Nope, Azure Web Console revealed that I have both REGION_NAME2 and HOME_EXPANDED defined

Resolved the startup issue by following some of the advice here https://github.com/aspnet/AspNetCore/issues/5919

Still having the SXS failure though even after a re-deployment and fixing the .NET Core 2.2 extensions on the Azure App Services instance.

Could the issue be, per the previous screenshots, is that the necessary DLLs and signservice.exe were never copied off of the Azure Pipelines build agent and onto the machine. I see the manifest files, but none of the DLLs themselves?

although it looks like that's done intentionally here:
https://github.com/dotnet/SignService/blob/c5b1c3de1f61a3b849bcad7dd8e1ae2ab9bdca77/src/SignService/SignService.csproj#L29-L60

Validated the contents via the build artifacts I downloaded from Azure Pipelines - all of the DLLs referenced in the manifests are there.

Got everything working on my end finally - I think the issue was how we're doing the Azure App Service deployment from Azure Pipelines. I'd recommend we take your advice from the Twitter thread we had earlier and add that to the development documentation:
https://twitter.com/onovotny/status/1125771206529961985

On the deployment, I always configure a slot and ensure the right settings are pinned to the prod slot. In the tasks, I stop the deployment slot, deploy, start, then do a slot swap. Makes sure nothing is in use.

I'm certain my issue was caused by a dirty deployment. But it's happily code-signing away now.

Apologies for playing necromancer here, but I've just deployed SignService to an App Service (on a B1 instance) and I'm seeing exactly the same problem. I saw the advice about using a deployment slot but it seems B1 instances don't support deployment slots, so I scaled up to S1 and did the following:

Created a slot named "staging" (copying the settings from production)
Stopped the "staging" slot
Deployed (using Azure Pipelines)
Started the "staging" slot
Swapped "staging" with "production"

I still can't get it to sign anything at all - it's consistently failing with the same "Sxs config is incorrect" error - and I'm beginning to lose the will to live 😆
Help me @clairernovotny, you're my only hope!


Check whether there’s an alternative for people who cannot see maps

Summary
Some websites contain interactive maps.
It’s very hard to make a map itself accessible to people who have visual impairments using some sorts of assistive technology. At the least, you should provide an alternative for users who cannot use the map.
Deadline
20/02/2021
What needs to be done?
It’s very hard to make a map itself accessible to people who have visual impairments using some sorts of assistive technology. At the least, you should provide an alternative for users who cannot use the map.
The Pension Wise website does this well. For instance, when you search for your nearest contact centre, you’re presented not only with a map you could use to navigate, but also with a text address any user could access.
Check any maps on your website to see whether you’re providing alternative routes for users who cannot use the map.
Who can help?

Made issue #369 to create an accessibility checklist that can be used for the MVP and going forward. Closing this along with other accessibility checklist items for now.


fix: compilation with default-features = false

Fix compilation with default-features = false, meaning without the k256 feature, also testing it in CI.
I took the opportunity to also improve the CI with more checks, I adapted it from what we have at alloy-rs/core.

Yes, these are intentional.

Binaries are also not compiled every time, but downloaded from GH releases, see taiki-e/install-action
I don't think having multiple action repos matters
cargo test is perfectly fine on a small repo with only unit tests; same for ungrouped actions, I'd prefer seeing what exactly failed (clippy vs fmt vs build) rather than having everything in one job

Running time went from 3m cached to 40s uncached (https://github.com/sigp/enr/actions/runs/8618027272).


code breaks if the word containing "exit" is used

If user intentionally or accidentally using word that containing "exit", the while loop will break even though the user does not intend to exit the program.
There are lots of german words that contain exit, link here (https://www.buchstaben.com/exit-enthalten).
while loop  in dailiza_bot.py need to be revised for better user experience

Good catch. That should be fixed.


envoy: Bump envoy version to v1.25.7

This is for latest patch release from upstream
https://github.com/envoyproxy/envoy/releases/tag/v1.25.7
https://www.envoyproxy.io/docs/envoy/v1.25.7/version_history/v1.25/v1.25.7
In addition the following changes are included:

update go.mod dependency to latest main
updated envoy extensions & enabled internal listener (https://github.com/cilium/proxy/pull/221)
co-signed image (https://github.com/cilium/proxy/pull/212)
updated ubuntu base image(https://github.com/cilium/proxy/pull/219)
installed ca-certificates package (https://github.com/cilium/proxy/pull/218)

/test

/test-runtime

/test-runtime


Update sql-server-privacy.md

 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.:x: v-brlaz sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@v-brlaz : Thanks for your contribution! The author, @craigg-msft, has been notified to review your proposed change.

Created pull request https://github.com/MicrosoftDocs/sql-docs-pr/pull/5655 in the private repository. #please-close


Add metrics on "Disable Plugin" button

With the changes in https://github.com/GoogleCloudPlatform/google-cloud-intellij/pull/803, we want to get metrics on user interaction with the "Google Plugin Conflict" dialog (to disable the bundled app engine support plugin).

It's probably good to also count the number of the notification pop-ups so that we know how much portion of people disable the old plugin.


Latest version forcingto use JVM 21

I am using this library in one project & its forcing me to use Java 21
Required by:
project :module > com.jeppeman.mockposable:com.jeppeman.mockposable.gradle.plugin:0.11
Caused by: org.gradle.internal.component.resolution.failure.exception.VariantSelectionByAttributesException: Dependency requires at least JVM runtime version 21. This build uses a Java 17 JVM.

Hey @malikmudassarhussain, thanks for reporting. 0.12 is available now where the requirement has been removed.

Ya, it works. Thanks for looking into this.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack 9dabf7c...4621ee6:
  > 4621ee6 Update community files (# 207)
bumping knative.dev/reconciler-test da2ce0e...8105a30:
  > 8105a30 Update community files (# 372)
  > 80f0d4d Fix String() call on nil events on event sender (# 371)
  > 091fb1a Update community files (# 370)
  > 8415d2f Update community files (# 369)
bumping knative.dev/eventing 15dd7ca...4e56370:
  > 4e56370 Update community files (# 6480)
  > aceb557 Knative port sink binding v1 deployment test to rekt (# 6479)
  > 37cb0e9 Knative port source at event types test to rekt (# 6476)
  > c884e27 Remove the filter restriction of only containing exactly one attribute (# 6477)
  > f7e8eef use go1.18 (# 6470)
  > d99685b Promote strict subscriber to beta (# 6473)
  > 5a8f707 Promote delivery timeout to beta (# 6472)
  > 840fce7 chore: update sugar controller ConfigMap example (# 6460)
  > 7e30288 Ignore trailing slashes for Broker addresses (# 6471)
  > 32dea96 update trace of apiserversource (# 6458)
  > 480c8cb Adds receiver options for configuring read and write timeouts message receiver (# 6306)
  > 37dc195 Port Pingsource test to rekt (# 6463)
bumping knative.dev/pkg a01dfed...52261a1:
  > 52261a1 Update community files (# 2569)
  > 7b8b060 Add tracing.SetupDynamicPublishingWithInitialValue (# 2558)

/retest
/lgtm
/approve

/retest

/retest

/override "Code Style / style / Lint"

/close
was replaced by @mgencur


adding long-form audio speaker diarization

https://github.com/NVIDIA/NeMo/pull/7737  can fix long audio clustering cuda ouf of memory  .
Theoretically, the maximum length of audio can be extended by a factor of unit_window_len/sub_cluster_n. For instance, by default, if the original clustering hits the memory limit at the 1-hour mark, the long-form clustering could handle up to 20 hours without exhausting the memory.

Sorry for the huge delay, I prefer waiting until this is merged with a stable release of NeMo before merging as NeMo is already problematic enough

This works well for me! I applied this fix in my fork of this repo.
This allowed me to download and diarize hundreds of podcast episodes 2-3 hour long on RTX2080.
Thank you!


[ENH] pivot_wider_spec pandas

PR Description
Please describe the changes proposed in the pull request:

implement pivot_wider_spec for pandas

This PR relates to #1352 .
Please tag maintainers to review.

@ericmjl

🚀 Deployed on https://deploy-preview-1427--pyjanitor.netlify.app

LGTM!


Pass input parameters to Azure devopspipeline trigger

Using Tool: https://marketplace.visualstudio.com/items?itemName=maikvandergaag.maikvandergaag-trigger-pipeline
I need to pass certain parameters with values to trigger another pipeline (like for example Json list of parameters or values ). How can we achieve that using the above tool. I do not see options for input parameters in this when i added this task. Could you please help ?

I marked it as feature request

this is build and in production


Update uk-tax-mtd.md

added links to new blogs

OPS Build status updates of commit a68948b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/financials/uk-tax-mtd.md
:white_check_mark:Succeeded
View

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.


Updated gatekeeper to 3.10

$subject
cc: @redhat-cop/rego-policies

@pabrahamsson can you review/merge when you've got 5


remove really big environment variables

Without this change, our direnv/nix combo can result in an error because the argument list passed to the server is too long.
We may need to filter these environment variables further depending on if we see the issue reoccur. However, this fixes it for me!
🍐 with @michaelglass

@asterite I forked this into NRI from your fork. What do you think is the best place for it to live going forward? This doesn't seem like a good patch for upstream.

@BrianHicks Nice! I guess we can push it to this fork, then release a new gem version (I can do it). Let me know! (also let me know if I got your question right)

closing in favor of #2