How do I change SiteDescription and siteImage in gatsby-config.js?

As per title. My website keeps defaulting to the generic lekoarts banner and description.

You overwrite the values in your gatsby-config.js file:
module.exports = {
  siteMetadata: {
    siteDescription: `Your description.`,
    siteImage: `/your-image-path.jpg`,
  },
}

Thank you for opening this!
I'm marking this issue as answered and closing it for now but please feel free to comment here if you would like to continue this discussion. Thanks for using my themes!

I'm having the same issue. I first tried adding my own banner.jpg image in static/, but my site continued to default to the LekoArt banner with my own site's description:

Then, I tried changing the value in gatsby-config.js to point to a PNG version of my banner instead: siteImage: "/banner.png", which is also in the static/ dir. This leads to no banner being shown in link previews:

I'm not quite sure what I'm doing wrong here. I cloned my repo from a friend's site repo, and he had just added a new banner.jpg in the same location without changing the value of siteImage (which I tried first, described above), and his site's link previews show his banner as expected:

Does anyone have a fix for this?

This is probably an intermittent issue with the scraping for the preview (cached response). Changing the file should be enough.

This is probably an intermittent issue with the scraping for the preview (cached response). Changing the file should be enough.

Good to know! And just to clarify, the preview image is format-agnostic, right? It should be ok to use a PNG instead of a JPG?

As long as you overwrite the siteImage property, yeah.

I tried clearing cache and building the repo again (on Netlify), but does not seem to work for me I still get the default banner.
Although I already replaced it with a new banner.
siteMetadata: {
    // Used for the title template on pages other than the index site
    siteTitle: `Secure Secrets`,
    // Default title of the page
    siteTitleAlt: `Securing the secentralized web`,
    // Can be used for e.g. JSONLD
    siteHeadline: `Secure your Secrets`,
    // Will be used to generate absolute URLs for og:image etc.
    siteUrl: `https://securesecrets.network`,
    // Used for SEO
    siteDescription: `Securing the decentralized web using Secret Network`,
    // Will be set on the <html /> tag
    siteLanguage: `en`,
    // Used for og:image and must be placed inside the `static` folder
    siteImage: `/banner.jpg`,
    // Twitter Handle
    author: `@mohammedpatla`,
    // Navigation links
  }

Any suggestions @LekoArts
Were you able to get yours to work @jidicula ?

I tried clearing cache and building the repo again (on Netlify), but does not seem to work for me I still get the default banner.
Although I already replaced it with a new banner.

siteMetadata: {

    // Used for the title template on pages other than the index site

    siteTitle: `Secure Secrets`,

    // Default title of the page

    siteTitleAlt: `Securing the secentralized web`,

    // Can be used for e.g. JSONLD

    siteHeadline: `Secure your Secrets`,

    // Will be used to generate absolute URLs for og:image etc.

    siteUrl: `https://securesecrets.network`,

    // Used for SEO

    siteDescription: `Securing the decentralized web using Secret Network`,

    // Will be set on the <html /> tag

    siteLanguage: `en`,

    // Used for og:image and must be placed inside the `static` folder

    siteImage: `/banner.jpg`,

    // Twitter Handle

    author: `@mohammedpatla`,

    // Navigation links

  }

Any suggestions @LekoArts
Were you able to get yours to work @jidicula ?

Not exactly, the link preview still doesn't show my banner image, but it's not displaying the template banner anymore.

@jidicula I think I fixed the issue, by creating a copy of the banner.jpg in the src/../sections/ folder

@mohammedpatla where is the src/.../sections/ folder? I'm using the minimal blog theme, my project repo can be seen here.
Good to hear that it works fine on Discord, so that means that LinkedIn and Telegram might be slower to refresh their caches for unravelling link previews?

Um not too sure where this would be for you @jidicula
But I think you can place it in /components or /src and try.

Update: seems fine for me now, with the banner card being generated as expected.

Update: seems fine for me now, with the banner card being generated as expected.

Hi @jidicula! Thank you for pointing out the issue. I meet the exact same issue as yours. May I ask that how did you solve the problem? Are you still using placing the banner.jpg at static/?

Update: seems fine for me now, with the banner card being generated as expected.

Hi @jidicula! Thank you for pointing out the issue. I meet the exact same issue as yours. May I ask that how you solved the problem? Are you still placing the banner.jpg at static/?

I am, and it fixed itself somehow - I think it's some kind of black-box caching on each platform that does card previews. You can see my site repo here.

@jidicula Thanks for your kind reply! I'll check the cache.


If a machine that should be online is offline

You end up with a SocketException...

In general the way to test this is to throw an exception during various places - like in HasFile in PlaceLinuxRepro - so there it should always return false!


google_sign_in build failure

Steps to Reproduce
Run: 'Flutter run' in the google_sign_in example app.
Logs
exampe/android/app/src/main/java/io/flutter/plugins/GeneratedPluginRegistrant.java:11: error: cannot find symbol
GoogleSignInPlugin.registerWith(registry.registrarFor("io.flutter.plugins.googlesignin.GoogleSignInPlugin"));
^
symbol:   method registerWith(Registrar)
location: class GoogleSignInPlugin
1 error
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.4 16E195, channel alpha)
• Flutter at /Apps/Flutter
• Framework revision 9f1d5ba50e (8 hours ago), 2017-05-09 17:06:52 -0700
• Engine revision cbbf956fcb
• Tools Dart version 1.23.0-dev.11.11
[✓] Android toolchain - develop for Android devices (Android SDK 25.0.2)
• Android SDK at /Users/pietertheron/Library/Android/sdk
• Platform android-25, build-tools 25.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
[-] iOS toolchain - develop for iOS devices
✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
Download at: https://developer.apple.com/xcode/download/
Once installed, run 'sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer'.
✗ ideviceinstaller not available; this is used to discover connected iOS devices.
To install, run:
brew update
brew install --HEAD libimobiledevice
brew install ideviceinstaller
✗ ios-deploy not installed. To install:
brew update
brew install ios-deploy
✗ CocoaPods not installed.
CocoaPods is used to retrieve the iOS platform side's plugin code that responds to your plugin usage on the Dart side.
Without resolving iOS dependencies with CocoaPods, plugins will not work on iOS.
For more info, see https://flutter.io/platform-plugins
To install:
brew update
brew install cocoapods
pod setup
[✓] Android Studio (version 2.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Gradle version 3.2
• Java version: OpenJDK Runtime Environment (build 1.8.0_112-release-b06)
[✓] IntelliJ IDEA Community Edition (version 2017.1.2)
• Dart plugin version 171.4424
• Flutter plugin version 13.1
[✓] Connected devices
• SM G920F • 06157df6ac48212c • android-arm • Android 6.0.1 (API 23)

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

There was one last change to plugin registration which might be related?
https://groups.google.com/forum/#!topic/flutter-dev/zba1Ynf2OKM

That was the issue for me.


Poor performance on cuda const memory

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

Hello, I'm now currently working on one numba project that will use a large const array. I tried to create the memory using cuda.const.array_like but it was somehow different from what i thought:

the const memory creation process took a really, really long time;
it cost a large amount of host memory.

Here is the testing code:
import numpy as np
import numba
from numba import cuda
import time

const_mem=np.arange(2**24,dtype=np.int8) # 16M memory

@cuda.jit()
def cuda_copy(copy_test):
  tx = cuda.threadIdx.x
  ty = cuda.blockIdx.x
  bw = cuda.blockDim.x
  pos = tx + ty * bw
  cuda_const_mem=cuda.const.array_like(const_mem)

  copy_test[pos]=cuda_const_mem[pos]

@cuda.jit()
def cuda_copy2(copy_test,source):
  tx = cuda.threadIdx.x
  ty = cuda.blockIdx.x
  bw = cuda.blockDim.x
  pos = tx + ty * bw

  copy_test[pos]=source[pos]

to_be_filled=np.empty((2**24,),dtype=np.int8)
threadsperblock = 512
blockspergrid = len(to_be_filled)//threadsperblock

start_time = time.time()
cuda_copy2[blockspergrid, threadsperblock](to_be_filled,const_mem)
print(f'cuda_copy2: {time.time()-start_time:.3f} seconds')

start_time = time.time()
cuda_copy[blockspergrid, threadsperblock](to_be_filled)
print(f'cuda_copy: {time.time()-start_time:.3f} seconds')

And the output:
cuda_copy2: 1.337 seconds
Traceback (most recent call last):
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\cudadrv\driver.py", line 2160, in add_ptx
    driver.cuLinkAddData(self.handle, enums.CU_JIT_INPUT_PTX,
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\cudadrv\driver.py", line 300, in safe_cuda_api_call
    self._check_error(fname, retcode)
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\cudadrv\driver.py", line 335, in _check_error
    raise CudaAPIError(retcode, msg)
numba.cuda.cudadrv.driver.CudaAPIError: [218] Call to cuLinkAddData results in UNKNOWN_CUDA_ERROR

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "U:\Projects\python\numba_const_test.py", line 36, in <module>
    cuda_copy[blockspergrid, threadsperblock](to_be_filled)
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\compiler.py", line 821, in __call__
    return self.dispatcher.call(args, self.griddim, self.blockdim,
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\compiler.py", line 964, in call
    kernel = _dispatcher.Dispatcher._cuda_call(self, *args)
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\compiler.py", line 972, in _compile_for_args
    return self.compile(tuple(argtypes))
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\compiler.py", line 1102, in compile
    kernel.bind()
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\compiler.py", line 590, in bind
    self._func.get()
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\compiler.py", line 441, in get
    linker.add_ptx(ptx)
  File "V:\Program Files\Python\lib\site-packages\numba\cuda\cudadrv\driver.py", line 2163, in add_ptx
    raise LinkerError("%s\n%s" % (e, self.error_log))
numba.cuda.cudadrv.driver.LinkerError: [218] Call to cuLinkAddData results in UNKNOWN_CUDA_ERROR
ptxas error   : File uses too much global constant data (0x1000000 bytes, 0x10000 max)

I was able to come up with a work around that pass the const array as a parameter of cuda function, but i believe the performance of cuda const array should be a improved.

Thanks for the report. @gmarkall could you take a look at this please? Thanks.

The constant memory size is only 64KB - see Table 15 in https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#compute-capabilities
Therefore you can't use more than 64KB of constant data. If you have this much data, you will need to put it in a device array and pass it into your kernel as a parameter.

Thanks for answering.the problem is solved 😄


Difficult to move items between lists

When using DragAndDropListExpansion, I have found it very difficult to drag an item from a list to another list especially if the destination list is empty. Problem is that the title widget of DragAndDropListExpansion and the contentsWhenEmpty widget do not accept the item. As a result of this it is quite difficult to find the correct location where the item should be dropped.

+1

+1

+1

Have you tried setting the lastItemTargetHeight and lastListTargetHeight, while setting contentsWhenEmpty: const SizedBox.shrink()?


Requiring v8-profiler-next fails across multiple threads.

The following script fails with "Module did not self-register" errors using v8-profiler-next 1.5.1. This is similar to https://github.com/hyj1991/v8-profiler-next/issues/9 and perhaps the fixes haven't been ported over?
I'm currently on Node v12.20.1.
$cat prof.js
const {Worker, isMainThread, parentPort} = require('worker_threads');
const fs = require('fs');

function work() {
    require('v8-profiler-next');
    parentPort.postMessage('hi');
}

if (isMainThread) {
    require('v8-profiler-next');
    new Worker(__filename).on('message', (result) => {
        console.log(result);
    });
} else {
    work();
}

$node --version
v12.20.1
$node prof.js

events.js:291
      throw er; // Unhandled 'error' event
      ^
Error: Module did not self-register: '.../src/hyperbase/node_modules/v8-profiler-next/build/Release/profiler.node'.
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:1057:18)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Module.require (internal/modules/cjs/loader.js:887:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.<anonymous> (.../src/hyperbase/node_modules/v8-profiler-next/dispatch.js:2:17)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
Emitted 'error' event on Worker instance at:
    at Worker.[kOnErrorMessage] (internal/worker.js:242:10)
    at Worker.[kOnMessage] (internal/worker.js:252:37)
    at MessagePort.<anonymous> (internal/worker.js:168:57)
    at MessagePort.emit (events.js:314:20)
    at MessagePort.onmessage (internal/worker/io.js:80:8)
    at MessagePort.exports.emitMessage (internal/per_context/messageport.js:11:10)

Hello, I'll soon refactor v8-profiler-next to be context-aware to support profiling in worker_threads.

landed in https://github.com/hyj1991/v8-profiler-next/commit/8eea60851be85e444cf933f838eba5c7a7bcfc70


[dataprotection] Minor bugfix for AKS Vaulted Backups and support for new paired region

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

az dataprotection backup-vault initialize-restoreconfig: Based on the existence of new parameters staging_resource_group_id and staging_storage_account_id, we correctly set the object type of the output JSON.
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

/azp run


feat: Improve log verbosity

Description:

Adds the output of the externally executed commands in DockerService to the debug level of logs.
Reduces the repeated error logs when waiting for a service to start up.

Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Attention: Patch coverage is 17.24138% with 24 lines in your changes are missing coverage. Please review.

Project coverage is 50.41%. Comparing base (d34c153) to head (da667d3).

Files
Patch %
Lines

src/services/DockerService.ts
0.00%
12 Missing :warning:

src/utils/debounce.ts
22.22%
7 Missing :warning:

src/services/ConnectionService.ts
16.66%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #577      +/-   ##
==========================================
- Coverage   51.39%   50.41%   -0.99%
==========================================
  Files          24       25       +1
  Lines        1181     1202      +21
  Branches      136      138       +2
==========================================
- Hits          607      606       -1
- Misses        550      572      +22
  Partials       24       24

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fixed typo in code example

Fixed a typo which could lead to some frustration if people just copy the string and try it.

thanks


Question on how to use the evaluate.py

Hi I have some issue when trying to run the evaluate.py. This is my commend:
python -m evaluate --config default_inference_args.yaml --complex_name DD_5R7Y_TC5_100_test --protein_path data/5r7y_rec.pdb --ligand_description data/TC5.sdf --out_dir results/user_predictions_small
And the error is:
Traceback (most recent call last):
  File "/home/tur54445/work/anaconda3_2023/envs/diffdock-gpu/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/tur54445/work/anaconda3_2023/envs/diffdock-gpu/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/gpfs/work/tur54445/git/DiffDock/evaluate.py", line 12, in <module>
    resource.setrlimit(resource.RLIMIT_NOFILE, (64000, rlimit[1]))
ValueError: current limit exceeds maximum limit

Hi, the evaluate.py file is to run the model on some of the standard datasets, to run individual predictions please use the inference.py file

Thanks!


Proposal for a better run-time handling of shared library versions

People testing my Rawtherapee and GIMP appimages on  various Linux distributions have sometimes reported conflicts between bundled and system libraries, typically showing up as missing symbols. I am currently using CentOS7 for building my packages.
Generally speaking, there is often a need to decide at run-time wether to use the system or the bundled version of some specific library, whichever is newer. One known example is libstdc++, but there are other cases as well.
Another example is the fontconfig library. Currently I am bundling the library, as well as its dependency libfreetype, into the appimages. However, I want to use the system-provided /etc/fonts/fonts.conf file, so that I do not need to bundle fonts into the AppImage itself.
Now the problem is that if the system fontconfig version is too new, the bundled library might give errors when parsing /etc/fonts/fonts.conf. In this case, the bundled libfontconfig should be skipped and the system one used instead. However, this might bring other issues with dependent libraries, like libfreetype: the system fontconfig library requires symbols that are not provided by the bundled libfreetype (because it is too old), and the system libfreetype should be used as well...
To solve this, I am developing some bash code that, for a given library, compares the versions of the system and bundled copies, and "disables" the bundled one when the system one is newer.
The library version is extracted from the suffix of the dereferenced *.so* file name.
For this to work, I had to introduce some changes into the way the libraries are bundled. In particular, I had to:

copy all files of a given library into the AppImage, preserving symlinks whenever possible, so that the  dereferenced *.so* file is available in the bundle (see here). All library files are copied directly into $APPDIR/usr/lib.
when deleting the blacklisted libraries, delete all files matching $LIBRARY.so* (see here).
when starting the appimage, create a temporary folder and symlink all bundled libraries there, so that some libraries can be deleted if system ones should be used. Add this temporary directory to LD_LIBRARY_PATH, but DO NOT ADD $APPDIR/usr/lib (see here).
for some specific libraries, use the full *.so* name to determine the library versions, and delete the copy in the temporary library dir if the system library is newer (see here)

An example of how this mechanism is used can be fount in the {Rawtheapee AppRun script](https://github.com/aferrero2707/rt-appimage/blob/master/ci/AppRun#L5).
With this, I was able to provide compatibility with "rolling" distributions like Sabayon and Majaro (in addition to "classical" ones like Debian stable/testing, Ubuntu and Fedora).
However, I would like to have the opinion of the experts (pinging @probonopd and @TheAssassin in particular) regarding this. Of course, if you find this useful, feel free to integrate it in the default AppImage scripts...
Thanks!

I guess we'd need collect empirical evidence, a.k.a. trial and error, as we have never experimented with this scheme.
While I am not opposed to "crude hacks", maybe there is a cleaner way to implement a similar scheme that does not need symlinks and such: Could we take the code of ld-linux.so and hack it into a custom AppRun that would compare the system version and the bundled version of each library and always load the newer one?


Ability to use NeoPixel leds

All by boards uses neopixel, this one allow to use them, may be implemented into yasolr :-)

Added NeoPixel possible use (even both same time)
Added NeoPixel Example
Added Adafruit NeoPixel Library

For now straight compatibility just drive one color on each led Green, Yellow and Red, if needed I'll add ability to traffic light only with one Led.

Thanks! I wasn't expecting some PR on such a small project :P I will review!
I have updated the CI files for now (which conflicts a little)

Thanks! I wasn't expecting some PR on such a small project :P I will review!
I have updated the CI files for now (which conflicts a little)

hidden goal is to be able to use them with small boards for yasolr like this one all small connectors to connect relays :-)

I have updated the CI files for now (which conflicts a little)

Sorry I'm far from expert in PR/Rebase/Conflict, don't know what I need to do on this one so solve conflict, I think I'll let you hand on this.

I have updated the CI files for now (which conflicts a little)

Sorry I'm far from expert in PR/Rebase/Conflict, don't know what I need to do on this one so solve conflict, I think I'll let you hand on this.

ok I will grab your changes then :-)

@hallard : please let me know when I can rebase your PR branch on main and add the CI :-)
I will need to force push into your branch for that (I see I should be able to do it).

@mathieucarbou should be good for now :-)

use the _gpioEnabled flag to check for lifecycle (end)

Not sure how to handle this one, in fact I though you could have both call to begin() one for gpio other for neopixel (all leds would be synched of course in this case)
Imagine call 1st for neopixel then call second with wrong ping, then 2nd set _gpioEnabled to fasle and by the way disable 1st one. I don't think it's the expected result.
May be we could handle another flag as _neopixelEnabled or use gpio mode OR neopixel but not both same time.

Do you know some NeoPixel leds with only 3 leds ? I saw strips and single led, but did not fidn a set of 3 ?

Nope usually I cut strip or use this kind of awesome, lot of nice models

got theese one in 5V can't find them now, looks like all 12V

https://www.aliexpress.com/w/wholesale-20pcs-rgb-pixel-led-ws2811-5V.html

Thanks for the hints! I will have a look. These are funny: I am so busy with other things... I should pause for a few weeks at some point just to make something fun for the kids with that ;-)

@hallard : I have rebased the PR, updated the CI and did some updates on the code for a better integration with what was existing. Would you be able to test ? If it works, I will merge and release.
Thanks for this addition!

@hallard : I have rebased the PR, updated the CI and did some updates on the code for a better integration with what was existing. Would you be able to test ? If it works, I will merge and release.
Thanks for this addition!

@mathieucarbou I'm very sorry, I'm totally lost, between my fork, rebase from main of my branch of for upstream, and merge sometime git drives me mad just for few line of code. I propose you to merge, then I will delete my branch and sync my fork and test. If any issue I will create a new branch if ok to you

Thanks for the hints! I will have a look. These are funny: I am so busy with other things... I should pause for a few weeks at some point just to make something fun for the kids with that ;-)

I know what it is ... Can send you some if needed, got some of old projects, looks great to put in Solar Router Enclosure

@mathieucarbou looks like I succeeded to resolv conflicts on my side, but since I can't build anymore nor test.
So for now I'm giving up, I don't need to spend more time on trying mess on my side than time to code feature.
Don't blame you of course, It's not the first time, each time I'm forking and creating branch while upstream is moving it's a mess on my side with rebase, merge, resync, conflict. I never have any issue on my repo or repos with permission, always on forked ones but may be I'm not smart enough for that.

@hallard : I have rebased the PR, updated the CI and did some updates on the code for a better integration with what was existing. Would you be able to test ? If it works, I will merge and release.
Thanks for this addition!

@mathieucarbou I'm very sorry, I'm totally lost, between my fork, rebase from main of my branch or for upstream, and merge. Sometimes git drives me mad just for few lines of code. I propose you to merge, then I will delete my branch and sync my fork and test. If any issue, I will create a new branch to fix if ok to you?

No worry I have the local copy.
You could have done a got fetch and git reset --hard to set you local copy up to date

v1.1.0 is released with he PR: I will integrate it in YaSolR now.

Just tested, it works.
Just you let know that default compile program in platformio.ini is set to TrafficLightNeoPixel not sure it's intentional or not.

Thanks!
Ye sit does not matter: this is for the devs ;-) the ci will force and execute all examples.

Oh! I added a Todo also in the kanban board of YaSolR (project tab) to add support for that once I will have the time.
This is low priority for now though.


How to get animation offsets from characters

Describe your problem here.
Boyfriend has multiple singing animations: left, down, up, right...
All of these animation have a X-offset and a Y-offset, that's in the .json of boyfriend.
Is there a way to specifically get the offset of an animation?
Something like "getProperty('boyfriend.animation.curAnim.offset.x)" or "boyfriend.leftSING.offset.x" or somthing like that to get the offset seen in the character editor?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

function onCreatePost()
    debugPrint(getProperty("boyfriend.animOffsets.singLEFT", true)[1]) -- X
    debugPrint(getProperty("boyfriend.animOffsets.singLEFT", true)[2]) -- Y
end

Oh neat! Thanks!
Is there also a way to get the name of the animation in the xml as well?


ADDED compatibility with focus point module

Fixed with https://github.com/silverstripe-australia/silverstripe-versionedfiles/commit/1fbe79df179451c2b5d15b455805e86e388b57f6


Method overloading in interfaces using constraints not working with inheritance

TypeScript Version:  2.3.3
Apparently, I'm trying to overload same interface method in child interface, and expect to get the same overloading as in the range of the same interface. But it seems like all overloads from parent interface are hidden by child overload.
Also, I've tried to reproduce this in TS Playground so I believe this is not version dependent.
Code
type AConstraint = { fieldA: number };
type BConstraint = { fieldB: number };

interface A {
    method<T>(map: (a: number) => T): T;
    method<T extends AConstraint>(map: (a: number) => T): T;
}

interface B extends A {
    method<T extends BConstraint>(map: (a: number) => T): T;
}

(a: A, b: B) => {
    a.method((p) => { return 3; });
    a.method((p) => { return { fieldA: 3 } });

    b.method((p) => { return 3; }); // Error 1
    b.method((p) => { return { fieldA: 3 } }); // Error 2
    b.method((p) => { return { fieldB: 3 } });
};

Expected behavior:
Compiler resolves overloads from parent interface
Actual behavior:
First two b.method calls throws errors

(1) Argument of type (p: number) => number is not assignable to parameter of type (a: number) => BConstraint. Type number is not assignable to type BConstraint
(2) Argument of type (p: number) => {fieldA: number;} is not assignable to parameter of type (a: number) => BConstraint. Type {fieldA: number;} is not assignable to type BConstraint

Are you missing an extends A clause in this example?
See #10229 and other similar issues. If you redeclare a method in a derived type, the parent signatures aren't copied down automatically.

Yes, I forgot to add extents in the example but the result is the same.
It's pity that there is no fast solution as far as I understand.

Is this the same issue as what I'm experiencing?
interface IBaseService {
  doSomething(action: 'start', callback: () => void);
  doSomething(action: 'end', callback: () => void);
}

// Error for this interface due to type mismatching of 'start' and 'end'
interface ISpecificService extends IBaseService {
  doSomething(action: 'start', callback: (time: number) => void);
}

See in Playground
Using a new argument type results in pretty much the same error, also if I provide a wider option with the action argument as string, it still errors out.
interface IBaseService {
  doSomething(action: 'start', callback: () => void);
  doSomething(action: 'end', callback: () => void);
  // doSomething(action: Function, callback: Function);
}

interface ISpecificService extends IBaseService {
  doSomething(action: 'something-else', callback: (time: number) => void);
}

See in Playground
Is there no way to achieve what I'm trying to do?
In my case, I have an EventEmitter with a method on(event: string, callback: Function): Function; and I want to implement this emitter in a certain class that has specific events, for which I want to specify type hints for the callback format instead of just Function.


Tstore

@Tushar-Naik @santanusinha
Changes for latest dropwizard and latest ranger

@santanusinha made the changes - migrated ranger to 0.4 (which was merged).


[C+ Checklist Needs Completion] [$250] Distance rates - Order of distance rates is not preserved after clearing cache and restarting

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.78-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to workspace settings > Distance rates
Add a few distance rates
Note that order of distance rates is A, B, C, D, E etc
Go to Troubleshoot > Clear cache and restart
Select Reset and refresh
Return to Distance rates page

Expected Result:
The order of distance rates will remain as A, B, C, D, E etc
Actual Result:
The order of distance rates is no longer A, B, C, D, E etc
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/72e9100f-8155-411a-9312-a5ca343c0a09
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0149693392cec4578c
Upwork Job ID:<PHONE_NUMBER>945052160
Last Price Increase: 2024-06-01
Automatic offers:

Krishna2323 | Contributor | 102631316

Issue OwnerCurrent Issue Owner: @greg-schroeder

@greg-schroeder  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Proposals under review

Proposal
Please re-state the problem that we are trying to solve in this issue.
Order of Distance Rates is not preserved after clearing of cache.
What is the root cause of that problem?
There is no order defined for displaying distance rates in the list view. As the code uses Map object, insertion order is defaulted as the order to display.
Creation of map:
https://github.com/kpadmanabhan/App/blob/0cb2dc052f01b6a024ac872ad8bf628159fe7873/src/pages/workspace/distanceRates/PolicyDistanceRatesPage.tsx#L101C1-L116C7
Display of distance rates:
https://github.com/kpadmanabhan/App/blob/0cb2dc052f01b6a024ac872ad8bf628159fe7873/src/pages/workspace/distanceRates/PolicyDistanceRatesPage.tsx#L300C1-L313C23
When cache is cleared, the Map is destroyed as well and when we navigate to the page, the Map is recreated from memoised version from Onyx or from the OpenPolicyDistanceRatesPage API response from backend. Map maintains insertion order of the keys and the same is the order in which distance rates are displayed in the UI.
What changes do you think we should make in order to solve the problem?
Sort order of displaying distance rates in the UI should be independent of the datastructure used underneath. Propose to define one of the following sort orders.

order in which distance rates are created, but persist this order into an Array based on index in the background (I do not personally recommend this, as this is not user friendly. It does not make sense especially when the list is huge and it will be difficult for users to find any particular distance rate when they revisit the page)
increasing or decreasing order of the distance rate, independent of the selection
sorted based on distance rate, but selected to be displayed on top sorted and non-selected at the bottom in sorted order
provide a sort option to the user based on criretia like Low - High, High - Low, Most recently added or something similar.

The above order must be preserved using a property in the Policy object and saved in the backend as well. When we make the OpenPolicyDistanceRatesPage API call, the same must be returned by the backend.
Hnece this bug needs implementation in both backend and frontend. Propose to split this into backend and frontend separately or make it as internal.
What alternative solutions did you explore? (Optional)
N.A.

Thanks for the proposals everyone. Since this is a straightforward issue, with the solution being almost the same across all the proposals, I'd suggest we go with the first proposal that mentions sorting.
As such, @Krishna2323's proposal looks good to me.
🎀 👀 🎀 C+ reviewed

@ShridharGoel It's still the same idea. I think we can handle it in the PR if needed.

If we evaluate based on this, we might end up with an arbitrary sorting that's not ideal for the user. It satisfies the "mentions sorting" part of the criteria, but far from the best we can do.

That's correct, but we really did not specify "What the sorting criteria should be" in the bug statement. As such, you're just throwing a guess on the best sorting criteria. In my opinion, its a little unfair to the first proposer as the new proposals are suggesting the same thing with different sort criteria.

That's correct, but we really did not specify "What the sorting criteria should be" in the bug statement. As such, you're just throwing a guess on the best sorting criteria. In my opinion, its a little unfair to the first proposer as the new proposals are suggesting the same thing with different sort criteria.

@allroundexperts : I agree with your statement regarding the fairness part to the first proposer. But if you look at this bug, it indeed is about sorting. However, what also needs to be looked into is the thought that others have put behind while proposing the same sorting solution.
@dominictb : what do you think about the suggestions in this proposal?
As you rightly pointed out regarding "best we can do", the value that we are giving to our customers especially in terms of usability and feature richness.

@allroundexperts : sorting based on created is not adding value.
I dont mind which proposal you go with finally. The solution should give graceful user experience and add value. As my proposal and @dominictb are suggesting, this needs backend implementation as well. Please consider that.

Ah, thank you for the bump. I was waiting for the discussions to end, but I agree with the original decision.

Nice, looks like we're on our way here! Let us know if you need any help @Krishna2323

Not overdue. We're still finalising the best sorting criteria. @tgolen Can you please check @Krishna2323's comment above?

OK, thanks. Let's go with your recommendation then for now.

@allroundexperts - $250 - You can make a manual request via ND
@Krishna2323 - $250 - Paid through Upwork

C+ checklist is all we need for tomorrow and we can close!

Checklist

We never implemented sorting of distance rates on the frontend. As such, this issue exists since the beginning when this feature was added ie here.
N/A
N/A
A regression test would be helpful here. The steps given in the OP look good enough to me.

Okay filing using OP steps

$250 approved for @allroundexperts


Update Spark to 3.2.3

CI is failing because Spark has removed 3.2.2 from the apache download mirrors. This PR updates Spark support to 3.2.3.

We've got to bump all the licenses like https://github.com/elastic/elasticsearch-hadoop/pull/1923/files right?

💚 All backports created successfully

Status
Branch
Result

✅
8.6

Questions ?
Please refer to the Backport tool documentation


Added a few more gems

A few more for the list and I'm done, I swear.

Ha, no worries.

Release the hounds!  (sorry to hound you, but I would like a release)

now released as part of version 1.1.3.


Automated cherry pick of #848: fix: baremetal enable serial not working

Cherry pick of #848 on release/2.9.0.
#848: fix: baremetal enable serial not working

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.9.0@d1b1df9). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff               @@
##             release/2.9.0    #849   +/-   ##
===============================================
  Coverage                 ?   7.56%
===============================================
  Files                    ?     342
  Lines                    ?   53481
  Branches                 ?       0
===============================================
  Hits                     ?    4045
  Misses                   ?   49057
  Partials                 ?     379

Flag
Coverage Δ

#aFlag
7.56% <0%> (?)

Impacted Files
Coverage Δ

pkg/hostman/guestfs/fsdriver/linux.go
0% <0%> (ø)

pkg/hostman/guestfs/sshpart/sshpart.go
0% <0%> (ø)

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 d1b1df9...f8ade84. Read the comment docs.

/area baremetal

/lgtm
/approve


from_glib_ptr_borrow : Consider taking a *mut pointer to object

Since we usually use *mut for GObjects, it would make sense to take a *const *mut or a &*mut. The latter one is what I usually have in my code so far.

Should ideally use a reference as that makes the lifetime tracking work better and makes it harder to use it wrong. Not sure why I didn't do that. I'll look into that again :)

Can you try if https://github.com/gtk-rs/gtk-rs-core/pull/1375 makes usage easy for you?


Wrong lables in Figure 8.1

Please check the position of the left saddle point lable in Figure 8.1.  The top saddle point lable should be  maxima.

fixed


Fix RST errors in Custom Serializers

Found some small typos in the RST for describing custom serializers. This small PR fixes it. Nothing more.
Thanks!

Coverage remained the same at 97.66% when pulling e3ae526cba37a4ea2d8a48b05aaeff062847c644 on parkerhancock:master into 79f2dd9707666ca8fb9e63e331d71c0a376c029e on reclosedev:master.

Thanks!


Add instruction to place ca cert in webhook configurations

This PR adds instruction to place ca cert in webhook configurations. This is a necessary step for the installing topology operator with generated certs. This documentation problem is reported by @evankanderson in issue: https://github.com/rabbitmq/messaging-topology-operator/issues/185

Merged manually.


TEST PR PLEASE IGNORE

This PR is only so that I can get full visibility into the build logs.

/test e2e-rpm-install


Question

Is there a way to make it so you can see inside the CEO offices from the outside like you can in normal GTA? Thanks.

I don't think I follow, you can see the inside of an office from the exterior? I don't remember this in GTA:O
I know GTA:O disable some 3D models when you are inside an interior to clear the view when looking at windows (ie: it generally disable the exterior of the building you are in, otherwise you would see textures from the inside when looking outside)
Do you have an example with screenshots?

What I am meaning, In normal GTA Online if someone was to be in a helicopter and fly up beside the building they could see inside through the windows this is not the case for me right now as if I no clip out of the building and look in from the outside it just shows the building but you cant see in. Does that make anymore sense? Sorry difficult to explain.

To be honest I really didn't know R* added that feature and it surprises me since these interiors are the same for everyone.
They are instantiated so multiple players can be at the same physical place without seeing each others.
If no one is talking about this on the internet, I would recommend looking for decompiled scripts that mention offices (look for the IPL name or maybe parts of the 3D model name).
It still triggers me a little bit, could you please provide a screenshot? I'm really curious, I've never suspected this behavior with the interiors seen from exterior.


Provide a function that calculates the difference in business days between two dates

Would be cool to have a function that calculates the number of business days between two dates.

totally, thanks for your request and for checking this out!

🤔 what might be a good name for this function...maybe count_days_until? So you'd write
df.with_columns(
    plb.col('start_date').bdt.count_days_until('end_dates')

Uh, naming discussions are always so dangerous... I would go with something simple like diff (used by many date related frameworks) and not fix the time unit (days) and the direction (until) in the method name.
pl.col("date").dt.offset_by("1mo_saturating")).bdt.diff(pl.col("date")).alias("businessDaysTillMonthEnd")

yeah diff would be good I just worry that it's easily confused with the already existing Expr.diff

You're totally right. The diff way to do it would even make perfect sense and could be helpful in the bdt namespace as well. It's just a different thing :-) Maybe sub would be better than? Because this would really be a special kind of subtraction (https://pola-rs.github.io/polars/py-polars/html/reference/expressions/api/polars.Expr.sub.html) and you could still reserve diff for subtracting with the preceding element.

ah, perfect! I wasn't actually aware of Expr.sub - cool, let's go with that then, thanks! going to try to get an initial solution out today

Most civilized naming discussion ever 🤣

I've ended up adding plb.workday_count too, that should be more familiar for anyone coming from numpy, e.g.
In [9]: df.with_columns(
   ...:     result = plb.workday_count(
   ...:         'Day1',
   ...:         'Day2',
   ...:         weekend=['Thu', 'Fri', 'Sat', 'Sun'],
   ...:     )
   ...: )
Out[9]:
shape: (6, 5)
┌────────┬──────────┬────────────┬────────────┬────────┐
│ Market ┆ Service  ┆ Day1       ┆ Day2       ┆ result │
│ ---    ┆ ---      ┆ ---        ┆ ---        ┆ ---    │
│ str    ┆ str      ┆ date       ┆ date       ┆ i32    │
╞════════╪══════════╪════════════╪════════════╪════════╡
│ AT     ┆ Standard ┆ 2022-01-02 ┆ 2022-01-05 ┆ 2      │
│ DE     ┆ Express  ┆ 2022-01-03 ┆ 2022-01-09 ┆ 3      │
│ AT     ┆ Standard ┆ 2022-01-04 ┆ 2022-01-09 ┆ 2      │
│ CZ     ┆ Standard ┆ 2022-01-05 ┆ 2022-01-09 ┆ 1      │
│ GB     ┆ Standard ┆ 2022-01-06 ┆ 2022-01-09 ┆ 0      │
│ CZ     ┆ Standard ┆ 2022-01-07 ┆ 2022-01-15 ┆ 3      │
└────────┴──────────┴────────────┴────────────┴────────┘


synthesize.sh script's argument -s does not work anymore

./synthesize.sh --num_samples 200 Test.java
[2017-10-10 13:42:21,611] ERROR in app: Exception on / [POST]
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.5/dist-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/vm17/Work/bayou/tool_files/build_scripts/out/python/bayou/server/ast_server.py", line 170, in <lambda>
    http_server.add_url_rule("/", "index", lambda: _handle_http_request(bp), methods=['POST'])
  File "/home/vm17/Work/bayou/tool_files/build_scripts/out/python/bayou/server/ast_server.py", line 37, in _handle_http_request
    asts = _handle_generate_asts_request(request_dict, predictor)
  File "/home/vm17/Work/bayou/tool_files/build_scripts/out/python/bayou/server/ast_server.py", line 53, in _handle_generate_asts_request
    asts = _generate_asts(evidence_json_str, predictor, num_samples=sample_count, max_ast_count=max_ast_count)
  File "/home/vm17/Work/bayou/tool_files/build_scripts/out/python/bayou/server/ast_server.py", line 64, in _generate_asts
    if num_samples < 1:
TypeError: unorderable types: str() < int()

Can reproduce. Acceptance test created.

accidentally did these changes on issue-91 branch, but awaiting merge

Merged PR #94.


Alternative Paste | Type clipboard

Description of the new feature / enhancement
Having the ability to use an "alternative" paste method could be very useful.
By this I mean having CTRL+V work as normal, then having the option to set a bind for an alternative paste (such as ctrl+alt+v) (or have it replaced the default paste method) that would simulate typing the clipboard text on your keyboard.
Scenario when this would be used?
if I am using a remote console (IPMI, IDRAC, ETC.), the clipboard is not passed through the console. But if the paste data was instead entered as keyboard presses, it would go through just fine.
Supporting information
This funstionality is possible in Auto Hot Key with "SendRaw, %Clipboard%"

/dup #18115


timers: add duration and cooldown timers for arceuus spells

Arceuus spells have two different timers: the duration of the effect, and the cooldown before they can be cast again. There are 5 types of spells (all 9 thrall spells share the same timers):

Shadow Veil
Ward of Arceuus
Death Charge
Thrall spells
Corruption (cooldown)

How Ward of Arceuus looks with both options enabled:

SpriteID also had a lot of bit-rot with the old arceuus spells

Can you make tests for these?

I went through and collected these:
<col=6800bf>Your thieving abilities have been enhanced.</col>
<col=6800bf>Your Shadow Veil has faded away.</col>

<col=6800bf>Upon the death of your next foe, some of your special attack energy will be restored.</col>
<col=6800bf>Some of your special attack energy has been restored.</col>

<col=ef0083>You resurrect a greater zombified thrall.</col>
<col=ef0083>Your greater zombified thrall returns to the grave.</col>

<col=0000b2>Your defence against Arceuus magic has been strengthened.</col>
<col=0000b2>Your Ward of Arceuus has expired.</col>

Tests added for checking the duration is set correctly, that Death Charge is removed when a kill is gotten, and that the Corruption Cooldown is correctly triggered.


update of readme files based on enus source

Closing this PR as we are archiving this sample. Please see https://github.com/microsoftgraph/msgraph-training-android for an Android sample.


Performance tests with query() instead of list()

Is your feature request related to a problem? Please describe.
Especially for large inboxes performance and processing time of the stats generation feels slow.
Describe the solution you'd like
According to this thread, there might be a simple fix to reduce processing time, by replacing
messenger.messages.list(folder)

with
messenger.messages.query()

Describe alternatives you've considered
This is worth some testing, since ThirdStats doesn't require to do the processing folder-wise (except when filtering messages for a special folder).
Additional context
None.

I just did some time measurements on retrieving all messages of an account holding 50k messages in 48 folders with different approaches:

Using messenger.messages.list(folder) on each folder gave a total average processing time of 15085.8ms
Using messenger.messages.query() gave an average processing time of 16805.1ms (note: a few seconds TB was unrespondend during processing)
Using messenger.messages.query({folder: folder}) on each folder gave a total average processing time of 15236.3ms

So unfortunately I cannot confirm the performance boost by using an empty query() on my end. It's surprising that approach 3 is actually a bit faster than 2. Still using list() is the fastest for me. I did 10 measurements for each approach
using the accounts refresh button.


fix: add more tests for curriclum testing

PR for https://github.com/freeCodeCamp/freeCodeCamp/pull/38456#issuecomment-604825040
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

This PR adds two tests to check that a challenge title and id are found in their meta.json files.
In making these tests, I found one challenge where a title didn't match.
It also adds some options to test a specific block or superblock of the curriculum...
npm run test:curriculum --block='Basic HTML and HTML5'
npm run test:curriculum --superblock=responsive-web-design
These commands will do that.
The lines I actually changed in test-challenges.js are 35, 103-131, & 205-238. The rest of the diff wasn't changed other than some indentation.
The code feels a little sloppy, but it looks like it's working. Any suggestions are welcome.

Fixed my awful naming with @moT01's much better one.


Clean up templates in config/templates and decide what to do with config/common/packages.yaml

Is your feature request related to a problem? Please describe.
There are too many, somewhat redundant templates in config/templates/. It's hard to keep them all up to date. Some of them pin versions of packages, which overlaps with the general config in config/common/packages.yaml.
Describe the solution you'd like
Clean up the templates in config/templates. Find a better solution for specifying package versions and variants - potentially get rid of config/common/packages.yaml and maintain only a handful of templates that specify their package versions and variants directly (i.e. fixed versions for operational implementations, latest develop).
Additional context
n/a

We're good on configs/common/packages.yaml - just need to clean up the templates in config/templates/ to close this issue.

This is an old issue. We should defer that to after the release. Some of the templates haven't been updated in years, and they will no longer build (because versions aren't available anymore etc). We shouldn't keep old templates around imo - if they aren't updated and don't work, they can be removed, and brought back later if really needed? An operational implementation of something with a specific template will also point to a specific tag of spack-stack and not to develop.


[FR] - Add CLI option to show the cardano-submit-api version number

External
Describe the feature you'd like
Add a --version option to cardano-submit-api that will show the version and git revision analog to cardano-node and cardano-cli.

Including a version option for the cardano-submit-api would be useful to identify its version, similar to the feature available in cardano-node:
$ cardano-node --version
cardano-node 9.1.0 - linux-x86_64 - ghc-8.10
git rev bfa1f18469de246369fa60112b98e87440501b93

While it's not a blocker, the cardano-node team should consider implementing this basic feature.


sys.maxsize() should not be used for seed generation

The problem is caused by line 502 in labs.py.
Replacing line 502 with seed = seed if seed else np.random.randint(np.iinfo(np.int32).max) fixes the issue.
@alex124585, would you mind submitting a PR with the above change?
Thank you!
See error below.
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[2], line 4
      2 p = 1
      3 gamma, beta = parameter_utils.get_best_known_parameters_for_LABS_wrt_overlap_for_p(N, p)
----> 4 terms = get_gate_optimized_terms_greedy(N) # indices of Pauli Zs

File ~\Dropbox\ANL\qaoaPortfolioOptimization\test\QOKit\qokit\labs.py:502, in get_gate_optimized_terms_greedy(N, number_of_gate_zones, seed)
    498         two_body.append((i, i + 2 * k))
    500 circuit = []
--> 502 seed = seed if seed else np.random.randint(sys.maxsize)
    503 print(f"seed: {seed}")
    504 rng = np.random.default_rng(seed)

File mtrand.pyx:746, in numpy.random.mtrand.RandomState.randint()

File _bounded_integers.pyx:1336, in numpy.random._bounded_integers._rand_int32()

ValueError: high is out of bounds for int32

@alex124585 would you mind implementing the suggested change above? Thank you!

k

Resolved by #15


Push the timeout-tracking logic inside Kurtosis API

This a) reduces complexity, because the client lib no longer needs to deal with multithreading and b) makes sense, because the Kurt API server has all the metadata information about the test anyways. This would kill multiple TODOs in the code

Completed by Lord @galenmarchetti


It does not work for me. Just nothing happens.

I only have a tabs and simulated browser window. Start tests button not working.

Yes

Same here, the docs are in the blog post and once I changed the line it worked fine:
In DuskTestCase.php Find and replace this line:
use Laravel\Dusk\TestCase as BaseTestCase;
with:
use BeyondCode\DuskDashboard\Testing\TestCase as BaseTestCase;

I did it before running the tests. It just does not work.

I can push the repo if You want.


Remove HappyBase code

Follow up from https://github.com/GoogleCloudPlatform/gcloud-python/issues/1762.
A new repo will be created called google-cloud-python-happybase and available under pypi at https://pypi.python.org/pypi/google-cloud-happybase.

Closed in favor of #2109.


jstree working unexpectedly

I have a long json that populate the tree using this code
jQuery('#data').jstree({
           'core': {
               "check_callback": true,
               "themes": { "stripes": true },

               'data': d

           }
       })

first load everything seem normal, but when trying to open child's node the tree working unexpectedly.
after opening one of node the tree disappear.on other node the order of the tree change incorrectly, place root inside other child nodes.
any idea what causing this bug?

This means you have duplicate IDs in the data you pass to jstree. Ensure that the IDs are unique (or omit them).


Access denied

When I run npm run test:client with my username and password, I'm getting the following error:
{ NikeLoginError: Can't login
    at /Users/ken/work/nikeplus-client/src/nike.js:89:15
    at process._tickCallback (internal/process/next_tick.js:178:7)
  name: 'NikeLoginError',
  data:
   { code: 400,
     uri: 'https://unite.nike.com/login?appVersion=358&experienceVersion=308&uxid=com.nike.commerce.nikedotcom.web&locale=en_US&backendEnvironment=identity&browser=Google%20Inc.&os=undefined&mobile=false&native=false&visit=1&visitor=3db2a130-513a-11e8-bbcb-9b3ebb5c260d',
     body: '\n<TITLE>Access Denied</TITLE>\n\n
Access Denied
\n \nYou don\'t have permission to access "http://unite.nike.com/login?" on this server.
\nReference #18.27010f17.1525617122.328da828\n\n\n' } }

Any idea why this might be happening?

I can't seem to login to nike's official my activity page due to https://awr.svs.nike.com/accesstoken throwing a 503 service unavailable.  I'm starting to think there's an issue on Nike's server side.
However, what I don't quite understand is how https://nike.bullrox.net is able to login and fetch my data.  From what I can tell, it's the only site that can successfully pull data from my nike+ run activity.
@pdupavillon If you can share details on how your site works, that would be tremendously helpful.  Apologies if you aren't the maintainer (I figured you are, given that the site is listed in this repo's description)


No license file

This is a really amazing method, but I am hesitant to use it without a provided license. Could a license file be added to this repository?

Better late than never, an MIT license has been added. Sorry about the slow response.


VSCode will stop working when laptop goes to sleep

Does this issue occur when all extensions are disabled?: Not Sure

VS Code Version: latest
OS Version: Win11 22H2 22621.521

Steps to Reproduce:

Start VSCode
Wait until laptop goes to sleep

Not every attemp reappears. Before submitting the issue, i've tried reinstall the OS (raw OS is also Win11) , then the problem still appears, but the frequency is much lower.
When the BUG occurs, it seems that I can't do actions like inputting, moving the cursor and even closing the window. If i minimize the window and recovery, it will responding to the last operation but still not react probably. Only when i restart the window it will behave correctly.

Merging to https://github.com/microsoft/vscode/issues/149247


Virtual calls in small-size Vector arithmetic

I noticed that a number of Vector methods were made virtual in da941d2be67f.
I think this has serious implications for the performance of small-size Vector methods like: https://github.com/mfem/mfem/blob/5b894c6541c61b82e09641b08275d1a3ba6fdf89/linalg/vector.cpp#L149-L156
and https://github.com/mfem/mfem/blob/5b894c6541c61b82e09641b08275d1a3ba6fdf89/linalg/vector.cpp#L207-L217

If I remember correctly this was needed to allow full compatibility between MFEM and PETSc methods;  PetscParVectors can be transparently used (as Vectors) inside MFEM methods as well as MFEM vector data can be wrapped into PETSc Vec. Have you timed this performance impact? I guess it is for CPU only builds, right? Would it make sense to write a small class for these small vectors?

The impact of this was benchmarked in https://github.com/mfem/mfem/pull/2279 (thanks @camierjs!) and there seem to be about a 2x penalty for these virtual calls.
@stefanozampini, can you think of an alternative implementation that reverts this change?

ping: @stefanozampini

Well, if we want full compatibility between MFEM's and PETSc GPU, we cannot avoid these virtual calls.
I see the performance regression from the unit tests, but what about in an actual app? What is your concern about these regressions? FEM assembly?

Yes, the concern is the cost of operations on small host Vectors which are used in many places in the library, and were specifically optimized to use these methods which were assumed to be inline.
One option is to re-introduce the inline methods with new names and use those where we need the inline versions.
I guess you are trying to keep PetscParVectors parts corresponding to mfem::Vector and petsc::Vec synchronized, is that why you made these methods virtual? I suspect that this synchronization has its limitations and is not 100% reliable, e.g. if someone uses the Vector's Memory directly to copy/move the Vector data, or on the PETSc side, if one modifies directly the petsc::Vec object.

I spent quite some time to have full compatibility between using mfem::Vector accessors and petsc::Vec accessors.
If one uses the proper accessors, the memories should be in synch. See e.g https://github.com/mfem/mfem/blob/master/linalg/petsc.hpp#L277 which is used e.g.  https://github.com/mfem/mfem/blob/master/linalg/petsc.cpp#L1920. It also works for time-dependent problems https://github.com/mfem/mfem/blob/master/examples/petsc/makefile#L90, where vector data is modified by PETSc (e.g. in RK) and by the mfem users (residual evaluation) https://github.com/mfem/mfem/blob/master/examples/petsc/ex9p.cpp#L569

One question: why did you need to make the UseDevice methods virtual? How does PetscParVector use that flag?
The intent for this flag (false by default) in mfem::Vector was to treat the Vector as a host Vector regardless of the mfem::Device configuration -- this way any old non-GPU code that uses a Vector will still work purely on host, and this includes all small host Vectors. To me it seems natural to always have PetscParVector tagged for use on device. For example, we do that for all (Par)GridFunctions. What do you think?

if you call UseDevice(false) on a PETSc vector, you will force petsc::Vec to use HOST methods.

Hmm, this creates an issue: if we want operations defined in the Vector class to work properly with small host Vector objects (using just inline methods) and with PetscParVector objects which have the UseDevice flag set to false. Basically, the question is how to implement methods like
Vector &Vector::operator*=(double c)
{
   const bool use_dev = UseDevice();
   const int N = size;
   auto y = ReadWrite(use_dev);
   MFEM_FORALL_SWITCH(use_dev, i, N, y[i] *= c;);
   return *this;
}

so that it only calls inline methods for small host Vectors?
I was thinking something like this:
Vector &Vector::operator*=(double c)
{
   const bool use_dev = UseDeviceInline(); // i.e. data.UseDevice()
   const int N = size;
   auto y = use_dev ? ReadWrite() : HostReadWriteInline();
   // HostReadWriteInline(); is the same as mfem::ReadWrite(data, size, false);
   MFEM_FORALL_SWITCH(use_dev, i, N, y[i] *= c;);
   return *this;
}

However, this will not work as expected for PetscParVectors which have UseDevice set to false.

I don't understand. If usedevice is false for a PetscParVector, you will always use the host memory and you will modify host memory in this function.
Why not having
Vector &Vector::operator*=(double c)
{
   const bool use_dev = GetUseDevice(); // no virtual function, access mfem::Vector class member which tells us the up to date location (always false for Small vectors)
   const int N = size;
   auto y = use_dev ? ReadWrite() : begin(); // safe to use begin() here
   // HostReadWriteInline(); is the same as mfem::ReadWrite(data, size, false);
   MFEM_FORALL_SWITCH(use_dev, i, N, y[i] *= c;);
   return *this;
}

operations with two vectors are trickier since now the Vector class always prefers device updated vectors

First, the two methods PetscParVector::UseDevice do not seem to sync with mfem::Vector::data, so data.UseDevice() may return something different from the virtual UseDevice().
Second, calling begin() will only work if data is valid only on host, which may not be always true. That is why I use mfem::ReadWrite(data, size, false) instead which will move data accordingly, if needed. The problem with that call and PetscParVector is that it may cause the mfem::PetscParVector::data (inherited from mfem::Vector) and mfem::PetscParVector::x to go out of sync.

In principle, a Vector is always aware of the most up-to-date memory location (or both). And GetData() on petsc vectors will return the mfem memory object.
The data member of Vector is an alias to the PetscMemory in PetscParVector. https://github.com/mfem/mfem/blob/master/linalg/petsc.cpp#L264. And the PetscMemory is in sync with whatever is the status of petsc::Vec
A small code example that I can run and that shows that what you are proposing will not work would be great.

To see the first issue I mentioned above, apply this patch to the current MFEM master branch:
diff --git a/examples/petsc/ex1p.cpp b/examples/petsc/ex1p.cpp
index 000c5bf48..e065a5704 100644
--- a/examples/petsc/ex1p.cpp
+++ b/examples/petsc/ex1p.cpp
@@ -146,6 +146,24 @@ int main(int argc, char *argv[])
    // 3b. We initialize PETSc
    MFEMInitializePetsc(NULL,NULL,petscrc_file,NULL);

+   if (1)
+   {
+      Vector x(1);
+      PetscParVector px(MPI_COMM_WORLD, x);
+      cout  << "[" << myid << "]: px.UseDevice() = "
+            << px.UseDevice() << endl;
+      cout  << "[" << myid << "]: px.GetMemory().UseDevice() = "
+            << px.GetMemory().UseDevice() << endl;
+      px = 1.0;
+      Vector &ref_px = px;
+      ref_px += 1.0;
+      cout << "[" << myid << "]: px[0] = " << px.HostRead()[0] << endl;
+   }
+
+   MFEMFinalizePetsc();
+   MPI_Finalize();
+   return 0;
+
    // 4. Read the (serial) mesh from the given mesh file on all processors.  We
    //    can handle triangular, quadrilateral, tetrahedral, hexahedral, surface
    //    and volume meshes with the same code.
diff --git a/linalg/vector.cpp b/linalg/vector.cpp
index 16f5c11ba..e71df961f 100644
--- a/linalg/vector.cpp
+++ b/linalg/vector.cpp
@@ -221,9 +221,9 @@ Vector &Vector::operator-=(const Vector &v)

 Vector &Vector::operator+=(double c)
 {
-   const bool use_dev = UseDevice();
+   const bool use_dev = data.UseDevice();
    const int N = size;
-   auto y = ReadWrite(use_dev);
+   auto y = use_dev ? ReadWrite() : mfem::ReadWrite(data, N, false);
    MFEM_FORALL_SWITCH(use_dev, i, N, y[i] += c;);
    return *this;
 }

then build mfem with cuda and with petsc (also built with cuda) and run
cd examples/petsc
make ex1p
mpirun -np 2 ./ex1p -d cuda

The contents of px will be 1 (or maybe undefined) and not 2, as it should be.
This should be easy to fix -- we just need to make sure px.GetMemory().UseDevice() is sync'd with px.UseDevice().
The second issue I mentioned above, can come up only if a PetscParVector can be in the following state: (1) px.UseDevice() is false, and (2) the PetscParVector::data is not valid on host. Is this possible for a PetscParVector?

@stefanozampini It is not correct to say that the Vector is always aware of the most up-to-date memory, by design there is several methods that will make a Vector unaware of the most up-to-date memory, a Vector is also unaware that he is unaware at this point, in such cases this is the responsibility of the user/dev to manually use SyncMemory or SyncAliasMemory to make the code correct.
I think this conversation shows that we're trying to make the Vector class do too many things, do it all classes rarely lead to good code in the long run. Having a small vector class (stack allocated, not heap allocated) would probably make more sense than adding yet another bandaid on the Vector class. It should be relatively straightforward to benchmark, and replace Vector with StackVector<N> where needed, while probably making MFEM also faster in these critical parts. The other advantage of stack allocated vectors is that they would be copyable from host to device without any pointer validity considerations.
I also want to reiterate that I really think that it was a mistake to make Vector not go on device by default, it would have been relatively easy to add a check during development to know if a small vector is being moved to device, and should be replaced with a StackVector<N>. This would have resulted in a much more user/dev intuitive API, it is a recurring problem to have user/dev having issues with unnecessary memcpy between host and device, and these are hard to track down.

@v-dobrev The vector constructor does not copy x by default. I was not copying the device flag either. Here you have a fix https://github.com/mfem/mfem/commit/ab73241e7ed4f10565a46c6c2904ce1b46d05bc7.
There is still an open issue: what to do when PETSc and MFEM devices do not talk to each other? What are the devices you support? Do you have a configuration script that I can test with?

@v-dobrev I mean, if PETSc supports KOKKOS (or HIP or whatever in the future) and MFEM supports UMPIRE  (just as an example, since I see it in the list of Devices). What is the intended mean of UseDevice in this case? Do you want the PETSc classes to still take advantage of running via KOKKOS and let them talk with MFEM through the host? As it is now, the code within _USE_DEVICE guards in petsc.cpp deal with the case PETSc with CUDA && MFEM with CUDA. But I was wondering what should be the returned value of bool UseDevice() if called on a PetscParVector if the two do not match.

There should be no conflict with regard to the UseDevice() flag -- for vector methods implemented in MFEM it means for MFEM to use its configured device/backend, and similarly for methods calling PETSc vector functions -- they should use the configured PETSc device/backend ("device" here includes the case where no actual device like GPU is being used -- in that case we just say that we are using the cpu device or other devices like omp, raja-cpu, etc).

Another question, @stefanozampini: why this change: b8281fa0? This changes the intended behavior for the method.


Jump between date search matches

It would be really handy to be able to jump from one match to the next or previous, analogous to the usual text search or navigating side-by-side diffs.

A good idea, but I am making the decision that we will no longer improve (but will continue to support) source dates. I think adding this feature will likely take away from other big issues we must tackle. Thanks!

Can we at least close as "not planned" instead of "completed"?


old RCs should be removed from dist.apache.org

https://dist.apache.org/repos/dist/dev/incubator/resilientdb/
Still contains an RC2 that was not released.
@cjcchen can you remove this?

https://dist.apache.org/repos/dist/dev/incubator/resilientdb/
removed

@cjcchen Can you remove the KEYS file from the dev folder too? You should only have the KEYS file in the releases folder.

@pjfanning
Removed.
https://dist.apache.org/repos/dist/dev/incubator/resilientdb/
I also added a 1.10.1-rc0 to remove the binary from 1.10.0.


Keyboard manager switches off occasionaly

Microsoft PowerToys version
0.56.2
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
I LOVE PowerToys, it's fantastic. Thank you so much for all you work making it rock.
From time to time my mapping of Caps Lock to Ctrl-L will malfunction.  I think (but again can't reproduce) that it happens just after I login.
Sadly I can't reproduce it regularly - I'm reporting to try to be helpful, it's not a big problem at all.
✔ Expected Behavior
My keyboard re-mappings should be permanent.
❌ Actual Behavior
The Caps Lock key reverts from my mapping to a it's (semi) normal behavior. It will enable Caps Lock, but not disable it.
Other Software
No response

I'm not sure from the description which direction the mapping is done in.
Perhaps a /bugreport would help so we can see the mappings.

Hi again - the mapping is Caps Lock -> Ctrl Left.  I'll send the report bug too.

PowerToysReport_2022-03-29-12-32-23.zip

Maybe this is being caused by there being some Window that is privileged being on focus sometimes. For example, if you use Task Manager and PowerToys isn't running as admin, the mapping won't work there. (This makes sense, unprivileged apps shouldn't be able to switch key presses on privileged windows).
Could you please try to run PowerToys as an administrator and seeing if this still occurs for you?
/needinfo

Hrm, sorry I got this wrong in the report - but I am actually running as administrator.

Couldn't replicate the described behavior. It's possible that if it only happens after just logging in that it's just the case that the Keyboard Manager Engine is still booting up.

Does this make sense to you @aaronelliotross ? /needinfo

No it  happens kind of whenever. Don’t sweat it, if I can figure out a
correlation I’ll let you know.
Thanks for checking it out !
On Thu 31 Mar 2022 at 13:57, Jaime Bernardo @.***>
wrote:

Does this make sense to you @aaronelliotross
https://github.com/aaronelliotross ? /needinfo
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/17309#issuecomment-1084485022,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA4RTTTSW2NUMPQYHYQ6BDVCWHLRANCNFSM5R2GQCFA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Aaron

https://github.com/microsoft/PowerToys/issues/9300 this is already in several issues going back a ways

@jaimecbernardo I can reliably duplicate this bug, I posted in the other thread with a high level overview of how to trigger it.
https://github.com/microsoft/PowerToys/issues/9300#issuecomment-1046316577
Basically, remap the Caps Lock key to something else, then hold it while launching something that takes over the full screen. For me it happens reliably with Dota 2 and some other games.

Hi @mrudden , Keyboard Manager conflicts with some games. It shouldn't be used while playing games as these might also use the same low level APIs we're using: https://docs.microsoft.com/en-us/windows/powertoys/keyboard-manager#known-issues

/dup #16173


RFC: Transparently use verbatim paths on Windows

(rendered)

+1 for the general idea; but we have to be very careful how we normalize the paths. Windows does more than the transformations listed in the RFC.
For instance, Windows strips trailing dots and spaces (in each path component): C:\a. . is the same as \\?\C:\a, which is a different file than \\?\C:\a. ..
And then there's also special files like CON and PRN. CreateFileW(L"c:\\con", GENERIC_WRITE, ...) actually opens a file handle for stdout; but \\?\c:\\con is a file on disk. It's debatable whether we want the rust stdlib to be able to open these (probably not, as these are just a relict from the DOS days...).
Not sure if there are any additional caveats. Do 8.3 short names interact with file name normalization, or are they handled completely in the file system driver?

I would personally be quite wary of doing this sort of manipulation in the standard library. It should never be the case that someone wants to subvert this behavior, because the purpose of the standard library is to provide the safe interface here so applications don't have to bind it themselves.
Along those lines I am not confident that we can convert to verbatim paths and continue to satisfy 100% of use cases of these APIs. I'd be more comfortable if there were other sources cited here such as:

MSDN articles
Official Windows documentation about this sort of transformation
Implementations in other languages or libraries

I'm supportive of this. I also did this for my Ruby implementation. It's annoying not being able to deal with and those silly special cases long paths in a portable way.  Could you be a bit clearer?
For reference: https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx?f=255&MSPPError=-2147217396

I'd personally prefer if there was first a crate on crates.io that provided path normalization so it could be carefully tested.

@Zoxc
I'm basically worried about the case where an API is explicitly passed a non-verbatim path which we will then silently translate to a verbatim path under the hood. This may have the possibility of breaking the semantics of using the API if we either do the conversion incorrectly or the API has some behavior encoded as part of a non-verbatim path which is impossible to encode as a verbatim path.
A semi-example of this is how abstract Unix sockets on Linux need a leading 0 byte to indicate this. By encoding the argument carefully this is requested, but if we had an API which did some silent translation of the argument it may break this.

I currently don't have time to update this.


Add configuration (via a TOML file)

Yeah, all in all this is looking good, and definitely does what you set out to do very well. A lot of the stuff Callum and I have mentioned have been stylistic things, or naming things - that's just stuff you'll get used to more in time with Go.
Also, we'll sort out goimports, but it is critical that you use it. I'd also recommend having golint running all the time during development too. I can show you how I usually work if that's any help.

Almost there! I do need to get Travis up before we merge so we can do the style checks automatically still though. I'll do that today nonetheless, probably over lunch.

Now then… how can we get Travis to build this?

Alright, Travis is running and building - seems like everything is in order so far :)


fix: correct braid(m, n) order

Rewrote the braid function to correct the history output

[ ]  Check history output
[ ]  Check that the drawn circuit is correct
[ ]  Check the braid_sequence

Resolves #29

The (n, m) structure needs to be preserved.


Fix import path for Remix 1.6.2+

Remix 1.6.2 changed the package layout, which breaks an import and causes typechecking errors:
node_modules/remix-flat-routes/dist/index.d.ts(1,57): error TS2307: Cannot find module '@remix-run/dev/config/routes' or its corresponding type declarations.

Ah, I see. This was used by the tests. Since actual usage will pass in defineRoutes provided by Remix, that's why it didn't break in general.
I should probably just create a dummy defineRoutes for tests instead of doing a deep import (which may break again in the future, since that's technically an implementation detail).


Fix vertical oscillation in density in the 2d_dry_bubble case

It looks like something went wrong in gravity in the previous setup
closes #255

bors try

theta, rhotheta, and rho after 500 secs

@trontrytel Could you take a look at the moist bubble as well?

bors try


Submission data on determination form page

Fixes #3971
Test Steps

Add or edit a determination (e.g. /apply/submissions/2/determination/2/edit/)
See the proposal information on the right for added context

Neat idea. We should have this on both determination and reviews I think.
Can we have it in a fly out tab that only loads when needed via htmx? I think the tab should be closed by default.
When the tab is clicked it takes 50 % of the width.

@frjo We'll have to come back to this, I suppose. Because our change shows it by default, I guess we will want that option too.

@frjo I spent some time looking into making the requested changes. The htmx part is easy and works great: add a button, tell it to get from a new html fragment that contains the submission data. As for getting the elements to flow the way I wanted them to, that seems like another story. There is quite a bit of CSS all the way up and down the elements that seems to push things wider or narrower than I was expecting. In any case, I like your suggestion.

I couldn't seem to get the flex items to match height (which is the default behavior of setting flex) but I discovered the flex-basis style. Using that flex-basis along with some Alpine.js and htmx it looks closer to what we want. What do you think, @frjo?

@bickelj Works really well. Played around with styling a bit and added the same feature to reviews. Please take a look at https://github.com/HyphaApp/hypha/pull/4142.

Superseded by #4142


Screenshot is broken

Somehow the simulator is not taking screenshot anymore. Save image is always black.

We should remove this feature. It is confusing. Save should be hex with the corresponding hardware configuration.

The game code is hardware agnostic

Screenshot seems to be taken only after keypress - it worked for me if I interact with the simulator first


Add reference app category

Add reference app category recommended in flutter officiail Website

This might help other people. Thanks for the contribution @Scienticious


Processing Rosbags in JavaScript

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
MacOS Mojave 10.14

SDK Version
2.16.5

Language
C++

Issue Description
Hi! I am looking to process librealsense's Rosbag outputs inside a React / Node JS app, and was wondering if you had any recommendations for how to process Rosbag files in JavaScript? One option that I found is the Cruise Automation Rosbag library, but was wondering if you've had success with applying that to librealsense's Rosbag outputs? Or perhaps do you know of any wrapper or library that allows use of the Librealsense library in JavaScript? Specifically, I'm looking to be able to load and playback depth frames in a given .bag file

For playback of recorded files functionality in JavaScript, please take a look at Librealsense Node.js wrapper:
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/nodejs

Hi Thomas,
Do you need any further clarifications regarding this topic?


refactor improve join node output when build table rows is 0

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.72 seconds
stream load tsv:          535 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size:<PHONE_NUMBER>5 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.77 seconds
stream load tsv:          547 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>6 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.4 seconds
stream load tsv:          534 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>5 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.66 seconds
stream load tsv:          533 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>8 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.99 seconds
stream load tsv:          529 seconds loaded<PHONE_NUMBER>9 Bytes, about 134 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          32 seconds loaded 861443392 Bytes, about 25 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>5 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.26 seconds
stream load tsv:          530 seconds loaded<PHONE_NUMBER>9 Bytes, about 134 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>8 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.99 seconds
stream load tsv:          533 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.8 seconds inserted 10000000 Rows, about 347K ops/s
storage size:<PHONE_NUMBER>0 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.78 seconds
stream load tsv:          534 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>1 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.75 seconds
stream load tsv:          528 seconds loaded<PHONE_NUMBER>9 Bytes, about 135 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size:<PHONE_NUMBER>1 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.79 seconds
stream load tsv:          529 seconds loaded<PHONE_NUMBER>9 Bytes, about 134 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.7 seconds inserted 10000000 Rows, about 336K ops/s
storage size:<PHONE_NUMBER>4 Bytes

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.73 seconds
stream load tsv:          542 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>1 Bytes


Add Keycloak provider

this adds a provider for Keycloak

Very cool, was just trying to set up this proxy with Keycloak!

@jehiah What is the procedure for getting pending PRs merged in? Is this PR missing something that is blocking it? We would love to see this brought in to the core. This repo has a ton of open issues and PRs, is it mostly unmaintained these days?

See issue #428

we have used keycloak with oauth2_proxy using OIDC provider and it worked well. do we need it still?


questions

The Essence of NFT on BTC is because currently btc does not have fungibility, and the smallest unit of btc, 1 satoshi, is marked as a specific serial number of NFT.
So if one day the BTC network is successfully upgraded to support fungibility, then the question arises, are these NFTs still NFTs?

Bitcoin is fungible today, at a degree that's at least comparable to other large assets. It's certainly more fungible than fiat, especially digital fiat, and I don't think many people would claim that fiat doesn't meet the definition of fungibility.
What you seem to be describing would be more like "perfect privacy", which would be very unlikely to ever be added to the base layer of Bitcoin. No asset in the world currently has anywhere close to that level of privacy, with the movement of gold, oil, and wheat, just as examples, routinely being tracked, even when the owners of the asset don't want it tracked.
So, I don't think the issue is a particularly worrying concern, and it reduces to "what if the fundamental structure of Bitcoin changes", which can't really be addressed prior to that happening. If it did happen, people interested in preserving the value of their NFTs would be incentivized to come up with something they view as a solution.


make release error

Date you used Cookiecutter PyPackage: 15-03-2018
Cookiecutter version used, if any: 1.6.0
Python version, if any: 3.6.4
Operating System: Linux Mint 18.3 Xfce

Description
make release calls clean, which calls clean-build, which removes the dist folder and twine upload dist/* fails.
What I Did
(taika) hector@hector ~/DATA/Documentos/projects/taika $ make release
rm -fr build/
rm -fr dist/
rm -fr .eggs/
find . -name '*.egg-info' -exec rm -fr {} +
find . -name '*.egg' -exec rm -f {} +
find . -name '*.pyc' -exec rm -f {} +
find . -name '*.pyo' -exec rm -f {} +
find . -name '*~' -exec rm -f {} +
find . -name '__pycache__' -exec rm -fr {} +
rm -fr .tox/
rm -f .coverage
rm -fr htmlcov/
twine upload dist/*
ValueError: Cannot find file (or expand pattern): 'dist/*'
Makefile:79: fallo en las instrucciones para el objetivo 'release'
make: *** [release] Error 1

Edit: bad command names.
Edit2: title typo.

I am seeing this issue as well.
My temporary fix looks like this:
release: upload, clean ## Upload a release, clean

upload: ## package and upload a release
	twine upload dist/*

My fix is:
release: clean dist upload clean ## clean, build, upload and clean again

upload: ## upload the contents of dist/* with twine
        twine upload dist/*

Looks like there's a PR #420 that addresses this also.

Closing this then, since there is a PR that addresses it.


Material Banner Issue

Hi , I was checking out flutter new futures . one of them is the material banner , it just works a great but with a little issue.
if we add a material app with transparent background, the app bar would have a white space , but if we set the property of Scaffold ( extendBodyBehindAppBar: true,). anything at body goes as expected behind the app bar, but if we show the material banner , the extendBodyBehindAppBar property looks like it set back to false unless i dismissed the material banner .
the image after showing the material banner

the image before showing material banner

@devnoaman
There have been recent updates made to Material Banner with this PR and merged into master, https://github.com/flutter/flutter/pull/90380.
I am not sure which version are you on, but if you are not on master, then please switch to it's latest version and try again to see if you get same behavior or not.

@darshankawar
@Calamity210
this is sample code
https://github.com/devnoaman/banner/blob/master/lib/main.dart

@Calamity210 the same thing also
i made it from 2 over 10

@Calamity210 the same thing also
i made it from 2 over 10

Hmm, I thought it might be because the MB pushes down content when it's elevation is 0.
Can you send the code for the elevated MaterialBanner? @devnoaman

Oh, yes. The MaterialBanner you pass to the showMaterialBanner call

@Calamity210 and what is the prober way to show it?

yeah about this
MaterialBanner(     elevation: 10,     ... )
i used this from 2 all the way to 10

@Calamity210

ok , do your best,

@devnoaman Giving the MaterialBanner an elevation > 0 fixes it. In your sample above:
ScaffoldMessenger.of(context).showMaterialBanner(MaterialBanner(
+           elevation: 1,
            content: Text('Hi from Material Banner'),
            leading: const Icon(Icons.info),
            backgroundColor: Colors.yellow,
            actions: [
              // TextButton(
              //     onPressed: () {}, child: const Text('Some Action')),
              TextButton(
                  onPressed: () {
                    ScaffoldMessenger.of(context).hideCurrentMaterialBanner();
                  },
                  child: const Text('Dismiss')),
            ],
          ));

@devnoaman
Check above solution and see if it works for you.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Add unit test framework

This would be a huge value added to this project.
Here's an example tutorial of setting up something similar:
https://medium.com/@aussy/lets-create-a-discord-music-bot-using-test-driven-development-tdd-part-1-9c95ea32766f

I am eager to work on this in order to get more experience with TDD and refresh my Jest development based on previous Nestjs projects.

Is this actively being worked on at this point?

Not by me at least, it was commented in the discord that other user would take the initiative with this one as got little time left. If it is still not assigned can start once more into it.

Not by me at least, it was commented in the discord that other user would take the initiative with this one as got little time left. If it is still not assigned can start once more into it.

Yep that was me asking about it, just didn't want to step on anyone's toes if they were pretty deep on this already.

Is this actively being worked on at this point?

Please have at it!

I would consider this feature accepted if we had coverage over at least one of the event handlers and its call tree. Just enough to demo how the test technique applies to our discord.js application at all layers

going to take a stab at this in the next few days if no one else is working on it!


Visual Studio Express 2010 is no longer available for download

All the docs here for getting node-gyp to work on Windows reference Visual Studio Express 2010. The download links no longer work and I don't think it is available any longer from MS. Can we update docs to refer to the steps require to use Visual Studio Express 2013 or some other freely downloadable version (like Visual Studio Community Edition).

closing this - my bad - docs clearly indicate 2013 version on home page


UpdateSubscriptionAsync method does not update AutoDeleteOnIdle properties

Description

i am trying to using the admin api to update some subscription AutDeleteOnIdel time span, code snip as:
var subs = admin.GetSubscriptionsAsync(topicName).GetAsyncEnumerator();
while (subs.MoveNextAsync().AsTask().Result)
{
var subdetails = admin.GetSubscriptionAsync(topicName, subs.Current.SubscriptionName).Result.Value;
subdetails.AutoDeleteOnIdle = TimeSpan.FromDays(90);

var updated = admin.UpdateSubscriptionAsync(subdetails).Result.Value;

Console.Out.WriteLine($"Sub {subdetails.SubscriptionName} in {subdetails.TopicName} updated autodelete on idle");

}
however the code runs fine but AutoDeleteOnIdle was not actually updated
using

Hey @liangdaxian , are you inquiring about the Azure.Messaging.ServiceBus library?
GetSubscriptionsAsync
UpdateSubscriptionAsync

@liangdaxian:  Please clarify which package and version you're using.

hi i think i put it under desc but get trunked:
using

I've confirmed that the autoDeleteOnIdle property is defined in the service spec and that the payload is being formatted correctly by the client.    This looks to be an issue with the Service Bus ATOM service.
@EldertGrootenboer:  Not sure why this was transferred over to the client repository, but I'm going to send it back, as it requires service investigation.

This issue has been transferred to the Azure Service Bus repository for service team investigation as #711.


Improve Error Handling

This PR improves on the error handling by:

Ensuring WindowsAPIError is more consistent and better documented
Simplifying error_check and ensuring it's easier to tell what the original error was.

Current coverage is 95.59%

Merging #59 into master will decrease coverage by -0.86% as of 1a26c85

Powered by Codecov. Updated on successful CI builds.