Implement password hashing and allow more variables to be set via environment variables

好耶ヽ(✿ﾟ▽ﾟ)ノ

@paz appreciate the effort for implementing hash to protected routes, but why are we adding these extra non-sensitive configs to environment variables?

vercel have a limited environment variable storage (#19) and it would be a waste to store these extra data
additionally, if users were to fork this repo and deploy it on their own, they can't avoid changing file contents and making customisations

my main point is that extracting these configs to environment variables does not help them avoid potential upcoming conflicts when they update their forks. hence i would prefer not making these extra changes to the way configurations are handled. thx

Ah i see, I did not know it is limited, well the environment variables are OPTIONAL, leaving them blank means it just uses the config file instead. I prefer to use environment variables, but that doesnt mean its forcing others to do the same.

我也这样觉得，环境变量空间有限，而且修改config也挺便捷。另外哈希密码挺不错的😋🌼

I will add password hashing option to config aswell

Core logics implemented in #83, closing this for now.


Next

Closes @wfp/ui#
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

@cassandrakesewa We may have to send a note to the developers that as a breaking changes they should use dart-sass instead of node-sass now. :)

@laurasilvani They need to change one dependency. Good idea to announce it on Thursday!


copy table example

It would be really cool if we could copy all the entities in one storage table to another. I'm guessing, it would need a couple of API additions that pass the unserialized json too. Here is a draft outline of what I was thinking.

Merged into #146.


[HOW TO] Delete a node from a tree

I want to delete a node from a tree. Given nodeToDelete, this method doesn't work
The splice method doesn't work, as soon as you call update() the item well be back
const itemIndex = nodeToDelete.index;
nodeToDelete.parent.children.splice(itemIndex, 1); // item disappear
this.tree.treeModel.update(); // item comes back again

If I use the array that I am binding to the tree then I can delete elements from the root as such
const itemIndex = nodeToDelete.index;
this.nodes.splice(itemIndex, 1);
this.tree.treeModel.update();

What if the node I want to delete is not in the root of the nodes array i.e. it is a child of another node?
The first option clearly much better but it doesn't work, the delete is not persisted in the tree.

@sulhome
You can try as below
deleteNode(node, tree){
  let parentNode = node.realParent ? node.realParent : node.treeModel.virtualRoot;
        _.remove(parentNode.data.children, function (child) {
            return child === node.data;
        });
        tree.treeModel.update();
        if (node.parent.data.children.length === 0) {
            node.parent.data.hasChildren = false;
        }
  }

and call deleteNode function from your HTML on node.
<tree-root   #tree
      [nodes]="nodes"
      [options]="customTemplateStringOptions">
      <ng-template #treeNodeTemplate let-node>
        {{ node.data.name }}
        {{ childrenCount(node) }}
        <button (click)="deleteNode(node, tree)">-</button>
      </ng-template>
      <ng-template #loadingTemplate>Loading, please hold....</ng-template>
 </tree-root>

Hope this will help
Pleas let me know output.
Thanks
Abhi.

@AbhiThakare what is _.remove. I get exception in

Cannot find name '_'.

Thanks

@AbhiThakare
'_' refers to a library called lodash
you need to install it in your angular project then import it like so
import * as _ from 'lodash'

note that the variable name _ is a convention and you can name it whatever you want

@sulhome , Thanks
I got it. I used import * as _ from 'lodash-es/debounce'; initially, It didn't work.
As you mentioned this worked
import * as _ from 'lodash';
Now is this lodash compatible for AOT ?

@AbhiThakare thanks for your input. I am a complete newbie to this library and would like to understand if there are any documentation that helps me to understand which object has what methods.
As an example, in your code you have fluently written node.realParent, I am curious to understand what is realParent ? I am finding it very difficult to understand the features of this library.
Also why are we doing this?
if (node.parent.data.children.length === 0) {
            node.parent.data.hasChildren = false;
        }

I am trying to write addNode method and I am clueless as to what it should look like. Here is my bad attempt in doing it
addNode(node, tree) {
  let parentNode = node.realParent ? node.realParent : node.treeModel.virtualRoot;
  parentNode.children.push({
    name: 'new child'
  });
  tree.treeModel.update();
}

@rakesh1988
Thanks!
You can find methods / properties here - https://angular2-tree.readme.io/docs/api
not all is mentioned here, sometime you need to dig into lib code as well.
if (node.parent.data.children.length === 0) {
    node.parent.data.hasChildren = false;
}

This condition is added because, If you delete last child from any parent node then collapse / expand icons  should get removed from parent node, in my case it was not working hence added this condition.
As you mention you try to add node, you no need to check for realParent
you can try like below
JS
addNode(node, tree) {
  tree.treeModel.setFocus(true);
  if (node.data.children === undefined) {
	node.data.children = new Array();
  }
  node.data.children.push({
		name: 'a new child',
		hasChildren: false
	});
  tree.treeModel.update();
}

HTML
<tree-root #tree
      [nodes]="nodes"
      [options]="customTemplateStringOptions">
      <ng-template #treeNodeTemplate let-node>
        {{ node.data.name }}
        {{ childrenCount(node) }}
        <button (click)="addNode(node, tree)">+</button>
      </ng-template>
      <ng-template #loadingTemplate>Loading, please hold....</ng-template>
 </tree-root>

Hope this will help
let me know output.
Thanks
Abhi.

Hi,
you should update the original tree and not the wrapped TreeNodes - they get rebuilt from the original nodes on each update

@adamkleingit are you suggesting that the suggested answer is not the right way to do things? can you please give an example about what you are talking about if this is the case

i search online this project has used add,edit,delete. works good
https://stackblitz.com/edit/github-33gw8i

i search online this project has used add,edit,delete. works good
https://stackblitz.com/edit/github-33gw8i

Hi @vijayuxd Please check back the URL and update it with working on. Thanks :smile:


Updated Content API to use members plans to determine permission

no-issue

As discussed on slack, this PR now doesn't error at all, just removes the content if it's not able to be seen.
@kirrg001 I've updated the isContentApi function - could you just double check that it seems good to you?

Yeah I think that looks good 👍DUH ✌🏻


Container nesting visual bug

Godot version
3.5.1.stable
System information
Gentoo Linux x86_64, AMD RX580
Issue description
Placing a container inside of another container makes it's placement not visualized.
Steps to reproduce
Add a child Container node to the main Interface node,
Add a child Container node to the said container.
Minimal reproduction project
container_bug.zip

Can you add a screenshot or a clip of what you describe?


Add snyk

https://snyk.io/
stacks-wallet uses this, I think we should add it here, too.

Is this a quick library integration or more intensive change?

Has already been set up


docs: configuration manual

Feature Request
Add a configuration manual to the documentation so people (especially node operators) have a reference manual to turn to when setting up their nodes.
Summary
A reference manual for the different configuration entries helps create an optimal configuration for node operators. It also allows developers to add more in-depth knowledge about the different entries than what the config.toml comment section allows.

Problem Definition
The config.toml comment section is not enough space for describing the configuration entries.

Proposal
Write a reference manual and add to it as we add new features.

If find this effort really important.
But I wonder whether you should split the work in several PRs. This is on main, we can have stale versions that are not in the documentation website.

Stretch goal:

[ ] Ensure the updated information in config.toml.md is added to the toml.go template (decide if separate issue should be opened for this)


Assignment 4

Please look at my Assignment 4 - part 1 new commit for the result of part 1 assignment and Assignment 4 for the part 2 assignment, thanks
The best ways to address under-fitting and over-fitting are to use
regularization. Regularization controls the penalty for complexity,
which (when successful) will prevent under- and over-fitting.
Bonus question: For the case of over-fitting only, obtaining more
training data will also help. By splitting more proportion of data to
training dataset before the training, we could solve over-fitting and
make training & tests datasets results more similar.

Lab 4 accepted.
Nice work!


Uses player name on join, uses player nickname on exit

Hi there! This plugin works great, exactly what I needed! However, on my server we use Essentials to nickname players. When joining the server it says their actual player name, but on exit it uses their nickname.
I was wondering if a change in the config would work. It currently uses %s but I couldn't find a list of character codes for this.

Sorry for the delay, for some reason I didn't get an email about this issue.
Thanks for the feedback!  It's really exciting to know that someone's using the plugin!
Can you give me some more information about Essentials? Maybe a link to the specific plugin and/or documentation for how it stores nicknames, as I'm not personally familiar with it.
Since I don't want to take a dependency / couple this plugin to any other plugin, what do you think about this "Notification nickname" feature that's separate from Essentials?
Commands that can be run by all players:
/setnotifynickname "Zach" # sets the current user's notify nickname to "Zach"
/deletenotifynickname # removes the current user's notify nickname
/deletenotifynickname <player name> # removes the targeted user's notify nickname. only executable by ops. necessary allow a way for bad actors / "nickname squatters" to be removed

I was wondering if a change in the config would work. It currently uses %s but I couldn't find a list of character codes for this.

To answer this specific question, on both player join and player leave, this plugin uses the Spigot "DisplayName", so I'm unsure exactly why it uses the playername on enter, and nickname on leave.
https://github.com/zdwolfe/minecraft-player-notification-plugin/blob/master/src/main/java/io/github/zdwolfe/mc/playernotification/event/PlayerLeaveListener.java#L21
https://github.com/zdwolfe/minecraft-player-notification-plugin/blob/master/src/main/java/io/github/zdwolfe/mc/playernotification/event/PlayerJoinListener.java#L21


Broken links

[Links for template are broken, getting: "Hmmm… can't reach this page"]

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

ID: 7b777f61-d4cf-bc8e-64b5-43c2e695051b
Version Independent ID: 20c2b71c-b805-b5e1-4de3-f5bc51de91e5
Content: Template apps samples - Power BI
Content Source: powerbi-docs/connect-data/service-template-apps-samples.md
Service: powerbi
Sub-service: powerbi-template-apps
GitHub Login: @paulinbar
Microsoft Alias: painbar

Hi @marianabarone, The samples article is indeed out of date and I just removed it. Those samples are no longer available, and in a number of cases the template apps don't exist anymore either.
You can find template apps on appsource, or by going into the apps page in Power BI and choosing Get apps.
Thanks for bringing this obsolete article to my attention.


Pandora Boxchain Meetup || Blockchain Week Berlin 2018

title: Pandora Boxchain Meetup || Blockchain Week Berlin 2018
Description: Pandora Boxchain welcomes all blockchain and AI enthusiasts to its second meetup in Berlin!
Join us for pizza & beer right after the Zero Knowledge Summit at Mindspace.
We will kick off with the presentation by Andrey Sobol on the topic ‘Randomness in POS blockchains’.
Afterwards, Sergey Korostelyov will shed some light on how blockchain can help build decentralized AI systems.
name: Pandora Boxchain Meetup || Blockchain Week Berlin 2018
synopsis:
Pandora Boxchain welcomes all blockchain and AI enthusiasts to its second meetup in Berlin!
  Join us for pizza & beer right after the Zero Knowledge Summit at Mindspace.

We will kick off with the presentation by Andrey Sobol on the topic
‘Randomness in POS blockchains’.

Afterwards, Sergey Korostelyov will shed some light on how blockchain can help build decentralized AI systems.
date: 2018-09-05
endDate: 2018-09-05
time: '19:00'
endTime: '21:00'
https://www.meetup.com/pandoraboxchain-berlin-meetup/events/254212923/
price:
venue: Mindspace - Coworking Space in Berlin
address: Krausenstraße 9-10, 10117 Berlin, Germany
category: event
speakers:
name: Andrey Sobol

title: Co-founder

company: Pandora Boxchain

companyURL: https://pandoraboxchain.ai/

twitter: andrENsobol

website:

bio: Andrey Sobol is a co-founder of Pandora Boxchain, CTO of Satoshi Fund,
  consensus protocol researcher and author of Hyperion consensus protocol.
  Andrey is an experienced software developer, focused on distributed technologies,
  artificial intelligence and cryptography. He also holds expertise in Game Theory,
  Austrian Theory on Business Cycle, and Decentralization Law.

name: Sergey Korostelyov
title: Researcher and developer
company: Pandora Boxchain
companyURL: https://pandoraboxchain.ai/
twitter:
website:
bio: Sergey Korostelyov is a researcher and developer at Pandora Boxchain,
co-author of pynode computing kernels on Ethereum based PoCW consensus,
blockchain and machine learning expert.
Sergey has more than 10 years of experience in software architecture design and
development focused on building complex enterprise systems

@cesalazar Carlosб please update the event.
I've edited info in the 1st comment.

Please don't use closed issues to request changes. Open a new one instead, using the current content of the page. Here's the current content: https://raw.githubusercontent.com/cesalazar/berlinblockchainweek/master/events/add-pandora-boxchain-meetup/README.md


Add revisionSHA when creating repositories

Fixes: https://github.com/kairos-io/kairos/issues/2393

nothing that can be tested in PRs..


Rely on "Auto-cancel redundant pipelines"

To simplify the code and reduce API calls, I propose relying on the behavior of the Auto-cancel redundant pipelines CI/CD setting instead of using cancel_prev_pipelines().

I like this. I vaguely remember GitLab doesn't cancel pipelines in all cases. Let's rely on the GItLab feature and if it turns out to be an issue, we can add cancel_prev_pipelines() back and add an option in the .gitreview to optionally run it.

I like this. I vaguely remember GitLab doesn't cancel pipelines in some cases.

I know that one requirement is that jobs must be marked [interruptible](https://docs.gitlab.com/ee/ci/yaml/#interruptible).

I see. Looks like the GitLab repos I contribute to are not doing that. We may want to use a config flag for running cancel_prev_pipelines(), which by default is disabled.

I'm retracting this proposal.  The existing approach is more likely to be useful to users.


Limelight renamed to moonlight

This isn't a real issue but recently Limelight got renamed to Moonlight. Maybe to prevent confusion you should update the naming here too?

Thanks for notifying, at least I updated the instructions at http://embedded-software-architecture.com/?page_id=140 . When I find some time I should update the github content too, you're right :)


testing vulnerability

dexter can you generate code to fix some of these changes

devape can you review the diff and provide feedback?

devape, can you review the diff for this pr?

devape, yes, can you do that?


Search keyword related issues

When I search for problems, such as C + +, the problem is that the keyword submitted to the search engine is C, which can't recognize symbols like "+".

Thank you for the bug report, I'll make sure to get this fixed!

Fixed in version 3.5.0, update coming soon.


TiledMapTileLayer tile width & height data types inconsistent #3821

libgdx#3821

Should change the method call to float; decreases the amount of changes we need to make in the code.
Still trying to find where this section of the code is called so I can test it.
Impact Analysis: Code calling the TiledMapTiledLayer() class may be affected. Why does tileHeight/tileWidth need to be called in int instead of float? More research required.
Also I'm slightly saltly (like a saltine cracker...!) on finding where this code is implemented.

Fix committed and posted in the code.


fix: generate all required templates

Checklist

[x] run make test
[x] PR title must follow conventional-commit-standard
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[x] commit message and code follows the Code of conduct

Updating examples and docs in a different pr. So that we can release the packages first.

Great! Docs will be updated in another PR?

Yep, so that we can merge as soon we release?


Post process errors out before file is completely converted

I am using sickrage to trigger the post process and on some files it errors out before completely converting the file.
2015-10-19 12:46:54 INFO     POSTPROCESSER :: Processing succeeded for /media/tv_complete/Horizon in the Middle of Nowhere - 1x01 - Those Lined Up Before the Horizon.mkv
AA
AAconverter.ffmpeg.FFMpegConvertError:
AA    total_output, pid=p.pid)
AA  File "/sickbeard_mp4_automator/converter/ffmpeg.py", line 549, in convert
AA    timeout=timeout, preopts=preopts, postopts=postopts):
AA  File "/sickbeard_mp4_automator/converter/init.py", line 243, in convert
AA    for timecode in conv:
AA  File "/sickbeard_mp4_automator/mkvtomp4.py", line 566, in convert
AA    outputfile, inputfile = self.convert(inputfile, options, reportProgress)
AA  File "/sickbeard_mp4_automator/mkvtomp4.py", line 150, in process
AA    output = converter.process(inputfile, original=original)
AA  File "/sickbeard_mp4_automator/postConversion.py", line 39, in
AATraceback (most recent call last):
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Starting conversion.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Creating subtitle stream 1 from source stream 4.

AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Creating subtitle stream 0 from source stream 3.

AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Reading subtitle streams.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Creating audio stream 1 from source stream 2.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Audio detected for stream #2: ac3 [jpn].
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Creating audio stream 0 from source stream 1.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Audio detected for stream #1: ac3 [eng].
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Reading audio streams.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Pix Fmt: yuv420p.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Video codec detected: h264.
AA2015-10-19 12:46:22 - mkvtomp4 - INFO - Reading video stream.
AA2015-10-19 12:46:22 - SickbeardPostConversion - INFO - Processing /media/anime/Horizon in the Middle of Nowhere/Horizon in the Middle of Nowhere - 1x01 - Those Lined Up Before the Horizon.mkv.
AA2015-10-19 12:46:22 - readSettings - WARNING - Audio bitrate was invalid, defaulting to 256 per channel.
2015-10-19 12:46:22 INFO     POSTPROCESSER :: Executing command [u'/usr/bin/python', '/sickbeard_mp4_automator/postConversion.py', u'/media/anime/Horizon in the Middle of Nowhere/Horizon in the Middle of Nowhere - 1x01 - Those Lined Up Before the Horizon.mkv', u'/media/tv_complete/Horizon in the Middle of Nowhere - 1x01 - Those Lined Up Before the Horizon.mkv', '252345', '1', '1', '2011-10-01']

I am able to manually convert with ffmpeg without any errors

Does it do this with every file? Or just certain files? and does it have this problem every time with those same files or is it intermittent?

Certain files and every time with those files.

What version of FFMPEG are you running?

And can you email me a link to one of the files that's giving you trouble?

ffmpeg version N-76045-g97be5d4
from ffmpeg -version
XUbuntu 14.04
if that helps.
As for the file in question it was from the following torrent (S01E01 is one of the files)
https://kickass.unblocked.la/horizon-in-the-middle-of-nowhere-1080p-5-1-aac-dual-audio-eng-jap-t8092576.html

I'm running N-71028-gaa65ff2 and I'm able to process the file you sent (S01E01) without any problems. I would just try a different build of FFMPEG and see if your problem resolves. Try grabbing some precompiled binaries and using those (especially if you built from source)

interesting. I did not build from source just updated from the available repo
I will try again and see what happens.

I just tried again the fiel again and I received the same error and it stopped converting a 977meg file at 174megs into the conversion. successfully processed 2x01
I am doing this via sickrage post processing.

Try one of these binaries:
http://johnvansickle.com/ffmpeg/releases/ffmpeg-release-32bit-static.tar.xz
http://johnvansickle.com/ffmpeg/releases/ffmpeg-release-64bit-static.tar.xz

Any updates?


Incompability issues with Node v.4.x.x

Currently there are certain compatibility issues with Node v.4.x.x and node-sass. This means that the NPM install will not work as expected, for those running the latest version of Node. To check which version of the node running, type node -v in your terminal.
There is however a fix for this, if you want to roll back to Node v.0.12.5:
1. Install Node version manager
curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.26.1/install.sh | bash
2. Activate nvm
. ~/.nvm/nvm.sh
3. Install Node v.0.12.5
nvm install 0.12.5
4. Use Node v.0.12.5
nvm use 0.12.5
5. Verify that you're using the correct version
node -v
6. Clone the FoundationPress repository
git clone git@github.com:olefredrik/FoundationPress.git
and enter the folder for your latest clone cd FoundationPress
7.  And hopefully it should work for you too. I'll keep an eye out and update dependencies as things become more compatible with the latest version of the node. Sorry for the inconvenience.

Using the latest version of node-sass seems to be a solution. Update package.json DevDependencies to:
    "node-sass": "3.3.3",

and npm install works on my machine™

I can verify that after following this and updating node-sass, I was able to to get npm install to work (though I still had to run bower install && grunt build separately--they did not install while running npm install).
Yosemite 10.10.4 mac mini

Fixed by updating dev dependencies in package.json https://github.com/olefredrik/FoundationPress/pull/478

I installed bower install globally (bower install -g) but running bower install in my project folder gives bower: command not found. @roborracle can you help?


0-Value Greeks For Contracts Expiring Today

Expected Behavior

Option contracts that expire on the current day in the backtest should have non-zero greek values when properly warmed up.
Actual Behavior

Option contracts that expire on the current day have 0-value greeks.
Potential Solution

Consider Ernest's recommendations on this discussion thread.
Reproducing the Problem

View the logs of this backtest.
System Information

Run in the browser QC terminal. Windows 10.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

Hi, looks like the non-0DTE options also have incorrect Greeks. For example their ATM options have Deltas very far from .50.
see backrest here: https://www.quantconnect.com/forum/discussion/10359/inaccurate-equity-option-expiry-time-and-incorrect-option-greeks

Hi, looks like the non-0DTE options also have incorrect Greeks. For example their ATM options have Deltas very far from .50.
see backrest here: https://www.quantconnect.com/forum/discussion/10359/inaccurate-equity-option-expiry-time-and-incorrect-option-greeks


Refactor liquidity params script.

Now get json file and supports several options.

done SPyrus requested one letter change,


URL CORRECT BUT RETURNING NOT VALID JSON

I added the .env details for the Url and \graphql as well however i get errors on the build and retrieval of postsm pages, authors and categories.
✓ Compiled /posts in 165ms
⨯ SyntaxError: Unexpected token '<', ")
at async getAllPosts (lib/wordpress.ts:35:24)
at async Page (app/posts/page.tsx:34:16)
33 |   const url = getUrl("/wp-json/wp/v2/posts", { author: filterParams?.author, tags: filterParams?.tag, categories: filterParams?.category });
34 |   const response = await fetch(url);

35 |   const posts: Post[] = await response.json();
|                        ^
36 |   return posts;
37 | }
38 | {
digest: '3000570245'
}

This usually occurs when the RestAPI is not enabled on your WP. Can you try this and see if it works?
🔗 https://www.perplexity.ai/search/how-to-enable-the-wordpress-re-olXm9RZlQtymdv12amrW0A
Check and see if plugins are disabling the API


Open > GitHub cursor wrong

I click open and see,
GitHub
▼積丹尼 Dan Jacobson
▶circe
▼mymaps
▼master
LICENSE
N.geojson.gz
O.geojson
README.md
map1.geojson
▶spectre-meltdown-checker
▶wayback

When the pointer is placed on any of them, the pointer symbol is all wrong. You are using the text entry symbol "I" instead of a correct 'arrow when over something clickable."

By the way, clicking my 3000 point .geojson file did nothing. But that is an issue for #600 .

No longer relevant after UI refresh


fix: fixed error check

Fixed comment from https://github.com/abcxyz/abc/pull/383#discussion_r1467205091

Thanks!


Ambient integration with Kiali

Opening an issue to track necessary ambient integration with Kiali, a few things I noticed when viewing ambient workloads from Kiali UI:

missing sidecars are flagged as warning. While this is useful for sidecars, may not be useful for ambient without sidecars.
would we show L4 data plane components - ztunnels?
would we show L7 data plane components - waypoints?

Also from @costinm: making sure Kiali is well integrated with OpenTelemetry and is on board with Istio ( and proxyless gRPC and other native-support telemetry ) generating metrics with OTLP and the common semantic conventions.

Hi @linsun ,  thanks for opening this on the Istio side.  On the kiali side we also have https://github.com/kiali/kiali/issues/5476 for reference.  It's a high-level epic where we track some tasks and have discussion about Ambient support.
We have spent a bit of time on Ambient but are also waiting for the dust to settle some, and for users to start providing feedback on Kiali behavior with Ambient.  Here is a blog about some of initial initial support https://medium.com/kialiproject/kiali-support-for-istio-ambient-mesh-alpha-7f91ea8c6f54.

missing sidecars are flagged as warning. While this is useful for sidecars, may not be useful for ambient without sidecars.

We agree that it's not useful to flag missing sidecars for ambient-enabled namespaces.  In the blog you can see how we are migrating to "out of mesh" warnings.  We have more to do here.

would we show L4 data plane components - ztunnels?
would we show L7 data plane components - waypoints?

We have added some labeling for ztunnel components, and also have added waypoints to our list view.    As for metrics in general, we're following the discussion here https://github.com/istio/istio/issues/42320 and also evaluating things as the metrics emerge.  The L4 TCP metrics are showing up in the graph as the ztunnel is generating those familiar metrics.  Although, the L4 metric reporting is now more complete, you don't have TCP or HTTP, you get TCP and HTTP, so currently you will see multiple edges for the different protocols, for a single HTTP request.
We welcome ideas and reports in the Kiali github Discussions and Issues!

We can probably let the bot close this one, please direct Ambient+Kiali issues to Kiali directly, unless it's directly related to support for Kiali on the Istio side.


Feat:Want to Create UI Of Profile page #823

Created Profile page made it responsive #823

@nishant0708,  I just wanted to check if you've had a chance to synchronize your fork. Please let me know once it's confirmed.

@mdazfar2 it is now


Confusing command-line parsing for --noheader

Thanks for creating and maintaining this package, it looks very useful!  Just a small suggestion from a newcomer who had a rather bumpy time getting started.  I tried:
$ csvtojson --noheader file.csv
Unknown boolean value  for parameter --noheader.
[... 33 lines of helptext ]

So then I tried
$ csvtojson --noheader true file.csv
Unknown boolean value  for parameter --noheader.
[... 33 lines of helptext ]

At this point I dug into the source and found that the accepted syntax is
$ csvtojson --noheader=true file.csv

Is there any chance you might consider making the option parsing compatible with getopt_long(3) (possibly by using a library like minimist, yargs, or commander)?  If not, perhaps a clearer error message?  Maybe also reducing the amount of helptext shown so errors fit on a 80x24 terminal?
Thanks for considering,
Kevin

@jfoclpf I don't see anything in the docs you linked about the command-line option parsing this issue describes (--noheader vs --noheader true vs --noheader=true).  Even if it were documented, I would argue it is better to follow the getopt parsing conventions users are likely to expect rather than documenting non-standard conventions.

you're right, I read it too quickly


New chinese simplified language package

New chinese simplified language package.
simplified Chinese translation for users of mainland China.

correct translation.


strange delays in message delivery to pubsub cloud function

Not sure if this is a bug report or a support request, sorry!
I'm trying to understand some strange behaviour I'm seeing from time to time.  In summary, I am occasionally seeing messages that are just sitting in the subscriber, for no apparent reason, until their acknowledgement deadline is reached, at which point they get delivered.  It doens't happen very often, but when it does it wreaks havoc on my system because it doesn't deal well with these huge (and unexpected) latencies.
Here is an example (apologies for using screen shots).
My GCE server sends messages and it does so, it them and logs them along with their pubsub message id:

The message in question (above) has id<PHONE_NUMBER>777511, and was sent at 2020-04-12T22:58.20...Z.
The consumer of this message is a cloud function that has an acknowledgement deadline of 600s, and is deployed with the following command:
gcloud beta functions deploy --quiet handleClientOrder --runtime nodejs10 --max-instances 10 --memory 512 --region europe-west1 --timeout 30 --trigger-topic client_orders --retry --set-env-vars=FUNCTION_REGION=europe-west1,GCLOUD_PROJECT=my-project-id,DEPLOYMENT_TIME=2020-04-12T11:21:56.603Z --entry-point app --allow-unauthenticated
Typically, 99+% of messages are delivered within a few milliseconds if there is no backlog, which there rarely is.  The cloud function also logs when it consumes the message:

You can see that the message in question was consumed at 2020-04-12T23:08:21...Z, exactly 10 minutes after it was sent.  When I search Stackdriver for that message id, I can see that it was delivered once and only once.
I can see at the time there is no backlog in the subscriber, but I do see a large spike in the oldest unacked message, and I also see my unacked message count flat lining above zero for a period of time:

It's as if the cloud function just decided not to consume this message until the last possible moment, possibly others too.
So my questions are:

is this expected behaviour?
if so, what causes it?  is there a way to avoid it
if not, what can I do to investigate the cause?

From the Pub/Sub perspective, it looks like this message was delivered right after it was published, but received a 503 error. It was then redelivered 10 minutes later. The 503 error could cause backoff behavior in redeliveries, particularly if there were other 503 errors happening at the time. Therefore, it looks like something happened outside of Pub/Sub itself. It could be something with the Cloud Function or with the application code processing the message.

Where do you see this 503 error?

You should be able to see it in the Stackdriver subscription/push_request_count metric (see pubsub metrics), which can be broken down by response_class and response_code.

  I didn't show any 503 in the data I posted in this ticket (did I?).  Do you have access to the data?

I was able to look at internal logs for the provided message ID.

Oh, thanks for taking the time!
Thanks for pointing me at the metrics explorer.  I can't see the problem though.  I filtered the time range down to that in question (one hour either side) and I don't see any errors.

It could be a transient issue with Cloud Functions infrastructure, e.g., if a server restarts at the moment the message is sent.

It's happening often enough that it's becoming quite a problem for me.  The messages doesn't get redelivered until the acknowledgment deadline, and that's set to a very high figure compared with the normal latencies that my app expects.
Is there a workaround?
How do I file a bug report?

It could happen occasionally because acks and nacks (which is what the 503 is interpreted as) can be dropped under certain circumstances. We are aware that the frequency of these has increased and are working to mitigate it, though it will take a few weeks for such changes to roll out. If you want to file a bug report, it is best to create a support case via the support page in the Cloud console.

Thanks.
https://issuetracker.google.com/issues/154260226


CFP: Cilium Document Localizations for Chinese

Cilium Feature Proposal
Is your feature request related to a problem?
No
Describe the feature you'd like
Fluent in English reading && writing is the best way to learn and get involved. But people can get start more easier and faster if they can read the document in their mother  language. So maybe we need a way to do some localizations work like https://github.com/cilium/cilium/pull/17542.

Hi @joestringer , I found we have 21 chapters need to be translated. Should we translated all of them this time? Maybe chapter like gettinghelp and contributing which less relevant with core concepts or usages of Cilium could put secondary priority and won’t block we publish the documents.


Enable interactive account creation

I'd love to have for public beta is provisioning a new account, ideally with an interactive prompt.
i.e.
$ rack register
Register a new Rackspace Cloud account:
Username: _

For now, I'm marking this as a question and assigning it to @kenperkins . We need to find out which API to query for this, and how it differs, if at all, from the signup process on rackspace.com. In addition, we should consider any vulnerabilities this feature may allow.

If we can address the questions/issues above :point_up: this week, we can have this for public beta.

Yup, I'm looking forward to this. Any word on the API @kenperkins?

@kenperkins was this determined as doable or not?


[Security] Exclude and remove freebuilder dependency

Motivation
Freebuilder is an annotation processor used in Bookkeeper's StorageClientSetting interface:
https://github.com/apache/bookkeeper/blob/16e8ba772bb5cf4c7546fb559bd9d455d4e42625/stream/clients/java/base/src/main/java/org/apache/bookkeeper/clients/config/StorageClientSettings.java#L27-L33
The annotation processor is only needed at compile time.
The Freebuilder library gets flagged as a vulnerable library by Sonatype IQ. This causes Pulsar distribution to be flagged as vulnerable since Freebuilder is a transitive dependency.
Additional context
There's a separate issue in Bookkeeper to change the dependency to optional / compileOnly: https://github.com/apache/bookkeeper/issues/2732
Modifications
Exclude freebuilder library and replace the code that used shaded dependencies from the freebuilder library.

Approved


Jhbertra/ps014

This PR:

Formats the codebase with Ormolu
Removes all bu nix support for building
Removes the stale example code
Adds a compilation test (to make sure the generated test can be built)
Adds golden tests
Inlines every dependency that used to require servant-support on the frontend
Fixes all issues associated with PS 14 upgrade
Bumps pinned dependency version for purescript-bridge

Good question... yeah, we'd need to add some checks ourselves. It looks like all it did before was build the project... which is all it could have ever done, seeing as there were no tests :sweat_smile:


fix: Make "Editor Settings" gear icon blue only when it's open

Fixes: #495

Can one of the admins verify this patch?


[iOs] Apple signin user details on authentication doesn't contain name

Bug report
CHECKLIST

[x] I have read the issue reporting guidelines

[x] I confirm this is a suspected bug or issue that will affect other users

[x] I have reproduced the issue using the example project or provided the necessary information to reproduce the issue.

[x] I have read the documentation thoroughly and it does not help solve my issue.

[x] I have checked that no similar issues (open or closed) already exist.

Current behavior:
Edited authenticateUserWithApple function to the code mentioned below
Expected behavior:
The object is supposed to contain a key with name
Environment information

Cordova CLI version
9.0.0<EMAIL_ADDRESS>

Cordova platform version
ios 6.1.0

Plugins & versions installed in project (including this plugin)
cordova-custom-config 5.1.0 "cordova-custom-config"
cordova-plugin-androidx 2.0.0 "cordova-plugin-androidx"
cordova-plugin-androidx-adapter 1.1.1 "cordova-plugin-androidx-adapter"
cordova-plugin-customfcmreceiver 1.0.0 "Custom FCM Receiver"
cordova-plugin-dialogs 1.3.2-dev "Notification"
cordova-plugin-enable-multidex 0.2.0 "Enable Multidex"
cordova-plugin-firebasex 10.0.0 "Google Firebase Plugin"
cordova-plugin-hello-c 1.1.1 "Hello"
cordova-plugin-whitelist 1.3.4 "Whitelist"

Dev machine OS and version, e.g.
Mac OS X, 10.15.6

Runtime issue

Device details
iPhone SE (2nd generation) simulator

OS details
iOS 13.6

Related code:
function authenticateUserWithApple(){
    FirebasePlugin.authenticateUserWithApple(function(credential) {
        authCredential = credential;
        FirebasePlugin.signInWithCredential(credential, () => {
            FirebasePlugin.getCurrentUser((user) => {
                console.log(user);
            }, function () {
                console.log('Error');
            });
        }, (error) => {
            console.error("Failed to sign in", error);
        });
    }, function(error) {
        logError("Failed to authenticate with Apple", error);
    }, 'en');
}

Console output
2020-08-05 20:39:08.847826+0530 FirebaseTest[60749:4515213] FirebasePlugin[native]: Enter foreground: FCM direct channel = false
2020-08-05 20:39:09.676455+0530 FirebaseTest[60749:4515213] {"email":"***@gmail.com","isAnonymous":0,"idToken":"***","uid":"***","emailIsVerified":true,"providerId":"apple.com"}
2020-08-05 20:44:06.225969+0530 FirebaseTest[60749:4515213] FirebasePlugin[native] LOG: Enter background: FCM direct channel = false
2020-08-05 20:44:06.399961+0530 FirebaseTest[60749:4515213] FirebasePlugin[native]: Enter background: FCM direct channel = false

The plugin will only return values in the user object in the getCurrentUser() callback which are not empty in the current Firebase user account.
In the case of user's name, this is not being returned because it is returning a nil value from the Firebase iOS SDK:

According to this open issue against the Firebase iOS SDK, this is expected behaviour because the Apple ID token doesn't contain the user's name.
Therefore rather than being a bug with this plugin, this is a current limitation of the Firebase SDK which may or may not be resolved in a future version.
There is a suggested workaround in the same issue which involves implementing the didCompleteWithAuthorization delegate method to extract the user's full name and update the user's Firebase account profile with it.
However this adds considerable complexity to the plugin and involves writing data back to the user's Firebase account profile because the Firebase SDK currently doesn't do this itself, so I consider this to be a low-priority enhancement as opposed to plugin bug.

Understood. Thank you for the quick response.
My bad. I should've searched the firebase sdk for similar issues. I think this issue can be closed.

@chaitanya-deep There's still the possibility of adding the workaround to this plugin as an enhancement if other plugin users think this is worthwhile.
So I'll leave this open as a possible future enhancement until either the Firebase SDK fixes it or we implement the workaround.

Hello, i saw a new version was released 6 days ago and full name or name is still not implemented. Is there any updates?

I have a fork with a few changes that I will split up into separate PRs after doing lots more testing, but what I have done is return the first and last name provided by Apple during sign up only. It doesn't attempt to write this to the Firebase displayName property but makes it available to your app to do so.
https://github.com/tzappia/cordova-plugin-firebasex/tree/auth-emulator-and-apple-info


Adding template Snowflake to GCS

Added Template Snowflake to GCS. Tested for all possible inputs i.e. json, parquet, avro, csv.

Awesome work Varunika! LGTM Obs: In line 129 of the readme it is written gs:test_bucket instead of gs://test_bucket

Noted. Will send in an edit in the next PR.


IOException when encoding with Base64Pipe - IOException: Attempted read on closed stream

❗ please do not add sensitive information in issues, you can provide extra information via email using issue number as reference ❗
Describe the issue
An IOException is being thrown when encoding with the Base64Pipe. The exact configuration was still working with the F!F Docker container from Nexus around 2023-04-17 7.9-SNAPSHOT. 2023-05-23 was the time our end-to-end test failed on this error.
The issue seems to happen in the pipe called EncodeInhoud, returning an empty string, while it should return a base64 string. The actual error seems to be thrown in the adapter calling the sub-adapter that executes the EncodeInhoud pipe.
Reporter
Marcel Lenterman

To Reproduce
zaakbrug.log
zaakbrug.log.1.log
Pipeline GetBas64Inhoud and 1 more.zip
Screenshots
If applicable, add screenshots to help explain your problem.
Configuration
https://github.dev/ibissource/zaakbrug/blob/ab08568891a020a7b47df09019d7fba1fc2db232/src/main/configurations/Translate/Configuration_GetBas64Inhoud.xml#L41-L42
Input
Please provide an example of the input message.
Alternatively, provide a Ladybug report.

Environment
FF! 7.9-20230607.190024: ZaakBrug
Running on f2c4ca8485e2 using Apache Tomcat/8.5.85
Java Version: OpenJDK Runtime Environment (1.8.0_362-b09)
Heap size: 396M, total JVM memory: 868M
Free disk space: 230GB, total disk space: 251GB
Up since: 2023-06-08 17:26:21 (18h)

Additional Environment

DBMS: [e.g. Oracle, MSSQL, MariaDB, MySql, PostgreSQL]
Browser: [e.g. Chrome, Safari, Edge, Firefox]

Additional Context
Add any other context about the problem here. (f.e. ladybug report / test adapter with larva test)

The cause of this exception must be in the result of the HttpSender that fetches the data.


Add loy bot + a Dockerfile for convenience

The flag --break-system-packages sounds scary and I didn't want to find out what it really does. Thus whipped up a Dockerfile to not break local setup.
My bot doesn't do anything beyond the SimpleBot yet, so commented out for the time being.

The flag --break-system-packages sounds scary

:laughing: This has actually been the default until 2024


Please add LICENSE file

At the moment, only the package.json specifies the license (MIT). To make the license clear it would be good to have a dedicated LICENSE file with the complete MIT license text, as it is (presumably) good practice at least for GitHub repos.
Anyway, thank you for your library!

License file added


Error when using multiple files in different languages

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Neovim version
NVIM v0.9.0-dev-329+ged27f0e93
Operating system/version
archlinux
Terminal name/version
WezTerm
$TERM environment variable
xterm-256color
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
First create a folder, then in this new folder create two new files in different languages, e.g. a.py and b .c. Then use nvim to edit one of the files, write something like print "hello world", then use the command :e b.c (assuming you edited a.py first). py), and then get an error and can't move through the text at all using hjkl.

Expected behavior
No errors were reported, it was able to be edited normally, and lsp was working properly
Actual behavior
Keeps popping up errors and can't move the cursor at all, let alone edit anything!
Additional information
No response

Possible duplicate of https://github.com/ayamir/nvimdots/issues/393.

another error

These two are essentially the same.

Possible duplicate of #393.

After using :TSUpdate it works fine, thanks!


Wsb warsaw university

Wyższa Szkoła Bankowa w Warszawie
WSB University in Warsaw
@student.wsb.warszawa.pl

@walim12 I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge too many files, most of them do not relate to the organization which you would like to add to this repository. Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


Webapp layout updates

Various layout changes to the webapp, including:

Colors matching IDM InfoHub
IDM logo
Sentence case
Content rename (e.g. "Advanced" > "Model parameters")
Collapsible cards for all configuration options
Removed history
Better backend error logging
Modified graph styling
Nicer font :)

Almost certainly some of the CSS/HTML could be tidied up (though I resisted the urge to use &nbps; and <br> to do layout formatting!).

Closing since included in #171


Ingest endpoint doesn't report document errors.

Describe the bug
When using the ingest endpoint to index a json formatted body and there is an error in one of the document, the error will not be reported to the user.
Steps to reproduce

Use the following mapping:

version: 0.5
index_id: friends
doc_mapping:
  mode: "lenient"
  field_mappings:
    - name: name
      type: text
    - name: points
      type: i64
      fast: true
    - name: last_modified_at
      type: datetime
      fast: true
indexing_settings:
    commit_timeout_secs: 1

Insert a set of documents with an invalid document see the PH entry with a last_modified_at date set to February 31.

curl -XPOST "http://ogdru-jahad.local:7280/api/v1/friends/ingest" -H 'Content-Type: application/json' -d '{ "id": 1, "name": "Bernard", "points": 10, "last_modified_at": "2023-04-17T20:40:00Z" }
{ "id": 2, "name": "Alice", "points" :  20, "last_modified_at": "2023-01-01T12:00:00Z" }
{ "id": 3, "name": "Caroline", "points": 50, "last_modified_at": "2023-04-18T13:00:00Z" }
{ "id": 4, "name": "PH", "points": 5, "last_modified_at": "2023-02-31T00:00:00Z" }
{ "id": 5, "name": "Bob", "points": 42, "last_modified_at": "2023-01-28T03:00:00Z" }'

The API response:
{
  "num_docs_for_processing": 5
}

But only 4 documents will be indexed and I do not see anything in the log but I am running at INFO level.
Expected behavior
I would expect to see some kind of parsing error returned to the user.
Configuration:
Please provide:

Output of quickwit --version
Quickwit 0.5 (docker image)

I think we can close this as a duplicate of #1524. @ph, ingest works a bit different in quickwit comparing to elasticsearch. please see this https://github.com/quickwit-oss/quickwit/issues/1524#issuecomment-1135448224 for more information.

Sound good to me, thanks for the details.


Fix generated PRODUCT_BUNDLE_IDENTIFIER

Underscores are illegal in bundle identifiers, so we should post process the C99 identifiers and replace them. Xcode uses dashes, so let's use those, too.

This matches what Xcode does to generate bundle identifiers now and should be good to go.

LGTM


Change player's EMO pool by rival's final boards

Currently, a player's EMO pool is constant, regardless of the activities of rivals. At the start of the match, the player's fixed EMO pool (EMOs not built in the player's deck) is determined.
I propose to make the EMOs in the rival's final boards being reduced from the player's pool.
This is for adding variety to each match and reduce mirror matches.

This issue has been inactive for a while, so I'm closing it.


v0.2.1

Bump to v0.2.1 as it is already published

Coverage remained the same at 99.408% when pulling 673de336b7bf149e21823f0b4f9cdab843279e4b on bump/v0.2.1 into 06ebfd8feffa827e53b9b9dc0eb1ec0b7de61d44 on master.


Windows Portable doesn't accept same parameters as web.py

For the portable Windows executable to really be portable (e.g. run off an encrypted volume on a USB stick), it'd be useful for it to accept the same parameters as the web.py launcher so all the identity, configuration, and storage can be stored alongside it.  Right now, it's hardcoded (in main.js) to ~/.reticulum-meshchat for storageDir with no option to set configDir.  It'd be useful to accept these:
  --identity-file IDENTITY_FILE
                        Path to a Reticulum Identity file to use as your LXMF address.
  --identity-base64 IDENTITY_BASE64
                        A base64 encoded Reticulum Identity to use as your LXMF address.
  --reticulum-config-dir RETICULUM_CONFIG_DIR
                        Path to a Reticulum config directory for the RNS stack to use (e.g: ~/.reticulum)
  --storage-dir STORAGE_DIR
                        Path to a directory for storing databases and config files (default: ./storage)

In the interim, a workaround is to extract the Electron app, edit resources/app/electron/main.js, and change line 83 to modify storageDir to the desired path.  Then, add code like this below it to set configDir to the desired path:
        // determine path for config
        const configDir = path.join('.reticulum'); // .reticulum

Finally, change the exeChildProcess call to:
        // spawn executable
        exeChildProcess = await spawn(exe, [
            '--headless', // reticulum meshchat usually launches default web browser, we don't want this when using electron
            '--port', '9337', // FIXME: let system pick a random unused port?
            '--storage-dir', storageDir,
            '--reticulum-config-dir', configDir,
        ]);

to use the new configDir path.  Then, just run the extracted Reticulum MeshChat.exe directly instead of the packaged portable file, and it'll use the modified JS.

Hey! Thanks for reporting this, I haven't forgotten about it. Just been a bit busy!
I want to add a UI for managing config paths etc, but for now it's probably easier to just passthrough any command line args passed to the .exe and merge them in to the default provided in exeChildProcess.
Will plan to sort this before the next release.

This is now supported and has been merged into the main branch with commit a804e16f3d28d011d410ca017eacbc7a97b6187f
I tested this on my Windows machine by creating a shortcut link to the portable .exe with a custom --storage-dir and --reticulum-config-dir. I did have to provide an absolute path and not a relative path, otherwise the folders were created elsewhere. Unsure where, because the shortcut had the working dir set to where it was opened from...
I have not tested this on an external USB drive yet, but will attempt this later on. Let me know if this works for you and I can close this issue :)


change label propTypes to allow element

According to https://github.com/react-toolbox/react-toolbox/issues/1306 when using a React element for the label (because of translations for example) raises an invalid proptype error.
I changed all the label proptypes to allow both, string and element.

Hi @rubenmoya.
Please check if you can use PropTypes.node for this.
// Anything that can be rendered: numbers, strings, elements or an array
// (or fragment) containing these types.
optionalNode: React.PropTypes.node,

React Proptypes

I thought about it @rafarovina but it says you can use arrays of those types, which I don't think would work for a label.

Thanks for the PR @rubenmoya but I'd like to evolve the discussion in the original PR to decide if this should be done and which components should be changed :)


How to extract private key as String

How to extract private key as String in this repo. Please share code in Swift ?.

@hboon

You want to look at EtherKeystore.signHash(_:for:) and how it gets the key.

so you don't know the exact code ?


GAME_MODS_IDS_LIST does not add mods

Describe the bug
Populate the variable GAME_MODS_IDS_LIST with mod IDs and after recreating the container, mods are not added to the server
To Reproduce
My docker-compose.yml

version: '3.8'
services:
arma-reforger:
image: ghcr.io/acemod/arma-reforger:edge
platform: linux/amd64
container_name: arma-reforger
ports:
- "2001:2001/udp"
- "17777:17777/udp"
volumes:
- /opt/reforger/configs:/reforger/Configs
- /opt/reforger/profile:/home/profile
- /opt/reforger/workshop:/reforger/workshop
environment:
- SERVER_PUBLIC_ADDRESS=foo
- GAME_NAME=foo
- SERVER_A2S_ADDRESS=<IP_ADDRESS>
- SERVER_A2S_PORT=17777
- GAME_PASSWORD_ADMIN=foo
- GAME_MAX_PLAYERS=40
- GAME_PROPS_DISABLE_THIRD_PERSON=true
- ARMA_MAX_FPS=60
- GAME_MODS_IDS_LIST=5994AD5A9F33BE57,5ED8BE1DDFEFBE4C,5D071A427403AB2C

Here are all logs from the container
console.log
error.log
script.log

I think I got the cause, working on a fix

Actually I though the cause was something else ( #26 ) but rather it's the fact that support for mods was introduced after edge->latest switch from #22 , so it's not present in edge tag.
Switch to image: ghcr.io/acemod/arma-reforger:latest will solve the issue.

I totally missed that, thank you! I used the docker-compose.yml template in the project, so I did not even look at the image tag properly :) Now I am just patiently waiting for #26 to be approved.


adding values on the bars

I feel like this would better live as a plugin. That would allow it to label all different types of charts rather than being tied specifically to horizontal bars

sure. i'm on it
Can you guide me a little bit on plugins though.

@apoorvasrinivasan I think elements should have a getCenterPoint() method but I don't think we've designed it to be publicly available so it could change

Are all these functions defined in core.elements ? Where are the functions
defined ?
Sending from mobile
Apoorva
@apoorvaShrini
On Apr 26, 2017 03:45, "Evert Timberg"<EMAIL_ADDRESS>wrote:

@apoorvasrinivasan https://github.com/apoorvasrinivasan I think
elements should have a getCenterPoint() method but I don't think we've
designed it to be publicly available so it could change
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chartjs/Chart.js/pull/4176#issuecomment-297181066,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALTQIwNW00g2UkCZnhYf01RKvXRqEWO6ks5rznB9gaJpZM4NGaM0
.

@etimberg  can you please have a look at the plugin.

Code looks OK. please add docs and tests

@etimberg i have added a basic test case. please have a look

An overall thought on these new features: it's a good start but since it doesn't work on any kind of chart, the API risk to change quickly (see #4169). That's why I would have preferred to see it implemented as an external plugin (e.g. chartjs/chartjs-plugin-datalabels) to give us a chance to not deal with deprecation.
@etimberg It could be a good process to incube substantial new features in external plugin first and if it makes sense, merge these plugins in core when feature complete and API stable enough.

@simonbrunel I'm ok with having a process like that. I definitely agree about the API changing rapidly as @apoorvasrinivasan iterates on features/chart types. I think this would make a solid external plugin.

@simonbrunel  @etimberg
i have shifted the options to the plugin and changed variable names.  I am not sure i quite understand what i need to do next

@apoorvasrinivasan We are thinking to move that PR to another repository (chartjs/chartjs-plugin-datalabels) as an external plugin. Though, we will need one (or more) maintainer(s) to make that feature stable, work on all types of charts and fully customizable, so if you are interested in contributing more, we can setup that new repository and give you full access to it. What do you think?
Could also interest @omarsedki

Sure :)
Sending from mobile
Apoorva
@apoorvaShrini
On May 9, 2017 02:04, "Simon Brunel"<EMAIL_ADDRESS>wrote:

@apoorvasrinivasan https://github.com/apoorvasrinivasan We are thinking
to move that PR to another repository (chartjs/chartjs-plugin-datalabels)
as an external plugin. Though, we will need one (or more) maintainer(s) to
make that feature stable, work on all types of charts and fully
customizable, so if you are interested in contributing more, we can setup
that new repository and give you full access to it. What do you think?
Could also interest @omarsedki https://github.com/omarsedki
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chartjs/Chart.js/pull/4176#issuecomment-299982706,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALTQI1RmK9v6K-S29TEgctSjjsUCILslks5r33xRgaJpZM4NGaM0
.

+1 to making it a plugin rather than core feature. This PR should probably be closed and the feature pursued in a separate repo

@apoorvasrinivasan I finally found some time to create this new repository and import the initial code. I rewrote most of your implementation to make it work with any type of chart but also to exposed more options (see options.js and the positioning and styling examples which are not yet in the repository).
All options also accept a function, which is evaluated for each data point, for example:
color: function(index, context) {
    return Math.abs(context.value) >= 50? 'white':
        context.value < 0? 'rgb(255, 128, 128)':
        'rgb(128, 128, 255)';
},
backgroundColor: function(index, context) {
    var value = context.value;
    if (Math.abs(value) < 50) {
        return null; // disable background
    }

    var factor = (context.value+100)/200;
    return Color('rgb(96,96,250)')
        .mix(Color('rgb(250,96,96)'), factor)
        .rgbaString();
},
borderColor: function(index, context) {
    return context.value < 0?
        'rgba(250, 50, 50, 1)' :
        'rgba(50, 50, 250, 1)';
},
borderWidth: function(index, context) {
    var ref = Math.abs(context.value);
    return ref >= 50 || ref < 25? 0 : 1;
}

Feedback and contributions welcome of course and since there is no release yet, we can still change options name/behavior if it can be better. Note that it requires a build of Chart.js from latest master.


Chore: Upgrade RabbitMQ to 3.6.10

Upgrade RabbitMQ from 3.6.9 to 3.6.10 (changelog).
This release contains bug fixes and usability improvements.

Blocked: waiting for CloudAQP to catch up.

This is not relevant anymore, RabbitMQ 3.7 has been released.


Update plugins

Updated plugins and I can't produce https://github.com/robolectric/robolectric/issues/2160

Why use CircleCI? They never have the latest build tools: https://circleci.com/gh/robolectric/deckard/32.

@mag We might want to remove the emulator. It does not work very well.

@jaredsburrows on the CircleCI front - sigh. I really like Circle but I wish they kept their android packages a little fresher. I will reach out to them - we use them for Android builds at my paying gig and have to do a lot of custom installation logic. In the meantime feel free to ignore the failing circle build or take it out entirely.
 It was working before this upgrade though.  Do you have any up-to-date knowledge of running the emulator on CI that makes you so pessimistic? It's been a while since I've looked at it.
I would love to keep the Espresso test in Deckard, since Espresso seems to have a lot of mindshare. But I still don't use it day-to-day because every time I try to get it running on CI I run into insurmountable problems. I keep hearing that GOOG has solved these problems but when I pester them I get nothing :rage4:
@seadowg @erd any insight? Should we give up on Espresso in Deckard?

Honestly? Given that it's light years easier to get Robolectric 3.0+ up and running with Android Studio, I don't really see much use for this project any longer. When people ask for examples of Robolectric tests, I usually point them at robolectric-samples. I do love the name Deckard, though (thanks @seadowg!). I'd hate to kill the project for that one fact alone.

@mag In that case the PR is ready for merge. Travis + Espresso + Emulator does not work very well.
@erd If you do not want this, you can delete this repo. I only made this PR based on an issue that someone else brought up in robolectric/robolectric#2160. For test examples, users should look at the tests within Robolectric.

@erd It's true, the need for Deckard has diminished. But I still have scars from when it was a giant pain. Until Google says that Robolectric is officially supported, I say we keep it. But in the mean time I'm OK if we pull out the Espresso tests and dependencies.

@mag Is Google looking into to saying that Robolectric officially supports Android? Like they did for Espresso? I will pull out the Espresso so we can finish this.

@jaredsburrows Sounds good. Not exactly sure what you mean by that question, but I certainly don't know of any plan by Google to support Robolectric. They try not to break us and that's been working out pretty well of late. And they do listen when we whine loud enough :wink:

@mag That's really good. Without Robolectric testing is extremely painful with normal "instrumentation" testing.


Assertion failure in snippet.ml

I don't have a MWE, but I suspect it is related to EOF. Here is the trace:
Uncaught exception:

  "Assert_failure lib/ansi_renderer/snippet.ml:319:4"

Raised at Grace_ansi_renderer__Snippet.Of_diagnostic.segments_of_labels in file "lib/ansi_renderer/snippet.ml", line 319, characters 4-40
Called from Grace_ansi_renderer__Snippet.Of_diagnostic.line_of_labels in file "lib/ansi_renderer/snippet.ml", line 326, characters 19-54
Called from Grace_ansi_renderer__Snippet.Of_diagnostic.lines_of_labels.(fun) in file "lib/ansi_renderer/snippet.ml", line 471, characters 12-148
Called from Iter.fold.(fun) in file "src/Iter.ml", line 77, characters 23-31
Called from Grace_source_reader.lines_in_range in file "lib/source_reader/grace_source_reader.ml", line 283, characters 4-59
Called from Iter.fold in file "src/Iter.ml", line 77, characters 2-32
Called from Grace_ansi_renderer__Snippet.Of_diagnostic.lines_of_labels in file "lib/ansi_renderer/snippet.ml", line 426, characters 6-1023
Called from Grace_ansi_renderer__Snippet.Of_diagnostic.block_of_labels in file "lib/ansi_renderer/snippet.ml", line 482, characters 4-33
Called from Grace_ansi_renderer__Snippet.Of_diagnostic.of_diagnostic.(fun) in file "lib/ansi_renderer/snippet.ml", line 493, characters 19-45
Called from Base__List.map in file "src/list.ml", line 433, characters 15-18
Called from Grace_ansi_renderer__Snippet.Of_diagnostic.of_diagnostic in file "lib/ansi_renderer/snippet.ml", line 487, characters 6-262
Called from Stdlib__Fun.protect in file "fun.ml", line 33, characters 8-15
Re-raised at Stdlib__Fun.protect in file "fun.ml", line 38, characters 6-52
Called from Grace_ansi_renderer.of_snippet_renderer in file "lib/ansi_renderer/grace_ansi_renderer.ml", line 14, characters 16-87
Called from Stdlib__Format.output_acc in file "format.ml", line 1355, characters 4-20

I am writing a mini-language. Here is the input, and the expected error output message:
Input:
x :

Expected output:
-error[EP004]: EOF found instead of type
- --> test/parsing/declaration_eof.kk:1:3
-  |
-1 | x :
-  |    ^ EOF found here
- ```

Maybe a duplicate of #1.


PPA updating ansible to this python version and running the QA deploy to confirm that everything works there

Please try updating ansible to this python version and running the QA deploy to confirm that everything works there. The python version is specified in group_vars/prosody/vars.yml — changing that should be all that's needed, since the ansible playbook installs and configures the version we have set.
Originally posted by @rlskoeser in https://github.com/Princeton-CDH/ppa-django/issues/495#issuecomment-1323749158

Resolved in PR: https://github.com/Princeton-CDH/cdh-ansible/pull/129

This should be resolved now. Deploys to QA have been working well and the python version was changed here.
Can also confirm we're using correct python on QA deploy:
Welcome to Ubuntu 18.04.6 LTS (GNU/Linux 4.15.0-39-generic x86_64)

 * Documentation:  https://help.ubuntu.com
 * Management:     https://landscape.canonical.com
 * Support:        https://ubuntu.com/advantage
Last login: Fri Jan 13 14:21:19 2023 from <IP_ADDRESS>
pulsys@cdh-test-prosody1:~$ conan
(ppa-3.9.0-dev-8aa36a) conan@cdh-test-prosody1:/srv/www/ppa/current$ python
Python 3.9.16 (main, Dec  7 2022, 01:11:58)
[GCC 7.5.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>


docs(turborepo): correct default ui config value

Description
v2.0.6 flipped the default ui config back to stream: #8631

Allow CI Workflow Run

[ ] approve CI run for commit: 7ba220e0e6efde6c2733f7a52dbb38fb80b89931

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer


Is it possible to dequeue NibOwnerLoadable?

If not, what is the recommended way to use NibOwnerLoadable on a cell(table/collection) class.

NibOwnerLoadable is for when you design a XIB with your class being set on the File's Owner.
The way UITableView and UICollectionView work in CocoaTouch, as designed by Apple, impose you that all your custom cells designed in dedicated XIBs have an empty File's Owner and have their cell's contentView as the root of the XIB. So this configuration corresponds to the NibLoadable protocol, not the NibOwnerLoadable protocol.
NibLoadable

You don't have a File's Owner in your XIB
The view you intend to load is at the root of your XIB
When Interface Builder, or your UITableView or UICollectionView, will load the NIB file, it will unarchive the NIB and instantiate the root view and all its subview. That's how UITableView works, you don't have a choice for that, that's how Apple implemented the dequeueReusableCell(…) method for cells registered with a XIB/UINib.

So it's suitable for UITableViewCells & UICollectionViewCells, or for custom views you plan to create by code
NibOwnerLoadable

That's for when you set your File's Owner of your XIB to your custom class
By definition, in a XIB, a File's Owner is an instance that is supposed to be already created by yourself and that you pass to the UINib.instanciate(owner:options:) when unarchiving the XIB. So the way it's supposed to work in this configuration, given how XIB work, is that you create an instance of your class, then you unarchive the XIB by giving it that newly created instance as parameter to take the place of the File's Owner.

So it's only suitable for views that you either:

create by code (let obj = MyClass(frame: someFrame) then obj.loadNibContent() to unarhive/load the NIB with obj as its File's Owner)
or that you will add to another storyboard or XIB (because that other storyboard will then create your instance of MyClass by calling its init(coder:) when the storyboard itself unarchives, then in that init(coder:) method you can call self.loadNibContent() to unarchive/load the content of the XIB with self as File's Owner)

But in both case with NibOwnerLoadable you create an instance of your class (by code or because it's in another storyboard or XIB) then load the XIB, which is suitable for custom views / widgets that you plan to reuse in multiple places in your app, but not for UITableViewCell / UICollectionViewCell because that's not how Apple's dequeueReusableCell method works (it expects the class to be part of the NIB's root view and be instantiated when the NIB is unarchived, not already existing and be passed by reference as File's Owner)

Thanks for the crystal clear explanation. I think this writeup should be put in somewhere in the documents, for teaching people how it works behind the scenes. It is super useful (at least for rookie like me).

Thank you for explanation!


Help people add their own macros

We could provide a folder to put your own macros in, and document how to use them.
Relevant support thing where a user tried to put custom components outside the app/views folder and Nunjucks refuses to load them.
This work might need to wait until we are more confident about macros, for example error handling.

Closing in favour of #585 which I believe covers this – this one came first, but that one has a PR associated with it.


HA not matching vera status for lights/switches

Home Assistant release (hass --version):
pi@hassbian:/home/homeassistant/$ hass --version
-bash: hass: command not found
Python release (python3 --version):
pi@hassbian:/home/homeassistant $ python3 --version
Python 3.4.2
Component/platform:
vera / raspberry pi2
Description of problem:
i walk up to my outdoor lights switch at the wall and turn them on. vera shows them as on (pretty much immediately), HA sometimes will show them as on but most of the time doesn't. i even waited a while on a few tests just to make sure it wasn't something where i needed to decrease the polling interval...i waited at least 30 seconds with no change in the HA display.
Expected:
HA display should match vera display (i.e. lights on/off should be reported instantaneously, or at least close to it)

Can you add this to your config and operate the switches - and then post the logs?
logger:
  logs:
      pyvera: debug

It should allow me to see what events are being received.
It's probably the same as https://github.com/pavoni/pyvera/issues/51

i need to set aside some time to test this, i'll see if i can do that this week.  it's definitely not the same as the issue raised by pavoni above though - in this instance with the issue i'm seeing, vera is correctly updating state immediately but there's a delay in HASS carrying that state update through.


syscall: exec_windows.go: arguments should not be escaped to work with msiexec

Problem: When I try to run a command in Go on Windows using exec.Command() and then exec.Run(), the arguments are escaped using this logic: https://github.com/golang/go/blob/master/src/syscall/exec_windows.go#L26. That logic escapes the quotes around TARGETDIR="%v" which need to be there. I currently am assigning to Cmd.SysProcAttr manually to get around the escaping.

What version of Go are you using (go version)?
Go 1.6.2
What operating system and processor architecture are you using (go env)?
Windows 10 Pro x64
What did you do?
The code is available here, but needs to be run on Windows: https://play.golang.org/p/aU1PlbNTqM
Download any MSI. I used https://fastdl.mongodb.org/win32/mongodb-win32-x86_64-2008plus-ssl-3.2.6-signed.msi and renamed it to package.msi.
You can run Error() and NoError() to see the difference
What did you expect to see?
I expect to see the MSI say: Completed the ... Setup Wizard.
What did you see instead?
I see the Windows Installer help Windows and when I click it, I get: exit status 1639 which is:
ERROR_INVALID_COMMAND_LINE 1639 Invalid command line argument. Consult the Windows Installer SDK for detailed command line help.

args := fmt.Sprintf(/a "%v" TARGETDIR="%v", msiFile, targetDir)
cmd := exec.Command("msiexec", args)

How many arguments do you want msiexec to get? Because you are passing one argument - args (args is a single string). If you want to pass many arguments, than you should pass them one by one:
cmd := exec.Command("msiexec", "arg1", "arg2", "arg3")

and so on. I think waht you want is:
cmd := exec.Command("msiexec", "/a", "package.msi", `TARGETDIR=C:\test folder`)

Not tested. I don't even know how msiexec works.
Alex

You are correct, I should have used that in the example. That is the way I originally tested and same error.

Actually, let me clarify because it is a different error, but same reasoning.
When I run:
cmd := exec.Command("msiexec", "/a", "package.msi", `TARGETDIR="C:\testfolder"`)

I get this error: "Could not access network location "C:\testfolder". And then: exit status 1603
I need quotes around the TARGETDIR value because there could be a space in the path.

I get this error: "Could not access network location "C:\testfolder". And then: exit status 1603

msiexec is, probably, confused about having " character in the file name. You confused it.

I need quotes around the TARGETDIR value because there could be a space in the path.

The exec.Command passes arguments to the program as is. It will deal with spaces, quotes, double quotes and everything else. That is what syscall.EscapeArg does. You job is to provide all parameters exactly as your child process expects them. Leave all the quoting to do for syscall.EscapeArg.

This is the command that ultimately I'm trying to run:
msiexec /a "package.msi" TARGETDIR="C:\test folder"

Like I suggested above, you want:
cmd := exec.Command("msiexec", "/a", "package.msi", `TARGETDIR=C:\test folder`)

syscall.EscapeArg will insert whatever escape characters to make sure that space in TARGETDIR=C:\test folder does make it across to msexec.
Alex

When run msiexec on cmd.exe, some of double-quotes will be removed.
dummy.c
#include <windows.h>
#include <stdio.h>

int
main(int argc, wchar_t* argv[]) {
  int i, narg = 0;
  LPWSTR* args = CommandLineToArgvW(GetCommandLineW(), &narg);
  for (i = 0; i < narg; i++) printf("%ws\n", args[i]);
  LocalFree(args);
  return 0;
}

C:\dev>dummy /a "package.msi" TARGETDIR="C:\testfolder"
dummy
/a
package.msi
TARGETDIR=C:\testfolder

But when run dummy.exe, those are not removed.
package main

import (
	"os"
	"os/exec"
)

func main() {
	cmd := exec.Command(`dummy`, "/a", "package.msi", `TARGETDIR="C:\testfolder"`)
	cmd.Stdout = os.Stdout
	cmd.Stderr = os.Stderr
	cmd.Run()
}

C:\dev>go run d.go
dummy
/a
package.msi
TARGETDIR="C:\testfolder"

When I run this command, it works properly:
cmd := exec.Command("msiexec", "/a", "package.msi", `TARGETDIR=C:\testfolder`)

When I run this command, I get the MSIEXEC WIndows Installer Help and then exit status 1639:
cmd := exec.Command("msiexec", "/a", "package.msi", `TARGETDIR=C:\test folder`)

When I run this command, I get the MSIEXEC WIndows Installer Help and then exit status 1639:

I actually tried running your program, and I agree that exec.Command escaping rules don't work with msiexec. It is second program (that I know of) that behaves this way - first one is cmd.exe (https://blogs.msdn.microsoft.com/twistylittlepassagesallalike/2011/04/23/everyone-quotes-command-line-arguments-the-wrong-way/).
The Go exec.Command escaping will work if child program reads its arguments with CommandLineToArgv. And most programs doo. But cmd.exe and msiexec.exe do not. So you have to use Cmd.SysProcAttr.CmdLine to build command line by hand. And you do that already. I don't see what else you could do.
Alex

But when run dummy.exe from go, those are not removed.

This is to be expected. Arguments are passed as is - quotes and all. If you don't want quotes, don't pass them inside parameters.
Alex

@alexbrainman This is my expected. Just explaining go's behavior. :)

Ah very interesting! I didn't know cmd and msiexec behaved differently. It looks like other languages have this problem as well. Luckily, we can specify the arguments in a raw format directly using cmd.SysProcAttr.CmdLine. Do you think a comment added to exec.Command would be sufficient enough?

Do you think a comment added to exec.Command would be sufficient enough?

Do you propose to say that it is a mess? :-)
Also #15588 might be of interest to you.
Alex

Haha yeah, that should do it! Or something like: The Cmd's Args are escaped and may not work properly with some programs. You can skip escaping them by manually assigning the Args to Cmd.SysProcAttr.CmdLine after prefixing with a space.
And I checked out 15588 - CommandLineToArgv() looks like a good way to test commands to ensure they will be split correctly when read by a C/C++ programs - thanks.
And for more reference, I ran tests with different syntax in command prompt and the results are below. It's a shame you can't put quotes around the entire property like this because this wouldn't be an issue, but it's a msiexec problem, no a Go problem: "TARGETDIR=C:\testfolder"
# MSI Success
msiexec /a package.msi TARGETDIR=C:\testfolder

# MSI Success
msiexec /a package.msi TARGETDIR="C:\test folder"

# MSI Error
msiexec /a package.msi "TARGETDIR=C:\test folder"

# MSI Error
msiexec /a package.msi TARGETDIR=\"C:\test folder\"

# MSI Error
msiexec /a package.msi TARGETDIR=""C:\test folder""

# MSI Error
msiexec /a package.msi TARGETDIR=^\"C:\test folder^\"

# MSI Error
msiexec /a package.msi TARGETDIR='C:\test folder'

The Cmd's Args are escaped and may not work properly with some programs. You can skip escaping them by manually assigning the Args to Cmd.SysProcAttr.CmdLine after prefixing with a space.

I am bit concerned about adding a lot of confusing mambo jambo because of two programs (cmd.exe and msiexec.exe). For most users we don't even need to say word "escape", and everything works perfectly.
Maybe we should specifically mention cmd.exe and msexec.exe as being different. And say that for some argument strings Cmd.Args will not work, and they should build command line by hand and pass it via  Cmd.SysProcAttr.CmdLine. What do you think?
Another alternative would be to create new function similar to syscall.EscapeArg, but specifically for cmd.exe. And ignore msexec.exe altogether.
Alex

I agree on preventing it from becoming confusing. Can we do both options? Since manually setting Cmd.SysProcAttr.CmdLine is an option, adding a comment about it would be helpful.
And for a more official way, I like the idea of adding an alternative syscall.EscapeArg for cmd. We could then run (hopefully) msiexec and other applications by passing them to cmd like this:
cmd /k "msiexec /a C:\package.msi TARGETDIR=^"C:\test folder^""

We could then run (hopefully) msiexec and other applications by passing them to cmd like this:
cmd /k "msiexec /a C:\package.msi TARGETDIR=^"C:\test folder^""

I don't see how having special function that handles cmd.exe arguments will help with msiexec.exe. The idea is that this function allows you not to worry about quotes and spaces and similar. If you are prepared to take care of quotes and space and others in your code, then Cmd.SysProcAttr.CmdLine is all you need.

I am going to take care of quotes and spaces in my code, I'm just trying to think about what makes the most sense for other users that will experience the same problem. A comment seems like it makes the most sense right now since I don't have any good use cases for a separate function for cmd.exe.

A comment seems like it makes the most sense right now since I don't have any good use cases for a separate function for cmd.exe.

Sounds good to me. Feel free to send the change.
Alex

I just sent CL 30947, which does the right quoting for cmd.exe at least. There is a test that now passes and did not before. The actual rules seem easy enough that we should provide them.
The hard question is how to know when to turn on the additional quoting. The CL looks for cmd or cmd.exe or %COMSPEC%, which is a bit of a hack. I would like to have something in the SysProcAttr that lets you opt in or out explicitly, but I would also like it if we could do the right thing by default in more cases than we do now. Is that possible?
Detecting explicit cmd is easy, if a bit hacky, but then in #17149 we have an ordinary batch file being executed and that has the same issue. We could also apply the rules for programs ending in ".bat". Is that enough? Do all batch files still end in ".bat"?

Nah, they also use .cmd.

I guess the bigger question is: is there a reliable way to detect batch files? Must they end in ".bat" or ".cmd"? And if something does end in ".bat" or ".cmd" is it guaranteed to be a batch file, or at least to be reinvoked using cmd /c ?

I'm not sure trying to detect this is the correct way. I suggest to add (somewhere) a CmdCommand() function that does the escaping, and then specify in the documentation that cmd requires a non-standard escaping and thus point users to CmdCommand()

I guess the bigger question is: is there a reliable way to detect batch files? Must they end in ".bat" or ".cmd"? And if something does end in ".bat" or ".cmd" is it guaranteed to be a batch file

I tried to see what cmd.exe does. I opened command prompt and renamed benchcmp.exe into benchcmp.bat, and run it:
C:\tmp>.\benchcmp.exe
usage: .\benchcmp.exe old.txt new.txt

  -best
        compare best times from old and new
  -changed
        show only benchmarks that have changed
  -mag
        sort benchmarks by magnitude of change

Each input file should be from:
        go test -run=NONE -bench=. > [old,new].txt

Benchcmp compares old and new for each benchmark.

If -test.benchmem=true is added to the "go test" command
benchcmp will also compare memory allocations.

C:\tmp>rename benchcmp.exe benchcmp.bat

C:\tmp>.\benchcmp.bat

C:\tmp>MZÉ$
'MZÉ$' is not recognized as an internal or external command,
operable program or batch file.

C:\tmp>L$↑ëD$♀â°☻}ïT$¶ì∟┬ë∟$ëL$♦ΦΘ♣█☼à8☺╓Γ╝┴┬☼i╥ßà☼╟ï☺1┬i╥!╓Γ╝┴┬☼i╥ßà☼╟Θx   ë▌ìL
)ⁿï‼1┬i╥!╓Γ╝┴┬☼i╥ßà☼╟ï☺1┬i╥!╓Γ╝┴┬☼i╥ßà☼╟ΘG   ï‼1┬i╥!╓Γ╝┴┬☼i╥ßà☼╟Θ/   ë▌ë╬╤Θ☺Θìl.
 ☼╢‼1┬☼╢☺┴1┬☼╢E¼$Çë¶$ΦÉ$╞D$♦ Φ2U♥8   â─(├Φ+5♥Φ╙"☺
'L$↑ëD$' is not recognized as an internal or external command,
operable program or batch file.

C:\tmpQñ┼ïI↑ïæ╘☺ëiL☼╢i=☼╢s=     ⌡òêA=òïk►)ΦëD$►ï-8 Q♂Φr 1>wu  | â└DëD$¶ëL$►â∙☺t8
à╔u_☼╢T$$ä╥tJë♦$ëL$♦☼╢┌ë\Φâ.♦
Qñ┼ïI↑ïæ╘☺ëiL☼╢i' is not recognized as an internal or external command,
operable program or batch file.

C:\tmp>

So it looks like cmd.exe checks for .bat extension.

at least to be reinvoked using cmd /c

I do not think you need to prefix every batch file with cmd /c. I am pretty sure CreateProcess will do the right thing. In fact I tried running both benchcmp.exe and benchcmp.bat with exec.Command (which uses CreateProcess) and it executed properly (as exe) both times.
We can also do something similar to what debug/pe does to determine if executable file is executable.
Alex

Windows only uses file extensions to determine which application should, by default, handle the file. If you have a batch file that you rename to a .txt file. It should always open with Notepad.exe unless you changed the default association in the Default Programs app. This is different from Linux and OS X (where the file extension doesn't matter). I think we are safe to assume .cmd and .bat are always and the only batch files.
As for should we detect, we probably shouldn't because we probably can't come up with a list of all the exceptions. Leave that up to the user to use the correct function. I agree with @rasky that there should be a separate CmdCommand() function and documentation to point to it.

Sorry I don't use/run the msi file above, so I don't make sure but how about C:\test^ folder?

Maybe the same issue as this: http://stackoverflow.com/questions/42327117/iis-appcmd-called-via-go-app-invalid-xml-input

Maybe the same issue

Probably is the same issue. Perhaps %windir%\system32\inetsrv\appcmd uses different command line parameters escaping rules.
Alex

I think that in the ideal world we would quote arguments in the safest way by default, and would provide fields in syscall.SysProcAttr to request less safe quoting mechanisms.  That would protect the naïve user and permit the expert to get what they need.
Is that possible without breaking backward compatibility?

Is that possible without breaking backward compatibility?

We already doing that as far as I am concerned. Executing most programs work. Except for cmd.exe and msiexec.exe (and maybe few others that we do not know about). For exceptions users can use syscall.SysProcAttr.CmdLine to format command line as they wish.
It is tempting to add code similar to CL 30947 to handle cmd.exe, but then we should also handle batch files. For simple cmd.exe invocations (where no escaping required) exec.Command already works, so changing the syscall package code will not change that. But if exec.Command("cmd.exe", ...) needs escaping, then it would be broken now - so fixing this code seems like a win / win.
Alex

For simple cmd.exe invocations (where no escaping required)

Considering that probably half of the billions of child processes spawned each day on Windows have something to do with cmd and spaces in paths are nothing unusual since at least the year 2000, I find this curiously optimistic.
And odd that the member CmdLine isn't even listed here: https://golang.org/pkg/syscall/#SysProcAttr
Also no example for how to use it to be found aside from e.g. here: https://medium.com/@felixge/killing-a-child-process-and-all-of-its-children-in-go-54079af94773
I probably won't use Go any more myself but still wanted to note that this is obviously a very common use case (simply check the stars on nvm-windows if you don't know what nvm is), so it would be for your own benefit to document this instead of dismiss it.
But that's just my two cents... Go has survived without this for 8 years already after all. :)

In #27199, @sebres proposes what looks like a clever algorithm for quoting arguments in such a way that they can survive being passed through cmd.exe, while also working fine in the normal case where cmd.exe isn't involved. If it works, this would obviate the need try and sniff out .bat files.

This breaks calling Unity installer from go too, as it quotes the arguments breaking the installer.


Set multiple values to edit dataset if URL is used

Hello,
i am using the flexdatalist with the URL to load my data. Now if I want to edit a dataset I have to load the existing values to the multiple flexdatalist to show previous selected values. I use the data-value-property and data-visible-properties. So I have to set the value and the visible value. Using the "add" method seems to allow only selecting from an existing datalist. As I load the datalist always only by enetering at least 4 characters no datalist is available. Was wondering if I can initialize with "data" option and set the previous data that they can be selected with "add" method. But there is no description how the to format or if it is possible
Using:
<input type="text"                                    name="invoices"                                    id="inputInvoice"                                    placeholder=""                                    class="flexdatalist form-control"                                    data-url='{{ route('invoice.json') }}'                                    data-search-in="invoice_id"                                    data-visible-properties='["invoice_id"]'                                    data-selection-required="true"                                    data-value-property="id"                                    data-min-length="4"                                    multiple="multiple">
Try to add old values something like:
 $('#inputInvoice').flexdatalist( {data: [                                                 {id : 21, invoice_id : "22842"}                                             ]});                                     $('#inputInvoice').flexdatalist('add', 21)

did you mean user can add some value if its not in databases?, if that your mean you can try this code
$('#inputInvoice').flexdatalist( {noResultsText: '<button> Trigger Some Button </button>'});

did you mean user can add some value if its not in database and store that to database?, if that your mean you can try this code :
$('#inputInvoice').flexdatalist( {noResultsText: '<button> Trigger Some Button </button>'});
if your mean user can add some value in multiple dataset just use comma or your valuesSeparator to add value
if your mean user can add some value that used before you can try this code :
$('#inputInvoice').flexdatalist( {allowDuplicateValues: true} );


[PATCH] Fix tests broken by latest PyTest version

PyTest 4.2.0 introduced a regression which broke the tests in our test plans, but wasn't a deal-breaker, so we simply adapted to the breakage. The regression was fixed in 4.2.1, but the fix introduced a new feature (not a bug) that @unittest.skip-skipped classes won't even be handled by our test plan code, so we can't measure them in the plugin-testing tests. This commit adapts to that change and removes the adaptation to the fixed regression. At also adds some new assertions and information to help ensure this is still working, even with the new changes.

looks good to me


tweepy can't be built for python35

Issue description
On the last stage when calling test I get the next errors:
[10:37] igor@nixos-pc tweepy-3.5.0 [nix-shell] % python3 setup.py test
running test
running egg_info
writing requirements to tweepy.egg-info/requires.txt
writing tweepy.egg-info/PKG-INFO
writing dependency_links to tweepy.egg-info/dependency_links.txt
writing top-level names to tweepy.egg-info/top_level.txt
reading manifest file 'tweepy.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'tweepy.egg-info/SOURCES.txt'
running build_ext
Traceback (most recent call last):
  File "setup.py", line 43, in <module>
    zip_safe=True)
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/nix/store/ah75wrpyp7fvs3jqxlfrbrnydf75cww4-python3.5-bootstrapped-pip-8.1.2/lib/python3.5/site-packages/setuptools/command/test.py", line 172, in run
    self.run_tests()
  File "/nix/store/ah75wrpyp7fvs3jqxlfrbrnydf75cww4-python3.5-bootstrapped-pip-8.1.2/lib/python3.5/site-packages/setuptools/command/test.py", line 193, in run_tests
    testRunner=self._resolve_as_ep(self.test_runner),
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/main.py", line 93, in __init__
    self.parseArgs(argv)
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/main.py", line 123, in parseArgs
    self._do_discovery([])
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/main.py", line 228, in _do_discovery
    self.test = loader.discover(self.start, self.pattern, self.top)
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/loader.py", line 341, in discover
    tests = list(self._find_tests(start_dir, pattern))
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/loader.py", line 398, in _find_tests
    full_path, pattern, namespace)
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/loader.py", line 475, in _find_test_path
    tests = self.loadTestsFromModule(package, pattern=pattern)
  File "/nix/store/ah75wrpyp7fvs3jqxlfrbrnydf75cww4-python3.5-bootstrapped-pip-8.1.2/lib/python3.5/site-packages/setuptools/command/test.py", line 40, in loadTestsFromModule
    tests.append(self.loadTestsFromName(submodule))
  File "/nix/store/1c2lsmy2g76kbs7r3y4m46qy9yd9s9xs-python3-3.5.2/lib/python3.5/unittest/loader.py", line 153, in loadTestsFromName
    module = __import__(module_name)
  File "/tmp/tt/tweepy-3.5.0/examples/oauth.py", line 29, in <module>
    print(api.me().name)
  File "/tmp/tt/tweepy-3.5.0/tweepy/api.py", line 338, in me
    return self.get_user(screen_name=self.auth.get_username())
  File "/tmp/tt/tweepy-3.5.0/tweepy/auth.py", line 134, in get_username
    user = api.verify_credentials()
  File "/tmp/tt/tweepy-3.5.0/tweepy/api.py", line 631, in verify_credentials
    )(**kargs)
  File "/tmp/tt/tweepy-3.5.0/tweepy/binder.py", line 245, in _call
    return method.execute()
  File "/tmp/tt/tweepy-3.5.0/tweepy/binder.py", line 229, in execute
    raise TweepError(error_msg, resp, api_code=api_error_code)
tweepy.error.TweepError: [{'message': 'Bad Authentication data.', 'code': 215}]

Steps to reproduce
nix-build '<nixpkgs>' -A python35Packages.tweepy
There is no problem for the python27. I noticed that for python27 tests are not performed:
[10:36] igor@nixos-pc tweepy-3.5.0 [nix-shell] % python setup.py test
running test
running egg_info
writing requirements to tweepy.egg-info/requires.txt
writing tweepy.egg-info/PKG-INFO
writing top-level names to tweepy.egg-info/top_level.txt
writing dependency_links to tweepy.egg-info/dependency_links.txt
reading manifest file 'tweepy.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'tweepy.egg-info/SOURCES.txt'
running build_ext
----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK

Technical details

System: NixOS: 17.03pre94121.210b3b3 (Gorilla) and on master
Nix version: nix-env (Nix) 1.11.4
Nixpkgs version: 17.03pre94121.210b3b3

there are no tests in the archive, so it needs doCheck = false;.

cc maintainer @garbas

:+1: disabled the tests in commit mentioned above


generate code request_test template error

When executing cargo loco generate scaffold post title:string content:text of https://loco.rs/docs/getting-started/tour/#adding-a-crud-api, the generated code is in tests/requests/post.rs now code is use blo::app::App This should be use {{pkg_name}}::app::App;

Can I submit a PR to fix it?

Yes, we'd love your help, thanks!


Fix issue on Bitcoin ListAddressGroupings method

Fix issue on Bitcoin ListAddressGroupings method where balance is not parsed from the response.

Time to make a NuGet release with recent updates?

Very good point. I plan to make a release soon, during the next few days.

BitcoinLib 1.7 is now live on NuGet Gallery: https://www.nuget.org/packages/BitcoinLib/ 🚀

It’s awesome to see this new release! Thank you.

From: Nikos Baxevanis<EMAIL_ADDRESS>Sent: Monday, April 2, 2018 3:42:45 PM
To: GeorgeKimionis/BitcoinLib
Cc: Woooof; Author
Subject: Re: [GeorgeKimionis/BitcoinLib] Fix issue on Bitcoin ListAddressGroupings method (#67)
BitcoinLib 1.7 is now live on NuGet Gallery: https://www.nuget.org/packages/BitcoinLib/ 🚀
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/GeorgeKimionis/BitcoinLib/pull/67#issuecomment-377876703, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA67jfmjeTddnLmey3MdvzPC5hQQ_NC8ks5tkdZ1gaJpZM4R4ObN.


Improved usability if 'racer' command fails

Now it is much easier to debug racer and figure out what's wrong.
Considering the alpha state of rust in general, better debugging capabilities will pay for itself quickly.
Additionally,

removed debug printing
silenced linter, as 'view' variable was redefined within a loop. It worked, but the linter had a point there

Thank you and sorry for such a late reply!


Delete manifests/j/JanDeDobbeleer/OhMyPosh/7.56.1 directory

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run


Add queryResource to stripes config. Part of STCOR-231

yarn run v1.6.0
$ eslint *.js settings
/home/jenkins/workspace/folio-org/ui-organization/PR-93/project/settings/Plugins.js
4:1  error  'stripes-config' should be listed in the project's dependencies. Run 'npm i -S stripes-config' to add it  import/no-extraneous-dependencies
✖ 1 problem (1 error, 0 warnings)
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


Release Google.Maps.AddressValidation.V1 version 1.3.0

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 7707366)

Created release for Google.Maps.AddressValidation.V1-1.3.0


[Actions] Auto-Sync cookiecutter template

Pull Request Test Coverage Report for Build 478557088

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.074%

Totals

Change from base Build 459712169:
0.0%

Covered Lines:
337

Relevant Lines:
351

💛  - Coveralls

Pull Request Test Coverage Report for Build 478557088

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.074%

Totals

Change from base Build 459712169:
0.0%

Covered Lines:
337

Relevant Lines:
351

💛  - Coveralls


Use correct media type

Fixes #55

@mikeminutillo Thanks! I dont use servicepulse so I wouldnt have noticed this.  I think the "json" was from an older version of NSB.Json which I was basing this section off of