Single Sign on with ADALiOS, loginWithProvider:token: not working

In the Azure Mobile Apps client for iOS (https://github.com/Azure/azure-mobile-apps-ios-client), the MSClient class provides a method
-(void)loginWithProvider:(nonnull NSString *)provider
token:(nonnull NSDictionary *)token
completion:(nullable MSClientLoginBlock)completion;
I am trying to use this method with a token retrieved from the ADALiOS. I am following this tutorial (https://azure.microsoft.com/en-us/documentation/articles/app-service-mobile-ios-aad-sso/). This tutorial is before the November update to Mobile Apps
Is it possible to use AD single sign on with Azure Mobile Apps?

Yes it should be possible.  I'd start by reviewing this updated AAD setup post here: https://azure.microsoft.com/en-us/documentation/articles/app-service-mobile-how-to-configure-active-directory-authentication/
What error are you getting back from that call?
I know there is a potential issue with the audience not matching.  I think the workaround that is to make sure the ResourceURI referenced from ADAL is the same as the App ID URI for the web application registration in AAD. Which should just be the URL of your site root (HTTPS scheme).
@mattchenderson can you comment?

this the error message I get
com.Microsoft.WindowsAzureMobileServices.ErrorDomain Code=-1302 "You do not have permission to view this directory or page." UserInfo={NSLocalizedDescription=You do not have permission to view this directory or page.}

@phvannor  Thanx for providing the information above.
Following the instructions in the link i am running into another issue.  If I have a mobile app configured with express authentication I am not able to see it as one of the options for under "Permissions to other Apps"

@phvannor I have followed the instructions at https://azure.microsoft.com/en-us/documentation/articles/app-service-mobile-how-to-configure-active-directory-authentication/ and still get the same error message

you checked the audience is correct as well?
You should have more details in your server logs, can you go to settings -> diagnostic logs, and turn on app logs at verbose.  Then open the streaming logs blade (you may need to add it via add tiles).  Once that says connected, try logging in again and see what gets output.

I had not checked the audience.
The streaming log do refer to the audience
JWT validation failed: IDX10214: Audience validation failed.
How do you set/configure the audience

I put some notes above, I copied from a different conversation: "make sure the ResourceURI referenced from ADAL is the same as the App ID URI for the web application registration in AAD. Which should just be the URL of your site root (HTTPS scheme)."
Unfortunately, I haven't personally tried this scenario so I can't clarify, if I get a chance I do mean to walk through it manually myself.  If it doesn't make sense hopefully @mattchenderson can fill us in, or you may try asking on StackOverflow/etc.
This is a temporary bug with EasyAuth on Web Apps, but this way will continue to work as well once the fix roles out as wel.

 Audiences: 'https://'. Did not match:  validationParameters.ValidAudience: '' or validationParameters.ValidAudiences: 'http://'.
2015-12-02T00:14:10  PID[9624] Information Sending response: 401.71 Unauthorized
So from above looks like it is failing because the validation parameter is expecting http but it is receiving https

This what I had to make following changes do...
In the portal I changed the APP ID URI  =  http://[app url]
In the mobile app I changed the resourceURI  =  http://[app url]
This did the trick.

Looks like there is another bug in the portal that caused you to get http:// as well.  I've raised that issue internally for review as well.

Closing here, since the fix needs done on the EasyAuth pipeline/etc.  I don't have a link to that work since  its not on GitHub.  @mattchenderson can we do a blog post once the fix goes live?

Same error https versus http in audiences. Workaround does not work because I get an application not found, even after several restarts ...


Empty quasi has no valid 007 value

Here's an uninitialized value that leaks through from Perl 6 land:
$ bin/007 -e='macro moo() { return quasi {}; }; say(moo())'
Use of uninitialized value @arguments of type Any in string context.
Methods .^name, .perl, .gist, or .say can be used to stringify it to something meaningful.
  in method call at /home/masak/ours/007/lib/_007/Runtime.pm (_007::Runtime) line 152

I think this code is at fault:
method run($runtime) {
    for $.statements.elements -> $statement {
        my $value = $statement.run($runtime);
        LAST if $statement ~~ Q::Statement::Expr {
            return $value;
        }
    }
}

It returns a value if there were any statements to run, but doesn't explicitly return (and so we get Nil) if there were none.

The value I would expect is None, by the way.


[CMake] Correct dependencies for clang headers components

When building swift as a part of LLVM (as opposed to standalone) the components
related to swift headers should explicitly depend on the clang target to produce
those. On LLVM 9 and up, that would be clang-resource-headers and on lower
versions it would be clang-headers. It is important that we check for
clang-resource-headers first because clang-headers refers to something
different in LLVM 9 and up.
cc @compnerd @gottesmm @Rostepher

@swift-ci please smoke test

@swift-ci please smoke test


Accept command-line arguments to module?

 Issue by alphapapa
Thu Feb 16 16:56:51 2017
Originally opened as https://github.com/codelucas/newspaper/issues/337

I've been using the https://github.com/buriy/python-readability module and am interested in moving to newspaper.  A nice feature that python-readability has is the ability to call the module directly from the command line, like:
python -m readability.readability -u "http://example.com"

Would it be possible to add support for this to newspaper?  Thanks.

added in v 0.9.2


Confetti Gun and the Funny Gunny shop

As requested, I have added a room on the third floor with the "Funny Gunny Shop." The player senses that someone else is in the room even before entering and must decide whether to enter or not. Once they choose "Yes," they encounter the mysterious merchant who randomly sells one of the following items: "Laser Sword," "Gum Cigarettes," and "Confetti Gun." If the player chooses to buy the "Confetti Gun" when prompted by the merchant, it will be added to their inventory. The other items cannot be added.

Alright , so let's add the text: "This item cannot be added, when choosing the following items: "Laser Sword," "Gum Cigarettes".

Looks good so far. I have tested your code and everything seems to be working. As @Salt-is-leaving suggested, you could add something like "this item is currently not available" to the items that are currently not usable. The rest of your code seems good, maybe add some more comments to the second while-loop, so we can see what is going on there at the first glance.


Shadcn Accordion components should use details and summary tags for better SEO

I wanted to use Accordion  component for my website, And when i inspected the code on Accordion component that i got from shadcn i noticed that shadcn was not using details and summary tags for Accordion,
So I think shadcn should use re-defined tags that html has for better seo, Let me know what you think.

This would be more of a enhancement then bug

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


[FEATURE REQ] Ensure all track 2 packages have package-info.java

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Ensure package-info.java is in all track 2 java packages mainly to provide documentation for the package when javadocs are generated. package-info.java can also be used for declaring package level annotations.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Every java package created as part of track 2 library should contain package-info.java file with appropriate documentation.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
None
Additional context
Add any other context or screenshots about the feature request here.
None
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

Done - https://github.com/Azure/azure-sdk-for-java/pull/3875


Relicense under dual MIT/Apache-2.0

Why?
The MIT license requires reproducing countless copies of the same copyright
header with different names in the copyright field, for every MIT library in
use. The Apache license does not have this drawback, and has protections from
patent trolls and an explicit contribution licensing clause. However, the
Apache license is incompatible with GPLv2. This is why Rust is dual-licensed as
MIT/Apache (the "primary" license being Apache, MIT only for GPLv2 compat), and
doing so would be wise for this project. This also makes this crate suitable
for inclusion in the Rust standard distribution and other project using dual
MIT/Apache.
How?
To do this, get explicit approval from each contributor of copyrightable work
(as not all contributions qualify for copyright) and then add the following to
your README:
## License

Licensed under either of
 * Apache License, Version 2.0 ([LICENSE-APACHE](LICENSE-APACHE) or http://www.apache.org/licenses/LICENSE-2.0)
 * MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT)
at your option.

### Contribution

Unless you explicitly state otherwise, any contribution intentionally submitted
for inclusion in the work by you shall be dual licensed as above, without any
additional terms or conditions.

and in your license headers, use the following boilerplate (based on that used in Rust):
// Copyright (c) 2015 t developers
// Licensed under the Apache License, Version 2.0
// <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT
// license <LICENSE-MIT or http://opensource.org/licenses/MIT>,
// at your option. All files in the project carrying such
// notice may not be copied, modified, or distributed except
// according to those terms.

And don't forget to update the license metadata in your Cargo.toml!
Contributor checkoff

[x] @tschottdorf

I have no license headers. Do I need any? I've added to Cargo.toml and the README. Let me know whether that's enough and I'll close & release.

I'm not comfortable making recommendation to leave out the license headers as I've received transitive legal advice to do it and the apache license appendix recommends it. But lots of people seem to think they are unnecessary.
On January 9, 2016 6:00:23 AM EST, Tobias Schottdorf<EMAIL_ADDRESS>wrote:

I have no license headers. Do I need any? I've added to Cargo.toml
and the README. Let me know whether that's enough and I'll close &
release.

Reply to this email directly or view it on GitHub:
https://github.com/tschottdorf/hlc-rs/issues/1#issuecomment-170225485

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


how to get cpuid?

What exactly do you mean by cpuid?
If you mean the x86 instruction: hwinfo uses it on x86 machines to get processor information and provides them in the ˋhwinfo::CPUˋ class.

yes, in x86， this code could get the cpu id:
void getCpuId(int cpuInfo[], int infoType)
{
#if 0
#ifdef Q_OS_WIN
// #include "intrin.h"
//__cpuid(cpuInfo, infoType);return;
#if _MSC_VER
int excValue = 0;
_asm {
mov edi, cpuInfo;
mov eax, infoType;
mov ecx, excValue;
cpuid;
mov [edi], eax;
mov [edi+4], ebx;
mov [edi+8], ecx;
mov [edi+12], edx;
}
#else
asm volatile
(
"cpuid"
: "=a"(cpuInfo[0]), "=b"(cpuInfo[1]), "=c"(cpuInfo[2]), "=d"(cpuInfo[3])
: "a"(infoType)
);
#endif
#endif
#endif
}
QString getCpuId2()
{
QString cpu_id;
int cpuInfo[4] = {0};
getCpuId(cpuInfo, 1);
QString str0 = QString::number((quint32)cpuInfo[3], 16).toUpper();
str0 = str0.rightJustified(8, '0');
QString str1 = QString::number((quint32)cpuInfo[0], 16).toUpper();
str1 = str1.rightJustified(8, '0');
cpu_id = str0 + str1;
return cpu_id;

}
but how to get the cpuid in linux?

Looks like this question is not related to hwinfo. Please consider using c++/x86/assembler related forums for this kind of questions.
However cpuid.h implements a platform independent cpuid call.


For steps applied several times, personalized brief  is only accessible for the first column

The personalized brief,  for steps applied several times (when there are several columns), is only accessible for the first column.
library(pointblank)

c_agent <- create_agent( tbl= small_table) %>%
  col_vals_regex(columns = vars(b, f),
                 #col_vals_regex(columns = vars(gnad_fin_url), # CORRECT
                 regex = "[a-z]",
                 step_id = "my_regex",
                 label = "This is my regex",
                 brief = "The current field must have at least one letter"
  )  %>%
  interrogate()

c_agent # tooltip over "This is my regex" (first column in report table) is only visilbe for the first step of the series

c_agent$validation_set$step_id[1]
c_agent$validation_set$label[1]
c_agent$validation_set$brief[1] #!is.NA (this is correct)

c_agent$validation_set$step_id[2]
c_agent$validation_set$label[2]
c_agent$validation_set$brief[2] # is.NA (this is the bug)

get_agent_x_list(c_agent)$briefs # second value is.NA (same bug)
c_agent$validation_set$brief # second value is.NA (same bug)

c_agent$validation_set$label # No NAs

The following is an extended short example with three groups of rules, and a temporary workaround to help correct the bug:
library(pointblank)

c_agent <- create_agent( tbl= small_table) %>%
  col_vals_regex(columns = vars(b, f),
                 #col_vals_regex(columns = vars(gnad_fin_url), # CORRECT
                 regex = "[a-z]",
                 step_id = "my_regex",
                 label = "This is my regex",
                 brief = "The current field must have at least one letter"
  )  %>%
  col_vals_regex(columns = vars(b),
                 #col_vals_regex(columns = vars(gnad_fin_url), # CORRECT
                 regex = "\\d",
                 step_id = "anoter_regex_1_column",
                 label = "Another regex for just 1 column",
                 brief = "The b field must have at least one number"
  )  %>%
    col_vals_regex(columns = vars(b, f),
                 #col_vals_regex(columns = vars(gnad_fin_url), # CORRECT
                 regex = "\\d",
                 step_id = "anoter_regex",
                 label = "Another regex",
                 brief = "The current field must have at least one number"
  )  %>%
  interrogate()

c_agent # tooltip over "This is my regex" (first column in report table) is only visilbe for the first step of the series

c_agent$validation_set$step_id[1]
c_agent$validation_set$label[1]
c_agent$validation_set$brief[1] #!is.NA (this is correct)

c_agent$validation_set$step_id[2]
c_agent$validation_set$label[2]
c_agent$validation_set$brief[2] # is.NA (this is the bug)

get_agent_x_list(c_agent)$briefs # every second value is.NA for steps with several "sub-steps" (same bug)
c_agent$validation_set$brief # same bug

c_agent$validation_set$label # No NAs

# Start of the workaround
steps_labels_briefs <- data.frame(step_id = c_agent$validation_set$step_id, label=c_agent$validation_set$label, old_brief = c_agent$validation_set$brief, new_brief = NA)

steps_labels_briefs$step_group <-
  sub(pattern = "^(.+)(\\.\\d{4})$", replacement ="\\1",  steps_labels_briefs$step_id, perl = T)

# First row (step) of each group of steps
steps_labels_briefs_0001 <-
  steps_labels_briefs[grepl(pattern = "^(.+)(\\.0001)$", steps_labels_briefs$step_id),]

# Assign the correct brief value (workaround)
steps_labels_briefs$new_brief <- steps_labels_briefs$old_brief
for (c_step_group in steps_labels_briefs_0001$step_group){
  steps_labels_briefs$new_brief[steps_labels_briefs$step_group == c_step_group] <-
    steps_labels_briefs_0001$old_brief[steps_labels_briefs_0001$step_group  == c_step_group]
}
c_agent$validation_set$brief <- steps_labels_briefs$new_brief

# Check that the workaround worked
get_agent_x_list(c_agent)$briefs # correct
c_agent$validation_set$brief #  correct (side note: much faster than previous line of code)

Thanks for catching this. This can definitely be fixed!


Fixed a bug on MacOS related to loading data from NSUserDefaults

Fixed a bug on MacOS that wasn't loading stored data properly from NSUseDefaults

Bug fix in Apple SaveDataHelper, reviewed.


Percentage tested column - Total Tests/Total population *100

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Seeing a column named percentage tested which is equal to
Percentage Tested  = Total Tests / Total population *100
Describe alternatives you've considered
Calculating it myself
Additional context
It would be nice to see that data. as i think it is important for checking state of districts and how they are doing.

Use the per million button (1M) to get tests normalized by population. It's essentially the same information


Cannot read property node of undefined

(This issue tracker is only for bug reports or feature requests, if this is neither, please choose appropriate channel from http://bluebirdjs.com/docs/support.html)
Please answer the questions the best you can:

What version of bluebird is the issue happening on?
3.5.2

What platform and version? (For example Node.js 0.12 or Google Chrome 32)
React Native 0.57

Did this issue happen with earlier version of bluebird?
Unknown

(Write description of your issue here, stack traces from errors and code that reproduces the issue are helpful)
When starting my React Native app with remote debugging on, I get this error from Bluebird as it attempts to check the process' version number:

bluebird.js:5637:35

var version = process.versions.node.split(".").map(Number);

When I disable remote debugging, the app works as expected. Remove the package that depends on Bluebird also re-enables the remote debugging feature.

Same here!

Hey @mlostekk i'm facing the same issues trying to debug RN trough VSCode :(
How did you managed to solve this issue?
Best!

See this commit.
https://github.com/petkaantonov/bluebird/pull/1576/commits/28683161d1fcb21524942e6ae44a2e2cfdfc6074
in the release from NPM you have to adjust this part in the file node_modules/js/browser/bluebird.js near the very end (in my case line number 5636)

@mlostekk thank you so much!
it's funny (and scary) that the only solution for this now is editing the node_module...
what would be the library using the outdated version of bluebird resulting into this exception? maybe we could find it and pull request the fix for it for a proper solution...
thx!!!

Well, its already fixed, but @petkaantonov needs to trigger a release and seem to be not available at the moment


[Pending Stalls] New stalls for a polling place with an already approved stall show up as 'NEW' on the pending stalls list, but in edit/comparison mode when you go to process them

e.g. Stall 4089 (Rye Community Hall)
The interface's behaviour showing edit mode when you click on it is correct, but maybe the list of stalls should be something other than 'NEW'?
AB#58

Closing and merging into https://github.com/keithamoss/demsausage/issues/1335


Bump fastg-parser to v0.6.1

Add additional workflow to avoid spades overlaps doubled.
Fix #101

Coverage decreased (-2.3%) to 68.859% when pulling 5712996b976ad083d1e15421684c8757643c8319 on hotfix/bump-fastg-to-v0.6.1 into a26162dece54986be27b1cb8b8ee4265b035f450 on master.


Investigate not holding hard reference to rebased starting point when regenerating

Description
When regenerating states we hold a hard reference to states supplied as a rebased starting point.  This significantly increases memory usage but ensures tasks.
See https://github.com/PegaSysEng/teku/pull/3007#discussion_r507929023 for discussion and things to consider.

We aren't seeing significant numbers of state regenerations in process now so this is less of a priority.

Going to close this as unnecessary.


[Server 2016] Script hangs and never completes

On Server 2016 I'm running the script as instructed but it never completes. See output below:

Git repo is cloned to c:\windows\temp
WindowsDefenderATPLocalOnboardScript.cmd was downloaded from security.microsoft.com via Settings->Onboarding and placed in c:\windows\temp\mdefordownlevelserver
*md4ws.msi downloaded from security.microsoft.com via Settings->Onboarding and placed in c:\windows\temp\mdefordownlevelserver

Install line (PowerShell, as administrator):
& "c:\windows\temp\mdefordownlevelserver\Install.ps1" -RemoveMMA '<MMA ID REDACTED>' -OnboardingScript "C:\windows\temp\mdefordownlevelserver\WindowsDefenderATPLocalOnboardingScript.cmd"
Log output (it just hangs a minute in):
[PCNAME:09 22/08/11T13:48:50.107 Install.ps1:503] [Net.ServicePointManager]::SecurityProtocol updated to 'Ssl3, Tls, Tls12'
[PCNAME:09 22/08/11T13:48:50.622 Install.ps1:613] Tracing session 'install-PCNAME-10.0.14393.5246' started.
[PCNAME:09 22/08/11T13:48:50.794 Install.ps1:629] Removing cloud workspace <ID REDACTED>...
[PCNAME:09 22/08/11T13:48:51.138 Install.ps1:637] Workspace <ID REDACTED> removed.
[PCNAME:09 22/08/11T13:48:51.203 Install.ps1:657] BuildLabEx: 14393.5246.amd64fre.rs1_release.220701-1744
[PCNAME:09 22/08/11T13:48:54.047 Install.ps1:660] There are 27 KBs installed.
[PCNAME:09 22/08/11T13:48:54.047 Install.ps1:663] KB5016058 was installed on 08/10/2022 00:00:00
[PCNAME:09 22/08/11T13:48:56.956 Install.ps1:286] 'HKCR' PSDrive created(script scoped)
[PCNAME:09 22/08/11T13:48:57.800 Install.ps1:961] Handle 3248 opened over C:\windows\temp\mdefordownlevelserver\md4ws.msi
[PCNAME:09 22/08/11T13:49:00.550 Install.ps1:972] 797261D33CB9160051DE3F2AFA691C63A5078ED8724A56A3322E4DE846DDCFA3 C:\windows\temp\mdefordownlevelserver\md4ws.msi
[PCNAME:09 22/08/11T13:49:00.597 Install.ps1:996] Running C:\Windows\system32\msiexec.exe /i C:\windows\temp\mdefordownlevelserver\md4ws.msi /lvx*+ C:\Users\<NAME REDACTED>\AppData\Local\Temp\8fe4e3b3-7114-4144-b31d-0fd0fcc888b2.log /quiet in C:\Windows\system32 ...
[PCNAME:09 22/08/11T13:49:05.727 Install.ps1:996] Command "msiexec.exe /i C:\windows\temp\mdefordownlevelserver\md4ws.msi /lvx*+ C:\Users\<NAME REDACTED>\AppData\Local\Temp\8fe4e3b3-7114-4144-b31d-0fd0fcc888b2.log /quiet" run for 00:00:04.7665445
[PCNAME:09 22/08/11T13:49:05.743 Install.ps1:998] install successful.
[PCNAME:09 22/08/11T13:49:05.743 Install.ps1:1020] Invoking onboarding script C:\windows\temp\mdefordownlevelserver\WindowsDefenderATPLocalOnboardingScript.cmd
[PCNAME:09 22/08/11T13:49:05.759 Install.ps1:1028] Running C:\Windows\system32\cmd.exe /c C:\windows\temp\mdefordownlevelserver\WindowsDefenderATPLocalOnboardingScript.cmd in C:\Windows\system32 ...
#30+ minutes pass, nothing happens past this point.

The temp log file states it successfully installed the "Microsoft Defender for Endpoint" application but seems to have ceased all activity afterwards. The "Sense" service did not start and errors out when I attempt to manually start it.
What could the issue be? Task manager is quiet and there's little indication of what is happening.

Hi, you are using WindowsDefenderATPLocalOnboardingScript.cmd which is interactive (requires user input). Please use the onboarding script for use with group policy instead as per documentation, thanks!
For the affected machine, running the group policy script should finish the onboarding step and the machine should be good to go.

The .cmd script isn't interactive if the pause/confirm statements are removed. Our org is not in a position to deploy this via GPOs and we need the script method to function.

Hi, please do not modify the onboarding scripts. The script marked for use with group policy can be used as-is, even if not deployed through group policy. Do not use the “local” onboarding script for anything other than manual, interactive deployment.


kaizen: Update config for kaizen

This was supposed to have been part of the big v1.0-prepare PR but I fat-fingered it out of the commit when fighting git problems co-operatively with Micha.
Signed-off-by: Tim Bray<EMAIL_ADDRESS>

Codecov Report
Base: 95.83% // Head: 95.83% // No change to project coverage :thumbsup:

Coverage data is based on head (8647ff5) compared to base (251dbe7).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #177   +/-   ##
=======================================
  Coverage   95.83%   95.83%
=======================================
  Files          17       17
  Lines        2067     2067
=======================================
  Hits         1981     1981
  Misses         62       62
  Partials       24       24

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Showcase Submission for 107-sandpiles

Thank you Paul Keller for your contribution! A member of the Coding Train team will review it shortly.

Sandpiles in Plain HTML and JS
Paul Keller

Lovely!


CLI is not sending CELO when you try to submit a Governance Proposals

Package
@celo/celocli
Have you ensured that all of these are up to date?

[X] the package you are using

What version of the package are you on?
@celo/celocli/4.0.0 darwin-arm64 node-v20.11.0
What command or function is the bug in?
celocli governance:propose
Operating System
macOS (Apple Silicon)
Describe the bug
When you try to submit a new governance proposal using this command: celocli governance:propose --jsonTransactions ./mainnet.json --deposit 10000e18 --descriptionURL https://github.com/celo-org/governance/blob/main/CGPs/cgp-XXXXmd --from XXXXXXXXXXXXXXXXXX --privateKey=XXXXXXXXXXXXXXXXXX
you got an error that says "Too small deposit"

And If you check, The cli is not sending any celo with the propose transaction for some reason.

relates to #154

Hi @juanjgiraldoc
i have found the source of the issue and am working on fix

reopened until we have outside confirmation that
<EMAIL_ADDRESS>fixes the issue
@juanjgiraldoc please try and let me know
<EMAIL_ADDRESS>

Seems working, thanks @aaronmgdr

Closing based on @juanjgiraldoc's feedback ✅


Keyboard shortcuts are changed en masse

Type: Bug
I've been using vscode for a long time. Now my keyboard shortcuts are changed en masse. For instance, ctrl+` no longer opens a new terminal. Almost all shortcuts are changed or removed.
I looked for settings and did not find anything related. Also looked for any extesions that might have changed these, again, no chance.
So:

How did this happen? I'm using vscode on windows+linux+mac, but I've not turned on sync.
How to fix this?

VS Code version: Code 1.91.1 (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:08:12.169Z)
OS version: Linux x64 6.9.8-200.fc40.x86_64
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 9 7950X 16-Core Processor (32 x 400)

GPU Status
2d_canvas: unavailable_softwarecanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onopengl: disabled_offrasterization: disabled_softwareraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_softwarewebgpu: disabled_off

Load (avg)
0, 0, 0

Memory (System)
30.55GB (13.76GB free)

Process Argv
--enable-features=UseOzonePlatform --ozone-platform=wayland --crash-reporter-id aae31f76-74e1-48d7-beaf-b93ecd51fa3f

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
wayland

Extensions (16)

Extension
Author (truncated)
Version

ng-template
Ang
18.1.2

markdown-mermaid
bie
1.23.1

mermaid-markdown-syntax-highlighting
bpr
1.6.6

dart-code
Dar
3.92.0

flutter
Dar
3.92.0

vscode-eslint
dba
3.0.10

gitlens
eam
15.2.0

vscode-solution-explorer
fer
0.8.6

copilot
Git
1.213.0

copilot-chat
Git
0.17.1

csdevkit
ms-
1.8.14

csharp
ms-
2.34.12

vscode-dotnet-runtime
ms-
2.0.9

remote-containers
ms-
0.375.1

vs-code-prettier-eslint
rve
6.0.0

svelte-vscode
sve
108.5.4

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonnoceb:30805159
asynctok:30898717
pythonregdiag2:30936856
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
chatpanelt:31048053
dsvsc021:30996838
da93g388:31013173
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecation:31068756
dwnewjupyter:31046869
impr_priority:31094925
pythonrstrctxtcf:31093870

Hi :-)
Thanks for filing this issue!

Could you provide at your keybindings.json file contents? (invoke command "Preferences: Open Keyboard Shortcuts (JSON)" to open it)

Could you please try the following:

2.1. Run the command "Developer: Toggle Keyboard Shortcuts Troubleshooting" (you can, for example, run it from the Command Palette).
This will activate logging of dispatched keyboard shortcuts and will open an output panel with the corresponding log file.
2.2. Press exactly the keybinding you desire
2.3. Check what VS Code detects
If you are having trouble understanding the output, please feel free to paste here and we'll be happy to help ☺

Before I go into your suggestion @ulugbekna
I uninstalled VsCode, removed ~/.vscode and ~/.config/Code, and reinstalled VsCode, which fixed the problem.
After logging in with copilot, things were messed up again.

// Place your key bindings in this file to override the defaults
[
]

This happened twice:
The problem is observed, I invoke "Developer: Toggle Keyboard Shortcuts Troubleshooting" and then then shortcuts are back! First time I thought it can be because of something else. This time I only did that and I'm back on track!

OK. It happened again. This time Developer: Toggle Keyboard Shortcuts Troubleshooting did not fix it.
2024-07-18 16:56:24.903 [info] [KeybindingService]: / Soft dispatching keyboard event
2024-07-18 16:56:24.903 [info] [KeybindingService]: \ Keyboard event cannot be dispatched
2024-07-18 16:56:24.903 [info] [KeybindingService]: / Received  keydown event - modifiers: [ctrl], code: ControlLeft, keyCode: 17, key: Control
2024-07-18 16:56:24.903 [info] [KeybindingService]: | Converted keydown event - modifiers: [ctrl], code: ControlLeft, keyCode: 5 ('Ctrl')
2024-07-18 16:56:24.903 [info] [KeybindingService]: \ Keyboard event cannot be dispatched in keydown phase.
2024-07-18 16:56:25.123 [info] [KeybindingService]: / Soft dispatching keyboard event
2024-07-18 16:56:25.123 [info] [KeybindingService]: | Resolving ctrl+[Backquote]
2024-07-18 16:56:25.123 [info] [KeybindingService]: \ No keybinding entries.
2024-07-18 16:56:25.123 [info] [KeybindingService]: / Received  keydown event - modifiers: [ctrl], code: Backquote, keyCode: 192, key: Unidentified
2024-07-18 16:56:25.123 [info] [KeybindingService]: | Converted keydown event - modifiers: [ctrl], code: Backquote, keyCode: 91 ('`')
2024-07-18 16:56:25.123 [info] [KeybindingService]: | Resolving ctrl+[Backquote]
2024-07-18 16:56:25.123 [info] [KeybindingService]: \ No keybinding entries.
2024-07-18 16:56:25.304 [info] [KeybindingService]: + Ignoring single modifier ctrl due to it being pressed together with other keys.

I'm still having this problem. It's more of a transient nature. I've been unable to guess what are the exact conditions (or even if there are).
Can someone help? This really causes hiccups while working.

Hi, I've just had a very similar issue. I think the root cause (and what we have in common) is wayland. Recently I managed to switch from running VSCode via XWayland fallback (current default) to native Wayland.

--enable-features=UseOzonePlatform --ozone-platform=wayland
by the way you don't need these, you can export ELECTRON_OZONE_PLATFORM_HINT=auto in your .bashrc

It looks like when you switch layouts (I use Dvorak + two cyrillic layouts) some state breaks and keybinding malfunctions. I also have Ctrl+` not opening the terminal when it happens.


Illegal offset type in isset or empty in taxonomy.php when using Timber\Menu

Hi,
I am using Timber(1.9.4) installed with Composer, in a Wordpress project version 5.1.1, with ACF plugin last version.
In my recent projects, I got a Wordpress warning which is:
"Warning:  Illegal offset type in isset or empty in wp-includes/taxonomy.php" in a custom post type content with a custom taxonomy.
When I comment the line in the foreach loop to retrieve all the menus, the warning disappears
 foreach (get_registered_nav_menus() as $key => $value) { $context[$key] = new Timber\Menu( $key ); }
Does somebody have the same issue?
Thanks!

Hi
Does this problem also exists in new versions of Timber? There were some changes with the menu so maybe it was fixed.
Also do you have any menus created?
And finally I can't find the timber-menus file? Are you sure about the version that you are using?

Sorry palmiak, I edited my comment.
I updated timber to version 1.9.4.
We use a foreach loop to retrieve all menus, and add each menu to the global Timber context.

I updated Timber to version 1.10.
When I add menu to the global context like below
foreach (get_registered_nav_menus() as $key => $value) { 		$context[$key] = new Timber\Menu( $key ); 	}
in some page, it throws an error from the wp core taxonomy.
If I comment this line
$context[$key] = new Timber\Menu( $key );
there is no more error.

Fist of all - could you paste the error code here?
Could you show me what will this code output:
var_dump( get_registered_nav_menus() );

It will should show you all the $keys. For example let's say it will output:

menu_1
menu_2
menu_3

next define all those menu manually - without the loop:
$context['menu_1'] = new Timber\Menu( 'menu_1' );
$context['menu_2'] = new Timber\Menu( 'menu_2' );
$context['menu_3'] = new Timber\Menu( 'menu_3' );

next go to the page where the error appears and start commenting all the menus one after another to see which one is causing the error.

Hi Palmiak, sorry for my late reply, just came back form holidays.


add contributing guidelines to readme

with 0 merge conflicts™

Good job :+1:

Wonderful. Thank you.


Add intermediate size to hf_olmo

intermediate_size is a commonly used field in transformers. We should support it in hf_olmo. The current vLLM implementation for hf_olmo does a try-except to handle the absence of this field.

Not tested but this is a pretty straightforward change and I'll fix anything that breaks

Closing as I will probably need to add a few more things.


fixes issue #1389

Fixes #1389
Changes: [Add here what changes were made in this issue and if possible provide links.]
Screenshots for the change:

@amitiwary999 Please follow the commit style guideline.  Please change the commit name as well.
See this

@the-dagger Please take a look


Add BuildingInsideVisualStudio condition check

Only exclude files for VS bug when actually building with VS.

Hi @JelleHissink - could you please also give this a check when you have a gap?


fix(ui): don't use Buffer for FNV hash

Fixes #11764
Motivation

Buffer requires a polyfill in client-side JS

#11628 upgraded to Webpack v5, which no longer auto adds Node polyfills

which ended up breaking 2 year old code from #6925

this seems to be the only one we use (as far as I can tell) -- and this code quite literally copies server-side Go code, so makes sense that it may use server-side libraries

add comments in the code about where this is copied from so that it is easier to track (I had to go through the blame + commit history)

Modifications

use charCodeAt to extract an int from a character of a string instead of converting the string to a Buffer

was inspired by looking at some JS FNV implementations and stumbled upon some lines like this: https://github.com/schwarzkopfb/fnv1a/blob/2c5ee2889e42a6bd38c64a29b227eaf9e048e40d/index.js#L22

also re-enable the tslint rule at the end of the code

Verification

unit tests for FNV hashing still pass, so think the implementation is equivalent and we're good!

Misc Notes
I decided not to replace the FNV hash implementation with a library as basically all of them are unmaintained and this change only ended up being a few LoC. Some of the libraries are also quite large (implementing many different algorithms and are not tree-shakeable), or have build issues, or use BigInt (which is not supported in older browsers... although those are all EoL and I wouldn't personally support such old browsers, but still something to consider).

But I wonder why we should calculate podName by nix hash in frontend? We may derive podName from nodeId...
nodeId=steps-dflj9-1988631110 podName=steps-dflj9-whalesay-1988631110

the logic, as far as I can tell, is that when the name is too long it needs to be shortened, so a hash is used. The back-end PR is #6712 which also has a few back-links to other issues and PRs. Could also ask JP if he remembers
I've definitely had some long names before and it's common to do some truncation or hashing when names are too long for k8s's limit. So it makes sense to me.

Thanks for clarifying!
Now also makes sense to me.


Update guava to 28.2-jre

Updates com.google.guava:guava from 25.1-jre to 28.2-jre.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.guava", artifactId = "guava" } ]

labels: library-update

not doing this because we're staying in sync with scio


New API testing module

While reviewing and merging #2306 I detected a tiny API violation of SemVer that was able to slip past detection earlier since we don't yet have any ability to actually test the module API on engine changes.
While we're in Alpha and flying half-blind like this it is probably very hard to avoid trivial violations like that which can be so subtle. Maybe we can be accepting of that on a case-by-case basis until Beta or when we no longer can say we are aware of and able to fix all live modules. Thoughts?
In either case we need a better way to detect API violations. We have the ApiScraper already to produce what to test, how about creating a unit test module that'll exercise every single entry minimally, including a bit of logic to compare what's tested vs. what exists? Maybe there's some clever way to work code coverage into it, this is something we should hit 100% on.
Related: #2159, #1975, https://github.com/MovingBlocks/Terasology/wiki/Modding-API

Heya :-)
I implement an API test for interface Context and class ComponenetFieldUri,  check it out:
ApiTest. In short it compares the signatures of current public & protected methods (method name, modifiers, parameter, return type, exceptions) to the signatures registered. Is it what we want? Now the signature registering stuff is kind of manual. I think maybe it's better to gather some comments before going on testing the huge size of API ;-)
Thanks for your suggestions!

Bit of a belated response here but yep, that seems about right :-)
At first I was thinking simply using each method in some test, since in that case any method signature change would cause a failure. But if there are validators like that it may be even more correct!
We still would want actual unit tests that exercise stuff too, eventually. I wonder if having the method signature validators would cause code coverage analysis to think that code is covered? That might be something to look into. IMHO simply validating the method signatures shouldn't count as covering the code itself.
This reminds me a bit about #2851 which was using a generator that makes unit tests. Really bare ones but it might have some potential for API testing I wonder? Pinging @iaronaraujo for reference.

I just made a pull request with a tutorial about how to use EvoSuite to create unit tests!
https://github.com/MovingBlocks/Terasology/pull/2906

Thanks @iaronaraujo for the nice write-up! And for popping up on this issue as I wanted to run something by both you and @GabrielXia :-)
Today during my day job I got into thinking about Semantic Versioning, auto-detecting the correct scope to increment, and guaranteeing that a pending release has bumped the version number accordingly (optionally over-scoping - such as incrementing major even if there isn't a technical API violation, but for instance all save games will fail to load in the new version)
Between the auto-generated tests from @iaronaraujo and the method signature detection by @GabrielXia could we combine the two to arrive at a complete snapshot of the entire public API? Then between scanning/scraping what is marked as public API vs what we have covered by those tests and comparing vs. state in the master branch we could in theory get a perfect image of API state. Plus a good scaffold to then improve the generated API tests to also include additional more meaningful unit tests that functionally validate API methods.
Image the automatic build test of a new engine PR:

If it introduces breaking changes by removing a parameter on a public method the API tests will fail. We know it'll need a major version increase.

Even if the author adjusts the tests we can run the tests as of the master branch against the new code and we know an incompatible change is being introduced

If it introduces a new public method (or variant) then it requires a minor version increase (comparing the public API scan before and after would detect it)
If it matches perfectly (so ignoring changes in method content or private/protected methods) then it remains a patch release

If any API change is thus detected the PR can be blocked until it includes a version bump appropriate for the change (which would need community approval). Comparing to master would show if said version bump has already happened, in which case the API change would be OK for SemVer (still subject to review) as we have already done the correct increment.
Going with that would guarantee that the version in develop is always correct, as it'll be bumped as part of a merged PR that required the bump. As such there's no actual need to worry about a bigger bump at build/release time (at present I vary minor vs patch manually, usually at release time, and had thought I'd need to set it as a Jenkins release action parameter)
We could eventually expand this to modules and other projects.
What do you think?

Thanks @iaronaraujo for the cool tutorial and @Cervator for the interesting propose, I like it !!! 👍
So if I understand correct, we are going to do:

For each API:

Auto-generated tests using EvoSuite for the code coverage
Method signature detection to API violation
More meaningful test

Compare the API we have vs API we tested vs the state in master branch for documentation

However, I have a few questions as follow:

Do we ignore protected methods?
About auto-detecting the correct scope to increment: Since unit returns 2 results: pass or not pass, how can we distinguish 3 scope: major, minor and patch? Or maybe we just print something to show which scope to increment?

Is there something else I miss? I guess I have some misunderstandings, thanks for pointing it out :-) Also sorry about the delay

That's a pretty nice idea! Regarding EvoSuite, I think it will fit this scope very well! It will still be necessary to check the tests generated to see if they make sense and also make some code adjustments, but after these tasks are done, the tests would be very useful to check how much a new PR would change the engine.
I also had the same doubts as @GabrielXia . I think it's easy to manually check the scope of the change by looking at the tests' results and the code changes on Github, but how would we do that automatically?

@GabrielXia that sounds about right yep

Yeah I think protected should be ignored since such methods wouldn't be part of the Modding API

That's where the PR vs master comes in handy:

If you run the generated API tests from the master branch (likely stored somehow in Jenkins or Artifactory as of the latest release) against the source of a PR then if the author changed a method signature in a backwards incompatible way the tests will fail. We know there is an API violation requiring a major version bump

If the tests included with the PR themselves do pass and the version has been bumped to the next major version since the prior release (from the master branch) then the PR does validate as a viable major release element. We may or may not accept that (community discussion & decision)

If you run master tests against the PR and they pass, but code coverage (maybe actual code coverage, or some custom scanning we do for the tests / API stuff) for public API methods isn't 100% then the PR is introducing new functionality (new methods / method variants). It indicates a minor release.

Again if tests included with the PR do pass + cover 100% + there is a minor version bump included then everything is prepared and we can merge if we choose to. Minor releases are routine.

If the master tests pass and coverage is still 100% then it is a patch release and we're fine (patch gets incremented by default after a release).

The API tests should be stored in a second source set distinct from the regular one in engine-tests I believe (src/api-tests/java maybe?) - they count as something other than unit tests. By keeping it as a different set of tests I believe we can do stuff like not counting it as test coverage for "regular" tests and such, but I haven't experimented with that.
That may also allow us to later run the tests from an archive somehow, produced by the previous release.

Making a quick site note here before I forget. There are two tricky places where API tests may need a little extra to detect full SemVer compliance:

A class previously whitelisted via @API or another method (like being whitelisted at the package level) has the API flagging removed. This would not be caught by an API test within the engine simply testing the class methods - the failure would only occur at regular runtime with a usage from a module. Not hard to catch, but an easy case to miss.
A service previously available within the game Context is no longer published there. This would (at first anyway) only affect engine-published services, like the WorldRenderer. Likely solution would be to simply have a big test that goes and retrieves every known service to make sure it is available, from a full environment. That would need a list of all "standard" services which would probably be pretty handy anyway for documentation, much like the ApiScraper. At least some can be found by looking for the @Share annotation but you can also register a service manually.

@GabrielXia & @iaronaraujo: I'm hoping we can finally get back to this now that the pressure from GSOC is over - might be a good option for a mini-GSOC type thing! Maybe the two of you would still be interested. @kaen and @oniatus have also made some neat progress on the automated testing angle as well.


Typo in main.py

In the Vod class in main.py "entry" is spelled as "entrie" instead of entry. This has no effect on the code I am simply being extremely pedantic.

thx


Support more general merging results with Rebot

Originally submitted to Google Code by filip.noetzel on 24 Mar 2014

Hey,
first, some background:
We're using RF to run selenium with ~100.000 test cases across 6 browsers. Obviously, that would take a while if we were to execute them in series. To speed things up, we're using Jenkin's matrix build ability to run test 1-1000 in on job, 1001-2000 in the second job, etc. After all individual jobs have run, we'd like to generate a report that looks as if the tests were executed in series (i.e. parallelisation happens transparently).
second, here's a reduced test case 'split_testcases.robot':
*** Testcases ***
Testcase A
Log        A
Testcase B
Log        B
When run like
$> pybot split_testcases.robot
one gets a report that has the following hierarchy:
Split Testcases

Testcase A
Testcase B

However, when run like this:
$> mkdir run{1,2}
$> pybot --outputdir run1 --test 'Testcase A' split_testcases.robot
$> pybot --outputdir run2 --test 'Testcase B' split_testcases.robot
.. and combined with
$> rebot run{1,2}/output.xml
i get a report that has this hierarchy:
Split Testcases & Split Testcases

Split Testcases

Testcase A

Split Testcases

Testcase B

I tried using --rerunmerge, but that fails because the test reports contain disjoint testcases, am I missing a command line flag ?
I think I was able to adapt merger.py posted in https://code.google.com/p/robotframework/issues/detail?id=1615 to create a merged result that resembles my wanted output, simply by removing the check. I think I can work with that for now. However, such functionality sounds useful to other RF users.
Thanks again for Robot Framework :)

Hi,
i have a requirement where i need to break a single testcase in multiple in report.xml.
i have a testcases A(Scenarios A)under suit A.
now in this testcase A i am covering many more testcases say A1,A2..
but in the report as expected the testcase count is 1.
Expected report.xml in my case.
for example.
total test case:3(A+A1+A2)
passed:2
failed:1
how do i achieve this??
do i need to write a custom result writer and rebot script??
any leads is appreciated,Thanks in advance.
failed:1

Hi,
I am also having the same requirement as 'simplyviks' posted here. I need to generate multiple logs by splitting the log file from one testcase run.
Any pointers are greatly appreciated. Thank you.

I tweaked the counting logic.though that's not the best solution.

I think you should use tags for reporting purposes (requiremetents etc.)  instead of counting test cases.
You can cover multiple tags with one test case and dynamically add and remove tags during test execution. Use "BuiltIn.Set Tags" and "BuiltIn.Remove Tags".

Hi, Thanks for sharing your comments.
@simplyviks, can you share how you tweaked the counting. In my case i'm using dummy testcases for getting the count and pass/fail result addressed. But for that dummy testcase, i also need to associate the respective log which is a snippet of the full log run in one major testcase. Are you also getting the log seperately?
@mkorpela, I am able to set/remove tags for sections of codes under same testcase. But i am unable to get seperate log files based on the tags. I tried rebot with different options like '--tagstatexclude', '--removekeywords tag:' but with no luck. Is there a way i can extract logs for only the code part which is under defined TAG?
Thanks again.


Rework sidebar styles

Changes the highlight color to blue
Uses chevrons
Reduces bolds

Before

After

Thanks @micahtaylor; updated to fix a JS bug.


[Web Components Template] Update config to automate language page creation

The problem
Translation data is currently captured in separate json files. In addition, the language/locale folders are also generated manually within the webpack config. This is overhead configuration that could lead to misconfigurations, as well as additional management for adopters of this template.
The solution
This should be updated so that the language/local folders are auto-generated based on the existence of the language files.
Additional information

We should also account for cases where languages are not necessary, so that the language folders aren't generated if the translation files are empty, or perhaps a configuration that would disable this feature and only take in the data from a single set of i18n data.

Include in the 8/2 engineering refinement meeting agenda.


OSError: [Errno 12] Cannot allocate memory

Hello,
I am getting Cannot allocate memory error;I understand this is something related to my GPU. But it is quite surprising that I should get this error because, I am training this on 3 1080TI GPUs, with a batch size of 64.
Traceback (most recent call last):
  File "train.py", line 162, in <module>
    train()
  File "train.py", line 113, in train
    for batch_idx, (data, target) in enumerate(train_loader):
  File "/usr/local/torch3/lib/python3.5/site-packages/torch/utils/data/dataloader.py", line 310, in __iter__
    return DataLoaderIter(self)
  File "/usr/local/torch3/lib/python3.5/site-packages/torch/utils/data/dataloader.py", line 167, in __init__
    w.start()
  File "/usr/lib/python3.5/multiprocessing/process.py", line 105, in start
    self._popen = self._Popen(self)
  File "/usr/lib/python3.5/multiprocessing/context.py", line 212, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File "/usr/lib/python3.5/multiprocessing/context.py", line 267, in _Popen
    return Popen(process_obj)
  File "/usr/lib/python3.5/multiprocessing/popen_fork.py", line 20, in __init__
    self._launch(process_obj)
  File "/usr/lib/python3.5/multiprocessing/popen_fork.py", line 67, in _launch
    self.pid = os.fork()
OSError: [Errno 12] Cannot allocate memory

CUDA_VISIBLE_DEVICES=0,1,2 python train.py ~/DATASETS/cifar.python cifar10 -s ./snapshots --log ./logs --ngpu 3 --learning_rate 0.05 -b 64
Please suggest what I could do to avoid this issue.
Thank You!

Hi! As @ZhuFengdaaa confirms,  it seems a peak memory problem although I am not able to reproduce it. Again as @ZhuFengdaaa suggests, this seems to be linked with the number of threads (also see https://discuss.pytorch.org/t/guidelines-for-assigning-num-workers-to-dataloader/813/6).

Another related thread https://github.com/ruotianluo/self-critical.pytorch/issues/11

Why does this have to do with permissions？

OSError: [Errno 12] Cannot allocate memory sounds more like a RAM problem, not a GPU problem. Check you have enough RAM/SWAP, and the correct user permissions.

Why does this have to do with permissions？and what should i do with permissions?

**Fixed this ** by allocating 4G swap memory. You can try allocating more memory if 4G does not suffice.
Follow this blog to allocate swap memory on your device.
https://www.digitalocean.com/community/tutorials/how-to-add-swap-space-on-ubuntu-16-04

Fixed the problem by allocating 64GB of swap memory from the external disk.

Why would some SWAP be needed? It slows down everything
I am having the same problem while trying to allocate 30 GB even though I have 1 TB free...

Fix, is almost always:
echo 1 > /proc/sys/vm/overcommit_memory
https://stackoverflow.com/a/52311756/1391392

This problem is come from CPU memory allocation. check CPU Ram Memory


Fixes multihardlink directory styling on unix

Hello.
Encountered strange behaviour of lscolors package regarding multihardlink formatting on unix systems.
It's perfectly normal for a directory to have more than one reference -- that's how Unix filesystem works (as far as I know), e.g. execution of /usr/bin/ls -hn --color / yields:
lrwxrwxrwx   1 0 0    7  1 Jul  2021 bin -> usr/bin
drwxr-xr-x   4 0 0 4.0K 29 Jun 06:36 boot
drwxrwxr-x   2 0 0 4.0K  1 Jul  2021 cdrom
drwxr-xr-x   3 0 0 4.0K 16 Mar 19:47 data
drwxr-xr-x  20 0 0 4.8K 14 Jul 13:12 dev
...

where 2nd column is reference count for each directory.
I've created a real multihardlink file and default ls displays it in red and yellow color (as configured in LS_COLORS env. var):
$ LS_COLORS="mh=1;33;48;5;52:" /usr/bin/ls -l --color

However, invoking pkg/cli/lscolors on the same directory results in:

That's why I propose to add extra logic into isMultiHardlink() function.

Further investigation showed up that the result also depends on current working directory. Excessive coloring happens when working directory is the same as listing directory, and vanishes when listing directory is different.

Apparently, that's because GetStyle() expects absolute paths, not relative ones.

Hmm, interesting. You can't normally create hard links to directories with ln but TIL the nlink field counts all the . and .. references as hard links too. Thanks for the fix. I'll add a regression test for it too.

Further investigation showed up that the result also depends on current working directory. Excessive coloring happens when working directory is the same as listing directory, and vanishes when listing directory is different.

Apparently, that's because GetStyle() expects absolute paths, not relative ones.

I don't get this part. Your screenshots show a l4 command, which I assume is a program you've built based on the lscolors package. Is there a similar bug in Elvish itself?

I don't get this part. Your screenshots show a l4 command, which I assume is a program you've built based on the lscolors package. Is there a similar bug in Elvish itself?

No, I walked through the code with debugger, and as far as I can tell Elvish uses absolute paths and therefore is not affected by this aspect. I changed my paths to absolute and that solved the 2nd issue.


Aruba not recognized as a country

The endpoint for /api/wikify_region returns QCausx-Aruba instead of (Q21203)[https://ringgaard.com/kb/Q21203] for Aruba. We probably need to add it to our country wikifier, is that correct?

@saggu do you know if we can add that qnode for Aruba to our country wikifier?

Already added to the main branch of t2wml-annotation. I will move it to the causeex branch as well.

@devowit Please use this file to get all the qnode mappings:
https://github.com/usc-isi-i2/t2wml-annotation/blob/master/annotation/generation/country-wikifier.csv


How to contribute

Hello all, i would like to repair some bugs regarding different boards which i have. Is there some guide how to create pull requests ?
For example one/bug feature one pull request or it all together ?
Regards, Ondra OK1CDJ

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Hi Ondra!
One bug and one pull request sounds fine!
The rest is the usual way -> Fork -> make changes or additions to the code -> create pull request.
BR
Rainer

Ok will make PR.

Hi Ondra,
i’ve pushed our latest updates (Version 4.33e) to the main branch. Please refer to this branch for your next requests.
Thank you for your help
73 de Kurt
OE1KBC
Ing. Kurt Baumann
T: +43 699 1200 3520
M:  @.> @.
H:  https://icssw.org https://icssw.org
Von: Ondrej Kolonicny @.>
Gesendet: Mittwoch, 28. August 2024 09:45
An: icssw-org/MeshCom-Firmware @.>
Cc: Subscribed @.***>
Betreff: [icssw-org/MeshCom-Firmware] How to contribute (Issue #45)
Hello all, i would like to repair some bugs regarding different boards which i have. Is there some guide how to create pull requests ?
For example one/bug feature one pull request or it all together ?
Regards, Ondra OK1CDJ
—
Reply to this email directly, view it on GitHub https://github.com/icssw-org/MeshCom-Firmware/issues/45 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2E5BZYBDBNMKJKQBKQKVTZTV5YVAVCNFSM6AAAAABNHVPFJ2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ4TCMZTGQYDCOI .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/AA2E5B5GCYV2INEV7DHLNILZTV5YVA5CNFSM6AAAAABNHVPFJ2WGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHJI7V5QM.gif Message ID: @.*** @.***> >


Code block is doubled in Instant Rendering Mode

Hello!
When working in instant rendering mode, code blocks are getting doubled displayed when working on them, I guess that is a bug.

Also: I cannot click on the closing ``` quotation marks to edit the line.

This is a display form, divided into two areas, the upper part is the editing area, and the lower part is the real-time rendering display area. If it is a graph, it may be more intuitive to understand.

I see. Hmm, I guess that was not clear enough so that I understood it 😅


PG native driver: cannot insert multiple commands into a prepared statement

Environment
Knex version: 1.0.4 and 0.95.15
Database + version: PosgreSQL 12.9
OS: Ubuntu
ping @machuga
Bug

Knex.raw should allow multiple commands when using pgnative driver

Error message: cannot insert multiple commands into a prepared statement

 knex.raw('SELECT 1; SELECT 1;')

origin: https://github.com/knex/knex/blob/master/lib/dialects/postgres/index.js#L218
This change seems to fix the problem:
      values: obj.bindings && obj.bindings.length ? obj.bindings : undefined,

@Awalgawe I have the same problem
await knex.raw(`
        DELETE FROM walhalla_main.report_type_user_column
        WHERE report_type_id = :reportTypeId:;

        INSERT INTO walhalla_main.report_type_user_column (report_type_id, column_name)
        SELECT :reportTypeId: AS report_type_id, unnest(:columns::text[])
      `, { reportTypeId, columns: params.columns.map(({ name }) => name) })
        .transacting(trx);

and after dividing delete and insert everything going well.
I suggest that it isn't normal behavior, mayby it's because of my knex version 0.21.16

I was able to reproduce this on my machine as well, and your recommended patch did cause the query to fail. I'm not sure why it resolves it on initial review, and why without the change it utilizes a prepared statement. I'll try to run the test suites to see if there is any other unintentional behavior. I may have to isolate that change to the pgnative driver specifically if there is some extra hiccup here.
@denbon05 given your version of connect is 0.21.16, as you mentioned, I think this is an unrelated issue. Same underlying protocol error message, but not caused by the same thing in the application code.


CMAKE The package name passed to find_package_handle_standard_args (OpenMP_CXX)   does not match the name of the calling package (OpenMP).

It first through the following warning when I try to compile pytorch using source file
CMake Warning (dev) at /usr/share/cmake3/Modules/FindPackageHandleStandardArgs.cmake:272 (message):
The package name passed to find_package_handle_standard_args (OpenMP_CXX)
does not match the name of the calling package (OpenMP).  This can lead to
problems in calling code that expects find_package result variables
(e.g., _FOUND) to follow a certain pattern.
Call Stack (most recent call first):
cmake/Modules/FindOpenMP.cmake:565 (find_package_handle_standard_args)
cmake/Modules/FindMKL.cmake:213 (FIND_PACKAGE)
cmake/Modules/FindMKL.cmake:307 (CHECK_ALL_LIBRARIES)
cmake/Dependencies.cmake:140 (find_package)
CMakeLists.txt:564 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.
And,

MKL OpenMP type: GNU
-- MKL OpenMP library: -fopenmp
-- Brace yourself, we are building NNPACK
-- NNPACK backend is x86-64
-- Failed to find LLVM FileCheck
-- git Version: v1.4.0-505be96a
-- Version: 1.4.0
-- Performing Test HAVE_STD_REGEX -- compiled but failed to run
-- Performing Test HAVE_GNU_POSIX_REGEX -- failed to compile
-- Performing Test HAVE_POSIX_REGEX -- compiled but failed to run
CMake Error at third_party/benchmark/CMakeLists.txt:231 (message):
Failed to determine the source files for the regular expression backend

and it exit with saying -- Configuring incomplete, errors occurred!
Any help how to configure OpenMP?

Hi there! Thank you for bringing this up. It seems that your issue is related to the installation of PyTorch via building it from source. This repo and its issue listings should contain useful info for trouble shooting: https://github.com/pytorch/pytorch#from-source . Compared with building from source, we recommend using Anaconda to install PyTorch which many people find less painful a process. Hope this helps!


Add HAR output to network traffic analysis

HAR scripts are easier to analyse. Additionally, it's nice to keep track of requests that were tried to be initiated but failed, due to certificate pinning or other reasons. The attached script provides a solution. You can run it like this, by appending to the mitmproxy call: -s ./har_dump.py --set hardump=./$appId.zhar It should be possible to run the script, too, after having already recorded network traffic.

We'd need to integrate this script with the existing analysis script in #68

This should be merged after #80 is merged.

@n-aggarwal, can you review (and merge)?

I've now integrated the HAR script into the current workflow. I haven't tested that yet. Files are directly stored as .har and not the compressed *.zhar, for simplicity.


Error on Icon.getImageSource

Hi, I'm getting the RNVectorIconsManager not available, did you add the library to your project and link with libRNVectorIcons.a? error when trying to use the Icon.getImageSource() method. Of course I'm aware of configuration. Everything works well when I do:
<Icon name="email" size={24} />

However, it throws error when I do:
Icon.getImageSource('email', 24).then((source) => {
});

And of course <Icon /> way is working on both iOS and Android.

You must integrate the ObjC/Java libraries for PNG support like the error suggests, see docs for more info on how to do this.


Clean install is weirdly borked

So I used npx sb@next repro, and it all works fine. But as I've commented in the following code block, a clean vite + sb-builder-vite install errors out
yarn create vite # I used the react-ts template
cd <folder name>
yarn
npx sb@next init --builder storybook-builder-vite # https://storybook.js.org/blog/storybook-for-vite/
yarn storybook # acorn jsx error
yarn add --dev @storybook/core-common webpack@^5 # Peer dependencies that yarn said were unmet, so this is an attempt to try to fix the error
yarn storybook # acorn jsx error still occuring

Here's a screenshot of the "acorn jsx error"

This error seems to make everything load infinitely. No error gets displayed in my shell, and everything comes to a standstill.

I think this happens when two incompatible versions of acorn are used. I'll see if I can reproduce

Here's a repo with the reproduction up to the first yarn storybook https://github.com/JamesYeoman/sb-vite-bug-repro if it helps. I'm also using node v16.10.0 and yarn v1.22.17, all installed using volta on Windows 10

I am getting this on a clean install - bit of a blocker :/

OK, thanks for the reproduction @JamesYeoman.  I've figured out what's happening.  If acorn-jsx is not hoisted up to the top-level node_modules directory, it isn't being pre-bundled (see https://github.com/vitejs/vite/issues/3024), and for whatever reason (probably changes to the dependencies used inside storybook), acorn-jsx is now only inside node_modules/@storybook/addon-docs/node_modules/acorn-jsx, so it isn't being processed/pre-bundled.
The temporary solution is to add '@storybook/addon-docs > acorn-jsx' to your optimizeDeps.include in viteFinal (being careful not to overwrite the existing values in the config). I'm working on adding it to our base config, but this is tricky to do with this > syntax.  I think I've found a way, though, and should have a PR up shortly.

Oh, and you'll probably also hit https://github.com/storybookjs/storybook/issues/17516 next.  I've got a PR up in storybook to fix it, but for now you can remove addon-interactions.


VSF2-121/Add variant selector to product listings

I'm closing, as the PR looks like a duplicated one.


Highlight discussion issue for dockershim deprecation

Link to k/kubernetes issue 106917
(various places)
Related rewording to make that extra link work in context

and also:

Replace alias for dockershim FAQ with a Netlify redirect

Previews:

/blog/2017/11/containerd-container-runtime-options-kubernetes/
/blog/2020/12/02/dont-panic-kubernetes-and-docker/
/blog/2021/11/12/are-you-ready-for-dockershim-removal/
/dockershim
/docs/tasks/administer-cluster/migrating-from-dockershim/

Fixes #30839

/sig node
Helps with https://github.com/kubernetes/kubernetes/issues/104878

/milestone 1.24

Is this PR intended for the 1.24 release? I noticed the 1.24 milestone.

@nate-double-u I assume you added the v1.24 to mark that this has some connection to the v1.24 release, rather than to indicate that we should treat this change as actually part of that milestone?

@nate-double-u I assume you added the v1.24 to mark that this has some connection to the v1.24 release, rather than to indicate that we should treat this change as actually part of that milestone?

Correct--there's likely a better way to do that. Sorry for the confusion.

The changes look good
/lgtm

This is ready for (another) review. The main change since my last push was to address https://github.com/kubernetes/website/pull/30840#discussion_r821928762

Thank you for addressing comments
/approve

/lgtm
Thanks everyone!


Add example for models with relationships or embedded objects (subdocuments)

I think the title says it all. Since I am new to mongoose and zod I am struggling with how to build a model that has embedded documents as well as relationships.

I assume you have some code samples at hand for this. Maybe you can share those?

As the OP, I need help working with relations.
I've navigated almost every option in the zod/mongoose schema, and can't see how to set the ref.
In Prisma is very easy to do, but I no longer want to work with Prisma because want to do everything with TypeScript code (also don't want to execute prisma generate every time I make a change in the schema...)
I have a UserValidationSchema (zod) with a property profile which is a UserProfileValidationSchema (zod).
When creating a user in users collection, it should create a profile in user-profiles collection, and when retrieving a user from users, it should bring the corresponding profile from user-profiles.
I will appreciate a lot any info. Thank you!

Hello, please see https://github.com/andreww2012/mongoose-zod/issues/5#issuecomment-1518685724
Defining relationships is fully handled by mongoose. Remember that mongoose-zod is just a convenient constructor of mongoose schemas, and if you can't find a feature that is supported by mongoose-zod, you should go down to mongoose level. Perhaps explicitly mentioning handling references in the documentation is a good idea. 👍


add a rough-and-tumble spawn_async API

For some use cases, its useful to be able to delegate work from your
main thread that can run asynchronously.
Some work items:

[ ] more tests =)
[ ] update docs to mention scope() as a preferred alternative
[ ] refactor HeapJob and StackJob a bit (probably defer for later)

After some discussion with @cuviper, we were thinking that -- before landing -- it'd be good to have this return something with which you can poll the result.

Gonna close this while I play around with how to rework it. I'm planning on also doing some (optional) futures-rs integration.


Login context does not have access to the TermGroup. If using app context, add app@sharepoint to the term group access list.

Debug code
powershell.exe Information: 0 : 2020-06-28 08:56:51.7431        [Term Groups]   [10]    [Debug] Code execution scope started    0ms     e13fb279-05ea-495d-b994-d6f0a745c30e                                                                    powershell.exe Warning: 0 : 2020-06-28 08:56:52.5948    [OfficeDevPnP.Core]     [0]     [Information]   Login context does not have access to the TermGroup. If using app context, add app@sharepoint to the term group access list.    0ms
powershell.exe Information: 0 : 2020-06-28 08:56:52.8630        [Term Groups]   [10]    [Debug] Code execution scope ended      1120ms  e13fb279-05ea-495d-b994-d6f0a745c30e
powershell.exe Information: 0 : 2020-06-28 08:56:52.8640        [ProvisioningTemplate WebHook Call]     [10]    [Debug]
Code execution scope started    0ms     e13fb279-05ea-495d-b994-d6f0a745c30e
powershell.exe Information: 0 : 2020-06-28 08:56:52.8640        [ProvisioningTemplate WebHook Call]     [10]    [Debug]
Code execution scope ended      0ms     e13fb279-05ea-495d-b994-d6f0a745c30e
powershell.exe Information: 0 : 2020-06-28 08:56:52.8650        [Provisioning]  [10]    [Debug] Code execution scope ended      217626ms        e13fb279-05ea-495d-b994-d6f0a745c30e
Apply-PnPTenantTemplate : Access denied. You do not have permission to perform this action or access this resource.
At line:1 char:1

Apply-PnPTenantTemplate -Path .\starterkit.pnp -Parameters @{"PORTALU ...

  + CategoryInfo          : WriteError: (:) [Apply-PnPTenantTemplate], ServerUnauthorizedAccessException
  + FullyQualifiedErrorId : EXCEPTION,SharePointPnP.PowerShell.Commands.Provisioning.Tenant.ApplyTenantTemplate

I am the proper admin, I have followed all instructions exactly. I am the term store admin with the credentials used for the process.

Hi Thank you. Could you please re-update your ticket using the template that was provided when creating our bug report please? All the information requested is important for us to classify and triage issues.
Thanks!

Hi,
I have had the same issue - should I raise as a new bug (and use the template as you point out)? Or wait to see if additional info is added to this one?
Tony

@WorTony please open a new issue following @bcameron1231 advice,

Closing to consolidate with another duplicate issue.
If you see this, please try adding app@sharepoint to the Term Store administrators group. If you need more help, please follow issue #416
Thanks!


Wrap lines in output

Wrap lines in output (only noticable for json)

Thanks for this. But there are a lot of changes here, and it's hard to see what does what. For example, it looks like rules for .container are simply dropped.  12pt turns into 12px.  Etcetera.  Could you create a minimal PR that makes the least changes needed, adding other things in separate changes?  Or explain more fully the rationale for all of these changes?

Updated to include the change I care about the most! I can add back the other changes later but would really love the word wrap (now or soonish) since I'm on trypandoc from my phone a lot

I'm hesitant about this, because pre-wrap can make it harder to see the difference caused by the wrap settings.  But let's try it.

Actually I have a better idea. I'll revert the pre-wrap. We can add special processing for JSON to format the JSON output using JS functions.

Implemented via commit 8fa927765fbc93dfacc302634aa52469e9d246b7

YES!!! your fix is way better


feat(trycmd): Support echo text | command

$ echo "hello" "second argument" | lines-from-stdin=1 bin-fixture
read line from stdin: hello second argument

Rather than attempting to do the common case and facing its security implications, this uses the fact that echo is defined in POSIX with a very small feature set, and that piping is a simple shell feature. The presence of the echo command is recognized (as any shell might recognize it) and evaluated into stdin data.
Closes: https://github.com/assert-rs/snapbox/issues/172 (thus CC'ing @jpmckinney)
[edit: Add POSIX link]

A test is included. In the project I'm using it in, switching from the published to the git version correctly shows that my README.md (which was ignored so far) is subtly lying, which is factually the correct result.

Thank you for your interest in this!  However, I feel it is premature to have a PR for this as we have not decided on a solution and solution discussions I feel are best left to issues.  I have also found the existence of PRs are manipulative of maintainers, whether intended by the author or not, as maintainers want to honor the work that was done by people and work to merge it.  Some maintainers recognize that and learn to not care at all.  For myself, I've taken a fairly hard stance that I push back and close PRs.

No pressure intended, happy to move the discussion back to #172. With the issue split out from #168, this sounded to me like the discussion was already complete by isolating the topic from command execution.


Bug: Monitoring methods date format

Steps to Reproduce:

Step
Action
Expected
Status

1
Navigate to EASEY Inhttps://easey-dev.app.cloud.gov/monitoring-plans

Pass

2
Select one facility from the table by clicking on Open Tab.
A new tab with the name of the facility should appear.
Pass

3
Click on the tab that just appeared, with the name of the desired facility. (Barry AL, Oris Code 3 was mainly used for this test)
The facilities information should appear
Pass

4
Under the Sections drop down, make sure that Monitoring Methods is always selected.
The data for Monitoring Methods should appear. This data is in accordance with the business glossary and wireframes.
Fail

5
Select all possible arrangements from the Configurations drop down and Locations drop down boxes
The correct data according to the chosen arrangements will appear.
Fail

6
Given a user is viewing monitoring methodsWhen the data is displayed in a table
The user will have the option to sort the monitoring method data in ascending or descending alphabetical order
Fail

Actual Results
Step 4 failed because date is in the wrong format
Actual Result:

Expected Result:

Requirements
Acceptance Criteria
Given monitoring methods section is selected
When monitoring method data is displayed
Then the data is displayed in accordance with the business glossary and wireframes
DATE FORMAT: MM/DD/YYYY HH:MM (hours in military time 1-23)

https://cvpcorp.testquality.com defect D8. Do not remove or edit.

Test case TC57 passed as a result of the bug fix
New date format:


Primitive Data Types: reference to IEEE floating-point standard in integers section

I'm confused

In section "i8, i16, i32, i64, i128":

Min and max values are based on IEEE standard for Binary Floating-Point Arithmetic.

In section "u8, u16, u32, u64, u128":

Same as signed numbers, min and max values are based on IEEE standard for Binary Floating-Point Arithmetic.

Does "IEEE standard for Binary Floating-Point Arithmetic" means IEEE 754? I think it doesn't specify integer numbers. It even says in section "2. Definitions, abbreviations, and acronyms ":

2.1.32 integer format: A format not defined in this standard that represents a subset of the integers and perhaps additional values representing infinities, NaNs, or negative zeros.

@demin-dmitriy Sorry for took more time to fix it. Thanks again for reporting this.


add check for setting in other place

Pull request checklist:

[ ] CHANGELOG.md was updated, if applicable

Description


[SW-94, SW-95] DEX transaction – Transaction detail (Swap Route & Rates)

For me swap route exist for the remove liquidity transaction, which is not an expected behaviour:

fixed


Q: how to serialize map of lists?

Hi,
I would like to use map DB for an event source database.
The main event storage is a map of UUID/Key and a List of events, something like this:

Map<UUID, List<Event>> data = new HashMap<> ();

Event is an interface
How would I go about defining the treemap and serializing it?
Thanks.

make handsels serializer and use it on make Map
        map = database.createHashMap("transactions")
                .keySerializer(SerializerBase.LONG)
                .valueSerializer(new TransactionSerializer())
                .counterEnable()
                .makeOrGet();


Create ShellSort.cpp

I want to contribute in your project under hacktoberfest2024

Thanks for your contribution!

PR has been merged!

Happy contribution!


Refactor: dry enhancements

what
This is a refactor of the ratelimitpolicy and authpolicy controller to use common code that was repeated.
For that, a new base reconciler is created called TargetRefReconciler which is meant for controllers reconciling CRD's having a targetRef from the Gateway API policy attachment. Also, a new file in the package commons called gatewayapi_utils.go with common functionality regarding gateway API resources.

Remove back reference of the authpolicy in the target object when the authpolicy is being deleted.
Unittests of the common code

This is good refactoring already. Perhaps one iteration further and we could maybe try to reduce the number of if..else and switch statements now checking the type of the target ref object. WDYT about an interface to abstract any target ref object for the functions we need here (i.e. GetRules(), GetHostNames(), GetKind(), GetTargetObject(), ValidSubdomains(), GetGatewayKeys(), etc), with proper implementations for Gateway and for HTTPRoute?

yeah, there is plenty of room for improvement. I will happily review that abstraction ;)


missing dependency for .deb

deb/fio/DEBIAN/control needs updating, at minimum libcurl4-gnutls needs to be added as a dependency, and the description is outdated.

done


Faster benchmarks

Removes the overhead from due to pausing/resuming in Google benchmarks, which allows removing the iteration count 10. This results in more stable benchmarks and easier control of timings. Note, the EvaluateModSwitch and EvaluateRelin benchmarks are therefore out-of-place. Also avoids using plaintext/ciphertext references, to make benchmarks independent of each other.
Interestingly, this change the performance significantly on some benchmarks. I compiled SEAL with and without HEXL on ICX with clang-10.
E.g., with HEXL=OFF, I'm seeing ~1.4x speedup on CKKS EncodeDouble/DecodeDouble.
With HEXL=ON, I'm seeing significant speedups in CKKS EvaluateMulCt, e.g. on N=8192, (~1.7x), EvaluteMutPt (~2.4x) and EvaluateSquare (~2.1x) speedups compared to before this PR.
I'm not sure why the difference, perhaps reducing the number of cache misses. Anyhow, both PALISADE, Lattigo, and HElib all seem to use the same approach as this PR.
See attached for more details: seal-bench-remove-random.xlsx

Thanks for the tip; I'll close this PR until I have time to investigate further.


OL8 vs OL9 benchmark

Hello,
it will be nice to find in docs some benchmarks of OL8 vs OL9.
At the moment, it will helps to me motivate  customer to upgrade.
made some synthetic tests by myself, like there
https://github.com/oracle/oracle-linux/issues/55
maybe I need to made in other way ?
Cheers,
GT

I used the same ESXi, the latest open-vmtools.
no other context, vm on that esxi env.
CPU is quite old, but the same as VM.
*-cpu:0        description: CPU        product: Intel(R) Xeon(R) Gold 6254 CPU @ 3.10GHz        vendor: Intel Corp.        vendor_id: GenuineIntel        physical id: 1        bus info: cpu@0        version: 6.63.0        slot: CPU #000        size: 3100MHz        capacity: 4230MHz        width: 64 bits        capabilities: lm fpu fpu_exception wp vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss syscall nx pdpe1gb rdtscp x86-64 constant_tsc arch_perfmon nopl xtopology tsc_reliable nonstop_tsc eagerfpu pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm invpcid_single ssbd rsb_ctxsw ibrs ibpb stibp fsgsbase tsc_adjust bmi1 avx2 smep bmi2 invpcid xsaveopt arat md_clear spec_ctrl intel_stibp flush_l1d arch_capabilities        configuration: cores=1 enabledcores=1 microcode=0

mutex performance:
 ./sysbench --test=threads --thread-locks=10 --max-time=60 run


Discussion: Config option for including .js on import paths

Summary
When Fuse generates files in a Node project with the module compiler option set to NodeNext, the import paths cause a TS error:
Relative import paths need explicit file extensions in ECMAScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean './tada.js'?
Code Generator has an emitLegacyCommonJSImports option for controlling whether to add the .js extension to import paths. If gql.tada will be preferred over codegen going forward, would it be possible to include a way to control the import paths in generated Fuse files?

That's a good point, we can def include that when we check the tsconfig


Scheduled exports always include transactions since the action's creation date

The transactions exported include the ones in the period starting at the date of creation of the scheduled backup instead of the date of the last time it was executed.
The export is done with the date in ScheduledAction.getTag() as the date at which it should start to export transactions. The problem is that this date isn't updated after an scheduled export is run and stays at the creation date.

Fixed in hotfix/patches.


Update SSD reference impl to add batchnorm after feature extraction layers

We drafted the proposal here with some experiment data:
https://docs.google.com/a/google.com/document/d/e/2PACX-1vTVvqd74nF5Elc6kVLP7OpOhpOgcKnrdlR-EPEAbGrrOQBWkAL2PgijPk5Po5DVeBtABghBljE-C4H7/pub

No more ref changes in v0.7 timeframe, examine for v0.8


Readme referring to incorrect property?

this part of the readme
"where the "application" is injected as the spring.config.name in the SpringApplication (i.e. what is normally "application" in a regular Spring Boot app), "profile" is an active profile (or comma-separated list of properties), and "label" is an optional git label (defaults to "master".)"
Isnt the correct property actually meant to be spring.application.name not spring.config.name
I just set up spring config server and thats how I got it to work

spring.config.name is used on the config server not the client


Treat Warnings as Build Errors - DotNetNuke.Abstractions

Description of problem
DNN has many build warnings riddled throughout the entire solution which are usually indicators of issues with the code. Be it legacy code that has not been cleaned up or wasted memory footprints when allocating data that is never used. We should be treating all build warnings as errors to ensure a high level of code quality.
This is a massive undertaking with the size of DNN, I recommend we do this for the DotNetNuke.Abstractions project as it is the newest project and smallest project.
Description of solution

Update .csproj to treat build warnings as erros
Update code to remove all build errors

Description of alternatives considered
N/A
Screenshots
N/A
Additional context
Code Quality Enhancement
Affected browser
N/A

We could also do this for DotNetNuke.DependencyInjection

I expect it'll be a while before we're ready to turn this on for any other projects, so I'm not sure how helpful that top-level task would be (it'll probably just get closed after getting stale)

See the SA0001 documentation.  We need to turn on generation of the XML doc file for the project for StyleCop to check the validity of the comments.

That helped, now I Just need to remove the warnings 👍

I agree on turning that on on small new projects like this, but yeah we still have a lot of work to do before even thinking about turning that on elsewhere 😬


Fix CVE dependency issue

Fix issue #276  by update dependency poi-ooxml.version to 5.1.0 @FrankSchnicke

Thanks for providing this PR. However, you're missing a signed ECA and the CI is failing with the upgrade. Would it be possible to take a look at these issues?

ECA is passing, thanks. However, in your latest commit, you reverted your change and added another library - was this on accident?

Because the vulnerability was introduced by org.apache.commons:commons-compress, which is a dependency of poi-ooxml. Due to significant changes in the first attempt to modify the poi-ooxml version, which caused a CI failure, I chose to directly upgrade the version of commons-compress in my second attempt at modification.

Thanks a lot!


Should we add this to the readme?

@yodurr @ferventcoder @mwrock what are your thoughts on this:
https://github.com/fireeye/flare-vm
I thought it might be useful to call this out as another example of how to set up a machine.

Is the motivation to show how to invoke a boxstarter script via script:
Set-ExecutionPolicy Unrestricted
. { iwr -useb http://boxstarter.org/bootstrapper.ps1 } | iex; get-boxstarter -Force
Install-BoxstarterPackage -PackageName https://raw.githubusercontent.com/fireeye/flare-vm/master/flarevm_malware.ps1

instead of the click once launcher?
I think that's a good idea. Especially to get around non IE/Edge browser issues or needing a scripted install.

I was thinking more around showing another example of setting up another VM for a specific purpose, in this case for malware analysis and investigation.  The installation issue with non IE/Edge browsers we definitely need to capture though, and provide information about how to run the installation without click once.


Use module property from dev webpack config in DLL webpack config

Closes #1468
To avoid a circular dependency, this relies on a check against the parent module's filename for webpack.config.renderer.dev.dll.js. It's expected that developers who rename config files will hopefully grep the codebase for this filename before changing it.
This change also includes this check again when configuring DllReferencePlugin, because it is no longer guaranteed that the DLL manifest file exists by the time the renderer config specifies plugins.
I played around with the idea of creating a webpack-merge strategy that plucks the value of module from the dev webpack config, but it wouldn't help much because, in webpack.config.renderer.dev.js, the DllReferencePlugin would still try to require the manifest (which would not exist).
Lmk if this should be rebased off a different tag.

I changed the base of this PR to the v0.13.3. PR's should be made against this branch instead of master. Can you please fix the merge conflicts.

Conflicts resolved

Thanks for this!


Improved docstrings of pair-counters in the double_tree_pairs module

@a-kravtsov - since you have been looking at this code most recently, would mind vetting these documentation improvements and providing feedback on whether this is sufficiently clear, or if further clarification is warranted?

I think it'd be good to explicitly mention that if rbin=0 and sample1=sample2 the code will count number of objects in the sample. As I mentioned, this is not obvious. To me the natural expectation for the number of pairs with zero separation is 0. So mentioning np.diff helps, but if someone uses zero's element of counts, they still may be puzzled.


CL/aarch64 back end: implement the wasm SIMD bitmask instructions

The bitmask.{8x16,16x8,32x4} instructions do not map neatly to any single
AArch64 SIMD instruction, and instead need a sequence of around ten
instructions.  Because of this, this patch is somewhat longer and more complex
than it would be for (eg) x64.
Main changes are:

the relevant testsuite test (simd_boolean.wast) has been enabled.

at the CLIF level, add a new instruction vhigh_bits, into which these wasm
instructions are to be translated.

in the wasm->CLIF translation (code_translator.rs), translate into
vhigh_bits.  This is straightforward.

in the CLIF->AArch64 translation (lower_inst.rs), translate vhigh_bits
into equivalent sequences of AArch64 instructions.  There is a different
sequence for each of the {8x16, 16x8, 32x4} variants.

All other changes are AArch64-specific, and add instruction definitions needed
by the previous step:

Add two new families of AArch64 instructions: VecShiftImm (vector shift by
immediate) and VecExtract (effectively a double-length vector shift)

To the existing AArch64 family VecRRR, add a zip1 variant.  To the
VecLanesOp family add an addv variant.

Add supporting code for the above changes to AArch64 instructions:

getting the register uses (aarch64_get_regs)
mapping the registers (aarch64_map_regs)
printing instructions
emitting instructions (impl MachInstEmit for Inst).  The handling of
VecShiftImm is a bit complex.
emission tests for new instructions and variants.

@yurydelendik test-mention.

Thanks for the review.  A couple of comments:

Also, is there an extra "preopt.serialized" that needs to be removed?

That is something to do with peepmatic -- a generated file, resulting from the change of the CLIF instruction-definition file.

there is no need to implement all VecShiftImmOp, only SShr is needed

That's true; but it was easy enough to implement and test UShr and Shl too.  I thought these would be useful.  In particular, for doing improved translations of CLIF vector shifts by immediate, at some future point.


Make tests more resistent against changes in pandoc

Many tests use golden files to check correctness of the produced output. This is fragile and breaks whenever the pandoc template changes for the given format. Tests should rather check for the existence of certain keywords in the output.
Filters with fragile tests:

[x] scrlttr2
[x] track-changes
[ ] table-short-captions

I've updated the testing for table-short-captions.
Thanks for the examples in the other two filters.


Questions: CR for artifacts

Is there a plan to support artifacts (e.g. a  AVRO schema) as CR? This way a AVRO schema could be defined over GitOps, this would help in our organization to enable change in a more controlled way.

This is an interesting idea, you suggest the operator would take care of pushing an AVRO artifact, defined as a CR, to the registry? I don't expect we will implement this feature in the near future, but we should keep it in the backlog. Could you please provide specifics - a more detailed use case, how would you expect it to work?

Maybe some more detailed background form where I'm standing. We're using Strimzi / AMQ Streams in a bigger Enterprise and are just at the beginning of using Kafka ;)
Right now most of our resources are controlled over gitops e.g. Topics, KafkaUsers or even our Keycloak configurations. With gitops we found a way of having the state of resource controlled and can implement various gates on top of it, like governance or even data owner approvals.
Our week spot is now the schema registry. We considered to protect the Schema Registry API but this is far more complex than gitops as we don't know and it would be home grown.
Currently we are looking at https://docs.confluent.io/current/confluent-security-plugins/schema-registry/introduction.html#sr-security-plugin because of the statement of your previous comment, that you won't have this in the near future. This could solve our security needs but not regarding other aspects like governance.

@dweber019 have you evaluated uploading artifacts to the registry at runtime by your kafka producers?
or maybe the maven plugin to upload the artifacts after deployment?

Closing this issue, the work on this idea moved to https://github.com/Apicurio/apicurio-registry-content-sync-operator


Storage end to end test can run without env var credentials ✨

Changes
The storage end to end test required that an environment variable be
configured that points to json credentials on disk which can be used to
pull from a private GCS bucket. These are not actually required to run
the test if the defualt credentials in the namespace have the
appropriate permissions - which is the case when the test runs in CI via
Prow!
With this change, the test will now be run in CI. This partially
addresses #821, but TestStorageBucketPipelineRun still requires this env
var to run.
Also removed the read-secret-env step from the storage test. If the
credential isn't present, the previous step in the Task would fail. Also
nothing is verifying the output of this step. (Maybe this was for
debugging?)
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide
for more details.

/lgtm
Nice!

Looks like you might need to change the email address on this one @sbwsg

I think the email address in the co-authored line is right but googlebot needs an explicit label of "cla: yes" to continue.  I forgot to do this Friday and am remembering now that I don't have the ability to do so through the GUI.  Checking if labelling via tekton-robot works:
/label "cla: yes"

/approve
/meow space

If I'm reading this right it looks like a docker pull hit network weather:
I0506 17:51:51.302] [test-taskrun-with-serviceaccount-docker-basic] [build-step-test] time="2019-05-06T17:49:10Z" level=fatal msg="Error reading blob sha256:06c6d2f5970057aef3aef6da60f0fde280db1c077f0cd88ca33ec1a70a9c7b58: Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: i/o timeout"
I0506 17:51:51.302] [build-step-test] Build Failed

And running the same TaskRun locally did not experience the issue. I'm going to try simply rerunning this.
/test pull-tekton-pipeline-integration-tests