[Bug]: grant_ownership does not work on procedure

Terraform CLI Version
1.8.3
Terraform Provider Version
0.89.0
Terraform Configuration
resource "snowflake_database" "test" {
  name = "DB"
}

resource "snowflake_schema" "test" {
  name     = "SCHEMA"
  database = snowflake_database.test.name
}

resource "snowflake_role" "test" {
  name = "TEST"
}

resource "snowflake_grant_account_role" "test" {
  role_name = snowflake_role.test.name
  parent_role_name = "SYSADMIN"
}

resource "snowflake_procedure" "test" {
  name        = "PROCEDURE"
  database    = snowflake_database.test.name
  schema      = snowflake_schema.test.name
  language    = "JAVASCRIPT"
  return_type = "VARCHAR"
  statement   = "return 'Hi'"
}

resource "snowflake_grant_ownership" "test" {
  account_role_name   = snowflake_role.test.name
  on {
    object_type = "PROCEDURE"
    object_name = "${snowflake_database.test.name}.${snowflake_schema.test.name}.${snowflake_procedure.test.name}()"
  }
}

Category
category:identifiers
Object type(s)
resource:grant_ownership
Expected Behavior
expected fully qualified name is "DB"."SCHEMA"."PROCEDURE"()
Actual Behavior
but actual is "DB"."SCHEMA"."PROCEDURE()"
│ Id: ToAccountRole|"TEST"||OnObject|PROCEDURE|"DB"."SCHEMA"."PROCEDURE()"
│ Error: 090208 (42601): Argument types of function 'PROCEDURE()' must be specified.

Steps to Reproduce

copy the following configuration
terraform apply

How much impact is this issue causing?
Medium
Logs
No response
Additional Information
The workaround for grant_privileges_to_account_role does not work because of the different parse logic of IDs.

grant_ownership - https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/57ee66bf76590da389b3343b5b6a0fd7d0e4836e/pkg/resources/grant_ownership.go#L403
grant_privileges_to_account_role - https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/57ee66bf76590da389b3343b5b6a0fd7d0e4836e/pkg/resources/grant_privileges_to_account_role_identifier.go#L145

Even if parse logic is the same, it does not work in a empty arguments because the () is removed.
The following error should occur
Error: 090208 (42601): Argument types of function 'SOME_PROCEDURE' must be specified.

Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

If I can figure out the details of identifiers rework, I can create a PR :)

Hey 👋
Thanks for creating an issue for this. The identifiers rework will happen this quarter, but we haven't started it yet. We have to analyze all the cases where we feel the identifiers are lacking and provide a proposal we can all agree on. Without the proposal, we cannot give any instructions, but I will link this issue to do it as part of the identifiers rework.

Hi @sfc-gh-jcieslak :)
Thank you for releasing version 0.93.0!
I am eager to use this version, but unfortunately, I cannot upgrade until the issue with the identifiers rework is resolved. Depending on when this will be addressed, we are considering excluding non-functioning grant resources from Terraform management.
Could you please provide an update on the current status of the identifiers rework?
Thank you for your assistance!

Hey @YAhiru. @sfc-gh-jcieslak is on a short vacation. He will come back on Monday and the identifiers rework will be his first focus. I will let him know about your request, so he can provide you with a better timeline.

Thank you!
I appreciate the quick response and am looking forward to the next release :)

We are also stuck on version 0.92 now, because the deprecated resource "snowflake_procedure_grant" has been removed in version 0.93. The new resource "snowflake_grant_privileges_to_account_role" does not work with procedures that do not have arguments. The old resource worked with it. Maybe you could leave the old resource available until the new resource is able to cover this case?
Thanks and kind regards
Susanne

Hey @susanneauxmoney, we are currently fixing the identifiers, including this case. It should be a part of the next release (Wednesday/Thursday next week).

@sfc-gh-asawicki great news and thanks a lot for your quick reply 👍 .

Hey @susanneauxmoney, we are currently fixing the identifiers, including this case. It should be a part of the next release (Wednesday/Thursday next week).

Any update when the fix will be released?

Hey @rahulgoyal01 👋 (cc: @YAhiru @susanneauxmoney)
This will be my next task after introducing a new identifier representation for functions/procedures/external functions in our SDK. Hopefully, the fix for this (and other privilege-granting resources) will land in the next release (and if not the next one, it should certainly be fixed in the one after).

Hi @YAhiru @susanneauxmoney @rahulgoyal01 👋 The fix was released in a new provider version (v0.95.0). Please upgrade using the migration guide.

@sfc-gh-jmichalak
Thank you!
In my environment, I was able to migrate without any problems!

@sfc-gh-jmichalak is the guidance here to upgrade directly from 0.92.0 to 0.95.0, as 0.93.0 and 0.94.0 no longer have the snowflake_procedure_grant resource and also do not support procedures with no arguments?
I'm trying to square what I'm reading here with your general guidance on https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/MIGRATION_GUIDE.md

Tip
We highly recommend upgrading the versions one by one instead of bulk upgrades.

Hi @jmkacz 👋
You can upgrade directly to v0.95.0, but please read the migration guide for all of the versions between and act accordingly. We've added such a tip because some users tried to bulk upgrade from, e.g. v0.60 to v0.90, which resulted in lots of problems.


safe copy for C++

Travis-ci got something wrong, need to restart build

I will take another look at this PR this week. Sorry for the delay.

There was conflicting, I create another pr latter


Expand results in GitLens Results view

An improvement for Compare working copy to merge-base #240 would be to have a quick way to expand the results. For example, if there are 20 files changes in various directories between my feature-branch and a master, right now I need to expand the results one by one.

Going to hold off on implementing this until https://github.com/Microsoft/vscode/issues/30288 get flushed out. I don't want to hack implement something, if the api support is coming.

Depends on https://github.com/Microsoft/vscode/issues/55879


Use PEP compatible version for wheel and sdist

Close #192

@MrMrRobat, check it, please

Great, thanks much!

Cool, thank you :)


Read across aligned pages on macOS and add some documentation

The bad news is that macOS only lets you read a page at a time with vm_read. However, the good news is that—after reading some of the JDK's debugger source for that platform (see here)—I put together a read_aligned function to handle the messy work of reading multiple pages and smushing together the results.
The result of that is that copy_address is now concerned with just the addresses (and slicing into the result buffer), and read_aligned is concerned with just the reading. 🎉
This still doesn't get it truly working on macOS, but it's a bit more progress (and served as a fun little exercise for the evening).

Nice! I have a couple of tiny comments but this looks great.
For later, I figure that the "right" way to fix the vm_read will be to use vm_read_overwrite which writes to a user-specified address instead of dynamically creating a mapping for the copy. It may be messy because the destination might also have to be page-aligned, and there's no way to allocate on a page boundary in stable Rust. Only thing I can see doing is create a 2x page size buffer and get a page-aligned slice of it. Anyway, getting ahead of things there for now. :-)

FYI I factored the process memory reading bits of this project out to a standalone crate: https://github.com/luser/read-process-memory
I don't have the automated tests running on OS X yet (haven't worked around the task_for_pid nonsense), but I did some manual testing and it looked like on my system (10.11.6) vm_read worked fine with a size > 4096. Am I missing something?
I've got a PR here to use that crate for ruby-stacktrace: https://github.com/jvns/ruby-stacktrace/pull/14

@luser does your change in #14 cover unaligned reading on OS X? If so I'll close this PR. If not, it's probably worth adapting @dirk's work the read-process-memory crate. Either way drop a note here so we know what to do!

Ugh reading comprehension. Saw your note about the size. I don't have an OS X system handy to test on, so I can't investigate.

I did manual testing with both >4k reads and unaligned reads and they seemed to work with the existing code on my machine. I don't know if that's because of changes in the kernel, or if there's a scenario that @dirk tested that I missed. If this code is needed for OS X I'd be happy to take it into read-process-memory.

Ok. I'm going to close this out. If a user finds it's not working properly we can revisit.

I did manual testing with both >4k reads and unaligned reads and they seemed to work with the existing code on my machine. I don't know if that's because of changes in the kernel, or if there's a scenario that @dirk tested that I missed. If this code is needed for OS X I'd be happy to take it into read-process-memory.

@luser: Honestly this patch was just to have it do The Right Thing like all the other code I'd seen using vm_read (mostly looking at the XNU source itself as well as the JVM HotSpot's debugger code). If the cases you tested work then I'd call the existing implementation "Right Enough". 😉


ModelBuilder: Allow sequence store type to be specified

We currently have...
modelBuilder.Sequence("MySequence").Type<short>();

...but we should also allow you directly specify the store type (similar to properties).
modelBuilder.Sequence("MySequence").StoreType("smallint");

A bit off topic, but why don't we also allow this syntax?
modelBuilder.Sequence<short>("MySequence");

The Type<T>() method seems a bit strange. This would also make Sequence more similar to properties.

I suspect because it's optional, and chaining is more consistent in that case. But there's no technical reason why we couldn't.

Optionality can be handled with a non-generic overload. Just saying.

Yeah, I think we should do it. I'll bring it up in API Review.

Triage: YAGNI. The default mappings for short, int, long, etc. should always be sufficient.


Extends out of the box

What an amazing work, thanks!
Ran into this glitch (notice the last line bleeding below the separation line):

Simply using this in my code solved the issue for me:
.emoji-picker__tab-body {
  margin-top: 0px !important;
}

Any idea what could be causing this, and if it is the right way to fix the issue?

I just released v2.4.0 that should fix this issue for you. Feel free to re-open if the issue still persists.


Fake HttpClient for Unit Testing

We need a fake HttpClient to ensure we can reliably run unit tests.

Completed in #15


use default staticPath

What type of PR is this?
/kind feature
Which issue(s) this PR fixes:
Fixes https://github.com/openyurtio/openyurt/issues/484

/lgtm
/approve


cjxl GIF to JXL crashes using VarDCT with effort-8 or effort-9

Describe the bug
Consider the following GIF: e4ec7c5d7af5342f57347c9ada429fba.gif

To Reproduce
cjxl -d 1 -e 8 e4ec7c5d7af5342f57347c9ada429fba.gif output.jxl

Expected behavior
cjxl should produce a JXL file as it does with -e 7 and lower.
Actual behavior
Using -e 8 or -e 9 crashes. Using -e 7 and lower does not crash.
$ cjxl -e 8 -d 1 e4ec7c5d7af5342f57347c9ada429fba.gif output.jxl
JPEG XL encoder v0.8.0 759a29f9 [AVX2]
./lib/extras/dec/color_hints.cc:54: No color_space/icc_pathname given, assuming sRGB
Read 600x563 image, 3032897 bytes, 3.8 MP/s
Encoding [VarDCT, d1.000, effort: 8],
./lib/jxl/image.h:57: JXL_CHECK: xsize <= orig_xsize_
Illegal instruction (core dumped)

Valgrind output: valgrind.log
Environment

OS: Arch Linux x86_64
Compiler version: gcc 12.2.0
cjxl/djxl version string: JPEG XL encoder v0.8.0 759a29f9 [AVX2]

Additional context
Here, -d 1 is just to force VarDCT. If you don't include that it implicitly defaults to -d 0 and uses modular lossless.

Thanks for the report!
Bisecting this shows that this used to work before we started using the render pipeline in the encoder: https://github.com/libjxl/libjxl/commit/60654d7e75829fa94e557e6cf3f1d47d5456fbe5.

Fixed in #2556?

Appears to be fixed :+1:


Handle Ctrl-C interrupt

Out of habit I'd often exit this script with Ctrl-C instead of by hitting q, but this would cause problems with the cursor never being unhidden and hitting Ctrl-C in diff view would cause the terminal to fill with junk. Fixed by trapping Ctrl-C and resetting cursor and colours before exiting.

Hey @ryanc414, thanks for your contribution, I find this one interesting, however it doesn't seem to work for me. I'm on ubuntu 14.04 with bash 4.3. What platform did you use to test it ? For me whenever I try to Ctrl-c, the cursor appears again but the terminal gets completely broken (can no longer see commands, pressing enter will print the prompt on the right instead in a new line, ...).

No problem. Ah that's unfortunate, it worked OK for me testing on Arch Linux 4.10.1 with Bash 4.4.12, running in xfce4-terminal 0.8.4. I've also been using it on Centos 7 without a problem. Maybe there's a Ubuntu-specific problem I could look into at some point...

I was able to reproduce the problem on Ubuntu 14.04. My most recent commit fixes the problem for me - the stty settings were wrong after exiting so I made sure to restore the original settings in the cleanup function. Let me know if this still doesn't work for you!

Hey @ryanc414, indeed, restoring the original settings fixed the problem, thanks for that :+1: , however, there's still something that I'm not convinced about. Hitting Ctrl-C while using less won't do anything, so I don't see the point to trap Ctrl-C within less and cleaning up everything after quitting. I think it would be better if we untrap Ctrl-C before using less (this command should do the job: trap - INT), and then trap again. What do you think about that ?
(sorry for the late response, busy week ...)

I'm gonna go ahead and merge it, but I'll add some changes, mainly the point that described above.

For me, hitting Ctrl-C when in the diff view using less does exit the program uncleanly. I think it depends on whether the interrupt is delivered to the git-recall script or the parent git process (not sure what determines this exactly). It's not a major problem though so I'm happy for this merge to be closed.


Warnings from aspnetcore Middleware.sln

Building Middleware.sln produces warnings from the TestServer project. It looks like various test related references are being added to the project incorrectly, e.g. the project doesn't need Moq and xunit. The actual warnings are happening because TestServer.csproj is not an SDK project and probably can't handle the type of references.
Fix TestServer project so test references are not added
Update TestServer to use SDK project format?

Is this still an issue?  @mkArtakMSFT can you take a look?

@BrennanConroy, @halter73 who is best positioned to look into this?

Assuming there's no rush, @Tratcher is generally the best person for looking into TestServer-related issues.

I don't think there is a big rush, considering this was filed back in Jan.  It might not even repro anymore.

Moving out of 3.0 as this is not blocking anything.

Note this is regarding a WebSocket manual test asset WebSockets.TestServer, not Hosting's TestServer. https://github.com/aspnet/AspNetCore/blob/1f6d235a5e7fd9427c96b7508610c5d3401a1571/src/Middleware/WebSockets/samples/TestServer/WebSockets.TestServer.csproj


refactor: cleanup minor part 1

Tracker can be seen here https://github.com/odpf/optimus/issues/545

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 9 (55.56%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 75.886%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

config/loader.go
5
9
55.56%

Files with Coverage Reduction
New Missed Lines
%

ext/notify/pagerduty/pagerdutynotifier.go
2
93.75%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
7408

Relevant Lines:
9762

💛  - Coveralls


modcc segfaults when compiled with -ffast-math

Reproduced with gcc 5.3 and gcc 6.3 doing:
modcc -t cpu mechanisms/mod/expsyn.mod
Configuration:
 CMAKE_BUILD_TYPE                 Debug
 CMAKE_INSTALL_PREFIX             /usr/local
 MODCC_BIN                        MODCC_BIN-NOTFOUND
 NMC_AUTO_RUN_MODCC_ON_CHANGES    ON
 NMC_BUILD_VALIDATION_DATA        OFF
 NMC_DISTRIBUTED_MODEL            serial
 NMC_SYSTEM_TYPE                  Generic
 NMC_THREADING_MODEL              tbb
 NMC_USE_OPTIMIZED_KERNELS        OFF
 NMC_VALIDATION_DATA_DIR          /users/karakasv/Devel/nestmc-proto/validation/data
 NMC_VECTORIZE_TARGET             none
 NMC_WITH_ASSERTIONS              ON
 NMC_WITH_CUDA                    OFF
 NMC_WITH_PROFILING               ON
 NMC_WITH_TRACE                   OFF
 unwind_library_generic           unwind_library_generic-NOTFOUND
 unwind_library_target            unwind_library_target-NOTFOUND
 CMAKE_CXX_FLAGS                  -ffast-math

Stack trace:
(gdb) bt
#0  0x00000000005325d9 in make_unique_local_assign (scope=..., e=0x0, prefix=...)
    at /users/karakasv/Devel/nestmc-proto/modcc/astmanip.cpp:16
#1  0x000000000051779d in CnexpSolverVisitor::visit (this=0x5acf80, e=0x5ab4d0)
    at /users/karakasv/Devel/nestmc-proto/modcc/solvers.cpp:55
#2  0x000000000053f893 in AssignmentExpression::accept (this=0x5ab4d0, v=0x5acf80)
    at /users/karakasv/Devel/nestmc-proto/modcc/expression.cpp:924
#3  0x00000000004f37a8 in BlockRewriterBase::visit (this=0x5acf80, e=0x5ab560)
    at /users/karakasv/Devel/nestmc-proto/modcc/visitor.hpp:114
#4  0x00000000005174ff in CnexpSolverVisitor::visit (this=0x5acf80, e=0x5ab560)
    at /users/karakasv/Devel/nestmc-proto/modcc/solvers.cpp:27
#5  0x000000000053f35f in BlockExpression::accept (this=0x5ab560, v=0x5acf80)
    at /users/karakasv/Devel/nestmc-proto/modcc/expression.cpp:849
#6  0x00000000004efcf8 in Module::semantic (this=0x7fffffffb990)
    at /users/karakasv/Devel/nestmc-proto/modcc/module.cpp:447
#7  0x00000000004ba223 in main (argc=5, argv=0x7fffffffd128)
    at /users/karakasv/Devel/nestmc-proto/modcc/modcc.cpp:108

The problem starts from symdiff.cpp, where this check does not succeed and does not populate the result.coef[id]. It then comes up in solvers.cpp here, where r.coef[s] will insert a new expression_ptr that wraps  basically nullptr, since s does not exist, which finally leads to the segfault two lines later.
I cannot figure out an obvious fix, since a "field" knowledge is needed.

On one hand, there is a bug in solvers.cpp which does not treat a missing coefficient as zero. On the other, that coefficient should never have been missing (or zero).
Running test_modcc when compiled with -ffast-math will also report a large number of errors.
The symbolic differentiation code uses NAN as a dummy value when testing expressions for constant simplification opportunities; unfortunately, in this context, -ffinite-math-only, which is included in -ffast-math, allows a test against a function returning NAN against zero (or, apparently, any other number) to return true.
The simplest workaround is to use a different dummy value, but really we should not be using -ffast-math on modcc, or in fact in any situation where it is applied outside of a particular well-defined source file. Is it possible to work around the SVML linking issue by using #pragma GCC optimize ("-ffast-math") or similar in just those source files using SVML routines?

Is it possible to work around the SVML linking issue by using #pragma GCC optimize ("-ffast-math") or similar in just those source files using SVML routines?

That's probably the best idea. I was trying sth quick that's why I ended up with the weary solution to just add the flag globally.
Let me report on the #pragma case and then we can mark it as wontfix.

There's also a real bug here that needed fixing :)


Allow isSupported to be customised

It currently checks for indexedDB which is not present in all environments. Either this function should be moved to the manners component so that it can be customised. Or, we could add a supported method to all components and then call of them instead of running this isSupported function.

Implemented on next branch.


FF tag

Dear Sirs,
I just trying to parse Visa CDET cards from https://play.google.com/store/apps/details?id=com.visa.app.cdet app. App's card 09 is marked as Card with '00' and 'FF' Padding.
When READ RECORD command performed:
00b2011400
Response from the card is:
700C5F340101FFFFFF9F570208409000
ber-tlv parser parses that string in the following manner:
BerTlvs{tlvs=[BerTlv{theTag=+ 70, theValue=null, theList=[BerTlv{theTag=- 5F34, theValue=[1], theList=null}, BerTlv{theTag=+ FFFFFF9F57, theValue=null, theList=[]}]}]}
But according Annex B of EMV 4.3 Book 3 value where bytes from 5 to 1 all = 1 mean "see subsequent bytes" ie all FF's should be ignored.
Could you fix this please?
Thanks in advance,
Igor

Hi Igor,
Thank you for your report.
It seems that 700C5F340101FFFFFF9F570208409000 is a Response APDU.
[R-APDU] 9000
  [70 (response template)] 5F340101FFFFFF9F57020840
    [5F34 (PAN sequence number)] 01
    [FFFFFF9F57 (?)] 0840
      [08 (?)]

This response contains the private constructed tag 'FFFFFF9F57' with the value length 2.
It contains value '0840' which we should parse as a TLV object. But it does not look like a valid TLV structure.
May be there are any rules regarding tags beginning with ‘FF’ ?

As I could see from
https://www.emvco.com/wp-content/uploads/2017/05/EMV_v4.3_Book_3_Application_Specification_20120607062110791.pdf
Annex B1 is when tag numbers ≥ 31 are used (that is, bits b5 - b1 of the first byte equal '11111'), subsequent bytes should be processed. Ie at least first FF should be ignored and processing should be move to the next byte. But it is unclear is the next byte is new first byte or no. Based on context Visa uses for test card I mentioned earlier I could guess they wait FF skip.

Igor,
The library processes subsequent bytes with 0b11111.
The problem is with the tag FFFFFF9F57 and its value 0840.
In the 'Annex B' they say that FF at the beginning means constructed private class tag.
So we need to parse  '0840' as TLV not as a value.
If we change BerTag code to this:
    public boolean isConstructed() {
        for(int i=0; i<bytes.length; i++) {
            if(bytes[i] == (byte)0xff) {
                continue;
            }
            return (bytes[i] & 0x20) != 0;
        }

        throw new IllegalStateException("No bytes or all bytes are 0xff");
    }

The test will pass.
But it's not correct because we need to process FFFFFF9F57 as constructed not primitive.

FF: 11 - private class, 1 - Constructed data object, 11111 - See subsequent bytes.
So could we treat this in the following manner:
FF
  ↳ FF
         ↳ FF
                ↳ 9F57 [length][data]

?

Because FF is a Constructed data object we need to process its data '0840' as another TLV object.
But '0840' does not look like valid TLV structure.

Yes, it looks like regular data

Hi, guys.
It looks like BerTlv is not as simple as it could be :) We also faced up with the problem -- and those bytes should be just skipped. The bad thing is that 00 and FF are able to be before, after, or between normal tags %)
Look, what I've found: "ISO 7816 Annex D.1: BER-TLV data object" -- https://cardwerk.com/iso7816-4-annex-d-use-of-basic-encoding-rules-asn-1/
NOTE – Before, between or after BER-TLV data objects, ’00’ or ‘FF’ bytes without any meaning may occur (e.g. due to erased or modified TLV-coded data objects).
FYI, another team also do things to work out the issue: https://github.com/Shopify/grizzly_ber/issues/13
Could you add fix for this?

I encountered the same issue with the '00'. Any fix for this?


Remove the deprecated API pages

Description:
Remove the following two pages as the previously deprecated APIs have been removed from 3.0.0 altogether.

docs/develop/product-apis/deprecated-apis/store-apis.md
docs/develop/product-apis/deprecated-apis/publisher-apis.md

Related to
https://github.com/wso2/docs-apim/issues/465

Fixed in https://github.com/wso2/docs-apim/pull/750


stringToUtf8ByteArray produces illegal UTF-8 for non-BMP characters

When using stringToUtf8ByteArray to convert a string into a sequence of UTF-8 encoded bytes, any characters outside of the BMP (i.e. characters with a code point > 0xFFFF) gets encoded as a sequence of two surrogate pairs.
Encoding surrogate pairs as UTF-8 is illegal and will cause a proper decoder to fail. See RFC 3629 which states:
"The definition of UTF-8 prohibits encoding character numbers between U+D800 and U+DFFF, which are reserved for use with the UTF-16 encoding form (as surrogate pairs) and do not directly represent characters."

I started working on this and I hit a wall because of issues with the charCodeAt function. In some email traffic, Lokedhs explained that the solution would involve extracting the high and low surrogate pairs that map to the "special" characters - those that have code points greater than 0xFFFF. The charCodeAt function would handle all this. I had trouble getting it to work and I finally built a small test file in my local Closure instance for development, testing in Chrome. No luck. Then I built this JSFiddle which does this:

I placed the code from that JSFiddle back in the local Closure test instance and in Chrome, it does this:

In IE, it does this:

I found the "overload" for charCodeAt here.
Bottom line: I don't see a reliable way to grab the surrogate pairs for characters outside the BMP. Until I figure this out, or someone has a suggestion, I can't proceed. I almost think I discovered a more fundamental bug.

I wouldn't use that one at all. I would rather simply iterate over the characters and do surrogate processing manually. Something like this (note, completely untested):
var i;
var pendingHighSurrogate = null;
var result = [];
for(i = 0 ; i < s.length ; i++) {
    var ch = s[i];
    // High surrogate?
    if(ch >= 0xD800 && ch <= 0xDBFF) {
        if(pendingHighSurrogate != null) {
            throw "Illegal format: Two high surrogates in a row";
        }
        pendingHighSurrogate = ch;
    }
    else if(ch >= 0xDC00 && ch <= 0xDFFF) {
        if(pendingHighSurrogate == null) {
            throw "Illegal format: Low surrogate not preceeded by high surrogate";
        }
        result.push(encodeCharAsUtf8(((pendingHighSurrogate - 0xD800) << 10) | (c - 0xDC00)));
        pendingHighSurrogate = null;
    }
    else {
        result.push(encodeCharAsUtf8(ch));
    }
}
if(pendingHighSurrogate) {
    throw "Illegal format: truncated surrogate pair";
}

Thanks for the report, this behavior should at least be documented on for the function.

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


Keyboard shortcut for Rescan MIDI

@krevis, thank you for creating MIDI Monitor! It's become a really helpful and indispensable tool for me.
It would be great if it were possible to rescan MIDI devices via a keyboard shortcut, e.g. ⌘R. RIght now, I can only trigger that via the menu.

Thanks for the patch, it looks fine to me. You could also just use Audio MIDI Setup, which already has command-R to "Refresh MIDI Setup", which I believe is the same thing.
I'm curious, though, why do you need to do this so frequently? What part of your setup doesn't update automatically?

It's more a convenience thing – I often skip opening Audio MIDI setup and use MIDI Monitor directly to verify proper MIDI connectivity. And with some USB devices, it can be helpful to do a manual rescan.

Which USB devices, in particular, have you found that need this?  Just asking out of curiosity, since I don't recall using any USB-MIDI interfaces that didn't automatically update when plugged or unplugged.


service "ingress-nginx-controller-admission" not found

Hi
doing a kubectl apply results in error
namespace/staging unchanged
deployment.apps/foo created
service/foo created
deployment.apps/bar created
service/bar created
Error from server (InternalError): error when creating "5-simple-fanout-ing.yaml": Internal error occurred: failed calling webhook "validate.nginx.ingress.kubernetes.io": failed to call webhook: Post "https://ingress-nginx-controller-admission.ingress-nginx.svc:443/networking/v1/ingresses?timeout=10s": service "ingress-nginx-controller-admission" not found

solved this with
kubectl delete -A ValidatingWebhookConfiguration ingress-nginx-admission


Test 4 fails

On Mac OSX 10.11 with GeForce GT 650M and CUDA 7.5, I see the following log output for test 4 which fails (all other tests pass):
4/9 Testing: 2D-improve
4/9 Test: 2D-improve
Command: "/Users/krismz/Software/SCI-Solver_Eikonal/build/test/bin/2D-improve"
Directory: /Users/krismz/Software/SCI-Solver_Eikonal/build/test/2d
"2D-improve" start time: May 04 10:05 MDT
Output:
Running main() from gtest_main.cc
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from Improve2D
[ RUN      ] Improve2D.DecreaseRMSError
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
NO cross edge!!! Maybe a hole!!
/Users/krismz/Software/SCI-Solver_Eikonal/src/test/2d/2D-improve.cc:105: Failure
Value of: rmsError[i - 1] > rmsError[i]
Actual: false
Expected: true
[  FAILED  ] Improve2D.DecreaseRMSError (513 ms)
[----------] 1 test from Improve2D (513 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (513 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] Improve2D.DecreaseRMSError
1 FAILED TEST

Test time =   0.54 sec
Test Failed.
"2D-improve" end time: May 04 10:05 MDT
"2D-improve" time elapsed: 00:00:00
`

This is a result of #6 . depending on the architecture, the # of blocks, and the weather, some 2D tests sometimes fail. I don't know if there is time or motivation to investigate this further. These tests work 80% of the time on my windows and opensuse machines.


AO-8901: print Go version in log, fix a code error in README

Minor fixes:

Print the Go version in the log when the agent is started, so it won't need to ask for this information while diagnosing issues.
Fix a code error in README: a right curly bracket is missing.  (Ref: https://swicloud.atlassian.net/browse/AO-8901)

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   84.01%   84.01%
=======================================
  Files          33       33
  Lines        3222     3222
=======================================
  Hits         2707     2707
  Misses        432      432
  Partials       83       83

Impacted Files
Coverage Δ

v1/ao/internal/reporter/reporter_grpc.go
66.43% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[Rust] Add branch profile discovery for other branching Expr

This PR fixes the branch file discovery to also analyse branching in other Expr in addition to the original ExprIf. Currently it supports ExprIf ExprLoop ExprMatch ExprForLoop ExprWhile.

Here is a snippet of the data.yaml showing BranchProfile details.
functionsReached:
 - Version
 - any
 - clone
 - env::var_os
 - into_string
 - is_none
 - is_some
 - map
 - split
 - trim
 - var_is
 functionDepth: 0
 visibility: private
 ICount: 12
 BBCount: 11
 EdgeCount: 18
 functionUses: 0
 BranchProfiles:
 - Branch String: /src/httparse/build.rs:34:4
   Branch Sides:
   - BranchSide: /src/httparse/build.rs:34:50
     BranchSideFuncs:
      - Version
     - any
     - clone
     - env::var_os
 - Branch String: /src/httparse/build.rs:38:4
   Branch Sides:
   - BranchSide: /src/httparse/build.rs:38:47
     BranchSideFuncs: []
 - Branch String: /src/httparse/build.rs:44:4
   Branch Sides:
   - BranchSide: /src/httparse/build.rs:44:32
     BranchSideFuncs:
         - into_string
         - is_none
 - Branch String: /src/httparse/build.rs:50:4
   Branch Sides:
   - BranchSide: /src/httparse/build.rs:50:36
     BranchSideFuncs: []


use one style of graphics for tuning and nested resampling

we currently use 4 very different looking types of figures to describe two concepts (tuning and nested resampling) that have a large overlap -- we should decide which style is best and use that consistently so that it's easier to see how this is related, see also #398:

see https://github.com/compstat-lmu/lecture_i2ml/issues/398#issue-516125617


how to run the deployment 'npx cdk deploy' without additional confirmation

Dear all,
I am trying to automate the deployment, but unfortunately even if I use 'npx cdk deploy --all --force' there's still a need for for confirmation: 'Do you wish to deploy these changes (y/n)?'
If there's a way to run the deployment without additional confirmation ?
best regards
Alexey Shishkin

Try with --method=direct

according to the 'npx cdk deploy --help':
-m, --method   How to perform the deployment. Direct is a bit faster but lacks progress information [string] [choices: "direct", "change-set", "prepare-change-set"]
it does not specify anything about confirmation, but OK, let me try ...
on the other hand:
--all   Deploy all available stacks [boolean] [default: false]
-f, --force   Always deploy stack even if templates are identical [boolean] [default: false]
they are supposed to work, but they did not

I forgot to mention to add --require-approval never

So, what is going to be the final suggestion from your side ?
Is it 'npx cdk deploy --method=direct --require-approval never' ?

this is to confirm the command 'npx cdk deploy --method=direct --require-approval never' does the job
pls. update the documentation mentioning this command as a way to run deployment of the application non-interactively


Null Reference in EnumProperty Editor (Lab #330)

System.NullReferenceException
HResult=0x80004003
Message=Referencia a objeto no establecida como instancia de un objeto.
Source=Xpand.ExpressApp.Win
StackTrace:
at Xpand.ExpressApp.Win.PropertyEditors.EnumPropertyEditor.ObjectSpaceOnCommitted(Object sender, EventArgs e) in E:\tfs\expand\Xpand\Xpand.ExpressApp\Xpand.ExpressApp.Win\PropertyEditors\EnumPropertyEditor.cs:line 115
The variable control is null, perhaps you should consider a null check as in OnCurrentObjectChanged?

Installer lab build 18.2.601.16 includes commit EnumPropertyEditor throws on ObjectSpaceOnCommitted #330 #341 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

build not ready yet sorry for this

Installer lab build 18.2.601.18 includes commit EnumPropertyEditor throws on ObjectSpaceOnCommitted #330 #341 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS

Installer lab build 18.2.601.22 includes commit EnumPropertyEditor throws on ObjectSpaceOnCommitted and when Enum is a Flag #330 #341 that relate to this task. Please test if it addresses the problem. If you use nuget add our NugetServer as a nuget package source in VS.
. Thanks a lot for your contribution.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Line symbol patterns?

Hello, what an interesting library and broader project! The PMTiles format seems particularly innovative.
I'm working on geologic maps and one of our key needs is line patterns with symbols and dashing patterns, to represent different types of folds, faults, etc. (See an example here: https://macrostrat.org/map/#x=-114.633&y=51.0354&z=8.6&show=line-symbols,geology).
Is this functionality available in protomaps, or do you see a straightforward pathway to adding them? I might be willing to work on a pull request if it fit into the roadmap.

That functionality is not available directly in this library, but you could implement it via custom Symbolizer that calls the Canvas API. It looks like your Mapbox/MapLibre solution is looking quite well through, and for most sophisticated use cases like yours I would recommend that over this library.

I'm making this library more focused for 2.0, and the built in set of Symbolizers to support the default basemaps will be the only ones, though anything on Canvas is possible with custom Symbolizers. #112


[Question] How to make authn request with client_assertion_type and client_assertion by default

standing at:
https://openid.net/specs/openid-connect-core-1_0.html#ClientAuthentication
there's the need of also:
client_assertion and client_assertion_type
The authentication token MUST be sent as the value of the [OAuth.Assertions] client_assertion parameter.
The value of the [OAuth.Assertions] client_assertion_type parameter MUST be "urn:ietf:params:oauth:client-assertion-type:jwt-bearer", per [OAuth.JWT].

Do we need to have this behaviour as default in:
https://github.com/openid/JWTConnect-Python-OidcService/blob/d06bf271d6de51682c6a8a34a202c612855ec56f/src/oidcservice/client_auth.py#L382

youth errors


Stage 2 of Port App B deploy failed

Touching base to see if we need to fix anything our end

typo my end, appears to be working now


Add way to control contract validation in testing

Should be possible to turn off automatic contract testing, we could still run it but it might just be a warning instead of an error.

Default is on


Implement MiddleMouseButton view

File: com.almasb.fxgl.input.view.MouseButtonView

Hi! Can you assign this to me?

Go for it 👍


Failover

Scenario: Server with open connection goes down or we have new container deployed. Connections would be lost. How can we fail over/move the connection over to another container/server?
I am using Node js and Redis and would like to store the connection in Redis and have the ability to move them over to another server. Per the readme a connection only requires 104 bytes, so is it possible to a) store these 104 bytes in Redis and b) reestablish the connection on another server/container?

No you cannot do that. I've never heard of such a feature before.


New version: DaCosySheeep.FactDownloader version 1.3.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Exception during executable launch operation System.AggregateException: One or more errors occurred. (An error occurred trying to start process 'C:\Users\Validator\AppData\Local\Programs\Fact downloader\Fact downloader.exe' with working directory 'D:\TOOLS'. The specified executable is not a valid application for this OS platform.)  ---> System.ComponentModel.Win32Exception (216): An error occurred trying to start process 'C:\Users\Validator\AppData\Local\Programs\Fact downloader\Fact downloader.exe' with working directory 'D:\TOOLS'. The specified executable is not a valid application for this OS platform.    --- End of inner exception stack trace ---    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)

@wingetbot waivers Add Validation-Executable-Error


Is it impossible to put charts in separate components?

First of all: This is an absolutely great project!
So I'm building a chart with different lines depending on the input data, so I need to split out charts in the ChartCanvas and map them in like below.
<ChartCanvas>
    {chartData.map(LineChart)}
</ChartCanvas>

function LineChart(data) {
   return <Chart>...</Chart>;
}

But it seems I can't do that at all because I get errors whenever I put Chart elements in separate components instead of directly in the ChartCanvas.
Uncaught TypeError: Cannot read property 'origin' of undefined
    at t.value (Chart.js:107)
    at ReactCompositeComponent.js:793
    at s (ReactCompositeComponent.js:73)
    at m._renderValidatedComponentWithoutOwnerOrContext (ReactCompositeComponent.js:792)
    at m._renderValidatedComponent (ReactCompositeComponent.js:819)
    at m.performInitialMount (ReactCompositeComponent.js:359)
    at mountComponent (ReactCompositeComponent.js:255)
    at Object.mountComponent (ReactReconciler.js:43)
    at m.performInitialMount (ReactCompositeComponent.js:368)
    at mountComponent (ReactCompositeComponent.js:255)
    at Object.mountComponent (ReactReconciler.js:43)
    at g.mountChildren (ReactMultiChild.js:234)
    at g._createInitialChildren (ReactDOMComponent.js:701)
    at g.mountComponent (ReactDOMComponent.js:520)
    at Object.mountComponent (ReactReconciler.js:43)
    at g.mountChildren (ReactMultiChild.js:234)

It seems the Chart isn't loaded in the context.chartConfig anymore. Is there any way around this?

you are correct, right now Chart has to be a direct child of ChartCanvas. I have some ideas on how to change this but it would be a breaking change.

If you could implement that at some point that would be great. Because now I have to hardcode some stuff and that's quite annoying. Thanks!

Yes that is the idea, cannot promise any timelines though

Hi, Jim! What kind of hard code did u do? :)

Well I wanted to put Charts in separate components so I could map an array with the chart data over to those react components, now I had to do it hard coded like:
<Chart>...</Chart>
<Chart>...</Chart>
<Chart>...</Chart>

Right now I use pure functions for separating charts which returns chart component with content. And will be waiting for breaking changes by Ragu.

Is there any workaround this?

Just in case this helps someone else, I managed to get charts in child components, as long as they are  functions, not objects:
<ChartCanvas height={...} ...>
{ Charts1(some props/args) /* <<< this has to be a function, returning an array of  / }
{ Charts2(some other props/args) / <<< or a single  */ }

/* This is the function that returns an array of Charts. */
function  Charts1(args) {
return [
<Chart id={...} ... >...,
<Chart id={...} ... >...
]
}
function  Charts2(args) {
return <Chart id={...} ... >...
}


Update Enumerable.php

Fix Deprecated: Return type of YaLinqo\Enumerable::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /vendor/athari/yalinqo/YaLinqo/Enumerable.php on line 50 1

This project seems to be dead to be honest

This project seems to be dead to be honest

@miszczu It would seem that you are right. Have you found a viable alternative? I have a project that still uses this and I'm likely going to have to upgrade before this warning becomes a real issue.

@ekazda In my case it is used in a Laravel project, so we plan to switch to Illuminate\Support\Collection. Haven't look for other alternatives.

@ekazda In my case it is used in a Laravel project, so we plan to switch to Illuminate\Support\Collection. Haven't look for other alternatives.

This is what I ended up doing also. Thanks for the confirmation.


Add items arrangement actions

Common actions like "snap to side", "center items" etc.

You thought as I had thought. I'd like to "Zoom Back" often. I suggest that "Fit to View" button changes to "Zoom Back" after "Fit to View" button is clicked, so that you can go to whole canvas and return to local canvas.


updated eleganrl model based on new version

I have updated eleganrl agents based on latest eleganrl version. I have tested this code in finrl and both finrl and finrl-meta have same code of line.
please let me know if I can improve any place.  I will be happy to contibute :)

Hi. good work. thanks very much.
finrl-meta needs more data sources. I will appreciate it if you can submit PRs.


Update Workspace kedzia666

979ef87b-e9bf-46d8-b76f-d3c118f8f041
Changes

Checks
By submitting your pull request for review, you agree to the following:

[x] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[x] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[x] To the best of my knowledge, all proposed changes are accurate.

This doesn't appear to be a valid addition in this way. Are you desiring to add a code sample?


Use json-schema 0.7 to benefit "optional": true

- What I did
replaced [ string, null ] type definition with optional strings
- How I did it
Updated json-schema requirement to 0.7 so we can rely on optional
- How to verify it
I used gojsonschema to generate code and check it match the one from repo
(makes me wonder we could adopt a contract-first approach here)
- Description for the changelog
schema updated to rely on json-schema draft-07
- A picture of a cute animal (not mandatory but encouraged)

got confused with json schema spec versions


Using ol-tilecache with EPSG:4326 Tiles

The library is really helpful.
I have used custom tilegrids also. It is displaying the map but not the correct tiles. In Zoom 0 it shows only partial map and the tiles seems to be distorted. I am using OL5.
The code snippet is below, Please suggest resolution-
var projection = ol.proj.get('EPSG:4326');
var map_extent=projection.getExtent();
var origin = ol.extent.getTopLeft(projection.getExtent());
var resolution_map= [0.703125, 0.3515625, 0.17578125, 0.087890625,
0.0439453125, 0.02197265625, 0.010986328125,
0.0054931640625, 0.00274658203125, 0.001373291015625, 0.0006866455078125, 0.00034332275390625, 0.000171661376953125,
0.0000858306884765625, 0.00004291534423828125, 0.000021457672119140625,0.000010728836059570,0.000005364418029785];
var tileGrid = new ol.tilegrid.TileGrid({ resolutions: resolution_map,
tileSize: 256,
origin: origin });
var maplayer = new ol.layer.Tile({
source: new ol.source.XYZ({
tileUrlFunction: createTileUrlFunction('https://URL_To_Tilecache/{0z}/{x1}/{x2}/{x3}/{-y1}/{-y2}/{-y3}.jpeg',tileGrid,map_extent)
})
});

Hi @sonalanshika,
from snapshots that you send me I see that something wrong with  raster reprojection.
Try to set 'EPSG:4326' projection to the XYZ source or map view.
const map = new ol.Map({
  ...
  view: new ol.View({
    projection: 'EPSG4326',
  })
})

// and for the tile source cause by default it use EPSG:3857
var maplayer = new ol.layer.Tile({
  source: new ol.source.XYZ({
    projection: 'EPSG:4326',
    tileUrlFunction: createTileUrlFunction('https://URL_To_Tilecache/{0z}/{x1}/{x2}/{x3}/{-y1}/{-y2}/{-y3}.jpeg',tileGrid,map_extent)
  })
});

Hi,
Thank you. however it solved the problem only partially. It is not showing the complete extent still. My code-snippet is below. Please suggest further.
var projection = ol.proj.get('EPSG:4326');
var map_extent=projection.getExtent();;
var origin = ol.extent.getTopLeft(projection.getExtent());
var maplayer = new ol.layer.Tile({
source: new ol.source.XYZ({
tileUrlFunction: createTileUrlFunction('https://URL_To_Tilecache/{0z}/{x1}/{x2}/{x3}/{-y1}/{-y2}/{-y3}.jpeg',tileGrid,map_extent),
projection: projection
})
});
var mapview = new ol.View({
center: [78.0000 , 21.0000 ],
projection: 'EPSG:4326',
resolutions: resolution_map,
zoom: 0,
minZoom: 0,
maxZoom: 17
});
map = new ol.Map({
target: 'map',
projection: 'EPSG:4326',
layers: [maplayer ],
view: mapview
});

Hmm, still looks like a problem with view configuration or resolutions.
Try initialize view like below
var mapview = new ol.View({
  center: [78.0000 , 21.0000 ],
  projection: 'EPSG:4326',
  zoom: 0,
  maxZoom: 17,
});

And tile grid
var tileGrid = ol.tilegrid.createXYZ({
  extent: ol.proj.get('EPSG:4326').getExtent(),
  maxZoom: 17,
  tileSize: [256, 256],
});

Hi,
thanks It works now.  By adding tileGrid: tileGrid to the ol.source.XYZ.  while tilegrid is kept "ol.tilegrid.TileGrid...." only.
Thank Again.
Regards,
Sonal


apt install --install-recommends -y winehq-stable did not complete successfully

I am creating this issue for better visibility, as I already described what it was about in my comment here
I was hoping for a workaround to install winehq on a Centos7 system. Not sure if this is at all possible / anyone has attempted.
Any help is much appreciated on this front.

It is ok to use CentOS 7 as the host system and use Ubuntu as the base image of the container.
The two do not conflict. Just install Docker to your host system and then build the image without changing the base image in dockerfile.
Please refer to What is a Container? for more details.

I have finally sorted out this issue: the images were created without issues.
To wrap up, one has to perform a few changes in order for this to work:

Drop the "=" sign on line 13 in dockerfile.gamecore
Execute the following commands, instead of those suggested in run_windows_gamecore_on_linux,  to properly perform a multi-stage build

base_image=tencentailab/hok_env:cpu_v2.0.1
target_image=final_gamecore
internal_image=gamecore

docker build -f ./dockerfile/dockerfile.gamecore --target gamecore -t ${internal_image} --build-arg BASE_IMAGE=${base_image} .
docker build -f ./dockerfile/dockerfile.gamecore -t ${target_image} --build-arg BASE_IMAGE=${base_image} .

What also helped me immensely, was the dockerfile reference, so that I could understand the content inside dockerfile.gamecore.

I have finally sorted out this issue: the images were created without issues.
To wrap up, one has to perform a few changes in order for this to work:

Drop the "=" sign on line 13 in dockerfile.gamecore
Execute the following commands, instead of those suggested in run_windows_gamecore_on_linux,  to properly perform a multi-stage build.
Finally make sure that the docker image name is final_gamecore, as shown below:
docker run --network host -e SIMULATOR_USE_WINE=1 -it final_gamecore bash

base_image=tencentailab/hok_env:cpu_v2.0.1
target_image=final_gamecore
internal_image=gamecore

docker build -f ./dockerfile/dockerfile.gamecore --target gamecore -t ${internal_image} --build-arg BASE_IMAGE=${base_image} .
docker build -f ./dockerfile/dockerfile.gamecore -t ${target_image} --build-arg BASE_IMAGE=${base_image} .

What also helped me immensely, was the dockerfile reference, so that I could understand the content inside dockerfile.gamecore.


No module named 'paho'

I'm getting the following error even though i have paho-mqtt installed via pip3
Requirement already satisfied: paho-mqtt in /home/pi/.local/lib/python3.10/site-packages (1.6.1)
Traceback (most recent call last):   File "/home/pi/airthings-mqtt-ha/src/./airthings-mqtt-ha.py", line 24, in <module>     import paho.mqtt.publish as publish ModuleNotFoundError: No module named 'paho'
I am running Ubunut 22.04 (jammy) on rpi3.

If I remember correctly the bluetooth stuff needs sudo to run, so you will need to install paho without the --user option.

You can definitely install the modules globally, but you do not need to run as root provided you have run setcap on the bluepy-helper binary as follows:
sudo setcap 'cap_net_raw,cap_net_admin+eip' `find ~/.local/lib/ | grep bluepy-helper | grep -v bluepy-helper.c`

It has been a while, but my recollection is that root access is only needed for bluetooth scanning and not where you know the device's mac address, so you could also just run the script as root for scanning purposes and then revert to a non-privileged user.

If I remember correctly the bluetooth stuff needs sudo to run, so you will need to install paho without the --user option.
I ultimately decided last night to go back to raspbian instead of ubuntu, but i started to have the same issue.
So i it looks like since i cant run the /airthings-mqtt-ha.py --generate_config without root. installing the pip3 packages as root and it worked, but somewhere in there not running root works too so im not sure what happened.

I did run the sudo setcap 'cap_net_raw,cap_net_admin+eip' find ~/.local/lib/ | grep bluepy-helper | grep -v bluepy-helper.c`` again with verbose to make sure it was "OK"
Thanks for your imput, i appreciate it!


SQFT Finetuning

Description
This PR enables SQFT's NLS fine-tuning as a microservice in OPEA.
SQFT details can be found in the paper: SQFT: Low-cost Model Adaptation in Low-precision Sparse Foundation
Models (https://aclanthology.org/2024.findings-emnlp.749/)
Issues
N/A
Type of change
New feature (non-breaking change which adds new functionality) as detailed in our paper https://aclanthology.org/2024.findings-emnlp.749.pdf
Dependencies
Transformers, PEFT, and NNCF.
Tests
We include a test to validate the fine-tuning, similar to other existing fine-tuning approaches in OPEA.

Please fix the code scan issue and e2e test issue in CI test.
Submit each commit with git commit -s -m xxx to avoid DCO issue.

pls hold on this PR. we are syncing on how to deal with this one.


Install hangs on "System will reboot and load Emulationstation now after you hit the A button." I press the A button and nothing happens.

Before submitting your issue or feature request

This is not Anbernic Stock Emuelec or 351Elec.  ArkOS is a different distribution from those systems and will have differing features.  The only somewhat similarity is that they both use EmulationStation as a frontend.  Even with that, the versions of EmulationStation that's used in between ArkOS and Anbernic Emuelec/351Elec is different with different capabilities.  If there are features you prefer with Anbernic Emuelec/351Elec, just use those distributions instead.  The choice is yours.

Have you reviewed the Frequently Asked Questions available at https://github.com/christianhaitian/arkos/wiki/Frequently-Asked-Questions---RG351P ?

If your issue is related to being able to run certain emulators or ports, have you reviewed the Emulators and Ports section available at https://github.com/christianhaitian/arkos/wiki/ArkOS-Emulators-and-Ports-information for needed bios files and supported rom extensions?

Describe the Issue (If applicable)
A clear and concise description of what the bug is.
How can the issue be reproduced? (If applicable)
What device are you using?
What version of ArkOS are you on?

You can find this during the initial boot process right before the Emulationstation loading screen

Anything else you'd like to include that may help to review this issue or feature request?

It's ok to Just reboot manually when that happens.

Okay, thanks! I did that and now it states that no gamepads detected. Advice? Thanks so much!

Awesome thanks. Now the os boots, but the EasyRoms drive does not have folders in it?

No rom folders?  That shouldn't have happened.  Are you looking at this from your computer?

Any update on this issue?

I reflashed the card and everything is great!

I reflashed the card and everything is great!


BUG: Navbar componentsare not working in contributor page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
https://github.com/user-attachments/assets/ee846a80-b3db-4e90-a1f9-dffa17222faf
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@PriyaGhosal  please assign a level 3 token to it


Revalidate model after successful custom validation event broadcasted

Description
Custom validation of input fails if event of successful validation is broadcasted after input validation is executed in onChange method (for example custom validation takes some time - timeout, async request etc.).
Reproduce
Enter "yzy" as name, wait a second till custom validation with timeout will ran (you will get "exists" error message). Enter one more character wait one more second. Input will be marked as invalid although it is valid.
https://jsfiddle.net/t1a8nhop/
Fix relates with issues
#577
#637
#440

Thanks!

No thank you for your effort to help, the more the better, sorry it took so long we are going through some changes and there will be major updates to this framework in the not too distant future :)

Hi guys.
I have compile the current development version with the modification, it didn't correct the error for me.
Can someone confirm that building the dev branch work for him/her ? Thank.

Do you configure properly form? You have to set ngModelOptions: { allowInvalid: true }, that custom error broadcasting would work.
       "items": [{
          key: "name",
          onChange: 'onNameChange(modelValue, form)',
          ngModelOptions: {
            allowInvalid: true
          }
        }]

sorry I miss-understand your exemple. I'm using asyncValidator on the field.
I will raise a new issue for this.

It would be great if you could create a new release for this fix. When do you plan to get it out?
Thx

@raoulus I will do a build this weekend.

 :)

Cool, looking forward!

@Anthropic what about the release? :)

@raoulus another pull I accepted had another pull added and I needed some clarification on the previous steps taken for this project, so it had to wait, but it will be this week/weekend :)

@raoulus @cepauskas 0.8.13 is now in bower and the npm release will be follow by tomorrow


Ubiquiti Data Connector

Proposed Changes

add Ubiquiti data connector
add Ubiquiti workbook
add Ubiquiti datection rules and hunting queries

@vu-socprime - Please fix for validation errors.
@shainw - Please review parser, analytic rules and hunting queries
@v-jayakal - Please review data connector and workbook
Thanks,
Preeti

@vu-socprime : Please fix validation errors

Please rename the sample data file to that same as custom logs table name.
here it should be as : Ubiquiti_CL


[NFR] PHP74 support for windows

For building Phalcon on AppVeyor we're using the pre builded windows DLL's. PHP74 isn't available (zephir_parser_x86_vc15_php7.4_1.3.2-505.zip)
Can we add them?

Implemented in v1.3.3


Stampede allocation in config file currently has -4 SUs

ahh.. its using the wrong project. I thought I changed it in all the files. which file is this ?

config in examples

Thanks Antons. Thought I had it updated, but I guess not. I am making a new branch for all the non-documentation related issues pointed out by you and George. I'll update the testing sheet to the same.


fix(#3881): allow exclude to work with node API

PR checklist

[x] Addresses an existing issue: #3881
[x] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

Overview of change:
Moves isFileExcluded from runner to configuration, then import it to both runner and linter.
No documentation update because this is already what is expected from the existing documentation.
CHANGELOG.md entry:
[bugfix] allow exclude to work with node API

Thanks for your interest in palantir/tslint, @nspaeth! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

looks great! thanks @nspaeth

@palantirtech Can you please include this fix in a release as soon as possible? It makes tslint almost unusable when yarn run lint produces a bunch of errors that should have been ignored by
  "linterOptions": {
    "exclude": [
      "**/auto-generated/*.ts"
    ]
  }

but is not.

I'm having issues with this using Angular8. I updated to the latest versions and tried all solutions but was still not able to lint successfully. My solution was in the end to exclude the file directly from the angular.json
"lint": {
"builder": "@angular-devkit/build-angular:tslint",
 ...
"exclude": [
            "**/node_modules/**",
            "**/*.json"
          ]

I've tried @ChrisBertrand's way, and it works. However, I'm just a newbie. Could somebody let us know if there will be any side effect (negative points) of doing so please?

I got it working after a bunch of different combinations:
"linterOptions": {
    "exclude": ["**/src/app/shared/services/generated/**"]
  },

thanks @mswilson4040 it's working!
@everyone: try the pattern [path/to/exclude], this seems to work fine!


Improve CloudOpts argument --identity with better help text

Improve CloudOpts argument --identity with better help text
Current behavior
The command ockam space show --identity <IDENTITY_NAME> runs the command with the given identity name. And this is what you see when you run ockam space show --help.

There are many other commands which end up using the same CloudOpts argument struct. And here are some of them:

ockam space list --help
ockam space delete --help
ockam project ticket --help

Desired behavior
The ockam space list --help command should provide some more information on the --identity flag. So should all the other commands in the list above.
To do this:

Add a comment to the field describing what it is: /// Run the command as the given identity name.
To make this change take a look at this file. Add a doc comment w/ the text above to the CloudOpts::identity field.

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

I have already started working on the issue. Can you please assign it to me?

@0scvr I just assigned this issue to you.
@ironspec07 I just assigned the issue to @0scvr. How far along are you in working on this issue? We have plenty of other issues for you to look at if you haven't spent too much time on this 👍🏽 . Do you have a PR raised against it?

ok no problem. I'll find another issue, Thanks

@ironspec07 Awesome! Thank you 👍🏽


add allocated bytes to runtime statistics

What problem does this PR solve?
Issue Number: close #7421
Problem Summary:
What is changed and how it works?
Add and log allocated_bytes in each executor in MPPTaskStatistics
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge


execute ./scripts/patch.sh error?

[jack@localhost cpp_workspace]$ git clone https://github.com/microsoft/MSVBASE.git
Cloning into 'MSVBASE'...
e
./scripts/patch.shremote: Enumerating objects: 433, done.
remote: Counting objects: 100% (15/15), done.
remote: Compressing objects: 100% (11/11), done.
remote: Total 433 (delta 8), reused 4 (delta 4), pack-reused 418
Receiving objects: 100% (433/433), 40.23 MiB | 1.21 MiB/s, done.
Resolving deltas: 100% (240/240), done.
[jack@localhost cpp_workspace]$ cd MSVBASE
[jack@localhost MSVBASE]$ git submodule update --init --recursive
Submodule 'thirdparty/Postgres' (https://github.com/postgres/postgres.git) registered for path 'thirdparty/Postgres'
Submodule 'thirdparty/SPTAG' (https://github.com/microsoft/SPTAG.git) registered for path 'thirdparty/SPTAG'
Submodule 'thirdparty/hnsw' (https://github.com/nmslib/hnswlib.git) registered for path 'thirdparty/hnsw'
Cloning into '/home/jack/cpp_workspace/MSVBASE/thirdparty/Postgres'...
Cloning into '/home/jack/cpp_workspace/MSVBASE/thirdparty/SPTAG'...
Cloning into '/home/jack/cpp_workspace/MSVBASE/thirdparty/hnsw'...
Submodule path 'thirdparty/Postgres': checked out 'e849f3f1f884ad140b60a24354c6371cbd2efbb6'
Submodule path 'thirdparty/SPTAG': checked out 'de5b7f8c57c45c31ac1caeaf3b9f1d0e1668c395'
Submodule 'ThirdParty/zstd' (https://github.com/facebook/zstd) registered for path 'thirdparty/SPTAG/ThirdParty/zstd'
Cloning into '/home/jack/cpp_workspace/MSVBASE/thirdparty/SPTAG/ThirdParty/zstd'...
Submodule path 'thirdparty/SPTAG/ThirdParty/zstd': checked out 'e47e674cd09583ff0503f0f6defd6d23d8b718d3'
Submodule path 'thirdparty/hnsw': checked out '443d667478fddf1e13f2e06b1da4e1ec3a9fe716'
[jack@localhost MSVBASE]$ ./scripts/patch.sh
../../patch/SPTAG.patch:253: trailing whitespace.

../../patch/SPTAG.patch:258: trailing whitespace.

../../patch/SPTAG.patch:260: trailing whitespace.

../../patch/SPTAG.patch:262: trailing whitespace.

../../patch/SPTAG.patch:276: trailing whitespace.

warning: squelched 27 whitespace errors
warning: 32 lines add whitespace errors.
../../patch/hnsw.patch:393: trailing whitespace.

warning: 1 line adds whitespace errors.
../../patch/Postgres.patch:44: indent with spaces.
        estate->is_index_inorder = scandesc->xs_inorder;
../../patch/Postgres.patch:45: indent with spaces.
        //elog(INFO, "node is in order : %d", estate->is_index_inorder);
../../patch/Postgres.patch:54: indent with spaces.
        estate->is_index_inorder = scandesc->xs_inorder;
../../patch/Postgres.patch:55: indent with spaces.
        //elog(INFO, "node is in reorder : %d", estate->is_index_inorder);
../../patch/Postgres.patch:81: indent with spaces.
        //int step = 0;
warning: squelched 39 whitespace errors
warning: 44 lines add whitespace errors.

### Tasks
- [ ] https://github.com/microsoft/MSVBASE/issues/7

Looks like all messages are warnings. Let's figure out how to resolve these warnings.

2023年12月12日 13:49，zqxjjj @.***> 写道：
Looks like all messages are warnings. Let's figure out how to resolve these warnings.
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/MSVBASE/issues/9#issuecomment-1851343998, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOKP45XI4H5GDBJZ6JOMP73YI7WANAVCNFSM6AAAAABAHLBC6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJRGM2DGOJZHA.
You are receiving this because you authored the thread.

Ok, there is an another problem, the docker build is too slow. Can you provide a local way to build this repo?


Denture Fix - Uninstall dentures option missing

Surgery option to uninstall dentures is not visible with Denture Fix mod enabled

Temporary fix re-enables uninstalling dentures but it grows the jaw back too.


Compile C / C++ code with fpic

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Follow-up for discussion https://github.com/Homebrew/homebrew-core/pull/112792:

Added --with-fpic for configure-based projects
Added CMAKE_POSITION_INDEPENDENT_CODE for cmake based projects.
For meson we don't have to add anything since static libraries are built with -fpic by default (see default value for b_staticpic)

@Homebrew/brew @Homebrew/core @Homebrew/linux casting a wide net as want a lot of opinions here:

does this make sense on Linux?
does this make sense on macOS?

My thoughts on this:

On principle, I tend to be hostile to changes that affect all 6373 formulas without being able to actually test them.
We currently have only 25 formulas that set fPIC in various ways (and a dozen more that use it in their tests, but that wouldn't be affected by this proposal). So the need does not appear to be great.
The burden of adding args << "--with-pic" if OS.linux? on formulas that actually need it is not that heavy.
I am not very involved with the Linux side of formulas, so if the maintainers who deal with it on a daily basis feel strongly about it, I won't object, if related failures in future PRs are fixed or do not block merging.

I am, however, strongly against it for macOS. Yes, it should be a no-op there, but given the number of weird-assed configure or cmake scripts we see around, I feel it's likely to break some stuff anyway. Even a 0.5% failure rate on 6373 formulas will mean 30 formulas to debug and fix.

Agreed with @fxcoudert and @sjackman that this should be macOS only.
This may be a terrible idea but I'll throw it out anyway - could this be added to the compiler shim instead for default arguments to build systems?  I'm only suggesting this because I think that would be more analogous to how -fPIE is automatically added by Ubuntu's GCC.
It's may also be worth noting here for the sake of record that the primary historical reason this probably isn't a default in Ubuntu and other distros was because on 32-bit x86 systems (and probably older ones as well), there was a non-trivial performance impact in some cases when -fPIC was used.  However this concern is irrelevant on 64-bit x86 systems, (or ARM, to my knowledge).  I do not know how many, if any, 32-bit x86 users are building formulae from source but that should not be a use case that we try to support.

-fPIC is stronger than -fPIE: the second one only allows the position-independent to be only linked into executables, while -fPIC allows the code to be linked in both executables and shared libraries.
PS: Given most linux vendors ship gcc configured with --enable-default-pie, why are we not doing it?

Given most linux vendors ship gcc configured with --enable-default-pie, why are we not doing it?

I'd say we probably should be.

Given most linux vendors ship gcc configured with --enable-default-pie, why are we not doing it?

I'd say we probably should be (in addition to this PR).

This is definitely a good idea, as it will avoid problems when we compile Fortran binaries on Linux with brewed GCC that link to C/C++ libraries that were built with the host GCC using -fPIE.  That's probably why https://github.com/Homebrew/homebrew-core/pull/115963 failed for example.

Homebrew/homebrew-core#112792 (comment)
When the problem can be solved by modifying std_*_args I'd prefer to do that. One reason is that it's easier to undo the change for a particular formula if needed. I'm not strongly against doing it in the shim though if that's the preference. As Mike said above though

we probably need a way to disable it too.

I agree that we need a way to disable it (maybe with an ENV.no_pic?).  The reason I was somewhat leaning towards adding it to the compiler shims is because we have our fair share formulae which either don't use std_configure_args/std_cmake_args, use a different build system, or use some handmade Makefile or custom build system.  Figuring out how to get -fPIC as a flag into all those different contexts could be a real pain, where as having it in a compiler shim saves a lot of trouble.

I'd like to see some testing of that before we make it the default.

💯 agree with that. We should base this decision on actual testing.

I agree that we need a way to disable it (maybe with an ENV.no_pic?)

If it's not a universal benefit, and there are cases where it might break stuff and need to be disabled… I don't think we should make it a default. It's not that hard to add to formulas that actually need it (instead of all formulas that might need it, plus an unquantified number that would break).

I agree with @danielnachun and @fxcoudert. Doing this at the compiler shim level makes more sense than changing every formula for a "maybe".

It would make some sense on compiler shim level, but we'd need to make sure -fno-pic is respected, if the build passes it. I say this makes sense because some "hardened" compiler setups in other distributions do something pretty much exactly like this, though that's sometimes only for the top 1000 packages or so, so we may need an opt-out.
I'd very much would like to try it, but I feel like this is something that may need to be tested a bit more before merging.
Though I will also say that we've had the request for exactly one formula (I think?) so far and I don't really agree with the objections made against the change in that formula and would suggest merging the fix for protobuf only (where we know it works) while we discuss whether it's possible to do so by default for things where we don't yet know if it universally works.

@fxcoudert wrote…

Given most linux vendors ship gcc configured with --enable-default-pie, why are we not doing it?

I wrote…

I'd say we probably should be (in addition to this PR).

@danielnachun wrote…

This is definitely a good idea, as it will avoid problems when we compile Fortran binaries on Linux with brewed GCC that link to C/C++ libraries that were built with the host GCC using -fPIE. That's probably why https://github.com/Homebrew/homebrew-core/pull/115963 failed for example.

@danielnachun Would you like to volunteer to open this PR to add --enable-default-pie to the GCC formulae?

@MikeMcQuaid wrote…

I'd like to see some testing of that before we make it the default.

@fxcoudert wrote…

💯 agree with that. We should base this decision on actual testing.

@Bo98 wrote…

If it can work, I'd very much would like to try it given some moves made by some Linux distributions, but I feel like this is something that may need to be tested a lot more before merging.

$ git grep -l std_configure_args | wc -l
     307
$ git grep -l std_cmake_args | wc -l
     811
$ calc 307+811
	1118

@ilya-lavrenov Would you be up for testing this PR by building from source ~100 of these formulae (selected randomly) that use either std_configure_args or std_cmake_args? I still can't guarantee this PR will be merged even if you do that work, but that testing is a requirement for merging this PR.

@MikeMcQuaid wrote…

It seems we have mild consensus that: this should be added per-formula, for now at least

Shall we circle back and merge @ilya-lavrenov 's original PR?

https://github.com/Homebrew/homebrew-core/pull/112792

I can definitely open the PR to --enable-default-pie for GCC on Linux, as there seems to be universal consensus that we should be matching what our own host compiler does.  I'll start with GCC 12 but I think we might also want to do GCC 11 for users building C/C++ from source.

Shall we circle back and merge @ilya-lavrenov 's original PR?

protobuf: added PIC compilation flags homebrew-core#112792

Yes 👍🏻

I still can't guarantee this PR will be merged even if you do that work, but that testing is a requirement for merging this PR.

Just for context: it probably won't be, sorry!

Shall we circle back and merge @ilya-lavrenov 's original PR?

protobuf: added PIC compilation flags homebrew-core#112792

Yes 👍🏻

I still can't guarantee this PR will be merged even if you do that work, but that testing is a requirement for merging this PR.

Just for context: it probably won't be, sorry!

I think the conclusion at the moment may be that moving forward, we should generally allow adding -fPIC on Linux if its absence is reported to cause issues for users provided that it does not cause a build failure.
Testing random formulae may not be terribly helpful - many if not most packages already add it so testing those packages isn't very informative in understanding if it causes failures.  For that reason it seems the right solution is just to deal with this on a formula by formula basis unless we encounter a scenario where a large number of formulae are affected.

Passing on this, sorry @ilya-lavrenov!

Passing on this, sorry @ilya-lavrenov!

great that we have the conclusion now, thanks!


The empty method of Selection not override the values

Describe the bug
When I called empty method after the filing methods like fromModel or options not overwritten the values
Expected behavior
I expect to when I use this method before filling methods will overwrite the values as shown in documents
Screenshots
orchid_issue
Desktop (please complete the following information):

Windows OS
Chrome Browser

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Server (please complete the following information):

Platfrom Version: [e.g. 10.4]
Laravel Version: [e.g. 8.40]
PHP Version:  [e.g. ^7.3|^8.0]
Database:  [e.g. MySQL]
Database Version:  [e.g. 10.4.18]

Hi @mahmoudmohamedramadan . Can you explain the problem in more detail.
Because the screenshot above shows that everything is working correctly.

Hi @tabuna, I mean that as you explain in Orchid Docs that I should suffix the filling methods like options in my code WITH the empty method otherwise the added value will be overwritten, BUT when I try to prefix the empty method WITH the filling methods as there is NO difference, the expected result that the key in the first param in the empty should overwrite the options values...it's true or NOT?

Hi this footnote is no longer relevant for the latest versions.  I didn't really like the fact that you need to follow a certain order of calls.  Therefore, I changed the honey empty, so that it would be executed last

Awesome, BUT sir, this method remains exist in DOCS


Failing e2e tests - scorecard-head on ossf-tests/scorecard-action

Matrix: null
Repo: https://github.com/ossf-tests/scorecard-action/tree/main
Run: https://github.com/ossf-tests/scorecard-action/actions/runs/5472166417
Workflow name: scorecard-head
Workflow file: https://github.com/ossf-tests/scorecard-action/tree/main/.github/workflows/scorecards-head.yml
Trigger: push
Branch: main

Was likely fixed by https://github.com/ossf-tests/scorecard-action/pull/4


Nuget restore fails locally using the CLI

Package Restore takes a dependency on kudu existing
You get a host error on attempting and failing:
2016-10-23T03:46:57.387 Executed: 'Functions.cs-http' (Succeeded)
2016-10-23T03:47:11.685 Restoring packages.
2016-10-23T03:47:11.685 Package restore failed:
2016-10-23T03:47:11.685 System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Program Files (x86)\siteextensions\kudu'.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileSystemEnumerableIterator`1.CommonInit()

Additionally, finding this error is hard since it is in the host logs. It might be good to surface host and function logs in the cli console window.

I was able to partially mitigate this by:

Setting my Nuget path here: $env:AzureWebJobs_NuGetPath
Running nuget.exe restore manually

I get this error when modifying the project.json, thus requiring manual restore:
2016-10-23T04:13:23.547 Restoring packages.
2016-10-23T04:13:23.549 Starting NuGet restore
2016-10-23T04:13:23.549 Package restore failed:
2016-10-23T04:13:23.553 System.ComponentModel.Win32Exception (0x80004005): Access is denied
   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start()
   at Microsoft.Azure.WebJobs.Script.Description.PackageManager.RestorePackagesAsync()

Glad to hear the 2nd issue was just a config problem, as I run that scenario frequently and haven't seen the issue.
The original issue has been addressed in the current runtime bits, so upgrading what the CLI is using will fix it.

@ahmelsayed Please close this issue when you update the current CLI


Add new quarkus benchmark parsers

-Added parser for crud/ restCrud/ pingperf/ getting-started quarkus benchmark parsers
related to issue #188
Signed-off-by: Dong Hoon Lee<EMAIL_ADDRESS>

@dhlee49 It's missing some metrics for benchmarks. For example, for getting started, we'll need the following metrics:

average latency
latency max
latency standard deviation
RSS after first request
RSS after the process started
RSS after the process was warmed up with requests
startup time
throughput

I've mentioned the metrics for each benchmark in their issue for adding them, so you can take a look what metrics are missing.

@dhlee49 I believe we're done writing parsers for all metrics related to Quarkus, so let's update this PR.

@piyush286
I made mistake with this PR using master branch so I created new branch for this but that lead to closing the issue, could you reopen the PR so I can swap out the branch?
Thanks in advance!

I don't see an option to reopen it. Feel free to open another PR for it.


[release-4.15] OCPBUGS-36606: enable audit log for oauth-openshift

This is a manual 4.15 cherry-pick of https://github.com/openshift/hypershift/pull/3994

/jira cherrypick OCPBUGS-33060

/jira refresh

/approve

/jira refresh

/retest-required

/retest-required

/retest-required


fix for issue #91 - Fade Out UI throws NPE for EventToastManagerFrame

some frames don't have parents I guess?

Thanks again! :-)


CSS rewrite for the Settings Page & updated EN and DE translations

Can you provide a screenshot of the new page?

Can you provide a screenshot of the new page?

Sure here :


Add some paths conditions to workflows

Some small improvements to actions:

Test invoke.py pip now ignores if all changes are in docs/** or invokeai/frontent/** as changes to these paths cannot affect generation
pyflakes same change as Test invoke.py pip
mkdocs-material now runs only if the push includes changes to docs/**

Closing per internal discussion

sorry, could have added the changes from #2688 here, did not realize it was still open :-(


Rotation speed issue / alert

I got this error (but no consequences in operational terms):
[12/19/2023, 11:06:04 AM] [homebridge-daikin-cloud] This plugin threw an error from the characteristic 'Rotation Speed': Unhandled error thrown inside read handler for characteristic: Cannot read properties of null (reading 'value'). See https://homebridge.io/w/JtMGR for more info.
homebridge.log.txt
Please check Homebridge's log I attached here.
Greetings, Simone!

@SAzzinelli this should be fixed in version<EMAIL_ADDRESS>Closing this for now. Come back to me if this problem would still exist.


SIP: Adding WebAuthn Support to zkLogin

This SIP proposes adding WebAuthn support to zkLogin, allowing the use of WebAuthn authenticators for managing ephemeral keys. This integration aims to leverage the security and convenience of WebAuthn to enhance zkLogin's functionality, making it more accessible and user-friendly.

Closing after discussion. Vanilla r1 signatures can be added inside zkLogin already today, whereas this proposal would require protocol changes.


Draft ppp

Codecov Report

Merging #227 (c812240) into master (7d672b1) will increase coverage by 11.95%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master     #227       +/-   ##
=============================================
+ Coverage     27.34%   39.29%   +11.95%
- Complexity      409      611      +202
=============================================
  Files           150      150
  Lines          3364     3364
  Branches        403      403
=============================================
+ Hits            920     1322      +402
+ Misses         2379     1941      -438
- Partials         65      101       +36

Impacted Files
Coverage Δ

...seedu/address/logic/commands/DeleteAB3Command.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...u/address/logic/parser/DeleteAB3CommandParser.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/seedu/address/logic/commands/EditAB3Command.java
0.00% <0.00%> (-95.53%)
:arrow_down:

...main/java/seedu/address/model/AB3ModelManager.java
85.10% <0.00%> (-10.64%)
:arrow_down:

src/main/java/seedu/address/model/tag/Tag.java
70.00% <0.00%> (-10.00%)
:arrow_down:

...c/main/java/seedu/address/model/person/Person.java
90.47% <0.00%> (-7.15%)
:arrow_down:

src/main/java/seedu/address/model/AddressBook.java
55.76% <0.00%> (-5.77%)
:arrow_down:

...address/model/assessment/UniqueAssessmentList.java
23.40% <0.00%> (+4.25%)
:arrow_up:

...in/java/seedu/address/logic/parser/ParserUtil.java
23.68% <0.00%> (+6.57%)
:arrow_up:

...va/seedu/address/logic/commands/CommandResult.java
83.33% <0.00%> (+8.33%)
:arrow_up:

... and 32 more

Continue to review full report at Codecov.

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


[Gold Custom]: SDPR/MCFD - Social Sector

Product Owner Contact Details
<EMAIL_ADDRESS>Technical Owner Contact Details
<EMAIL_ADDRESS>Secondary Technical Owner Contact Details
<EMAIL_ADDRESS>Silver custom realm name
gde0rjxl, secimg
New realm name
social
Gold environment needed

[X] Dev
[X] Test
[X] Prod

Code of Conduct

[X] I agree to follow Pathfinder SSO's Gold Custom Realm Community Ways of Working

Can new realm name in gold cluster be more meaningful, something like socialSector or ss for short?

Can new realm name in gold cluster be more meaningful, something like socialSector or ss for short?

@f-w  and @ll911  can you please suggest the new name?

updated to social

provisioned.


WebXR option to display hand mesh instead of controllers

Is your feature request related to a problem? Please describe.
As a player of a first person shooter game, when playing WebXR immersive mode, I would like to see hand meshes representing my players hands even though i'm holding on to Quest Controllers, so that I can see my hands animate when I make a fist, point, wave hi at someone, grab onto objects, hold a gun, hold a sword, etc.
Describe the solution you'd like
I would like the default webXR experience to have an option for setting up default generic hands for me.  useDefaultHands: true or something like that.  I would like the mesh hands to automatically animate to "a pointing" finger if the grip button is down and the trigger button is up.  Or a "fist" if both the trigger and the grip button are held down.
In terms of games, I think I might need the ability to dial in certain hand shapes when certain actions take place.
Idea is to be able to dictate: When hand mesh position is close to lever mesh position and grip button is down, then close the fist by 50% (like pulling down a lever).
Just a handful of stock poses might just cover 90% of all hand use cases in which the hands hold things (relaxed hand, fingers spread, closed fist, index finger point, gun grip).  For example fingers spread wide can hold a softball.  Then we just parent the hand to the thing we want to hold and dial in the pose to 30% fingers bent.  Whereas holding a sword or a bat has a smaller diameter so we parent the hand to the object we want to hold then dial the pose down to 75% of a closed fist.
I would also like a clear API for adding additional animations or overriding a mesh with my own custom hand meshes.
Discussion
https://forum.babylonjs.com/t/are-there-any-existing-models-for-webxr-hands-available/14584/7

Keeping it for future releases ATM, becuase i can't still estimate how many tasks I will finish by code freeze for 4.2.

Hello All,
Is there any update on this ? Can we have Hand Menu instead of controllers ?

You can use the controllers on the hand menu as well (enabling the near-interactions feature). Or is your goal to show hands instead of controllers?

Hey Raanan, can u please share HandMenu Playground (with near-interaction feature on).. I am able to see hands instead of controllers but i want some kind of menu (3D GUI buttons or anything else) to open on hands gesture

This would be really helpful for the baseball game I'm working on. I'd like the controllers to look like hands so that I could hold the ball and bat. The hand tracking feature is cool, but I feel like that would turn off certain people from playing.

that's not an easy feature to implement, and IMO should actually come directly from the vendors. Otherwise we will need to have hand positions for each type of available controller. I don't remember seeing this feature suggested for the WebXR specs, but I will try researching some more

Require supports from vendors.


Fix the way HustleBook parses numbers

Right now, when the user inputs a number, it will show inconsistent error messages depending on the number they put.
For example, if they run edit david p/123 and they are asked to give a confirmation of the index of the client to edit, the number "0" will give Unknown command as an error message instead of The person index provided is invalid. Error messages for wrong index is also inconsistent across edit, delete find and meet commands.
Let's,

Fix the way HustleBook recognises numbers
Fix the logic of the commands to ensure consistent error messages

Closes #209, Closes #195

Codecov Report

Merging #234 (d75f047) into master (a90ffcb) will decrease coverage by 0.38%.
The diff coverage is 20.00%.

@@             Coverage Diff              @@
##             master     #234      +/-   ##
============================================
- Coverage     58.51%   58.12%   -0.39%
- Complexity      473      474       +1
============================================
  Files            96       96
  Lines          1991     2006      +15
  Branches        226      227       +1
============================================
+ Hits           1165     1166       +1
- Misses          744      758      +14
  Partials         82       82

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/FlagCommand.java
0.00% <0.00%> (ø)

...java/seedu/address/logic/commands/MeetCommand.java
0.00% <0.00%> (ø)

...a/seedu/address/logic/parser/HustleBookParser.java
66.66% <0.00%> (ø)

src/main/java/seedu/address/ui/CommandBox.java
0.00% <0.00%> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
67.74% <40.00%> (-7.26%)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java
88.88% <40.00%> (-2.54%)
:arrow_down:

...in/java/seedu/address/commons/util/StringUtil.java
95.00% <100.00%> (+0.26%)
:arrow_up:

Continue to review full report at Codecov.

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


Release 1.10.2 hotfix

https://project-sunbird.atlassian.net/browse/SB-8367

Wrong pull request

wrong pull request


VAT number can't be optional in Laravel 5.4

Hello
I just upgraded a project to Laravel 5.4 and it seems the VAT number field is now always required even if it's empty and no required validation rule is set. I don't think this was the case before?
As a quick work around, I copied your validation rule and added a check if the value is empty:
Validator::extend('vat', function($attribute, $value, $parameters, $validator) use ($cooldown) {
    if ( ! $value) {
        return true;
    }

    $validator->setCustomMessages([
        'vat_number' => $validator->getTranslator()->get('vatnumber-validator::validation.vat_number'),
    ]);

    try {
        return VatCalculator::isValidVATNumber($value);
    } catch (VATCheckUnavailableException $e) {
        return false;
    }
});

If I want it to be required, I can now add the required rule manually.

Doh, sorry... just found this in the Laravel docs:

By default, Laravel includes the TrimStrings and ConvertEmptyStringsToNull middleware in your application's global middleware stack. These middleware are listed in the stack by the  App\Http\Kernel class. Because of this, you will often need to mark your "optional" request fields as nullable if you do not want the validator to consider null values as invalid.

The nullable rule did the trick... 👍


Create dataset loader for indo_story_cloze

Dataloader name: indo_story_cloze/indo_story_cloze.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?indo_story_cloze

Dataset
indo_story_cloze

Description
This is the first commonsense reasoning dataset in Indonesian. IndoStoryCloze is a story in our dataset that consists of the four-sentence premise, one-sentence correct ending, and one-sentence incorrect ending. In total, we have created 2,325 Indonesian stories with the train/dev/test split 1,000/200/1,135.

Subsets
-

Languages
ind

Tasks
Commonsense Reasoning

License
MIT (mit)

Homepage
https://huggingface.co/datasets/indolem/indo_story_cloze

HF URL
https://huggingface.co/datasets/indolem/indo_story_cloze

Paper URL
https://aclanthology.org/2022.csrr-1.2.pdf

#self-assign

Hi @chenxwh, may I know the progress of this dataloader creation? Since it has passed 2+2 weeks of expected completion of dataloader, I will clear the assignee if no update is received by Monday 12 PM UTC.

Hi @sabilmakbar,
It is implemented and the PR is here: https://github.com/SEACrowd/seacrowd-datahub/pull/323

Thanks @chenxwh, didn't see that.
For coming PR, mind addisng this in the initial PR message:
Closes #179
so that it will be linked to this issue and checking process will be easier


Change executable template to use main.swift

The recent changes to templates switched this template away from using @main which seems to not be compatible with Xcode's build system. The only non-@main way for executables that is supported is naming the primary source file "main.swift", so that is what this PR does.
rdar://106044582
(cherry picked from commit 6fd2633da998331adeab25db4bf1df29fa11437d)

@swift-ci please smoke test


Fix failing Nox tests session on Windows

This PR will close #307
Description
Fixes the failing nox tests session on Windows.
Testing Instructions

[ ] Review code
[ ] Check GitHub Actions build
[ ] If icatdb Generator Script Consistency Test CI job fails, is this because of a deliberate change made to the script to change generated data (which isn't actually a problem) or is here an underlying issue with the changes made?
[ ] Review changes to test coverage
[ ] Does this change mean a new patch, minor or major version should be made? If so, does one of the commit messages feature fix:, feat: or BREAKING CHANGE: so a release is automatically made via GitHub Actions upon merge?
[ ] nox tests session should not fail
[ ] The version of the installed setuptools should be <58.0.0

From GitHub Actions


Style clone function doesn't copy the renderer function

I find this:
The new style options don't have a renderer property.
https://github.com/openlayers/openlayers/blob/1a952e7a41a030791780af6086ca4ad91e30e6f0/src/ol/style/Style.js#L213-L226

Someone correct me if I'm wrong, but wouldn't this require a deep copy of the renderer property?
The obvious solution, of course, would be to just add renderer: this.getRenderer() ? this.getRenderer() : undefined, into the properties of the new Style, but that would create issues because it would only pass the renderer function by reference.
To properly clone the renderer function, I think a deep copy of the renderer function object would be needed.  Core JavaScript doesn't provide a good way of creating a deep copy of an object that contains functions, so a true solution would require either a custom deep copy function to be added to OpenLayers, or the use of a third-party library that provides a deep copy function.  A third-party library would add another dependency, since I don't believe any of the current OpenLayers dependencies provide deep copy functions.

I've wondered before whether a deep copy of a function is necessary.If a function is defined and used according to the standard specification, there is no need for a deep copy.I understand that many circumstances need to be considered for the convenience of deep copy of the function. In this case, I suggest some supplementary instructions on style.clone in the API documentation to make it clear that no copy will be carried out for the renderer function.

There seems to be no attempt to deep clone functions in the library.  If geometry is a function there is also no deep clone.  The style of a feature is not deep cloned, regardless of whether it is a function.

In that case, taking a cue from how clone() handles the geometry property, we should be able to do renderer: this.getRenderer() to get the same kind of copy as geometry would get if it were a function.
Would that be an appropriate fix for this issue, or would it be better to leave clone() as is and simply state in the API that the renderer function is not copied?

I created PR #11893 to move the issue forward.

I created PR #11893 to move the issue forward.


Added feature to create a remote port forwarding session to a selected RDS instance

The user is now able to start a remote port forwarding session from a selected remote port to a selected local port for any given RDS instance (Only RDS instances allowed as of now).
General workflow:

Bastion instance is created
User selects a RDS instance to connect to
The security group of the RDS instance is changed to allow traffic from the bastion security group
Session is started/active
User terminates session when they finish their work
Security group changes are reverted
Bastion instance and session are terminated

Example : Create a session with an RDS Instance

User calls the function 'port-forward --remote-port 5432 --region ap-southeast-2' which prompts them to create a new bastion instance

Once the bastion is launched the user is asked to select which RDS instance they wish to connect to.

We observe that the RDS instance security group adds a rule to allow traffic from the bastion security group

The session is started once the connection is established

We connect to the RDS instance via our localhost on the selected local port which in our case is 5432 (When localhost is not explictly set it defaults to the remote port)

We are able to successfully connect to the RDS instance via our localhost on port 5432

After the user completes their work they terminate the session.

We observe that the bastion instance is terminated & the security group rule has been removed.

Added following changes:

Created new file 'portforward.go' which contains functions for creating port forward session & bastion

Created new file 'rds-authorisation' which contains functions for finding rds instances and modifying their security groups.

Refactored 'CreateDefaultBastion' to replicate functionality of CmdLaunchBastion and utilise the cli context. As such is able of using all same configurable flags.

Added prefix Cmd to StartRemotePortForwardSession function.

Added remote host flag which can be used to specify specific instance to connect to.

Refactored port rules on security group to only open the remote port specified on TCP rather than all ports for all protocols.

Re-ran workflow test and was able to connect to RDS instance on localhost as expected.