Enable Biological Consumers to select a version of the Cell Census as basis data in Gene Expression

Description: Ability for users to select different versions of the Cell Census from GE. Software versioning is crucial because it makes it easier for our users and software developers to keep track of the many releases that we will make. Users want a logical manner to understand when and what updates are made, and they rely on us to keep them up to date. Currently, our process is to release a new version of the census every 6 months. Therefore versioning each census release will be crucial for users in between cycles, publications, and collaborating with other users etc.
Full Product Options:
Ability to select different versions in a dropdown within GE
Documented version changes included in each census version should be in a change log
Priority:
Must Have
User Segment:
Computational Biologists
Expert Users
Experimental Biologists
First Time/Naive Single Cell Biologists
General Genomic Researchers
Dependencies
Updating GE API for census version selection in collaboration with Data Team
Risks
Need to decide with the Data Team and other stakeholders which type of versioning practice we want:
Semantic (SemVer)
Date of Release
Sequential Numbering
Hybrid
UX/UI Wireframes:
Not available
Design Sizing Effort
Design- small
UXR - none
Eng Sizing Effort
XS
Timeline
Delivery in Q2 2023

We did some discovery already and came up with this document https://docs.google.com/document/d/1QnsBbo4c7hCjIGy2It6sLIru7UEHmQp6SJfJvs9SFcg/edit#heading=h.deg5w3fvyfym
That said, due to the complexity of reproducibility and the requirements we might need to refine, we are removing this requirement for the half.


mock locations from "Fake GPS JoyStick" app are not being rejected

My device is an LG G3 with Android 5.0
Were you aware of this issue?
Thank you for your work!

Hi @shadowpt, thanks for pointing this out. I installed the app and can confirm the locations manage to sneak by the mock detection. This seems to be due to an exploit that allowed apps even on non-rooted devices to hide the mock flag:
https://pokemongohub.net/android-7-1-security-update-disables-gps-spoofing-pokemon-go/
As part of an Android security patch this exploit has been fixed in Android 7.1. and according to the article might be fixed in updates for older Android versions.
https://android.googlesource.com/platform/frameworks/base/+/d22261fef84481651e12995062105239d551cbc6
I am not sure how many "old" devices these updates will effectively roll out to, but the problem will (slowly) heal over time...
The LocationAssistant relies on the mock flag being set at least for some locations, so there is not much we can do for devices that are either rooted or make use of the exploit. In theory, one could implement a more advanced logic to detect fake locations (compare locations from the WiFi / non-GPS providers against possibly fake GPS location readings), but this would mean building a full-blown location fusion logic from scratch and would go way beyond what I intended the LocationAssistant to do.
I'm closing this issue for now. If you have suggestions for a possible fix or want to investigate deeper, feel free to comment / reopen it.


[Bug] Keyboard issue on mobile

Describe the Bug
Keyboard doesn't work correctly (or at all) on android phone (redmi 8). Wont type in chat or and wont type in username field.
Steps to Reproduce
Start launcher, click keyboard button try to type and see it works broken or dont work at all.
Additional Information
Browser google and chrome. Latest version available from google play market. Phone redmi 8 using google keyboard and android version 10, MIUI 12.5.3 (stable version)

idk if it will work i was the one who added the keyboard button and i only have iOS so i only tested it there

Like keyboard opens but with it also whole game moves down, and secondly it wont even type so idk. ChatGPT says that it technically muss work but it doesn't.

moves the game down? if its moving the game out of the way for the keyboard then thats probably built into the os/browser.

Yeah but it also makes like idk white gap between keyboard and game it can be also bug of this js feature idk but anyways it isnt such critical bug the most critical that it wont type at all

But the keyboard opens right?

It opens keyboard but it wont type leme attach photos

Also new bug button goes off screen before it wasn't happening and I typed "Bello" and it didn't wrote as you see

I have several ideas I can try, but support is limited since we currently don’t have any android devices to test with.
Several ideas:
[ ] Test if changing input opacity from 0 changes anything
[ ] Add type="text" to input element
Also, this may be an issue with the Chrome browser on android (see https://github.com/nosir/cleave.js/issues/147)

I mean we could test in android studio just running a browser

Okay, I hope y'all will be able to fix it. By the way you can emulate android using BlueStackX or smth

I mean we could test in android studio just running a browser

Also good option

I just quickly added both ideas, holler if it makes a difference

Leme test it rq

Now buttons are a bit out of their places
And Now I can't close keyboard using button it keeps appearing after closing it using < or button

Oh wait forgot to rotate phone my bad leme test it rotated

and the button doesn't close it it only opens it

I need to do some more research on this. I’ll get back to you.

Ok so. It still bugged and buttons are out of screen. Secondly its impossible now to close keyboard beacuse it loops opens. Thirdly it wont type like it was. White gaps became smaller but still exist

and the button doesn't close it it only opens it

Then we need add for mobile android devices close button beacuse it loop opens it

Well by what i have tested its impossible to close the keyboard with js so idk

Then uh need unloop it somehow, anyways but it still wont type its also critical bug now there is 2 critical bugs

okay...

I think now its kinda chrome issue fully..

let me test it on chrome on ios and see if i get the same issues

no i dont get the issue but its probably because apple makes google use webkit on ios chrome but on android im guessing it uses chromium?

Yeah it definitely uses Chromium ig. Anyways it still happens probably even android bug idk. The fact keyboard is hella buggy is fact

try using another browser that has another engine like firefox and see if it happens there

Same issue

hmmmmm

No clue it muss be fixed somehow... there is a lot more android users than iOS users so yeah

yea

Alright I have a solution but it will take me some time to fix regardless.

Looping open keyboard
This flaw appears to have been here since the button was added. I’ll have to keep track of the keyboard state and toggle between open and closed, which is fairly easy to implement.
Android key presses
It appears that we can’t trust android to dispatch the proper key events. Eagler needs key, keyCode, and which. My current idea is to completely eliminate android’s keyboard events. Instead, we read changes to the hidden keyboard input and dispatch the event programmatically ourselves. Here’s the general outline of what needs to change

[ ] override key event completely to include an extra don’t ignore flag.
[ ] Update all script key events to include the don’t ignore flag
This allows us to disregard any keyboard inputs
[ ] add an event listener to the keyboard input that has an onchange event listener.
[ ] When the event listener fires, we read the new value from the input, pass it through to our normal keyEvent function, and then clear the input
We also need to allow there to be a character within the input so we can detect the delete key press through the length of the input decreasing.

This would eliminate the need for constantly implementing changes to jump through each devices requirements for reading the key events, and feels fairly straightforward to implement.

Yeah I would have made the keyboard button do that and track the state but it just worked for me when clicking the done button on the top of the keyboard it would be fine so that's why I didn't

android doesn't have such button

i fixed the looping check my pull request

oh and completely off topic but i've found by using shadow client i can get at least 30 fps so i think thats the next thing to try to make it automatically optimize the settings

completely off topic thing was useless because with this client im getting 190+ fps on pc and 40-60+ fps on mobile

well good for you i was just saying from my personal experience

Y’all keep it on topic in the issues. I can open up discussions if you really want to do that

okay sorry but can you?

Anyways lets move to issue. Any progress on fix?

no not really

(for me)

Actually, I could use all of your help. If anyone has an android device, could they go to https://w3c.github.io/uievents/tools/key-event-viewer.html and type “e”, and post a screenshot of the result here? I’m close to a solution but I want to make sure it will work.

i dont have an android device but here's it in bluestacks

Here is android device

any updates?

Oh wow, that’s not good. Most of  your UIEvents are undefined, and that makes this extremely hard to work with. This is what it ideally should be closer to:

uh then its android issue

we kinda cant do anything about that so kinda this project is broken for android

If someone can figure out how to detect android devices, I’ll start working on the bloated solution.

ik how to do it leme send u code rq

function getDeviceType() {
var userAgent = navigator.userAgent || navigator.vendor;
if (/android/i.test(userAgent)) {
return "Android";
}
if (/iPad|iPhone|iPod/.test(userAgent) && !window.MSStream) {
return "iOS";
}
if (/Win|Mac|Linux/i.test(userAgent)) {
return "PC";
}
return "Unknown";
}
console.log(getDeviceType());
this muss work i hope

yeah but it still wont fix for everyone

If someone can figure out how to detect android devices, I’ll start working on the bloated solution.

If android doesn’t support the basic Keyboard Events, it means that any project that detects key presses will be broken. Looking at other forums, some android users have recommended this: https://play.google.com/store/apps/details?id=org.pocketworkstation.pckeyboard&hl=en

anyways need try to fix that issue and not be like "other projects"

Just do it via the user agent

???? you mean about detection of android yeah can use that too

To detect android

yeah i literally gave code above how to do that

function getDeviceType() {
var userAgent = navigator.userAgent || navigator.vendor;
if (/android/i.test(userAgent)) {
return "Android";
}
if (/iPad|iPhone|iPod/.test(userAgent) && !window.MSStream) {
return "iOS";
}
if (/Win|Mac|Linux/i.test(userAgent)) {
return "PC";
}
return "Unknown";
}
console.log(getDeviceType());

Oh sorry I didn’t see

by the way why we cant use keydown or input? It will probably fix bug

When android dispatches keydown events, everything is either undefined or zero regardless of what key is pressed. I’ll have to use input which will make us lose a lot of events such as shift keys

leme see i saw on stackoverflow translator of keydown thingy for android

https://stackoverflow.com/questions/30743490/capture-keys-typed-on-android-virtual-keyboard-using-javascript <- here check this out

I think I managed to get it working. Can someone check https://github.com/FlamedDogo99/EaglerMobile/tree/keyboard-fix that has an android device and report back?

Idk if it works on android but I know it messes it up on iOS so you might need to add a detection to use the old method for iOS and the new one for android

Idk if it works on android but I know it messes it up on iOS so you might need to add a detection to use the old method for iOS and the new one for android

Before I merge I'll definitely do that. Right now I'm just wanting to make sure I'm not wasting my time with this attempt.

Yea and that new commit you just did fixed it but now you can clearly see the input in the corner

leme test it on android

Yeah, I just hid the input again, we'll see what happens

write here when u fixed and ill test it

Alright it should (in theory) work. Currently the space key creates a "_" but that can be fixed after we make sure this works.

backspace works strange i need open and close keyboard to do single backspace

testing ...  but typing works fine looks like just backspace broke but other stuff fixed

i got error after setting name leme ss it and send from realplurr

Aka, it fixed keyboard but backspace works strange (can be used only one time other wise need reopen keyboard), and this update broke server list

I have no idea how you have a browser that doesn’t support ES3, but I guess I can I can try and add a polyfill method for apply and call functions. How long has is been since you’ve updated your phone?

Alright I added polyfill functions for apply and call, if someone can test https://raw.githubusercontent.com/FlamedDogo99/EaglerMobile/keyboard-fix/eaglermobile.user.js, I would be greatly appreciative

I have no idea how you have a browser that doesn’t support ES3, but I guess I can I can try and add a polyfill method for apply and call functions. How long has is been since you’ve updated your phone?

Like im on stable release of MIUI, means its release of MIUM (redmi/Xiaomi OS based on android)  what has got nearly everyone who isnt in beta testing programm

Alright I added polyfill functions for apply and call, if someone can test https://raw.githubusercontent.com/FlamedDogo99/EaglerMobile/keyboard-fix/eaglermobile.user.js, I would be greatly appreciative

Aka, it fixed keyboard but backspace works strange (can be used only one time other wise need reopen keyboard), and this update broke server list

For some reason the added function that ignores keyboard events was causing that issue for you, although I have no clue why

Bug with keyboard backspace still exist

can someone else fix this then?

Someone else is welcome to try. If they can figure out why your specific device doesn’t handle EventTarget.prototype.addEventListener the same as most other devices, they’d probably be able to help. However without this function (or something similar) you’ll not be able to type in inputs (which is needed because your device just doesn’t work with keydown events), and even if you could type, I’m pretty sure you’d get repeated key presses randomly.

Alright, this is my last attempt on this particular solution for the android keyboard problems. It works on Safari, macOS chrome, and even Orion, so if it throws an error on your device, I’m out of ideas.
https://raw.githubusercontent.com/FlamedDogo99/EaglerMobile/keyboard-fix/eaglermobile.user.js

Testing...

I think its beacuse android doesn't understand "fn"? All other errots contained fn and here to.

It throws error in multiplayer tab mostly

Wait it does this without you even typing?

Hi yeah managed to recreate it in other pages too, anyways yeah it crashes without any interaction i guess i explained really bad. My bad, I'll wait for a fix and sorry that I explained without big ammount of details :)

I think I fixed it. Currently anything besides a-z, A-Z, 0-9, and the delete key will probably not work, but if there’s no crashes that will be easy to fix.

I think there’s a syntax error in there somewhere, so if it looks broken it’s my fault. I’ll fix it this afternoon

Okay testing updated version

New issue, typing works fine in profils page but when it goes to server (chat) you cant select chat line and / or type there

But yeah it fixed error and I managed to join server ^^^

But yeah it fixed error and I managed to join server ^^^

That’s a step in the right direction then.

New issue: typing works fine in profile page but when it goes to server (chat) you can’t select chat line or type there

I’ll look at it closer Tom

Out of curiosity, when you go to chat is there a blinking "_" that displays where you would be typing?

Hello im back I'll test in minutes

Wait where is latest code just it shows yesterday at all js codes if you mean at all version no its not blinking _ if you about new version idk where it is located at

I'm going to close this issue and open a new one specifically for not being able to type in chat on android, since we have solved the problem with not being able to dispatch valid keyboard events.

I just realized that this is not fully fixed with some Android devices, although I think got it working on the input-fix branch.


Fix GitHub Casing

Hello @varghesejose2020,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Please make sure to test your changes locally or with the preview environment. Here's what I see in the preview:

Changes updated @cwarnermm

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@cwarnermm

@cwarnermm  changes updated


How i can get Token was used?

Can you provide more context and clarify your question please ? thanks!

I wanna get 'usage': {'prompt_tokens': 56, 'completion_tokens': 31, 'total_tokens': 87}, but it alway return null

Hi @ziuzangdev, please check my response here


Winter 2020 updates

Goals:

Initially comment out the forcing of sign in code. Students can opt in to this behavior by un-commenting it if they want it. Thoughts?

Make the naming of routes more consistent between draft:resource and draft:account while still being appropriately descriptive of each.

Switch from match back to get and post.

Switch from the _from_path and _from_query suffixes of parameters to path_ and query_ prefixes; I believe this is a bit more clear because the former only makes sense when you're in the controller fetching, after the values have been placed into params. When you're initially labeling a flexible route or naming an input, it seems confusing why we're calling them e.g. "from query" when they're not in the query yet, they're in the form.
Perhaps we should instead consider naming these things _from_form (not strictly accurate if the query string was typed or e.g. API request) and from_uri. Is that better than query_ and path_?

@raghubetina I generated a resource and account to test and wrote some feedback here:
https://github.com/jmw686-appdev/test-draft-account-gen/pull/2

@jelaniwoods Tyvm. I made some changes; can you take another look?

@raghubetina I generated another resource and account. The PR is here.

@raghubetina Everything looks good to me.  I generated a User and another table and tested all routes and sessions for the user.  Here is my workspace.
https://github.com/pmckernin/test_draft_resource

@raghubetina is it okay if I merge this into master?

Sure, thank you! Don't remember how this slipped through.
On Fri, Oct 21, 2022 at 12:19 PM Jelani Woods @.***>
wrote:

@raghubetina https://github.com/raghubetina is it okay if I merge this
into master?
—
Reply to this email directly, view it on GitHub
https://github.com/firstdraft/draft_generators/pull/75#issuecomment-1287238563,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGGBUOQ5NWHJDRCG7CWXR3WELGDRANCNFSM4KW6JOMA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Suggestion] Make text look better for mobile?

If you can, I'd suggest moving the logo and menu button both to the right, and make a sign or something that says "turn landscape for faq section" and maybe make the

look a little formatted and a little nicer. That should be all for mobile. :smiley:

Mobile layout is good enough unless there are some major issues other than just not pretty enough text, or not enough pixels in X direction, etc. I don't expect a large mobile audience downloading and burning Twister via their phones. As such is the case, the mobile layout mainly exists only as a brief information piece, landscape mode people ... (most people at least) ... should know to turn their phones for a different view be it a desktop mode or a landscape targeted layout offering more. I have brought text and img elements in closer that this screenshot shows though to attempt to make it 'nicer looking'.


Error while deploying the application to Cloud Foundry

Hi,
I am an amateur to IBM cloud services. I followed the exact steps given in the Read me file. When I tried to deploy the application in IBM Cloud Foundry (Step 7), I am getting an error message saying that you have exceeded your organization's memory limit.
I have attached a screenshot capturing the error message. Let me know how I can make this work.

Hi,
Replace the manifest.yml(https://github.com/IBM/offline_order_processing_during_pandemics/blob/master/manifest.yml) content with the following content and repeat step 7.
applications:
 - name: order-processing-pandemic
   random-route: true
   memory: 512M

Thank You


Add noindex to taxonomy pages

Currently our taxonomy pages are taking away from the ranking of some of more core pages pages for certain terms. To avoid it, this PR adds the noindex tag to the taxonomy pages so search engines will crawling those pages.
Based on Hugo's docs, taxonomy pages have specific added variables that we can key off to add the tag: https://gohugo.io/variables/taxonomy/#example-usage-of-site-taxonomies

@zchase @sean1588 is this good to merge?

I'd like to talk through this one with Zack and Sean before merging. Will add an item to our sync this week for it.

i added the do-not-merge label until yall figure it out

Chatted through this today, and we will do it, so removed the do-not-merge.

cool, is this good to merge then?


FIOCLEX and FIONCLEX undefined building under MSYS2

git clone https://github.com/libuv/libuv.git
cd libuv
sh autogen.sh
./configure
make
CC       src/unix/libuv_la-core.lo
src/unix/core.c: In function ‘uv__cloexec_ioctl’:
src/unix/core.c:542:25: error: ‘FIOCLEX’ undeclared (first use in this function)
r = ioctl(fd, set ? FIOCLEX : FIONCLEX);
^
src/unix/core.c:542:25: note: each undeclared identifier is reported only once for each function it appears in
src/unix/core.c:542:35: error: ‘FIONCLEX’ undeclared (first use in this function)
r = ioctl(fd, set ? FIOCLEX : FIONCLEX);
^
make: *** [Makefile:1962: src/unix/libuv_la-core.lo] Error 1

Are you using the MSYS shell of the MinGW shell?

MSYS.

Can you try this patch?  You'll need to do the autogen+configure+make dance afterwards.
diff --git a/configure.ac b/configure.ac
index d3787c0..074f00f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -60,8 +60,8 @@ AM_CONDITIONAL([NETBSD],   [AS_CASE([$host_os],[netbsd*],       [true], [false])
 AM_CONDITIONAL([OPENBSD],  [AS_CASE([$host_os],[openbsd*],      [true], [false])])
 AM_CONDITIONAL([OS390],    [AS_CASE([$host_os],[openedition*],  [true], [false])])
 AM_CONDITIONAL([SUNOS],    [AS_CASE([$host_os],[solaris*],      [true], [false])])
-AM_CONDITIONAL([WINNT],    [AS_CASE([$host_os],[mingw*],        [true], [false])])
-AS_CASE([$host_os],[mingw*], [
+AM_CONDITIONAL([WINNT],    [AS_CASE([$host_os],[mingw*|msys*],  [true], [false])])
+AS_CASE([$host_os],[mingw*|msys*], [
     LIBS="$LIBS -lws2_32 -lpsapi -liphlpapi -lshell32 -luserenv -luser32"
 ])
 AC_CHECK_HEADERS([sys/ahafs_evProds.h])

Many errors, starting with:
CC       src/win/libuv_la-async.lo
In file included from src/win/internal.h:29:0,
from src/win/async.c:25:
src/win/winapi.h:4151:6: warning: ISO C90 doesn’t support unnamed structs/unions [-Wpedantic]
};
^
src/win/winapi.h:4157:5: error: unknown type name ‘NTSTATUS’
NTSTATUS Status;
^
Many of the subsequent errors look like they are consequent of NTSTATUS

What is CC?  make V=1 should tell you.  If it's mingw, does make -f Makefile.mingw work?

make V=1
libtool: compile:  gcc  ....
$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-pc-msys/5.3.0/lto-wrapper.exe
Target: x86_64-pc-msys
Configured with: /msys_scripts/gcc/src/gcc-5.3.0/configure --build=x86_64-pc-msys --prefix=/usr --libexecdir=/usr/lib --enable-bootstrap --enable-shared --enable-shared-libgcc --enable-static --enable-version-specific-runtime-libs --with-arch=x86-64 --with-tune=generic --disable-multilib --enable-__cxa_atexit --with-dwarf2 --enable-languages=c,c++,fortran,lto --enable-graphite --enable-threads=posix --enable-libatomic --enable-libcilkrts --enable-libgomp --enable-libitm --enable-libquadmath --enable-libquadmath-support --enable-libssp --disable-win32-registry --disable-symvers --with-gnu-ld --with-gnu-as --disable-isl-version-check --enable-checking=release --without-libiconv-prefix --without-libintl-prefix --with-system-zlib --enable-linker-build-id --with-default-libstdcxx-abi=gcc4-compatible
Thread model: posix
gcc version 5.3.0 (GCC)
Admin@Admin-PC MSYS ~/libuv
$ make -f Makefile.mingw
cc -Wall -Wextra -Wno-unused-parameter -Iinclude -Isrc -Isrc/win -DWIN32_LEAN_AND_MEAN -D_WIN32_WINNT=0x0600 -c -o src/fs-poll.o src/fs-poll.c
cc -Wall -Wextra -Wno-unused-parameter -Iinclude -Isrc -Isrc/win -DWIN32_LEAN_AND_MEAN -D_WIN32_WINNT=0x0600 -c -o src/inet.o src/inet.c
cc -Wall -Wextra -Wno-unused-parameter -Iinclude -Isrc -Isrc/win -DWIN32_LEAN_AND_MEAN -D_WIN32_WINNT=0x0600 -c -o src/threadpool.o src/threadpool.c
cc -Wall -Wextra -Wno-unused-parameter -Iinclude -Isrc -Isrc/win -DWIN32_LEAN_AND_MEAN -D_WIN32_WINNT=0x0600 -c -o src/uv-common.o src/uv-common.c
cc -Wall -Wextra -Wno-unused-parameter -Iinclude -Isrc -Isrc/win -DWIN32_LEAN_AND_MEAN -D_WIN32_WINNT=0x0600 -c -o src/version.o src/version.c
cc -Wall -Wextra -Wno-unused-parameter -Iinclude -Isrc -Isrc/win -DWIN32_LEAN_AND_MEAN -D_WIN32_WINNT=0x0600 -c -o src/win/async.o src/win/async.c
In file included from src/win/internal.h:29:0,
from src/win/async.c:25:
src/win/winapi.h:4154:5: error: unknown type name ‘NTSTATUS’
NTSTATUS Status;
^

I don't think we support the MSYS gcc. We do support MinGW (the w64 variant).  Why are you using MSYS gcc?

Does this patch help?
diff --git a/src/win/winapi.h b/src/win/winapi.h
index 341fcd0..0b87c3f 100644
--- a/src/win/winapi.h
+++ b/src/win/winapi.h
@@ -24,6 +24,9 @@

 #include <windows.h>

+#ifdef __MINGW32__
+#include <ntdef.h>
+#endif

 /*
  * Ntdll headers

I'll second what @saghul said that we don't support mingw32 (and therefore msys-gcc) so if this doesn't fix it, you should switch to mingw-w64.

At some point, some piece of documentation said (or I read it as saying) to use the MSYS shell..
I am a UNIX guy; I know nothing about Windows. I don't know the difference between the two shells.
I have a huge UNIX application that I would like to get running under windows. It was written with portability in mind; i.e. using libraries like libuv  to isolate the OS specific functions, so this should all be feasible.
I would prefer a MinGW like solution that would allow me to cross-compile on my UNIX build engines; also my experiments with VS make me want to hang myself.

Switched to the mingw shell; prompt is now "Admin@Admin-PC MINGW64 ~/libuv"
Restored to unpatched configure.ac, sysgen, configure, make clean, make yields NTSTATUS undefined.
Applied the first patch, sysgen, configure, make clean, make: same error.
Reverted, applied second patch; "patch" rejects it, I think because the git index is for a unknown (to me) branch; applied the patch manually, sysgen, configure, make clean, make: NTSTATUS still undefined.
-- Charles

Hey @charlesUnixPro, let's see if we can sort this out :-)
Yes, you want to use MSYS2, but when compiling stuff you plan on distributing, you want to use the MinGW-w64 toolchain.  It's confusing, I know, but that's the way it is.
I think you are missing the win32api-headers package, which you can install with pacman.  Install it and let's see if that works.

Also, use make -f Makefile.mingw when building with mingw-w64.

With the second patch, sysgen, configure, make clean -f Makefile.mingw, make -f Makefile.mingw: NTSTATUS undefined.
pacman -Q shows:
msys2-w32api-headers <IP_ADDRESS>32.6172d2f-1
pacman -Ss doesn't show any other w32api headers.
-- Charles

I suspect the problem is a broken MinGW-64 install -- the installer gave me a lot of grief, and I may very well have busted something, so I think that should wait until the broken installer gets sorted.
So I'm solving this from the other direction. I have successfully built my application (and libuv) on a UNIX box using the MinGW cross-compiling tools, and am perfectly content with not being able to build on Windows if I can do it from UNIX, as Windows makes my eyes bleed.
I copy the .exe over to Windows and run it: fails with "libuv-1.dll" missing. That makes sense; I need to copy the DLL over to the windows box, but I don't know where to copy it to,
-- Charles

Google for "dll load order", there are some good articles out there.  The directory of the .exe is usually a safe bet.

Copied libuv-1.dll to the cwd...
Operator's console...
C:\Users\Admin\Desktop\QuickStart_MR12.6e>dps8.exe  MR12.6e_boot.ini
DPS8/M emulator (git d496e423)
Production build
Please register your system at https://ringzero.wikidot.com/wiki:register
or create the file 'serial.txt' containing the line 'sn: 0'.

DPS8M simulator V4.0-0 Beta
base_system.ini-54> set absi nunits=1
Non-existent device
! was unexpected at this time.
error: sys_cable: invalid switch name <absi>
base_system.ini-1004> cable absi,0,0,032,0
Invalid argument
SUPER_USER deprecated
TRLSB deprecated
TAPE: unit is read only
FNP telnet server port set to 6180
listening to 6180
Debug output to "STDOUT"
Debug output to "STDOUT" at Fri Dec 09 12:00:35 2016
CONSOLE: ALERT
bootload_0: Booting system MR12.6e generated 04/10/16 2159.2 pdt Sun.
2000.6  announce_chwm: 428. pages used of 512. in wired environment.
2000.6  announce_chwm: 706. words used of 1024. in int_unpaged_page_tables.
find_rpv_subsystem: Enter RPV data: M-> [auto-input] rpv a11 ipc 3381 0a

2000.6  load_mst: 946. out of 1048. pages used in disk mst area.
bce (early) 2000.6: M-> [auto-input] bce

Multics Y2K.  System was last shudown/ESD at:
Saturday, April 16, 2016 16:00:20 pst
Current system time is: Friday, December 9, 2016 12:00:40 pst.
Is this correct? M-> [auto-input] yes

The current time is more than the supplied boot_delta hours beyond the
unmounted time recorded in the RPV label.  Is this correct? M-> [auto-input] yes

The current time I'm using is more than 12 hours
after the last shutdown time recorded in the RPV label.
Are you sure this is correct? M-> [auto-input] yes

bce (boot) 1200.6: M-> [auto-input] boot star

Multics MR12.6e - 12/09/16  1201.0 pst Fri
1201.1  Loading FNP d, >system_library_unbundled>site_mcs 7.6c
Received BOOTLOAD command...
1201.1  FNP d loaded successfully

Volumes to be Scavenged:

   rpv    (root)
   root2  (root)
   root3  (root)

Ready
 1201  as   as_init_: Multics MR12.6e; Answering Service 17.0
 1201  as   LOGIN              IO.SysDaemon dmn cord (create)
 1201  as   LOGIN              Backup.SysDaemon dmn bk (create)
 1201  as   LOGIN              IO.SysDaemon dmn prta (create)
 1201  as   LOGIN              Utility.SysDaemon dmn ut (create)
 1201  as   LOGIN              Volume_Dumper.Daemon dmn vinc (create)
 1201  as   LOGIN              Scavenger.SysDaemon dmn scav1 (create)
 1201  as   as_mcs_mpx_: Load signalled for FNP d.
M-> CONSOLE: RELEASED

 1201  cord Enter command:  coordinator, driver, or logout:
-->  cord
 1201  prta Enter command:  coordinator, driver, or logout:
-->  prta
 1201  vinc
 1201  vinc r 12:01 0.687 31
 1201  vinc
-->  vinc
 1201  scav1 Created >system_control_1>scav1.message
 1201  bk
1201.2  scavenger: Begin scavenge of dska_00a by Scavenger.SysDaemon.z
 1201  scav1 Scavenging volume rpv of logical volume root
 1201  bk   r 12:01 0.613 24
 1201  bk
-->  bk
 1201  ut   copy_dump: Attempt to re-copy an invalid dump.
1201.9  scavenger: Scavenge of dska_00a by Scavenger.SysDaemon.z completed.
 1201  scav1 Scavenging volume root2 of logical volume root
1201.9  scavenger: Begin scavenge of dska_00b by Scavenger.SysDaemon.z
1202.6  scavenger: Scavenge of dska_00b by Scavenger.SysDaemon.z completed.
 1202  scav1 Scavenging volume root3 of logical volume root
1202.6  scavenger: Begin scavenge of dska_00c by Scavenger.SysDaemon.z
 1202  as   sc_admin_command_: Utility.SysDaemon.z: delete_old_pdds
 1202  ut   send_admin_command: Execution started ...
 1202  ut   completed.
 1202  ut   monitor_quota: The requested action was not performed.
 1202  ut   The quota of >dumps is 0, a record limit needs to be specified.
 1202  ut
 1202  ut   Records   Left  %    VTOCEs   Left  %   PB/PD  LV Name
 1202  ut
 1202  ut   166172   99898  60   42218   34090  81  pb     root
 1202  ut
 1202  ut   r 12:02 3.428 494
 1202  ut
-->  ut
1202.7  RCP: Attached tapa_00 for Utility.SysDaemon.z
1202.7  RCP: Detached tapa_00 from Utility.SysDaemon.z
1202.7  RCP: Attached rdra for Utility.SysDaemon.z
1202.7  RCP: Detached rdra from Utility.SysDaemon.z
1202.7  RCP: Attached puna for Utility.SysDaemon.z
1202.7  RCP: Detached puna from Utility.SysDaemon.z
1202.7  RCP: Attached prta for Utility.SysDaemon.z
1202.7  RCP: Detached prta from Utility.SysDaemon.z
CONNECT <IP_ADDRESS>
evHandler WILL 3
CONNECT <IP_ADDRESS> to d.h000
1203.4  scavenger: Scavenge of dska_00c by Scavenger.SysDaemon.z completed.
 1203  as   LOGOUT             Scavenger.SysDaemon.z dmn scav1   1:52 $7.52 (log
o)
 1203  as   LOGIN              Repair.SysAdmin int d.h000 (create)

Telneting in:
HSLA Port (d.h000,d.h001,d.h002,d.h003,d.h004,d.h005,d.h006,d.h007,d.h008,d.h009,d.h010,d.h011,d.h012,d.h013,d.h014,d.h015,d.h016,d.h017,d.h018,d.h019,d.h020,d.h021,d.h022,d.h023,d.h024,d.h025,d.h026,d.h027,d.h028,d.h029,d.h031)?
Attached to line d.h000

Multics MR12.6e: Installation and location (Channel d.h000)
Load = 5.0 out of 90.0 units: users = 5, 12/09/16  1209.3 pst Fri
l Repair

Password:
You are protected from preemption.
Repair.SysAdmin logged in 12/09/16  1209.4 pst Fri from ASCII terminal "none".
Last login 12/09/16  1203.6 pst Fri from ASCII terminal "none".

r 12:09 0.534 19

It works...
libuv rocks!
Thank you all very much for the help and advice.
-- Charles

I'm glad you go it working Charles! Closing this then.

I scrubbed every trace of MSYS and MinGW from my system and reinstalled MSYS2, carefully following the instructions, and am now able to build.
This issue was apparently caused by an installation error on my part.
-- Charles

Good to know! 👍


Mutable histograms?

Like d3/d3-collection#5 but for histograms.

Like d3/d3-collection#31 but for histograms.

(Which makes me wonder whether d3.histogram should be in d3-collection…)

Closing this issue due to inactivity. Please do not hesitate to add a comment or reopen the issue if you think it is still worth pursuing.
The reference would now be https://github.com/d3/d3-hexbin/issues/3 or https://github.com/d3/d3-quadtree#quadtree_add


Survey.Ask always fails under git bash (Mingw64)

Survey.Ask() consistently returns "Incorrect function" when used under the (very common) git bash for WIndows, also known as Mingw64, it fails completely.
I tested under cmd and it worked ok.

Thanks for reporting this @rfay - i'll try to reproduce this on my windows machine and see if I can come up with a solution relatively soon.

Sorry that I haven't dived into the code, but is it possible you're using a OS-level detection instead of terminal capabilities detection? (with 	golang.org/x/crypto/ssh/terminal)
Thanks so much for your work on this, and for the rapid response here. We absolutely love the functionality of survey, it's beautiful, far nicer than anything we've ever worked with. We have a lot of trouble testing it, were unable to get go-expect to work with or without our binary, couldn't figure out why and couldn't figure out how one debugs go-expect.

@hinshun is the guy to ask about integrating with go-expect. Hopefully he can address the issues you are having. I will try to get to the bottom of the terminal issues. We're not using golang.org/x/crypto/ssh/terminal and are just relying on suffixing the name of the files with the build target. This obvously doesn't work to differentiate  Mingw64 from cmd.exe. I'll try to look into this soon to get you a fix ASAP.
Would you mind posting the full list of questions and the call to survey.Ask so I can make sure that your code works as expected.

I opened https://github.com/Netflix/go-expect/issues/5 on go-expect, asking about timeouts, but the bottom line is if something's wrong, you can get a hang in your tests, and they're enormously difficult to debug, or see state, or figure out where to go. If you think more would be useful in this project (new issue?) or in go-expect, I'm happy to write up more. go-expect just (currently) looks like a rathole for maintenance because of these issues.

I think it makes a lot of sense to keep that issue as the go-to place for handling the debugging issue. I think the place where survey is impacted (apart from recommending go-expect as the testing framework) is to ensure that we support Mingw64.

I think go-expect definitely have a long way to go in terms of ease of debugging. Unfortunately because go test hides away your raw os.Stdout, os.Stdin, it's hard to provide a way to interactively continue from where a test is hanging. You can definitely do this if go-expect was running as a regular go binary.
Let's continue discussing improvements about go-expect in its git repository, I'd be happy to brainstorm with you on how we can make it better.

Discussing testing with @hinshun over in https://github.com/Netflix/go-expect/issues/5 he mentioned that go-expect definitely does not currently support Windows.  And I'm going to jump to the assumption that https://github.com/hinshun/vt10x also doesn't support Windows. So then that would mean that survey doesn't have any capability of running tests under Windows either right?

Strange it works on the integrated git bash terminal in vscode but not in the actual ming terminal

Is there any solution for this?

The issue seems to be with MinTTY running the bash command, as starting "c:\Program Files\Git\bin\bash.exe" from the Command Prompt does not have the same issue.

The line that fails is: https://github.com/AlecAivazis/survey/blob/4de88574e78145bdcfa6627ed766564231588b31/terminal/runereader_windows.go#L94

Filed: https://github.com/mintty/mintty/issues/906

Another workaround might be to run the affected app under winpty - I tried this with saml2aws (above linked issue) and it appears to work around the issue. AFAICT winpty is bundled with Git for Windows.

@rfay If testing survey on Windows is still a priority for you: we have managed to enhance Netflix/go-expect, such that it can use conpty on Windows. The result is ActiveState/termtest . It works very well for testing survey driven menus for us.

Thanks so much for letting me know @mdrohmann - Didn't end up using any of these approaches.

When trying to debug interactive applications in Visual Studio Code or IntelliJ (such as saml2aws), the prompts are skipped in the Debug console.

In gitbash
If executed as winpty binary it works perfectly

Authorization flow affected as well.

@AlecAivazis, thanks for your hard work on this. I'm sure it's not a good time to have so many people breathing down your neck.
Is there anything we can do to help?

@AlecAivazis, thanks for your hard work on this. I'm sure it's not a good time to have so many people breathing down your neck.
Is there anything we can do to help?

Probably @dscho is able to give some advice about what exactly needs to be fixed.

I lack the context and the time... :sad:

Seeing the nice mention in Golang Weekly reminded me of survey, which I loved except for this problem. Is this issue any better at this point? I see that the Windows capability is highly promoted in the README now, not sure if it was back then.
The project I work on is cross-platform, so anything has to work so many places: macOS (arm64/amd64), linux (everywhere), Windows (traditional and WSL2). And most people using it in traditional windows will be using git-bash.

Unfortunately, im not too familiar with the Mingw64 terminal and very much lacking in time. If someone wants to take this on, i'm happy to provide any guidance I can

I see that winpty has been suggested as a working work-around. This makes me believe that you could imitate how Git for Windows itself (which also does not have direct access to MinTTY's pseudo terminal) handles this. In a nutshell, it spawns cat >/dev/tty && read -r -s line </dev/tty && echo \"$line\" && echo >/dev/tty and reads its output, which works because the bash.exe shipped with Git for Windows does have access to /dev/tty. Good luck... 😃

It's hard to claim support for Windows when the basic git-bash terminal can't work; so many people in the dev world use that to interact with Windows.

@rfay i understand where you're coming from. The claim to support windows comes from supporting the native windows terminal and cmder. There are so many different terminals in the windows ecosystem its hard to track them all. I understand that git-bash is a very common one which is why I'm offering to help you or anyone else that wants to add support as it would clearly help a large number of users

Yeah, totally understood. And of course I have the same problem and same limited resources and same spread-out-all-over-the-place problem with the projects I maintain. Thanks for all your effort on this! It's a beautiful tool.

Why don't you simply add the suggestion to use winpty ... in Git Bash? That way, you still support Windows, and you do not have to change even as much as a line of code.

@AlecAivazis Any news regarding this issue ?

bubbletea looks like a good and well maintained alternative.


Non-backwards compatible change in moving_averages.assign_moving_average

A zero_debias argument has recently been added to moving_averages.assign_moving_average. zero_debias=True significantly changes the behavior of moving_averages.assign_moving_average, in an unexpected way.
The default value for zero_debias is True. It should be False. The True default breaks existing code (it was breaking Keras), and it also a bad default value on its own because it introduces unexpected behavior (one would expect assign_moving_average to merely assign a moving average, i.e. return the equivalent of an assign_add op. Instead it creates weights, etc).

It's good to discuss this before 1.0 release.  Most people do want debiasing.  We should fix keras to use this function properly or support debiasing as well.

(we should mention this as a breaking change in RELEASE.md)

Is there any update on this since 1.0 is already in alpha now.

Sorry, I'm not sure what the action item is. Keras, in particular, was fixed internally on Dec 14, 2016 and was included in the next rollout. If that was the action item, then this can be closed.
If instead, the action item was to change the default of zero_debias in assign_moving_average to False, then I think more discussion is required, especially since it apparently improves some of the standard implementations of batchnorm, which in turn has statistically significant quality improvements on things like benchmark inception.
I will close this thread, assuming it is the first. If it is not, please reopen, assign to me, and perhaps give more details as to why the default should be moved to false.


Terminal crashed on lunch while accessing system through RDP

Environment
Windows build number: Microsoft Windows [Version 10.0.18362.295]
Windows Terminal version (if applicable): Windows Terminal (Preview) Version: 0.5.2681.0

Steps to reproduce

Install the latest (0.5.2681.0) windows terminal on windows 10 (1903), launch it and it works.
RDP (using window RDP tool) your system from another windows computer
Try to launch the windows terminal, it opens and crashed instantly

Expected behavior
Terminal should work while accessing it through RDP session
Actual behavior
Terminal crashed with below given windows event log entry:
Faulting application name: WindowsTerminal.exe, version: 1.0.1909.25001, time stamp: 0x5d8b1a61
Faulting module name: TerminalControl.dll, version: 1.0.1909.25001, time stamp: 0x5d8b194b
Exception code: 0xc0000409
Fault offset: 0x0000000000052691
Faulting process id: 0x1d3c
Faulting application start time: 0x01d579d25edff7c0
Faulting application path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_0.5.2681.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
Faulting module path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_0.5.2681.0_x64__8wekyb3d8bbwe\TerminalControl.dll
Report Id: 98b3041c-8f29-4d12-a639-8deebd0f828f
Faulting package full name: Microsoft.WindowsTerminal_0.5.2681.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App

/feedback

It's funny -- my terminal instance crashed while I was on lunch also. Seriously.

I also get a crash on startup. It was working fine a few days ago, but now it crashes every time I try to start it. Possibly caused by a recent update to Windows - I installed an update recently.
Currently OS build is: Windows 10 x64 version 1903, OS build: 18362.356

/feedback
This still needs a feedback item, but the bot un-marked it as such. Sorry/thanks!


Fix multiple strokes occuring when multiple clients are attached

attach with two separate socket clients
start a race and simulate a stroke
two strokes occur
only one should

Issue only occurs when connected to localhost (or when ngrok'ing localhost)
does not affect ui.waterbug.site, so I'm closing it


What does this error mean?

Error Domain=com.google.comGDataXML Code=-1 "The operation couldn't be completed. (com.google.GDataXML error -1)" at XMLTools.js (line 1)
I'm using XMLTools to parse my KML files and convert them into an object.

Thanks!


Bumping chart version to 0.5.2

Signed-off-by: Gustavo Carvalho<EMAIL_ADDRESS>

/approve

/merge


tlock-js: design flaws in hash to field function

The plan is to simply use the hash to field function from the IRTF


Convert an existing account to FxA login

We should have a mechanism for a logged in user to convert their old account to an FxA account. This will likely involve logging into the old username/password account and then logging into FxA. This should allow for the user to change the email address stored in olympia.
We should discuss with UX the logistics of this. Should we require a user to log in to their old account to convert it?

There have been more specific bugs filed on this issue for implementation. Closing. See https://github.com/mozilla/olympia/issues/987.


replacements for system_command and system_command_nb

Right now system_command and system_command_nb use os.system and subprocess.Popen(shell=True) respectively.
These both can have some undesirable behavior:

os.system and Popen(shell=True) launch an OS-native shell, which will behave differently between windows and posix, and it's difficult to safely escape shell input.
os.system leaks Talon's file handles to the child, which may cause unexpected behavior in both Talon and the launched app.
Popen(shell=True) may close the launched process when Talon itself closes, which is likely undesirable. This behavior also depends on your platform.
Popen(shell=True) can leak resources inside Talon with each subprocess launched.

I propose improving these in a couple of ways:

Use shlex.split instead of shell=True. shlex will behave the same across platforms, and doesn't do shell expansion of variables and subcommands, so is generally safer.
Use subprocess.run(check=True) instead of os.system(). subprocess.run doesn't leak Talon's file handles, doesn't invoke an extra /bin/sh process, and check=True means you'll get an exception if the subprocess fails, which is usually a good thing.
Use talon's system.launch instead of subprocess.Popen(shell=True). This spawns the process separately from Talon, so it won't close when Talon exits, and won't leak resources.

Please test this on each platform before merging.
from talon import Module, system
import shlex
import subprocess

mod = Module()
@mod.action_class
class Actions:
    def system_command(cmd: str) -> None:
        """Run a command and wait for it to finish"""
        args = shlex.split(cmd)
        subprocess.run(args, check=True)

    def system_command_nb(cmd: str) -> None:
        """Run a command without waiting for it"""
        args = shlex.split(cmd)
        cmd, args = args[0], args[1:]
        system.launch(path=cmd, args=args)

Had a go at this on the branch mentioned above. The implementation is the same as lunixbochs' with an attempt to allow ~ to expand to the user's home directory. There are some issues with it though relative to the current implementation:

shlex.split doesn't work properly with Windows it seems. system_command("WScript C:\\Users\\Administrator\\Desktop\\test.vbs hello") came out of shlex.split as something like ["WScript", "C:UsersAdministratorDesktoptest.vbs", "hello"].
Arguments with quotes and ~ behave differently, for example ~/my-script.sh "~/.bashrc" would previously pass a literal ~/.bashrc to the script. shlex.split doesn't let us find out if an argument was quoted however, so my implementation expands it to /home/my-user/.bashrc.

The first issue I think requires a different parser to shlex.split. This would probably also let us fix the second (more minor) issue.
The following were the set of test cases (with per-platform variants) I was using to verify the behaviour:

Run actions.user.system_command("false") (failing command) and check you get a stacktrace.
Run actions.user.system_command_nb("false") (failing command) and check you don't get a stacktrace.
Run actions.user.system_command("~/my-script.sh ~/.bashrc") and check the ~ are expanded correctly and the shell script is executed if it has the execute mode set.
Run actions.user.system_command("~/my-script.sh 'with quotes'") and  actions.user.system_command("~/my-script.sh '~/.bashrc'")  and check the quotes are handled correctly. Also run the _nb version with same.
Run actions.user.system_command("bash -c \"false || ~/my-script.sh 'with quotes'\"") and check the quotes are handled correctly. Also run the _nb version with same. This is the accepted way of running an ad-hoc shell command.
Run actions.user.system_command("sleep 5") (or equivalent) and check the action doesn't return until the command exits.
Run actions.user.system_command_nb("gedit"), check the gui comes up, and doesn't close when you exit Talon.

FWIW, there only two files that use system_command, which could well use a rewrite for the affected commands anyway:

apps/i3wm/i3wm.talon
apps/dunst/dunst.talon

OK, I ended up writing my own lexer/parser. I've heard of enough people using this action in their scripts that it seems worth implementing. It was also fun to write it ;).

Ok, let me think about this and review it.

Listing out some options for how to proceed with this issue:

Change the action to def system_command(cmd: str, arg1: str="", arg2: str="", arg3: str="", arg4: str=""):. You could then call it from .talon like user.system_command("echo") or user.system_command("echo", "foo") etc. up to four arguments. We could deprecate the existing system by checking if there was a space in the first argument and using the existing os.system path if so.
Don't change the implementation. lunixbochs has identified some theoretical issues with it, but I haven't heard of it being too problematic so far.
lunixbochs makes a 1st party implementation in Talon.
Use my implementation in #1140. Hopefully it's pretty much a set and forget situation; I doubt we'd want to change the behaviour again (unless there are bugs). I am doing some model based testing against shlex.split, so hopefully there aren't many issues.
Deprecate then remove this API, as identified in this comment it's only used in a couple of places in this repo. I think it is used by a fair few knausj users for custom commands though, so we'd be making them all do some work.
Use shlex.split for Unix and a .dll call on Windows.
Try and find a third party pure Python implementation with compatible licensing and copy paste it in to knausj. This doesn't seem like an improvement on #1140 though.

I've sorted them in my order of preference. The fixed args count thing seems kind of neat, and if people want more they probably want to drop in to Python anyway.

One thing we could try is to replace all of knausj's uses with some other, safer API/implementation, and keep but deprecate the existing actions.


how to make a slackbot only respond to DM with itself through the app.message method?

Description
I'm having a problem and I don't know how to deal with it.
I have several bots connected to my Slack that perform different actions. However, what these bots have in common is that people should be able to send messages to them via DM (Open a conversation with the bot), and through these messages each bot should be able to perform something.
Today, to read these messages, I have the following code:
app.message('', async ({ message, say, body }) => {
  if((message.channel_type === "im") && (message.channel.startsWith('D'))){
    const channelInfo = await slack.getChannelData(app, message.channel)
    const userInfo  = await slack.getUserData(app, message.user)

    if ((typeof channelInfo != 'undefined') && (channelInfo !== null)) {
      if((channelInfo.channel.is_im == true) && (channelInfo.channel.num_members == 2) && (channelInfo.channel.is_open == false)){
            await say({ text: "resposta", thread_ts: message.ts});
      }
      else {
          await say({ text: "Sem Resposta", thread_ts: message.ts});
      }
    }
  }
});

The problem I'm facing is that many times the message sent to Bot A is received by Bot B, the message sent to Bot C is received by Bot A, and this way the bot is not able to respond to the message.
My question is, how can I make a bot listen to events of messages received only from its DM, without having to use any words or anything like that. Do you have any ideas?
Reproducible in:
@slack/bolt": "^3.22.0",
node version: v20.13.1
OS version(s): MACOS 14.6.1 (23G93)
Thank you!
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Hi @seratch , thanks for your answer
This is not the case =/
The communications are between a regular user and a bot, but another bot is also able to receive them.
This problem may occur due to some access that my bots have that they shouldn't (To test some features I granted excessive permissions). See the permissions section of my app manifest below.
        "scopes": {             "user": [                 "admin.teams:read",                 "admin.users:read",                 "im:history",                 "reactions:read",                 "team.billing:read",                 "team.preferences:read",                 "team:read",                 "usergroups:read",                 "users.profile:read",                 "users:read",                 "users:read.email",                 "chat:write",                 "mpim:read",                 "im:read",                 "im:write",                 "channels:history",                 "mpim:history"             ],             "bot": [                 "app_mentions:read",                 "bookmarks:write",                 "calls:read",                 "calls:write",                 "channels:history",                 "channels:join",                 "channels:manage",                 "channels:read",                 "chat:write",                 "chat:write.customize",                 "chat:write.public",                 "files:read",                 "files:write",                 "groups:history",                 "groups:read",                 "groups:write",                 "im:history",                 "im:read",                 "im:write",                 "incoming-webhook",                 "links:read",                 "links:write",                 "mpim:history",                 "mpim:read",                 "mpim:write",                 "pins:read",                 "pins:write",                 "reactions:read",                 "reactions:write",                 "reminders:read",                 "reminders:write",                 "team.billing:read",                 "team:read",                 "usergroups:read",                 "usergroups:write",                 "users.profile:read",                 "users:read",                 "users:read.email",                 "users:write",                 "workflow.steps:execute",                 "conversations.connect:read",                 "commands"             ]         }     },     "settings": {         "event_subscriptions": {             "user_events": [                 "message.channels",                 "message.im",                 "message.mpim"             ],             "bot_events": [                 "app_home_opened",                 "app_mention",                 "message.groups",                 "message.im"             ]         },
Regarding the applications, each one has its own code and its own bolt object, but both run on the same VM instance. The code to capture the message from both bots is exactly the same as the one I published in the first message.

@lincoln19922 OK, I figured out the cause. The "message.im" event in "settings.event_subscriptions.user_events" is likely the reason for your situation. Please remove the event subscription from all of your apps. Subscribing to user events means that those apps receive message events for your human account (because you've granted the apps to subscribe to your own message events). This is why you notice that a few apps receive the same message events, even when some of them are not involved in the DM with you.


SQLite - createMany support

Problem
CreateMany is a really useful feature for inserting many rows at once without creating many queries and pushing them into a transaction.
I know the createMany method has been disabled on sqlite datasource manually by https://github.com/prisma/prisma/pull/5389, but I'm not sure about the technical reasoning behind it.
As far as I know (sorry for my naivity) SQLite supports INSERT INTO _table_name VALUES(...) natively, so it should be, in theory, implementable without any horrifying workarounds.
Suggested solution
Make createMany allowed method for sqlite datasources. As for implementation, use INSERT INTO.
Alternatives
I'm currently using $transaction as a workaround. Not terrible, just annoying to not be able to use the same clean syntax like for other datasources.
await prisma.$transaction(
    [...values].map((value) =>
      prisma.tableName.create({
        data: { column: value }
      })
    )
  );

Additional context
This request started as a discussion: https://github.com/prisma/prisma/discussions/7372

The error message is also misleading in that it still refers (permanently) to a preview feature that does not exist any more since 2.20.0: https://github.com/prisma/prisma/blob/ca5111d07b2c4fc71a147ea90af2776de78683a6/src/packages/sdk/src/engine-commands/getConfig.ts#L50-L57

Any update about this issue?

More popular issue over here: https://github.com/prisma/prisma/issues/10710 I'll close this one as a duplicate, if some of the info you added here @tajnymag is missing over there, please leave a comment. Thanks!


Release the gem to RubyGems.org

Suggest a 0.10.0 version, or a 1.0.0 version for this major milestone
Will need to tell people to update their Gemfile (to no longer use the git repository and a tag) in the release notes
Should automatically publish new versions using Github Actions

With release 0.10.0: https://github.com/govuk-ruby/govuk-design-system-rails/releases/tag/0.10.0 the gem is available on RubyGems: https://rubygems.org/gems/govuk-design-system-rails 🥳


check for minimum  wayland-client version (>= 1.20)

build fails if the installed wayland-client version is less than 1.20

Thanks! Also, btw, I updated the main PR's OP to contain a general idea of my plans with the implementation.


Ethan: Blacklist fly4lesstravels.com

Ethan requests the blacklist of the website fly4lesstravels\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fly4lesstravels\.com has been seen in 5 true positives, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Slow from VM machine in Gcloud

Currentely we are using google storage in the same zone.
I see really slow connection (using PHP and using BASH COMMANDs)
We can´t find the origin of the problem, but i have find a rare extreme latency to googleapis.
PING storage.l.googleusercontent.com (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from ce-in-f128.1e100.net (<IP_ADDRESS>): icmp_seq=1 ttl=51 time=47.8 ms
64 bytes from ce-in-f128.1e100.net (<IP_ADDRESS>): icmp_seq=2 ttl=51 time=47.1 ms
64 bytes from ce-in-f128.1e100.net (<IP_ADDRESS>): icmp_seq=3 ttl=51 time=47.1 ms
64 bytes from ce-in-f128.1e100.net (<IP_ADDRESS>): icmp_seq=4 ttl=51 time=47.0 ms
64 bytes from ce-in-f128.1e100.net (<IP_ADDRESS>): icmp_seq=5 ttl=51 time=47.0 ms
If that possible?

@mcontartesi, I see you opened a similar issue here. I'm going to close this one out as this issue doesn't seem specific to the library. Please feel free to re-open if you discover any issues targeted towards this code base :).


feat(cli): change options from openai to openAI

transmart not support openaiApiKey but openAiApiKey so change cli
options
Closes https://github.com/Quilljou/transmart/issues/12

Thanks. Bumped to v0.4.3


TiDB as a data source for streamlit

Hi streamlit folks,
I am from TiDB community. TiDB is an open source distributed HTAP NewSQL database and fit for real-time analytics. There is also a TiDB Cloud. Streamlit is wonderful especially I am also a Python developer. :-) I wonder if I could submit a PR to add TiDB to the data source list. How do you think?
Thanks!

Hi @zhangyangyu 👋
TiDB looks like a great project, please go ahead and make a tutorial in a similar style to the ones that already exist!
Best,
Snehan 🎈


Consider adding local-account ID into the userHandle of the passkey

The user.id (aka userHandle) of a passkey is currently filled with the 32 bytes of the IV/seed value for the key, as well as 2 bytes for a counter. This leaves 30 bytes (max is 64 bytes) available.
Currently, the only linkage between a passkey credential and its local-account is via the localStorage entries. But if we put the accountID into the user.id / userHandle field pin those remaining 30 bytes, then a local-account ID is retrievable from any passkey authentication (even if the localStorage entry has been lost -- although that loss would have dropped the initial public-key for the credential, so verifications would no longer be possible).
Advantages:

more durable design (less reliant on localStorage)
more secure storage of the account ID than in localStorage -- although it likely would still be stored there for public-key verification purposes

Disadvantages:

backwards-incompatible breaking change -- though such risk is low, likely nobody is using this in production yet -- although the code could probably work around this by detecting the difference in length of returned userHandle values.
would imply a stricter limit on the maximum length of the account ID -- and maybe we shouldn't take up all 30 remaining bytes, in case we need more space for something else later?
account ID is not really "personally identifying information" so it's probably technically OK to put in this field, but it's maybe not as preferred.

An added benefit is that password managers which allow you to access your passkey across devices will be able to sync the account id as well.  This should make provisioning a new device easier for apps that would otherwise have to figure out the account being provisioned with some other means

Hmmm, that's an interesting point. The wallet is definitely planned to handle synchronization of the accounts/keys, so it would handle this. But yeah, it might make non-wallet out of band synchronization easier.


PR for Docker Digest

Issue addresed
Enter the issue number of the bug(s) that this PR fixes

Template should be strictly Closes #16

!bounty 200

!bounty 50


[WIP] Deduplicate metadata path structure to use file path  = "root/directory/file"

What changes were proposed in this pull request?
Changes in this PR:

Index metadata will store index file contents as below (note the <root>/<directory>/<file> structure:

  "content" : {
    "root" : "file:/C:/Users/apdave/github/hyperspace-1/src/test/resources/indexLocation/index1",
    "directories" : [ {
      "path" : "v__=0",
      "files" : [ {
        "name" : "part-00007-ec24eb01-d061-4cd7-b10f-9ba9c3fdc972_00007.c000.snappy.parquet",
        "size" : 926,
        "modifiedTime" :<PHONE_NUMBER>839
      }, {
        "name" : "part-00011-ec24eb01-d061-4cd7-b10f-9ba9c3fdc972_00011.c000.snappy.parquet",
        "size" : 926,
        "modifiedTime" :<PHONE_NUMBER>730
      }, ...

JoinIndexRule, FilterIndexRule (and future rules) will read from a list of index files which comprise of the latest snapshot of the index. This was not true in previous versions. Earlier, the index was represented by just its root directory.

Why are the changes needed?
For support of future features like #110 and #60 , it's now required to read a subset of index files from a list of index directories. For this, we are updating the metadata to store a complete list of files which represent the current state of the index.
Does this PR introduce any user-facing change?
yes, the metadata layout is changed. Previously created indexes will be unreadable.
How was this patch tested?
unit tests

@apoorvedave1 will come up with a better structure in index metadata to capture file structures. I am closing this for now. Thanks!


[API] Create stable APIs for PyTorch 2.6

optimize is turned on. It will be controlled by an option in PyTorch
Remove the _TORCH_ONNX_SAVE_EXTERNAL_DATA_WITH_IR flag

Each pytorch release will use a version that is design for that release. We are responsible to update the pytorch code to use the correct version. For example, we should go in now and change pytorch to use the 2.6 api.


Cndb 11133

Substitute ant script and scriptdef with custom ant tasks - port CASSANDRA-17224
Set correctly keepBriefBrief property - port CASSANDRA-18267
Ran into an odd situation where, on jdk21, antlr crashed first time through then worked 2nd. Forking it fixes this.
Fix classpath file creation for eclipse. - port CASSANDRA-17294

Pending CI, eclipse changes tested locally by Berenguer.

CI doesn't show any related failures, not to mention most seem like flaky tests, timeouts I see repeatedly on PR gate

Maybe that comment could be a TODO?

I am not sure why a TODO, the fork is done.

Not sure what the situation is with support for JDK 21. I have tried to build with it and there are some compile errors in BTreeSet.java. No problems with JDK 17, though. Is this expected?

Valid point, I had to mention here: main branch is not compiling on JDK21+, this was tested on the JDK22 POC branch which compiles with newer JDK versions -  jdk22-october-for-poc.
Does changing <!-- Ran into an odd situation where, on jdk21, antlr crashed first time through then worked 2nd. Forking it fixes this. --> to

I was mentioning the TODO because I think we don't know why it's needed, do we? So in case we want to investigate in the future. But I won't insist. The update comment looks good, +1

I was mentioning the TODO because I think we don't know why it's needed, do we?

Ofc, you are right! Updated accordingly, rebased and force-pushed. Going to commit on clean CI

I also ran locally below commands(and received the expected outcome) as per the below "table":
Ant command                           | No args        | -Dcassandra.keepBriefBrief=true | -Dcassandra.keepBriefBrief=false
ant testclasslist                        | brief output | brief output                                      | long output
ant testsome -Dtest.name=... | long output | brief output                                      | long output
ant test                                      | brief output | brief output                                      | long output
PS GH can really benefit of the option to add tables...similar to how Jira has them

CI had a really bad day, and many builds errored out. I just pushed an empty commit to trigger it again.

Looks good to me, +1.

CI completed despite one job being marked as Pending here.
There is one non-timeout failure which is unrelated:
https://jenkins-stargazer.aws.dsinternal.org/job/ds-cassandra-pr-gate/job/PR-1365/3/testReport/org.apache.cassandra.metrics/TrieMemtableMetricsTest/tests_stage_1___jvm_unit_tests___jvm_unit_compression_tests___testContentionMetrics_compression/
Moving to commit


Recommended Updates

I think the Readme file needs to be updated for better user readability and understanding the things properly.

@Itsparththing assign this to me!


[Feature Request]  Docker Application with simple UI

I feel most of the people who just want to spin up a kubernetes cluster using terraform would like to have a UI , where they can input all the OCI values and terraform will run and display the results.
So a simple web application with fields from the tfvars file which will be filled by the user and on clicking create terraform init, plan and apply will run and the results or the logs will be outputed to the user.
user will just run the run the docker-image and they will run the application locally , fill in the fields and the OKE cluster gets created. It will also eliminate any errors related to missing tfvars variables by the user.

What's the use case for this?
The whole point of Terraform is that you manage your infrastructure as code - I agree there is huge room for improvement in the way this module does (or doesn't) do things but a web app is the very last thing the module needs in my opinion.

i agree the whole point of doing this thing is having a UI that can help the developer/USER/newbie to simply put in the values and create the cluster.
So I wanted to assign it to myself actually as I am already working on it  . the End goal is to give developer the UI interface from where he can spin up the infra and deploy the application to the cluster only from the UI without knowing about terraform , dockerfile, yaml files. <<<< this is the end use case. i can close the issue and create a pull request instead after creating the application... what say ?

It seems to me that a UI to provision k8s clusters would be an entirely separate project.
I'm just a user of the module though - not an Oracle Dev. Not my call. :)

ohh ok I will talk with @hyder on this separately.

@cawilliamson if you have ideas on how we can improve, I'm very happy to listen

@saiyam1814 The paid version of Terraform, TFE, has a decent Web UI. The issue with Web UI’s though is they are a security nightmare.
If they are spinning up a Kubernetes cluster they should know how to use basic command line.
Also long-term Oracle will probably launch a browser-based IDE, a way to do builds and runs via the Cloud Console, etc. All of these tools will provide additional graphical ways to run Terraform.

@saiyam1814 : https://docs.cloud.oracle.com/iaas/Content/ResourceManager/Concepts/resourcemanager.htm

@dan yes if the Ui nit used properly it will create clusters just like that
so mainly its targeted for developers and I working at oracle have recieved
lots of inputs for creating this
@ali yeah I have seen the resource manager, mine is a bit different thougb
from infra to app deployment via UI , like I told you oN slack the other
day. You can compare that to somewhat like rancher for oracle oke .
On Tue, 14 May 2019 at 6:11 AM, Ali Mukadam<EMAIL_ADDRESS>wrote:

@saiyam1814 https://github.com/saiyam1814 :
https://docs.cloud.oracle.com/iaas/Content/ResourceManager/Concepts/resourcemanager.htm
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oracle-terraform-modules/terraform-oci-oke/issues/26?email_source=notifications&email_token=AB6PRIUJD2ASQPCWUKVT6BTPVIDENA5CNFSM4HLI7YW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVJ56AQ#issuecomment-492035842,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6PRIW5VOZIOJMERY4WDS3PVIDENANCNFSM4HLI7YWQ
.

--
https://www.linkedin.com/in/saiyam-pathak-97685a64/

@saiyam1814 I am external to Oracle so I can't speak of your internal discussions on this but this repository sits under a user called "oracle-terraform-modules" and in a repository called "terraform-oci-oke".
In it's current state it does not work properly as a Terraform module in any meaningful way (i.e. it is nowhere near production ready.) With that in mind - I think this is entirely the wrong place to even think of adding GUI elements and "quick start" type setups.
A terraform module is 100% about creating a set of resources on a specific provider but presenting the TF developer with a simple frontend to that (i.e. the module may create ~30 resources but from a Terraform point of view this is "one cluster".) It should not include any kind of "quick start" or "gui" elements at all as that runs entirely at odds with what Terraform as a product is and does.

@cawilliamson I think I mentioned that before also to you that I will work internally will Ali and then 7 days ago only this issue was closed(since I am not able to delete) not sure if you saw the status already. I feel there is jo point in bringing up same things on closed issue.

@saiyam1814 Sure, no worories. Was just responding to your update added 7hrs ago. 👍


[DNM] Allow incompatible slots when using swap-move

Update swap with move strategy to use a build time size configuration to use for moving up and swapping, so incompatible slots, eg primary with 4096 byte sectors and secondary with 8192 sectors can work together. Sectors must still be all linear with the same size on each slot, both have to have sectors with sizes that are multiples, etc.
TODO - Must add Zephyr config
WIP - To test on sim: MCUBOOT_SWAP_MOVE_SIZE=8192 cargo test --features "swap-move sig-rsa validate-primary-slot"

@d3zd3z @nvlsianpu While testing this PR I had the same issues we discussed in the meeting this morning while running on the simulator, perm_with_random_fails sometimes fails, I will investigate this but we should not release a new version before it is known why this is happening.

@d3zd3z @nvlsianpu While testing this PR I had the same issues we discussed in the meeting this morning while running on the simulator, perm_with_random_fails sometimes fails, I will investigate this but we should not release a new version before it is known why this is happening.

I believe, although I'm not 100% sure, this is an issue with the test itself, not with the bootloader, I'll see into fixing it tomorrow. It only happens when using swap-move.


THRIFT-3514: Add PHP 7 version of php_thrift_protocol

This is an initial port of php_thrift_protocol to PHP7 and an RFC how to approach it.
However as we deal with zval's all over the place, we opt for separating
the C files completely leading to some overhead. However this
is a good start to see the differences in the implementation. From
there we should follow up with a more unified approach by refactoring
parts of the zval handling to be more generic so we can plug it
into PHP 7 and PHP 5 extensions.
Tested this by running with TestClient.php against a CPP server
and using TBinaryProtocolAccelerated.

Awesome!! :cake: :beers:

The contbuild error seem to be in java-csharp and perl-py3 which are unrelated.

Great work :8ball:


Scanning QR code stopped working on Mac

Yubico Authenticator version: 5.1.2
Operating system and version: Mac 10.15.1
YubiKey model and version:
Bug description summary:

Can't scan QR code like I used to before 10.15 & 5.1.2
Steps to reproduce

Scan QR code

Expected result
QR code scanned
Actual results
QR code not scanned
Other info
I would like to scan QR codes so I can use my device again.

Thanks for the report.
Can you please double check what version of Yubico Authenticator you are using? The latest should be 5.0.1.
Could you also please check if Yubico Authenticator has access to Input Monitoring? This is new in MacOS Catalina. You can find it in "Security & Privacy" > "Privacy" > "Input Monitoring"

Oh sorry, that was firmware version. It's 5.0.1.
And yeah, my Yubico Authenticator requested that permission, I went there to checkbox it, and then restarted the app.

Did it work after restarting or do you still have the same problem?

Problem still persists.

Could you run the application from a terminal with the --log-level DEBUG commandline option, try the QR scanner, and post the output here?

I'm having the same problem. I'm happy to run from the terminal if you provide the command.

It would be nice if there was an image import as a backup

Scanning doesn't work for me either. I'm on MacOS 10.15.4, is there anything I can do to help get this fixed?

@wade41 A standard otpauth url should work, following the format described here. The issue you linked is about handling those links directly, not when parsed from QR code.
Could you give us any more info about the QR code so we can look into it? Maybe name the service, or post the string in the QR code with the secret removed.

It's from a project I'm working on. Utilizing ttps://github.com/endroid/qr-code to generate the QR code and https://github.com/Spomky-Labs/otphp to generate the provisioning uri.
Here's the otpauth url:
otpauth://totp/admin.Application%3Asam%40u-lab.nl?issuer=admin.Application&secret=

uri looks fine, I'm able to add it with ykman:
$ ykman oath uri "otpauth://totp/admin.Application%3Asam%40u-lab.nl?issuer=admin.Application&secret=abba"
$ ykman oath code
<EMAIL_ADDRESS>742944

Do you have an example QR code picture as well?

Sure thing, here you go. I shortened the secret bit in this to abba like your comment.

I am on MacOS Big Sur, and this still doesn't work. I have all permissions set properly. Please let me know anything I can do to help troubleshoot the issue.

just bought a new Yubikey, just ran into this issue when trying to setup with https://www.docusign.com/. here is the output from /Applications/Yubico\ Authenticator.app/Contents/MacOS/yubioath-desktop --log-level DEBUG
2021-03-03T21:11:36+0100 ERROR [yubikey.parse_qr:558] Failed to parse uri
Traceback (most recent call last):
  File "qrc:///py/yubikey.py", line 556, in parse_qr
    CredentialData.from_uri(qrdecode.decode_qr_data(qr))))
  File "/Applications/Yubico Authenticator.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/ykman/oath.py", line 124, in from_uri
    raise ValueError('Invalid URI scheme')
ValueError: Invalid URI scheme
2021-03-03T21:11:40+0100 ERROR [yubikey.parse_qr:558] Failed to parse uri
Traceback (most recent call last):
  File "qrc:///py/yubikey.py", line 556, in parse_qr
    CredentialData.from_uri(qrdecode.decode_qr_data(qr))))
  File "/Applications/Yubico Authenticator.app/Contents/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages/ykman/oath.py", line 124, in from_uri
    raise ValueError('Invalid URI scheme')
ValueError: Invalid URI scheme

Same issue, MacOS 12.2.1 and authenticator 5.1.0
2022-03-31T09:11:51-0400 ERROR [yubikey.parse_qr:925] Failed to parse uri
Traceback (most recent call last):
  File "qrc:///py/yubikey.py", line 923, in parse_qr
    CredentialData.parse_uri(data)))
  File "/Applications/Yubico Authenticator.app/Contents/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/yubikit/oath.py", line 110, in parse_uri
    raise ValueError("Invalid URI scheme")
ValueError: Invalid URI scheme

The app only requested screen recording, but I granted it input monitoring as well. No change with both.

Had an issue but once I added:
Screen Recording permission set for Yubico Authenticator (System Preferences -> Security & Privacy -> Privacy -> Screen Recording). Additional required settings will also be in Input Monitoring.
Running Monterey 12.3.1
Yubico Authenticator v5.1.0

@vosscodes Can it scan some qr codes or do you run into the same issue with every single qr code?

sorry for the delay - not sure how I missed this. I only attempted two sources (several times with regenerated codes), but no luck with either. Unfortunately this was a corporate device that I no longer have, so I can't attempt again or get any more details.

I also have this issue. Both Screen Recording and Input Monitoring permissions have been gived to Yubico Authenticator v5.1.0. I am running macOS 12.5.1

FWIW I have a QR code scanning app "QR Capture v2.5.6 (43) by Marek Hrušovský" (from the App Store), which also uses Screen Recording permissions read QR codes on screen. It can scan these no problem. Indeed this is how I am now getting the details across and into Yubico Authenticator on sites that do not give me a way to copy thr raw secret.
I mention this to point out it is not some new limitation in recent macOS. This is possible, it is just that Yubico Authenticator v5.1.0 fails due to some bug.

@ruario We are working on a newer version of the app. If you have the time you can try out the beta here https://www.yubico.com/products/yubico-authenticator/yubico-authenticator-desktop-beta/ . It would be interesting for us to know if this works better or not. Please keep in mind that this is a beta version. Thanks!

Yubico Authenticator 6.0 has now been released and uses a new codebase. As such, this issue has been marked with the legacy label, and will be automatically closed in 7 days. If this issue is still relevant to Yubico Authenticator 6, please comment on the issue saying so, and it will be kept open (or be re-opened). Sorry for the inconvenience!


Remove the Set-ExecutionPolicy command

Remove the Set-ExecutionPolicy command. irm 'https://aka.ms/install-azd.ps1' | iex doesn't write anything to disk, so the execution policy doesn't affect it.
Also, if you run install-azd.ps1 like this in a new PowerShell process, the following code block doesn't make sense, because you won't see that change in the caller's PowerShell session:
            # Also add the path to the current session
            $env:PATH += ";$InstallFolder"

It only makes sense, if you start Windows PowerShell and run irm 'https://aka.ms/install-azd.ps1' | iex directly.

@puicchan
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@hhunter-ms could you take a look at this? Thanks!

Hey @alexandair  - Thank you for your feedback! It looks like our docs have since been updated to reflect the development team's changes to the Azure Developer CLI GitHub readme.

Thanks, @hhunter-ms! Closing as fixed.

@hhunter-ms @KarlErickson
The article that @hhunter-ms has mentioned is still incorrect. The current change of the installation command is making the same mistake as the previous one. irm 'https://aka.ms/install-azd.ps1' | iex doesn't write anything to disk, so the execution policy doesn't affect it.

Thanks for the extra explanation, @alexandair. Reopening.

Thanks for opening @alexandair. I can help answer some questions about the PowerShell install stanza. In short, we're trying to maximize installation success and usability in all the various use cases of our installation script.
Setting the ExecutionPolicy to Bypass or AllSigned
It is true that a script downloaded and piped directly into iex is not signature checked before execution. Our use of setting execution policy in the PowerShell process is meant to ensure that the script's dependencies execute as well.
We're setting the ExeuctionPolicy for the PowerShell process to ensure that cmdlets that the PowerShell install script invokes (e.g. Expand-Archive) are not blocked by more restrictive policies (e.g. Restricted) set at a higher scope. If the policies on the machines on which this command are executed are not so restrictive, then the command this PR proposes will work.
More on signatures: We do sign the script for scenarios that involve downloading the script to a file and executing the downloaded script file. If you want to verify the signature of the installation script before executing the install script, download the install script to a file and execute that file in PowerShell. PowerShell will verify the Authenticode signature of the script file on load and act according to execution policies already set.
To see this behavior locally:
PS> Set-ExecutionPolicy Restricted -Scope Process
PS> irm 'https://aka.ms/install-azd.ps1' | iex
VERBOSE: Downloading latest build from https://azure-dev.azureedge.net/azd/standalone/release/latest/azd-windows-amd64.zip
VERBOSE: Unzipping artifacts
Write-Error: Cannot expand C:\Users\djurek\AppData\Local\Temp\wzfcuxwn.vrz\azd-windows-amd64.zip
Write-Error: The 'Expand-Archive' command was found in the module 'Microsoft.PowerShell.Archive', but the module could not be loaded. For more information, run 'Import-Module Microsoft.PowerShell.Archive'.

Our documentation for the install scripts now describes these scenarios: https://github.com/Azure/azure-dev/blob/main/cli/installer/README.md#verifying-the-install-script-in-powershell
Setting the $env:PATH
In cases where the script is executed in a PowerShell session that does not end (e.g. the user has downloaded the script, opens a PowerShell session, and runs the script from that PowerShell session), setting $env:PATH directly in the script ensures that azd is available for use in the current session. This is meant to make it easier to use azd in this scenario.
In scenarios where the shell closes when the script ends, setting $env:PATH does nothing material.

@danieljurek
First, it's great that the installation script is signed. No question about that.
I'm surprised that the Expand-Archive function is blocked by the execution policy. Execution policy affects the scripts not the cmdlets/functions.
On my machine, the execution policy for Windows PowerShell is set to Restricted. As you can see it, that doesn't block install-azd.ps1 to install Azure Dev CLI. Maybe it's not a bad idea to first run Unblock-File before you try to expand the .zip with Expand-Archive.
PS C:\> Get-ExecutionPolicy
Restricted
PS C:\> irm 'https://aka.ms/install-azd.ps1' | iex
VERBOSE: Downloading latest build from
https://azure-dev.azureedge.net/azd/standalone/release/latest/azd-windows-amd64.zip
VERBOSE: Unzipping artifacts
VERBOSE: Installing azd in C:\Users\alex\AppData\Local\Programs\Azure Dev CLI
VERBOSE: Cleaning temporary install directory: C:\Users\alex\AppData\Local\Temp\f30j12m4.5yi
An entry for C:\Users\alex\AppData\Local\Programs\Azure Dev CLI is already in PATH
Azure Developer CLI (azd) installed successfully. You may need to restart running programs for installation to take effect.
- For Windows Terminal, start a new Windows Terminal instance.
- For VSCode, close all instances of VSCode and then restart it.
PS C:\>

As we are discussing the installation script, the way how PATH env variable is set should be changed. [Environment]::SetEnvironmentVariable converts the raw registry type from REG_EXPAND_SZ to just a REG_SZ. That change is actually very distractive despite the script's comment.

@alexandair, thanks for the clarification.
I think I see what's happening. The error about Expand-Archive does occur in pwsh (v7.2) but not in PowerShell (v5.1). Since we're executing powershell and not pwsh, this behavior shouldn't come up. I'll do some more checking on my side and update our documentation if that's the case. @KarlErickson, let's keep this PR open.
https://github.com/Azure/azure-dev/issues/244
As far as the environment variable: I see that my local Path in the registry is indeed a REG_SZ and my read of the SetEnvironmentVariable method is that it's setting the type to REG_SZ. This definitely requires further investigation.
https://github.com/Azure/azure-dev/issues/245

@alexandair -- Thanks again for pointing out the issue involving destructive changes to the registry. I'm proposing a PR here that updates PATH without changing its value type and still sends the WM_SETTINGCHANGE message to top level windows so that azd is usable without a restart in later terminal sessions.

@alexandair @hhunter-ms @danieljurek is this issue now resolved? Can we close? Thanks.

@alexwolfmsft can you please assess this PR and close if it's obsolete? Thanks!

The install docs have been updated since this was opened, I believe we can close this.
#please-close


Restore proper fallback gradient for IE8 and IE9.

Changes

Restore proper fallback gradient for IE8 and IE9.

For review: @DoSomething/front-end

LOL. Awesome pic! :+1:


BIGTOP-4189: Mock Data Implementation for AI Chat API in Bigtop-Manager

https://issues.apache.org/jira/browse/BIGTOP-4189
API Doc: https://apifox.com/apidoc/shared-1b4f589f-27ef-4bb6-9c9c-e3680d3fca79/api-205377679
Commits: https://github.com/lhpqaq/bigtop-manager/commits/aichat/mock?author=lhpqaq

Please fix the ci


AuthenticationError: Client sent AUTH, but no password is set

[root@iZm5edcqv0ak6r3z1ayc8vZ proxy_pool-master]# python proxyPool.py schedule

*** ______  ********************* ______ *********** _  ********
*** | ___ _ ******************** | ___ \ ********* | | ********
*** | |/ / _ __   __  _ __   _ | |/ /__ * ___  | | ********
*** |  /|  // _ \ \ / /| | | ||  __// _ \ / _ \ | | ********
*** | |   | | | () | >  < \ || || |  | () | () || |  ****
*** _|   ||  ___/ //_\ __  |_|   _/ _/ __/ ****
****                       __ / /                          *****
************************* / / *******************************

2022-01-11 15:41:15,913 launcher.py[line:45] INFO ProxyPool Version: 2.4.0
2022-01-11 15:41:15,914 launcher.py[line:50] INFO ProxyPool configure HOST: <IP_ADDRESS>
2022-01-11 15:41:15,914 launcher.py[line:51] INFO ProxyPool configure PORT: 5010
2022-01-11 15:41:15,914 launcher.py[line:52] INFO ProxyPool configure PROXY_FETCHER: ['freeProxy01', 'freeProxy02', 'freeProxy03', 'freeProxy04', 'freeProxy05', 'freeProxy06', 'freeProxy07', 'freeProxy08', 'freeProxy09', 'freeProxy10']
2022-01-11 15:41:15,957 launcher.py[line:58] INFO ============ DATABASE CONFIGURE ================
2022-01-11 15:41:15,957 launcher.py[line:59] INFO DB_TYPE: REDIS
2022-01-11 15:41:15,957 launcher.py[line:60] INFO DB_HOST: <IP_ADDRESS>
2022-01-11 15:41:15,957 launcher.py[line:61] INFO DB_PORT: 6379
2022-01-11 15:41:15,957 launcher.py[line:62] INFO DB_NAME: 0
2022-01-11 15:41:15,957 launcher.py[line:63] INFO DB_USER:
2022-01-11 15:41:15,957 launcher.py[line:64] INFO =================================================
2022-01-11 15:41:15,959 redisClient.py[line:149] ERROR redis connection error: Client sent AUTH, but no password is set
Traceback (most recent call last):
File "/opt/proxy_pool-master/db/redisClient.py", line 144, in test
self.getCount()
File "/opt/proxy_pool-master/db/redisClient.py", line 130, in getCount
proxies = self.getAll(https=False)
File "/opt/proxy_pool-master/db/redisClient.py", line 112, in getAll
items = self.__conn.hvals(self.name)
File "/usr/lib/python2.7/site-packages/redis/client.py", line 3084, in hvals
return self.execute_command('HVALS', name)
File "/usr/lib/python2.7/site-packages/redis/client.py", line 898, in execute_command
conn = self.connection or pool.get_connection(command_name, **options)
File "/usr/lib/python2.7/site-packages/redis/connection.py", line 1379, in get_connection
connection.connect()
File "/usr/lib/python2.7/site-packages/redis/connection.py", line 567, in connect
self.on_connect()
File "/usr/lib/python2.7/site-packages/redis/connection.py", line 643, in on_connect
auth_response = self.read_response()
File "/usr/lib/python2.7/site-packages/redis/connection.py", line 739, in read_response
response = self._parser.read_response()
File "/usr/lib/python2.7/site-packages/redis/connection.py", line 340, in read_response
raise error
AuthenticationError: Client sent AUTH, but no password is set
2022-01-11 15:41:15,963 launcher.py[line:39] INFO exit!

AuthenticationError: Client sent AUTH, but no password is set   配置正确密码


Decrease database size

Less data to download, smaller indexeddb size, etc.
Also, use better quality images for zoomable imgs.
Not sure about runtime performance but it shouldn't be worse, I guess.

Thanks!


Document how to link metadata tsv download  to matrix files

+1 . The matrix .mtx contains only numerical information, and thus the .mtx depends on information in the .tsv to understand the nature of the contents. (at least via my navigation of the data browser and matrix service documentation).
I wondered when I was browsing the data if they should be bundled together.


Upgrade Ledger dependencies

I did these upgrades to try and reoslve https://github.com/SetProtocol/index-ui/issues/514. They didn't resolve that issue but I think we can upgrade them anyway.
This targets website-v2-master. I can duplicate for master if requested.

Hmm, when I tried pulling this down, it looks like it breaks the buy/sell flow. Can you confirm?


Fixed a double assignment

Corrected a autogenerated code issue where a local variable
was generated twice within the same scope.
Signed-off-by: Toure Dunnon<EMAIL_ADDRESS>

Unfortunately this doesn't seem like a good long term solution as any subsequent codegen executions will overwrite this...we need to figure out a better way to fix the issue in the codegen client:

either we can edit the swagger spec to make it work OR
need to make a change to fix the bug in the codegen code itself (which is it fails to generate good code when a request body has two file parameters)

@dakaneye issues addressed.


Sksinghspecs

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-node

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

The title of the PR is clear and informative.
I don't think this is done yet :) can you edit title to summarize the change?

fixed the title

The title says "change to async for create API" but I don't see any changes to the create API.  Is that still in progress?

Actually it was already pushed to main branch, I will correct the title :)


Fix weibull lcdf

Fix for #2810.
Release notes
Makes the Weibull cdf/lcdf more stable by using log1m_exp function.
Checklist

[x] Math issue #2810

[x] Copyright holder: Sean Pinkney
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@rok-cesnovar I don't know how to fix the expression tests for changes up to commit 9c1f7ace7ba39b9f41c3832c84e75565f253173a (2nd to last commit). Though the changes are minimal. For the final commit, I ended up following the neg_binomial_2_l/cdf files which meant overhauling the files. In my opinion, the file is more readable but would like you or others to take a look. There aren't great tests for these functions so my plan is to add them as well.
@nhuurre for y values of 0 I have the function returning 0 for the cdf and negative infinity for the lcdf. I believe the cdf is right. I'm wondering if the lcdf is correct or if it should also return 0.

Negative infinity for the lcdf is correct. The derivatives should be well-defined (and zero) when alpha > 1.

Negative infinity for the lcdf is correct. The derivatives should be well-defined (and zero) when alpha > 1.

Shoot, I was hoping it was for all alpha because this raises a question about what we do with the pow() function. In the case of the model in the discourse post I get the same answer when I return a 0 for all alpha as compared to the UDF defined in the same post. When I limit the implementation here to alpha > 1 I get divergences and a different answer, unsurprisingly. The question then is, do we update pow or do I leave it for all alpha here?

@rok-cesnovar I don't know how to update the opencl version of weibull cdf. It needs to return 0 for the gradients when y is 0 and I'm getting header check errors when I try to update.

Looking into it now.

Obsoleted by #3037