fix: migrate prettier auto endofline to LF as it's the default

Summary
I tried to migrate from prettier, and the migration failed because of unsupported option
#2138
Test Plan
you can migrate a prettierrc like this
{
	"printWidth": 120,
	"useTabs": true,
	"tabWidth": 2,
	"singleQuote": true,
	"semi": false,
	"trailingComma": "all",
	"bracketSameLine": false,
	"arrowParens": "avoid",
	"endOfLine": "auto"
}

I KNOW that auto won't be supported, but i think it's better for the UX to continue the flow with the best option possible instead of stopping in error
feel free to close if you disagree

I think it’s a sensible approach. Just wondering if we should log a warning. What do you think, @ematipico ?

Yeah I think we should log a warning at least.

hum... as a nooob in rust i'm not really sure how should i do that...
not print like an animal right? I saw the console thingy but unsure how to trickel it down the transmformation 🤔
if someone could give a hint i can do that (or someone push it and i 🤓 )

@eMerzh I'll give you some pointers soon :)

Here
https://github.com/biomejs/biome/blob/644d2f429ebf3ec2fa8b424478fc0540197b1c2a/crates/biome_cli/src/execute/migrate/prettier.rs#L344-L346
You'll have to read the value of prettier_configuration.end_of_line and check it is equals to the new value you added. If so, than you should add something like this:
console.log(markup! {
	<Warn>"Warning message here"</Warn>
});

I leave the contents of the message to you

oh ok, as simple... :)
thanks for the help 🙌 ... i pushed an attempt then :)

LGTM. Thank you! Do you mind adding a changelog entry? You can check the instructions in CONTRIBUTING.md.

You'll also have to run just gen-web afterwards so the changelog file on website can be updated as well. :)

aaarg ...omg sorry sorry 😅

aaarg ...omg sorry sorry 😅

No worries! I tweaked a few places and added a test. Hope you don't mind.


Add errno constants for Linux: AArch64, MIPS64, PPC64, SystemZ

Look good.  Though I have today keep reminding myself what systemz is.

Auto-merge toggled on

Would "Big Iron" be a better name? ;-)

Would "Big Iron" be a better name? ;-)

Oh, so I can use it to press shirts and trousers?  :-P


Key management overview step trimming

Rearranged the "Key management" overview step copy to fit a little nicer and remove some redundancy. This prevents the "Continue" button from shifting downward too much during this flow.
Also adds an instance of intl to acknowledgement index page

Approved as I understand the titles in pageContent are covered by <FormattedMessage> in an other PR.


Add copy code button to code samples

Description

This updates the code samples to also have a "copy sample code" button to make it easier to copy sample code.
To make development of this and future features easier, the code samples have been moved into their own control.
Motivation and Context

How Has This Been Tested?

Tested manually.
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Sorry about the large PR @gregwoo-microsoft but I think that this step would make future changes a lot easier. Happy to hear your thoughts about the copy code button and it's UX!

Thanks a lot @chingucoding! I ran the XCG and the changes look great! I want to spend more time to review/better understand your code changes before I approve this PR. Amazing work! This is awesome :)

This looks great Marcel! Thanks a lot for this! I changed the copied text to "Copied!" (one problem though is that I can't get it to be at the center of the control tip). I think that this phrasing is more concise than "Successfully copied sample code!" and is also more subtle/discrete. Let me know what you think :)


Unable to set different line colors based on marker values and color in chart js line graph

Expected behavior
The line chart should render with the line color changing based on the marker colors.
This is an example screenshot of what the output should be.
https://pasteboard.co/6PKRCJzsaKa6.png
Currently the code that I have written using chart JS (v 4.2.1) is not rendering in the browser.
I am trying to do this using vanilla javascript. I am welcome to any suggestions. If there is an alternate method, kindly suggest it to me also.
Current behavior
The code is not rendering in the frontend and the there is error in console window. (screenshot attached below.)
https://pasteboard.co/Rma2XuY1jAz7.png
Reproducible sample
https://codepen.io/Thomas-1/pen/oNPVqQp
Optional extra steps/info to reproduce
No response
Possible solution
No response
Context
This is an example screenshot of what I am trying to achieve.
https://pasteboard.co/HgPrWMUziKIS.png
chart.js version
v4.2.1
Browser name and version
Microsoft Edge Version 111.0.1661.44 (Official build) (64-bit)
Link to your project
No response

@ttvettuparambil I think you could the segment option (instead of custom plugin) in line chart: https://www.chartjs.org/docs/latest/charts/line.html#segment

@ttvettuparambil another possible solution is to use chartjs-plugin-gradient.

@ttvettuparambil I think you could the segment option (instead of custom plugin) in line chart: https://www.chartjs.org/docs/latest/charts/line.html#segment

Hello stockiNail, thank you for the reply. I have managed to use segments to bring colors in different segments but still have not gotten the required result.
currently I have modified my code as can be seen in the link below.
https://pastebin.com/JdFtjE4s
I am trying to use ctx object to access p0 and p1 points and give a gradient.addColorStop to p0 and p1 points so that theoretically the gradient should appear in each segment depending on the color of the points, but unfortunately let gradient = ctx.createLinearGradient(ctx.p0.x, ctx.p0.y, ctx.p1.x, ctx.p1.y) in test() is having an error in console although I can access p1 and p1 objects just fine.
Could you please take a look into the above pastebin url and see if I am missing anything. The chart designed using my code currently looks like this (screenshot attached below).
https://pasteboard.co/QoxILiee5MVP.png

const test = (ctx) => {
// let gradient = ctx.createLinearGradient(ctx.p0.x, ctx.p0.y, ctx.p1.x, ctx.p1.y);
gradient.addColorStop(0, ctx.p0.options.backgroundColor);
gradient.addColorStop(1, ctx.p1.options.backgroundColor);
console.log(ctx p0 color::::>,ctx.p0.options.backgroundColor);
console.log(ctx p1 color::::>,ctx.p1.options.backgroundColor);
return gradient;
}
When I uncomment the first line in the function, I am getting "ctx.createLinearGradient is not a function" error in browser console. Could somebody tell me why it is like that.

const test = (ctx) => {
// let gradient = ctx.createLinearGradient(ctx.p0.x, ctx.p0.y, ctx.p1.x, ctx.p1.y);
gradient.addColorStop(0, ctx.p0.options.backgroundColor);
gradient.addColorStop(1, ctx.p1.options.backgroundColor);
console.log(ctx p0 color::::>,ctx.p0.options.backgroundColor);
console.log(ctx p1 color::::>,ctx.p1.options.backgroundColor);
return gradient;
}
When I uncomment the first line in the function, I am getting "ctx.createLinearGradient is not a function" error in browser console. Could somebody tell me why it is like that.

On logging the ctx object in console window for both segment{ borderColor } as well as datasets { borderColor } I have found that addcolorstop property is not available in segment{ borderColor }. So I guess segment method to add gradient for line is out of the question. Can you show me as to how to implement chartjs-plugin-gradient in vanilla js.
Also related question, how to make segment line color same as that of p1 point using segments ?

I have found the solution (kind of.) Since gradient is not possible in ctx (context) object in borderColor in segments object, I decided to apply a solid color for each segment in the graph with the color changing based on the ctx.p1.options.backgroundColor value. This is the code that I have used to acheive this.
It now looks something like this.
https://pasteboard.co/EbJAm3uL4xji.png
function drawCustomScoreChart(panel) {
vals = setDataForReadinessScore()
var data = vals["y"];
console.log(data is:::::::>,data);
var colors = getReadinessScoreColors()
console.log(colors::::::::::>,colors);
const test = (ctx) => {
console.log(ctx object::::::>,ctx);
console.log(ctx p0 color::::>,ctx.p0.options.backgroundColor);
console.log(ctx p1 color::::>,ctx.p1.options.backgroundColor);
return ctx.p1.options.backgroundColor;
// return gradient;
}
  var myChart = new Chart(panel, {
    type: 'line',
    data: {
      labels: vals["x"],
      datasets: [{
        label: 'Sales',
        data: data,
        tension:0.4,
        borderColor:'#000',
        pointBackgroundColor:colors,
        borderWidth: 3,
        segment: {

            borderColor: ctx => test(ctx),

        }
      }],

    },
    options: {
        maintainAspectRatio: false,
    //   aspectRatio: 3/1,
      responsive: true,
      showScale: false,
      scales: {
        y: {
            beginAtZero: true
        },
        x: {
            beginAtZero:true
        },
        yAxes: [{
          ticks: {
            beginAtZero: true
          }
        }],
      },
    //   extra code
    plugins: {
        legend: {
          display: false
        }
      },
    },

  });

}
I would like it if gradient option would be applied by default to chart.js library.

On logging the ctx object in console window for both segment{ borderColor } as well as datasets { borderColor } I have found that addcolorstop property is not available in segment{ borderColor }. So I guess segment method to add gradient for line is out of the question. Can you show me as to how to implement chartjs-plugin-gradient in vanilla js. Also related question, how to make segment line color same as that of p1 point using segments ?

@ttvettuparambil apologize for late reply but I was busy on other stuff.
Well, the ctx that you are getting as argument is not the context of the canvas but the context of the scriptable option.
See the doc: https://www.chartjs.org/docs/latest/charts/line.html#segment
To have the canvas context, you should access to the chart instance, as following:
const canvasContext = ctx.chart.ctx;
canvasContext.createLineagGradient...

@ttvettuparambil another possible solution is to use chartjs-plugin-gradient.

@stockiNail @can you give me a working example of this plugin along with a quick summary on the plugin options ?

Yes, I can but let me take time a bit. But having a look to your code, I see (but maybe I'm wrong) that the gradient must be applied to the single segment  and based on the values of p0 and p1, therefore the plugin is not a good solution for this use case because the plugin is applied to the whole line based on values of dataset.

const test = (ctx) => {
// let gradient = ctx.createLinearGradient(ctx.p0.x, ctx.p0.y, ctx.p1.x, ctx.p1.y);
gradient.addColorStop(0, ctx.p0.options.backgroundColor);
gradient.addColorStop(1, ctx.p1.options.backgroundColor);
console.log(ctx p0 color::::>,ctx.p0.options.backgroundColor);
console.log(ctx p1 color::::>,ctx.p1.options.backgroundColor);
return gradient;
}

Hello @stockiNail, Thank you for your response, Seems however the snippet is not working properly. I called the function in segment: { borderColor } and put the snippet but now it is rendering only green color gradient. This is the screenshot of the output now.
https://pasteboard.co/4VDXH8dIiBag.png
If you look at the screenshot, the first 2 points have orange color. The third one is having dark green color and the fourth point having green color, but the entire gradient line is having green color.
segment: {
borderColor: ctx => test(ctx),
}
const test = (context) => {
const {chart, p0, p1} = context;
const ctx = chart.ctx;
let gradient = ctx.createLinearGradient(p0.x, p0.y, p1.x, p1.y);
gradient.addColorStop(0, p0.options.backgroundColor);
gradient.addColorStop(1, p1.options.backgroundColor);
return gradient;
}

@ttvettuparambil another possible solution is to use chartjs-plugin-gradient.

@stockiNail @can you give me a working example of this plugin along with a quick summary on the plugin options ?

Yes, I can but let me take time a bit. But having a look to your code, I see (but maybe I'm wrong) that the gradient must be applied to the single segment and based on the values of p0 and p1, therefore the plugin is not a good solution for this use case because the plugin is applied to the whole line based on values of dataset.

Ok thank you for clarifying this point for me.

On logging the const ctx = chart.ctx; I dont see any p0 and p1 objects in the output. So I am not sure how the green gradient color is even rendering.

I dont see any p0 and p1 objects in the output.

yes, because p0 and p1 are int not chart.ctx. Anyway I have created a codepen and I have got the same weird behavior that only the last gradient is applied.

@ttvettuparambil no at the moment, may be anything will come later to my mind. Anyway there is something weird that I cannot understand.

ok. Thanks a lot for the clarifications. Hope you have a nice day. Do let me know if this feature is implemented in chart.js in the future.

Nice day to you too.  I need to know why it's not working!

@ttvettuparambil  have a look to the codepen: https://codepen.io/stockinail/pen/rNZgxae
It's working as you expected.
But I think there is a bug in CHART.JS when a segment callback is returning a gradient (but also a canvas pattern).
https://github.com/chartjs/Chart.js/blob/82aca7852f56457bf6482348f4b189bfd21dbeda/src/helpers/helpers.segment.js#L348-L350
The changed style is applied if the JSON stringify value of prev and new values are different. But the stringify of a gradient is always and empty object string representation.
As workaround, as you can see in codepen, I set a random value in order it's different for all segments and then the JSON stringify is also different then the gradient is always applied.
Play on it if you can and let me know. In the meanwhile I have to find time to fix it.
Let me know

@stockiNail you are a legend. Put the function in my code and working like a charm. Can't tell you how much I appreciate this. Do let me know when the code bug is fixed. Was racking my brain for past few days to find a solution for this issue.

Could we leave this issue open for 1 more day. Just in case I have any doubts.

@ttvettuparambil remember that when the PR will be released you should remove the random value from gradient instance to be clean

@ttvettuparambil PR submitted:  #11217

I should only remove the random gradient value; the rest of the code stays the same right?

YES! The rest will remain the same. To be more precise (see comment):
const test = (context) => {
  const {chart, p0, p1} = context;
  const ctx = chart.ctx;
  const {x: x0} = p0.getProps(['x'], true);
  const {x: x1} = p1.getProps(['x'], true);
  const gradient = ctx.createLinearGradient(x0, 0, x1, 0);
  gradient.addColorStop(0, p0.options.backgroundColor);
  gradient.addColorStop(1, p1.options.backgroundColor);
  gradient.random = Math.random(); /// <--- This row could be removed after PR will be released
  return gradient;
}

Is it also clear to you why the code is accessing to the p0/p1 X properties by getProps instead of using directly the properties?
Because I didn't comment and maybe it's not clear.

I should only remove the random gradient value; the rest of the code stays the same right?

YES! The rest will remain the same. To be more precise (see comment):
const test = (context) => {
  const {chart, p0, p1} = context;
  const ctx = chart.ctx;
  const {x: x0} = p0.getProps(['x'], true);
  const {x: x1} = p1.getProps(['x'], true);
  const gradient = ctx.createLinearGradient(x0, 0, x1, 0);
  gradient.addColorStop(0, p0.options.backgroundColor);
  gradient.addColorStop(1, p1.options.backgroundColor);
  gradient.random = Math.random(); /// <--- This row could be removed after PR will be released
  return gradient;
}

Is it also clear to you why the code is accessing to the p0/p1 X properties by getProps instead of using directly the properties? Because I didn't comment and maybe it's not clear.

@stockiNail , I confess I do not know how the above code snippet works. Could you please explain it to me?

const test = (context) => {
  const {chart, p0, p1} = context;
  const ctx = chart.ctx;

context is the CHART.JS context for segment callback (NO the Context2d of a canvas) and is described here.
The Context2d instance (ctx) is retrievable from the chart instance.
  const {x: x0} = p0.getProps(['x'], true);
  const {x: x1} = p1.getProps(['x'], true);

p0 and p1 are data point element.
As you can see, the code is not accessing to properties of the element directly (i.e. p0.x). If you will do, you will see that at the first chart loading, the segments will not be drawn, if you enabled the animation.
The properties of the point elements are affected by animation and  i.e. p0.x is providing the current position of the point in the animation cycle. Furthermore, the segment callbacks are invoked at the beginning of dataset drawing.
In this use case, it's better to use the final position of the point to create the gradient and the way to get it is to use getProps method of data element instance, passing the array of properties you need and true which means you need the final value.
  const gradient = ctx.createLinearGradient(x0, 0, x1, 0);
  gradient.addColorStop(0, p0.options.backgroundColor);
  gradient.addColorStop(1, p1.options.backgroundColor);
  return gradient;
}

I think the last part doesn't need any comment. Be aware that the code is creating a gradient where the color is ONLY horizontally distributed (see y1 and y2 set to 0).

@ttvettuparambil just FYI, I used the same code you have as segment callback in the PR as additional test case for Chart.js:
https://github.com/chartjs/Chart.js/pull/11217/files#diff-c6a1b50ea001b54e20c01e22400f72a72c54408b1f2411c08c1f71ebb495b202


[Issue]: ROCm Install - Use case Graphics -- Required/Optional

Problem Description
Does RPP require the graphics use case or just the rocm use case?
If this does require graphics use case what are the functionality support or rocm graphic libs required?
Operating System
ALL
CPU
ALL
GPU
Other
Other
ALL
ROCm Version
ROCm 6.0.0
ROCm Component
rpp
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

graphics use case is not required for RPP. Just "rocm" use case is enough

TODO - Remove graphics from install instructions


Suggestion: Restructure project

Hi,
I would suggest following changes:
name folders "nodes" to "dyf" and "samples" to "extra" to fit the Dynamo Package convention.
Move "wiki_imgs" either to Dynanimato.wiki (if intended to use it in the wiki) or "extra" (if intended to use them in the package).

This has been addressed in 2db18137cb9074901e8fadf53c7c2ad5f559f2eb and c694e23af3317084e5e987d4aa45b31a1a91dfa4
Everything that is supposed to end up in the actual package is now organized in the folder aptly named package


CS lime-212, Redundant Check on Strategy List

Description
Several functions in SavingsAccount and CreditLine contracts iterate through the list of strategies. The functions make two sanity checks:
• balanceInShares[_user][_token][_strategy] != 0,
• _strategyList[i] != address(0).
The second check is unnecessary because if _strategyList[i] is equal to address(0), the first check would be false.
Integrations Checklist

[ ]  Have any function signatures changed? If yes, outline below.
[ ]  Have any features changed or been added? If yes, outline below.
[ ]  Have any events changed or been added? If yes, outline below.
[ ]  Has all documentation been updated?

Changelog

Removing redundant checks for strategy in CreditLine.sol.
Removing redundant checks for strategy in SavingsAccount.sol.

Event Signature Changes
Function Signature Changes
Features
Events

Codecov Report

Merging #272 (bce1977) into v1-alpha (2230ce8) will increase coverage by 0.11%.
The diff coverage is 80.00%.

:exclamation: Current head bce1977 differs from pull request most recent head 4b8cab3. Consider uploading reports for the commit 4b8cab3 to get more accurate results

@@             Coverage Diff              @@
##           v1-alpha     #272      +/-   ##
============================================
+ Coverage     78.40%   78.52%   +0.11%
============================================
  Files            16       16
  Lines          1681     1676       -5
  Branches        310      306       -4
============================================
- Hits           1318     1316       -2
+ Misses          363      360       -3

Impacted Files
Coverage Δ

contracts/Pool/Extension.sol
84.05% <ø> (ø)

contracts/Pool/Pool.sol
94.48% <ø> (ø)

contracts/Pool/PoolFactory.sol
68.96% <ø> (ø)

contracts/SavingsAccount/SavingsAccount.sol
67.18% <0.00%> (+0.52%)
:arrow_up:

contracts/SavingsAccount/SavingsAccountUtil.sol
86.04% <ø> (ø)

contracts/yield/NoYield.sol
78.57% <ø> (ø)

contracts/CreditLine/CreditLine.sol
83.91% <100.00%> (+0.39%)
:arrow_up:

contracts/PriceOracle.sol
61.29% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2230ce8...4b8cab3. Read the comment docs.


Genuary workflow fails

What happened?
Our daily Genuary workflow seemed to fail this morning: https://github.com/EthanThatOneKid/acmcsuf.com/actions/runs/5365380964/jobs/9734171837#step:3:27. It seems like the API keys may need to be rotated.

Specifically, the DISCORD_TOKEN variable.

Specifically, the DISCORD_TOKEN variable.

Updated https://github.com/EthanThatOneKid/acmcsuf.com/settings/secrets/actions/GENUARY_DISCORD_TOKEN.

No longer fails!
Action required: Set up 2FA on the account that owns the secret Discord token.


Support configuration file

To be honest, I don't have much experience in this. I think there is a ConfigParser and I don't know about OSes other than Linux, but even Linux there might be various places for configuration files. I think the order of search could be

$XDG_CONFIG_HOME/<PyClock_DIR>/<CONFIG_FILE>
$HOME/.<CONFIG_FILE>

Notes:

The XDG is part of XDG Base Directory Specification.
<PyClock_DIR> can be

PyClock
pyclock

<CONFIG_FILE> can be

PyClock.conf
pyclock.conf
pyclockrc

Configuration file should be first to be loaded to override default options, then command-line options override thereafter.

I have experience with ConfigParser. It should be very straightforward.

@Regaerd Good, then this is your task (Feeling like I'm the boss here, just kidding), if you are up to it, and I can prepare a simple setup.py, #12

It seems we need -s, -p, --center (-c is used), and -a, so the configuration can be overridden by command-line option. Thoughts?

So -s, -p, etc. would set the properties to true?
Yes, I guess that would be worthwhile.
I would rather -c and -C be for center in that case.
Color could then be -i or -ci with --color-index
Which would work nicely, since I was considering adding --color-value for terminals with support for 256 colors.

I would rather -c and -C be for center in that case.

I would prefer that, too.

Color could then be -i or -ci with --color-index

On GNU/Linux, I personally dislike option name like -ci, or
-foobar, it would confuse single letter option combination, e.g.
-SPCA. Single letter and long option name, just these two types, no
mixed usage in my opinion.
How about -k and --color. --color-index is bit of wordy.

Which would work nicely, since I was considering adding --color-value for terminals with support for 256 colors.

Do we really need two option? You can just add the support and use the
color option. They are both index anyway, in fact, the current colors
are the first ten colors in 256-color, so the same thing. You just
initialize the full set of 256-color, and that should be it.
So, if you agree, then I shall leave the renaming to you and -spca
to you, just duplicate the lines, edit option names and help, flip
store_*. And don't forget to update README for new -spca.
(Just realized that is "spca," feeling we need a mascot, I vote for a kitten)

Yeah -k should be good.

They are both index anyway

Right, I'm not sure what I was thinking.

And don't forget to update README

Well, now there's two README's to forget about.

We can just switch to reST, or don't use reST the same way since it's only for PyPI, just short description and a link to GitHub.

I prefer md actually. I would use pandoc to handle the conversion automatically, but it's been consistently making mistakes.
It's not a huge deal yet.

Here is an example of a package I maintained.
Since it has documentation, so I don't even put much stuff in README.rst, which is used on both PyPI and project website. A quick example usage and links section and that's it.
A rule of thumb is to not make your life harder. I could have put the information on README and docs, but that's just asking for more works that really ain't necessary.

Yes, good advice.
Next time I need to update the readme's I'll consider stripping down the rst version to something that doesn't need updating.
The largest chunk is just controls and commands.


SX [User Story] [UI] Chatbot interaction 6

[User Story] [Epic]
Multi-party approval, Approval configuration

Epic Title
(Epic NAME)

User Story Title
(Story NAME)

T-shirt size: S(mall), M(edium), L(arge)

Story Points - Fibonacci (1,2,3,5,8,13)

User Story
[Value Statement] "As a user of a certain type (specify), I want to do something (action) so I can provide business value"

Description
A short description

Business Need (the why):
from epic, if needed

Details
All the details needed to accomplish the story. Tasks - can be linked

Additional User Story
linked

Acceptance Criteria/Tests
The list of tests or criteria to be used to validate the story has been completed. May still need UAT or SIT / integration testing with other stories.

Failure
if any, fold in negative tests

Definition of Done
e.g.Passes all regression tests, Passes testing per acceptance criteria, Approved by UI team, Able to show feature in demo

UX/UI Details and/or mockups
if UI change, a mockup or description of what to change, attach image/file

Applications or Systems impacted
upstream/downstream applications/systems

Dependencies
non story dependencies

Assumptions
if any

Performance Considerations
Will the work performed as part of this story impact system performance. If there is potential for a performance hit, where would you expect it to manifest in the system.

Security Considerations
E.g., does this story involve working with Personal Identifying Information (PII)? Note any such security sensitive aspects.

QA Considerations
what might be hard to test or test setup criteria

Acrhitectural/System or Component Impacts
if we're adding something new or significantly changing architecture flag for architect review

Reverse Engineering Required
to document areas not known well by dev team

Questions/Clarifications
Create new issue (add label: question) in this repo and link to this issue

Intent addressed in #943.


SSH executing commands to Palo Alto firewall

I am trying to issue the "show system info" command via SSH on a Palo Alto firewall - but no success
# Opening session
$session = New-SSHSession $Script:computer -Credential $Script:credential -Port $Script:port -AcceptKey  -ErrorAction "Stop"
$sessionId = $session.SessionId

# Execute command
$stream = New-SSHShellStream -SessionId $sessionId -Columns 80 -Rows 256
$stream.Write("show system info")
$stream.read()

This just gives me:
Last login: Fri Jun  7 11:35:07 2024 from <IP_ADDRESS>

Number of failed attempts since last successful login: 0

show system info

Trying to do an additional $stream.read() a few seconds later:
user@fwl01> show

user@fwl01> show system

user@fwl01> show system info

I never get the expected output.
Any hints on how to handle this ?

Remeber to add a carriage return in the command. So it is interpreted as an enter and have a sleep after the commandSent from my iPhoneOn Jun 7, 2024, at 5:40 AM, bipsendk @.***> wrote:
I am trying to issue the "show system info" command via SSH on a Palo Alto firewall - but no success
Opening session
$session = New-SSHSession $Script:computer -Credential $Script:credential -Port $Script:port -AcceptKey  -ErrorAction "Stop"
$sessionId = $session.SessionId
Execute command
$stream = New-SSHShellStream -SessionId $sessionId -Columns 80 -Rows 256
$stream.Write("show system info")
$stream.read()
This just gives me:
Last login: Fri Jun  7 11:35:07 2024 from <IP_ADDRESS>
Number of failed attempts since last successful login: 0
show system info
Trying to do an additional $stream.read() a few seconds later:
@.***> show
@.***> show system
@.***> show system info
I never get the expected output.
Any hints on how to handle this ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Adding a `r in the command then gives this read output:
show
exivity_RO@nCNM-fwl-07-01> show system
exivity_RO@nCNM-fwl-07-01> show system info
WARNING: terminal is not fully functional
-  (press RETURN)

Update - adding rn helped ...


[CFM_214] feat: Create SearchInfo popup

https://user-images.githubusercontent.com/41050673/222772578-b53b38f9-118f-4316-8042-4f8c4b41d975.mp4

Left a few comments.

I pushed the changes, not sure if I did it correctly. Can you see the changes?


How-to guide to install all AWS modules

This might change with multi-region

The idea is a "How-to" guide with just the SQL to install all modules or it makes sense to create a guide on how to manage modules in general? Showing the iasql_modules(), iasql_installed_modules(), iasql_install() and iasql_uninstall() methods?

I leave here the query to install/uninstall all modules: select iasql_install(variadic array(select module_name from iasql_modules_list()))

Yes! Though I think @dfellis had something in mind to make the query even simpler?

Closed in favor of https://github.com/iasql/dashboard/issues/280


Change window title on new message

Operating system: ubuntu 16
Messenger for Desktop version: 2.0.3 - beta

I just installed messenger for desktop via dpkg and the messenger window does not indicate whether I ahve some new unread messages.
Prior to installing messenger via dpkg I used downloaded .deb package, which changed title depending on new message.

Fixed in 2.0.5, will be released soon


Tslint warnings

Thanks for this PR. However, on the develop branch I have made some significant changes to the example app that makes this PR obsolete. Since I will soon release version 3 I will not merge this PR.


added data loaders for stringer datasets

New branch for pushing data loaders.

Good to go?

I need to add another data loader for the Steinmetz dataset. Should that go in this pull request, or clean branch? I noticed the automated workflow keeps changing the files, and I don't know if it's safe to resubmit to this PR.

With #189 merged, the automated workflow should run checks on tutorials, but not modify them. You should be able to push additional notebooks to this PR if you want to do everything in one place. But can you say why you think the files are getting changed?

For some reason, the pull request diff is picking up all the files added to master since you made your branch, but the compare view doesn't see them:
https://github.com/NeuromatchAcademy/course-content/compare/master...data_loaders2

This is maybe a known issue in github: https://stackoverflow.com/questions/16306012/github-pull-request-showing-commits-that-are-already-in-target-branch
But that Stackoverflow thread is a little old.

Originally I thought I had to make a new branch because I had to pull changes, but it turns out it was just the CI script and a yaml file. If you could merge in these three notebooks that would be great.
Re: new issue, if you don't think it's safe to proceed I can just make a new branch and a new pull request.

I did a local checkout of your branch and the diff against master also only shows the files you've added. So I think this is just a github bug.

To confirm, the merge commit only has your files: https://github.com/NeuromatchAcademy/course-content/commit/2e1be413bc70d229d743ef45335fe5091cd1f025


PDF export of Microsoft .NET documentation doesn't generate code colorization

PDF export of the .NET documentation doesn't colorize highlighted portions of code:

Comparison between PDF and HTML output:

So, reading the docs offline it's hard to grasp the important parts of changed code.

NB: This issue is not about full code colorization, but only about colorizing the highlighted portions of code:
Current PDF Result

Expected PDF Result

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

ID: ca6139d4-022c-395a-0abf-539660550df1
Version Independent ID: 3dd23f8a-300e-3f78-1dc5-8b759aabdccc
Content: Get started with ASP.NET Core
Content Source: aspnetcore/getting-started/index.md
Product: aspnet-core
Technology: aspnetcore-getstarted
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thanks. This has been reported

@Rick-Anderson : None of the issues in the list you pointed to deals with colorization.

Can you create one in that repo? That's where it needs to be reported.

Sure. Thanks for pointing me to that repo. I created a corresponding issue there.


New release including the latest features

Currently, Bower finds release 2.0.0 when asked to pull this project as a dependency. Unfortunately, that version does not seem up-to-date (in particular, it does not have support for the cs-depends-on attribute).
@axel-zarate Is it possible for you to add a new release with the current master?

@axel-zarate Many thanks. We've just seen that you've published a new release many days ago. What an exceptional reactivity! Cheers.

Yes, and I forgot to update this thread. Sorry about that. By the way, I was unaware that the bower integration was working. Good to know!


Infinite loop in BigDecimal.isValidLong

The following causes an infinite loop in SJS:
BigDecimal("9.223372E+285625056").isValidLong

On JVM, it correctly returns false:
Welcome to Scala 2.11.8 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_77).
Type in expressions for evaluation. Or try :help.

scala> BigDecimal("9.223372E+285625056").isValidLong
res0: Boolean = false

The loop is in fact not infinite. It is just EXTREMELY slow. isValidLong uses longValueExact which uses valueExact. The last one computes the normilized version of the BigDecimal using toBigIntegerExact. It looks like the computation in toBigIntegerExact is slow due to the size of the BigIntegers involved in the computation.
A simple solution would be to test some bounds in valueExact before computing the toBigIntegerExact and fail faster for large numbers.

Erm this is still a problem in 0.6.9.
BigDecimal("9.223372E-2147483642").isValidLong

Btw, large numbers (eg. 9.223372036854776E+79031823) are now fast, just not slow ones by the look of it.

Oh, we had some issues with the implementation of that fix and ended up removing the part that was supposed to take care of those cases rather than fixing it. This should be simple to fix.
THe issue with the small one is that it is relying on BigInteger arithmetic, which is a "bit" slow. The used only the scale and precision of the BigDecimal to determine if there is no way that the number will fit in a long. If you really need the performance we could explore the possibility of implementing valueExact without using BigInteger, but that would be for another issue.

ended up removing the part that was supposed to take care of those cases rather than fixing it

Not really. It was supposed to be handled by toBigIntegerExact(), which should fail fast if the number is not integer. The problem likely lies around there, then.

If you really need the performance

To clarify, I'm not sure if you're seeing the same results as me but on my machine, I've let the call run for about 3, 4 minutes before I killed it.

@japgolly if I run your example I get on JVM, Rhino, Node (fastOpt/fullOpt) the result is always an exception on the creation creation of the number.
scala> BigDecimal("9.223372E-2147483642")
java.lang.NumberFormatException: Scale out of range.
  at java.math.BigDecimal.adjustScale(BigDecimal.java:609)
  at java.math.BigDecimal.<init>(BigDecimal.java:562)
  at java.math.BigDecimal.<init>(BigDecimal.java:383)
  at java.math.BigDecimal.<init>(BigDecimal.java:806)
  at scala.math.BigDecimal$.exact(BigDecimal.scala:125)
  at scala.math.BigDecimal$.apply(BigDecimal.scala:283)
  ... 33 elided

@japgolly did you found other cases that end up in an infinite (or extremely long) loop?

It's a test in Monocle that comes up with these cases. I locally put in a few hacks to print out what it's doing and what the numbers are and then run the test and see where it hangs.
This is the test:
https://github.com/julien-truffaut/Monocle/blob/master/test/jvm/src/test/scala/monocle/std/BigDecimalSpec.scala
And this is the little local hack I used to print useful stuff:
diff --git a/core/shared/src/main/scala/monocle/std/BigDecimal.scala b/core/shared/src/main/scala/monocle/std/BigDecimal.scala
index 0a4132d..379f787 100644
--- a/core/shared/src/main/scala/monocle/std/BigDecimal.scala
+++ b/core/shared/src/main/scala/monocle/std/BigDecimal.scala
@@ -6,9 +6,41 @@ import monocle._

 object bigdecimal extends BigDecimalOptics

+object Ahhh {
+  def time[A](name: String)(a: => A): A = {
+  printf("[%s] Starting...\n", name)
+  val start  = __start
+  val result = a
+  __stop(name, start)
+  result
+}
+
+import System.{nanoTime => clock}
+
+private type Clock = Long
+
+private def __start: Clock =
+  clock()
+
+private def __stop(name: String, start: Clock): Unit = {
+  val end = clock()
+  val timeInMs = (end - start).toDouble / 1000000
+  printf("[%s] Completed in %.3f ms.\n", name, timeInMs)
+}
+
+}
+import Ahhh._
+
 trait BigDecimalOptics {
   val bigDecimalToLong: Prism[BigDecimal, Long]  =
-    Prism[BigDecimal, Long](bi => if(bi.isValidLong) Some(bi.toLongExact) else None)(BigDecimal(_))
+    Prism[BigDecimal, Long]{bi =>
+      if( time(s"$bi.isValidLong")(bi.isValidLong) )
+        Some( time(s"$bi.toLongExact")(bi.toLongExact) )
+      else
+        None
+      }{ l =>
+        time(s"BigDecimal($l)")(BigDecimal(l))
+      }

   val bigDecimalToInt: Prism[BigDecimal, Int] = bigDecimalToLong composePrism long.longToInt

Thanks for that code. I found that the value that we used to compute the fast path was overflowing. The fix is simple and I tested it on Monocle.

Very nice, thanks @nicolasstucki. With this fix, Monocle's entire test suite will pass in Scala.JS. :+1:


Full encoder and decoder for little endian uint12 on little endian

The code added to encode-decode.ipynb handles encoding and decoding
of arrays with arbitrary length with all index calculations
and treatment of dangling blocks. It was written to closely
mimick the way it would be written in C or C++ so that
it can serve as a prototype for a correct C/C++ implementation.

Merged, thanks!


Write a Julia program to convert seconds to hours

Description
Write a Julia program to convert seconds to hours
1 hour = 60 minutes = 3600 seconds
Input  : 18000
Output : 5

How to contribute

Save the solution in program/convert-seconds-to-hours.jl file
Add convert-seconds-to-hours.jl file in convert-seconds-to-hours folder

!assign


Readme

Added a readme with game instructions. Let me know if you have changes or just merge and change on your end.

thanks for adding this


Add type safety to UserModule

Created from #274. See also #7 .
Use the models from #301.

Please create a PR and get a green build.

PR: GH-373


fix plot_network bottom data drawing

Fix mx.viz.plot_network() function that does not plot the bottom data blob.

mxnet doesn't differentiate between input and weight. Is this going to plot all weights?

@piiswrong It will only additionally plot the bottom data, for example:
def get_mlp():
    """
    multi-layer perceptron
    """
    data = mx.symbol.Variable('data')
    fc1  = mx.symbol.FullyConnected(data = data, name='fc1', num_hidden=128)
    act1 = mx.symbol.Activation(data = fc1, name='relu1', act_type="relu")
    fc2  = mx.symbol.FullyConnected(data = act1, name = 'fc2', num_hidden = 64)
    act2 = mx.symbol.Activation(data = fc2, name='relu2', act_type="relu")
    fc3  = mx.symbol.FullyConnected(data = act2, name='fc3', num_hidden=10)
    mlp  = mx.symbol.SoftmaxOutput(data = fc3, name = 'softmax')
    return mlp

should plot like:
(data) -> (FullyConnected) -> (Activation) -> (FullyConnected) -> (Activation) -> (FullyConnected) -> (SoftmaxOutput)
However, the original version plot like:
(FullyConnected) -> (Activation) -> (FullyConnected) -> (Activation) -> (FullyConnected) -> (SoftmaxOutput)
It lacks plotting "(data) -> "
BTW, how can I directly insert a picture in the comment?
￼

@taoari This PR is good to merge. Please update to the latest master to merge.

You may need to update again...


Get tests passing on 2.13.

This PR gets Dagon cross-building for 2.13.
We use type aliases and methods to paper over pre- and post-2.13 changes, including IterableOnce and LazyList. This is similar to what we do in Paiges but a bit more involved.
This PR does disable a few compiler flags that aren't supported on 2.13. I could have hacked SBT to make them conditional but I didn't have the energy (but would be fine if someone else wanted to do this). I left them commented out for now.
cc @johnynek

Good feedback, I took your suggestions.

two of the tests timed out. I wonder if the change to Iterable did it somehow making really giant lists that are eagerly constructed?
I restarted them, but if this is the new normal we may need to limit them somehow.
Looks good to me to merge though and see if we continue to see the issue.

actually, 2.11 js failed twice in a row.
I think we need to figure it out.
The diff coverage is 90.9%.

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
- Coverage   82.71%   82.58%   -0.13%
==========================================
  Files          13       13
  Lines         376      379       +3
  Branches       16       11       -5
==========================================
+ Hits          311      313       +2
- Misses         65       66       +1

Impacted Files
Coverage Δ

core/src/main/scala/com/stripe/dagon/HMap.scala
96% <100%> (ø)
:arrow_up:

core/src/main/scala/com/stripe/dagon/Dag.scala
85.38% <90%> (-0.27%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d1faf1...aceb0e5. Read the comment docs.


Check for package.json parse errors in isAtomRepoPath

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change
Wrap the call to the season parser in a try/catch block to prevent errors in parsing any package.json file in the current directory from crashing the entire Atom process.
Alternate Designs
Switching to the built in JSON.parse would remove the need to bring in the season parser, but it would still require a try/catch block around the parsing.
We could try manually parsing the file to avoid any of the validity checks since all we care about is the name, but that's more effort than it is worth.
Why Should This Be In Core?
It fixes a bug in core.
Benefits
Atom will no longer crash when launched from a directory with an invalid package.json.
Possible Drawbacks
The function now has a try/catch block, preventing some runtime optimizations from being possible on the function, however as it is only run once per window initialization the runtime will never optimize it in the first place.
Verification Process
I created a build with these changes on top of e6507c59cb846cd02b84f4a3b10dd43b2572a939 and ran it in a directory that was crashing the previous version without issues.
Applicable Issues
Fixes #18058.

Duplicate of #18060.


Buggy stop() on Android 6.0.1

Hi!
stop() function works buggy for me on Android 6.0.1: when I call it, it starts playing sound from the beginning, plays in for about 1-2 seconds and stops it. Sample code:
window.plugins.NativeAudio.preloadComplex('1.mp3', '1.mp3')
window.plugins.NativeAudio.play('1.mp3')
setTimeout(function() {
    window.plugins.NativeAudio.stop('1.mp3')
}, 1000)

+1 I have the same issue on Android 6.0.1 (haven't tried other versions though) when using preloadComplex(), using loop() then, any time I call stop() the sound stops then plays once more for about a second or so.

Same exact problem on Android 6.0.1 when using preloadComplex(), it keep playing after hit stop for 1 second.

+1 same problem here.


Demo - OpenCV Error (Windows)

Hi, when I run the demo the main GUI comes up and when I try to select 'take photo' button the photo window can't be closed and the input image is never updated. When I force close the progrm I see the following error:

File "[MyFileTree]\dynamic_net\dynamic_style_transfer\gui\webcam_style_transfer_widget.py", line 23, in on_take_photo_click
self.input_image = self.webcam.take_photo()
File "[MyFileTree]\dynamic_net\dynamic_style_transfer\gui\webcam\webcam.py", line 17, in take_photo
cv2.imshow(winname, image)
cv2.error: OpenCV(3.4.7) C:\projects\opencv-python\opencv\modules\highgui\src\window.cpp:358:
error: (-215:Assertion failed) size.width>0 && size.height>0 in function 'cv::imshow'

Which version of OpenCV are you using and what should be the correct result of pressing 'take photo'?

Hi, thanks for raising the issue. you should use "space bar" or "enter key" to capture your image after clicking 'take photo'. I updated the instructions in the readme.
The error you get is probably because you forced closed the program.


Build failure: prusa-slicer 2.7.1 (aarch64-darwin)

Steps To Reproduce
Steps to reproduce the behavior:

Build prusa-slicer 2.7.1 on aarch64-darwin

Build log
error: builder for '/nix/store/p9rhlppb7z6zc87dhpvykcsb6pgwjixl-prusa-slicer-2.7.1.drv' failed with exit code 2;
       last 10 log lines:
       > include/boost/functional.hpp:557:74: error: no template named 'binary_function' in namespace 'boost::functional::detail'; did you mean '__binary_function'?
       >     class pointer_to_binary_function : public boost::functional::detail::binary_function<Arg1,Arg2,Result>
       >                                               ~~~~~~~~~~~~~~~~~~~~~~~~~~~^
       > include/boost/functional.hpp:46:24: note: '__binary_function' declared here
       >             using std::binary_function;
       >                        ^
       > 16 errors generated.
       > make[2]: *** [src/libslic3r/CMakeFiles/libslic3r.dir/build.make:77: src/libslic3r/CMakeFiles/libslic3r.dir/cmake_pch.hxx.pch] Error 1
       > make[1]: *** [CMakeFiles/Makefile2:1105: src/libslic3r/CMakeFiles/libslic3r.dir/all] Error 2
       > make: *** [Makefile:146: all] Error 2

Additional context
Builds on x86_64-linux w/o problems (local & Hydra).
Notify maintainers

@moredread @tweber @tmarkus
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 23.2.0, macOS 14.2.1`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.19.2`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Add a :+1: reaction to issues you find important.

Consider opening a PR for making it broken.

Fixed by #283675.


Regenerate key pairs in Frankfurt and extend expiry date to 6 months

https://github.com/relaycorp/relaynet-internet-gateway/blob/b7aafc931f013c7ebb9e1599434d9cd464866e8b/src/bin/generate-keypairs.ts#L21-L22

Extended to 1 year instead: https://github.com/relaycorp/relaynet-internet-gateway/commit/e8949e1ddb46f3e911d504d6637f3a11ea0d58c9

Done in:

https://github.com/relaycorp/relaynet-internet-gateway/commit/e8949e1ddb46f3e911d504d6637f3a11ea0d58c9
https://github.com/relaycorp/relaynet-internet-gateway/commit/bf8297b9d01606df444ada4f8138a82094b4743c
https://github.com/relaycorp/cloud-gateway/commit/e506dcc05bdeef92d5e75b724e1cda970c27b7ee

Hmm, didn't quite work because the key id wasn't picked up because I'm using a ConfigMap that's only updated upon install.


Write a program to check whether a given string is anagram

Description
Two strings are said to be anagrams if they make a meaningful word by rearranging or shuffling the letters of the string. In other words, we can say that two strings are anagrams if they contain the same characters but in a different order.

Can i be assigned this issue?

!assign

Hey @ShadyResurrected, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.

ok

!assign

Hey @Coder-12-ux, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.

!assign

Hey @sohamsahare123, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.

i would like to work on this project if its still open

!assign

Hey @Sumsum1000, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.

!assign

Hey @allan-anilson, !assign command is only for good first issue !!!
Please choose another issue.
Thanks for your interest in contributing to this project.

Program added successfully :tada:
Thanks for your contribution :hugs:


Add a testing script

Requires Process.chdir(_) from wren-console and Strings.globMatch(_, _) from wren-essentials

Ok, now we have an issue where this wouldn't just work with the release version of wrenc... is it's the script job to check the runtime version, or should the package manager handle that?  I worry someone doing development of the library isn't using the package manager though so they would never flag that...
I'm wondering if we should have some sort of Runtime.assertVersion perhaps - since we're still so young and changing?

I'm wondering if we should have some sort of Runtime.assertVersion in wren-console itself perhaps - since we're still so young and changing?

I'd say it's the script's job, and that assertVersion is a good idea.

https://github.com/joshgoebel/wren-console/pull/15/files

Oh that might be nice. Excited to try this later.


Fixex #18499 - Query::$indexBy is auto-inserted into Query::$select in Query::all()

When using yii\db\Query::all(), the yii\db\Query::$indexBy is auto inserted into yii\db\Query::$select - The same as in yii\db\Query::column()
If it works, there should be probably made some changes in docs in Query::$indexBy, Query::all() and original Query::column().

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
sorry, dunno

Fixed issues
#18499

@bizley would you please help adding some tests to the pull request?

Sure thing, first time possible.

Merged. Thanks!


Don't install x64 framework package on Windows 10 ARM64 devices as it…

Don't install x64 framework package on Windows 10 ARM64 devices. Win 10 ARM64 doesn't support x64 apps.
Tested IsWindows10_21H2OrGreater API on Windows 10 and Windows 11 and works as expected.
And fixed an unused variable deploymentOptions in two places (in one place, it is used now. In another, it is removed altogether).
https://github.com/microsoft/WindowsAppSDK/issues/2564

/azp run

/azp run

/azp run

This seems very fragile. If downlevel gets serviced with these APIs, will all of this behave as expected?

This seems very fragile. If downlevel gets serviced with these APIs, will all of this behave as expected?
Even if suppose the GetMachineTypeAttributes get serviced downlevel, it will still work as expected. Because I am not only checking if the API is available BUT we are also calling the API to check applicability of a given architecture compatibility on the device.

/azp run

/azp run


Legit clippy::bad_bit_mask complain

error: incompatible bit mask: `_ | 128` can never be equal to `0`
   --> quinn-proto/src/tests.rs:407:17
    |
407 |         assert!(packet[0] | 0x80 != 0);
    |                 ^^^^^^^^^^^^^^^^^^^^^
    |
    = note: #[deny(clippy::bad_bit_mask)] on by default
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#bad_bit_mask

So, this one is the only clippy lint that is flagged as an error.
This assert appears to be effectively assert!(true).
If the intention here was to check for the long header, perhaps it should be instead
assert_ne!(packet[0] & 0x80, 0);

Yeah, I think your suggested fix is correct. PR, please?


Where are other (wow) classes ?

Add some tabs or reload the same Datatable with new content (from other classes)

Dropdown menu with all classes is ready. Need datas and loading function (javascript + datatable)

It's over 😸


OCM-1018: Change provider logger to tflog

Implement new internal logger uses "github.com/hashicorp/terraform-plugin-log/tflog" This is done in order to have better logging
With previous logger implementation, all log levels are printed in DEBUG log level
Also found two issues in Terraform:

If user not setting the TF_LOG environment variable, the logs are not printed
The log prints the caller, in that case it line in the logging file

Need to check if those issues are already fixed in later versions

/retest

/retest


[REG 2.067.0] std.getopt, problem with the automatic handling of "h"

b2.temp reported this on 2016-11-14T12:21:51Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=16684
CC List

andrei (@andralex)

Description
Compile this program:

===============
void handleH(){assert(false);}
void handleG(){}

void main(string[] args)
{
    import std.getopt;
    getopt(args, config.passThrough, "g", &handleG);
    getopt(args, config.passThrough, "h", &handleH);
}
===============

invoke with the -h option.
An assert error should happen.

There's a problem with the help system. If you replace "h" by "H" and call the program with -H, then the assertion fails as expected.

dlang-bugzilla (@CyberShadow) commented on 2017-07-21T09:16:57Z
This is a regression.

Simplified test case:

/////////////// test.d ///////////////
void main()
{
    import std.getopt;

    string[] args = ["program", "-h"];
    getopt(args, config.passThrough);
    assert(args == ["program", "-h"]);
}
//////////////////////////////////////

Introduced in https://github.com/dlang/phobos/pull/2072

(In reply to b2.temp from comment #1)
> @Alexandru, I think that the best fix is to
>
> 1/ add a static assert() that prevents to specify "-h" or "--help"  (for
> example there's already "optionValidator" that does a few static checks)

What if I want to generate my own help screen? E.g. if I'm using a library around std.getopt that generates its own enhanced help screen (e.g. ae.utils.funopt).

b2.temp commented on 2017-01-05T09:45:05Z
@Alexandru, I think that the best fix is to

1/ add a static assert() that prevents to specify "-h" or "--help"  (for example there's already "optionValidator�" that does a few static checks)

2/ also update the documentation, saying that this switch is handled automatically

The real problem I had before reporting was 1 hour lost trying to get why a function pointer was never called. With a static check this will never happen anymore to anyone.

b2.temp commented on 2017-01-08T09:19:30Z
Sorry, I allow myself to remove the assignee because of https://github.com/dlang/phobos/pull/5024 and also to prevent any double PR.


Add ghost role whitelist and enables drones for whitelisted players

About the PR
Adds role whitelisting from Nyanotrasen and reenables drone roles, requiring their players to be whitelisted.
This involves a lot of cherry-picking commits from Nyanotrasen so there's a decent potential for issues. I'm planning on testing it with that in mind but any additional help is apreciated.
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Drones are reenabled as ghost roles for players who are whitelisted.

As a disclaimer the one thing about Nyano WL that I'm not super happy with is that it's 1 whitelist for everything that requires whitelisting. I didn't want to touch DB stuff.
It works on Nyano because the way the server is administrated is designed with that restriction in mind. I'm not sure if it's a good general solution

Also we just disabled drones again today on Nyano because it's clear they need reworks even with the WL restriction

@Chief-Engineer you coming back to this?


windmill: 1.219.1 -> 1.246.15

Description of changes
this PR is waiting for the nsjail build fix PR to be tested, but essentially this should just work.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 282065 run on x86_64-linux 1

1 package built:

windmill


Support Play Framework 2.9

Update Play 2.8 to 2.8.20

Nice! Does this PR supersedes #2517? A test using Scala 3, which is supported since Play 2.9, would be nice, but if you don't have the bandwidth to add it, we can add it later.


error code for errors like block number not yet indexed

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
Currently when making a query with the block parameter, like
{
  tokens(block: {number: 10000000}) {
    id
    owner
  }
}

We get an error like :
{
  "errors": [
    {
      "message": "Failed to decode `block.number` value: `subgraph QmX2cspb9qty6nzUEwf7iw6jU7ptwk6hBp9VVhsHdHHcMA has only indexed up to block number 9753342 and data for block number 10000000 is therefore not yet available`"
    }
  ]
}

What is the expected behavior?
It would be great to have error code so we can easily differentiate between different type of error, instead of parsing the error message

Hi, is there any interest in that.
I just encounter the issue again and was going to submit a new issue when I realised I already submitted the same here :)

Hi, is there any interest in that. I just encounter the issue again and was going to submit a new issue when I realised I already submitted the same here :)

I definitely have interest in this too.  I imagine most people are interested in getting better errors from The Graph.

thanks @chanhosuh and @wighawag
@Jannis @lutter this seems useful, but also relevant to the attestable errors work?

Yes, I think machine-readable error codes would be very worthwhile to have


Handle situations where language client is disposed earlier than expected

Partial fix for #6865
This fix merely ensures we handle situations where language client is disposed earlier than expected (and prevent null object references).
Currently when language client gets disposed early we get null object reference errors. This will ensure we do not get such errors (ie gracefully handle disposing objects).

Codecov Report

Merging #7388 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7388      +/-   ##
==========================================
- Coverage   58.15%   58.06%   -0.09%
==========================================
  Files         493      493
  Lines       21778    21782       +4
  Branches     3487     3488       +1
==========================================
- Hits        12665    12648      -17
- Misses       8326     8343      +17
- Partials      787      791       +4

Impacted Files
Coverage Δ

...client/activation/languageServer/languageServer.ts
92.06% <100%> (+0.53%)
:arrow_up:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

src/client/common/platform/pathUtils.ts
66.66% <0%> (-5.56%)
:arrow_down:

src/client/linters/pydocstyle.ts
84.61% <0%> (-5.13%)
:arrow_down:

src/client/common/logger.ts
68.55% <0%> (-4.41%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
66.66% <0%> (-2.9%)
:arrow_down:

src/client/interpreter/locators/helpers.ts
70.73% <0%> (-2.44%)
:arrow_down:

src/client/common/process/proc.ts
13.28% <0%> (-1.57%)
:arrow_down:

src/client/common/installer/productInstaller.ts
76.71% <0%> (-0.53%)
:arrow_down:

src/client/datascience/constants.ts
99.63% <0%> (-0.37%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba52496...a978190. Read the comment docs.


Adding support for inline attachments replacements in MailboxPreview

I use swoosh for sending emails with inline-attachments. It is difficult to preview the email with images when using MailboxPreview.
I decided to do some string replacement to link to the attachments correctly when viewing in browser. But not sure if it is something we would like to enable by default or just enable with a flag (config, specific-route or query-parameter)?
Please let me know what you think about this please?

Hi @princemaple - I've merged your refactor PR. Thanks for having a look and suggesting those edits.
They look good!
Are you happy to merge or do you want me to review anything else?
If you also don't mind adding hacktoberfest-accepted label to this PR it would be much appreciated 😄

Thanks @theodowling !
One last thing, could you post a screenshot of this feature?


cache: pre-cache Plex On Deck content

Considering plex integration is available albiet in its infancy, can we look at seeing how to have rclone get an upcoming list of what is "On Deck" and download / pre-cache this data?
Similar/Same idea to the unionfs script i have seen floating around which does much the same thing, but it requires a unionfs mount and the script to run.
Thanks

@remusb?

Implementing more features based on Plex seems like a bad idea to me. This would increase the complexity of the cache even more would force us to keep up to date with the Plex development to keep this feature working. rclone should remain to be a common tool for working with storage providers. Besides that, there are other media solutions like Plex (for example Emby) which could also benefit from such a feature.
How about a new rc command to trigger caching of specific files:
rclone rc cache/fetch chunks=10 file=movie1.mkv file2=movie2.mkv

This could would keep rclone general and provide a way for external scripts/tools to fill the cache.

@jkaberg fair point, im sure rclone in all efforts are trying to improve this just as continued development.  This isnt particarly for load times, as a lof o people who use rclone may just have poor bandwidth.  This saves the use of bandwidth being used to watch media, rather than having the prepopulated (eg when lots of people at home at night, bandwidth is important, but when its not being used during the day, bandwidth can be used).
@B4dM4n i hear your point, but there is already plex integration.  This service could be dependant on if plex integration is active, then action it.  Your idea of doing a combined call for scripts isnt a bad option, but this could just be done at the OS level rather than integration into rclone.  My thoughts would be that if you are going to amend rclone, you would amend it for the integration to get it done right rather than dependant on something external.  just my opinion but.

@remusb @ncw - any chance of this being considered?
Appreciate may be a bit of work to do - but............ would be great to have this done.

This feature would be great.
Let's hope to have an answer.

The command is not merged yet. We need to wait for a review of the code.

Ok, tnx

I think now command has been committed.
I tried to run the .py file after changing settings and got no errors.
But how i know it is working?
What is it caching exactly?

This could be my bad. I messed up the error return value of the fetch command.
There should be a corrected version at https://beta.rclone.org/v1.43-027-gdeda0936-beta/ soon.
To get the python script working only the following variables have to be set:
PLEX_URL = 'http://<IP_ADDRESS>:32400' # the plex server address
PLEX_TOKEN = '0123456789abcdef' # your plex token
RCLONE_MOUNT_PATH = '/mnt/data/' # the rclone mount point that plex is using
RCLONE_RC_ARGS = ['--rc-addr=<IP_ADDRESS>:1234'] # custom rclone arguments or [] if not needed
RCLONE_RC_CHUNKS = ':100' # the cache chunks to get

The chunks= parameter is documented in the rclone rc help. It is based on the chunk_size configuration of the cache remote and specifies the chunk range to download.
The default value :100 will download the first 100 chunks of each file. To download the whole file use :  as the range.

I have this for every object:
Any idea?
{
"status": {
"erie_Tv/Chicago Med/Season02/Chicago Med.S02E08.L'escamotage.mkv": {
"Error": "object not found",
"FetchedChunks": 0
},
"erie_Tv/Grey's Anatomy/Season01/Grey's Anatomy.S01E01.Quando il gioco si fa duro.avi": {
"Error": "object not found",
"FetchedChunks": 0
},

But after some time i got this:
2018/09/03 20:56:38 Failed to rc: connection failed: Post http://<IP_ADDRESS>:5572/cache/fetch: net/http: timeout awaiting response headers
Traceback (most recent call last):
File "plex_ondeck.py", line 28, in
('file%d=%s' % (i, f) for i, f in enumerate(files))
File "/usr/lib/python2.7/subprocess.py", line 190, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '<itertools.chain object at 0x7fefa5ceac90>' returned non-zero exit status 5

2018/09/03 20:56:38 Failed to rc: connection failed: Post http://<IP_ADDRESS>:5572/cache/fetch: net/http: timeout awaiting response headers

This means that the rclone mount process took to long to fetch all the chunks and the rclone rc command canceled the request after the configured --timeout value (default 5m). The rclone mount will still continue to process all files, but you wont see the return status.
Your best option is probably to increase the --timeout setting.

files already cached are overwritten or are skipped?

All chunks found on disk will not be downloaded again.
But they will be read from the disk, which is a bit inefficient.

this sounds pretty promising.
@B4dM4n my apologies for not being able to understand what you are saying around the implementation but it sounds pretty sweet by the idea of getting X number of chunks in adv based on "On deck" contents.  This is a good implementation from what i can grasp.
Can you please advise me how to implement it and test?
I am running rclonev1.43-039 if that is of any benefit.
this is my config:
drpau@Syn:/$ sudo cat /volume1/rclone1.43.conf
Password:
[gdrive_mount]
type = drive
client_id = *******.apps.googleusercontent.com
client_secret = *******
scope = drive
root_folder_id =
service_account_file =
token = {"access_token":"*******}

[cache]
type = cache
remote = gdrive_mount:
plex_url = http://<IP_ADDRESS>:32400
plex_username = drp
plex_password = *******
chunk_size = 5M
info_age = 12h
chunk_total_size = 400G
plex_token = *******

[this_one]
type = crypt
remote = cache:
filename_encryption = standard
directory_name_encryption = true
password = *******

and my command to use rclone is:
/volume1/rclone1.43-039/rclone mount this_one: /mnt/gdrive.de/ --allow-other --config=/volume1/rclone1.43.conf --cache-db-path=/volumeUSB1/usbshare > /tmp/rclone.log 2>&1 &

To get the script from this post working, you first need to add the --rc flag to your mount command.
I explained the settings for the python scipt in this post.
In your case this would be the correct settings for these two settings:
RCLONE_MOUNT_PATH = '/mnt/gdrive.de/'
RCLONE_RC_ARGS = []

@B4dM4n thanks for that. Just fyi im on a synology nas and had to do a lot of (annoying) things to get this somewhat working on my device. long story short, had to get python 3 installed, then some modules, then install pip to get the plexapi stuff working and then edit some paths to make all the things work.
But........ finally got there.
I too am having what appears to be timeout issues as you mentioned above.  I am on a ~8megabit down, and 1megabit (ADSL) internet connection and my guess is that this is what doesnt have enough bandwidth to complete the process therefore - timeout occurs.
I have 2 follow up questions
1: where do i add the --timeout to?  my rclone mount or the execution of the script?
2: is there a way of getting verbose output / any feedback when running the script to confirm what is happening?

1: where do i add the --timeout to? my rclone mount or the execution of the script?

The --timeout flag can go into the RCLONE_RC_ARGS setting:
RCLONE_RC_ARGS = ['--timeout=1h']

2: is there a way of getting verbose output / any feedback when running the script to confirm what is happening?

rclone rc does currently not provide a live feedback. Only the final result is send back and then print out.
If you enable debug logging with -vv on your rclone mount command, you can watch the log output and see the downloaded chunks there, but this is not user friendly.
To get feedback after every file you could use this loop to download each file with it's own rc call:
    for video in plex.library.onDeck():
        for media in video.media:
            for part in media.parts:
                check_call(chain(
                    ['rclone'],
                    RCLONE_RC_ARGS,
                    ['rc', 'cache/fetch', 'chunks='+RCLONE_RC_CHUNKS],
                    'file='+part.file[len(RCLONE_MOUNT_PATH):]
                ))

Thanks very much for the feedback - appreciate it.
I presume its working as i get a quick response from playing items on deck - however is there a way of verifying it?  And by that - i mean a simple way :) thanks :)

The rclone rc response will tell you how many chunks of the file have been checked and are now in the disk cache (they could have been there already). FetchedChunks should have been named ProcessedChunks, which better describes it.
{
	"status": {
		"series/file2.mkv": {
			"Error": "",
			"FetchedChunks": 100
		}
	}
}

To check which specific chunks are cached, you need to go tn the --cache-dir and find the chunk files (named after their offset from the beginning of the file) manually.

1: where do i add the --timeout to? my rclone mount or the execution of the script?

The --timeout flag can go into the RCLONE_RC_ARGS setting:
RCLONE_RC_ARGS = ['--timeout=1h']

2: is there a way of getting verbose output / any feedback when running the script to confirm what is happening?

rclone rc does currently not provide a live feedback. Only the final result is send back and then print out.
If you enable debug logging with -vv on your rclone mount command, you can watch the log output and see the downloaded chunks there, but this is not user friendly.
To get feedback after every file you could use this loop to download each file with it's own rc call:
    for video in plex.library.onDeck():
        for media in video.media:
            for part in media.parts:
                check_call(chain(
                    ['rclone'],
                    RCLONE_RC_ARGS,
                    ['rc', 'cache/fetch', 'chunks='+RCLONE_RC_CHUNKS],
                    'file='+part.file[len(RCLONE_MOUNT_PATH):]
                ))

You mean using that instead of:
`    for video in plex.library.onDeck():
for media in video.media:
for part in media.parts:
files.append(part.file[len(RCLONE_MOUNT_PATH):])
check_call(chain(
    ['rclone'],
    RCLONE_RC_ARGS,
    ['rc', 'cache/fetch', 'chunks='+RCLONE_RC_CHUNKS],
    ('file%d=%s' % (i, f) for i, f in enumerate(files))
))

`
Changing that part i have this error
2018/09/12 08:34:11 Failed to rc: No '=' found in parameter "f" Traceback (most recent call last):   File "plex_ondeck_single_file.py", line 26, in <module>     'file='+part.file[len(RCLONE_MOUNT_PATH):]   File "/usr/lib/python2.7/subprocess.py", line 190, in check_call     raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '<itertools.chain object at 0x7fdad9d426d

just closing the loop on my end and requests - i have had the script running successfully for a few days now and have now incorporated it into my crontab schedule.
Thank you  for dev'ing this and your help in implementation

@B4dM4n sorry 1 last point in response to this:

The rclone rc response will tell you how many chunks of the file have been checked and are now in the disk cache (they could have been there already). FetchedChunks should have been named ProcessedChunks, which better describes it.

my log is outputting "FetchedChunks": 100 rather than ProcessedChunks even though they have been previously Fetched.
from cronjob which ran at 13:00 daily (13/9/18).
 "TV/Who Is America/Season 01/Who Is America - S01E04 - Episode 4 WEBDL-1080p.mkv": {
                        "Error": "",
                        "FetchedChunks": 100
                }

from manually running at 14:30 (today - 13/9/2018) - approx 1.5hrs later
},
                "TV/Who Is America/Season 01/Who Is America - S01E04 - Episode 4 WEBDL-1080p.mkv": {
                        "Error": "",
                        "FetchedChunks": 100
                }

Its not erroring or causing issues - just worth a mention.  im not sure if you intended on just renaming Fetched to Processed or were expecting both to be displayed in different scenarios.
thanks

@memnos
I updated the gist with the working script (there was a misplaced bracket in the last one).
@drpau
The result currently shows FetchedChunks, but this name is misleading.
The value returned is the number of ProcessedChunks, the number of chunks checked to be in the disk cache.

Tnx, now all seems to work properly.
Just 2 other question.

Is there the possibility to fetch only series on deck and not movies?
Is there the possibility to give cache/fetch for a dir instead on for single files?

Is there the possibility to fetch only series on deck and not movies?

Sure, the python plexapi can differentiate between movie's and shows. A check like isinstance(file, plexapi.Show) should do the trick.

Is there the possibility to give cache/fetch for a dir instead on for single files?

This is not implemented yet, but it is definitely possible. I will put it on my to-do list, unless someone else wants to try this.

Sorry, but where shouls i put isinstance(file, plexapi.Show)
I'm not a coder and i'm not able to code.
Tnx

@memnos
There is a new version of the python script in the gist. It will only download show episodes, but can be changed easily to filter movies.

@B4dM4n unsure what has occured lately but it appears that the script is now erroring.
i am on rclone 1.46
ash-4.3# ./rclone --version
rclone v1.46
- os/arch: linux/amd64
- go version: go1.11.5
ash-4.3#

and the script executes but logs this:
ash-4.3# cat /tmp/plex_ondeck.log
Traceback (most recent call last):
  File "/volume1/plex_scripts/plex_ondeck.py", line 28, in <module>
    ('file%d=%s' % (i, f) for i, f in enumerate(files))
  File<EMAIL_ADDRESS>line 579, in check_call
    retcode = call(*popenargs, **kwargs)
  File<EMAIL_ADDRESS>line 560, in call
    with Popen(*popenargs, **kwargs) as p:
  File<EMAIL_ADDRESS>line 950, in __init__
    restore_signals, start_new_session)
  File<EMAIL_ADDRESS>line 1483, in _execute_child
    restore_signals, start_new_session, preexec_fn)
UnicodeEncodeError: 'ascii' codec can't encode character '\u2019' in position 104: ordinal not in range(128)
ash-4.3#

is this due to new rclone or do i have something locally broken?


Could not deploy this app to heroku

$ git push heroku dev:master

Counting objects: 33, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (26/26), done.
Writing objects: 100% (33/33), 2.81 KiB | 0 bytes/s, done.
Total 33 (delta 12), reused 0 (delta 0)
remote: Compressing source files... done.
remote: Building source:
remote:
remote: -----> Node.js app detected
remote:
remote: -----> Creating runtime environment
remote:
remote:        NPM_CONFIG_LOGLEVEL=error
remote:        NPM_CONFIG_PRODUCTION=true
remote:        NODE_ENV=production
remote:        NODE_MODULES_CACHE=true
remote:
remote: -----> Installing binaries
remote:        engines.node (package.json):  unspecified
remote:        engines.npm (package.json):   unspecified (use default)
remote:
remote:        Resolving node version (latest stable) via semver.io...
remote:        Downloading and installing node 5.10.0...
remote:        Using default npm version: 3.8.3
remote:
remote: -----> Restoring cache
remote:        Skipping cache restore (new runtime signature)
remote:
remote: -----> Building dependencies
remote:        Pruning any extraneous modules
remote:        Installing node modules (package.json)
remote:
remote:        ><EMAIL_ADDRESS>install /tmp/build_7142a4220421d6ea6b117ccf7865cb41/node_modules/contextify
remote:        > node-gyp rebuild
remote:
remote:        make: Entering directory `/tmp/build_7142a4220421d6ea6b117ccf7865cb41/node_modules/contextify/build'
remote:        CXX(target) Release/obj.target/contextify/src/contextify.o
remote:        SOLINK_MODULE(target) Release/obj.target/contextify.node
remote:        COPY Release/contextify.node
remote:        make: Leaving directory `/tmp/build_7142a4220421d6ea6b117ccf7865cb41/node_modules/contextify/build'
remote:
remote:        ><EMAIL_ADDRESS>postinstall /tmp/build_7142a4220421d6ea6b117ccf7865cb41
remote:        > typings install && npm run clean-ng2 && gulp check.versions && gulp postinstall
remote:
remote:        sh: 1: typings: not found
remote:
remote:        npm ERR! Linux 3.13.0-79-generic
remote:        npm ERR! argv "/tmp/build_7142a4220421d6ea6b117ccf7865cb41/.heroku/node/bin/node" "/tmp/build_7142a4220421d6ea6b117ccf7865cb41/.heroku/node/bin/npm" "install" "--unsafe-perm" "--userconfig" "/tmp/build_7142a4220421d6ea6b117ccf7865cb41/.npmrc"
remote:        npm ERR! node v5.10.0
remote:        npm ERR! npm  v3.8.3
remote:        npm ERR! file sh
remote:        npm ERR! code ELIFECYCLE
remote:        npm ERR! errno ENOENT
remote:        npm ERR! syscall spawn
remote:        npm ERR<EMAIL_ADDRESS>postinstall: `typings install && npm run clean-ng2 && gulp check.versions && gulp postinstall`
remote:        npm ERR! spawn ENOENT
remote:        npm ERR!
remote:        npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'typings install && npm run clean-ng2 && gulp check.versions && gulp postinstall'.
remote:        npm ERR! Make sure you have the latest version of node.js and npm installed.
remote:        npm ERR! If you do, this is most likely a problem with the angular2-seed package,
remote:        npm ERR! not with npm itself.
remote:        npm ERR! Tell the author that this fails on your system:
remote:        npm ERR!     typings install && npm run clean-ng2 && gulp check.versions && gulp postinstall
remote:        npm ERR! You can get information on how to open an issue for this project with:
remote:        npm ERR!     npm bugs angular2-seed
remote:        npm ERR! Or if that isn't available, you can get their info via:
remote:        npm ERR!     npm owner ls angular2-seed
remote:        npm ERR! There is likely additional logging output above.
remote:
remote:        npm ERR! Please include the following file with any support request:
remote:        npm ERR!     /tmp/build_7142a4220421d6ea6b117ccf7865cb41/npm-debug.log
remote:
remote: -----> Build failed
remote:
remote:        We're sorry this build is failing! You can troubleshoot common issues here:
remote:        https://devcenter.heroku.com/articles/troubleshooting-node-deploys
remote:
remote:        Some possible problems:
remote:
remote:        - Node version not specified in package.json
remote:        https://devcenter.heroku.com/articles/nodejs-support#specifying-a-node-js-version
remote:
remote:        Love,
remote:        Heroku
remote:
remote:
remote:  !     Push rejected, failed to compile Node.js app
remote:
remote: Verifying deploy...
remote:
remote: !       Push rejected to oama.
remote:
To https://git.heroku.com/oama.git
 ! [remote rejected] dev -> master (pre-receive hook declined)
error: failed to push some refs to 'https://git.heroku.com/oama.git'

we don't know how heroku work and form your error log typings not found so you need to install typings npm install typings --global


[WIP] Add basic documentation for side games

This is an early PR just to have feedback on what info to include, layout etc. Atm only the build list is planned for the other games as I don't know much about the file types the Nintendo DS uses. If you have a docs about it feel free to just add them to the PR.
For the DS games I'm gonna do 1 index.md file with links to common info and game specific info because those 2 games are so similar internally.

Excited for this one :)

I didn't have time to look into the file types for the DS games. So it's mostly builds, some link fixes and basic layout.


mempool: trap on double-free instances

A double-free could cause very hard to find bugs when using the mempool
allocator as the same memory would end up being allocated twice
afterwards.
Now that bits in the block bitmap are cleared only when actually freeing
a block, we may simply ensure those bits are still set before clearing
them, effectively catching most double-free cases.
The alloc_bit_is_set() function is made static inline so that when
assertion checks are disabled the compiler won't complain about unused
code.

Hmm, looks like this patch may have found some stuff based on the posted CI failure.
Run tests under tests/kernel/pipe/ to see what I mean

On Fri, 4 Oct 2019, Andrew Boie wrote:

Hmm, looks like this patch may have found some stuff based on the posted CI failure.
Run tests under tests/kernel/pipe/ to see what I mean

I filed #19535 already, hoping that someone more knowledgeable with
pipes would check it out.
In the mean time, this CI failure shouldn't prevent the merging of
this PR. This is not creating a bug but rather exposing an existing one.

In the mean time, this CI failure shouldn't prevent the merging of
this PR. This is not creating a bug but rather exposing an existing one

Unfortunately, unless the test is also temporarily disabled, simply merging this will cause CI to be in a non-passing state, which we can't have.
However, in this case the fix is easy; the extra free in the test needs to be removed.

Hmm, after looking at the pipe code some more, I am less sure. But at any rate we need to either disable the test or fix the issue

The double-free issue in CI is now fixed, and I just pushed a slightly
more efficient version that avoids the (1 << bit) construct. That won't
make MISRA happy about the rest of the file, but that really ought to be
addressed in a separate PR.


Improve performance of getting parsed data

It's more efficient to do an array access per data point than a function call per data point
I didn't add to the migration guide since it's a protected method new in 3.0

The engine probably inlined that function already, so probably makes no difference at all.
Readability and maintainability should be considered before this kind of optimization. That said, this change does not make a huge difference. 5 extra lines of probably no real value. Did you try to measures it?

Yeah, I was debating this one. I'll go ahead and close it


Use new Azure Pipelines domain in build badges

Azure Pipelines now uses https://dev.azure.com/pypa/pipenv instead of https://pypa.visualstudio.com/pipenv (even though redirects will work for some time).  This updates the build badges to use that new domain.

LGTM, merging! Interestingly, visualstudio.com does not actually redirects, only shows an identical page. Is this intentional? (Just curious.)


Raw morse input

Allow the user to paste a series of .-/ as input - for instance, if it was copied from a puzzle - and validate and parse it as if the user had entered with the buttons.

Has this come up much? I feel like most of the time there isn't Morse just presented directly and I have to transcribe anyway.


moving content for volume mounts from cf docs into spec

The spec for the volume_mounts field in the bind response was documented as a separate file, then removed from the CF docs by accident so it was missed when we migrated the spec.
We have since added the doc back to the CF docs: http://docs.cloudfoundry.org/services/volume-services.html. Once this is merged, we'll remove the doc from the CF docs. I will submit a PR to the 2.12 branch as well.

LGTM

LGTM

lgtm

one more l.g.t.m needed

LGTM


Add Heroku rollback instructions to the deployment.md

https://github.com/opencollective/opencollective/issues/4477

We need to run prettier

Thanks. Missed that one. 👍🏽


Update Dependencies

...in preparation for the 1.7.0 release (#369).

Release binary size increased from 3.66 MB to 3.80 MB. Largest 10 crate-level contributors after the updates:
$ cargo bloat --release --crates -n 10
    Finished `release` profile [optimized] target(s) in 0.06s
    Analyzing target/release/tldr

 File  .text     Size Crate
 8.8%  27.3% 704.1KiB std
 4.2%  13.0% 336.3KiB rustls
 3.1%   9.5% 244.5KiB clap_builder
 2.6%   8.0% 207.0KiB ring
 1.9%   5.9% 151.4KiB toml_edit
 1.6%   4.9% 127.3KiB tldr
 1.5%   4.5% 116.9KiB [Unknown]
 1.2%   3.6%  92.8KiB reqwest
 1.0%   3.2%  82.3KiB hyper
 0.8%   2.4%  61.9KiB hyper_util
 5.4%  16.7% 430.4KiB And 51 more crates. Use -n N to show more.
32.5% 100.0%   2.5MiB .text section size, the file size is 7.8MiB

Note: numbers above are a result of guesswork. They are not 100% correct and never will be.

RusTLS is quite significant, but I think having it bundled is worth it because it gets rid of many platform-specific issues.
A release build with --no-default-features --features native-tls has 2.61 MB, which is significantly smaller. If some folks want to have a small binary and have a proper TLS provider on their system, that is a good option.


Move Linux Jumpbox from 14.04 to 16.04

While deploying the shared services environment the linux jumpbox had issues
running cloud-init to get the initial provisioning information.
Moving to 16.04 resolved this issue.
14.04 reached end of support this month as well, so probably a good thing
to move away from.
Signed-off-by: Graham Hayes<EMAIL_ADDRESS>

We are making an update to all the existing modules to add:

Logging & Monitoring
Update management
Increase the VM size
Use latest ubuntu version (18)

It'll be complete in a few more days.


Ldap groups phase1

Please make sure you've read the pull request section of our code contribution guidelines.
When filling in a section please remove the help text and the above text.
Summary
[A brief description of what this pull request does.]
Ticket Link
[Please link the GitHub issue or Jira ticket this PR addresses.]
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Ran make check-style to check for style errors (required for all pull requests)
[ ] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)
[ ] Needs to be implemented in mobile (link to PR or User Story)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[ ] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

@hmhealey Thanks! I'll try to address all this stuff during today, to try to merge to master tomorrow, if it is possible :)

@hmhealey I reviewed all your comments and fixed them. Thanks a lot, a very detailed review! :1st_place_medal:

Does this need to have the version of mattermost-redux updated before it can be merged?

@hmhealey Once the other related PRs are approved I will update Redux and merge them all at once.


Last test fails to recognize 3 required calls to random number code

Challenge http://www.freecodecamp.com/challenges/waypoint-create-a-javascript-slot-machine has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
I am unclear on why this is not passing the final test case, as all three slots are correctly populated with a random number

 slotOne = Math.floor(Math.random() * (3-1+1) + 1);
  slotTwo = Math.floor(Math.random() * (3-1+1) + 1);
  slotThree = Math.floor(Math.random() * (3-1+1) + 1);

I reset the challenge and now it passes all tests.
There must be some underlying bug in the validation code that was stuck in a non-passing state.

Oddly enough I'm having this problem on my mobile phone...my best guess for my situation is that the browser doesn't support .floor


Remove Unnecessary Data

The NLDI system only uses a subset of the information being processed. Tables, columns, and plpgsql code not required for the NLDI navigation process can be removed.

@EthanGrahn -- do you think this issue is worth keeping around?

@dblodgett-usgs I'd need more specifics on which is considered not necessary. It's not all being used for navigation, but we use it for other queries. I think we could close this.


Esc 296 already have undertaking

ticket details -> https://jira.tools.tax.service.gov.uk/browse/ESC-296

@carwynellis All the comments are addressed here


bug fix in base iscached

fixed a typo
added tests for caching and uncaching

Nice fix @boazmohar !


fix live tl showing on video that is upcoming

This ensures that videos which are upcoming won't have live TL showing (the live TL button won't show, either). It will appear when the video status changes to live.

Actually my bad, I forgot these values can be toggled and we need initial values. In that case it's a bit tricky because video is not defined at initialization...
You might need to separate the variables to a new toggleable one that is button controlled, and then add that to the computed (showTL)

Yeah, I realised that too. My focus is not that great today. I tested it out a bit more, and it's just not behaving correctly. I made the live TL always closed by default, and always show the live TL button (as long as hasLiveChat is true, which isn't true in mugen for example)

And once again, forgot about MugenClips. Live TL button only shows on videos with video status live or upcoming.

To summarize the changes in this PR:

Live TL is closed by default on all platforms
Live TL button is only available on videos that actually have live TL, which means:

It doesn't show in MugenClips
It doesn't show in past videos
It shows in videos that are upcoming
It shows in videos that are live
Hopefully that's everything


Adds ActiveHash#find_by functionality

Functionality similar to ActiveRecord::FinderMethods#find_by (https://github.com/rails/rails/blob/a8827cb9afdf122c003fbff1bfcc28bda2994d04/activerecord/lib/active_record/relation/finder_methods.rb#L82).
Includes updated specs.

This functionality looks to be merged. please re open if I'm mistaken
Thanks


Display of contact emails in Project summary page

As a corresponding contributor to a project, I do not want my email address to be scrapable in the Browser by potentially malicious tools.

Currently, corresponding contributor emails are viewable in a form that could easy be scraped by tools. They should be masked somehow.
┆Issue Number: HCAB-311

It's not easily scrapable because it's not embedded in javascript or HTML served by the server, it's in the JSON response returned by Azul and dynamically rendered on the client.


[Feature][Connector]add s3file save mode function

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.


[BUG]  Insert link without selection returns nil error

Please confirm

[x] I am running the latest version of this plugin

Describe the bug
When executing insert_link and nothing is selected in Telescope, this error is shown:
E5108: Error executing lua: ...local/share/nvim/lazy/telekasten.nvim/lua/telekasten.lua:1284: attempt to index local 'selection' (a nil value)
stack traceback:
        ...local/share/nvim/lazy/telekasten.nvim/lua/telekasten.lua:1284: in function 'run_replace_or_original'
        ...re/nvim/lazy/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
        ...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:257: in function <...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:256>

To Reproduce
Steps to reproduce the behavior:

Start nvim
require'telekasten'insert_link()
type something so that the fuzzy picker will show 0 results and press enter
See error

Expected behavior
Not sure? But my suggestions would that the note is created if it does not exists, or a link is created that points to the non-existing note, so later we can create it.
Screenshots

Operating system (please complete the following information):

OS: osx

Additional context
Add any other context about the problem here.

When executing insert_link and nothing is selected in Telescope, this error is shown

I confirm that I can reproduce this bug.

But my suggestions would that the note is created if it does not exists, or a link is created that points to the non-existing note, so later we can create it.

I believe this was the original behavior yes. I will have a look at it. Thank you for reporting that.

All right! When nothing is selected, this should use what was typed in the prompt for the link. The file will not be created straight away, but if you use follow_link it will then create it automatically as usual.

can confirm it works, great job and quick fix :)


Define Scene Configuration as a Dictionary

I should add Scene Configuarion as an empty Dictionary to not receive this warning
Info.plist contained no UIScene configuration dictionary (looking for configuration named "(no name)")?
With the following definition, I'm not able to make it.
 info:
      path: FOLDER/Info.generated.plist
      properties:
        CFBundleDisplayName: "APPNAME"
        CFBundleIconName: Assets.xcassets/AppIcon.appiconset
        CFBundleName: APPNAME
        CFBundleShortVersionString: 1.0.0
        CFBundleVersion: "010000"
        UIApplicationSceneManifest:
          UISceneConfigurations:

Do you have any ideas or suggestions?
Related link

This structure worked for me:

In yaml if you want an empty dictionary use {}
UIApplicationSceneManifest:
    UISceneConfigurations: {}


[Bug]: Permanent redirection for https does not work

Description
I‘m trying to get a 301 redirection instead of a 302 for the http to https redirection for a static service.
I added this to the Traefik labels:
traefik.http.middlewares.redirect-to-https.redirectscheme.permanent=true
But then the redirect isn’t working at all and I get a 404 instead if I access the site with http.
Minimal Reproduction (if possible, example repository)

Create a static service
Enable http > https redirection
Add the label provided above
Redeploy
Visit the http url

Exception or Error
No response
Version
beta.323
Cloud?

[X] Yes
[ ] No

I solved this. This has to be added to all services because that’s a global rule and then there is a mismatch.
Can this label be moved out of the service definition into the global config? Or should it be scoped to each service?

The redirect should probably be permanent by default.
Does a non default redirection from http to https even makes sense for anything?


Add function update

Adding the function listed in the title would be beneficial for development processes with vide.
update(instance: Instance)
Updates an existing instance by applying the properties and states passed.
Type
type Properties = Map<string|number, any>

function update(instance: Instance): (Properties) -> nil

Details

It follows the same rules as creating a property but it simply just updates an existing instance passed with those properties.

Example
local frame = script.Parent
print(frame.Name) --> "Frame"

update(frame) {
    Name = "SomeFrame"
}

print(frame.Name) --> "SomeFrame"

There are some issues and concerns I have that need to be addressed before
adding this function:

The name update() is a bit overloaded in the context of this library and
could cause some confusion, a better name could be hydrate() or apply().

What should the behavior be in the following case?
local label = Instance.new("TextLabel")

local countA = source(0)
local countB = source(100)

update(label) {
    Text = countA
}

update(label) {
    Text = countB
}

What are the use-cases for this function?
Currently create() can accept an instance to clone and apply properties to
which serves the purpose of allowing us to use pre-made instances in the same
way as Vide-made instances and avoiding the issue in 2..`