[cw310] Annotate pin-out with peripheral map and fix up the PMOD2 mapping

Annotate the hyperdebug pin-out with the peripheral map used in the
default test configuration. This should help with future synchronization
efforts for pinmux configs across the different targets.
Fix PMOD2 mapping for the hyperdebug variant.
The pins mapped to PMOD1, when PMOD2 was the intention.

Looks good to me, with one minor suggestion. For blocks which are instantiated multiple times (I2C, UART etc), could you add the index of the block to the comments too? E.g. I2C0, UART1

Was there a specific motivation you had in mind for that?
From my POV, it isn't the responsibility of this file to decide the specific peripherals software uses through the pinmux, and it is acceptable from the FPGA's perspective to use any I2C peripheral for a given I2C-marked interface. Instead, these annotations declare what the board supports.
The software mapping for a given top (i.e. for multi-top handling) would probably decide the exact set, but I didn't want to make the XDC file go too far, else it'll be yet another thing to update if we decide not to go with a one-to-one mapping (for whatever reason).

From my POV, it isn't the responsibility of this file to decide the specific peripherals software uses through the pinmux, and it is acceptable from the FPGA's perspective to use any I2C peripheral for a given I2C-marked interface. Instead, these annotations declare what the board supports.
The software mapping for a given top (i.e. for multi-top handling) would probably decide the exact set, but I didn't want to make the XDC file go too far, else it'll be yet another thing to update if we decide not to go with a one-to-one mapping (for whatever reason).

Ah okay, in this case it would be helpful to add a comment to explain this. My thinking was to over-specify the mapping to avoid confusion, but a helpful description would make it clearer that this is really a board-support annotation.


VS Code Installer Stops Interacting With Windows (Hangs) on Windows 11 24H2 (Reproducible Every Single Time At-Will)

Does this issue occur when all extensions are disabled?: Not Applicable (N\A)

VS Code Version:  1.93, 1.92.2, 1.92.1 (Same result if attempt to install VS Code via Microsoft Store or winget; installs fail)
OS Version:

Edition	Windows 11 Pro
Version	24H2
Installed on	9/6/2024
OS build	26100.1742
Experience	Windows Feature Experience Pack 1000.26100.18.0
Steps to Reproduce:

Clean install Windows 11 24H2 on system or use production Windows 11 24H2 system (do NOT use Virtual Machine)

Download VS Code installer (version 1.93.1, 1.93, or 1.92.2, 1.92.1 - or - Insider Build 1.94.0, etc) to test or production system

Execute VS Code installer

VS Code installer hangs at "Preparing to Install: Setup is preparing to install..."

Cannot cancel install; selecting Cancel button does not work - must terminate install via End Task in Task Manager

Windows 11 24H2 generates WER

Repeat test by attempting install via Microsoft Store and winget; VS Code install hangs and fails with Windows Error Reports

Partial snippet of Windows Error Report (WER):
FriendlyEventName=Stopped responding and was closed
ConsentKey=AppHangXProcB1
AppName=Setup//Uninstall
AppPath=C:\Users<user>\AppData\Local\Temp\is-E71M8.tmp\VSCodeSetup-x64-1.93.1.tmp
ReportDescription=A problem caused this program to stop interacting with Windows.

I have the same issue, in safe mode the installer works. I suspect it has something to do with the AMD drivers, do you have and AMD card?

Still occurs every time on my Win11 with AMD CPU and NVidia GPU.


Details rendered event

Is there any details rendered event for responsive extension ?
I need to re-bind some plugin to cells content when the new details has been rendered to dom.
I tried to listen column-visiblity.dt event but it didn't work.

As in child rows? No there isn't at this time since the child row is under control of the calling software, so it would generally know when it is displayed so there is no need for DataTables to emit an event.
In future, per the contributing document that GitHub links to on the "new issue" page, could you open support requests / questions in the DataTables forums please.


Features/dispatch plot

Fixes #123 by introducing dispatch plots.
Todo:

[ ] Find a better implementation for labels
[ ] Define colors
[ ] Define the order (non-dispatchable first, then stack dispatchables on top)

Closed in favor of https://github.com/rl-institut/oemoflex/pull/2


Don't use mozGetAsFile when quality is passed

mozGetAsFile doesn't accept a quality argument like toDataURI as a result the quality argument on toBlob won't have an effect on jpeg and webp images.

The performance benefits greatly outweigh the slight performance increase of using quality+toDataURI as the base64 encoding overhead is still much much greater. Unfortunately no matter what (reasonably high enough) quality is selected in toDataURI, it will always be slower than mozGetAsFile. Do you have any significant non-performance reasons to use quality+toDataURI?

Performance? Stripping the quality gives you the wrong file. The point of quality is to affect the image quality / filesize of the file. If you strip it out then you're not giving the file that toBlob is expected to give out.
Local performance is irrelevant if you upload the wrong quality blob to the server.
It's pretty irrelevant anyways since Firefox has native toBlob now. The only people using mozGetAsFile are using outdated Firefox. Trying to optimize performance for them at the cost of getting the correct output is pointless.

The whole point of using toBlob instead of toDataURL is performance, and it's not like I added the moz method to support old Firefoxes--it was added back when it was the only performant method in any Firefox. You are still getting the correct image format, just not at the desired quality.
I understand your concern though and I will accept a patch that uses toDataURL instead of the moz method when the format is image/jpeg and a non-1 quality is specified.

The whole point of using toBlob instead of toDataURL is performance, and it's not like I added the moz method to support old Firefoxes--it was added back when it was the only performant method in any Firefox. You are still getting the correct image format, just not at the desired quality.

To me the whole point of using toBlob instead of toDataURL is getting a format that XHR2 will accept and upload with xhr.upload.
In any case thanks for responding. I don't need the IE9- base64 support or toBlobHD so I just went with using canvas-to-blob for my project.


debug_toolbar.middleware.DebugToolbarMiddleware is missing

following command
$python manage.py migrate
gives error:
SystemCheckError: System check identified some issues:

ERRORS:
?: debug_toolbar.middleware.DebugToolbarMiddleware is missing from MIDDLEWARE_CLASSES.
	HINT: Add debug_toolbar.middleware.DebugToolbarMiddleware to MIDDLEWARE_CLASSES.

I am not sure where to add this

following the new setting as per #65  I ended up with
 django.core.exceptions.ImproperlyConfigured: Application labels aren't unique, duplicates: debug_toolbar

solved by commenting out debug_toolbar from INSTALLED_APPS in <project>/src/<project>/settings/base.py:
  INSTALLED_APPS = (
                 # ...
                 # 'debug_toolbar',
                 # ...
   )


F-Droid

Is your feature request related to a problem? Please describe.
Could you please publish the app on F-Droid? Thanks!
Describe the solution you'd like
I can help package and submit it.

I would love some help. I've looked into it and couldn't really understand what needed to be done. However, I haven't really been able to work on this project the last couple of days and the next few weeks. I should be primed to continue working by next month.

I can help you! I'll package it and make an MR. Thanks!

Is that possible to remove Firebase? It's non-free and not allowed in F-Droid.

It is...The reason why I added it in the first place is so that people can log in and gain access to their favorites and things read/watched without needing to do steps to transfer data. Plus, crashlytics is a huge plus for when debugging crashes.
What I can try to do is modify the builds so there's a firebase version and a non-firebase version with descriptions of what the difference between the two are.

What I can try to do is modify the builds so there's a firebase version and a non-firebase version with descriptions of what the difference between the two are.

A flavor without firebase/google play service/mlkit would be good. Thanks!

I'll see what I can do. It probably won't be easy.

Thanks!

Somehow I think I was able to do it!
https://github.com/jakepurple13/OtakuWorld/tree/no_firebase_variant
That's the branch right now. I'll merge it either later today or tomorrow. This should also make it easier for others to help work on the app too since the no firebase variant doesn't need firebase.

Hm...I'll see what I can do then.

Thanks!

I think it should be all good now.

It works, thanks! It seems there are several different apps in this repo. Is com.programmersbox.otakuworld a unified app? Should I package all of them?

There's three (technically four) apps in this project. AnimeWorld, AnimeWorld TV, MangaWorld, and NovelWorld. OtakuWorld WILL NOT work. It was an attempt at viewing favorites from all the other apps in one place but it didn't work out too well.

Got it! Thanks!

AnimeWorld depends on com.google.android.exoplayer:extension-cast which depends on GMS. MangaWorld build failed:
> Task :mangaworld:processNoFirebaseReleaseResources FAILED
FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':mangaworld:processNoFirebaseReleaseResources'.
> A failure occurred while executing com.android.build.gradle.internal.res.LinkApplicationAndroidResourcesTask$TaskAction
   > Android resource linking failed
     ERROR:/builds/linsui/fdroiddata/build/com.programmersbox.mangaworld/mangaworld/src/main/res/layout/page_end_chapter_item.xml:9: AAPT: error: attribute adSize (aka com.programmersbox.mangaworld:adSize) not found.

     ERROR:/builds/linsui/fdroiddata/build/com.programmersbox.mangaworld/mangaworld/src/main/res/layout/page_end_chapter_item.xml:9: AAPT: error: attribute adUnitId (aka com.programmersbox.mangaworld:adUnitId) not found.

     ERROR:/builds/linsui/fdroiddata/build/com.programmersbox.mangaworld/mangaworld/src/main/res/layout/page_next_chapter_item.xml:8: AAPT: error: attribute adSize (aka com.programmersbox.mangaworld:adSize) not found.

     ERROR:/builds/linsui/fdroiddata/build/com.programmersbox.mangaworld/mangaworld/src/main/res/layout/page_next_chapter_item.xml:8: AAPT: error: attribute adUnitId (aka com.programmersbox.mangaworld:adUnitId) not found.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
* Get more help at https://help.gradle.org/
BUILD FAILED in 8m 6s

Could you please take a look? Thanks!

Mmmm...That's annoying. Guess I have to limit the ads too then...Not like they were in the way! If you were wondering how could I do such a thing. They are SO out of the way, you kinda need to try to find them. But I can try something.
As for casting, that's really annoying. That would mean we won't be able to cast. I guess I can, but then need to make sure to explicitly point out that in the no firebase version, casting is not available.

Yes, there are many convenient functions provided by GMS and Firebase. Instead of remove all GMS from the flavor, we can also use https://github.com/microg/GmsCore.


2.2.0 is not 1.0-beta compatible on IntelliJ / Android Studio

A gradle clean before build will fix this problem.

Can you try with 2.3.0

It works! Thank you very much.


[Proposal]: Chekcout PR command

Problem Description
Currently, if you need to check out a branch from a PR to test it locally, you need to figure out if it is on a fork or a branch that exists in the repo. Then, if it is a fork, you need to add a remote to that fork and then checkout the relevant branch.
Proposed solution
Add a command that would operate something like git checkout-pr <pr_url>, and figure out whether or not the branch referred to in the url is on a fork or the main repo. It would set up any necessary remote and then checkout the appropriate branch. It is also worth considering adding a "cleanup" command or option here, that removes the remote and deletes the branch once you are done.

Thoughts @Leo6Leo ?

@Leo6Leo I imagine that:
Scenario 1: this would go and see if pull/22 was from a fork or not. If it was from a fork, it would add the appropriate remote and then create a new branch from that remote.
Scenario 2: this would again go and see if pull/1 was from a fork or not. Also, if Leo6Leo/git-utils is not a remote on the current repo, it would create a remote to it if pull/1 was not from a fork.
Scenarios 3 & 4: no pull URL was provided, so it will tell them to have an appropriate PR url

Wdym "if pull/22 was from a fork or not"? If it is from a fork, do you mean PR#22 is a PR in Leo6Leo/git-util?
Let me make this more clear by having some more cases:
Case 1
Cali0707 created the upstream repo Cali0707/git-utils, and Leo6Leo forked it and has Leo6Leo/git-utils
And currently, Leo6Leo is running git-utils commands in ~/Downloads/git-utils, and this is Leo6Leo/git-utils.git
The PR links I used in the text below are valid.
Scenario 1
Leo6Leo try to git checkout-pr a pr from cali0707/git-utils because he needs to review one of Cali's PR.
This PR is not from a fork, and as expected, it should clone the Cali0707/git-utils repo, and checkout to add-make-pr-command branch.
Scenario 2
Leo6Leo try to git checkout-pr a pr from Leo6Leo/utils because he needs to review one of his own PR.
This PR is from a fork, and as expected, it should just directly checkout to leo/test branch.
Scenario 3
Cali0707 try to git checkout-pr a pr from Cali0707/git-utils because he needs to review one of his own PR. This PR is by Leo6Leo, and Leo6Leo is trying to merge Leo6Leo/git-util.git leo/test branch to Cali0707/gitutil.git main.
This PR is not from a fork, and as expected, it should clone the Cali0707/git-utils repo, and checkout to add-make-pr-command branch.
...... A bit sleepy rn, let's discuss this in person cali 🥱 @Cali0707

Per conversation with @Leo6Leo :

Find the remote for the PR branch
Find the branch name for the PR
Add remote if needed
Checkout remote/branch


Reservation order depending on Availability Date

Hello,
We need to be able to change the reservation order depending on how close the Availability Date is.
To do so we would need the following :
The parameter AvailabilityDate to be passed into event OnAutoReserveOnBeforeSetValueArray so we could set a custom "ValueArrayNo" depending on the value of AvailabilityDate
Like so :
OnAutoReserveOnBeforeSetValueArray(ValueArrayNo,AvailabilityDate);
[IntegrationEvent(false, false)]
local procedure OnAutoReserveOnBeforeSetValueArray(var ValueArrayNo: Integer,AvailabilityDate : Date)
begin
end;

So critical - we were trying to mimic Netsuite reservation / allocation strategy functionality (which is excellent) but ran into this issue with our own PTE. Prefer it is just implemented, but support this as a fix for customisations.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43428.


Keyboard issue on Tag Input

📒 Description
This PR introduces the fix for keyboard issue in Tag input
Problem
It's all about the design. The "Add tags" isn't the TextField, it's the NSButton, so we're unable to type any text during focusing here.

Solution
I introduce the AddTagButton as a subclass of NSButton, then it will pass all characters (exclude some ignore characters) from -keyDown to their delegate.
It means that user can start typing without any disruption.
🕶 Types of changes
Bug fix (non-breaking change which fixes an issue)
🤯 List of changes

[x] Introduce the AddTAgButton
[x] Receive the characters from AddTAgButton and present on Tag Input
[x] Minor cursor improvements

👫 Relationships
Closes #3028
🔎 Review hints

Able to navigate throughly by pressing Tab
Able to start typing when focusing on Tag Button
When typing text on Tag Button -> The Tag AutoComplete view will appear and filter properly
Click on Tag Button -> Open the view as normal

GIF

Thank @skel35, I will fix this minor issue by focusing on TextField again. So we can exit or enter new tag quickly 👍

@skel35 I've address your feedback with the last commit. After creating a new tag, we can continue typing or pressing the ESC to exit the windows.
It's ready for reviewing now 👍

GIF


Add Wrapped ETH to WETH9 addresses on BASE

https://github.com/Uniswap/sdk-core/blob/be6b14fa6f9223e0483389d792a9e07eab557a27/src/entities/weth9.ts#L6
Can you add weth on base mainnet below to the linked code:
[8453]: new Token(8453, '0x4200000000000000000000000000000000000006', 18, 'WETH', 'Wrapped Ether'),
Thanks

Hello, have you resolve your issue?


Tutorial Javalin JWT Example it is not clearly explained

Tutorial with JWT is not clarity for me.

1st code block with MockUser I should put into of course in new class MockUser, but where I should put 2nd block of code (with 4 steps) ? Also in MockUser class ?

I can't import JWTProvider from 2nd block (exacly 4 step:
JWTProvider provider = JWTProvider(algorithm, generator, verifier); )
thought I have in gradle:
compile 'com.auth0:java-jwt:3.4.1'
What can be wrong ?

@DavidApi if you tag the author you'll have a better chance of getting help, I don't think he watches this repo.

Apologies for the late response.

That block of code is yours to put wherever suits your project best. You can initialize it in the same place as you initialize your app and pass the provider to your controllers. The examples assume everything is define in one class (all handlers are defined in-line and all initialization done at once) so we do have access to the provider directly.

You shouldn't need to define a dependency to Auth0 library yourself since the extension already defines it with a specific version. If the problem still persists please let me know.


fix(drawer): unsubscribe

make sure subscription on fromEvent observable is unsubscribed when the
component is destroyed

@erhise, this PR has passed google test, can you resolve the conflict and push again

@vivian-hu the merge conflict was resolved but the ci-tests seem to have failed due to other issue? Is it possible to do retry-build? Otherwise let me know and I can push again.

Go ahead and rebase and push again since our CI on master is currently green.

Can you please push this fix? 10x


Linux workspace build failing

On Windows the workspace builds okay.
On WSL2 using Ubuntu and GCC 11.1 or Xubuntu 20.04 and GCC 11.1 I get the following error when I try to build via the wxdap-linux.workspace:
/bin/sh: 1: cd: can't cd to build-debug
=== build completed successfully (0 errors, 0 warnings) ===
On Linus should I be using using the workspace or cmake or make?
The reason I am trying to build on Linux is that when I try to build the DLL using the linux C::B project I am working on I get the error below and I want to check if I get the same error with the CodeLite workspace.
-------------- Build: Debugger WXDAP in Debugger WXDAP DLL unix (compiler: GNU GCC Compiler)---------------

g++ -Wall -std=gnu++17 -g -O0 -ggdb -fPIC -isystem/usr/lib/x86_64-linux-gnu/wx/include/gtk3-unicode-3.0 -isystem/usr/include/wx-3.0 -D_FILE_OFFSET_BITS=64 -DWXUSINGDLL -D__WXGTK__ -pthread -DBUILDING_PLUGIN -I. -I/mnt/d/Andrew_Development/Work_Installers/CodeBLocks_Private_Experimental/src/include -c /mnt/d/Andrew_Development/Work_Installers/CodeBLocks_Private_Experimental/src/plugins/contrib-wip/DebugAdapterProtocol/dap.cpp -o /mnt/d/Andrew_Development/Work_Installers/CodeBLocks_Private_Experimental/src/.objs31_64/debugger_WXDAP/dap.o

/mnt/d/Andrew_Development/Work_Installers/CodeBLocks_Private_Experimental/src/plugins/contrib-wip/DebugAdapterProtocol/dap.cpp: In static member function ‘static dap::ObjGenerator& dap::ObjGenerator::Get()’:
/mnt/d/Andrew_Development/Work_Installers/CodeBLocks_Private_Experimental/src/plugins/contrib-wip/DebugAdapterProtocol/dap.cpp:87:25: error: use of deleted function ‘dap::ObjGenerator::ObjGenerator()’
   87 |     static ObjGenerator generator;
      |                         ^~~~~~~~~

There are 2 workspaces provided. You should use the one named wxdap-linux (which is a CodeLite file system workspace that I configured to run  cmake command line commands). If you are not using CodeLite, then you should use the CMake:
mkdir -p build-debug
cd build-debug
cmake .. -DCMAKE_BUILD_TYPE=Debug
make -j$(nproc)

(same goes for macOS)

I am still working on this, but I think I may have a lead on the underlying issue I am seeing on Linux and it looks very similar to the following CodeLite issue:
https://github.com/eranif/codelite/issues/2769
I did a quick and dirty hack and changed the std::unordered_map to std::map along with the include files and I get the DAP so to build. This is not the right way, but it indicated to me that the project file I am using is okay and that the issue is code related. I need to look at issue 2769 and see what was done and give it a try.

Checked out issue 2769 and ported the change and created https://github.com/eranif/dbgd/pull/6 with the changes in it.
Once PR merged this issue can be closed.

Thanks very much for working on this yesterday. The compilation now works on all three OS's (Windows, Linux and MacOS).
I can now do some rudimentary debugging on the MacOS using C::B now. Breakpoints, simple watches and call stack are wired with about the same level of functionality as the demo app.

That's great to hear. I too did some progress wiring it into CodeLite. Attached is a screenshot. I am also attaching a screenshot of the settings dialog which I am currently using to define a dap server within CodeLite


Not possible to change websocket port in client

the wsPort parameter works as expected on my custom server, but it seems is not possible to change it on the client.
https://github.com/Code-Forge-Net/Remix-Dev-Tools/blob/50d55572cde95f6514d0f1cccb184cc01575632c/src/RemixDevTools/hooks/useDevServerConnection.ts#L52
Although the parameter can be passed to useDevServerConnection its always called with no params, so default 8080 is used

@keul I will be looking at this / fixing it very soon! Thank you for the report

Last time I encountered this, turns out I was passing in the initial build without withServerDevTools.
https://github.com/manawiki/core/commit/18f85565c0823085b54ddb19edaf28c78a7b3ae3#diff-81f1e1d9edc082e5508df8a7d86d802b361e59aa76c17abf37619e50fefcb134L139

@keul I've tested it out yesterday, in order fori t to work you need to pass it into the config as the second parameter,
withDevTools(App, { wsPort: 5000 })

I tried both this…
if (process.env.NODE_ENV === "development") {
  const { withDevTools } = await import("remix-development-tools");
  AppExport = withDevTools(AppExport, {
    wsPort: 9999,
  });
}

export default AppExport;

…and this…
if (process.env.NODE_ENV === "development") {
  const { defineClientConfig, withDevTools } = await import("remix-development-tools");
  const config = defineClientConfig({
    wsPort: 9999,
  });
  AppExport = withDevTools(AppExport, config);
}

export default AppExport;

But I still see it trying to call on ws://localhost:8080/

Uhm, after upgrading to latest version (3.3.0, I was on 3.2.x) the behaviour changed a bit, but maybe now is more clear what is the problem.
I was confused by seeing failed connections to localhost:8080: is not my remix app, but an API rest endpoint I'm using from remix! Someway it's trying to connect to this address using websockets. If I disable RDT, these connections attempts disappears. 🤔

Interesting, well the server part of rdt exposes a websocket to communicate with the client and the client uses this to fetch server timings etc. So you should be able to connect by changing the port number tp the same free port on both server and client, I'll be releasing a new version tomorrow that allows you to not connect at all

I see both successful and failing WS connections: on 9999 they succeed

on 8080 (obviously) the fails…

@keul I moved it to the settings page, merging the pr and releasing, you shouldn't have these problems anymore

@AlemTuzlak I've to be honest, now I don't get anymore how to change the WS port on the client

@keul go to settings tab on the client, enable swrver dev tools and change it (enabled by default)

OK, not exactly convenient as every developer in the team should do this (on every browser), but it works now! 🙌

OK, not exactly convenient as every developer in the team should do this (on every browser), but it works now! 🙌

Oh man that's a good point 🤔 I'll see if I can bring back the old approach + the new one

Just jumping in, had the same issue and given there's no documentation on it I didn't even think to check the UI settings for the port setting 🤷
I think setting your own port would be common enough to warrant a tiny section in the readme.
I think it should be a config passed to withDevTools for sure same as withServerDevTools... and then show up in the settings for you to change on the fly if you need to... although I don't understand why you'd ever want it to not match the wsPort?

I do confirm that that is not intuitive and having a param in the client side config would be more


Classifer creation (no coding, just 🤔🤔🤔)

Investigate what type of classifier we want to end up using. Right now, I'm thinking SVM, but there may be faster options out there. This investigation should be done for both hand/not-hand (binary) and ASL translation (multi class). Remember, we're using HoG features.

Need two classifiers, one for hand/not hand, one for the letters of the alphabet. SVM suggested for Hand/Not, random forests for letters.

Try next: use a smaller image, or downsize the HOG somehow.

https://stackoverflow.com/questions/28390614/opencv-hogdescripter-python
This shows how to get the hog of a smaller block.

Added an example of a two class SVC with webcam feedback on branch "alex". The SVC is trained with HOG computed from two images.

Zip and put dataset in slack. Mess with numbers to try and get a higher score.

http://scikit-learn.org/stable/modules/model_persistence.html
Use this to try and save the model so we don't have to train it on load every time.

I've pushed my updated hog example file to branch Alex.
Here's the data I was using
6ClassData.zip.
Things to try: crop our training data and our real-time image using the team's way of detecting the hand bounding box.


feat: add victoriametrics crds

Hi, here are the crds obtained with the crd-extractor script for the victoria metrics operator repo
Thanks in advance :)

https://github.com/VictoriaMetrics/helm-charts/blob/master/charts/victoria-metrics-k8s-stack/crds/crd.yaml


setjump / longjump (Lua) crash (IDFGH-3232)

There's something wrong with setjump / longjump. It seems that if a context switch occurs during a longjump, the context will get corrupted sometimes, resulting in stack pointer corruption and an awesome crash.
The problem doesn't seem to occur if I disable interrupts (or task switching) around the setjump / longjump calls, but that's not a proper solution.
Lua relies on setjump / longjump, so these (standard C) functions are required. I've also tried building Lua as C++ and using exceptions instead, but it doesn't work at all (immediate crash).
Test case:
https://github.com/szmodz/esp-idf/commits/sjlj_crash
Also related:
https://github.com/espressif/esp-idf/issues/5227
Example output:
Hello world!
This is esp32 chip with 2 CPU cores, WiFi/BT/BLE, silicon revision 1, 4MB external flash
Free heap: 4492760
[1000] sjlj tick 0
[1001] sjlj tick 1
[2000] sjlj tick 0
[2001] sjlj tick 1
ets Jun  8 2016 00:22:57

rst:0x8 (TG1WDT_SYS_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:4
load:0x3fff0034,len:5132
load:0x40078000,len:14484
load:0x40080400,len:4068
entry 0x40080624
W (172) boot.esp32: PRO CPU has been reset by WDT.
W (173) boot.esp32: WDT reset info: PRO CPU PC=0x4008b3f6
0x4008b3f6: _xt_context_save at /home/szmodz/invendyne/esp-idf/components/freertos/xtensa/xtensa_context.S:142

E (173) boot.esp32: WDT reset info: &s CPU not support!

Hello world!

The problem exists in the stable SDKs as well (though the symptoms might look a little different).
Do you know what is the exact cause? Any ETA?

I have the exact same problem. Longjmp does not appear to restore the stack correctly.

Just tried the latest commit 625bd5e (full checkout including compiler) and my Lua VM now seems to work just fine.

Hi @szmodz @surfskidude, We've probably found what the issue is and have been working on this already. It seems to be some corruption if a context switch occurs during the longjmp.

The issue has been fixed in https://github.com/espressif/esp-idf/commit/fc22e3c645bbdd5568d52eff77f69431776fd212.


Should notebooks be completed or clear on launch?

Currently they have been run, including the exercises!

I'll recommend clear on launch?

done - added solutions, which have solutions and have been run. made originals cleared and took out solutions


Trouble running with multiple GPUs

Howdy!
I'm able to successfully run the deep_stable example on 1 GPU, but run into problems when I try 2 GPUs or more.  For the 2 GPU example, I run into a ValueError for the following at the "Processing 2 sets of gradients." step:
 File "/global/scratch/tsmidt/harmonicConvolutions/model_assembly_train.py", line 276, in construct_model_and_optimizer     loss = tf.reduce_mean(tf.concat(0, lossesPerGPU))
The error message:
ValueError: Shape must be at least rank 1 but is rank 0 for 'concat_45' (op: 'Concat') with input shapes: [], [], [].
I've included the output from running the train command for deep_stable on 1 and 2 gpus.
https://gist.github.com/blondegeek/b2bb5266ad2bc83812366f839533a749
Any ideas why? I'm using tensorflow 0.11.0rc0, cudnn 4.0 and cuda 7.5.18.
Thanks!
Tess

Hi Tess,
Thank you very much for getting in touch. This bug has been fixed in our latest versions but not on the current state of master. I will make a fix for this release tomorrow.
However, multi-GPU is experimental for training at the moment in any case. For speed improvements, we are very soon releasing a much fast version of the code (should be this week).
I will keep you posted! Thanks again for bringing that to our attention.
Best regards
Stephan

Thanks very much for you response!


Restart Indexer agent after ETH-node problem

Noticed the following problem: I am using not very stable OE 3.0.1
In case of any problems with the ether node , the agent (and Graph-node) stops working, this is understandable.
But when the ETH node came in order (I use watchdog to restart), the Graph-indexer-agent continues to not work and, accordingly, the entire Graph-node also does not work, and this continues until it is manually restarted.
I think most indexers have this problem.

Noticed the following problem: I am using not very stable OE 3.0.1
In case of any problems with the ether node , the agent (and Graph-node) stops working, this is understandable.
But when the ETH node came in order (I use watchdog to restart), the Graph-indexer-agent continues to not work and, accordingly, the entire Graph-node also does not work, and this continues until it is manually restarted.
I think most indexers have this problem.


[BUG] Upgrade iphyton to version 7.7

Describe the bug
Some NVIDIA DLIs make use of RAPIDS containers.
In those notebooks, they use the following code in order to terminate the Jupyter notebook, to release resources for the next notebook.
import IPython
app = IPython.Application.instance()
app.kernel.do_shutdown(True)

There is a known bug (https://github.com/ipython/ipython/issues/11646) when using that command in any version of ipython prior 7.7.
I have workarounded it by manually updating the ipython version to v7.7.
conda install ipython=7.7
Steps/Code to reproduce bug

Expected behaviour
No error message displayed when executing the cells containing the code above.

Tested in 0.15, and already updated to the latest version. Thanks @kkraus14 !


[Feat] - Add Contributors Page, Updated Docusaurus, Fix Linters

Description:

This PR updates docusaurus to v2.4.3
Adds a page for thanking contributors
Updated Notes Admonitions
Adds Markdown Linter - Fixes #160 & Fixes #117

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!

This PR is a WIP as Linter is Failing

Hi @adithyaakrishna
Could you please take a look at the CI tests? Thanks.


Slow queries on larger result sets

Hi there,
I am new to this addon after working with emberfire for a good while (looking to move on due to lack of maintenance). Appreciate the work and the addon looks great!
However, one thing I've noticed is that when using query with a result set in the 1000s there is a fairly significant difference in the amount of time it takes for both addons to return the documents.
In my simple test, to retrieve 2000 documents using this.store.query, both addons took roughly the following amount of time to return:
ember-cloud-firestore-adapter: ~ 18 seconds
emberfire: ~ 4-5 seconds
I also would periodically come across the following error:
[2022-01-11T14:25:46.640Z]  @firebase/firestore: Firestore (9.6.2): FirebaseError: [code=resource-exhausted]: Too many outstanding requests.
[2022-01-11T14:25:46.641Z]  @firebase/firestore: Firestore (9.6.2): Using maximum backoff delay to prevent overloading the backend.

I have only briefly looked at the code of the addon, but is the individual fetching of docs in query() -> findQueryRecords() in adapters/cloud-firestore-modular.ts the cause of these issues?

Yes, there are individual fetches after a query but if you notice, the query uses onSnapshot and does the individual fetching inside of it. Then at the very end, it does a unsubscribe to it.
Because we kept a listener running, the individual fetches will attempt to fetch from the cache making it resolve the request significantly faster.
To be honest, it's been a while since I did this simple benchmarking and I'm not sure if it's still the case or if Firestore has changed that behavior already. Maybe it's time for me to check that out soon.

I've checked this again, and it does seem to not act as I would've remembered then. I'm gonna try to revisit the implementation to improve ng querying.

Great, thanks for looking into it!

#237 introduced the refactor


Tests?

I wonder why there are no tests at all? Or am I missing something?

This was discussed briefly in #14

Tests are lacking at the moment. I plan on adding a good suite shortly. The app was put together quite quickly but tests are going to be very much needed to maintain any sort of ability to keep the project maintained well. Expect to see these in the next few weeks.

@catphish Thank you for answer!


Beta loading/slowness issues

Clicking "View" button in search result table is slow to load
Clicking "View" Scan button is also slow to load

Hi @DeniseColbert ,
I am fixing the reference entry, I noticed the slowness issue in the Entries with reference records.
Could you confirm it, please? Please also let me know if it's still slow with records that do not have a reference entry.

Still slow with records that have no reference (linked) entry, and also still slow with records that do have a linked entry.
Maybe a little bit faster, but still not great.

@Vino-S has this now been fixed?


Resolve whether types in certain packages should be added to gti by default

The same question is also relevant for methods and functions.

All types that should be added to gti are being added to gti correctly, and package-wide -add-types flags are set whenever necessary. Therefore, there is no further action required.


Support Arch linux flavors in scripts

Generic Issue

[ ] Ensure this issue is not a bug report or feature request

Description
A clear and concise description of the issue.
Possible Solution

Additional context/Screenshots
Add any other context about the problem here. If applicable, add screenshots to help explain.

Hello there! Is this issue related to porting the Makefile to arch linux(or other non-debian based linux distros)? If so, I'm interested in it.

Do you have any pointer to which scripts I should begin porting with?

@vina-fda The Makefile has all the scripts for installing under make environment . This line starts the linux section. This should also handle arch linux and other non-debian systems.
https://github.com/microsoft/electionguard-cpp/blob/fe0286a25c65cac4375ac2c0fee3495d1dc24244/Makefile#L43

This will need to be looked at again to see if we still want/need this for version 2.0

@SteveMaier-IRT
Kindly have a look at the attached PR to resolve this issue.


colour_changed

Targets #1686
Description
Changed background color of both search and add client to the active clients.
Checklist:
have performed a self-review of my own code.

These code changes are already in review in another branch


release.nix not working

Cloning master and trying to build a release does not work:
nix-build ./release.nix -A channel
error: attribute 'channel' in selection path 'channel' not found

Hm, the channel is available in the  cfg.prefix subset, thus nix-build ./release.nix -A qchem.channel. But you are right, at least our README is incorrect in this case.


Files/folders marked as ignored in IntelliJ's project are shown as 'never used' in the .gitignore file

Specifically, I have marked the .idea folder as ignored in the project panel (or project structure -- it's the same.)
Now the /.idea entry in the .gitignore file has a yellow squiggly line saying:

'/.idea' entry is never used

Also, IntelliJ generates classes into the /out folder, and this is marked as 'never used' as well.

Other files in there (such as /nbproject, *.project and so on) apply to other developer's project structures which are not in the repo. I don't understand the value of marking these to look like errors. It might be more sensible to display them as redundant (greyed out).

Hi, I just started using PHPStorm and found this great plugin, but immediately stumbled upon this issue, my first guess was that it has something to do with excluded folders, which it obviously does.
I just wonder why is this issue closed? I think it is still reported with false positives and previous comments here tell the same tale?


Support for Angular 8

In Angular 8, there were some changes in the builder API and it's no longer compatible with the new version besides package.json version requirements.
Error:
Could not find the implementation for builder angular-builder-custom-terser-options:browser-custom-terser
Error: Could not find the implementation for builder angular-builder-custom-terser-options:browser-custom-terser
    at WorkspaceNodeModulesArchitectHost.resolveBuilder (/Users/foo/bar/node_modules/@angular-devkit/architect/node/node-modules-architect-host.js:49:19)
...

👍 I've upvoted to bring  terser customization into angular's default builder, as this I'm finding this builder really useful for my project where I want to see which component is performing the logging. So I agree that support for angular 8 would be a good enhancement to implement soon. I'll take a look to see whether I can submit a PR to help this along 😄

Hello @keenondrums - do you have an idea of when the Angular 8 support will be released? Would like test this, our project was recently upgraded to Angular 8 and the constructor.name is required. Thanks!

Is it as simple as creating a PR with peerDependencies tweaked as follows?
  "peerDependencies": {
    "@angular-devkit/build-angular": ">= ^0.13.2",
    "@angular/compiler-cli": ">= ^7.2.5",
    "@angular/compiler": ">= ^7.2.5",
    "@angular/core": ">= ^7.2.5",
    "terser-webpack-plugin": "^1.2.2"
  }

If so, I can do that pretty easily?

If the fix is in master, and you look at the recent commits, I doubt those are the only changes. Not too sure though

@JAspeling, yes you are absolutely right,  I didn't notice the latest commit which indeed contains more changes.

Guys, the fix is indeed in master, but I didn't have time to test it. As soon as I do, I'll make it an official release.

Just a heads up - I compiled the code using npm run prepack and copied the dist folder into the package in my node_modules with, and it's still giving the same error. Not sure if that's the right way of using the master branch though.

It seems it was complaining about the implementation being missing because node-modules-architect-host.js is looking for the implementation property on the builder which doesnt exist. Added that, the build on my angular 8 project succeeds.
However the issue seems to still persist.

Upvoting!
Since ng 8.2.4+ there is an issue with Terser and tinymce on a unicode regex, the only workaround is to set ascii_only: true using your builder.

I published 2.0.0-alpha.0. It works if es5 differential loading is disabled. Unfortunately, I don't have time to make it work as it seems to require to re-write too much code. My bud reported these set of builders works well. Try it instead of this one. Another way to go around the issue is to monkey-patch minify from Terser.

Hi @aigoncharov  can you please give as some repo example how to use angular-builders to keep class names?

@yantrab which version of Angular are you using?

We are actually migrating our current codebase to be minify-friendly so it doesn't depend on relaxed terser options. That's the other option people have and actually should go for.
Thanks for the plugin, but we are no longer using it :)


Create Expense is not getting reflected

I could post the create expense request successfully but not reflected in Spiitwise
Expense create completed without error but not reflected in Splitwise
OAuth: Oauth 1.0
Api: create_expense
Request (Body parameters)
users__0__owed_share : 75.0
cost : 150.0
users__1__owed_share : 75.0
users__1__user_id : my user id
users__0__paid_share : 0.0
description : "test"
payment : false
users__1__paid_share : 150.0
users__0__user_id : friend user id
Response
Response{code=200, message='OK', body='null', headers={Transfer-Encoding=chunked, null=HTTP/1.1 200 OK, Server=nginx, X-Request-Id=2915a687-0d9b-4e6e-a9f0-18bf835d1dd5, X-Runtime=0.037593, X-Content-Type-Options=nosniff, Connection=keep-alive, Pragma=no-cache, Date=Fri, 03 Jan 2020 23:32:03 GMT, Via=1.1 vegur, Referrer-Policy=origin, X-Frame-Options=SAMEORIGIN, Strict-Transport-Security=max-age=31536000, Cache-Control=no-cache, no-store, max-age=0, must-revalidate, Content-Disposition=inline; filename="response.json", X-Xss-Protection=1; mode=block, Set-Cookie=, Vary=Accept-Encoding, Expires=Fri, 01 Jan 1990 00:00:00 GMT, Content-Type=application/json; charset=utf-8}}

Does this java api support createExpense. As they have not mentioned it in the Readme


fix: add DEBUG for runner errors like :rotating_light: FOREIGN KEY constraint failed

logging FOREIGN KEY violations to error logs would have saved time debugging https://github.com/directus/directus/discussions/15020
Still ends up throwing an error.
Before
# no logs on foreign key constraint violations
...
13:05:25 🚨 FOREIGN KEY constraint failed
Error: FOREIGN KEY constraint failed
at client.transaction.connection (/home/michael/.npm/_npx/.../node_modules/knex/lib/execution/runner.js:254:35)
After
knex:runner FOREIGN KEY constraint failed [
{ table: 'platereadersample', rowid: 1, parent: 'file', fkid: 0 },
...
13:19:47 🚨 FOREIGN KEY constraint failed
Error: FOREIGN KEY constraint failed
at client.transaction.connection (/home/michael/.npm/_npx/.../node_modules/knex/lib/execution/runner.js:256:23)
...

@kibertoad this is another sqlite fix/feature, any chance of taking a look?

Coverage decreased (-0.003%) to 92.267% when pulling 549162d1dd69212371b6feef3e564d5921e6b4cd on barakplasma:patch-2 into 2dadde4214d9ee333adccfa517089647e94d23be on knex:master.

thanks for the PR but code coverage decrease, any chance of add test for the case?

@OlivierCavadenti I would be happy to add a test, but I can't find any existing test which covers that method. There was an uncovered line next to the line I added. If you could tell me which test get's to L252 of lib/execution/runner.js, that would help me improve this PR. I tried adding a test to test/integration2/schema/foreign-keys.spec.js but couldn't figure out if any of them covers L252


BEST BEST BEST!!

WORST PERFECTLY! THX!

Thank you, glad to hear it's working well for you! Feel free to open a new GitHub issue if you encounter any problems with the bot.


Breaking change in 0.2.1 from 0.2.0

After updating to 0.2.1 I am getting this error
(node:9432) UnhandledPromiseRejectionWarning: TypeError: scdl.isValidUrl is not a function
Usage:
const scdl = require("soundcloud-downloader");
scdl.isValidUrl(url)

ive found an fix for this. change all scdl to scdl.default.isValidUrl, scdl.default.getInfo, etc. hope i helped you.
cheers

idk why owner changed to this in the docs and in package but, thats it.

With the new update I introduced a new function to create new instances of SCDL and that changed the exports. You must do this now:
const scdl = require("soundcloud-downloader").default

Thanks will try this.


Fixed errors in .md files

Fixed table name syntax in GenomicAndExpression_T_test.Rmd,
and fixed errors in Somatic_Mutations.Rmd

lgtm!

I added the Processing_Raw_Data_With_Bioconductor.md file.


After upgrading supabase i get a _computedKey error

Bug report
Similar to: https://github.com/supabase/supabase/issues/27108. Open AI library causes issues due to the older version of deno used
When hitting my edge function i get the following error:
../wowzer-utopia-one [develop|✚6…2] ❯❯❯ supabase functions serve
Setting up Edge Functions runtime...
Serving functions on http://<IP_ADDRESS>:54321/functions/v1/<function-name>
Using supabase-edge-runtime-1.53.3 (compatible with Deno v1.40.3)
serving the request with /home/deno/functions/generate
runtime has escaped from the event loop unexpectedly: event loop error: ReferenceError: _computedKey is not defined
    at https://deno.land/x/openai@v4.26.0/lib/ChatCompletionStream.ts:4:14
failed to send request to user worker: event loop error: ReferenceError: _computedKey is not defined
    at https://deno.land/x/openai@v4.26.0/lib/ChatCompletionStream.ts:4:14
user worker failed to respond: event loop error: ReferenceError: _computedKey is not defined
    at https://deno.land/x/openai@v4.26.0/lib/ChatCompletionStream.ts:4:14
InvalidWorkerResponse: event loop error: ReferenceError: _computedKey is not defined
    at https://deno.land/x/openai@v4.26.0/lib/ChatCompletionStream.ts:4:14
    at async Promise.allSettled (index 1)
    at async UserWorker.fetch (ext:sb_user_workers/user_workers.js:77:21)
    at async Object.handler (file:///home/deno/main/index.ts:171:14)
    at async respond (ext:sb_core_main_js/js/http.js:162:14) {
  name: "InvalidWorkerResponse"

-->

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
When using OpenAI the code fails. This is apparent in our application code, and in the examples. It seems due to issues in the older version of deno used by supabase edge function runtime - as per the thread https://github.com/nberlette/deno_fmt/issues/2
To Reproduce
Follow steps in https://supabase.com/docs/guides/ai/examples/openai
Serve the edge function locally
Make the POST request as described in the guide
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Version of supabase-js: [e.g. 6.0.2]
Version of Node.js: [e.g. 10.10.0]

Additional context
Add any other context about the problem here.

Hello @tonypee 😁
This is an upstream issue.
It will resolve when the ongoing Deno codebase version up PR has been merged.

Yup - looks like a dup of https://github.com/supabase/edge-runtime/issues/355 anyway - ill close. thanks


No password security

No password security measures have been implemented. Do not provide any real passwords when creating user accounts.

Password hashing implemented, much appreciated Chris!


Create zxcvbnm

Collaborator: Review for this pull request was created. Link to review

Collaborator cannot find remote user mapping for pull request creator, review created by system administrator


Fix pycodestyle exclude option

This fixes issue https://github.com/OrkoHunter/pep8speaks/issues/52.
It looks like the problem is that pep8speaks is just running pycodestyle on a file called file_to_check.py so the exclude option never matches anything. We need to filter the files before this point.
I tweaked the filename_match function from pycodestyle.
NOTE: I have not tested this code.

Ah! Thanks for digging into this. :)

@OrkoHunter was this change deployed yet?


Does not work in windows

The problem is that windows paths start with a drive letter.
You are escaping the path in the function regExpQuote(str), but i think it escapes too much.
When I changed according to http://stackoverflow.com/a/6969486/1353412 to
function regExpQuote(str) {
  return (str + '').replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, '\\$&');
}

it worked well.

I have no Windows environment. Could you please change this regular expression and make PR to this repo?


feature req: ability to declaratively define sanitize/transform rules

Consider the example of storing/ quering an email field on a User. Common practice is to have this email-address lowercased and truncated before storing.
Similarly, each findX method querying for an email-address should be preceded by some code to properly transform the email-address (lowercase + truncate).
This obviously isn't DRY.
What if you could declaratively define transformation/sanitization rules per field which would kick in on CUD + all Find-methods?
Again email is just an example here. Imho it's broadly applicable.

Hi @gebrits, beforeFind (#902) and afterFind (#923) will added in waterline 0.11. I believe those would be enough to address this. Please reopen if you feel otherwise, thanks.


Fixes #3, add syntax error handling on messages

@kofno please consider this for inclusion.
Thanks for developing this project!

Thanks for your contribution!
I pulled this branch down locally to test it, and then I did a squash and merge locally, which github didn't seem to like. I'm going to close this PR, but your changes have been merged.
Thanks again!


Ion 1.1 spec needs to be updated to reflect changes in macro signatures and grouped parameters

In a macro signature, macro parameters are contained in a s-expression rather than a list
Cardinality of parameters is no longer supported
Parameter types are no longer grouped in a s-expression (e.g.: (foo int))—the type is annotated on the parameter name (e.g. int::foo)
(:), (:void), and (:values) macros are superseded by the stream syntax [: ... ]
A grouped parameter is indicated by putting [] around the variable name (e.g. [foo]). When the variable has a type annotation, the type annotation stays with the variable name (e.g. [int::foo]).
The binary encoding uses presence bits for encoding macro arguments
Presence bits are required in the binary encoding if and only if there are 3 or more parameters
For a non-grouped parameter, a 1 indicates the parameter is present, and 0 indicates argument is not present.
For a grouped parameter, 1 indicates a present expression group, and 0 indicates no expression group for that parameter.
The text encoding no longer requires delimiting containers around the arguments for a grouped parameter. Instead, the arguments should be enclosed in the stream syntax.

I'm wary of several of these proposals.  It's going to be hard to discuss with all of this under one issue.

We agreed to delay implementation of parameter cardinality, not eliminate it.  (As I work on the denotational semantic model, I'm becoming more convinced that we need more strong typing, not less.)
Removing the S-expression from parameter type declarations eliminates a critical extension point.  As an annotation, there's no good way to do anything but a type-name.
I'm not immediately in favor of the stream syntax and would like to understand its rationale.
Even given that, we still need void/values as system-defined macros for use in the template language.
We should still have both void and values.  I want to allow the former to accept and ignore any* and thus act as an easy, structure-aware mechanism to discard a chuck of data. I find myself needing this almost daily when working with Ion-based DSLs.

We agreed to delay implementation of parameter cardinality, not eliminate it. (As I work on the denotational semantic model, I'm becoming more convinced that we need more strong typing, not less.)

That's my mistake.
I've created the following issues to handle these discussions independently from each other.

#289
#290
#291


Adding wavedrom images of the encoding

Good to have these added, do we also have a pipeline to build it and see a rendered version?

Nope. I do think this is needed however, and i had sent a mail to riscv-help earlier today

Good to have these added, do we also have a pipeline to build it and see a rendered version?

Nope. I do think this is needed however, and i had sent a mail to riscv-help earlier today

for now, best thing during review is to build locally or to pase into https://wavedrom.com/editor.html

Note that if you are building locally, be sure to do git submodule update --remote --init to pull in the submodule code or the local build will fail to find the theme and fonts.
As for the wavedrom, if you add --verbose to the Makefile and build locally, it spits out these errors:
asciidoctor: DEBUG: images/wavedrom/load_store_pair.adoc: line 4: unknown style for literal block: wavedrom
asciidoctor: DEBUG: images/wavedrom/load_store_pair.adoc: line 15: unknown style for literal block: wavedrom
asciidoctor: DEBUG: images/wavedrom/c-load_store_pair.adoc: line 4: unknown style for literal block: wavedrom
asciidoctor: DEBUG: images/wavedrom/c-load_store_pair.adoc: line 15: unknown style for literal block: wavedrom
asciidoctor: DEBUG: images/wavedrom/c-load_store_pair.adoc: line 25: unknown style for literal block: wavedrom
asciidoctor: DEBUG: images/wavedrom/c-load_store_pair.adoc: line 37: unknown style for literal block: wavedrom
I'm working on figuring out why.

I have done a rework of the spec to get it closer to what current specifications look like. Previously, it was in the style of RV64 & RVC, as this is where the encodings come from. Please have a look.
riscv-zilsd-v0.0.0.pdf
Built pdf attached for those who cannot reproduce easily.

I hope this has had sufficient review. Thanks for the work on this. If you still spot an issue, please flag in a new issue.


Support 'displayName' property on custom types

Motivation
This helps the API return more useful errors when using a factory pattern to create custom types. Here's an example of how it would work:
function makeEnum(name, values) {
  function Enum(value) {
    // Validate that the assigned value is one of the possible enum values.
    return value >= 0 && value < values.length;
  }
  Enum.prototype = Number();
  Enum.displayName = name;

  return Enum;
}

const store = fortune({
  item: {
    state: { type: makeEnum('ItemState', ['open', 'closed']) },
  },
});

Changes
With this change, instead of returning
BadRequestError
The value of "state" is invalid, it must be a "Enum".

The API will now return
BadRequestError
The value of "state" is invalid, it must be a "ItemState".

Notes
displayName is a common non-standard property. You can find docs here: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/displayName
I'm happy to make any tweaks if you need me to. Thanks!

Cool, I didn't know that there was already a non-standard convention for that. I'll release it in the next patch version, thanks.


fix issue HTTPS/SSL not supported

it can be better, but work
I rework https test, and also test in my project, it works
i'm sorry about workflow, its my first pull request in public rep

@RealVlad322 for some reason, the npm build fails

@RealVlad322 thanks!


Contribution: Maurer Rose linked to scroll

Link to demo here: https://codepen.io/leimapapa/pen/PooWmow?editors=0010

@leimapapa Great project! We're experimenting with sharing projects on the Coding Train social media. Would it be alright to share this project? If so, and you would like to be tagged in any posts let me know what your instagram and/or twitter username.

Feel free to share it on social media. No need to tag me. Thanks though. 👍
-Sam
On Tue, Oct 22, 2019, 7:10 AM Violet<EMAIL_ADDRESS>wrote:

@leimapapa https://github.com/leimapapa Great project! We're
experimenting with sharing projects on the Coding Train social media. Would
it be alright to share this project? If so, and you would like to be tagged
in any posts let me know what your instagram and/or twitter username.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CodingTrain/website/pull/1809?email_source=notifications&email_token=AIU2SVBXPTVFLTSE6O6TWDLQP2DMFA5CNFSM4JDFFCZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB4RQWQ#issuecomment-544807002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIU2SVDAGGHYDZNP7JCJ5DDQP2DMFANCNFSM4JDFFCZQ
.


json.Marshal() on a result.Aggregations returns invalid JSON

When marshaling to the result of a aggregation to JSON it produces invalid JSON. The code below is the relevant snippet from my code:
termsAgg := elastic.NewTermsAggregation().
        Field("_type").
        OrderByAggregation("topScore", false).
        SubAggregation("topScore", elastic.NewMaxAggregation().Script("_score")).
        SubAggregation("types", elastic.NewTopHitsAggregation().Size(5))

searchRequest := client.Search().
        Index("myindex").
        Query(query).
        Aggregation("top_results", termsAgg)

result, err := searchRequest.Do()
if err != nil {
        //Actually doing real error handling here, but I cut that for readability
        log.Println("Error: SearchHandler:", err)
        return
}

aggRes := result.Aggregations

topResults, found := aggRes.Terms("top_results")
if !found {
        log.Println("Error: no results found")
        return
}

resultJson, err := json.Marshal(topResults)

Result (trimmed):
{
   "Aggregations":{
      "buckets":[  ],
      "doc_count_error_upper_bound":0,
      "sum_other_doc_count":0
   },
   "DocCountErrorUpperBound":0,
   "SumOfOtherDocCount":0,
   "Buckets":[
      {
         "Aggregations":{
            "doc_count":39,
            "key":"ne",
            "topScore":{
               "value":2.060434579849243
            },
            "types":{
               "hits":{
                  "total":39,
                  "max_score":2.0604346,
                  "hits":[
                     {  },
                     {  },
                     {  },
                     {  },
                     {  }
                  ]
               }
            }
         },
         "Key":"ne",
         "KeyNumber":ne,
         "DocCount":39
      }
   ]
}

Notice the unquoted value of KeyNumber.
It look like this is an oversight when solving #51, but I'm not sure.

As I said in #51: Serializing data back to JSON is not the primary use case of Elastic.
What's your use case for serializing results back into JSON?

No, I noticed that, and I realize it is a low priority bug. I thought I'd still report it though, as it still is a bug.
I use it to verify I built correct requests. I have a system generating searches, and while developing I find it handy to be able to read the response.

I recognize it'd be handy to have something like that. But I'm not sure if I'd call it a bug. Can you do the same with official clients in Ruby, Python, or JavaScript?
If you want to read the response, you can enable tracing. If you want even more control, you can specify your own http.Transport.

I think I have expressed myself unclear. The only thing I think is a bug is the unquoted string value of KeyNumber, which causes the Marshal() to generate invalid json. It's not part of the curl response, so it must be something added by elastic (the library). I'm honestly not even sure what KeyNumber is there for. Should it be some iteration value maybe? Or a numeric representation of the key?
KeyNumber is defined as a json.Number, which is really a string. However, the string in a json.Number is supposed to represent a number. There is apparently no check that the string you set a json.Number to actually do represents a number.
So, my conclusion is that KeyNumberis set to an incorrectly (as it is set to a non-number string), but I don't know what is was intended to contain, so I can't fix it for you.
I have worked around this problem, so it is in no way a critical bug (for me at least) but I still think it is a bug.

See json.Number and UseNumber as well as #26 and #37 for insight as to why there is a KeyNumber in there. IIRC the tl;dr version is that keys can be numeric values as well as strings, and we were looking for a convenient way of extracting the numeric value, which is exactly what json.Number does. So sometimes, you need use Key, sometimes you use KeyNumber.
As I said, marshaling is out of scope for Elastic (as it probably is for all official clients). Aggregations have proven to be especially tricky to deserialize, so serializing those is an extra effort I probably won't go.
However, I do wonder why json.Number does serialize as "ne" in json.Marshal. I'll leave this issue open and look into it when I have more time.

I see. Thank you for explaining!

This issue is rather old and I'm not even sure what's the problem even after re-reading the discussion :-) FWIW marshaling is out of scope for Elastic as it's just a client library.
Please re-open if this is still an issue.


FR interact with contract

Hey @cawfree - first of all, thanks for putting this together!
I was wondering if you had already considered extending the Web3 object to allow for interactions with contracts

Hey @JavierTrujilloG, sorry for taking so long to get back to you.
Yes, this is definitely on the roadmap! Unfortunately I'm busy for the next six weeks, but once that's over I'll definitely be back to introduce this functionality.
I'm currently working on an authentication solution so that we'll be able to use web3-react-native to interact with real existing wallets that are tied to user friendly login solutions like Twitter/Facebook/Google, which will make this library a lot more useful.


jigsawstack[major]: Add Jigsawtack Integration Docs

This PR includes documentation for #6665

Hey! Sorry about the delay!
We want this PR to just be the docs - the code should go into your fork at https://github.com/JigsawStack/langchainjs/
If you do that I'll merge just the docs and ship a release!

Hey! Sorry about the delay!
We want this PR to just be the docs - the code should go into your fork at https://github.com/JigsawStack/langchainjs/
If you do that I'll merge just the docs and ship a release!

Hi @jacoblee93, I have updated this PR to only contain the docs. The dir libs/langchain-jigsawstack has been removed from this PR.
Let me know if it's all good. Merci.

Hey @iamNarcisse I am so sorry for all the delays - just shipped 0.0.1:
https://www.npmjs.com/package/@langchain/jigsawstack
And will merge docs here once CI passes.
Thank you again!


[Core]RequestMetrics add preempt metrics

In vllm, when resources are insufficient,
preemptive scheduling will be performed,
so monitoring this indicator is also very
important. New preempt related indicators
are added to the metrics in RequestOutput,
including:

preempt_recompute_count: the frequency at which preemptive recomputation occurs.
preempt_swap_count: the frequency of preemptive swapping.
preempt_swap_block_count: the number of blocks preemptively swapped.

/ready

/ready

@zhouyuan  @tmm1  @orsharir  Ping !


Fixed a bug with the exec command with paths containing spaces

Fix for issue #56

thanks! on npm at version 1.10.3


Helper to know if a slot has been filled

In my component I have a wrapper around an optional slot.

      {{ @page_title }}

    <slot name="right"/>

As the slot is optional, if the user does not fill it, the result contains an empty div:

I would like to remove this empty div by knowing if the slot has been filled. Today I created a helper for this:
  defp has_slot?(%{__surface__: %{provided_templates: slots}}, slot) do
    slot in slots
  end

  defp has_slot?(_, _), do: false

And I use it like that

      {{ @page_title }}

    <slot name="right"/>

Does Surface have a more idiomatic way to know is a slot has already been filled? If not, what about adding a helper in Surface?
I would be happy to implement this feature if you wish.

Hi @Malian!
Since named slots are stored as assigns, you can do the following:

  <slot name="right"/>

However, I agree we could have a more explicit way to do that. Maybe we should create a macro so we could additionally be able to validate at compile-time if the slot has been declared or not. I'm not sure about the name though because in this case, we actually have a slot (it has been defined by the <slot> tag). It just hasn't been filled yet so has_slot? might be misleading. Here're some alternatives suggestions:

@lnr0626 any other suggestion?

has_slottable?(:name) or slottable?(:name) is consistent with the terminology on mdn about slots (https://developer.mozilla.org/en-US/docs/Web/Web_Components/Using_templates_and_slots and https://developer.mozilla.org/en-US/docs/Web/API/Slottable); though I've found that terminology to be a little bit confusing.

Thank you both for your prompt responses.
@msaraiva In the beginning, I had indeed used assigns[:right] to check whether the slot had been filled or not. Two weeks later I went back to my code and I no longer understood this condition. What was my intention? I understand that "slots" are stored in assigns from a technical point of view but it seems strange from a DX point of view. So I decided to use the provided_templates key because it seems more explicit in context and better describes my intentions.
I am willing to spend some time on this feature if you wish. In that case, could you quickly walk me through the steps needed to get things done?

I am willing to spend some time on this feature if you wish. In that case, could you quickly walk me through the steps needed to get things done?

Sure. Something like this should work:

  alias Surface.API
  alias Surface.IOHelper

  defmacro slot_assigned?(slot) do
    defined_slots = API.get_slots(__CALLER__.module) |> Enum.map(& &1.name)

    if slot not in defined_slots do
      IOHelper.warn("Undefined slot #{inspect(slot)}", __CALLER__, & &1)
    end

    quote do
      unquote(__MODULE__).slot_assigned?(var!(assigns), unquote(slot))
    end
  end

  @doc false
  def slot_assigned?(%{__surface__: %{provided_templates: slots}}, slot) do
    slot in slots
  end

  def slot_assigned?(_, _), do: false

You can add the code above to lib/surface.ex so the macro can be imported by default.
And optionally, If you really want to shine here, you can improve the warning message doing something similar to: https://github.com/msaraiva/surface/blob/911a73e705795232523e703998c2ae47e188fa6c/lib/surface/compiler.ex#L719-L747

One thing to note is that the default slot's name in the provided_templates map is :__default__

@lnr0626 good point. Maybe we should do something about that too. The main reason we use :__default__ is to keep the name :default available as it might be a common name for a property, however, I think this solution is inconsistent as it does not prevent other good names from falling into the same problem. Ideally, we could have an optional option :as for slots that would be used to name the assign. Something like:
# Defines a :default assign (not :__default__)
slot default

# Defines a :default_slot assign if you want to reuse :default as a property
slot default, as: :default_slot

This would make the behaviour of all slots, named or not, consistent.
@Malian no need to do this in the scope of this PR as it may require a few changes in many different places.


Updated README.md to refactor git commands markdown

Updated code syntax with required markdown.

Thanks again! It looks so much better.


How to access value from editor?

Thank you for creating this editor.
I have successfully implemented this editor with limited toolbar functions.
I want to know that how to get value (text that we have typed) from editor.
I have new in RN, have tried to get value but didn't get success.
Could any one of you please help to resolve this issue, as I want to sent this text to server and then do further process.
Please please help me, After successful implementation I really don't want to stuck on this point.
any help appreciate.
Thanks in advance.

You should use the onValueChanged prop of the CNRichTextEditor to listen for value changes and then store it your component state, just as the sample on the README.md


Corporate Console - When one CLA Manager is accepted/declined no other CLA Manager is able to request access for the project

Background
nirupamavkrsna CLA Manager of the Company

Launch Corporate Console
Login as CLA Manager
Create a company Samsung Data Systems
Sign CLA for project Testopia

nirupamavmentee5 Request Access to project
5. Login to corporate console as nirupamavmentee5
6. Join Company Samsung Data Systems
7. nirupamavkrsna accepts the request of nirupamavmentee5 to join the company
8. nirupamavmentee5 request access for project Testopia
9. nirupamavkrsna accepts the request of nirupamavmentee5 to become CLA Manager for the project Testopia
nirupamavmentee6 Request Access to project
5. Login to corporate console as nirupamavmentee6
6. Join Company Samsung Data Systems
7. nirupamavkrsna accepts the request of nirupamavmentee6 to join the company
8. nirupamavmentee6 request access for project Testopia
9. Observed that Request Access fails with 409 and there is no error message

Tested and verified that the issue is fixed


Prevent volume creation when heketi pod is deleted

Hello.
I noticed that deleting the heketi pod while somes PVCs are Pending  leads to heketi.db inconsistency. Repairing the heketi.db is possible but time consuming and prone to errors.
Looking for a way to prevent this to happen...
Maybe by introducing a pre stop lifecycle hook in the heketi deployment so that I can put the heketi server in read-only mode ?
Ideas welcomed here :)

This is probably something that could be explained in the heketi documentation. Maybe you want to move the conversation to the heketi project?
/cc @phlogistonjohn


[Relay][Quantization] KL-divergence-based per-layer calibration

KL divergence algorithm ported from MXNet quantization
CollectStats pass that collects input of each simulated_quantize in annotated graph into a tuple output
Support floating-point scale
max_scale as an alternative for power2_scale in weight quantization

Evaluation code
https://gist.github.com/vinx13/6f1eb1f9e2c0a8786149ee881bfcd6aa
What's left:

I added QAnnotateKind.BIAS. I'm not sure whether it is necessary. Currently there are a few tricks in handling bias (nbit_bias, valid_range, ...). It would be good to find a better solution and avoid these tricks.
In my evaluation script, I have to write quantization workflow by myself (optimize, annotate, calibrate, realize). Please also share your thought on the design of calibrate function. We need to decide how users can specify different ways of quantization (max, power2, KLD, ...)

Evaluation result on ImageNet:
max_scale for weights, KL divergence for activations:
resnet18_v1, 0.70642 / 0.89702
resnet50_v1, 0.73682 / 0.91664
resnet101_v1, 0.74484 / 0.9208
resnet18_v2, 0.70794 / 0.89832
resnet50_v2, 0.7691 / 0.93268
resnet101_v2, 0.78204 / 0.94124
power2 for weights, KL divergence restricted to power2 value for activations (use --eval-power2 option in my evaluation script):
resnet18_v1, 0.70332 / 0.89526
resnet50_v1, 0.73426 / 0.9146
resnet101_v1, 0.72434 / 0.91058
resnet18_v2, 0.70314 / 0.89618
resnet50_v2, 0.76486 / 0.93108
resnet101_v2, 0.78066 / 0.94002
These experiments are done under opt_level=2. When opt_level=3, FoldScaleAxis might generate some outliers in bias vector and cause significant accuracy drops. We should use different scales than taking the maximum for bias in this case.
cc @tqchen @ZihengJiang @eqy @ajtulloch @antinucleon @FrozenGene

This one is ready. Please review and share your thoughts on calibration api design.

@vinx13 Could you please address other comments?
We can change the calibrate API like you did for now. In long term, we should think about like calibrate(graph, mod, ctx, fcalibrate), where fcalibrate(sq_op, stats) is a callback function which can be provided by user.

@ZihengJiang @vinx13 please followup on this and let us merge soon

@ZihengJiang @jackwish please https://docs.tvm.ai/contribute/code_review.html#approve-and-request-changes-explicitly


[core] fix urljoin to not add unnecessary joiners

fixes #21273

API changes have been detected in azure-cosmos. You can review API changes here
API changes
+             enable_automatic_id_generation: bool = ...,
+             enable_automatic_id_generation: bool = ...,

/azp run python - storage - ci


Added script url as embedded.

The raw link is intended.
I would really prefer to discuss these things in an issue rather than getting unexpected pull requests, if not for the simple reason that you went to a lot of trouble for a change which wasn't needed.
Sorry that you wasted your time, but please next time open a ticket before doing the work.


updatequeries my component is not unmounting

i am using update queries and after update data is coming but on update both ui is coming means before and after rendering.
Kindly suggest what to do to unmount before rerendering ui.
Regards
gmchaturvedi

my issue is solved i am updating my updatequey with unshift in place of set method.
reagrsd
gm

Thanks!


Adding item page

Proposed Changes

On budget list is possible to click on an item taking you to the item page

Checklist

[x] Feature developed
[x] Created/updated unit tests
[ ] Comments in code
[ ] Documentation written

Coverage decreased (-1.5%) to 76.471% when pulling 71d06471e889581e195461df1a65271245a90718 on rssilva:item-page into ddf7c8e1ee591036dbf2f6285e5ae412423f4f6e on ModusCreateOrg:master.


Completness

Hi 👋
I've been developing a Firebase app, and thought about publishing my Firebase bindings, but I'm wondering if I should merge them with this package instead, however, I wonder if you plan on completely cover the Firebase SDK, and updating the firebase and purescript version or if it is ok to publish another firebase package with a different name I guess?

Hi Edgar,
thank you for reaching out. Publish new bindings, I'm not planning to
update this, let alone extend it. I'm moving away from Firebase and have no
intention of using more than RTDB and Auth.
Let me know when you've published them, I'll link to them from this
repository.
Cheers,
Willem
On Sun, Jan 16, 2022 at 5:20 PM Edgar Rodriguez @.***>
wrote:

Hi 👋
I've been developing a Firebase app, and thought about publishing my
Firebase bindings, but I'm wondering if I should merge them with this
package instead, however, I wonder if you plan on completely cover the
Firebase SDK, and updating the firebase and purescript version or if it is
ok to publish another firebase package with a different name I guess?
—
Reply to this email directly, view it on GitHub
https://github.com/mostalive/purescript-firebase/issues/18, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABLXOAH3ZW6B4MKYQZ2ISLUWL46JANCNFSM5MC7WGMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hello @mostalive , I'm trying to publish https://github.com/codingedgar/purescript-firebase.git but I cannot as the name is already reserved, may I ask you to unpublishing your package? or rather I pick a different name?


ran npm install with npm 8.3.0

Codecov Report

Merging #10976 (00aef9a) into main (84625f0) will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #10976   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        482      482
  Lines      33851    33851
  Branches    5518     5518
======================================
+ Hits       21463    21514   +51
+ Misses     10344    10291   -53
- Partials    2044     2046    +2

Impacted Files
Coverage Δ

src/standalone/api/kernelApi.ts
72% <0%> (-2%)
:arrow_down:

src/platform/msrCrypto/msrCrypto.js
16% <0%> (-1%)
:arrow_down:

src/notebooks/debugger/debuggerVariables.ts
67% <0%> (+<1%)
:arrow_up:

src/kernels/errors/kernelErrorHandler.ts
63% <0%> (+1%)
:arrow_up:

.../notebooks/controllers/vscodeNotebookController.ts
82% <0%> (+1%)
:arrow_up:

src/notebooks/debugger/kernelDebugAdapterBase.ts
79% <0%> (+2%)
:arrow_up:

src/kernels/kernelFinder.base.ts
85% <0%> (+2%)
:arrow_up:

src/kernels/jupyter/remoteKernelFinder.ts
92% <0%> (+3%)
:arrow_up:

...kernels/jupyter/preferredRemoteKernelIdProvider.ts
97% <0%> (+4%)
:arrow_up:

src/notebooks/debugger/debuggingManagerBase.ts
74% <0%> (+5%)
:arrow_up:

... and 6 more


Some homebrew statbocks not properly importing

I tried importing a couple homebrew monsters and they're showing up in Foundry with CR 1, AC 14, 10 hp, 0 ft movement, no alignment, no creature type, no languages, no resistances/immunities/conditions, and no features (other than Multiattack). Quoted below is one of the homebrew monsters, an advanced bodak. I'm using 5E Statblock Importer ver <IP_ADDRESS>

Bodak Custodian
Large Undead, Chaotic Evil
Armor Class 16 (natural armor)
Hit Points 212 (25d10 + 75)
Speed 30 ft.
STR
24 (+7)
DEX
18 (+4)
CON
17 (+3)
INT
7 (-2)
WIS
14 (+2)
CHA
18 (+4)
Skills Perception +12, Stealth +13
Damage Resistances Cold, Fire; Bludgeoning, Piercing, and Slashing from Nonmagical Attacks that aren't Silvered
Damage Immunities Necrotic, Poison
Condition Immunities Charmed, Frightened, Poisoned
Senses Darkvision 120 ft., Passive Perception 22
Languages Abyssal, Common The languages it knew in life
Challenge 14 (11,500 XP)
Proficiency Bonus +5
Death Gaze. When a creature that can see the bodak’s eyes starts its turn within 30 feet of the bodak, the bodak can force it to make a DC 17 Constitution saving throw if the bodak isn’t incapacitated and can see the creature. If the saving throw fails by 5 or more, the creature is reduced to 0 hit points unless it is immune to the frightened condition. Otherwise, a creature takes 16 (3d10) psychic damage on a failed save.
Unless surprised, a creature can avert its eyes to avoid the saving throw at the start of its turn. If the creature does so, it has disadvantage on attack rolls against the bodak until the start of its next turn. If the creature looks at the bodak in the meantime, that creature must immediately make the saving throw.
Sunlight Hypersensitivity. The bodak takes 5 radiant damage when it starts its turn in sunlight. While in sunlight, it has disadvantage on attack rolls and ability checks.
Unusual Nature. The bodak doesn’t require air, food, drink, or sleep.
Actions
Multiattack. The bodak makes two Fist attacks and can use Wither Gaze.
Deathly Fist. Melee Weapon Attack: +6 to hit, reach 10 ft., one creature. Hit: 14 (2d6 + 7) bludgeoning damage plus 18 (4d8) necrotic damage. The target must succeed on a DC 16 Constitution saving throw or its hit point maximum is reduced by an amount equal to the damage taken. This reduction lasts until the target finishes a long rest. The target dies if this effect reduces its hit point maximum to 0.
Withering Gaze. One creature that the bodak can see within 60 feet of it must make a DC 17 Constitution saving throw, taking 22 (4d10) necrotic damage on a failed save, or half as much damage on a successful one.
Bonus Actions
Aura of Annihilation. The bodak activates or deactivates this deathly aura. While active, the aura deals 5 necrotic damage to any creature that ends its turn within 30 feet of the bodak. Undead and Fiends ignore this effect.

Thanks, it should be fixed now in v1.91.2. Let me know if you still have issues after updating the module.

That appears to work! Thanks again for the quick fix!


Add CI testing to Gitjob

Add automated CI testing.
There are currently minimal tests (only one package has them and it's less than 4% coverage). But, we should run tests and add tests. At the least, it will make sure the code builds.

[ ] add tests for the other examples (they require much more setup)
[ ] port tests from fleet here (might be faster, but would reduce coverage of fleet's setup)
[x] add test for ssh (seems necessary to catch that early) https://github.com/rancher/gitjob/pull/83
[x] fix helm package script while at it (helm chart release is unusable without deleting the crd folder) https://github.com/rancher/gitjob/pull/80

Missing / postponed:

[ ] test for webhook, termination examples
[ ] behavior might be better tested by integration tests (external git repo control and setup is too much infrastructure)

[ ] crd spec fields (by commit)
[ ] triggers, enqueues, ..


Make it look nice on mobile

Assuming scatter works nicely with Firefox for Android and other mobile browsers, the side should adapt to the device screen size

Would absolutely love to see better mobile integration, we'll look further into it when we see Scatter come to mobile.

Depends on https://github.com/EOSEssentials/Scatter/issues/106

This also affects desktop users

Not only nice but explore alternatives with IOS Secure enclave maybe. add PKs with battle and pet permissions only. let's explore.


Ansible Exchange system install ISO error

Operating System Version: 20H2 19042.1110
Deploying via (VirtualBox/VMWare/AWS/Azure/ESXi): ESXi
Vagrant Version (if applicable):

Please verify that you are building from an updated Master branch before filing an issue.
Description of the issue:
While building the exchange host, I'm running into the following error message that causes the build to stop:
TASK [exchange : Installing Exchange] ****************************************************************
changed: [<IP_ADDRESS>] => {"changed": true, "cmd": ".\\install-exchange.ps1", "delta": "0:04:21.994670", "end": "2021-08-11 08:14:16.792901", "failed_when_result": false, "rc": 0, "start": "2021-08-11 08:09:54.798231", "stderr": "", "stderr_lines": [], "stdout":

"[20:09] [+] Attempting to install Microsoft exchange.\n[20:09]
[+] Please note, you will have to reboot and re-run this script after the prerequisites have been installed.\n[20:09]
[+] Failure to reboot will cause the Exchange installation to fail.\n[20:09]
[+] Temporarily re-enabling TrustedInstaller and Windows Update services...\n[20:09]
[+] It appears the Exchange prerequisites have been installed already. Continuing installation...\n[20:09]
[+] The Exchange ISO was already downloaded. Moving On.\n
[20:09] The Exchange ISO doesn't appear to be mounted.\n
[20:09] Mounting the Exchange 2016 ISO...\n
[20:09] Something went wrong downloading or mounting the ISO...\n\r\n
SDelete v2.04 - Secure file delete

The ISO has been successfully downloaded, however there appears to be an issue with recognizing that the ISO has been mounted.
Logging into the VM and running the install script manually, I am able to see the ISO has been successfully downloaded and mounted.
The script appears to fail at line 185 in the "install-exchange.ps1" script: https://github.com/clong/DetectionLab/blob/master/Vagrant/scripts/install-exchange.ps1#L185
The installer script appears to create an additional drive letter (E: then F:) to mount the ISO per each attempt, and the script as written is failing due to check that it mounted to the D: drive letter.

Attempted to recreate and resolve this issue. New problem: Script errored at the dotnet offline installer download. The url has changed.
Line 18 should now be:
$dotNetInstallerUrl = 'https://download.visualstudio.microsoft.com/download/pr/4312fa21-59b0-4451-9482-a1376f7f3ba4/9947fce13c11105b48cba170494e787f/ndp471-kb4033342-x86-x64-allos-enu.exe'

Fixed in your PR! #702


feat: allow react/jsx-dev-runtime imports

Description
This PR contains the following changes:

Imports of react/jsx-runtime and react/jsx-dev-runtime are now resolved with the same module ID and globally deduplicated
Skip over the actual react/jsx-runtime.js module (which uses process.env.NODE_ENV checking) in favor of manual resolution using the isProduction local variable
Ensure both react/cjs/react-jsx-dev-runtime.development.js and react/cjs/react-jsx-runtime.production.min.js are processed by Vite's optimizer

Additional context
Fixes #6215

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Not sure how those failing tests could possibly be related to this PR 🤔

Closing as we have now merged https://github.com/vitejs/vite/pull/8546, and test are still failing here. @aleclarson if you think we should continue to discuss the dedupe part of this PR, let's check it in a new PR against main.


⚠ Bofrost.hr has degraded performance

In 2310b7e, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 2137 ms

Resolved: Bofrost.hr performance has improved in 0059a06 after 6 minutes.


fix: the behavior of SpinOp.to_matrix for multiple spin sites

Summary
Closes #1187
Details and comments
The expected results of the unittests which I added here have been verified using the older SpinOp as implemented in Qiskit Nature 0.4

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 87.472%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
9405

Relevant Lines:
10752

💛  - Coveralls


Hide packages structure from the import declarations

Currently, we need way too many imports to access the different packages in the library, e.g., data structures, distributions, etc. It would be better to have just one import with everything.

I believe I included this in the last PR, so you just need to write from privugger.transformer import * and then you get all the stuff.

I believe I included this in the last PR, so you just need to write from privugger.transformer import * and then you get all the stuff.

This seems to work for some, but not all, functions. For instance, I cannot access the constructor Dataset. I think we should configure modules so that we simply write import privugger as pv and then use pv to call the corresponding function.

Fixed on #33


License

What is the license for this gem? MIT?

Yes. Can't believe I overlooked that. :) Will put that in the repo ASAP.
--
Stephen Touset
<EMAIL_ADDRESS>On Nov 3, 2011, at 1:59 PM<EMAIL_ADDRESS>wrote:

What is the license for this gem? MIT?

Reply to this email directly or view it on GitHub:
https://github.com/stouset/twitter_bootstrap_form_for/issues/17

Fixed. 5c3e7c606fa50712067458957e82fd500a1c857b

Thanks!

1.0.3 has been released with this fix included.


Removed uphill creation of parent objects.

When we do this, the corresponding elements are NOT created in the neutron DB and that causes big problems.
Of 189 tests, we have...
170 Passing
7 Failing
12 skipped

Travis bombed the last test run due to a connection timeout.


fix kubectl apply cmd in ctrlmesh try demo doc

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N

[ ] Y

2. What is the scope of this PR (e.g. component or file name):
docs/ctrlmesh/started/try.md

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

I have read the CLA Document and I hereby sign the CLA

recheck

Doc has been updated in

#421


Add typescript Support

If I try to use the library in a typescript project I get the following compilation error:
TS7016: Could not find a declaration file for module 'lottie-web-vue'. './node_modules/lottie-web-vue/dist/lottie-web-vue.umd.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/lottie-web-vue` if it exists or add a new declaration (.d.ts) file containing `declare module 'lottie-web-vue';`

My main.ts (vue 3):
import { createApp } from 'vue'
import LottieAnimation from 'lottie-web-vue'
// ...

createApp(App).use(LottieAnimation);

It would be great, if you could adopt typescript support for the library :)

It turned out there is a pretty simple solution :), thx to https://blog.kewah.com/2018/typescript-declaration-file/
in the tsconfig.json:
{
  "compilerOptions": {
    "noImplicitAny": false
  }
}

I leave the ticket open. It would be great to have the d.ts file for this lib eventually.

Hi @Andreas-Schoenefeldt - I've been working on a new Typescript declared plugin that has just been released (v2.0.0). I'm testing it now and it's been entirely rewritten to support this ticket.

Great news 🥳


Add an explicit type on enumMaps

This works around a bug in the NNBD SDKs. It should not be necessary in
the long term, but also shouldn't hurt anything.
Work around https://github.com/dart-lang/sdk/issues/41180

@natebosch – I can wait publishing 3.3.0 until this lands...if you like
...but I'm guessing you'll need to regenerate code

I'm guessing you'll need to regenerate code

Yeah probably. Wanted to get this to you ASAP so you could decide if you want to run with it or not.
Another option is if we know the types of the values we could use a real type instead of dynamic and add it as an additional generic on the other methods. I don't know if we always have a type for that.
Yet another option would be to ignore it and hope that the SDK gets fixed soon. I mostly had this change because when this fails it's really annoying to figure out what's happening as @jakemac53 and I discovered.

I'm guessing you'll need to regenerate code

Yeah probably. Wanted to get this to you ASAP so you could decide if you want to run with it or not.
Another option is if we know the types of the values we could use a real type instead of dynamic and add it as an additional generic on the other methods. I don't know if we always have a type for that.
Yet another option would be to ignore it and hope that the SDK gets fixed soon. I mostly had this change because when this fails it's really annoying to figure out what's happening as @jakemac53 and I discovered.

Where are we hitting this w/ the migration?

I ran into this because I happened to be using the NNBD sdk when trying to build build_modules.
All json serializable things that use enums are broken in the NNBD sdk.

I ran into this because I happened to be using the NNBD sdk when trying to build build_modules.
All json serializable things that use enums are broken in the NNBD sdk.

This is going to blow up all tests that validate build when it's released.
I'd rather wait for the SDK to get fixed. Doesn't seem urgent at all now.

I'd rather wait for the SDK to get fixed. Doesn't seem urgent at all now.

Fair enough. We'll just need to make sure we're not using the NNBD SDK locally when we try to do builds until this is fixed.


Fixed NPM link

Old link seemed to be pointing to a 404 page.

Coverage remained the same at 96.816% when pulling ef212af282ade7eeb094fe185a24df881924ebf3 on AlexanderGrooff:patch-1 into 56fdb81fc1613870eb4540ccc291cdd387812378 on btelles:master.


Add different versions of SPM

Currently, the dev version of SPM is installed by default. Users should be able to choose the version of SPM they want, including dev.
Todo

[ ]  Determine which MCR version goes with the different versions of SPM.

Addressed by #151


Update Helm release topolvm to v11

This PR contains the following updates:

Package
Update
Change

topolvm
major
10.0.0 -> 11.0.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Caution: https://github.com/topolvm/topolvm/blob/main/CHANGELOG.md#0160---2022-12-05 (!)

Caution: https://github.com/topolvm/topolvm/blob/main/CHANGELOG.md#0160---2022-12-05 (!)

TL;DR: Make this upgrade a breaking change, and strongly recommend to set helm_values.useLegacy: true for existing setups in the major update migration guide, but let the component setup TopoLVM with the topolvm.io apiGroup for new installations.


Allow for tags to be marked as alternatives.

So
function what() {
  //<< tagname
  // will show up by default
  //>>
  //<< tagname:altname
  // will show up when using { tagname: 'altname' }
  //>>
}

Already had this as a ticket.


Convert as conversions to From/TryFrom

The as conversions in our protocol code our quite dangerous and I think we might have a few integer overflow bugs in there. In general I think we should replace all as conversions with From and TryFrom to make it clear if the conversion can fail or not. There is also a clippy lint as_conversions which should be added to our code base to prevent regressions.

👋 I'd like to work on this.


Make Dialyzer-safe

We want this project to pass through Dialyzer/Dialyxer without complaint.

Done.


display browser version in jlineup report

It would be great to see the browser version in order to see if the bug shown in a report depends on a certain version.

Good idea, already implemented! Thank you!


Profile and check performance

This is a very small tool, yes. I still want to check how much CPU and RAM it uses. I strongly believe that any software should try to optimize the usage of resources so that user's resources are not hogged :)

Should we run these tests for every push? Just to see how the numbers are? Hmm

Should we run these tests for every push? Just to see how the numbers are? Hmm

Info - https://github.com/karuppiah7890/issues-info/tree/main/karuppiah7890/helm-schema-gen/7

Info - https://github.com/karuppiah7890/issues-info/tree/main/karuppiah7890/helm-schema-gen/7

The numbers look sensible for a tool that executes one time and then stops in a glimpse. I'm still going to check where the memory and CPU is used for a good understanding :)

The numbers look sensible for a tool that executes one time and then stops in a glimpse. I'm still going to check where the memory and CPU is used for a good understanding :)


[Bug] [deploy] Database setting invalid when deploy with install.sh

Search before asking

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

What happened
I change the config in https://github.com/apache/dolphinscheduler/blob/2ff76c7e7330c166fba1e514aba6ddf2dbe74f57/script/env/dolphinscheduler_env.sh#L30-L35 to mysql it work when I directly run command dolphinscheduler-daemon.sh start <server> but failed in install.sh
ref: https://github.com/apache/dolphinscheduler/pull/7549#issuecomment-1006639653
What you expected to happen
ATT
How to reproduce
ATT
Anything else
No response
Version
dev
Are you willing to submit PR?

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

Code of Conduct

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

Fix by #8100