Desktop ScreenRect crash error

I am using Unity 2020.3.30f1 version. I installed Agora SDK and started screen sharing. I used the StartScreenCaptureByScreenRect function to share the monitor area and Unity crashes.
What's the problem?
I have 2 monitors and each has a different resolution.

hi @seungsunkim  could you pls share the crash stack trace with us?

hi @seungsunkim could you pls share the crash stack trace with us?

This issue also happens on me with Unity 2020.3.2f1.
And the stacktrace is as bellow:
Stacktrace:
at  <0xffffffff>
at (wrapper managed-to-native) agora_gaming_rtc.IRtcEngineNative.updateVideoRawData (intptr,uint) [0x0000a] in :0
at agora_gaming_rtc.VideoRender.UpdateVideoRawData (uint,intptr,int&,int&) [0x00013] in D:\Agora_Unity_SDK_FULL\samples\Hello-Video-Unity-Agora\Assets\AgoraEngine\Scripts\AgoraGamingSDK\videoRender\VideoRender.cs:93
at agora_gaming_rtc.VideoSurface.Update () [0x000f4] in D:\Agora_Unity_SDK_FULL\samples\Hello-Video-Unity-Agora\Assets\AgoraEngine\Scripts\AgoraGamingSDK\videoRender\VideoSurface.cs:122
at (wrapper runtime-invoke) object.runtime_invoke_void__this__ (object,intptr,intptr,intptr) [0x00020] in <695d1cc93cca45069c528c15c9fdd749>:0
=================================================================
Got a SIGSEGV while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application.
Received signal SIGSEGV
Stack trace:
0x00007ffb7f6b7e46 (agoraSdkCWrapper) deleteEngine
0x0000018ba2f5db86 (Mono JIT Code) (wrapper managed-to-native) agora_gaming_rtc.IRtcEngineNative:updateVideoRawData (intptr,uint)
0x0000018ba2f5d973 (Mono JIT Code) [VideoRender.cs:93] agora_gaming_rtc.VideoRender:UpdateVideoRawData (uint,intptr,int&,int&)
0x0000018ba2ef786b (Mono JIT Code) [VideoSurface.cs:122] agora_gaming_rtc.VideoSurface:Update ()
0x0000018b94aed3f8 (Mono JIT Code) (wrapper runtime-invoke) object:runtime_invoke_void__this__ (object,intptr,intptr,intptr)
0x00007ffb9e79f280 (mono-2.0-bdwgc) [mini-runtime.c:2827] mono_jit_runtime_invoke
0x00007ffb9e722ac2 (mono-2.0-bdwgc) [object.c:2921] do_runtime_invoke
0x00007ffb9e72bb1f (mono-2.0-bdwgc) [object.c:2968] mono_runtime_invoke
0x00007ff7172e29c4 (Unity) scripting_method_invoke
0x00007ff7172daeb5 (Unity) ScriptingInvocation::Invoke
0x00007ff7172865f4 (Unity) MonoBehaviour::CallMethodIfAvailable
0x00007ff7172866fc (Unity) MonoBehaviour::CallUpdateMethod
0x00007ff71673d398 (Unity) BaseBehaviourManager::CommonUpdate
0x00007ff7167472ea (Unity) BehaviourManager::Update
0x00007ff716bbb73d (Unity) InitPlayerLoopCallbacks'::2'::UpdateScriptRunBehaviourUpdateRegistrator::Forward
0x00007ff716b9c19c (Unity) ExecutePlayerLoop
0x00007ff716b9c273 (Unity) ExecutePlayerLoop
0x00007ff716ba30a9 (Unity) PlayerLoop
0x00007ff717ffc4a1 (Unity) PlayerLoopController::UpdateScene
0x00007ff717ffa109 (Unity) Application::TickTimer
0x00007ff71899b4a1 (Unity) MainMessageLoop
0x00007ff71899f4e1 (Unity) WinMain
0x00007ff71a7fdef6 (Unity) __scrt_common_main_seh
0x00007ffc2db854e0 (KERNEL32) BaseThreadInitThunk
0x00007ffc2eea485b (ntdll) RtlUserThreadStart

I am using Unity 2020.3.30f1 version. I installed Agora SDK and started screen sharing. I used the StartScreenCaptureByScreenRect function to share the monitor area and Unity crashes.
What's the problem?
I have 2 monitors and each has a different resolution.

hi @seungsunkim could you pls share the crash stack trace with us?

I have solved this by realloc the buffer size.
In the VideoSurface.cs of the demo, there is a constexpr private System.IntPtr data = Marshal.AllocHGlobal(1920 * 1080* 4);
For the client(audience, receiver):
It will crash while the size of buffer sent by the native method updateVideoRawData is greater than the allocated size.
For the server(broadcaster, sender):
The third parameter of method StartScreenCaptureByScreenRect is the struct ScreenCaptureParameters.
It will crash while the ScreenCaptureParameters.dimensions.width * ScreenCaptureParameters.dimensions.height is greater than the allocated size.

@Arlen-LT - not sure if you only want to share an area of the screen or it is the display you want to share.  In the latest SDK version, you may call startScreenCaptureForDisplay() now for the display capture.


use string template instead of pystache for script/param substition in Python client

Coverage remained the same at 88.845% when pulling 1cf219a109423a7661595b3965ce35cf3fd5cb69 on irontable:develop into 12acfcf32c6d8d66322daa8bc08c2bad1cf4ccda on Netflix:release/3.0.0.


Callback when start/end dragging from any snapping point

onOpenStart only fire when dragging from bottom most snapping point
onCloseStart only fire when dragging from top most snapping point

Please help me with this callback

Facing the same issue, have you found a solution/alternative regarding this issue?


New Flutter project templates shouldn't reference dependencies that aren't in the standard Flutter package

The latest Flutter Project template in Android Studio contains a dependency reference to flutter_lint which isn't included as a part of the standard Flutter Installation. This can cause issues when attempting to start a new project while not connected to the internet since Android Studio doesn't support pub get in 'offline' mode and running flutter pub get --offline in Chipmunk builds isn't picked up by the IDE. This leaves you unable to use code inspection, build runners, or other dart lang features needed to work on your project.   It would be helpful in the future to not include references to libraries or packages that aren't a part of the standard Flutter environment.
flutter_doctor.txt
Flutter Plugin 70.0.2
Android Studio Chipmunk | 2021.2.1 Patch 2
Build #AI-212.5712.43.2112.8815526, built on July 10, 2022
Runtime version: 11.0.12+0-b1504.28-7817840 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 11.6.8
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 8
Registry: external.system.auto.import.disabled=true
Non-Bundled Plugins: Jetbrains TeamCity Plugin (2021.2.99602), Dart (212.5744), io.flutter (70.0.2)

Hello @strangedreamsNY. This issue is related to the IntelliJ plugin for Flutter, therefore, it might be better if it was raised in flutter-intellij. Closing as this is not an issue with Flutter itself.

Just ran a quick check and I can verify that this is behavior of Flutter itself and not limited to any plugins or 3rd party providers.
Running the following command: flutter create demo --project-name demo_app --template=app --offline results in the error message; "Because demo_app depends on flutter_lints any which doesn't exist (could not find package flutter_lints in cache), version solving failed."

Looks like from to error, you need to cache flutter_lints. You can see how to add a package into your cache here. Basic gist of it is, you'll need to run flutter pub cache add flutter_lints while being connected to a network in order for the pub tool to download it, and then you should be able run that command.


Support signed commit in trunk-io/trunk-action/upgrade

Let me open the feature request in the repository, since trunk-io/trunk-action doesn't allow me to create issues.
Currently, trunk-io/trunk-action/upgrade doesn't support signed commit. If we turn on the setting that commit is required in a GitHub repository, the check doesn't pass. So, it would be great to support signed commit in the action.

Hi @yu-iskw! Thanks for bringing this up! I'll talk to the owning team about making sure that repo supports issues.
In the meantime, we'll look into seeing if we can get signed commits working for that action.

@yu-iskw we've released<EMAIL_ADDRESS>which has an option for signoff. This should be sufficient for the DCO app.
If this is insufficient and you also need the commit to be signed and verified, please let us know and we can look into supporting that.

@TylerJang27 Thank you so much. I will give it a try.


How to get this keyboard to work with shadow-dom?

Hello!
First of all, great work on this keyboard, I have it fully functional on my application with react and it works great.
My question is, I have another project that uses shadow-dom, and when I implement it the same way I did on the other project I get a Keyboard_dom_error.
I've tryed to add the simple-keyboard div inside the shadow-root-document but that didn't work, could you shed some light as to how to get this to work? This app is also build in react with shadow-dom
Thanks

Hey @robergwillian,
You normally get KEYBOARD_DOM_ERROR when the container div is not found when the constructor is fired. This is not likely to happen in React (where react-simple-keyboard is likely to be used) since the div is added in the renderer and the constructor is called in the useEffect.
So my questions are:

Are you using react-simple-keyboard?
If not, is there a reason why you chose this approach? (Generally React users opt for react-simple-keyboard).
Could you provide a repro that I could look into (e.g.: a test git repo or fork of one of the demo sandboxes?)

There's many things that could cause an issue, but they're most likely related to setup so a repro will definitely help.
I will reopen once I get the repro, thanks!

Hey, thanks for you quick reply.
I'm using the react-simple-keyboard. I created a copy of the repo I'm using, its a private, but I've added you to have access to it.
branch: ew-249-custom-keyboard
my keyboard component is at components/common/keyboard
and the keyboard is used at components/signup/primaryinfo/index
on the app main index you will see some commented lines, those lines switch between using the regular dom or the shadow-dom, if you uncomment those lines and comment the shadow-dom, the keyboard works fine, but when using the shadow-dom we get the error and the keyboard will not show up.
let me know if I can clarify anything else

Hi @robergwillian,
I have pushed an update to<EMAIL_ADDRESS>which should partially address this use-case.
The keyboard should now render normally (no more KEYBOARD_DOM_ERROR). However, imported styles are not being added at the ShadowRoot level. I'm not sure if this is due to missing Webpack configuration or otherwise, but I think diving into potential Webpack tweaking goes beyond my scope.
If you do find a solution for this please let me know. I will make sure to update this thread if I come across any new info.
Regards,
Francisco Hodge


Redefined function call with proper parameters

The following error in make has been fixed @tillkamppeter @michaelrsweet

Have you tested building with both libcups2 and libcups3?

@tillkamppeter also updated install.md

Note that the CUPS master branch is NOT libcups3, the CUPS master branch is CUPS 2.5.x.
To compile with libcups3, install libcups3 (does not overwrite anything of installed CUPS 2.5.x, then, in libcupsfilters and libppd do
make distclean
./configure
make
sudo make install

Check the output of ./configure to see whether it really selected libcups3 and not libcups2.

Compiled with libcups 3 .... works perfectly @tillkamppeter


DEVX-2365

Description

Related Issue
https://jira.corp.adobe.com/browse/DEVX-2365

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@AmyJZhao , I'll create a separate ticket for that, new functionality added in this PR are covered by 100%


(#5489) Use yaml in e2e install

This may have failed for unrelated reasons, could we pls give this another shot?

Let me know if this is good to merge, thanks!

Ready to merge


Refactor test modules

There was a lot of code duplication in the various test modules. This PR creates a base class that contains most of the duplicated code. Modules used for testing now inherit from that base class, then add extra methods via mixins, which eliminates most of the duplication. This should make maintaining and writing new test cases easier in the future.
For example, if we want to test a LightningModule withtest_step and test_end implemented, but no validation, we can just do
class CurrentTestModel(LightningTestMixin, LightningTestModelBase):
    pass
model = CurrentTestModel(hparams)

A couple other minor changes along the way:

Modified how Trainer.__is_overriden works so that it functions correctly with more complicated inheritance. It now compares sub-class code to the original LightingModule code, rather than the sub-class's direct parent, which is what super gives you.
Found, fixed, and added a new test for a bug that would occur when calling test on a model with no val_dataloader

@neggert this is brilliant :)  thanks for simplifying this! mind deconflicting?

@williamFalcon Rebased and fixed the conflicts. Also did a little more work. Editing the original message to reflect the new stuff.

@neggert cool mixins. I'm torn about the mixins. While I agree they're a nice clean solution, it seems like it will make it harder for your non-engineer to add tests when contributing. Since Lightning code is designed to be readable by non-engineers, I'm not sure whether it overcomplicates things.
Maybe the way to overcome this is to document the mixins in the test README?
Any thoughts on that?

Hey, sorry for vanishing. I think in the long run this will be simpler for non-engineers (and maintainers), but yes, the docs should be updated. Looks like that README is getting a bit out of date anyway. Might be worth giving some thought to how tests are organized. It looks like at the moment, everything is just getting crammed into test_model.py.


updated npm broken link issue documentation #6556

Updated link  in readme files for displaying across npm
1)https://www.npmjs.com/package/@storybook/addon-actions
2)https://www.npmjs.com/package/@storybook/addon-storyshots
Let me know if any doubts.
Fixed #6556
Thanks

No problem , most welcome.
Just wanted to get an idea when it will be reflected to npm pages?

Maybe next time I publish the packages? I'm not sure how it works tbh

Oh cool.. :D


Fix yq stanza for generating repo files

Fixes https://github.com/rancher-sandbox/cOS-toolkit/issues/1339
Signed-off-by: Itxaka<EMAIL_ADDRESS>

not wiating for tests, it makes no sense, release pipeline


TypeError: Data must be a string or a buffer

Library Affected:
workbox-webpack-plugin v3.4.1
Browser & Platform:
N/A
Issue or Feature Request Description:
I'm using the workbox-webpack-plugin in a Typescript project like so:
new WorkboxPlugin.InjectManifest({
      swSrc: path.resolve(__dirname, 'src', 'app', 'utils', 'service_worker.js'),
      swDest: path.resolve(__dirname, 'static', 'service_worker.js'),
      globDirectory: path.resolve(__dirname, 'src'),
      globIgnores: ['**/*.tsx?'],
      globPatterns: ['**/*.{png,jpg,ico,svg,html,css}']
    })

However, when I boot up Webpack I get the following exception:
TypeError: Data must be a string or a buffer
    at Hash.update (crypto.js:99:16)
    at module.exports (/Users/jakemarsh/repos/monolist/web-client/node_modules/workbox-webpack-plugin/build/lib/get-asset-hash.js:28:35)
    at generateMetadataForAssets (/Users/jakemarsh/repos/monolist/web-client/node_modules/workbox-webpack-plugin/build/lib/get-manifest-entries-from-compilation.js:239:15)
    at getManifestEntriesFromCompilation (/Users/jakemarsh/repos/monolist/web-client/node_modules/workbox-webpack-plugin/build/lib/get-manifest-entries-from-compilation.js:317:23)
    at InjectManifest._callee$ (/Users/jakemarsh/repos/monolist/web-client/node_modules/workbox-webpack-plugin/build/inject-manifest.js:143:27)

When I added some logging, it seemed to be throwing this error when attempting to process a Typescript definition file. I assumed that would be covered by globIgnores? I also tried using just exclude. Am I missing something here?
I also ran into the same issue trying to use next-offline, which uses GenerateSW rather than InjectManifest.

Hello @jakemmarsh!
Taking a step back from your question, the glob-related options are meant to be used when you need to precache assets that exist outside of the webpack asset pipeline. So I wanted to confirm that you actually need to cache non-webpack assets for your web app—if not, you can just leave that out, and that might clear up your issue.
(The planned revamp of the webpack plugin will make this configuration more explicit, as it's unfortunately confusing at the moment.)
If you do need to use the glob options to cache non-webpack assets, it would be good to see the full webpack config that you're using, if possible, as well as how you're invoking webpack (using the dev server, for instance, or some other method).


ci: also gate on latest Java

See https://github.com/tigerbeetle/tigerbeetle/pull/1357 for discussion.

(will hold off merging until we fixed the java failure, to not increase flakiness rate)

(will hold off merging until we fixed the java failure, to not increase flakiness rate)

Let's merge it!


Package not showing up in pypi search.

Hi sorry, I could not find anyone to contact hence I am opening this issue. I released a package name "torchflare" just about 3 hours ago. I can see the package listed when I log into my account and also accessible when I go the directlink: https://pypi.org/project/torchflare/ but when I search it on pypi website I cannot seem to find it.
Please help me

Looks like it works now. Maybe there was just a backlog of re-indexations. Can you confirm it works for you now ?

It works now thank you ! I am closing this issue now.


mans : (little) Network Port Scanner

Feature description

Ability to launch network scans "like" nmap.
Not all features will be available at first.
mans is only available through mashREPL

For @isontheline work :

Create classes that receive delegates (scan updates / errors)
For mans command line : show progressive scans (refresh lines)
For GUI : show progressive scans via tableview ?

https://stackoverflow.com/questions/56167320/ios-swift-how-to-check-if-port-is-open
https://github.com/swiftsocket/SwiftSocket
https://stackoverflow.com/questions/54640996/how-to-work-with-udp-sockets-in-ios-swift
https://nmap.org/book/man-port-scanning-techniques.html

Args parser : https://github.com/apple/swift-argument-parser

[ ] 36 concurrent instead of 4
[ ] Not shown : 996 filtered tcp ports (no-response)
[ ] And it’s apps
[ ] top ports by default
[ ] mans —help : mans-options
[ ] mans -Pn
[ ] mans : argument timeout (filtered)

Common Ports : https://github.com/pymike00/p_scan/blob/main/src/common_ports.json
IANA Ports : https://github.com/itsalljustdata/ports/blob/main/ports.json
Commonly Open Ports : https://www.speedguide.net/ports_common.php
Common Ports : https://packetlife.net/media/library/23/common_ports.pdf
Common Ports Cheat Sheet : https://www.stationx.net/common-ports-cheat-sheet/

Nice!
Testing now!
Does it support domain names or just IPs?
Have some inconsistency with nmap output when trying to scan google.com

It supports both IP/DNS
Right now you need to specify ports to scan.
In the next beta release (few days) it will work same as nmap for default ports to scan.

Hello! Me again!
I did some interesting research and here are results that I got in nmap VS mans  battle! First I tried to scan 100 google.com ports and then I made 1k port scan in local network. In worldwide web mans results are a little bit slow, but in local it’s amazing! The same as nmap! Wonderful!
By the way, noticed one Inconvenient  thing: after scanning mans  returns all ports results. It would be better to return just open ports.
P.S. Appending results. Sorry, I like to draw 😁
P.S.S. Thanks for the great job you did! Can’t wait to test some new features!


Initialized Create React App + Tailwind

@A91y we're not going to use Tailwind as per our last discussion. Also, I don't see tailwind installed in package.json so where have you added it?

Wrong PR name, Tailwind should be replaced with Typescript


Can not open specific directories

I'm submitting a ...

[x ] bug report
[ ] feature request
[ ] support request

General infomration
kodi 18 rc1
Prerequisites

[ x] Are you running the latest version?

Description
I can not get to various sections in the menu (e.g. "Continue watching", "Genre" etc)
Steps to Reproduce

Try to open some directories like "Continue watching shows for user X", "Genre", etc.
Most of the times the Working icon will appear but the directory does not open

Some directories never open (e.g. Genre), some open all the time (E.g. Documentaries)
Expected behavior: [What you expected to happen]
Open the directory and show the list of items
Actual behavior: [What actually happened]
E.g. log output when I can not open " "Continue watching shows..."
20:42:08.173 T:1386095344 WARNING: CPythonInvoker(23, /root/.kodi/addons/plugin.video.netflix/addon.py): the python script "/root/.kodi/addons/plugin.video.netflix/addon.py" has left several classes in memory that we couldn't clean up. The classes include: N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon9xbmcaddon5AddonE,N9XBMCAddon9xbmcaddon5AddonE
20:42:08.183 T:1520431856   ERROR: GetDirectory - Error getting plugin://plugin.video.netflix/?action=video_list&type=continueWatching&video_list_id=b6e456f1-0577-4c52-aa94-f97cdebcf4af_42864075X3XX1544299765377
20:42:08.185 T:1900084880   ERROR: CGUIMediaWindow::GetDirectory(plugin://plugin.video.netflix/?action=video_list&type=continueWatching&video_list_id=b6e456f1-0577-4c52-aa94-f97cdebcf4af_42864075X3XX1544299765377) failed

Context (Environment)
raspberry pi 3
Installation

[ ] I installed the plugin via zip from the Releases page
[x] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

Additional informatin on the environment
[Descripe your environment a bit more detailed, are you using LibreElec f.e.]
Debug log
[Please include a link to your debug log (use something like http://sprunge.us/) or similar, please do not paste]
Other information
[e.g. detailed explanation, related issues, suggestions how to fix, links for us to have context, etc.]
Screenshots
[Please add a screenshot if that helps understanding your problem]
[You can erase any parts of this template not applicable to your Issue.]

Try adding the timeout parameter for open_url calls, on Navigation.py

Edit the line 1053 of the Navigation.py file, add the timeout parameter and set it to 60 seconds, it is enough, but it could delay some other actions in case of real timeouts.. but it will solve your issue.
data = urllib2.urlopen(full_url , timeout = 60 ).read()

thank you very much. Indeed it solved the issue.

Why was this change not added in the latest version (0.13.22)
At least as a setting in the addon options?
I also had to add the timeout to 60 to solve this issue si I guess alot of people have this and not everyone knows how to do this.
Is there a negative site in adding this timeout parameter to the open call?


x86: clear EFLAGS on double fault

In the event of a double fault, we do a HW task switch to
a special _df_tss hardware task which resets the stack
pointer to the interrupt stack and otherwise restores
the main hardware task to a runnable state so that
_df_handler_bottom() can run.
However, we need to make sure that _df_handler_bottom()
runs with interrupts locked, otherwise another IRQ could
corrupt the interrupt stack resulting in undefined
behavior.
We have very little stack space to work with in this
context, just zero it. It's a fatal error for the thread
in any event.
Fixes: #7291
Signed-off-by: Andrew Boie<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #13339   +/-   ##
=======================================
  Coverage   48.76%   48.76%
=======================================
  Files         319      319
  Lines       46892    46892
  Branches    10840    10840
=======================================
  Hits        22867    22867
  Misses      19391    19391
  Partials     4634     4634

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9072d34...0c0daa0. Read the comment docs.


Fix large memory usage

@vam-google : for this, we want to at least figure out what the cause might be before GA; after we figure out the cause, we can then decide if we want to actually fix it before GA or not.

@vam-google Can you also look into the breakdown between heap and off-heap memory usage as part of this?

Profiling is completed.
@meltsufin the results are sent to your mailbox, including the non-heap usage results.

The found memory issues are test-related and are not present in the actual clients code.

Closing as the memory issues are caused by testing code only.


Mark Classes as serializable

Hi, I am working with big data Spark.NET and It seems to complain when we try to serialise the code to our worker nodes, Is it possible to mark the classes as serialiazable. Ideally would like to try this on a beta version. I am happy to do a PR to try this concept out.

https://docs.microsoft.com/en-us/dotnet/spark/how-to-guides/dotnet-interactive-udf-issue


RFC: New input format for starfish

First commit updates the only producer of the input data.  Second commit updates the existing consumers.

Codecov Report

Merging #83 into tonytung-missing-notebook will decrease coverage by 0.09%.
The diff coverage is 100%.

@@                     Coverage Diff                      @@
##           tonytung-missing-notebook      #83     +/-   ##
============================================================
- Coverage                      68.71%   68.62%   -0.1%
============================================================
  Files                             23       23
  Lines                            975      972      -3
============================================================
- Hits                             670      667      -3
  Misses                           305      305

Impacted Files
Coverage Δ

starfish/image/_stack.py
87.5% <100%> (-0.18%)
:arrow_down:

starfish/starfish.py
92.12% <100%> (ø)
:arrow_up:

starfish/io.py
92.04% <100%> (-0.18%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d19db84...dcd3779. Read the comment docs.


Typo: Should say "70l Wasser"

https://github.com/technologiestiftung/giessdenkiez-de/blob/4d126460707110f707efa29c5650cb177993be4d/src/assets/content.tsx#L173

This has been taken care of at some point. Closing this issue.


hpcrun -h in hpctoolkit/2021.10 does not show anything about gpu profiling

It should list the same -e options as the man page does.

This repository has moved to GitLab, please continue this issue there.


Save tokens

Issue solved
Closes: #109

Changes Proposed
added save tokens code and fixed some errors

Types of changes made:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

@ChetnaNSharma, @yash982000 please review the pull request

@yash982000, can you please add the appropriate labels?

@yash982000, can you please add the appropriate labels?

Done :)


fix word-wrap and word-wrap-all

Fix #643
changed word-wrap to word-wrap-all and added break-word value for word-break in _word-break.css
(Also included it in the imports, although a cleaner non-breaking change is in the following: related to Issue #641 and PR #642 )

+1

Don't want this partial included - thanks.


Remote authentication for Spotify

Prevent users from having to create their own Spotify App authentication code and domain by creating an authentication layer on a shared domain name.

Included in release 2.4.4


Fixed #30153: Fixed some cases of erroneous ordering introduced by too eager merging of forms.Media objects

https://code.djangoproject.com/ticket/30153

I'd prefer to see a test in forms_tests/tests/test_media.py since this isn't an admin issue.
The code isn't straightforward for me -- perhaps some comments would be useful.
Any documentation updates needed? (c19b56f633e172b3c02094cbe12d28865ee57772)

Thanks to all of you for the review and helpful comments!
@timgraham The problem manifested itself with inlines in the administration panel that's why I started writing the tests there as well. In the end I only had to change the media class to fix it. I worry a bit that the problem may seem artificial when presented outside the administration panel ("why don't you just reorder assets then?") but at the same time the problem may be clearer so it may be very well worth it.
I don't think documentation updates are required. The code now does what it's supposed to do.

I've split the change into two commits:
The first commit only adds a test. The test passes but shows a broken result (inlines.js before jquery.js)
The second commit changes forms.Media and the test. The test passes again but this time the result also works in the browser.

I simplified the test a bit. Could you add one for the CSS changes?

Thanks for simplifying the test. I added the CSS test you requested: https://github.com/matthiask/django/commit/cca826805e447c11c658043ae3e08e4802cb04ab


Include types in our type-checking

Much of our code that looks typed actually hasn't been.  We have a
lot of our types concentrated in this one file types.js, and we
haven't marked it as a file Flow should look at -- which means that
when we import a type from there, Flow treats that as a totally
unknown type.  Any property we get off of such a type, or the return
value from any method call on it, then gets the same untyped treatment.
So, mark the file with @flow.
Just doing that alone brings 1564 (!) errors.  (Figures based on a
version from the other day; may vary slightly after rebase.)  So in
order to make this mergeable, I dialed it back:

out of about 77 types in the file, knocked out about 44 of them
back down to any
added 5 more types, all as any
sprinkled a couple of mysterious anys in notifications.ios.js --
I don't really understand why these give an error while the Android
counterparts don't.

None of these anys represent a regression in our type coverage, I
believe -- they just do the same thing more narrowly that was already
happening as a consequence of the whole file being untyped.  They were
any, they're still any, but at least some of the other things in
the file now are not.
It'd be a pretty great project for a bunch of people to come change
the various anys to real types, keeping the type-checker happy the
whole time.  That'll give us real, truthful types in a lot more of the
code -- which (a) will probably find some live bugs we have right now,
and more importantly (b) will give us much better information about
what the code says as we make changes day to day, which will help us
make fewer bugs in the future.

I fully agree with the approach and am quite happy that we are improving the type coverage.


|                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
/home/user/zephyrproject/zephyr/kernel/mutex.c:148:2945: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  148 |         LOG_DBG("on mutex %p got_mutex value: %d", mutex, got_mutex);
      |                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
/home/user/zephyrproject/zephyr/kernel/mutex.c:150:3857: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  150 |         LOG_DBG("%p got mutex %p (y/n): %c", _current, mutex,
      |                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
/home/user/zephyrproject/zephyr/kernel/mutex.c:160:2945: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  160 |         LOG_DBG("%p timeout on mutex %p", _current, mutex);
      |                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
/home/user/zephyrproject/zephyr/kernel/mutex.c:175:2041: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  175 |                 LOG_DBG("adjusting prio down on mutex %p", mutex);
      |                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
/home/user/zephyrproject/zephyr/kernel/mutex.c: In function 'z_impl_k_mutex_unlock':
/home/user/zephyrproject/zephyr/kernel/mutex.c:231:2945: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  231 |         LOG_DBG("mutex %p lock_count: %d", mutex, mutex->lock_count);
      |                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
/home/user/zephyrproject/zephyr/kernel/mutex.c:251:3857: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  251 |         LOG_DBG("new owner of mutex %p: %p (prio: %d)",
      |                                                                                ^
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
[132/145] Building C object zephyr/kernel/CMakeFiles/kernel.dir/work.c.obj
[133/145] Building C object zephyr/kernel/CMakeFiles/kernel.dir/sched.c.obj
/home/user/zephyrproject/zephyr/kernel/sched.c: In function 'k_sched_unlock':
/home/user/zephyrproject/zephyr/kernel/sched.c:989:2945: warning: the comparison will always evaluate as 'true' for the address of 'data' will never be NULL [-Waddress]
  989 |         LOG_DBG("scheduler unlocked (%p:%d)",
      |                                                                                ^
In file included from /home/user/zephyrproject/zephyr/include/zephyr/logging/log_core.h:9,
                 from /home/user/zephyrproject/zephyr/include/zephyr/logging/log.h:11,
                 from /home/user/zephyrproject/zephyr/kernel/sched.c:17:
/home/user/zephyrproject/zephyr/include/zephyr/logging/log_msg.h:94:17: note: 'data' declared here
   94 |         uint8_t data[];
      |                 ^~~~
[134/145] Linking C static library zephyr/kernel/libkernel.a
[135/145] Linking C executable zephyr/zephyr_pre0.elf

[136/145] Generating dev_handles.c
[137/145] Building C object zephyr/CMakeFiles/zephyr_pre1.dir/misc/empty_file.c.obj
[138/145] Building C object zephyr/CMakeFiles/zephyr_pre1.dir/dev_handles.c.obj
[139/145] Linking C executable zephyr/zephyr_pre1.elf

[140/145] Generating linker.cmd
[141/145] Generating isr_tables.c, isrList.bin
[142/145] Building C object zephyr/CMakeFiles/zephyr_final.dir/misc/empty_file.c.obj
[143/145] Building C object zephyr/CMakeFiles/zephyr_final.dir/isr_tables.c.obj
[144/145] Building C object zephyr/CMakeFiles/zephyr_final.dir/dev_handles.c.obj
[145/145] Linking C executable zephyr/zephyr.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:        9956 B       512 KB      1.90%
            SRAM:        3968 B        96 KB      4.04%
        IDT_LIST:          0 GB         2 KB      0.00%

Environment (please complete the following information):

OS: Arch linux
Zephyr SDK 0.15.1
tag v3.2.0

I believe it's medium because enabling this option is how you debug macro problems.

One of the messages is from the line below in zephyr/sys/cbprintf_internal.h.
size_t _pmax = (buf != NULL) ? _inlen : INT32_MAX; \

As the warning says buf is a fixed memory location, data, and not a pointer.
This kills it, but I'm not sure this is a right way to fix it.
diff --git a/include/zephyr/sys/cbprintf_internal.h b/include/zephyr/sys/cbprintf_internal.h
index 5334bef1937..af5302f1f1e 100644
--- a/include/zephyr/sys/cbprintf_internal.h
+++ b/include/zephyr/sys/cbprintf_internal.h
@@ -402,6 +402,7 @@ do { \
 do { \
        _Pragma("GCC diagnostic push") \
        _Pragma("GCC diagnostic ignored \"-Wpointer-arith\"") \
+       _Pragma("GCC diagnostic ignored \"-Waddress\"") \
        Z_CBPRINTF_SUPPRESS_SIZEOF_ARRAY_DECAY \
        BUILD_ASSERT(!IS_ENABLED(CONFIG_XTENSA) || \
                     (IS_ENABLED(CONFIG_XTENSA) && \

This should always work.
diff --git a/include/zephyr/sys/cbprintf_internal.h b/include/zephyr/sys/cbprintf_internal.h
index 5334bef1937..a19946b238d 100644
--- a/include/zephyr/sys/cbprintf_internal.h
+++ b/include/zephyr/sys/cbprintf_internal.h
@@ -382,6 +382,11 @@ do { \
                 _name##_buf32)))
 #endif

+static inline bool ___is_null(void *p)
+{
+       return p == NULL;
+}
+
 /** @brief Statically package a formatted string with arguments.
  *
  * @param buf buffer. If null then only length is calculated.
@@ -430,7 +435,7 @@ do { \
        Z_CBPRINTF_ON_STACK_ALLOC(_ros_pos_buf, _ros_cnt); \
        uint8_t *_rws_buffer; \
        Z_CBPRINTF_ON_STACK_ALLOC(_rws_buffer, 2 * _rws_cnt); \
-       size_t _pmax = (buf != NULL) ? _inlen : INT32_MAX; \
+       size_t _pmax = ___is_null(buf) ? _inlen : INT32_MAX; \
        int _pkg_len = 0; \
        int _total_len = 0; \
        int _pkg_offset = _align_offset; \

___is_null() should be somewhere else, I suppose.

@yashi Could you please submit a PR with the above fix? It seems reasonable.

Ok.  Give me a few days.


Recommended AviatorScript, high-performance, lightweight scripting language hosted on the JVM.

AviatorScript is a high-performance, lightweight scripting language hosted on the JVM.
https://github.com/killme2008/aviatorscript

What's the ask here?


add raster colorizer for bands

[ ] I added an entry to CHANGELOG.md if knowledge of this change could be valuable to users.

Here is a brief summary of what I did:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

615 of 665 (92.48%)  changed or added relevant lines in 30 files are covered.
8 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.03%) to 89.685%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

datatypes/src/primitives/query_rectangle.rs
9
10
90.0%

operators/src/processing/raster_vector_join/non_aggregated.rs
6
7
85.71%

operators/src/processing/temporal_raster_aggregation/first_last_subquery.rs
1
2
50.0%

services/src/datasets/external/netcdfcf/mod.rs
0
1
0.0%

datatypes/src/operations/image/colorizer.rs
1
4
25.0%

services/src/api/model/datatypes.rs
19
62
30.65%

Files with Coverage Reduction
New Missed Lines
%

operators/src/util/stream_zip/vec_zip.rs
1
90.65%

services/src/api/handlers/layers.rs
1
87.16%

services/src/api/handlers/projects.rs
1
98.59%

services/src/pro/api/handlers/users.rs
1
96.62%

services/src/api/handlers/workflows.rs
2
90.08%

services/src/pro/contexts/postgres.rs
2
96.36%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
113270

Relevant Lines:
126298

💛  - Coveralls


How do you acquire semantic_info?

I wonder how did you obtained lists of semantic relation pairs in the folder semantic_info?

Hi @AlfredWGA,
We got them from these sources:
https://github.com/mfaruqui/retrofitting
https://github.com/suhaibani/JointReps/tree/master/work

Hi @AlfredWGA,
We got them from these sources:
https://github.com/mfaruqui/retrofitting
https://github.com/suhaibani/JointReps/tree/master/work

Thank you for your reply!


Update project structure to support Airbyte and dbt for ELT workflows

To form a more robust foundation for our data platform work we have selected Airbyte for our E(xtract)L(oad) stages and dbt for T(ransform).
As we adopt those tools we need to update the project structure of this repository to store the necessary code and configurations related to those tools. This includes bootstrap scripts to get local development environments set up, as well as creating the initial implementations to serve as a reference for other contributors.
To that end we need:

[ ] An install script to set up and configure the octavia-cli for Airbyte
[ ] A directory structure for storing Airbyte configurations to manage connectors
[ ] A directory structure for our dbt project(s)
[ ] Initialize the dbt project and populate the configuration for executing it against our data lake
[ ] Document the steps required to create a new Airbyte sync connection
[ ] Document the steps involved in wiring together an Airbyte -> dbt execution pipeline

Started a discussion in this repository to document the steps for configuring an Airbyte connection https://github.com/mitodl/ol-data-platform/discussions/231
By using the discussion feature it will be easier to edit and update those instructions as they evolve with our infrastructure development.


New (alternative) function for animated gifs

I've made use of images2gif lib to create gifs and make the whole library more "pythonic". I also made it an alternative for now because it's also not perfect.
I've pulled MAGICK_HOME check from main() and moved it to the legofy_gif(). New function (legofy_gif_pythononly) triggers when no MAGICK is found.
Pros:

python only solution
makes use of build-in PIL ImageSequence
no temp dir is created - everything happens i the memory

Cons:

uses numpy - great library, but its heavy and generally has a different purpose.

Other stuff:
Pycharm .idea/ added to ignore
Pulled apply overlay as a separate function to make it more visible on how does it work. Also makes it easier to debug.

Not sure how to pass travis test before the actual commit (library that is "missing" is a part of previous commit)

You can move /images2gif/images2gif.py to just /images2gif.py as you can import files in python, then letting you do import images2gif

also, its failing to find images2gif because its not in scope. If it was installed in pip it would find it, but since the file is in /legofy/ and trying to find /images2gif/ it doesn't.
Solution: if images2gif is on pip, have it installed, OR add the file in /legofy

I did move it to the main dir - sorry for delay.
There is a pip version but well, it doesn't work with py 3+

This pull request have conflict,
Please fix it before @JuanPotato merge it.

I'm sorry for messy commit . I'm currenty struggling with git to do what I want. I'm not sure how to remove failing check in Travis build.
I'm still getting the same: ImportError: No module named 'images2gif'
And images2gif.py is clearly in the main directory. For my project runs without any problem with this structure... Perhaps this library should be manifested ?

This is an install bug. Try adding the file to MANIFEST so that it gets
copied over in the site pakages.
Nick
On Nov 19, 2015 05:38, "Piotr Wołoszyn"<EMAIL_ADDRESS>wrote:

I'm sorry for messy commit . I'm currenty struggling with git to do what I
want. I'm not sure how to remove failing check in Travis build.
I'm still getting the same: ImportError: No module named 'images2gif'
And images2gif.py is clearly in the main directory. For my project runs
without any problem with this structure... Perhaps this library should be
manifested ?
—
Reply to this email directly or view it on GitHub
https://github.com/JuanPotato/Legofy/pull/71#issuecomment-158058946.

Conflicts and build fail, fixing should be easy since @niroyb has been helpful to give you some information

Now travis fails on Python 2.6, 2.7 - I'll try to adopt older versions of images to gif.

Hey,
Can you explain to me why Travis CI build failed this time ?

File "/home/travis/build/JuanPotato/Legofy/legofy/images2gif_py2.py", line
449, in writeGifToFile
fp.write(globalPalette)
TypeError: must be string or buffer, not None
Looks like the variable fp is unset.
On Nov 25, 2015 02:16, "Piotr Wołoszyn"<EMAIL_ADDRESS>wrote:

Hey,
Can you explain to me why Travis CI build failed this time ?
—
Reply to this email directly or view it on GitHub
https://github.com/JuanPotato/Legofy/pull/71#issuecomment-159562206.

@niroyb
globalPalette was unset - because images2gif for Py 2 is ... well it's old :/ anyways, found the issue and fixed it.

Doesn't look fixed to me...

@nabijaczleweli
Care to elaborate? I've tested it both with Py2.7 and 3.4, also the Travis passes
The check is not successful because of the test coverage. (Coveralls) But it's something that shouldn't be considered in this case. (it's an outside library adopted for legofy). Is it possible to exclude particular files from test coverage ?

@CasualBeer The Travis was failing

@nabijaczleweli but it isn't failing :\

It was (above the message previous to mine)

oooh

Coverage decreased (-61.06%) to 24.324% when pulling cfbbddefd09de137e1ad06e83c7e4f21916fe4e9 on CasualBeer:master into 3d5a214b0828687eb555705c39b824bd73250466 on JuanPotato:master.


Fails to compile on recent nightly

When building on rustc 1.22.0-nightly (277476c4f 2017-09-16) tarpc-plugins fails with
error[E0061]: this function takes 2 parameters but 1 parameter was supplied
  --> src/plugins/src/lib.rs:68:52
   |
68 |                     *doc = Symbol::intern(&str_lit(&doc.as_str()).replace("{}", &old_ident));
   |                                                    ^^^^^^^^^^^^^ expected 2 parameters

error: aborting due to previous error

error: Could not compile `tarpc-plugins`.

Thanks for the report! Will try to get it fixed today.


example: 100mb of analytics data in pglite

narrative:

loads relatively fast
client-side interactivity is very fast
works offline
loads very fast with cached data/code

Would this be like a table/graph frontend? How would the story go other than "useShape to sync data in, the rest is all SQL/Postgres in the client"?

Yeah some sort of table graph frontend where the user can change some options or whatever so triggers a new client-side query/render. The goal is to show a) 100mb is actually a reasonable amount of data to load and b) client-side interactivity is still really fast.
But the UI itself is whatever strikes your fancy.

@KyleAMathews as previously discussed, should we morph this into a more "live streaming HackerNews data using Electric and PGlite?" rather than a plain one-time analytics workload?

I like the idea of something live streaming but the main point here is to show that the sync engine & pglite can easily load large amounts of data. Live streaming is nice but not the point.
A HN clone doesn't necessarily show that but it could e.g. if you include some charts about posts / month over the past 10 years or whatever that shows off an aggregation query. So yeah an aggregation query + full-text search could be fun (though perhaps creating the full-text index would be excessively slow?).
What do you think?

Hmm if the main point is still the loading of large amounts of data then I think it's better to stick with the thing I've already been building (large fixed dataset) and do something with that - probably no point in reworking it for a different data source. We can think of different examples to highlight the live streaming analytics use case.

👍 yeah, good to keep examples focused

It's easy to find large dataset from HN and GH which would allow to do some nice demos with loads of data that interests the community. The problem with these dataset is that they are all a bit old and therefore there is a gap between the collected data and the data that we would load live.
If we're looking into synching large amounts of data I'd still look into one very large dataset. In the order of GB (or the max the browser allows).
As discussed, what's special is the ability to also get live data for your analytics, I'll create a separate issue for that.

https://github.com/electric-sql/electric/issues/1745

This is done enough for now — https://linearlite.examples.electric-sql.com/ loads 150mb of data into pglite so good demo


razer-cli: init at 2024-04-04

Python CLI utility to control various razer devices via openrazer. Useful for changing mouse dpi, rgb, etc with command line and user scripts/services.
https://github.com/LoLei/razer-cli

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 348822

x86_64-linux

:white_check_mark: 2 packages built:

razer-cli
razer-cli.dist

It's worth noting that this requires openrazer running.
Maybe in the future create a module with an option programs.razer-cli.enable that will also enable openrazer? Not a blocker though this looks good to me.

Could also be an option within the hardware.openrazer module

Hmm I don't think this is a bad idea either. Like hardware.openrazer.razer-cli.enable

Should probably be up to the maintainer of openrazer module @roelvandijk if this makes sense. Otherwise I think there is plenty of convention of packages which do not work without various services behaving in a certain way.

Thanks for the suggestions, I'll try to post this on the discourse thread for packages ready to be merged when I have some more time.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4874

LGTM except I still don't really understand we can't just package the stable version

I don't have a strong reason, but the last git commit is many months old, so I would rather have those small fixes in the package when it seems unlikely they'll show up in a stable numbered version any time soon if ever.

The thing is that AFAICT the latest commit is literally just 2.2.1 with a minor CI fix and it's not at all functionally different


Add missing features for overview

Motivation and Context
The first version of the overview screen went in incomplete to make the PRs smaller/easier to understand and get it in sooner knowing we could iterate on it.
This PR contains the missing features from the prototype (https://rops-prototypes.netlify.app/prototypes/overview/version-one/index.html).
What has changed

How to test?

Links

/deploy terryj


Wrap each question in a  with a proper class name.

In attempts/new view:
Wrap each question (along with it's answers) in a special 'div' element.
The div's class name matches the question type ('checkbox_question', 'radio_question' etc...)
This can be beneficial for applying styles to questions.
No spec written, since it's a small view related change,  I can write one if needed.

@bnadav Can you explain where you wanted this? Thanks.

@iffyuva I don't know why @bnadav wanted it, but generally speaking, it would be nice to have so we can style our apps a little better without needing to generate the views. That would mean when we upgrade, we don't need to check for new question types and add new partials to cover them.
In fact, if it were me, I'd like to see a general CSS class like 'rapidfire' in addition to this specific class, so we could target all generally, and then get specific if needed.
Or in the case of the admin section, if the fields there were wrapped, we wouldn't need to check for new fields to add to our own copies of the views -- we could just make a CSS class in our app, and let Rails find the HTML inside the gem.


Optimize not working after updating to 8.5 Self-Managed

I opened a topic in the forum and some users have claimed they have had the same issue. Nobody from the team answered though.
I have just updated my Camunda SM to 8.5 using docker compose and after the successful installation, I see that the Optimize container is unhealthy.
The logs say:
Caused by: org.camunda.optimize.service.exceptions.OptimizeRuntimeException: The database Optimize schema version [3.11.5] doesn't match the current Optimize version [3.13.1]. Please make sure to run the Upgrade first.
	at org.camunda.optimize.service.db.schema.DatabaseMetadataService.lambda$validateMetadata$0(DatabaseMetadataService.java:56)
	at java.base/java.util.Optional.ifPresent(Optional.java:178)

I checked the docker -compose configuration file and under healthcheck, I saw that this container ought to run this command:
curl -f http://localhost:8090/api/readyz

I headed into the container and ran the command. It was not successful. It seems like the problem is really with the component itself.
I did not have the problem before the upgrade. What is the issue?

@okaeiz Is that issue related to Helm or Docker Compose?

@okaeiz Is that issue related to Helm or Docker Compose?

It's related to Docker Compose

In that case, I move this issue to the Docker Compose repo.


Stalled provisioning of new hosts and existing hosts disconnected

Rancher Version:
1.2.1
Docker Version:
1.12.3
OS and where are the hosts located? (cloud, bare metal, etc):
Rancher on Joyent, machines on vultr
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single external db
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Unknown, hosts get disconnected, provisionig new hosts stalls in provisioning state
Results:
�2016-12-22 19:00:24,051 ERROR [82924fec-edce-458a-ac99-fa8221be6092:217422] [instance
:5618] [instance.start->(InstanceStart)] [] [cutorService-13] [i.c.p.process.instance.
InstanceStart] Failed to Scheduling for instance [5618]
�2016-12-22 19:00:31,485 ERROR [04e812b9-3423-430a-92a0-506af1a9524e:217442] [instance
:5620] [instance.start->(InstanceStart)] [] [ecutorService-4] [i.c.p.process.instance.
InstanceStart] Failed to Scheduling for instance [5620]
�2016-12-22 19:00:32,615 ERROR [a8b953f7-431f-479b-836c-1b29aafeaf00:217440] [instance
:5619] [instance.start->(InstanceStart)] [] [cutorService-20] [i.c.p.process.instance.
InstanceStart] Failed to Scheduling for instance [5619]
�2016-12-22 19:00:37,208 ERROR [43c0b7ef-f98c-42c1-a188-d03f192a661b:217459] [instance
:5621] [instance.start->(InstanceStart)] [] [ecutorService-2] [i.c.p.process.instance.
InstanceStart] Failed to Scheduling for instance [5621]
�2016-12-22 19:00:46,167 ERROR [e233f8f5-fed1-4024-aa21-c05a66aa9913:217571] [instance
:5622] [instance.start->(InstanceStart)] [] [cutorService-14] [i.c.p.process.instance.
InstanceStart] Failed to Scheduling for instance [5622]
�2016-12-22 19:00:46,167 ERROR [d8a33aa7-6ac7-43d0-95c7-3b9780472efc:217570] [instance
:5623] [instance.start->(InstanceStart)] [] [cutorService-12] [i.c.p.process.instance.
InstanceStart] Failed to Scheduling for instance [5623]

Expected:

Closing due to inactivity. There have been numerous improvements in rancher. Let us know if this exists in a modern version of rancher.

I havent seen this lately, so most likely fixed, thanks


Allow terraform init to pull 'backend' info from terraform.backend config

Please allow to the terraform init command to optionally pull the backend info from the 'terraform.backend' block, as we always like to have 1 source of truth and always should be the terraform configs.

aws <<
terraform {
backend "s3" {
bucket         = ""
key            = "< terraform state file key/name>"
region         = "us-east-2"
}
}

azure <<
terraform {
}
backend "azurerm" {
resource_group_name  = "< resource_group_name >"
storage_account_name = "< storage account name >"
container_name       = "terraformstate"
key                            = "< terraform state file key/name >"
}
}

So much better than the way we're doing it through input nodes.  I agree.
But until then, you can use parameters and templates to abstract down these settings to a config.yml:
Reference: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/templates?view=azure-devops
Literally magic for this type of inconsistency or potential spelling mistake across a hundred terraform templates.
Example parameterized step template teraform-init-template.yml
parameters:
  - name: ProjectFolder
    type: string
  - name: AzureServiceConnection
    type: string

steps:
  - task: TerraformTaskV4@4
    displayName: 'Azure service connection initialization'
    inputs:
      provider: 'azurerm'
      command: 'init'
      workingDirectory: ${{ format('{0}/{1}', '$(System.DefaultWorkingDirectory)', parameters.ProjectFolder) }}
      backendServiceArm: ${{ parameters.AzureServiceConnection }}
      backendAzureRmResourceGroupName: $(BackingStorageResourceGroup)
      backendAzureRmStorageAccountName: $(BackingStorageAccount)
      backendAzureRmContainerName: $(BackingStorageContainer)
      backendAzureRmKey: $(BackingStorageKey)

Example use-case (anywhere)
        steps:
          - template: ./terraform-init-template.yml
            parameters:
              ProjectFolder: ${{ parameters.ProjectFolder }}
              AzureServiceConnection: ${{ parameters.AzureServiceConnection }}

Example config.yml
variables:
  AzureServiceConnection: 'S7T-DevOps'
  BackingStorageResourceGroup: 'S7T-TFState-RG'
  BackingStorageAccount: 'S7T-BuildStorage'
  BackingStorageContainer: 'S7T-TFState'
  BackingStorageKey: 'S7T.Terraform.TFState'

At the very first root (here I'm using a deploy.yml which is what I make my pipeline out of), you should reference the config.yml as a template under variables.  This way, they will propogate down form the top level root of the yaml forest:
# Advanced multimodal deployment template
trigger: none

pool:
  vmImage: 'ubuntu-latest'

variables:
  - template: ../config.yml

extends:
  template: ../templates/my-anywhere-deployment-template.yml
  parameters:
    ProjectFolder: 'terraform/core'
    AzureServiceConnection: ${{ variables.AzureServiceConnection }}

Thus you can shorten down how you reference them to make this consistent and packed into a neat global variable config.yml using templates.  Any time you change these, they take affect across the entire terraform node forest at pipe time.

📝 This is snippet code but was pasted from some production code to give you a general idea.  My template library is much larger...

Hi, I will add it to our board

This would be useful for us also.  When we only have one environment, and we use AAD (for developers) and workflow identity (for pipelines), we don't have any secrets in the backend configuration anyways so leaving it in the terraform file directly is more convenient for us.
Another related request - to support backend files (possibly coming from a Secure File or from the repository) would be useful for us.
@mericstam is someone actively working on this?  If not I might be able to contribute a pull request for the above two feature requests.

@andrewluotechnologies, No one is looking at this at the moment.  I am responsible to keep the task updated with security fixes but new features are low priority right now.  I would be happy If you could contribute, I will assist as much as I can. review PR, Test in our test environment, ETC.

closing this due to inactivity


Use @electron/remote instead of importing it from electron as it will be deprecated in Electron version 14

resolve #76

Nice, I was about to do it myself! +1

Hi, thanks for the PR. Since workflows seemed to be gone on PRs, please rebase on develop, that might fix it. You also forgot to remove a leftover from your tests, please remove the 'yarn.lock' file

Hi, thanks for the PR. Since workflows seemed to be gone on PRs, please rebase on develop, that might fix it, but you may have to fix your code style to fix the errors. You also forgot to remove a leftover from your tests, please remove the 'yarn.lock' file

Okay, it's basically just the double-quoted and single-quoted string problem tho haha, I'll fix it, btw for the yarn.lock, why don't you just put it into .gitignore. In addition, for the workflow, I think it's better to create a GitHub workflow to let future PR get checked immediately.

@kimlimjustin Yes there's a workflow and that's why I asked you to rebase. You don't get to have it automatically happen since it's your first contribution though. LGTM


Metrics and Observability endpoints

Is your feature request related to a problem? Please describe.
Add Prometheus endpoint based on Starlette exporter and OpenTelemetry collector for a proper management on clustered environments.

Hi @pdrivom
The idea is a good one though I've got too much core stuff to work on right now though.
I'm very open for a PR to implement this :smiley:

Hello @seapagan, ok I will work on it and open a PR  😃

Hello @seapagan, ok I will work on it and open a PR smiley

Thanks! :smiley:


Efficient prometheus

Reason for Change
Number 1 and 2 are important for configuration.
Number 3 and 4 are important for scraper performance.
Changes:

Moves the port number of the Prometheus endpoint to a high value next to our node port used in CDN
Adds a service that forwards a single, random NPM Prometheus endpoint so that a scraper can scrape only one NPM pod for metrics that are redundant across nodes.
Removes old IPSet counts when the number of entries is 0 to reduce the amount of lines to scrape.
Adds total ipset entries as a Prometheus metric so customers can have this number without having to include the advanced metric - number of entries per ipset.

Requirements:

[X] perf
[X] includes documentation
[X] adds unit tests

Notes:
Current metrics that are redundant across nodes:

npm_num_policies
npm_num_iptables_rules
npm_num_ipsets
npm_num_ipset_entries (new)
npm_ipset_counts (unlikely, but worst case can have up to 60,000 lines for scraping)

Remaining metrics that are node-specific:

npm_add_policy_exec_time
npm_add_iptables_rule_exec_time
npm_add_ipset_exec_time

How did we achieve No.2's goal? What's the service you mentioned in No2?
Does it achieved by ipset_inventory.go file?

How did we achieve No.2's goal? What's the service you mentioned in No2?
Does it achieved by ipset_inventory.go file?

Achieved through the new service in azure-npm.yaml: npm-metrics-cluster-service, and by the way you configure the container insights agents, but that's a job for their team. The ipset_inventory.go deletes the metric (line in the scrape file) for every ipset that has no entries/is deleted.

ran load tests showing that npm with/without prometheus still act the same

also noting here that having 1 vs 2 HTTP endpoints for an NPM pod seems not to affect performance


[SwipeableDrawer] Decorrelate swipeAreaWidth and initial jumping amount

This PR removes dependency between swipeAreaWidth prop and the ready-for-dragging signal of drawer related to
Closes #23037

@omidtajik It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!

It's my pleasure to contribute in this awesome project ❤


hide the experimental _format command

hide the experimental _format package command
Motivation:
I want to clean up the CLI face of swift package, and the _format command stood out undocumented, likely experimental, and with no explanation. Based on the forum thread inquiring about _format and it's status, this PR hides the command from being displayed in --help and such.
Modifications:
Hides the _format command
Result:
swift package invocation no longer shows _format in the list of available commands.

@swift-ci smoke test


Passing on State to BatchNorm when it is integrated with nn.Squential

Hi,
This in continuation to the effort of bringing eqxvision up to date with changes in equinox (https://github.com/paganpasta/eqxvision/issues/72).
One of the changes required now are to pass the state explicitly to the Normlayer's call.
Quite often the models are recursive collections of bunch of layers with BatchNorm integrated in them somewhere. For example,
ResNet utilises mainly 4 blocks where each block is implemented as a Sequential and within each sequential block there is a BatchNorm.
One solution I see is to either modify the equinox.nn.Sequential to accomodate an optional State input or simply implement a modified Sequential in the Eqxvision repository.
@patrick-kidger, I wanted to seek your advice on the best way to go about and supporting such scenarios before making any drastic changes.

Heyhey! I think we could probably do something like:
class StatefulLayer(eqx.Module):
    @abc.abstractmethod
    def __call__(self, x, state):
        ...

class BatchNorm(StatefulLayer):
    ...

class Sequential(eqx.Module):
    ...

    def __call__(self, x, state=sentinel):
        for layer in self.layers:
            if isinstance(layer, StatefulLayer) and state is not sentinel:
                x, state = layer(x, state)
            else:
                x = layer(x)
        if state is sentinel:
            return x
        else:
            return x, state

In other words: (a) use the same trick that LayerNorm uses to optionally pass through stateful input, and (b) add an explicit way of checking whether a layer should take an extra state input.
I'd be happy to take a PR on this to Equinox if you want. (I'd also be happy to see this appear in Eqxvision; I don't have strong feelings either way.)

Yup! Problem solved. :)


Follow-up items for PR2321 (--store-constants-to-file)

Below are the follow-up items to support --store-constants-to-file in PR #2321.

Support memory-mapping on Windows
Add a metadata in the constant file to indicate endianness
Improve the way to find the constant file if possible, e.g. a way to find out from which directory a .so file is loaded.

Currently the constant file should be at the same folder as the user program or a folder specified by OM_CONSTANT_PATH environment variable.

@tungld
If the onnx file is generated on x86 little endian, and then I compile it on s390x, is there any endianess issue?

@chenqiny no, there is no problem, and it's often the case.


incomplete sentence

URL to the relevant documentation
https://docs.quantum.ibm.com/api/qiskit/0.42/qiskit.transpiler.preset_passmanagers.generate_preset_pass_manager
Select all that apply

[X] typo
[ ] code bug
[ ] out-of-date content
[ ] broken link
[ ] other

Describe the fix.
The second sentence describing this method ends abruptly.  There should be more ...

This function provides a convenient and simple method to construct a standalone PassManager object that mirrors what the transpile

Hi @dr-kinder and thanks for your issue! We will transfer it to the qiskit/qiskit repo where the API reference docs live, but in the meantime I asked some colleagues their opinion on what the rest of the sentence is. The consensus is that it's probably along the lines of "This function provides a convenient and simple method to construct a standalone PassManager object that mirrors what the transpile function does."


Republish with new github name

Hi! I'm using this package in one of my applications. I can't build the new version, as you changed your username from jasonliang512 to jasonliang-dev. This messes up some checksums.
Can you republish this package so it's in the elm package registry under the name jasonliang-dev/elm-heroicons?
Thanks!

Hey 👋 I've been scratching my head for the past few days trying to find the best solution for this issue. It seems to be that the best thing I can do right now is to republish the package under my new username.
jasonliang-dev/elm-heroicons is now published. I wish I could somehow remove/update jasonliang512/elm-heroicons on the package registry though.

I wish I could somehow remove/update jasonliang512/elm-heroicons on the package registry though.

I agree. It makes sense that the elm package repository is immutable, as then it would be possible to install elm packages forever (in theory), but in practice it's so tightly linked to github, it doesn't make sense that it's immutable.
Evan said he is working on a solution for this, but it's a large issue.


BitBucket Integration Setup Not Working if Logged In

The "Link Bitbucket" button on https://hub.docker.com/account/authorized-services/ works properly if you are not already logged in to Bitbucket. However if you ARE logged in it just pops open a new window with the same page.
The temporary workaround is to log out of Bitbucket first, but clearly this is a bug that you might want to address.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Thanks for reporting @aytawn. Unfortunately, hub.docker.com is handled by a different team and we cannot help you here.
For support issues on hub.docker.com, please contact<EMAIL_ADDRESS>There's also an issue tracker for feedback here; https://github.com/docker/hub-feedback/issues?utf8=✓&q=is%3Aissue+is%3Aopen+bitbucket
Can you report this issue using either one of the above? Thanks!


Question about  action_tokens usage in _assemble_input_token_sequence

Hi, thanks for your great job!
I have read all the code and found that in the function transformer_network::_assemble_input_token_sequence,  action_tokens  are used to be a kind of input, but the values all are set to be zero(**action_tokens = tf.zeros_like(action_tokens)  **).
My question is that why not use the data here? Have you done any experiment proving that it works bad?
Thanks for your attention and keep waiting for your kind response!
`
def _assemble_input_token_sequence(self, context_image_tokens, action_tokens,   batch_size):
action_tokens = tf.one_hot(action_tokens, self._vocab_size)

action_tokens = self._action_token_emb(action_tokens)

action_tokens = tf.zeros_like(action_tokens)

action_tokens = tf.expand_dims(action_tokens, axis=-2)

input_token_sequence = tf.concat([context_image_tokens, action_tokens],  axis=2)

input_token_sequence = tf.reshape(  input_token_sequence, [batch_size, -1, self._token_embedding_size])

return input_token_sequence

`

I have the same question.
I was wondering how actions are specifically input into the model since the paper lacks that detail and I was looking into this repository.
And what is b/260260205 and b/254902773 referring to?


Could not save lang.yml

[00:21:27 WARN]: [AdvancedArmorStands] Could not save lang.yml to plugins/AdvancedArmorStands/resources/lang.yml because lang.yml already exists.

You can ignore this error it's nothing important. But that you for reporting it, i will investigate further. Also make sure you are using the most recent version of the plugin (in console it prints out "v.<IP_ADDRESS> for bukkit 1.17+") and the most recent version of bukkit/ spigot/ paper/ or whatever you use of that.
If the error still keeps appearing, delete the whole folder "AdvancedArmorStands" in your "plugins"-folder and restart the server. That should fix it.


Xnero: Watch designpluz.com.au

Xnero requests the watch of the watch_keyword designpluz\.com\.au. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

designpluz\.com\.au has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


osqp_vendor: 0.1.2-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository osqp_vendor to 0.1.2-1:

upstream repository: https://github.com/tier4/osqp_vendor.git
release repository: https://github.com/tier4/osqp_vendor-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.0
previous version for package: null

I see that the version is 0.1.2 but the cmake version is 0.6.1. Should the package/release version be 0.6.1?

I see that the version is 0.1.2 but the cmake version is 0.6.1. Should the package/release version be 0.6.1?


Copy/paste between editors strips soft returns

When copying from one draft editor and then pasting into another, I am seeing soft returns get stripped. The soft returns were '\n' characters added via RichUtils.insertSoftNewline.
https://github.com/facebook/draft-js/blob/213cd764f61cc64552bddd672ae1748529d55333/src/model/modifier/RichTextEditorUtil.js#L82
Here is a small test that can be added to convertFromHTMLToContentBlocks-test.js which demonstrates the problem:
  it(`must not strip soft returns`, () => {
    const contents = 'a\nb'
    const html_string = `
      <div>${contents}</div>
    `;

    const blocks = convertFromHTMLToContentBlocks(html_string);

    expect(blocks.contentBlocks.length).toBe(1);
    expect(blocks.contentBlocks[0].text).toBe(contents);
  });

result:
    Expected value to be (using ===):
      "a\nb"
    Received:
      "a b"

This happens on account of this line of code:
https://github.com/facebook/draft-js/blob/507925ad60638734872dba52efa2d0b66cd94e4b/src/model/encoding/convertFromHTMLToContentBlocks.js#L357
Is there a way we can change this behavior to handle copy paste between draft editors?
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
0.11.0-alpha
Chrome, MacOS

Same issue as the one I filed just now? https://github.com/facebook/draft-js/issues/1780

same here


Location of julia.h

When installing julia manually to /opt I can find the header files /opt/julia-1.7.1/include/ and other stuff like a global startup.jl in /opt/julia-1.7.1/etc/julia/.
Where does juliaup put these files? I can seem to find anything in ~/.juliaup

Opps nevermind I found them at .julia/juliaup


Please create a support group

a lot of people faced with problems like this:

!Атака проводиться успішно! Руський воєнний корабль іди нахуй!
!Attack is successful!
---------Traffic stats---------
 [     Generated ] 39643.71 MB |<PHONE_NUMBER>5 bytes
 [      Received ]     0.09 MB |       96869 bytes
 [ Response rate ]      0.0%
-------------------------------

why 0.0% ? how to fix? why so low?
Please create a support group in the telegram

Low response rates usually mean that targets can't manage the load we're producing

Probably best to just add that last sentence to FAQ
As for support group - there no groups dedicated to just this tool alone but considering the spread of it you can probably get some initial help in IT Army of Ukraine (I'll probably add this to the config repo since it's more relevant there)

fe0c8c90e1178e57408031c7e85baf1cb1352f79 for the FAQ part

Enabled github discussions for community help


Fixed positioning for overlay

This PR changes the positioning of the overlay from absolute to fixed.
This ensures that no overflow will be triggered in surrounding elements, which is important especially for mini TextEditors.
cc @nathansobo

Tested this out locally and everything seems to be working. @atom/core if anyone sees anything weird with overlay decorations, this may be a culprit.


Equivalent to createURLRoute in new version?

i've installed in a project with electron-vite and react
everything seems to be ok except
I can't seem to match the homepage in dev mode
<Router basename="/main"> is not able to match the URL "/" because it does not start with the basename, so the <Router> won't render anything.
Seems to be related to the lack of ability to load a url vs a htmlFile:
  registerRoute({
    id: 'main',
    browserWindow: mainWindow,
    htmlFile: join(__dirname, '../renderer/index.html'),
  })
  // HMR for renderer base on electron-vite cli.
  // Load the remote URL for development or the local html file for production.
  if (is.dev && process.env['ELECTRON_RENDERER_URL']) {
    mainWindow.loadURL(process.env['ELECTRON_RENDERER_URL'])

  } else {
    // mainWindow.loadFile(join(__dirname, '../renderer/index.html'))
  }

ok figured it out
had to registerRoute at the end
and then find the vite port and paste into the config

figured it out!

not quite - i'm noticing HMR is not working
seems to be using the index.html file and not the local dev server

this is the bottom of my createWindow function

  // HMR for renderer base on electron-vite cli.
  // Load the remote URL for development or the local html file for production.
  if (is.dev && process.env['ELECTRON_RENDERER_URL']) {
    mainWindow.loadURL(process.env['ELECTRON_RENDERER_URL'])

  } else {
    mainWindow.loadFile(join(__dirname, '../renderer/index.html'))
  }

  registerRoute({
    id: 'main',
    browserWindow: mainWindow,
    htmlFile: join(__dirname, '../renderer/index.html'),
  })

and things load but no hot module reloading

ok turns out HMR works in the top level App / Layout files
But not for the individual route content itself  (most of the app)
i know its a recent version! hope this is helpful
ty

Hi @sebbean , you no longer need to do this manually:
  if (is.dev && process.env['ELECTRON_RENDERER_URL']) {
    mainWindow.loadURL(process.env['ELECTRON_RENDERER_URL'])

  } else {
    mainWindow.loadFile(join(__dirname, '../renderer/index.html'))
  }

registerRoute handles this logic internally, in the development environment it will be used loadURL, in production loadFile as you can see here:
https://github.com/daltonmenezes/electron-router-dom/blob/main/packages/electron-router-dom/src/index.ts#L104-L121

Also, could you please test/check the with-multiple-windows example?

@daltonmenezes it does look like this is actually an issue with vite + react-dom-router not triggering page content refreshes
i see the reload happen on both the server terminal as well as the dev console in the app
the content inside the page content stays stale
but content in the wrapper layout changes (HMRs) fine
thanks again!
i'll check out


Multiple clusters for dev, staging, and production?

Does anyone have any strategies they use for deploying the same deis app across multiple clusters? I stumbled across #1155 but the suggested fix of using the --cluster flag doesn't seem to appear anywhere in the documentation.

@glenwong maybe this question is better suited to the deis-users mailing list? You would probably get better responses there.

I'm going to close this as it hasn't had any followup in several weeks, and may be better suited for discussion on the deis-users mailing list.


Update stitching_Matterport3D.m

To aviod some possible problems.

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


Fix IAE caused by effectively empty context value

This PR fixes the console spam of an IAE caused by the luckperms api rejecting context values that are effectively empty.
ex.
[07:12:19] [Craft Scheduler Thread - 41 - FeatherBoard/WARN]: [LuckPerms] An exception was thrown by me.lucko.extracontexts.calculators.WorldGuardFlagCalculator whilst calculating the context of subject CraftPlayer{name=Sxtanna}
java.lang.IllegalArgumentException: value is (effectively) empty
	at me.lucko.luckperms.common.context.AbstractContextSet.sanitizeValue(AbstractContextSet.java:81) ~[?:?]
	at me.lucko.luckperms.common.context.ImmutableContextSetImpl$BuilderImpl.add(ImmutableContextSetImpl.java:281) ~[?:?]

The problem is solved by pre-checking the validity of the value using what luckperms uses.
Luckperms API
5.0 -> 5.3

I found this issue after solving the NPE issue, but I forgot to update my initial PR, sorry. @lucko

No problem, thanks again :)


Allow JWT signing method to be configurable.

This change creates a new Signer interface which encapsulates jwt.SigningMethod + the key material use to sign JWT tokens.
This allows clients to modify how JWT tokens are signed by passing in their own Signer. In particular, I'm interested in coupling this with something like
https://github.com/golang-jwt/jwt#extensions to allow for JWT signing backed by KMS systems (Vault, Cloud KMS, etc) where the private key never resides on the local client.
Introduces a new AppsTransportOptions to make it easier to make new transport creation options without needing to make new funcs each time. For now only added WithSigner, but we could easily extend this out to other config options (Client, BaseURL, etc.)
Finally, upgrades deprecated jwt.StandardClaims -> jwt.RegisteredClaims.

This looks good and the I like the AppsTransportOptions. The smallest nit would be whether you wanted to add something to README.

Sure thing! Coming right up.

@bradleyfalzon This is good for another look! 🙏

@bradleyfalzon ping for review

Sorry @wlynch! I've completely missed this, should we merge? It looks good to me.

oh sure! (I also forgot about this 😅 )


Update VDI connection instructions

NCI has upgraded the VDI, and connecting to the VDI is now slightly different
https://opus.nci.org.au/display/Help/4.+VDI+FAQ
These instructions are in the doco as well as the wiki.

@CEKrause I think I've added comments in the wiki and put in a request to update the doco, did I miss anything?


GSLUX-585: Layer comparator

JIRA issue
https://jira.camptocamp.com/browse/GSLUX-585
Description

[x] Add slider bullet in layer manager (only for the first layer)
[x] Add slider splitter on the map
[x] Accessibility on splitter (handle tab/left/right/esc keydown)
[x] Attach permalink params: lc (for on/off) and sliderRatio
[x] Check result in v3 integration

@mki-c2c Thanks for your review and your suggestions.
Added some improvement regarding accessibility.

Can you check please?

@mki-c2c can you review this PR and approve the changes (if ok with you) please?

Thanks for your approval.

Same as tobias. The slider has become decently smooth now. There is no more a frustrating behavior. Thanks. And it's true, the labelling is weird, because the corresponding layer is on the opposite side (label left, layer right). As it has always been ??

Label and layer are now on the same side, check my last commit: https://github.com/Geoportail-Luxembourg/luxembourg-geoportail/pull/55/commits/456bb783c4cf7f3b7e0f6c7a7da07b5bec0d3009


Open a modele

Hi,
is there a method to save a SQL sheet as a Model and then open it everytime I open some connections?

Could you clarify what exactly do you mean by "Model"?

Hello @Ferrox12
Could you clarify - what sheet? What instruction? Please, provide more examples.

In my case when I open a connection I must execute a SQL command. Is there a method to open in the sheet this query?

@Ferrox12 Is Datasets sound similar to your request?
https://dbeaver.com/2023/03/24/roles-in-team-edition-manager-editor-viewer/#:~:text=The concept of Datasets

Sorry for "Model" I mean a Script that has to be run everytime I open a connection

@Ferrox12 You can try adding bootstrap queries in the connection settings

Thank you very much!!!

@Ferrox12 I am closing the issue. I wish a nice experience working with the DBeaver