Still having issue after following your "NPM Debugging Guidelines"

Hi Ben,
I still have issue after following your suggestion.
Below is the setup and error info
$ node -v && npm -v && gulp -v
v4.1.0
3.3.3
[09:44:29] CLI version 3.9.0
$ npm cache clear && rm -rf node_modules && npm install
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

<EMAIL_ADDRESS>install /Users/myname/code/sage/wp-content/themes/sage-nest/node_modules/bufferutil
node-gyp rebuild

CXX(target) Release/obj.target/bufferutil/src/bufferutil.o
In file included from ../src/bufferutil.cc:16:
../node_modules/nan/nan.h:261:25: error: redefinition of '_NanEnsureLocal'
NAN_INLINE v8::Local _NanEnsureLocal(v8::Local val) {
^
../node_modules/nan/nan.h:256:25: note: previous definition is here
NAN_INLINE v8::Local _NanEnsureLocal(v8::Handle val) {
^
../node_modules/nan/nan.h:661:13: error: no member named 'smalloc' in namespace 'node'
, node::smalloc::FreeCallback callback
../node_modules/nan/nan.h:672:12: error: no matching function for call to 'New'
return node::Buffer::New(v8::Isolate::GetCurrent(), data, size);
^~~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:31:40: note: candidate function not viable: no known conversion from
'uint32_t' (aka 'unsigned int') to 'enum encoding' for 3rd argument
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
^
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:43:40: note: candidate function not viable: 2nd argument
('const char *') would lose const qualifier
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
^
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:28:40: note: candidate function not viable: requires 2 arguments, but 3
were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate, size_t length);
^
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:36:40: note: candidate function not viable: requires 5 arguments, but 3
were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
^
In file included from ../src/bufferutil.cc:16:
../node_modules/nan/nan.h:676:12: error: no viable conversion from 'v8::MaybeLocal<v8::Object>' to 'v8::Local<v8::Object>'
return node::Buffer::New(v8::Isolate::GetCurrent(), size);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:210:7: note: candidate constructor (the implicit copy constructor) not viable: no
known conversion from 'v8::MaybeLocal<v8::Object>' to 'const v8::Local<v8::Object> &' for 1st argument
class Local {
^
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:210:7: note: candidate constructor (the implicit move constructor) not viable: no
known conversion from 'v8::MaybeLocal<v8::Object>' to 'v8::Local<v8::Object> &&' for 1st argument
class Local {
^
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:214:13: note: candidate template ignored: could not match 'Local' against
'MaybeLocal'
V8_INLINE Local(Local<S> that)
^
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:326:13: note: candidate template ignored: could not match 'S *' against
'v8::MaybeLocal<v8::Object>'
V8_INLINE Local(S* that)
^
In file included from ../src/bufferutil.cc:16:
../node_modules/nan/nan.h:683:26: error: no member named 'Use' in namespace 'node::Buffer'
return node::Buffer::Use(v8::Isolate::GetCurrent(), data, size);
~~~~~~~~~~~~~~^
In file included from ../src/bufferutil.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Primitive *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:414:12: note: in instantiation of function template specialization
'v8::Local<v8::Primitive>::Local<v8::Value>' requested here
return NanEscapeScope(NanNew(v8::Undefined(v8::Isolate::GetCurrent())));
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
In file included from ../src/bufferutil.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Boolean *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:424:12: note: in instantiation of function template specialization 'v8::Local<v8::Boolean>::Local<v8::Value>'
requested here
return NanEscapeScope(NanNew(v8::True(v8::Isolate::GetCurrent())));
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
In file included from ../src/bufferutil.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Function *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1514:12: note: in instantiation of function template specialization
'v8::Local<v8::Function>::Local<v8::Value>' requested here
return NanEscapeScope(NanNew(handle)->Get(kCallbackIndex)
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
In file included from ../src/bufferutil.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Object *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1632:12: note: in instantiation of function template specialization 'v8::Local<v8::Object>::Local<v8::Value>'
requested here
return NanEscapeScope(handle->Get(NanNew(key)).As<v8::Object>());
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
9 errors generated.
make: *** [Release/obj.target/bufferutil/src/bufferutil.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/myname/.npm-packages/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "/usr/local/bin/node" "/Users/myname/.npm-packages/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/myname/code/sage/wp-content/themes/sage-nest/node_modules/bufferutil
gyp ERR! node -v v4.1.0
gyp ERR! node-gyp -v v3.0.1
gyp ERR! not ok
npm WARN install:bufferutil<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN install:bufferutil Exit status 1

><EMAIL_ADDRESS>install /Users/myname/code/sage/wp-content/themes/sage-nest/node_modules/fsevents
> node-gyp rebuild

SOLINK_MODULE(target) Release/.node
ld: library not found for -lgcc_s.10.5
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Release/.node] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/myname/.npm-packages/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "/usr/local/bin/node" "/Users/myname/.npm-packages/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/myname/code/sage/wp-content/themes/sage-nest/node_modules/fsevents
gyp ERR! node -v v4.1.0
gyp ERR! node-gyp -v v3.0.1
gyp ERR! not ok
npm WARN install:fsevents<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN install:fsevents Exit status 1

><EMAIL_ADDRESS>install /Users/myname/code/sage/wp-content/themes/sage-nest/node_modules/utf-8-validate
> node-gyp rebuild

CXX(target) Release/obj.target/validation/src/validation.o
In file included from ../src/validation.cc:15:
../node_modules/nan/nan.h:261:25: error: redefinition of '_NanEnsureLocal'
NAN_INLINE v8::Local<T> _NanEnsureLocal(v8::Local<T> val) {
^
../node_modules/nan/nan.h:256:25: note: previous definition is here
NAN_INLINE v8::Local<T> _NanEnsureLocal(v8::Handle<T> val) {
^
../node_modules/nan/nan.h:661:13: error: no member named 'smalloc' in namespace 'node'
, node::smalloc::FreeCallback callback
~~~~~~^
../node_modules/nan/nan.h:672:12: error: no matching function for call to 'New'
return node::Buffer::New(v8::Isolate::GetCurrent(), data, size);
^~~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:31:40: note: candidate function not viable: no known conversion from
'uint32_t' (aka 'unsigned int') to 'enum encoding' for 3rd argument
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
^
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:43:40: note: candidate function not viable: 2nd argument
('const char *') would lose const qualifier
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
^
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:28:40: note: candidate function not viable: requires 2 arguments, but 3
were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate, size_t length);
^
/Users/myname/.node-gyp/4.1.0/include/node/node_buffer.h:36:40: note: candidate function not viable: requires 5 arguments, but 3
were provided
NODE_EXTERN v8::MaybeLocal<v8::Object> New(v8::Isolate* isolate,
^
In file included from ../src/validation.cc:15:
../node_modules/nan/nan.h:676:12: error: no viable conversion from 'v8::MaybeLocal<v8::Object>' to 'v8::Local<v8::Object>'
return node::Buffer::New(v8::Isolate::GetCurrent(), size);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:210:7: note: candidate constructor (the implicit copy constructor) not viable: no
known conversion from 'v8::MaybeLocal<v8::Object>' to 'const v8::Local<v8::Object> &' for 1st argument
class Local {
^
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:210:7: note: candidate constructor (the implicit move constructor) not viable: no
known conversion from 'v8::MaybeLocal<v8::Object>' to 'v8::Local<v8::Object> &&' for 1st argument
class Local {
^
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:214:13: note: candidate template ignored: could not match 'Local' against
'MaybeLocal'
V8_INLINE Local(Local<S> that)
^
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:326:13: note: candidate template ignored: could not match 'S *' against
'v8::MaybeLocal<v8::Object>'
V8_INLINE Local(S* that)
^
In file included from ../src/validation.cc:15:
../node_modules/nan/nan.h:683:26: error: no member named 'Use' in namespace 'node::Buffer'
return node::Buffer::Use(v8::Isolate::GetCurrent(), data, size);
~~~~~~~~~~~~~~^
In file included from ../src/validation.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Primitive *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:414:12: note: in instantiation of function template specialization
'v8::Local<v8::Primitive>::Local<v8::Value>' requested here
return NanEscapeScope(NanNew(v8::Undefined(v8::Isolate::GetCurrent())));
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
In file included from ../src/validation.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Boolean *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:424:12: note: in instantiation of function template specialization 'v8::Local<v8::Boolean>::Local<v8::Value>'
requested here
return NanEscapeScope(NanNew(v8::True(v8::Isolate::GetCurrent())));
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
In file included from ../src/validation.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Function *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1514:12: note: in instantiation of function template specialization
'v8::Local<v8::Function>::Local<v8::Value>' requested here
return NanEscapeScope(NanNew(handle)->Get(kCallbackIndex)
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
In file included from ../src/validation.cc:7:
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:221:5: error: assigning to 'v8::Object *volatile' from incompatible type
'v8::Value *'
TYPE_CHECK(T, S);
^~~~~~~~~~~~~~~~
/Users/myname/.node-gyp/4.1.0/include/node/v8.h:180:37: note: expanded from macro 'TYPE_CHECK'
*(static_cast<T* volatile*>(0)) = static_cast<S*>(0);      \
^ ~~~~~~~~~~~~~~~~~~
../node_modules/nan/nan.h:1632:12: note: in instantiation of function template specialization 'v8::Local<v8::Object>::Local<v8::Value>'
requested here
return NanEscapeScope(handle->Get(NanNew(key)).As<v8::Object>());
^
../node_modules/nan/nan.h:398:30: note: expanded from macro 'NanEscapeScope'
# define NanEscapeScope(val) scope.Escape(_NanEnsureLocal(val))
^
9 errors generated.
make: *** [Release/obj.target/validation/src/validation.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/myname/.npm-packages/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR!

Node 4.1.0 is brand new. Maybe there's a compatibility issue with an npm package. Why you don't try with an older version first?

Ok.. I just install nvm change to 0.12 then redo it again, it works.
thanks
On Sat, Sep 19, 2015 at 10:52 AM, Scott Walkinshaw<EMAIL_ADDRESS>
wrote:

Node 4.1.0 is brand new. Maybe there's a compatibility issue with an npm
package. Why you don't try with an older version first?
—
Reply to this email directly or view it on GitHub
https://github.com/roots/sage/issues/1543#issuecomment-141604936.

--
Best Regards,
Ferry

I am also facing these type of issues. Re-Installation not working in my case. roots/sage is wonderful, but only when they work. Its forcing me to move from roots.
https://github.com/roots/sage/issues/1535
my issue is still pending, but here's the issue is closed.
:thumbsdown:

@webtuts4u the OP's issue looks like it might be related to a new version of node and was fixed when using the recommended version. i don't think you're running into the same thing.
i just googled the error message from your screenshot in #1535 and came across these:
https://github.com/BrowserSync/UI/issues/19
https://github.com/BrowserSync/browser-sync/issues/809

roots/sage is wonderful, but only when they work. Its forcing me to move from roots.

this should be changed to say "npm/node packages are wonderful, but only when they work on my environment". hopefully you can figure out a way to solve the issues you're running into, but this isn't really our problem.
waking up on a saturday morning to an email with your ":-1:" was kinda shitty. you know that this is an open source project that myself and others work on in our free time, and support for free... right?

Also, npm is throwing errors recently due to the new version of Xcode.  Make sure you open that up and accept the new license agreement, then try running things again.


echo off if "%PATH_BASE%" == "" set PATH_BASE=%PATH% set PATH=%CD%;%PATH_BASE%; chcp 65001 2>nul >nul java -jar -Duser.language=en -Dfile.encoding=UTF8 "%~dp0\apktool.jar" %*

Information

Apktool Version (apktool -version) -
Operating System (Mac, Linux, Windows) -
APK From? (Playstore, ROM, Other) -

Stacktrace/Logcat
Include stacktrace here

Steps to Reproduce

apktool

Frameworks
If this APK is from an OEM ROM (Samsung, HTC, LG). Please attach framework files
(.apks that live in /system/framework or /system/priv-app)
APK
If this APK can be freely shared, please upload/attach a link to it.
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything?
If you are trying to install a modified apk, did you resign it?
Are you using the latest apktool version?

I don't like leave a comment


Error with some png image

Hi I got error when I using some .png image when I want to convert to html (not all png image, but I don't know what different between the image that error and the image that not) . This the errors look like

php esc2html.php temp_print.bin > receipt-with-logo.html
PHP Fatal error:  Uncaught ImagickException: UnableToReadImageData `' @ error/pnm.c/ReadPNMImage/1319 in C:\xampp\htdocs\MyProject\public\escpos-tools-master\src\Parser\Command\StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php:81
Stack trace:
#0 C:\xampp\htdocs\MyProject\public\escpos-tools-master\src\Parser\Command\StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php(81): Imagick->readimageblob('P4\n764 830\n\x00\x00\x00\x00...', 'pbm')
#1 C:\xampp\htdocs\MyProject\public\escpos-tools-master\esc2html.php(59): ReceiptPrintHq\EscposTools\Parser\Command\StoreRasterFmtDataToPrintBufferGraphicsSubCmd->asPng()
#2 {main}
thrown in C:\xampp\htdocs\MyProject\public\escpos-tools-master\src\Parser\Command\StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php on line 81

Thank you!

Are you able to share the contents of temp_print.bin either on this issue tracker or via the email address on my profile?
I can't see a way to debug this without a sample file which shows the issue.

Hi Mike,
I already send to your email

Received.
Your file is malformed: You are using the GS ( L command, which only allows you to specify a data length up to 65kb of data. Your image is around 80kb, so you should be using GS 8 L, which allows for larger data sizes
The file contains 1d 28 4c 4a 37 (GS ( L data header for 14154 bytes of data), while the actual image command contains 764x830 dimensions. We can detect how many bytes of image that adds up to, and ignore the data header.
So with this small patch, your file will process:
diff --git a/src/Parser/Command/StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php b/src/Parser/Command/StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php
index dcf9e47..41d4ced 100644
--- a/src/Parser/Command/StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php
+++ b/src/Parser/Command/StoreRasterFmtDataToPrintBufferGraphicsSubCmd.php
@@ -51,6 +51,7 @@ class StoreRasterFmtDataToPrintBufferGraphicsSubCmd extends DataSubCmd implement
             return true;
         } else if ($this -> y2 === null) {
             $this -> y2 = ord($char);
+            $this -> dataSize = intdiv($this -> getWidth() + 7, 8) * $this -> getHeight();
             return true;
         } else if (strlen($this -> data) < $this -> dataSize) {
             $this -> data .= $char;

I have not tried to print your file, but if it works on an Epson printer, I would consider applying this patch in some form, otherwise we should return only part of the image.
In either case, we can detect the problem and log a warning when this situation arises.

Thanks! its working to process to html, I don't have Epson printer, but I think my client using an Epson printer, so if it's working I will tell you.
Thank you


Fix inverse logic of ValidAPIKey

This fix should be self-explanatory.

Self explanatory indeed. Thank you!

Thanks for the fix! Could you also please tag a new release? Would be awesome :).

All set! https://github.com/briandowns/openweathermap/releases/tag/0.12

Thanks! :+1:

Am 12.06.2018 um 23:43 schrieb Brian Downs<EMAIL_ADDRESS>All set! https://github.com/briandowns/openweathermap/releases/tag/0.12
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Number of acked messages does not correspond to metrics in Stackdriver

I have this trivial script that just consumes messages from PubSub
from google.cloud import pubsub

subscriber = pubsub.SubscriberClient()
subscription_path = 'projects/myproject/subscriptions/mysub'

def callback(msg):
    msg.ack()

subscriber.subscribe(subscription_path, callback=callback)

I'd expect that all messages would be acked without any problem, but when I look at Stackdriver, I see a huge number of Deadline Operations and very few StreamingPull Acknowledge Operations. See screenshot of Stackdriver https://imgur.com/Z4ZK2l1 (couldn't upload it here for unknown reasons).
Is this expected? Is it fault of Stackdriver or can there be a problem with acking?
Tested on Google Dataproc with google-cloud-pubsub-0.28.4.
Thanks!

Forgot to mention that this happens for topic that has 7 days old messages, could that be an issue? I tried it on queue that is not full and the difference between acked and pulled operations were about 10% (way lower than in previous case, but still unexpected).

This looks like it might be related to this question on Stack Overflow.
I will look into this further.

Thanks. We have a similar use case to batching https://github.com/GoogleCloudPlatform/google-cloud-python/issues/4141 and weren't able to get even your sample code working (ratio of acked / pulled messages was extremely low). Had to switch to 0.27.0 with sync pulls which work just fine.

I am going to close this, since it seems very similar to other reported issue and the Consumer.pause() and Consumer.resume(), which was resolved by #4558.
@Marigold If I'm closing this in error, I'm happy to work with you to get a case we can reproduce. (Having to wait until messages are 7 days queued will obviously take longer to reproduce :grinning:.)


VSCode runs slow when connected to personal hotspot

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.58.2
OS Version: 11.5.1 (20G80)

Steps to Reproduce:

Connect VSCode to a 'slower' (I noticed that this issue occurred when I was connected to 3G instead of 5G). When attempting to debug the debugger was awfully slow to start.
Disconnect the Mac from network and the debugger starts normally.

I think its important to note that this issue occurred on an Apple Silicon Mac which usually starts the debugger fast.

what debugger are you using?

Hey Andre,
Sorry for the very slow reply. I am using the ordinary VSCode Python debugger.

On 29 Jul 2021, at 23:57, Andre Weinand @.***> wrote:
what debugger are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/129799#issuecomment-889512386, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHPCJQ7HVKCQF5IH76WCZ4TT2HMEFANCNFSM5BHEPHJA.

/extPython


DUPLO-13077 fixes broken elastic search creation with no dedicated master type and fixes elasticsearch_version mapping

Overview
Creation of Elastic search cluster was crashing when dedicated_master_type field of cluster config was not defined because it was implicitly being assigned { Value: null } by Go. The unexpected structure made AWS throw a non descriptive error message Value cannot be null, parameter : key. This PR makes this field into a pointer so we may properly assign it null when it is not defined. AWS lets us create it properly with no errors if dedicated_master_type is null.
This PR also fixes attribute mapping between API response and TF state. The correct version field is called EngineVerion instead of ElasticSearchVersion.
Summary of changes
This PR does the following:

Makes dedicated_master_type into a pointer so it is null if absent from parameters when creating DuploElasticSearchDomainRequest
fixes mapping between elsaticsearch_version and the BE's EngineVersion fields

Testing performed

[ ] Using unit tests
[x] Manually, on my local system
[ ] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-13077 Terraform Provider - Elasticsearch/Opensearch cluster continuously flagged for destroy/recreate

I guess @Duplo-nikv fixed the mapping before me here. Good thing I uncovered this dedicated master bug so this work wont have been for nothing :)

@komalkadam-dc You did something very similar to this in this PR. would you review this too?


WHERE IS DNS SETTINGS ?????????????? WHERE ????????

On the Domains page, in the list of domains, select the domain you're using for your website, and then select DNS settings in the management panel
WHERE IS DNS SETTINGS ?????????????? WHERE ????????

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

ID: f087dd43-183d-08bb-37d7-d893bede6efb
Version Independent ID: be429837-0300-3de8-6c7d-2da2fabc2c17
Content: Update DNS records to keep your website with your current hosting provider - Microsoft 365 admin
Content Source: microsoft-365/admin/dns/update-dns-records-to-retain-current-hosting-provider.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

Looks like they changed it from DNS Settings to DNS records, perhaps? When I go there, the rest of the instructions seem fine.
The only other thing I can think of for your problem is you clicked the tick mark next to the domain rather than actually clicking the domain to get into the domain's DNS settings?

Step 2 in the link is wrong. You need to click on the domain name (not just select), then click on DNS Records tab. The next step 3 is correct.
additionally this page is hard to find. it should be linked to the page where you change your DNS Name server (https://docs.microsoft.com/en-us/microsoft-365/admin/get-help-with-domains/change-nameservers-at-any-domain-registrar?view=o365-worldwide) as the next logical step.

@ragnarokas Thank you for your feedback. We have modified the document with the correct settings and the same can be tracked via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/2573
Thanks
Sri

@msbemba  Thank you very much for the contribution and updating the documentation with PR. @ragnarokas Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Release 0.2.0rc6

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@j-wags feel free to merge once tests pass if I'm not back at my computer soon - I don't think deps changed significantly


Add a custom hook before checker run

Feature motivation

Sometime we want to delay type checking until some tasks done. In my case, it should wait for CSS module type definition files generated. It can be done with serviceBeforeStart hook in production build, but not for watch mode because this plugin use single checker service in watch mode and serviceBeforeStart hook triggered only in initial build.
Feature description

Call a custom hook before type checker run
Feature implementation

Just like serviceBeforeStart hook but called when type checker is about to be run

Would you like to try something out and raise a prospective PR?

Sorry about my misunderstanding, the serviceBeforeStart hook should work as expected.


k8 liveness endpoint

Hello folks,
first of all, congrats for this amazing project, it's really works for us sending transactions and marketing messages thought n8n workflows.
The last month we're deployed codechat in kubernetes infra, and it's working very well. But, for working 100% we need to define the liveness endpoint for k8.
Question: have any global endpoint to returns if all instances are connected and working well?
Thanks!
Refs: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/#define-a-liveness-http-request

Good morning!
Exit the route fetchInstances. In it, you pass your globalApiKey to retrieve all instances.


Set up project with webpack and jest

Set up the project with webpack, jest, babel, and linters.

STATUS: APPROVED!!!  🎉 🎉 🎉 :champagne: :champagne: :champagne:
Hi Team,
Good job !!!
To Highlight! :clap:  :green_circle:
:heavy_check_mark:  Linter is working right
:heavy_check_mark:  Project web pack and jest setup is fine
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉 🎉 🎉 :champagne: :champagne: :champagne:

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

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


Any way to hide a disabled scrollbar?

Works great except the not needed scrollbar? Any way to hide this?
The options "scrollBar" and "scrollable" are set to false by default. What am I missing?

Hi @gsirin,
Can you provide the sample code such as JSFiddle?

Hi Nick,
was able to fix my issue by simply setting"freezeColumn" to false.


tab key not working

Issue Type: Bug
tab key not giving tab spaces in vscode
VS Code version: Code 1.33.1 (51b0b28134d51361cf996d2f0a1c698247aeabd8, 2019-04-11T08:27:14.102Z)
OS version: Windows_NT x64 10.0.17134

Are you familiar with the designed behaviour of VSCode in relation to tabs and spaces? This Stack Overflow article may help. And SO is a good place to ask questions.

@srinivasan-groupten You can control this behavior with these settings, as stated in the Stack Overflow question @gjsjohnmurray linked to:
// Insert spaces when pressing Tab. This setting is overriden
// based on the file contents when `editor.detectIndentation` is true.
"editor.insertSpaces": true,

// When opening a file, `editor.tabSize` and `editor.insertSpaces`
// will be detected based on the file contents. Set to false to keep
// the values you've explicitly set, above.
"editor.detectIndentation": false

Please let us know if you see unexpected behavior after updating those settings. Thanks!


pls help, make video

make a video on YT with detailed explanation

I'm sorry we're not fully supporting people who doesn't know at all about it.
If you don't understand how by reading readme, meaning you're not qualified.


Cannot find module 'lodash/defaultsDeep' error in latest check in ('98f5c0e')

Hello,
I've simply cloned react-boilerplate repo and ran npm run setup to see an error that lodash/defaultsDeep module could not be found. How do I work around this problem? Shouldn't the installation take care of any missing modules?
I'm using node v8.4.0, npm v6.1.0, and yarn 1.3.2.
Please see the full commands and output below. Please tell me if you need anything else:
mxsdev2 ➜  git clone --depth=1 https://github.com/react-boilerplate/react-boilerplate.git
Cloning into 'react-boilerplate'...
remote: Counting objects: 389, done.
remote: Compressing objects: 100% (346/346), done.
remote: Total 389 (delta 69), reused 176 (delta 21), pack-reused 0
Receiving objects: 100% (389/389), 1.04 MiB | 335.00 KiB/s, done.
Resolving deltas: 100% (69/69), done.
mxsdev2 ➜  cd react-boilerplate
mxsdev2 ➜  react-boilerplate git:(master) npm run setup

<EMAIL_ADDRESS>presetup /home/jchukwum/temp_test/react-boilerplate
npm i chalk shelljs

npm notice created a lockfile as package-lock.json. You should commit this file.

<EMAIL_ADDRESS><EMAIL_ADDRESS>added 23 packages from 10 contributors and audited 26 packages in 1.996s
found 0 vulnerabilities

<EMAIL_ADDRESS>setup /home/jchukwum/temp_test/react-boilerplate
node ./internals/scripts/setup.js

Do you want to clear old repository? [Y/n]
Removing old repository ✓
Installing dependencies... ✓This might take a while)
Initialising new repository
Done!

<EMAIL_ADDRESS>postsetup /home/jchukwum/temp_test/react-boilerplate
npm run build:dll

<EMAIL_ADDRESS>build:dll /home/jchukwum/temp_test/react-boilerplate
node ./internals/scripts/dependencies.js

module.js:491
throw err;
^
Error: Cannot find module 'lodash/defaultsDeep'
at Function.Module._resolveFilename (module.js:489:15)
at Function.Module._load (module.js:439:25)
at Module.require (module.js:517:17)
at require (internal/module.js:11:18)
at Object. (/home/jchukwum/temp_test/react-boilerplate/internals/scripts/dependencies.js:13:18)
at Module._compile (module.js:573:30)
at Object.Module._extensions..js (module.js:584:10)
at Module.load (module.js:507:32)
at tryModuleLoad (module.js:470:12)
at Function.Module._load (module.js:462:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build:dll: node ./internals/scripts/dependencies.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:dll script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postsetup script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!  However, the problematic require (/internals/scripts/dependencies.js:13) has been in the code for a very long time and if it were the problem, it would have been reported either by someone else in the past or at least by the CI tools. Most likely, your dependencies weren't installed correctly.
Something slightly strange in your report: You list Yarn 1.3.2 but the script is using npm instead of Yarn. (You can tell because a package-lock.json file was created.) So there could be something not totally correct in your computer setup. You might want to look into that. Maybe make sure you update Yarn to 1.7.0 and check that it's available in your PATH.
Another thing to try would be to run yarn install (or rerun npm install if you prefer). It should also take care of running npm run build:dll (because of the postinstall script in package.json) but make sure you do it manually if you don't see it happen. Does this fix your issue?
Good luck!
Julien

Hello Julien,
Thanks for a quick response.
Yes, it is odd that this is happening especially if it's old code that hasn't been touched. Something to note is that when I pull down the older checkin by running 'git checkout 904c8a878279e3eedcbab59ed6934ac08c9eb610' and then 'npm run setup', it works. So it looks like something on the latest checkin doesn't work well on my linux machine.
I'll update my yarn and also try to run yarn install. If I have any success I'll post it.
Thanks

I've finally resolved the issue. The solution is:

Delete the whole directory: rm -rf react-boilerplate
Clone the repo again: git clone --depth=1 https://github.com/react-boilerplate/react-boilerplate.git
Go into the directory and call npm install: cd react-boilerplate && npm install
Now run the setup: npm run setup

This time you don't see the errors.
Thanks for assisting in initial investigation.

Good to know you fixed it. Still strange that you got this issue in the
first place because the setup script is supposed to install dependencies
for you. Will refer back to this if someone else reports the same issue.
Closing in the meantime.
On Sun, Jul 1, 2018, 9:50 PM John Chukwuma<EMAIL_ADDRESS>wrote:

I've finally resolved the issue. The solution is:

Delete the whole directory: rm -rf react-boilerplate
Clone the repo again: git clone --depth=1
https://github.com/react-boilerplate/react-boilerplate.git
https://github.com/react-boilerplate/react-boilerplate.git
Go into the directory and call npm install: cd react-boilerplate
&& npm install
Now run the setup: npm run setup

This time you don't see the errors.
Thanks for assisting in initial investigation.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/react-boilerplate/react-boilerplate/issues/2263#issuecomment-401625429,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIiJn2v8kdRx9DGRxcZ67YkajdFoL_Fdks5uCRoLgaJpZM4U-Idc
.

I ran in to this same issue today. After upgrading NPM, Yarn and Node, it looks my Node version was the issue. Upgrading from Node v8.9.0 to v8.11.3 resolved the error.
I was able to identify the need for an upgrade by running yarn install, which reported this error:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=8.9.4".

Hopefully that helps someone.

Thanks, Indeed it's also expected by the new version of eslint. Will plan to update the engines section in package.json.


Missing: --secrets-file FILENAME

I'm using version 0.6.0
and I'm getting the message Missing: --secrets-file FILENAME when the command is executed as follows:
vaultenv --host $VAULT_ADDR --token $TOKEN --secrets-file /etc/secrets.d/app.secrets /usr/bin/env

I've confirmed the file /etc/secrets.d/app.secrets exists and has the following content
MY_USERNAME=app/production#username
MY_PASSWORD=app/production#password

Any idea why this is happening, is it a bug?

Thanks, I populated the TOKEN environment variable manually to test and I got a much better response.

A follow question, can I do a TLS_SKIP_VERIFY with vaultenv?


[new feature] any chances to get all outputs inherited from modules directly?

when deploy via terragrunt with many modules, I need write seperate module to handle all outputs. that's not convenience.
Will it be good feature to export the outputs directly from modules?
For example, in a module, there is output from that module
module.ecs.this_name

So currently I have to add below codes to get that output as module.ecs.this_name
output "module.<module_name>.<same_output_name>" {
  value = module.<module_name>.<same_output_name>
}

Are there any easy ways to export all outputs from reference modules?
different modules have multiple outputs, how to easily export all of them after terragrunt deployment?

I think this is by design in Terraform: they want you to be explicit with all inputs and outputs. That does mean it's more verbose, but less "magic" makes the behavior more predictable.

I am fine with this feature now. I will work it with dependency. But I found a bug in it.
https://github.com/gruntwork-io/terragrunt/issues/955


Enable Sourcegraph

I want to use Sourcegraph for redux code search, browsing, and usage examples. Can an admin enable Sourcegraph for this repository? Just go to https://sourcegraph.com/github.com/gaearon/redux. (It should only take 30 seconds.)
Thank you!

suggest install sourcegrapph tool in local to search / browsing the redux code.
:)

Duplicate of #108, btw.
I'm happy to do this from another account later.

Seems like it's somehow already there.
Whatevs!
Closing.


Add linux course in hindi

What does this PR do?
Add course
For resources
Description
This is a linux youtube tutorial.
Why is this valuable (or not)?
This covers all important concepts about linux.
How do we know it's really free?
Its available on youtube.
For book lists, is it a book? For course lists, is it a course? etc.
It is a youtube tutorial.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Make work with XCode 9 beta3+

Renamed all instances of  `Type` to ProtoType, which fixes issue #213

Need to change the tests


Add functionality to allow dependency injection via ServerCallContext

What version of gRPC and what language are you using?
1.12.0 csharp
What operating system (Linux, Windows, …) and version?
Linux, Windows
What runtime / compiler are you using (e.g. python version or version of gcc)
msbuild
What did you do?
We wanted to implement a server application that makes some nifty use of dependency injection. To do so we make use of the Inceptor api that was added with 1.10.0. Currently there's however only a hacky way to add context aware objects to the ServerCallContext instance as outlined below:
    public interface IServerCallContextAccessor
    {
        ServerCallContext ServerCallContext { get; }
    }

    public interface IServerCallContextProvider
    {
        ServerCallContext ServerCallContext { get; set; }
    }

    public class ServerRequestScope : Interceptor
    {
        public IServiceProvider ServiceProvider { get; }

        public ServerRequestScope(IServiceProvider serviceProvider)
        {
            ServiceProvider = serviceProvider ?? throw new ArgumentNullException(nameof(serviceProvider));
        }

        public override async Task<TResponse> UnaryServerHandler<TRequest, TResponse>(TRequest request, ServerCallContext context, UnaryServerMethod<TRequest, TResponse> continuation)
        {
            using (var serviceScope = ServiceProvider.CreateScope())
            {
                // initialize the server call context to allow future invocations to GetService<IServerCallContextAccessor>() to return the current server call context
                var serviceProvider = serviceScope.ServiceProvider;
                serviceProvider.GetRequiredService<IServerCallContextProvider>().ServerCallContext = context;
                context.RequestHeaders.Add(new ServiceProviderMetadataEntry(serviceProvider));

                return await base.UnaryServerHandler(request, context, continuation);
            }
        }
    }

    public class ServiceProviderMetadataEntry : Metadata.Entry
    {
        public const string MetadataKey = "X-ServiceProvider";

        public IServiceProvider ServiceProvider { get; }

        public ServiceProviderMetadataEntry(IServiceProvider serviceProvider)
            : base(MetadataKey, string.Empty)
        {
            ServiceProvider = serviceProvider ?? throw new ArgumentNullException(nameof(serviceProvider));
        }
    }

        public static IServiceProvider GetServiceProvider(this ServerCallContext context)
        {
            if (context == null)
            {
                throw new ArgumentNullException(nameof(context));
            }

            foreach (var requestHeader in context.RequestHeaders)
            {
                if (ServiceProviderMetadataEntry.MetadataKey.Equals(requestHeader.Key, StringComparison.InvariantCultureIgnoreCase))
                {
                    if (requestHeader is ServiceProviderMetadataEntry serviceProviderMetadataEntry)
                    {
                        return serviceProviderMetadataEntry.ServiceProvider;
                    }
                }
            }

            throw new InvalidOperationException($"{ServiceProviderMetadataEntry.MetadataKey} request header metadata not found.");
        }

    class Program
    {
        static void Main(string[] args)
        {
            // configure service collection
            var serviceCollection = new ServiceCollection()
                .AddScoped<IServerCallContextProvider, ServerCallContextProvider>()
                .AddScoped<IServerCallContextAccessor>(t => new ServerCallContextAccessor(t.GetRequiredService<IServerCallContextProvider>().ServerCallContext));

            // build service provider
            var serviceProvider = serviceCollection.BuildServiceProvider();

            // initialize the server along with service controllers
            int port = 50051;
            Server server = new Server
            {
                Services =
                {
                    SecurityService.BindService(new SecurityService()) // inner-most service
                        .Intercept(new GenericExceptionHandler()) // a middlware that gracefully swallows all exceptions that could possibly occur
                        .Intercept(new ServerRequestScope(serviceProvider)) // inject the request scope, providing the service provider request scope from here on
                },
                Ports = { new ServerPort("<IP_ADDRESS>", port, ServerCredentials.Insecure), },
            };
            server.Start();

            foreach (var serverPort in server.Ports)
            {
                Console.WriteLine($"Server listening on {serverPort.Host}:{serverPort.BoundPort}");
            }

            Console.WriteLine("Press any key to stop the server...");
            Console.ReadKey();

            server.ShutdownAsync().Wait();
        }
    }

Yes, we indeed hack the dependency injection mechanism into the current implementation of the ServerCallContext by means of a custom request header metadata entry.
What did you expect to see?
We expected to have a way to inject a scoped System.IServiceProvider into ServerCallContext instances. Having a possibility to deep clone an existing ServerCallContext instance we could pass it on to services via the Interceptor api.
Adding a property with type System.IServiceProvider to ServerCallContext and making the constructor of ServerCallContext public would allow effective dependency injection mechanisms to be implemented. The following is an example of how
    public class ServerRequestScope : Interceptor
    {
        public IServiceProvider ServiceProvider { get; }

        public ServerRequestScope(IServiceProvider serviceProvider)
        {
            ServiceProvider = serviceProvider ?? throw new ArgumentNullException(nameof(serviceProvider));
        }

        public override async Task<TResponse> UnaryServerHandler<TRequest, TResponse>(TRequest request, ServerCallContext context, UnaryServerMethod<TRequest, TResponse> continuation)
        {
            using (var serviceScope = ServiceProvider.CreateScope())
            {
                // initialize the server call context to allow future invocations to GetService<IServerCallContextAccessor>() to return the current server call context
                var serviceProvider = serviceScope.ServiceProvider;
                serviceProvider.GetRequiredService<IServerCallContextProvider>().ServerCallContext = context;

                // clone copy server call context with a customized service provider
                var scopedServerCallContext = new ServerCallContext(context, serviceProvider);
                return await base.UnaryServerHandler(request, scopedServerCallContext, continuation);
            }
        }
    }

+1

Yeah, either a copy constructor on the ServerCallContext or some sort of "view bag"-like functionality on it would help a lot not just with schlepping the service provider around, but possibly other things like user identity, etc.
Would you accept a PR for this (and by this I'm referring to the copy constructor as it's super-trivial and the least intrusive of the two options, plus it's not going to change the public API)?
By the way, @dpsenner I think it's better to use a trailer + lowercase metadata key (gRPC lowercases everything anyway) as request headers are not guaranteed to be there. You do have to clean the trailers up afterwards, though, so that's not so nice.

We chose to inject the service provider in the metadata headers because:

we prefix the service provider header key with a prefix that should never fly in from a request
we would have to clean up the trailers if we did inject the service provider there, creating yet another point of failure
if request headers are not present, we create them on the fly

This work around gets the job done for our functional requirements. Despite that, it is a hack that only exists because it works and is the only solution we could come up with.
@cearny I do not grasp how a copy constructor would help if a middleware creates the service provider by calling ServiceProvider.CreateScope(). There's no way to override the ServerCallContext instance of a request, is there? Would you please explain your thoughts?

@dpsenner as I see it, looking at the ServerCallContext sources, there is no way to be sure that RequestHeaders is null or not -- and I see no way of setting it if it is, as it's getter-only. Am I missing something?
The "trailers" thing is precisely the same hack that works that you made, only since the trailers are instantiated by default, they're never null and what we're using is something like:
try
{
    return await base.UnaryServerHandler(request, context, continuation);
}
finally
{
    // Clean up after ourselves, we don't want to send that (empty) trailer to the client
    context.ResponseTrailers.Remove(metadata);
}

so they should get cleaned up properly. But, again as you've mentioned, this is just a working hack.
As for the benefits of having a copy constructor on ServiceCallContext, it would allow us to write something like:
public class ScopeAwareServiceCallContext {
    // ...

    public ScopeAwareServiceCallContext(IServiceScope serviceScope, ServiceCallContext originalContext) : base(originalContext) {
        serviceScope = _serviceScope;
    }
}

// and in the interceptor write something like:

var scopeAwareContext = new ScopeAwareServiceCallContext(serviceScope, context);
return await base.UnaryServerHandler(request, scopeAwareContext, continuation);

So, a riff on the decorator pattern, really.
Add to this an extension method on ServiceCallContext to try and extract the service scope, similarly to what you've written but in this case just trying to cast to ScopeAwareServiceCallContext instead of iterating on the headers, and you could have most of what you need.
Ideally I'd like to have some sort of "context metadata" that one could add whatever one needs to (sort of like a view bag, as I've mentioned), but -- having a copy constructor on the (non-sealed!) ServerCallContext class -- all this could be solved just with extension methods. No new fields on the original ServerCallContext would need to be introduced. Neat & tidy -- I think.

Now I get your idea of the copy ctor. That could work. I rather thought of replacing ServerCallContext with IServiceProvider and let the grpc internals look up the original ServerCallContext by invocation of serviceProvider.GetRequiredService<ServerCallContext>() . A developer could wrap the service provider without even having to know about the ServerCallContext existence.
A wrapping interceptor that does nothing would then look like:
public override async Task<TResponse> UnaryServerHandler<TRequest, TResponse>(TRequest request, IServiceProvider serviceProvider, UnaryServerMethod<TRequest, TResponse> continuation)
{
    return await base.UnaryServerHandler(request, serviceProvider, continuation);
}

And another one that adds a scope could look like:
public override async Task<TResponse> UnaryServerHandler<TRequest, TResponse>(TRequest request, IServiceProvider serviceProvider, UnaryServerMethod<TRequest, TResponse> continuation)
{
    using (var serviceScope = ServiceProvider.CreateScope())
    {
        return await base.UnaryServerHandler(request, serviceProvider, continuation);
    }
}

This means that dependency injection becomes a integrated component of the grpc csharp implementation.

That would work neatly, yes.
Though I've had a discussion on this topic with a colleague and the end result of that was that we both agreed this would tie "out-of-the-box" gRPC to Microsoft's own DI container way too much, making it an intrusive change. The point being that some people use Autofac, Ninject, etc.
Until we'll have something like per-scope metadata separated from headers and trailers, which could enable most any sort of customization, I can live with the most minimal change to the gRPC codebase, which would be a copy constructor.

It's great news that you are considering this addition as a team!
If portability and interopability is a concern, then grpc could come with it's own dependency
injection interface. All that interface has to define is a method signature object GetService(Type t). It is straight forward to implement adapters from [microsoft DI, ninject, autofac] to grpc di. I however have to insist that it is a tedious task to reinvent an existing wheel. That's just yet another api to the same requirements.
This link points the documentation of the IServiceProvider interface, note the "Applies to" section.
Further this link points to a property how the asp net core devs implemented a way to access a request-scoped serviceprovider. Adding this (or something like this) to the ServerCallContext could be an option that's backwards compatible.

Hm, the thing is that I don't really want to see gRPC as "just a sort of ASP.NET Core", convenient as that might be. I feel like it should focus on what it does specifically and provide extension endpoints where more is required of it. This may be because I'm coming to it from an Express.js mindset. Just to put my thoughts into context.
By the way, I've taken a look at maybe adding a service provider to the ServerCallContext class and it would be quite awkward to do it directly right now, as it is only created via HandlerUtils.NewContext, which is called directly in the HandleCalls of ClientStreamingServerCallHandler, DuplexStreamingServerCallHandler, ServerStreamingServerCallHandler and UnaryServerCallHandler. Shipping a service provider to those would mean either having some sort of static master-class (like the Logger, something that I don't really like already) or quite a lot of rework. Not sure I, at least would be able to tackle that task.
I still feel like just being able to swap the server call context with a sub-class that is able to provide a service provider using an interceptor (sort of like the interceptor already does with InterceptingCallInvoker on the client side) would be the path of least resistance. Also, maybe adding a "view-bag" of sorts on the server call context :-)

I like the idea that grpc allows injecting arbitrary data into the methods that process requests and allowing the interceptors to manipulate that arbitrary data. After all that is what dependency injection is about.
This solution is similar to what we hacked with the custom meta data header. It is just bad to abuse request headers for details that matter only to the server side implementation and, as pointed out earlier, there's no functional requirement to make use of any existing dependency injection framework. From this point of view it is better that grpc is standalone, not depending on any other framework but netstandard.

@dpsenner I am currently tunneling the IServiceProvider into the call handler with a similar interceptor. Instead of using a header, I use an AsyncLocal<IServiceProvider>:
public static class ServerCallContextExtensions
{
    internal static AsyncLocal<IServiceProvider> ServiceProviderInstance { get; set; } = new AsyncLocal<IServiceProvider>();

    public static IServiceProvider ServiceProvider(this ServerCallContext _)
    {
        return ServiceProviderInstance.Value;
    }
}

// In the same assembly as ServerCallContextExtensions so it can access the internal field
public StartScopeInterceptor : Interceptor
{
    private readonly IServiceProvider _services;

    public StartScopeInterceptor(IServiceProvider services)
    {
        _services = services ?? throw ArgumentNullException(nameof(services));
    }

    public override async Task<TResponse> ClientStreamingServerHandler<TRequest, TResponse>(
        IAsyncStreamReader<TRequest> requestStream,
        ServerCallContext context,
        ClientStreamingServerMethod<TRequest, TResponse> continuation)
    {
        using (var scope = _services.CreateScope())
        {
            ServerCallContextExtensions.ServiceProviderInstance.Value = scope.ServiceProvider;
            try
            {
                return await continuation(requestStream, context);
            }
            finally
            {
                ServerCallContextExtensions.ServiceProviderInstance.Value = null;
            }
        }
    }

    // ...
}

public ServiceImpl : TheService.TheServiceBase
{
    public override async Task<TheResponse> TheMethod(TheRequest request, ServerCallContext context)
    {
        IServiceProvider sp = context.ServiceProvider();
        // ...
    }
}

@chwarr yeah, AsyncLocal and extension methods look like the least-intrusive and "nicest" option ATM and AsyncLocal is used for this in a few other places -- the zipkin4net library uses a similar concept, for example, and that's how we added instrumentation for StackExchange.Redis.
Interesting info regarding the team possibly having considered a Dictionary at one point, by the way 👍 I guess a ConcurrentDictionary could be used instead in case synchronization issues were a thing. But if an object is provided, then any library consumer could decide what they actually need.
I've actually had this lying around for a while now but never got around to actually running the test suite locally (to formally check that it doesn't break anything -- it really shouldn't though): https://github.com/cearny/grpc/commit/91bc5d6272e408372c5465552e452e4316af2cb9

Looks like the use case has been sufficiently covered by https://github.com/grpc/grpc/pull/18086?

That woks handsomely for me!

Great news that this issue was fixed, thanks!


Remove stars from RuneWatch plugin

WDR doesn't support, and it's confusing to players.

fixed in v2.0.2


SyncTeX works tex->pdf, but not reverse

After running atom-latex:preview to preview the file, atom-latex:synctex takes me from a .tex file to the right place in the pdf, but if I click on text in the pdf pane and invoke atom-latex:synctex (via Ctrl-L Ctrl-S), it brings up a Save Dialog.
I'm wondering if this is user error, but I've run out of ideas.

Have you tried SyncTeX from here right click menu? That should determine if it' a key binding issue, or an actual syncTeX issue.
You can always change the keybinding if you find the defaults not suitable for you :-)
PS: I like to think of it more as ctrl - L - S i.e holding down ctrl and hitting l s ;-)

Ok, I missed that Cmd-click is the way to go. I overlooked that in the documentation because I jumped straight to the keybinding table in my eagerness to edit LaTeX in Atom. I actually tried Cmd-click on my machine at work before opening this issue (because that's how it is invoked in TeXShop), but it did nothing; works fine on my laptop at home right now, however.
So, it's user error of a sort :) Had I read the documentation thoroughly, I would have persevered with Cmd-click on my machine at work. Perhaps a restart of Atom would have fixed it.
So, Ctrl-L Ctrl-S is a chord?! Having grown up on Emacs, I can't help but see it as two keystrokes ;-)
Thanks very much for your help, I'm fairly new to Atom and this atom-latex package looks really nice.

If there are no .snynctex files, then the ctrl - click will fail, but it should also pop up an error message with why it failed. :-)


Fix compatibility with diffusers < 0.25.0

Following https://github.com/huggingface/optimum-intel/pull/952

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Chapter 8 Part: Creating a topic model using NMF for documents

When I try to run the following line of code:
W_text_matrix = nmf_text_model.fit_transform(tfidf_text_vectors)
I get the following error: ValueError: array must not contain infs or NaNs
As far as I can see there aren't any.

Dear Christian,
got the example running in the end, but then on running the code again, got the nans & infs error again. This morning I re-installed Anaconda and got the example running repeatedly. So, nothing to do with the data I guess, but with my previous Anaconda setup.


Endpoint for a users previous purchases.

Description of changes
Resembles the endpoint used to get order statistics displayed in OW4, only difference is that you pass a user id and get that users history (compared to the other endpoint where an authenticated user can get his/her data). Can (should) only be used by nibble, requires nibbles oath token with the "shop.readwrite" scope. To be used in Nibble4
Code Checklist

[x] I have added tests
[x] I have provided documentation

@kharann LGTM? MERGE?

sorry had syrup in caps lock

javel

jævla codeql tar jo 5 år

Improvements for public repository forksKan dette være en forklaring? Se særlig på Improvements for public repository forks


Hint when noSuchMethod only invokes a noSuchMethod inherited from somewhere other than Object

Given code like:
class B extends A {
  noSuchMethod(x) => super.noSuchMethod(x);
}

Produce a hint that the method can be removed as long as the noSuchMethod on the right-hand side is not referring to the one inherited from Object.

https://codereview.chromium.org/1506903005

https://github.com/dart-lang/sdk/commit/1b0a5c313f94620289db6a1b3328d3b3399fc57d


Terminal command success decorations still render after running clear

Testing #143433

cd ..
clear
:bug: success terminal command decorations are still rendered

only reproes sometimes - regression related to the eraseInDisplay change that was reverted in xterm

https://github.com/xtermjs/xterm.js/pull/3649/commits/a4fee6d4b5c8dd4f740cdf0b5e221377020c1f34

I bet I caused this in https://github.com/xtermjs/xterm.js/pull/3654, need i-- after a splice...

Upstream fix merged, updating now...

I'm having the same error on macOS.

Still seeing this:

Version: 1.65.0-insider (user setup)
Commit: 8440b8f47231147823ba9696913d642e5f2d9d58
Date: 2022-02-25T05:17:33.544Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.22000

@joyceerhl if you run Terminal Clear, you should see them all gone from the terminal. running clear in the terminal only clears the viewport.

are you scrolling? a gif might be helpful because I cannot reproduce

You might also be seeing https://github.com/microsoft/vscode/issues/75141

Oh--I thought running clear in the terminal was supposed to clear the decorations too. Seems a little weird that the command decorations would stick around--I would disable the decorations if that is the behavior moving forward since the dots become quite distracting:

Terminal: Clear does clear out the decorations, btw.

I don't think we can do anything about clear not working as expected in pwsh/windows. @Tyriar can we?

@meganrogge conpty sends \x1b[3J, should we be clearing markers whenever erase in display clears any line?

I think it would be easy to handle by checking if the command entered was clear or similar (could even do this per shell type since the command name may differ). More complicated and potentially destructive to clear the markers whenever erase in display is called imo. We can test it though.

Reopening to cover cases like this specially


Fitbit oauth2

I would like to ask you if you have the intention to support oauth2 authentication for fitbit.
Best regards,
David

Fitbit will be retiring OAuth1.0a soon (as yet undefined date though) and OAuth2 is required for heartrate & location access.

Fitbit OAuth 1 retirement is now set for Mar/Apr 2016:
https://dev.fitbit.com/docs/oauth2/
Is there any plan to support oauth2?


Fix for new MELA

Coverage remained the same at 84.053% when pulling 9b15d8953c82e995a74c99cace6e64321e3ea95a on usarica:master into b36b313d0caf261e4c22e34f5fd52e71c7773084 on aminnj:master.


fix: improve default exports for CommonJS and ESM

Fixes https://github.com/isaced/appstore-connect-sdk/issues/28

See https://blog.andrewbran.ch/default-exports-in-commonjs-libraries/#the-simplest-solution
And https://github.com/microsoft/TypeScript/issues/2719#issuecomment-310969161

@isaced Amazing! I will try it out!


xterm: 320 -> 325

Motivation for this change
Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Updates xterm to 325, and also enable sixel support for drawing images on the screen.

Does this need #16136 ? Also no need to add a dependency on libsixel?

@groxxda The sixel support in xterm is independent of the libsixel. The libsixel provides an encoder/decoder for images from/to sixel format. The xterm support is for graphics display.

From the travis logs: make: *** No rule to make target 'install'.  Stop.

Just ran nox-review locally and it was all fine

@vrthra ./sixel-256.support.patch is no longer needed.
https://github.com/saitoha/libsixel/issues/47
It also can be set with the following xterm-specific escape sequence by applications.
printf '\033[?1;3;256S'

@saitoha thanks, I think 256 colors is a reasonable default in this case.


VersionDotNetCoreAssemblies "Add a Version field if missing" parameter cannot be enabled via the GUI

Describe the bug
I am using the MS hosted Azure DevOps. In the Version .Net Core Assemblies task, when using a yaml pipeline, the Add a Version field if missing checkbox is always disabled.
To Reproduce
Steps to reproduce the behavior:

Create a new yaml build pipeline.
Add the VersionDotNetCoreAssemblies task by using the Visual Assist editor.
Try to enable the Add a Version field if missing field; you can't.

The resulting yaml would look something like this:
    - task: VersionDotNetCoreAssemblies@2
      inputs:
        Path: '$(Build.SourcesDirectory)'
        VersionNumber: '$(versionNumber)'
        Injectversion: true
        FilenamePattern: '.csproj'
        OutputVersion: 'OutputedVersion'

Expected behavior
You should be able to check the box to enable the Add a Version field if missing behaviour.
Screenshots
You can see the checkbox is disabled in this screenshot.

I tried adding text to the The version field to update textbox as well thinking that the 2 controls might be linked, but it did not work.
Logs
N/A
What is the issue see:

Azure DevOps Services (VSTS)

Additional context
The work around is to not use the Visual Assist GUI editor and update the yaml text directly by adding the AddDefault: true input parameter.
e.g.
    - task: VersionDotNetCoreAssemblies@2
      inputs:
        Path: '$(Build.SourcesDirectory)'
        VersionNumber: '$(versionNumber)'
        Injectversion: true
        FilenamePattern: '.csproj'
        AddDefault: true
        OutputVersion: 'OutputedVersion'

Once you enable it via the yaml text, then the GUI checkbox is enabled and allows you to turn it on and off via the GUI.

That looks to be a bug in the  Visual Assist GUI editor, don't think I can do anything about that other than report to Microsoft, which I will do.
Remember I do provide details YAML documentation that gives all the options, though looking at it for this task the example is wrong for Add Default, there should be no e.g. at the end of the  description.

It is a known issue with the editor, I am told it should be fixed soon, you can track the progress at https://developercommunity.visualstudio.com/content/problem/1008735/pipelines-some-checkboxes-grayed-out-in-yaml-pipel.html
I will close this issue as it is not this extension at fault.


Local variables or user defined scope

Narrow scope for variables

Current template
Current page/head/etc content
Current template defined as template:declare (so not parser's class)

Or manually by using var:scope. It will act as a new scope when reading/writing variables. We could even named them and use them in multiple places theoretically. We can also nest scopes
<var:scope>
    All properties set here will be "unset" at the of tag
</var:scope>

Going with the user defined scopes without names for now


Don't strip ember-test-selectors from production builds

Purpose
Never strip ember-test-selectors (even in production builds), because we want to use them for Selenium tests.
Summary of Changes
Add
'ember-test-selectors': {
    strip: false,
},

to ember-cli-build.js
Side Effects
Slightly larger bundle.
Feature Flags
N/A
QA Notes
Should start seeing data-test-* attributes on elements that have been marked with them.
Try:
document.querySelector('[data-test-quick-search-input]');

on the dashboard.
Ticket
N/A
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Pull Request Test Coverage Report for Build 1479

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.067%

Totals

Change from base Build 1476:
0.0%

Covered Lines:
2288

Relevant Lines:
3547

💛  - Coveralls


small css things

Just putting this up because I know @kylgarmor is working on some CSS as well and want to make sure we don't step on each other's toes!

remove bottom border from headline to match new mock up
make page section top margin consitent with all components

so nice to see overloaded css go away


chore: Remove test-js-packages (docker) workflow

Description
Related to #3831
This removes the docker workflow and we can add it back when the issue is fixed
Problem*
Resolves
Summary*
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This might have ramifications for #3807 -- if so, it can be added back there with a fix


clair: add the additional legacy empty layer

This layer was used in docker prior to being able to support truly empty
layers. It is a 1k tarball of 0s.

OMG HERO!


Add support for htmlAttributes (or className) parameter.

Unless there already is a way to add classes to the containing div, I would suggest adding support for the (pretty common) htmlAttributes parameter.
@Html.React("MyComponent", new { data = Model }, htmlTag: "div", containerId: "my-container", htmlAttributes: new { @class = "my-class-needed-on-container" })

I can see this could be problematic if one should try to add an id using the htmlAttributes even if containerId is provided.
An alternative solution is to add a specific parameter className that takes a string of classes to add to the container. This is probably the most "sane" way to do it, since you probably do not want to allow other attributes than class to a React-container.
@Html.React("MyComponent", new { data = Model }, htmlTag: "div", containerId: "my-container", className: "my-class-needed-on-container" )

Feel free to send a pull request adding this if you want :)
Are there any use cases that can be solved by this that can't be solved simply by adding the attributes onto the React component instead?

My current problem is the use of child selectors in css. Since React.NET "adds an extra element" around the component, the child selector does not work unless I add another child level in the css. I could of course rewrite my other "components" to include the extra child level in the HTML also, but adding a class to the container seems like the easier path.  ;-) )
I will try to submit a pull request. It's my first ever, so I hope I get it right! :-)

Added in #211, thanks @jovnas


Tree View for Flow items

Description
closes #5627
I am starting this draft PR so you can start reviewing the code right now
There are a lot of things to be done yet but a very basic prototype is done
Checklist

[ ] Add Flow sorting for table
[x] Add onClick Flow event
[x] Added Highlight flow
[x] Filters for flows.
[ ] I have updated tests where applicable.
[ ] I have added an entry to the CHANGELOG.

I will be updating the code and features that are missing
Any kind of help/suggestion will be good

Edit: sorry for my dumb mistake on merge of mitmproxy (not really familiar with forks) 😥

I have added TreeView for flows but tests are not updated yet if everything seems fine in the code you can give me green light and i will update test


Weird behavior with update params

I didn't had the time to investigate deeper, but the following is happening:
👍 Works fine
api.update('characters', { id: 1 })

❌ Doesn't work
api.update('characters', { id: 1, character: { firstName: 'John' } })

The second scenario tries tries to request the /characters/:id URL (seems like it didn't replace the :id with the given id).

Fixed on 9600fb5e28a59abad8a8d7e7ced2b93a34b10aa9


Fixes assert being hit when entering a conversation on iOS 8

Current coverage is 64.34%

Merging #109 into dev will decrease coverage by -0.02% as of 0bdaaa5

@@              dev    #109   diff @@
=====================================
  Files          59      59
  Stmts        3098    3096     -2
  Branches        0       0
  Methods         0       0
=====================================
- Hit          1994    1992     -2
  Partial         0       0
  Missed       1104    1104

Review entire Coverage Diff as of 0bdaaa5

Powered by Codecov. Updated on successful CI builds.


Modifications in ETS - Date and date type should be checked if the thesaurus is "GEMET - INSPIRE themes, version 1.0"

Date and title should be correct with respect to the given controlled vocabulary. For the "GEMET - INSPIRE themes, version 1.0"

A new branch has been created for helpdesk issues. Issue solved and changes have been pushed into helpdesk branch.

ref #130
This issue should probably have been created in the ETS repository

[2017.4 meeting 2018-04-19] JRC to review the proposed change. The issue should be moved into the ETS repository (if possible in Github)


不知道有没有人遇到过_NestedScrollMetrics这个问题

flutter: The following _CastError was thrown while handling a gesture:
flutter: type '_NestedScrollMetrics' is not a subtype of type 'ScrollPosition' in type cast
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #0      RefreshPhysics.createBallisticSimulation

配合NestedScrollView确实有这个错误抛出,但没造成致命奔溃,下个版本修正

配合NestedScrollView确实有这个错误抛出,但没造成致命奔溃,下个版本修正

谢谢

已修正


[Doc][S3] Add docs about how to use minio s3 as storage type

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Add docs about how to use minio s3 as storage type
Documentation Links
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I found that we've already enable withPathStyleAccessEnabled in https://github.com/apache/dolphinscheduler/blob/efd68f480548839fe6c21dcdf3836afa42bc9029/dolphinscheduler-common/src/main/java/org/apache/dolphinscheduler/common/utils/S3Utils.java#L90-L95
Then this problem will no longer exist.


Getting error when running any phala commands

Hello,
I keep getting this error when I run any phala commands
/usr/bin/phala: line 75: syntax error near unexpected token `}'
/usr/bin/phala: line 75: `}'

@gngchrs Thank you for letting us know. We are aware of this issue and working on a fix. We apologize for the inconvenience.

This should be solved since 12/01/2021. If it persist please reinstall your scripts.


Add simple dropdown example.

What: Added a simple dropdown example

Why: Couldn't find one

How: Added a story to storybook

Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged
[ ] Added myself to contributors table

Codecov Report

Merging #219 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #219   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines         269    269
  Branches       61     61
=====================================
  Hits          269    269

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 8232cfd...268ed85. Read the comment docs.

Would you like to add yourself to the contributors table?

Yes please :-) created another PR for that, totally missed it.


Have the Windows home dir as an environment variable

Is your feature request related to a problem? Please describe.
I'd like to be able to access C:\Users\JohnDoe from WSL without having to do start a subprocess with wslpath -ua $(powershell.exe 'echo ${HOME}' |  tr -d '\r').
Describe the solution you'd like
Have an environment variable such as WSL_WINDOWS_HOME with the path /mnt/c/Users/JohnDoe.
Describe alternatives you've considered
wslpath -ua $(powershell.exe 'echo ${HOME}' |  tr -d '\r')

Additional context
An example application is sharing configuration or other sort of files accross multiple distros by storing them at some location in the Windows filesystem.
It would also be helpful if systemd had access to this variable so services could access the same functionality (In this regard related to #9719).

It is possible with WSLENV environment variable. More info here https://devblogs.microsoft.com/commandline/share-environment-vars-between-wsl-and-windows/
Example:

From Windows side:

> set WSLENV=USERPROFILE/u
> wsl.exe

From WSL side:

$ echo $USERPROFILE
C:\Users\x

Agreed with @Biswa96 here: WSLENV is the best way to access the Windows user profile path.


how use USE INDEX (....) and IGNORE INDEX (....) with MySQL

MySQL supports the use of USE INDEX and IGNORE INDEX (known as index hints)
See link here https://dev.mysql.com/doc/refman/8.0/en/index-hints.html
Example here.
SELECT * FROM table1 t1 USE INDEX (col1_index,col2_index)
  WHERE t1.col1=1 AND t1.col2=2 AND t1.col3=3;

How do I do this with KnexJS

Usage questions should go to stackoverflow (as it is mentioned in issue template).
Anyways I don't think there is any other way to achieve this, but putting the whole query to knex.raw.
knex.raw('SELECT * FROM ?? USE INDEX (??,??) WHERE ??=? AND ??=? AND ??=?', ['table1 as t1', 'col1_index', 'col2_index','t1.col1',1,'t1.col2',2,'t1.col3',3])


Widget class

Part of #690.
cc @sdroege @EPashkin

@GuillaumeGomez, Thanks, LGFM
But seems we need add #![cfg_attr(feature = "cargo-clippy", allow(derive_hash_xor_eq))] to lib.rs too.
Currently 3 clippy error like this:
error: you are deriving `Hash` but have implemented `PartialEq` explicitly
  --> src/auto/paper_size.rs:16:38
   |
16 |     #[derive(Debug, PartialOrd, Ord, Hash)]
   |                                      ^^^^
   |
   = note: #[deny(derive_hash_xor_eq)] on by default
note: `PartialEq` implemented here
  --> src/auto/paper_size.rs:182:1
   |
182| / impl PartialEq for PaperSize {
183| |     #[inline]
184| |     fn eq(&self, other: &Self) -> bool {
185| |         self.is_equal(other)
186| |     }
187| | }
   | |_^
   = help: for further information visit https://rust-lang-nursery.github.io/rust-clippy/v0.0.212/index.html#derive_hash_xor_eq

@EPashkin Weird... I'll make another PR to do it.
@sdroege: Ok! Can't wait to see what you'll come with. :)

This is not really fixed btw :) You would now add the functions to gtk::WidgetClass, which exists now (and there's a way to get it from an instance). See for example here https://gitlab.freedesktop.org/gstreamer/gstreamer-rs/blob/7a69a1137c190927e351205074b641d525e6ce25/gstreamer/src/subclass/element.rs#L245-274


[error message feedback] provide feedback for file-not-found

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
Problem:
I can't figure out why I'm getting this error and a broken link with file not found error.
The file is here, in my patch-11 branch, and that's where the PR is coming from.
https://github.com/LukaszPawlowski-MS/powerbi-docs-pr/blob/patch-11/powerbi-docs/media/service-admin-portal/power-bi-admin-block-classic-workspaces.png
The image link refers to it seemingly correctly as below.
media/service-admin-portal/power-bi-admin-block-classic-workspaces.png
The reference to the image seems ok too, and matches what other links in the same pages are using.
Log code info
(This is auto filled, no need to modify)

Severity
Type
Code
Category
File
Message
Source
Line
Time

Warning
User
file-not-found

powerbi-docs/admin/service-admin-portal.md
Invalid file link: 'media/service-admin-portal/power-bi-admin-block-classic-workspaces.png'.

216, 1
07/22/2020 16:22:52

Link to report
(This is auto filled, no need to modify)
report url

@LukaszPawlowski-MS Should this issue be moved to the repo at https://github.com/MicrosoftDocs/powerbi-docs ?


add v in from of require commands of go.mod in README.

Would save a lot of time.

Closing this in favor of #429


5.1.5 FIA_AFL_EXT.1.1

Change the list of remote authentication methods to "remote authentication based on [selection: username and password, username and a PIN that releases an asymmetric key stored in OE-protected storage, X.509 certificates, SSH public-key credential]"
As currently worded, FIA_UAI.5.1 and FIA_AFL_EXT.1.1 are in disharmony; there are FIA_UAU.5.1 remote login methods for which no failure handling is required! This change synchronizes the authentication methods between FIA_AFL_EXT.1.1 and FIA_UAU.5.1 (as previously existed in v1.0 of the PP).

Made a note of this, but I think it is intentional that certificate- and credential-based failures are not counted.


Non-resizable textcats to legacy

Adding spacy-legacy.TextCatCNN.v1 and spacy-legacy.TextCatBOW.v1

One final thing: can you add the architectures to the tests here?
https://github.com/explosion/spacy-legacy/blob/master/spacy_legacy/tests/test_legacy.py
(I can do it later today if you're short on time.)


Support Python 3

What does this PR do?
Moves GoExpvar to Python3! This one was a straightforward one. I was worried about that, but it seems like the string manipulation issues are mostly handled in this one.
Motivation
🐍 3⃣ 🚂
Review checklist

[x] PR has a meaningful title or PR has the no-changelog label attached
[x] Feature or bugfix has tests
[x] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
[ ] If PR adds a configuration option, it has been added to the configuration file.

Additional Notes
Anything else we should know when reviewing?

Codecov Report

Merging #2833 into master will increase coverage by 3.94%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##           master    #2833      +/-   ##
==========================================
+ Coverage   84.69%   88.63%   +3.94%
==========================================
  Files         662        7     -655
  Lines       37611      264   -37347
  Branches     4507       40    -4467
==========================================
- Hits        31853      234   -31619
+ Misses       4436       19    -4417
+ Partials     1322       11    -1311


[research] API analytics solution

We want to ensure we have a robust solution for API analytics and monitoring.
Analytic data points we would like to track:

Requests and traffic per version / URL
Requests and traffic per endpoint
error logs

Requirements

[ ] Evaluate Vercel as a solution
[ ] If Vercel does not appear to be a good direction, make other suggestions

Vercel Evaluation

[X] Requests per version/URL : https://vercel.com/dashboard/klimadao/monitoring/01HFRNPP02VREA6ZDGWAHH65F4
[X] Requests per endpoint: https://vercel.com/dashboard/klimadao/monitoring/01HFRPFZ4YB8XWZ0PQ6A0180CJ
[X] Traffic per version/URL : https://vercel.com/dashboard/klimadao/monitoring/01HFRQ14WCWB3S4M1NMJ97DV4P
[X] Traffic per endpoint: https://vercel.com/dashboard/klimadao/monitoring/01HFRPYWSY9X9978P8JB8PCKDW
[X] error logs: https://vercel.com/klimadao/carbonmark-api/logs?page=2&timeline=past30Minutes&startDate=1700561147638&endDate=1700562947638&levels=error&levels=warning

Pros:

The solution is non intrusive (No library or extra code to add)
The monitoring solution seems solid and customizable enough through queries configurable through the UI.
The query system is well documented: https://vercel.com/docs/observability/monitoring/monitoring-reference#where .

Cons:

The query system being UI based. We may hit a wall if we need very complex queries.
The data retention period is only 30 days on the Pro plan (90 days with the enterprise plan)
The error analysis is poor:

Only 1 day of data retention
No stack traces 5xx errors

Provisional conclusion

The Vercel analytics solution seems very good in terms of reporting traffic and requests even though it is not 100% customizable.
There is no real error analysis, but it is normal since it does not capture  code exceptions. It is only based on HTTP status code.
There are integrations possible between Vercel and other platforms to improve error analysis. I am going to explore that next.

@Atmosfearful for your review


How to Get GPT Engineer to Implement "Complex" Algorithms?

When asking GPT Engineer to perform a minor complex code edit I get the following message:
Please note that the actual logic to do X, Y, and Z  is not included in the edit blocks above. This is because the implementation of these algorithms can be quite extensive and would require a significant amount of code. However, the structure and method signatures have been provided to guide the implementation.
Is this the desired behavior? Is there any way to force GPT Engineer to implement more complex code edits?
Note: I've replaced the actual complex operations names with X, Y and Z.
Thank you

This is unfortunately a feature of chatGPT rather than a direct thing with gpt-engineer. gpt-engineer tries to be quite clear that it expects a complete implementation, but sometimes chatGPT just really doesn't want to. From case to case, breaking down a complex program in multiple prompts could be a way, but I understand that this is not a very satisfactory answer.


EBS data volume not mounted for some Nextflow jobs

Some of the jobs submitted with Nextflow do not mount the 200GB data disk, resulting in "No space left on device" error. This happens only with a few random jobs.
Steps to Reproduce
agc workflow run demo-nextflow-project --context spotCtx
Relevant Logs
I get this below error:
|abort: write error ....(No space left on device)
Doing a df -h on the instance shows the 200GB disk (/dev/nvme1n1) not being mounted:

df -h
  | Filesystem Size Used Avail Use% Mounted on
  | overlay 30G 9.4G 20G 33% /
  | tmpfs 64M 0 64M 0% /dev
  | tmpfs 5.0G 0 5.0G 0% /sys/fs/cgroup
  | /dev/nvme0n1p1 30G 9.4G 20G 33% /opt/aws-cli
  | shm 64M 0 64M 0% /dev/shm
  | tmpfs 5.0G 0 5.0G 0% /proc/acpi
  | tmpfs 5.0G 0 5.0G 0% /sys/firmware

Expected Behavior
Mount the 200GB disk (/dev/nvme1n1 ) on /

df -h
| Filesystem Size Used Avail Use% Mounted on
| /dev/nvme1n1 200G 12G 188G 6% /
| tmpfs 64M 0 64M 0% /dev
| tmpfs 3.8G 0 3.8G 0% /sys/fs/cgroup
| /dev/nvme0n1p1 30G 6.4G 23G 22% /opt/aws-cli
| /dev/nvme1n1 200G 12G 188G 6% /etc/hosts
| shm 64M 0 64M 0% /dev/shm
| tmpfs 3.8G 0 3.8G 0% /proc/acpi
| tmpfs 3.8G 0 3.8G 0% /sys/firmware

Actual Behavior
See above expected behavior
Operating System:
AGC Version: agc version: 1.0.0-41-gc5ac696
Was AGC setup with a custom bucket: No
Was AGC setup with a custom VPC: No

I noticed that this is mostly happening with the M5n,  R5n, C5n instances types.
Here is the error message:

+ systemctl enable amazon-ssm-agent
+ systemctl start amazon-ssm-agent
Job for amazon-ssm-agent.service canceled.
 util.py[WARNING]: Failed running /var/lib/cloud/instance/scripts/runcmd [1]
 cc_scripts_user.py[WARNING]: Failed to run module scripts-user (scripts in /var/lib/cloud/instance/scripts)
util.py[WARNING]: Running module scripts-user (<module 'cloudinit.config.cc_scripts_user' from '/usr/lib/python2.7/site-packages/cloudinit/config/cc_scripts_user.pyc'>) failed

``

Hello snamburi3! Could you please help us tracing this issue down by pointing us which workflow ends up giving you this error? Is it any of the workflows  defined in the demo-nextflow-project or anything else that you have tried to run?

Thank you, Andrey. It is the workflow defined in the demo-nextflow-project. Also, please ignore my above comment that this happens only on the M5n, R5n, C5n instances types. I have seen this happen with other instance types as well like M5, R5, C5 instances types. Some of the instances just fail to mount the  /dev/xvdba disk as user scripts on the instance fail to run.

Both of this and #40 indicate that the EC2 instances are not correctly provisioning their dependencies. I will add a new feature request for better validation of provisioning and close these issues in favor of that.

Checking in on this ticket. Is this a problem that you have had consistently with this workflow or is this a sporadic issue? If it is the latter we are working on some solutions to make provisioning of the dependencies more reliable. If it is a consistent issue there may be a different problem we need to investigate.

Hi @markjschreiber
Thanks for your work on this. I have been experimenting the usage of AGC with NextFlow. The EBS autoscale generally works really well. However, it fails when multiple large files are being staged by NextFlow at the same time. I think the source of failure is this line in awslabs/amazon-ebs-autoscale: https://github.com/awslabs/amazon-ebs-autoscale/blob/a55e7cc8913dd89e42b4668e4e8883209db26e9c/bin/ebs-autoscale#L209
STATS=$(df -BG  ${MOUNTPOINT} | grep -v Filesystem)
TOTAL_SIZE=$(echo ${STATS} | awk '{print $2}')
USED=$(echo ${STATS} | awk '{print $3}')

The stats are based on the df command which under reports when multiple files are being copied to a directory inside ${MOUNTPOINT}. I think it can be fixed by using du as in:
USED=$(du -BG -d 0 ${MOUNTPOINT} | awk '{print $1}')

This patch can be applied to the awslabs/amazon-ebs-autoscale copy included with AGC: https://github.com/aws/amazon-genomics-cli/blob/main/packages/cdk/lib/artifacts/batch-artifacts/amazon-ebs-autoscale.tgz
Cheers
Usman R.


Visual Studio triggers changes check multiple times in nodemon for one save

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.62.0 (Universal)
OS Version: OS: Darwin x64 20.6.0

This is similar to #48501, except it's not related to the debugger.
Steps to Reproduce:

Open Visual Studio Vode
Open Terminal
Run nodemon
Save a file (doesn't even require changing the file)

It happens more often than not, but sometimes it'll take multiple saves to get the avalanche of restarts. Running nodemon --verbose tells me it's the same file each time. I originally had the project folder on Google Drive, and to rule out that as the culprit I copied the folder to my desktop and restarted VS Code and used the desktop version. The problem continued to exist. The problem does go away, so it does seem to be the Google Drive.

I hope there is something VS Code can do to mitigate the extraneous file change notifications when saving the file? Otherwise, feel free to close the ticket.

Maybe related to #106879?

Any save ends up to be a truncate and 1-many write calls. Quite possible that this emits more than one event but maybe nodemon could add a bit of delay and only react after having listened to events for a bit?

@bpasero Good idea. I also created https://github.com/remy/nodemon/issues/1952.


admins option seems useless

For enterprise chef is there anyway to give clients permission to read the users endpoint?  I can't find anywhere set permission for user objects on the chef server (11.14).  If I set admins to anything other than '' I get a permission denied error and the chef run errors out.

Not to mention admin seems required:
module ChefVaultCookbook
  module Resource
    class ChefVaultSecret < Chef::Resource::LWRPBase
      self.resource_name = 'chef_vault_secret'
...
      attribute(:admins, kind_of: [String, Array], required: true)

It was suggested that I use knife-acl to add read permissions for the users container to the client, but it doesn't look like that's possible with enterprise/hosted chef.

Looks like it's a known access to user keys issues: http://onddo.github.io/chef-encrypted-attributes/#chef-user-keys-access-limitation
I'm guessing there aren't many folks using chef-vault (creating/encrypting in a recipe on a node) or they'd have similar issues.

Can you chime in here @thommay


Warning: Function components cannot be given refs. Attempts to access this ref will fail. Did you mean to use React.forwardRef()?

I am seeing this warning/error coming from this library. Any idea what's going on here?

Please try to play with the alpha v3.0.0 version of the library, which uses a hook that passes props to your native TextInput component.


Detect DB2's Procedure language (SQL-PL)

I have developed a couple of projects in IBM's SQL-PL, which is a Procedure language. However, the language analyse in GitHub says this is SQL, and I know it is based on linguist.
The language documentation is here: http://pic.dhe.ibm.com/infocenter/db2luw/v10r5/topic/com.ibm.db2.luw.apdv.sqlpl.doc/doc/c0011916.html
My projects use many elements of this language:

https://github.com/angoca/log4db2
https://github.com/angoca/db2unit

This should be fixed in https://github.com/github/linguist/pull/2175 . These changes should be live on GitHub in the next week or so.


CallToActionSection

Based on the most up to date designs: https://www.figma.com/file/KE2Sw6ob07agN25RmeNuFg/GiveCycle-wireframes?node-id=4076%3A12228

large Card components with animated button

separate CTACard component
make sure to put all static text in common file

Hello everyone, I'm not sure how to assign myself to this but I will be working on this.


Question about Edge Conv

Hi, I have a question about Edge Conv implementation of yours.
EdgeConv constructs K number of local graph making the shape as Batch_size * C * Pts * K.
According to your implementation, kernel 1 * 1 is applied and it means that the local kernel weights are all same.
However, in image convolution, weights in each kernel is different.
For example, if there is 3 * 3 kernel conv, current dgcnn has same weight across the kernel.
Then, doesn't it make more sense to use (1 * K) like x-conv from PointCNN to capture local features?
What is your reason behind using 1 * 1 convolution rather than 1 * K convolution?
Thank you!

Suppose you want to transform features from (Batch_size, C1, Pts, K) to (Batch_size, C2, Pts, K). In convolution, the size of each nn.Conv2d kernel is (1, C1, 1, 1). There are C2 kernels.
See https://pytorch.org/docs/stable/generated/torch.nn.Conv2d.html?highlight=nn conv2d#torch.nn.Conv2d for more details.

Thank you for the reply.
Maybe I am misunderstanding something, but so I know that there will be C2 different kernels as output features will be C2.
However, 1 * 1 kernel only extracts features from individual point like pointnet, so it can not extract any local features.
How does DGCNN achieve this by using 1 * 1 kernel?
I presume it achieves it by calculating edge features and max-pool operation.
Is this correct?

There is a max-pooling after each convolution to aggregate local features.

I suggest you refer to the DGCNN paper on how DGCNN extracts local features. My code strictly follows the paper for implementation.


chore: Merge release into pg

Description
PR to merge release into pg
/test Sanity
:mag: Cypress test results

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10401977495
Commit: 6d67be7ee6462fae3e6c6c54813dd601c13766ea
Cypress dashboard.
Tags: @tag.Sanity
Spec:
Thu, 15 Aug 2024 10:32:01 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

Merging this as the test is passing in local.