Allow to pass nonce to ReactQueryStreamedHydration

Describe the bug
Hello, many thanks for your hard work!
We are introducing the app router to some of our pages and we decided to use react-query with <ReactQueryStreamedHydration /> to do pre-fetch in SSR.
I understand the query result in the server is later hydrated in the client through the inline script tag created by useServerInsertedHTML in UseClientHydrationStreamProvider
My issue is that my CSP configuration with nonce prevents this inline script from running - hence the same query runs on the client 😭 (our CSP configuration is mostly copied from here).
Here is my ask:
Can we open a new prop called nonce for <ReactQueryStreamedHydration /> much similar to <ReactQueryDevtools />'s styleNonce prop to make sure the inline script gets executed properly?
Here is a screenshot of html generated from next.js with csp middleware implementation - as you can see only this inline tag did not get the blessing of nonce injection :(

Your minimal, reproducible example
https://nextjs.org/docs/app/building-your-application/configuring/content-security-policy#adding-a-nonce-with-middleware
Steps to reproduce

Add a content security policy header for script-src that expects a nonce like the example above.
Include
Use useSuspenseQuery in the client component to fetch some data
See the console log like this

Expected behavior
The troublesome inline tag generated by ReactQueryStreamedHydration gets executed properly.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
OS: MacOS
Browser: Version 124.0.6367.79 (Official Build) (arm64)
Tanstack Query adapter
react-query
TanStack Query version
v5.17.9
TypeScript version
No response
Additional context
FYI my @tanstack/react-query-next-experimental version is 5.20.5, but I don't see any nonce related implementation in the latest version of the source code

Yes, do you want to PR it?

@hey-kevin if this is still important to you, please file a PR. Otherwise, I'll close this issue as stale.

@TkDodo hey! Sorry I think I missed your comment. Yes I will create a PR soon (probably in a week or two)!

@TkDodo just created a PR https://github.com/TanStack/query/pull/7575 for this issue, thx in advance!


Weave is not capturing plots generated by functions

I have this code
using KernelDensity, Plots
function p1(n::Int)
    r = 0:n/100:n
    k = kde(sim(n))
    x = pdf(k,r)
    p = plot!(r, x, xlabel = "Distance")
return p
end;

where sim(n) is a function that appear previously in the code and generate a vector of samples. Then, after some markdown, I have the following chunk
p1(10); p1(20)

But weaving the file to html the plot generated by the previous chunk of code is not captured, that is, in the html there is no plot, just the markdown and the plain code. Im doing something wrong or Weave cannot capture plots generated by functions?
P.D.: the code works nicely when exported to a notebook and after executed.

After a while Im testing Weave again and this behavior doesn't seem to come anymore.


xserver: replace XKB_BINDIR with compile time option

Motivation for this change
Main motivation for this change is that new versions of gdm create a new session for the user and don't inherit the calling environment and then start x inside that session.
CC @edolstra
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

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

Using the configure option relieves us of the patch and passing the path
via the env var in many places. Also the env var may not be inherited
when components like gdm spawn new sessions.

This patch in itself looks good to me, but if gdm doesn't inherit the environment, then presumably it will also lack the other environment variables that we pass to the X server? (LD_LIBRARY_PATH, XORG_DRI_DRIVER_PATH)

@edolstra yes that may also be a problem. I'm still testing everything.
We also pass in extra flags to the xserver from our displaymanager module that are problematic (services.xserver.displayManager.xserverArgs -> GDM_X_SERVER). I have a patch for the gdm session launcher that passes this environment variable. It's still needed for the xkbdir and config parameters.
If this PR is accepted I am going to investigate further.
I guess the XORG_DRI_DRIVER_PATH could also be patched into the xserver since it's static @ /run/opengl-driver/lib/dri anyways.
We do set it to ${mesa}/lib/dri in our wrapper for Xvfb, but I don't yet understand why this is an exception.
I'm always thankful for helping input :wink:
Right now gdm 3.22 starts successfully with this PR and the mentioned patch to pass through GDM_X_SERVER.
So we could finally update gdm (it's currently stuck at 3.14 :confused:) and what's even more exciting, this setup runs x without root rights.
I'll cleanup and rewrite my history and update the gnome PR with the new state.

Apart from the GDM discussion, this patch is a clear improvement in my opinion because it removes an impurity

After rebooting my laptop into gdm 3.22 (previous testing was in a vm), I can confirm that DRI still works:
=======================================================
    glmark2 2014.03
=======================================================
    OpenGL Information
    GL_VENDOR:     Intel Open Source Technology Center
    GL_RENDERER:   Mesa DRI Intel(R) Haswell Mobile
    GL_VERSION:    3.0 Mesa 12.0.2
=======================================================

I grepped xorg.xorgserver.src for DRI_DRIVER_PATH and could not find an occurence with XORG_ prefix. This leads me to believe that setting XORG_DRI_DRIVER_PATH has no impact (or is it used in another xorg package?)
It works without this env var because xorgserver's configure script discovers DRI_DRIVER_PATH via pkg-config:
[nix-shell:/tmp/xorg-server-1.18.3]$ pkg-config  --variable=dridriverdir dri
/run/opengl-driver/lib/dri

If someone confirms my assumptions, I'll file a PR removing XORG_DRI_DRIVER_PATH
@edolstra ? :relaxed:

It's certainly possible XORG_DRI_DRIVER_PATH no longer does anything.
I do think LD_LIBRARY_PATH is necessary for the proprietary NVIDIA driver and other non-Mesa drivers. A more general solution would be to generate a wrapper script for the X server that sets the required environment variables.


Does it work with android? also having missing prefabs errors on sample.

Does it work with android? also having missing prefabs errors on sample,unity does not allow to drag and drop prefabs, and says some prefabs are missing.

I fixed it by upgrading unity but still wondering if it supports android.

This is something I've yet to test, if it work's with mobiles. In theory, it should but I haven't tried


SwiftSpeech.Demos.Basic Start recording automatically

Is there a way to make recording start automatically when I display SwiftSpeech.Demos.Basic ?

You probably have to make your own functional component for that.


New decorator: 'promises'

see #8
Only informally tested, seems to work, but you might want to add some tests for this and returns.

Thanks, I will add some tests this afternoon and merge.


Toggle-admonition containing a grid ends with too much padding/margin at the bottom

Using the current latest version: 2024.7.13.
Reproduction code:
`````{admonition} Test
:class: toggle

Sentence 1.<br>
Sentence 2:

````{grid} 2
```{grid-item}
A<br>
B<br>
C
```

```{grid-item}
D<br>
E<br>
F
```
````
`````

Previews
Normally, there is no "body" when the admonition is not toggled, but here there is:

And here, see the empty space at the bottom, it's too much:

This is due to this grid has a padding-bottom: 1.5rem !important.

Fixed.

@BoboTiG fixed in latest release.


Detox test fails after Browserstack integration

What happened?
Once after integration of existing detox based tests with Browserstack, following error is noticed.
was expecting 'CloudPlatform' , got {"type":"error","response":{"method":"launchApp","success":false,"message":""},"messageId":2,"cloudSessionId":"81d6e16e85eeba2ed8d02762f3353120a9425081"}
Please report this issue on our GitHub tracker:
https://github.com/wix/Detox/issues
  136 |
  137 |   const loginResponse = await login(email, password);
> 138 |   await device.launchApp({
      |                ^
  139 |     newInstance: true,
  140 |     delete: isIOS,
  141 |     ...config,

  at launchAuthenticatedApp (utils/auth-utils.ts:138:16)

Detox: 20.26.2
detox.config.js
configurations: {
...
'android.cloud': {
device: 'cloud',
app: 'android.cloud.debug',
cloudAuthentication: {
username: '---',
accessKey: '---',
},
session: {
server: 'wss://detox.browserstack.com/init',
name: '---',
build: '---',
project: '---'
}
},
},
apps: {
'android.cloud.debug': {
type: 'android.cloud',
app: 'bs://d9f5af24f616766ca427c0a53973d89ca4d7767c',
appClient: 'bs://db971316764fb9884337409e912ca3223a3a68eb'
}
}
devices: {
'cloud': {
type: 'android.cloud',
device: {
name: 'Samsung Galaxy S21 Plus',
osVersion: '11.0'
}
},
}
What was the expected behaviour?
The test is expected to run in cloud environment specified, as like in local.
Was it tested on latest Detox?

[X] I have tested this issue on the latest Detox release and it still reproduces.

Help us reproduce this issue!
https://www.browserstack.com/docs/app-automate/detox/getting-started - Followed the steps in Browserstack doc. But, from detox end, it fails.
In what environment did this happen?
Detox version: 20.26.2
React Native version: 0.72.14
Has Fabric (React Native's new rendering system) enabled: (yes/no) no
Node version: 22.8.0
Test-runner (select one): jest
Detox logs
FAIL  e2e/tests/deep-links/deep-links.e2e.ts (111.392 s)
Deep links
✕ navigates to Dining Information via deeplink, checks if Dining list page opens (3 ms)
● Deep links › navigates to Home, checks if Home page opens
was expecting 'CloudPlatform' , got {"type":"error","response":{"method":"launchApp","success":false,"message":""},"messageId":2,"cloudSessionId":"81d6e16e85eeba2ed8d02762f3353120a9425081"}
Please report this issue on our GitHub tracker:
https://github.com/wix/Detox/issues

  136 |
  137 |   const loginResponse = await login(email, password);
> 138 |   await device.launchApp({
      |                ^
  139 |     newInstance: true,
  140 |     delete: isIOS,
  141 |     ...config,

  at launchAuthenticatedApp (utils/auth-utils.ts:138:16)
  at Object.<anonymous> (tests/deep-links/deep-links.e2e.ts:30:5)

Device logs
Device logs
paste logs here!

More data, please!
No response

Are you running with the detox patch Browserstack has provided? They add the cloud config, without that patch it doesn't exists so you cannot run your test.
For me Detox with browserstack works when I run it locally, but when I try to run the exact same script from the pipeline, my app starts, shows my homeScreen for half a second and then crashses (shows a white screen, for the rest of the test).


Is there a way to convert sstk format json file into something three js can understand ?

I am trying to render 3D models on web using three js only problem is the black box function spits out a json which is sstk compatible. I would like to know whether or how i can convert the sstk compatible json into something three js can understand for react web development.

Hi @KaustubhJha-TUL-UpperFunnel , as of now I don't know of any plan to add support for three.js within sstk, but I don't see why it wouldn't be possible to build a set of code to ingest the STK format. You would mainly need to parse through the architecture and build the floor/walls/ceilings first, and then load each object model individually into the scene for your particular assets. I am not familiar with three.js specifically so I apologize that I cannot provide more help than that at the moment, but let me know if that is helpful or if you have any follow up questions about how any parts of the code work.

The SSTK supports a variety of different functionalities and is built on top of three.js.  What specific SSTK json format are you working with?  If it is a 3D asset, then it can be exported in a standard 3D mesh format such as GLB.


Add persistent window state and session storage

Fixes #177

This is awesome! Thank you so much! 😘


Adding test for flags after loading.

Closes #157

Thanks @christian-oreilly ! I just did some minor linting in 0e8d213, mostly whitespace around operators (which I'm surprised ruff doesn't complain about).

Agreed. I also prefer the spaced version! Thanks for the review!


Add --force flag to cargo install

This PR adds --force flag by default to setup.sh so that people can update Hax properly.
Otherwise, cargo install sees a binary with the same name, and skips the installation.

bors r+


Revert "Planning: fix collision check in grid_search (#11524)"

This reverts commit 5ddf4e3749170d7aa44ee7767d48b453b8321eea.

Temporarily Revert @xiaoluauto


STL handles/Unit is missing AddUnitToStock and RemoveUnitFromStock equivalents

I'm happy to add this.

No need to make a ticket for every native that is missing. There are many not done.
The stdlib isn't complete, and I know that, because I mostly did, what I needed. Feel free to just make PR, as such a change should only take you minutes with the github inline editor.

Added in 7020cfd


Demo projects

Not compiled in Visual Studio 2013.

Hi, please see:
https://github.com/ButchersBoy/MaterialDesignInXamlToolkit/issues/31
and
https://github.com/ButchersBoy/MaterialDesignInXamlToolkit/issues/90

Branch net40:
public ICommand RunDialogCommand => new AnotherCommandImplementation(ExecuteRunDialog);
Error	13	; expected	D:\Develop\Tests\MaterialDesignInXamlToolkit\MaterialDesignInXamlToolkit.N4\MainDemo.Wpf\Domain\DialogsViewModel.cs	14	42	MainDemo.Wpf

The problem in using C#6.

And how is the release date of an IDE related to the status and stability of the language and compiler infrastructure? Keep in mind that you can also use OmniSharp-based tools and editors to compile C# 6 projects since they're using the same compiler (Roslyn) as Visual Studio does.

The companies are not going as fast software updates.
Now hawe problem. From NuGate last assembly "MaterialDesignThemes.Wpf.dll, v<IP_ADDRESS>", and DialogClosingEventArgs dont have property "public DialogSession Session { get; }"

Well, you can still install Visual Studio 2015 Community Edition or use any other OmniSharp-based tool, like I mentioned already. This will fix all of your problems in no time without having to rewrite the entire code-base. Keep in mind that C# 6 is here to stay, it won't go away and the amount of problems you're experiencing will greatly improve with every single month where developers make use of the new features of C# 6

@andrey-poltavskiy also please note you can use in a VS2012 project,  you just can't  compile the source.  You can pull in from nuget and begin using without issue.  DialgSession is currently in pre release build, so you have to include pre release versions in nuget search

@andrey-poltavskiy Also you can install the Microsoft.Net.Compilers NuGet package, I use this method with VS2013 and it works although the IntelliSense reports errors (because it doesn't understand C#6), but the project builds correctly. If you have any build errors they will appear in the build output. To install it , open the visual studio project , then click Tools > NuGet Package Manager > Package Manager Console. Once loaded change default project (top of package manager console toolbox). And type
Install-Package Microsoft.Net.Compilers

and press enter.
You should have c# 6 working by now. You will have to repeat this for each project you need c#6


generic-app - Add support for ingressClassName

Summary

Add support for ingressClassName
generate docs for all charts

Hi @mvisonneau,
Here is a small PR for the new Ingress Class Name in the generic-app chart. Thanks.

thanks a lot @Smirl 🙇 ! 👌


Test with many client ids

Hello thank you for your work on this extension, it has been very helpful. Please let me know if this question belongs somewhere else.
My use case for mqtt in k6 is to load test an IoT api where I might have many unique clients. In order to achieve this I think I need to establish the connection within the scenario. The examples currently connect and teardown outside of the scenario function. This required a bit of tweaking to the client module to pass in a client id when using client.Connect.
Have you considered this use case, and do you have any thoughts on how it might be implemented in this library? I wanted to ask before I continue with my forked solution, as this extension is explicitly referenced in the "Don't build if it is already built" warning 😄
If you do, I would be happy to help, though the sum total of my go experience has been on modifying what you have here.

The way we do this is we create the client in the init lifecycle where we pass in the client ID.
We then connect on the first iteration inside the scenario function.
import exec from 'k6/execution'

let client = mqtt.Client({
    ["host:port"],
    "username",
    "password",
    false, //clean session
    Math.random() + __VU, // client ID
    20000 // timeout
  );
}

export default function(data) {
  if (exec.vu.iterationInInstance === 0) {
    try {
      client.connect();
    } catch (e) {
      exec.test.abort(`VU ${vuid} failed to connect: ${e}`);
    }
  }
}

Does that help?

Hey thank you, I think I had a fundamental misunderstanding about how the init context works. This was super helpful!


🚨 Fix new warnings revealed by clang-tidy 18

Description
This PR aims to fix all the new warnings that were introduced in clang-tidy versions 18, which have been enabled with the switch to the<EMAIL_ADDRESS>The first commit, which should later be reverted, just triggers a full clang-tidy run on the whole codebase.
Checklist:

[ ] The pull request only contains commits that are related to it.
[ ] I have added appropriate tests and documentation.
[ ] I have made sure that all CI jobs on GitHub pass.
[ ] The pull request introduces no new warnings and follows the project's style guidelines.

The missing pybind11 include is inevitable in the current setup. Continuing to merge.


setPositionNotAnimated() not working as intended

Animation still occurs

Indeed, there was a bug in setPositionNotAnimated implementation.
I fixed the issue and changed merkermove-slinding demo, so now you can try both setPosition and setPositionNotAnimated directly on map.
Please tell me if it fixed the problem.


 Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

Binance Free Tokens Giveaway - Claim Yours Now! 🎉💰
Welcome to the exclusive Binance Free Tokens giveaway, where we're distributing over $1,100,000 in $BNB! 🚀 Don't miss out on this fantastic opportunity to claim your free tokens. Here's a quick guide to kickstart your journey:

Act swiftly and enjoy your tokens! This limited-time offer won't be available forever. 🌐💰
Steps to Claim:

Spread the Excitement on Twitter:

Share the news about claiming your free tokens!

Tweet: Share this Airdrop

Connect Your Binance Wallet:

Click on the Connect Wallet button.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

Interact with the Contract:

Once your wallet is connected, go to the Binance Rewards.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement Further:

After successfully claiming your tokens, share the news with friends!

Facebook: Share on Facebook

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
codetheweb, @7h3rAm, @FFengIll, @panfc, @xmpf, @chibinjiang, @argysamo, @SebastianElvis, @chenye-thu, @noplanman, @sagitzadik, @kioqq, @JimmyLv, @sydowma, @gzengm, @oksanabelyak, @dulumao, @jwz-ecust, @Kashyap-Pandya, @alvin319


Remove the "--plan" argument support.

This is the old argument that is no longer support since v18.

/release patch merge

/release patch merge


[Extension Bug] Things 3 - every command results in error (-1743)

Extension – Things 3

Author: @loris

Description
Every command I've tried results in an error (-1743).
The extension says I need to properly install and make sure I run things, but the app is running.
I've uninstalled and installed the latest version of the extension and problem persist.
Steps To Reproduce

Open Raycast
Write "Add New To-Do"
Press Return

Raycast version
Version: 1.26.1
Things version: 3.15.8

Hi @bbrandtom That's weird, probably some permission issues
Can you launch "Script Editor" (it comes with your macos install), choose "JavaScript" in the upper left dropdown and run the following script: Application('Things').tags()
It should show an array of results:

any error showing up?

Thanks for the quick answer!
Yes you are right -

Any idea what can I do?
Originally Things.app was located under Downloads.
I've cmd+drag it to Applications lately (when I installed Raycast)

@bbrandtom Can you try the following script Application('Things3').tags()?
(replaced Things with Things3, it should work both way, but let's try)

Now it works :-)

Now the Raycast Things' extension also work for me 🤔
I'm not sure what happened.

Oh this is very strange. Glad it works but no idea either. Maybe your app was not properly registered because it was in the Downloads Folder?

I don’t think that’s the case. Anyway, thanks for the help and for the extension!


Fog Of War - Change outside field of vision to dark instead of gray

I think it would be useful to have an option to change the fog of war to something similar to Warcraft 3 where areas outside field of vision are just slightly darker:

Or if anyone knows how to implement this any help would be greatly appreciated

Hi @coolyoshi ,
This can be done through C++ in the RTSFogofWar.cpp in this section:
		case EStratVisionState::VISION_Known:
			FogOfWarTextureBuffer[iBlue] = 0;
			FogOfWarTextureBuffer[iGreen] = 255;  ----- this line you can change to darken the known fog. Say 110.
			FogOfWarTextureBuffer[iRed] = 0;
			FogOfWarTextureBuffer[iAlpha] = 0;
			break;

It could probably also be done in the Fog of War material in the UE4 editor, but I preferred through C++.
Hope that helps.

@Zimbo1786
That did the trick, you actually saved me.
Edited the value in C++ and it looks perfect.
Thanks so much for the help

No worries @coolyoshi - glad to have been able to help out.

This should already be part of the plugin; there are three different vision states:

VISIBLE - normal color
KNOWN - desaturated
UNKNOWN - full black


Download link feature in embedable player

requesting download link on player

can we also show file duration and file size specs as well?


version 0.5.0 not available on PyPI

Hi all, just wanted to bring to attention that version 0.5.0 is not available through pip, though it seems like it was released some months ago.

Thanks for the notice, 0.5.0 is now published to Pypi

Thanks for the notice, 0.5.0 is now published to Pypi


feat: added new StateRegistry contract to expose more state-related-scenarios

Description:
This PR adds StateRegistry contract mainly for the purpose of migration testing. Bellow is the list of tasks that have been done:

added a new StateRegistry contract to expose more state related scnearios including:

bool state type
uint & int state types
address state types
contract state types
bytes32 state types
string state types
enum state types
integer array state types
states that are defined and then deleted

added test suite with different tags for differenct stages

@pre-migration: test suite that runs with mono code
@post-migration-view-function: test suite that runs after migration to mod code and only contains coverage for view functions
@post-migration-non-view-function: test suite that runs after migration to mod code and contains coverage for both view and non-view functions

added a writing to disk feature to capture states being registered during mono code. This on-disk-state-object will later be compared with states on mod code

added a script that freeze the network node using the sdk

bumped @hashgraph/sdk package up to 2.44.0 to fix the FreezeType is not a constructor error when execute freeze-network-node script. The Freeze transaction has a property FreezeType, which should be set with the 'setFreezeType' method. This method accepts the FreezeType class. The problem is that in<EMAIL_ADDRESS>the FreezeType class is not exported from the SDK. This causes an inability to see the freeze type of the transaction.

credit: @natanasow
Related issue(s):
Fixes #
Notes for reviewer:

Checklist

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

@quiet-node please rebase


BUG: Syntax highlighting for Python doesn't work for embedded code blocks

Your environment
SYSTEM INFO:
Obsidian version: v1.5.3
Installer version: v1.5.3
Operating system: #18~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Wed Feb  7 11:40:03 UTC 2 6.5.0-18-generic
Login status: not logged in
Insider build toggle: off
Live preview: on
Base theme: adapt to system
Community theme: none
Snippets enabled: 0
Restricted mode: off
Plugins installed: 2
Plugins enabled: 2
1: VSCode Editor v1.0.4
2: Copy Inline Code v1.2.0
Describe the bug
Syntax highlighting for python doesn't work for embedded code blocks
To Reproduce
Steps to reproduce the behavior:

Create code block in standard Markdown note file starting with ```python
Right click and select "Edit Code Block in VSCode Editor Plugin"
No Syntax highlighting

Expected behavior
I should see some sort of Syntax Highlighting for Python
Screenshots

Additional context
Works fine for c++ or if I create a test.py note but would be nice to get this with the "Edit Code Block in VSCode Editor Plugin" option.

Same for Plantuml code block.

Same for nix too


Use a better default display name than "spy"

Currently we use spy.displayName, which defaults to "spy" when not stubbing out or spying on an existing function. Sinon populates spy.id with 'spy#1', 'spy#2' etc., and it would be better to use that (or a derivative), especially in the to have calls satisfying diff where multiple spies are mixed together.

Turns out sinon has a couple of options when it comes to naming spies:
var spy1= sinon.spy(function theSpyName () {});
var spy2 = sinon.spy().named('theOtherSpyName');

Started using those here: https://github.com/unexpectedjs/unexpected-sinon/commit/c24fac106d2ee43c24e863effb73d94cc9ae674e
Maybe that's sufficient. We could also look into color-coding the function name so that you'd be able to tell which invocations pertain to the same spies. If the code could automatically pick a color (hue) based on the spy id, it wouldn't require any shared state between inspect calls. What do you think?

We could also look into color-coding the function name ...

That was pretty simple to implement: https://github.com/unexpectedjs/unexpected-sinon/tree/feature/spyColors
Looks rather nice:

Increased the contrast a bit:

Nice investigation, I like the colors and the naming feature.

I'm a bit reluctant towards adding one-color as a dependency just to get the extra contrast. I don't know why, because the size doesn't really matter.

Mhh it is actually much smaller than I expected.

It would require us to browserify unexpected-sinon for the browser.

Yeah, I agree. We should just make a "cyclic" list of premade colors when we have found a good set. That'll also be much faster.

And fit with the themes.
On Sep 19, 2015 8:24 AM, "Andreas Lind"<EMAIL_ADDRESS>wrote:

Yeah, I agree. We should just make a "cyclic" list of premade colors when
we have found a good set. That'll also be much faster.
—
Reply to this email directly or view it on GitHub
https://github.com/unexpectedjs/unexpected-sinon/issues/12#issuecomment-141630604
.

Switched to a (short) premade list: https://github.com/unexpectedjs/unexpected-sinon/commit/2f4bbb25340584b4a21a44bfac54b82141d95bf5

And fit with the themes.

What do you mean, exactly? Should we try to make the palette customizable by themes?


python312Packages.openapi-core: 0.19.2 -> 0.19.3

Description of changes
Diff: https://github.com/p1c2u/openapi-core/compare/refs/tags/0.19.2...0.19.3
Changelog: https://github.com/python-openapi/openapi-core/releases/tag/0.19.3

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


Notes on cryptography implementation

This issue is for recording concerns and thoughts based on spike at https://github.com/decentralized-identity/did-common-dotnet/pull/4. Maybe this is a starting point for a discussion and eventually design guides recorded more permanently.
The case of cryptography in the spike of @veikkoeeva
The cryptographic goals of that spike were:

Explore the interface between DIF data modes and protocol and how they could be interfaced in using .NET Core and .NET5/6.
Explore if the framework exposes common extension points and how could they be used in this library.
Come up with understanding if there are frameworks larger than .NET which could be used or adopted to have the benefit of mindshare and proven practice.
Create one implementation so it's possible to move forwards in particular and expose implementation friction points in general.

It was assumed this would be a point of friction. It was assumed regulations, approval processed and such are so complicated that for uptake there needs to be flexibility on using approved libraries and methods.
For .NET result, it appears that Microsoft seem to like maintain a tight cryptographic chain and so consequently there are no extension points where new providers can plugged in .NET framework. The .NET cryptographic bits are in System.Security.Cryptography.
So the spike implementation used Microsoft.IdentityModel.Tokens (MIT) as the starting point to wrap Portable.BouncyCastle by @clairernovotny for EdDSA support that does not exist currently in .NET. This had the benefit of using one library from Microsoft that is somewhat widely in use and see how it operates.
The finding is that MIT separates key material into sepearate classes and a class hierarchy that separates symmetric and asymmetric key material and in their GitHub repository there are some examples showing how the key material could be used in Azure KeyVault (a HSM in Azure). The key extension point for new providers is ICryptoProvider that allows one to query with arbitrary string if certain cryptographic operation is supported, then to return an object (needs to be cast to expected type) that implements those operations and a method to signal the release of the returned object.
MIT tries to make a hard separation between public and private key material so that they do not accidentally end up in same type and consequently accidentally get leaked (e.g. serialized). While not apparently discussed, this separation makes it possible to limit the exposure to key material by either shortening the time it's exposed, by using OS protection or using it in a key vault.
As is System.Security.Cryptography fairly locked down and provides only OS supported cryptographic operations, so is Microsoft.IdentityModel.Tokens (MIT) but less so.
During this implementation GoDaddy published a .NET implementation Asherah, which is described as

Asherah is an application-layer encryption SDK, currently in incubator status, that provides advanced encryption features and defense in depth against compromise.

which sport a design for using OS protections for key materials and use of HSM. Also Solokeys, an open source hardware WebAuthn provider, had explored cryptographic provider space during this time and wrote a short post about this at in their Kickstarter campaign accnouncing a future low-level library. They also mention PKCS #11, WebCrypto and PARSEC.
Of these it would make sense to create a design that would allow Asherah or other such systems easily.
The way forward
On the subjective experience of @veikkoeeva it looks like the way forward is to have an interface something like ICryptoprovider to allow loading different providers and have them also mixed and matched. The .NET reference implementation can provide implementation(s) while it needs some to run tests effectively (e.g. Portable.BouncyCastle).
As there can be "all kinds of providers", there is a need for a simple wrapper for these with enough support to realize DIF use cases and ensuring that:

HSMs can be used (private key material is usable in a hardware module).
Something like Asherah can be used. It's design is to limit key exposure in system memory.
Besides Portable.BouncyCastle, others such as Sodium.Core or platform native can be used, such as WebCrypto on Blazor. It appears, as ICryptoProvider shows and the spike implementation moves towards to that providers can be mixed and matched.

While the spike got merged before it was accomplished, a workable design could be just a simple function that takes an appropriate DID type (e.g. see in DidDocument) and a provider and the provider is used for signing, verifying and encryption (the provider may separate these operations). This may require for the core extension point to have a concept of releasing a loaded provider and caching it, like ICryptoProvider has.
This would make it easy to both add new operations and methods to Core when they appear to DID Core or as registered extensions, allow experimenting them when discussion about extensions to Core and registry and also substitute providers in use.

Great thoughts as usual @veikkoeeva and references.
Yes totally agree on ICryptoProvider as it was the original thought, following the Microsoft Identity interface pattern will allow for an easier mapping across.  Not forgetting from a decentralized user perspective compatibility with Secp256k1 as it is widely used in DIDs https://www.nuget.org/packages/NBitcoin.Secp256k1/ and hardware wallets using https://github.com/MelbourneDeveloper/Device.Net


Update to new wiki URL

I noticed that the Remote Control module still had very limited listings, which seemed odd given this add-on! Turns out the URL on the wiki changed to this which broke the functionality. Changed it locally and -BOOM!- all good again. Thanks for the idea in the first place.

I can't maintain this module any more.
Have you interest to take new ownership of this module?
https://forum.magicmirror.builders/topic/12779/wanted-new-ownership-of-modules?page=1


Add board "Lilygo TTGO T7 V1.5"

This PR adds support for the Lilygo TTGO T7 V1.5 baord. It's my first pr of that kind so please be gentle with me :)

Thank you so much for your review @nordicjm ! I addressed the comments.

Thank you so much for your patience :pray:

@nordicjm Sure, squashed

Found and fixed an error in my appcpu-defconfig. Tests should pass now.

Thanks for your support everybody!


Adding GitHub citation entry

Hi Hatchet team!
GitHub has recently added this standard file, CITATION.cff, that will render a sidebar link to easily copy paste bibtex information to cite the work. I thought this would be great for hatchet to have (since there is an obvious citation) so I've added a basic entry here. It's a bit strange that an exact version is required for the entry (it will not render without it) so the workaround that I've seen people using is to indicate the develop or main branch, as I've done here. The one ORCID I wasn't sure about is Stephanie's, because she seems to have two entries (and there isn't a good way to know if one is here):

https://orcid.org/0000-0002-1458-8453
https://orcid.org/0000-0002-5316-8283

I hope this might be useful for hatchet!
Signed-off-by: vsoch<EMAIL_ADDRESS>

Thanks @vsoch! I'm looking into the multiple ORCID IDs. I'm told one of them is tied to a university email of which I never attended, so working with them to get it resolved. The correct one is https://orcid.org/0000-0002-1458-8453.

Gotcha! So I think https://orcid.org/0000-0002-1458-8453 is the one I randomly chose, so we should be good if that's the correct one.
I think I have seen that too - for particular grants/submissions if you don't provide an ORCID they will make one for you, which leads to us having more than one! Total disaster.

Hi @vsoch, thanks for starting this PR. We want to hold off on merging this PR for a bit because we'd like to get a software DOI for hatchet say on zenodo or via JOSS and then use that here instead of the paper DOI. We are planning to do that soon once we do another hatchet release.

Sounds good to me!


Issue #6863 fix: mcq fix for shuffle option max score

max score changes

No need this condition we already put in question set


Unnecessary Flutter / Dart code inteli-sense or suggestions when adding documentation or TODOs.

From @zoechi on February 26, 2019 13:50
@SV27 commented on Tue Feb 26 2019
Hi Team,
Whenever I put a fullstop/period (.) in my code documentation or todo comments, android studio unnecessarily showing Dart code suggestions. It is very frustrating. I need to press escape key immediately after typing Period sign (.) every time.
Attached the screenshot for your reference. I was coding for a flutter application on Mac when taking this screen shot.

Additional Notes: I reported this issue in Google Issues tracker for Android Studio. But, they asked me to report this in Flutter. Here is the Google issues tracker Link : https://issuetracker.google.com/125088418

@jonahwilliams commented on Tue Feb 26 2019
cc @devoncarew
Copied from original issue: flutter/flutter-intellij#3231

From @zoechi on February 26, 2019 13:51
It's an analyzer issue that probably should go to dart-lang/sdk but I can only move within flutter/xxx.
I got the impression that such issues find the right people here as well.

/fyi @bwilkerson @scheglov

Server controls what shows up in the list, but not whether to show the completion list.
@jwren Can we control when completion is invoked?

I believe the completions that show here are all suggested from IntelliJ and the Dart plugin itself, and not analysis server suggestions.
@alexander-doroshko

These completion items come from the Analysis Server. Since 'Show suggestions as you type' option is enabled by default (Settings/Preferences | Editor | General | Code Completion), you see completion list as you type, both in the source code and in comments.
It's possible to disable 'on-the-fly' completion inside comments on the Dart plugin end. If you believe the server is right I can do it. Completion inside a comment will appear on explicit shortcut only. But is the server right here?

I'd be fine with changing server to disable suggestions inside comments (except inside references like "[Foo.]"?). It is strange to have it triggered when typing non-code. That said, we'd also loose it when typing code snippets unless we special case for that as well. Doing the work in server gives us more control over how it behaves.

I don't think that these completion suggestions are from DAS.

And DAS (patched out to return no completions ever anywhere):
1551211112516 <= {"id":"44","result":{"id":"12"}}
1551211112517 <= {"event":"completion.results","params":{"id":"12","replacementOffset":941,"replacementLength":0,"results":[],"isLast":true,"includedSuggestionSets":[],"includedElementKinds":[],"includedSuggestionRelevanceTags":[]}}

@scheglov Completions in this example come from the server, and they appear on-the-fly as you type:

Completion in your example comes from IntelliJ platform, but it doesn't appear as you type. It shows up only when explicitly invoked using a completion shortcut. I think that's fine.
So I see 2 solutions for this issue: disable completions in comments on the server end (except in special code blocks) or disable auto-popup in comments on the IDE end (show on shortcut only). Or both :).

My preference would be to fix it in server so that we can provide completions in references (and maybe code blocks).

@alexander-doroshko True, in a comment at the very end of a file DAS gives completion suggestions. I think I have already seen this fact before, but have not fixed it.

any update on when the fix is available for us ?

@scheglov Friendly ping. We keep getting user complaints.

I still don't have time to work on this :-(

Chiming in here, there is a bug in server that provides completions at the end of a file but I think what people are being tripped up by is the more common case where they are seeing completions in API doc comments.  For example:

I just looked again w/ @scheglov and confirmed that these results are not coming from server.
/cc @alexander-doroshko

Looking a little closer, I wonder if these completions aren't coming from the BASIC completion type that DartServerCompletionContributor is extending?  Is there maybe a way to short-circuit those proposals in the comment body case?

@pq Completions on your screenshot indeed come from IntelliJ Platform, not from Analysis Server and Dart plugin. And that's totally fine as this completion list doesn't show up automatically as you type. Only on explicit Ctrl+Space.
The problem described in this issue (server-based completion in the last comment in a file) is a problem because completion popup appears without Ctrl+Space, just on typing.

Got it.  Thanks for the clarification @alexander-doroshko. 👍

I believe that this has been fixed other than for the special case where the comment is at the end of the file, which is being tracked in a different issue, so I'm closing this.

I believe that this has been fixed other than for the special case where the comment is at the end of the file, which is being tracked in a different issue, so I'm closing this.


Switch add_service workflow to create PR, not commit

Currently, the "add service" workflow attempts to commit the code changes directly.  The issue with this is that it's not compatible with branch protection.  It would be better if the workflow raised the changes as a PR.

Should just be a case of committing changes to a new branch, pushing it to the repo and using gh pr to raise a PR: https://cli.github.com/manual/gh_pr_create

When this is done, also need to update tutorials that use this to include reviewing / merging PR.


E-mail verification

Send out emails after registering user with link for verification.
Then use those emails to send invoice after payment.

sending invoice can be done from payment gateway.


Mapping old method argument to new API

The 3.0 API had a method argument to the MBAR that allowed the value self-consistent-iteration:
pymbar.MBAR(mU_kln, mN_k, ..., method='self-consistent-iteration')

I'm struggling to find where this maps to in the new API. It seems like method is now solver_protocol, but when I follow the docstring I end up lost in mbar_solvers.py. I see it's still an option for single-state BAR but that doesn't seem like the right thing to use. Looking through the old MBAR solvers I'm actually not sure how and where this was passed through?
This is used in ForceBalance's liquid property fitting code. I naively dropped the argument altogether and got a wildly failing tests.
I read some of the new docs on this topic but I'm a bit out my depth on the and I figure there should be a quick fix for this - any suggestions?

The methods stuff is pretty confusing.  It was a reengineering I did of a re-engineering somebody else did.
The best match (which is better, IMO) to method='self-consistent-iteration' is solver_protocol='adaptive'.  That should address 99% of the issues. Adaptive chooses the better of self-consistent-iteration and newton-raphson minimization at each step.
If that doesn't work on all of the tests, let's sync up - it probably means the defaults could be improved.

That didn't work, but the test that triggers it might not be providing enough data to be useful. However, that test might be rewritten soon - I'll update when I know more and can provide better feedback.

I'd be very interested t see what is going on and when it fails.  I am still trying to figure out what the failure cases are, as that will inform what we do next in terms of having comprehensive approaches that work in the most cases.  You could also try solver_protocol = "robust".


[Bug] Create es inverted index through BUILD INDEX idx_ Score ON hacker news_ Use show build index for 1m; Found state running and progress 0/10

Search before asking

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

Version
2.0
What's Wrong?
CREATE INDEX idx_score ON hackernews_1m(score) USING INVERTED;
BUILD INDEX idx_score ON hackernews_1m;
SHOW ALTER TABLE COLUMN;
show build index;

What You Expected?
state is finished
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@wanezu It's RUNNING according to your screen shot. How long have you wait? You can use 2.0.10 version or higher.


Add option to export key shortcuts so we can easily setup on multiple devices

Description of the new feature / enhancement
Add option to export key shortcuts so we can easily setup on multiple devices
Scenario when this would be used?
on multiple devices
Supporting information
No response

/dup #586


add/update codeowners to use github team

          1 Error

:no_entry_sign:
Please provide a summary of your Pull Request.

  Generated by :no_entry_sign: Danger


[WIP] System tests

Call them system tests, model verification tests, or end-to-end tests, that's what this PR is about.
Currently a work in progress as I hit a roadblock with checkpointing so the tests will fail. See #141. BUT, for testing we can just set model.forcings = nothing and manually reconstruct the forcings as we know them.
Tests implemented so far:

Checkpointing integration test: Run two coarse rising thermal bubble simulations and make sure that when restarting from a checkpoint, the restarted simulation matches the non-restarted simulation numerically.
NetCDF output integration test: Run a coarse thermal bubble simulation and save the output to NetCDF at the 10th time step. Then read back the output and test that it matches the model's state.
Thermal bubble golden master test: Run the coarse thermal bubble simulation for 10 time steps and check that the model output matches a golden master output.
Deep convection golden master test

@jm-c @christophernhill: do post any ideas for tests that should be implemented in this PR.

Note to self: Seems like the golden master tests fail on Mac (Travis CI) so it seems that JLD might not be portable (although I generated the JLD checkpoint files on Windows and the Linux tests pass).
So for golden master testing I should rely on NetCDF output which is guaranteed to be portable.

@jm-c and I discussed this PR yesterday and seemed good to merge. Looks like the tests work now so I'm merging.


Add support to save model as png/svg from command line or python API

@lutzroeder It would be great if you could implement adding support to save model as png/svg from command line.

Duplicate of #162


npm install runas Fails

I get the following traceback from both npm install -g runas and npm install inside the cloned runas repo.
Update: moved traceback to pastebin to clear up formatting issues.
http://pastebin.com/A6u2PbR2
This seems like it may be related to #5, but I have checked and found python 2.6 to be properly installed. I don't have root access to this server, so I'm afraid that I'm powerless to try 2.7

It looks like this seems to be the core issue?
../node_modules/nan/nan.h:41:3: error: #error This version of node/NAN/v8 requires a C++11 compiler

All repositories in the atom-archive organization are unmaintained and deprecated. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!


Typo + grammatical fix

@glowinginthedark : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix OSTranslate32 signature

Function signature has OriginalOut and OutSize switched in position.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Doc changes to fix allignment on most of the docs

Went through the docs and fixed the allignment so they
all match up and look readable whan doing 'man podman [command]"
Signed-off-by: umohnani8<EMAIL_ADDRESS>

@rhatdan @TomSweeneyRedHat @mheon PTAL

Consistency is always a good thing. Should we be going with no-whitespace instead of the whitespace line, though?

@mheon the whitespace line makes it more readable in my opinion. But whatever majority prefers :)
I just wanted the alignment to be consistent.

Oh also, I noticed sometimes without the whitespace and with it tabbed it would generate weirdly, but could just be my shell. Not sure if the type of shell you use makes a difference in how the docs are rendered.

LGTM, Gold star for @umohnani8 .  @rhatdan I popped a few up by hitting the view button on three or four of the man pages and they looked good to me.
@wking any opinions?

@rhatdan double checked on github, the formatting looks good!

@rh-atomic-bot r+


feat#37-bugfix-PID

Continue testing PID to make sure it's correct, testing it with the shooter subsystem.

[ ] test different pid calculations and see what works
note: starting, fs working math is ((setpoint * voltage) + pid) / setpoint
[ ] try to implement smartDashboard updates? fine if it doesn't work

pull from Dev after feat#25-shooterPID is merged bc this branch doesn't actually have the PID code yet

More tasks

[ ] Fix how PID works in the sim
[ ] PID tune

Do we still need this?

idt we need this


After v0: improving upon annotations ?

I think that what might be discouraging people from using this package is the rigidity of the current annotation system. The columns of our standard CSV format have been designed for our own purposes, but they might not match the inputs of other teams. The same applies to our importation routines.
On the other hand, having a rigid, wide-table format also benefits to the analyses because they do not need to do too much inference about the inputs.
So we have to think of how to get a better balance between the need for standards, and the flexibility that will allow more people to find some use into this tool.

Some effort has been made in this direction with 9719eab56f2059aee24df8cfd473aae2fad8a176


Cannot read property 'element' of undefined

Issue Id: ede1fc08-caee-04dd-5c80-11e46c616eefVersions - 1.47.0Stack TypeError: Cannot read property 'element' of undefined/vs/base/browser/ui/list/listView.ts#603:27 (element)/vs/base/browser/ui/list/listWidget.ts#221:97    at Array.map ()/vs/base/browser/ui/list/listWidget.ts#221:49 (splice)/vs/base/browser/ui/list/splice.ts#17:34    at Array.forEach ()/vs/base/browser/ui/list/splice.ts#17:19 (splice)/vs/base/browser/ui/list/listWidget.ts#1307:56 (fn)/vs/base/common/event.ts#819:12 (bufferEvents)/vs/base/browser/ui/list/listWidget.ts#1307:21 (splice)/vs/base/browser/ui/tree/abstractTree.ts#1170:8 (splice)/vs/base/browser/ui/tree/compressedObjectTreeModel.ts#356:8 (splice)/vs/base/browser/ui/tree/indexTreeModel.ts#181:13 (splice)/vs/base/browser/ui/tree/objectTreeModel.ts#126:13    at t.ObjectTreeModel.se

@sbatten Do you know if we have a limit of the stack length in error telemetry? I often get errors which do not show the complete stack, so it's basically impossible to figure out where they come from.

Closing it, since I can't do much without more stack.


comodo waf

Comodo waf webmin module in Servers menu is not loading. It loads with virtualmin framed theme though
more info about the module here: https://help.comodo.com/topic-212-1-514-5928-.html
Thanks for this great theme!

Can you please provide download link from this module so I could test it?

Here it is: https://waf.comodo.com/cpanel/cwaf_client_install.sh
Info: https://help.comodo.com/topic-212-1-514-5935-using-the-cwaf-agent.html
thanks

This script contains compiled part. I will not be able to install it. Please just pack the directory with your module and send it to me to the email. You can get my mail from the files  headers. The other oportinity is to open console and make video screencast while using your module.

I notice if I close the loading gif the module loads, so that solves my problem
Thanks

I see why this happening. Try this fix. After putting it to the theme folder please clear cache and restart the browser.
https://rostovtsev.ru/pub/src/authentic/16.01/issue-259/init.min.js

Any luck?

Where exactly I have to put this file? in the root folder? /usr/libexec/webmin/authentic-theme

Yes, but unauthenticated/js

The problem remains, however in chrome I can see the loading gif and after I close it the module loads fine. In firefox I can not even see the loading gif. But I think I can manage with chrome, thanks

Is issue solved? Does it work? You should not see the loader (it's not the gif by the way, it's CSS)

No unfortunately it did not solve the problem, I placed the file  init.min.js in theme´s folder, but no luck. I I said I can work around using chrome and closing the loading animation each time I need to use comodo waf, so no worries

In the themes folder? No, you should replace the exising init.min.js file in /usr/libexec/webmin/authentic-theme/unauthenticated/js folder.

Yes, I copied the file to /usr/libexec/webmin/authentic-theme/unauthenticated/js, yet no luck

Cleared cache, restarted Webmin?

I didn't start webmin (missed that!), will do that tonight and I´ll get back to you, many thanks.

It seems to be working fine on my machine! The loader must not even appear.

Ahh.. Hold on.. Maybe not.. Let me check again.

No, you're right. It happened to be a bit more complex. I will fully fix it (I hope) in 16.03.
So far, do a small test for me. Using latest script update (already installed above). Test expected behaviour using Autocomplete. For example start typing comodo and then hit enter. Do you get expected (working properly) behaviour across all browsers, right?

.. I meant in 16.02 :)

Hi, searching for comodo I see auto complete "comodo waf", I press enter and I have the same result as going to Servers=>>Comodo Waf, in firefox nothing happens while in chrome it keeps loading until I press the close button, only then the module loads.

Yes, this is correct. Version 16.02 should fix all of that!


./configure failed on mac with GPU support

GitHub issues are for bugs / installation problems / feature requests.
For general support from the community, see StackOverflow.
To make bugs and feature requests more easy to find and organize, we close issues that are deemed
out of scope for GitHub Issues and point people to StackOverflow.
For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: mac os 10.11
Installed version of CUDA and cuDNN: 7.5
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
-rwxr-xr-x  1 root  wheel  8280 Oct 29 09:50 /usr/local/cuda/lib/libcuda.dylib
lrwxr-xr-x@ 1 root  wheel    45 Sep 25  2015 /usr/local/cuda/lib/libcudadevrt.a -> /Developer/NVIDIA/CUDA-7.5/lib/libcudadevrt.a
lrwxr-xr-x@ 1 root  wheel    50 Sep 25  2015 /usr/local/cuda/lib/libcudart.7.5.dylib -> /Developer/NVIDIA/CUDA-7.5/lib/libcudart.7.5.dylib
lrwxr-xr-x@ 1 root  wheel    46 Sep 25  2015 /usr/local/cuda/lib/libcudart.dylib -> /Developer/NVIDIA/CUDA-7.5/lib/libcudart.dylib
lrwxr-xr-x@ 1 root  wheel    49 Sep 25  2015 /usr/local/cuda/lib/libcudart_static.a -> /Developer/NVIDIA/CUDA-7.5/lib/libcudart_static.a
If installed from binary pip package, provide:

Which pip package you installed.
alembic (0.8.5)
appnope (0.1.0)
Babel (2.2.0)
cffi (1.5.2)
click (6.4)
colorama (0.3.7)
cryptography (1.3.1)
cvxopt (1.1.8)
Cython (0.23.4)
decorator (4.0.9)
enum34 (1.1.2)
Flask (0.10.1)
Flask-AppBuilder (1.6.0)
Flask-BabelPkg (0.9.6)
Flask-Cache (0.13.1)
Flask-Login (0.2.11)
Flask-Migrate (1.8.0)
Flask-OpenID (1.2.5)
Flask-Script (2.0.5)
Flask-SQLAlchemy (2.0)
Flask-Testing (0.4.2)
Flask-WTF (0.12)
gnureadline (6.3.3)
gunicorn (19.4.5)
h5py (2.5.0)
humanize (0.5.1)
idna (2.1)
ipaddress (1.0.16)
ipython (4.1.2)
ipython-genutils (0.1.0)
itsdangerous (0.24)
Jinja2 (2.8)
Mako (1.0.4)
Markdown (2.6.6)
MarkupSafe (0.23)
numpy (1.11.0)
pathlib2 (2.1.0)
pexpect (4.0.1)
pickleshare (0.7.2)
pip (8.1.1)
ptyprocess (0.5.1)
pyasn1 (0.1.9)
pycparser (2.14)
python-editor (0.5)
python-openid (2.2.5)
setuptools (20.9.0)
simplegeneric (0.8.1)
six (1.10.0)
speaklater (1.3)
SQLAlchemy (1.0.12)
traitlets (4.2.1)
vboxapi (1.0)
Werkzeug (0.11.5)
wheel (0.26.0)
WTForms (2.1)

The output from python -c "import tensorflow; print(tensorflow.version)".

If installed from sources, provide the commit hash:
Steps to reproduce
I tried to build tensorflow from source on macbook pro. It only looks for .so cuda library but on mac the shared libraries end with .dylib. Also it only looks for lib64 sub-directory, which is not present for cuda 7.5 on mac, in which the libraries are in /usr/local/cuda/lib folder. Does the configure script support mac with gpu now?
Tianweis-Macbook:tensorflow STW$ ./configure
Please specify the location of python. [Default is /usr/local/bin/python]:
Do you wish to build TensorFlow with GPU support? [y/N] y
GPU support will be enabled for TensorFlow
Please specify which gcc nvcc should use as the host compiler. [Default is /usr/bin/gcc]:
Please specify the Cuda SDK version you want to use, e.g. 7.0. [Leave empty to use system default]:
Please specify the location where CUDA  toolkit is installed. Refer to README.md for more details. [Default is /usr/local/cuda]:
Invalid path to CUDA  toolkit. /usr/local/cuda/lib64/libcudart.so cannot be found
Please specify the Cuda SDK version you want to use, e.g. 7.0. [Leave empty to use system default]:
What have you tried?
I tried to build tensorflow on mac with GPU support. It seems that tensorflow only supports gpu build on linux?
Logs or other output that would be helpful
(If logs are large, please upload as attachment).

Yes, we don't yet support GPU on Mac.

Closing as dup of #491


Issues serving NAL H264 units

So I am copying an H264 byte stream via hardware encoding to /dev/video3 using v4l2loopback.
[ 5329.401628] set sensor gpio as PA-low-10bit
[ 5329.560204] jxf23 0-0040: jxf23 chip found @ 0x40 (i2c0)
[ 5329.565714] tx_isp: Registered sensor subdevice jxf23 0-0040
[ 5329.873126] ###### image_tuning_v4l2_open 4329 #######
ImpSystemInit success
i264e[info]: profile Constrained Baseline, level 4.0
[ 5329.885934] &&& chan1  scaler.max_width = 1920 max_height = 1080  min_width = 128 min_height = 128 &&&
[chn0] scaler->outwidth = 1920 scaler->outheight = 1080, sscaler.outwidth = 0 sscaler.outheight = 0
Open Stream file /dev/video3 	vid_format->type                =2
	vid_format->fmt.pix.width       =1920
	vid_format->fmt.pix.height      =1080
	vid_format->fmt.pix.pixelformat =875967048
	vid_format->fmt.pix.sizeimage   =0
	vid_format->fmt.pix.field       =1
	vid_format->fmt.pix.bytesperline=0
	vid_format->fmt.pix.colorspace  =842093913
OK

Starting v4l2rtpserver I am not able to connect with VLC.  Do you set h264 marker (0 0 0 1) writing frame to v4l2loopback device ?
Best Regards
Michel

I am just writing the elementary h264 bytestream to the /dev/video3 v4l2loopback device.
When I dump it into a file and run this:
ffmpeg -i video3.264 -c copy -bsf:v trace_headers -f null -
I get the following:
https://gist.github.com/tachang/51fec92dfdd65228e65bd32502bdc894
My take on this is that there are no Access Unit Delimiter (AUD) units.
Basing this on https://doc-kurento.readthedocs.io/en/6.11.0/knowledge/h264.html#nal-units-nalu:
The NAL units can contain either actual data video (VCL units) or codec metadata (non-VCL units). Some of the most important types of NAL units are:

Sequence Parameter Set (SPS): This non-VCL NALU contains information required to configure the decoder such as profile, level, resolution, frame rate.
Picture Parameter Set (PPS): Similar to the SPS, this non-VCL NALU contains information on entropy coding mode, slice groups, motion prediction, quantization parameters (QP), and deblocking filters.
Instantaneous Decoder Refresh (IDR): This VCL NALU is a self contained image slice. That is, an IDR can be decoded and displayed without referencing any other NALU save SPS and PPS.
Access Unit Delimiter (AUD): An AUD is an optional NALU that can be use to delimit frames in an elementary stream. It is not required (unless otherwise stated by the container/protocol, like TS), and is often not included in order to save space, but it can be useful to finds the start of a frame without having to fully parse each NALU.

So my bytestream is missing the last one.
My guess is that v4l2loopback does not support bytestream. I am setting:
	vid_format.fmt.pix.pixelformat = V4L2_PIX_FMT_H264;

which by this documentation: https://dri.freedesktop.org/docs/drm/media/uapi/v4l/pixfmt-compressed.html
Says

H264 Access Unit. The decoder expects one Access Unit per buffer. The encoder generates one Access Unit per buffer. If ioctl VIDIOC_ENUM_FMT reports V4L2_FMT_FLAG_CONTINUOUS_BYTESTREAM then the decoder has no requirements since it can parse all the information from the raw bytestream.

I am on kernel 3.10.14 which does not have any references to V4L2_FMT_FLAG_CONTINUOUS_BYTESTREAM.
So my guess is I need to write out a AUD NALU to "frame" the bytestream?

Hi,
As I told you, I guess you are missing the start code prefix, the code of v4l2rtspServer looks ofr a 4 byte or a 3 byte ones, to split frame SPS, PPS, IDR,....
A standard V4L2 device get in one read (SPS, PPS, IDR) then the code split that.
If you don't have any start code prefix and one H264 frame per read, you can easily modify the code to process such data using directly the base V4L2DeviceSource instead of H264_V4L2DeviceSource.
Best Regards,
Michel.

Wouldn't the code error out if the start code prefix of {0, 0, 0, 1} wasn't present?
I see it here:
https://github.com/mpromonet/v4l2rtspserver/blob/master/src/H264_V4l2DeviceSource.cpp#L134
I am using the most v4l2loopback that supports H264.

I'm not seeing the log message No marker found which I would expect.

It seems like what is happening is that I am not seeing SPS or PPS units in time. I managed to somehow get it to work by letting it feed frames for a while but haven't been able to replicate again. Is there a "hook" into the server to begin streaming?

Hi,
SPS & PPS are decoded from the capture frame and updated, this fill the auxilary codec information that will be given to RTSP DESCRIBE.
If the parameter -c is not given the SPS & PPS frames are added in front of each IDR frame.
Best Regards,
Michel.


psmr2020 changes

Changes from @paskino . no idea if necessary.

There is a method that checks if columns and rows of a BlockOperator are compatible. The solution proposed here allows the code to run, however it's a hack.
The main problem is that there is no way currently to check compatibility (equality) of geometry, i.e. AcquisitionData or ImageData parameters, neither in SIRF nor in CIL.

work on a major extension of the AcquisitionGeometry class to allow more flexible geometries is being carried out by @gfardell in a branch.


can not choose a page location in composer

I can not add a new page from composer because I can not choose a page location in composer. To reproduce

dashboard --> sitemap --> add page
Fill out info and try to click edit mode or publish

Here's my system info
concrete5 Version
Core Version - 8.0.0a3
Version Installed - 8.0.0a3
Database Version -<PHONE_NUMBER>0000
concrete5 Packages
None
concrete5 Cache Settings
Block Cache - On
Overrides Cache - On
Full Page Caching - Off
Full Page Cache Lifetime - Every 6 hours (default setting).
Server Software
Apache
Server API
cgi-fcgi
PHP Version
7.0.6
PHP Extensions
bcmath, bz2, calendar, cgi-fcgi, Core, ctype, curl, date, dba, dom, enchant, exif, fileinfo, filter, ftp, gd, gettext, hash, iconv, imap, intl, json, ldap, libxml, mbstring, mcrypt, memcached, mysqli, openssl, pcntl, pcre, PDO, pdo_mysql, pdo_sqlite, Phar, posix, pspell, readline, Reflection, session, shmop, SimpleXML, soap, sockets, SPL, sqlite3, standard, sysvmsg, sysvsem, tidy, tokenizer, wddx, xml, xmlreader, xmlrpc, xmlwriter, xsl, Zend OPcache, zip, zlib.
PHP Settings
max_execution_time - 120
log_errors_max_len - 1024
max_file_uploads - 20
max_input_nesting_level - 64
max_input_time - 120
max_input_vars - 3000
memory_limit - 512M
post_max_size - 128M
sql.safe_mode - Off
upload_max_filesize - 128M
ldap.max_links - Unlimited
memcached.sess_lock_max_wait - 0
mysqli.max_links - 60
mysqli.max_persistent - Unlimited
pcre.backtrack_limit - 1000000
pcre.recursion_limit - 100000
session.cache_limiter - no value
session.gc_maxlifetime - 1440
soap.wsdl_cache_limit - 5
opcache.max_accelerated_files - 10000
opcache.max_file_size - 0
opcache.max_wasted_percentage - 5

Can others reproduce this? I can't as reported – I'm wondering if it's because I'm on PHP 5.6 and this is a PHP 7 environment?

I've tried this in PHP 5.5 and PHP 7 and couldn't reproduce either.
Might be worth Jessica testing it again in case it was some sort of one-off javascript error that was killing the initialization of the page selector (or checking the console for JS errors).

Ok, I retested on a new install and I could not reproduce the error. Everything worked.  sorry

Thanks for testing again @jessicadunbar :+1:


How to enlarge the image size automatically rather than zoom handly？

Label Studio version: 1.5.0
I get the image like this

and I want to make the image full of screen automaticlly like this (below example is zoomed handly)

I try to set "width" and "maxwidth" like #1146 , but it seems didn't work, there is my setting
<View>
  <Image name="image" value="$imgage" width="1000%" maxWidth="1500px"/>
  <RectangleLabels name="label" toName="image">
    <Label value="Airplane" background="green"/>
    <Label value="Car" background="blue"/>
  </RectangleLabels>
</View>

Hi! We finally fixed this issue and also improved zoom control. You can check this in develop.
Just be sure that fflag-feat-frontend-dev-2504-image-enhancements-zoom-center is enabled.

We will release some fixes tomorrow regarding fflag-feat-frontend-dev-2504-image-enhancements-zoom-center flag, so I would recommend that you wait a little and use the day-after-tomorrow build of LS (from 27 July).


New update isn't published because of the same version

Hey there @xrr2016!
Thanks for merging my PR, but I noticed that the new version isn't published
because the version is the same on the loc and pub.dev site.
Here are the publish logs from your GitHub action.
Local version: [1.3.3]
37
Remote version: [1.3.3]
38
Remote & Local versions are equal, skip publishing.

Updated.


Put "style" parameter on write_report

Allow user to apply different styles to each call to write_report.

Need to think about this a lot.

Added add_style function to report.


EME Clear Key Demo

Hi Sam,
Thanks for nice article! https://www.html5rocks.com/en/tutorials/eme/basics/
I am able to follow the instructions and able to encrypt the video using webm_crypt with some key and later I am using same key for decryption & video playback, the encrypted is getting fired, but decryption is not happening successfully.
my question is once we generate the key - how we are converting that to following uint8Array format?
if I convert the encryption key to base64 format & use it, it still does not work. Please help with your inputs/suggestions.
// Define a key: hardcoded in this example
// This corresponds to the key used for encryption
//var KEY = new Uint8Array([
//  0xeb, 0xdd, 0x62, 0xf1, 0x68, 0x14, 0xd2, 0x7b,
//  0x68, 0xef, 0x12, 0x2a, 0xfc, 0xe4, 0xae, 0x3c
//]);
Thanks,
Kiran

Thanks Kiran.
I'm not working so much on EME stuff now, so just checking for the best answer to your question.
Will keep you posted...

@Kiranmyway Did you manage to get this working? I'm looking at the same thing and wondering where I'm going wrong trying to playback an MP4 file encrypted with CENC


[NTR/ACD] ACD-related telomere biology disorder

Preferred gene-related syndrome label
ACD-related telomere biology disorder
Synonyms
ACD-related telomeropathy
Parent term (use OLS, or your favorite ontology browser)
MONDO:0100137 (telomere syndrome)
Definition
A complex set of inherited conditions caused by a pathogenic variant(s) in the ACD gene that results in aberrant telomere biology.
**Definition source (Please give PubMed ID, if applicable, in format PMID:#######)
PMID: 20301779
Children terms (if applicable) Should any existing terms be re-classified as children underneath this new proposed term?
ACD-related short telomere syndrome (not currently existing. will also need a new term created-happy to submit a separate request if needed)
Definition: A spectrum of conditions, including dyskeratosis congenita, Hoyeraal-Hreidarsson syndrome, hereditary aplastic anemia, and pulmonary fibrosis, typically characterized by shortened telomeres due to a pathogenic variant(s) in ACD that results in impaired telomere maintenance.
Child terms under this new term: MONDO:0014690 (Dyskeratosis congenita, autosomal dominant 6), MONDO:0800370 (dyskeratosis congenita, autosomal recessive 7), MONDO:0018045 (Hoyeraal Hreidarsson syndrome), MONDO:0001713 (hereditary isolated aplastic anemia)
ACD-related long telomere syndrome (not currently existing. will also need a new term created-happy to submit a separate request if needed)
Definition: A telomere biology disorder typically characterized by increased telomere length due to a pathogenic variant in ACD
Child terms under this new term: MONDO:0018961 (Familial melanoma)
Your nano-attribution (ORCID) or URL for a working group
https://orcid.org/0009-0007-0138-2054
I've added a screenshot below of the proposed tree that the GCEP envisions. Please let me know if you are not able to see it. It's just showing up as text for me, so I'm happy to send over email if that would be helpful and works better.

This request is being made on behalf of the ClinGen Interstitial Lung Disease GCEP.
@courtneythaxton
@nicolevasilevsky
@ahamosh

@awaddell01 I created the three new terms.
It doesn't make sense ontologically to classify MONDO:0018961 (Familial melanoma) as a child of ACD-related long telomere syndrome because this would be interpreted as ACD is the only gene implicated in familial melanoma and that is not the case (there are 11 genes according to Orphanet.)
This is also the case with MONDO:0018045 (Hoyeraal Hreidarsson syndrome) and MONDO:0001713 (hereditary isolated aplastic anemia) as children of 'ACD-related short telomere syndrome'. There are other genes besides ACD that are implicated in these diseases.
I added MONDO:0014690 (Dyskeratosis congenita, autosomal dominant 6) and MONDO:0800370 (dyskeratosis congenita, autosomal recessive 7) as children of 'ACD-related short telomere syndrome'
Here is a screenshot of the hierarchy:

I'm happy to have a call to discuss further if there are any questions.

@nicolevasilevsky That makes sense. Thanks so much for doing this! For ACD-related long telomere syndrome, could we possibly edit the definition to include familial melanoma since it can't be a child term? Something like "A telomere biology disorder typically characterized by increased telomere length due to a pathogenic variant in the ACD gene that may cause familial melanoma"

Hi @awaddell01 - sure thing, I will edit the definition.


Fix: Add missing karma service and configuration

Karma was partially missing from the prometheus_server role.
karma.yml file is from @sobrase

Important: the current "defaults" variables are for a single alertmanager to monitor.
In the futur, if we desire to be able to gather more alertmanagers from karma, a small update of the role will be needed.


Add bash-completion

/etc/bash_completion.d
Autocomplete for kafka-topics and kafka-console-consumer. It also completes properties
For when we are interactive within the docker
docker run --rm -it --entrypoint bash landoop/fast-data-dev

Added with f3c19fd5975663701c4ffd8b8488cc9a50f68f53.


Additional service checks for ingestion server health endpoint

Fixes

Fixes #2019 by @sarayourfriend
Description

This PR adds checks for Elasticsearch and both databases to the ingestion server health check endpoint behind the check_deps query parameter. This also refactors the database connection into a db_helpers module similar to the es_helpers one that already exists. This helped simplify some of the code and centralize where the database connection logic was being defined.
The failed response has a status field with a list of possible failures due to the number of upstream services to be checked. If there's a better way to structure that, let me know!
Testing Instructions

just i to start the stack
Run curl -L http://localhost:50281/ and verify the output is {"status": "200 OK"}
Run curl -L http://localhost:50281/?check_deps=true and verify the output is {"status": "200 OK"}
Run docker stop openverse-db-1 openverse-es-1 openverse-upstream_db-1 to simulate the resources becoming unavailable
Run curl -L http://localhost:50281/ and verify the output is {"status": "200 OK"} (unaffected)
Run curl -L http://localhost:50281/?check_deps=true and verify the output is {"status": "503 Service Unavailable", "dependencies": {"es": ["Elasticsearch could not be reached"], "db": ["Database connection for 'upstream' could not be established", "Database connection for 'api' could not be established"]}} (error messages shown)

Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Thanks for the suggestion @dhruvkb! I updated the response to the following, since the presence of the dependency in the response should indicate whether or not it's OK:
interface Response {
  status: "200 OK" | "503 Service Unavailable"
  dependencies: { // only if `check_deps=true`
    es: string[], // only if es problems
    db: string[], // only if db problems
    // ...
  }
}


Metadata API: Make consistent_snapshot optional

Fixes #1374
Description of the changes being introduced by the pull request:
From chapter 7 in the spec (version 1.0.17)
Finally, the root metadata should write the Boolean
"consistent_snapshot" attribute at the root level of its keys of
attributes.
If consistent snapshots are not written by the repository,
then the attribute may either be left unspecified or be set to the
False value. Otherwise, it must be set to the True value.

We want to make sure we support repositories
without consistent_snapshot set.
Signed-off-by: Martin Vrachev<EMAIL_ADDRESS>Please verify and check that the pull request fulfills the following
requirements:

[x] The code follows the Code Style Guidelines
[x] Tests have been added for the bug fix or new feature
[x] Docs have been added for the bug fix or new feature

It's not a matter of choice where the consistent_snapshot should be.
It's not allowed to have arguments without default values after arguments with:

Will update clarify in the commit that this is a breaking change.


MakeRedisAuthStore Implementation

New dependencies:

cache-manager
cache-manager-redis-yet
cache-manager-ioredis-yet

Goal:
Enable Redis to be used natively to save instance keys and credentials.
Way of use:
import { redisStore } from 'cache-manager-ioredis-yet'
import { makeRedisAuthState } from '@whiskeysockets/baileys'

Start connecting to Redis
const redisInstace = await redisStore({
     host: '<IP_ADDRESS>',
     port: 6379,
     password: '',
})

Instantiate the makeRedisAuthState function and inform the Redis connection instance and a key referring to the WPP instance
const { state, saveCreds, clearState } = await makeRedisAuthState(redisInstance, key)

Little remark here. Redis is not good option for heavy whatsapp accounts. Especially for a lot accounts on a single redis node. It is verified on my personal experience.

What do you suggest in place of Redis? I have 700 instances connected and I never had a problem, but I would like to know what you suggest that could be better than Redis for me to study in case of any problems?

Little remark here. Redis is not good option for heavy whatsapp accounts. Especially for a lot accounts on a single redis node. It is verified on my personal experience.

What do you suggest in place of Redis? I have 700 instances connected and I never had a problem, but I would like to know what you suggest that could be better than Redis for me to study in case of any problems?

Little remark here. Redis is not good option for heavy whatsapp accounts. Especially for a lot accounts on a single redis node. It is verified on my personal experience.

What do you suggest in place of Redis? I have 700 instances connected and I never had a problem, but I would like to know what you suggest that could be better than Redis for me to study in case of any problems?

I'd recommend move to mongodb

If you save and maintain all the keys in Redis, it won't be optimal because Redis will take longer to perform queries. Samuel's approach is to set a 24-hour expiration time for keys other than "creds", thus avoiding having multiple keys in Redis.
let ttl: number | undefined = 86400 // 24 hours
if(file === 'creds') {
	ttl = undefined
}

This way, keys other than "creds" will have a 24-hour expiration time, while the "creds" key will have no defined expiration time.

Be careful, deleting the keys have it's own down side. I have ever did what you suggest to just because i don't like to save few GB of keys data. But it got me hard. Incoming message will need more time to be decrypted first every the period (in the suggeston, every 24 hours). Some incoming messages would be failed/unable to decrypted due to invalid something or no session (sorry forgot the exact reason) especially from IOS and sending message could take longer to failed. Sometime even return the id when sending mesage but not got delivered in the recipient's phone. I personally wouldn't mess again with the keys.
All keys should be stored in the database so it could be loaded only when needed. Only after doing that, clearing the data periodically from redis should be okay as it will be loaded again from database when needed.

Be careful, deleting the keys have it's own down side. I have ever did what you suggest to just because i don't like to save few GB of keys data. But it got me hard. Incoming message will need more time to be decrypted first every the period (in the suggeston, every 24 hours). Some incoming messages would be failed/unable to decrypted due to invalid something or no session (sorry forgot the exact reason) especially from IOS and sending message could take longer to failed. Sometime even return the id when sending mesage but not got delivered in the recipient's phone. I personally wouldn't mess again with the keys.

All keys should be stored in the database so it could be loaded only when needed. Only after doing that, clearing the data periodically from redis should be okay as it will be loaded again from database when needed.
Thanks for the suggestion, I'm going to create a parameter that lets the user define how long he'd like to keep the keys and default to 24 hours.  if the user does not set the key expiration time, the default will be 24 hours.


feat(pie-textarea): DSW-2132 add textarea form functionality

Describe your changes (can list changeset entries if preferable)
[Added] - textarea component form functionality
Author Checklist (complete before requesting a review)

[ ] I have performed a self-review of my code
[ ] I have added thorough tests where applicable (unit / component / visual)
[ ] I have reviewed the PIE Storybook PR preview
[ ] I have reviewed visual test updates properly before approving
[ ] If changes will affect consumers of the package, I have created a changeset entry.
[ ] If a changeset file has been created, I have used the /snapit functionality to test my changes in a consuming application

Reviewer checklists (complete before approving)
Reviewer 1

[ ] I have reviewed the PIE Storybook PR preview
[ ] If there are visual test updates, I have reviewed them

Reviewer 2

[ ] I have reviewed the PIE Storybook PR preview
[ ] If there are visual test updates, I have reviewed them

/test-aperture

/spapit

/snapit

This is looking great @leksaBoiko!
One thing: I've discovered a bug which I think needs to be fixed in a follow-up PR.
Scenario: The textarea is focused inside a form, when you press the enter key
Expected behaviour: A new line is added to the text inside the textarea
Current behaviour: The form is submitted

/snapit

/snapit


Block: Social media embed

So far I have seen Twitter and Facebook embeds. The links are missing from the import.

such as Twitter on this page https://main--sling--aemsites.aem.page/whatson/sports/combat-sports/stream-bkfc-48-live (live page at https://www.sling.com/whatson/sports/combat-sports/stream-bkfc-48-live)

And these Facebook embeds https://main--sling--aemsites.aem.page/whatson/sports/wrestling/stream-aew-revolution-live (live page at https://www.sling.com/whatson/sports/wrestling/stream-aew-revolution-live)
https://main--sling--aemsites.aem.page/whatson/sports/college-football/rece-davis-college-gameday-interview
(https://www.sling.com/whatson/sports/college-football/rece-davis-college-gameday-interview)
https://www.sling.com/whatson/sports/college-football/noah-eagle-interview
https://www.sling.com/whatson/sports/nba/doris-burke-interview
https://www.sling.com/whatson/sports/soccer/stream-premier-league-live-sling-tv
https://www.sling.com/whatson/entertainment/comedy/stream-dave-season-3

also we need to know which other social media posts should be able to be embedded, and how to handle them.

Update: The sling team confirmed they need Instagram too. Also they have a new embed component which allows them to embed a player from watch.sling.com, but this will be done in another ticket.


[.NET SDK, AADv1] Problems when testing out the AADv1 Auth Sample

I have been following the Microsoft tutorial here and I used "AADv1" connection as sample.
I don't know if anyone has read or done this article demo before, however I feel there are some things not clear in it.
When I run the bot, there are two things don't seem to fully work and they are:

Retrieving the mails from last 30 mins

Displaying logged in user information

Retrieving the mails from last 30 mins
When I type "recents", the bot only displays an adaptive card with the "Here are all of your unread Inbox emails from the last 5 minutes:" message only. There are functions in the source code to show any mail in the last 30 mins and yet nothing appears. Perhaps I'm missing something or misunderstanding?

When I try to write "me" to display my information, the bot emulator only displays "Response code 500 (Internal Server Error) " with "Sorry, my bot code is having an issue". It feels like there is no connection to Azure AD?

The only task the bot is able to do is send an email message on the logged in user's behalf.
AADv1 Root Dialog.
Lines 46 and 66 are the calls to "recents" and "me" respectively.
Functions related to finding recent mails:
Line 95 is private GetTokenDialog CreateGetTokenDialog()
Line 107 for ListRecentMail(IDialogContext context, IAwaitable<GetTokenResponse> tokenResponse)
The SimpleGraphClient class
The web.config file
Anyone know that's up?

Thanks for reporting this, we will investigate.

I would like to update this issue saying that I was able to get "recent" working. I am able to see unread emails from the last 30 minutes.
However I am still having problems when typing "me", still getting the same error.

Have you stepped through the code to see that you are getting an access token? That would be the first thing I'd check because there isn't a lot of error handling in the sample. I'd be curious if you do get an access token if you get an exception when trying to call into the graph api, or if the issue is that you are just not receiving a token.

Yes, I do get an access token to authenticate and I am able to send email or see my unread emails, however the bot doesn't let me see who i am and who i report to

@Rykimaruh can you post the contents of that token in here. I want to make sure the scopes are correct in the token. Perhaps this is just missing one of the scopes?

@Rykimaruh a 6 digit number sounds like the magic code issued from the token service. The Token should be a JWT in the form of:
eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c
You should be able to see it when debugging here:
private async Task ListMe(IDialogContext context, IAwaitable<GetTokenResponse> tokenResponse)
        {
            var token = await tokenResponse;
            var client = new SimpleGraphClient(token.Token);

            var me = await client.GetMe();
            var manager = await client.GetManager();

            await context.PostAsync($"You are {me.DisplayName} and you report to {manager.DisplayName}.");
}

by inspecting token.Token

@ukphillips
Would you like me to send it privately or something? I am not sure if posting the token that gives personal info like email and other things related would be ok in a public forum.
However, do let me know if it's ok or safe to do so, just wanted to be careful of posting info that pay show personal info.

@Rykimaruh fair enough, can you decode it and redact the sensitive information for posting? Really I'm just interested in the claim types and the scopes it contains - both of which should be insensitive data.

{   "typ": "JWT",  }

{
"scp": "email Mail.Read Mail.Send profile User.Read",
}
is t his enough? let me know

i take this was not enough information?

@JasonSowers , yeah I still do. I've yet to receive any additional information regarding as to why the "me" function doesn't work and just gives the error

@Rykimaruh sorry for the delay, I am looking into this today and trying to troubleshoot for you.  Right now I am blocked by a tenant issue, but will hopefully have something for you later today.

@JasonSowers wel, they are two different issues within same bot, not sure if I should consolidate it into one  issue or have them apart to tell the different issue type.

Ok we will reopen this one.
I'm not sure where you are seeing the "Unread email from the last 5 minutes" from. I am seeing 30 minutes everywhere.
in the bot:

in the code :
    Text = "Here are all of your unread Inbox emails from the last 30 minutes:",

    DateTime from = DateTime.Now.Subtract(TimeSpan.FromMinutes(30));

    if (message.ReceivedDateTime.HasValue && message.ReceivedDateTime.Value >= from)

The "recents" command is working fine for me and I cannot find anything that references 5 minutes rather than 30 minutes.  Could you please add more detail here?
For the "me" command, I am seeing the same issue you are.

I wrapped the ListMe function's code in a try catch and got this error:

Code: Authorization_RequestDenied
Message: Insufficient privileges to complete the operation.
Inner error

Could you please add a try catch and confirm you are getting the same error? Thie is all I did:
        private async Task ListMe(IDialogContext context, IAwaitable<GetTokenResponse> tokenResponse)
        {
            try
            {
                var token = await tokenResponse;
                var client = new SimpleGraphClient(token.Token);

                var me = await client.GetMe();
                var manager = await client.GetManager();

                await context.PostAsync($"You are {me.DisplayName} and you report to {manager.DisplayName}.");
            }
            catch (Exception e)
            {
                Console.WriteLine(e);
                throw;
            }
        }

I will investigate this further. As this should be working out of the box as per the walkthrough instructions.

If you update your AADv1 app to include the read all user' basic profiles  permissiong the "me" command should work as well.

@JasonSowers  i will check it out and update as soon as I can

@JasonSowers
Update: So I tested the new pushed version and I can confirm that the cached token does work correctly. I tested with asking to send email or check recent emails after initial sign in and it doesn't ask me for sign in again as it is recognizing previous token cached.
This is more of a question, but because I am in wifi (does the cached token stay as long as my O365 session? For example I have it that i only can last 8 hours before I have to re-sign in to my O365 account using MFA. Is the token lifecycle similar? I'm assuming that if I'm connected with SSO in my workspace LAN it won't require even a login).
On the other hand, I tried the "me" command it still gives me the same error. The message is the same: not enough privileges for some odd reason. Does it have to with using emulator and not some "authenticated" channel?
I used fiddler to determine the exact error message and it's this:
{"Message":"An error has occurred.","ExceptionMessage":"Code: Authorization_RequestDenied\r\nMessage: Insufficient privileges to complete the operation.\r\n\r\nInner error\r\n","ExceptionType":"Microsoft.Graph.ServiceException","StackTrace":"   at Microsoft.Graph.HttpProvider.<SendAsync>d__19.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown --.
Upon closer inspection it makes me think it has to do MS Graph. Could it be the App registered in the AD?

I did 100% of my testing in the emulator, so this will all work completely fine in the emulator.  I will hae to ask about the token timeout.
Yes the AAD app needs this permission added to it: read all users' basic profiles

make sure to grant permissions after you save:

After you have enabled this permission, make sure to sign out of the sample so it forces you to sign back in.  You can sign out by simply typing signout

Glad we could get this working for you.  I have also had the docs team submit an update to the walkthrough you were using to include that permission in the list of permissions.  Hopefully, it will help others.  Apologies that it took so long to get a response from us, thank you for being so patient.

@Rykimaruh For tokens,  when you sign in you get 2 tokens a sign in token and a refresh token.  The refresh token duration is set by your tenant admin, so I am unable to tell you how long that will last.  The sign in token lasts 1 hour but will be refreshed without having to sign in again while your refresh token is still valid.


Assertion Failure in pty.node

I get the following exception when running tasks:

Setup:

unzip folders.zip
create a workspace with the folders empty2 and empty2

Steps:

Tasks>Task2 (continue without scanning)
Tasks>Task4 (continue without scanning)

The following output shows up in the terminal and then the assertion failure dialog appears

->Assertion Failure

Duplicate https://github.com/Microsoft/vscode/issues/31782