Add debugging support

This is of course predicated on rustc emitting debuginfo (pdb?) that Visual Studio can understand, but there's nothing nicer than Visual Studio's debugger.

Looks like Microsoft will be doing our work for us:
http://blogs.msdn.com/b/vcblog/archive/2015/04/29/open-sourcing-visual-studio-s-gdb-lldb-debug-engine.aspx

As of the 1.2 release the other day the 1.3 beta and 1.4 nightly have support for the msvc toolchain which emits a pdb. I've tested my project and I'm able to debug my project with VS.

The MSVC ABI version of Rust for Windows indeed creates PDB files on build, however debugging doesn't seem to work with Visual Rust when F5-ing; breakpoints are ignored.

New Rust coder and experiencing the same as @sunsided points out.
Visual Studio 2015, using a simple 'hello world' project.

With #199 debugging under MSVC ABI (both 32 and 64 bit) should work now.

I'm using the latest AppVeyor build based off #200 and Rust stable 1.5 x86_64-pc-windows-msvc. The call stack and break points work (they worked in the November build as well) however watch windows don't work for variables. I'm using the Rust code below modified from the application template and compiling/running in debug mode and get identifier "xxx_this_is_my_symbol_xxx" is undefined in the watch window. Am I doing something wrong?
    let mut xxx_this_is_my_symbol_xxx = 5;
    println!("Hello, world! {}", xxx_this_is_my_symbol_xxx);
    xxx_this_is_my_symbol_xxx += 1;
    println!("Hello, world! {}", xxx_this_is_my_symbol_xxx);

Rust MSVC debug info is currently lacking due to LLVM being kinda awful at it still. It doesn't know how to emit debug info for local variables yet. You're not doing anything wrong, and there's not much Rust or VisualRust can do here other than wait for LLVM to get their act together.


Update the func.yaml File Reference to include Default and Range values

The following file should document all the possible values in the func.yaml file:
https://github.com/fnproject/docs/blob/master/fn/develop/func-file.md
However, the file does not document default, maximum, or the valid range of values for an attribute. Where relevant, the file should be updated to include these values.

PR: https://github.com/fnproject/docs/pull/116

PR: #116


Logitech G305 Support

Tested under libratbag 0.9.903 and git 44d36d2. Able to get it to show up in ratbagctl with:
Name=Logitech Gaming Mouse G305
DeviceMatch=usb:046d:c53f
Svg=logitech-g303.svg
Driver=hidpp20
LedTypes=dpi

[Driver/hidpp20]
DeviceIndex=1

The name showing up is just under the receiver though, "Logitech USB Receiver Mouse", and it does not seem to properly detect my DPI changes or be able to set them. Will attach lsusb output or anything else needed to help get this added.

g305.txt

Hello,
Have you found a way to get it working since then?

Yeah, it uses the same Lightspeed connector as the G900, G903, G603 and G403. Can you build the project and invoke ./ratbagctl.devel -vvv resolution active set 1600 in the build directory?

ratbagctl list does not see my device, so I'm not sure what to input for that.
I've tried with the above device file but it's not much better:
https://gist.github.com/John-Gee/188bef6d6078ff776fc2e92c7b833c0f

It does see the mouse but there's an issue with the protocol. This is weird since the Lightspeed connector is working fine in my G603. I think libratbag is trying to read an invalid onboard profile.

Any update on this?

Working on it.

I managed to get ratbagd to talk with the device. Now I am getting an HID++ error, I'll need to explore this.

Is that patch a kernel patch? I have the same issue with my g305, piper or ratbagd
"New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 046d:c53f

So we just have to wait for the patch to be applied to the kernel or how does it work?

Yes, it will be out in 5.4.

Looks like the patch applies cleanly on 5.3, I'm running it now and it does resolve the issue on the ratbag side. Piper is missing the UI for G305 still but at least it shows up now! Thanks.
Not sure if I should close this before it's landed in a stable kernel or not.

@Vash63 https://github.com/libratbag/piper/pull/445 should fix the missing SVG in piper.
I will close this now Linux 5.4 is out and the latest libratbag release includes support for the G305. People with issues, please go to #846.


Use CSS-in-JS instead of plain CSS

This project uses Semantic UI, but for a few cases where we need custom CSS I think it would be nice to use styled-components or a similar library

I got it


TypeError: Expected signal to be an instanceof AbortSignal

Hi,
I'm getting "TypeError: Expected signal to be an instanceof AbortSignal" when running "Azure Pipeline: Configure Pipeline". I select "Simple Application to Windows Web App", then select my Subscription, then the error happens.
Console shows:
mainThreadExtensionService.ts:64 TypeError: Expected signal to be an instanceof AbortSignal
	at new H (c:\Users\pkruk\.vscode\extensions\ms-vscode.azure-account-0.9.8\dist\extension.js:253:17134)
	at c:\Users\pkruk\.vscode\extensions\ms-vscode.azure-account-0.9.8\dist\extension.js:253:18457
	at new Promise (<anonymous>)
	at G (c:\Users\pkruk\.vscode\extensions\ms-vscode.azure-account-0.9.8\dist\extension.js:253:18420)
	at t.<anonymous> (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:726791)
	at c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1649941
	at Object.next (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1650046)
	at c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1648983
	at new Promise (<anonymous>)
	at n (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1648728)
	at t.fetch (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:726717)
	at t.<anonymous> (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:725345)
	at c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1649941
	at Object.next (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1650046)
	at n (c:\Users\pkruk\.vscode\extensions\ms-azure-devops.azure-pipelines-1.188.1\dist\extension.js:2:1648785)
	at processTicksAndRejections (internal/process/task_queues.js:93:5)

From what I google it might be problem with webpack minimalization in production build...
Versions:
Azure Pipelines 1.188.1
VSCode:
Version: 1.58.2 (user setup)
Commit: c3f126316369cd610563c75b1b1725e0679adfb3
Date: 2021-07-14T22:10:15.214Z
Electron: 12.0.13
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043

I have the exact same problem.
VSCode:
Version: 1.60.0 (user setup)
Commit: e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff
Date: 2021-09-01T10:41:52.311Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19042

Azure Pipelines extension:
v1.191.0

Version 1.195.0 of the extension has been released. Let me know if you still have issues after updating.


Handle CSV uploads with a UTF8 BOM prefix

This can be happen when exporting CSVs from Excel.

Original implementation: 226fdd47


GRPC Calls to Bigtable & Datastore fail

Using:
gcloud==0.18.1
grpcio==1.0.0

All the RPC calls we do on Bigtable via the Python client seem to fail, e.g. listing the instances of a project returns:
In [1]: from gcloud import bigtable

In [2]: client = bigtable.Client(project=u'project-id', admin=True)

In [3]: client.start()

In [4]: client.is_started()
Out[4]: True

In [4]: client.list_instances()
AbortionError: AbortionError(code=StatusCode.INTERNAL, details="{"created":"@1472199496.200250000","description":"RST_STREAM","file":"src/core/ext/transport/chttp2/transport/frame_rst_stream.c","file_line":107,"http2_error":1}")

We've tried downgrading the gcloud or the grpcio library but without success. Also, the Bigtable Golang client seems to function correctly.
Edit: The issue is also present when querying Datastore over GRPC.

Other folks are seeing this problem as well.  We are actively investigating this issue.

Also seeing this. Started yesterday/last night as far as I can tell.
We attempted to move back to
grpcio (1.0.0rc2)

with no improvement.
Thanks!

We think we found the culprit.  We are rolling out a server side fix now.

@sduskis Great, thank you for the quick resolution!

@tseaver: I think that there's follow up work.  gRPC should handle RST_STREAM more gracefully.

I am assuming this must be out of date. Please re-open if it is still an active issue.


Cookie window prevents use of Visual Filter Selector

Describe the bug
When I use the visual file selector the cookie settings window prevents me from using it.
See example: https://imgur.com/a/XGW1bkO
Version
v<IP_ADDRESS>
To Reproduce
Example site: https://www.notebooksbilliger.de

Describe the bug
Same as original post. Visual selector doesn't 'see' the cookie window. Therefore, can't use visual selector.
Version
v0.45.25
To Reproduce
Example site: https://www.keimling.de/tribest-shine-multi-batch-compact-cold-press-juicer-weiss/


I rerun the code by cellranger3.01 get 12308 cells

I rerun the code by cellranger3.01 with eWAT Lin- data. In total, I got 12308 cells, 6246 in the Control and 6206 cells in CL  treatment. But in your paper, you just have 9915 cells. Did you filter again. what's more, I also want to know if you plot the Figure1A with the cluster result produced by cellranger. Because I  can't get the similar figure like yours. Can you give me code to plot  Figures 1, 2, S2, S3, S5，Figures 3, S4， Figures 4, S6，Figure 5， Figure 6.
and I saw you used the " kmeans_10_clusters" file. But in your figure just 8 clusters. I want to know the reson. Did you remove the other clusters?

Hello,
The data for this paper was processed using cellranger v. 2.0.1. For cellranger v. 3, 10X Genomics changed their cell-calling algorithm. And while I haven't used v3 on our gWAT dataset, it is my experience that v3 calls more cells than v2. You can read up on the new cell algorithm here: https://support.10xgenomics.com/single-cell-gene-expression/software/pipelines/3.0/algorithms/overview#cell_calling. So I would guess that most of your problems with replicating the figures in our paper comes from the difference in cellranger version. Otherwise the code posted here should completely replicate the figures in the paper. Figure 1, and most of the figures, comes directly from the .cloupe file generated by cellranger. And for the question about the kmeans_10_clusters, for Figure 1 the clustering algorithm was Kmeans 10 clusters, and this data has 8 main clusters, where Clusters 9 and 10 contain only one cell each. We did not remove any clusters. Hope this is helpful!


invoke the API callback with a mocked response upon Error

fixes #5 -- using the 'patch' release strategy.  i suggest that you clone the issue if the CPS strategy is more desirable in the long-run, since it suggests a 'major' release
the 'patch' strategy was taken to minimize impact and maximize upradeability within the SendGrid Node.js suite of packages.
given an emitted new Error('MESSAGE'), the response Object passed to the callback will include
{
  statusCode: 500,
  body: {
    message: 'MESSAGE',
    name: 'Error',
    stack: 'Error: MESSAGE\n...',
  },
  headers: {},
}

Thanks for the pull request and detailed issue report!
Could you please sign our CLA so we can get this merged? Thanks!

i have sent a signed copy of your CLA to<EMAIL_ADDRESS>i'm glad to be of help 😌

hello, @thinkingserious .  we would like to know what an ETA would be on the release of this patch.  all indications are that i have provided you with the CLA information that you requested

i believe i can only vote once 👍

👍

Thanks for the vote @firebait!

@cantremember,
Thanks again for your contribution!
When you get a moment, please fill out this form so we can send you some swag :)


Is there a better way to separate questions and answers than two parallel files?

there is jupytext which i've put a PR in for. if you could have a play make sure this works on your machine, i think that would be a big step towards solving this problem @am-bean


UIwebview change to WKwebview

Issue Type
Apple are refusing UIwebview built apps in favour of WKwebview. Apps using UIwebview will not be allowed on the appstore from April 2020.
This plugn has a refererence to UIwebview that should be changed to WKwebview ASAP.
Description
UIwebview reference found in file CDVCamera.h
line 75 @property (strong) UIView* webView;

UIView is not deprecated, UIWebView is deprecated. UIView is the base class of both UIWebView and WKWebView
https://developer.apple.com/documentation/uikit/uiview
https://developer.apple.com/documentation/uikit/uiwebview
https://developer.apple.com/documentation/webkit/wkwebview

Thank you for your prompt clarification :)


Minor Problems API cleanup

@bot-gradle test rFM

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Superseded by https://github.com/gradle/gradle/pull/27318


How is the firebase store data structure?

Just add bookmarks collection,and a doc with one url: https://xx. But give json error when access the /bookmarks

Or what should do to use the bookmarks feature?

Hey @inevity I don't have documentation up for the bookmarks feature, but yes it's all through Firebase. I built a separate app to post to Firebase, but there's no web functionality for it yet. The body for Twilio is just a URL right now :)

Thanks.


Npm call

Hi,
i've installed Slip by npm (https://www.npmjs.com/package/slipjs/tutorial). As i'm using browserify, i would like to know how to call Slip by require / import instruction. It's for just a part of a SPA, so i don't want to use a script html tag.

Try global.Slip or window.Slip.


[usbdev] Minimum, Maximum & Random length Out trans seq

This PR contains usbdev Minimum, Maximum & Random length Out transaction sequences.

@jdonjdon now after usbdev_pkt_sent seq merged so there is conflict in this branch. i will rebase it


put back autocomplete="off" warning and add it on all checkboxes and radio?

I haven't find it in v4 documentation and all input checkboxes and radio doen't have the autocomplete attribute but as mentioned in v3 :
https://getbootstrap.com/docs/3.4/javascript/#callout-buttons-ff-autocomplete firefox persist state during browsing https://bugzilla.mozilla.org/show_bug.cgi?id=654072
So maybe need to add autocomplete="off" on all radio and checkboxes in V4/V5 documentation + put back the callout?

Inclined to leave it off—haven't seen any reported issues I think on this?

there's no issues you're right but maybe need to make a choice? remove all autocomplete on checkboxes and labels, or put it quite everywhere with an explaination?
I've open that question because I've question myself about that attribute on a radio group in bootstrap and find the reply in v3.
But for sure it's a desired functionnality of Firefox so maybe remove all autocomplete attribute would make sense. Let you decide :-)

FWIW as a Firefox user I truly appreciate those persistent states.
I'd recommend to completely remove autocomplete on radio/checkbox groups.

I'm in favor of removing them all.

FWIW as a Firefox user I truly appreciate those persistent states.
I'd recommend to completely remove autocomplete on radio/checkbox groups.

However...do they cause issues? I've not tested, but could these lead to situations where a programmatically set CSS style (like active) is NOT present, but the underlying input/button has been autocompleted by Firefox silently when returning to the page?

Using the back button will sometimes persist input states on Firefox. which is not always what is wanted (specifically with dynamically generated pages, and when the user cannot control the caching headers that served the page)  I've had to use autocomplete="off" many times where I couldn't set caching headers to no-cache/must-revalidate/etc.

Also, isn't is needed specifically for button styles checkboxes/radios?

In my opinion the problem is that there is no visual indication that firefox autocompleted a radio or checkbox. The text boxes are at least marked yellow. As I recently was looking for a bug in my code that didn't exist because of this I'm also voting for the autocomplete=off or at least something mentioned in the documentation.

trying to tackle the fundamental problem here https://github.com/twbs/bootstrap/pull/28952

Closed via https://github.com/twbs/bootstrap/pull/28952


Consolodate data cleanliness sections into more succinct single section

The current sections exist in the current guide:

Clean Up Your Data
Sample Data Quality Problems
Fix Data Quality Problems in Your Existing Data

Consolidate these sections to make them more succinct with tips on identifying issues and how data must be formatted for certain fields.

Link to section about cleaning data: https://s3-us-west-2.amazonaws.com/sfdo-docs/npsp_data_import.pdf#page=11


Update PInvoke examples

Contributes to #11396, #11772

These examples can be migrated to the samples repo.

@rpetrusha

@rpetrusha Shall we move the examples to the samples repo? If so, which directory should they be in?

Yes, @NextTurn, moving them to the samples repo is an excellent idea. They could go in the snippets/standard/interop/pinvoke directory.


Bug in expected improvement computation?

Hi,
I may have found a bug, or maybe just an incoherence between the notation of the paper and the code. In the computation of the expected improvement, both in DDP and in FDDP, the value dq (or d1) is computed as
dq_ -= k_[t].dot(Quuk_[t]);

In the paper however, the reported sign is a plus (rather than a minus), so it should be:
dq_ += k_[t].dot(Quuk_[t]);

Hi @andreadelprete
The problem is a changed of notation inside the Crocoddyl code and paper (and also Tassa notation). Unfortunately, I didn't notice this different when I wrote the paper.
Concretely speaking, dV in our code has opposite sign of dV in the paper (and also Tassa notation). Therefore, the liner quadratic representation of the expected improvement has opposite sign.
Additionally, you need to note that the feed-forward k and feed-back K terms are without the negative sign.
Let me know if you agree with me.
Yours,
Carlos


Backend tests fail 1_divideupmarketpoolsharesdbpm_test, wpammarketprobabilities_test

Currently 1_divideupmarketpoolsharesdbpm_test.go and wpammarketprobabilities_test.go are failing, and it looks like the test scenarios might be the issue?
--- FAIL: TestDivideUpMarketPoolSharesDBPM (0.00s)
    --- FAIL: TestDivideUpMarketPoolSharesDBPM/Prevent_simultaneous_shares_held (0.00s)
        1_divideupmarketpoolsharesdbpm_test.go:14: Prevent simultaneous shares held: expected (3, 1), got (10, 4)
FAIL
FAIL    socialpredict/tests/dbpm        0.286s
--- FAIL: TestCalculateMarketProbabilitiesWPAM (0.00s)
    --- FAIL: TestCalculateMarketProbabilitiesWPAM/Prevent_simultaneous_shares_held (0.00s)
        wpammarketprobabilities_test.go:22: at index 1, expected probability 0.875000, got 0.615385
        wpammarketprobabilities_test.go:22: at index 2, expected probability 0.700000, got 0.571429
    --- FAIL: TestCalculateMarketProbabilitiesWPAM/infinity_avoidance (0.00s)
        wpammarketprobabilities_test.go:22: at index 1, expected probability 0.750000, got 0.545455
        wpammarketprobabilities_test.go:22: at index 3, expected probability 0.250000, got 0.454545
        wpammarketprobabilities_test.go:22: at index 5, expected probability 0.250000, got 0.454545

Try pulling from branch /golang123 and let us know the results of the tests...I updated our Go version to 1.23, it was on 1.22.5 before and using an older Go version might have been an issue
(and thanks for your work on GetSetupHandler!)

It looks like a logic error in the test cases. I set appConfig.Economics.MarketCreation.InitialMarketSubsidization to 0 and TestDivideUpMarketPoolSharesDBPM passes. I don't know the math well enough to confirm. It's probably a similar issue for the other tests. The app itself is fine.

Yes, the test cases should be updated to have the expected outcome that is
showing be the same as the actual output.
On Tue, Aug 27, 2024 at 12:00 AM j4qfrost @.***> wrote:

It looks like a logic error in the test cases. I set
appConfig.Economics.MarketCreation.InitialMarketSubsidization to 0 and
TestDivideUpMarketPoolSharesDBPM passes. I don't know the math well enough
to confirm. It's probably a similar issue for the other tests. The app
itself is fine.
—
Reply to this email directly, view it on GitHub
https://github.com/openpredictionmarkets/socialpredict/issues/283#issuecomment-2311637312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADFQCVMKTB2AWH5ZRNPLGNDZTQIXRAVCNFSM6AAAAABNE2TKICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJRGYZTOMZRGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Adding Reference to Promregator

Promregator provides discovery and facilitates scraping of Cloud-Foundry-based applications. Amongst another mechanism, it also uses the file-based service discovery approach to communicate the result of discovery to Prometheus.
After some time of incubation of the project, I (as the maintainer of Promregator) would like to ask for referencing the project in Prometheus' docs.
@brian-brazil ping to you, as requested in CONTRIBUTING.md.
Signed-off-by: Nico Schmoigl<EMAIL_ADDRESS>

Is this actually a file_sd? Your docs indicated that you rejected the (recommended) approach of having on target per target.

Is this actually a file_sd?

Valid question, but one which requires a little longer answer:
In fact, in the beginning, I thought that the target audience of Promregator would mainly be operators of individual apps run on Cloud Foundry. Support from operators of the Cloud Foundry Platform was explicitly out of scope for Promregator ("low touch" applications). As consequence, the number of apps/targets to scrape then would be low (per Promregator instance). Initial configuration effort then is one major concern in the entire process. That is why I initially started with an approach of having only one single endpoint in Promregator, which Prometheus would scrape (and includes the super-set of all metrics demanded).
Over the past months, however, it turned out that people want to use Promregator in environments with many more applications/targets involved. Promregator is run as central instance and may even feal like a service on the platform available to many applications (there is even a feature request asking to separate the selection of applications from Promregator's configuration file). Thus, it's not just about the operators of the applications anymore, Moreover, operators of the underlying Cloud Foundry platform are starting to indicate interest in supporting the configuration and scraping processes. With that the "low-configuration-cost approach" does not work out anymore due to scalability issues (most likely due to the same reasons Prometheus is recommending it like it does). That is why Promregator (since 0.2.0) supports two modes in parallel (consumers are free to choose and change back and forth):

Single Endpoint Scraping: all-in-one endpoint (initial approach)
Single Target Scraping: Discovery happens via file_sd approach exposed via endpoint; Prometheus receives a set of URLs at Promregator - one URL for each potential target to scrape. Promregator then behaves like a proxy for Prometheus (similar to what you know from PushProxy) and "only" does the enrichment of labels for each CF application instance separately without aggregating metrics from different targets.

For more details on these modes, you might be interested in having a look at https://github.com/promregator/promregator/blob/643d2c6ac0ab895ba08e11c37e18a8c85d05e090/docs/documentation.md. A discussion of the Single Target Scraping mode (with pros and cons) is also available at https://github.com/promregator/promregator/blob/643d2c6ac0ab895ba08e11c37e18a8c85d05e090/docs/singleTargetScraping.md .
Do you think that it would be helpful to mention the Single Target Scraping mode here in the line of integrations.md?

Your docs indicated that you rejected the (recommended) approach of having on target per target.

As laid out above, documentation should not reject this approach altogether anymore (in the past it did for the reason and history described above). Still being the default for a quick start, Single Endpoint Scraping is only recommended in case that you have a low number of applications. If you don't like a statement in the documentation, please feel free to give me feedback - then I should have another round over that.
Alternatively, if you don't feel that Promregator fits in the file_sd's category in general (e.g. as it also facilitates the scraping process), I am also fine with seeing Promregator being mentioned in a different section. The other sections in place today, however, were even less appropriate to me - and adding another category just for Promregator did not feel right from my perspective either. If you see that different (or you have some further ideas how to convey the message better), I am willing to listen to your arguments.

"only" does the enrichment of labels for each CF application instance separately without aggregating metrics from different targets.

It is something that should be done via relabelling on the Prometheus side.

Prometheus does not support setting the metrics path for a target directly when defining targets using the file_sd_configs approach.

__metrics_path__ should work in file_sd_configs, though a URL parameter would be safer from an escaping standpoint.

Do you think that it would be helpful to mention the Single Target Scraping mode here in the line of integrations.md?

It's not exactly a pure file_sd, I'm thinking that Other might be more appropriate.

Hi Brian, I saw your reply, but due to a personal topic, I can't have a look at it appropriately. I will come back to you, once I had proper time to look at your criticism.

With your comment, you have made a point: Single Target Scraping mode should enable users to fully take advantage of the Prometheus' way of handling labels (With Single Endpoint Scraping you can't, though). With branch disableEnrichment I am currently trying to permit a way of configuration where Prometheus would be in charge of adding the labels via relabeling.

This is something that should be done via relabelling on the Prometheus side.

I would be interested to hear your arguments on this. Besides flexibility due to configuration are there any "good reasons" for doing it like this?
Background to my question is that metric values scraped will not be unique (and thus clashes may happen), if someone forgets to relabel them in Prometheus. I feel uneasy with that.

__metrics_path__ should work in file_sd_configs, though a URL parameter would be safer from an escaping standpoint.

On the escaping: Where do you see the risk exactly?
Internally, Promregator anyhow needs to match __metrics_path__ against a list of "known targets". Moreover, the identifiers of which __metrics_path__ consists are all pure-ASCII characters only (i.e. it's a UUID plus some integer).

It's not exactly a pure file_sd, I'm thinking that Other might be more appropriate.

I will move it to the "Other" section, if you feel better with that. However, give me a little time for this - I'd like to first see that the relabeling stuff runs smoothly.

It's not exactly a pure file_sd, I'm thinking that Other might be more appropriate.
I will move it to the "Other" section, if you feel better with that. However, give me a little time for this - I'd like to first see that the relabeling stuff runs smoothly.

In the meantime I could finish that part. There will be an option called promregator.scraping.labelEnrichment which can be set to false. Then, relabeling is done in Prometheus using relabel_configs. Due to compatibility reasons the user of Promregator needs to explicitly enable the mode.
I have updated the documentation and created a note describing when and how it can be used. Feedback is appreciated.
The feature will be available starting with 0.5.0 (the version is not released, yet - I hope that this will happen soon).

A file_sd should ensure that the instance label ends up being unique, and thus that things work correctly without relabelling.

Well, it would be an easy thing for Promregator do take care of that. But unfortunately, instance may not be specified by scraping, but may only be accessed during rewriting.

On the escaping: Where do you see the risk exactly?
If there were unusual characters in there, they would not get escaped.

That cannot happen - at least as no user would be post-processing (and thus mangle) the file downloaded.
So, I think we are safe here.

Thanks!

But unfortunately, instance may not be specified by scraping, but may only be accessed during rewriting.

You can provide an instance label in file_sd. It's not usual for a generic file sd, but it makes sense here.


Feat: Support basic-auth or reuse chartmuseum as addon registry

Signed-off-by: StevenLeiZhang<EMAIL_ADDRESS>Description of your changes

Fixes #3618
I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
0.install Harbor to be private registry, or you could also use official demo site https://goharbor.io/docs/2.3.0/install-config/demo-server/

install KubeVela v1.3.1
delete default addon registry

➜  bin git:(fea-auth) ✗ ./vela addon registry delete KubeVela
Successfully delete an addon registry KubeVela

3.upload fluxcd addon and velaux addon to Harbor
please refer to https://github.com/oam-dev/kubevela/issues/3618#issuecomment-1110794857 for details
4.add Harbor to be addon registry
➜  bin git:(fea-auth) ✗ ./vela addon registry add harbor --endpoint="https://hub.harbor.com/chartrepo/oam-addon" --username=xxxxxxxx --password=xxxxxxx --type=helm
Successfully add an addon registry harbor

5.list Harbor
➜  bin git:(fea-auth) ✗ ./vela addon registry list
Name    Type    URL
harbor  helm    https://hub.harbor.com/chartrepo/oam-addon
➜  bin git:(fea-auth) ✗ ./vela addon list
NAME    REGISTRY        DESCRIPTION                     AVAILABLE-VERSIONS      STATUS
fluxcd  harbor          A kubevela addon for fluxcd     [1.1.0, 1.0.0]          disabled
velaux  harbor          A kubevela addon for velaux     [v1.3.1, v1.3.0]        disabled

6.enable velaux from Harbor
➜  bin git:(fea-auth) ✗  ./vela addon enable velaux --version=v1.3.1
I0428 18:20:30.802898   61217 apply.go:107] "creating object" name="addon-uischema-velaux" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.816572   61217 apply.go:107] "creating object" name="component-uischema-k8s-objects" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.826370   61217 apply.go:107] "creating object" name="component-uischema-task" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.838713   61217 apply.go:107] "creating object" name="component-uischema-webservice" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.850772   61217 apply.go:107] "creating object" name="trait-uischema-scaler" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.861949   61217 apply.go:107] "creating object" name="trait-uischema-storage" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.876154   61217 apply.go:107] "creating object" name="workflowstep-uischema-deploy2env" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.903761   61217 apply.go:107] "creating object" name="workflowstep-uischema-notification" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.916231   61217 apply.go:107] "creating object" name="workflowstep-uischema-webhook" resource="/v1, Kind=ConfigMap"
I0428 18:20:30.927497   61217 apply.go:107] "creating object" name="addon-secret-velaux" resource="/v1, Kind=Secret"                                                                                Addon: velaux enabled Successfully.
Please access the velaux from the following endpoints:
+---------+-----------+----------------------------+---------------------------+
| CLUSTER | COMPONENT |  REF(KIND/NAMESPACE/NAME)  |         ENDPOINT          |
+---------+-----------+----------------------------+---------------------------+
|         | velaux    | Service/vela-system/velaux | http://<IP_ADDRESS>:31788 |
+---------+-----------+----------------------------+---------------------------+

7.check velaux status, and it should be enabled
➜  bin git:(fea-auth) ✗ ./vela addon list
NAME    REGISTRY        DESCRIPTION                     AVAILABLE-VERSIONS      STATUS
fluxcd  harbor          A kubevela addon for fluxcd     [1.1.0, 1.0.0]          disabled
velaux  harbor          A kubevela addon for velaux     [v1.3.1, v1.3.0]        enabled (v1.3.1)

8.go to velaux webpage->addons to list all addons

click the card of fluxcd, go to its detail page and click button enable to enable fluxcd. Fluxcd addon should be enabled successfully.

10.check Fluxcd addon status , it should be enabled
➜  bin git:(fea-auth) ✗ ./vela addon list
NAME    REGISTRY        DESCRIPTION                     AVAILABLE-VERSIONS      STATUS
fluxcd  harbor          A kubevela addon for fluxcd     [1.1.0, 1.0.0]          enabled (1.1.0)
velaux  harbor          A kubevela addon for velaux     [v1.3.1, v1.3.0]        enabled (v1.3.1)

try to disable fluxcd. It should be disabled successfully

➜  bin git:(fea-auth) ✗ ./vela addon disable fluxcd
Successfully disable addon:fluxcd

delete Harbor addon registy, it should be deleted successfully

➜  bin git:(fea-auth) ✗ ./vela addon registry delete harbor
Successfully delete an addon registry harbor

Add KubeVela official registry back, and enable Fluxcd.It should works well.

➜  bin git:(fea-auth) ✗ ./vela addon registry add KubeVela --endpoint="https://addons.kubevela.net"  --type=helm
Successfully add an addon registry KubeVela
➜  bin git:(fea-auth) ✗
➜  bin git:(fea-auth) ✗ ./vela addon registry list
Name            Type    URL
KubeVela        helm    https://addons.kubevela.net
➜  bin git:(fea-auth) ✗  ./vela addon enable fluxcd
➜  bin git:(fea-auth) ✗ ./vela addon list | grep flux
fluxcd                          KubeVela        Extended workload to do continuous and progressive delivery                                             [1.1.0, 1.0.0]                  enabled (1.1.0)
➜  bin git:(fea-auth) ✗

Special notes for your reviewer

I  think this PR hasn't fully fixed the issue https://github.com/kubevela/kubevela/issues/3618 , it's part of it. We need to generate chart.yaml automatically for the rest part.

The code is generally LGTM. But I'm curious about how the index.yaml in harbor be generated? Do you write it manually?

I think this PR hasn't fully fixed the issue #3618 , it's part of it. We need to generate chart.yaml automatically for the rest part.

Ok, Let me add vela addon package command to make this PR to be complete implementation of #3618

The code is generally LGTM. But I'm curious about how the index.yaml in harbor be generated? Do you write it manually?

Ok, Let me add vela addon package command to make this PR to be complete implementation of #3618


Edit some minor issues that prevents running the app locally

This PR resolves some minor issues that prevents user from running the app locally.
User can now run the flask app by running py run.py from the base directory.

ah my mistake, i was trying out ways of fixing the errors, and forgot to remove redundant import on my latest push.


Fix typos in rcore.c comments

corrects all instances of:
'acording' > according
'usualy' > usually
'aprtially' > partially
'dettach' > detach

@RadsammyT thank you very much for reviewing all those typos! there are probably some more around the code!


Recreate the component if new set of options are being applied

closes #130
related #128

@dkamburov Should I tag with this PR?

In my opinion - yes.

@dkamburov I will wait for all of Maya's changes to make it in, and then I will tag.


Improved gradle setup. Removed unnecessary artifact creation in root …

…project and fixed jacoco report. Updated gradle wrapper to latest 4.2.1.

Coverage remained the same at 38.903% when pulling 81c1cfc14059386c9c33ac05dadc120f0722bf78 on Hillkorn:dropwizard-service into b0f33a6ff2dcd101612203fffe9471ed788fdb53 on kermitt2:dropwizard-service.

I merged the best parts (excluded local maven libs since it's too verbose, more places + didn't work for me for some reason)
Also lucene was shaded only in the onejar, but for dependent project we needed it to be shaded in the main jar. I added it too


Tip of LambdaNet + GHC 7.10.1 does not work

When trying to use LambdaNet tip together with GHC 7.10.1 (hmatrix tip supports GHC 7.10) first AI.DemoLayer complains about missing "RandomTransform" and "RandomGen" imports.
Subsequently AI.Network fails with:
AI/Network.hs:14:1:
    The associated type ‘Parameters’
    mentions none of the type or kind variables of the class ‘Network a’
    In the class declaration for ‘Network’

Alright, so the most recent commit should fix DemoLayer -- which is basically a scratchpad -- and I'm updating to GHC 7.10 to attempt to recreate the Network issue.

The same with GHC 7.10.2

Should work with the newest merge.


Open Node while using lazy loading

Hello,
I am using lazy loading to display my jstree since the tree can be really huge. Now i wanted to implement the ability to select the recent 10 nodes the user expanded. I am saving the nodes in an array and can select them, so everything looks fine.
Unfortunately since I am using lazy loading i only can open nodes which are already displayed.
Is there anyway If the user chooses one of the last 10 nodes from the select box, to navigate the tree to this node and load all parent nodes on the fly ?
Kind Regards,
David

No, jstree has no idea that a node exists if its parent is not loaded. That is like asking your OS to search a flash drive that you have not yet connected.
Maybe the state plugin will work for you?
Also - the _load_nodes function accepts an array of IDs to load (and the IDs may not be currently present), so if you know the parent IDs of the node you want to select you can easily call:
$('#tree').jstree(true)._load_node(['parentID1', 'parentID2', 'parentID3'], function () {
  this.select_node("CHILD_ID_OF_PARENTID3");
});

Best regards,
Ivan

I tried that but it is not working for me. Here is my setup:
For example i have a node with id "/branches" and one which is a child with id "/branches/Bug".
So I am trying your code like:
$('#tree').jstree(true)._load_node(['/branches'], function () { //Also tried with # too
  this.select_node("/branches/Bug");
});

I receive an error like this: https://localhost:8443/ETLTool/serverInterface.jsp?childsForId=undefined 500 (Internal Server Error)
This is how i initialize my jstree :
	$("#treeview").jstree({
		"plugins": ["types", "wholerow","state"],
		"types": {
			"folder": {
				"icon": "./images/folder.png"
			},
			"file": {
				"icon": "./images/file.png"
			}
		},

		"core": {
			"data": {
				"url": function(node) {

					if($("#treeDatasourceSelect").val() == "SVN") {
						if (node.id === "#") {
							return showOnlyScripts ?  "serverInterface.jsp?getRootScripts=YES"
									: "serverInterface.jsp?getRoot=YES";

						} else {
							console.log(node);
							return showOnlyScripts ? "serverInterface.jsp?scriptsForId=" + node.id
									:"serverInterface.jsp?childsForId=" + node.id;

						}
					}
					else {
						if (node.id === "#") {
							return"serverInterface.jsp?explorerRoot=YES";

						} else {
							return "serverInterface.jsp?explorerChildsForId=" + node.id;

						}
					}
				},
				"data": function(node) {

						return {
							"id": node.id
						}

				}
			}
		}
	});


myarts app | form_valid implementation suggestion

In the myarts/views.py module we can implement 'form_valid', in other manner, without to do a double 'save()' calls.
Implementation suggestion : https://github.com/LiorA1/Django/blob/main/dj4e-samples/myarts/owner.py#L23

Thanks - this is a lot prettier and better from an OO perspective.  I will make this change when I can update the course materials.


[new model] our model achieves an 0.473 Spearman's Correlation on the DMS substitution benchmark

Hi ProteinGym Team,
Thank you for updating the ProteinGym Benchmark Dataset. We have evaluated our model (ProtSSN, on https://github.com/tyang816/ProtSSN) on the updated ProteinGym for DMS substitution scoring. Our model obtained an overall Spearman's correlation of 0.473 with the ensemble version. We kindly request you to double-check this result and, if confirmed accurate, update our performance on the leaderboard.
For your convenience, we have provided the mutant-specific scores by email. Please let us know if more information or documents are required from us. Thank you!

Hi @bzho3923,
Thank you for raising the issue and congrats again on ProtSSN!
Following up on our email exchange, I am responding here with the same information for the benefit of other research teams interested in contributing new baselines (just updated the instructions on README accordingly). If you would like new baselines to be included in ProteinGym (ie., website, performance files, detailed scoring files), please submit a PR to our repo with the following two elements:

A new subfolder under proteingym/baselines named with your new model name. This subfolder should include a python scoring script similar to this script, as well as all code dependencies required for the scoring script to run properly
An example bash script (e.g., under scripts/scoring_DMS_zero_shot) with all relevant hyperparameters for scoring, similar to this script

Regarding model performance on DMS benchmarks, please ensure you are using the performance scripts we provide for consistency (e.g., for zero-shot DMS benchmarks). These scripts correct for various biases (e.g., number of assays per protein family and function groupings). As a result, the aggregated performance post-correction is not the same as the arithmetic average across assays. For instance, the aggregate Spearman performance of TranceptEVE on the zero-shot DMS substitution benchmark is 0.474 pre-correction (ie., arithmetic average across all 217 assays), but 0.457 post-correction (reported performance in paper / website).
Best,
Pascal

Hi @bzho3923 -- I have just merged in the PR with the ProtSSN dependencies added in. We will be refreshing our zero-shot substitution benchmark with ProtSSN scores shortly. Congrats again on the great work!

Hi Pascal,
Thank you very much for the update. Looking forward to seeing the new
leaderboard. :D
Best,
Bingxin
On Wed, 6 Mar 2024 at 12:27, Pascal Notin @.***> wrote:

Hi @bzho3923 https://github.com/bzho3923 -- I have just merged in the
PR with the ProtSSN dependencies added in. We will be refreshing our
zero-shot substitution benchmark with ProtSSN scores shortly. Congrats
again on the great work!
—
Reply to this email directly, view it on GitHub
https://github.com/OATML-Markslab/ProteinGym/issues/11#issuecomment-1980067002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ2SYD7E6OMDWTMGCWDKKZLYW2LJ5AVCNFSM6AAAAABBATX7ZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBQGA3DOMBQGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix a typo in the comment for blockRenderMap

FWIW, this doesn't appear to be used in any generated documentation.

@facebook-github-bot shipit

#387 adds docs for this.

Thanks for importing. If you are an FB employee go to Phabricator to review.


[release-1.5] 🐛 Fix race condition getting IPAddress for Metal3Data

This is a manual cherry-pick of https://github.com/metal3-io/cluster-api-provider-metal3/pull/1361
It is manual since the original PR didn't have the DCO...

/approve


Make modification to allow support for Heterogeneous (Periodic) Hidden Markov Models

As we discussed, that would be nice (when you have time) to make the changes to allow support of time dependent HMMs (transition matrices + emission distribution) is possible.
Then, check that these changes do not affect the performances of homogeneous HMMs.

Done on the main branch, awaiting feedback before tagging release


Update uuid to 1.0

Updates uuid dependency to the latest released version (1.0) . This is a breaking change if you export the dependency.
I don't think any more changes are required

Hello, when are you releasing the next minor version? Since this is a breaking change and I have another dependency (sqlx) which uses uuid < 1.0 I have to lock the rkyv version to =0.7.37 Due to my case I think it would be better if we had jumped from 0.7.38 to 0.8. Thanks

I'll add a feature specifically for uuid <1.0 with the next minor release so you can use uuid_0.8 until sqlx updates.

Alright, that's a good plan. Thank you


fix(readme): fix labels docs

:tada: This PR is included in version 1.87.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SPARK-48399][SQL] Teradata: ByteType should map to BYTEINT instead of BYTE(binary)

What changes were proposed in this pull request?

According to the doc of Teradata and Teradata jdbc, BYTE represents binary type in Teradata, while BYTEINT is used for tinyint.

https://docs.teradata.com/r/Enterprise_IntelliFlex_VMware/SQL-Data-Types-and-Literals/Numeric-Data-Types/BYTEINT-Data-Type
https://teradata-docs.s3.amazonaws.com/doc/connectivity/jdbc/reference/current/frameset.html

Why are the changes needed?

Bugfix
Does this PR introduce any user-facing change?

Yes, ByteType used to be stored as binary type in Teradata, now it has become BYTEINT.
(The use-case seems rare, the migration guide or legacy config are pending reviewer's comments)
How was this patch tested?

new tests
Was this patch authored or co-authored using generative AI tooling?

no

cc @dongjoon-hyun @cloud-fan @LuciferYang thanks

Thank you @dongjoon-hyun, We can add a follow-up on the legacy configuration if someone considers it necessary.
Merged to master

late LGTM


fix(3187136947): align Account menu to Figma design

Description of the Changes
Align the Account menu to the Figma design
Solves #3187136947

Checklist

[ ] Manually tests of the main Application flows are done and passed.
[ ] New unit / functional tests have been added (whenever applicable).
[ ] Docs have been updated (whenever relevant).
[ ] PR title is follow the Conventional Commits convention: <type>[optional scope]: <description>, e.g: fix: prevent racing of requests

This change is

NVM, saw now that you use default values.

:tada: This PR is included in version 1.9.0-dev.26 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[ problem ] S3 Rag docs mix in GitHub loader info

We need to update / rewrite https://www.eidolonai.com/docs/recipes/s3-rag. Links are broken at the top, and further down the page it refers to the GitHub loader and interacting with the Repo Expert.

Requirements to run the docker script: make and poetry.
KDE Wallet Service pop-up: won't cancel, but can ignore. Should go away once installation is complete. https://pypi.org/project/keyring/
sudo keyring disable


Adding a tool for building bot.

I have added Haven OnDemad APIs to the list. I personally found them very cool.

Thanks :+1: I have squashed and merged the commits.


How to handle cudnn version conflict?

Description
2023-03-27 03:15:47.851349: E external/org_tensorflow/tensorflow/compiler/xla/stream_executor/cuda/cuda_dnn.cc:417] Loaded runtime CuDNN library: 8.0.5 but source was compiled with: 8.6.0.  CuDNN library needs to have matching major version and equal or higher minor version. If using a binary install, upgrade your CuDNN library.  If building from sources, make sure the library loaded at runtime is compatible with the version specified during compile configuration.
2023-03-27 03:15:47.854454: E external/org_tensorflow/tensorflow/compiler/xla/status_macros.cc:54] INTERNAL: RET_CHECK failure (external/org_tensorflow/tensorflow/compiler/xla/service/gpu/gpu_compiler.cc:641) dnn != nullptr
What jax/jaxlib version are you using?
jaxlib  0.4.6+cuda11.cudnn86
Which accelerator(s) are you using?
GPU
Additional system info
Python3.8 Ubuntu18.04
NVIDIA GPU info
Mon Mar 27 03:17:33 2023
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 530.30.02              Driver Version: 530.30.02    CUDA Version: 12.1     |
|-----------------------------------------+----------------------+----------------------+

I don't know why but installing tensorflow-cpu==2.4.0 is enough to solve the problem.

I couldn't use latest version.. How to resolve it?

https://github.com/google/jax/issues/14480
0.4.2 doesn't have any error.

For whatever reason, JAX is picking up an older cudnn that you have installed. I suggest finding the older version on your system and removing it.
The next release of jax and jaxlib will allow installation of cudnn via pip wheels, which may help.
Hope that helps!

Your options are:
a) use a sufficiently new CuDNN, at least as new as specified by the jaxlib release, or
b) build jaxlib from source
However, note that jax 0.4.7 now supports pip wheel installation of CUDA and cudnn. Try
# Installs CUDA and CUDNN using pip, needs *nothing* but the NVIDIA driver
pip install --upgrade "jax[cuda11_pip]" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html

It is aaaaaaaaaawesome. Works great! Thank you for your kindness response. @hawkinsp


Rust : Adding README for topics

Description
Description
Add README , giving a brief of what Rust is and how it works .
Also the topics to be covered
Domain
Rust
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign documentation


community-operators assisted-service-operator (0.0.2)

Since this operator currently only supports basic install, ie. there is
no upgrade path from one version to the next, this PR just updates the
operator package to use the operator/operand relationship with the
operand being managed via an AgentServiceConfig.
Thanks submitting your Operator. Please check below list before you create your Pull Request.
Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[ ] ~Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?~ This operator is basic install only at this time.
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] ~Have you tested an update to your Operator when deployed via OLM?~ This operator is basic install only at this time.
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

this PR now requires #3605 before it'll be happy with the ci.yaml.

/hold cancel

/hold cancel


TableView should render an endash with aria-label for cells with missing values

🙋 Feature Request
Spectrum recommends: "When there are gaps in the data, use an en dash (–) to represent null or not applicable (N/A) values."
https://spectrum.adobe.com/page/table/#Use-an-en-dash-(–)-for-missing-values
It would be great if TableView code handled this internally so implementors don't have to. @reidbarber had the good idea to wrap the endash in <span aria-label="No value">, but I could see implementors either choosing different strings or not localizing it. I've seen many cases where implementors either don't know about the endash guidance or don't consider the aria-label (so it doesn't announce "endash").

🤔 Expected Behavior
If a TableView cell is empty, then a Cell is automatically rendered with an endash and a localized aria-label

😯 Current Behavior
Implementors have to manually insert the endash and a localized aria-label

💁 Possible Solution
Hopefully Cell code can handle this

🔦 Context
A Quarry contributor opened a PR with manually inserted endashes for empty fields, then @paulkenney recommended we figure out how AT should announce it

💻 Examples
From an implementor's PR:
// Use an en dash (–) for missing values.
// https://spectrum.adobe.com/page/table/#Use-an-en-dash-(%E2%80%93)-for-missing-values
const EN_DASH = '–';
...
function cellMapper(item, columnKey) {
  switch (columnKey) {
    ...
    default:
      return item[columnKey] || EN_DASH;
  }
}

🧢 Your Company/Team
Adobe/Quarry

🎁 Tracking Ticket (optional)

This seems reasonable to me since Spectrum mentions this, and I can't think of any downsides. Maybe these cells can use a default string for an aria-label like "No value", unless one is provided by the user.


Tile boundary artifacts when using web_optimized

Seen using rio_cogeo 1.1.8
When using the web_optimized=True there is a visible tile boundary seen, especially at higher zoom levels.
Here is web_optimized=True

Here is web_optimized=False

Imagery is SAR, from SpaceNet 6

👋 @matthewhanson using https://github.com/cogeotiff/rio-cogeo/tree/v2 I could not reproduce the artefact. I'm seeing some resampling artefacts but not at tile intersection. Could you share the exact file you are using and the command line you used ?

@matthewhanson ping 😄


SAMZA-2758: Upgrade vulnerable versions jetty and jackson

Issues: Upgrade dependencies for security fixes and enhancements for jetty and jackson
In current dependencies there are security vulnerabilities
Jackson: https://security.snyk.io/package/maven/com.fasterxml.jackson.core:jackson-databind/2.12.2
Jetty: https://security.snyk.io/package/maven/org.eclipse.jetty:jetty-server/9.4.38.v20210224
Upgrade Jackson to version: 2.13.3
Upgrade Jetty to 9.4.48.v20220622
Changes: Describe major changes, listing each separately.
Upgraded dependency management versions
Tests: Existing Tests
API Changes: None
Upgrade Instructions: None
Usage Instructions: None

@Sanil15 @mynameborat can you please take a look

@perkss Have you tested with deploying jobs on top of the existing tests?

@mynameborat yeah tested against local jobs that I run as samples


Can't play in mpv player for movie streaming

Go to "mov-cli -e"
Change the mpv to vlc
Ctrl + O , enter and Ctrl X
Download vlc player
mov-cli -s films (name of the movie)

Send a full log please.

Do you mean "can't play in vlc player"?

Have you figured out the issue?

Stale


Allow theme specific config for editor.rulers

It is already possible to have language specific options for editor.rulers:
  "[git-commit]": {
    "editor.rulers": [
      {
        "column": 72,
        "color": "#F00"
      },
    ]
  },

It would be great to allow theme specific settings like this:
  "[Default Dark Modern]": {
    "editor.rulers": [
      {
        "column": 80,
        "color": "#540"
      },
      {
        "column": 100,
        "color": "#530"
      },
      {
        "column": 120,
        "color": "#520"
      }
    ]
  },
  "[Default Light Modern]": {
    "editor.rulers": [
      {
        "column": 80,
        "color": "#FED"
      },
      {
        "column": 100,
        "color": "#FDC"
      },
      {
        "column": 120,
        "color": "#FCB"
      }
    ]
  },

@hediet maybe we should we have rulerNColor theme keys?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Fix the ‘jump’ of the input field when editing a series title

Reassigning to @Coenego for review

Coverage remained the same at 100.0% when pulling ce739a15aa93e261ee5782f2f8a56ac33ef8b56f on bp323:issue-179 into 38e7bb03c9e48b623fedec7993cb0fa79765db84 on CUL-DigitalServices:master.


Broken Icons in remote container

Version: 1.53.0-insider
Commit: ed72c64b87ede96fb51bc16870f9d2ecbf4fcc57
Date: 2021-01-22T17:18:37.386Z
Electron: 11.2.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-40-generic
Steps to Reproduce:

Install vscode-insiders
Install Python Test Explorer for Visual Studio Code (https://marketplace.visualstudio.com/items?itemName=LittleFoxTeam.vscode-python-test-adapter)
Install ms-vscode-remote.remote-containers extension
Create a new workspace, an empty python project, and a pytest file.
Open the test explorer UI. The icons are missing.

Does this issue occur when all extensions are disabled?: N/a (I think this might be related to the ms-vscode-remote.remote-containers extension.)
Additional Info:
I just got an update to my vscode-insiders IDE, and now the icons in the test explorer UI seem to be hidden or missing. I restarted my computer, uninstalled and reinstalled the extension, but they're still missing. I downloaded the standard version of vscode (1.52.1), and I can see the icons, so it's possible that there is a breaking change coming.
Here is the output of my code-insiders > help > about menu:
Version: 1.53.0-insider
Commit: ed72c64b87ede96fb51bc16870f9d2ecbf4fcc57
Date: 2021-01-22T17:18:37.386Z
Electron: 11.2.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-40-generic
2021-01-23_21-32

I pulled up the developer tools and found one of the icons' CSS
background-image: url("vscode-remote-resource://<IP_ADDRESS>:45595/vscode-remote-resource?path=%2Froot%2F.vscode-server-insiders%2Fextensions%2Fhbenl.vscode-test-explorer-2.19.5%2Ficons%2Fpassed.svg&tkn=b0ced289-13e9-4798-9071-2d43f43cf8eb")
Now I am not so sure if this is an extension issue or an issue with the vscode remote server. I'm using a dev container, and this extension is running inside the container. I see the request is blocked by the client, which explains the missing icon.

/duplicate of #114743

/duplicate of #114743


build(package): upgrade devDependencies

What is the motivation for this pull request?
build(package): upgrade devDependencies
 @commitlint/cli                   ^17.0.3  →   ^17.3.0
 @commitlint/config-conventional   ^17.0.3  →   ^17.3.0
 @types/jest                       ^29.0.0  →   ^29.2.3
 @types/node                       ^18.0.1  →  ^18.11.9
 @typescript-eslint/eslint-plugin  ^5.30.5  →   ^5.44.0
 @typescript-eslint/parser         ^5.30.5  →   ^5.44.0
 eslint                            ^8.19.0  →   ^8.28.0
 husky                              ^8.0.1  →    ^8.0.2
 jest                              ^28.1.2  →   ^29.3.1
 lint-staged                       ^13.0.3  →   ^13.0.4
 prettier                           ^2.7.1  →    ^2.8.0
 ts-jest                           ^28.0.5  →   ^29.0.3
 typescript                         ^4.7.4  →    ^4.9.3

Checklist:

[x] Conventional Commits

Codecov Report

Merging #69 (f26b364) into master (1d2c8a5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #69   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           63        63
  Lines          314       314
  Branches        12        12
=========================================
  Hits           314       314

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


Can't run Unsloth locally in windows

It say's no module named Triton

Oh ye sadly Windows is an issue - I'm working on something to make Triton an optional dependency

Is there any way around?

Oh ye sadly Windows is an issue - I'm working on something to make Triton an optional dependency

Is there any way around?

Actually there is - in theory one can install Triton via windows - maybe try https://github.com/unslothai/unsloth/issues/210

Actually there is - in theory one can install Triton via windows - maybe try https://github.com/unslothai/unsloth/issues/210

The links he provided doesn't work

I maked a docker image, you can use it directly. follow the guide below.
https://github.com/Jiar/jupyter4unsloth


primer.md Code Fix

primer.md Code Fix

@Afanyiyu : Thanks for your contribution! The author(s) have been notified to review your proposed change.


New-NavContainer Stuck at Loading Symbols for TestLibraries image BC 14.10

I am executing the following command:
New-NavContainer -accept_eula -containerName test -imageName 'mcr.microsoft.com/businesscentral/onprem:14.10.40471.0-de'  -doNotExportObjectsToText -TimeZoneId (Get-TimeZone).Id -shortcuts "Desktop" -alwaysPull -updateHosts -useBestContainerOS -accept_outdated -includeCSide -enableSymbolLoading -includeTestToolkit -includeTestLibrariesOnly

It runs forever (>1h whereas container creation usually only takes 5min) with the following output:
NavContainerHelper is version <IP_ADDRESS>
NavContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - ltsc2019
Docker Client Version is 19.03.5
Docker Server Version is 19.03.5
Pulling image mcr.microsoft.com/businesscentral/onprem:14.10.40471.0-de-ltsc2019
14.10.40471.0-de-ltsc2019: Pulling from businesscentral/onprem
Digest: sha256:b248e37d07443a19b04e43b2dcf5757de817bb68d8ca31faae89dfb4db3e30b1
Using image mcr.microsoft.com/businesscentral/onprem:14.10.40471.0-de-ltsc2019
Creating Container test
Version: 14.10.40471.0-de
Style: onprem
Platform: 14.0.40464.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.973 (ltsc2019)
Host OS Version: 10.0.17763.973 (ltsc2019)
Using locale de-DE
Using process isolation
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\NavContainerHelper\Extensions\test\my:
- AdditionalOutput.ps1
- AdditionalSetup.ps1
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container test from image mcr.microsoft.com/businesscentral/onprem:14.10.40471.0-de-ltsc2019
e13b0fb69ee978688a8c8df35f63cd07f65cfdfbab939943a3caa9213f57f1ba
Waiting for container test to be ready

WARNING: DNS resolution not working from within the container.
Using Windows Authentication
Starting Local SQL Server
Starting Internet Information Server
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Creating http download site
Creating Windows user mmenninghaus
Setting SA Password and enabling SA
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : test
Container Dns Name  : test
Web Client          : http://test/NAV/
Dev. Server         : http://test
Dev. ServerInstance : NAV
Setting test to <IP_ADDRESS> in host hosts file

Files:
http://test:8080/al-3.0.193882.vsix

Initialization took 130 seconds
Ready for connections!
Set TimeZone in Container to W. Europe Standard Time
Reading CustomSettings.config from test
Creating Desktop Shortcuts for test
Importing Objects from C:\TestToolKit\CALTestLibraries.DE.fob (container path)
Generating Symbols while importing

All of my colleagues are experiencing the same issue.
It does not get stuck if I do not set the Symbol Loading flag. The 14.10 containers get also stuck when setting up without Symbol Loading or TestLibraries but trying to export C/AL files.
Thank you for any help on this issue!

is there anything in the event log? You can use Get-NavContainerEventLog to fetch the event log (remember to delete it again from the event viewer)

See below, nothing that I am aware of.
{
    "container.logs":  [
                           "Initializing...",
                           "Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)",
                           "Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file",
                           "Starting Container",
                           "Hostname is test",
                           "PublicDnsName is test",
                           "Using Windows Authentication",
                           "Starting Local SQL Server",
                           "Starting Internet Information Server",
                           "Modifying Service Tier Config File with Instance Specific Settings",
                           "Starting Service Tier",
                           "Registering event sources",
                           "Creating DotNetCore Web Server Instance",
                           "Creating http download site",
                           "Creating Windows user mmenninghaus",
                           "Setting SA Password and enabling SA",
                           "Creating SUPER user",
                           "Container IP Address: <IP_ADDRESS>",
                           "Container Hostname  : test",
                           "Container Dns Name  : test",
                           "Web Client          : http://test/NAV/",
                           "Dev. Server         : http://test",
                           "Dev. ServerInstance : NAV",
                           "Setting test to <IP_ADDRESS> in host hosts file",
                           "",
                           "Files:",
                           "http://test:8080/al-3.0.193882.vsix",
                           "",
                           "Initialization took 128 seconds",
                           "Ready for connections!"
                        Zeitangaben in Millisek.:",
                           "    Minimum = 0ms, Maximum = 0ms, Mittelwert = 0ms"
                       ],
    "container.env":  [
                          "username=mmenninghaus",
                          "ExitOnError=N",
                          "useSSL=N",
                          "accept_eula=Y",
                          "securePassword=\u003cspecified\u003e",
                          "passwordKeyFile=c:\\run\\my\\aes.key",
                          "removePasswordKeyFile=Y",
                          "accept_outdated=Y",
                          "licenseFile=\u003cspecified\u003e",
                          "auth=Windows",
                          "locale=de-DE",
                          "databaseServer=",
                          "databaseInstance=",
                          "enableApiServices=Y",
                          "enableSymbolLoading=Y",
                          "COMPLUS_NGenProtectedProcess_FeatureEnabled=0",
                          "DatabaseName=Cronusde",
                          "NAVDVDURL=\u003cspecified\u003e",
                          "VSIXURL=\u003cspecified\u003e",
                          "IsBcSandbox=N",
                          "CountryCode=de",
                          "COUNTRYURL=\u003cspecified\u003e"
                      ]
}

Sorry, but that isn't the event log

I can repro this on my machine.

Something must have changed, if I do this:
if (-not $windowscredentials) {
    $windowscredential = Get-Credential -UserName $env:USERNAME -Message "Enter your Windows Password"
}
New-NavContainer `
    -accept_eula `
    -accept_outdated `
    -containerName test `
    -imageName 'mcr.microsoft.com/businesscentral/onprem:1904-cu8-de-ltsc2019' `
    -doNotExportObjectsToText `
    -shortcuts "Desktop" `
    -updateHosts `
    -includeCSide `
    -enableSymbolLoading `
    -includeTestToolkit `
    -includeTestLibrariesOnly `
    -Credential $windowscredential

It works fine.
If I replace cu8 with cu9 - it hangs.

The bug is in Microsoft.Dynamics.Nav.Ide.psm1 in 14.10, and it actually causes Import-NavApplicationObject, Export-NavApplicationObject, Compile-NavApplicationObject and more standard cmdlets to fail.
Reason is a new parameter added to RunNavIdeCommand, which doesn't get a value and subsequently all functions calling RunNavIdeCommand will fail/hang.
I have filed a bug to get this fixed for 14.11. In the meantime, I have uploaded a patch for Microsoft.Dynamics.Nav.Ide.psm1 here:
https://bcdocker.blob.core.windows.net/public/14.10.40471.0-Patch-Microsoft.Dynamics.Nav.Ide.psm1
And I have added code to New-NavContainer, which downloads and patches the container if 14.10 is being instantiated, you should see this when you have the right version:

Will ship in the next NavContainerHelper version <IP_ADDRESS> (today) and with that all pipelines etc. should work as before...

This commit fixes the issue: https://github.com/microsoft/navcontainerhelper/commit/ff02d655ea49021cf10affedc180e57b8d42ad51

Shipped in <IP_ADDRESS>

Described here: https://freddysblog.com/2020/02/22/small-issue-in-bc-2019-wave-1-cu9/

We had the same sort of behavior with Import-ObjectsToNavContainer.ps1 since last thursday. We are not using CU9 yet, but CU5 for reasons. After long investigation we have noticed that on the host machines the Windows Updates had run in the night of wednesday and thursday.
Result was that Import-ObjectsToNavContainer was just hanging.
After reverting Security Update KB4532691 the issue was fixed. Where can we address this ?

@mboon75 - you are running into the other problem we had over the last two weeks: https://freddysblog.com/2020/02/14/hyperv-isolation-to-the-rescue/ - where the February update caused things to fail.
There is no better solution to this than what's mentioned in the blog post above.

ah, sorry @freddydk ! I should read more of your blogs before asking. Thanks a lot.


Update DynamicResponseModel.php

In PHP 8.0.13+ and 8.1 error:
Fatal error: Declaration of Postmark\Models\DynamicResponseModel::offsetGet($offset) must be compatible with Postmark\Models\CaseInsensitiveArray::offsetGet(mixed $offset): mixed in vendor/wildbit/postmark-php/src/Postmark/Models/DynamicResponseModel.php on line 56

Please see my feedback in https://github.com/wildbit/postmark-php/pull/89#issuecomment-987872368 , this is same/similar issue.

Addressing via https://github.com/wildbit/postmark-php/pull/95

Thanks for the input. I am closing the ticket due to age. If this is still a needed change - please let us know.


Unexpected pixel jitter when using 2d stretch mode the keep stretch aspect

Godot version:
3.2.3.stable.official
OS/device including version:
Windows 10 20H2, GTX 1080ti, R7 3800X
Issue description:
The bottom and right side of the Godot application jitter when being resized if you enable 2D Stretch mode with Aspect locking.
This only occurs on the bottom and right side. Left and top work as expected without jitter.
Steps to reproduce:
Under DIsplay -> Window:
Size
Width: 1280
Height: 720
Stretch
Mode: 2d
Aspect: keep
(This enables letterboxing like Celeste on ultrawide monitors)
Under Rendering -> Quality:
2d
Use Pixel Snap
Drag Window around and look at bottom / right side letter boxing.
Minimal reproduction project:
Demo.zip
https://user-images.githubusercontent.com/34455502/110059334-66ba6980-7d3a-11eb-8d94-fa748577d81c.mp4
https://user-images.githubusercontent.com/34455502/110059314-5e622e80-7d3a-11eb-9cca-0dc53a3d8b6e.mp4

i believe the issue starts at line 615
in window.cpp
https://github.com/godotengine/godot/blob/a6ec6521fae92a245c9d395efda01a120a44324d/scene/main/window.cpp

@Skullfurious window.cpp only exists in the master branch, not 3.2.
Also, if you want to avoid any kind of pixel jitter, you need to enforce integer scaling using a script: https://github.com/godotengine/godot-proposals/issues/1666

The jitter doesnt come from the actual canvas resolution changing. Not demonstrated in the videos is the top and left sides not jittering. None of the pixels within the bounds of the viewport jitter.
The only jitter is when the letterboxing is shifting on the bottom and right side of the window.
https://user-images.githubusercontent.com/34455502/110198486-3ef5ff00-7e29-11eb-9d6f-9e88a52971a6.mp4


additional parameters (scopes) as settings constant

I've read the example over the docs of how we can override the get_additional_parameters function. Since most of the backends are now running version 2.x i think we can provide a less "intrusive"  way to pass extra scopes.
The proposal is to change the default get_additional_parameters from OAuthRedirect to
    def get_additional_parameters(self, provider):
        "Return additional redirect parameters for this provider."
        scopes = {}
        try:
            delimiter = settings.ALLACCESS[provider.name]['scopes']['delimiter']
            scopes = {
                'scope': delimiter.join(settings.ALLACCESS[provider.name]['scopes']['values'])
            }
        except AttributeError:
            print('no settings for {}'.format(provider.name))
        return scopes

so we can just append something like this over the settings:
ALLACCESS = {
    'facebook': {
        'scopes': {
            'delimiter': ',',
            'values': ['email']
        }
    },
    'google': {
        'scopes': {
            'delimiter': ' ',
            'values': [
                'https://www.googleapis.com/auth/userinfo.profile',
                'https://www.googleapis.com/auth/userinfo.email'
            ]
        }
    }
}

I think most of the devs will need extra scopes always (the email address is always out of the default one) so this prevent to build a custom  OAuthRedirect object just because we need more scopes.

I think improving the way scope or other parameters are added would be great but I don't like the idea of doing this via settings. It would be more fitting to the approach of this project to either allow this to be passed in the url conf such as
...
url(r'^login/facebook/$', OAuthRedirect.as_view(params={'scopes': 'email'})),
url(r'^login/(?P<provider>(\w|-)+)/$', OAuthRedirect.as_view(), name='allaccess-login'),
...

or by configuring on the Provider record. I would lean toward the former since I don't think there is a great way to store this mapping in the database.

I really like setting's variables :) but yeah the URLs params seems more appropriated to the project. I will cast the modifications to this on my fork. Thanks again.


[BUG] github mermaid

Describe the Bug
Since mermaid get updated on github this one breaks all templates made since now..

Github use mermaid version: v11.3.0
Gitlab use mermaid version: v10.7.0
Gitlab no issues only on github.


Is the audio data available anywhere?

Hi there,
I was looking into DeepChroma and wanted to see if the dataset from http://isophonics.net/ for Beatles, Queen, Wweieck, etc is still available or not.
Thank you.

Sorry, seems like I did not get a notification for this.
The audio data is unfortunately copyrighted, and cannot be shared freely.
Sorry about that.


Handle color theme dependent on excel theme

How properties of the current excel theme can be accessed:
https://docs.microsoft.com/en-us/javascript/api/office/office.officetheme?view=word-js-preview

This should be at least acceptable now!


Table Storage Type not exported/imported

When a table is defined with storage type "Column Store" in source system, the attribute is not exported to the git repository. When this table is imported, the table is handled as "Row Store" table because DD09L-ROWORCOLST field is initial.
Error happens in serialization method of ZCL_ABAPGIT_OBJECT_TABL:
    ASSIGN COMPONENT 'ROWORCOLST' OF STRUCTURE ls_dd09l TO <lg_roworcolst>.
    IF sy-subrc = 0 AND <lg_roworcolst> = 'C'.
      CLEAR <lg_roworcolst>. "To avoid diff errors. This field doesn't exists in all releases
    ENDIF.

I would recommend to keep the field ROWORCOLST as it is and live with the diff. errors in old releases...

as far as I remember, importing without the field imports it as 'C' by default?

Sorry, I have not checked in detail. The affected table was exported as RowStore table and by mistake I thought it was a ColumnStore table that has been exported with wrong attributes. And yes, 'C' is the default if ROWORCOLST is initial. I'll close this issue!


Dynamic custom theming

This should not affect existing ui at all, did some basic testing and components changed from lock file seems to be not affected.

Testing on Monday 4/25


IP Addresses in Screenshot Don't Match Document

The first diagram, and the document say that the external address space for the NAT rules should be 100.0.X.0/24 but the first screenshot under "Part 2: NAT Rules" shows 10.100.X.0/24.

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

ID: 1130e194-0e0f-d7e9-1f5f-2101a173f9ff
Version Independent ID: 1f8e045c-4c83-f414-08aa-d68352e6abec
Content: Configure NAT on Azure VPN Gateway - Azure VPN Gateway
Content Source: articles/vpn-gateway/nat-howto.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang

Hello @matthansen0
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

#reassign:deeharris-23

#assign:cherylmc

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20221009154223734 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20221009154223734 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 42.351%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
29389

Relevant Lines:
64565

💛  - Coveralls


CI: Finish setting up GitHub actions

Starting by adding a test so pytest doesn't fail due to finding nothing.

Codecov Report

Merging #9 (f0d70ed) into main (8f3441b) will increase coverage by 1.78%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   85.71%   87.50%   +1.78%
==========================================
  Files           1        2       +1
  Lines          14       16       +2
==========================================
+ Hits           12       14       +2
  Misses          2        2

Impacted Files
Coverage Δ

nireports/tests/test_nireports.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

What do we think about only running oldest and newest Python on Windows/OSX?


[PUI] Add licenses texts to PUI

This PR will:

[x] Add license extraction steps to frontend and backend builds
[x] Adds an API that exposes them
[x] Adds a licenses screen to show frontend and backend licenses (closing PLH component tin nav)

@matmair I believe that this requires some further consideration regarding generation of the license file on systems which do not have the node packages installed.
For example in a docker based setup, this would error out as the frontend packages are discarded from the final build image. Maybe we need to update the docker image to pre-compile this file and COPY it across?
Additionally, this means that the invoke install step fails unless the required node packages are on the system. As this PR introduces a new package requirement, the initial testing on my system failed.
git checkout -b matmair-pui-licenses-text master
git pull git@github.com:matmair/InvenTree.git pui-licenses-text
invoke install

results in:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'rollup-plugin-license' imported from /home/inventree/src/frontend/vite.config.ts.timestamp-1711495260776-0c5a4685b33bb.mjs

But after running invoke frontend-install the process seems to work (however I do get the following warning message):

Running the server, (invoke server and invoke frontend-dev) I get the following error when trying to view the licences:

After running:
invoke frontend-compile
invoke static -f

And then, we have success!

With regard to the actual implementation - It is very hard to look at a particular license because you have to scroll through the entire text. Can we split it up via package, and the user can click on a particular package to see the license available for that package?
This would likely require rendering the file as JSON (which is better anyway IMO) so that the frontend can split by package name.

Can we potentially make the license file generated via github actions, and remove the need for local generation?

The license file for the frontend is part of the PUI static bundle (in the .vite like the manifest required to render the right entry points) and is automatically included so it is not an issue on packaged or docker installs.
When updating an install we recommend running inv update - which will install the new frontend package. What I do not understand is how to invoke install triggers any frontend compilation.
This PR is mainly intended to fulfil the requirement to distribute copies of the licenses visible for the end user for some of the dependencies, I do not think it needs to be pretty to fulfil this legal obligation.
The Python dependencies will always need to be generated on the backend, we do not know in advance what plugins with what dependencies are installed on the server.

When updating an install we recommend running inv update - which will install the new frontend package. What I do not understand is how to invoke install triggers any frontend compilation.

After attempting again, I may have got my order of operations wrong in the first comment - I cannot reproduce this on calling invoke install.

@matmair I have added some error handling and split the licenses up so they render like this:

Having tinkered a bit more I understand the implementation better. With the addition of error handling (so that dev setups don't break for example) I am happy with this.

Very nice @SchrodingersGat that you have split up by package now.
Maybe as a last thing it would be good to get line wrapping working, for that a simple white-space: pre-line css attribute on the license body container should be sufficient.

@matmair happy to merge?

@SchrodingersGat I will try to fix this up soon

This is now ready for merge from my POV

@matmair please address conflicting files and then I will review again

@SchrodingersGat merge is in!

@SchrodingersGat this is ready now, the 4 errors in docker seem to be related to https://github.com/inventree/InvenTree/issues/6943

@SchrodingersGat CI is successful now

Any idea why the migrations checks were skipped here? They should be running due to the change in requirements.txt...

Probably because the diffset does not contain the file but not sure

Seems like that change is not working. Can we either roll back #6942 or merge this so that master is building again and try to figure out the migrations in a dedicated PR?

Yep, I'll merge this in.

@matmair cheers :)


[realsense2] fix osx build

Submitted upstream as https://github.com/IntelRealSense/librealsense/pull/11997

Thanks for the fix and for following up with upstream!


Map results in surprising key case

Map() results in a map with keys that are surprising. For example A struct with member CIDR, its resulting key in the map is cDIR. A member with Name results in a key with name, less surprising but still odd. It's like the Map() call is taking the inverse of the first character always. Is this by design? I think this behavior is in conflict with the documentation: "Keys are capitalized to find each corresponding exported field". The godoc is closer to the behavior.
import (
  "fmt"
  "testing"
)

type Record struct {
  Data    map[string]interface{}
  Mapping map[string]string
}

func StructToRecord(in interface{}) *Record {
  rec := Record{}
  rec.Data = make(map[string]interface{})
  rec.Mapping = make(map[string]string)
  typ := reflect.TypeOf(in)
  for i := 0; i < typ.NumField(); i++ {
    field := typ.Field(i)
    dbFieldName := field.Tag.Get("db")

    fmt.Printf("%d %v, tags: %v\n", i, field.Name, dbFieldName)
    if dbFieldName != "" {
      rec.Mapping[field.Name] = dbFieldName
    }
  }

  mergo.Map(&rec.Data, in)
  return &rec
}

func TestStructToRecord(t *testing.T) {

  type A struct {
    Name string `json:"name" db:"name"`
    CIDR string `json:"cidr" db:"cidr"`
  }

  type Record struct {
    Data    map[string]interface{}
    Mapping map[string]string
  }

  a := A{Name: "David", CIDR: "<IP_ADDRESS>/8"}
  rec := StructToRecord(a)
  fmt.Printf("rec: %+v\n", rec)
  if len(rec.Mapping) < 2 {
    t.Fatalf("struct to record failed, no mapping, struct missing tags?, rec: %+v, a: %+v ", rec, a)
  }
}

Output:
go test -v
=== RUN   TestStructToRecord
0 Name, tags: name
1 CIDR, tags: cidr
rec: &{Data:map[name:David cIDR:<IP_ADDRESS>/8] Mapping:map[Name:name CIDR:cidr]}
--- PASS: TestStructToRecord (0.00s)
PASS
ok             0.003s

Works in current master (future 0.3.10).


George/fix dogfooding

The dogfooded graphs weren't matching the standard Graphite versions because:

the full res granularity was different.
Graphite interpolates Null data points, (which makes NonNegative Derivative work better.)

This PR addresses both those issues.
However it is still not perfect, and there are still some discrepancies between the two that will require investigating

+1


Unable to read suppression file after upgrading from 7.1.2 to 7.2.1

Using maven I used to have a working config using ${project.basedir}/owasp-suppressions.xml in the path to suppressionFile:
    <plugin>
        <groupId>org.owasp</groupId>
        <artifactId>dependency-check-maven</artifactId>
        <version>7.1.2</version>
        <configuration>
            <suppressionFile>${project.basedir}/owasp-suppressions.xml</suppressionFile>
        </configuration>
        <executions>
            <execution>
                <goals>
                    <goal>check</goal>
                </goals>
            </execution>
        </executions>
    </plugin>

after bumping the version up to 7.2.1 I got the following error
Error:  Failed to execute goal org.owasp:dependency-check-maven:7.2.1:check (default) on project chargebee-client: One or more exceptions occurred during dependency-check analysis: One or more exceptions occurred during analysis:
Error:  	InitializationException: Warn initializing the suppression analyzer: Failed to load /home/github/_work/my-project/my-project/sub-module/owasp-suppressions.xml, caused by Unable to read suppression file.
Error:  		caused by SuppressionParseException: Failed to load /home/github/_work/my-project/my-project/sub-module/owasp-suppressions.xml, caused by Unable to read suppression file.
Error:  	InitializationException: Warn initializing the suppression analyzer: Failed to load /home/github/_work/my-project/my-project/sub-module/owasp-suppressions.xml, caused by Unable to read suppression file.
Error:  		caused by SuppressionParseException: Failed to load /home/github/_work/my-project/my-project/sub-module/owasp-suppressions.xml, caused by Unable to read suppression file.

I was able to work around it by changing to using ./owasp-suppressions.xml
    <plugin>
        <groupId>org.owasp</groupId>
        <artifactId>dependency-check-maven</artifactId>
        <version>7.2.1</version>
        <configuration>
            <suppressionFile>./owasp-suppressions.xml</suppressionFile>
        </configuration>
        <executions>
            <execution>
                <goals>
                    <goal>check</goal>
                </goals>
            </execution>
        </executions>
    </plugin>

${project.basedir} might not be what you think in a multimodule build, but is seems you have redacted the logging so... here's the doc: https://maven.apache.org/guides/introduction/introduction-to-the-pom.html#Available_Variables

the build log is for a private project. It's not the use of ${project.basedir} that was the problem though, that's worked perfectly fine since dependency check was added to the project. The only thing that changed when the problem occurred was the change to the version number

Maven plugins never see ${project.basedir} the pom file is expanded during parsing, so it is outside the control of any plugin, plugins just get the path injected.
In any case you will have to provide some means of reproducing your problem.
all I can tell it works without any issues for me in eg. this project:
https://github.com/B3Partners/brmo/blob/05fb71489fa67a621a1b96e1c837a4f96a080215/pom.xml#L1079-L1090
as you can see in https://github.com/B3Partners/brmo/actions/runs/3079008323/jobs/4974916567

The reason this "worked" before is that ODC would have logged this as a warning and not bubbled the exception up. If a suppression file is not found the build will fail. Sorry for not making that clear in the release notes - the change was buried in a larger change around how suppression rules are handled.
Consider using ${session.executionRootDirectory} for aggregate builds? see https://stackoverflow.com/a/7676673/1995422
Alternatively, one could use the <inherited>false</inherited> in the plugin configuration for an aggregate build so that ODC is not directly executed in all child projects - rather just executed once at the parent level.


new version of module not published to registry

I just created a new release of my module, Capi. Every time we've published a new version, it's been visible on deno.land/x/capi within a minute or so. 40 minutes has passed and still the new version isn't reflected (https://deno.land/x/capi). I'm concerned there may be an issue.

Have you checked the webhook delivery status?

I told the webhook to redeliver and it reported a success. I don't see a new version yet, but it's only been a few minutes – I'll give it a little while longer and check back in.

Ah, looks like it gave an error: https://deno.com/status/cd1ce4a8-1d47-47d7-a62c-3810ebd52700.

Module too large (28003800 bytes). Maximum allowed size is 20971520 bytes.

Thank you for the assistance!


How to resize blui canvas ?

by resizing blui widget ,mouse cursor not working  i can't  click on input fields it's look like the browser widget was been shifted  idk.
more info : i don't won't blui widget to be full screen i won't it to specific position
HELP !!!

If you use the BluiWorldWidgetActorExample, inside its begin play

you can see that the widget draw size is passed into Init Blu Eye

which sets the correct draw size in the Blu Eye instance.

If both the world widget draw size and the blu eye draw size are set correctly, your rendering will be aligned.

hey Thank you so mush

i want the widget to be in the middle of screen is that possible

@getnamo hey what about webrtc and access to mice  how can i do that ? it only work with apprtc
but in speech to text doesn't work thank in advance

Moving the widget should be done by the owner of the widget, just move the actor if it's a world widget. WebRTC I believe requires a newer chromium build see https://github.com/getnamo/BLUI/issues/11. I'm probably not going to update chromium for a while (not enough time), but the general instructions to do that can be found here https://github.com/AaronShea/BluBrowser

if i click up it show cursor

Plz Help

i'm only using widget to viewport not the actor

https://api.unrealengine.com/INT/BlueprintAPI/UserInterface/Viewport/SetAlignmentinViewport/index.html

hey bro thx for your help i tried what you give me but no chance :(
the problem is that blu render the widget according to the size given but i translate the widget with y + 200 i lose  click on my inputs field  but if i click on top as i showed you in picture it work so what's wrong plz help


Error while computing document links for ...

I get this error every time I save a *.scss file:
[Error - 1:41:25 AM] Error while computing document links for file:///c%3A/Users/myuser/folder/scss/amp/_header.scss: URI malformed
URIError: URI malformed
    at decodeURIComponent (<anonymous>)
    at Function.e.parse (c:\Users\myuser\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\css-language-features\server\dist\cssServerMain.js:1:738848)
    at t.<anonymous> (c:\Users\myuser\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\css-language-features\server\dist\cssServerMain.js:1:744433)
    at c:\Users\myuser\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\css-language-features\server\dist\cssServerMain.js:1:743828
    at Object.next (c:\Users\myuser\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\css-language-features\server\dist\cssServerMain.js:1:743933)
    at s (c:\Users\myuser\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\css-language-features\server\dist\cssServerMain.js:1:742702)

It causes the bottom (output) panel to pop up every time I hit save and this is quite annoying TBH.
(I'm not sure if this is the correct repo for this bug, should I create it in the vscode repo instead?)
VS Code version:
1.37.1
f06011ac164ae4dc8e753a3fe7f9549844d15e35
x64

Edit:
1.36 works fine

I am having the same issue. Here is my environment:
Version: 1.38.0
Commit: 3db7e09f3b61f915d03bbfa58e258d6eee843f35
Date: 2019-09-03T21:47:00.464Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0

Do you have a repro case?

Same problem here!
Environment:
Version: 1.38.1 (user setup)
Commit: b37e54c98e1a74ba89e03073e5a3761284e3ffb0
Date: 2019-09-11T13:35:15.005Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.17763

I've opened a pull request for this bug: #180
Steps to reproduce the error:

Create a file named test.scss
Paste:

.test {
    background: url('<%=invalid%>');
}

Open the file in VS Code


Profiler filters by FQN

Profiler should not filter by table and/or schema, but directly by FQN

done in https://github.com/open-metadata/OpenMetadata/pull/4325


Configuration property

In document you suggested to set configuration for AccountId with Azure Domain Account instead of AccountDomain property

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

ID: d0b937c5-e104-7ad9-249c-4594f95298b8
Version Independent ID: 0b9f9820-e418-08f3-9d67-356b81cb1d0e
Content: Quickstart: Create a Xamarin Android app - Azure Spatial Anchors
Content Source: articles/spatial-anchors/quickstarts/get-started-xamarin-android.md
Service: azure-spatial-anchors
GitHub Login: @craigktreasure
Microsoft Alias: crtreasu

Ah! Good catch. Thanks for reporting this. I'll get a fix out.


Unit tests assume app:install

Add phake app:install step to Travis CI install process
Test that .env file has been generated as part of the install.

:+1:


Fixes to trivy parsing

Trivy reports sometimes miss the Title property causing the script to convert to SecurityHub to fail.

Thanks for this PR - fixed an issue I had with title parsing:
#python3 sechub_parser.py Traceback (most recent call last):   File "sechub_parser.py", line 43, in <module>     cveTitle = str(p['Title']) KeyError: 'Title'