Header or request information in hooks

Is there a way to access this in a hook? If not, would this be possible?

@RickEyre Yes these are accessible.
On the client they are just under hook.params.headers, they will get passed appropriately to your REST client transport (ie. superagent, request, etc.)
On the server see: http://docs.feathersjs.com/middleware/express.html#setting-service-parameters

I'm going to assume you are good based on my last comment. We can re-open if necessary.

Yes, forgot to respond back, sorry. Thanks for the answer :)
On Sun, 27 Mar 2016, 02:12 Eric Kryski<EMAIL_ADDRESS>wrote:

I'm going to assume you are good based on my last comment. We can re-open
if necessary.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/feathersjs/feathers-hooks/issues/61#issuecomment-201999692

I can't seem to find hook.params.headers.
When I printed out, it is undefined.
Maybe something got changed?

@parnurzeal on the client or on the server. On the server they will not exist unless you use an Express middleware like this:
function(req, res, next) {
  req.feathers.headers = req.headers;
  next();
}

@ekryski - i'm on the server, and hooks.params is always empty. i'm coming from the graphql endpoint and using the hooks in the services, but params is always empty.
trying to figure out exactly where to put that middleware function. could you give me a better idea?
thanks for your help

to anybody coming to this thread, i got this to work by defining a function and using that as middleware.
app.use('/graphql', middleware, graphqlExpress((req) => {})

function middleware(req, res, next) {
  req.feathers.headers = req.headers;
  next();
}

same here, after adding req.feathers.headers = req.headers in my middleware, all I got in hook.headers is an undefined object. am I missing anything? @ekryski
by the way, I am using 2.1.1 featherjs and 1.8.1 for feather-hook

The data you are looking for will be in hook.params.headers (not hook.headers).

why can't get headers when the path is specified:
app.use('/path',function(req, res, next) {   req.feathers.headers = req.headers;   next(); })
while the following code still works:
.use(function(req, res, next) { 	req.feathers.headers= req.headers; 	next(); })
@ekryski @daffl

That should work as long as you're declaring that before any other use of /path.

.use(function (req,res,next) {
	req.feathers.a = 'a';
	next()
})
.use('/path',function (req, res, next) {
	req.feathers.b = 'b' ;
	next();
})
.use('/path',function(req, res, next) {
	req.feathers.c = 'c';
	app.handle(req, res);
})

I found only a is in params, b and c are lost. @marshallswain

You can if you register it at the same time as the service:
app.use('/path', function(req, res, next) { req.feathers.headers = req.headers; next(); }, service)

Ideally as also pointed out in this FAQ the service method calls should stay as transport independent as possible however.

Same here. I have
  app.use(function(req, res, next) {
    req.feathers.fromMiddleware = 'Hello world';
    req.feathers.headers = req.headers;
    next();
  })

before app.use(notFound()); in src/middleware/index.js
In service hook, console.log(hook.params) show:
{ query: {}, provider: 'socketio' }

What am I doing wrong?
Thanks in advance

You have to register that middleware before any service.

@daffl. Thanks for for your help but I can't get it to work.
middleware.js is registered after the services in app.js. I've tried it in app.js before app.configure (services); but does not change the result. Where should it be put?
app.use(function(req, res, next) {
  req.feathers.fromMiddleware = 'Hello world';
  req.feathers.headers = req.headers;
   next();
});//Same result
// Set up our services (see `services/index.js`)
app.configure(services);
// Configure middleware (see `middleware/index.js`) - always has to be last
app.configure(middleware);
app.hooks(appHooks);

What other plugins are you registering before this?

None. It is a new generated app

I think you'll need at least this much before that for this to work:
import feathers from 'feathers';
import bodyParser from 'body-parser';
import rest from 'feathers-rest';

const app = feathers()
  .configure(rest())
  .use(bodyParser.json())
  .use(bodyParser.urlencoded({ extended: true }))
  .use(function(req, res, next) {
    req.feathers.data = 'Hello world';
    next();
  });

What @marshallswain posted should work. It is definitely being tested in https://github.com/feathersjs/feathers-rest/blob/master/test/index.test.js#L399

Hi :-)
The "feathers generate app" command creates a different source code.
app.configure(rest()) is after .use(bodyParser.urlencoded({ extended: true })), so test fails: https://travis-ci.org/kfern/test-feathers-hooks-issues-61/builds/242706813

Interesting, we'll have to look into it. Can you create an issue in https://github.com/feathersjs/generator-feathers/ so this doesn't get lost?

Done :-)

Here again @daffl
Adding in src/middleware/index.js
  app.use(function(req, res, next) {
    req.feathers.data = 'Hello world';
    next();
  });

req.feathers is undefined in middleware/index.js, so test fails. with 500 error: https://travis-ci.org/kfern/test-feathers-hooks-issues-61/jobs/245581258
Test repo: https://github.com/kfern/test-feathers-hooks-issues-61/blob/master/src/middleware/index.js#L7

Will be fixed via https://github.com/feathersjs/generator-feathers/pull/232. Just move your app.configure(middleware); after app.configure(rest());

Ok. Now the tests do not fail, but the data is not available. In a before all hook, console.log(hook); return {}
https://github.com/kfern/test-feathers-hooks-issues-61/blob/master/src/middleware/index.js#L8
https://github.com/kfern/test-feathers-hooks-issues-61/blob/master/src/app.hooks.js#L8
https://github.com/kfern/test-feathers-hooks-issues-61/blob/master/src/hooks/test-61.js

I just created a new app with<EMAIL_ADDRESS>then a middleware like:
module.exports = function (options = {}) {
  return function test(req, res, next) {
    req.feathers.message = 'Hello world';
    next();
  };
};

And a before all hook like
module.exports = function (options = {}) { // eslint-disable-line no-unused-vars
  return function (hook) {
    console.log(hook.params);
    // Hooks can either return nothing or a promise
    // that resolves with the `hook` object for asynchronous operations
    return Promise.resolve(hook);
  };
};

And it shows message: 'Hello world' when going to the service endpoint.

Hi @daffl
It's working for me now! Thanks for the nice and direct feedback :-)
I followed these steps:
$ feathers -V
2.2.1
feathers generate app
feathers generate middleware (middleware-test-61)
feathers generate service (service-test-61)
feathers generate hook (hook-test-61, before all service-test-61)
https://github.com/kfern/test-feathers-hooks-issues-61/commit/5b22ba6043854971504c8d95af73a26683511d5f
Added req.feathers.message = 'Hello world'; in src/middleware/middleware-test-61.js
Added console.log(hook.params); in src/hooks/hook-test-61.js
https://github.com/kfern/test-feathers-hooks-issues-61/commit/f6a68909ba98c3f3b1d34d6975d10e225740d374
Test's output:
ok 1 Feathers application tests starts and shows the index page
middleware-test-61 middleware is running
ok 2 Feathers application tests 404 shows a 404 HTML page
middleware-test-61 middleware is running
ok 3 Feathers application tests 404 shows a 404 JSON error without stack trace
undefined
ok 4 'hook-test-61' hook runs the hook
ok 5 'service-test-61' service registered the service
https://travis-ci.org/kfern/test-feathers-hooks-issues-61/jobs/245929395
Request http://localhost:3030/service-test-61 with Postman:
info: Feathers application started on localhost:3030
middleware-test-61 middleware is running
{ query: {}, provider: 'rest', message: 'Hello world' }
info: after: service-test-61 - Method: find

Using feathers-client and socket.io-client, console.log(hook.params) output is
{ query: {}, provider: 'socketio' }
info: after: service-test-61 - Method: find

Is the expected behavior?
feathers-client and socket.io-client versions are:
<script<EMAIL_ADDRESS><script<EMAIL_ADDRESS>

Glad it worked and yes, that is expected. Socket middleware has to be registered with their own provider.

Ok. Thanks again


Why is create-react-app installing an old version of webpack?

When I run create-react-app, it's using an older version of webpack, despite using the latest version of create-react-app:
create-react-app version: 1.3.0
react-scripts version: 0.9.5
In my yarn.lock file it's set to<EMAIL_ADDRESS>I would like to be able to use webpack > 2.0 if possible
Also getting a bunch of unmet peer dependency warnings when creating a new app. Not sure if it's causing the issue:
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency "webpack@1 || 2 || ^2.1.0-beta || ^2.2.0-rc".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^3.0.0 || ^4.0.0".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS><4.0.0".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>- 3.x".
warning<EMAIL_ADDRESS>has unmet peer dependency "webpack@1 || ^2.1.0-beta".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| 3.x".
warning<EMAIL_ADDRESS>has unmet peer dependency "file-loader@*".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^3.0.0".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS><3".
warning<EMAIL_ADDRESS>has unmet peer dependency "webpack@1 || ^2.1.0-beta || ^2.2.0-rc.0".
warning<EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>

This is completely intentional. We are shipping the version of Webpack that we know to work well. There are unresolved issues with Webpack 2 (for example broken sourcemaps) that have been stopping us from rolling it out. We found some workarounds, but in general we try to ship you the stable tools rather than latest tools. We take care to ensure they all work together well as a cohesive package. When we are confident about shipping Webpack 2 we will definitely release it. I hope this helps!

Also getting a bunch of unmet peer dependency warnings when creating a new app.

This is just a Yarn bug. It will be fixed in Yarn 0.22.

Makes sense. I was under the impression that Webpack 2 was being used because it was defined in the package.json in the scripts: https://github.com/facebookincubator/create-react-app/blob/0c0c5c1f24c38feb57bea54795a11c0e0d0a2300/packages/react-scripts/package.json#L59

In master, yes. This lets us work on the issues we encounter, and keep track of whether they get solved. When it’s ready, it will be released as 0.10.


Following README.md File Instructions Results in Failure to Compile App

Description
When following the instructions contained within the README.md file within this repository an error is thrown.
Go Version
go version go1.14.1 darwin/amd64
Current behavior
When copying/pasting the example in README.md and running, the following occurs.
colinjohnson@cjohnson07 newrelic_api_connector % go run newrelic_api_connector.go
# command-line-arguments
./newrelic_api_connector.go:9:2: imported and not used: "github.com/newrelic/newrelic-client-go/pkg/config"
./newrelic_api_connector.go:14:5: assignment mismatch: 1 variable but newrelic.New returns 2 values
./newrelic_api_connector.go:14:21: undefined: ConfigAPIKey

Expected behavior
When copying/pasting the example in README.md it is expected that the example will run properly (including connecting to NewRelic and, potentially, printing a list of NewRelic applications.
Steps To Reproduce
Steps to reproduce the behavior:

Copy the text from README.md into a .go file.
Run the command go run filename.go
The result is a file that is not able to be compiled.

Debug Output (if applicable)
NA.
Additional Context
This occurs with the following newrelic-go-client: github.com/newrelic/newrelic-client-go v0.19.0.
I've been running with a configuration that looks as follows:
package main

import (
	"fmt"
	"os"

	"github.com/newrelic/newrelic-client-go/newrelic"
	"github.com/newrelic/newrelic-client-go/pkg/apm"
)

func main() {
	apiKey := os.Getenv("NEW_RELIC_API_KEY")
	nr, _ := newrelic.New(newrelic.ConfigAdminAPIKey(apiKey))

	params := apm.ListApplicationsParams{
		Name: "RPM",
	}

	apps, err := nr.APM.ListApplications(&params)

	if err != nil {
		fmt.Print(err)
	}

	fmt.Printf("%#v\n", apps)

}

I believe the sample code I've written is not ideal because the configuration does not reflect the different methods of providing API Keys to a NewRelic API client - either a ConfigPersonalAPIKey or a ConfigAdminAPIKey - I simply assume the user is providing an environment variable named NEW_RELIC_API_KEY.
References or Related Issues
Are there any other related GitHub issues (open or closed) or Pull Requests that should be linked here?

Thanks @colinbjohnson for the issue, and a example solution.  I added a few error statements for context too.

Just tested!!! Thank you!


local error in the translator core.

I forgot to enable the GS-SequenceTranslator debugging option, did that now. In the meantime below the error I keep getting since today's update.
Below a copy & paste from bugsack about the error.
2x GS-SequenceEditor\translator-core.lua:158: attempt to index field '?' (a nil value)
GS-SequenceEditor\translator-core.lua:158: in function GSTRTranslateSpell' GS-SequenceEditor\translator-core.lua:99: in function GSTranslateString'
GS-SequenceEditor\translator-core.lua:37: in function <GS-SequenceEditor\translator-core.lua:32>
(tail call): ?
GS-Core\Core.lua:442: in function GSUpdateSequence' GS-Core\Core.lua:133: in function GSReloadSequences'
GS-SequenceEditor\translator-core.lua:287: in main chunk
Locals:
str = "Heroic Throw"
fromLocale = "deDE"
toLocale = "enUS"
cleanNewLines = false
output = ""
found = false
conditionals = false
mods = "|cff55cc55|r"
etc = "Heroic Throw"
(*temporary) = nil
(*temporary) = "HASH"
(*temporary) = "!"
(*temporary) = ""
(*temporary) = "Heroic Throw"
(*temporary) = 0
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field '?' (a nil value)"
GNOME = "GS-SequenceTranslator"
language =  {
HASH =  {
}
KEY =  {
}
SHADOW =  {
}
}

I don't know how but your language files are blank.

HASH =
{
}
KEY =
{
}
SHADOW =
{
}

These should all contain values.
TimothyLuke

On 2 Sep 2016, at 4:47 AM, DrMengede<EMAIL_ADDRESS>wrote:
I forgot to enable the GS-SequenceTranslator debugging option, did that now. In the meantime below the error I keep getting since today's update.
Below a copy & paste from bugsack about the error.
2x GS-SequenceEditor\translator-core.lua:158: attempt to index field '?' (a nil value)
GS-SequenceEditor\translator-core.lua:158: in function GSTRTranslateSpell'
GS-SequenceEditor\translator-core.lua:99: in functionGSTranslateString'
GS-SequenceEditor\translator-core.lua:37: in function
(tail call): ?
GS-Core\Core.lua:442: in function GSUpdateSequence'
GS-Core\Core.lua:133: in functionGSReloadSequences'
GS-SequenceEditor\translator-core.lua:287: in main chunk
Locals:
str = "Heroic Throw"
fromLocale = "deDE"
toLocale = "enUS"
cleanNewLines = false
output = ""
found = false
conditionals = false
mods = "|cff55cc55|r"
etc = "Heroic Throw"
(temporary) = nil
(temporary) = "HASH"
(temporary) = "!"
(temporary) = ""
(temporary) = "Heroic Throw"
(temporary) = 0
(temporary) = nil
(temporary) = nil
(*temporary) = "attempt to index field '?' (a nil value)"
GNOME = "GS-SequenceTranslator"
language =
{
HASH =
{
}
KEY =
{
}
SHADOW =
{
}
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

HI @DrMengede the only way I seem to be able to replicate this is if did you upgrade the mod while the game was open from a version like 1.0 or 1.1?

Hi Timothy,
I think I found the error.
You can add a language selection between the sequence and pre macro.
I noticed that one of the sequences had the language indication "deDe"
in it. When I removed that line the error was gone.!
So I think that it just conflicted with the global "enUS" language setting.?
Regards,
Huub Mengede
(a.k.a. Dr.Mengede / Dr.Esqueleto / el_Padre)
On 2016-09-03 14:57, Timothy Minahan wrote:

HI @DrMengede https://github.com/DrMengede the only way I seem to be
able to replicate this is if did you upgrade the mod while the game
was open from a version like 1.0 or 1.1?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GnomeSequenced-Enhanced/issues/66#issuecomment-244544914,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJrziGIwhacbxLNhoQWFd-UOJfhIAbOgks5qmW6igaJpZM4JzCN1.

It thought that that Macro was in German but unless you have the translator installed it doesnt have any German references.

I've released this in the Beta build on Curse and the updated Translator. If there are no issues overnight the Beta will become the final version


Useful source material

NHSX have got loads of useful material for this repo, such as:

https://nhsx.github.io/AnalyticsUnit/experiences.html
https://nhsx.github.io/AnalyticsUnit/toolstackrequests.html

On installing R Goldacre review, and the relevant material is accepted implicitly or explicitly in Data Saves Lives

NHS Pycom page on areas that use GitHub


Svelte Store undefined despite logging to dev server.

Describe the bug
I am setting the value of a Store within a pages load function and I have run console.log on the store within the load function to ensure it is being set correctly, the dev server log is correct. However when the page actually loads it logs to the console on the page as undefined.
Any idea what could be causing this?
export async function load() {
	let homeTown;
	userStore.subscribe((value) => (homeTown = value.profile?.homeTown));
	const weatherData = await searchWeatherLocations(homeTown);
	weatherStore.set(weatherData);
}

Reproduction

Set the value of a store in a load function.
Log the store value on a page.

Logs
No response
System Info
System:
    OS: macOS 13.0
    CPU: (10) arm64 Apple M1 Pro
    Memory: 64.89 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 19.4.0 - ~/.nvm/versions/node/v19.4.0/bin/node
    Yarn: 1.22.19 - ~/Library/pnpm/yarn
    npm: 9.2.0 - ~/.nvm/versions/node/v19.4.0/bin/npm
  Browsers:
    Chrome: 109.0.5414.87
    Safari: 16.1
  npmPackages:
    @sveltejs/adapter-auto: ^1.0.0 => 1.0.2
    @sveltejs/kit: ^1.0.0 => 1.2.1
    svelte: ^3.54.0 => 3.55.1
    vite: ^4.0.0 => 4.0.4

Severity
blocking an upgrade
Additional Information
No response

I realised I should be returning an object from the load function and then setting the store in the client side code.
<script>
	import { enhance } from '$app/forms';
	import { locationsSearchStore } from 'store';
	export let data;
	locationsSearchStore.set(data.locations);
</script>

export async function load() {
	let homeTown;
	userStore.subscribe((value) => (homeTown = value.profile?.homeTown));
	const locations = await searchWeatherLocations(homeTown);

	return { locations };
}


Tests fail or pass depending on how runtests.jl is executed

FYI, @Datseris and @maximilian-gelbrecht : https://discourse.julialang.org/t/unable-to-reproduce-unit-test-outcomes-from-package-managers-clean-environment/79772

Can you point me which test in the source code is the failing one? Maybe that will help me identify the problem...

See the last commit; I commented everything out, but the not-working test. When running it via ] test TreeEmbedding I'll get as delay values [0,1,5]. So does @maximilian-gelbrecht on his machine.

After hours of diggin into the code I found the error, eventually. - And it does not make me happy at all! Instead of
costs = zeros(T, NN, D)
@inbounds for (i,v) in enumerate(ns)
   costs[i,:] = (Vector(prediction[i]) .- Vector(Y[v+Tw])).^2
end

this here
costs = abs2.(Matrix(prediction) - Matrix(Y[ns.+Tw]))

works perfectly on all machines and runners. Why is this? Anyway, I'll accept this and close the issue.


Set variables within block if needed

1. Why is this change necessary?
To be able to overwrite the block buy_widget_data and still use buy_widget_reviews.
2. What does this change do, exactly?
It moves the two variables that are only used within the block into the scope of the block.
3. Describe each step to reproduce the issue or behaviour.

Create yourPluginsrc/Resources/views/storefront/component/buy-widget/buy-widget.html.twig
Extend the parent template {% sw_extends<EMAIL_ADDRESS>%}
Overwrite block buy_widget_data to only contain buy_widget_reviews:

{% block buy_widget_data %}
    {% block buy_widget_reviews %}
        {{ parent() }}
    {% endblock %}
{% endblock %}

Open a product with reviews
Click the reviews link

Expected behavior: The link still works and scrolls to the reviews tab.
Actual behavior: Nothing happens
4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

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

Hey @DanieliMi,
moving twig variables into a new block is a breaking change, therefore we can't merge your changes.
As a workaround you could copy & paste the variable declarations into your template override.
To fix this in the platform you can't remove the variables in the original block, but you have to deprecate them there for the 6.5.0 version, and wrap the variable declaration in the new block in a {% if var is not defined %} to not override the declaration in the parent block if it was already declared there.

Hi @keulinho,
I have changed the PR accordingly. Will I have to do anything for the 6.5 release or will they be removed automatically due to the deprecation?

LGTM! :blue_heart:
No nothing to do from your side. We will remove all places that are marked as deprecated when we are starting the work for the 6.5 release.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-22951
Please use this issue to track the state of your pull request.

Thanks for your contribution 🎉


Fix typographical error(s)

@igreulich, I've corrected a typographical error in the documentation of the angular-truncate project. Specifically, I've changed charachters to characters. You should be able to merge this pull request automatically. However, if this was intentional or if you enjoy living in linguistic squalor, please let me know and create an issue on my home repository.

Thanks! I am not the greatest of spellers..


Reaction Role Edit

should make a command to edit reaction roles so its more clear rather than posting multiple reaction embeds if you add more reactions

reaction roles are on my list of things to fix anyways. I'll keep this issue open till it's fixed/updated


Add option to show forecast icon

This is text mode goddammit!
Defaults to true so behaviour is as before.

Hi,
Realise this is probably pretty low priority.
Any thoughts on this?
Cheers.

thanks!


[Rails][Administrate] Add CSV importer format in Administrate UI

For Places, Stories, and Speakers, admin are able to upload a CSV to bulk import data. However, it’s not clear what format the CSV needs to be in for the importer. Let’s add some UI to each of the admin pages that describes the necessary CSV format.
Sample CSV file: https://drive.google.com/file/d/1Ir6TC5THEoQKk-98JZ5KpTjXd-R6yMmA/view
Administrate view for the importer can be found here
Acceptance Criteria:
For the Place importer, the CSV format is described next to the import field.
For the Speaker importer, the CSV format is described next to the import field.
For the Story importer, the CSV format is described next to the import field.
Design is up to you! Please add where you think would make the most sense. :)
from @slaloggia (Thank you!):
Currently, the system is expecting a csv with column headings and is matching the values to attributes by location, not by the heading name. So names do not have to be exact, but the order matters.
Title
Description
Speakers (if multiple, separate with commas)
Places (if multiple, separate with commas)
Interview_location
Date_interviewed
Interviewer
Language
Media (this field currently cannot be empty, see #534 #535 #536)
Permission_level (leave blank if no restrictions, ANYTHING in this column will set permission level to "user_only")

If this is still up for grabs, I'd like to give it a shot!


[feature/update-license-header-year] Update year of license header

Description
Spotless updates the year at the head of your license based on the year of your last commit.
./gradlew spotlessApply -PspotlessSetLicenseHeaderYearsFromGitHistory=true

I used this script.

Thank you for your contribution @l2hyunwoo !
LGTM 👍


Middleware loader expect lowercase path

Hi,
I faced a weird bug, while trying to load some custom made middleware. The console output an error about my middleware not being found. After some investigation I found that custom middleware are expected to be in a path in lower case (see this line)
This is a very big assumption as many filesystem are case sensitive. Would it be possible to get rid of that limitation ?

Fixed and released in lws v2.0.6 - sorry about that. Please re-install.
Let me know if you find any other issues. 👍


Add development infrastructure to maven or using docker-compose in order to remove embedded connectors

I suggest we add some docker config to spin-up development Kafka cluster + Debezium + optional database/s to get rid of the dependency to one static version of connectors.
It's relatively easy to add new endpoints to Kafka Connect for Debezium and instead of accessing the embedded connectors we can query the connected Kafka Connect instances.

In a recent call @gunnarmorling decided in favor of docker-compose.


allow posting images to private DMs, groups and channels

Added support for groups here: https://github.com/PublicRelay/com.publicrelay.www/pull/2709/files

Should be fully implemented by this https://github.com/lgbtq-technology/lgbtq-helper-app/pull/23/files

Fixed by #23!


Add support for gitweb-style projects.list

It's useful to have web UI display only repositories with configured public access. This is particularly important when hosting git using http://gitolite.com/gitolite/index.html or similar solutions.
In case of gitolite, public visibility is controlled by giving access to a special user called "gitweb". This makes gitolite add the repository to projects.list file, which is read by gitweb (https://www.kernel.org/pub/software/scm/git/docs/gitweb.conf.html variable $projects_list).
I've added an option to point gitlist to the same file. If the file exists, only repositories listed there will be shown.

That would be great!

Hey @mrzechonek, can you rebase this into master? Ready to merge.

Done, but master seems broken :(

Master needs a cleanup. Soon! :)

:+1:


add more helpful message to upgrade base class

https://issues.apache.org/jira/browse/CASSANDRA-11126?focusedCommentId=15404041&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15404041
If you try to run tests in one of the base test classes, the error message is just incidental and doesn't help much if you don't understand the way the tests are generated:
RUN_STATIC_UPGRADE_MATRIX=true CASSANDRA_VERSION=git:trunk JAVA8_HOME=/usr/lib/jvm/java-8-openjdk JAVA7_HOME=/usr/lib/jvm/java-7-openjdk python2.7 ./run_dtests.py --vnodes false  --nose-options='-v' upgrade_tests/cql_tests.py:TestCQL.select_distinct_with_deletions_test
[...]
======================================================================
ERROR: select_distinct_with_deletions_test (upgrade_tests.cql_tests.TestCQL)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/pcmanus/Git/cassandra-dtest/upgrade_tests/upgrade_base.py", line 53, in setUp
    .format(self.UPGRADE_PATH.starting_version, self.UPGRADE_PATH.starting_meta.java_version))
AttributeError: 'NoneType' object has no attribute 'starting_version'
-------------------- >> begin captured logging << --------------------

So, this improves that error message:
CASSANDRA_VERSION=git:trunk ./run_dtests.py --vnodes false --nose-options= --with-id --verbosity=2 --nocapture upgrade_tests/cql_tests.py:TestCQL.select_distinct_with_deletions_test
[...]
======================================================================
ERROR: upgrade_tests/cql_tests.py:TestCQL.select_distinct_with_deletions_test
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/mambocab/cstar_src/cassandra-dtest/upgrade_tests/upgrade_base.py", line 52, in setUp
    self.validate_class_config()
  File "/home/mambocab/cstar_src/cassandra-dtest/upgrade_tests/upgrade_base.py", line 211, in validate_class_config
    self.assertIsNotNone(self.UPGRADE_PATH, no_upgrade_path_error)
AssertionError: No upgrade path specified. <class 'upgrade_tests.cql_tests.TestCQL'> may not be configured to run as a test. Did you mean to run one of its subclasses, such as <class 'abc.TestCQLNodes3RF3_Upgrade_current_2_2_x_To_in
dev_2_2_x'>?

@knifewine could you review please?

great idea, +1

Thanks for the review! Merging.


No return from setter

For some cases it is very inconvenient to have return value in setter,
for example:
fn f() {}
fn g() -> i32 {
    17
}

fn main() {
    let x = 1;
    match x {
        1 => f(),
        2 => g(),
        3 => println!("Hi"),
        _ => println!("ohter"),
    }
}

this code doesn't compile, because of g return value,
I don't use value from setters, so it would be convenient to have such option
to make it possible for easy way to deal with generated code.
Depend on #5

I will merge this PR after this discussion: maybe the skip feature is redundant.

#[property(set(noret))] is little shorter then #[property(set(type = "none"))],
but I usually use it on the top of struct so it is doesn't really matter for me.
Close in favor of #8


Machine learning

Description 📜
I have created a new branch in machine learning that is OpenCV and i ve  created a documentation of basics <issue_number_here>

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

can anyone review my PR

What is the issue number you're working on? @kuwarkapur

Basically it's not an issue and I am working on machine learning directory
and I am adding a topic in a new folder opencv
On Sun, 26 Sep, 2021, 17:40 Abhishek Sharma, @.***>
wrote:

What is the issue number you're working on? @kuwarkapur
https://github.com/kuwarkapur
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/2844#issuecomment-927295079,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKBNMPROTLBL27N4ZW6KNLUD4EU3ANCNFSM5EYRTJ5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

We can't accept this issue as we are following our roadmap, and also we have a fixed path of this program, follow the roadmap and if you think you can work on any issue created by us, then comment out that, we'll evaluate and then only you can be assigned to an issue. @kuwarkapur


pulumi.com/engineering strategy blog

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

What: Post engineering design decisions, how we made them, and why.
Where:  Create a page such as pulumi.com/engineering (throws a 404 currently)
Affected area/feature

Things that can be added would be decisions we made and why when we implemented the following.

Pulumi Imports
Managing GitHub at Pulumi with Pulumi
Automation API
Multi Language Components
Helm Release Design
Update Plans

@tusharshahrs Do you imagine this as a separate blog altogether, or a filtered view on our existing one (say, all posts tagged with a certain tag)? I can definitely imagine the latter.

happy to collab on this with you Christian

Great, sounds good, thanks!


Correct PostScript trigonometric operators

PDF 32000-1:2008 <IP_ADDRESS> "Type 4 (PostScript Calculator) Functions" defers to the PostScript Language Reference for the description of these functions. The PostScript Language Reference, third edition chapter 8 "Operators" defines the angle type as a "number of degrees". Section 8.1 defines "angle sin real", "angle cos real", and "num den atan angle". The documentation for atan further states that it will return an angle in degrees between 0 and 360.
Handle these operators correctly in PostScriptEvaluator.execute. Convert the inputs to sin and cos from degrees to radians for use with Math.sin and Math.cos. Correctly pop two values from the stack for atan, use Math.atan2, and convert from radians to (positive) degrees.

Added some reference PDFs. These PDFs contain a 256x256 bitmap with component color values of x,y,0 and a DeviceN color space with a Function which calculates the alternate color space components using the operators in question (with appropriate scaling). Before this change pdf.js draws these quite interestingly different from other PDF viewers. After this change pdf.js draws these similarly to other PDF viewers.

For future reference, these PDFs were generated using a hack of Skia at https://skia-review.googlesource.com/c/skia/+/651756 .

Hmmm... even after sprinkling Math.fround all over it doesn't happen. The actual sampled values go from 0 to 255 which should then map to 0 to 1.0. 1.0 should have the same value as 0, so should be wrapped around. It seems most likely that we didn't get 1.0 to begin with as an input maybe? It looks like 1.0 should come in from AlternateCS::getRgbBuffer from the 255 value. Upon inspection it appears this does happen (I'm getting 1.0 there locally). Stepping through the function I see that we're not getting exactly 0 for Math.sin(a * (Math.PI / 180)). Re-arranging things to Math.sin(a / 180 * Math.PI) doesn't help. In the end, Math.sin(2*Math.PI) also isn't 0, so not much help there. However, since these are degrees and so rotations happen on an integer, changing to Math.sin(a % 360 / 180 * Math.PI) seems to work, since it keep everything perfectly regular. Have to make sure JavaScript's remainder is the right operator (do we want remainder or mod?).

After a careful re-reading of https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Remainder#description it looks like the remainder operator does the right thing here.

/botio test

/botio makeref


feat: workload generator add container dirs implementation

What type of PR is this?
/kind feature

What this PR does / why we need it:
add container dirs implementation
Which issue(s) this PR fixes:

Fixes #674
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

27 of 31 (87.1%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 74.014%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/modules/generators/workload/workload_generator.go
27
31
87.1%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
7585

Relevant Lines:
10248

💛  - Coveralls


Which config can reproduce the results in paper?

Hi,
For training on Audioset see this section
The following parameters control how many time/frequency frames in patchout:
Unstructured:
models.net.u_patchout=400

Structured:
models.net.s_patchout_f=4
models.net.s_patchout_t=400

For fine-tuning check the folder of each dataset, for example fsd50k


dapp: add check for required browser features

Fixes #2497
Short description
Please try it yourself in your web-browser an make sure that the application gets served for you.
Definition of Done

[x] Steps to manually test the change have been documented
[x] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

Run the dApp
Make sure you get not shown the error message.

Looks good on the first view, but you should add a changelog entry.
And do you know of any browser that doesn't support the feature set to test the bad case?

Looks good on the first view, but you should add a changelog entry.
And do you know of any browser that doesn't support the feature set to test the bad case?

Looks good on the first view, but you should add a changelog entry.

Good idea. I'll do so!

Looks good on the first view, but you should add a changelog entry.

Good idea. I'll do so!

And do you know of any browser that doesn't support the feature set to test the bad case?

Hmm. Maybe an old Internet Explorer which stopped development some years ago.
@christianbrb as the only Windows user in the team: could you try install the old IE?

And do you know of any browser that doesn't support the feature set to test the bad case?

Hmm. Maybe an old Internet Explorer which stopped development some years ago.
@christianbrb as the only Windows user in the team: could you try install the old IE?

@palango I got an old Firefox v41.0 from an achieve. This works. Means I get the message and not the application.
Unfortunately this fu**** up my actual Firefox setup (the config files, plugins, user data,,,). I'll need some time to fix it.
@christianbrb forget about my comment. I did it already.  --^

@palango I got an old Firefox v41.0 from an achieve. This works. Means I get the message and not the application.
Unfortunately this fu**** up my actual Firefox setup (the config files, plugins, user data,,,). I'll need some time to fix it.
@christianbrb forget about my comment. I did it already.  --^

@palango I got an old Firefox v41.0 from an achieve. This works. Means I get the message and not the application.
Unfortunately this fu**** up my actual Firefox setup (the config files, plugins, user data,,,). I'll need some time to fix it.

Oh damn, sorry for that! But thanks for testing.

@palango I got an old Firefox v41.0 from an achieve. This works. Means I get the message and not the application.
Unfortunately this fu**** up my actual Firefox setup (the config files, plugins, user data,,,). I'll need some time to fix it.

Oh damn, sorry for that! But thanks for testing.

I'll rebase to resolve the merge conflicts.

I'll rebase to resolve the merge conflicts.


Revamp content and design for ReportStream About page

Problem statement
User: As a prospective or existing user, when I visit the ReportStream website I want to be able to learn about ReportStream so that I can determine why it should matter to my job.
What you need to know
We can adapt some of previously revised (but not yet published) content work for this upcoming redesign so we don't have to reinvent the wheel.

Here's the Content Design folder that contains previously revised web copy prior to this current June 2022 redesign.

Here's the IA mural board for reference.

Here's the ReportStream Content Style Guide we adhere to for public-facing content

Acceptance criteria
Content:

adheres to ReportStream Content Style Guide
follows federal plain language guidelines

Design:

follows USWDS guidelines
if diagrams/graphics are needed where indicated by Content, include them on the relevant page/section
if template is not built, design needs to create it

User voices: Sender is an API user (phase out CSV uploader content), Receiver represents a public health dept (STLTs)
Review/QA:

Content shares work with Exp. Product and Design team to review before publishing
Design shares work with Exp. Product and Design team to review before publishing (staging for review? how does this work / let's discuss)

[Need more info - this page should be worked on last since it's dependent on the rest of the website content]

Note: The "About" page might just be a section within the future "Product" page -- however, if we do decide to keep "About" as its own static page, then we can keep this ticket open. Will flesh out details once "Website Updates MVP" is done.

We are supposed to include an about page of some sort, so we need to keep it around in some fashion.
Some options for easy moves we could make here:

Keep the existing page at /about. Link to it in the footer.
Keep the existing about page, move it into the new /product sidebar. Keep the path at /about or redirect from /about
Move the existing /about content to the bottom of the /product page content. Redirect /about to this new path

Updates 7/29/2022: Per our convo today @a-chappelear -- the About page can probably keep the same content for now and then have it linked up in the side-nav on the Product page. Ideal future state for content and design would be to revamp the "one-pager" that's linked up on the About page since it's fairly outdated.

This has also been added to the /product sub-nav (as it gets pretty lost in the footer.) Possible adjustments later - update or review the 1-pager that is attached to the page. Get feedback on the language used here from stakeholders during the review/research phase. Closed!


Add update domain list workflow

This pull request adds a new workflow called "Update Domain List". The workflow is triggered on a schedule and when changes are pushed to the .github/workflows/update_domein_list.yml file. It runs on the latest Ubuntu environment and performs the following steps:

Checks out the code using actions/checkout@v4

Sets up Node.js using actions/setup-node@v4

Installs dependencies using npm install

Generates a disposable email blocklist JSON using npm run generate:disposable_email_blocklist_json

Fixes linting issues using npm run lint:fix

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (5411195) to head (4b40c44).

:white_check_mark: All tests successful. No failed tests found.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            8         8
=========================================
  Hits             8         8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add Cloudsmith to docs as an alternative Package Manager to host Choc…

…olatey packages

Trigger the GHA for this contribution. PR didn't touch the workflow, so safe to do.

@ciaracarey thanks for taking the time to get this added.  Assuming the CI build passes, we can get this merged into the production site.


Investigate action failures

Expected Behavior

Actions should run without failures.
Current Behavior

A few of the update actions are failing. See here and here and here.
They do not seem to be consistently failing, but it's happening frequently enough that we need to investigate and see what is happening.

This looks like it's caused by a change in the format of the filenames for the v9 RC:
apache-tomee-9.0.0.RC1-plume.tar.gz
vs
apache-tomee-9.0.0-M8-plume.zip
I'm guessing the action matches based on expecting -x.y.z- so the apache-tomee-9.0.0-RC1-plume.tar.gz that gets constructed per the action error would usually be correct, but not for this version due to the 0.RC1.


This PR fixes issues with images

The edits here , add in more images whilst fixing issues with the original ones.
Content was modified too


Example for AWS Service Integrations ( S3 )

Description:
I do not find any documentation or example how we can define the sam template for AWS service integration such as S3

@ashishusoni SAM has an easy integration between Lambda Functions (AWS::Serverless::Function) and S3 buckets.
The spec for AWS::Serverless::Function can be found here: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#awsserverlessfunction
S3 event structure: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#s3
Here's a sample application that uses S3 events: https://github.com/awslabs/serverless-application-model/tree/master/examples/2016-10-31/image_resize_python


Annotation Editor: Fix query editor to support new react annotation handling

What this PR does / why we need it: This PR is to fix the Annotation query editor to support the new json structures for storing the annotation query in the dashboard json.
This PR is to fix the issue - https://github.com/grafana/opensearch-datasource/issues/143
the dashbord json is sticking to the new format of having the query filed nested into the target after the fix

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

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

I have signed up the CLA but somehow it is not reflecting here.

Hi @Thamizhvanan-R can you double check that you've signed the CLA?
Afaik, CLA needs to be signed for each repo under Grafana org, so if you've signed it in another one, it won't reflect in open-search

Hi @Thamizhvanan-R can you double check that you've signed the CLA?
Afaik, CLA needs to be signed for each repo under Grafana org, so if you've signed it in another one, it won't reflect in open-search

I think I have signed up the right one.

My Bad, Seems the commit I made is from my other github account - https://github.com/Thamizhpriyan
I have signed up the CLA with my other github account.

Hi @Thamizhvanan-R thanks for the contribution!

Can we get this merged? hoping to get this bug fixed in next release.

Hi @Thamizhvanan-R we just released v2.14.6 with the fix. Thanks again!


Update yumrepo.gemspec

Updating the version to 0.2.2
To get these changes, https://github.com/rubiojr/yumrepo/pull/6

@vinayakvakkund thanks!
I don't have plans to maintain this gem going forward, but I can grant you access to https://rubygems.org/gems/yumrepo if you want, so you can bump it yourself. Let me know.

Thanks @rubiojr , I am fine with that.

Hi @rubiojr  , Could You please upload new RPM to the repository?

@vinayakvakkund email me your rubygems account/email and I'll grant you access to be able to push it yourself, can't guarantee I'll be able to do that this week.

Maybe there's a workflow we can add to this repo to automatically push gems every new release?

Maybe there's a workflow we can add to this repo to automatically push gems every new release?

Yeah sure we can do that, I am not sure how to do it from Github but as of now I can build gem manually and upload it to rubygems. Please find below details of my username / email
usrername : vinayak3700
email<EMAIL_ADDRESS>

done.


containter in ERROR state

Hi!
After upgrade LXD from 2.5-0ubuntu1~ubuntu16.04.1~ppa1 to
2.6.2-0ubuntu1~ubuntu16.04.1~ppa1 all containters have ERROR state.
In log I see problems with lxc_parse:
lxc<PHONE_NUMBER>4719.275 ERROR    lxc_parse - parse.c:lxc_file_for_each_line:57 - Failed to parse config: lxc.start.auto =
Some info:
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.1 LTS
Release:	16.04
Codename:	xenial

apiextensions:
- storage_zfs_remove_snapshots
- container_host_shutdown_timeout
- container_syscall_filtering
- auth_pki
- container_last_used_at
- etag
- patch
- usb_devices
- https_allowed_credentials
- image_compression_algorithm
- directory_manipulation
- container_cpu_time
- storage_zfs_use_refquota
- storage_lvm_mount_options
- network
- profile_usedby
- container_push
- container_exec_recording
- certificate_update
- container_exec_signal_handling
- gpu_devices
- container_image_properties
- migration_progress
- id_map
apistatus: stable
apiversion: "1.0"
auth: trusted
environment:
  addresses:
  - X.X.X.X:8443
  architectures:
  - x86_64
  - i686
  certificate: |
    -----BEGIN CERTIFICATE-----
    ...
    -----END CERTIFICATE-----
  certificatefingerprint: ...
  driver: lxc
  driverversion: 2.0.6
  kernel: Linux
  kernelarchitecture: x86_64
  kernelversion: 4.4.0-47-generic
  server: lxd
  serverpid: 2479
  serverversion: 2.6.2
  storage: lvm
  storageversion: 2.02.133(2) (2015-10-30) / 1.02.110 (2015-10-30) / 4.34.0
config:
  core.https_address: '[::]'
  core.trust_password: true
  storage.lvm_fstype: xfs
  storage.lvm_thinpool_name: lxdpool
  storage.lvm_vg_name: vg0
public: false

Hi, can you please attach/copy the containers config which can be founder /var/lib/lxd/$CONTAINER/lxc.conf? This is likely related to changes made in liblxc.

You meant /var/log/lxd/*/lxc.conf

I'm also a bit confused because lxc.start.auto isn't understood by LXD at all so I'm not sure how it'd have ended up in your containers' config to begin with...

Right, sorry.

The only two ways I can think of to end up with this case would be, having the container or one of its profile use raw.lxc to set lxc.start.auto or having something in /usr/share/lxc/config/common.conf.d/ which does.
The latter seems a bit more likely.

No activity in almost 3 months, closing.


Can't disable https and use http

Hi
I've installed via the Quick Install option on Ubuntu and edited the config/postal.yml configuration file and changed the protocol option under web to http.
I also edited the app/config/environments/production.rb to config.force_ssl = false
Changed the nginx configuration so to list on port 80 and not redirect to 443
But I'm still being redirected to https,
Postal rails.log give me this:
07:23:24 web.1              | => [web.1:114382] [2019-08-17T07:23:24.927] INFO -- : Redirected to https://xxx.xxx.xxx.xxx/login?return_to=%2F
07:23:24 web.1              | => [web.1:114382] [2019-08-17T07:23:24.927] INFO -- : Filter chain halted as :login_required rendered or redirected
07:23:24 web.1              | => [web.1:114382] [2019-08-17T07:23:24.927] INFO -- : Completed 302 Found in 4ms (ActiveRecord: 0.6ms)

Have you restarted both Postal and nginx (or even the entire server) since you made the changes?

Hi, yes restart and even tried a reboot, works as soon as HTTPS is enabled again.

I'm not a ruby developer.
but try enabling rails debug.
config.action_view.debug_rjs = true
config.logger = Logger.new(STDOUT)
config.logger = Log4r::Logger.new("Application Log")
grabbed from the following website.
https://guides.rubyonrails.org/v3.0/debugging_rails_applications.html

I have been using custom ports on our Postal testing server, https://postalserver:444 to make space for the click tracking.
Now if I try to go back to the normal ports, I find myself redirected to 444 essentially replicating the above issue.
Presumably there is a cache somewhere or I hardcoded the port but I can't find it any more :-/

@willpower232 where is the 30x happening? Browser, nginx/apache or ruby?
Try searching for 444 in web root?
find / -name '*.rb' -exec grep -Hn "444" {} ;

As highlighted by @gonace the redirect includes the login URI and query string which makes me think the redirect comes from within Rails, most likely here https://github.com/atech/postal/blob/634d3af59d392e84bdcd2b843b427f3a79d3ff4c/app/controllers/application_controller.rb#L19
There is no mention of the URL or 444 in the ruby files so presumably its cached somewhere more specific.
I'm willing to bet that atech, or someone who knows Rails, will know where the issue is. I hasten to add this isn't a real problem for us but someone will eventually need to move Postal to a new URL and having the old one embedded somewhere isn't a good thing.


Remove Oomph docs

Remove Oomph setup page, since it is deprecated now and subsumed by the wiki page in the Epoch repo.

Done.


Vendor extension for example parameter value

I noticed that when generating an example request, path parameters are filled with a selected value based on their type.  So, for an integer, its 0, for strings, its 'string', and so forth.  Seems like it would be useful to have a vendor extension, say x-sw2mk-example, that allowed you to specify the example parameter value that would be displayed.  It'd be relatively trivial to add (simply look for the vendor extension and use that object instead of the value based on type).
That being said, I do realize that this would be solved if swagger2markup ever fully supports OAS 3.  And, by adding a 'supported' vendor extension, you could easily open yourself up to a flood of 'extensions' in the future.

I did discover that swagger2markup supports x-example in parameter definitions.  So, this issue should be closed.


Remove Makefile because it is out of date

We are using the compose flow now and we don't want to maintain two version.
@kkunal1408 Are you okay with this?

@kkunal1408
I'm guessing that this Makefile has been updated new functionality since being fork to the compose flow (e.g., FLAT=1). We'll need to get all the changes into the new flow.

It's fine to remove this Makefile. "FLAT variable I have already been integrated in the compose flow".


Rejected Ethereum when bridging from Solana to Ethereum

Hi,
I failed to approve the bridging tx when bridging funds from Solana to Ethereum.
The funds are not in my Solana wallet anymore, but didn't arrived as i did not approve the Ethereum tx in Metamask.
The table to the bottom of the UI is not showing anything (loader keeps spinning) so I can't retry the tx from there.
How to recover my funds?

The transaction was successfully processed by the Wormhole network. Try clearing your cache and reloading wormholebridge.com to see if the transaction can be completed there.
If wormholebridge.com doesn't work, you can manually submit the signed VAA bytes below to the submitVAA method on the Wormhole contract: https://etherscan.io/address/0xf92cd566ea4864356c5491c177a430c222d7e678#writeContract

Digest: 398a41f836ce5ed67468f6ea78c3e8c722412b056100bcee0c2376b117b35114
Signed VAA bytes:

01000000010d005d1acdf37381f00d5c02f3e0c2b7f08ac0410e814864248f7b815b126a4a79d020a9a2c5bbced25f7a3bd7bdfd7bf1c1b55d9ccfb3444a5867ade77b46b40ef8000122aa637ad26387e912aa972ac7058f83cb0f67442332b63d99fe218a5ae6b25457d148e9bdab09defa25e36c260bb5e9a9cd2045baebfb9302f8f0537a18466f0103ab4426fe5edb8470ac26edb79563cae5b00f8f91e5986fe517fca416168e0edd33938c69313d93850681b336734e38318ccf6e6e2d8dccc61b45a5ba8f310b1f0005e6892c05c2b191e747629329accbaf4a96f140e1c16ba712bf379f0c2c32300a39023d7e4198ca68b58b791232d949f9360be11f93ca86ea1bd316b7812a25410006092749ec930369fd437a88760d0671d43ed0d80b4ea3414e2c696f4b962cc9b75804496800a4904ac4a75c7e9f3069021201eb7f2590abfcda1761a8c95020050107fa7c578190a89b289baae34a6af7858981e3e828376d4cdb849a14509e8b488f64dfebfc2f409f6296d94feb99cb60f83080e55795b821c5f4a711542c89a8090008ed17e93e0109e09ec76969d989a11a1f39737032df5d54568e8f8ec57b458e5b5f599c095f602cf696d5494c79edba783d0125d6c01f6aaf85c2850a2129bfc0000a522951192e10d738eb7efea3b4d0b33174a3bb3f9f2a1c18afb8d4e633c551f24a515c71d9ebc8b4a1dd264a123982912071314174369e72e92f78b8ae7d434f000be56b936bf6b0c654081805738469980c7c39fac2a23d6d4a055d92a02e2a224a7c1d70fd0ea5b541dbfc3c5c2ac706a09f13770db465a00c1e669f3c11708f44010dd88959ff1a6ea581ba4b67b873e7b6f01cbfef3e8e7ea5400359ee2761ae319732b8c322376f6cc71235c581e2245c2921e9859257b7cb4566a6f7eec295c2d80010318aa2c894982083fce22810e35c893dcbf5817a435024190c3be66d51adb2037802447e16c7a752d45ebd6862034834f1d18efd00d9eb25703afe1a15797e16001114b6a055fdd8ce52e8979022ed9fc68598e2007b71955fe4d0a49d5f478a1a392a7f7c827b2a1dde44138c7fed5153823b716f1bbddab182e2247a666179ab1001126dd8a122700af86c3f353b0ac3d636845fdd9c39f47fe8d5ed4b0f75d052ea904a1dd87dd3bd0fbd70f114cb37da18d41fecdeea52e0c0c70377a08b8158f4c500612a8a8c100000587b0102f9d283c6eda59f293bd082203cd3989f27487eb8d0732df589afffb069c11bd60000000000000000000000002992ae1837fd3e5722495d078b599e0b9c5b2cb5020000000000000000000000005228a22e72ccc52d415ecfd199f99d0665e7733b09000000000000000000000000000000000000000000000000000000013d830e2d

Solved :)


Property 'cache' does not exist

Property 'cache' does not exist on type 'Query<Document<unknown, {}, { files: DocumentArray<any> | any[] | { [x: string]: any; }[] | any[]; options: DocumentArray<any> | any[] | { [x: string]: any; }[] | any[]; key?: unknown; name?: unknown; description?: unknown; createdAt?: unknown; }> & Omit<...>, Document<...> & Omit<...>, {}, { ...; }, "findOne">'. Did you mean 'catch'? ts(2551)

This can be simply said that Query.cache() is not a function.
Versions:
"mongoose": "^7.2.2",
"recachegoose": "^9.2.0"

Reproduction
(Using Typescript)

middleware.ts

import mongoose from 'mongoose';

import cachegoose from 'recachegoose';

cachegoose(mongoose, {
  engine: 'memory'
});

fetch.ts

const result: Board = await Schema.findOne({ key: queries.id }).cache(0);

ERROR:
const result: Board = await Schema.findOne({ key: queries.id }).cache(0);
                                                                ^^^^^
Property 'cache' does not exist on type 'Query<Document<unknown, {}, { files: DocumentArray<any> | any[] | { [x: string]: any; }[] | any[]; options: DocumentArray<any> | any[] | { [x: string]: any; }[] | any[]; key?: unknown; name?: unknown; description?: unknown; createdAt?: unknown; }> & Omit<...>, Document<...> & Omit<...>, {}, { ...; }, "findOne">'. Did you mean 'catch'?

Edit:
Tried with a implementation similar to your documentation README.md. Its the same
Schema.findOne({ key: queries.id }).cache(0).exec(function(err, records) { console.log('Yay i work') });

Property 'cache' does not exist on type 'Query<Document<unknown, {}, { files: DocumentArray<any> | any[] | { [x: string]: any; }[] | any[]; options: DocumentArray<any> | any[] | { [x: string]: any; }[] | any[]; key?: unknown; name?: unknown; description?: unknown; createdAt?: unknown; }> & Omit<...>, Document<...> & Omit<...>, {}, { ...; }, "findOne">'. Did you mean 'catch'?

Have you tried with the latest version? coz there is someone has created the PR 2 weeks ago.

I'm sorry I moved to an entirely new database now. Thank you if you fixed it.


JAVA-3098: Improve ability to trace active connections across Control…

…Connection, BasicLoadBalancingPolicy, DefaultTopologyMonitor and NodeRefresh classes

Test comment


Cannot call closure returned by bindTo

Closures returned by bindTo cannot be directly called.
Source:
class a {
    public static function init() { var_dump("test"); }
}

(function () {
    static::init();
})->bindTo(null, a::class)();

Result:
[daniil@daniil-arch jav]$ php src/q.php
string(4) "test"

[daniil@daniil-arch jav]$ jppm start
-> linux
-> app:run
-> install

Fatal error: Uncaught ParseError: Syntax error, unexpected 'function' in res://q.php on line 6, position 2
Stack Trace:
#0 {main}
  thrown in res://q.php on line 6

Storing the return value of bindTo in a variable and calling that works just fine

Workaround:
class a {
    public static function init() { var_dump("test"); }
}

((function () {
    static::init();
})->bindTo(null, a::class))();```


Abhishesingh msft/continuous recognize and send dtmf

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.CallAutomation

hi @cochi2, @minwoolee-msft: can you please merge this pr? Thanks!


:technologist: Add a flux example of a setup using rig

We need an example of how a flux setup with rig works. This includes how to install and manage the rig components as well as the rig-platform gitops support.
We should update the gitops documentation and link to this example.

Fixed in https://github.com/rigdev/docs/pull/36 and the https://github.com/rigdev/flux-demo repo


Embed project monitoring metric dashboards

Summary
This embeds the project monitoring Grafana metric dashboards if they are present and user has no access to cluster monitoring.
Fixes #7286
Technical notes summary
To test:

Create a cluster
Install cluster monitoring
Install project federator
Install project monitoring into the default project
Create a user in Rancher that only has access to the default project in the cluster as a project member
Deploy a workload into the default project
Check that the limited user can see grafana dashboards under the pod detail page and workload detail page metrics tabs

Areas or cases that should be tested
Pod detail page and workload detail page.
Areas which could experience regressions
Embedding of Grafana metrics dashboards on pod and workload detail pages
Screenshot/Video

@bashofmann A merge conflict has popped up. Can you take a look? @codyrancher @rak-phillip if it's a simple can you address?

Branding e2e tests failed. Attempting another run
  1 failing

  1) Branding
       Private Label:

      Timed out retrying after 60000ms
      + expected - actual

      -'Rancher e2e'
      +'Rancher'

@bashofmann can you provide a little more detail for the testing steps?
It appears that I can't get any grafana dashboards to render in my environment. I'm not 100% sure what "Install project monitoring into the default project" and I want to make sure I'm testing this properly.

rerunning tests due to another unrelated e2e error
  1 failing

  1) User can update their preferences
       Can navigate to Preferences Page:
     AssertionError: Timed out retrying after 60000ms: Expected to find element: `.tooltip.popover.vue-popover-theme`, but never found it.
      at UserMenuPo.isOpen (https://<IP_ADDRESS>/__cypress/tests?p=cypress/e2e/tests/pages/user_menu/preferences.spec.ts:1323:34)
      at Context.eval (https://<IP_ADDRESS>/__cypress/tests?p=cypress/e2e/tests/pages/user_menu/preferences.spec.ts:1346:30)

@rak-phillip For the general project monitoring docs see https://ranchermanager.docs.rancher.com/v2.7/pages-for-subheaders/prometheus-federator-guides

Rerunning failed validate job before merging


Included Resources only included in first request

With .Net Core and a default formatter set as shown in #35, I am getting the included resources only on the first request. Additional requests to the same endpoint do not have the included resources in the returned json. I am eagerly loading the related entities and they do indeed appear in the query response before serialization every time. Is this expected behavior? Do I have something misconfigured?
Thanks!

Thanks for pointing this out.
I think I see where the issue is. Looks like the default behaviour on the formatters is to cache the Serializer (https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Formatters.Json/JsonOutputFormatter.cs#L122-L130). However the json.net ReferenceResolver is created once per serializer. The JsonApiSerializer library makes heavy use of a customer ReferenceResolver to track included items. So issue is that it is not creating a new ReferenceResolver for each serialization.
I need to investigate a bit further to either make this library work with a single ReferenceResolver or reproduce a similar issue with vanilla json.net and raise a bug against the MVC framework .
But until I work out an in library fix, you can apply a quickfix by modifing the JsonOutputSerializer to not cache the serializer
    public class JsonApiOutputFormatter : JsonOutputFormatter
    {
        private JsonApiSerializerSettings _serializerSettings;

        public JsonApiOutputFormatter() : this(new JsonApiSerializerSettings(), ArrayPool<Char>.Shared)
        {

        }

        public JsonApiOutputFormatter(ArrayPool<Char> charPool) : this(new JsonApiSerializerSettings(), charPool)
        {

        }

        private JsonApiOutputFormatter(JsonApiSerializerSettings jsonSerializerSettings, ArrayPool<Char> charPool) : base(jsonSerializerSettings, charPool)
        {
            _serializerSettings = jsonSerializerSettings;
        }

        protected override JsonSerializer CreateJsonSerializer()
        {
            return JsonSerializer.Create(_serializerSettings);
        }
    }


Record input breadcrumbs from contentEditable targets

What it says on the tin. If the target's isContentEditable property is true, then record the breadcrumb.

Totally onboard with this PR, but it needs to be covered by one of our integration tests. I can do that but it might take me some time to get around to it.

Integration test added!


expose privkey_tweak_add and pubkey_tweak_mul functions

See #1

Yes, that last point is a good one. I may be misusing the underlying function; I'm not "tweaking" it.

I believe you're, but it's returning a new one instead of modifying it in place. I think I like that more actually, thanks for submitting.

Is it necessary to call secp256k1_ec_seckey_verify after secp256k1_ec_privkey_tweak_add assuming it succeeds?

I think not, hence the comment. See secp256k1_scalar_add, there are checks there. I just wasn't 100% sure.

Oh, so the comment was about that. Ok, I'm dropping it and will publish on pypi the updated version.


Support cl.exe (MSVC)

Although we use gcc to compile our code,
We should support Windows' cl.exe (i.e. MSVC) compiler too.
Thus, when we export the finaly static library to oop.a file, we should also export it to oop.lib file
So Windows MSVC users could use our library with:
cl.exe main.c oop.lib

See https://stackoverflow.com/a/8140514/14427765
See https://mottosso.gitbooks.io/cl-exe/content/building_a_static_library.html

How to link a static library in MSVC
https://stackoverflow.com/a/23882710/14427765

UPDATE:
How to run with visual studio:
https://stackoverflow.com/a/4446095/14427765

Bug when building MSVC in x64 (while x86 works fine)
https://social.msdn.microsoft.com/Forums/en-US/c4f72667-e2a2-4902-9e28-9762918ca524/visual-studio-communityprofessional-20172019-cant-build-in-x64?forum=visualstudiogeneral


Scheduling in Operating System (Operating System Concepts)

Description
What is scheduling in Operating System?
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

It is closed cause topic already taken.


Incorrect SQL when using Linq 'Any'

_context.Sites
.Where(b => b.WorkTasks.Any(c => c.WorkProgramId == workProgramId))
.Where(c => c.IsDeleted == false)
.Count()
Generates the following SQL
SELECT COUNT(*)
FROM [Site] AS [b]
INNER JOIN [Location] AS [b.Location] ON [b].[LocationId] = [b.Location].[Id]
INNER JOIN [SiteClient] AS [b.SiteClient] ON [b].[SiteClientId] = [b.SiteClient].[Id]
INNER JOIN [SiteCategory] AS [b.SiteCategory] ON [b].[SiteCategoryId] = [b.SiteCategory].[Id]
WHERE (
    SELECT CASE
        WHEN EXISTS (
            SELECT 1
            FROM [WorkTask] AS [c]
            WHERE ([c].[WorkProgramId] = @__workProgramId_0) AND ([b].[Id] = [c].[SiteId]))
        THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
    END
) AND ([b].[IsDeleted] = 0)

When it should have the '=1' after the WHERE (..)? i.e.
SELECT COUNT(*)
FROM [Site] AS [b]
INNER JOIN [Location] AS [b.Location] ON [b].[LocationId] = [b.Location].[Id]
INNER JOIN [SiteClient] AS [b.SiteClient] ON [b].[SiteClientId] = [b.SiteClient].[Id]
INNER JOIN [SiteCategory] AS [b.SiteCategory] ON [b].[SiteCategoryId] = [b.SiteCategory].[Id]
WHERE (
    SELECT CASE
        WHEN EXISTS (
            SELECT 1
            FROM [WorkTask] AS [c]
            WHERE ([c].[WorkProgramId] = @__workProgramId_0) AND ([b].[Id] = [c].[SiteId]))
        THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
    END
) **= 1** AND ([b].[IsDeleted] = 0)

Can you share your model & dbcontext classes?

It is for a customer who would not be happy to share the model with public. Can I do this privately?

No problem. Which version of EF are you using?

{
"version": "1.0.0-*",
"description": "Apollo1.Web.Data Class Library",
"authors": [ "mark.chidlow" ],
"tags": [ "" ],
"projectUrl": "",
"licenseUrl": "",
"dependencies": {
"Apollo1.Web.Domain": "1.0.0-*",
"EntityFramework.Commands": "7.0.0-rc1-final",
"EntityFramework.MicrosoftSqlServer": "7.0.0-rc1-final",
"Microsoft.AspNet.Identity.EntityFramework": "3.0.0-rc1-final",
"Microsoft.Extensions.Configuration.Json": "1.0.0-rc1-final"
},
"commands": {
"ef": "EntityFramework.Commands"
},
"frameworks": {
"dnx451": { },
"dnxcore50": {
"dependencies": {
"Microsoft.CSharp": "4.0.1-beta-23409",
"System.Collections": "4.0.11-beta-23409",
"System.Linq": "4.0.1-beta-23409",
"System.Runtime": "4.0.21-beta-23409",
"System.Threading": "4.0.11-beta-23409"
}
}
}
}

I have a 'using AutoMapper.QueryableExtensions'. I'll check that this isn't causing the issue.

This looks similar to #4877 which is fixed in latest codebase (but causes issue in rc1 release). I will test if above query works in latest code or not.

OK I have tested and it has nothing to do with  AutoMapper.QueryableExtensions.

Repro code:
public class Program
{
	public static void Main(string[] args)
	{
		using (var _context = new BloggingContext())
		{
			_context.Database.EnsureDeleted();
			_context.Database.EnsureCreated();
			var workProgramId = 1;
			var query = _context.Sites
				.Where(b => b.WorkTasks.Any(c => c.WorkProgramId == workProgramId))
				.Where(c => c.IsDeleted == false)
				.Count();
		}

	}
}

public class Site
{
	public int Id { get; set; }
	public bool IsDeleted { get; set; }

	public ICollection<WorkTask> WorkTasks { get; set; }

}

public class WorkTask
{
	public int Id { get; set; }

	public int SiteId { get; set; }

	public int WorkProgramId { get; set; }
}

public class BloggingContext : DbContext
{
	public DbSet<Site> Sites { get; set; }

	protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
	{
		optionsBuilder.UseSqlServer(@"Data Source=(localdb)\MSSQLLocalDB;Database=test;Integrated Security=True;Connect Timeout=30");
	}
}

Generated query in latest code
exec sp_executesql N'SELECT COUNT(*)
FROM [Sites] AS [b]
WHERE EXISTS (
    SELECT 1
    FROM [WorkTask] AS [c]
    WHERE ([c].[WorkProgramId] = @__workProgramId_0) AND ([b].[Id] = [c].[SiteId])) AND ([b].[IsDeleted] = 0)',N'@__workProgramId_0 int',@__workProgramId_0=1

The issue is fixed in latest code. Feel free to re-open, if you still see issue happening after upgrading to RC2 (after it releases).
As a work-around in RC1 you can convert Any(...) to Count(...) > 0

Excellent thank you.


Fix #141: Replaced broken and old links in README.md

Issue Number
fixes #141
Describe the changes you've made
I have removed the broken links from README.md and replaced them with working links. I have also substituted the old GitLab links with the corresponding GitHub links.
Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
NA
Test plan (OPTIONAL)
NA
Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

Provide a Deployed link of route/page that needs to review
NA

Thank you @kailash360 . Merging now!


[TEVA-1819] Job alert account creation ab test

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1819
Changes in this PR

Some small design tweaks
Use Create an account instead of Continue when creating an account
Add ab test for job alert account creation prompts

Review app deployed to https://teaching-vacancies-review-pr-2913.london.cloudapps.digital


cmake: do not require dependencies on unsupported systems

Do not require dependencies on unsupported systems.
Display a warning instead.
This allows to build the native binaries, for example to run a server using exe game code, even if there is no provided deps and especially no NativeClient binaries.

Seems reasonable. lgtm


Error when following installation instructions

When I follow the installation instructions from this project, I get the following errors:
After typing: pip install -U turicreate
Could not fetch URL https://pypi.python.org/simple/turicreate/: There was a problem confirming the ssl certificate: [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:590) - skipping
Could not find a version that satisfies the requirement turicreate (from versions: )
No matching distribution found for turicreate
After typing: pip install turicreate==5.0b1
Could not fetch URL https://pypi.python.org/simple/turicreate/: There was a problem confirming the ssl certificate: [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:590) - skipping
Could not find a version that satisfies the requirement turicreate==5.0b1 (from versions: )
No matching distribution found for turicreate==5.0b1

Had the same problem. Updating pip worked for me. Good luck!

@jonhull Thanks for letting us know what the solution was. I'm closing this for now.


Parse forms mails with smarty

Maybe 5.5 Branch?
1. Why is this change necessary?
You cannot do something with the submited data like if conditions etc
2. What does this change do, exactly?
Parses mail template as smarty
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
That the field is now smarty and the Compatibility is not the best, so after update it should be updated 😈
6. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hi @shyim, thanks for your PR!
I'm sorry i have to close it, but we think it has limited benefit while adding a bit of a security risk and possible confusions for shop owners and other developers (because custom modifiers won't work here).
Sorry I don't have any better news for you! :(


Compiler Version mismatch

When using the latest GCC_ARM compile, the following is output. It isn't causing a failure but is troublesome to see displayed during compilation.
[Error] @,: Compiler version mismatch: Have 7.2.1; expected version >= 6.0.0 and < 7.0.0
ARM Version info:
arm-none-eabi-gcc (GNU Tools for Arm Embedded Processors 7-2017-q4-major) 7.2.1 20170904 (release) [ARM/embedded-7-branch revision 255204]
C

@jflynn129 That's a mirror of what we test with in CI. There are small corner cases where you can get incorrect output with Mbed OS + GCC 7. I'm not going to lay blame on which one is at fault.

ARM Internal Ref: MBOTRIAGE-1546

@theotherjimmy Given that this is not stopping/aborting compile, should this be converted to [Warning]?

If it going to be announced as an error or warning then sufficient information should be provided to allow a user to take action (or not if they choose). At this point, this is simply an annoyance to the user.

@jflynn129 That information depends on how you want to install the compiler. We could include some text like "please configure a version of GCC_ARM within the expected version range".

@theotherjimmy the problem is that most people will go to https://launchpad.net/gcc-arm-embedded/+announcement/15015 OR https://developer.arm.com/open-source/gnu-toolchain/gnu-rm and download the ARM GCC tools.  Then when you run the mbed compiler that error message is displayed.  I know that keeping up with the GCC releases is probably something you don't want to do, but rather than announce "[Error]@" it would be better to say something like "[WARNING] mbed os version x.x has only been tested with ARM GCC tools version x.x through x.x, unexpected errors may occur with later version."  or something like that.  Announcing it as an error isn't true, because everything appears to work just fine. The latter statement is more descriptive of what the problem is. That is just MO though.

@jflynn129 Thanks for the suggestion on text. I like it and I can make a PR to update the text to something similar. RE: warning/error that was requested of me. I don't recall who, but my gut says that @sg- will have an opinion on the warning/error status.

Hi, guys. So what is the solution for this issue? Downgrade the GCC compiler? Or the "ERROR" is just a warning if compile succeed nothing is going to be wrong?

Moreover, we've been using GCC7 since we started to use mbed-os 5. Was this problem only discovered recently or may it have been defect all along? What problems could we aspect from products that are already deployed?

@M-ichae-l @geoffrey-vl Your concerns are valid.

Or the "ERROR" is just a warning if compile succeed nothing is going to be wrong?

I recall at least one instance where GCC 6 emitted correct instructions and GCC 7 emitted bad instructions. I don't recall the details though, other than the device catastrophically failed (Hard fault I think) in < 1sec. At best, compile success with GCC 7 may work. As we test with GCC 6, we can only make claims about what it willl do.

Was this problem only discovered recently or may it have been defect all along?

The defect is something that I have been aware of for about 2 months prior to the PR that added that ERROR/WARNING at the beginning of the compile.

What problems could we aspect from products that are already deployed?

Given that the only defect that I'm aware of crashed the device on boot, I think you should be pretty safe. Don't take my word for it though; you should have testing in place to verify that your devices boot and function correctly.

RE: WARNING vs. ERROR: I'm open to changing the text of the message and changing it to a warning. It my observation: Mbed OS Hal's can produce many warnings, so making something like this an Error may have been requested of me (by parties I don't recall) to make a conversation like this happen.

If I were to change the text, it would be to:
"[Warning] This version of Mbed OS has only been tested with GCC version 6, and unexpected errors may occur, even when compilation succeeds, when using the currently selected version: x.x.x."
I think that's accurate to the defect we have/had and provides enough information to allow a user to make an informed decision.

That said, note that today marked the last release of GCC6 (https://gcc.gnu.org/ml/gcc/2018-10/msg00221.html).
Wouldn't that make it a good reason to support GCC7?

Is there an GitHub issue that is tracking progress towards GCC 7 support?

Hi Yanyenem,
Visit, https://developer.arm.com/open-source/gnu-toolchain/gnu-rm/downloads and then scroll down to the "6-2017-q2-update" should do it.

change file can close the compiler waring tips
mbed-os/tools/toolchains/gcc.py
"GCC_RANGE = (LooseVersion("9.0.0"), LooseVersion("10.0.0"))"


Clear button (e.g. X) inside input field (e.g. search)

Hello!
I was wondering if there is any easy way to put an clear button (e.g. X) inside of an input field. I want to do so for searching and noticed there is a clear button available, but I want something that appears in the input box on the right side when I type. Is this possible given the existing framework or do I have to attach a custom component myself. Thanks!

@RyanThomas95 I think it's possible, but not sure if it's easy to you. The only way is to custom the search input field, you can check this
But I'll not enable options.clearSearch in your case, instead, I will try to add a custom button inside the input field and handle the click event.
There are some search component customization examples.
Hope this will help you and let me know if you have any problems, thanks


Convert rewards value to real dollar value

As a user I want to see real dollar value of reward, so I know how big the reward is in USD instead of SBD.
We can just replace currently displayed rewards with USD equivalent.
Inside tooltip that shows up when reward is hovered we should display additional information:
X SteemPower
X SBD

Below that we display the same information as before.
We can use the same rate as the one used for estimated account value.
https://busy.org/steemit/@trafalgar/can-we-change-the-ui-to-display-rewards-in-usd

I think we shouldn't freeze payout after it's paid. We shouldn't assume that users paid out immediately. They can hodl as well.

Are we planing something like this for payout hover state?


auto integrate of youtube-dl on server

i am developing a package to editing video and i need to integrate youtube-dl library with package installation. package developed in PHP. can we do auto integration of library on server

this can be done simply calling exec function in php.   sample code :

other projects based on similar ideas are https://github.com/labzone/Youtube-dl-php ,https://github.com/norkunas/youtube-dl-php.

Thanks @siddht1. For further discussions, please move to #152.


Update Pill styles in Salt (Next) Components

Due to the changes to Button the variable styles used by the Pill component also need to be updated.

Pill component in Salt (Next) component library updated. Branch merged, but not yet published.


Some way of updating phone when ipad makes post (or vice versa)

pull to refresh?
push notifications?

push notifications:
https://facebook.github.io/react-native/docs/pushnotificationios.html
https://developer.apple.com/library/ios/documentation/IDEs/Conceptual/AppDistributionGuide/AddingCapabilities/AddingCapabilities.html#//apple_ref/doc/uid/TP40012582-CH26-SW6
https://parse.com/tutorials/ios-push-notifications


api: support IdleTimeout in clientTrafficPolicy

What type of PR is this?
Support idle timeout in ClientTrafficPolicy.
Fixes #2861

one nit to fix godoc strings, else looks good !

/retest


Unable to install matplotlib-base from pypi

matplotlib-base seems to be conda only.
$ pip install matplotlib-base
ERROR: Could not find a version that satisfies the requirement matplotlib-base (from versions: none)
ERROR: No matching distribution found for matplotlib-base

Forgot to close. #161


final implementation

updated run.py to use bauhaus, nnf, sat solver. the code works.

Don't think this is the target repository you're looking for :P


Troubleshooting for LoRA Fine-tuning of MiniCPM-V-2.5 -- ERR: (FAILED: multi_tensor_adam.cuda.o & fused_adam.so: cannot open shared object file: No such file or directory)

Environment Details

OS: Ubuntu 22.04
NVIDIA Driver: 535.54.03
CUDA Version (nvcc -V): 12.2
Python Version: 3.10
PyTorch Version: 2.3.0+cu121
GCC Version: 13 (initially)

Issue Description
Encountered errors when attempting to use DeepSpeed for LoRA fine-tuning of MiniCPM-V-2.5.
Error 1: Ninja Build Failure (FAILED: multi_tensor_adam.cuda.o)
Error Message
FAILED: multi_tensor_adam.cuda.o
ninja: build stopped: subcommand failed.

Root Cause
Issue with the ninja build command in PyTorch's C++ extension compilation process.
Solution
Modified the ninja command in the PyTorch utility script:
File: /envs/xxx/lib/python3.xx/site-packages/torch/utils/cpp_extension.py
Change: ['ninja', '-v'] to ['ninja', '--version']
Error 2: Shared Object File Not Found (fused_adam.so)
Error Message
ImportError: /home/xxxxx/.cache/torch_extensions/py310_cu121/fused_adam/fused_adam.so: cannot open shared object file: No such file or directory

Attempted Solution (Unsuccessful)
pip uninstall deepspeed
DS_BUILD_FUSED_ADAM=1 pip install deepspeed

Successful Solution

Clone DeepSpeed repository:
git clone https://github.com/microsoft/DeepSpeed.git
cd DeepSpeed

Install with specific build flags:
DS_BUILD_UTILS=1 DS_BUILD_FUSED_ADAM=1 pip install .

Resolved CUDA and GCC version conflict:

Lowered GCC version to 11.3

Reinstalled DeepSpeed:
DS_BUILD_UTILS=1 DS_BUILD_FUSED_ADAM=1 pip install .

Outcome
After implementing the above solutions, the DeepSpeed installation was successful, and the LoRA fine-tuning code ran without errors. Hope that helps a little.

Thank you！


sidebar

Sidebar can be a really cool feature for this to be included :)

Looking at @nixmd's page, this seems to be a satisfactory answer. Closing.


is it possible to specify a license file from within Julia?

So I have a package which uses gurobi which I'm trying to make a lot more portable for containerization purposes, but Gurobi has been a thorn in my side when trying to do this in a number of ways.
It seems like the license file must be specified by GRB_LICENSE_FILE and that this must happen in the shell.  This makes it impossible for me to build a truly portable container, because it would need to have different license files in different contexts.
It doesn't seem like the Gurobi API exposes this selection, but I thought I'd ask, is there any way of doing this?

The real issue you have is the need to generate the license via grbgetkey, which ties the license to a particular host.
Does setting GRB_LICENSE_FILE via ENV["GRB_LICENSE_FILE"] = " ... " before you run using Gurobi not work?

My plan was to keep all licenses with my package and then have it select them depending on the host.  This would allow me to, e.g. deploy docker images at different locatiosn (where gurobi is allowed to run by virtue of the licenses) seamlessly.  Gurobi is really not cooperating with my trying to do this, as I also wanted to set GUROBI_HOME via Julia Pkg artifacts, but the check for GUROBI_HOME during the build step of this package seems to preclude that.
Ideally, the thing I most want is to be able to do a ]add SomePackage from any docker container where gurobi is allowed to run and have it "just work", but gurobi is presenting lots of obstacles to this.

Feel free to modify the build.jl.

but gurobi is presenting lots of obstacles to this.

I think this is by design.
I take it there is a limited number of places this is going to be deployed to? Couldn't you just figure out where the default license file location is, then copy the gurobi.lic file to that location on install.

Yeah, what you're saying is certainly doable, I'm able to get portable containers but gurobi is mostly preventing me from making a truly portable package.  This isn't the end of the world, but complicates the environment build process (I have multiple containers across multiple projects).
Ok, not sure there is anything to be done here, but I may make a PR to modify the build.jl to make things a little easier if I can think of a way of doing that without breaking anything.

This likely isn't a new request for Gurobi. I'd recommend contacting them directly.

Yeah, my company has a lot of licenses with them, so we get their attention, I may re-open this conversation if I get too aggravated.
We've had discussions about these sorts of IT issues with them in the past, but they weren't particularly helpful (even though, as you said, it was clear we weren't the only people they were hearing this from).  It's gotten bad enough that my colleagues and I have semi-seriously considered starting to look for Gurobi alternatives, even though Gurobi itself is obviously fantastic as a solver.
My hope was that eventually they'd realize that all of their clients are doing "microservices" and need to make things easily portable and containerizable, and their current setup is a big obstacle to that, so they'd move toward trying to make things easier, but I don't think anything has changed in the last few years.
There's also the related issue that the current system strongly incentivizes using "compute servers".  All of this has been a nightmare for trying to set up proper test environments, or even to do a staged upgrade of Gurobi binaries.
TL;DR: Gurobi: fantastic solver, terrible for IT issues.

Closing since this is not a Gurobi.jl issue. Please re-open if there is any movement upstream.


[Bug]: Skip unsupported transformation of DocumentClient

Self-service

[ ] I'd be willing to implement a fix

Describe the bug
The v2-to-v3 transform unsuccessfully tries to transform DocumentClient
Steps to reproduce
Input code:
import DynamoDB from "aws-sdk/clients/dynamodb";

const docClient: DynamoDB.DocumentClient = new DynamoDB.DocumentClient();

Observed behavior
Output code
import AWS_DynamoDB from "@aws-sdk/client-dynamodb";

const docClient: AWS_DynamoDB.DocumentClient = new DynamoDB.DocumentClient();

Expected behavior
Ideal behavior would be to support transformation for DocumentClient, but it's tracked in https://github.com/awslabs/aws-sdk-js-codemod/issues/51
While it's supported, the transformation of DocumentClient should not happen.
Environment
aws-sdk-js-codemod: 0.10.5
- jscodeshift: 0.14.0
- recast: 0.21.5

Additional context
No response

Skipping unsupported transformation in service module import/require or global require property would need either renaming the identifiers after transformation, or renaming current identifiers whose transformation is not done.

renaming the identifiers after transformation

import DynamoDBClient from "aws-sdk/clients/dynamodb";

const client = new DynamoDBClient();

becomes
import { DynamoDB as DynamoDB_transformed } from "@aws-sdk/client-dynamodb";

const client = new DynamoDB_transformed();

renaming current identifiers whose transformation is not done

import DynamoDB from "aws-sdk/clients/dynamodb";

const docClient: DynamoDB.DocumentClient = new DynamoDB.DocumentClient();

becomes
import DynamoDB_old from "aws-sdk/clients/dynamodb";

const docClient: DynamoDB_old.DocumentClient = new DynamoDB_old.DocumentClient();

It would be easier to prioritize DynamoDB transformation itself in https://github.com/awslabs/aws-sdk-js-codemod/issues/51
In the meantime, if you have any files where DocumentClient is created using service import/require or global require property, the workaround would be to either not run transformation on those files or unstage those files after transformation.

Re-opening, so that issue is not locked.
We can close it after transformation for DynamoDB DocumentClient is added.

Support for DocumentClient was added in https://github.com/awslabs/aws-sdk-js-codemod/pull/450, and released in https://github.com/awslabs/aws-sdk-js-codemod/releases/tag/v0.14.0


base validator

Variable Validator
Overview
The variableValidator class is designed to provide a flexible and robust mechanism for validating input data against a set of predefined rules. It's particularly useful in contexts where inputs need to be dynamically validated, such as in template processing or API input validation.
Features

Customizable Validation Rules: Set specific validation functions for individual variables or define a default validation function applicable to all variables not explicitly covered.
Exclusion Mechanism: Specify variables that should be excluded from validation, offering flexibility for diverse input scenarios.
Dynamic Rule Management: Add or remove validation rules for variables on-the-fly, adapting to changing requirements.

Usage
To use variableValidator, initialize an instance with an array of variable names you intend to validate and an optional set of rules.
Initialization
import { variableValidator } from './path/to/variableValidator';

const validator = new variableValidator(['name', 'age'], {
    default: value => typeof value === 'string',  // Default rule
    specific: {
        age: value => typeof value === 'number' && value >= 18  // Specific rule for 'age'
    },
    exclude: ['nickname']  // Exclude 'nickname' from validation
});

Adding and Removing Rules
You can dynamically add or remove validation rules for specific variables or modify the exclusion list:
import { variableValidator } from './path/to/variableValidator';
// Add a specific rule
validator.addSpecificRule('email', (input) => /\S+@\S+\.\S+/.test(input));

// Remove a specific rule
validator.removeSpecificRule('username');

// Add a variable to the exclude list
validator.addExcludeVariable('token');

// Remove a variable from the exclude list
validator.removeExcludeVariable('password');

Validating Input
To validate an input object against the defined rules, use the validate method. It returns a ValidationResult object indicating whether the validation passed and an optional error message:
const input = { username: 'user1', password: 'pass', email<EMAIL_ADDRESS>};
const result = validator.validate(input);

if (!result.isValid) {
    console.error(result.errorMessage);
} else {
    console.log('Validation passed!');
}

Please add documentation of the BasePromptTemplate and Validator. It would be very helpful if use cases are included. Thanks!

Base Prompt Template
Introduction
The basePromptTemplate is a core component designed to streamline the integration and manipulation of dynamic content within template strings. It serves as the foundation for creating customizable text outputs that adapt based on specific input variables, making it an essential tool for applications that require dynamic text generation, such as automated messaging, configurable email templates, or personalized user interfaces.
A base prompt template enables developers to define a template with placeholders that can be programmatically replaced with actual data at runtime. This approach allows for a high degree of flexibility and reusability in applications where the content needs to be tailored to individual users or contexts without hardcoding every possible variation.
Use Cases

Automated Email Generation: Automatically generate personalized email content for marketing campaigns, user notifications, or system alerts by populating predefined templates with user-specific data.
Content Management Systems (CMS): Enable non-technical users to define templates for various content types, such as blog posts or product descriptions, that can be dynamically filled with actual data from a database.
Chatbots and Virtual Assistants: Use templates to generate varied responses based on user input or conversation context, enhancing the natural language processing capabilities of the bot.
Configuration Files: Dynamically generate configuration files for software or hardware based on user inputs or environmental variables, ensuring that each deployment is correctly configured.
Code Generation Tools: Simplify the process of generating boilerplate code for new projects or modules by defining templates that include common patterns and structures, filled in with specific details for each use case.

The basePromptTemplate class facilitates these and many other use cases by providing a robust framework for working with template strings. It includes features for input validation, template manipulation (such as adding prefixes and suffixes), and error handling to ensure that templates are correctly processed and output as expected.
Features

Dynamic Template Processing: Easily define and manipulate template strings with dynamic variables.
Input Validation: Leverage the integrated variableValidator to ensure input variables meet specified validation rules.
Flexible Template Manipulation: Add prefixes or suffixes to your template strings for enhanced customization.
Error Handling: Rigorous checks and error messages for undeclared or unused variables in the template.