Bionic Text

Problem/Concern
There is an opportunity to enhance the product to include an accessibility feature for Neurodivergent Folks.

link to problem content

Proposed Solution
Create the solution to have Bionic Text as an accessibility feature.

@lwnoble had discussed this a little in our 09/05/24 Theme Builder status call.  The idea here is to introduce support for Bionic Reading (https://bionic-reading.com/br-method/) in Theme Builder.  Lise thinks that this could possible be done as an overlay (similar to the dyslexic overlay) if bionic reading fonts exist.  Otherwise, we talked about introducing a component that could render text in a Bionic Reading format.  In the latter case, its usefulness would be limited to only those who use our components in their applications.

@lwnoble believes that this needs to be a component so overlay won't be a possibility.
Because no studies have been concluded that endorse Bionic Reading, we should maybe do some more research on this prior to implementing a component.


u64 might not be enough for higher difficulty

Would you  consider to use U256?
    public struct Block has key, store {
        nonce: u64,
        ...,
    }

@neeboo good point, I was thinking about this too. But take in account that there's payload and meta involved in hashing. So if u64 will be not enough - you can just push something there  (actually anything) - and try nonces again in the u64 range. So we are ok here. BTC has the same and it's ok just because of this - payload.


fix: fixed GraphQL Access query resolver to return the correct data

Description
Fixes #1338
This fixes the object returned from the resolver and also fixes how the resolver is registered.
I used the graphql-playground to verify the fix. I'm unsure if an automated test should be set up in access-control, auth, or in a completely new test directory.

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

Hey @nborko thank you very much for the PR!
Merged!


Add support for card-mod, in order to customize font-size, or set font-size directly

Hi! Great card.
Using card-mod, I'm trying to reduce font size in the list:
style: |
  ha-card {
    font-size: 10px;
  }

But it doesn't have any effect.
However in your list with tracks I see smaller fonts, how did you do it?

Thanks!
I don't alter the font-size of the items so I guess it's the default font-size.
Using card-mod I successfully can change the font-size by using:
style: |
  paper-item {
    font-size: 10px;
  }

Cool, thanks.
What parameter should I change to decrease the height of a line?

I'm not at my pc right now, I guess the height and line-height properties of the paper-item.
Since I use a hardcoded value to calculate height for the overflow this will give issues when you alter the height via card-mod.
Maybey I'll make css variable for it, otherwise a config option.

That would be so cool.

Just a quick note from the latest HA release:

The theme variable paper-card-background-color is removed. You can use ha-card-background or card-background-color as a replacement.
In general, all variables that start with paper will be removed at some point.


Hotfix for panic when config dir does not exist

This PR fixes the issue where the config file was written to a directory that may not exist, leading to a panic.  This commit creates the directory if it does not exist and improves error handling if something goes wrong, avoiding the panic.

This should resolve #19


Remove history truncation

Remove history truncation. As far back in the Go SDK as I can find we have this code to try to truncate any events past atWorkflowTaskCompletedEventID. It is not clear what the point of this code is and it has a number of bugs. Some of the issues are:

It can panic if h.Events[0].GetEventId() > atWorkflowTaskCompletedEventID
It is trying to check if the last event in history after truncation is a EVENT_TYPE_WORKFLOW_TASK_COMPLETED, but atWorkflowTaskCompletedEventID is from the workflow tasks GetStartedEventId so it should be a EVENT_TYPE_WORKFLOW_TASK_STARTED.
If history does go beyond GetStartedEventId then that indicated this workflow task is stale and will be rejected by the server anyway

Note: The cadence team also pointed out the strangeness of this code https://github.com/uber-go/cadence-client/blob/4d4c09f5053b53caea43100e170c327058731f5c/internal/internal_task_pollers.go#L961

If history does go beyond GetStartedEventId then that indicated this workflow task is stale and will be rejected by the server anyway

What if there is a signal sent between when we first got partial history from PollWorkflowTaskQueueResponse and started iterating to get the rest? Wouldn't then the signal be past the last task completed we expected? I wonder if this can be somehow tested by having a history too large to fit in PollWorkflowTaskQueueResponse.history and have first history page iteration be held up, then send a signal, then let history iteration continue.

Wouldn't then the signal be past the last task completed we expected

When we are processing a new workflow task we never expect any event past the last workflow task started event because the server will buffer the signal

When we are processing a new workflow task we never expect any event past the last workflow task started event because the server will buffer the signal

:+1: So to confirm, server guarantees that if a worker is processing a task, and that task is still valid, fetching history will never contain any events after the task being processed?


"TÂRGU-NEAMȚ", "Blebea", "Humulești" and "Humuleștii Noi" are listed in the drop-down menu

Description:
"TÂRGU-NEAMȚ", "Blebea", "Humulești" and "Humuleștii Noi" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "BICAZ" in the search bar.

Expected results
The location is listed in the drop-down menu as "Oraș TÂRGU-NEAMȚ, NEAMȚ (TÂRGU-NEAMȚ)" and "Oraș TÂRGU-NEAMȚ, NEAMȚ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Blebea" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Blebea, NEAMȚ (TÂRGU-NEAMȚ)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Humulești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Humulești, NEAMȚ (TÂRGU-NEAMȚ)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Humuleștii Noi" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Humuleștii Noi, NEAMȚ (TÂRGU-NEAMȚ)".


Tabnine still "STARTING" (for hours)

OS version: Win 10
Editor: vscode-stable
Editor version: March 2021 (version 1.55)
Programming language: js, php
TabNine extension version: v3.2.16
Engine version (could be located within Tabnine Hub): cannot open

Issue Details:
I have a problem with tabnine for 2 days, it's only "starting" when I run VSCODE:

But in Jetbrains PHP Storm it works fine.
I made no changes, no settings, nothing. Only things I ussually do is to open VSCODE, there is update very often and then all my extensions load.
I tried to restart every app, disable enable extension, reinstall extension, restart PC - nothing work

My coworker with macOS has the same issue

We had an issue that could cause this in the engine version 3.3.103. It is possible that you are stuck in that version.
Please remove tabnine & its folder under vscode (which is under %USERPROFILE%\.vscode\extensions) and reinstall.
Let me know if that didn't work for you.

We had an issue that could cause this in the engine version 3.3.103. It is possible that you are stuck in that version.
Please remove tabnine & its folder under vscode (which is under %USERPROFILE%\.vscode\extensions) and reinstall.
Let me know if that didn't work for you.

I confirm, this works.

Thanks :-)

We had an issue that could cause this in the engine version 3.3.103. It is possible that you are stuck in that version.
Please remove tabnine & its folder under vscode (which is under %USERPROFILE%\.vscode\extensions) and reinstall.
Let me know if that didn't work for you.

Hey, I can't clear your solution. Please explain that. :( My vscode editor inside not showing tabnine logo and there create a responsive performance issue. How can I fix it?

Great


⚠ Bofrost.be has degraded performance

In 2cdfb69, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1595 ms

Resolved: Bofrost.be performance has improved in 051cec4 after 6 minutes.


Extend custom tooltip example with more values

Apex Charts tooltips are very powerful and I like that we can customize them with the Apex.tooltip.custom method. A regular Apex Charts tooltip contains lots of information in it, however in the custom tooltip example you have (https://codepen.io/apexcharts/pen/NBdyvV) you only show how to get the current value which is the easiest to find. I tried to find the category names but the w object is so huge it's very hard to find the info you need there. It would be great if you could extent the custom tooltip example with all the info you have in the regular tooltip.

Sure, I will update the example.

Updated the codepen.
X-values or categories are accessible in w.globals.labels
tooltip: {
  custom: function({ series, seriesIndex, dataPointIndex, w }) {
    return (
      '
' +
      '' +
      w.globals.labels[dataPointIndex] +
      ': ' +
      series[seriesIndex][dataPointIndex] +
      '' +
      '
'
    )
  }
}

Thanks for that. Problem with this approach is that when you switch to line charts, category names will be presented as numbers: https://codepen.io/Deka87/pen/YzPExYx?editors=0010. Any workaround for this?

Hi there! First of all I want to say that ApexCharts is amazing and allow you make so great things custom.
In this case i have some issues for my custom tooltip...
I'm trying to add an arrow at the bottom of the tooltip, but the problem is that I can't center the tooltip when I mouse over the series data on the right side of the chart... Can you help me please.
I know other graphing tools use the property yAling:bottom.
By the way I'm applying ApexCharts  "apexcharts": "^3.35.5", "react-apexcharts": "^1.4.0" with React.js.
Finally i can't apply the mode and behavior:
highlightingMode="FadeOthersSpecific"
highlightingBehavior="NearestItemsAndSeries"
Thanks in advance for all the support you can give me.

tooltip: {
theme: 'light',
x: {
show: false,
},
custom: function ({ series, seriesIndex, dataPointIndex, w }: any) {
const month = w.globals.labels[dataPointIndex];
const value = series[seriesIndex][dataPointIndex];
const color = w.globals.colors[dataPointIndex];
const circleColor = w.globals.stroke.colors[seriesIndex]; // Use stroke color for the circle
      return (
        '
' +
        '

' +
        "" +
        month +
        ": " +
        value.toFixed(2) +
        "%" +
        "
"
      );
    }
  }

  // css
  .arrow_box {

position: relative;
background: #fff;
/* border: 2px solid #000000; */
margin: 5px 10px 5px 10px;
display: flex;
align-items: center;
}
.arrow_box:after, .arrow_box:before {
right: 100%;
top: 50%;
border: solid transparent;
content: " ";
height: 0;
width: 0;
position: absolute;
pointer-events: none;
}
.arrow_box:after {
border-color: rgba(85, 85, 85, 0);
/* border-right-color: #555; /
/ border-width: 10px; /
margin-top: -10px;
}
.arrow_box:before {
border-color: rgba(0, 0, 0, 0);
/ border-right-color: #000000; /
/ border-width: 13px; */
margin-top: -13px;
}
.circle {
width: 10px;
height: 10px;
border-radius: 50%;
margin-right: 8px;
}


canvas.getContext on IE

Fix according to https://caniuse.com/#feat=webgl

Thank you, I will add this in the next release. Travis was executed with an error, I can not merge it.

@kutuluk do you still develop this project?


ncgen.exe on Windows fails with trailing '-o' argument.

While debugging issue #103, I've discovered that the following command fails on Windows:
$ ncgen -b small.cdl -o output.nc
g:\netcdf-c\build\ncgen\ncgen.exe: only one input file argument permitted

Whereas the following command works properly:
$ ncgen -o output.nc -b small.cdl

This may be related to the XGetOpt code (which replaces the absent getopt in Visual Studio)

That sounds right, the option processor
is assuming that all flags precede all non-flag
arguments, so it treats -o output.nc as two non-flag
arguments.
Not clear if it is worth fixing.
=Dennis
Ward Fisher wrote:

While debugging issue #103, I've discovered that the following command fails on Windows:
$ ncgen -b small.cdl -o output.nc
g:\netcdf-c\build\ncgen\ncgen.exe: only one input file argument permitted

Whereas the following command works properly:
$ ncgen -o output.nc -b small.cdl

This may be related to the XGetOpt code (which replaces the absent getopt in Visual Studio)

Reply to this email directly or view it on GitHub:
https://github.com/Unidata/netcdf-c/issues/104

Since the behavior on Linux is different, I'd be inclined to bring them in to sync. Since the Linux version is more flexible I'll argue that that's the behavior we should standardize with, although I'll agree it's not a high priority.
Sent from my iPhone

On Jan 28, 2015, at 12:50 PM, DennisHeimbigner<EMAIL_ADDRESS>wrote:
That sounds right, the option processor
is assuming that all flags precede all non-flag
arguments, so it treats -o output.nc as two non-flag
arguments.
Not clear if it is worth fixing.
=Dennis
Ward Fisher wrote:

While debugging issue #103, I've discovered that the following command fails on Windows:
$ ncgen -b small.cdl -o output.nc
g:\netcdf-c\build\ncgen\ncgen.exe: only one input file argument permitted

Whereas the following command works properly:
$ ncgen -o output.nc -b small.cdl

This may be related to the XGetOpt code (which replaces the absent getopt in Visual Studio)

Reply to this email directly or view it on GitHub:
https://github.com/Unidata/netcdf-c/issues/104
—
Reply to this email directly or view it on GitHub.

I would suggest a Jira issue for it so we don't forget.
=Dennis
Ward Fisher wrote:

Since the behavior on Linux is different, I'd be inclined to bring them in to sync. Since the Linux version is more flexible I'll argue that that's the behavior we should standardize with, although I'll agree it's not a high priority.
Sent from my iPhone

On Jan 28, 2015, at 12:50 PM, DennisHeimbigner<EMAIL_ADDRESS>wrote:
That sounds right, the option processor
is assuming that all flags precede all non-flag
arguments, so it treats -o output.nc as two non-flag
arguments.
Not clear if it is worth fixing.
=Dennis
Ward Fisher wrote:

While debugging issue #103, I've discovered that the following command fails on Windows:
$ ncgen -b small.cdl -o output.nc
g:\netcdf-c\build\ncgen\ncgen.exe: only one input file argument permitted

Whereas the following command works properly:
$ ncgen -o output.nc -b small.cdl

This may be related to the XGetOpt code (which replaces the absent getopt in Visual Studio)

Reply to this email directly or view it on GitHub:
https://github.com/Unidata/netcdf-c/issues/104
—
Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub:
https://github.com/Unidata/netcdf-c/issues/104#issuecomment-71904406

Maybe we should consider alternatives to Xgetopt.
Here are two I found:

https://svn.apache.org/repos/asf/qpid/proton/trunk/examples/include/pncompat/internal/getopt.c
http://research.microsoft.com/en-us/um/redmond/projects/invisible/src/games/ping/getopt.c.htm

Sounds good to me. The Apache version seems preferable at a glance; the MS version has a license that seems to imply any time we talk about NetCDF we need to mention that it uses software from UC Berk (clause 3).  I'll see about merging in the former once I get the cdf5 stuff wrapped up.


fix: [lw-11808]: set min width for assets icons

Checklist

[ ] JIRA - LW-11808
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Allure Report
allure-report-publisher generated test report!

processReports: ✅ test report for ee171b6d

passed
failed
skipped
flaky
total
result

Total
32
0
4
0
36
✅


PHOENIX-6054 Phoenix-spark connector doesn't work with phoenix master…

… branch

There is a minor code java change compared to the existing phoenix-spark connector, because of the different Hadoop version. So I created it as a new module.
Extra Jackson dependency has been added and I disabled testOrderByNullable test in BaseOrderByIT because it was using a SYSTEM table and spark does not allowed to create a temporary view on that.

I am going to close this PR because I am handling phoenix-spark part of the PHOENIX-6057.


builder error: URL scheme is not allowed

I'm not sure if this is a bug report or feature request, but when I try to build an rss feed from an xml file I'm getting the following error:
ERROR rsspls       > error processing feed for file:///[local path]/A-245-24.xml
Caused by:
0: unable to fetch file:///[local path]/A-245-24.xml
1: builder error for url (file:///[local path]/A-245-24.xml): URL scheme is not allowed
2: URL scheme is not allowed
My situation is this: I'd like an RSS feed of court filings in certain proceedings before the Federal Court of Canada . You can look these up by court file number on the court website, but it does this through some javascript stuff, and essentially just feeds the browser all of the necessary information through an easy to identify JSON GET request.
I've set up a crontab to pull the JSON to a local file, then transmute JSON to XML, then I tried to have rsspls create a feed out of the resulting XML, with the error message above as the result.
In case more specifics would help, the URL for the JSON GET is:
https://www.fct-cf.gc.ca/CourtFilesAndDecisions/proceedingQueriesRE?division=A-245-24&courtnumber=A-245-24
And then I'm just  trying to do:
item = "data"
heading = "RECORDED_ENTRY"
summary = "DOCNO"
date = "FILING_DATE"
Thanks!

Do you have file-urls enabled at the top of the config file? E.g.
[rsspls]
output = "/some/path"
file_urls = true

Yes, I do

Hmm it's going down the http code path, not the file path for some reason. Just to confirm some things:

What OS are you using and now did you install rsspls?
What's the output of rsspls --version?

I'm using Ubuntu v24.04.01. It's been a while since I installed rsspls but it looks like I just downloaded the linux binary and put it into a contrib/bin type directory in $PATH.
$ rsspls --version
rsspls version 0.8.1
Looks like the current version is 0.10 based on the website so maybe that is my problem, I'll give an "upgrade" a shot.

Yes, that seemed to fix this problem! I'm now getting a properly formed feed with no items, which still was not the desired output, but that is presumably also something on my end. Thanks! Incidentally I don't suppose there is an rss feed to follow new versions of rsspls so I don't fall behind again, is there?

Yes, getting a weird error there but I'll open a new issue since it's unrelated.

Yes, that seemed to fix this problem!

Right, the file url support was only added in the 0.10 release.


Added tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@da6fa54). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #1   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?        42
  Lines             ?      1001
  Branches          ?         0
==========================================
  Hits              ?      1001
  Misses            ?         0
  Partials          ?         0

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 da6fa54...5665514. Read the comment docs.


pbccs4.0.0 Error when input bax2bam results.

ccs --version
ccs 4.0.0 (commit v4.0.0)
ccs bax2bam.subreads.bam ccs.bam

|> 20190917 01:56:17.433 -|- FATAL -|- ValidateChemistries -|- 0xfc5940|| -|- unsupported sequencing chemistry combination:
binding kit:        100356300
sequencing kit:     100356200
basecaller version: <IP_ADDRESS>.140640

But I tried ccs 3.4.1 (commit c1a7cc8) without this error.

This is known and on purpose.
https://github.com/PacificBiosciences/ccs/blob/develop/README.md#help-i-am-getting-unsupported-

Old chemistries: With ccs 4.0.0, we have removed support for the last RSII chemistry P6-C4. The only option is to downgrade ccs with conda install pbccs==3.4.


Support pipelines

Sometimes, I want to execute a shell command from within gitsh and the ! prefix makes this very convenient. Sometimes, the shell command I want to execute includes a pipeline and when I try executing it with ! it fails with a parse error:
pick master@ !git log | wc -l
gitsh: parse error

I've read the man page, the README and the parser code but can't figure out if there's a way to execute a pipeline.
If there isn't, would you be interested in supporting this? If so, I'd be happy to take a stab at implementing it. Do you have any pointers for how to best approach implementing this?

It's not currently possible to use any kind of pipeline in gitsh.
I'd definitely be interested in supporting this, but I think we should look at the possibility of gitsh natively supporting pipelines rather than leaning on an external shell for them. That'd allow for pipelines that mix Git commands, gitsh commands, and shell commands. You example would probably be re-written as:
pick master@ log | !wc -l

The parser already supports the ||, &&, and ; operators to join multiple commands together, which the transformer turns into the relevant subclass of Gitsh::Commands::Tree::Branch, depending on the operator. We could probably use a similar structure could be used to support the | operator, perhaps producing a Gitsh::Commands::Pipeline object which could do something with IO.pipe and Process.spawn to connect up the pipeline.
What do you think?

Closing in favour of #299.


Some TUI programs does not work well

Mprocs can not send "Ctrl+h" signal to the programs running in it:

("Ctrl+h" should open Help in termusic, but it does not work while termusic is running in mprocs)
(However, other signals like "Alt+h" are ok.)
And cursors in nvim does not show in correct ways:
first picture: nvim INSERT mode cursor while nvim is not running in mprocs(correct):

second picture: nvim INSERT mode cursor while nvim is running in mprocs(incorrect, the cursor at line 21):

my guicursor config:
vim.o.guicursor="n-v-c-sm:block-nCursor-blinkwait100-blinkon200-blinkoff300,i-ci-ve:ver25-iCursor,r-cr-o:hor20"
Platform
Windows 11 21H2 22000.856
Terminal Emulator
Windows Terminal 1.1422.82.0
mprocs version
0.6.3
termusic version
0.7.3
nvim version
0.7.2

Thank you for reporting the issues.
I've been investigating the issue with the <C-h> key and still can't find a way to encode <C-h>. I created an issue to track thick problem separately.
As per the cursor problem, the cursor shape is not implemented yet.

 can be used in termusic running in mprocs(0.6.4) now.


RFC: Mitigating DST troubles

For some reason, I've lately been thinking a lot about the troubles we had earlier this year with the gigasecond exercise, and I've come up with some ideas about detecting and preventing future issues with that or perhaps with future exercises (todo: make list of other datetime exercises).
Now that we have exercise/example testing setup, the first is simply to schedule a TravisCI build for the DST switch dates in the US. If the tests fail for some reason, the scheduled build should let us know almost as soon as can be known. I poked around the TravisCI settings and didn't see anything like this, so I thought about setting up an IFTTT solution (I'm pretty sure I have an account, but I don't remember the password, as I don't think I've ever used it). Any other suggestions welcome.
The second is that it's seems possible that perhaps only a few CL implementations could be affected by the DST switch due to a bug in time handling. It would be nice to figure out how to conditionally allow failures on a per-test-case basis, so that even if there's a problem with the DST affected exercise tests on that implementation, we can still get feedback on the other exercises until the issue is resolved (or until the switch back).
If more serious problems turn up, we could also look for a date-time library for CL that's well maintained and, if that works out, recommend it to users.
Lastly, I think we should also consider a ---well maybe "policy" is too strong a word, but something between a recommendation and a policy, where we prefer time-dependent test implementations to work for standard time, if for some reason we can't make them work for both DST and ST.
I don't suggest that any of these be permanent or immediately acted upon, I mainly wanted to get my thoughts on the topic out there and collect some ideas going forward.
http://www.nist.gov/pml/div688/dst.cfm
Keep the faith, someday, sometime, the tyranny will end.

On Mon, 22 Jun 2015 22:59:23 -0700, William Clifford<EMAIL_ADDRESS>said:

wobh> [...] to schedule a TravisCI build for the DST switch dates in
wobh> the US. If the tests fail for some reason, the scheduled build
wobh> should let us know almost as soon as can be known.

I like this idea of setting it up so that the Travis job fires at the
DST change times. It appears that you are not the first to ask how to do
this
http://stackoverflow.com/questions/20395624/travis-ci-builds-on-schedule.
wobh> The second is that it's seems possible that perhaps only a few
wobh> CL implementations could be affected by the DST switch due to
wobh> a bug in time handling.

I had not thought about this possibility. I don't remember asking the
people submitting the reports of broken tests (is there a name for a
person who does Exercism exercises?) what implementations they were
using - but maybe one of us did and we have some data?
wobh> It would be nice to figure out how to conditionally allow
wobh> failures on a per-test-case basis, so that even if there's a
wobh> problem with the DST affected exercise tests on that
wobh> implementation, we can still get feedback on the other
wobh> exercises until the issue is resolved (or until the switch
wobh> back).

It seems to me (without deep review / thought) that the existing test
harness runs all tests and then reports pass/fail. So if that is correct
then even if a time related test failed due to DST/TZ issues we'd still
get feedback from the other tests. Or are you asking about allowing such
a test to fail - but allow the entire run for that implementation to
pass?
This sort of feature sounds like something we could build into the test
harness. But it also feels like the sort of thing that we should not add
unless we are sure we need it. Allowing some tests to sometimes fail and
not fail the test run sounds like a good way to not notice a problem.
wobh> Lastly, I think we should also consider a ---well maybe
wobh> "policy" is too strong a word, but something between a
wobh> recommendation and a policy, where we prefer time-dependent
wobh> test implementations to work for standard time, if for some
wobh> reason we can't make them work for both DST and ST.

I think a better policy is to not have time related exercises in
Exercism.IO. While time issues always seem easy, they never are. It
seems to me that complicated topics such as time or threading should be
kept out of these exercises.
wobh> Keep the faith, someday, sometime, the tyranny will end.

The tyranny of time? Down with clocks!
Suddenly I just thought of Metric Time - maybe that is what you mean by
'standard time'?

Good points, especially the last. I think we're agreed that we should at least set up some automatic test runs starting on 2015-11-01T02:00:00 (maybe a little later) so from here on out it's just a question of means and procedures.
TronCI seems like a good find, but I find myself a bit uncomfortable turning over my github credentials for this. I suppose it's a baseless worry though.
I think we should setup the test runs to use Eastern time per http://stackoverflow.com/a/23374438 and I think it might be a good idea to setup up the run time for a couple minutes after the change over. I wonder if we could set the datetime on the Travis VM and run some tests in advance?

Perhaps another way to approach this problem is to change the gigasecond interface to include a timezone parameter?

Hey, guess what came and went without any fuss?

Happy winter, everyone.


[Docs] Add decision record for generic string type

Part of #193.
We must decide what generic string type should be exposed in the API(s).
Strings are a core data type of OpenAssetIO, used as both input and output of entity references and URIs, as well as nested within dictionary types for entity attributes and specifications.

The latest fixup! focusses on C++ with only lip-service given to C bindings.  This reduces the options down to three.  It also includes a tentative decision.

@feltech feel free to squash and merge at your leisure.


Update embed-pages-to-teams.md

Add stream web part to list of things that don't work
Category

[x] Content fix
[ ] New article

Related issues
https://portal.microsofticm.com/imp/v3/incidents/details/234186801/home
What's in this Pull Request?
added Stream web part to list of things that don't work.

Docs Build status updates of commit add5484:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/features/embed-pages-to-teams.md
:warning:Warning
View
Details

docs/features/embed-pages-to-teams.md

Line 63, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 67, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 71, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 94, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 107, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 116, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 120, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 125, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 129, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.
Line 133, Column 5: [Warning-duplicate-alt-text] Alt text 'teams menu apps' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


PR: Fix completion popup dismissed after typing a space

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)
[x] Included a screenshot (if affecting the UI)

Dismiss the completion popup if a space is typed in the editor

Issue(s) Resolved

Fixes #8859
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: steff456

@steff456, does this also cover the other delimiters mentioned by @CAM-Gerlach on issue #8859? In other words, do we hide the completion widget after users press (, ), {, }, [ or ]?

@ccordoba12 At least in my testing, it does not.

Ok, @steff456, we need to hide the completion widget in those cases too. Please address that in this PR.

Hello @steff456! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/editor/widgets/codeeditor.py:

Line 2782:77: W291 trailing whitespace

@CAM-Gerlach, could check again this PR? It should be working now.

Any chance we can get a test for this?

Tests for code completion take a lot of time to run so I think we can merge this one without one for now.

@steff456, on a second thought, please add a simple test for this. You can add it here:
https://github.com/spyder-ide/spyder/blob/master/spyder/plugins/editor/tests/test_introspection.py
and you can test that after writing a delimiter (e.g. [) the completion widget is hidden.
Thanks!


Exclude config.h.in in sgx_unwind

Missed this in the previous PR: https://github.com/baidu/rust-sgx-sdk/pull/162.

Thanks again!


Prevent accidental seed mingling

Currently there are no safeguards against using multiple seeds with a single walrus server. To catch this, the server could store a hash of the user's seed. Then, the client would include this hash in each query to the server. If the hashes differ, the server can return an error.
The one twist here is with Ledger wallets. We can't hash the Ledger's seed directly, so we need to get a unique identifier some other way. (An obvious choice would be to generate an address with a very high key index.) Unfortunately, this means we need an extra setup step for Ledger users. Not a huge deal, but possibly confusing.

Another option would be to query the set of current addresses and try to regenerate one (using its associated keyIndex). This means you can't detect the wrong seed when you have 0 addresses, but that shouldn't be very common.


Adding metadata as author and description

Hello,
I have tried with different options to add the metadata labels.
The examples in internet are those:
core_model.author = 'Hincat'
core_model.license = 'BSD-3'
core_model.save('Favourite.mlmodel')

mlmodel.author = ‘tpeter’
mlmodel.license = ‘MIT’
mlmodel.short_description = ‘This model takes a picture of a leopard and predicts its gender’

model.author = 'Author'
model.license = 'BSD'
model.short_description = 'Predicts the price of a house in the Seattle area.'

But none of these options worked. (Obviously changing the text after the equal symbol.)
Please, can you inform about how to set the metadata?
Regards,
Jorge

Hello Aseemw,
Maybe my question was not clear.
Please, can you answer the following?
How can I add the author and description inside a coreml file that is build from a .pb file?
Regards,
Jorge

You edit the core ml spec and then save the model, like so:
model  = coremltools.models.MLModel(path)

#get the raw spec if you want to adjust layer names
#spec = model.get_spec()
# do something with the spec
# make a new model with the spec
#model = coremltools.models.MLModel(spec)

# Set the model metadata
model.author = 'Your Name'
model.license = 'Your License'
model.short_description = 'Description'
model.versionString =  '1.0 Beta 1'
model.save(new_path)

Hello Vade,
Thank you for your answer.
It works, it adds the author, license and description metadata.
But the problem is that the original mlmodel file has as Type parameter:
Input  Image (Color 300 x 400)
Output Image (Color 400 x 300)
But the new mlmodel file has:
Input  Image (Color 300 x 400)
Output MultiArray (Double 3 x 400 x 300)
Please, why the output has changed?
Regards,
Jorge

Did you adjust the spec at all?

If I add:
spec = mlmodel.get_spec( )
output = spec.description.output[0]
as used to create the original file, it launch and error. Do I need to specify something more?

You shouldn't need to touch the spec
just
# Set the model metadata
model.author = 'Your Name'
model.license = 'Your License'
model.short_description = 'Description'
model.versionString =  '1.0 Beta 1'
model.save(new_path)

without you posting code its impossible to know what is going on. Please post some code.

I have created the new file to add the metadata with the folllowing code:
import tfcoreml as tf_converter
import coremltools
model  = coremltools.models.MLModel('frozen5.mlmodel')
Set the model metadata
model.author = 'Jorge Gregorio'
model.license = 'Do not sell'
model.short_description = 'Neural art'
model.save('frozen5b.mlmodel')
It works, but it changes the output Type as I specified before.
Regards,
Jorge

That sounds like a bug, you can get the model spec before you change anything, print it, and what changed before and after, and try to force setting the spec values from before the change?
Thats odd. Maybe open a new issue just for that.

Thank you Dave for your answer.
I will try what you say tomorrow. I have very few knowledge with coreml.
If It does not work I will open a new issue.
Thank you. Regards.
Jorge

This issue is being closed due to inactivity. Please feel free to re-open if you still have questions surrounding this issue. Thanks!


fix: use json-bigint

from https://github.com/ExodusMovement/react-native-base/pull/23

merged first, can't get CI working


bug fix in tutorial documentation of univariate time series classification.

Reference Issues/PRs

What does this implement/fix? Explain your changes.

In tutorial documentation for sktime univariate time series classification, the legend in the plots of features for the Time-series forest classifier doesn't have slope. Since the feature name is _slope, matplotlib is not able to add it automatically to the label and throwing a warning message. I have renamed the _slope feature to slope before plotting to resolve this issue.
Does your contribution introduce a new dependency? If yes, which one?
No

What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[x] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[x] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

@mloning @TonyBagnall Please review this.

Good catch.
Failing code quality tests, but should be a quick fix with instances of #renaming _slope to slope. -> # renaming _slope to slope.

Thanks a ton for review. I have

Good catch.
Failing code quality tests, but should be a quick fix with instances of #renaming _slope to slope. -> # renaming _slope to slope.

Thank you so much for reviewing it!  I have made the suggested changes.


CLIMATE-598 - Update time attribute of Dataset when values are reshaped

Update the function reshape_monthly_to_annually in utils.py to edit the times attribute of Dataset when reshaping occurs.

Merged build triggered. Test Failed.

Merged build started. Test Failed.

Merged build finished. Test Passed.

I realize that providing the functionality in reshape_monthly_to_annually instead of in calc_climatology_year and calc_climatology_season is not the best approach. I am updating accordingly.

I realize that instead of updating reshape_monthly_to_annually,  I should have done it in calc_climatology_monthly, etc.
I am updating this accordingly.


While using customize directory tree together with searchable Tree, color of the filter node will not change to red

Version
1.6.0
Environment
Angular
Reproduction link
https://ng-zorro-antd-start.stackblitz.io/
Steps to reproduce
While using customize directory tree together with searchable Tree, color of the filter node will not change to red
What is expected?
While using customize directory tree and searchable Tree, color of the filter node will change to red
What is actually happening?
While using customize directory tree and searchable Tree, color of the filter node will not change to red
Other?

if u use NzTemplate to render nodes,you must render it yourself.(with NzTreeNode.isMatched)

Thanks for your answer.

Thanks for your answer.


Combine the Lawyer showcase to the main (student) showcase

Turn back on the Lawyer showcase, adding back the lawyer (Joyce) option on the character selection screen.

CSB has approved the use of their use case.

are we going back to the character selection screen?


Timer Trouble

As noted in #224 we might eventually face a choice between more accurate clocks provided by core Java classes vs more portable clocks in LWJGL expected to work the same everywhere for LWJGL's needs. That's not really a rush item, however, so leaving this here for later tweaking, especially when we've got more exposure to the platforms we might encounter issues on (for this one it was Mac)
Copy pasted links:
http://lwjgl.org/forum/index.php?topic=3177.0
http://lwjgl.org/forum/index.php/topic,4452.0.html

http://lwjgl.org/forum/index.php/topic,4452.msg24209.html#msg24209

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/487323-timer-trouble?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

I think we can close this issue. Pinging @mkienenb for confirmation.

Closing as (most likely) obsolete.


Datepicker clears old dates

Imagine a form where you have a collection of dates on it. And the datepicker configured not to allow dates older than today.
Now if you fast forward a couple of months and try to edit that form. Everything will be ok unless you accidentally set a focus to one of those fields. After the focus will be lost, the date field will be cleared because the datepicker is configured not to allow dates older than today. IMO this should no happen if the input value hasn't changed.
So shorting the above story. The datepicker should not clear/auto correct the input field unless the actual value has been changed.

I had the same problem, and  forceParse: false  not working for me.
Can anyone provide any kind of help?


activation logs --limit enhancements

Is your feature request related to a problem? Please describe.

If an activation has no logs, you get no output. This might be OK when doing a single activation --logs or with id but if doing a multi-fetch, does it make sense to at least show a marker for the activation that includes a "no logs for this activation" message?

The existing markers are hard to spot (maybe use chalk to bold them, also add activation version and date. These are some of the tweaks I think we dropped in the big cherry pick.

The best place to patch this is in the aio-lib-runtime repo. I've opened an issue https://github.com/adobe/aio-lib-runtime/issues/27 and have an implementation on this fork https://github.com/adobe/aio-lib-runtime/compare/master...rabbah:log-banner?expand=1.

The best place to patch this is in the aio-lib-runtime repo. I've opened an issue https://github.com/adobe/aio-lib-runtime/issues/27 and have an implementation on this fork https://github.com/adobe/aio-lib-runtime/compare/master...rabbah:log-banner?expand=1.

@joshuaauerbachwatson are we still dependent on a lower version of chalk (v2.4.2 vs latest which is v4.0)?
In the linked branch, I used the latest version of chalk.
Also, if the patch isn't accepted upstream, is there a convenient way to apply the patch in our own build without having to also fork the aio-lib-runtime repo?

@joshuaauerbachwatson are we still dependent on a lower version of chalk (v2.4.2 vs latest which is v4.0)?
In the linked branch, I used the latest version of chalk.
Also, if the patch isn't accepted upstream, is there a convenient way to apply the patch in our own build without having to also fork the aio-lib-runtime repo?

@joshuaauerbachwatson are we still dependent on a lower version of chalk

We are not.

is there a convenient way to apply the patch in our own build without having to also fork the aio-lib-runtime repo

Yes, using patch-package.    There are numerous patches against aio-lib-runtime in the cloud-workbench build (see cloud-workbench/patches).  Those particular patches are aimed at avoiding webpack / workbench initialization problems.  We so far have none that we apply at CLI level.   But, the nim build is set up to use patch-package, and we could easily be patching aio-lib-runtime there.   You could also use patch-package in the aio repo.   Adobe used to use it and abandoned it.  I don't know why.   Given the way we package things the only advantage to doing it in the aio repo is independent testing.   But, if this is something that has been rejected upstream we could decide that testing it as part of nim instead of as part of aio is adequate.

@joshuaauerbachwatson are we still dependent on a lower version of chalk

We are not.

is there a convenient way to apply the patch in our own build without having to also fork the aio-lib-runtime repo

Yes, using patch-package.    There are numerous patches against aio-lib-runtime in the cloud-workbench build (see cloud-workbench/patches).  Those particular patches are aimed at avoiding webpack / workbench initialization problems.  We so far have none that we apply at CLI level.   But, the nim build is set up to use patch-package, and we could easily be patching aio-lib-runtime there.   You could also use patch-package in the aio repo.   Adobe used to use it and abandoned it.  I don't know why.   Given the way we package things the only advantage to doing it in the aio repo is independent testing.   But, if this is something that has been rejected upstream we could decide that testing it as part of nim instead of as part of aio is adequate.

Thanks for the explanation. Will see how the issue is received upstream.
Re chalk, should this commit be dropped then https://github.com/nimbella/aio-cli-plugin-runtime/commit/490312d498ea81c0cc6fa0fb6c286ae9cfc30e06 (at least the chalk downgrade).

Thanks for the explanation. Will see how the issue is received upstream.
Re chalk, should this commit be dropped then https://github.com/nimbella/aio-cli-plugin-runtime/commit/490312d498ea81c0cc6fa0fb6c286ae9cfc30e06 (at least the chalk downgrade).

Re chalk, should this commit be dropped then 490312d

I believe and chalk workarounds can be dropped.  The reason for the problem was a glitch in dependency handling in oclif which has since been fixed.

Re chalk, should this commit be dropped then 490312d

I believe and chalk workarounds can be dropped.  The reason for the problem was a glitch in dependency handling in oclif which has since been fixed.

The upstream change is now merged. The solutions evolved through feedback on the PR so I will adapt our cli fork accordingly. There is a question of how to pick up the change in aio-lib-runtime which I'll take up in Slack for now.

Rodric fixed things and re-pushed.   I did the adoption on (local) branches.   There was no problem with the workbench build.
I am proceeding to merge the adoption.

The adoption is merged.  @rabbah I don't know what follow-up testing you want to do.  Since this is in the aio repo I will leave the closing timing to you.

Changes are already tested. Thank you for rolling it into our cli.


Communication channel link is missing in Readme

Do we have a communication channel on gitter or slack channel for new Contributors and discussion of new features and bugs?

openfoodfacts.slack.com

I can not join it. Can you send me an invite at<EMAIL_ADDRESS>


Status bar to display number of Partitions (besides routers, leaders, etc)

Note that sometimes nodes are in a leader-less partition. In that case, the number of counted Partitions in the network is higher than the number of Leaders.
Hence, showing this number separately is useful.


Static files

Should look like this:
from view import new_app

app = new_app()
app.static("/hello", path="/files")  # will use the route path by default (i.e. /hello in this context)

#82 would have to be finished to implement this.


Updates/submodule update 102

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;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
the PR must not fail unit testing
if you're adding/updating documentation of an endpoint, make sure you add/update the necessary parameters to the (template) configuration files in the ensembl-rest_private repo

Description
Updates the wiki submodule to match the previous release (101), and updates the version to 13.1
Use case
We need to keep the wiki submodule synced up with the version, and the REST version needs to be incremented to indicate a non-breaking change in functionality
Benefits
See use case
Possible Drawbacks
The wiki will still need to be updated to reflect release/102 functionality when it goes live. We may go back to having the wiki reflect the most recent release on GitHub, not the release available on the live server, and just inform outreach so that they can answer user questions appropriately
Testing
N/A, documentation change only
If so, do the tests pass/fail?
N/A, documentation change only
Changelog
N/A

Coverage decreased (-0.05%) to 44.406% when pulling 5b5e3b00f77e5dedad20ddb0f4daf1a5a8639bb5 on updates/submodule_update_102 into b6235e4b4b353c6583fb2f5d53e8335c2084036c on release/102.


setting display: block on inline targets breaks things badly

We're using sphinxcontrib-bibtex to handle citations; notice how the left bracket on the reference link is no longer inline, it shows up separated from its number and crashing into the title, and the remainder of the text starts a new paragraph:

I think this problem was introduced with #283 (or possibly #280).
this is how it should look (ignore the different spacing between top navbar and section title, that's just a page scrolling difference):

A proposed fix using javascript instead of CSS (note that the various disjuncts in the conditional may need to be made more general for it to work for all sphinx sites, not just ours):
<!-- this snippet prevents in-text anchors from being hidden under the -->
<!-- navbar when the page scrolls to them (citation/equation backlinks -->
<!-- are worst affected). -->
<script type="text/javascript">
    if ($(".navbar.fixed-top").length > 0) {
        var navHeight = $(".navbar").height();
        var shiftWindow = function() {
                var ourURL = document.URL;
                if ( (ourURL.search(/#[a-z]+\d\d\d\d$/) > 0) ||  // footbib item (bibtex key)
                     (ourURL.search("#id") > 0) ||               // footbib backreference
                     (ourURL.search("#equation") > 0) ) {        // numbered equation
                    scrollBy(0, -navHeight - 12);
                };
            };
        if (location.hash) shiftWindow();
        window.addEventListener("hashchange", shiftWindow);
    };
</script>

Thanks for the report!
Does one of the preview builds in https://github.com/mne-tools/mne-python/pull/8757 exhibit this, so it can be inspected live?
We actually have some footnote examples in the demo docs at https://pydata-sphinx-theme.readthedocs.io/en/latest/demo/demo.html#footnotes, so we should need to figure out what the difference is with the ones generated by sphinxcontrib-bibtex, as it seems to be working in our demo.

Here it is: https://24938-1301584-gh.circle-artifacts.com/0/dev/overview/implementation.html#id1
Sorry, I should have thought to include that to begin with.

Hmm, how did you get to that link?
If I just go to the demo page (without #id25 in the url), it renders fine. And don't directly see where I need to click to get that id25. When clicking the actual reference to go to the footnote, that works fine (although the offset doesn't seem to be working in this case, as it's hidden behind the navbar .. so that's another case that is not yet fixed).

OK, I see, it's the "back" reference from the footnote to where it is referenced. That indeed triggers the issue in our demo docs as well. Thanks for pointing that out!


Linux: Wrong paths lead to asset error

Hi Dan,
when I go get this project, go build and afterwards open up http://localhost:8081, I get an 400 error with following message:
Asset static/index.html can't read by error: Error reading asset static/index.html at /Users/sosedoff/go/src/github.com/sosedoff/pgweb/static/index.html: open /Users/sosedoff/go/src/github.com/sosedoff/pgweb/static/index.html: no such file or directory
Taken from the closed issues, I guess #171 happened again.
Also: kudos for this great project!

You're right. Fixed the issue and will probably do something about it for the long run.


DR2-1964 Update byIdentifiers and refactor tests

The entitiesByIdentifier method in the Preservica client has been
updated with a slightly different name and signature. Fixing the tests
to cope with this was really difficult because of all the mocking we've
got everywhere so I've replaced this in the three lambdas that call this
method.
The tests won't pass until https://github.com/nationalarchives/da-aws-clients/pull/318 is merged and deployed but this is ready for review otherwise.

I most definitely don't want a common method in the utils package. Reason is that most of then have slightly different requirements, different input objects etc. and it we try to make a generic one, it's going to end up very complicated and this is all about simplifying things. So I can live with the duplication as a trade for simplicity.
What I will probably end up doing is creating a trait in the utils packge for every trait we override with notImplemented for all the methods and then we can override that one which should shorten things considerably but I want to get all the changes in first before I do that.


Time conversion issue on OSX

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10.1 darwin/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/vk/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/vk/Work/Languages/Go/gopath"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/vp/p89gqpvd4f93zzqpz4g5jnvm0000gp/T/go-build932640703=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
package main

import (
        "fmt"
        "time"
)

func unix2Str(ts int64) string {
        const layout = "20060102"
        t := time.Unix(ts, 999999999)
        return t.Format(layout)
}

func unixTime(ts string) int64 {
        const layout = "20060102"
        t, err := time.Parse(layout, ts)
        if err != nil {
                fmt.Println(err)
                return 0
        }
        return t.Unix()
}
func main() {
        ts1 := "20110320"
        ut1 := unixTime(ts1)
        fmt.Println(ts1, ut1, unix2Str(ut1))
}

When I run this code on OSX it prints
20110320<PHONE_NUMBER> 20110319

When I run this code on Linux it prints
20110320<PHONE_NUMBER> 20110320

What did you expect to see?
I should expect that output from string format to unix and back to string format will produce the same date, and it is not the case on OSX only.
What did you see instead?
I see that dates didn't match each other since I convert date in string format into Unix timestamp I would expect that converting Unix timestamp to string date format should use UTC instead of local timezone.
It seems to me that the problem is OSX specific and t.Unix uses time.Local on darwin. If I modify t.Format(layout) in unix2Str function to t.In(time.UTC).Format(layout) it fixes the problem. Therefore, I'm not sure if it is considered as a bug or a feature.
Best,
Valentin.

I think the code I posted works as expected since documentation said that Parser returns time in UTC while Unix returns local time, closing the issue.


Fix documentation for druid.query.scheduler.numThreads.

It has nothing to do with druid.server.http.enableRequestLimit. They are independent implementations of similar ideas.

strange, it used to (poorly) just say it was similar to the other setting until #14324

Yeah, I guess that PR got done because the original wording was kind of confusing: it was trying to make an analogy to druid.server.http.enableRequestLimit but could have been interpreted as saying it's actually connected to druid.server.http.enableRequestLimit. Better to avoid mentioning that property at all, IMO.


[BUG] DeepInfra BGE-m3 embedding upsert generate [ERROR]: Error: 404 status code (no body)

Describe the bug
Trying DeepInfra as OpenAI Embedding Custom I got always this error trying to upsert both on Qdrant and In Memory
To Reproduce
Steps to reproduce the behavior:

Test using https://deepinfra.com/BAAI/bge-m3/api?example=openai-emb-js
Try to upsert from any source

Expected behavior
Complete the upsert in the vectordb
Screenshots

Flow
DeepInfra test Chatflow.json
Setup

Installation npx flowise start
Flowise Version 2.0.1
OS: Linux
Browser firefox

Additional context
Add any other context about the problem here.

Wa my fault I needed to use the Base URL and the apiKey shown after the model deployment
Closed


[BUG] Extention type inheritance and pickle

Describe the bug
The inheritance with/from extension types and python classes as well as pickling seems broken.
To Reproduce
Code to reproduce the behaviour:
import cython
# next cell
# %%cython -3
cimport cython

cdef class Base_A():
    cdef double _a(self, double x):
        print("Base_A_.a()")
        return -99

    def a(self, x):
        print("Base_A.a()")
        return self._a(x)

cdef class A(Base_A):
    cdef double _a(self, double x):
        print("A._a()")
        return x / 2.

    def a(self, x):
        print("A.a()")
        return self._a(x)

class Base_C(Base_A):
    def c(self, x):
        print("Base_C.c()")
        return self.a(x)

class C(Base_C, A):
    pass

Then
C().c(1)

produces
Base_C.c()
A.a()
Base_A_.a()
-99.0

And
import pickle

pickled_C = pickle.dumps(C())
pickle.loads(pickled_C)

produces
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/var/folders/1c/ynyjslg54654183kt57yfsdw0000gn/T/ipykernel_43218/2044510036.py in <module>
      2
      3 pickled_C = pickle.dumps(C())
----> 4 pickle.loads(pickled_C)

~/.ipython/cython/_cython_magic_[...].cpython-37m-darwin.so in _cython_magic_[...].__pyx_unpickle_A()

TypeError: _cython_magic_[...].A.__new__(C) is not safe, use C.__new__()

Expected behavior
A pure python implementation
# Pure Python implementation
class Base_A():
    def _a(self, x):
        print("Base_A_.a()")
        return -99

    def a(self, x):
        print("Base_A.a()")
        return self._a(x)

class A(Base_A):
    def _a(self, x):
        print("A._a()")
        return x / 2.

    def a(self, x):
        print("A.a()")
        return self._a(x)

class Base_C(Base_A):
    def c(self, x):
        print("Base_C.c()")
        return self.a(x)

class C(Base_C, A):
    pass

with
C().c(1)

produces
Base_C.c()
A.a()
A._a()

0.5

and pickle does not throw an error.
Environment (please complete the following information):

OS: macOS
Python version: 3.7
Cython version: 0.29.21

class C(Base_C, A):
    pass

Python allows this because it can be made to work from an inheritance point of view because A and Base_A have compatible layouts. If you change Base_C and C to cdef class then Cython disallows it with Only one extension type base class allowed. That suggests to me that we aren't expecting this inheritance to happen.
Uses a "vtab" of function pointers to call the correct cdef function in derived classes. My guess is that the second Base_A ends up overwriting the "vtab" that was set up by A. If I add a __cinit__ in the Cython classes I can see that A.__cinit__ is not being called.
I'm not quite sure what's happening for pickle, but I'd guess it's also to do with Cython not expecting multiple inheritance of cdef classes.

I'm not quite sure what the solution is at this stage, but I recommend not inheriting from multiple cdef classes!

Similarly A.__dealloc__ is never called either

I looks like tp_new is always inherited from whatever is picked as tp_base (https://github.com/python/cpython/blob/f9242d50b18572ef0d584a1c815ed08d1a38e4f4/Objects/typeobject.c#L6303):
type->tp_new = base->tp_new;

and Python choses tp_base.
That doesn't leave much scope for Cython to override what's going on here. The best we can do is probably some validation to raise an error given this pattern of inheritance.

Final comment (for now):

The best we can do is probably some validation to raise an error given this pattern of inheritance.

I think this could either be done by:

giving all cdef classes a custom metaclass that does this validation in __call__ - this should probably work on all supported Python versions, but is fairly intrusive and risks breaking user code that inherits from a cdef class and defines a metaclass (I thinK)
Checking in __init_subclass__ (which we define for each cdef class). This would only work in Python 3.6+ (but that's OK because it's a "nice to have" sanity check. It'd possibly prevent users from defining their own __init_subclass__ on a cdef class.
Check in __new__, which would be the wrong place to check because it'd happen each time an object is created rather than each time a class is created. But it'd work everywhere

https://github.com/scikit-learn/scikit-learn/pull/20567#issuecomment-907289384
@lorentzenchr

Quote: "I'm not quite sure what the solution is at this stage, but I recommend not inheriting from multiple cdef classes!"
This means the class inheritance design has to be reworked!

Your example works correctly (i.e. all the correct __cinit__ and __dealloc__ functions are called, the right cdef function is called, and pickle works) with
class C(A, Base_C):
   pass

Although obviously that changes your __mro__ so it may have other consequences for real-world examples. Essentially the rule to follow is that the most derived Cython class should end up in C.__base__. I still don't recommend it, but I think it'll work if you can follow that. Hopefully that helps the problem you're really trying to solve.
(I'm not sure whether this comment is most useful here or on the scipy issue...)

@da-woods Thank you so much for investigating this so quickly and even recommending solutions for my original problem in scikit-learn!


safety_margin implementation in bounded_runs_iter() is broken

In some cases, the results from running bounded_runs_iter() can be different each time it is run when safety_margin is provided. This has also resulted in weird differences between weakly hard constraints (i.e., (2, 3) having greater deviation bound than (2, 4)). I'll need to investigate what's wrong with my early termination code in bounded_runs_iter().
https://github.com/Ratfink/ControlTimingSafety.jl/blob/f5979fa8a4f526b41fb01409c0618b4cbde39fd5/src/safety.jl#L163-L175

What Automaton are you using, and what parameters to bounded_runs_iter are causing the odd behavior?  A minimal working example would be helpful.


Failed to upgrade Chef Server in hardened node

Chef Server Version
Current running version: Chef Server V12.19.26
I'm trying to upgrade to: V14.8.7
Platform Details
RHEL7
Configuration
Hardened node.
Scenario
I was trying to upgrade Chef Server from V12.19.26 to V14.8.7, following the steps in https://docs.chef.io/server/upgrades/
On one of the node, I cannot change user in Step 1.
sudo su - opscode-pgsql
Steps to Reproduce
Follow the upgrade steps in https://docs.chef.io/server/upgrades/
Run sudo su - opscode-pgsql
Expected Result
Chef Server upgrade successful.
If Chef Server upgrade may fail on some hardened nodes, can we have some pre-upgrade check script to check for the necessary dependencies?
Actual Result
It says:
exec: /var/opt/opscode/postgresql/.profile [Permmission denied]

@zx-progress We'll need more information on what a hardened node means before we can dig in here.

Please feel free to reopen with the additional information.


Add warning and extra flag about disabling Dataflow Runner v2 for Python.

The vast majority of pipelines are already running on Runner v2 (possibly opted in by the service) and the majority of those that are not	are on old SDKs.
This paves the way for removing	obsolete Runner	v1 support from	newer SDKs, starting with Beam 2.50.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python_Integration PreCommit


I have successfully completed the demo1

I have successfully completed the demo1
The "hello world" is worked,but when I want to flash a new hid-payload,I must  shorting pins the usb drive.Direct flash a new fw.bin without shorting pins will appear the error "0079",why?Can I flash a new hid-payload without shorting pins?thanks
By the way,What the demo2、demo3、No Boot Mode Patch for?what's they use?

There is no need for shorting pins again. Once you have your Demo1 hid-payload running you can just install another hid-payload by following the same procedure using EmbedPayload and your FW.bin

@brunomarx Must I install new hid-payload when it is running or it Finished running?BTW, What the demo2、demo3、No Boot Mode Patch for?what's they use?

I have successfully the demo1 too.But,I try other payload,the "DriveCom.exe" output error.
Like that:
Action spceified:SendExecuteable
Fatal:System.InvalidOperationException:DeviceIoControl failed:0079
How can I do fixed this error. ：）

@jearyorg can you tell me where to buy the 2303 usb ? can you give me the taobao link? thank you very much!


It doesn't work on Windows11.

Describe the bug
It doesn't work on Windows11.
To Reproduce
Steps to reproduce the behavior:

Upgrade to Windows11
Open the software.
And find it doesn't work.

Expected behavior
It works as it was on Windows 10.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Version: latest

Additional context
I know that windows 11 has provided central taskbar.
But the start icon is also moved to the center, that's not good.

Windows 11 will not be supported. See previous issues regarding this. The start button and icons are merged under the same UI group now and cannot be separated using the UIAutomation techniques this program utilizes. Sorry

Thank you.

Yeah it's a shame and they could easily make it an option with much cleaner native animations than these programs but instead they removed all kinds of functionality


Data is loaded right from beginning, loadMore does not work as expected

I want to implement an infinite scroll with https://github.com/CassetteRocks/react-infinite-scroller in my React component.
I am fetching my data from my IndexedDB in the useEffect method and save it in my dbScans store.
The goal is to show the first three objects from the dbScans array at the beginning, and then load more with the package.
I already got some help from @Venkat with the loadProducts function, but my current problem is, that at page load all objects are rendered.
If I scroll up/down, I see my console logs from my loadProducts function but it doesn't affect anything.
import React, { useEffect, useState } from 'react';
import InfiniteScroll from 'react-infinite-scroller';
import db from '../../services/index.db.service';

export default function ProductHistory() {
  const [dbScans, setDbScans] = useState<IProductClient[]>([]);
  const [loadedProducts, setLoadedProducts] = useState<any>([]);

  const perPage = 3;
  const [lastObjectPosition , setLastObjectPosition ] = useState(0);

  const { t } = useTranslation();
  let history = useHistory();

  useEffect(() => {
    (async function getDataFromDB() {
      setDbScans(await db.yourScans.toArray());
    })();
  }, []);

  const loadProducts = () => {
    setLoadedProducts((currentProducts: any) => {
      console.log("currentProducts", currentProducts)
      return [
        ...currentProducts,
        dbScans.slice(lastObjectPosition, lastObjectPosition+perPage)
      ]
    });
    setLastObjectPosition(currentValue => {
      console.log("curr", currentValue);
      return currentValue + perPage
    })
  }

  return (
    <>
      { dbScans.length > 0 ?

          { dbScans.length > 0 ?
            <InfiniteScroll
              pageStart={0}
              initialLoad={false}
              loadMore={loadProducts}
              hasMore={lastObjectPosition < dbScans.length}
              loader={
Loading ...
}
            >
              {dbScans.map((product: any) =>
                product.image
              )}
              </InfiniteScroll>
            : ''
          }

      : ''
      }
    </>
  )
}

This are my console logs after scrolling up/down:

How'd you fix it?


Tests is missing

What about tests?
Do you have a plans to add any test examples?

I haven't had the opportunity just yet - but do have plans to do so (please feel free to submit a PR).
I would prefer one of the following two options:

Recommended: HTTPX (an async replacement for requests) lets you mount a ASGI/WSGI application directly to a client instance. (API Reference)
FastAPI has a test client built into it, which directly subclasses Starlette's TestClient. To use this, you must add requests to your project.


[Chore] Increase coverage of exam client components

Checklist

[x] Client: I followed the coding and design guidelines.
[x] Client: I added multiple integration tests (Jest) related to the features (with a high test coverage)

Test Coverage
quiz-exam-summary.component.ts -> 98%
create-test-run-modal.component.ts ->

@stefanwaldhauser  and I expanded the test coverage of the same component :D. I will revert my changes, as this was @stefanwaldhauser  component.

@stefanwaldhauser  and I expanded the test coverage of the same component :D. I will revert my changes, as this was @stefanwaldhauser  component.


MudHighlighter: Add nullable annotation.

Description
Part of this issue https://github.com/MudBlazor/MudBlazor/issues/6535
I changed HighlightedTexts to have empty collection instead of null.
It's known that returning null instead of empty collection is considered a bad practice. Framework Design Guidelines: Conventions, Idioms, and Patterns for Reuseable .NET Libraries: "DO NOT return null values from collection properties or from methods returning collections. Return an empty collection or an empty array instead."
How Has This Been Tested?
No tests required.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change, improve null diagnostic warnings)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.

Updated PR.


docs: Update middleware attribute example to proper access modifier

Summary of the changes (Less than 80 chars)

Updates the example middleware attribute to use the proper access modifier.

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


Offline CSV Report Generation

Story: https://www.pivotaltracker.com/story/show/93451218
This PR changes the flow for these reports so that they are no longer generated on demand, but once per hour as a cron job, and stored in /data/uploads/publisher/reports on the NFS mount.
When the controller serves the stored file, it will add the generation time to the filename it sends to the browser.
There's a fair bit of refactoring in this too, the reports are now a lot more standardised around the CSVPresenter class.
This PR is best reviewed commit-by-commit
TODO:

[x] race condition lock
[x] deployment change: symlink NFS to folder in Rails.root
[x] cronjob to run the rake task every hour

Requisite deployment change: https://github.gds/gds/alphagov-deployment/pull/968

Related puppet change https://github.gds/gds/puppet/pull/2985
/cc @benilovj @jamiecobbett

@benilovj Your changes look good to me :+1:

The Puppet and Deployment changes have been merged, I've reviewed @boffbowsh's changes and @fofr has reviewed my changes. Merging.

FWIW: :+1:


Add possibility to remove modules

I.e. (kit/remove-module ) would do the inverse of kit/install-module
Would be useful in the case of failed experiments with a certain module.

That might be a bit tricky because modules write to files in the project, and if the file is edited later then it could be difficult to tell where the code from the module is and whether it's safe to remove it.

Yeah I can understand that this may be non-trivial to implement. One way would be to try to undo the module changes but bail out if there are any issues. I guess on underlying problem is the choice if the term "module", in that in reality the functionality is not really that modular at all (in the sense "self-contained"). And then there is the distinction to Kit "libraries" (which are actually modular in that sense), and the "profiles"...

Yeah, ideally there would be a way to make modules truly modular, but that's a tricky problem in itself. The current modules are probably better described as templates. One thing that may be a reasonable compromise would be to remove the files added when a module is installed and then let the user know what code to remove from the existing project files. The modules already list the changes they make during the installation, so it would be easy enough to list that out. In most cases, the changes to existing files involve adding dependencies, requiring namespaces so they get loaded, and injecting components into system.edn. So, it shouldn't be too onerous on the user to roll these back manually.

This is something I've missed, and have been thinking a bit about:

Any sort of removal should check the git status is clean, for safety.
For full files that are created, we could store the sha in the log, after it's been generated.

On removal, checksum is verified, and either removed or not, with a log message that it needs to manually removed, as it's been modified.

For file modifications, we could save a patch, if the file hasn't been changed, it's easy to reverse, and if it has, we can print out the reverse patch and tell them that's what needs to be removed.

This changes are carried out with rewrite-clj/zippers, no?  I'll have to review the implementation to be sure, but I seem to remember that changes are stored separately until ->root is called?  That seems like it has potential.

Writing this out, maybe a starting place would be to generate a "Removal report".  People can follow it manually, and if it works well enough, then we could automate it?

The removal report would probably be a good place to start, and we already produce a report for what's been installed. So, and easy quality of life improvement could be to persist the report from the installation and then add a command to print it out.


Lazy property decoding

Experimental

After some real-use cases testing, it seems this actually makes things worse


Support for TTGO Watches

Hi,
Thank you for this amazing project! I truly appreciate all the hard work and dedication put into it.
I would like to inquire if there are any plans to support the TTGO Watch product lines. They have some excellent devices in their lineup, that are also well supported within the Arduino framework. It would be wonderful to see these devices integrated with your project.
Thank you for your time and consideration.
https://github.com/Xinyuan-LilyGO/TTGO_TWatch_Library

Thank you.
Unfortunately, I do not have any TTGO boards as of now therefore I'm not able to test and add support

I might be able to add support for the TTGO T-Watch v2 as I have this version available. I will start by getting the user interface working. TTGO watches require initial configuration due to their Power Management Unit (PMU), but that should not be an issue. After that, do I need to configure the IMU for step counting? Is this feature available in the current firmware? Could you provide additional instructions?

The project does not include any external sensors/peripherals interface. It is only the UI interface based on LVGL. This is because the original C3-Mini board does not have any IMU. This can however be included based on the board.
You can fork the repo, add the configurations then test the board.


Audit Log; filter by User

Describe the feature you'd like
In the admin Audit log, add an option to filter actions by user, in addition to the existing event filter and date ranges.
Describe the benefits this feature would bring to BookStack users
Currently, there is not - to my knowledge - a convenient way of reviewing a specific user's activity in the audit log.
This makes it difficult to determine a user's contributions as well as the nature of their contributions.
Having the ability to sort by a particular user's actions would make admin review of malicious users significantly easier. It would also make the general administration significantly easier by giving a convenient overview of an individual user's contributions (or lack thereof), aiding in quality control.
Using their profiles is sub-ideal as it is only recent activity and not a complete log.
Example of what I imagine:
https://puu.sh/H5G0z/2cfa5fa13e.png

Thanks for the suggestion @SiddoSnow.
I totally agree, I didn't add this filter before as we didn't have any pre-built "User Select" control but I had to add one for the content-owner changes in v0.31 which we can re-use here. Have marked for the next feature release.

Thanks for the suggestion @SiddoSnow.
I totally agree, I didn't add this filter before as we didn't have any pre-built "User Select" control but I had to add one for the content-owner changes in v0.31 which we can re-use here. Have marked for the next feature release.

Thanks again @SiddoSnow.
This has now been implemented, to be part of the next feature release.


Configure ECS to Deploy Updated LibreChat with JWT Support

Discovery:
The updated LibreChat application now supports JWT authentication, requiring changes to the deployment configuration on Amazon ECS. Environment variables related to JWT authentication (e.g., secret keys, token expiration settings) must be properly configured and securely stored in the ECS environment to ensure smooth deployment and operation of the updated version of LibreChat.
Problem Definition:
The current ECS configuration does not account for the new JWT authentication requirements. Environment variables such as the JWT secret key and token expiration time need to be added to the ECS task definition and managed securely. Without updating the ECS configuration, the application will not function correctly in the production environment.
Motivations:
By updating the ECS configuration to support JWT authentication, we can:

Ensure the proper deployment of the updated LibreChat with the required JWT authentication capabilities.
Securely manage environment variables for JWT secret keys and related settings, minimizing security risks.
Maintain consistency between the local development environment and the production ECS deployment.

Proposed Solution:

Update ECS Task Definition:

Modify the ECS task definition to include the following new environment variables required for JWT authentication:

JWT_SECRET_KEY: The secret key used for signing and validating JWT tokens.
JWT_EXPIRATION: The token expiration time (e.g., "24h" for 24 hours).

Ensure these environment variables are securely stored using AWS Secrets Manager or SSM Parameter Store, and that the ECS task can access them at runtime.

Use Secure Secrets Management:

Ensure that sensitive environment variables, such as the JWT_SECRET_KEY, are not hard-coded in the task definition or Docker container. Instead, use AWS Secrets Manager or SSM Parameter Store to inject these variables securely into the ECS tasks.
Update the ECS task definition to retrieve the secrets from AWS Secrets Manager or SSM Parameter Store during task execution.

Test JWT Authentication in ECS:

After deploying the updated task definition, test the JWT authentication flow in the ECS environment to ensure the tokens are validated correctly and that users can authenticate successfully.
Test token expiration and renewal (if applicable) to confirm that the JWT configuration works as expected in the deployed environment.

Update CI/CD Pipeline (if needed):

If environment variables or secrets are set up through CI/CD, ensure that the pipeline correctly passes the necessary JWT configuration settings during deployment.
Verify that the CI/CD pipeline triggers a redeployment of the ECS service after the task definition is updated.

Add Monitoring and Logging:

Ensure that logging and monitoring are enabled for JWT-related authentication failures or issues, such as token validation errors or missing environment variables, via Amazon CloudWatch.
Add CloudWatch metrics and alarms for key indicators (e.g., JWT validation errors, ECS task failures) to monitor the health of the JWT authentication in production.

Definition of Done:

[ ] ECS task definition is updated with environment variables required for JWT authentication (e.g., JWT_SECRET_KEY and JWT_EXPIRATION).
[ ] Environment variables for JWT authentication are securely managed using AWS Secrets Manager or SSM Parameter Store.
[ ] The updated LibreChat application with JWT support is successfully deployed to ECS, and JWT authentication works as expected in the production environment.
[ ] Logging and monitoring are configured to track JWT-related issues in the ECS deployment.

Additional Information:

Secrets Management: Ensure that the JWT_SECRET_KEY is stored securely in AWS Secrets Manager or SSM Parameter Store, and that permissions are correctly set up to allow the ECS task to retrieve the secret.
Testing: Conduct thorough testing after deployment to confirm that JWT authentication functions as expected, including token validation, user creation, and token expiration.

#time 2
Tasks updated for the log in override


Legacy React APIs.

Dear Team,
As we continue to develop and maintain our applications, it's important to adhere to modern best practices. Therefore, I would like to highlight that it is not advisable to use Legacy React APIs in our future projects. These APIs are deprecated and may be removed in future versions of React, which could lead to potential issues and increased maintenance effort.
By using the latest React APIs, we can ensure our codebase is more robust, maintainable, and aligned with the latest standards. This approach will also help us leverage new features and improvements, contributing to better performance and developer experience.
Let's prioritize updating any existing code that relies on Legacy APIs and focus on utilizing the current React API for all new development.

I'm not sure what this is about...


Introduce support for str_sub(str, pos, len) (substring function)

This function takes in 3 arguments: the string, the position and the length (of the substring). As a quick example, a call to the function
str_sub('c programming', 3, 11)
will return the string programming as the substring to the string that is provisioned to the function call.

@ThakeeNathees, can you review this whenever you have time please. Thanks!

Thanks for the review @ThakeeNathees. I think latest patch should be good now. 💯

Thanks for the reviews. All request for changes applied 💯


Fix issue 383

Closes #383

Codecov Report

Merging #385 (637dcbe) into main (9c3242e) will increase coverage by 0.63%.
The diff coverage is 92.73%.

@@            Coverage Diff             @@
##             main     #385      +/-   ##
==========================================
+ Coverage   91.30%   91.94%   +0.63%
==========================================
  Files          17       16       -1
  Lines        1576     1688     +112
==========================================
+ Hits         1439     1552     +113
+ Misses        137      136       -1

Flag
Coverage Δ

unittests
91.94% <92.73%> (+0.63%)
:arrow_up:

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

Impacted Files
Coverage Δ

setup.py
0.00% <0.00%> (ø)

grayskull/base/github.py
72.83% <72.83%> (ø)

grayskull/license/discovery.py
94.87% <78.12%> (-2.28%)
:arrow_down:

grayskull/strategy/abstract_strategy.py
83.33% <83.33%> (ø)

grayskull/__main__.py
95.60% <93.18%> (-2.55%)
:arrow_down:

grayskull/base/factory.py
91.17% <93.33%> (+1.17%)
:arrow_up:

grayskull/strategy/py_base.py
95.00% <95.00%> (ø)

grayskull/config.py
96.96% <96.96%> (ø)

grayskull/strategy/pypi.py
97.57% <97.57%> (ø)

grayskull/utils.py
98.12% <97.58%> (-1.88%)
:arrow_down:

... and 7 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Cool! Thanks for your contribution again! :)


can't install slap-cli from nix flake

I've kept trying to install this and can't get it to work - would anyone be able to provide a working example?

Here's a complete flake, albeit not for the newest version.
{
  description = "slap-cli example flake";
  inputs = rec {
    nixpkgs = {
      url = "github:NixOS/nixpkgs/22.05";
    };
    flake-utils = {
      url = "github:numtide/flake-utils?rev=7e5bf3925f6fbdfaf50a2a7ca0be2879c4261d19";
    };
    mach-nix = {
      url = "github:DavHau/mach-nix?rev=8d903072c7b5426d90bc42a008242c76590af916";
      inputs.pypi-deps-db.follows = "pypi-deps-db";
    };
    pypi-deps-db = {
      url = "github:DavHau/pypi-deps-db?rev=ba35683c35218acb5258b69a9916994979dc73a9";
      inputs.mach-nix.follows = "mach-nix";
    };
    pypipegraph2 = {
      url = "github:TyberiusPrime/pypipegraph2?rev=e665acfe3d272a2d166b30f542ac592a47f7e26e";
    };
  };

  outputs = {
    self,
    nixpkgs,
    flake-utils,
    mach-nix,
    pypi-deps-db,
    pypipegraph2,
  }:
    flake-utils.lib.eachDefaultSystem (system: let
      pkgs = import nixpkgs {
        inherit system;
      };
      mach-nix_ = (import mach-nix) {
        inherit pkgs;
        pypiDataRev = pypi-deps-db.rev;
        pypiDataSha256 = pypi-deps-db.narHash;
        python = "python310";
      };

      slapcli = mach-nix_.buildPythonApplication rec {
        pname = "slap-cli";
        version = "1.7.1";
        format = "pyproject";
        # mach-nix currently can't extract requirements from
        # pyproject.toml
        requirements = ''
beautifulsoup4>=4.10.0
cleo>=1.0.0a4
databind>=2.0.0,<3.0.0
flit>=3.6.0
nr.util>=0.8.12,<1.0.0
poetry-core>=1.1.0a6
ptyprocess>=0.7.0
pygments>=2.11.2
PyYAML>=4.0
requests>=2.27.1
setuptools>=39.1.0  # Needed for pkg_resources
tomli>=2.0.0
tomlkit>=0.10.1,<0.11.0
twine>=3.7.0,<4.0.0
tqdm>=4.64.0
build>=0.8.0,<0.9.0
nr.python.environment>=0.1.4

        '';
        src = pkgs.python3Packages.fetchPypi {
          pname = "slap_cli"; # because dashes and underscores are the same thing except when they ain't
          sha256 = "sha256-s2Uyh+5wxx7V/t4DCpJL9f/JC+q8WXtsMi8NQosgr1I=";
          inherit version;
        };
      };
    in {
      defaultPackage = slapcli;
    });
}


Remote Configuration for ETC

What's the correct format for remote configuration for ETC? I saw you post about it on another issue but you weren't sure about it (It was similar to the XMR remote configuration format) but in the wiki of this repository it has a different format.
I can't really try it on my PC to check which one is correct (I probably can, but because of the low specs of my device it would take forever) and I don't really want to spread it just to check if the remote configuration works.

This is the correct format for the ETH miner: https://github.com/UnamSanctam/SilentETHMiner/wiki#remote-configuration, the examples there are for ETC (note the "etchash" algorithm). You can change it to what you want and then post it here and I can check if it's correct.

{
"url": "stratum://0xe8D349AC6087d114B1EF41a2E51aa442a3EA50F6.Worker/123456@etc-eu2.nanopool.org:19999",
"algo": "etchash"
}
Would this work?
Also, is it true that ETC is more profitable compared to ETH?

Yes that will work (though replace the wallet address with your own wallet address). ETH is the most profitable GPU coin at the moment https://whattomine.com/coins but there will be a hard fork for ETH in the next week or two which will reduce miner profits (since ETH wants to move to PoS instead of PoW). ETC isn't really that much less profitable than ETH due to the fact that ETC only requires 2.8GB currently to be mined while ETH requires around 4.4GB (their DAG sizes differ, also ETC has a smaller DAG size due to the etchash algorithm halving the size of if required on the GPU).

Thanks


Preventing Idle Sleep on macOS

I'm running 2.3.0 on 10.12.6, and I noticed over the last few days that my desktop isn't sleeping after 10 minutes like it used to. Some investigation is pointing me at the Keybase client / Electron.
pmset -g assertions says this with the Keybase client open:
pid 5142(Keybase): [0x0000ed0c0001a045] 00:30:02 NoIdleSleepAssertion named: "Electron"
And fully quitting keybase (removing keybase from the menu bar) removes that line and allows the machine to sleep at idle again.
It looks like 2.3.0 was released a few days ago, which corresponds to the timeline of my sleep issues.

cc @mmaxim

It looks like https://github.com/keybase/client/pull/12721 has introduced this "feature". According to the discussion in the PR it was to fix something that happened because the app went sleeping. I couldn't find any info why this is a bad thing, I understand why the daemon shouln't sleep but not why the app shouldn't. This makes the current version of keybase unusable for me.

+1

Backed this out in https://github.com/keybase/client/pull/12826.
The behavior we saw was that after electron came back from sleep, it was in a broken state. Hard to test.


MINOR: Removed deprecated schedule function

While working on this, I also refactored the MockProcessor out of the MockProcessorSupplier to cleanup the unit test paths.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax @vvcephei @bbejeck for reviews.

starting review now, but just a heads up the failure is from an unused import

Jenkins failures not related.

retest this please


PERF: DataFrame transform

[x] closes #12737, #13191
[ ] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

    before     after       ratio
  [2de2884 ] [4b352d9 ]
-  164.84ms     1.73ms      0.01  groupby.groupby_transform_dataframe.time_groupby_transform_dataframe
     4.44ms     3.77ms      0.85  groupby.groupby_transform_series.time_groupby_transform_series
     6.40ms     4.76ms      0.74  groupby.groupby_transform_series2.time_groupby_transform_series2

This also fixes (unless it was deliberate?) the issue of SeriesGroupBy.transform losing the Series's name, xref #9862

[Current coverage][cc-pull] is 84.12%

Merging [#13192][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master     #13192   diff @@
==========================================
  Files           138        138
  Lines         50385      50382     -3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          42383      42381     -2
+ Misses         8002       8001     -1
  Partials          0          0

Powered by Codecov. Last updated by [2de2884...9d78f65][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/2de2884a7e7abf64f9967f6d8bc05a2d45f59bb4...9d78f659f55128e55d0b062f30fb04ae68994d3e
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13192?src=pr

thanks @chris-b1 very nice PR as usual!


Question: Can I use the

I have a custom font I'd like to use for my app and specifically Storybook.js Any good solutions out there for this?

Yep, you can use @font-face in the Style component. Here's an example with PT Serif dropped in from Google fonts:
<Style
  rules={{
    '@font-face': {
      fontFamily: 'PT Serif',
      fontStyle: 'normal',
      fontWeight: '400',
      src: 'url(https://fonts.gstatic.com/s/ptserif/v8/I-OtoJZa3TeyH6D9oli3iXYhjbSpvc47ee6xR_80Hnw.woff2) format("woff2")'
    },
    body: {
      fontFamily: 'PT Serif'
    }
  }}
/>

That said, it might be simpler to use normal CSS for @font-face. I like to keep that stuff separate from my application logic and use some of the techniques outlined here for better loading performance: https://www.zachleat.com/web/comprehensive-webfonts.
But, if you want to stick to JS-only, or if you're using a component in a context where you don't have
access to CSS, this will do the trick 🤘


Update Maps with new art assets

Change wooden sign to new academy sign in courtyard.json
Add new center platforms underneath objective statues in the lovelace tower (whichever of the two colours looks better)
Add new lounge assets to lovelace tower (blue couches, blue recoloured chairs and coffee table)

@kerrielake I made an attempt at adding the platform underneath the objective statues, but except for the final one in the library map, the first four come out a bit strange. Feel free to take a look and update if you have a better solution to mine!


fix(chart): only show two rows max in legend [MA-2670]

Summary
When legend wraps to a third line, it should force a scroll to see the third row

Resources

Creating a package docs

Closing in favor of https://github.com/Kong/public-ui-components/pull/1204#pullrequestreview-1914759768


[AKS] Add Key Management Service feature for AKS to encrypt the Kubernetes data in etcd using Azure Key Vault with KMS plugin

Related command

az aks create --enable-azure-keyvault-kms --azure-keyvault-kms-key-id {} --azure-keyvault-kms-key-vault-network-access {} --azure-keyvault-kms-key-vault-resource-id {}
Description

This PR allows the customer to enable KMS feature for AKS cluster, which encrypts the Kubernetes data in etcd using Azure Key Vault with Key Management Service (KMS) plugin.
Testing Guide

History Notes

[AKS] az aks create/update: Add new parameters --enable-azure-keyvault-kms, --azure-keyvault-kms-key-id, --azure-keyvault-kms-key-vault-network-access, --azure-keyvault-kms-key-vault-resource-id and --disable-azure-keyvault-kms to support Key Management Service feature with Azure Key Vault

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

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

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AKS

https://dev.azure.com/msazure/CloudNativeCompute/_build/results?buildId=58198057&view=results

@bingosummer Could you please add the description of those new parameters in the PR description. Please note that these contents will be written into the history notes.
For more details, please refer to submitting-pull-requests

@zhoxing-ms
Updated the PR description. Does that meet the requirements?


fix: only remove leading 'v' from version

While playing around with this action, I stumbled upon a weird issue where my pre-releases (versioned according to Python's PEP440, e.g. v0.1.0.dev0) aren't picked up.
This PR changes the processing of the version string such that only leading 'v' characters are removed (instead of replacing all 'v' characters in the version string).
PS.: Thanks for your awesome action!

I've released your fix, you can either give the v1 tag another go in a few minutes or use the new explicit v0.16.7 tag of this action. Best!


Update antlr4 to match the one use in k8s 1.25

as per title.

/assign @n3wscott

/cc @duglin

/LGTM


[13.0.x] ISPN-13923 Support embedded query with Protobuf-encoded caches

https://issues.redhat.com/browse/ISPN-13923
https://issues.redhat.com/browse/JDG-5320

rebased on top of the current master

Looking a at https://issues.redhat.com/browse/JDG-5320, the target release is RHDG 8.4 GA, so we don't need any backport to the 13.0.x branch. I'm closing it!