Step 65 - lambda function trigger "invocation error"

I am getting on the lambda function: Batch size: 1Last result: PROBLEM: Function call failed
I think without a Node "console", debugging gets tricky - will be nice to have some kind of console to help debug as a feature in AWS Lambda.
Regardless, any idea if the Node function having some problem, or somewhere else?

Hi! Do you see any logs in CloudWatch Logs for your function with any
further details?
On Fri, Jan 27, 2017 at 12:18 AM ZUnshakable<EMAIL_ADDRESS>wrote:

I am getting on the lambda function: Batch size: 1Last result: PROBLEM:
Function call failed
I think without a Node "console", debugging gets tricky - will be nice to
have some kind of console to help debug as a feature in AWS Lambda.
Regardless, any idea if the Node function having some problem, or
somewhere else?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-serverless-website-workshop/issues/2, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABNPySqHUf11MHH6f2HjvvNow7H-OPIks5rWX4jgaJpZM4LvcUy
.

Ahh... I was new to the CloudWatch and was not aware of the log. Here is the exception I have:
{
"errorMessage": "Email address is not verified. The following identities failed the check in region US-WEST-2<EMAIL_ADDRESS>"errorType": "MessageRejected",
"stackTrace": [
"Request.extractError (/var/runtime/node_modules/aws-sdk/lib/protocol/query.js:40:29)",
"Request.callListeners (/var/runtime/node_modules/aws-sdk/lib/sequential_executor.js:105:20)",
"Request.emit (/var/runtime/node_modules/aws-sdk/lib/sequential_executor.js:77:10)",
"Request.emit (/var/runtime/node_modules/aws-sdk/lib/request.js:668:14)",
"Request.transition (/var/runtime/node_modules/aws-sdk/lib/request.js:22:10)",
"AcceptorStateMachine.runTo (/var/runtime/node_modules/aws-sdk/lib/state_machine.js:14:12)",
"/var/runtime/node_modules/aws-sdk/lib/state_machine.js:26:10",
"Request. (/var/runtime/node_modules/aws-sdk/lib/request.js:38:9)",
"Request. (/var/runtime/node_modules/aws-sdk/lib/request.js:670:12)",
"Request.callListeners (/var/runtime/node_modules/aws-sdk/lib/sequential_executor.js:115:18)"
]
}
Maybe some issue with my Conigto_UnAuth role?

Gotcha. It's more likely that your email address is not verified via SES. If you go to the SES console and verify that email address, the function should succeed. You should be able to follow the first few steps of lab 2 to do this. After that, you may need to delete and recreate the trigger.
Hope that helps!


Using static directives must be placed after other using directives

I would like to propose a new diagnostic that covers a new language feature of C# 6:
Category: Ordering
Rule: Using static directives must be placed after other using directives
Description: A violation of this rule occurs when a using static directive is placed before a normal or an alias using directive. Placing the using static directives together below all other using-directives can make the code cleaner and easier to read, and can help make it easier to identify the static members used throughout the code.
Rationale: This proposed rule is an extension in line with rules SA1208 and SA1209 to cover a new language feature in C# 6. My proposal is to put the using static directives after the existing using directives in order to have minimal disturbance to patterns that developers are used to.

:+1: This seems consistent with where the C# language service adds them too, so it passes the "IDE default behavior" test.

I agree on this as well. It's obviously non-breaking since it's a new language feature.

Grabbing this one for implementation


Bot failing

Needs investigation. Reported by @Septimus via Discord.

Bot Crashing
if (guild.channels.exists("name", config.auditChannel) && config.adminTransparency) {
                   ^

TypeError: guild.channels.exists is not a function

Marking as "urgent" because it is crashing the bot.

This appears to be the faulty line:
https://github.com/campDevs/DiscordBot/blob/6d2dcb0773c3e626e318191a3f505dc80fcf14fd/src/bot.js#L65

Based on the error message, we know that

guild.channels exists
guid.channels.exists might exist, but it might not -- but it's not a function.

Clearly, there's a malformed guid object. It's true that .exists is currently deprecated. However, it does appear to still exist. We should file a non-urgent issue to migrate away from this later.
To find this malformed guild object, we should take a look at how each event will respond, both in a server and in a DM.

[ ] guildMemberRemove
[ ] guildBanRemove
[ ] guildBanAdd

All looked okay. However, it's recommended to test guild.available before assuming their data is intact:

If it is not available, it indicates a server outage

Could this be the issue?

An update to the current hypothesis has been made. It is available in the first comment of this issue, as well as here.

Collection#exists was removed in master but remains in stable. The hypothesis is that the resolution of #88 will resolve this issue; because an unstable version is currently being used when attempting to run.


[Timeline] Video playing transition is not smooth

When playing a video in the timeline, the progress is not smooth. The screen goes through the following intermediate phases:

Or see the video:
https://github.com/android/socialite/assets/6952116/f3974d65-0b77-4ff0-af94-18a63cceb268

I'm seeing something similar but the video gets stuck playing on the left side of the card only (similar to image 5 above), which squishes the content:


Frontend_with_Framework: React 7.1 -> State Management in React

Description 📜
Created documentation for State Management in React.
Fixes #7590

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)
[x] 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
[x] Frontend Dev React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] 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) 📸

@heet-kumar Please review.

@heet-kumar  Done the required changes.

@anandhu720 once review this PR from your side.


Update cron module to latest

Fix for https://github.com/kelektiv/node-cron/issues/414 looks like it was done here:
https://github.com/kelektiv/node-cron/commit/57632b0c06c56e82f40b740b8d7986be43842735

thanks


getWallets() generates the same wallets regardless of ganacheOptions

Hey guys,
Just wanted to tell you I lost an awkwardly innumerable amount of hours trying to get MockProvider to do something very simple:
Creating more than 10 funded wallets
At first I was using Waffle 3.4.0. I noticed that following the docs to the letter wouldn't work:
let ganacheOptions = {
    total_accounts: 100,
}
const provider = new MockProvider({ ganacheOptions })
const wallets = provider.getWallets()

So I migrated to Waffle 4.0.0 (updating the necessary dependencies of course), and made the proper changes to ganacheOptions as thoroughly docummented by means of a link to Ganache code GitHub:
let ganacheOptions = {
    wallet: {
         totalAccounts: 100,
    }
}
const provider = new MockProvider({ ganacheOptions })
const wallets = provider.getWallets()

Which [you guessed it] didn't change the behavior of MockProvider whatsoever, because MockProvider would just create the 10 default accounts... But then I tried:
const accounts = Array(100).fill({ balance: '0xff' })
let ganacheOptions = {
  wallet: {
    accounts,
    defaultBalance: 100,
  },
}

Which would yield the following error:
/home/.../node_modules/ethers/node_modules/@ethersproject/bytes/src.ts/index.ts:58
    return !!(value.toHexString);
                    ^
TypeError: Cannot read property 'toHexString' of undefined
    at isHexable (/home/.../node_modules/ethers/node_modules/@ethersproject/bytes/src.ts/index.ts:58:21)

Fantastic! That's some progress right?....
Ended up adventuring all the way into the woods: I read through the unbundled code and added logs to several lines. That made me realize the docs were a lie all this time! This is the implementation for getWallets():
    getWallets() {
        var _a, _b, _c;
        const items = (_c = (_b = (_a = this.options) === null || _a === void 0 ? void 0 : _a.ganacheOptions.wallet) === null || _b === void 0 ? void 0 : _b.accounts) !== null && _c !== void 0 ? _c : defaultAccounts_1.defaultAccounts;
        return items.map((x) => new ethers_1.Wallet(x.secretKey, this));
    }

First point: WHY?
Second: This piece of code will NEVER generate accounts either from a seed, or a mnemonic, or anything whatsoever. Nedless to say, passing options object with a config that tells the damn thing to create whatever number of accounts will be utterly and epicly ignored. Someone had previously talked about this but just addressed the case of the mnemonic.
At this point I contemplated pivoting my career into content-writing or something like that, but then I remembered the fact that Waffle's MockProvider was just a wrapper for Ganache provider, and I could in fact cut the corner and instantiate Ganache provider myself [Thank you very much Waffle]:
const accounts = Array(100).fill({ balance: '0xff' })
let ganacheOptions = {
  wallet: {
    accounts,
  },
}
const ga = ganache.provider(ganacheOptions).getInitialAccounts()
const wallets = ga.map(x => new Wallet(x.secretKey))

I though Waffle was supposed to take complexity out of using Ganache directly, but it looks like we're all better of just diching MockProvider and using Ganache directly to generate the wallets.
Just FYI

Hey @cNoveron,
I'm sorry you had a hard time with this, and thank you for the detailed writeup.
I have pushed a fix in https://github.com/TrueFiEng/Waffle/pull/770 with missing tests for getWallets() against different ganacheOptions.wallet config.


Update references in custom interpreter notebook

Looks good, thanks! The CI failure issue should go away if you rebase on master.

Thanks! Just rebased.


Error when updating metafields on a product

When I try to do this:
shopify_product = ShopifyAPI::Product.find(123456)
shopify_product.metafields = [{
    key: "someKey",
    value: "new value,
    value_type: "string",
    namespace: "MyNamespace"
}]
shopify_product.save

I'm getting this error: :key=>["must be unique within this namespace on this resource"]
This namespace/key already exists on this product, I'm just trying to update its value to "new value", but this error is preventing it. How do I do this?
#419 could be related.

This method worked for me, but uses extra API calls. Is there a way to make my original request work?
metafield = ShopifyAPI::Metafield.find(:first, :params=>{:resource => "products", :resource_id => 123456, :namespace => "MyNamespace", :key => "someKey"})
metafield.value = "new value"
metafield.save

This isn't ideal, because in my full code (I only posted a snippet for clarity above), I'm updating other order properties such as the product title and product vendor, and it would be ideal to only have to hit the API at ShopifyAPI::Product.find(), instead of having to also hit ShopifyAPI::Metafield.find() for all of the metafields I need to update, which is actually 3 of them in my actual scenario.
So I'm having to go from this:
product = ShopifyAPI::Product.find()
product.save

to
product = ShopifyAPI::Product.find()
product.save

metafield1 = ShopifyAPI::Metafield.find()
metafield1.save

metafield2 = ShopifyAPI::Metafield.find()
metafield2.save

metafield3 = ShopifyAPI::Metafield.find()
metafield3.save

Unless I'm missing something.

Hi, @skillmatic-co! That sounds super frustrating. I'll take a look into this for you and see if I can find you a better solution.

Something I did think of, if you're willing to delve into GraphQL, you may be able to use [this productUpdate mutation] (https://shopify.dev/docs/admin-api/graphql/reference/products-and-collections/productupdate) to update multiple metafields in a single request. This mutation accepts both metafields and privateMetafields as a part of the mutation input fields.

Same problem for me, I can confirm the issue

Unfortunately the REST API doesn't support the action of updating metafields through the Product resource, you're only able to create new ones. Let me know if there are any more questions! I'll be closing the issue for now since this isn't a shopify_api bug.

Well, can it become a feature request? ;)

Pass this suggestion along to to Partner Support! :)

Shopify API should really support this. It takes extreme overhead to update a single entity.

I think I could solve it!
First I got the id of the metafield like this:
endpoint = "metafields.json?metafield[owner_id]= (customer_id) &metafield[owner_resource]=customer"
Then with help of this reference: https://shopify.dev/api/admin-rest/2021-10/resources/metafield#put-metafields-metafield-id , I could update my customer's metafield.
endpoint = "customers/" + (customer_id) + "/metafields/" + (metafield_id) + ".json"  json = {"metafield":{"id":(metafield_id) ,"value": (metafield_value),"type":"single_line_text_field"}}  response = requests.put(url + endpoint,json=json)
I hope it helps!

Hello guys. This is Bernard.
There is a way to update the metafields without so many problems. I hope my example helps you.
To update the metafield through the customer API it is necessary to pass the id of the metafield that will work.
Enter in:
{domain}/admin/customers/{customer_id}/metafields
So that all metafields created for the customer are listed. In this list, you can find out which is the id of the metafield that you need to update.
Then just run this code snippet that updates the customer data:
const body = {
	customer: {
		id,
		first_name,
		last_name,
		phone,
		metafields: [
			{ id:<PHONE_NUMBER>8307, namespace: "my_fields", type: "date", key: "bithday", value: bithday }
		],
	}
}

try {
	const response = await this.shopifyStore.put({
		path: `/admin/api/2022-04/customers/${id}.json`,
		data: body,
		type: DataType.JSON
	});

	const customer = response.body;
	console.log(customer);

} catch (error) {
	throw new Error(error.message);
}

Once there is a namespace/key combination for a product, you need to send the ID of the metafield along with the changes. Otherwise, you will get the "Key must be unique within this namespace on this resource" error. The best is to store the metafield ID locally when you first create the product.

If you are not updating other fields of the product then you can use metafieldsSet instead.
https://shopify.dev/docs/api/admin-graphql/2024-01/mutations/metafieldsset
Sets metafield values. Metafield values will be set regardless if they were previously created or not.
Manage metafields section of the documentation doesn't even mention this mutation.


Archetype: Martial Artist / Mauler / etc. and Fighter - Weapon Proficiency not increasing

If you build a fighter with an Archetypem like Mauler or Martial Artist, which increases your proficiency in certain weapons if you have a class feature that does so for other weapons, those proficiencies are not applied currently.
For example:
Fighter Level 5, Selected Weapon Group: Swords
Martial Artist Archetype
Unarmed Strikes should have Master Proficiency, but are listed as only Expert

Duplicate of #7195


Update README.md : blocking example

Hey!
I'm new to Rust. So, I might be wrong about this.
Trying to used the line in the README to install with the blocking feature gave me an error. I think this fixes it.
Thanks for your time,
Nate

Yes, that's true, thank for the hint and the PR, have merged your changes.


Bot API 5.0

Squashed fixes, implemented proposed fixes.

Hi, When does this PR apply?

Added baseUrl param to constructor (needed for private bot api servers) tested different methods with it... it works...
EXCEPT DownloadFile method when server runs in --local mode. in that case it will give you local link to local file, for example on ubuntu:
/home/ubuntu/telegram-bot-api/bin//documents/file_2
And we cant modify DownloadFileAsync because .NetStandard 1.1 doesnt support any file operations...

When will this PR be merged? :)

Any ETA for new release?


Created ChiragKhatri.json

Fixes Issue
Created data/chiragkhatri19.json
Changes proposed

Check List (Check all the applicable boxes)

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

Screenshots

Note to reviewers

your file name needs to be same as you Github username. you need to change it to data/chiragkhatri19.json. Thank You!

Did both the changes


DMIdecode - missing information

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs
Describe the bug
Brand new installation of OPNsense v20.1 on a new hardware build, restored config from old firewall. After installing the os-dmidecode plugin, the widget is missing a great deal of information.
To Reproduce
Steps to reproduce the behavior:

Install os-dmidecode
Visit lobby dashboard

Expected behavior
More than what is being shown
Screenshots

Relevant log files
Tell me where they are or how to generate logging from the plug-in and I will add it to the report.
Additional context
My previous hardware was an older Jetway SoC build, and it had only a little bit more showing, which is why after this new build I thought I would be report it. I appreciate that the information may not actually be there to obtain, but I'd like to know for sure, otherwise I'll just remove the plug-in.
Environment
Jetway JNF795T2-Q170 with i3-7100, 8GB RAM, 60GB SSD
OPNsense 20.1.8_1-amd64
FreeBSD 11.2-RELEASE-p20-HBSD
LibreSSL 3.0.2

@Taomyn can you provide the output of these two:
/usr/local/sbin/dmidecode -q -t system
/usr/local/sbin/dmidecode -q -t bios

Cheers,
Franco

Here you go, I did obfuscate the UUID but the rest is from the commands:
root@bart:~ # /usr/local/sbin/dmidecode -q -t system
System Information
		Manufacturer: _
		Product Name: _
		Version: _
		Serial Number: _
		UUID: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
		Wake-up Type: Power Switch
		SKU Number: _
		Family: _

System Configuration Options
		Option 1: _

System Boot Information
		Status: No errors detected

root@bart:~ # /usr/local/sbin/dmidecode -q -t bios
BIOS Information
		Vendor: American Megatrends Inc.
		Version: 5.12
		Release Date: 12/06/2019
		Address: 0xF0000
		Runtime Size: 64 kB
		ROM Size: 16 MB
		Characteristics:
				PCI is supported
				BIOS is upgradeable
				BIOS shadowing is allowed
				Boot from CD is supported
				Selectable boot is supported
				BIOS ROM is socketed
				EDD is supported
				5.25"/1.2 MB floppy services are supported (int 13h)
				Print screen service is supported (int 5h)
				8042 keyboard services are supported (int 9h)
				Serial services are supported (int 14h)
				Printer services are supported (int 17h)
				ACPI is supported
				USB legacy is supported
				BIOS boot specification is supported
				Targeted content distribution is supported
				UEFI is supported
		BIOS Revision: 5.12

BIOS Language Information
		Language Description Format: Long
		Installable Languages: 1
				en|US|iso8859-1
		Currently Installed Language: en|US|iso8859-1

Looks like dmidecode can't gather any data on the platform. Can't do anything about this from a plugin level unless dmidecode needs to be started with a different command sequence or something?

I've sent an email to Jetway support to see if they can explain if it's on the BIOS side, will update here if I get a response


Fix cronjob example

In the case of "* */5 * * *", it runs every minute past every 5 hours, so it is not true that it runs every 5 hours.
To solve it set 0 as minutes (0 */5 * * *)
(may the documentation of default value is also incorrect)

Unfortunately this typo is in the code itself at https://github.com/k3s-io/k3s/blob/454440f9a3b585bd038293f9de5acb470380c3fa/pkg/cli/cmds/server.go#L342 - it should be changed there, at which point we can update the help text in the docs.


Stolen artwork

Hey guys,
You stole (an old version of) my artwork and made and modifications without notification or attribution.
Yours:
https://github.com/soundbio/ministat-1/wiki
https://github.com/hivebio/ministat-1/raw/master/Docs/Images/MinistatOverview.png
Mine:
http://depts.washington.edu/soslab/turbidostat/pmwiki/pmwiki.php?n=Main.About
http://depts.washington.edu/soslab/turbidostat/pmwiki/pub/uploads/Main/About/overview.png
http://pubs.acs.org/doi/abs/10.1021/sb500165g
Had you asked I wouldn't have minded but I now have a few issues

It's based on an older slightly uglier version that was on my wiki vs what was improved for the paper.
You didn't ask and that's not very polite.

Can you please remedy this.
Chris

Hello.
HiveBio is not the owner of the listed repository anymore. It has been
deleted from our repositories.
The people who worked on this project have moved to SoundBio.
Please continue to work with SoundBio on the ministat issue.
Thank you.
On Feb 10, 2017 12:42 PM, "Chris Takahashi"<EMAIL_ADDRESS>wrote:

Hey guys,
You stole (an old version of) my artwork and made and modifications
without notification or attribution.
Yours:
https://github.com/soundbio/ministat-1/wiki
https://github.com/hivebio/ministat-1/raw/master/Docs/
Images/MinistatOverview.png
Mine:
http://depts.washington.edu/soslab/turbidostat/pmwiki/
pmwiki.php?n=Main.About
http://depts.washington.edu/soslab/turbidostat/pmwiki/pub/
uploads/Main/About/overview.png
http://pubs.acs.org/doi/abs/10.1021/sb500165g
Had you asked I wouldn't have minded but I now have a few issues

It's based on an older slightly uglier version that was on my wiki
vs what was improved for the paper.
You didn't ask and that's not very polite.

Can you please remedy this.
Chris
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/soundbio/ministat-1/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/APlpUkJ_Yn-eyMzplYxM0DT3FO1ZVnPEks5rbMvOgaJpZM4L91QW
.

This as a github.com/soundbio repository.  Is this not Soundbio's repo?

I apologize. I will stand down. I saw a hyperlink at the top for
GitHub.com/HiveBio for a raw image.
On Feb 10, 2017 12:56 PM, "Chris Takahashi"<EMAIL_ADDRESS>wrote:

This as a github.com/soundbio repository. Is this not Soundbio's repo?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/soundbio/ministat-1/issues/3#issuecomment-279063635,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APlpUqRjqTjPSYzl9APJBbSsCLmOt6dmks5rbM8IgaJpZM4L91QW
.

Chris,
I was involved with the ministat project when it was at Hivebio. Right now the project is moved to SoundBio, but inactive.
I'm not the culprit who modified the image and put it up on the wiki, but before I chase the culprit down can you provide a URL to the document that contains your artwork we "stole"? In our wiki page we name the open source projects published by the Klavins and Dunham labs as our inspiration. If your image was part of the documentation of either of these projects, I speculate that naming these projects was considered attribution. If the image came from somewhere else, then I don't know about it.
In either case, I'm happy to add whatever attribution you want or even take the image down. If you want attribution added, please provide verbiage so I have to modify this only once.
Thanks,
Paul Schnackenberg

I just found the image. I started navigating from Klavin's lab website documenting the 'stats. I clicked on one of the derivative 'stats and then the about page for that. So, I imagine the open source nature of the project, mention on the wiki of Klavin's lab and the link to Klavin's website was considered attribution. (BTW, the link to the license on Klavin's lab homepage is broken.)
Regardless, please provide the text or remedy you want to see.

The links are in the above issue.
The problem with "inspired by" is that the artwork was derived and not simply an inspired work or "original re-implementation".  In general, though I think "inspired by" is the right description for the project and that shouldn't change.
The licence doesn't cover artwork, just designs and code.  Also no OS licence I know of allows for copying without reproducing the original licence and appropriate attribution.
But the above are really just technicalities.  Personally I consider artwork a separate animal especially when related to academic work.  The last thing I want is for some entity to claim I'm not the copyright owner to my own work because of some other image on the internet.
In any case the person who used the image has contacted me and we're working it out.  I think the final result should be better for everyone and quick to resolve.
Thank you for giving this your attention though.  I really appreciate it.

@turnipsoup
huh.  apparently the soundbio github page links to the hivebio image.  Probably some residual from the migration as I've seen a mix of hivebio and soundbio text/links.

Glad Chuck (I assume) contacted you. If you have access to Klavin's site, it would be good to fix the link to the license on their homepage since there's no ability to find out exactly what it covers right now. If it doesn't already, it might be worth adding a paragraph about copyright of web properties just to remove any ambiguities.
FWIW, your work on the 'stats and documentation are all top-notch.

Glad Chuck (I assume) contacted you. If you have access to Klavin's site, it would be good to fix the link to the license since there' no ability to find out what it covers right now.
FWIW, your work on the 'stats and documentation are all top-notch. Congratulations on

The links to images on the hivebio site are residuals left from the migration. Wiki markup was copied over unmodified, so links still point to the hivebio ministat repository which apparently still exists. When I (or somebody) gets to it we'll modify the wiki markup to point to soundbio repository.
In the meantime, just ignore reverences or links to Hivebio. Hivebio is not responsible nor involved in this project.


Add a name to a Sunalyzer instance

This PR introduces a new parameter in the config.yml to specify a custom name for each Sunalyzer instance. This name is shown on the website in both the titlebar and html title.

@BorisBrock not sure if you want to have this feature, however as I've added it to my internal fork of sunalyzer I'd like to offer it back as a contriubtion. 😄


fix(question line): add specific text for Novi Sad

:tada: This PR is included in version 1.33.12 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


check_iaas should rise a warning in case of different OS on nodes

Initial set-up:
node1: Ubuntu 22.04
node2: Ubuntu 20.04
node3: Ubuntu 20.04
Run ./kubemarine check_iaas
Check output for this task
2022-11-08 15:07:35,111 k8s.example.com INFO *** TASK system.distributive ***
2022-11-08 15:07:36,378 k8s.example.com DEBUG Host <IP_ADDRESS> running "ubuntu 22.04"
2022-11-08 15:07:36,378 k8s.example.com DEBUG Host <IP_ADDRESS> running "ubuntu 20.04"
2022-11-08 15:07:36,378 k8s.example.com DEBUG Host <IP_ADDRESS> running "ubuntu 20.04"
System      OK     008  Distibutive .......................................... ubuntu 20.04, ubuntu 22.04
Expected result: Output for this check should rise a warning since having different OS on nodes is not quite good idea (but still workable, I guess)
Actual result: Test passed.

Issue reproduced & working on code change


Unable to fetch outgoing transactions

Problem statement
The listtransactions RPC lists outgoing transactions that match the imported descriptors, but the address is that of the destination. As a result, it is impossible to index the transaction by the sender's address, something that the libcore relies on.
Solution
This is solvable by fetching the verbose raw transaction for send entries, and also index the input address.
How to handle multiple inputs
It's okay  to index the transaction by addresses not belonging to the libcore wallet. The non-wallet address will eventually get filtered out, since the libcore always requests transaction IDs for a finite set of addresses (20, in most cases).

Fixed in b1532ab


Multiple Marketplace

In amazon documentation, it's stated that we just have to connect one marketplace of that region and all other marketplaces of that region will be accessible by that, this works for the UK when I connect with the UK I got all EU regions (who have EU in there amazon seller central URL) but when I connect JAPAN for FE it connected only japan in getMarketplaceParticipation API it returns only japan, not Australia, etc.

Hi @iamsr,
could it be possible that the seller account is only registered in Japan and does not participate in the other FE marketplaces?

No, it is registered in another marketplace, I can add a listing in Australia but can't access those listing when I am connected with the JAPAN marketplace. If I connect with Australia then I am able to get Australia listing data in calls but then it's only 1 marketplace at a time not another marketplace of that region for FE.

@iamsr, thats interesting. We have never used the FE endpoint, so I can't really check if that's an account issue or if this happens with all accounts in FE. But that might be a good question to ask the Amazon support.

@iamsr yes, Countries in the Fast East site require separate authorization

Amazon generates separate refresh tokens for Japan, Australia and Singapore (Far East endpoint) and the UAE (Europe endpoint) - see screenshot below.
I can't find much about it in the SPAPI documentation, I found out when opening a new Seller Central account - I would assume it's similar for Egypt, Turkey, India (Europe endpoint) and Brazil (North America endpoint).


/lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.30' not found

./curl-impersonate-ff: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.30' not found (required by ./curl-impersonate-ff) ./curl-impersonate-ff: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.28' not found (required by ./curl-impersonate-ff)
./curl-impersonate-ff: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.25' not found (required by ./curl-impersonate-ff)
curl chrome works !
The Error is with curl ff (Firefox)...

You probably get this error because your local libc version does not match the one with which curl-impersonate was compiled. This can happen if you use an older Linux distribution (the binaries in this repository are compiled on Ubuntu 20.04). What Linux distribution and version are you running on?

Ubuntu Xenial (16.04 LTS)

Your glibc version is too old. I'd recommend to build curl-impersonate from source locally on your machine (or upgrade to a newer Ubuntu of course :) )


Allow 2 users of the same group to share a temp cache.

Right now, the default configuration is that the manifest cache is stored in a user-specific, php-version-specific folder.
It should also be possible to configure this to be a folder specific to a system group, with the files set as group-writeable rather than user-writeable. The system should probably detect whether a given user is a member of this group, and only use this location for such users. Alternatively, the configuration can also specify a whitelist of configuration.
Presumably this will need to be configured in _ss_environment.php since the yaml config relies on the cache being available.

One problem is that in TempPath.php getTempParentFolder() it's using sys_get_tmp_dir() to get the base path to the cache storage location, which differs between CLI and web server. This would need to be rewritten in a way that is the same for both before this change would be useful, otherwise you're still getting two separate cache storage locations.
Environment: OSX with local Apache

Just had a look at my local sandbox and it looks like this is still a problem in SS4.

This is a stale enhancement ticket imo; closing.
It could arguably be used to simply Linux user management (you don't need to run dev/build as www-data) but we've set up the status quo deployment chains for so long now I can't see much value in keeping this open


Demo doesn't work

Hey,
Reproduce

Do steps 1-3 in the demo instructions.
Try to do node tools/analyse/

Expected result
App should run
Actual result
Error: Cannot find module '/Users/User-Name/Documents/dev/modulebox/tools/analyse'
There is no tools folder in the root folder, or any other folder I can see. The files in the folder are:
LICENSE.md	README.md	lib		node_modules	package.json	test

That is very odd. Clearly the tools/analyse folder is there (https://github.com/AndreasMadsen/article/tree/master/tools). Are you sure you are in the correct folder modulebox seams like the wrong name. Also I tried running the demo steps my self, it worked fine.

Hmm, I followed the instructions on the NPM site here: https://www.npmjs.com/package/article

Oh, haha. That documentation is out of date. Thanks for posting the issue, I have updated the documentation on npm now.


Freely Create Attestations (relies on custom trust anchor)

just check if i didn't mess anything up, it's your contribution anyway.

I'm confused, there are 14 commits, but only some minor changes in 4 files ... and the title of this MR doesn't align with the changes.

I'm confused, there are 14 commits, but only some minor changes in 4 files ... and the title of this MR doesn't align with the changes.
Apparently, adverse effects of sleep depravation are  a thing. Sorry!


best way to package a theme?

What is the preferred way to package a theme that includes templates and resources as well as extensions?

Create a zip file that people can download from GitHub and unzip in the root directory of their project?
Create a Python package that has Ivy as a dependency and its own init command to install bundled resources?
Something else?

I'd use something like option 1. A theme is just a directory so you can give it to people as a zip file or as a GitHub repository they can clone.
That's what I've done here: https://github.com/dmulholl/outline

:+1: thanks


Update object.get first parameter

the first parameter of the object.get function updated with value cloud_trail.config
fix proposed in issue: False positive for cloudTrailMultiRegionEnabled (AC_AWS_0448) #1432

Codecov Report

Merging #1433 (e6c1f3c) into master (87cddb0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1433   +/-   ##
=======================================
  Coverage   77.34%   77.34%
=======================================
  Files         279      279
  Lines        7866     7866
=======================================
  Hits         6084     6084
  Misses       1424     1424
  Partials      358      358


stats - missing total volume data

no error in console.
we see volume on the pool level.
it is only missing on the stats summary.

@Velua this might be something with the multicall?

It looks like we are getting faulty data for some pools. One example is ANT/BNT pool.
I have added another check to the calculation to skip the faulty pools. But we need to get to the bottom of this issue of receiving faulty data. It keeps breaking stuff.


feat(paeckchen-core): add logger interface

relates to #88

Coverage remained the same at 100.0% when pulling 99613950d051070048b1b2977830534522cfa8f4 on core/logger into a0c0bea5371bd5903e3683ee92bbaf977777bb76 on master.


Tab character in a value in a JSON file results in a truncated data import

When importing JSON to OpenRefine through 'Create Project', if there is a tab inside a value in the JSON file, the import succeeds but only imports data up to the tab character. e.g. JSON:
{   "rows": [     {       "Content": "A"     },     {       "Content": "	B"     },     {       "Content": "C"     }   ] }
(tab character is before the 'B')
Since tab is an illegal character in this position (should be escaped as \t) it makes sense for the import to fail - but failing silently is a problem.
Not sure what the best solution is - three possibilities:

Import should fail completely with an error
Import should succeed with the data truncated as currently but with feedback to the user that the truncation has occurred and why
Import should succeed with full data, converting illegal character to escaped version

I think 3 would be most appropriate given OpenRefine is a data cleansing tool.  And we should try to cleanse a bit of that.  Folks expect it not to fail for a String value and we can just escape for those cases.

Looks like we might be able to just expand on the CASE logic for VALUE_EMBEDDED_OBJECT
https://github.com/OpenRefine/OpenRefine/blob/master/main/src/com/google/refine/importers/JsonImporter.java#L125
using JsonToken
http://fasterxml.github.io/jackson-core/javadoc/2.2.0/index.html?com/fasterxml/jackson/core/JsonParseException.html
or perhaps using JsonParser.Feature methods might do it
http://fasterxml.github.io/jackson-core/javadoc/2.2.0/index.html?com/fasterxml/jackson/core/JsonParseException.html

Hi, I am now working with this issue and had make some progress.
Just double check if I understand the expected effect correct:  file with { "rows": [ { "Content": "A" }, { "Content": " B" }, { " Content": "C" } ] }  should be able to get loaded and there are two types of Content, one with a tab and one do not. And there is a tab before value B .

I am not sure what you mean by "two types of Content": I would expect A,  B and C to end up in the same column.

I see, so unquoted control chars (like \t, \n) should be ignored during the parsing right?

I would not ignore these unescaped control characters - see the 3 options mentioned by Owen above.

This currently does produce an error:

Illegal unquoted character ((CTRL-CHAR, code 9)): has to be escaped using backslash to be included in string value

although it doesn't happen until you make your record selection (ie not in initial preview).
Is there ever a time that we don't want to be permissive in parsing? ie Do we really need an import option for this? It's an invalid JSON file, but we're a data wrangling tool, not a validation tool.

We clean messy data. We DO NOT clean messy formats? I guess that's what we should adhere to since we are dependent on a large ecosystem of helper libraries for parsing and validating before importing the data.
In general, I take back my comment above that we should offer an option to cleanse unescaped values to clean messy JSON, since I was not aware that this was in fact an invalid or unknown format parsing error for JSON (Thanks to @tfmorris for helping me understand now on this)
Let's close this issue if we all agree and that our existing error message is adequate enough for this.

I think option 3, permissive parsing, always enabled with no option setting is the right answer here.

Import should succeed with full data, converting illegal character to escaped version

More generally, I'd really like to avoid the situation where someone works on a PR that addresses an issue only to have the code review devolved into a labored discussion as to whether the original issue was valid or not. Let's get that resolved up front and not flip flop on our decisions. We have a number of open PRs where this is happening and it's got to be discouraging to new contributors.

@tfmorris Ah, so you agreed with my original feeling, OK.  I thought you were disagreeing.  The flip-flopping happens because I read one thing instead of hearing you voice the comment.  So I think about it more, because I value your opinion @tfmorris and think around it.  No problem.  I'm very used to code review opening up more questions back to the original design issue #.  I think new contributors should be comfortable with some of that; they'll navigate that in the real world soon enough.  But agree that the issues are where we should do the collaborating on the design or on the dev mailing list.

Closed by #2431


fix: default Button to type=button

This avoids accidental form submissions

PR checklist

[x] I have followed the "Committing and publishing" guidelines;
[x] I have added unit tests to cover my changes;
[x] I updated the documentation and examples accordingly;

Changes description
Fixes default button type
Affected packages

@quid/react-core
@quid/react-date-picker (will fix bug with accidental submit on month change)

We got affected by a Yarn bug during the dependencies installation.
The workaround is to limit the network concurrency to 1, this will slow down the build process, but since we cache the packages it shouldn't be much of an issue.
I reported the issue in the related thread:
https://github.com/yarnpkg/yarn/issues/2629


Endless ui refresh

Reported by @abey79
Something is calling request_repaint way too often.
Needs a repro.

Could be https://github.com/emilk/egui/pull/3930
This would help diagnose it: https://github.com/emilk/egui/issues/3931

repro on main:

just py-build
python examples/python/face_tracking/main.py
let the script run for a few seconds then kill it
reset viewport

=> no endless UI refresh
mess just a little bit with the space view:

move the 3D space view tile around (tab / un-tab it)
zoom/scroll  the 3D space view a little bit

After a few of those ops, the infinite UI refresh triggers.

Still using the face tracking example, when setting the blueprint as follows and restarting, I can trigger the infinite refresh by just rotating/zooming the 3d view—no tile dragging needed.

This issue is still affecting head of main.  It doesn't impact all scenes, but for example in arkit_scnes the scene constant repaints.

@jleibs you just need to uncheck this checkbox:

It is off by default, but you ran an earlier version on main where it was on by default, and that is now saved. Or just reset your Rerun.
In other words: this won't be a problem for our users.

nope, wrong again


Tracksuite assumes the Git default branch is 'master'

What happened?
The repository created by tracksuite-init honours the init.defaultbranch option in the Git configuration, meaning that the default branch may have any name depending on configuration (most likely, main). However, tracksuite.GitDeployment.check_sync_local_remote assumes the branch is called master, hence failing because the branch is not found.
What are the steps to reproduce the bug?
git config --global init.defaultbranch main
tracksuite-init -t /path/to/target

Then, try deploying a suite
Version
develop
Platform (OS and architecture)
Linux
Relevant log output
No response
Accompanying data
No response
Organisation
No response

Should be solved in https://github.com/ecmwf/tracksuite/commit/90b738124f2c08e051707477ce58b25dfbc6bf9a
Can you give it another  try?

Yep, that works!


Style scoping bug when moving to v0.11.x

Given two webc components
c-red.webc

  <slot></slot>

c-blue.webc

  <slot></slot>

and their usage:
page.webc
<c-red>Hi I am red</c-red>
<c-blue>Hi I am blue</c-blue>

<c-red>
  I am red.
  <c-blue>Hi I am blue</c-blue>
  still red
</c-red>

<c-blue>
  I am blue.
  <c-red>Hi I am red</c-red>
  still blue
</c-blue>

Prior to v0.11
would result in:

  Hi I am red

  Hi I am blue

  I am red.

Hi I am blue

  still red

  I am blue.

Hi I am red

  still blue

.wxarocnxj{background-color:blue}
.wpqw-v5wk{background-color:red}

but after moving to v0.11
it results in the same html but extra/borked css styles, it seems like the :host selector leaks upwards (if that makes any sense).
.wxarocnxj{background-color:blue}
.wpqw-v5wk{background-color:blue}
.wpqw-v5wk{background-color:red}
.wxarocnxj{background-color:red}

I guess the clearer description is that if you style using :host in a component and you using that component in another (via a slot) then the :host css for that component makes its way into the parent somehow.

I just noticed the same behavior. :host seems to "bubble up" to the highest parent, meaning that things like :host > div behave erratically

apologies, I think this may be the regression introduced in #152 but I will investigate further.

I did confirm that this is fixed by #152 with the test case you supplied, thank you!
Shipping with v0.11.2 today.

Thanks @zachleat! Killing it as always 🚀

Thank you @rijkvanzanten!!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/65093

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74404

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74517

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74862

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74931

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/107080


Adding new game known as DOT and BOXES

This is a very interesting game. I still play it using pen paper.
If you don't know about this game click here.
I would like to work on this game, please assign me this under DCP'21.

@sumitvajarinkar go ahead

@sumitvajarinkar any update?

@sumitvajarinkar any update?

Actually I haven't already prepared game. I'm working on it. I'll push it in upcoming 5 days.


JavaDoc Page

Not sure how useful it'll be, but I'd like to get a JavaDoc webpage up of the library.  I like how ButterKnife implemented it with GitHub Pages.  May use my blog instead.  Anyway this is all new territory for me again.  Will need to experiment, see what's better.  I've used Doxygen before so may use that here?

Looks like Android Studio has a one click export JavaDoc webpage.  Sweet!  Now...where to put it?

Finished.  Added to the new github pages website. Direct link here: http://jaysoyer.github.io/Advanced-Adapters/javadoc/com/sawyer/advadapters/widget/package-summary.html


updated code to handle unfiltered parameters

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Trello Card: https://trello.com/c/bP7hTCJg/343-finder-frontend-does-not-gracefully-handle-unpermitted-parameters
URL with Unfiltered parameters - Before

URL with Unfiltered parameters - After handling the exception


How to use CURLOPT_ERRORBUFFER

Please let me know how to use https://curl.haxx.se/libcurl/c/CURLOPT_ERRORBUFFER.html this via guzzle.

What you're asking for as an internal implementation in php => if you call curl_error(), it works because "under the hood" it was already using CURLOPT_ERRORBUFFER =>https://github.com/php/php-src/blob/f982038fb676089b594b2a94f5c7edb14bf212b5/ext/curl/interface.c#L1825
If you ask about Guzzle and curl_error => https://github.com/guzzle/guzzle/blob/33ddb580d0d88242fa9d597b4715f5f89c5e56a1/src/Handler/CurlFactory.php#L153
I'd like to point out that, without further context, it's hard to give you a better answer.
HTH

Sorry for not elaborating.
I use an extension in my php app or website which uses guzzle. It gets curl error 35. When I checked the website it mentioned in the error it showed to use CURLOPT_ERRORBUFFER to get human readable error.
I'm not very much experienced using guzzle.
So how do I use CURLOPT_ERRORBUFFER only through guzzle. Also is there any documentation for that. As I was not able to find any in guzzle documentation.
Thanks!

Can anyone help me?
My question is "How do I use CURLOPT_ERRORBUFFER?"

You guys don't help!


Update Debug Notes for Dinamica Plus

Supports work on #73

Lint failure is from unrelated files in the rest of the repo.


latest tag on docker runs 0.13.0 .. it should run 0.14.0

just as the title says. when using the tag latest, 0.13.0 gets run; i've verified this by checking my note version from https://bitnodes.21.co/

Thanks. I set the tag correctly but looks like I needed to trigger a rebuild for it to take effect. Should be fixed shortly.

Fixed!


Update MarketStoreProductsPostFunction BusinessHandler to wait 60 seconds

to simulate timeout for the caller of native lambda proxy. This is used to test apiCallTimeout and apiCallAttemptTimeout in the lambda-proxy.yml

https://github.com/networknt/lambda-market/commit/75e5309eada1e6e925fe6791fc5d658c04bbcae2


feat: add support for symfony 7

same as #301 but without dropping support for symfony 4

bors merge

bors merge

bors merge-

bors merge

bors ping

bors cancel

FFS

Thank you @Chris53897


Implementing plugins for better separation

I want to include midi functionality through DryWetMIDI, but that got me thinking: I'm creeping in so many features and dependencies, perhaps some users don't want all that functionality. Or perhaps they want custom features that I'm not adding. I feel plugins would be a good solution to this, besides being fun to implement.
Key2Joy.Plugin.Midi in this branch shows an early example (e.g: Key2Joy.Plugin.Midi /Mapping/Actions/GetHelloWorldAction.cs). I hope to expand it to a plugin that will add Midi Controller trigger and some Midi writing action capabilities.
Plugins would add:

More customization options
Lower barrier to contribute
Possibility to add features without bloating the main project
Possibility to add features without having to wait for me to merge your PR
Ability to license plugin code differently from the main project

Some negatives:

More complexity
More work to maintain
Versioning challenges

PR Checklist

[ ] Mitigate security risks
[ ] Inform users of security risks
[ ] A plugin example (Key2Joy.Plugin.HelloWorld)
[ ] A plugin example that adds Midi Controller trigger and Midi writing action capabilities (Key2Joy.Plugin.Midi)
[ ] Considered how to version plugins and implement versioning
[ ] Moved the FFMPEG dependency to a plugin (Key2Joy.Plugin.Video) while tackling #36
[ ] Written documentation on how to create a plugin
[ ] Designed a way to share and distribute plugins

Along with this plugin system a lot of refactoring is happening. I'll track bug fixes, features and breaking changes in this post.
⚠ Breaking Changes

Old presets will no longer work and will have to be reconfigured. This is due to heavy refactoring and moving from Newtonsoft.Json -> System.Text.Json.
The legacy PressAndRelease has been removed. I doubt anyone used it and I shouldn't be supporting legacy stuff before 1.0.0 launch anyway.

New features

Plugins
In Tools there's a new function to quickly generate the opposite mappings of all mappings selected.

Bugs

Fixed crash caused by mouse button trigger capture (when switching to other trigger and clicking somewhere over the mouse button textbox)
Fixed action group headings appearing twice in the main form


enhancement: Add a config-sample for multiple AWS Organization accounts use case

Closing this issue as proposed pull request #4 was merged.


feat(stages): track hashing stages progress

Codecov Report

Merging #2820 (740d5df) into main (0b81096) will decrease coverage by 25.47%.
The diff coverage is 31.95%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #2820       +/-   ##
===========================================
- Coverage   71.41%   45.95%   -25.47%
===========================================
  Files         505      505
  Lines       66035    66261      +226
===========================================
- Hits        47162    30450    -16712
- Misses      18873    35811    +16938

Flag
Coverage Δ

integration-tests
17.34% <0.00%> (-0.03%)
:arrow_down:

unit-tests
38.34% <31.95%> (-28.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

bin/reth/src/args/stage_args.rs
0.00% <ø> (ø)

bin/reth/src/stage/drop.rs
0.00% <0.00%> (-3.51%)
:arrow_down:

bin/reth/src/stage/run.rs
0.00% <0.00%> (-1.64%)
:arrow_down:

crates/stages/src/stage.rs
88.46% <ø> (-5.77%)
:arrow_down:

crates/stages/src/stages/hashing_storage.rs
0.00% <0.00%> (-95.59%)
:arrow_down:

crates/stages/src/stages/sender_recovery.rs
89.63% <0.00%> (-2.57%)
:arrow_down:

crates/storage/db/src/abstraction/mock.rs
0.00% <0.00%> (ø)

crates/stages/src/pipeline/sync_metrics.rs
45.83% <14.28%> (-15.28%)
:arrow_down:

crates/primitives/src/checkpoints.rs
43.77% <45.00%> (-47.14%)
:arrow_down:

crates/stages/src/stages/hashing_account.rs
70.24% <59.09%> (-26.16%)
:arrow_down:

... and 7 more

... and 217 files with indirect coverage changes


How to get current location and passing these coords into lat,lng

hye i am just trying to get current location and pass these coords into lat, lng
can you please guide me how to do this
many thanks

Here's a service to do that: https://gist.github.com/vunguyenhung/792bc4cddcfc1f33b90e32834b43d4d1
This is not an issue of AGM, please ask this somewhere else. I think Gitter is a better place for the question like this.

Although this is definitelly possible to do by hand, as @vunguyenhung has shown, it might be a nice addition to add this somewhere in the lib, as it's a very common use-case.
@SebastianM What do you think?


How to get "processed" parameter values for in:body

I have a complex swagger schema for a JSON in:body POST request.  I can get the body value like:
var val = op.getParameter('body').getValue(req);
var body = val.value;

where op is the specific api.getOperations() element.  How can I get the processed values of the body though?  For instance I have a body like:
{
  time: '2016-04-01T00:00:00Z'
}

with "time" specified as a type:"string" and format:"date-time" in the api specification.  I see that op.validateRequest recursively validates the document but the .value property does not contain the recursively processed body.
I think it comes down to this line in helper.js (function convertValue):
if (_.isUndefined(type)) {
    return value;
  }

which stops deeper recursion.  Can you give me a pointer on where to start?

What's your schema look like?  If your type for the parameter is not available, as in you've got a schema that does not set the type, you can end up in this situation.  But I can't be sure without seeing your parameter schema.

You are correct, I do not have a "type" listed.  I was more wondering if adding something like:
return _.mapValues(schema.properties, function (property_schema, property) {
      return convertValue(property_schema, null, value[property]);
    });

would correctly process the structure recursively.  Do you forsee side-effects for other types?  Except for the obvious one if value is null (though solved by swapping the order of the _.isUndefiend checks for type and value).

I tried this and it causes all kinds of issues with validation since it uses the processed value and not the raw value.  Though changing to
value = this.rawValue

in parameter-value::valid does appear to solve that.

It should recursively convert values as long as the schema provides the type.  We do not make any assumptions for type when it is missing because JSON Schema does not require it.

noted.  I have type on subproperties and that that is what I want process.  e.g. type:string, format:date-time within the JSON document.
Thank you for the help.

Let me know if I can help further or if you find anything not working like you expected/wanted.


plugins/cmp-tabby: refactor

Do you think this would be more maintainable as its own plugins/by-name module?
It can still be a cmp plugin in the same way that (e.g.) otter does it.

This can be rebased without bf6a68f406577718d31c67103769b7816e46b420, right?

@mergify queue


Module '"remix-routes"' has no exported member '$path'.

I just wanted to try out this package as I'm searching for a way to have type-safety on routes.
Unfortunately, I'm getting a Typescript error when I want to import $path.
Module '"remix-routes"' has no exported member '$path'.

When going to remix-routes/index.d.ts in node-modules I'm getting this error:
Cannot find module '.remix-routes/types' or its corresponding type declarations.ts
Any idea how to fix this? I would really like to test this.
I installed it with yarn and didn't get any errors during installation.

Have you run remix-routes after installation? What does the command output?

It just outputs ✨  Done in 0.XXs.

Does node_modules/.remix-routes exit?

Yes, and it contains a types.d.ts with the correct routes inside.
The index.js on the other hand does not contain any routes.

Just fixed a bug that watch mode does not generate types at first run. Could you upgrade remix-routes to the latest version and try again? If the problem still exits, could you create a reproduction repository?

Just updated and it seems to work now in build and watch mode.
Thanks 👍
By the way, that's a great package. I would have created it on my own if I hadn't found yours.


Unhelpful error when beginning an argument name with a number

I found it difficult to decipher what exactly was going on from this compiler error. It would be more helpful if the compiler could hint that you may be trying to declare/use a variable whose name begins with a number.
Error Message:
-- TYPE MISMATCH ----------------------- ./src/elm/Authenticated/People/View.elm

The right side of (>>) is causing a type mismatch.

180|                                                    .twoFactorAuthEnabled >> Maybe.withDefault False >> (\2fa -> if 2fa then 1 else 0)
                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
(>>) is expecting the right side to be a:

    Bool -> c

But the right side is:

    number -> a -> number1

Hint: With operators like (>>) I always check the left side first. If it seems
fine, I assume it is correct and check the right side. So the problem may be in
how the left and right arguments interact.

-- TYPE MISMATCH ----------------------- ./src/elm/Authenticated/People/View.elm

You are giving an argument to something that is not a function!

180|                                                                                2fa
                                                                                ^^
Maybe you forgot some parentheses? Or a comma?

I reduced this to the following http://sscce.org/
import Html exposing (text)

f =
  (\2fa -> if 2fa then 1 else 0)

main =
  text <| always "hello" f

And based on the parser improvements in the development build, I am getting:
-- PARSE ERROR -------------------------------------------------------- temp.elm

Numbers cannot have letters or underscores in them.

5|   (\2fa -> if 2fa then 1 else 0)
        ^
Maybe a space is missing between a number and a variable?


Use streams instead of strings for response and request data

As pointed out in issue #2 and issue #9 issues, String is not an alternative for the request and response body type
Neither the response or request body are limited to string data. Data also does not need to be sent all at once, but can be sent as a continues stream, which needs to be supported as a lot of web applications takes advantage of this to do realtime communication.

Personally I don't think we should use foundation for this, but I agree with the premise.

Thats fair, And I do agree that using a library in the specification is kind of bad. But I'm having a hard time of being another solution.
A Input/Output stream class/spesification/interface is needed no matter what? right?
So the two solutions are
a) Defining your own
b) Using foundation

I'd I though that maybe, using foundation would be the easier of the two options.
I would at least like two see this problem be tackled as soon as possible, as I would very much like to see a standard like this be written.

Unfortunately depending on Foundation would mean we won't be able to support other platforms such as Linux/Windows when Swift goes open source, which is on track for December.
I propose we have a minimal protocol which just has a single function, which may look something like:
protocol Streamable {
  func readBytes(count: Int) throws -> [Int8]
}

We can have a couple of convenience extensions in Inquiline, along with making String itself conform to Streamable so that people can just return strings.
What do you think?

Very good point, and I also personally think that it's kind of 'ugly' to use Fondation anyway.
Thats a good idea, although we would need both a OutputStreamable and an InputStreamable.
We also need to make sure to specify how and when the thread should be blocked if it can not read the bytes.
I suggest we use the same as most 'standard input' implementation; "If it cannot read, it blocks the current thread and waits until the data requested is available."

I've updated the pull request, and removed the use of foundation.

Thanks @sigurdsvela, would you also be able to add these protocols and updated to Nest.swift. Then the CI will catch these syntax problems.

I'll do that. (Sorry, Java background)

I've updated the pull request again. What I'm undecided about is how this will work in practice.
In the README file there is an example how to write an application, which is written as a function, that returns the response. This will obviously not work with the body being a stream instead of a string.
What I'm thinking is that the response can be a second parameter to the application function. as such.
fun application(request : HttpRequest, response : HttpResponse) {
}

This would require the properties in HttpResponse to be read and write.

Hmm, looking though this. There should be one stream not two. The underlying web server will be calling readBytes on the body you've returned. Where there must be an implementation of readBytes reading the blocks, reading from disk, etc.
I'm just working though building an implementation of this to test it all out.

After thinking about this over the weekend, I think it would make more sense of both of the bodies become a GeneratorType where Element is CChar or Int8. This would fit Swift better.
Something like:
protocol Body : GeneratorType {
  typealias Element = Int8
  mutating func next() -> Self.Element?
}

Or even:
 /// Represents a HTTP Request
-public protocol RequestType {
+public protocol RequestType : GeneratorType {
+  typealias Element = Int8
+
   var method:String { get }
   var path:String { get }
   var headers:[Header] { get }
-  var body:String? { get }
+
+  mutating func next() -> Self.Element?
 }

@sigurdsvela @Danappelxx what's your opinion on this?

I think it's important that the streaming interfaces are also capable of expressing error conditions, such as short reads/writes. An Optional might not be sufficient for this purpose.
Go's Reader and Writer interfaces seem related and come along with some clear documentation.

I think that making the body a generator is a very neat idea, especially with how that makes it to work with strings.
I am not convinced that the request itself needs to be a generator, as that could be misleading (request.next() does not clearly imply that it is getting the next character of the body).
@mickeyreiss, I don't think it's necessary for next() to throw - a quick look at the documentation linked shows me that the only errors for read/write are EOF, which could be simply be expressed as nil (but perhaps I'm missing something, let me know if I am). Furthermore, if the function threw, we would not be able to conform the body to GeneratorType (at least, not without protocol extensions using try? under the hood, which is messy).
Also, @kylef, what's the best way to reach you? There are a couple things I would like to let you know about.

To try move this forward, I've created a NEP 2 to propose streamable bodies and to supersede this pull request.

I'm fairly inexperienced in a lot of this stuff, but is there a reason not to use UInt8 over Int8?

Nest 0.3.0 has solved this problem.


Changes to a header in a .systemLibrary target's sources are not detected

I have a package with a .systemLibrary target, and in Sources/MySystemLibraryTarget there is a MySystemLibrary.h header and a module.modulemap (which references the header via header "MySystemLibrary.h"). If I change MySystemLibrary.h, sourcekit-lsp will not pick up the changes until I both clean the build folder and restart sourcekit-lsp (restarting alone is not enough). An easy way to test this is to add an inline function to the header file, then try to call it from Swift. swift build will succeed without any additional effort (same for building/running the tests via VSCode's UI) , but sourcekit-lsp will report "no such function" until the build folder is cleaned and sourcekit-lsp is restarted.
Expected behavior:
Changes are detected by sourcekit-lsp as they occur.
Notes:
While this may sound like a weird use case, I find it very helpful for making a system library I work on work better with Swift. If I can test some functionality just by changing this header, that is preferable to having to change the system library and re-install. Also, in these situations it is often the case that you need to add #include statements as you build out the Swift wrapper for a system library.

Tracked in Apple’s issue tracker as rdar://118263993

Thanks for filing the issue @GeorgeLyon. However, I can’t reproduce it using the following steps:

echo "void systemTest() {}" > /tmp/systemlib/MyLib.h
Unzip package.zip
Open the package in VS Code and build
Open /tmp/systemlib/MyLib.h and add void test2() {} at the end
Call test2 from main.swift
I don’t see any diagnostics about test2 not existing

Could you check if these steps reproduce the issue for you? If not, could you amend them with the steps that I need to take to reproduce the issue?

Hmm just tried on your small project and it does indeed work, though it still fails on my larger project. My guess is that because I'm importing a significant amount of headers it is just really slow to update the index, making it seem like it never updates and resetting the LSP server makes it update quicker. I'll do some more testing and see if I can find the exact issue, but can close this for now.

If you can reproduce this in a project you can share, I would be very interested to hear about it and figure out what’s going on.


Add link to Cake.Markdownlint addin

Add link to Markdownlint addin for Cake build automation system

Could you please send this PR against “next” branch instead?

@DavidAnson Sure, rebased and changed target of PR.

I've some other markdownlint related repos. If you want I can add them here too (or in markdownlint-cli, whatever you prefer):

Chocolatey package for installing markdownlint-cli
Chocolatey package for installation markdownlint extension for Visual Studio Code
Cake addin for reading and processing issues reported by markdownlint / markdownlint-cli

I’d say:

CLI
Extension
CLI


Radius Graph with Random K-Points Picking

Dear Matthias,
Thanks for reading my post. I have a quick question regarding the implementation of radius_graph(r, k). As I noticed that scipy.spatial.ckdtree is used, does this function would actually return the k-nearest-neighbours whose distance w.r.t "src_point" is less than r?
I am trying to implement a function to random pick k points from all neighbours who lie in the distance r. Does it mean that I cannot use radius_graph() you completed?
Many thanks!

Hi, you can still use the radius_graph method by setting max_num_neighbors to a high value. However, you need to randomly pick k neighbors afterwards on your own. We sadly do not provide a functionality for this.

If you use the radius_graph method on the GPU, it is however sufficient to set max_num_neighbors to k because of non-determinism in the output :)

Thanks for your thunder reply! Just would like to confirm that Is it saying that since the GPU-version of radius_graph() method would not sort the neighbors according to their distances, it is therefore possible for us to use this version to achieve a random picking effects? Thanks!

From: Matthias Fey<EMAIL_ADDRESS>Sent: Thursday, September 26, 2019 10:19:19 PM
To: rusty1s/pytorch_geometric<EMAIL_ADDRESS>Cc: Chaoyi Zhang<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [rusty1s/pytorch_geometric] Radius Graph with Random K-Points Picking (#704)
If you use the radius_graph method on the GPU, it is however sufficient to set max_num_neighbors to k because of non-determinism in the output :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rusty1s/pytorch_geometric/issues/704?email_source=notifications&email_token=AJWFC4D5DRT7UGJECOHEBCLQLSSEPA5CNFSM4I2YPQ5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7VLSHQ#issuecomment-535476510, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJWFC4CHWUN2QR6FRW3FWZLQLSSEPANCNFSM4I2YPQ5A.

Yes :)

Thanks Matthias! Have a good day!

From: Matthias Fey<EMAIL_ADDRESS>Sent: Thursday, September 26, 2019 10:28:33 PM
To: rusty1s/pytorch_geometric<EMAIL_ADDRESS>Cc: Chaoyi Zhang<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [rusty1s/pytorch_geometric] Radius Graph with Random K-Points Picking (#704)
Yes :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rusty1s/pytorch_geometric/issues/704?email_source=notifications&email_token=AJWFC4CFY44VUHG37MGXVEDQLSTHDA5CNFSM4I2YPQ5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7VMIYY#issuecomment-535479395, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJWFC4FFAHAS7SPJWFHE5ADQLSTHDANCNFSM4I2YPQ5A.


MaterialUI tooltip component crashes in context editing

Hi,
inside my project I have a Tooltip component (Tooltip) that wraps an icon and I'm trying to localize it. The simplified structure is as following:
const t = useTranslate()
return (
  <Tooltip title={t("topbar.account")}>
    <IconButton>
      <AccountBox />
    </IconButton>
  </Tooltip>
)

The problem arises at the moment in which I hover the icon, the error returned is:

Cannot read property 'attributes' of null

Under the hood Material translate the tooltip in a div so I can't see why it isn't working properly.
I already tried the passToParent and tagAttributes options.
Have you ever faced something similar? Can I provide you other useful infos?
Dependencies versions:
@tolgee/react: 2.8.0
@tolgee/ui: 2.8.0
react: 17.0.2
material-ui: 4.11.3

Hi!
Thanks for submitting. It's strange since we use exactelly the same stack and tooltips work fine.
Can you please provide a stacktrace of the error? Minimum example project would be also great.

OK, so I reproduced locally on sample project. Going to fix it.

I was here writing the repro, glad to hear you managed to reproduce it! Let me know if you need any help.

It seems like this happens, because Material UI is removing the title attribute from the child while it renders the tooltip. I am looking into it, but if you need quick workaround, you can add aria-label attribute to the child element, so in-context localization stays working and use noWrap in the tooltips title attribute.
import './App.css';
import {Tooltip} from "@material-ui/core";
import {useTranslate} from "@tolgee/react";

function App() {
  const t = useTranslate()

  return (

        <Tooltip title={t({key: 'topbar.account', noWrap: true})}>

Hello

        </Tooltip>

  );
}

export default App;

I didn't notice that Material UI was removing the title attribute, anyway I'm gonna use the workaround for the time being. Thanks!

Do you have an account where I can test? Because I have another issue with the MaterialUI library and for the repro I need to link the api key. Actually I can create an account with a temp email. Is that ok?

Please open new issue for that :) Just add comment, that api key needs to be added. I will use my account to test it.

It's fixed in the PR, but I still recommend to use the workaround. Wit noWrap it will be wrapped, so the position tooltip will be counted with longer text and it's not going to be positioned well.


DRY-up logic for enabling fake S3 functionality

I have a number of motivations for doing this:

There is currently some implicit duplication in logic for this - in config/routes.rb and in S3Storage.build
I want to be able to enable the fake S3 functionality for an integration test and this might help
It addresses the concern I raised in this PR review comment.

I've ended up doing a bit of related tidying up along the way:

Extract S3Configuration class c.f. FakeS3Configuration and add specs for it
Remove S3Storage::Null which is no longer needed

Note that this PR is based on the work in https://github.com/alphagov/asset-manager/pull/344, so it should ideally be rebased against that branch before merging.

Now that #344 has been merged, I've rebased this against master and am force-pushing.

@chrisroos Thanks for reviewing. I'll get it merged.


MINOR: [C++] Ensure public API headers don't include Windows.h

@github-actions crossbow submit -g cpp

+1, will merge if CI is ok.


[AMBARI-23144] Remove obsolete modules from the Ambari source code

What changes were proposed in this pull request?
The following obsolete modules should be removed from Ambari.  Since they are not used or maintained there is no need to keep them around:

Groovy Shell (ambari-shell/ambari-groovy-shell) [last updated 9 months ago*]
Groovy Client (ambari-client/groovy_client) [last updated 2 years ago]
Python Shell (ambari-shell/ambari-python-shell) [last updated 4 years ago*]
Python Client (ambari-client/python_client)  [last updated 2 years ago]

How was this patch tested?
Executed unit tests. ambari-agent module had issues. Re-running.
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/974/
Test PASSed.


tsc problems

noted below, I seemed to have a fix for this, but noting in case it continues to be a problem / for others.
Run npm run doctor or wechaty run doctor(for docker user), paste output here
Wechaty Doctor

Wechaty version: #git[bebce38 fix ci test]
Darwin x64 version 16.1.0 memory 107/8192 MB
Docker: false
Node version: v7.2.1

Expected behavior
tsc command should compile .js files without complaining (too much!)
Actual behavior
tsc generated a lot of errors
Steps to reproduce the behavior (and fixes, if any)
checkout master
git pull
tsc

npm run dist gives the same errors
Paste the full output logs here with WECHATY_LOG=silly set
$ tsc
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/brolog-env.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/config.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/contact.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/friend-request.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/io-client.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/message-media.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/message.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/bridge.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/browser-cookie.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/browser-driver.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/browser.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/event.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/friend-request.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/index.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/puppet-web.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/server.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/watchdog.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/room.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/state-monitor.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/util-lib.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/wechaty.d.ts' because it would overwrite input file.

possible issues. seems I had extra generated .js files as src/*.js
I rm'd these and now tsc seems to run ok.
➜  wechaty git:(master) ✗ gst
On branch master
Your branch is up-to-date with 'origin/master'.
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	src/brolog-env.js
	src/config.js
	src/config.spec.js
	src/contact.js
	src/friend-request.js
	src/io-client.js
	src/io.js
	src/message-media.js
	src/message.js
	src/message.spec.js
	src/puppet-web/bridge.js
	src/puppet-web/browser-cookie.js
	src/puppet-web/browser-driver.js
	src/puppet-web/browser.js
	src/puppet-web/browser.spec.js
	src/puppet-web/event.js
	src/puppet-web/firer.js
	src/puppet-web/firer.spec.js
	src/puppet-web/friend-request.js
	src/puppet-web/friend-request.spec.js
	src/puppet-web/index.js
	src/puppet-web/index.spec.js
	src/puppet-web/puppet-web.js
	src/puppet-web/server.js
	src/puppet-web/ssl-pem.js
	src/puppet-web/watchdog.js
	src/puppet.js
	src/puppet.spec.js
	src/room.js
	src/state-monitor.js
	src/state-monitor.spec.js
	src/util-lib.js
	src/util-lib.spec.js
	src/wechaty.js

nothing added to commit but untracked files present (use "git add" to track)
➜  wechaty git:(master) ✗ rm src/*js
➜  wechaty git:(master) ✗ tsc
➜  wechaty git:(master) ✗ gst
On branch master
Your branch is up-to-date with 'origin/master'.
Untracked files:
  (use "git add <file>..." to include in what will be committed)

	src/puppet-web/bridge.js
	src/puppet-web/browser-cookie.js
	src/puppet-web/browser-driver.js
	src/puppet-web/browser.js
	src/puppet-web/browser.spec.js
	src/puppet-web/event.js
	src/puppet-web/firer.js
	src/puppet-web/firer.spec.js
	src/puppet-web/friend-request.js
	src/puppet-web/friend-request.spec.js
	src/puppet-web/index.js
	src/puppet-web/index.spec.js
	src/puppet-web/puppet-web.js
	src/puppet-web/server.js
	src/puppet-web/ssl-pem.js
	src/puppet-web/watchdog.js

nothing added to commit but untracked files present (use "git add" to track)

also
✗ tsc -v
Version 2.1.4
 npm -v
3.10.10

no npm run dist is still failing
$ npm run dist

><EMAIL_ADDRESS>dist /Users/dc/dev/rikai/ta-bot/wechaty
> tsc && shx cp package.json dist/ && shx cp src/puppet-web/*.js dist/src/puppet-web/

error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/brolog-env.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/config.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/contact.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/friend-request.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/io-client.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/message-media.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/message.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/bridge.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/browser-cookie.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/browser-driver.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/browser.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/event.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/friend-request.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/index.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/puppet-web.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/server.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet-web/watchdog.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/puppet.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/room.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/state-monitor.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/util-lib.d.ts' because it would overwrite input file.
error TS5055: Cannot write file '/Users/dc/dev/rikai/ta-bot/wechaty/dist/src/wechaty.d.ts' because it would overwrite input file.

npm ERR! Darwin 16.1.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "dist"
npm ERR! node v7.2.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dist: `tsc && shx cp package.json dist/ && shx cp src/puppet-web/*.js dist/src/puppet-web/`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dist script 'tsc && shx cp package.json dist/ && shx cp src/puppet-web/*.js dist/src/puppet-web/'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the wechaty package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     tsc && shx cp package.json dist/ && shx cp src/puppet-web/*.js dist/src/puppet-web/
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs wechaty
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls wechaty
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/dc/dev/rikai/ta-bot/wechaty/npm-debug.log

Short answer: run npm run redist instead.
Long answer: tsc will fail if the output will overwrite other files. So we need to delete the dist directory before run tsc again.

ok! that was why it worked once. my mistake. in that case maybe npm run dist should clean always?

@dcsan agree! :)


Params are not passed (hence neither the cmdline args)

Hi Sahloul,
I tried to launch the single_board node with a custom marker size, but I noticed that the param still has the default value specified in the code (0.05).
The whole namespace thing might need a rework.

I just encountered the same problem. To save someone else a few hours, the marker_size type should be double instead of string in the launch file.
<param name="marker_size" type="string" value="$(arg marker_size)"/>
should be
<param name="marker_size" type="double" value="$(arg marker_size)"/>


Add request header to disable the cache in response

As defined in https://github.com/adobe/helix-pages/issues/17, we need at the project level a way to control the cache headers from response of the dispatch method. This would allow a project to define the cache strategy if the default one is not satisfying.
I initially though that adding a request header to define the Cache-Control response would be a good strategy. But I spent half a day testing many combinations of the Cache-Control and Surrogate-Control and it turns out that both are needed and some cases do not work as expected:
Works and tells the browser that he can keep the file for a while:
resp.headers['Cache-Control'] = 'max-age=604800, private';
resp.headers['Surrogate-Control'] = 'max-age=0';

Works but browser will request the files each time:
resp.headers['Cache-Control'] = 'no-cache';
resp.headers['Surrogate-Control'] = 'max-age=0';

Followings do not work, Fastly still caches the response (based on the spec, I would assume this should work):
resp.headers['Cache-Control'] = 'no-store';
resp.headers['Surrogate-Control'] = 'max-age=0';

resp.headers['Cache-Control'] = 'no-cache';

Based on the complexity of this, I suggest to simply implement an on/off flag: if you add the X-Dispatch-NoCache request header (value does not matter), the response will set the appropriate headers so that it does not get cached by Fastly. Gives less control but at least it is easy to enable.

Retuned headers are now:

Cache-Control = max-age=604800, private
Surrogate-Control = max-age=0

if X-Dispatch-NoCache header is part of the request.


Update Envoy SAML imagery & URL

Updating Envoy imagery with SAML2 config and latest Envoy web dashboard UI.

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

@bheddens : Thanks for your contribution! The author, @jeevansd, has been notified to review your proposed change.

@chetansriv Can we please test the app again?

@bheddens As the Identifier and Reply URLs are fixed values for every customer we dont want customers to populate them. WE can autofill these values for the customers at the back but we are not showing them in the UI. The integration will still work without these values.

@jeevansd - That would be preferable, can you handle updating these values for the application itself? Envoy has a number of customers using the SSO app and getting these values updated to our newer endpoints would be ideal. I will update my screenshot to note include the advanced URLs.

@bheddens : Thanks for your contribution! The author, @jeevansd, has been notified to review your proposed change.


Build on msys2 gives the following "error: static assertion failed: "Structure must be same size""

In a fresh msys2 installation, after issuing the commands below:  (its is required to close the msys2 windows after each of the following two commands)

pacman -Syu
pacman -S git cmake make gcc dtc diffutils ncurses-devel python3 gperf tar isl

I did run the following commands/script:

export BASE_DIR=`pwd`
export RISCV_PREFIX=riscv-none-embed-
export RISCV_ISA=riscv32im
export XLEN=32
export PATH=$PATH:/apps/compilers/riscv-none-embed/bin
export RISCV=/apps/compilers/riscv-none-embed

git clone git://github.com/ninja-build/ninja.git && cd ninja
git checkout release
./configure.py --bootstrap
cp ninja.exe /usr/bin/
cd ..

curl -O 'https://bootstrap.pypa.io/get-pip.py'
./get-pip.py
rm get-pip.py

export ZEPHYR_TOOLCHAIN_VARIANT=cross-compile
export CROSS_COMPILE=$RISCV/bin/$RISCV_PREFIX
export ARCH=riscv32
git clone --config core.autocrlf=false https://github.com/zephyrproject-rtos/zephyr
pip3 install -r zephyr/scripts/requirements.txt
cd zephyr
source zephyr-env.sh
cd samples/hello_world
mkdir build && cd build
cmake -GNinja -DBOARD=qemu_riscv32 ..
ninja

I found the following error:

In file included from ../../../include/toolchain.h:21,
from ../../../include/kernel_includes.h:19,
from ../../../include/kernel.h:17,
from C:/msys2/home/myuser/test001/zephyr/arch/riscv32/core/fatal.c:7:
../../../include/logging/log_msg.h: At top level:
../../../include/toolchain/gcc.h:28:37: error: static assertion failed: "Structure must be same size"
#define BUILD_ASSERT_MSG(EXPR, MSG) _Static_assert(EXPR, MSG)
^~~~~~~~~~~~~~
../../../include/logging/log_msg.h:151:1: note: in expansion of macro 'BUILD_ASSERT_MSG'
BUILD_ASSERT_MSG((sizeof(union log_msg_head_data) ==
^~~~~~~~~~~~~~~~

How do I fix this compilation error without hacking into ../../../include/toolchain/gcc.h file?
Nelson

Hi.
I may have used a cross-compiler which is not supported by Zephyr.
Tried others 2 different compilers for  riscv32 arch and now it works without modifications.
Closing.
Nelson


agent: Support fetching LFC metrics (but don't use them yet)

Probably best to review each commit separately.
Most of the commits here are just repositioning to make the final commit, which adds LFC metrics collection, as simple as possible.
Builds on #892, must not be merged before then.

Discussion from the 1-1 meeting: scraping sql_exporter from the autoscaling-agent is the right way to go. I'll add a separate sql_exporter solely for autoscaling metrics. For now, we can just scrape the current one with a reasonable interval.

Discussion with @Omrigan : It's fine to just hard-code the metric names, and that makes it a lot simpler. Some discussion about whether to encapsulate the callbacks for metrics collection into a single struct with func(...) fields or an interface.

New changes:

Move metric names into the code (rather than config) and remove the generics it required. Diff here: 4b06d09..0bb2b59.
Fixed a bug where we'd always use the "system" metrics port, even for LFC metrics: da48a4cd3f89ef8eaceaf620e7e6faff781aef77 (NOTE: this was from reading the code; I haven't yet tested it)
Switch from unnamed callbacks to a new metricsMgr[M] struct to contain them and the kind string: 5dc98a95675fcb6573278834a688b123eda89149
Switch from the emptyMetrics() constructor callback to a generics-based approach: 031cc2d13a74e4c232122053404b6119cae04f47

cc @Omrigan particularly want your feedback on this one. I could go either way on this -- left it as a separate commit so it's easy to revert.

Deduplicate metrics config parsing: 86cb9f4a277a5357f95920a82d25b074eef3e60a + c1ae8d3b1ed5c2badcf0d3873b939b6be70137a9

Continuing with the changelog approach, if only because it helps me manage the rebasing:

Reverted 031cc2d13a74e4c232122053404b6119cae04f47 ("tmp: Switch from emptyMetrics() callback to generics"): 60f835b5fc21bc867d8b3a466f1896a22d44f6dc
Squashed everything back down: 60f835b..666e11f
Rebased onto main: 666e11f02bae311c0ac57dbe129e42f1487262db -> 34afd60dc06f5db3551ae1893054716a65cf5b2e
Renamed newMetrics -> updateMetrics: cb14dafe0dcefdb61bec02e365c9626af81b2cfa+b9e3ebd1ea51a7d3730b965d3ecd77c0318776f4
Fix outdated comment on FromPrometheus: 662613d170363fa9cf240e5a0225e0c2f1c09674
Renamed the c/l context/logger variables to ctx2/logger2: efdda7c819bbfe1a5178b884d9803438b5cebba1

Plus a couple more changes to #948 based on review here.

Final changes:

Squash previous commits. Diff here (empty): 66b9289..20bd2ba
Rebase onto main (dropping #948 from this branch). Diff here: 20bd2ba..fb2f910
Fix compile error from rebase: 689e18d0d66b935f9c0ffba6cae5df2e4f8f8306
Change metricsMgr[M any] -> metricsMgr[M core.FromPrometheus]: d0991a9d2447ff1c2a37e0cf2241422df51dea52

Waiting on some issues with latest release. Will hold off on merging for now to avoid accidentally adding much larger changes alongside a small fix.


Updated Highcharts from 6.0.4 to 7.0.2

Highcharts version updated from 6.0.4 to 7.0.2
And also updated package name to avoid name conflict on npm.
Suggestion : Please move highcharts from dependencies to peer dependencies to minimize scenarios like this.

I would also be interested in this! If you need any help or support, just let me know! 👍

I just had a closer look at this PR again and it looks to be in a very early stage, for example some files are added that should not be added. @noumanaltaf are you still working on this?

how do we update this? trying to update to version 8

this isn't a clean upgrade path. This pull request should be closed


Test/add controller key stroke tests

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 96.62921% with 3 lines in your changes missing coverage. Please review.

Project coverage is 73.61%. Comparing base (7a212ba) to head (c3f577b).
Report is 2 commits behind head on master.

Files
Patch %
Lines

src/controller.rs
96.59%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   68.84%   73.61%   +4.76%
==========================================
  Files           6        6
  Lines         735      792      +57
==========================================
+ Hits          506      583      +77
+ Misses        229      209      -20

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


x/tools/playground: program does not stop

What version of Go are you using (go version)?

The playground uses the latest stable release of Go.
The current version is go1.14.7.

Does this issue reproduce with the latest release?
There is as yet insufficient data for a meaningful answer.
What operating system and processor architecture are you using (go env)?
That running on https://play.golang.org/.
What did you do?
https://play.golang.org/p/kQraatQeinz
package main

import (
	"fmt"
	"time"
)

func main() {
	ticker := time.NewTicker(time.Second)
	for {
		select {
		case <-ticker.C:
			fmt.Print(".")
			break
		}
	}
	fmt.Println("Hello, playground")
}

What did you expect to see?
Program not starting or program running into timeout.
What did you see instead?
Playground reports a timeout and then does not stop generating output:
./prog.go:17:2: unreachable code
Go vet exited.

timeout running program
......................................................[...]

Program is still running as of finishing writing this issue. (More than 15 minutes.)

cc @toothrot

Hey @Teelevision, this is actually part of our faketime simulation of how the program output would look had it been running. As the clock and time advancement has been faked, we simulate delays in our output so it tries to look like what your program would have done.
If you change your code slightly to put the time output by ticker.C, the final message from your program before it was terminated would look similar to this:
# our API response:
{"Message":"2009-11-11 02:42:25 +0000 UTC m=+13345.000000001\n","Kind":"stdout","Delay":1000000000}

In this case, you can see that we weren't actually sleeping between loop iterations, and the clock has moved forward quite a lot further than our 5 second timeout! Our javascript UI simulates this in your browser, and calculates how long to sleep between each message based on the Delay field.
In any case, this is working as intended, so I will close this issue. As the output states, there was a timeout running your program, it was really killed (I verified locally and in production to be extra sure), but we fake your output so it looks real.
I hope that helps! Please comment if something isn't clear.


global: bump dependencies and add config [more]

Add SERVER_HOSTNAME to invenio.cfg in order to pass
same origin policy

This is part of making a working August release and will be merged and deployed when it passes.

Coverage remained the same at 0.0% when pulling 0376b3b71f08b8b0fa9591839ccdafc7cdb91c66 on fenekku:setup_august_release into f0f2b5fe2bbff4594d874bb1420e5f9525cf4e57 on inveniosoftware:master.

Coverage remained the same at 0.0% when pulling 0376b3b71f08b8b0fa9591839ccdafc7cdb91c66 on fenekku:setup_august_release into f0f2b5fe2bbff4594d874bb1420e5f9525cf4e57 on inveniosoftware:master.


Allow selection of specific BMH per compute node in OSBMS

Adds an optional bmhLabelSelector field in the instanceSpec OpenStackBaremetalSet so that a specific BMH can be chosen per instance if desired.
jira: OSPRH-10282
Co-Authored-By: @rabi<EMAIL_ADDRESS>

recheck

/test all

@rabi I tested your changes in my local environment in a few different ways.  Everything seemed to work properly, so I am good with this.

Merging this as it has been tested by both Andrew and me and +2 from Andrew.