Document Building From Source

Please document how to build elm from source
Judging by the complexity of the old BuildFromSource.hs provided for 0.18 and older, how to build from source is not always totally obvious.
I had to ask the elm slack, and it seems like I'm not the only one having issues
(see e.g. https://github.com/elm/compiler/issues/1892).
In the end, a custom stack.yaml and stack.build did it for me (Ubuntu 16.04, x86-64)
resolver: lts-13.27
projects:
   - '.'
extra-deps:
   - containers-<IP_ADDRESS>@sha256:28ad7337057442f75bc689315ab4ec7bdf5e6b2c39668f306672cecd82c02798
   - http-client-0.6.4@sha256:a74b417345857d4250bdb6d4ac7fd4a89fd2ad8efab3164c1a2b843398d5c13a
   - language-glsl-0.3.0@sha256:85c1e7bf2cf5d6e604b7a2899c27e2935033425944db200798e57849e64d4c81
   - network-<IP_ADDRESS>@sha256:1ca79d81af02d7acd6032d5e6c9bde4618a8fdcfbe19bd42b49d420183975df0
   - time-1.9.3@sha256:8f1b5448722a12a952248b356c9eb366e351226543d9086a2da71270522d5f45
   - binary-<IP_ADDRESS>@sha256:ae3e6cca723ac55c54bbb3fa771bcf18142bc727afd57818e66d6ee6c8044f12,7705
   - directory-<IP_ADDRESS>@sha256:500019f04494324d1df16cf83eefeb3f809b2b20b32a32ccd755ee0439c18bfd,2829
   - process-<IP_ADDRESS>@sha256:77a9afeb676357f67fe5cf1ad79aca0745fb6f7fb96b786d510af08f622643f6,2468
   - unix-<IP_ADDRESS>@sha256:e69269a17b9fa26cb77f3f55e86c39d0a4940ccfa0c4bc20826919d2572076ad,3496

(thanks to pd-andy and eh from the slack beginners channel!)
Also, there is a link loop:
If I follow the link under "Install" in the README.md, I get to the instructions from the documentation. If I follow the link to the npm package from there and go to troubleshooting (since my platform is not supported), when I follow "build from source" I get to the elm-platform repo which shows a link that it has moved here...

Sorry, duplicate of https://github.com/elm/compiler/issues/1894

This project prefers to have more new issues from more perspectives, so there is no need to close for "duplicates" here.
I added this to https://github.com/elm/compiler/issues/1949 so it is recorded in a more organized way.


Update ocLazyLoad links with the revised links created by grunt-rev

I am using yeoman (angular-fullstack) for my project. I want to use ocLazyLoad (link) in it with grunt-rev. But how do I update the asset links written in the js files by the new filenames created by grunt rev ? The links in the html file are update but how do I update the links in js file?

Hi, I have no idea, it's probably something that you should ask in the grunt-rev repository :)


Allow Disable of Grafana Dashboards / External Target Configuration

Signed-off-by: Oliver Bähler<EMAIL_ADDRESS>This PR adds two new features.
Chart [monitoring]:

Adds .Values.grafana.dashboards.enabled value which is set to trueby default. This allows users to disable the deployment of grafana dashboards.

Bumped to version 1.5.0
Chart [policy-reporter]:

Updated Dependency for monitoring chart to 1.5.0
Add new configuration key which allows the reference of an existing target configuration under .Values.existingTargetConfig. Is disabled by default and can be enabled. We need to deploy the secret ourselves, that's why we need such an integration

Bumped to version 1.12.0

Hey, thanks for you contribution. I think you missed the updated of Charts.lock, you can do this with helm dependency update charts/policy-reporter
If the test pass I will a short manual test and release it soon.


Pick Fonts

Allow choice of "Hand-Written", "Neat", "Calligraphy" fonts.
On windows: UD Digi Kyokasho NK-R, Yu Gothic, Yu Mincho
Other systems???

Apparently this is very tricky to do with CJK. The large number of glyphs results in a huge download...


Prevent opening files in terminal buffers

Hi! At first thanks for nvr it helped me a lot to get our of nested neovim sessions...
My current workflow for a react app looks like that:

single neovim instance (started with NVIM_LISTEN_ADDRESS) with source code files
split terminal (using neoterm) with build and watch tool running (yarn start)

The tool Create React App which is used for the React project is able to open up the editor directly from a browser tab if an error occurs. With setting the environment variable REACT_EDITOR="nvr -l" this works as expected: if the term split was focused it switches the window (-l flag) and opens the file. But if the term split is not focused (which is more often the case) it openes the file in the term split.
So basically the question is: is there a way to tell nvr it shouldn't open files inside terminal buffers?

At first thanks for nvr it helped me a lot to get our of nested neovim sessions...

Thanks. :)
Okay, the issue is that -l (which really is just a shortcut for -cc 'wincmd p'), is used unconditionally here. Try this instead:
export REACT_EDITOR="nvr -cc 'if &buftype ==# \"terminal\" | wincmd p | endif'"

It's working interactively from the shell, so it should work for REACT_EDITOR as well. Closing. :)

Sorry for the late reply!
Yes, the workaround works, but one has to add the parameters to the lauchEditor.js file of the Create React App react-dev-utils.
Thanks again for nvr and helping :+1:

Just in case somebody ends up here while searching for a solution when developing a Create React App, I submitted a PR to the CRA repo to add nvr to the list of editors.


Improve Branch Coverage for executeOGC in OperatorImportFromWkbLocal.java

Add 4 tests that improve branch coverage.

Closed on branch-cover-tests


Renaming organization to institution

Description
This PR renames the term 'organization' to 'institution' in our codebase
Motivation and Context
The change addresses outstanding technical debt and is required for better readability and maintainability. Currently there were both terms used to describe the same concept. The term 'institution' more accurately reflects the entities interacting with our system, providing clarity for future development and debugging processes.
Changes

Renamed all instances of 'organization' to 'institution' in the codebase.
Added db_patch script to reflect this change in the database schema.
Updated the duo-validation dependency from 3.5.1 to 4.0.0.

How Has This Been Tested?
Locally and with E2E
Fixes Jira Issue
https://jira.esss.lu.se/browse/swap-3712
Depends on
https://github.com/UserOfficeProject/user-office-factory/pull/368

Check with Emma

Check with Emma

Hello @UserOfficeProject/stfc-user-programme,
We've consulted with Emma regarding the proposal to standardize the naming convention to consistently use the term "institution". Emma confirmed that there are no objections to this change. While it appears that the Pull Request (PR) involves numerous files and changes, the core logic remains largely unchanged. Therefore, the review process should not be overly time-consuming. When you have a moment, please review the PR. 🎉

Hello @Junjiequan,
I have added you as a reviewer from the ESS side, as I think you are the best candidate. Please have a look when you've got a bit of time.
Thanks in advance :tada:

Are aesthetic occurrences of "organisation" staying such as here?

I spotted some old references to "organisation" .

Thanks for taking my feedback on board. I spotted one last use of "organisation".


Don't check reviews in a merge queue

Currently neither prcr, not review-bot don't support merge queues. We would like to turn them on in the polkadot-sdk repo (related issue can be found here).
Review bot should return green status when it's running in a merge queue. Since this check is going to be required I can't add simple skip check in a github action (like here).

Done with workaround, thanks @Bullrich !


Expose ImportDock and its children

Part of godotengine/godot-proposals#2537
Indipendent from #47520
Expose ImportDock, including a bunch of method for interacting with it.
Considerations for docs are the same of #47767.

Code seems fine. Needs rebase, so it can be tested etc. Also like in #47767, I don't see a way to actually get the import dock.

Closing per discussion in the Godot contributor chat.
Part of this PR will be extracted to make a cleanup PR.


Remove the DEFAULT_CPU ifdef from enqueue_deallocate

This PR fixes the memory leak issue with USM intermediate memory allocated within the operator implementations.

I approved assuming the test failed due to timeout
[ RUN      ] Trsm/TrsmFloat.test/alloc_buf__m_1027__n_1027__trans_t__side_l__diag_u__uplo_l__alpha_2__ldaMul_1__ldbMul_2__unusedValue_nan
2023-06-07T12:40:44.7882792Z 22/25 Test #22: blas3_trsm_test ..................***Timeout 1200.04 sec

Thanks @pgorlani. Yes, the tests fail because now we have to run 2304 x 2 tests for TRSM which makes the CI timeout.

You can increase the timeout in https://github.com/codeplaysoftware/sycl-blas/blob/cd37f15075d0e477345ee1f020327582fe6caffb/Dockerfile#L72
It is not ideal, but we can explore a better solution in a follow up PR. Maybe we can run USM tests in a separate job.

Hi @pgorlani @aacostadiaz, I have reduced the number of trsm tests in the latest commit to avoid timeout.


Include some library to repl

Hi @princejwesley. How can I include some library like a lodash to repl?

@xgrommx Use File > Add module Path... menu item to add lodash module path to REPL or set NODE_PATH env variable.

No, I want add it permanently =) @princejwesley

@xgrommx I'll add support for that and let you know :+1:

It would be perfect if it can be done like import from the REPL like
import "https://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.10.1/lodash.min.js"

@maowug Version 2 features are completed. Its available in v2 branch.
You can do require('lodash') or any npm module to download to application's user data path.
> let _ = require('lodash')

Great! :+1:

Merged to master

@princejwesley How about another think like a RxJS or smth like?

@xgrommx I am not sure what you are asking. loading reactive js?

@princejwesley now I can load const _ = require('lodash') but I also would like use Rx like so const Rx = require('rx')

@xgrommx  it will work in version 2.0.1.

@princejwesley thanks!


add the methods' doc link

Add the corresponding doc link for most methods.

However, it seems that this pull request is conflict with the code base
Could you please fix the conflict before we can be able to better it?

Seems that i have no write access to merge the PR.

Oh,  i have sync the changes to latest version.

Awesome.
Thank you very much for the contribution!


Failed build pipeline for application helm default versioning type

STR:
Create helm application codebase with default versioning type
Run build pipeline
ER: Build pipeline should be passed
AR: Build pipeline failed with an error
invalid result reference in pipeline task "helm-push": "BUILD_VERSION" is not a named result returned by pipeline task "get-version"
Same issue for Helm Charts library.


Social Ledger for RChain (on Discord Github Telegram Matrix and similar platforms)

Explore RChain requirements and architecture alternatives to implement social ledger on platforms of interest to RChain, in their priority order.  Deploy current SL on Discord, so RChain team can play with it and provide feedback and requirements from that context -- i.e. prototype based requirements gathering, rather than from cold.  Discord does not support markdown (no way to embed URL, click on it and have it do something).  We may need to provide folks a link that they can follow to act on those.
A strech goal is to explore SL implementation using Rholang (eat our own dog food).

@dckc Just to clear things, I did not assign any reward neither did I set a budget on this issue. I saw a budget of $400 under my name I dont know how that came to be as I did not participate in this issue and I am not among the participants or assignees. Please edit your post above.

Will do. I'm on mobile just now.

Looks like we ought to keep a keen eye on the budget program.
@jimscarver I'd like to propose a working group on this while @kitblake is in Tampa next week.

@patrick727, Yes, lots happening on the sheet lately from disappearing and appearing names, to random budget setting like in my case above. I really like what @dckc is doing on issue #260. Hopefully, we can get to adopt that when he is done.
@dckc I suggest creating a new issue for your work.

@dckc, will follow those guidelines for signature, status, links to context etc.. getting up to speed on that :-)

perhaps SL on github may be of most interest to the board and members at this time, since a lot of activity monitoring and analysis are focused there?

@jimscarver @9rb I'm interested in looking at the source code for social ledger -- 1.x, 1.5, and 2.x. I'm struggling to find it. Got any clues for me?

The social ledger prototype code is not ready for public access. Send me
your preferred email access and I will add you to our gitlab repository
@dckc
I've created the github repo
https://github.com/DigitalLifeCollective/basecamp for social ledger 2.0
development but haven't committed code yet. I should get around to it soon.
There is not yet any real code as we are still making decisions about the
architecture and tools

yawn. Wake me when it's shared publicly.


Fix conda recipe for Windows build

Sync Windows build script options with Linux/OS X build script options.
Packages can be tested by running the following on Windows:
conda install -c koverholt ibis-framework

thanks!


Rust package

This fixes rust builds on travis and adds mandatory changes for publishing on https://crates.io

Coverage remained the same at 93.548% when pulling f3a107356250ce61eae43d412fa215264e276a67 on narekgharibyan:rust_package into 580664d7c0c52ea2b4f702c28b20e7bb37285a16 on KeyviDev:master.


How to speed up rendering of many plots

ScottPlot Version: 4.1.16
Operating System: WIN10
Application Type: WinForms
Issue Description: In my application I create a list of plots that are linked. Basically I allow the user to create as many plots as he wants. The drawback is that with the increased number of plots and signals the interaction with the plots becomes slower and slower.
Questions: Do you have suggestions on what to do to speed up the rendering process? (compute in parallel, threads, use GPU, etc.)
Below is a snippet of the code that is excuted every time the use interracts with one of the plots. Works decent up to 4 plots and moderate number of signals (up to 10 signals/plot, and let's say up to 10000 points/signal). However, above that I start to feel degradation of speed that affects user interraction.
Using Parallel.ForEach doesn't help much. Especially for the Vertical Lines dragged linked I get strange delayed behavior if I use  Parallel.ForEach.

In the past I have used a graph tool that used the Arction engine. It was very fast, I must admit even, with many many plots. It is using DirectX in the background. Maybe we can learn from it.

Then I was triggered by Matlab, which I intensively use. Matlab uses OpenGL to render and it renders quite fast. Do you think OpenGL would increase rendering time in C#, see:
https://github.com/dwmkerr/sharpgl
If we could use it in ScottPlot, then 3D plots and surface plots will become possible too. What is your opinion on that?
This topic is related to #1113

Or is there a posibility to use the GPU power in C# for rendering?

For me any option that helps to speed up the rendering of many plots is ok. Because only 20% of processing power is used at the moment on my laptop. So, I think we can do more to use the available resources to our advantage.

Other links that I found:

https://github.com/sharpdx/SharpDX
https://github.com/microsoft/Win2D
https://www.reddit.com/r/csharp/comments/5uvyw9/what_can_i_use_for_basic_fast_2d_graphics/?utm_source=amp&utm_medium=&utm_content=post_body

For X Axes linked:
public static void OnAxesChanged(object sender, EventArgs e)
{
    FormsPlot changedPlot = (FormsPlot)sender;
    axisXMin = changedPlot.Plot.GetAxisLimits().XMin;
    axisXMax = changedPlot.Plot.GetAxisLimits().XMax;

    foreach (FormTimePlot tp in timePlot)
    {
        if (tp.formsPlot == changedPlot)
            continue;

        // Disable events briefly to avoid an infinite loop
        tp.formsPlot.Configuration.AxesChangedEventEnabled = false;
        tp.formsPlot.Plot.SetAxisLimits(xMin: axisXMin, xMax: axisXMax);
        tp.formsPlot.Render();
        tp.formsPlot.Configuration.AxesChangedEventEnabled = true;
    }

    //Parallel.ForEach (timePlot, tp =>
    //{
    //    // Disable events briefly to avoid an infinite loop
    //    tp.formsPlot.Configuration.AxesChangedEventEnabled = false;
    //    tp.formsPlot.Plot.SetAxisLimits(xMin: axisXMin, xMax: axisXMax);
    //    tp.formsPlot.Render();
    //    tp.formsPlot.Configuration.AxesChangedEventEnabled = true;
    //});

}

For Vertical Lines dragged linked:
public static void OnAxesDragged(object sender, EventArgs e)
{
    VLine changed_Line = (VLine)sender;
    vPosition = changed_Line.X;

    foreach (FormTimePlot tp in timePlot)
    {
        if (tp.vLine == changed_Line)
            continue;

        tp.vLine.X = vPosition;
        tp.formsPlot.Configuration.AxesChangedEventEnabled = false;
        tp.formsPlot.RenderRequest();
        tp.formsPlot.Configuration.AxesChangedEventEnabled = true;
    }

    //Parallel.ForEach(timePlot, tp =>
    //{
    //    tp.vLine.X = vPosition;
    //    tp.formsPlot.Configuration.AxesChangedEventEnabled = false;
    //    tp.formsPlot.Render();
    //    tp.formsPlot.Configuration.AxesChangedEventEnabled = true;
    //});
}

One thing that might help instead of Parallel.ForEach is several Task.Runs. Or writing your own multithreaded code.
Parallel.ForEach has weird characteristics, it generally uses as many threads as reasonably possible, and those threads are often not that busy. This means that if your CPU has other things to do they may not be scheduled as often. It also may spawn more threads than is efficient for a given workload. You can also pass in a ParallelOptions and play with MaxDegreeOfParallelism (or TaskScheduler if you want to get that low-level) and see if that helps.
As for ScottPlot, there's been talk for a while of moving to Skia, which is anticipated to have better performance than System.Drawing. I also wonder if ScottPlot may perform differently on Linux or Mac as libgdiplus uses Cairo which uses OpenGL (or sometimes another backend), and I don't know what GDI+ uses on Windows. There was a hacky way of forcing a program to use a dedicated GPU discussed in #413, however, that only works on Windows systems with NVIDIA GPUs (although on other systems it shouldn't break anything). Note that ScottPlot will still use hardware acceleration regardless, just not necessarily on the primary GPU.
There are general optimizations that could be done within ScottPlot. I don't think that there are many algorithms that are majorly unperformant, at least from a time complexity perspective. However, I don't touch signal plotting very often, and that would be the most performance-sensitive area. But in C# things like reducing heap allocations can be big performance wins, even if an algorithm is theoretically optimal. That's why things like LINQ are controversial among some developers.
Another optimization was discussed in #978. Unfortunately, it's not really feasible with GDI+ as GDI objects must only be used on the thread they're created on. It may be worth considering with Skia, although Skia may use object pools internally.

Hi @EmanuelFeru, @bclehmann,
A lot of performance optimizations have been made.
In particular, Signals use multithreading by default, which is why the additional Parallel loop does little to nothing.
And at the moment, in fact, the bottleneck in ScottPlot is GDI+. It does not use hardware acceleration to draw lines in any way. Therefore, each signal from its 1000+ lines adds its additional 10-50 milliseconds to rendering.
The transition to another library is always in my head, but all the time it is postponed. Since this is not a simple matter, and the current performance of the ScottPlot is quite satisfactory for most even heavily loaded applications.

As for ScottPlot, there's been talk for a while of moving to Skia, which is anticipated to have better performance than System.Drawing.

There is a positive experience with SkiaSharp #114, but it is already very outdated, at the moment there may be something simpler and more interesting.

I will do some more tries. For now I will close the topic. Thank you for your answers.

Hi Scott,
It's really good to see that you found time to work on Scottplot again. I'm reopening this topic because I just read your full discussion in #114 and in my opinion there is a lot of potential in using something like SkiaSharp  or SharpDX
As I said I used a tool in the past based on SharpDX and worked fantastically well, even with huge data points and many signal. On top, no difference was felt when going to large screens.
At the moment, with GDI+ I do feel a drop in speed when I go or maximize to a big screen, even with 0 signals in the plot (only my usual vertical Draggable line).
It would be great I would really like to encourage to continue on SkiaSharp, I am sure something great will come out. Processing power is there we just need to use it. I can be the first one using ScottPlotSkia.

It is quite amazing what it can do:
https://skia.org/docs/user/modules/particles/
Let's resume Skia developments :)

Linked to #1036

@EmanuelFeru a while back I did some experimentation with SkiaSharp and plotting and it was fantastic performance https://github.com/swharden/quickplot
The problem at that time was the build process was non-trivial (compared to System.Drawing which just seemed to work everywhere).
With #1036 and discussion emerging there this issue is on its way to being solved! I'll close this issue for now, and hopefully tackle this around the time .NET 6 and Microsoft.Maui.Graphics.Skia is out of preview 👍


client/coap.py: remove logging init

Current code creates file logging.conf in current dir, which is an unexpected (and unnecessary) side effect.

Coverage increased (+0.01%) to 90.104% when pulling c7379f509639cf4f6a6a6176f6def4b15d118110 on iot-lab:fix_client_logging into 379b5c6fc02ae968851cabfe58b3660cfd8945de on Tanganelli:master.


changing HashMap to LinkedHashMap to maintain execution order of deployment

currently deployment order is not maintain, in order to overcome this we can change HashMap to LinkedHashMap.
So that the deployment order in  maintained.

Can someone review this.

That is a backwards incompatible change so, if made, it will only be made to the 10.1.x branch.
Web applications are meant to be independent. If there are dependencies between two or more web applications then those applications should be coded so that they correctly handle starting in different orders.
Generally, the Tomcat project does not implement features that enforce this sort of ordering as it undermines portability.
I am close to -1 on this proposal but will leave the PR open for others to comment.

I agree to your point, but again since web applications are independent many applications may be running in order.
So I hope by doing this it doesn't affect the existing feature and it will add more advantage to it.

What exactly does this accomplish? LinkedHashMap just produces iterators which maintain insertion-order. It doesn't impose any order on when they were inserted.
Is this intended to maintain redeployment-order when auto-deployment is enabled and all applications are updated simultaneously? If so, it's entirely possible that your application updates would be processed across multiple background-processing executions and therefore wouldn't necessarily be redeployed in the order you expected, anyway.
I must not understand the use-case, here.

No response to Chris's questions so closing this as WONTFIX. Maintaining deployment order of applications is something probably best discussed on the users list in the first instance.


[Question] Code for MEGA/OMEGA

Hi, I'm really impressed by the result obtained in https://arxiv.org/abs/2007.02832, congratulations. I would like to test this approach in a navigation environment with obstacles (HER fails). Can you provide the code? Thanks.

Hi, the experiment directory for MEGA is here: https://github.com/spitis/mrl/tree/master/experiments/mega, and the commands to actually run the experiments are in this file: https://github.com/spitis/mrl/blob/master/experiments/mega/commands_for_experiments.txt. Happy to assist further if you have difficulty adapting the code.

That's awesome, thank you for the quick (light speed) response!


Enable Developer Tools or Inspect for theming

A Front End Developer here. I would love to have an option to be able to use developer tools to make it easier to make a custom CSS theme, since I'm not an app developer and you have already provided a way to upload your own theme. I didn't see Inspect feature anywhere, so could you add it? If not, what methods do you recommend for creating your own theme?

Use "Control + Shift + G" (or command on macOS) and type in "DEV_MODE" then click "Go"
The program will relaunch and you can then use the standard dev tools shortcut to launch the dev tools.  To open the devtools for Google Play Music you need to run the command openGPMDevTools() in the console of the first dev tools window 👍

Oh. Didn't found this in the documentation. Enlightening, thank you!

Yeah I should probably add it to the FAQ somewhere


Table is not getting formatted

1- installed the plugin
2- restarted Atom
3- added the table:
| Number_of_Players |Type_of_Players|
| 10 | Fat|
| 10 | Skinny|
4- Selected the table
5- Did CTRL+SHIT+P, followed by searching for Gherkin Table Formatter: Format and pressed the button
Result:
Table is not formatted.

The default keybinding is alt shift t.  The package relies on a gherkin grammar scope being installed, I'm currently using the "cucumber" package that includes one.

Table is not being formatted for me too.

Found it! dont forget to add the colon ad the end of a step ie.
When this happens:
|username|password|
|sdsd|asdasd|
then alt+shift+t


Fix invalid level error handling

Fixes #186. This was caused by the wrong type of exception being thrown. Also added a test to assert that invalid levels are being handled correctly.

Not sure if we agreed on this, but are we allowing blank fields in level and subject to denote empty?
To put it another way, we currently allow blank fields in address and phone for example, so a command like
add n/person p/ is accepted, and it treats the phone as blank
are we intending to have consistency across all fields or are we leaving it special for level and subject and if so, why?

@TopKec Yes we should be allowing blank fields in level and subject. That was done in #195 (but it may not be reflected in this PR until merged).

Alright, in that case I'd assume a subject version of allowing empty subjects will come soon


Add setup_shard_creator

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

Project coverage is 100.00%. Comparing base (617ed09) to head (78c4c0a).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #97   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           57        57
  Lines         1173      1186   +13
  Branches        92        94    +2
=========================================
+ Hits          1173      1186   +13

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


cnvkit -> pandas dependency issue

Hello Brad,
I had to do a manual pandas update from 0.18.1 to 0.19.2 due to an error AttributeError: 'Series' object has no attribute 'is_monotonic_increasing' originating from cnvlib.genome.subtract.py. That attribute is new in 0.19.1 and cnvkit seems to require it. I am using the  cnvkit from conda that bcbio installed (also, there was no newer version available through conda). Could there by any dependency issues either in cnvkit or bcbio? Mentioning @etal here as well.

Sven-Eric -- sorry about the issue. I'll bump the minimum pandas version requirement for cnvkit to be sure it installs an up to date version is is_monotonic_increasing included. I'll follow up here when the updated package is in bioconda.

Eric - I've been working on this conda package and realized it's a bit challenging to move to 0.19.x right now. bioconda builds against numpy 1.10 and there are only conda packages for pandas built against numpy 1.11 and 1.12, so trying to increase the pandas version causes conflicts and will lead to needing to rebuild other packages with updated numpy. Is there a larger dependency on having 0.19.2 or could we adjust CNVkit to avoid needing this 0.19.1+ only function?

OK, I think I can work around it. The purpose of that line is to safely handle nested intervals (which happen sometimes in users' target BED files and more often in gene annotations), in particular to decide whether it's safe to take a fast path to compute interval intersections. I'll add a check for that attribute and if it's missing, always take the slow path.
Is there an ETA on when it's safe to bump the minimum pandas version from 0.18.1 back to 0.19.1+, or some way I can watch for it? The pandas API moves faster than numpy and I would prefer to minimize the range of expected behaviors.
I'm working up to properly triaging chapmanb/bcbio-nextgen#1799, and once that looks OK I'll roll another point release of CNVkit including both that and the pandas shim.

Eric -- thanks for the quick fix, I'll update the cnvkit conda package with that as a temporary workaround. I'm not sure what it entails to build numpy so raised an issue to discuss bioconda/bioconda-recipes#3961 and will try to get that in place for forward usage. Thanks again.

I've updated the bioconda package with Eric's fix to support older pandas version so that should be good to go now. We'll work next on updating numpy in bioconda so that we can also use newer versions of pandas as the longer term fix. Thanks again for all the help with this.


No documentation on handling return values from calling a hub method from client

Title basically says it all. E.g. what if "SendMessage" would return a bool indicating success? Where can I handle the return value?

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

ID: 3cf283c9-5cd2-a9ff-b8c8-372d8a04356d
Version Independent ID: 729fddcf-24e4-b563-4952-a8045b231868
Content: ASP.NET Core SignalR JavaScript client
Content Source: aspnetcore/signalr/javascript-client.md
Product: aspnet-core
GitHub Login: @rachelappel
Microsoft Alias: rachelap

Meanwhile I managed to figure out that if I have async Task<string> MethodWithReturnValue() in the Hub I need to use  connection.invoke("MethodWithReturnValue").then(function(retval){/*do thing*/});,  but it'd be nice if there would be a code sample for this.

@anurse Should we add this?

@anurse 	let me know when you have sample code and we'll get this added.

Yes we should add this.

@anurse 	can you provide a sample that @tdykstra 	can add?

Same is needed in .NET (which also doesn't include specific info about catching exceptions)
JavaScript:
connection.invoke("Add", 40, 2)
    .then((result) => alert("40 + 2 = " + result))
    .catch((err) => alert("An error occurred: " + err));

.NET:
try
{
    result = await connection.InvokeAsync("Add", 40, 2);
}
catch(Exception ex)
{
    // Handle exceptions
}

I've been following all the documentation to implement the JavaScript client in one of the solutions I have and I think it is not clear what to do in terms of registration and existing handlers when a disconnection occurs. It's really good the start example but in my opinion it is not enough since I'm not entirely sure what's the best practice here to either reconnect or start a fresh connection by de-registering everything and start all over again. I take from the docs that a simple start would be sufficient?
Thanks,
Carlos.

Filed https://github.com/aspnet/AspNetCore.Docs/pull/12015 for the JS docs.


adding details to the contact form

👷 Deploy Preview for luxury-sherbet-3b08a4 processing.

Name
Link

🔨 Latest commit
736b64717b06b0affc5face0ca944b20bb87b2d6

🔍 Latest deploy log
https://app.netlify.com/sites/luxury-sherbet-3b08a4/deploys/65d1e6bb0c0e560008ebdb37


Migrating from GitLab

I am migrating this repo to have it along with other open-source SD tools and as a test for migrating PySD. The migration is still in process.

[x] pep8speaks
[ ]  coveralls
[ ]  ReadTheDocs
[ ]  conda-forge feedstock

Error when trying to add the webhook integration in RTD
Could not add webhook for excels2vensim. Make sure you have the correct GitHub permissions.
The project excels2vensim doesn't have a valid webhook set up, commits won't trigger new builds for this project. See the project integrations for more information.


fix compiling without gstreamer, libpeas

I wasn't able to compile the latest master on GNU/Linux without the optional dependencies gstreamer and libpeas installed.
It works with these changes.

Looks good, thanks.


Spark client can create/alter/drop a view

What changes were proposed in this pull request?
Apart from super users, normal users can create/alter/drop views via HMS

Why are the changes needed?
Even normal users can create/alter/drop views

Does this PR introduce any user-facing change?
No

How was this patch tested?
On local cluster and Unit tests. New unit tests are added.

Merged the patch into master. Please create followup JIRA for the test fixes as discussed.


docs: Fix typo

Correct a spelling mistake.
Fixes: #309.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

/test

Does this wfy @grahamwhaley ?


Update standard to v17

Rebased

Something is flaky with the Windows build on CI (happens almost on every PR).

This change is relatively big and doesn't really provide anything new so if it's OK let's skip this for now, at least in terms of releasing prebuild v13.

Sure, I don't mind.

Rebased.

Rebased.


Introduce computeGenTxSize

Closes #700.

@dcoutts I addressed your remark, can you approve so I can go ahead with #739?


Feature/group talk ai care

Codecov Report
Attention: 135 lines in your changes are missing coverage. Please review.

Comparison is base (902bd1d) 13.98% compared to head (871aa66) 13.78%.

Files
Patch %
Lines

src/gptui/models/jobs.py
0.00%
60 Missing :warning:

src/gptui/models/role.py
0.00%
44 Missing :warning:

src/gptui/models/handlers.py
0.00%
13 Missing :warning:

src/gptui/controllers/openai_chat_manage.py
0.00%
12 Missing :warning:

src/gptui/controllers/notification_control.py
0.00%
3 Missing :warning:

src/gptui/views/tui.py
0.00%
2 Missing :warning:

src/gptui/plugins/GroupTalk.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #29      +/-   ##
==========================================
- Coverage   13.98%   13.78%   -0.20%
==========================================
  Files          80       80
  Lines        7681     7790     +109
==========================================
  Hits         1074     1074
- Misses       6607     6716     +109

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


Create returnLessThanOrEqualToZero.cpp

Description
Program Name - Is The Number less than or Equal to Zero?
Issue No.- #156
Checklist(self check)

[ ] Yes, this task assigned to me.
[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] My Filename is in camelCase format
[x] I have added Problem statement, Problem link in file.

sorry, unassigned task.


Fix/date order

Fixes tab ordering issue and also ignores the file that I end up creating during interactive work

Merged manually to fix the conflict


Migration/market - test_build_creates_position

Adds the first test case of Market.
Uses Solady's FixedPointMathLib to verify FixedPoint.sol
Adds recommended plugins for VSCODE
Modifies foundry.toml

I took a quick look and wanted to say that maybe we should use camelCase everywhere instead of keeping the snake_case from Python.

@EperezOk I agree but maybe at the end when the code is reviewed, cause this way is easier to compare brownie tests against foundry test, but I also copied the comments, so maybe you are right and I can change it. I like your idea but lets look what Tomas says.


docs: Clean up README for Jakarta

Upgrade the ref of Walkthrough from 1.3 to 2.1 docs
Correct the edgex-compose link for downloading docker-compose file
Correct the broken link on LTS Policy
Close https://github.com/edgexfoundry/edgex-go/issues/3795
Signed-off-by: Cloud Tsai<EMAIL_ADDRESS>

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) N/A, only update README
[ ] I have fully tested (add details below) this the new feature or bug fix (if not, why?) N/A, only update README
[ ] I have opened a PR for the related docs change (if not, why?) N/A, only update README

Codecov Report

Merging #3817 (32cb9f1) into main (f7c7132) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3817   +/-   ##
=======================================
  Coverage   45.77%   45.77%
=======================================
  Files         112      112
  Lines        9640     9640
=======================================
  Hits         4413     4413
  Misses       4844     4844
  Partials      383      383

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 f7c7132...32cb9f1. Read the comment docs.


feature: Allow specifying import aliases in plugin config

Before this PR
Generated code chose default aliases which can conflict with hand-written ones and fail the importalias lint check.
After this PR

==COMMIT_MSG==
Allow specifying import aliases in plugin config
==COMMIT_MSG==
Possible downsides?

Released 0.41.0


import and not found any module in entry.js

when i import any module in entry.js, i get an error
'This relative module was not found:
./router in ./.temp/index.js, ./.temp/components/bill/index.js and 2 others'
i can't find 'router.js' in '.temp', but i have a 'router.js' in 'src'

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


Cube bounding box for 3 dimensions

Related Issue:
17
Description
This PR  adds the Cube as a bounding box that can act as an envelope in 3 dimensions. It also contains tests to ensure the correctness of the Cube bounding box.

@kushalpokharel I think some of these review comments look good to me, shall we incorporate them ?

@patricoferris I have incorporated those changes with the above commit, please review. sorry for the delayed response

Thanks LGTM :))


Add GltfLoader::new.

Objective
In my application, I'm manually wrapping the built-in Bevy loaders with a wrapper loader that stores some metadata before calling into the inner Bevy loader. This worked for the glTF loader in Bevy 0.10, but in Bevy 0.11 it became impossible to do this because the glTF loader became unconstructible outside Bevy due to the new private fields within it. It's now in fact impossible to get a reference to a GltfLoader at all from outside Bevy, because the only way to construct a GltfLoader is to add the GltfPlugin to an App, and the GltfPlugin only hands out references to its GltfLoader to the asset server, which provides no public access to the loaders it manages.
Solution
This commit fixes the problem by adding a public new method to allow manual construction of a glTF loader.

instead of adding a new method, I see no good reason to not make the field public

This should be ready to merge, I think.


Jenkins integration (CI integration)

Is your feature request related to a problem? Please describe.
I am currently using Jenkins to work on new project and automate the CI part of the project. I can't seem to find aws-amplify/cli plugin in Jenkins.
Describe the solution you'd like
A plugin for aws-amplify/cli to use with Jenkins
Describe alternatives you've considered
A way to integrate or use aws-amplify/cli with Jenkins
Additional context
I am not quite sure how to integrate aws-amplify/cli with Jenkins to automate the build and deploy and any of the commands that aws-amplify/cli can do.
I checked the headless mode but I am not sure how Jenkins will understand the amplify commands without installing aws-amplify/cli plugin - which is not available.

@totaland I have typically seen CI systems install the CLI on every build. So you can use npm install -g @aws-amplify/cli to install the CLI as a part of your build step.

@kaustavghosh06 thanks mate it works


Update develop.md

Proposed changes

provided more clarity to the structure of the mongo database connection string used as CONNECTIONSTRING environment variable to run the docker image in the sample node.js project.

Close and reopen the PR to trigger CI checks


Curriculum update needed on flexbox-alignment.md

The Odin's file, flexbox-alignment.md is updated. Please update the Kampus' file, checkout file here flexbox-alignment.md
Latest commits:

Flexbox Alignment: Mention to not focus on media queries in linked article's examples (#27184) (additions: 17, deletions: 17) on Jan 24 2024, 24:57 UTC
Merge pull request #27100 from fabulousgk/fabulousgk/issue27075 (additions: 1, deletions: 1) on Jan 13 2024, 14:49 UTC

Dublicated issue


modestBranding option set to true does not hide the youtube logo

setting this property doesn't seem to change anything. I tried camel case as specified in the readme, and also tried modestbranding all lowercase, neither hides the branding.
Thanks

Ive got the same issue! :(

Oh, I figured this out, should have posted an update. Setting modest branding to true does nothing  when showinfo is false, the two are mutually exclusive. When show info is true then setting modestbranding true hides a youtube link that shows up as part of the info bar. Basically, youtube is forcing you to either show the info bar at the top, or the youtube logo bottom right, you can't hide both of them. This is a limitation of the youtube iframe player itself, not of the react-native-youtube component.

I am unable to hide the youtbe button which is present at the bottom right of the video. Even when i set
modestbranding={true} and showinfo={false} still I am not able to remove the youtube button ?


Increment/Decrement Elapsed Time Shortcuts

[x] Increment by 1 minute using Ctrl + W/Up Arrow
[x] Decrement by 1 minute using Ctrl + S/Down Arrow
[x] Increment by 5 minutes using Alt + W/Up Arrow
[x] Decrement by 5 minutes using Alt + S/Down Arrow

Ended up using shift to increment/decrement by one, since ctrl + S was already used for restoring last stopwatch time state. Also Ctrl + W was being used to close the window


Composition of ValueChoice

Description
This PR adds support of composition (transformation) of one or several value choices. For example,
nn.ValueChoice([1, 2]) + nn.ValueChoice([3, 4]) + 5

Checklist

[x] test case
[ ] doc: pending #4399 to avoid conflicts.

How to test
See test_highlevel_apis.py.
Adding a full example will be the next task.

@ultmaster  in which cases this composition is useful?

@ultmaster in which cases this composition is useful?

For example, search for a channel scale factor.
class Module:
  scale = nn.ValueChoice([1.0, 1.5, 2.0])
  self.conv1 = nn.Conv2d(3, round(scale * 16))
  self.conv2 = nn.Conv2d(round(scale * 16), round(scale * 64))
  self.conv3 = nn.Conv2d(round(scale * 64), round(scale * 256))

  # forward
  return self.conv3(self.conv2(self.conv1(x)))

Or when kernel size and padding are coupled, in order to keep the output size constant.
class Module:
  ks = nn.ValueChoice([3, 5, 7])
  self.conv = nn.Conv2d(3, 16, kernel_size=ks, padding=(ks - 1) // 2)

  # forward
  return self.conv(x)

Or when several layers are concatenated for a final layer.
class Module:
  self.linear1 = nn.Linear(3, nn.ValueChoice([1, 2, 3], label='a'))
  self.linear2 = nn.Linear(3, nn.ValueChoice([4, 5, 6], label='b'))
  self.final = nn.Linear(nn.ValueChoice([1, 2, 3], label='a') + nn.ValueChoice([4, 5, 6], label='b'), 2)

  # forward
  return self.final(torch.cat([self.linear1(x), self.linear2(x)], 1))


Add scripts to start and populate an instance of Gerrit

Uses openfrontier/gerrit:2.13.x
Will generate an SSH key that can be used by the bot
Creates 2 fake users
Clones this repo inside Gerrit
Creates an empty review to play with

Thanks a lot. This makes integration testing much easier.


feat: support new dry run interface

Closes: https://github.com/FuelLabs/fuels-ts/issues/1700

Blocked by the next fuel-core release that will include: https://github.com/FuelLabs/fuel-core/pull/1631

Closing this in favor of https://github.com/FuelLabs/fuels-ts/pull/2021


Import Issue on codebases with both module and common js files

Hello! Let me start by saying this is a great fork, I love the isWithinTokenLimit utility!
We are having an issue in my organization when using this library, our codebase is split between commonjs and module javascript files, and for reasons outside of the scope here, we cannot define our project as modules on package.json.
The issue is as follows: When we import the library from an .mjs file, node tries to import the files under /esm/.js as common js files because of the extension, so it throws an error about invalid syntax.

This doesn't happen when importing from a *.cjs file

We managed to fix by "proxying" the library through a .cjs file, which works but it is quite unelegant.
Possible solution: The generated files under /esm/ could have the *.mjs extension, so node can infer the proper type of file by the extension alone or something like this https://github.com/microsoft/TypeScript/issues/18442#issuecomment-1377529164
How to reproduce: https://github.com/gbrlmtrz/minimum-reproducable-tokenzier/tree/main  on node 18 and 16

For what it's worth - I'm experiencing a similar problem but the other way around. We use modules in a typescript environment and it can't find the exports :\

Thanks for reporting, should be fixed in the next version.

For any one else hitting this in TypeScript but using ESM, try importing like this:
import { encode } from 'gpt-tokenizer/cjs/main'


Coords.py

Partial implementation of #114 and #124.
Moved coordinate transformations from utils.py to coords.py.
Added inverse capability to geodetic_to_geocentric transforms.
Added unit tests to check that running a transform and its inverse will produce the original inputs.

@aburrell, can you look over the unit tests?  Everything in coords.py should be tested, but we may want to expand testing based on earlier versions.  Specifically:

The 'targets' in test_local_horizontal_to_global_geo used the same function to generate.  Is there an external calculator for checking these?
The other functions all use reversibility as the test, ie, does applying the inverse of the test get you back to starting point.  Could add single transforms similar to cartesian to spherical tests.
Could add array versions of all these tests.
Anything else I missed?

Note: Leaving this here for now while we discuss #124.  Once we have a plan forward, will close and reopen a new pull request.

Closing in lieu of upcoming major update to utils (see #124).


Adds pills type selection and pills canister

About the PR
Adds pills type selection in ChemMaster window and pills canister entity. You can click on tile with pills canister and it will collect all nearby pills, holds up to 12 pills.
Screenshots

Changelog
:cl: artak10t

add: Added pills types selection in ChemMaster
add: Added pills canister

For naming pills w/o hand labelling each individually and showing popup I'll work on it in different PR.


Tab Styling Issue Fix

Right now, the TabPanel has overflow set to visible because of being classed as jupyter-widgets. The same styling is set explicitly for the TabBar. But the TabPanel contents has overflow set to hidden.

@jasongrout @SylvainCorlay

On the other hand, perhaps this calls into question why jupyter-widgets sets overflow to visible. Perhaps it shouldn't override phosphor's override: hidden?

On the other hand, perhaps this calls into question why jupyter-widgets sets overflow to visible. Perhaps it shouldn't override phosphor's override: hidden?

As far as I remember, several widgets in the widgets list were broken if we did not do so. It was a big difference with the earlier version.
In any case, if we make it "hidden", tooltips in bqplot figures will be come a lot harder to make work in the classic notebook at the same time as in JLab...

In any case, if we make it "hidden", tooltips in bqplot figures will be come a lot harder to make work in the classic notebook at the same time as in JLab...

IIRC, the tooltips in bqplot are done 'right', in that they are absolute positioned divs at the top layer, not relatively positioned divs next to the plots, so they shouldn't be affected at all.

IIRC, the tooltips in bqplot are done 'right', in that they are absolute positioned divs at the top layer, not relatively positioned divs next to the plots, so they shouldn't be affected at all.

Not anymore, they are in the figure's div at the moment.

@jasongrout @SylvainCorlay The tooltips right now are placed in the same div as the svg. So changing the overflow to hidden will affect the tooltips, i.e., they will be clipped.

@SylvainCorlay @jasongrout so what is the final consensus on this?

We're currently factoring out the tooltip logic used in jupyterlab so it can be used in other projects.

I think we should keep it the way it is, and maybe even set other containers to overflow:hidden...

Closing as a 'we should fix the real issue with tooltips'

Along with tooltips, we should also make TabPanel consistent with the rest of the containers. I thought that was the main issue here :)

Good call. I think we ought to test taking off the overflow:visible.


Update user page with total hours and join date

Objective
This PR updates the user page to include details about how much hours the user has in total and the date they joined Hours
Related Issue
See related issue here
Screenshots
Total hours change with filter, just like the charts.

Looks good to me, could you add some tests for it?


Permissions error on Wiki build webhook

The gollum_hook, which is responsible for triggering a website rebuild every time the wiki is updated, is returning an error every time Github attempts to send a payload:
(source)
Travis::Client::Error
      in GithubController#gollum_hook

...

You do not seem to have sufficient permissions.

Looks like some sort of authentication error between metasmoke and Travis?

It appears that we now use GH actions instead of travis for this so I assume this is resolved


need to re-generate byte code for intl

@curtisman can you review?

Please also Regen NoJIT bytecodes to make those checks pass as well.
@dotnet-bot test nojit tests please

this is not needed any more


Task 3 - Set up GitHub Pages

Add at least 3 tasks to Homepage project and convert them to issues (every issue should be linked to the homework's repository)

Go to repository Settings and set up GitHub Pages (from the main branch), update repository description with the proper website link. Follow the link to check if everything is fine - you should see your index.html content.

Done!
https://unitys22.github.io/homepage/


Token Logo Update

Please approve my PR #8637 I've changed the logo @keone
Thank You.

can you please rebase to main?

can you please rebase to main?

okay i will do that
Thank You


feat: make optional tech reviews setting

Description
This PR introduces an optional setting for technical reviews in the proposal workflow.
Motivation and Context
The change is required to provide flexibility in the proposal workflow, allowing certain proposals to bypass the technical review stage based on their workflow status. This feature is particularly useful when only a subset of proposals require technical review.
Changes

A new setting TECH_REVIEW_OPTIONAL_WORKFLOW_STATUS has been added to the settings table, which defines the workflow status that triggers the creation of a technical review.
The PostgresProposalDataSource class has been updated to check this setting when determining if a proposal needs a technical review.
The e2e tests have been updated to accommodate the new optional technical review setting.

How Has This Been Tested?

Fixes Jira Issue
https://jira.esss.lu.se/browse/
Depends On

Tests included/Docs Updated?

[ ] I have added tests to cover my changes.
[ ] All relevant doc has been updated

In general it looks great! I left some comments that might need little attention but otherwise I am happy to approve. I hope for us at ESS this setting will be empty by default right? And it will include all proposals for tech reviews

Cool, thanks, that was the idea if it is null or "" it should make a tech review.


FrameReceiverAdapterController should have _unique_config set

This will prevent creating config fan out PVs for configs that should be unique between processes, e.g. ctrl_endpoint
FrameProcessorAdapterController does this here.

I think it would make sense to have _unique_config on each specific class, rather than on the root adapter controller.


JQuery requires version bump

What version of UAA are you running?
74.21.0
How are you deploying the UAA?

using a bosh release

What did you do?
Call the following URL via browser:
https://<uaa URL>/vendor/jquery/javascripts/jquery.js
What did you expect to see? What goal are you trying to achieve with the UAA?
Newest JQuery version (currently 3.5.1)
What did you see instead?
Older JQuery version (3.4.1)
The UAA file that requires the version update is:
https://github.com/cloudfoundry/uaa/blob/develop/uaa/src/main/webapp/vendor/jquery/javascripts/jquery.js
Note:
It seems that JQuery 3.5.1 is not completely backward compatible:
http://blog.jquery.com/2020/04/10/jquery-3-5-0-released/
The jQuery team agreed it was necessary to fix this in a minor release, even though some code relies on the previous behavior and may break.

Thanks for the report @tack-sap, we should see if dependabot can be used to keep this up to date.  Until that happens, if you'd like to submit a PR we'd happily accept it.


An argument named raises confuses ruff's pydocstyle rules

Running:
⊙  ruff check --select D417,D214,D405 =(printf '
def f(a, raises) -> None:
    """
    Do really important stuff.

    Arguments:

        a:
            a super important integer

        raises:

            uh oh, some exceptions!
    """
')

produces:
/tmp/zsh82UkZp:2:5: D417 Missing argument description in the docstring: `raises`
/tmp/zsh82UkZp:3:5: D405 [*] Section name should be properly capitalized ("raises")
/tmp/zsh82UkZp:3:5: D214 [*] Section is over-indented ("raises")
Found 3 errors.

where ruff is expecting raises: to be a Raises: Napoleon block, but it isn't, it's simply the name of the argument, which indeed is described and properly capitalized.

I think this is consistent with pydocstyle. If I run pydocstyle foo.py --convention=google, I get:
foo.py:1 at module level:
        D100: Missing docstring in public module
foo.py:2 in public function `f`:
        D212: Multi-line docstring summary should start at the first line
foo.py:2 in public function `f`:
        D405: Section name should be properly capitalized ('Raises', not 'raises')
foo.py:2 in public function `f`:
        D214: Section is over-indented ('Raises')

The issue is the space between arguments which AFAICT is not "compliant" with the Google spec, although I'm not sure that part is "formally" defined anywhere. E.g., see this example from the docs:
def fetch_smalltable_rows(
    table_handle: smalltable.Table,
    keys: Sequence[bytes | str],
    require_all_keys: bool = False,
) -> Mapping[bytes, tuple[str, ...]]:
    """Fetches rows from a Smalltable.

    Retrieves rows pertaining to the given keys from the Table instance
    represented by table_handle.  String keys will be UTF-8 encoded.

    Args:
        table_handle: An open smalltable.Table instance.
        keys: A sequence of strings representing the key of each table
          row to fetch.  String keys will be UTF-8 encoded.
        require_all_keys: If True only rows with values set for all keys will be
          returned.

So, e.g., this works as expected:
def f(a, raises) -> None:
    """
    Do really important stuff.

    Arguments:

        a:
            a super important integer
        raises:
            uh oh, some exceptions!
    """

I'm gonna close just because these regexes are hard to get right and the goal is mostly to be consistent with pydocstyle right now (until we implement a formal parser for docstrings), so if pydocstyle behaves the same way, I don't want to mess with the regexes too much.


Update cti.yml

Can one of the admins verify this patch?

Closing as re-created the PR as #198 and merged.


fix(natds-icons): add icons

DSY-2469
Description

filled-action-privacy
filled-action-register
filled-content-violence
filled-product-fashion
outlined-action-privacy
outlined-action-register
outlined-brand-avondesenvolve
outlined-content-violence
outlined-product-fashion

[update regression]


Report output to CSV

The problem
As mentioned in https://github.com/infracost/infracost/issues/811#issuecomment-861522629 I use infracost breakdown in a daily scheduled pipeline that generate a daily report about our infrastructure total approximated cost. For the moment, we have three state files that generates a uniform report in JSON. I send this uniform report via cURL to our Discord Channel (slack would work as well) and Google Sheets using Pipedream.
For the Discord webhook, this methods works fine, it's easy to parse a Discord message using nodeJS but inserting this JSON report in Google Sheets is very difficult.
My proposal
A CSV --format option for diff breakdown and output to make easier to insert these informations in a datasheet.

Awesome! This would probably work nicely with --fields too so you can select the CSV fields.
Any suggestions on what the CSV output could like for the breakdown and the diff? The diff might be challenging

Maybe something like this could work: simple example.

The output command's format is making me wonder if there's a need to split between format and report type, e.g. we have a breakdown and diff report types, and those can be in text, HTML or CSV format (so you have csv_diff, or html_breakdown). JSON has been our machine format so it has all the data required to create either of those report types.
Can you possibly share what roles in the company use the Google Sheets output, and what they do with it?

making me wonder if there's a need to split between format and report type

Yeah this is a very good point, but I wonder if we need different formats for the CSV diff and breakdown. We handle both already in the JSON, so maybe one CSV format that has both would work. For each resource we could show the current, projected and diff cost in a single row. Maybe something like this:

project
resource
costComponent
price
unit
monthlyQuantity
hourlyQuantity
monthlyCost
hourlyCost
pastMonthlyCost
pastHourlyCost
diffMonthlyCost
diffHourlyCost

my/tf/code

742.64
1.0173
0
0
742.64
1.0173

my/tf/code
aws_instance.my_instance

742.64
1.0173
604.64
0.8283
138.00
0.1890

my/tf/code
aws_instance.my_instance
Instance usage
0.77
hours
730
1
560.64
0.768
422.64
0.5789
138.00
0.1890

Similar for the HTML output, maybe. It could show the diff and then an option of expanding that to show the full breakdown.

Hah, you're right, we already have the fields in JSON so we can use the same in CSV. For HTML, do you mean an HTML tab for each project inside the HTML page? We can default to the breakdown tab being visible.

do you mean an HTML tab for each project inside the HTML page? We can default to the breakdown tab being visible.

Yeah something like that could work. Or showing the diff as a summary at the top and then a bit you can expand to view the whole details.

Can you possibly share what roles in the company use the Google Sheets output, and what they do with it?

I explain my two use-cases in this comment here: https://github.com/infracost/infracost/issues/811#issuecomment-861522629
In this case, i'm planning to append our costs daily into to a Google Sheets using this pipedream integration and handle this sheet to our non-profit financial people so they can keep a history of the expected costs of the 3 environments we have as we create the resources (the project is in early stages).
Here's an example from the top of my head to what I'm expecting this sheet from finances to look like:

Date
Global $/h
Global $/month
Dev $/h
Dev $/month
Main $/h
Main $/month
Total $/h
Total $/month

18/06/2021
$0.5
$360
$0.5
$360
$0.5
$360
$1.5
$1080

19/06/2021
$0.6
$432
$0.5
$360
$0.5
$360
$1.6
$1152

20/06/2021
$0.6
$432
$0.55
$396
$0.5
$360
$1.65
$1188

@1mamute awesome - thanks! I've added this to the project board so we/community can get to it when there's bandwidth.


[WFLY-17605] Add in topology update wait time to stabilize before beginning test to avoid race condition.

This PR does the following:

for the test case TwoConnectorsEJBFailoverTestCase, which tests a single EJB client interacting with cluster members via the remoting protocol (on port 4447) and the remote+http protocol (on port 8080), adds in a delay to the first client interaction with the clustered deployment, to allow the cluster time to stablize when run on slow servers and prevent the client invoing on the cluster before it is ready.

For more details, see: https://issues.redhat.com/browse/WFLY-17605

Adding hold label.  I'll trigger a few runs to verify stabilization.

@rachmatowicz @pferraro Any news on this one?

I think I triggered extra runs but forgot to verify them.  I'll have to go through the run history to verify (or re-run if those are lost).

@pferraro @rachmatowicz Can we get this one through?

@bstansberry @pferraro @rachmatowicz Here's the solution to this problem of now knowing if this is stabilized - opened https://github.com/wildfly/wildfly/pull/18213 which incorporates this stabilization fix and marks the test as intermittent so that we can continuously test this in our clustering lab before enabling on upstream CI.

The change has been incorporated. This can be closed now.


feature/CPLP-1555-CreateOfferAssigned-AppLeadImageDocument

https://jira.catena-x.net/browse/CPLP-1555

currently on hold since there was a smaller change to the logic in the task


The replies indicator line is too low contrast to discern

Describe the bug
On iOS (at least), the line connecting profile picture to indicate a reply is too low contrast to reasonably discern:

To Reproduce
Steps to reproduce the behavior:

Reply to a post
Navigate to post on iOS, observe line

Expected behavior
Contrast should be higher, or indicator should be thicker/redesigned
Details

Platform: iOS (maybe other mobile; very screen dependent)
Platform version:
App version: 1.68.0

I can't actually tell. Is this dim or dark? We're about to fix up dim right now and hopefully get it released soon, but if this is dark we might need to look at that as well.

The screenshot is from dim. Dark is a little better:
Dark:

Dim:

This is a biggish a11y issue

We made some changes to contrasts in this upcoming release, going to leave this open though for feedback after 1.69 comes out, so feel free to respond or @ when it does.

This is much better for me in 1.69 - my concern in raising the ticket has been addressed.
Not sure if anyone is able to comment on the accessibility aspect?

@jghiloni Pinging you since you had mentioned it. Curious if you have any thoughts on whether this is better and/or if there are any additional changes that could help?

Orders of magnitude better, thanks!


How can I change the units 'm' to 'mm' for ERA5 precipitation dataset.

Generic Issue

xclim version:0.46.0
Python version:3.11
Operating System:Linux

Description

I download a ERA5 daily_mean precipitation dataset. I want to use this dataset to calculate the AnnualPrecipitation. I use the command as below. Here is the information of precipitation dataset after I revise the units. Before the changing activity, the units is 'm'.

What I Did

pre_ERA5.tp.values = pre_ERA5.tp.values*24*1000
pre_ERA5['tp'].attrs['units'] = 'mm'

What I Received

ValidationError: Data units millimeter are not compatible with requested [precipitation].

What I found
I don't konw why I got this error, even I changed the units to 'mm'. I want to know how can I fix this problem.
Then I search the xclim user guide and find the function xclim.core.units.convert_units_to(source, target, context=None)
Can I change the units of precipitation like:
pre_ERA5 = xclim.core.units.convert_units_to(pre_ERA5, 'mm', context=None)

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @CGL5230 !
This is because xclim calls "precipitation" the precipitation flux. We follow language from the "Climate and Forecast" (CF) conventions where "flux" is a mass per area per time, usually "kg m-2 s-1". Xclim can also receive a "rate", i.e. "length per time", like "mm/day", because the transformation from rate to flux is always the same for liquid water : multiply by the density of water which is 1000 kg/m³.
In your case, the data you have seems to be a total thickness of liquid water for the current timestep, which seems to be a day. If you confirm that the dataset is daily then an explicit and easy way out would be to patch the units like this:
pre_ERA5['tp'].attrs['units'] = 'm / day'

Xclim will convert to "kg m-2 s-1" under the hood.
But in this precise case, convert_units_to should be able to handle it, you simply need to request a precipitation flux unit:
pr = xclim.core.units.convert_units_to(pre_ERA5.tp, 'kg m-2 s-1')

Be aware however, that this works because the input has all needed metadata for xclim to understand how to handle the conversion! Xclim will read the standard_name  and understand that this variable speaks of liquid water. It will also look at the frequency of the data. It will then enable the conversion from  "[length]" to "[mass]/([area][time])".
pre_ERA5 = xclim.core.units.convert_units_to(pre_ERA5, 'mm', context=None)

Thanks for quickly reply. I use the command as pre_ERA5['tp'].attrs['units'] = 'm / day'  to change the units. And I got the waring like below when I run the ERA5_rx1day = xclim.indicators.atmos.max_1day_precipitation_amount(pr='tp',freq='YS',ds=pre_ERA5). I think the reason is the units  and variable name not the standard name for xclim. So,the result will be correct for R1XDAY index ?
/home/anaconda3/envs/py311/lib/python3.11/site-packages/xclim/core/cfchecks.py:41:
UserWarning: Variable does not have a `cell_methods` attribute.
 _check_cell_methods(
/home/anaconda3/envs/py311/lib/python3.11/site-packages/xclim/core/cfchecks.py:45: UserWarning: Variable has a non-conforming standard_name: Got `lwe_thickness_of_precipitation_amount`, expected `['precipitation_flux']`
 check_valid(vardata, "standard_name", data["standard_name"])

The warnings are indeed simply because the metadata of your dataset is not exactly what xclim expected, especially the standard_name that becomes wrong after our unit change.
If you use convert_units_to, the standard name will be modified as well and the warning should go away. But it's only a warning, and if you know that your data is valid, then it's safe to ignore.
I don't have access to the first link you gave, but from the metadata you showed, it seems that pre_ERA5['tp'] is the total precipitation accumulated over one timestep. So the liquid-water-equivalent thickness, in "m". To transform to a rate, you simply need to divide by the length of time of that timestep, which here is a single day.
precipitation_rate = total_precipitation  / 1 day

Which is why simply changing the units from "m" to "m/day" should work. I'm not sure I understand why you wanted to multiply by 24 ? That would mean the value is the mean hourly precipitation accumulation, which is not what I infer from the attributes.

Thanks for your quick reply. I offer the document link to show the precipitation dataset actually is daily mean dataset which would mean the value is the mean hourly precipitation accumulation. Because I want to get a daily-sum precipitation but hourly precipitation. The daily-sum precipitation can be used to calculate the annual precipitation.

Which is why simply changing the units from "m" to "m/day" should work. I'm not sure I understand why you wanted to multiply by 24 ? That would mean the value is the mean hourly precipitation accumulation, which is not what I infer from the attributes.


[BUG] [v1.5.5-rc1] rwx volume fails to mount on pod

Describe the bug
When creating a cluster on AWS using the ubuntu-jammy-22.04-amd64-server-20240301 image, the kernel version is 6.5.0-1014-aws.

Testing with Longhorn versions v1.5.5 and v1.5.4 on this kernel resulted in failure.
However, when installing to Longhorn v1.6.1 without updating the kernel, the issue was resolved, and RWX volumes could be successfully mounted on pods.
Updated kernel version to 6.5.0-1017-aws + Longhorn versions v1.5.5 and v1.5.4 passed successfully.

After discussing with @derekbit , for this issue, I believe we need to discuss whether to backport the v1.6.x NFS-Ganesha version to v1.5.x or continue recommending users to upgrade the worker node kernel version.
cc @james-munson @derekbit @innobead
Events:
  Type     Reason                  Age               From                     Message
  ----     ------                  ----              ----                     -------
  Normal   Scheduled               62s               default-scheduler        Successfully assigned default/ubuntu-mountvol0 to ryao-155-w3-86892745-sxf82
  Normal   SuccessfulAttachVolume  25s               attachdetach-controller  AttachVolume.Attach succeeded for volume "vol-0"
  Warning  FailedMount             3s (x6 over 25s)  kubelet                  MountVolume.MountDevice failed for volume "vol-0" : rpc error: code = Internal desc = mount failed: exit status 32
Mounting command: /usr/local/sbin/nsmounter
Mounting arguments: mount -t nfs -o vers=4.1,noresvport,timeo=600,retrans=5,softerr <IP_ADDRESS>:/vol-0 /var/lib/kubelet/plugins/kubernetes.io/csi/driver.longhorn.io/0507c11c5cae88bed1ea382c2b36f2faa90ff78759812a6c34599ece4336fb48/globalmount
Output: mount.nfs: Protocol not supported

To Reproduce

create a rwx volume vol-0
Create pod mount vol-0

kind: Pod
apiVersion: v1
metadata:
  name: ubuntu-mountvol0
  namespace: default
spec:
#  nodeSelector:
#    kubernetes.io/hostname: ryao-160-w2-d001851e-gl4nt # worker node
  containers:
    - name: ubuntu
      image: ubuntu
      command: ["/bin/sleep", "3650d"]
      volumeMounts:
      - mountPath: "/data/"
        name: vol-0
  volumes:
    - name: vol-0
      persistentVolumeClaim:
        claimName: vol-0

Expected behavior
The volume should get mounted normally and the pod start
Support bundle for troubleshooting
supportbundle_866bb13a-2fb6-40db-9dce-d70cb7c60c1d_2024-04-11T04-30-11Z.zip
Environment

Longhorn version: v1.5.5-rc1
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.26.15+k3s1

Number of control plane nodes in the cluster: 1
Number of worker nodes in the cluster: 3

Node config

OS type and version: Ubuntu 22.04
Kernel version: 6.5.0-1014-aws

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): AWS
Number of Longhorn volumes in the cluster: 2

Additional context
https://github.com/longhorn/longhorn/issues/6857
https://github.com/longhorn/longhorn/issues/6887

cc @PhanLe1010 @james-munson

The issue is caused by the regression in the NFS client of linux kernel6.5.0-1014-aws and fixed in 6.5.0-1014-aws.
I'm fine with either upgrading nfs-ganesha in v1.5.x or recommending users to upgrade the kernel (we don't need to do anything).
Before proceeding, I'd like to check @PhanLe1010 and @james-munson's opinion.

BTW, @innobead what's your suggestion for the issue?

@derekbit what's the regression issue?

Why does 1.6 work without the kernel fix?

@derekbit what's the regression issue? Is that specific to Ubuntu distro?

We don't know which commit causes the regression because we cannot find the source codes of 6.5.0-1014-aws for now.

What's the default kernel of 22.04

6.5.0-1014-aws for the AWS Ubuntu 22.04.

Why does 1.6 work without the kernel fix?

Well, I still have no idea about it.
Not sure if nfs-ganesha v5 is more resilient to the client error
[ 1128.246737] nfs4_destroy_session Destroy backchannel for xprt 000000005d180e7b
[ 1128.246741] RPC:        destroy backchannel transport
[ 1128.246742] RPC:        req=00000000667cb67b
[ 1128.246744] RPC:        free allocations for req=<PHONE_NUMBER>7cb67b
[ 1128.246746] RPC:        backchannel list empty= true
[ 1128.246762] RPC:       xs_tcp_send_request(116) = 0
[ 1128.255951] NFS: destroy per-net callback data; net=f0000000
[ 1128.255990] svc: svc_destroy(NFSv4 callback)
[ 1128.255998] nfs_callback_down: service destroyed
[ 1128.256055] RPC:        destroy backchannel transport
[ 1128.256056] RPC:        backchannel list empty= true
[ 1128.256058] RPC:       xs_destroy xprt 000000005d180e7b
[ 1128.256060] RPC:       xs_close xprt 000000005d180e7b
[ 1128.256090] RPC:       xs_tcp_state_change client 000000005d180e7b...
[ 1128.256092] RPC:       state 4 conn 1 dead 0 zapped 1 sk_shutdown 3
[ 1128.257086] NFS4: Couldn't follow remote path
[ 1128.257088] <-- nfs4_try_get_tree() = -93 [error]

I see. I would recommend users just update the kernel if they encounter this issue using that distro on AWS. Updating nfs ganesha major/minor version in a patch seems risky and not our dependency update policy. @derekbit WDYT?

I see. I would recommend users just update the kernel if they encounter this issue using that distro on AWS. Updating nfs ganesha major/minor version in a patch seems risky and not our dependency update policy. @derekbit WDYT?
However, we can have a KB if appropriate in this case.

Same here. I recommend updating the kernel as well.

Sounds good 👍

Close this ticket as we made documents for user to update kernel when encounter RWX mount failure on Ubuntu 22.04 6.5.0-1014-aws


Networking improvements

Description

Resolves # (issue)
How Has This Been Tested?

Due Dilligence

[ ] Breaking change
[ ] Requires a documentation update

Approved by @jackpooley


How to skip some fields in fragment

I have a fragment, but some fields aren't used on some case.
How to exclude it?
Thanks in advance!

Are you referring for http://graphql.org/learn/queries/#directives

Yes, Thank you for your answers.


[WIP] DL4J SameDiff loss function

Blocked on: https://github.com/eclipse/deeplearning4j/issues/8784


Limit Concurrent Tests

An option such as parallel. concurrency(4) to limit four tests running at the same time would be helpful. I've implemented this change and can submit a pull request if that'd be easier, but I'm using an npm library called semaphore as a module dependency that would be added to the package.json to limit the tests.

I think I'd be open to it :) Could I see the changes you made? Would be curious how you've implemented it so far

Yeah fill free to look at the changes. I made the pull request so you can review them. Let me know if there are any coding standards you follow, or implementation changes.

Thanks again!


Add support for GitLab flavored markdown

Subject of the feature
GitLab uses their own flavor of markdown. They call if GitLab Flavored Markdown (GFM). This is basically an extension of GitHub Flavored Markdown (also called GFM).
In addition to the linked specification, they markdown also supports quick actions inside their markdown in some contexts.
Problem
These special syntaxes are not recognized. This causes tooling such as remark-lint and Prettier to not work properly for them.
Expected behavior
All syntaxes, special GitLab references, and slash commands can be recognized.
Alternatives
Avoid using GitLab specific syntax. However, it would be nice if GitLab issue templates can be formatted, linted, and spell checked.

For the big question, “support all of gitlab-fm”, I’m not really interested in doing that 😅 It’s a lot of work to reverse-engineer something like GFM. I don’t hear a lot of folks wanting it, and I don‘t use it myself. And generally, I’d say it’s wiser to use as few extensions as possible, because it makes the markdown not portable.
What is missing in micromark? What features are not working as expected? E.g., colors chips and diagrams are not changes to the language, but changes in the renderer. Math and frontmatter have separate extensions already.
For quick actions, are you 100% that those are in the language too, and not in the renderer? E.g, mentions on GH are similar, but they are an extension to the renderer. Take for example: \@wooorm (@wooorm). If it was a markdown construct, the at-sign would be escaped and the mention not work, but it does!
Depending on further information, I would point you to either make your own micromark extensions, or remark plugins. This is fine in userland, or maintained here, but I’m not interested in writing code for it.

I’m not 100% sure which parts are language extensions, and which are part of the renderer.

emoji
frontmatter (JSON specific or language annotations)
inline diff
[math](https://docs.gitlab.com/ee/user/markdown.html#math
GitLab references
Table of Contents
Multiline blockquote
Code blocks (Specifically using tildes)
Footnotes (I think they are just links that are rendered differently)

I avoid GitLab specific syntax, except maybe emoji in issues, not committed files.
The two things that annoy me personally are that given the following issue template:
/estimate 1d
/label Bug

Prettier will format this as:
/estimate 1d /label Bug

This can be avoided by adding an empty line in between actions, so although annoying, it’s not a big deal.
And retext-spell will mark 1d as a spelling error. A workaround is to add 1d to the personal dictionary, but this isn’t very flexible (although this syntax only occurs once per issue template).
I had these small annoyances and hoped you might be interested to support more markdown flavors. Perhaps I’ll look into it some day, but I’ll close the issue for now.

:+1:
For future reference:

(g)emoji: are supported by several remark projects too
frontmatter: all except for custom languages (---php) syntax are supported with remark-frontmatter
inline diff is not supported, it looks a bit like critic markup btw
math: a different syntax is supported with remark-math
references: some of these are the same on GH and supported in remark-github. GH does this on the AST, so a similar project like remark-github could also do it with regexes instead of integrating with the parser
toc: not supported (remark-toc can generate one for a heading though, which would work everywhere)
multiline blockquote: not supported
code blocks w/ tildes: standard commonmark, so already supported
footnotes: can be done w/ remark-footnotes

The slash syntax seems to be something else? “Quick Actions” apparently. Uhmmm. I don’t know what their limitations are. Indeed, loose paragraphs might be fine:
/estimate 1d

/label Bug

And for spelling, you could turn them into some custom node type with a regex based AST traversal project (similar to remark-github, through mdast-util-find-and-replace). Then, remark-retext will ignore it because it’s syntactical and not prose, and thus it won’t be spellchecked!


release: v1.1.0

See CHANGELOG

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 1e2e61a2ca7e3c03059171b510e7e427364b8661
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Improve aspect ratio and size controls

Closes #226
This PR reworks the aspect ratio/size selector to be more maneable and less predicting for users.
A written up explanation is available in the accompanying issue that this PR closes.

Github hides reviews for outdated files, but there's one review that's still relevant: https://github.com/wulkano/kap/pull/251#discussion_r147531078

I get a RangeError: Maximum call stack size exceeded when removing the contents of one of the inputs.
When 16:9 is selected it should be possible to have 1 width and 1 height.
Select 5:3 in the dropdown, then 20 width and 16 height in the inputs, then click the switch button, and notice that each time you switch the number in the inputs decreases.

As per the last two commits:

Switching values also switches the aspect ratio (as discussed with @skllcrn) and doesn't decrease any values
stopped Maximum call stack size exceeded from occurring on removing a value from the height/width inputs when the ratio isn't locked

Things that need to be done now:

Apply a minimum value to width/height when the aspect ratio is locked and make the mimum the corresponding numerator/denominator
When oninput is called with no value in the width/height inputs we need to update the aspect ratio selector with the last valid input value (having a bit of trouble with this at the moment)

Another issue. Select 3:2, then clip the flip button and you'll end up with this:

Also, changing the width/height with the cropper open no longer changes the cropper size. But I don't think that's from this PR 🤔

Merging this. We can do follow-up PRs with improvements.

@CodeTheory, I think it is.

@kevva No, it worked before https://github.com/wulkano/kap/commit/0464f23fef059201c47aea5dc54a10e3330509d7, so the issue was introduced in this PR.


play-by-play actual time

I'd like to have a feed of all events across games.  But there doesn't seem to be a "global" eventId, the sortOrder and eventIds seem to be game specific, so I'm having to parse periods, and event times to guess actual event time so i can sort all events.  Anyone see another way?
Also, is the API delayed?
Thanks!

I don't personally have an answer for the first question, but re: delay,
I haven't seen what seems to be any artificial delay or even rate-limiting with any API requests I've made. It's either an oversight or just the NHL being nice to us, so best not to abuse it.

Thanks,  the delay proved very temporary.
As far as the "real event-time", I'm trying to figure out a proxy - counting forward from puck drop is terribly inaccurate.  So next I'll be trying to work backwards from current game-time.


Bug: Tasks generally need to be able to handle snapshots being deleted by the UI mid-run

Many archivebox tasks (e.g. update, add, etc.) fail mid-run if Snaphsots are deleted from the UI while archivebox is iterating over them.
We should do a pass over the codebase and find all the for Snaphot.objects... loops and add try:/except: within them to handle the case where the snapshot dissapears because it was deleted by another process.
root@kiwi /o/archivebox.un# docker-compose run archivebox update --index-only
Creating archiveboxun_archivebox_run ... done
find: '/.config/chromium/Crash Reports/pending/': No such file or directory
[i] [2023-03-18 06:07:27] ArchiveBox v0.6.3: archivebox update --index-only
    > /data

find: '/.config/chromium/Crash Reports/pending/': No such file or directory
Traceback (most recent call last):
  File "/usr/local/bin/archivebox", line 33, in <module>
    sys.exit(load_entry_point('archivebox', 'console_scripts', 'archivebox')())
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/cli/__init__.py", line 140, in main
    run_subcommand(
  File "/app/archivebox/cli/__init__.py", line 80, in run_subcommand
    module.main(args=subcommand_args, stdin=stdin, pwd=pwd)    # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/cli/archivebox_update.py", line 119, in main
    update(
  File "/app/archivebox/util.py", line 114, in typechecked_function
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/main.py", line 797, in update
    write_link_details(link, out_dir=out_dir, skip_sql_index=True)
  File "/app/archivebox/util.py", line 114, in typechecked_function
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/index/__init__.py", line 335, in write_link_details
    write_json_link_details(link, out_dir=out_dir)
  File "/app/archivebox/util.py", line 114, in typechecked_function
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/index/json.py", line 99, in write_json_link_details
    atomic_write(str(path), link._asdict(extended=True))
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/index/schema.py", line 193, in _asdict
    'snapshot_id': self.snapshot_id,
                   ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/utils/functional.py", line 48, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
                                         ^^^^^^^^^^^^^^^^^^^
  File "/app/archivebox/index/schema.py", line 265, in snapshot_id
    return str(Snapshot.objects.only('id').get(url=self.url).id)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/django/db/models/query.py", line 429, in get
    raise self.model.DoesNotExist(
core.models.Snapshot.DoesNotExist: Snapshot matching query does not exist.

Closing as duplicate of https://github.com/ArchiveBox/ArchiveBox/issues/1309


Fix missing Secure paramater in cookies

Description
This PR fixes bugs observed by Sonatype Lift over here
Signed commits

[x] Yes, I signed my commits.

We rely on cookies for authentication. This PR does addresses Sonatype's issue however this isn't something we can do right now in my opinion.

Thank you @100mik for raising this PR, will have to close this one but keep on bashing these bugs 😄 !


Added STL/CPP/returnDistanceToTheNearestVowel.cpp

Added a c++ program in STL/CPP to find Distance of each character in the given string in respect to the nearest character in the string.
Also Added a Animated FrontEnd Homepage to the FrontEndWebsites Folder.

@Jay206-Programmer Can you please your live link of website just create README.md file in that add your website's live link..
I merged this PR ..
but update Live link then again make your PR


Fixed imports with late binding and this

Fixes #4214
The problem in #4214 was that function like document.write can't be imported as a function reference into WASM directly, because document.write does not capture the this argument (here document). Or in other words, there is a difference between:
export function __wbg_reload_90d82b22b83c1d99(text) {
    document.write(text); // `write` has `document` as the `this` arg
};

and
// there is no `this` arg, so `write` will fail when invoked in WASM
export const __wbg_reload_90d82b22b83c1d99 = typeof document.write === "function" ? document.write(text) : notDefined('document.write');

However, aside from this not being captured, the (let's called it) direct import style also didn't support late binding. If the object or function is only defined after the WASM module is instantiated, then the direct import style will cause an error.
This PR fixes both issues by always generating a wrapper function for imported function from global scope.

Added changelog.


[Question] How do I access the feathers client with FeathersVuex

Hello,
in the feathers docs I have found the following code for client side usage:
const io = require('socket.io-client');
const feathers = require('@feathersjs/feathers');
const socketio = require('@feathersjs/socketio-client');

const socket = io('http://api.my-feathers-server.com');
const client = feathers();

My question is - is there a way to access this client variable also using FeathersVuex?
Since FeathersVuex has a socket already ... it must be somewhere
I could use the code above but i guess i will have 2 soeckts in this case running.

I wanna know how I can disable the real time feature for a specific Model.
I am using the Model mainly for CRUD actions and dont need real time updates for it


automatic DCU test suite FAILS| protectli v1210|Dasharo 0.9.2

Device
v1210
RTE version
0.7.5
OSFV version
branch: regression_protectli_v121x, commit<PHONE_NUMBER>17cbb5dab6f24b1b9c6d8dda23778c
Affected component(s) or functionality
No response
Brief summary
DCU 001&2.001 FAIL with the prompt of a missing keyword and DCU003.001 cannot find a specific String while performing the test automatically
How reproducible
happens every time
How to reproduce
Run dcu test suite (or any individual test case) on protetcli v1210 with dasharo 0.9.2 firmware
Expected behavior
the test should run just like his manual counterpart
Actual behavior
-missing uuid4 keyword in DCU001.001
-missing Random Int keyword in DCU002.001
-DCU003.001 is unable to recognize 'Success' string
Link to screenshots or logs
dasharo-compatibility_.zip
Additional context
No response
Solutions you've tried
No response

Should be fixed by this commit: https://github.com/Dasharo/open-source-firmware-validation/pull/361/commits/04e410031775b1ee51d8b9c97915d27696856bb8
DEVICE_IP=<IP_ADDRESS> FW_FILE=protectli_v1210_v0.9.2.rom RTE_IP=<IP_ADDRESS> CONFIG=protectli-v1210 SNIPEIT_NO=true ./scripts/run.sh dasharo-compatibility/dcu.robot
==============================================================================
Dcu
==============================================================================
DCU001.001 Change the UUID :: This test case verifies that the UUI... | PASS |
------------------------------------------------------------------------------
DCU002.001 Change the serial number :: This test case verifies tha... | PASS |
------------------------------------------------------------------------------
DCU003.001 Change the bootsplash logo :: This test case verifies t... | PASS |
------------------------------------------------------------------------------
DCU004.001 Verify SMMSTORE changes :: This test case verifies that... | SKIP |
smmstoretool ins not yet supported in DCU
------------------------------------------------------------------------------
Dcu                                                                   | PASS |
4 tests, 3 passed, 0 failed, 1 skipped
==============================================================================


Deprecate ListCatalogEntries in rill-load-test

This API previously had some latency issues. This is now deprecated so we didn't look at improving its performance as well.
Migrate to v1/instances/{instance_id}/resources?kind=rill.runtime.v1.MetricsView API to list the dashboards in rill-load-test

Updated the API here here


fluxctl: 1.24.0 -> 1.24.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/weaveworks/flux/releases.
meta.description for fluxctl is: "CLI client for Flux, the GitOps Kubernetes operator"
meta.homepage for fluxctl is: "https://github.com/fluxcd/flux"
meta.changelog for fluxctl is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/hgm0pjzj4s6akgp1hl508nmfz8yq9grr-fluxctl-1.24.1/bin/fluxctl passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 1.24.1 with grep in /nix/store/hgm0pjzj4s6akgp1hl508nmfz8yq9grr-fluxctl-1.24.1

found 1.24.1 in filename of file in /nix/store/hgm0pjzj4s6akgp1hl508nmfz8yq9grr-fluxctl-1.24.1

directory tree listing: https://gist.github.com/330d9ccf542e80fa9e799341c5062bd4

du listing: https://gist.github.com/b8b1663a95b1eff540d97caa7ddd7fbc

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
fluxctl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hgm0pjzj4s6akgp1hl508nmfz8yq9grr-fluxctl-1.24.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fluxctl https://github.com/r-ryantm/nixpkgs/archive/a3205773b02dda37000b78d3a0df3e1d350d2190.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hgm0pjzj4s6akgp1hl508nmfz8yq9grr-fluxctl-1.24.1
ls -la /nix/store/hgm0pjzj4s6akgp1hl508nmfz8yq9grr-fluxctl-1.24.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fluxctl

Maintainer pings
cc @Gonzih @Br1ght0ne @kalbasit for testing.

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

1 package built:

fluxctl


surfactant homeostasis

Hi,
Could I request a new process term entitled "surfactant
homeostasis", as an is-a child of homeostasis
(GO:0042592).
Definition:
Any of the processes involved in the maintenance of an
equilibrium of the surface-active lipoprotein mixture
which coats the alveoli.
PubMed Ref: 9751757
cheers,
Emily
(for annotation of P35247 + IMP)
Reported by: edimmer
Original Ticket: "geneontology/ontology-requests/1560":https://sourceforge.net/p/geneontology/ontology-requests/1560

thanks www.movied.org


Missing information on bitshifting

If the bits are bitshifted by a number greater than 32, for example (int)8 << 33, it is equivalent to bitshifting once to the left.
When writing (int)8 << 33, what really happens is (int)8 << 33 % 32
This is not specified anywhere on the page.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 02b8f7c2-379e-7876-0d36-25b06a294baa
Version Independent ID: 2410c828-0c66-9efe-ae87-9b07708aa731
Content: Bitwise and shift operators - perform boolean (AND, NOT, OR, XOR) and shift operations on individual bits in integral types
Content Source: docs/csharp/language-reference/operators/bitwise-and-shift-operators.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @pkulikov
Microsoft Alias: wiwagn

This is not specified anywhere on the page.

Except in section Shift count of the shift operators later down that page.

int or uint, the shift count is defined by the low-order five bits of the right-hand operand. That is, the shift count is computed from count & 0x1F

And the & 0x1F <=> & 31 is the bit-hack for % 32.
For completeness, for long, ulong it's analogous.

You are absolutely correct, thanks for pointing this out.
I will explain how my misunderstanding came to be, and maybe that will help improve the docs.

Left-shift operator <<
Right-shift operator >>

From reading these paragraphs i had the impression that this is all there is to it, and there is no disclaimer or a link in a close proximity to the info that's waaay down below.
I would suggest putting a note/disclaimer/link in that paragraph or somewhere in close proximity to the info you pointed out.

I can't follow. In

there's a link to that section (highlighted in yellow by me). Do you means some different place?

No, this is perfect, it somehow escaped me. It is phrased so banal that it would have never occurred to me that there are "tricks" to it.
When i read it, i thought, "what is there to read about how the count is defined, its a number okay, and it shifts the thing the number of times you define" and went on reading further.
There is a phrase, "never underestimate the banality of evil", i guess this will be a lesson to me to never underestimate the banality of phrasing.
Sorry about bothering you, turns out its all my monkey brain at fault.

Na na, no problem. I got trapped by similar things several times too (reading too fast, can skip that section because I know it...in essence read it thorough to really know it 😉).


fix: content transformers command

Q
A

Bug fix?
y

New feature?
n

BC breaks?
n

Deprecations?
n

Fixed tickets
n

@theus77 in the dataService -> lockRevision, added a null === $username, because the appTwig inMyCircles only works for authenticated users. When we pass a username we are running from cli. Aggree?

I agree.

I agree.

👍


cookies POPup button change the word

different tests to guarantee better functionality in cookies and privacy

As a user, I want the necessary changes to the website to be implemented to comply with the legal requirements identified during the legal investigation.