Help!

Hey, when I add this part of code below vvvvvvvv in my original code, it gives me this error in my command prompt:

let member = msg.mentions.members.first();
if(member)
return msg.reply("You haven't mentioned a member yet, command declined.");
let muteRole = msg.guild.roles.find("name", "Muted");
if(muteRole)
return msg.reply("You haven't got a role by the name of 'Muted', command declined.")
let params = msg.content.split(" ").slice(1);
let time = params[1];
if(time) return msg.reply("There is no time specified, command declined.")
member.addRole(muteRole.id);
msg.channel.send(You've been muted for ${ms(ms(time), {long: true})} ${member.user.tag});
setTimeout(function() {
member.removeRole(muteRole.id);
msg.channel.send(${member.user.tag} You've been unmuted.)
-11.2.1 Discord.js
-8.4.0 Node.js

Windows 8.1 64-bit

[ ] I found this issue while running code on a user account

[ ] I have also tested the issue on latest master, commit hash:

The issues page is for issues with the library.  Please go to the server to inquire about your code.

Please read the issue template you deleted in future.

Go to which Server?
Okay.

Sorry, which template and how do I view it?

From: Amish Shah<EMAIL_ADDRESS>Sent: Tuesday, September 5, 2017 12:19:01 PM
To: hydrabolt/discord.js
Cc: MrFifth; Author
Subject: Re: [hydrabolt/discord.js] Help! (#1891)
Please read the issue template you deleted in future.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hydrabolt/discord.js/issues/1891#issuecomment-327105490, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AeLqGJKoCFyOG4tqXvNsWwlc3k43wZklks5sfQP0gaJpZM4PMlFh.

Which server?

From: Isabella<EMAIL_ADDRESS>Sent: Tuesday, September 5, 2017 12:09:15 PM
To: hydrabolt/discord.js
Cc: MrFifth; Author
Subject: Re: [hydrabolt/discord.js] Help! (#1891)
The issues page is for issues with the library. Please go to the server to inquire about your code.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hydrabolt/discord.js/issues/1891#issuecomment-327103228, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AeLqGFE0OYzZnXUqH-UTzotKUUhMg_Q3ks5sfQGrgaJpZM4PMlFh.

The issue template that comes up right when you try to make a new issue.

thank you.

From: Dragon Fire<EMAIL_ADDRESS>Sent: Tuesday, September 5, 2017 3:26:36 PM
To: hydrabolt/discord.js
Cc: MrFifth; Author
Subject: Re: [hydrabolt/discord.js] Help! (#1891)
The issue template that comes up right when you try to make a new issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/hydrabolt/discord.js/issues/1891#issuecomment-327147922, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AeLqGCvkc4fI3xu49H0LkSEQTPGToq2Bks5sfS_sgaJpZM4PMlFh.


service doesn't terminate on sigterm

service-lb-postgres-akrobateo-8s7bs   0/1     Terminating   0          38s

I think it ignores SIGTERM signal

it's especially annoying because it's created by the operator and I can not force terminationGracePeriodSeconds:  as with other misbehaving images

I think this ignores TERM https://github.com/kontena/akrobateo/blob/master/lb-image/entrypoint.sh#L24
here's what I always do:
tail -f /dev/null &
wait $!

fixed in #28


Url Check shows broken link for event-image, but it's not!

This is a back-end admin panel bug.
Not sure if it really is.
Even though  I'm using the same image link that has been earlier used in past events. while creating a new event, it shows me this:
Failed to update record. This URL appears to be a broken link: HTTP Error 403: Forbidden
So, if the URL was actually broken it shouldn't show the image for past events.

Closing this down, since the back-end is shifted to MozStar repository


Multiselect filter doesn't search consecutively

Describe the bug
https://github.com/marimo-team/marimo/assets/29712687/4f9e64b8-f7a3-49ea-87ae-74a235955f5f
Not clear how the filtering works. I would expect it to search by substring, but searching foo also returns the result farboo.
Environment
{
"marimo": "0.1.65",
"OS": "Darwin",
"OS Version": "22.6.0",
"Processor": "arm",
"Python Version": "3.11.4",
"Binaries": {
"Chrome": "120.0.6099.71",
"Node": "v20.3.1"
},
"Requirements": {
"black": "23.10.1",
"click": "8.1.7",
"jedi": "0.19.1",
"pymdown-extensions": "10.3.1",
"tomlkit": "0.12.1",
"tornado": "6.3.3",
"typing_extensions": "4.8.0"
}
}
Code to reproduce
import marimo as mo
multiselect = mo.ui.multiselect(["foo", "farboo"])
multiselect

@delennc, we use a library for this which does fuzzy matching.
we can override this to lowercase + exact match as that is likely what is expected, and can later add back other filtering kinds when requested.


[Enhancement] Support full text search in new query engine.

Is your feature request related to a problem? Please describe.
Legacy engine and new engine behavior is not compatible. For example, the following query use QUERY function which is only support in legacy engine. Legacy will return 200 records and it is not configured.
SELECT *
FROM opensearch_dashboards_sample_data_flights
WHERE DestAirportID='SYD' AND QUERY('Cancelled:false')

In new engine, there is a setting plugins.query.size_limit which could be used to control return size.
Describe the solution you'd like
Support full text search functions are implemented in legacy query engine.
Describe alternatives you've considered
Support plugins.query.size_limit in legacy engine.
Additional context
Add any other context or screenshots about the feature request here.

We should also clarify the difference between https://github.com/opensearch-project/sql/blob/main/docs/user/admin/settings.rst#plugins-query-size-limit and LIMIT clause

Full support is already added to SQL engine v2 in https://github.com/opensearch-project/sql/issues/182


Rally improve artifact download headers

To prevent Rally artifact downloads from being counted in statistics add parameter to download url: x-elastic-no-kpi=true

closing as duplicate of https://github.com/elastic/rally/pull/1159

closing as duplicate of https://github.com/elastic/rally/pull/1159


Minor improvements/cleanup to the README.md and CONTRIBUTING.md

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #107 (7b88fb4) into main (568f1bc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #107   +/-   ##
=======================================
  Coverage   70.34%   70.34%
=======================================
  Files          14       14
  Lines        1359     1359
=======================================
  Hits          956      956
  Misses        330      330
  Partials       73       73

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 568f1bc...7b88fb4. Read the comment docs.


Shell login as root

I have a fresh installation of arch in a VM for testing. I can login to a live tty as root and my user from the default linux login prompt, but I cannot login as root from ly. I get the error "pam transaction aborted". Loggin in as my user works just fine in ly.
In a different issue somebody commented on an unrelated problem that they too couldn't login as root into the shell from ly because of pam. It seemed they thought that was normal and not a error.
My question is, whether the problem lies with me, as this is simply a config issue on my side, whether this is a error, or whether it is impossible to log in to the shell as root from ly.

I have a fresh installation of arch in a VM for testing. I can login to a live tty as root and my user from the default linux login prompt, but I cannot login as root from ly. I get the error "pam transaction aborted". Loggin in as my user works just fine in ly.
In a different issue somebody commented on an unrelated problem that they too couldn't login as root into the shell from ly because of pam. It seemed they thought that was normal and not a error.
My question is, whether the problem lies with me, as this is simply a config issue on my side, whether this is a error, or whether it is impossible to log in to the shell as root from ly.

edit /etc/pam.d/login
remove or comment line include word "require"

FYI @josephholten check out the file at /etc/securetty, this is the file regulating whether root can log in on a specific TTY or not, it's invoked by auth       required     pam_securetty.so in /etc/pam.d/login


Language support for D&D file input

Today the input button overlay is hard coded to a specific width, this breaks for other languages and rtl support.
Making these changes http://www.elkarte.net/community/index.php?topic=4315.msg31205#msg31205 works well but does require us to break the current txt string into two strings.   No drama for 1.1 but would be a language change for 1.0.
@emanuele45 thoughts on pulling this in to 1.0.10?

If I'm getting it, the main difference is that the input is inside the span that wraps the text, right?
If so, can't we use some js trickery to move the input inside the <a> tag?
It shouldn't be too difficult (though with js you never know what can happen lol), something like:
			// Rather click and select?
			$input = obj.find('#attachment_click');
			$input.change(function(e) {
				e.preventDefault();
				var files = $(this)[0].files;
				handleFileUpload(files, obj);
				this.value = null;
			});
			$input.clone().appendTo('.drop_area_fileselect_text');
			$input.hide();

If I'm getting it, the main difference is that the input is inside the span that wraps the text, right?

correct .. I tried a couple of things, but not having that span to target I could not get things to work.
Humm .. had not considered doing that.  I had done a position css trick like
		// Adjust the input overlay to the underlying link
		$text = $('.drop_area_fileselect_text');
		$('.drop_area_fileselect').css({
			"width"       : $text.outerWidth(),
			"min-height"  : $text.height(),
			"margin-left" : $text.outerWidth() * -1,
			"margin-top"  : $text.height() - $text.outerHeight()
		});
		$('#chrome .drop_area_fileselect').css({
			"padding-left": $text.width()
		});

to adjust the overlay to whatever text was in place.   I can give your suggestion a try as well, or do we just leave it kind of messed up from 1.0.10 at this point and fix it proper in 1.1

Dunno about 1.0...
I'd say let's fix 1.1 and we can consider later on a 1.0.11?

Sounds find to me, its been fragile since forever ... for 1.1 I'd go for breaking up the txt string so it just works.   Will make the PR

I did play a bit with my suggestion and pushed something that seems to work in all the conditions I tried (i.e. different sizes and rtl).
It's js-based, but since the functionality relies on js it's not an issue.


show computed type

Search Terms

Suggestion

Use Cases

Examples
type User = {
  id: number
  name: string
  birthday: number
}
var xxx: Partial<User>

show the type that computed type for xxx as blow
{
  ?id: number
  ?name: string
  ?birthday: number
}

Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[ ] This wouldn't change the runtime behavior of existing JavaScript code
[ ] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[ ] This feature would agree with the rest of TypeScript's Design Goals.

It's difficult to make everyone happy with when type aliases are expanded or not.
As a workaround, I've used an Id type to force expansion of a given type:
type User = {
  id: number
  name: string
  birthday: number
}
type Id<T> = {} & { [ P in keyof T]: T[P]}
var xxx: Id<Partial<User>>
// quick info:
// var xxx: {
//     id?: number | undefined;
//    name?: string | undefined;
//     birthday?: number | undefined;
// }

This works at the moment, no guarantees about the future.

and does is a bug?
ts dont given a error
var q1: number[] = []

var ww1 = q1.pop()

ww1.toFixed()

ww1 was undefiend
@dragomirtitian

Turn on your strictNullChecks compiler option

It's difficult to make everyone happy with when type aliases are expanded or not.

This x 1,000. For a large type, Partial<T> is much easier to comprehend.


i can not load the dict.en.th7,what is the structure of the dict.en.th7 file

file=torch.load('dict.en.th7')
the result is
/home/linhanxiao/torch/install/bin/luajit: /home/linhanxiao/torch/install/share/lua/5.1/torch/File.lua:343: unknown Torch class
stack traceback:
[C]: in function 'error'
/home/linhanxiao/torch/install/share/lua/5.1/torch/File.lua:343: in function 'readObject'
/home/linhanxiao/torch/install/share/lua/5.1/torch/File.lua:409: in function 'load

You will need to require the fairseq module before loading the dictionary file. The .th7 files are produced with torch.save(), but when you call torch.load() the class definitions must be available.

thanks for your answer


which version of waymo-open-dataset-tf?

thank you for your excellent code. Can you please tell me which version of the dataset you are using, and which version of waymo-open-dataset-tf?

Thanks! We've used Motion v1.2.0's scenario representation (i.e., files with names like training.tfrecord-XYZ). The repo was tested waymo-open-dataset-tf-2-12-0==1.6.4.

thanks for your prompt reply!


Define u5 type to distinguish between 5bit and 8bit slices

In rust-lightning-invoice I pass around slices of the data part as &[u8], but each byte is guaranteed to be in the range 0..32. I'd like to encode this guarantee in the type by defining pub struct u5(u8).
The traits FromBase32 and IntoBase32 could be introduced and implemented for &[u8] and &[u5] respectively and replace convert_bits (From<&[u8]> for &[u5] and From<&[u5]> for &[u8] can't be implemented since type and trait are foreign and it would be quite ambiguous). Furthermore  FromBase32 could be implemented for data structures that get parsed from bech32 data (like lightning invoice fields).
The best case UX would be that you can now implement FromBase32 for your types and don't have to worry about out-of-range errors anymore. The worst case UX would probably be that every use of an u5 (formerly u8) will now require an .into::<u8>().
What do you think about this proposal @clarkmoody and @apoelstra (I know I brought this up already in #5 but it got lost when we released the new version)?

I agree. I'd like to see the implementation details and hope we can avoid that worst-case UX.


[BUG]  Issue when installing on Windows 10 IoT LTSC 2021 (Windows10 10.0.19044.4412 x64)

Office Tool Plus Version, e.g. <IP_ADDRESS>:

<IP_ADDRESS>

Operating System, e.g. Windows 10 20H2 or Windows 7 SP1:

Windows 10 IoT LTSC 2021 10.0.19044.4412 x64

Error description (required):

Clicking Start Deploy on Deploy page throws an error.

Reproduction steps (suggested):

Configure Architecture, Products, Languages and Advanced Settings

Set Architecture to 64-bit
Choose Products: Microsoft 365 apps for Enterprise, Project Professional 2016 and Visio Professional 2016
Choose Language: English US (en-us)
Choose Advanced Settings > Activation Settings > Automatically accept the EULA

Click Deploy
Throw error

Screenshot (suggested):

Additional information (optional):
This has been going on for all the v10 releases I have tried (at least 3 different versions). Not sure what is causing it. Tried over several different installs of Windows 10, one windows 11 install, and two different devices.

The issue was fixed. The new version will be released on the next week.

10-4. I'll keep an eye out for it.

New release was out.


Re-Compile for Kibana 5.6.0

Hey Gang,
Just curious if you were going to release this plugin so we could run it against Kibana v5.6.0, since the versions are supposed to stay in lock step in regards to kibana and its plugins.
Cheers,

Sure, we just need to change a couple of lines and run the CI again.
(Usually just package.json and travis.yml environmental variables)
Give me a couple of days, I am out of the country. If you're in a hurry
please submit a PR.
Il mar 17 ott 2017, 00:51 Matt West<EMAIL_ADDRESS>ha scritto:

Hey Gang,
Just curious if you were going to release this plugin so we could run it
against Kibana v5.6.0, since the versions are supposed to stay in lock step
in regards to kibana and its plugins.
Cheers,
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fermiumlabs/mathlion/issues/49, or mute the thread
https://github.com/notifications/unsubscribe-auth/AIot_is_75xlTr1xdYuMRBRX2vw9UdJwks5ss94DgaJpZM4P7Wa_
.

Any updates on this?  Really looking forward to running with kibana 5.6.4

Very sorry for the delay guys. FIxing it ASAP.
If you've ever in a hurry, i've extended the docs in the README regarding opening a PR when you need to update it, to not have to wait us.

Fixed https://github.com/fermiumlabs/mathlion/releases/tag/v0.2.11


Error when executing the info call with a client credentials

2017/02/03 10:33:27 runtime error: invalid memory address or nil pointer dereference
::1 - - [03/Feb/2017:10:33:27 +0100] "GET /api/users/rob/info HTTP/1.1" 500 0
branch 0.9.10

Caused by https://github.com/itsyouonline/identityserver/commit/cce12d6533d3ca1912759cf5dfc9e4e8aace7619


Remove space before and after the slider bar

When a new Slideer is created, it is created inside a view and I can see an space before and after the slider (just set backgroundColor='red' on the style)
This is quite annoying in order to make it if with other components as the slider is always a bit smaller
Is this done in purpose? Can those spaces (padding or margin) be removed and make the bar use 100%
I tried setting the padding, margin, left, right to 0, but nothing...

You could temporarily put it inside a view with negative margins:
<View style={{ marginLeft: -10, marginRight: -10 }}>
  <Slider />
</View>

This behavior affects both platforms?

Is this issue still alive? I'd like to work on this issue but the example app doesn't reproduce this.

Or with marginHorizontal: -10

Or with marginHorizontal: -10

I see this issue only on Android. I applied the following style to the slider:
marginLeft: Platform.select({
      ios: 0,
      android: -15,
}),
marginRight: Platform.select({
      ios: 0,
      android: -15,
})

Kinda annoying but it works.

I see this issue only on Android. I applied the following style to the slider:
marginLeft: Platform.select({ ios: 0, android: -15 }),
marginRight: Platform.select({ ios: 0, android: -15 })

Kinda annoying but it works.

it's works, thanks

The issue seems to be still there. The "margin trick" works on android for me but it causes that one tick extra is added to the upper/right limit - with no change in value. so not very useful. :(

Anyone got an idea of how to select the margin values - are you just doing it by eye? It looks better with -3 for iOS on our app, but unsure if this is due to different screen sizes/resolutions etc.
Is setting a constant value for each platform robust?

Any update on this, I still see this on version 4.5.2


Damage not displaying on some targets

I've tested this addon in many instances and sometimes the damage displays on a different target, or it doesnt display at all. It works how it should sometimes, but most of the time it is either not showing damage at all or shows it on a different target. Link to example: https://youtu.be/TkhY1_hInwQ

Hello @TrippyDedd
Try to download this version and let me know if it works better than the one you have.

Hello @bkader I'll try this version out. I've tested it before with KUI Nameplates instead and had the same problem (on the version that I have). Will try with this version with and without ElvUI when I get the chance. Thank you for the quick response, cheers^^

@bkader  I see it's just a conflict with ElvUI, thank you :)

Hello again @TrippyDedd . I have completely rewritten the library and tested it on most addons, even on ElvUI, as long as GUIDs are cached. Let me know if it works on your end with the new changes.
Cheers.


<EMAIL_ADDRESS>hash check failed

Updating 'spotify-latest' (<IP_ADDRESS>8.g4b100958 -> <IP_ADDRESS>2.g1e7e78a4)
Downloading new version
SpotifyFullSetup.exe (78.3 MB) [==============================================================================] 100%
Checking hash of SpotifyFullSetup.exe ... ERROR Hash check failed!
App:         spotify/spotify-latest
URL:         https://download.scdn.co/SpotifyFullSetup.exe
First bytes: 4D 5A 90 00 03 00 00 00
Expected:    752f3b4b5e41cc7948ed8050099bdca1bb109a3b64d81292b1da3486a5a55e59
Actual:      8067c8dafde4ce9f065fa1079f2cee14ad97dab62f3901fb5569286371c88879

Fixed.


No ticker-messages on WebSocket when using API-Key that is restricted to an IP-address

I have just created an API-Key on CoinbasePro that it is restricted to a specific IP-address (for security reasons).
But the problem is now that now I don't get any ticker messages via the WebSocket anymore. Calls to CoinbasePro with the same api key are working perfectly, so the credentials are fine.
When I switch back to an API-Key that is NOT restricted to an IP-address, I am receiving the ticker messages without any problems.
So my questions is, is this a bug or a intended behaviour of CoinbasePro?
Best,
Michael

Since this feature is very important to me, I already digged into the code. In fact the message is received (a websocket message was sent by CoinbasePro):

{"type":"error","message":"Sorry, you could not be authenticated: Unauthorized","reason":"IP does not match IP whitelist"}

This results in a ResponseType.Error and therefore no event was fired.
The IP-address is definitly on the whitelist and all others calls except WebSocket does work.
Is the fault on the CoinbasePro-side? Maybe there is a 2. whitelist for WebSockets?

hey @michaelhuber1937. I tried looking around and can’t seem to find anymore information around it elsewhere. I can also try later today using my own credentials to see if I get the same error.
Did this work for you in previous versions of this library? If no then I would probably reach out to support on Coinbases end. They usually respond quickly if you use the reason for contact under API.
The only other thing I can recommend is to maybe try removing your key and creating a new one if you haven’t already.
I’ll get back to you later today.

I am seeing the same issue. Digging around some more and it looks like it's a known issue: https://github.com/coinbase/gdax-node/issues/201
Depending on what you are trying to do, a workaround "solution" from that thread is to use a read-only API key without an IP filter for web socket and then use the IP filtered key via REST to do any trade operations.
Wish I had a better answer for you!

I have a better idea. You overload the contructor with another parameter Authentication-Object, e.g. "authenticationWebSocket" and use that for the websocket. If this parameter is null, no authentication will be done, so the public WebSocket is used.
The standard-constructor just passes the same object of authentication to both internally.

Depending on what you are trying to do, a workaround "solution" from that thread is to use a read-only API key without an IP filter for web socket and then use the IP filtered key via REST to do any trade operations.

Good idea as well, but I don't prefer this solution since then we have two WebSocket-connections. Although it works.

@dougdellolio
What do you think?


Split into multiple modules

The functions and macros remain accessible under the old names, so this is transparent to existing users of the crate.

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is 94.08%.

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   94.17%   94.17%
=======================================
  Files           2        5    +3
  Lines         378      378
=======================================
  Hits          356      356
  Misses         22       22

Impacted Files
Coverage Δ

src/contains_regex.rs
100% <100%> (ø)

src/html_root_url.rs
90.97% <90.97%> (ø)

src/markdown_deps.rs
93.79% <93.79%> (ø)

src/helpers.rs
97.56% <97.56%> (ø)

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 5a40af0...8619244. Read the comment docs.


EPERM: operation not permitted, copyfile  - When working in DevContainer and using a google drive local map

Please delete, i opended a ticket using vscode here:  https://github.com/microsoft/vscode/issues/212417

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


[metrics] AUROC Metric cant handle batch with single class samples

🐛 Bug
During calculation of AUROC for a batch, if all samples of particular batch are of same class, it raises following error
`/usr/local/lib/python3.6/dist-packages/pytorch_lightning/metrics/functional/classification.py in roc(pred, target, sample_weight, pos_label)
    548
    549     if fps[-1] <= 0:
--> 550         raise ValueError("No negative samples in targets, false positive value should be meaningless")
    551
    552     fpr = fps / fps[-1]

ValueError: No negative samples in targets, false positive value should be meaningless`

This is tensor for labels : tensor([1., 1., 1., 1., 1., 1.]) for this batch all samples were of same class,
The metric being used : pytorch_lightning.metrics.classification.AUROC
Expected behavior
I don't know much about the calculation of auroc but it possible to batch have a single class only, and the metric should somehow handle this

In such case it shall be 1
@justusschock @SkafteNicki

Well this is to be consistent with sklearn:
>>> roc_auc_score([1, 1, 1, 1, 1], [0.9, 0.8, 0.5, 0.7])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/justusschock/Workspace/conda/envs/leg/lib/python3.7/site-packages/sklearn/utils/validation.py", line 73, in inner_f
    return f(**kwargs)
  File "/Users/justusschock/Workspace/conda/envs/leg/lib/python3.7/site-packages/sklearn/metrics/_ranking.py", line 393, in roc_auc_score
    sample_weight=sample_weight)
  File "/Users/justusschock/Workspace/conda/envs/leg/lib/python3.7/site-packages/sklearn/metrics/_base.py", line 77, in _average_binary_score
    return binary_metric(y_true, y_score, sample_weight=sample_weight)
  File "/Users/justusschock/Workspace/conda/envs/leg/lib/python3.7/site-packages/sklearn/metrics/_ranking.py", line 223, in _binary_roc_auc_score
    raise ValueError("Only one class present in y_true. ROC AUC score "
ValueError: Only one class present in y_true. ROC AUC score is not defined in that case.

And I also think it should be handled that way

@justusschock we calculate metrics after each validation step, and we cannot insure them to be of a single class, this way training would never complete due to possible error in every step. I think sklearn metric is designed keeping in mind to calculate auroc over a complete dataset not a batch.

Yes, and we basically do the same.
This implementation just provides the implementation. In the next version, we will provide an automated evaluation plan that caches multiple batches. In the meantime you can return them and aggregate them in at_epoch_end for metric computation.
I designed it that way on purpose, since an AUROC (or a ROC in general) is not descriptive in a single batch

Oh, that sounds right, currently, i just increased the validation batch size very much to decrease such probability of such cases:D, but I will add it to val epoch end, Thanks @justusschock

What I am doing now is using the following code for aggregation:
class Metrics(nn.Module):
    def __init__(self, num_classes):
        super().__init__()
        self.trues = []
        self.preds = []
        self.metrics = [
            M.Precision(num_classes),
            M.Recall(num_classes),
            M.AUROC(),
            M.Accuracy(),
            # M.AveragePrecision(pos_label=0),
            # M.AveragePrecision(pos_label=1),
        ]

    def forward(self, y_pred, y_true, prefix='train'):
        self.trues.append(y_true.cpu())
        self.preds.append(y_pred.detach().cpu())

        trues = torch.cat(self.trues, dim=0)
        preds = torch.cat(self.preds, dim=0)

        results = {}
        if len(trues.unique()) != 1:
            preds = nn.functional.softmax(preds, dim=-1)
            for metric in self.metrics:
                results.update({f"{prefix}_{metric.name}": metric(preds.cpu(), trues.cpu())})
        return results

    def reset(self):
        self.trues = []
        self.preds = []

Can you think of a good workaround for the multiclass scenario? I'm running into an issue where there are classes that are very rare.
In the case of 0 observations, I feel the average='weighted' should work, since the contribution to the final AUROC should be 0 regardless. One can think of other scenarios where there are a very high number of classes, some of which will happen to not be seen in some dataset.

Hi, please has this been fixed?
run into the same error while working with a binary classification problem
No positive samples in targets, true positive value should be meaningless

@Troublem1 if you do not have any positive samples then the score is meaningless. Are you trying to evaluate this per batch?

Yes i was.. should i calculate it per epoch instead?

For anyone else looking this up, here's how I did it inside a LightningModule:
def __init__(self, ...):
        self.valid_auc = torchmetrics.AUROC(pos_label=1)

def validation_step(self, batch: Tuple[torch.FloatTensor, torch.LongTensor], *args) -> None:
        self.valid_auc.update(logits.cpu().detach(), y.cpu().detach())
        self.log("validation_auc", self.valid_auc, on_step=False, on_epoch=True, prog_bar=True)

a solution I used in 3D medical is to compute the metric over epoch instead of single batch
https://github.com/Borda/kaggle_brain-tumor-3D/blob/58185ed24f58f06c2675728a7743a46dacce5af2/kaggle_brain3d/models.py#L52

@Borda thanks.. i am current using PL lightning for that competition.. i got a bit scared when my model starting reaching nans.. seeing as yours did as well.. its a bit surprising..
thanks for sharing the AUROC solutions...

@Borda thanks for sharing the AUROC solution!
I'm little concerned about using try / except statement.
how about this one?
def __init__(self, ...):
    ...
    self.auroc = torchmetrics.AUROC(..., compute_on_step)
...

def validation_step(self, batch, _):
    ...
    self.auroc(y_hat, y)
    self.log("val_auroc", self.auroc.compute(), on_step=False, on_epoch=True)

please tell me if there is something wrong


Mono Support

Is this lib Mono compatible?
Seems to create a client fine on Windows but I get this on OSX
var client = new DockerClientConfiguration(new Uri("tcp://<IP_ADDRESS>:2376")).CreateClient();
Could not load type 'Docker.DotNet.AnonymousCredentials' from assembly 'Docker.DotNet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.

Interesting. What version of Mono are you working with? Are you going
command line or using Xamarin Studio or another IDE?
On Tue, May 19, 2015 at 6:39 AM Jonathan Channon<EMAIL_ADDRESS>wrote:

Is this lib Mono compatible?
Seems to create a client fine on Windows but I get this on OSX
Could not load type 'Docker.DotNet.AnonymousCredentials' from assembly
'Docker.DotNet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
—
Reply to this email directly or view it on GitHub
https://github.com/ahmetalpbalkan/Docker.DotNet/issues/24.

I'm on 3.12.1 on OSX with Xamarin
On Tuesday, 19 May 2015, Josh Garverick<EMAIL_ADDRESS>wrote:

Interesting. What version of Mono are you working with? Are you going
command line or using Xamarin Studio or another IDE?
On Tue, May 19, 2015 at 6:39 AM Jonathan Channon<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
wrote:

Is this lib Mono compatible?
Seems to create a client fine on Windows but I get this on OSX
Could not load type 'Docker.DotNet.AnonymousCredentials' from assembly
'Docker.DotNet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
--
Reply to this email directly or view it on GitHub
https://github.com/ahmetalpbalkan/Docker.DotNet/issues/24.

--
Reply to this email directly or view it on GitHub
https://github.com/ahmetalpbalkan/Docker.DotNet/issues/24#issuecomment-103517380
.

Interesting, I developed this library in Mono on OS X.

Not sure what the issue was, seems ok now

I've got the error back here's a screenshot with code and result in the terminal

Looks like the issue is Xamarin Studio in that it wasn't copying System.Net.Http into the bin directory

I have the same problem while running tests on our bamboo
02-Nov-2015 18:18:56	Runtime Environment -
02-Nov-2015 18:18:56	   OS Version: Unix <IP_ADDRESS>
02-Nov-2015 18:18:56	  CLR Version: 4.0.30319.17020 ( Mono 4.0 ( 4.0.4 (Stable <IP_ADDRESS>/5ab4c0d Tue Aug 25 23:11:51 UTC 2015) ) )
02-Nov-2015 18:18:56
02-Nov-2015 18:18:56	ProcessModel: Default    DomainUsage: Single
02-Nov-2015 18:18:56	Execution Runtime: mono-4.0
02-Nov-2015 18:18:57	.F....
02-Nov-2015 18:18:57	Tests run: 5, Errors: 1, Failures: 0, Inconclusive: 0, Time: 0.1476902 seconds
02-Nov-2015 18:18:57	  Not run: 0, Invalid: 0, Ignored: 0, Skipped: 0
02-Nov-2015 18:18:57
02-Nov-2015 18:18:57	Errors and Failures:
02-Nov-2015 18:18:57	1) TearDown Error : <Testclass>
02-Nov-2015 18:18:57	   SetUp : System.TypeLoadException : Could not load type 'Docker.DotNet.AnonymousCredentials' from assembly 'Docker.DotNet, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.

Based on @jchannon posts I checked for the System.Net.Http.dll:
packages/Microsoft.Net.Http.2.2.28/lib/net40
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.dll
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.xml
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.Primitives.dll
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.Primitives.xml
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.WebRequest.dll
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.WebRequest.xml
packages/Microsoft.Net.Http.2.2.28/lib/net40/System.Net.Http.Extensions.dll
packages/Microsoft.Net.Http.2.2.28/lib/net40/ensureRedirect.xml
packages/Microsoft.Net.Http.2.2.28/lib/net45
packages/Microsoft.Net.Http.2.2.28/lib/net45/System.Net.Http.Primitives.dll
packages/Microsoft.Net.Http.2.2.28/lib/net45/System.Net.Http.Primitives.xml
packages/Microsoft.Net.Http.2.2.28/lib/net45/System.Net.Http.Extensions.dll
packages/Microsoft.Net.Http.2.2.28/lib/net45/ensureRedirect.xml

The net45 System.Net.Http is missing. Any ideas what could be the problem?


chore: add eslint with carbon config

Part of #6
Adds eslint and carbon config
Testing / Reviewing
Ran and no issues highlighted. Switched to airbnb config to see issues before switching back.

I pulled this down, ran the lint command, and saw only done output, but when i look at a component like the ExampleComponent in VS code I see a lot of lint error highlighting showing up. so i guess my question is what is the intended result from running yarn lint?

I pulled this down, ran the lint command, and saw only done output, but when i look at a component like the ExampleComponent in VS code I see a lot of lint error highlighting showing up. so i guess my question is what is the intended result from running yarn lint?

Needed quotes around 'packages/**/*.js'

Are you able to take another look at this @davidmenendez


partners[anthropic]: adding a default chat model for ChatAnthropic

Description: Adding a default chat model for the ChatAnthropic class
Issue: No default model set for the ChatAnthropic class
Dependencies: None
Lint and test: make format, make lint and make test have been run
The chat model was set to claude-3-sonnet-20240229, the one with the cheapest cost among the current Claude-3 models.

not adding a default is intentional, since it'll go out of date at some point and updating the default is a breaking change

not adding a default is intentional, since it'll go out of date at some point and updating the default is a breaking change

ok, I'll close this pr.


Support bazel-watcher (iBazel)

Are there any plans to support using bazel-watcher through bazelisk?
Adding something like a "--watch" option to bazelisk would be awesome in my opinion.
I'd be happy to try and contribute it myself if given a few pointers on what the solution should look like :)

Hey,
sorry for not following up on this earlier. I just tried to use bazel-watcher with Bazelisk (installed on my PATH as ~/bin/bazel) and it seems to work fine.
What exactly is the problem and what needs to be done here? Is this issue about some incompatibility between bazel-watcher and Bazelisk, or is it a feature request to better integrate bazel-watcher functionality with Bazelisk? :)

It might be useful to have bazel-watcher read an env var like IBAZEL_BINARY so you could say IBAZEL_BINARY=bazelisk for those who don't have bazelisk installed as bazel on their machine. Don't think that would require any changes on the bazelisk front though.

Related to https://github.com/bazelbuild/bazel-watcher/issues/339

There is maybe alternative option here - rather than adding a --watch flag to bazelisk itself, we could have bazelisk also manage ibazel installation.
E.g. having a .ibazelversion file, have bazelisk also install a binary called ibazelisk that just downloads and runs the correct version if ibazel, in the same way it runs bazel.
For bonus points, we could also actually install the a symlink for ibazel in the same way it now installs a symlink for bazel, so with npm i -g @bazel/bazelisk you have effectively installed ibazel and bazel with one command.

bump


CUDACC seems not defined on non-jetbrains ide?

https://github.com/apache/incubator-mxnet/blob/67af47c889fb5f54cc2b512ab8b453ff5070a1cc/src/common/cuda_utils.h#L35
The "JETBRAINS_IDE" macro may cause some common ops dose not use CUDNN acceleration?

Sorry, It may be an intellisense error of VS2017.

Has the problem been solved?

It's an error of VS2017. It's not solved.


IT SPEC(INFOSEC), SR Security Architect and Engineering Support.

Where is the rest of the information to allow me to use this property correctly? You know examples and the .text, .replacement. and .wrap, as well as all of the examples of how to use them in some non-trivial way!!!!

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

ID: b777d89e-f8ef-0f2b-d701-397cbca909c9
Version Independent ID: 530e90fb-556d-2ec8-3008-c6039f0a7cb2
Content: Range.Find Property (Word)
Content Source: api/Word.Range.Find.md
Product: word
GitHub Login: @o365devx
Microsoft Alias: o365devx

@poundjd
Thanks for your feedback. The other information is in the left navigation. Please use the filter option at the top of the table of contents to find what you're looking for. Hope this helps.


"Jvm target compatibility should be set to the same Java version" build error

After cloning the project, setting current java to version 18 and building it, I constantly get the error below. Any ideas what can cause it or what could be a solution for it?
* What went wrong:
Execution failed for task ':shared:compileDebugKotlinAndroid'.
> 'compileDebugJavaWithJavac' task (current target is 1.8) and 'compileDebugKotlinAndroid' task (current target is 18) jvm target compatibility should be set to the same Java version.
  Consider using JVM toolchain: https://kotl.in/gradle/jvm/toolchain

Many thanks! I have just pushed un update.
I have specified Java 18 in the shared gradle file.
The desugaring is enabled so that also older Android devices can run the latest Java language feature.


Archive unmaintained repositories

Describe the bug
It looks the following repositories are no longer maintained since the code is moved to this repo:
https://github.com/confidential-containers/peerpodconfig-ctrl
https://github.com/confidential-containers/cloud-providers
https://github.com/confidential-containers/peerpod-ctrl
Can they be archived?
How to reproduce
Observe repository activity and notice no updates in 9+months.
CoCo version information
cloud-api-adaptor HEAD
What TEE are you seeing the problem on
None
Failing command and relevant log output
No response

Sure, I will action this now.

Hey @mythi - we had a chat on the peer pods community call and agreed that we don't need these repos at all, so I've since deleted them to avoid any archived cluster. Thanks for bringing it to our attention.


"micalg" digest property

The micalg digest property is a property of digest, not the digest context, for all known to me digests. It was treated as a part of context because nobody wanted to add an extra field into the EVP_MD structure. My suggestion is moving it from the digest context parameters to digest parameters.

@beldmit I can see no reference to micalg as part of struct evp_md_ctx_st anywhere, going back as far as 1.0.2, so I'm not sure what you mean by the above
Given that the only references I can find to micalg are in some param construction calls, some asn1 writers and a pkcs7 test, I'm assuming this has been resolved
Marking as inactive, to be closed at the end of 3.4 dev, barring further input

The purpose of this PR is eliminating a long case from
https://github.com/openssl/openssl/blob/master/crypto/asn1/asn_mime.c#L158-L234
See also
https://github.com/openssl/openssl/blob/master/crypto/evp/digest.c#L904-907

I'm not sure I see the relation of that code to the statement
The micalg digest property is a property of digest, not the digest context, for all known to me digests.

Are you asking to have the EVP_MD_CTRL_MICALG eliminated and in its place an EVP_MD api added to generate it?

Well. In ideal world yes. But if there is a context-level control function already, we have to keep the API and there is no sense in this refactoring.


Automated cherry pick of #2608: Fixed "There are pending workloads due to capacity maxed by

Cherry pick of #2608 on release-0.7.
#2608: Fixed "There are pending workloads due to capacity maxed by
For details on the cherry pick process, see the cherry pick requests page.
NONE

/assign @tenzen-y


Add type annotations

First draft of some type annotations.
This will make Mypy complain a lot, but it's a first step on making the cobase leverage type annotations benefits.
I'll keep this as a draft for now. One of the decisions to make is whether to add lots of Any for variables with unknown types (or multiple type possibilities), or just to not add the annotation for now.
Ran all tests (both unit and end-to-end with ssh and docker) and everything seems fine (I'm on a macOS with Python 3.10). I also could add tox with the other Python versions to increase test reliability, I don't think I used anything version specific (such as | or TypedDicts and such), so this should be safe.

Thank you for splitting this out @lowercase00! Pushed a few bits and a workflow change so we can start iterating on the mypy warnings.
I'm keen to get this merged sooner than later - to avoid code divergence. I would rather we used Any where types are hard currently to make this process quicker. Then we can revisit all the Any instances one by one off the main branch. Also means the project can benefit from type checking sooner!

@Fizzadar sorry the delay! Sure, just to make sure I understood correctly, should we add Any anywhere Mypy shows warnings to merge asap (making sure there are no mypy warnings at all), and then gradually review each Any, is that correct?

@Fizzadar sorry the delay! Sure, just to make sure I understood correctly, should we add Any anywhere Mypy shows warnings to merge asap (making sure there are no mypy warnings at all), and then gradually review each Any, is that correct?

@lowercase00  nw at all! Yes I think that's reasonable to avoid diverging the branches? Maybe I'm wrong though what are your thoughts?

I would say this is mostly personal preference.
I usually prefer to gradually adopt type annotations leaving the warnings while things are not 100%. There are a few reasons for that: a non-typed repo already has a bunch of warnings (with MyPy on), Any types makes potential errors/bugs silent, and while adapting type annotations some architectural/behavior decisions change (initializing a parameter on the instance vs the class, using property decorators to access more critical attributes and etc), so I tend to prefer to have MyPy warn about potential problems. In the future, if we are working on a branch with MyPy on, it will be inevitable to fix a couple of things here and there (and this would slowly make the codebase reach 100% of annotations), and if we were to ignore MyPy (same effect as using Any), then we might as well turn MyPy off anyways.
Again, I think this is purely personal preferences, and to have Any everywhere and then slowing adding the correct annotation is completely valid as well. I'll be happy to work on this either way!

In the future, if we are working on a branch with MyPy on, it will be inevitable to fix a couple of things here and there (and this would slowly make the codebase reach 100% of annotations), and if we were to ignore MyPy (same effect as using Any), then we might as well turn MyPy off anyways.

Totally agree @lowercase00, let's not go down the Any route! I wonder what's best here for CI - would like to merge this branch in ASAP, maybe we just have mypy generate warnings and pass CI even though it has issues, which gives visibility without blocking any ongoing development.

@Fizzadar I've been thinking about this for a while, here are the options I could think of:
1. Let things as is with MyPy included in the CI
Ignore MyPy warnings and leave MyPy on the CI without breaking anything. This wouldn't allow us for a clean CI with MyPy, neither to implement a "no warnings" policy, but would already make MyPy a part of the application lifecycle. There's always a risk to "normalize" issues on the CI, which can be mitigated by actually dealing with types asap.
2. Let things as is and remove MyPy from CI
Keep MyPy warnings and remove MyPy from the CI (for now). This would help while developing, wouldn't hurt CI, but also wouldn't make the codebase "typed" and would be harder to enforce typing for new contributions.
3. Adopt Any:
This would allow us to have a "no warnings" codebase, add MyPy to the CI flow, and enjoy the benefits of the types we already have. The downside is that it would shadow potential issues, so we would have to actively/manually find places where Any is being used and slowly change it since we would have no warnings.
4. Remove types
When looking at incremental adoption of MyPy type annotations, one of the recommendations (MyPy docs) is to add annotations incrementally, making sure everything everything passes (no warnings) with each new added type. The upside is that it would be explicit: things are typed and predictable where we have types, and we still need to work where there aren't any types.
I'm not sure I would go path 4, especially since State/Host/Inventory is used pretty much everywhere, so I'm not sure how feasible it would be.
About 3, thinking about this as I write, I don't have a good experience using Any everywhere. What I've found when I was using it a lot is that it was tricky: I thought I had MyPy as a safety net to warn me about issues, but at the end of the day there were so much Any around the codebase that a whole bunch of issues were just being ignored, so I had a false sense of security. The same goes for # type: ignore. In my last projects I've been pretty strict about typing, and even going the extra mile to make sure things pass (like using if xyz is None: raise even when we know that wouldn't happen), so I'm probably more biased to keep warnings.
Between path 1 and 2, i have to say I'm not totally sold on either. To be honest I've never given much though to MyPy in the CI flow until you mentioned, but i do agree there's value to it. I'm also not a huge fan of having a whole lot of warnings on the CI, but weighting the pros and cons, I think I would rather to have MyPy already on the CI then to push it any further. Wondering if anybody else have experience adopting typing on larger codebases (I don't).
Again, your call, I'll be happy to help regardless of the path taken!

Leaning towards 1 I think. Agree that none of the options are ideal, so suggest we go with 1 and then iterate on fixing up the warnings ASAP.
At least having the warnings, even if not blocking CI, will make it possible to (manually) identify changes that increase the warning / issue count.

@Fizzadar let me know if you want more changes in here before the final review

Hi, based on this PR, I tried to add some types to get_facts – https://github.com/simon04/pyinfra/commit/dd02a7ab62494b38002b90fe4924f9f98208c633
The idea is that the following example snippet can infer the type of linux_distribution and the existence of the key "release_meta"
https://github.com/Fizzadar/pyinfra/blob/2d4792eba3233c77a298d86e9638434688e3a1c5/examples/dnf.py#L18-L19

Found a bit of time to sink into this! Have now fixed all the mypy errors, I attempted to add types where possible. There are a few ignore statements in there, some are validish (setting attribute on a function kind of thing), others indicate areas where the code should be rewritten such that there's no reliance on conflating types together.
There's also a bunch of horrible looking nested/union types which would also make excellent candidates for refactoring. Now this is passing it can be merged, and subsequent PRs can iterate on types to expand the coverage. Running mypy in strict mode demonstrates the effort required here (😅): Found 1437 errors in 109 files (checked 120 source files).
Huge THANK YOU @lowercase00 for working on this and iterating through the issues faced, I think this is in a great spot now and am thrilled to be adding type checking to the CI workflow 🎉

@simon04 that change looks fantastic, I'm about to merge this in if you wouldn't mind PR'ing that against 2.x that would be amazing!

100% agree, definitely a challenge worth persuing!


Scroller is arranged incorrectly

Describe the bug

Scroller isn't being a good citizen in the layout process which causes unexpected behavior when doing things like setting a Horizontal/VerticalAlignment.

In the picture above the left side is using Scroller and the right side is using the old ScrollViewer, but otherwise the markup is identical.  You can see that the ActualHeight of the Scroller ends up being the MaxHeight value that is set on the parent Border rather than the size of the content.  This causes its content to appear incorrectly positioned even though it is Top aligned.
Steps to reproduce the bug

Steps to reproduce the behavior:
Install a recent prerelease package and use this markup.
    <Grid>
        <Grid.ColumnDefinitions>
            <ColumnDefinition/>
            <ColumnDefinition/>
        </Grid.ColumnDefinitions>

        <Border MaxHeight="380"
                VerticalAlignment="Center"
                BorderBrush="Purple"
                BorderThickness="12"
                Grid.Column="0">
                <muxcp:Scroller ContentOrientation="Vertical"
                                VerticalAlignment="Center">

                    <StackPanel VerticalAlignment="Top"
                                BorderBrush="Lime"
                                BorderThickness="4"
                                Margin="12">
                        <Rectangle Width="296" Height="96" Fill="DarkRed" HorizontalAlignment="Right"/>
                        <Rectangle Width="296" Height="96" Fill="Red" HorizontalAlignment="Left"/>

                    </StackPanel>
                </muxcp:Scroller>
        </Border>

        <Border MaxHeight="380"
                VerticalAlignment="Center"
                BorderBrush="Purple"
                BorderThickness="12"
                Grid.Column="1">
                <ScrollViewer VerticalAlignment="Center">

                    <StackPanel VerticalAlignment="Top"
                                BorderBrush="Lime"
                                BorderThickness="4"
                                Margin="12">

                        <Rectangle Width="296" Height="96" Fill="DarkRed" HorizontalAlignment="Right"/>
                        <Rectangle Width="296" Height="96" Fill="Red" HorizontalAlignment="Left"/>

                    </StackPanel>
                </ScrollViewer>
        </Border>

Expected behavior

The left side looks like the right side.
Version Info

NuGet package version:  Microsoft.UI.Xaml 2.1.190319001-prerelease

Windows 10 version
Saw the problem?

October 2018 Update (17763)
Yes

Device form factor
Saw the problem?

Desktop
Yes

@regisb want to take a look?

Hey Jevan, yeah feel free to assign to me.
-Régis at the fabulous Ocean Shores
Get Outlook for iOShttps://aka.ms/o0ukef

From: Jevan Saks<EMAIL_ADDRESS>Sent: Friday, April 12, 2019 11:25:21 AM
To: Microsoft/microsoft-ui-xaml
Cc: Regis Brid; Mention
Subject: Re: [Microsoft/microsoft-ui-xaml] Scroller is arranged incorrectly (#551)
@RBridhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRBrid&data=02|01|regisb%40microsoft.com|8c46fc5cc4314ef7552308d6bf7439ca|72f988bf86f141af91ab2d7cd011db47|1|0|636906903237628335&sdata=u8d9GWTgYbT9XCHV3t6OSodxivN9hv1um0hTAFzugv8%3D&reserved=0 want to take a look?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fmicrosoft-ui-xaml%2Fissues%2F551%23issuecomment-482675877&data=02|01|regisb%40microsoft.com|8c46fc5cc4314ef7552308d6bf7439ca|72f988bf86f141af91ab2d7cd011db47|1|0|636906903237638331&sdata=ZWuMxs2%2FzLiRwAxQrQz0xdhauXvk2%2BW%2FmuHbagoq4ss%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAi2p98LoApBjb9tel4IDHbI7jQZ30v2-ks5vgM-RgaJpZM4csiRc&data=02|01|regisb%40microsoft.com|8c46fc5cc4314ef7552308d6bf7439ca|72f988bf86f141af91ab2d7cd011db47|1|0|636906903237638331&sdata=NyGVlR3syUhmmhLHlvkzTMMsAGMl6hvERAjrF87ti6k%3D&reserved=0.


serve

Open by mistake, can't find a way to remove it, please delete this issue

@jomendez  please click close issue button


import_set not working under Python 3.4.10 with xlsx files

I'm currently testing my code in a virtualenv using Python3. It works just fine when I run it under Python2 but when I open an xlsx using tablet.import_set using Pythong3, I get this stack error below.
python3
[unsignedzero 02:07:45]> import tablib
[unsignedzero 02:07:47]> tablib.import_set("temp_test_output.xlsx")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/core.py", line 1005, in import_set
    (format, stream) = detect(stream)
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/core.py", line 996, in detect
    if fmt.detect(stream):
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/formats/_xlsx.py", line 31, in detect
    openpyxl.reader.excel.load_workbook(stream)
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/packages/openpyxl3/reader/excel.py", line 85, in load_workbook
    raise e
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/packages/openpyxl3/reader/excel.py", line 81, in load_workbook
    _load_workbook(wb, archive, filename, use_iterators)
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/packages/openpyxl3/reader/excel.py", line 114, in _load_workbook
    new_ws = read_worksheet(archive.read(worksheet_path), wb, sheet_name, string_table, style_table)
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/packages/openpyxl3/reader/worksheet.py", line 116, in read_worksheet
    fast_parse(ws, xml_source, string_table, style_table)
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/packages/openpyxl3/reader/worksheet.py", line 81, in fast_parse
    source = _get_xml_iter(xml_source)
  File "/home/unsignedzero/.local/py_env/lib/python3.4/site-packages/tablib/packages/openpyxl3/reader/worksheet.py", line 43, in _get_xml_iter
    return StringIO(xml_source)
TypeError: initial_value must be str or None, not bytes

This looks to be a bug using the xml reader?

Seems like I had a buggy pip install and it work just fine, don't know what happened but at least it's fixed.

Problem still persists.... ... Had a fluke and though it was fixed.

Same troubles here. Did you found a solution?

I'm getting the same exception under Python 3 importing a CSV file. I think a clue about how to solve it in this StackOverflow anwer

import_set requires a stream input, not a path string.
https://tablib.readthedocs.io/en/stable/api/#tablib.import_set
I plan to suggest a patch to allow paths and file-like objects in input, but this is not the subject of this issue.

@claudep actually, the problem persists even with stream input or rather the problem is as a result of the byte stream input. If you check the StackOverflow question linked by @tadeo you will see that the issue is that StringIO expects a str object in Python 3 and not a byte object. So when a byte stream is passed, it fails. The Python docs suggest using the io module in Python 3

Are you able to reproduce the issue with a recent version? Did you try with master code?

@claudep I reproduced with version 0.12.1

Please try with the latest, 0.14.0


Refactor code

See new-ui branch

[x] #9
[x] #10
[x] #17
[x] #19
[x] #16

Closed without test odudex binaries because he will change the way beta releases will published (it will not more inside his repo).


feat: SQLAlchemy and alembic integration

Adds Python integration for pgai that allows defining and managing vectorizers through SQLAlchemy models and Alembic migrations. The integration provides a declarative interface for vector embeddings and automated migration support.
class BlogPost(DeclarativeBase):
    content_embeddings = VectorizerField(
        source_column="content",
        embedding=EmbeddingConfig(
            model="text-embedding-3-small",
            dimensions=768
        )
    )

# Semantic search query
results = (
    session.query(BlogPost.content_embeddings)
    .order_by(
        BlogPost.content_embeddings.embedding.cosine_distance(
            func.ai.openai_embed('text-embedding-3-small', 'search query')
        )
    )
    .limit(5)
)

Features:

Declarative SQLAlchemy field for vectorizer configuration
Alembic operations for creating and dropping vectorizer
Support for migration generation for vectorizer changes

Known missing features:

Downgrading of migrations
Custom sqlalchemy functions for ai.xxx functions
Comparison of existing vectorizer does not work yet! (Working on it)
Chunking configuration is not separated for character and recursive splitter (will fix)
Scheduling config only supports default and specific scheduling not no scheduling right now (will fix)

How to review:

Have a look at some tests and the docs to understand how it is supposed to work. If you don't like this or have questions, stop here and leave a comment of what to improve/change
Implementation wise:

sqlalchemy/__init__.py: Contains the VectorizerField that defines sqlalchemy embedding models and registers the configuration for alembic.
alembic/autogenerate.py: Contains the comparison functions as well as the python code generation.
alemic/operations.py: Contains the SQL to be executed (and some alembic boilerplate).
configuration.py: Contains dataclasses that map to all the sql parameters for create_vectorizer which act as the input for both the VectorizerField as well as the alembic create_vectorizer and drop_vectorizer operations.

I made the autogen work but have now separated this rather large PR in 3 smaller ones:
#265 #266 and #267
Closing this PR.


Save message_key in Grape::Exceptions::Validation

This way you can make better API errors, that have a stable message_key, alongside a translated message.

This needs tests and a CHANGELOG entry.
Also take a look at https://github.com/ruby-grape/grape/pull/1203, maybe we can do something similar?

Bump @Darksecond

Closing in favor of https://github.com/ruby-grape/grape/pull/1227.


E2E accuracy "RuntimeError: Eager run failed" with PyTorch 2.5

The following E2E tests fail:

GPTJForCausalLM
GPTJForQuestionAnswering

for at least the following scenarios:

E2E accuracy huggingface, training, float32, LTS, PyTorch 2.5
E2E accuracy huggingface, training, float32, Rolling, PyTorch 2.5

Error:
RuntimeError: XPU out of memory, please use `empty_cache` to release all unoccupied cached memory.
...
RuntimeError: Eager run failed

Note that with IPEX the error is slightly different:
RuntimeError: XPU out of memory. Tried to allocate 256.00 MiB (GPU 0; 64.00 GiB total capacity; 63.22 GiB already allocated; 63.70 GiB reserved in total by PyTorch)
...
NotImplementedError: Eager model failed to run

@riverliuintel, @Stonepia do you observe the same error?

Hi @vlad-penkin , Yes, we have the same error and we have a tracker at https://github.com/intel/torch-xpu-ops/issues/701.
Besides, formerly these test don't OOM may because we didn't implement some XPU backend kernels (Thus they fall to CPU kernels). After we implemented more kernels, the  GPU memory may not be sufficient, thus OOM.


update readme file with compiler method for all language

Yeah, why not. Would you like to take some?

will try

label it to hacktoberfest


Query by block/hash/rcpt ctxdata

Pull Request Test Coverage Report for Build 583034186

0 of 35 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 27.493%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

services/indexes/avax/reader.go
0
35
0.0%

Files with Coverage Reduction
New Missed Lines
%

services/indexes/avax/reader.go
2
10.27%

Totals

Change from base Build 582841682:
-0.1%

Covered Lines:
1533

Relevant Lines:
5576

💛  - Coveralls


Request a Copy sends same email regardless of accept or reject (in pre-7.6)

Describe the bug
May have been accidentally caused by #8833
The bug is simple. If you reject a request for copy, then the requestee is still sent the acceptance email (with no files attached). There should obviously be two separate email templates
To Reproduce
Steps to reproduce the behavior:

Use main code with #8833 installed & latest main of dspace-angular.

It's easiest to test if you set mail.server.disabled=true in your local.cfg on the backend. This lets you see the full text of the mail in dspace.log

Submit a request for copy for a restricted item.
In the request email, click on the link to respond.
Reject the response.  Notice that the email sent says "You request for copy....has been approved"

Expected behavior
It appears that the code in RequestItemEmailNotifier is always using the request_item.granted email template... and there's no corresponding request_item.rejected email template.  This is obviously incorrect.
See code at https://github.com/DSpace/DSpace/blob/main/dspace-api/src/main/java/org/dspace/app/requestitem/RequestItemEmailNotifier.java#L178-L179
Ideally, there should be a check similar to:
String emailTemplateName = ri.isAccept_request() ? "request_item.granted" : "request_item.rejected"

Then we'd need to create a request_item.rejected email template
Related work
May have been caused by #8833

@mwoodiupui : Just found this bug today.  It appears to be caused by your changes in #8833 (though it may have existed prior, not sure).  Any chance you could fix it quickly for us?
If not, I may be able to get to it next week.  I think the fix is rather simple... see "Expected Behavior" section above.


[Request] Check URL before submit if exist

Hello,
I have a user who keeps abusing the site by submitting a specific URL multiple times each day.
How can I check the URL before submitting if it already exists or not, and simply showing an error message "URL has been submitted before. Check this URL: http://shorten.li/bHNv1P" ??
Thank you.

Thank you for your suggestion. This feature is on the plan, but I haven't finished it yet because I've been busy these days. I hope that I am able to complete the feature in late December, so stay tuned!

This feature has been added, but is turned off by default, as turning it on will increase kv quota consumption. If you need to use this feature, make sure to change config.unique_link=true in index.js.
https://github.com/xyTom/Url-Shorten-Worker/blob/779653193d066a7b1267c2288a46d36b171a1c9e/index.js#L5


Use Makefile syntax to define defaults

Makefile uses ?= to define a variable that is set if the variable on the left side is not defined. This also fixes the binary name when running make build.
Additionally, ensure that the docs directory exists before updating.
Before:
$ make
--- Building ionosctl via go build ---
built /home/mv/Projects/CORE/ionosctl/builds/ionosctl__
DONE

After:
make
--- Building ionosctl via go build ---
built /home/mv/Projects/CORE/ionosctl/builds/ionosctl_linux_amd64
DONE

This also is broken is the release/v6 branch. Should I open an extra PR to v6 branch or do you cherry-pick commits?

This also is broken in the release/v6 branch. Should I open an extra PR to v6 branch or do you cherry-pick commits?

Please open an extra PR for the release/v6 branch as well.


brother-hll6400dw: init at 3.5.1-1

add printer driver for Brother HL-L6400DW
limitation: works only with socket protocol, does not work with ipp protocol
this is documented in the driver's meta.longDescription
i was not 100% sure how to package this
because brother distributes this as 2 packages: cupswrapper and driver
currently, the cupswrapper is the user-facing package
{
  nixpkgs.config.allowUnfreePredicate = pkg: builtins.elem (lib.getName pkg) [
    "brother-hll6400dw-lpr"
  ];

  services.printing.drivers = [
    pkgs.brother-hll6400dw
  ];

and the actual "driver" (mostly the filter executable rawtobr3)
is a dependency of the cupswrapper
anyway, works for me
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

user was banned https://github.com/NixOS/moderation/blob/main/moderation-log.md


photo dates not visible on mobile, difficult to know the most up-to-date version

Needs https://github.com/openfoodfacts/openfoodfacts-dart/pull/892

off-dart v3.5.0 is not public yet, but we can start to play with the uploaded timestamp using a local off-dart up-to-date package.
Of course several options are available regarding dates:

sorting pictures by date (which is probably already the case, but it's not visibly obvious)
displaying the date on each picture (on top? below with a Text?)
specific icon for out-of-date pictures (on top)
listing pictures with "last week", "last month", "last year", "older" list subtitles
strongly preventing users from using out-of-date pictures when trying to refresh an existing picture

If we put an extra "wrong!" icon on older images (e.g. more than 1 year old), the order of images gets implicit and the "good" and fresh pictures too.
@teolemon @jusdekiwi Would that be good?

1
2
3

Maybe a "warning" icon would be better since old doesn't mean wrong.
Also as shown in your previous message, it seems like clicking on the old images would delete them. That's not what we want, is it?
Last thing, would it be possible to tell if one "old" image is obsolete or? Because it would be useful to be able to tell that the product has not changed. Otherwise we would have to take new pictures all the time.
Thank you for your work anyway!! :)

@jusdekiwi It's just a matter of icon: a "warning" icon would be more appropriate indeed. No specific action is intended for the moment anyway (i.e. no "image deleting").
This issue is only about upload timestamps. There is no obsolete flag triggered by a power-user or a producer in Smoothie.
The only "obsolete" flag we use (and actually we currently don't) is "OBSOLETE_NUTRITION_IMAGE", that is true if states contain either 'en:nutrition-photo-to-be-selected' or 'en:photos-to-be-uploaded'. Doesn't look very reliable IMHO.
Perhaps we could work on the "upload timestamp" aspect only, then in another issue be more specific about "obsolete" images if needed.

[ ] I would add the explicit date on the photo, like we do on the web.
[ ] I would avoid radius on photos if they are not spaced. Edge to edge without radius if they are stuck together (like the Google Photos pattern), or with a radius, and clear padding.
[ ] I would take the occasion to allow multi-select on long-press for image moderation (no other usecase) on the whole gallery
[ ] We could add moderator only features like moving images to another product, or image deletion
[ ] We should add support for image reporting using Nutri-Patrol (first as a deeplink to the Nutri-Patrol using a web-view, then using the API) for everyone
[ ] We should refrain from using icons only to indicate stale status. The idea of separating image by their quarter, with headers could be interesting (Q1-2024, Q2-2019…) and more value adding

Web reference

Old mockup I did (without headers)

@monsieurtanuki on the webapp it's kind of okay since i can see the dates by hovering the pictures with my cursor.
However, on the mobile app, I can't see them.
What I would like is having a visible timestamp over or under the pictures uploaded in the mobile app when selected an existing picture. I've made an example below. I am not sure about the warnings, maybe we would need more opinions of contributors.

@jusdekiwi Something like that?

Yeah, that would be really good. I'm not sure about the exclamation marks for the old ones though. Maybe a light red/pink background for the text would be more obvious.


Can not register Touch Id fido key in Safari browser(Mac M1 chip)

Describe the issue:
Can not register fido key in Safari browser.
https://user-images.githubusercontent.com/39077751/165265524-597ca5ae-7caf-4b67-820b-a3f272811b2f.mov
How to reproduce:

Login to myaccount
Navigate to the security page
Click add button in Security key/biometric section
Allow add touch ID
Insert fingerprint

Expected behavior:
Device should be registered sucessfully.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: 5.12.0-alpha20
OS: Mac
Database: MSSQL
Userstore: JDBC

Optional Fields
Related issues:

Suggested labels:

This is a known issue in the platform authenticators with Safari browser and MacOS with M1 chip. External USB keys are working fine. The original issue is reported here [1] and another discussion can be found in fido-dev mailing list [2]. As per comment [3] this will be fixed in a future release.
[1] https://bugs.webkit.org/show_bug.cgi?id=224042
[2] https://groups.google.com/a/fidoalliance.org/g/fido-dev/c/BHYtIkLTZbM
[3] https://bugs.webkit.org/show_bug.cgi?id=224042#c8

This is fixed with macOS Monterey 12.3.1 update. Tested with macOS Monterey 12.3.1 (M1 pro) and safari. Refer [1].
[1] https://bugs.webkit.org/show_bug.cgi?id=224042#c13
@aaujayasena can you also verify with M1 chip?

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[FN] Upgrade to netcore30

Lots of breaking changes
https://docs.microsoft.com/en-us/aspnet/core/migration/22-to-30?view=aspnetcore-3.0&tabs=visual-studio
https://docs.microsoft.com/en-us/dotnet/core/compatibility/2.2-3.0

Just wondering what are the direct benefits of upgrading?
We had some discussion here
https://github.com/block-core/blockcore/issues/8
It would be nice to know of any additional benefits.

Just wondering what are the direct benefits of upgrading?
We had some discussion here
block-core#8
It would be nice to know of any additional benefits.

This article explains it pretty well: https://hackernoon.com/features-to-build-better-applications-with-net-core-3-dc320740e0a9

Implemented in SFN <IP_ADDRESS> https://github.com/stratisproject/StratisFullNode


Witness private key may be lost if wallet crashes

cli_wallet does not save the private key for a new witness until the witness is registered on chain; this is wrong.

what's more, even after the witness is registered on the chain and the key is imported into the wallet, it doesn't trigger an automatic save or backup (like import_key does).  I just lost a few witness keys because I really like to hit ctrl-c when I make a typo.

This issue was moved to bitshares/bitshares-core#71


Deleted redundant code in MIQ Policy Controller / MIQ Actions

Check for generated flash messages no longer needed with the code fix in https://github.com/ManageIQ/manageiq/pull/6525
New begin/rescue block in ci_processing/process_elements generates flash messages

@dclarizio @h-kataria Redundant code removal, please review.

LGTM


drivers: can: Convert drivers to new DT device macros

Convert CAN drivers from:
DEVICE_AND_API_INIT -> DEVICE_DT_INST_DEFINE
DEVICE_GET -> DEVICE_DT_INST_GET

Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

@galak Did you mean to include both CAN and DMA drivers in this PR?

@galak Did you mean to include both CAN and DMA drivers in this PR?

No, need to fix that.


DS 2654 common config object

Replace the new DSpace() usage to retrieve the serviceManager / configurationService by a factory call to DSpaceServicesFactory.getInstance().getServiceManager() & DSpaceServicesFactory.getInstance().getConfigurationService()
This will limit the creation of "new DSpace()" objects in the codebase.

Thanks @KevinVdV !  I had to make minor updates in my branch to get it to pass unit tests, but the core code was all correct. Merged


Support ES version 7.7.0

Issue #, if available:
https://github.com/opendistro-for-elasticsearch/job-scheduler/issues/57
Description of changes:

Bump ODFE version to 1.8.0
Bump Elasticsearch version to 7.7.0
Bump JDK version to 14
Bump Gradle version to 6.4
Bump nebula.ospackage version to 8.3.0
Change the parameter of JobSchedulerPlugin#createComponents & SampleExtensionsPlugin#createComponents (https://github.com/elastic/elasticsearch/pull/52604)
Change the RestHandlers to use new way to register with the RestController (https://github.com/elastic/elasticsearch/pull/51950)
Fix an issue with plugin installation order when using Elasticsearch testClusters to do integration test, the issue probably caused by this change (https://github.com/elastic/elasticsearch/pull/54852)
Update the requirement of JDK 14 in README and workflow files
Update Release Notes

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

Thanks zengyan-amazon and qreshi for the tricky fix for the testClusters issue.


QSM Challenge 2.0

Thanks very much for compiling this list!
Additional data for the QSM challenge 2.0 could be added:
http://www.listsnu.org/qsm-challenge/

Awesome, thanks! I'll parse through the details and add this to the list soon!


[Bug]: Incorrect Lap Time Calculation

🐞 Bug Report
📋 Description
The application is not calculating lap records correctly when recording lap times. Instead of logging individual lap durations, it records the total elapsed time, leading to inaccurate analysis and visualization of lap data.
⚙ Steps to Reproduce
Steps to reproduce the behavior:

Start the timer in the application.
Record multiple laps using the lap button.
Observe the recorded lap times in the lap analysis section.

✔ Expected Behavior
Each lap should be recorded with its individual duration, allowing for accurate analysis of lap times.
❌ Actual Behavior
The application records the total elapsed time instead of separate lap durations, resulting in incorrect lap analysis and visualization.
📷 Screenshots
https://github.com/user-attachments/assets/aa98b7a2-171f-43f0-9ebf-f8fa7a39f9e1
🖥 Environment (if applicable):

OS: Windows, macOS
Browser: Chrome, Safari

🔧 Additional Information
This bug significantly affects the analysis features of the application, making it difficult for users to track their performance accurately.

@param-code please assign it soon

@param-code hey param. I found a new and more problematic bug in the application  while solving this issue. The problem is the counter of this application is NOT updating with time and the counter is pretty much static. I found the component which is causing this problem and its the particle effect you have running on the page. The effect keeps re-rendering the page which leads to memory overload and issues in rendering much important component of the page which is the counter.
https://github.com/user-attachments/assets/3d9d9fa3-88a4-4fee-b213-0b1f3401cc30

As of now I would recommend removing this component from the page

https://github.com/user-attachments/assets/a7944feb-30f2-4849-9daa-b1c1cd746147
So what should I do? should I remove it or I just pr what my issue was about

@param-code hey param. I found a new and more problematic bug in the application while solving this issue. The problem is the counter of this application is NOT updating with time and the counter is pretty much static. I found the component which is causing this problem and its the particle effect you have running on the page. The effect keeps re-rendering the page which leads to memory overload and issues in rendering much important component of the page which is the counter.
render.mp4

As of now I would recommend removing this component from the page

Screen.Recording.2024-10-18.at.7.09.15.PM.mov
So what should I do? should I remove it or I just pr what my issue was about

Remove it if you think it will solve the issue.


Solution for secrets in DevOps?

I noticed that my project with secrets which I committed to DevOps does not maintain the secrets file when pulling the project to another PC.  What solutions are available to allow secrets to be accessible for projects on DevOps?

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

ID: 9c99993c-c983-6fda-6c3a-ce4ec8c1d5db
Version Independent ID: ba688ac2-e90e-fce5-f60d-40c2553c3efd
Content: Safe storage of app secrets in development in ASP.NET Core
Content Source: aspnetcore/security/app-secrets.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: scaddie

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.


Fixed username bug.

Hey, I didn't make an issue for this. But I wanted to fix the delete bug the other day, where the user doesn't actually get deleted from the local storage because of inconsistencies of the API response (whether a user is ranked or not).
So to fix this whenever the cards are created in the createCard method, I put all the username in the correct format and inject it into the card's dataset. So whenever you need the username in the correct format you can reference the card's dataset with the username property. I then used this value in the deleteUser function so it actually removes the user from the local storage.
Please have a look and see if you are okay with the fix. Thanks

Thanks for contributing, check other issues for contributing more👍🏼


Update wording.

Change "Where" to "Description" in table and info winow.
elm-format web/elm/Eqdash.elm.

I really don't like elm-format formatting, too many whitespace but meh, I'll give it a try.


My Resources Filtering: Search by Filenames

This is not NFIE Required.  See #376 for my suggestions on required search fields.  I am open to Filename being added to the list as a "nice to have" feature.

Filtering has been revamped in the My Resources page.  Assigning to @horsburgh to decide if there is still an enhancement here to consider (which should be filed as a separate issue) or whether this can just be closed.

@dtarb - I don't think this is something we should support.  The filenames are not exposed on the My Resources page. We should enable filter on the metadata items one can actually see on the page. Objections?

Lets close.


feat: add delay seconds

Description

Motivation and Context

Breaking Changes

How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[ ] I have executed pre-commit run -a on my pull request

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


(bugzilla 6874) disable the broken optimized path and fall back to append

(bugzilla 6874) disable the broken optimized path and fall back to appender
Also add a unittest.
This doesn't properly resolve the issue, of course.
http://d.puremagic.com/issues/show_bug.cgi?id=6874
I'm not so confident of the unittest's quality. Someone should have a look at that. I feel like it could be much simpler. And I suspect that the array length could take up more than 1 byte on another compiler/machine, but I couldn't find anything on that.

@schveiguy?

I'll take a look on Monday. Can't right now

Definitely realloc is not expecting pointers that are not at the base of the memory block.  My thoughts are first that realloc should just fail if the pointer was not returned by gc.malloc.  This is easily checked with the data already obtained by findPool.  That would be a fix to druntime.
Then, insertInPlace's optimized path should be changed to avoid realloc.  It shouldn't be using realloc anyway.
Using appender should be doable (which would merge the optimized and non-optimized case), but we need a stack-created version.  The current heap-allocated version is too expensive for this simple operation.
However, this temporary fix is fine, the only thing I'd suggest is simply wiping out the realloc code.  It's not viable, and leaving it with a version notation looks like it could be reenabled in the future...

followed @schveiguy's suggestions

@schveiguy Would you do another review pass through the changes based on your comments.  Sounds like this one is ready and fixes an important bug.


Create a consolidated docker image for CI

add scripts/docker_run

this passes ${DB} for switching db variant, default is hsqldb

add scripts/gradle to unify test dependency startup

all test scripts call into this script

deletions

scripts/ldap/install-ldap.sh
=> replaced by ldap setup in Dockerfile
run-integration-tests.sh
=> use scripts/docker_run scripts/integration-tests.sh
run-unit-tests.sh
=> use scripts/docker_run scripts/unit-tests.sh

[#168655249]

This PR doesn't address any possible changes/improvements/consolidations that could be made in CI tasks/configuration.

I wonder if the following files should just be copied in instead of echoed in:

/bin/start-mysql
/bin/start-postgresql
/etc/ssl/ca.info
/etc/ssl/ldap01.info
/etc/ssl/certinfo.ldif
/bin/start-slapd

Also I tried to build the image but I seem to be missing two files:

ldap_db_init.ldif
copy_of-uaa_src_main_resources_ldap_init.ldif

should we move those files into the uaa/ci/image folder?
PS: I did find ldap_db_init.ldif  in 4 different places

uaa/out/production/resources/ldap_db_init.ldif
uaa/build/resources/main/ldap_db_init.ldif
uaa/build/libs/WEB-INF/classes/ldap_db_init.ldif
uaa/src/main/resources/ldap_db_init.ldif

@sjolicoeur uaa/ci/image/ldap_db_init.ldif was removed in https://github.com/cloudfoundry/uaa/pull/1107/commits/a0b2ee71f75b568ceadeb06391c97503df9465df (but the reference is still in the Dockerfile)

@sjolicoeur uaa/ci/image/ldap_db_init.ldif was removed in a0b2ee7 (but the reference is still in the Dockerfile)

yeah, is it still needed though?


Disable LangSmith

Hi, I'm getting the following errors logged when running eval:
Connection error caused failure to post http://localhost:1984/runs  in LangSmith API. Please confirm your LANGCHAIN_ENDPOINT. ConnectionError(MaxRetryError("HTTPConnectionPool(host='localhost', port=1984): Max retries exceeded with url: /runs (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x2d0c00d90>: Failed to establish a new connection: [Errno 61] Connection refused'))"))
Connection error caused failure to patch http://localhost:1984/runs/8703dda5-af15-4f7c-adeb-9636a07f847b  in LangSmith API. Please confirm your LANGCHAIN_ENDPOINT. ConnectionError(MaxRetryError("HTTPConnectionPool(host='localhost', port=1984): Max retries exceeded with url: /runs/8703dda5-af15-4f7c-adeb-9636a07f847b (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x2d0bdd640>: Failed to establish a new connection: [Errno 61] Connection refused'))"))

How can I disable LangSmith to prevent these?

unfortunatly there is no fix as of now and it is an annoying set of error messages. We will change that shortly
for now, you can disable the logs if you want
import logging
logging.getLogger('langchain.callbacks.tracers.langchain').setLevel(logging.WARNING)
logging.getLogger('urllib3.connectionpool').setLevel(logging.ERROR)

will put a long term solution in the mean time

Thanks a lot! 🙏🏻

I also faced the same issue and using the given suggestion to disable the log for now. Thanks @jjmachan

applogies for this annoyance @ranjanritesh89 but we will also have a permanent fix with #394 too


Installation problem in Lumen

Hello,
I installed the laravel-module component in a clean installation of lumen, I configured the autoloading. Then I followed the steps described in the following URL: https://nwidart.com/laravel-modules/v3/lumen
After all the installation I created my first module with the command "php artisan module: make blog", after creating it I receive the following error:
In BlogServiceProvider.php line 73:

   Class 'Config' not found

Line 73 of the BlogServiceProvider.php file contains the following:
$ this->loadViewsFrom(array_merge (array_map (function ($path) {
             return $path.'/modules /blog';
         }, \Config::get ('view.paths')), [$sourcePath]), 'blog');

Can anybody help me?
Regards, and thank you very much

I'm not understanding what you're asking me. The error code is generated by the command mentioned above. The version of lumen is 5.5.
Thank you very much

I can't help you without more information.
This is not related to this package, please refer to the lumen docs.

The error I get after installing the laravel-module package, and generating a new module with the command available in your package. That's why I post it here. The same I explained wrong, my English is pretty bad ...
I get the error after executing the command:
php artisan module: make
It is in the generated module where said error appears.

Yes but it's not related to this package.
Your issue is with php and lumen usage.

Hello,
I installed lumen, then installed the laravel-module I also followed the steps described at https://nwidart.com/laravel-modules/v3/lumen
run command "php artisan module:make User" as per instructions module generated
after this trying to create migrations by following doc https://nwidart.com/laravel-modules/v3/advanced-tools/artisan-commands getting error
In Application.php line 176:
Class 'Modules\User\Providers\UserServiceProvider' not found
Can anybody help me?
Thanks in advance


Issue with Node.js 4.1.0, sass-lint 1.2.2, and Webpack 1.12.2

I got the following error trying to use sasslint-loader as preloader in my Webpack config file using the suggested Webpack configuration over the .js files.
Syntax "js" is not supported yet, sorry
Hash: 8f21d01e87f9ca7b3987
Version: webpack 1.12.2
Time: 634ms
    + 1 hidden modules

ERROR in ./lib/src/index.js
Module build failed: TypeError: Cannot read property 'toString' of undefined
    at module.exports (/project_path/node_modules/sass-lint/lib/groot.js:25:89)
    at Function.sassLint.lintText (/project_path/node_modules/sass-lint/index.js:36:13)
    at lint (/project_path/node_modules/sasslint-loader/index.js:21:25)
    at Object.module.exports (/project_path/node_modules/sasslint-loader/index.js:99:7)

Thanks for the report. This was probably one of the few things I punted on while waiting for some fixes in sass-lint, but I will revisit this loader as a whole.

@dmachat if there's anything in sass-lint holding you back get an issue up and we'll look into it as a priority.. Integrations like this are really great and we'll be looking to support here as best as we can.

@davegomez that's my shoddy doc work. You need to limit this loader to scss or sass files, eg https://github.com/alleyinteractive/sasslint-loader/blob/master/example/webpack.config.js#L18. I've also updated the README.
@DanPurdy caught up today and it doesn't look like there's anything actually blocking me on your end anymore. sasstools/sass-lint#62 is still an issue for stray tabs, but I've removed them from tests for now and can live without them.

Srry @dmachat  but I'm not getting even an error output now.
I have tried all the loader options and this is my webpack config now:
/* eslint indent: 0 */
// jscs:disable requireCamelCaseOrUpperCaseIdentifiers

const path = require('path');
const ExtractTextPlugin = require('extract-text-webpack-plugin');
const BrowserSyncPlugin = require('browser-sync-webpack-plugin');
const merge = require('webpack-merge');

const TARGET = process.env.npm_lifecycle_event;
const LIB = path.resolve(__dirname, 'lib/src');
const BIN = path.resolve(__dirname, 'bin');

const common = {
  entry: LIB,
  resolve: {
    extensions: ['', '.js']
  },
  output: {
    path: BIN,
    filename: 'js/main.js'
  }
};

if (TARGET === 'start' || !TARGET) {
  module.exports = merge(common, {
    devtool: 'eval-source-map',
    module: {
      preLoaders: [
        { test: /\.js?$/, loaders: ['eslint', 'jscs'], include: LIB },
        {
          test: /\s[a|c]ss$/,
          exclude: /node_modules/,
          loader: 'sasslint'
        }
      ],
      loaders: [
        {
          test: /\.js?/,
          loaders: ['babel'],
          include: LIB
        },
        {
          test: /\.scss?$/,
          loader: ExtractTextPlugin.extract('style-loader',
            'css-loader?sourceMap' +
            '!autoprefixer-loader?browsers=last 2 version' +
            '!sass-loader?outputStyle=expanded&sourceMap&sourceMapContents')
        }
      ]
    },
    sasslint: {
      configFile: './.sass-lint.yml',
      emitError: true,
      emitWarning: true,
      failOnError: true
    },
    stats: {
      children: false
    },
    plugins: [
      new ExtractTextPlugin('bin/css/main.css', {
        allChunks: true
      }),
      new BrowserSyncPlugin({
        host: 'localhost',
        port: 3000,
        server: { baseDir: ['./bin'] },
        files: ['bin/*.html']
      })
    ]
  });
}

The console is clear while I get some errors and warnings using the sass-lint CLI tools.
I don't understand what i'm doing wrong here.
Also, this is how I'm downloading the module since the NPM support doesn't work yet:
"sasslint-loader": "git://github.com/alleyinteractive/sasslint-loader",


New package: Sentinel v0.2.0

Registering package: Sentinel
Repository: https://github.com/mhudecheck/Sentinel.jl
Created by: @mhudecheck
Version: v0.2.0
Commit: a155b01cf49c35b7afa927f8e8a64786e872b636
Reviewed by: @mhudecheck
Reference: https://github.com/mhudecheck/Sentinel.jl/issues/1
Description: This is a Julia library for processing ESA Sentinel 2 satellite data.
Release notes:

New features

Superseded by #49082, I suppose.
@mhudecheck note that if you do not change the version number every time, you update the same pull request instead of opening multiple times that then have to be closed.


Update @types/markdown-it to aligned and pinned version

🧹 Tech Debt
The @types/markdown-it updates were disabled due to an issue in prosemirror-markdown, see https://github.com/ProseMirror/prosemirror-markdown/issues/124
When addressed we can re-enable the updates and switch from @types/markdown-it v13 to v14 (which we actually end up relying on v14  of the actual markdown-it library transitively due to prosemirror-markdown).
Additional discussion here: https://github.com/ni/nimble/pull/2318#discussion_r1700853278

https://github.com/ProseMirror/prosemirror-markdown/issues/124 was merged but blocked on release. After merge the pinned version of @types/markdown-it should be updated to align with the version of markdown-it used by prosemirror-markdown.
Unfortunately we should not enable renovate as we need to manually align the @types/markdown-it version to what is used by our version of prosemirror-markdown so it should be a manual task added to https://github.com/ni/nimble/issues/1747


chore: change doc comment for quickstart command

Description
Adds the dfx quickstart to the dfx help menu.
Fixes # (issue)
https://dfinity.atlassian.net/browse/SDK-998

Looks good (shortens the help text), but the bug in question seems to be out of date.  The command does show help in dfx 0.15.1:
$ dfx --version
dfx quickstadfx 0.15.1
$ dfx quickstart --help
Use the `dfx quickstart` command to perform initial one time setup for your identity and/or wallet. This
command can be run anytime to repeat the setup process or to be used as an informational command, printing
information about your ICP balance, current ICP to XDR conversion rate, and more.

If setup tasks remain, this command is equivalent to running `dfx identity set-wallet` (for importing) or `dfx
ledger create-canister` followed by `dfx identity deploy-wallet` (for creating), though more steps may be
added in future versions.

Usage: dfx quickstart [OPTIONS]
...
$ dfx --help
The DFINITY Executor

Usage: dfx [OPTIONS] <COMMAND>

Commands:
...
  deps        Pull dependencies and integrate locally
  diagnose    Detects known problems in the current environment caused by upgrading DFX, and suggests commands to
                  fix them. These commands can be batch-run automatically via `dfx fix`
  fix         Applies one-time fixes for known problems in the current environment caused by upgrading DFX. Makes
                  no changes that would not have been suggested by `dfx diagnose`
  extension   Manages the dfx extensions
  generate    Generate type declarations for canisters from the code in your project
  identity    Manages identities used to communicate with the Internet Computer network. Setting an identity
                  enables you to test user-based access controls
  info        Get information about the replica shipped with dfx, path to networks.json, and network ports of
                  running replica
  ledger      Ledger commands
  new         Creates a new project
  ping        Pings an Internet Computer network and returns its status
  quickstart  Use the `dfx quickstart` command to perform initial one time setup for your identity and/or wallet.
                  This command can be run anytime to repeat the setup process or to be used as an informational
                  command, printing information about your ICP balance, current ICP to XDR conversion rate, and more
  remote      Commands used to work with remote canisters
  schema      Prints the schema for dfx.json
...
...

Thank you for reviewing Eric. I will close this PR as it looks like the issue is resolved in the latest.


arrow-schema-52.2.0 breaks build

Describe the bug
Version 52.2.0 of arrow-schema breaks the CI build as it deprecated the method Schema::all_fields, which is treated as an error by the cargo clippy CI step.
To Reproduce
Run cargo clippy
Expected behavior
Build succeeds
Additional context
No response

I see this was already done, and my failing PR just needs to be rebased.


Cleanup awsapi leftovers

Just a couple of things that were still in the code :)

I swear I did remove them, probably some merge issue. LGTM, merging now.

If if comes back we will just kill it again :)


Unit tests for parse_json_markdown function

Type of change

[x] Unit tests
[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] Optimization
[ ] Documentation Update

Description
Unit tests for parse_json_markdown function. Checks correct inputs and also some incorrect ones.
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

It is part of unit testing job

/assign @bparees

/lgtm
/approve


AWS Config Rule Compliance Overlay

Feature name
AWS Config Rule Compliance Overlay
Is your feature request related to a problem? Please describe.
Users want to view the current compliance of resources to help understand the environment visually where issues may lie with configuration.
Describe the feature you'd like to see implemented
The ability to turn off and on an visual indicator, maybe a red line or similar to quickly visualisally grasp non-compliant resources.

This is a great idea.
I'm going to add your feedback to this ticket as i think the two are related and close this one. Is that ok with you?

Also, if you upvote the issue Matt has linked to it will help us with prioritisation.

We've evaluated this request and added this to our backlog but it is not prioritized in the immediate roadmap for the solution.