TOC (Table of Contents Support)

Just hoping anyone can help with using TOC (Table of Contents) in DinkToPDF.
Currently there is no settings for it (like there is for Header and Footer etc).
Any ideas of how to wrap it in so we can use the feature that is available would be great

In case anyone else ends up here needing a solution to this...
Subclass ObjectSettings and add a property to set the table of contents flag to true:
public class TableOfContentsSettings : ObjectSettings
{
    [WkHtml("isTableOfContent")]
    private bool IsTableOfContents
    {
        get => true;
    }
}

You can then add this object wherever you want the ToC to appear in the pdf. Something similar could probably be done with the Cover object type, but I've not got round to trying that (yet).

@garymcleanhall thanks, It worked but just the first time. I'm using .net core 2.2. After implemented, I call API to generate the PDF file. It works well  for the first time and the next time, pdf file shows the blank page for TOC

It's working for me. Use the suggested class as the following.
var pdf = new HtmlToPdfDocument()
{
    GlobalSettings = globalSettings,
    Objects = { coverSettings, new TableOfContentsSettings(), objectSettings }
};

Same here,
It works well for the first time and the next time, pdf file shows the blank page for TOC


Could we extend all normal knex query builder methods to support json field expression

Json query API could be even easier to use if all knex query builder methods can take in Json field expressions in addition to plain knex column references. In that case instead of knex implementation our json query versions would be called.
Somewhat related to #18

+1

This needs to be done before #187 so this is now in progress

Implemented in https://github.com/Vincit/objection.js/pull/270


Add possible to pass another value to initialState param

In my feature I need to instance the treegrid and keep the current state.
But currently, when I add one more row and apply the treegrid, the another rows are open or close.
I think in accept another value like "keep" or simply false. Currently is only ("expanded" and "collapsed").

hello I think I'm having a problem similar to yours, basically I have a ajax that actuliza my table but do not know how to maintain the state of the rows, you know how?


Sometimes the notification does not appear and goes to the notification area

As a title sometimes the notification does not appear and goes directly into the notification area without sounding or appearing

increases the notification counter and does not show it

Change the focus assistance in the settings of the windows to show all the notifications not only for Alarm
settings -> focus assist settings -> Get all notification from all apps
BTW I think you are using dual screen monitor or more than one monitor that's why it is happening

Yes i use two monitor, thanks to answer me. I didn't known this option of win10 and i thought that the problem was related to the thread.


💬 [Question] User never defines Elopage Password

💬 Question

The user never defines the password for Elopage when registering his Member area - it is to clarify if we can configure this differently and if we can make sure the user knows he gets an EMail to set his password.

This is correct and we adjusted our application to behave similar


Do you want Python 1st-class support?

And what does that even mean? Did you read this blog post?

I would be really happy of this feature also.
I use almost only python scripts and I was also thinking about a way to ease the coding, since from a IDE (like Visual Studio Code) where your python code is just a string of a yaml field, you don't get all the help like syntax highlighting etc.
One "hack" would be to store locally workflow definition using kustomize. And with json patches, adding .py files in the content of the "script" field. It would just need a kustomize build before applying to the argo server.
I didn't know the argo dsl option. I'll check.

There is now an interactive Katacoda course that teaches step-by-step on how to write and submit your first Argo workflow using Couler Python SDK. Thanks @alexec for setting it up!
You can play with it right from your browser so please check it out and let us know what you think! https://github.com/couler-proj/couler#try-couler-with-argo-workflows


Add the tutorial 'Ruby On Rails in Production'

This PR adds the tutorial 'Ruby On Rails in Production'
I have read and understood the Contributor's Certificate of Origin at the end of the template and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: Vincent Thelang<EMAIL_ADDRESS>

I don´t know why, but I am able to run the compiler but I can´t see it in the browser. I´m getting no errors. I had access to it before we removed the default configuration of nginx.

@raphi-0901

I don´t know why, but I am able to run the compiler but I can´t see it in the browser. I´m getting no errors. I had access to it before we removed the default configuration of nginx.

I have tested it again on the latest version, there the same behavior. I have remove the part with the deletion of the default config (
7bf2899). I think it is because of the settings of the default_server names.

I now tried it again from scratch and it still does not work. Now when I try reaching my server I just get the default nginx site.

@raphi-0901

I now tried it again from scratch and it still does not work. Now when I try reaching my server I just get the default nginx site.

Hmm, i tried it with my domain and there it works as described above. I have changed it in the tutorial to a domain.
I have added the part

Are you using an extra domain or the default one provided by netcup?

@raphi-0901
In this case an external one but it should work the same way with the internal one from netcup.

Hey!
Could you please add Steps to your headings  as shown in the template file?

@raphi-0901

Hey! Could you please add Steps to your headings as shown in the template file?

Done ✅

Hi vincent-the, thank you for contributing! It's been a while since you opened this PR and it probably is an open secret we're currently overwhelmed with reviewing all the awesome tutorials we got. Please be patient - we're about to publish tutorials this and next week. Thank you!


TopHitsAggregation don't support custom gson Builder

If you have custom Deserializer for the Gson builder, it will not work with the TopHitsAggreagtion (implemented by me :) ). I need a possibility to pass the custom gson object to the TopHitsAggregation to build custom POJO's from this aggregation.
Thanks
Marcel

Accidentally i put the changes in my other Pull Request, Hope this is ok.  #535

Any news ?


PSF artifacts

There are some SNe where the PSF model is pretty obviously wrong. Of the 30 SNe I test on locally:
LSQ12dbr (R):

SN2005ki (B):

SN2005ki (R):

SN2006ac (B):

SN2011by (R):

SNF20060512-002 (B):

SNF20060512-002 (R):

SNNGC4424 (R):

SNNGC6801 (B):

SNNGC6801 (R):

Note that the ringing is caused by FFT shifting the PSF, but this is a separate issue. For example, running SN2005ki (B) with a fully resampled PSF (not FFT shifted) removes the ringing, but not the large scale residuals:
Fully resampled PSF:

FFT-shifted PSF (same as above):

I don't understand how you can say this is coming from the PSF: which PSF, of which epoch? All epochs seem equally bad. Furthermore, the final model for the "fully resampled" and "FFT-shifted" PSF look rather different, while the PSF is the same.
PS: could you change the scale for the reference images, so that we could see if there's any structure in there?

As one concrete example, take this epoch from SNNGC4424 (R) as an example:

It seems to me that the PSF in this epoch is too wide, causing positive residuals at large radius and negative residuals at small radius. I suggested that it has at least something to do with the PSF because the only free parameters in cubefit at this stage are the position of the galaxy, the position of the SN, and the SN amplitude. Those all seem to be optimized pretty well - or at least, there's no way to change those parameters to make this residual go away.
Of course, this could be a problem with how cubefit is treating the PSF model, rather than the PSF model itself.

But then I'm wondering why all the PSF of a SN sequence be wrong at the same time.

Note: the scale labels on the residuals in the plots above are wrong. The residuals are actually typically on the scale of 1-5% (of the maximum value in the scene model). I won't repost all the images, but here's LSQ12dbr (R) for example:

(This is with resampled PSF fitting and subpixel sampling of 3x3, which is now the default.)


Bug fixes + enhanced font features

Uses correct method to extract user configured settings
Uses correct config declarations
Replaces font-scale with font-size and line-height to make it easier to use
Adds font-family setting to allow users to customize the feel

Just opened an issue for this (#42) and submitted an alternate solution (#43). I should have checked open PR's first.
+1 on this PR: Font size was not being calculated properly because the settings API now uses atom.config.get(key) (See Configuration API). The call to atom.config.getSettings() returned the default settings (no user overrides).
@NikhilKalige, I leave the choice of which PR to merge up to you. Great work on the plugin!

@komputerwiz Thank you...
@itsthatguy Thanks for the PR,, merged


Webviews and settings editor fail to move to an aux window

Also, there's a strange issue that's tricky to replicate, but it has happened to me multiple times. It stops commands from working.

Thanks for testing this and reporting the issue.
Supporting editors that are iframe based will be a larger task: https://github.com/microsoft/vscode/issues/195715
I have pushed changes to enable settings editor to move and prevent iframe based editors to open in floating windows to not break the workbench layout.


Bug try to fix pv operator of work thread pool

Proposed changes
I think get_cv is not notified enough
errCode = 2, detailMessage = (<IP_ADDRESS>)[CANCELLED]exchange req success but status isn't ok: [CANCELLED]fail to offer request to the work pool, pool=PriorityThreadPool(name=brpc_heavy, queue_size=10240/10240, active_thread=3/128, total_get_wait_time=132638923328525, total_put_wait_time=350913870127)
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.36 seconds
stream load tsv:          554 seconds loaded<PHONE_NUMBER>9 Bytes, about 128 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>1 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.52 seconds
stream load tsv:          552 seconds loaded<PHONE_NUMBER>9 Bytes, about 129 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>5 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.01 seconds
stream load tsv:          555 seconds loaded<PHONE_NUMBER>9 Bytes, about 128 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          29.8 seconds inserted 10000000 Rows, about 335K ops/s
storage size:<PHONE_NUMBER>0 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 49.02 seconds
stream load tsv:          535 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>9 Bytes


Unable to add text before *Series databinding.

Instead of "Title Value" in the IndexedToolTip, there's no option to add binding text before the Value.
For example:
"High Temp 90 degrees" could be changed to "High Temp Dry 90 degrees", or "Low Temp 30 degrees" could be "Low Temp Frosty 30 degrees".
If there is some kind of overload I can use to add a descriptor before the value, that would be fantastic but I'm not quite sure how to implement it. I'd suffice with the tooltip, but the tooltip doesn't seem to display at all.

How about this: https://github.com/beto-rodriguez/Live-Charts/wiki/0.4-Labels
LineChart.PrimaryAxis.LabelFormatter = value => "High Temp Dry " + value + " degrees"

The problem with that is if along the same line the temperature drop from 90 degrees to 30 degrees, the label will be formatted as "High Temp Dry 30 degrees". I'd like to be able to insert my own text with data binding along with the values along the axis.

I am sorry, but I think I don´t understand what you are trying to do. can you please explain me better so I can help you.

Perhaps this graph with my previous charting software will best illustrate what I am trying to do. I want to add text along with the value to specify who/what that value belongs to.
Specifically, in this example, users tip a user, the graph will show how much they were tipped and when they were tipped but can not show who they were tipped by.

You can add a custom Tooltip, setting the DataToolTip property of a chart, I am not sure if that will be enough to solve your case, I will work on this and some other issues now, another important point I see, is that right now this library does not supports negatives in bar charts,

I am courious about how did you achive that in the library you plot that, right now the solutions I see are not easy, or I dont like them.

Currently I use a pointvalueevent with ZedGraph, which basically lets me know what index number the bar I am currently hovering over is and I pull the data from a List of string using that index: http://zedgraph.sourceforge.net/documentation/html/E_ZedGraph_ZedGraphControl_PointValueEvent.htm

OK, one moment and I will make a commit to solve this.

I have to say that that I don't really like the solution I committed, I think it is not easy enough, can you please let me know what you think? I will describe the steps to make it work, I also added an example in test project, to Custom Bar Chart example it contains a custom tooltip now.
First of all this is the legend that explains the problem I solved https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomBar.xaml.cs#L17

Create a custom tooltip class that inherits from IndexedTooltip

An indexed Tooltip DataContext will be injected when mouse is over a point with an IndexedTooltipData Instance (https://github.com/beto-rodriguez/Live-Charts/blob/master/Charts/Tooltip/IndexedTooltipData.cs)
It will expouses some useful properties,

Property
Description

Series
The Series that Data point belongs to.

Index
The index of the Series in the Series collection of the chart.

Point
The data point values (x,y)

Value
Y value formatted.

here is the XAML https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomTooltip.xaml
here is the Code behind https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomTooltip.xaml.cs

Now you need to set that new tooltip to a chart, really simple, just this https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomBar.xaml#L22 and load your extra data to that tooltip https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomBar.xaml.cs#L24

And that's it, The tricky part happens here https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomTooltip.xaml#L34
The multivalue converter injects IndexedTooltipData and ExtraData so you can play with those properties. the corverter just maps them here https://github.com/beto-rodriguez/Live-Charts/blob/master/ChartsTest/BarExamples/CustomTooltip.xaml.cs#L30
If I were clear enough this is what you read in this line
var maxSales = values[0] as decimal[][];
var data = values[1] as IndexedTooltipData;
return maxSales[data.Index][(int) data.Point.X].ToString("C");
//max sales is an array containing the max price sold by Maria, John and Erick.
//maxSales[data.Index]  =>  gets the arraf of Maria, John or Erick according to data.Index
//[(int) data.Point.X]  => data.Point contains the value of the current point, the X property is the index.

Alright, after a little trickiness, I got the example working, and if I understand correctly, I need to handle my strings inside MyConverter? Right now by default through your example it only passes numeric values, correct?

Yes, this is a tricky solution, it should work, but I am not really happywitht this, does this solves your problem? do you have an idea to make it easier?

@cglmrfreeman to your question in this example yes, but ExtraData Property is IEnumerable you can pass any object you need.

@cglmrfreeman I have pulled a lot of code that will make this easy now, here is an example
https://github.com/beto-rodriguez/Live-Charts/tree/master/ChartsTest/Line Examples/Mvvm


console.logs for env

Description
Brief description of changes
Risks
Potential areas of concern that you would like looked at more closely
Validation
How you verified that these changes work
Issue
Link to corresponding issue
Operating System
If you're on Mac, please include whether you are on an Intel or M-series CPU

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Introduce stash_count(&self) -> Result<usize, Error>

Would it be possible to introduce something like this to Repository? I'm not sure if the goals of this project are to keep it mapping to the libgit2 directly or not. I'm also not sure if allowing not requiring a mutable reference here is sound.
/// Count number of git stashes
pub fn stash_count(&self) -> Result<usize, Error> {
    unsafe {
        let mut count = 0;
        let mut data = StashCbData {
            callback: &mut |_, _, _| {
                count += 1;
                true
            },
        };
        let cb: raw::git_stash_cb = Some(stash_cb);
        try_call!(raw::git_stash_foreach(
            self.raw(),
            cb,
            &mut data as *mut _ as *mut _
        ));
        Ok(count)
    }
}

Based of the stash_foreach implementation.

I like the train of thought here and I appreciate you took the time to work this out, but I am sceptical about this because:

It introduces a helper that is actually not present in the raw libgit2 that we bind to.

It increases the surface area of using unsafe functions even though the same can be achieved by using the already existing wrapper around git_stash_foreach  namely  Repository::stash_foreach docs

is a decision that has to be made on the repo owners level because it depends on the vision for this crate - right now it looks like a 'safe wrapper' around the bindings to libgit2 and not more which is nice because it basically means whatever method u come across here you can reason about it being in a 1:1 relationship with one c function in the underlying lib

independent of 1. I would argue this does not need to be implemented in a n unsafe way.

just my two cents :)

I thought that might be the case but I didn't see it stated explicitly (in my quick look)
It might not need unsafe but ideally we don't need a mutable reference to self which is my main reason for wanting to add another function.

Both of these reasons are why I opened an issue rather than a PR :)


The list of Codes

A link to a relevant list of error codes as shown in the arcticle's screenshot in "Code 39" might be handy.

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

ID: 38b3139a-27f1-d868-b971-2255ffe4dcc2
Version Independent ID: 2ba8b67a-ee31-2fce-7143-48df43bfc3ce
Content: Detecting Driver Load Errors - Windows drivers
Content Source: windows-driver-docs-pr/install/detecting-driver-load-errors.md
Product: windows-hardware
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Again, thanks @jurva for the suggestion and apologies it took us a while.  The link you want is https://docs.microsoft.com/en-us/windows-hardware/drivers/install/device-manager-error-messages.  I've added that to the doc page above, fix should be live in 15 min or so


Some identifiers renders type that don't need to be in output

After update with adding type to output (https://github.com/metanorma/pubid-core/commit/8882ea33d38ab4005dd8cdd4250b16cc3ccbbfdd)
Now "IS" type included to output:
       expected: "IEC 60050-111/AMD1/FRAG1 ED2"
            got: "IEC is 60050-111/AMD1/FRAG1 ED2"

Yes "IEC 60050-111/AMD1/FRAG1 ED2" is correct.


[BEAM-8643] Update elasticmq-rest-sqs

Update elasticmq-rest-sqs

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

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

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

R: @iemejia


Outstanding API documentation changes

[x] Write/find section comparing positional to non-positional tracking

[x] Link it in the set_store_position docstring

[x] Test nullptr -> None conversion in parent() and beyond
[x] Add support for Sphinx Tippy or some other tool tip extension
[x] Add Sphinx bibliographies (can be based on Empirical implementation)
[ ] Change all "systematics manager" to "phylotrackpy"

I don't think we actually need to change mentions of the systematics manager to phylotrackpy, since it's the phylotrackpy systematics manager

Last fixes done in #84


Exit in Web UI not working

Trying to exit the UtilityBox via browser UI does nothing.
Clicking on Exit in the browser UI neither closes the browser tab, nor does it exit the desktop app, that runs in tray/background.
Besides that at first glance everything seems to be working.
I'm running Windows 10 Pro 20H2 (19042.867) have the browser UI opened in Google Chrome x64 version 89.0.4389.90
Kind regards
Eddga

Hello,
That is correct, the background service is a webserver that is runned on your local machine. This way the tool is available through a browser with a nicer interface than a usual "windows form" which looks quite boring if you were to ask me.
It also simplifies the process of adding new functionality in an easier manner if someone wish to edit the code base of their own.
To close the service, you'll just as any regular "tray" program, right click it and close it.
Br

But why is the "Exit" option even there in the browser UI if it is intended to do nothing?
Or am I misunderstanding something?

Oh I see what you mean now haha, that is not even meant to be there, I think I was using it during debugging.

Will be fixed part of V.0.3.0


User domain page shows no social Icons

Details
User doamin interface has no social options to verify them
it's just blank were the Twitter / Discord / Github / Anima ICONS must be present
User info :
Address : 0x0177f618d683bf567227b3fea497b0fa35a0cb90c16042a9f039180d48e699a4
Domain name : decentralizeddreams.stark
Screenshots :

Comment :
As user domain is too long , those icons have moved out of the size given in for the identity info in our code  ?
becoz the domain name  is displayed at the bottom of the NFT image why ?
usually all the info will be on the right side of NFT
Default domain info : ALL DETAILS ON THE RIGHT SIDE

Discord ticket link :
https://discord.com/channels/1025416762835095674/1198717277395243108

Please i would love to be assigned this.


MAID-3125: docs/license: update LICENSE details

updated readme license text and license files to updated license details

@pravarseth , I just checked and Beaker has only MIT license, can you please confirm if it's correct we add the BSD license to our fork of it and where you get this understanding from?

Adding BSD makes this consistent with the rest of our licensing strategy @bochaco. There is very little difference between MIT and BSD but keeps thing simpler.

Ok thanks for the confirmation @NickLambert.


Can't call API from react native

I used Postman call to API(https://laravel.qrcode.test/api/qrcodes/5555) then it success. But when
call the API from React Native app then it's appear error "Network request failed". Can you help me, please?(url)
Error on app:

Call from Postman:

Docker running:

The issue here is, from wherever you're running the react native app, that environment doesn't know how to resolve the DNS qrcode.test
Whereas postman is able to do that because of the host file entry.


Using hideEdgesOnMove does not work with defaultEdgeType set to "arrow" in v2

With the below set:
hideEdgesOnMove: true,
defaultEdgeType: "arrow",
It looks like edges will not "unhide" after moving the graph if I hover a node just after moving the graph but before edges have unhidden.

Actually, it looks like I get the same behaviour without setting defaultEdgeType to arrow.

I have verified that this happens for me in the settings.ts example in the examples folder (after adding hideEdgesOnMove: true and moving the graph around a lot).

Thanks @apitts for the bug, I can reproduce it, and I corrected it.
The code will be reviewed before to merge it in the v2 branch.

Terrific - thanks @sim51!


issue FDSN#18

@andres-h if you are comfortable with these PRs go ahead and merge then with the draft branch and then you can see the rendered docs at:
https://seedlink.readthedocs.io/en/draft/

OK, I merged the PRs into draft. I think ideally the eval team should check if the PRs are OK and close the corresponding tickets if yes.


Setting an x and y offset when cropping

Is there a way I can crop out an x, y, width and height from an image?

I assume you are trying to say, crop out the middle of the image?
Thanks to this awesome pull request by @waterskier2007 - you now can specify a UIBezierPath to mask out the image... https://github.com/gavinbunney/Toucan#path-mask
Would just be a case of changing together a maskWithPage then a resize as required.


"About" screen

Add an "about" screen to settings with app info.

Added the "community" screen in 2.0 which performs this function.


Pasting Tab character leads to Tab Completion

Windows Terminal version
1.13.10395.0
Windows build number
10.0.19044.0
Other Software
No response
Steps to reproduce
Copy the command below. The command is echo<space><tab><space>test.
echo 	 test
(I now realize github strips the tab character, but you can write this in your text editor and copy it to your clipboard)
If you are adventurous,
rm 	 thisfiledoesntexist (this will delete the first file in your directory!!)
Expected Behavior
It should paste
echo 	 test verbatim.
Actual Behavior
echo <first-file-in-current-directory> test
The Tab completion is invoked which replaces the tab with the first entry in tab completion.

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #8993 for more discussion.
/dup #8993

@zadjii-msft I think maybe you meant #6134.

oh my god I so did, I even had that tab open and everything. How... ugh. Thanks.


Keep state lock for duration of journal context

cherry-pick of #11255

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9044/
Test PASSed.

alluxio-bot, merge this please


DocumentBuilder merges numbering definitions referencing the same abstract numbering definition

When building a document, DocumentBuilder merges numbering definition instances referencing the same abstract numbering definition. This leads to bugs if the numbering definitions contain numbering level definition overrides.
For an example, see the example document and the processed version. When opened in Word, the example document displays three lists, each numbered from 1 to 3. After being included in a document built with DocumentBuilder, however, the processed version displays as three lists with a consecutive numbering from 1 to 9.
Technically, this is due to the CopyNumbering method in WordBuilder only creating a new numbering definition if it can't find one using the same abstract numbering definition. Instead, it should use a map like CopyComments.
This issue is the cause of a bug in our system, so I'll will start working on a fix.

Fixed by #204.


Create a directory and a file inside

Issue: #5

Added list of commands (task4.txt)
Attaching screenshots

Screenshots

Hey @Skr-suyash
Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.
Thanks for contributing in OpenCode'24 ✨✨!

Hey @Skr-suyash
Your PR has been merged 🥳🥳 and you have earned 10 points.
Thanks for contributing in OpenCode'24 ✨✨


DNS resolution failed for :80:

UserWarning: failed to set default signal handler: ValueError('signal only works in main thread of the main interpreter')` (raised from /usr/local/lib/python3.10/site-packages/jina/init.py:150)
ERROR  AsyncGRPCClient@ 1 gRPC error: StatusCode.UNAVAILABLE [10/28/22 03:09:07]
DNS resolution failed for :80:
C-ares status is not ARES_SUCCESS qtype=A
name= is_balancer=0: DNS query
cancelled
The ongoing request is terminated as the server is
not available or closed already.
WARNI… AsyncGRPCClient@ 1 process error: CancelledError()
ConnectionError: DNS resolution failed for :80: C-ares status is not ARES_SUCCESS qtype=A name= is_balancer=0: DNS query cancelled
Traceback:
File "/usr/local/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 558, in _run_script
self._session_state.on_script_will_rerun(rerun_data.widget_states)
File "/usr/local/lib/python3.10/site-packages/streamlit/runtime/state/safe_session_state.py", line 73, in on_script_will_rerun
self._state.on_script_will_rerun(latest_widget_states)
File "/usr/local/lib/python3.10/site-packages/streamlit/runtime/state/session_state.py", line 543, in on_script_will_rerun
self._call_callbacks()
File "/usr/local/lib/python3.10/site-packages/streamlit/runtime/state/session_state.py", line 556, in _call_callbacks
self._new_widget_state.call_callback(wid)
File "/usr/local/lib/python3.10/site-packages/streamlit/runtime/state/session_state.py", line 278, in call_callback
callback(*args, **kwargs)
File "/app/gui.py", line 186, in click_handler
asyncio.run(prompt_handler())
File "/usr/local/lib/python3.10/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
return future.result()
File "/app/gui.py", line 139, in prompt_handler
await asyncio.gather(
File "/app/gui.py", line 39, in disco_request
async for resp in client.post(
File "/usr/local/lib/python3.10/site-packages/jina/clients/mixin.py", line 346, in post
async for result in c._get_results(
File "/usr/local/lib/python3.10/site-packages/jina/clients/base/grpc.py", line 142, in _get_results
raise ConnectionError(my_details)

Hello,
You need to first publish your DiscoArt server first before using this UI. https://github.com/jina-ai/discoart

Got it worked. Thanks a lot. And more parameters need cloud you please add more?

I only added the most basic ones to not overwhelm the students.
Any specific parameters you would like to see?

Here many DD users in China appreciated a lot for your beautiful and convenient UI because only few people still using DD rather than SD.
As we are all deep users with DD, here is a list we want to use and hope it doesn't bother you too much:
init_image(link of image), init_scale, tv_scale, range_scale, sat_scale, cutn_batches, cut_overview, cut_innercut, cut_icgray_p, eta
and if possible maybe a gif window(when generation complete) is useful to check the progress for optimizations.
Thanks for your reply.

Got it, I will include those in the next iteration.

@kuyin The extra parameters and GIF preview are added.

Nice bro!


HGTector stuck at "Step 1: Searcher - batch protein sequence homology search."

Hi Qiyunzhu,
As you suggested, I have created, without any issue, a local database using the databaser.py. However,  HGTector fail to progress through the first step. After 24 hours, it is still working on the first set of 10 proteins:
"Validating task...
Done.
Step 1: Searcher - batch protein sequence homology search.
-> Searcher: Batch sequence homology searching and filtering. <-
Reading input data...
WP: 7278 proteins.
Done. 7278 proteins from 1 set(s) to query.
Reading search results from previous run(s)...
Done. 0 results found, remaining 7278 proteins to search.
Reading taxonomy database... done. 1491434 records read.
Reading protein-to-TaxID dictionary... done. 34512566 records read.
Reading taxonomy records from previous run(s)... done. 1 taxa and 13 ranks read.
Enter the TaxID of WP, or press Enter if you don't know:578459
Taxonomy of input protein sets:
WP: Rhodotorula graminis WP1 (578459)
Batch homology search of WP (7278 queries) started.
BLASTing KPV71431.1,KPV71432.1,KPV71433.1,KPV71434.1,KPV71435.1,KPV71436.1,KPV71437.1,KPV71438.1,KPV71439.1,KPV71440.1..."
This sound suspicious.

Hello Kafker,
Thank you for letting me know this. I looked at your configuration file. It
seems that everything is correct. Let us first suspect that the
computational load may be the problem. When you run this forever-running
Blast, how much CPU load and memory does it take? (e.g., by unix command
"top")
Best,
Qiyun
On Sun, Jul 10, 2016 at 10:26 AM, kafker<EMAIL_ADDRESS>wrote:

Hi Qiyunzhu,
As you suggested, I have created, without any issue, a local database
using the databaser.py. However, HGTector fail to progress through the
first step. After 24 hours, it is still working on the first set of 10
proteins:
"Validating task...
Done.
Step 1: Searcher - batch protein sequence homology search.
-> Searcher: Batch sequence homology searching and filtering. <-
Reading input data...
WP: 7278 proteins.
Done. 7278 proteins from 1 set(s) to query.
Reading search results from previous run(s)...
Done. 0 results found, remaining 7278 proteins to search.
Reading taxonomy database... done. 1491434 records read.
Reading protein-to-TaxID dictionary... done. 34512566 records read.
Reading taxonomy records from previous run(s)... done. 1 taxa and 13 ranks
read.
Enter the TaxID of WP, or press Enter if you don't know:578459
Taxonomy of input protein sets:
WP: Rhodotorula graminis WP1 (578459)
Batch homology search of WP (7278 queries) started.
BLASTing
KPV71431.1,KPV71432.1,KPV71433.1,KPV71434.1,KPV71435.1,KPV71436.1,KPV71437.1,KPV71438.1,KPV71439.1,KPV71440.1..."
This sound suspicious.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DittmarLab/HGTector/issues/5, or mute the thread
https://github.com/notifications/unsubscribe/AEMVN32SvUr6kmgaXe6Nl02380EDIB7nks5qUQEOgaJpZM4JI2Fx
.

Hello,
Thank you for the quick reply. I will check that and let you know. However, I am working on a workstation with 64 Gb of RAM, so I do not think it should be a problem.
Best,
Andrea

Dear Qiyun,
Problem solved, now it work. Seems that writing blastp outputs into an external archive was the problem.
Follow the CPU and Memory usage while it is running now on my workstation:
top - 14:41:54 up 6 days,  3:32,  5 users,  load average: 26.98, 24.73, 21.45
Tasks: 544 total,   1 running, 543 sleeping,   0 stopped,   0 zombie
%Cpu(s): 95.4 us,  0.8 sy,  0.0 ni,  3.8 id,  0.0 wa,  0.0 hi,  0.0 si,  0.0 st
KiB Mem:  65924036 total, 63918632 used,  2005404 free, 12727252 buffers
KiB Swap: 67054588 total,    70676 used, 66983912 free. 40191152 cached Mem
It's is really high!
Thanks
Andrew

Hello Andrea, Great to know that! As I remember, Blast is slow but does not
take to much memory, since the databases are stored as multiple volumes
(1GB each by default?) on the disk. Modern alternatives such as Diamond,
however, cache the entire database in memory. Those program will cause a
substantial burden on desktop computers. Meanwhile, another source of
memory usage (probably your situation) is that HGTector caches the entire
GI-to-TaxID table in memory, which consumes dozens of GBs. This is
optional, and can be switched off if the Blast database already contains
TaxID information (for example, NCBI nr has it). In GUI.html there is a
guideline of how to append TaxID when constructing Blast database. Best,
Qiyun
On Sun, Jul 17, 2016 at 6:04 AM, kafker<EMAIL_ADDRESS>wrote:

Dear Qiyun,
Problem solved, now it work. Seems that writing blastp outputs into an
external archive was the problem.
Follow the CPU and Memory usage while it is running now on my workstation:
top - 14:41:54 up 6 days, 3:32, 5 users, load average: 26.98, 24.73, 21.45
Tasks: 544 total, 1 running, 543 sleeping, 0 stopped, 0 zombie
%Cpu(s): 95.4 us, 0.8 sy, 0.0 ni, 3.8 id, 0.0 wa, 0.0 hi, 0.0 si, 0.0 st
KiB Mem: 65924036 total, 63918632 used, 2005404 free, 12727252 buffers
KiB Swap: 67054588 total, 70676 used, 66983912 free. 40191152 cached Mem
It's is really high!
Thanks
Andrew
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DittmarLab/HGTector/issues/5#issuecomment-233181063,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEMVN4ewIt7u9oPAojkgvsueC9jn6Svyks5qWih7gaJpZM4JI2Fx
.


v1.3.0 not available on conda-forge

hi, I saw the warning that maintenance is down to bare minimum but I was wondering about the process for conda-forge builds. I cannot find v1.3.0 on conda-forge, but I don't know what's required to have that build.

@nicoddemus did you provide the conda firge build or was it a random 3rd party?
I never did a conda build for this

Sorry, I must have missed the notification.
https://github.com/conda-forge/pytest-forked-feedstock/pull/9
Should be up in an hour or so. 👍


Implemented basic SMS support in the API project.

In this case, only the from, to, type and text are implemented for an SMSRequest. By using JsonProperty attributes the response is deserialized and mapped to class properties. Maybe someone else (or maybe even myself) might be motivated to add the other functionality that your API provides. if this gets merged. Hope you'll have a look at it and consider integrating this contribution.
Regards,
Felix Mann
ThisIsWhyTheInternetExists.com

Shit, just saw that someone else already did this. It seems that this project is kinda abandoned, so I don't think I'll put any effort in this. If you're still interested in merging my contributions instead of the already existing pull request, please let me know through this ticket.

Not abandoned, just getting started actually. Should have things running more smoothly in a little while.

Sorry for the delay. As @tjlytle said we've had a slow start. I ended up merging the other PR but pulled the SMS type and response enums from yours for future use. Thanks for the commit!

guess we are all working for the same @DJFliX  :P


Support customized headers

We are using docker engine swarm mode and one scenario is:
On one swarm worker, our service, which is running inside a docker container, will make some remote api calls to swarm manager leader. Now we want to enable the TLS for docker daemon on the manager leaders.
To make those remote api calls, we use dockerode module, and what we want to do is to set the headers(Host: ), then make the remote call by IP, not the common name, so we don't have to set it in /etc/hosts to tell what IP the common name will be resolved to.
e.g.
https://github.com/apocas/dockerode
var docker = new Docker({
  protocol: 'https',
  host: '<IP_ADDRESS>',
  headers:{Host: 'my-common-name'}, // we need it!!!
  port: process.env.DOCKER_PORT || 2375,
  ca: fs.readFileSync('ca.pem'),
  cert: fs.readFileSync('cert.pem'),
  key: fs.readFileSync('key.pem')
});

And in docker-modem:
var optionsf = {
    path: address,
    method: options.method,
    // headers when making this dial, if not, then the headers when new the modem object, and
    // then no headers
    headers: options.headers || self.headers || {},
    key: self.key,
    cert: self.cert,
    ca: self.ca
  };

This seems not necessary for docker-modem since it supports setting headers per request, but it's good to have this.


Need a hand?

@zargony
I'm working on ide-rust in conjunction with the Atom IDE and Rust Dev Tools movements.
I have some pain points with language-rust and I was taking a look but it seems like at least one of them ( #82 ) has an open PR.
I'd love to help you with adding syntax and improvements to the grammar here. Thoughts?

Any help would be much appreciated. Unfortunately I currently don't find much time to spend on improving language-rust. Most open issues are "complicated", i.e. they're not simple grammar rules problems but need a deeper look into how atom packages work (which I unfortunately don't have). If you'd like to stay for a while and help to move things ahead, I could as well add you as a collaborator. We should just make sure to discuss bigger changes openly first and to have working specs all time to prevent regressions.
#82 looks nice, but is missing some automated specs. It adds some lines to test.rs which was used for manually reviewing syntax highlighting before we had specs.

Any help would be much appreciated. Unfortunately I currently don't find much time to spend on improving language-rust. Most open issues are "complicated", i.e. they're not simple grammar rules problems but need a deeper look into how atom packages work (which I unfortunately don't have).

Is this still the case?

Yes. I think I know Rust pretty well, but my knowledge of Atom’s JS internals is limited and RL keeps me busy, so I appreciate any help.

ide-rust directly depends on this and @alexheretic now (and for quite some time) maintains it so he might have time and interest to help here

That would be great, to further improve this project! :)

Yes I, and all atom rust users, am interested in improvements. I don't have a lot of free time myself at the moment though.
Interestingly it looks like they'll also be official rust support for atom when they adopt the tree-sitter parsing system: https://github.com/atom/language-rust. But who knows when that'll be.

Thanks. Looks like there’s a tree-sitter based grammar for Rust now.


Add ability to use GitHub merge queues

fixes #289

Stack:

#314
#311 ⬅
#313
#310

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

✓ Commit merged in pull request #314


Add a test for #328

This change introduces a test for the code change of #328 (There were no tests originally for this functionality).
Adds to the fix for #278

Test failures don't seem related.


Retries failed pod submissions

Changes proposed in this PR
Running process for jobs that Cook wants to start and that don't yet have a corresponding pod in k8s.
Why are we making these changes?
Without this, jobs that have failed pod submissions for a transient reason (e.g. a timeout when talking to a mutating admission controller) will get a job instance in the Unknown state forever, and will not continue trying to do the pod submission.

Closing this in favor of PR #1614


about simulation

Hello sir:
All launch files in the upo_launchers fold  can't be used correct because there are no file "teresa.urdf" and e.t. I want to know if I rewrite the launch files with the same style by changing the teresa model to turtlebot model, it should work well?

Hello,
besides the need of the teresa model, there are other packages needed that
are not currently included in our repository.
We made some changes in the launch files because of the final demo of the
TERESA project. I'm fixing these dependencies and other issues probably on
Monday or Tuesday. So, you could try again with the new updates next week.
Thank you for your interest.
Best regards,
Noé Pérez
2017-05-04 14:39 GMT+02:00 CxyCH<EMAIL_ADDRESS>
Hello sir:
All launch files in the upo_launchers fold can't be used correct because
there are no file "teresa.urdf" and e.t. I want to know if I rewrite the
launch files with the same style by changing the teresa model to turtlebot
model, it should work well?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/robotics-upo/upo_robot_navigation/issues/1, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AOit-B9zJvDyR8H47FyJdcejcYyWfNVHks5r2cbxgaJpZM4NQnNf
.

--
Noé Pérez Higueras
PhD student - Service Robotics Lab
Robotics, Vision and Control Group
Pablo de Olavide University, Seville, Spain

Hi noeperez:
Thanks for your reply so soon.
There is some problem at package nav_features with that error message:
/home/cxy/catkin_ws/build/upo/navigation_features/setup_custom_pythonpath.sh: 4: exec: /home
/cxy/catkin_ws/src/upo/navigation_features/cfg/nav_features.cfg: Permission denied
[  5%] Built target upo_msgs_generate_messages_nodejs
upo/navigation_features/CMakeFiles/navigation_features_gencfg.dir/build.make:63: recipe for
target '/home/cxy/catkin_ws/devel/include/navigation_features/nav_featuresConfig.h' failed
make[2]: *** [/home/cxy/catkin_ws/devel/include/navigation_features/nav_featuresConfig.h] Er
ror 126
CMakeFiles/Makefile2:4446: recipe for target 'upo/navigation_features/CMakeFiles/navigation_
features_gencfg.dir/all' failed
make[1]: *** [upo/navigation_features/CMakeFiles/navigation_features_gencfg.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
Last changing of the nav_features package a month ago can be 'make' well, so I compare the last package and this package with 'diff', but I can't find any problem so could you please help me and check what happen ?  Thank you very much.

Hi,
the problem is the permission of the .cfg files. You can fix it by giving
execution permission to all the .cfg files (sudo chmod +x  filename.cfg).
I will change them as soon as possible. Anyway you can change them in your
local copy if you are in a hurry.
Sorry for the inconvenience.
Best regards,
Noé
2017-05-22 16:46 GMT+02:00 CxyCH<EMAIL_ADDRESS>
Hi noeperez:
Thanks for your reply so soon.
There is some problem at package nav_features with that error message:
/home/cxy/catkin_ws/build/upo/navigation_features/setup_custom_pythonpath.sh:
4: exec: /home
/cxy/catkin_ws/src/upo/navigation_features/cfg/nav_features.cfg:
Permission denied
[ 5%] Built target upo_msgs_generate_messages_nodejs
upo/navigation_features/CMakeFiles/navigation_features_gencfg.dir/build.make:63:
recipe for
target '/home/cxy/catkin_ws/devel/include/navigation_features/nav_featuresConfig.h'
failed
make[2]: *** [/home/cxy/catkin_ws/devel/include/navigation_features/nav_featuresConfig.h]
Er
ror 126
CMakeFiles/Makefile2:4446: recipe for target 'upo/navigation_features/
CMakeFiles/navigation_
features_gencfg.dir/all' failed
make[1]: *** [upo/navigation_features/CMakeFiles/navigation_features_gencfg.dir/all]
Error 2
make[1]: *** Waiting for unfinished jobs....
Last changing of the nav_features package a month ago can be 'make' well,
so I compare the last package and this package with 'diff', but I can't
find any problem so could you please help me and check what happen ? Thank
you very much.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/robotics-upo/upo_robot_navigation/issues/1#issuecomment-303121694,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOit-Blk2GmLneX50PNUy1p1BBtSgfqiks5r8Z-wgaJpZM4NQnNf
.

--
Noé Pérez Higueras
PhD student - Service Robotics Lab
Robotics, Vision and Control Group
Pablo de Olavide University, Seville, Spain

Hello sir:
There is some problem at package upo_robot_navigation-master/assisted_steering with that error message:
upo_robot_navigation-master/assisted_steering/include/assisted_steering/AssistedSteering.h:18:54: fatal error: assisted_steering/AssistedSteeringConfig.h: No file or directory
compilation terminated.
I tried to find this file, but con't.Can you give some advice?
Looking forward to your repley.
Thanks
sheng zhang

Good morning Sir,
I have downloaded the package and I had not problem with the compilation.
Anyway, it seems a problem with the file of the reconfiguration of the
dynamic parameters.
Could you check the permissions of your file
assisted_steering/cfg/AssistedSteering.cfg? Please try to give it write
permissions: chmod a+rw AssistedSteering.cfg and try to compile again.
Best regards,
Noé Pérez
2018-04-16 3:43 GMT+02:00 zhangshengHust<EMAIL_ADDRESS>
Hello sir:
There is some problem at package upo_robot_navigation-master/assisted_steering
with that error message:
upo_robot_navigation-master/assisted_steering/include/assisted_steering/AssistedSteering.h:18:54:
fatal error: assisted_steering/AssistedSteeringConfig.h: No file or
directory
compilation terminated.
I tried to find this file, but con't.Can you give some advice?
Looking forward to your repley.
Thanks
sheng zhang
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/robotics-upo/upo_robot_navigation/issues/1#issuecomment-381457236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOit-IwKtmCrRBnayZIpxG6MQev3LAE9ks5to_dOgaJpZM4NQnNf
.

--
Noé Pérez Higueras
PhD student - Service Robotics Lab
Robotics, Vision and Control Group
Pablo de Olavide University, Seville, Spain

Hello,
It's very strange. I have downloaded the package and I compile it without
errors.
I will go deeper and I will try with another computer in the following days.
Meanwhile, which ROS distribution are you using? Could you check that you
have installed the package ros-<ros_distro>-dynamic-reconfigure?
Sorry for the inconvenience, I'm very busy this week but I will try to fix
the problem as soon as possible.
Best regards,
Noé
2018-04-16 10:22 GMT+02:00 zhangshengHust<EMAIL_ADDRESS>
Hi noeperez:
Thanks for your reply so soon.
I try to give it permission,like the picture below.
[image: sdnaz936ejgx gu_jeg5 4i]
https://user-images.githubusercontent.com/33928697/38797358-ca61adb0-4190-11e8-9fec-02212ba5b153.png
But the above error still exists
[image: pymify 20kopb 581p a 0]
https://user-images.githubusercontent.com/33928697/38797723-063fba06-4192-11e8-8d2d-89b7f09d212e.png
Best regards,
sheng zhang
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/robotics-upo/upo_robot_navigation/issues/1#issuecomment-381518903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOit-ClWoReC6kWvV8mmDmwvhXbi3d6dks5tpFSxgaJpZM4NQnNf
.

--
Noé Pérez Higueras
PhD student - Service Robotics Lab
Robotics, Vision and Control Group
Pablo de Olavide University, Seville, Spain

Hi,
My ROS distribution is kinetic. dynamic_reconfigure package can be found.
I run them on a virtual machine , I don't know whether it result in  the error.
I am not in a hurry.Thank you again.
Best regards,
sheng zhang


Reduce memory consumption of baseX

Currently requires over 8 GB with a 1.6 GB database - I can't run it on a NIST computer
This can probably be fixed by outputting to a stream instead of holding all results in memory and then outputting at once.

I guess most of the memory is consumed by basex itself. I think this is currently not high priority. For Wikipedia we will manage to fit it into memory on 16G instance. For ArXiv we use stratosphere anyhow.


__builtin_offsetof generates to Marshal.OffsetOf

Marshal.OffsetOf is both going to be slow, and potentially incorrect in some cases (Although I'm not sure the generator can generate these invalid cases).
In newer versions of the windows sdk, FIELD_OFFSET changed from
#define FIELD_OFFSET(type, field)    ((LONG)(LONG_PTR)&(((type *)0)->field))

to
#define FIELD_OFFSET(type, field)    ((LONG)__builtin_offsetof(type, field))

which causes any macro using FIELD_OFFSET to now be generated with Marshal.OffsetOf

This is expected. The (nuint)&(((T*)0)->field) trick doesn't work in .NET and will generate a NRE due to the dereference required to get the field.
The next closest thing that .NET could do that isn't Marshal.OffsetOf is:
Unsafe.SkipInit(out T tmp);
return ((nuint)&tmp.field - (nuint)&tmp))

But this has its own complications.


Update MarkdownTextInput.web.tsx

Details

useLayoutEffect is a version of useEffect that fires before the browser repaints the screen.

It's ideal for cases where dimensions are derived from a DOM element and then updates some other element before react paints the screen. These seem like good use cases.
Related Issues

GH_LINK
close #166
Manual Tests

I didn't add any tests, I'll try a video comparison maybe?
Linked PRs

CLA throws a 404

Oops - CLA should is added now!

I have read the CLA Document and I hereby sign the CLA

Thanks!


Add Support X86 and ARM64 on big platforms

In this task it necessary to create libs to x86 for Linux, macOS, Windows OS

[x] X86 Windows
[x] X86 Linux
[x] X86 macOS
[x] AArch64 Linux
[ ] AArch64 Android

you may want to add aarch64 macOS aka Apple M1...
ps: https://github.com/Mizux/dotnet-native/blob/2c4737b1eb29bc3a93911a632520774fe1b74e2f/cmake/dotnet.cmake#L28-L30
:wink:

@Mizux, yes it would be great to add osx-arm64, but the current project is a full cpu_features binding, so first of all I'm looking forward to the day we add arm64 support to macOS :smile:. I didn't find emulation about apple m1, only bitrise CI/CD bitrise-apple-silicon-m1-stacks
in hobby plan of pricing, I see free use of M1 https://www.bitrise.io/pricing, but for me this is not a convenient docs/UI, maybe we need to check if we can integrate into cpu_features, but again there will be a mess with CI/CD services
also thanks a lot for your project https://github.com/Mizux/dotnet-native this project helps me a lot to understand how to integrate native code in .NET


Version 2 kick off with docker support

@rbmy FYI

will this be merged or replace what's in the master branch? I believe this is the feature/docker-v2 branch only.

I'd like to get this merged into master quickly.


Fix migration of VM with volume on Ubuntu

Description
This PR attempts to fix: https://github.com/apache/cloudstack/issues/6114

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 2869

@blueorangutan test ubuntu20 kvm-ubuntu20

@nvazquez a Trillian-Jenkins test job (ubuntu20 mgmt + kvm-ubuntu20) has been kicked to run smoke tests

@Pearl1594 @nvazquez @sureshanaparti
this sounds like a critical issue in 4.16 as well
(it seems this operation is not supported in centos7 with stock qemu)

@Pearl1594 can you re-target to 4.16 ?

@blueorangutan test ubuntu20 kvm-ubuntu20

@Pearl1594 a Trillian-Jenkins test job (ubuntu20 mgmt + kvm-ubuntu20) has been kicked to run smoke tests

Trillian Build Failed (tid-3614)

the issue seems to be caused by #5410
@GabrielBrascher could you please review this PR ?
@Pearl1594 I am wondering if this is better
        if (canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost) == StrategyPriority.CANT_HANDLE) {
            return StrategyPriority.CANT_HANDLE;
        }

I am not sure if it would be correct @weizhouapache (I may be wrong), but based on the 4.15.2 code , it looks like if the strategy returned by canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost is CANT_HANDLE, then there are some subsequent checks done before returning can't handle
if (canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost) == StrategyPriority.CANT_HANDLE) {
                Set<VolumeInfo> volumeInfoSet = volumeMap.keySet();

                for (VolumeInfo volumeInfo : volumeInfoSet) {
                    StoragePoolVO storagePoolVO = _storagePoolDao.findById(volumeInfo.getPoolId());
                    if (storagePoolVO.getPoolType() != StoragePoolType.Filesystem && storagePoolVO.getPoolType() != StoragePoolType.NetworkFilesystem) {
                        return StrategyPriority.CANT_HANDLE;
                    }
                }
            }

I am not sure if it would be correct @weizhouapache (I may be wrong), but based on the 4.15.2 code , it looks like if the strategy returned by canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost is CANT_HANDLE, then there are some subsequent checks done before returning can't handle
if (canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost) == StrategyPriority.CANT_HANDLE) {
                Set<VolumeInfo> volumeInfoSet = volumeMap.keySet();

                for (VolumeInfo volumeInfo : volumeInfoSet) {
                    StoragePoolVO storagePoolVO = _storagePoolDao.findById(volumeInfo.getPoolId());
                    if (storagePoolVO.getPoolType() != StoragePoolType.Filesystem && storagePoolVO.getPoolType() != StoragePoolType.NetworkFilesystem) {
                        return StrategyPriority.CANT_HANDLE;
                    }
                }
            }

@Pearl1594
OK, so the logic in this PR looks like same as what before #5410 , right ?

@weizhouapache thanks for pinging me.
I will double-check the file history to get some context and remember what changed over time and why.
Thanks for stepping up with this PR, @Pearl1594!
I believe that most of the work I've done on that PR was related to extracting a few validations added by @nvazquez in https://github.com/apache/cloudstack/commit/0fbf5006b87b6c5bc6188e3e1dd226a98c3bb453 and align with the workflow in case of Local volume + attached data disks. Might be good if @nvazquez can join us and double-check it as well.
I have some meetings and tasks to run. Get back as soon as I can to refresh my memory and re-debug this flow.
Regarding @Pearl1594`s comment in https://github.com/apache/cloudstack/pull/6116#issuecomment-1068867617
@Pearl1594 I believe that you are right indeed. There are some special cases to be handled, with regards to being NFS migration or a local storage migration, which led to the subsequent checks you've referenced.

I am not sure if it would be correct @weizhouapache (I may be wrong), but based on the 4.15.2 code , it looks like if the strategy returned by canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost is CANT_HANDLE, then there are some subsequent checks done before returning can't handle
if (canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost) == StrategyPriority.CANT_HANDLE) {
                Set<VolumeInfo> volumeInfoSet = volumeMap.keySet();

                for (VolumeInfo volumeInfo : volumeInfoSet) {
                    StoragePoolVO storagePoolVO = _storagePoolDao.findById(volumeInfo.getPoolId());
                    if (storagePoolVO.getPoolType() != StoragePoolType.Filesystem && storagePoolVO.getPoolType() != StoragePoolType.NetworkFilesystem) {
                        return StrategyPriority.CANT_HANDLE;
                    }
                }
            }

@Pearl1594 OK, so the logic in this PR looks like same as what before #5410 , right ?

Yes @weizhouapache

I am not sure if it would be correct @weizhouapache (I may be wrong), but based on the 4.15.2 code , it looks like if the strategy returned by canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost is CANT_HANDLE, then there are some subsequent checks done before returning can't handle
if (canHandleKVMNonManagedLiveNFSStorageMigration(volumeMap, srcHost, destHost) == StrategyPriority.CANT_HANDLE) {
                Set<VolumeInfo> volumeInfoSet = volumeMap.keySet();

                for (VolumeInfo volumeInfo : volumeInfoSet) {
                    StoragePoolVO storagePoolVO = _storagePoolDao.findById(volumeInfo.getPoolId());
                    if (storagePoolVO.getPoolType() != StoragePoolType.Filesystem && storagePoolVO.getPoolType() != StoragePoolType.NetworkFilesystem) {
                        return StrategyPriority.CANT_HANDLE;
                    }
                }
            }

@Pearl1594 OK, so the logic in this PR looks like same as what before #5410 , right ?

Yes @weizhouapache

@Pearl1594
OK, looks good to me. let's wait for the smoke test results

@blueorangutan test ubuntu20 kvm-ubuntu20

@Pearl1594 a Trillian-Jenkins test job (ubuntu20 mgmt + kvm-ubuntu20) has been kicked to run smoke tests

Trillian Build Failed (tid-3635)

@blueorangutan package

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 2904

@blueorangutan test ubuntu20 kvm-ubuntu20

@Pearl1594 a Trillian-Jenkins test job (ubuntu20 mgmt + kvm-ubuntu20) has been kicked to run smoke tests

@Pearl1594 @weizhouapache tested and it is working fine when migrating VMs with KVM + Root on Local.
In case tests are all good, I am LGTM in having this one merged.

Trillian test result (tid-3636)
Environment: kvm-ubuntu20 (x2), Advanced Networking with Mgmt server u20
Total time taken: 32410 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6116-t3636-kvm-ubuntu20.zip
Smoke tests completed. 92 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Tests on ubuntu are fixed, started a new round of tests for centos to ensure no regressions are added - can be merged after that

@blueorangutan test

@Pearl1594 a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3662)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46441 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6116-t3662-kvm-centos7.zip
Smoke tests completed. 86 look OK, 6 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_add_primary_storage_disabled_host
Error
0.58
test_primary_storage.py

test_01_primary_storage_nfs
Error
0.10
test_primary_storage.py

ContextSuite context=TestStorageTags>:setup
Error
0.18
test_primary_storage.py

test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL
Failure
581.46
test_vpc_redundant.py

test_02_redundant_VPC_default_routes
Failure
343.92
test_vpc_redundant.py

test_reboot_router
Failure
461.39
test_network.py

test_01_secure_vm_migration
Error
161.28
test_vm_life_cycle.py

test_02_unsecure_vm_migration
Error
266.01
test_vm_life_cycle.py

test_03_secured_to_nonsecured_vm_migration
Error
142.81
test_vm_life_cycle.py

test_08_migrate_vm
Error
44.96
test_vm_life_cycle.py

test_02_list_snapshots_with_removed_data_store
Error
9.53
test_snapshots.py

test_02_list_snapshots_with_removed_data_store
Error
9.53
test_snapshots.py

test_hostha_kvm_host_degraded
Error
698.20
test_hostha_kvm.py

test_hostha_kvm_host_fencing
Error
681.64
test_hostha_kvm.py

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3673)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31731 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6116-t3673-kvm-centos7.zip
Smoke tests completed. 91 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_hostha_enable_ha_when_host_in_maintenance
Error
300.68
test_hostha_kvm.py

Merging based on approvals and test results on ubuntu and KVM (error not related to the PR)


When I run it prompts

gu@gu:~/a$ cargo run --release
warning: /home/gu/.cargo/config is deprecated in favor of config.toml
note: if you need to support cargo 1.38 or earlier, you can symlink config to config.toml
warning: /home/gu/.cargo/config is deprecated in favor of config.toml
note: if you need to support cargo 1.38 or earlier, you can symlink config to config.toml
error: could not find Cargo.toml in /home/gu/a or any parent directory
I'm sure I have installed the necessary requirements. Can you help me solve this problem? Thank you very much.

What did you try to do? To install memflowup please follow the instructions from the readme:
The recommended way is to install it through our automated script:
curl --proto '=https' --tlsv1.2 -sSf https://sh.memflow.io | sh

Alternatively you can manually install it via cargo as well:
cargo install memflowup --force

Best,
ko1N


Render kcp documentation under kyma-project.io/docs/control-plane pages

Description
Changes proposed in this pull request:
Render kcp documentation under kyma-project.io/docs/control-plane pages:

add .kyma-project-io dir with important files to able render documentation in kyma website under /docs/control-plane pages

Related issue(s)
kyma-project/website#603

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

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


Cant deploy elk as azure-cli wheezy InRelease is not signed in the repo

Hi,
The deployment fails when the controllervm wants to install the azure-cli package:
nThe following additional packages will be installed:n aufs-tools cgroupfs-mount libapr1 libaprutil1 libltdl7 openresty-openssln openresty-opm openresty-pcre openresty-resty openresty-zlib pigznSuggested packages:n mountall openresty-restydoc zipn. The following NEW packages will be installed:n apache2-utils aufs-tools azure-cli cgroupfs-mount docker-ce libapr1n libaprutil1 libltdl7 openresty openresty-openssl openresty-opmn openresty-pcre openresty-resty openresty-zlib pigz unzipn0 upgraded, 16 newly installed, 0 to remove and 86 not upgraded. Need to get 111 MB/111 MB of archives.nAfter this operation, 486 MB of additional disk space will be used.nWARNING: The following packages cannot be authenticated!n azure-clinn[stderr]ngpg: requesting key 417A0893 from hkp server packages.microsoft.comngpg: key 417A0893: public key "MS Open Tech<EMAIL_ADDRESS>importedngpg: Total number processed: 1ngpg: imported: 1 (RSA: 1)nW: GPG error: https://packages.microsoft.com/repos/azure-cli wheezy InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY EB3E94ADBE1229CFnW: The repository 'https://packages.microsoft.com/repos/azure-cli wheezy InRelease' is not signed.
Any help?
Thanks!

Thanks for reporting. We'll take a look soon. @yuwzho

HI @caoladder , thanks for your reporting. We dig it and find this is a known issue from Azure CLI https://github.com/Azure/azure-cli/issues/6383 . Once the issue is fixed, this will be fine.

This still appears to be a problem even after the aforementioned azure-cli issue has been closed.  Are any updates @yaweiw or @yuwzho ?


fix: revert using merged options from generator

Fix regression in #364.
@antfu Do you think the options merging should be reverted too?

The options in each preset shouldn't be merged together (as it's the preset's custom & specific options). For those meant to be shared should be configure top-level.


Issue found on page 'SQLite Scanner Extension'

Please describe the issue on the page and include the "Page URL" link shown below.
Page URL: https://duckdb.org/docs/archive/0.9.0/extensions/sqlite.html
Hello Team,
We are using DuckDB in our application. Currently we are trying to implement the SQLite extension as it is mentioned that we execute both read and write operations concurrently.
We are using DuckDB driver to connect to SQLite DB and trying to execute an Update query and always getting an error that database is locked.
“duckdb.duckdb.Error: Invalid Error: database is locked”
Sample python code snippet is provided below as well
conn = duckdb.connect()
conn.install_extension("sqlite")
conn.load_extension("sqlite")

conn.execute("ATTACH 'sqlite_name.db' AS sqlite_db (TYPE SQLITE)")

conn.execute("<UPDATE_QUERY>")

Let us know how to resolve the issue of concurrently executing the Select and Update at the same time and also if this can be achieved using SQLite extension to DuckDB?
Appreciate your help in this regard!
-Naveen

Hi @yeturi, this repository is intended to host the documentation of DuckDB. Issues should be files related to the documentation itself – please do not file questions about DuckDB features.


Steam not logged in on MacOS

Describe the bug
Steam.steamInit() returns a .status 20 (Steam not running) on OSX. Windows and Linux exports are working fine. Running through Steam, so Steam IS running - GodotSteam just seems unable to see it?
Version of GodotSteam:
3.5.2
I'm no OSX expert, so I'm not sure how to debug this. Any idea what could be wrong?

Hmm, that's incredibly weird.  What version of OSX is this?

Also what version of GodotSteam are you running?  Is it the GDNative plug-in or the pre-compiled module?  The newest 3.19 or an older version?  The 3.5.2 you specified is, I think, the Godot engine version.

This is the release I'm using: https://github.com/Gramps/GodotSteam/releases/tag/g352-s155-gs319

Ah, OK, so the newest GodotSteam.  I was gonna say we (foolishly) don't use the GodotSteam versioning in the actual file names.  Your export thing looks fine thus far but I'll double check my project to be sure.

Hmm, you're export settings look like mine.  Maybe this is Ventura-specific for some reason?  I'd definitely find out what the OSX version numbers of other users are since I can't duplicate this error in Monterey (12.x).  Trying to find someone who might have Ventura to test this still.
Can you do a minimum test version?  Basically a new, blank project with just a print(Steam.steamInit()) in the _ready() function; with logging enabled.  Export it as your game's name and drop it into its Steam folder then run int.

How'd the minimum test go?

Our Mac user is on vacation, so it will be a few weeks before I can test anything Mac related sadly.

Finally got around to doing a minimum version. No change. Output from Steam.steamInit() is still {status:20, verbal:"Steam not running."}

I thought I had responded to this but apparently only in my mind. I haven't had any other Mac users mention this so I wonder if it is some localized setting.  Kind of like Ubuntu's Snap version of Steam where Godot cannot talk to the Steam client.  Perhaps some "security" feature of OSX since GodotSteam isn't really a signed executable?

When we had the build released we didn't hear from any Mac user who had it actually working. So I think it affects everyone. Would it change anything if I made the Mac build from a Mac instead of from my Windows pc?

I got it working! Fix was to simply disable Codesign:

Hey-o!  Glad it was an easy fix.  I had asked some folks if they had any ideas.  I added this to the Mac exporting tutorial for future generations!


[SPARK-22876][YARN] Respect YARN AM failure validity interval

What changes were proposed in this pull request?

Adds support for using the YARN application master failure validity interval for determining whether an attempt is the last attempt during shutdown. During the application master shutdown hook, if the validity interval is specified, we get previous attempt information from the resource manager to make a best attempt at mirroring whether YARN is going to retry the application or not. This is because, AFAIK, there is no way for Spark to know 100% if YARN will retry the application or not.
This can lead to two scenarios where Spark is wrong about whether YARN will retry it or not:

Spark thinks it is the last attempt but YARN will retry the application. This is what can already happen by not respecting the failure validity at all.
Spark thinks the application will retry but YARN thinks it was the last attempt. This will result in the staging directory not be cleaned up, but that can already happen if Spark gets killed during the last attempt and doesn't run it's shutdown hooks.

Why are the changes needed?

Spark supports passing the AM application validity interval to YARN using the spark.yarn.am.attemptFailuresValidityInterval setting, but then ignores this during shutdown when determining whether it's the last attempt and whether to cleanup the staging directory. Currently if you try to use this setting, YARN will respect the setting and attempt to retry the application, but only if the last attempt was killed with a hard shutdown (like OOM) that prevents the shutdown logic from occurring. During a graceful shutdown, Spark will delete the staging directory and unregister the application from YARN, preventing any further attempts.
Does this PR introduce any user-facing change?

Yes, Spark on YARN better supports graceful recovery and retry of jobs to better enable long running Spark Streaming jobs.
How was this patch tested?

New UT. It unfortunately relies on sleeping to test the validity timing, which seems very prone to flakiness, but I couldn't think of any other way to test it. If the approach seems sound, it might be worth just removing the test. I don't see any tests for simply the max YARN attempts.
Was this patch authored or co-authored using generative AI tooling?

No

+CC @tgravescs

so to clarify the issue here, the original PR that added this validityInterval config (https://github.com/apache/spark/pull/8857/files) just seems to call the YARN setAttemptFailuresValidityInterval().  So you are saying that config works on the YARN side but on the Spark side we think its the last attempt when it really isn't. That makes sense.
I'm not sure I understand how your second point above though (Spark thinks the application will retry but YARN thinks it was the last attempt.) happens with this config?  what is the scenario there?

I'm not sure I understand how your second point above though (Spark thinks the application will retry but YARN thinks it was the last attempt.) happens with this config? what is the scenario there?

Theoretically that should only happen if there are inconsistencies/bugs in the implementation on the Spark side compared to the YARN side. Just included it has a general "risk" of adding this.

This has been working great in production for us for nearly a year now


Tests

Highlights

[x] Write unit tests using Rspec for the following classes:
[ ] Book and the result after running the test is shown in the image below.

[ ] Person and the result after running the test is shown in the image below.

[ ] Student and the result after running the test is shown in the image below.

[ ] Teacher and the result after running the test is shown in the image below.

[ ] Classroom and the result after running the test is shown in the image below.

[ ] Rental and the result after running the test is shown in the image below.

Hi @kennedyowusu,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

The Pull Request has a proper title and description. ✔
You have added a descriptive Readme file. Good Job. ✔
There is no linter error is present. ✔
Project has met all the technical aspects you needed for this project. Keep rocking! ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you for approving the project. @arslanbisharat


fix🐛:  DriverClassName

基于问题而修复
[11:22:39] [Server thread/ERROR]: [STDERR] [com.mysql.jdbc.Driver] Loading class `com.mysql.jdbc.Driver'. This is deprecated. The new driver class is `com.mysql.cj.jdbc.Driver'. The driver is automatically registered via the SPI and manual loading of the driver class is generally unnecessary.

有没有一种可能，老版本服务器无法识别这种新版本驱动

出现这个问题的让用户自己改配置就行了


module data source is not found when running a terraform refresh on a code that is calling a module!

Hello Team, Basically I have run the code given below through the Azure DevOps pipeline already. Got few existing resources so want to import the state file locally . However when I am running the terraform refresh, I get the below error ;
" Error: Route Table: (Name "platform-rt-uks-legacy-prd-zscl" / Resource Group "platform-rg-uks-legacy-prd") was not found
│   with module.landing_zone.data.azurerm_route_table.zscl["uks"],
│   on ..\module-tf-landing_zone\networking.tf line 43, in data "azurerm_route_table" "zscl":
│   43: data "azurerm_route_table" "zscl" {
"
Although the above route exists in the portal.
Here is my code below:
resource "azurerm_route_table" "rt" {
for_each = var.Location
name                          = replace(replace(replace(local.roleCount, "type", "rt"), "role", "lz"), "count", "0${1 + index(keys(var.Location), each.key)}")
location                      = azurerm_resource_group.rg[each.key].location
resource_group_name           = azurerm_resource_group.rg[each.key].name
disable_bgp_route_propagation = false
tags                          = local.tags
lifecycle {
ignore_changes = [
tags["FirstCreated"]
]
}
}
resource "azurerm_route" "routes" {
for_each = local.rt_routes
name                   = each.value.rt_key
resource_group_name    = azurerm_resource_group.rg[each.value.loc_key].name
route_table_name       = azurerm_route_table.rt[each.value.loc_key].name
address_prefix         = each.value.prefix
next_hop_type          = each.value.hop_type
next_hop_in_ip_address = each.value.region == "UK South" ? local.firewalls.uksouth : local.firewalls.ukwest
}
data "azurerm_route_table" "zscl" {
provider = azurerm.prd-hub
for_each = local.zscl-rt-table
name                = each.value.name
resource_group_name = each.value.rg
}
resource "azurerm_route" "zscl-routes" {
provider = azurerm.prd-hub
for_each = local.zscl-routes
name                   = each.value.loc_key == "Primary" ? replace(local.baseName, "type", "RouteTo") : replace(local.baseNameSecondary, "type", "RouteTo")
resource_group_name    = data.azurerm_route_table.zscl[each.value.zscl_key].resource_group_name
route_table_name       = data.azurerm_route_table.zscl[each.value.zscl_key].name
address_prefix         = each.value.address_space[0]
next_hop_type          = "VirtualAppliance"
next_hop_in_ip_address = each.value.zscl_key == "ukw" ? local.firewalls.ukwest : local.firewalls.uksouth
}

Please open this issue in Terraform repo

Please open this issue in Terraform repo

May I know where please?


Fix offset_col to match label with for non-sm column classes

When building horizontal forms with a label_col other than col-sm-*, the offset would be calculated incorrectly. I changed the method offset_col to take other column breakers into account.

Very nice! ✨
Can you add a CHANGELOG entry so I can merge this? Thanks!

Could you fix the merge conflict in the CHANGELOG? Please don't delete the "your contribution here" line. Thanks!

Thanks for your contribution! 🙇


Remove #![feature(alloc_layout_extra)]

A subset of this feature was stabilized in https://github.com/rust-lang/rust/pull/69362,
and none of the still-unstable methods are in use in blog_os

Thanks a lot!
I merged this manually in 7bfacb39c46c8f99d82b1c146a46222ebc41a39b since this also affects the post-11 branch. (Unfortunately, Github doesn not seem to recognize this.)


Update Provider.js

Any particular reason you prefer this style?

Less variable declarations, smaller code

I wouldn’t think either is more efficient but I’m OK with this, it does read easier on eyes.


witness_only still mentioned in Python docs

Background
"witness_only" is still mentioned in the Python docs, even though the parameter was removed in 0.4 according to the release notes (https://github.com/lightningnetwork/lnd/releases)
https://github.com/lightningnetwork/lnd/blob/117441b7d0b9ccbffdc24e47776c7df81d47f7e7/docs/grpc/python.md
Your environment
master. Other details irrelevant since this is a doc error.
Steps to reproduce
Irrelevant since doc error.
Expected behaviour
No use of "witness_only"
Actual behaviour
"witness_only" mentioned

Ah, yes. Want to make the easy doc correction PR?

decided to pick this quick one myself at  #1051


yle-dl: 2.31 -> 20200807

Motivation for this change
https://r.ryantm.com/log/2020-10-09.log
Things done

Updated to latest version and fixed tests when enabled.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@SuperSandro2000 Please, update license to licenses.gpl3Plus, since licenses.gpl3 is deprecated (https://discourse.nixos.org/t/lib-licenses-gpl3-co-are-now-deprecated/8206/4).
See https://github.com/aajanki/yle-dl/blob/3100fa5f4f82583d092e580930b84cd3214860f1/setup.py#L56:
        'License :: OSI Approved :: GNU General Public License v3 or later (GPLv3+)',

@SuperSandro2000 Please, update license to licenses.gpl3Plus, since licenses.gpl3 is deprecated (discourse.nixos.org/t/lib-licenses-gpl3-co-are-now-deprecated/8206/4).
See<EMAIL_ADDRESS> 'License :: OSI Approved :: GNU General Public License v3 or later (GPLv3+)',

@sikmir Changed that. Thanks for the notice.

20201022 is out.

20201022 is out.

Updated. Thanks for the notice!

yle-dl moved to python3 https://github.com/NixOS/nixpkgs/issues/101964

I was wondering why this looked so familiar https://github.com/NixOS/nixpkgs/pull/67656


Invalid ARIA roles

I was recently looking through a list of the most used ARIA roles according to HTTP Archive and found that the 15th most common one is role="bar". Doing some BigQuery sleuthing I discovered that much of the usage may be coming from this library.
To my knowledge, ARIA doesn't support a role of either bar or spinner. A full list of roles can be found here. My concern is that, from the accessibility side of things, it may be teaching folks to use roles which don't convey any meaningful information to assistive technology like a screen reader.
There is a progressbar role, which seems closer to what you're after. I guess since a spinner is a kind of indeterminate progressbar you could also use that role, combined with the recommendation from the spec:

The author should supply values for aria-valuenow, aria-valuemin, and aria-valuemax, unless the value is indeterminate, in which case the author should omit the aria-valuenow attribute.

For anyone that finds this in the future I have forked this repo and created accessible-nprogress that adds accessibility support for this library. - https://github.com/nmackey/accessible-nprogress


Encourage Users To Specify Versions When Using ignite.addModule

Deprecation warning:
ignite.addModule should require version number.  Unless yolo: true
Update Docs
https://github.com/infinitered/ignite/blob/master/docs/advanced-guides/creating-plugins.md#edit-the-pluginjs

@GantMan Can you explain further what you mean here?

ah yes, sorry.
So addModule is willy nilly about the version number.   So whatever code you wrote, could be defunct if the module has any major breaking changes!!!!   Not specifying a version number on the module is pretty careless.  SO, we should enforce version numbers!!!
This ticket is asking for a deprecation warning on any line of code that uses this without a version being passed in.
Now, if a person feels they should NOT put a version number because they like to live life fast and crazy, we require yolo: true to supress the warning.   But it's highly recommended that every module added is the specific version that has been tested.

We could place a warning if version is not detected.
Updating [the plugin template] would be another nice touch.

Thanks for the additional context, @GantMan. Good thoughts @skellock .


Getters in manually implemented get-set properties are not translated

My use case is that I want to effectively add an alias for a member. So here for example I add an Id property which just points to DocumentId. I need the getter of Id to be projectable for it to get translated correctly in queries, but I also happen to need a setter (it's implementing an interface). I cannot separate out the two, because if an interface defines a property as get; set; you have to implement both in one go.
        [NotMapped] public string Id { [Projectable] get => DocumentId; set => DocumentId = value; }
        public string DocumentId { get; set; }

For reference, the id comes from an interface like this:
interface IId<T>
{
    T Id { get; set; }
}

I do need both get and set in the interface and I can't separate them out.

This sounds like a source generation issue and/or a runtime resolution issue and should be relatively easy to fix. Happy to take a PR here.


[Component] Slider

On the web side the design specs seems to be ready? wdyt @JeremiasUX ?

Yea, I would say so

Yea, I would say so

It seems we only miss the disabled state update (from DIM4 to DIM3) and you'll have an approval on the web side :tada:

@elisa-hery The component is ready to test on web side, be my guest! 🙂

Moving back to dev stage, as there has been some changes on halo styles.

Hi everyone :wave: Following last design updates, here is the last version of web implementation: https://github.com/adevinta/spark/pull/1406#issuecomment-1737554821 Thx in advance for your reviews :pray:

@elisa-hery Last changes have been approved by team mates, so we're good for a new testing session! :tada:

@soykje here are my comment after testing :

in responsive and on chrome only, it's almost impossible to move the "handler" (except when clicking on the track, where it does move - see video below)
about focus, figma says "keyboard navigation only", but on storybook you can see the focus when you click. I don't know if this is normal! --> it's a radix bug, seen with @turolopezsanabria and @JeremiasUX we can merge like this https://github.com/adevinta/spark/issues/1471
track colour: on Storybook the onBackground is black #000000, whereas on figma it's the colour #1D1F2A --> slack thread here
the specs clearly state that the border-radius is customisable, shouldn't this also be written in storyBook ? --> will be updated
perhaps to check during design review : if you want to click on the track to move the handler, the click zone is very small (4px high) and it's not easy. I don't know if we could/would enlarge this area

https://github.com/adevinta/spark/assets/142396647/e9802694-f4b2-425e-a94e-def25ed2fd83

@soykje thanks, thanks ! My comments :

fixed ✅ !
ok
i'll talk about it with @JeremiasUX
done ✅  thanks !
i'll talk about it with @JeremiasUX

@JeremiasUX --> component ready for design review !

Design review:

"Pressed" should not have a focus. Focus is only for keyboard navigation
Range Slider: As soon as you move one handle above the other, the design changes of the second handle (image attached)
Shape naming should be Square and Pill
"Prevent thumb overlap" is not part of the spec, is that feature necessary?

What do you mean by "pressed" here? It seems like the same as focus, or am I missing something?
Not sure to get your point... Maybe we could discuss this directly?
N/A
Not necessary at all, but a suggestion as the base component from Radix UI proposes it :slightly_smiling_face: It seems to me a nice to have, but we can omit it if you prefer :+1:

Hello 🙋🏻♂
yes confirmed, we can move forward, just make sure we have the 2 remaining issues documented in tickets so we don't lose track of them


Docker-Compose example for instant run of SecHub Server, PDS, and Databases locally

Proposal
We could offer a docker-compose.yml example file that allows to start everything necessary to run, develop and debug SecHub locally.
The example code could be placed in a subfolder examples:
- sechub
    - examples
    - ...

The corresponding docker-compose.yml could look like this:
version: "3.9"

services:
    sechub-server:
        build:
            context: ../
            dockerfile: ./examples/sechub-server.dockerfile
        ports:
            - "8443:8443"
        profiles: ["full", "sechub-server", "pds"]
        networks:
            - sechub
            - pds

    sechub-db:
        image: postgres
        profiles: ["full", "sechub-server"]
        networks:
            - sechub

# ---------------------------------------------------------------------
# -------------------------------- PDS --------------------------------
# ---------------------------------------------------------------------

    pds:
        build:
            context: ../
            dockerfile: ./examples/pds.dockerfile
        ports:
            - "8444:8444"
        profiles: ["full", "pds"]
        networks:
            - pds

    pds-db:
        image: postgres
        profiles: ["full", "pds"]
        networks:
            - pds

networks:
    sechub:
    pds:

Starting only SecHub-server with a database could be done like this:
docker compose --profile sechub-server up --build
or with all components
docker compose --profile full up --build

This issue is solved. One can start multiple PDS-Solutions and SecHub is containerized.
The only missing part is, that the approach is not yet described.
This will change with: #1646.


Add ratings: true for developer call

The same as the one in the app, would be great for fetching all developer apps info.

Any thoughts @facundoolano? This would be a great addition and achieve feature parity with play store scraper 😁

Any updates @facundoolano?


The JWT string must have two dots

Hello,
I tried using this SDK to try and verify token which I get when user subscribes to web push notification. But I'm getting this error: The JWT string must have two dots.
Am I missing some step here? The token I get from Firebase indeed does not look like JWT.
Do I need to do something with it before or this SDK can't be used to verify Firebase web notification token?

Hey @paradox37, and thanks for reaching out. Token verification can be applied only to ID tokens as described in https://firebase.google.com/docs/auth/admin/verify-id-tokens

Then how do we get the User ID to verify the userID?

The IdToken is authenticated by another application and you can verify that it has been authenticated correctly. If you just want to get a user by ID, you don't need to generate an IdToken to then verify it - just use the UID directly.

How


Release comparison 'Cannot read property 'map' of undefined'

No output file is created and the log shows the error as per title.
Not sure if this something my side I need to look and what the extension is expecting to continue successfully.
This is occurring in a new Release Definition so could be something I still need to configure?
Appreciate any guidance on the issue you can give.
VSTS Extension you are using
GenerateReleaseNotes Extension (XPlat)
Where are you running it?
*VSTS, run on agent agent = on prem.
Version of Extension/Task
1.1.5
Expected behavior and actual behavior
No output file generated.
Last few lines of log:
2017-05-08T10:19:54.8705134Z Getting the details of release 616
2017-05-08T10:19:55.3952718Z Getting the details of build 3250
2017-05-08T10:19:56.3495633Z Getting the details of last release using definiton 'ESE Client (test)' that was successful in environment 'Release Notes'
2017-05-08T10:19:57.2240861Z Can find not successful release to the request stage, using first release as baseline
2017-05-08T10:19:57.2240861Z Getting the details of release 562
2017-05-08T10:19:58.2369425Z Getting the details of build 2892
2017-05-08T10:19:59.0864186Z Getting the workitems associated between release 616 and release 562
2017-05-08T10:20:00.2065207Z ##[error]Unhandled: Cannot read property 'map' of undefined
2017-05-08T10:20:00.2065207Z ##[section]Finishing: Generate Release Notes based on Release Comparision API
Steps to reproduce the problem
Not sure

Could you run again adding the global release variable system.debug = true to get a more detailed log file.
From this log I can see that it getting the build, but seems to have an issue looking up the work item.
There is the chance it is the template you are using, is it one of the samples in the the repo or are you looking for different fields? Do you get the same issue with a sample template?

Hi
Thanks for the quick response but seems I wasted your time - my apologies.
I'm using the sample template at the moment  but found the issue.
The build it fell back to (build 2892) completed on the build server (this branch using Jenkins), but the tfs side is incomplete/failed. Did not notice this earlier as I'm still finding my way in vsts.
I removed all releases and started from scratch using a successful build and we have success!
Do have a another question though if you would indulge me please.
Is it possible to select which previous build to compare to vs the last successful one?
Trying to setup that this would not be required, but with my testing I created for argument sake the following:

RTM release
Beta release
Update Release
RTM release.

Say I want to compare the two RTMs instead of adding together the diffs of the other releases - as you would do in VSTS UI?
Just something I was wondering while trying to setup and test.
Thanks
Jacques

Glad you got sorted.
As to what to compare too, I work out the last successful build of the current stage, or a stage you define with an overriding parameter e.g.you are in 'test', but you want to compare to 'release'
As I  use the same calls as the VSTS UI in this task I can do the same compares it can. The problem is how to get the IDs of the releases/builds to compare into the actual build process?
When designing the task I could think of no options other than the current release ID and the last successful, which I use. Any other would need an ID to be manually entered on a per build basis, at which point using the VSTS UI seemed a better choice.
Open to ideas.

Closing as I think this is addressed now, open it again if I am wrong

It was working but now it fails again.
Getting this after template process:
2017-05-09T08:01:57.1069063Z Getting the details of release 628
2017-05-09T08:01:58.5757832Z Getting the details of build 2945
2017-05-09T08:01:59.4194895Z Getting the workitems associated between release 629 and release 628
2017-05-09T08:02:00.6224847Z Getting the workitem details for 5096,4950,5122,5132,4034,4975,5041,4033,5009,4992,4973,1419,1420,2901,3159,3756,3758,4032,4653,4656,4802,4817,4831,4863,4919,4938,4941,4970,4972,4035,1375,3045,3112,3714,3893,4307,4798,4844,4847,4865,4691,4935,4927,3947,4898,3968,3993,4458,3050,4884,3975,4866,4872,4874,4828,3946
2017-05-09T08:02:02.1225203Z Repository type TfsGit
2017-05-09T08:02:02.1225203Z Getting the commits between commit ID 71ca630049ac2ebf23b6438ea7df497ff33f5a74 and 0e179b4f3e394f6fefbfcdfb73c88a71ef1852b4 from repo eada8c5a-2b41-4b24-9a3b-d6b301c5b9da
2017-05-09T08:02:03.4506783Z ##[debug]Using template mode InLine
2017-05-09T08:02:03.4506783Z Using in-line template
2017-05-09T08:02:03.4506783Z Processing template
2017-05-09T08:02:03.4506783Z ##[debug]task result: Failed
2017-05-09T08:02:03.4506783Z ##[error]Unhandled: Unexpected token {
2017-05-09T08:02:03.4506783Z ##[debug]Processed: ##vso[task.issue type=error;]Unhandled: Unexpected token {
2017-05-09T08:02:03.4506783Z ##[debug]Processed: ##vso[task.complete result=Failed;]Unhandled: Unexpected token {
2017-05-09T08:02:03.4663032Z ##[section]Finishing: Generate Release Notes based on Release Comparision API
2017-05-09T08:02:03.4820882Z ##[debug]Cleaning agent temp folder: L:\VSTSAgent2_work_temp
2017-05-09T08:02:03.4820882Z ##[section]Finishing: Release
Not sure what changed before this.

I assume the template has not changed, so my guess is a property you reference in the template is null, or any array when unexpected so we end up with a stray {.
I usually remove lines from the template to isolate the problem line

Correct, template unchanged.
Removed lines as suggested and did spot something I'm not sure off. Mentioned to one of my colleagues and he has an idea. Something they changed I think so something we have to sort.
Thanks again for the help.
Regards
Jacques