Reset mocks?

I have a "clock" module that provides a singleton instance, to use instead of calling "new Date()" directly. To unit test other modules that make use of the clock module, I have to mock out the singleton instance, and mock its "now()" method.
This means each invocation of "clock.now()" needs a different value. To achieve, this, for each invocation, I need to create a mock of the Clock class and inject it into the system under test (using mockery). This is not ideal; I'd much rather create a placeholder mock, inject it once, and apply a different setup within each unit test.
Is there some way to reset mocks? Or, at least, to apply "setup()" calls without falling into the "record/replay" functionality?

Latest typemoq (v0.3.3) includes changes to allow resetting mocks. Please give it a try.

Oh fantastic, thanks very much! I'll give this a go. Very much appreciated! :balloon: :dancers: :+1:


Switch build to Babel 7?

There is a babel-upgrade tool to automate upgrade and full changes in babel 7 here https://babeljs.io/blog/2018/08/27/7.0.0
We'd like to remove babel 6 from next version of debian and switched to using babel 7 for building es6-promise but it'd be nice to stay close to the versions you use.

Switching to babel 7 seems great, I welcome it. If i have some spare time, I'll doit myself.


feat(unzip): support for lzh files

Description

Supports lzh files in addition to zip files. This makes it possible to install "DirectShow File Reader プラグイン for AviUtl".
This PR contains commit #88.
Related Issue

Motivation and Context

How Has This Been Tested?

You can install the zip plugins.
You can install the lzh plugins.
It works not only with yarn start, but also with yarn package.

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

7zip-bin contains a lightweight version of 7zip that cannot handle minor formats such as lzh, while win-7zip contains the normal version of 7zip.

I didn't know that.
We certainly need to do so.
However, win-7zip only supports Windows, right?
If so, we feel like I have to use p7zip for UNIX as well.

Sure, it's Windows only. I don't have a mac, but if apm works on Linux, I'll consider introducing node-p7zip.
Does apm work on Linux or Mac?

I think we're making the current code cross-platform by using electron's features, but I haven't tested it yet.
Testing on Linux is easy, but testing on a Mac requires me to find someone who has one since I don't have a Mac either.

It was harder than I thought to create a Linux development environment, so I'm not going to do it.
I have pushed a version that fixes the regression caused by this PR. Is this acceptable?


Check what fields can be used as entity properties

Comment from Matt

Should we validate the type of fields that can be used as entity properties? My understanding is that groups and repeat groups are fields that can't be used as entity properties.
Less sure about this one: for now, should we prohibit any fields within a repeat group from being used as entity properties?

check what happens when a structural field is used in a bind that gets picked up by entities
validate and don't allow binds on structural fields if they are for some reason allowed currently
don't allow binds on fields within repeat groups
keep all this in mind for future versions of entities that use repeats and can make multiple entities per form

Check what happens when we try to add binds about entity properties to structural fields or fields within repeat groups and don't allow if they don't make sense.
We'll come back to an updated version of the entity XForm spec that allows for a form to create multiple entities in the future.

Bonus things to test:

try out a form that both produces and consumes a dataset
try out a form that tries to have multiple fields bind to the same dataset property

try out a form that tries to have multiple fields bind to the same dataset property

I do think we should address this one, as it seems important to the mental model. I also don't think that it'll really work as-is today. If a user really wants to feed multiple fields into a single dataset property, they can do so by combining the fields in a separate calculate field, then saving that calculate field to the property. I'm hopeful that it wouldn't take much to address this, maybe just adjusting a database constraint.

check what happens when a structural field is used in a bind that gets picked up by entities
validate and don't allow binds on structural fields if they are for some reason allowed currently

@lognaturel, do you know offhand whether these are possibilities? If not, then I think the only thing we need to do is disallow fields in repeat groups.

I do think we should address this one, as it seems important to the mental model. I also don't think that it'll really work as-is today.

I was imagining the last one parsed would win and that would be fine. But maybe that doesn't go well with the way user-specified properties are stored as json?

validate and don't allow binds on structural fields if they are for some reason allowed currently

pyxform will not allow this and I think that's enough.

disallow fields in repeat groups

We could do this at the pyxform level as well. Depending on what the parsing code does I imagine it will either crash or save every repeat instance (similar to the multiple matching savetos case above).
It feels ok to me to do nothing for 2022.3 and follow up either with a point release or in a later release.

As long as pyxform is doing this validation, it feels OK to leave most of this for later. :+1: Though I see that @ktuite has assigned herself to this issue. @ktuite, how do you think we should prioritize this?
I'd still like to make the following change for v2022.3:

try out a form that tries to have multiple fields bind to the same dataset property

I think it's important to do that because it involves modifying a database constraint. I think we don't want to have to deal with making that change retroactively. It'll be a super small code change, but I think worth doing.

I was imagining the last one parsed would win and that would be fine. But maybe that doesn't go well with the way user-specified properties are stored as json?

Hmm that part of the workflow might do what you say and be fine. But I strongly suspect that some SQL joins are assuming that the field:property relationship is 1:1 for a given form def and won't work if it's many:1. I'd be happy to make this change!

For release 2023.4, I'm thinking I need to remember what I informally tested before and add real tests.


Password Org Reset Design (web)

Functional Requirements
The web app must offer a password reset feature.
The app must send a password reset link to the organizer's email.
Non-Functional Requirements
The password reset process must adhere to security best practices.
The password reset email should be received by the user within 5 minutes.
Dependencies
Existing log-in functionality.
Supabase configuration for email services to handle password reset.
Resources
https://supabase.com/docs/guides/auth/auth-password-reset
https://supabase.com/docs/guides/auth/auth-email-templates
Design password reset page and email template - 3 hours

Completed


Enrichment Chamber won't accept certus/fluix crystal seeds

I'm trying to use Mekanism's Enrichment Chamber to grow Certus/Fluix in a processing chain. First I tried having the Thermal Expansion Sequential Fabricator auto-export directly into it without anything in between. That didn't work so I tried with itemducts. Still nope. Tried with Logistical Transporters, and still nope. Tried with a vanilla hopper, and nope. I tried reconfiguring the input to a different side because maybe it needed to be updated or something, and then I tried all those again with different side inputs. Again, didn't work. I ended up changing it out with the standard way of growing pure Certus/Fluix, which causes more lag and takes more time, maybe takes more energy idk. It very well might be an issue with one of the other mods in my custom modpack but I haven't tested to see what it could be yet.
Forge: <IP_ADDRESS>68 (current recommended version)
Mekanism: Mekanism-1.12.2-<IP_ADDRESS>9.jar
Mod List: https://hastebin.com/yubefomuca.md

Are they partially grown seeds? Could you please let me know the AE2 version, seems your hastebin expired

appliedenergistics2-rv6-stable-3


Fix: max- width instead of width and fade in and out on waiting text and images

This Pr references #587  and solves  on the styling  of chapter 3 challenge 2.

Thanks for submitting this pr. Let's put a pause on this until @GBKS can clarify on a couple issues. This isn't a blocker for launch.
I can confirm the waiting fades in and out which is nice. I'm not sure how the centered text should look though.

Thanks for working on this. I still see some layout issues and left-aligned text. Screenshots are via the Responsive design mode on Safari on Desktop.

Generally, it is not a good idea to use fixed width or height for content containers (especially height). Text will render at different lengths across translations, people change text size, etc.

Thanks for working on this. I still see some layout issues and left-aligned text. Screenshots are via the Responsive design mode on Safari on Desktop.

Generally, it is not a good idea to use fixed width or height for content containers (especially height). Text will render at different lengths across translations, people change text size, etc.

hey @GBKS  before i commit my changes i do what to know if this  loading style works
https://github.com/saving-satoshi/saving-satoshi/assets/90271995/90704101-840d-411a-aea6-380566542467

@GBKS  can i get your feedback on the new fix

Thanks for the tweaks. I still see layout issues on mobile.

Before any of the other 3 miners are loaded, the grid nicely fits into the screen. As soon as one of them is loaded, it expands to the screen edges (it should keep the first state with the padding). This is likely due to the boxes responding to the width of the labels. The layout logic should be reverse where the box defines the width, and the label resizes its width accordingly.
Again, it is not a good idea to use fixed heights. Using a min-height is a better idea, because it allows the container to resize if the content happens to be longer (localization to another language, for example, might cause this).

Thanks for the tweaks. I still see layout issues on mobile.

Before any of the other 3 miners are loaded, the grid nicely fits into the screen. As soon as one of them is loaded, it expands to the screen edges (it should keep the first state with the padding). This is likely due to the boxes responding to the width of the labels. The layout logic should be reverse where the box defines the width, and the label resizes its width accordingly.
Again, it is not a good idea to use fixed heights. Using a min-height is a better idea, because it allows the container to resize if the content happens to be longer (localization to another language, for example, might cause this).

Alright, i have been able to restraint from using fixed width, and i have been able to fix the layout issues mentioned, you could review


buffer completion for large CJK document leads to segmentation fault of neovim

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
I'm using LazyVim, which enables the buffer completion source for blink.cmp
https://github.com/LazyVim/LazyVim/blob/dc4345a5ee23ffb4aad50c01eca49d5bc47a96c3/lua/lazyvim/plugins/extras/coding/blink.lua#L76
when editing a large CJK document and modifying a long sentence in insert mode, neovim crashes with a segmentation fault right after the modification.
I can no longer reproduce the crash once I disable buffer from completion sources, so I suspect there's something wrong with it
this is a random large Chinese document for reproducing the issue
https://github.com/ArcturusZhang/Classical-Mechanics-Lecture-Notes/raw/refs/heads/master/chapter/chapter2.tex
here is a cast for the issue

below is the stack trace
#0  __strlen_avx2 () at ../sysdeps/x86_64/multiarch/strlen-avx2.S:76
No locals.
#1  0x0000650858389eea in xstrdup (str=0x0) at /usr/src/debug/neovim/neovim/src/nvim/memory.c:469
No locals.
#2  0x0000650858356744 in nlua_luv_thread_common_cfpcall (lstate=0x72535a3e0380, nargs=1, nresult=-1,
    flags=1, is_callback=is_callback@entry=false) at /usr/src/debug/neovim/neovim/src/nvim/lua/executor.c:279
        error = <optimized out>
        retval = <optimized out>
        top = 2
        status = 2
#3  0x0000650858356895 in nlua_luv_thread_cfpcall (lstate=<optimized out>, nargs=<optimized out>,
    nresult=<optimized out>, flags=<optimized out>)
    at /usr/src/debug/neovim/neovim/src/nvim/lua/executor.c:244
No locals.
#4  0x0000725360b2b644 in luv_work_cb (L=0x72535a3e0380) at /usr/src/debug/libluv/luv-1.48.0-2/src/work.c:107
        i = <optimized out>
        req = <optimized out>
        work = 0x6508705fac00
        ctx = <optimized out>
        lctx = 0x72535a3ea4d8
        top = 0
#5  0x0000725360a17f06 in lj_BC_FUNCC () at buildvm_x86.dasc:857
No locals.
#6  0x0000725360a2c0aa in lua_pcall (L=L@entry=0x72535a3e0380, nargs=nargs@entry=1,
    nresults=nresults@entry=0, errfunc=errfunc@entry=0)
    at /usr/src/debug/luajit/LuaJIT-fe71d0fb54ceadfb5b5f3b6baf29e486d97f6059/src/lj_api.c:1122
        g = 0x72535a3e03e0
        oldh = 0 '\000'
        ef = <optimized out>
        status = <optimized out>
#7  0x00006508583566e7 in nlua_luv_thread_common_cfpcall (lstate=lstate@entry=0x72535a3e0380,
    nargs=nargs@entry=1, nresult=nresult@entry=0, flags=flags@entry=1, is_callback=is_callback@entry=false)
    at /usr/src/debug/neovim/neovim/src/nvim/lua/executor.c:263
        retval = <optimized out>
        top = 2
        status = <optimized out>
#8  0x0000650858356895 in nlua_luv_thread_cfpcall (lstate=lstate@entry=0x72535a3e0380, nargs=nargs@entry=1,
    nresult=nresult@entry=0, flags=flags@entry=1) at /usr/src/debug/neovim/neovim/src/nvim/lua/executor.c:244
No locals.
#9  0x00006508583568dd in nlua_luv_thread_cfcpcall (lstate=0x72535a3e0380, func=<optimized out>,
    ud=0x6508705fac00, flags=1) at /usr/src/debug/neovim/neovim/src/nvim/lua/executor.c:252
        retval = <optimized out>
#10 0x0000725360b2b8aa in luv_work_cb_wrapper (req=0x6508705fac00)
    at /usr/src/debug/libluv/luv-1.48.0-2/src/work.c:155
        work = 0x6508705fac00
        L = 0x72535a3e0380
        lctx = <optimized out>
        i = <optimized out>
#11 0x000072536090030a in worker (arg=0x0) at src/threadpool.c:122
        w = 0x6508705fac58
        q = 0x6508705fac70
        is_slow_work = 0

Relevant configuration
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
	spec = {
		{ "LazyVim/LazyVim", import = "lazyvim.plugins" },
		-- uncomment the following block avoids segmentation fault
		-- {
		-- 	"saghen/blink.cmp",
		-- 	opts = {
		-- 		sources = {
		-- 			default = function()
		-- 				return { "lsp", "path", "snippets" }
		-- 			end,
		-- 		},
		-- 	},
		-- },
	},
})

neovim version
v0.10.2
blink.cmp version: branch, tag, or commit
v0.7.6

Do we really need this '+ 1'?
It seems that removing this line solves the problem.
https://github.com/Saghen/blink.cmp/blob/217f670d85d8b56f48146636b61019ccd072f9cc/lua/blink/cmp/sources/buffer.lua#L21-L24
It also produces a (seemingly) weird chunk:
Typing ipsum on a text:
Lorem {cursor} dolor sit amet, consectetur adipiscing elit. Gravida mus sociosqu

With + 1:
{
  _line = "Lorem i  dolor sit amet, consectetur adipiscing elit. Gravida mus sociosqu"
}

Without + 1:
{
  _line = "Lorem   dolor sit amet, consectetur adipiscing elit. Gravida mus sociosqu"
}

Do we really need this '+ 1'? It seems that removing this line solves the problem.

confirmed

Thanks for debugging this @itepechi!


os error 87

Describe the bug
I'm trying to download "Jobless Reincarnation" with dual audio and brazilian subtitle, but it always five me this error after donwloading video and audio. It says incorrect parameters.

To Reproduce
Steps / command to reproduce the behavior:
$ crunchy archive -a ja-JP -a pt-BR -s pt-BR -m audio -o "../_Novos/Mushoku Tensei: Jobless Reincarnation (2021) {tmdb-371310}/S{season_number} E{episode_number} - {title}-org.mkv" https://www.crunchyroll.com/series/G24H1N3MP/mushoku-tensei-jobless-reincarnation[S01E01-E100,S02-S50]

Expected behavior
Donwload the anime, with audio in Japanese and Portuguese and Brazilian subtitles.
Client (please complete the following information):

OS: Win10
Version 3.0.3

I was unable to reproduce this on macOS using the latest commit of crunchy-cli, I installed two episodes before I interrupted the command.

This is a Windows specific error. When looking for the error on the internet and given that I cannot reproduce this on Linux too, the cause is probably that the underlying ffmpeg command is too long for Windows (once again a weird windows-only edgecase 🙃).
The command can be shortened down by using ffmpegs' -f concat, I can see if it really works/fix it later today.

=P
I will get the linux version then and run in my other computer
@ByteDream if you want me to test something here on Windows, just ask.

(once again a weird windows-only edgecase 🙃)

Gotta love those Windows-only bugs lol

Gotta love those Windows-only bugs lol

Yes windows got me once or twice with its weird behavior (e.g. d090a5a1daa781be9aac83950a8318267ccd5317 I was so pissed when I did this commit)

So I've tried the command on a Windows VM and I also got an error but with a different error code. The cause may not be the length of the underlying command but special characters in the filename which are forbidden on Windows (there are plenty). Can you re-try the command without any of the colon : character in the filename?

Ok @ByteDream now I'm just feeling dumb. The anime name have a ":" and this was the cause of the error.
But it still is a strange behavior, since it downloads all the video and audio tracks and just throw an error when trying to download the subtitle, as show in my screnshot up there.

But it still is a strange behavior, since it downloads all the video and audio tracks and just throw an error when trying to download the subtitle, as show in my screnshot up there.

It's probable that it had finished downloading the subtitles and had gone into the FFmpeg phase, which is right after.

Makes sense @valentinegb since the subtitles are what is faster to download (I believe)

I had sanitation already in place to prevent such errors but it would only sanitize the replaced output template options instead of the full output argument.
The commit above fixes this

I reverted the commit in v3.2.0. There are simply too many cases where the fix breaks other parts/possibilities (on Windows 🙃), e.g. Windows named pipes, locations on other drives (#289), ...

 Maybe add something on the readme about it?
"The path of the output can't contain any : or /"


_eventCallback(): Reason: 2 - AUTH_EXPIRE (MEGH-2614)

I tried two correct Access Point  with same ERRORS
I cleared the flash and re-flash the RmakerSwitch a few times..
Im using this Wemos R32 ( ESP32 in Uno form factor )
https://www.hackster.io/NYH-workshop/wemos-r32-with-arduino-startup-guide-7bc841
[ 70766][I][RMaker.cpp:19] event_handler(): RainMaker Claim Successful.
[ 95845][W][WiFiGeneric.cpp:829] _eventCallback(): Reason: 2 - AUTH_EXPIRE
E (102590) wifi_prov_mgr: STA Disconnected
E (102590) wifi_prov_mgr: Disconnect reason : 2
E (102594) wifi_prov_mgr: STA Auth Error
[ 95853][E][WiFiGeneric.cpp:448] _arduino_event_cb(): Provisioning Failed: Reason : Authentication Failed
[ 97022][W][WiFiGeneric.cpp:829] _eventCallback(): Reason: 2 - AUTH_EXPIRE
E (103767) wifi_prov_mgr: STA Disconnected
E (103769) wifi_prov_mgr: Disconnect reason : 2
E (103771) wifi_prov_mgr: STA Auth Error
[ 97030][E][WiFiGeneric.cpp:448] _arduino_event_cb(): Provisioning Failed: Reason : Authentication Failed
[ 98149][W][WiFiGeneric.cpp:829] _eventCallback(): Reason: 2 - AUTH_EXPIRE
E (104894) wifi_prov_mgr: STA Disconnected
E (104895) wifi_prov_mgr: Disconnect reason : 2
E (104897) wifi_prov_mgr: STA Auth Error
[ 98156][E][WiFiGeneric.cpp:448] _arduino_event_cb(): Provisioning Failed: Reason : Authentication Failed

@stanleyseow , can you add handling for the event ARDUINO_EVENT_PROV_CRED_RECV similar to this, by editing the sysProvEvent handler and see if you are getting the correct credentials? Can you also once try connecting to an Open AP (no security)?

Closing this issue as it is stale and no response was received. Please update here if the issue still persists

I just got my LOLIN esp32-c3 mini WIFI working by digging this piece of errata from their site. So ANNOYING!!!
//You have to add this code after running WiFi.softAP or WiFi.begin for the Wifi to work.
WiFi.setTxPower(WIFI_POWER_8_5dBm);             //https://www.wemos.cc/en/latest/tutorials/c3/get_started_with_arduino_c3.html#wifi


Desync - Another extension

I ran into synchronization issues when using Live Share with x extension that modifies files.
If you run into this issue, please follow the instructions here to capture diagnostic information that will help us figure out the cause.

Closing as stale.


On Fuse Crash when Unmounting Views

Issue: Views with an onFuse callback, that has code to unmount itself, crashes.
For example:
_getImage() {
    if (this.state.showImage) {
      return (
        <ViroImage position={[0, 0, -1]}
          source={{uri: "https://images.sftcdn.net/images/t_optimized,f_auto/p/50d2f4ce-96d1-11e6-8a53-00163ed833e7/3980672443/google-earth-screenshot.jpg"}}
          onFuse={()=>{this.setState({showImage : false})}} />
      );
    } else {
      return;
    }
  }

This should have been fixed by build 2.0.1 as mentioned at http://docs.viromedia.com/docs/releases. Closing.


Other bottom panels are not reshown when the console is automatically closed after stopping the running project

Godot version
v3.5.beta5.official [815f7fe63]
System information
win10 64
Issue description
The animationplayer dock is not activated if the console is closed after executing.
https://user-images.githubusercontent.com/19652075/167369803-d8b1d83e-6314-4d98-bf13-d63fdd5b736c.mp4
Steps to reproduce
See the video.
The animation dock was open.
Run the project.
The output was opened automatically (the option open console automatically is activated)
Execution stopped.
The console was closed automatically (the option to close output automatically is activated).
The animationplayer node is pressed but its dock does not open.
Minimal reproduction project
...

I thought it was a bug.
I click on the animationplayer node but its dock doesn't open until I click on another node and click on the animationplayer node again.


Match requests for path subpaths

Swagger does not allow you to have a single path to match all subpaths below it.  The purpose of this issue is to allow this at a route matching level in swagger-tools.  The way this will work is you will annotate your path with the x-swagger-router-handle-subpaths vendor extension and all requests for subpaths of said path, that do not match an explicit path defined in Swagger, will be associated with the annotate path so that swagger-tools can do the appropriate routing.
This is much like #159 but in reverse.
/cc @theganyo

Minor correction: The marked path will match the path as specified + all subpaths.


Add Metal API for GRBackendRenderTarget to all platforms

Description of Change
Add public GRBackendRenderTarget (int width, int height, GRMtlTextureInfo mtlInfo) to the netXX builds.
Bugs Fixed

Fixes #3069

API Changes
Added:
public GRBackendRenderTarget (int width, int height, GRMtlTextureInfo mtlInfo)
Behavioral Changes
None.
Required skia PR
None.
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[ ] Merged related skia PRs
[x] Changes adhere to coding standard
[ ] Updated documentation

/azp run

/azp run

thanks @mattleibow !


Cannot use useTrackRenders when component is used in the wrapper option

When using renderStream.render, I am unable to to use components that use the useTrackRenders hook in the wrapper option. As an example, see this test from an Apollo Client bug that I'm working on. I'd like to move the <Suspense /> and <ErrorBoundary /> components to the wrapper option. When doing so however, I see the error:
useTrackRenders: A Render Stream must be created and rendered to track component renders

Ideally useTrackRenders can be used for components in the components passed to render and wrapper. This would also make it easier for more of these complex cases to use rerender to avoid the need to pass that whole component tree again.

Good point. We probably need to swap ParentWrapper and Wrapper for that here:
https://github.com/testing-library/react-render-stream-testing-library/blob/c4b152e54c2816ae17ef35d9ccef893cf5c14ac8/src/renderStream/createRenderStream.tsx#L259-L261

Closing as the argument against this change makes sense. Sticking with the rule of "components that track renders should be part of the test" makes a lot of sense to me and should help reduce confusion.


Prevent crash in dataclasses with no-init params

@cgarciae

Thanks @NeilGirdhar ! Can you fix the pre-commit issue?

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a572f6a) 53.80% compared to head (2ef8783) 53.80%.

Files
Patch %
Lines

flax/experimental/nnx/nnx/module.py
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3514      +/-   ##
==========================================
- Coverage   53.80%   53.80%   -0.01%
==========================================
  Files          98       98
  Lines       11524    11526       +2
==========================================
+ Hits         6201     6202       +1
- Misses       5323     5324       +1

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

Thanks @NeilGirdhar !


Error: Error installing package 'protolite':

Hi, seem to have an issue when trying to build using renv, I've had a google around but the only fixes I've seen seem to apply to fixing this on a local machine rather than with this build pack. Any ideas how to go about fixing this?
`       - Installing protolite ...                      FAILED
Error: Error installing package 'protolite':
=====================================
   * installing *source* package ‘protolite’ ...
   ** package ‘protolite’ successfully unpacked and MD5 sums checked
   ** using staged installation
   Package protobuf was not found in the pkg-config search path.
   Perhaps you should add the directory containing `protobuf.pc'
   to the PKG_CONFIG_PATH environment variable
   No package 'protobuf' found
   Using PKG_CFLAGS=
   Using PKG_LIBS=-lprotobuf
   Found C++17 compiler: g++ -std=gnu++17 -g -O2
   ------------------------------[ ANTICONF ]-----------------------------
   Configuration failed to find protobuf. Try installing:
    * deb: libprotobuf-dev (Debian, Ubuntu, etc)
    * rpm: protobuf-devel (Fedora, EPEL)
    * brew: protobuf (OSX)
   If protobuf is already installed, check that 'pkg-config' is in your
   PATH and PKG_CONFIG_PATH contains a protobuf.pc file. If pkg-config
   is unavailable you can set INCLUDE_DIR and LIB_DIR manually via:
   R CMD INSTALL --configure-vars='INCLUDE_DIR=... LIB_DIR=...'
   ----------------------------[ ERROR MESSAGE ]----------------------------
   <stdin>:1:10: fatal error: google/protobuf/message.h: No such file or directory
   compilation terminated.
        Push rejected, failed to compile R (shiny) app.
!     Push failed`

Hi @benjcowley
The error is likely due to missing the libprotobuf-dev system dependency.
You can install libprotobuf-dev by adding an Aptfile to the root of your project directory which lists the dependency, as described in the README under Installing Binary Dependencies.
See the aptfile test for an example.


Support for Swift Package Manager

It would be great to have support for Swift Package Manage as an alternative to CocoaPods/Carthage, now that it is built into Xcode and supports binary frameworks:
https://developer.apple.com/videos/play/wwdc2019/416/

Hi @pierceglennie .
We just published v4.0.0-beta.1 . Check the Migration Guide, integration of the SDK should have become way easier.
Now that we are distributing the SDK as an xcframework we will look into adding support for Swift Package Manager in an upcoming release.
Kind regards,
Lukas

Thanks. Good to see some updates coming through. Will try to migrate in next couple weeks and let you know if I face any issues.

Hey @pierceglennie
We published v4.0.0. We've outlined support for Swift PM in the README. It's not seamless yet but we hope for SR-13343 to be addressed soon.
Best,
Lukas

@mollidor Just to let you know I upgraded to v4.0.0 and migrated to Swift PM. I've not been able to test everyone yet but it seems to have gone smoothly and it's meant I'm no longer getting any warnings on build (which the old SumUp SDK gave me). Thanks for your efforts

That's great to hear 🎉 . Thanks a lot for coming back to us, much appreciated.


[QUESTION] Access Path Mapping from Filters

There is a way to access a path mapping in the filter?
For example
 METHOD_ADD(Foo::func,"/foo/{1}",Get,"FooFilter");

Can I access mapped parameter {1} inside doFilter()?
I know how can I access it in a HttpController using
 void func(const HttpRequestPtr& req, std::function<void (const HttpResponsePtr &)> &&callback, int arg1);

I need to add int arg1 in doFilter as well to achieve this?

Currently, you cannot do this, the doFilter method is a virtual function of the base class, so its signature is fixed. I will consider adding a method to the HttpRequset class to access these parameters.

@TheEnigmist I made a patch #1765 for this, please check it out. thanks.

@TheEnigmist I made a patch #1765 for this, please check it out. thanks.

OH thx. I use vcpkg to dev drogon on my windows system (and maybe I will use it on target linux too for build it). Do I need to manually build drogon to test this patch?
Can I use vcpkg to build drogon src?

@TheEnigmist I made a patch #1765 for this, please check it out. thanks.

OH thx. I use vcpkg to dev drogon on my windows system (and maybe I will use it on target linux too for build it). Do I need to manually build drogon to test this patch? Can I use vcpkg to build drogon src?
You can add drogon source code to your project.

You can add drogon source code to your project.

Ok, tested with last code downloaded from routingparams branch and it works like a charm.
Tested on Windows.
I think we can close this issue and thanks for the fast reply.


⚠ Bofrost.fr has degraded performance

In ed80443, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1015 ms

Resolved: Bofrost.fr performance has improved in 25b6c22 after 1 hour, 58 minutes.


fix error message

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.192%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2343

Relevant Lines:
3487

💛  - Coveralls


Add test coverage

@daniel-thom great, this PR restores the Coverage levels.

@daniel-thom do you mind removing nightly from https://github.com/NREL-SIIP/PowerSystems.jl/blob/master/.github/workflows/pr_testing.yml also?

@daniel-thom great, this PR restores the Coverage levels.

I'm not sure about restoration. This code was not tested previously. I don't know what caused the coverage drop.


Add CONTRIBUTING.md file

Since this was referenced in the PULL_REQUEST_TEMPLATE example and this project did not have a CONTRIBUTING.md, figured I'd take a shot at adding one.

Wondering if something isn't configured correctly in Travis (since I just added a text file).

Hello @jeffgerlach,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Remove NO_DEBUG define

The define is never used anywhere.

Good catch, it also seems that NDEBUG is never used anywhere.
We could also replace UPNP_HAVE_DEBUG and DEBUG defines by a single define as both are changed at the same time.


Load configuration from /config.json

Renames the /config.js endpoint. Returns json
instead of setting window.config. This is more
flexible, but also somewhat more involved, since
we have to load configuration asynchronously
from the client code.
Also adds source map generation for js and css
(easier debugging)

I was going to suggest this! Thanks for implementing before I had the
chance to.

Do we want to call this config.json or just config? I can see the case for the former, as it looks like a file and could be implemented by one. The latter looks more like other things like /healthcheck (assuming content-type is correct)
Thoughts?

The idea was that, like you said, people could host the ui from a web server like apache httpd, and simply configure it by creating their own config.json.

WFM


fix: [#4483] Switching npm dependency bcrypt to bcryptjs

Fixes # 4483
#minor
Description
This PR changes the use of bcrypt for bcryptjs to improve the portability of botbuilder-js.
Specific Changes

Replaced bcrypt ^5.0.1 by bcryptjs ^2.4.3.

Testing
The following image shows the CosmosDbKeyEscape unit tests working with the hashSync method from bcryptjs.

Promoted /4669


AC: Create initial webpage with header, content section, and footer

WHEN I load the portfolio
THEN I am presented with a page containing a header, a section for content, and a footer

Done


Clear expiration with update cmd doesn't work as documentated

Under Update cmd for expiration argument it says "To unset the expiration for the banner, supply an empty value to this argument.".
I tried

--expiration
--expiration ''
--expiration ""

as in

az devops admin banner update --id '' --org https://dev.azure.com/MyOrgUrl/ --expiration ''

None of these worked, got an error msg:

az : ERROR: az devops admin banner update: error: argument --expiration: expected one argument

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c28dc091-8cff-450b-14a8-ef2399b80a25
Version Independent ID: 85f56f24-276c-a2d7-eccd-cb877c65b1d6
Content: az devops admin banner
Content Source: latest/docs-ref-autogen/ext/azure-devops/devops/admin/banner.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

I raised two related issues that went under a different page path for some reason, perhaps this one should move under the same path:

https://docs.microsoft.com/en-us/cli/azure/devops/admin/banner

As this may be a bug, I opened an issue in the code repo, https://github.com/Azure/azure-devops-cli-extension/issues/867.

Closing this issue as you have created an issue in our repo - https://github.com/Azure/azure-devops-cli-extension/issues/867. Please track that issue for all future updates.


GEOPY-693: version 0.9.0

merging branch release/0.9.0

see https://github.com/MiraGeoscience/geoapps/pull/437 instead


Update readme for finance-vix data package

The finance-vix data package is used in tutorials
Update the README.md to present a better example and mention that the data package is used in tutorials

I don't believe we should publish this data given the copyright notice.
I believe the licence applied and the statement on https://datahub.io/core/finance-vix#license is incorrect.
Seeking advice @rufuspollock

@Stephen-Gates can you clarify what about that copyright notice (which i understand covers the content of that site not data). Plus this is the US and this is a simple collection of factual data so I think this is PD 😄

@rufuspollock you may be right but I read section 2 differently. Emphasis mine

Copyrights, Trademarks, Service Marks, And Proprietary Rights.

The design of, as well as the materials, content, text, graphics, and information contained in, the Website ("Materials") are owned, licensed, or controlled by Cboe or the party credited as the owner, licensor, or provider of such Materials. The Materials on the Website, including, but not limited to all menu pages, content, text, video, audio and graphics are protected by copyrights, trademarks, service marks, and/or other proprietary rights and laws of the United States and other countries. The Website is also protected as a collective work or compilation under United States copyright and other laws. You agree to comply with all applicable copyright and other laws. You agree to comply with any guidelines for linking to the Website that may be posted on the Website. You may view, print and download one copy of the contents for your personal noncommercial use in connection with products and services offered by Cboe, provided, you maintain all copyright, trademark and other notices contained on the Materials. You may not otherwise copy, reproduce, alter, store either in hard copy or in an electronic retrieval system, license, transmit, display, broadcast, create a derivative work (for example, a financial product, service or index) from, publish, rent, sublicense, distribute, or otherwise use in whole or in part in any other manner the Materials without Cboe's prior written consent except to the extent that such use constitutes "fair use" under the "Copyright Act of 1976", as amended from time to time. Request for such consent must be made in writing and should be mailed to Cboe Options Exchange, Attention: Internet Marketing, 400 S. LaSalle Street, Chicago, IL 60605; or you may send an e-mail via the "Contact Cboe" form located at http://www.cboe.com/Contact. Any Materials contained in the Website that are owned by, or about, you shall not be subject to the limitations on use contained in these Terms.

The data is not credited to another party, so I can only assume it is copyright Cboe.
Cboe are a US company, not government, so I believe copyright applies.
The data may be facts but perhaps the work is in how the data was selected and organised.
Rufus would you please edit the README.md as I don't believe I have the knowledge to address licensing correctly.

@Stephen-Gates this talks about content not data. And the point i made about copyright is not about federal public domain dedication but about the Feist decision.

Thanks @rufuspollock
https://github.com/frictionlessdata/example-data-packages/pull/46


Odd culture conversions in colonies

Reported here: https://forum.paradoxplaza.com/forum/index.php?threads/eu4-to-vic2-converter-release-thread.973026/page-166#post-25861384
Mod and save here: https://forum.paradoxplaza.com/forum/index.php?threads/eu4-to-vic2-converter-release-thread.973026/page-166#post-25861396

This is an 17th Oct 1792 save of a game where player created a custom norwegian germanic nation of Sundaland at over existing south american clay recently (1690s) - some colonized, some not, resulting in a messy history. Sample example:

For example this province was supposed to be owned by someone else, but is not.
Also since culture is converted 17 days prior to save date, there is no noticeable conversion.
Nothing to fix here, can close.


Add support for checking doctest strings in user defined functions

Description
I usually put doctest strings when I define a new (non-trivial) function.
def euclid_mcd(a: int, b: int) -> int:
    """Return the MCD between positive a, b.

    >>> euclid_mcd(42, 24)
    6
    >>> euclid_mcd(24, 42)
    6
    >>> euclid_mcd(42, 42)
    42
    """
    assert a > 0
    assert b > 0
    if a < b:
        a, b = b, a
    if (a != b):
        r = a - b
        return euclid_mcd(b, r)
    return a

I was not able to easily check the doctest in marimo. I tried the following strategies:

import doctest; doctest.testmodule This normally works in a Jupyter notebook, in marimo I get "This cell raised an exception: TypeError('MarimoPdb.init() got an unexpected keyword argument 'nosigint'')", with Python 3.11
python -m doctest notebook.py This doesn't work since the doctest string is hidden inside a __ upper level function and ignored by doctest
marimo export script notebook.py -o notebook_test.py; python -m doctest notebook_test.py This fails with a "ContextNotInitializedError" on a call to mo.status.progress_bar

Any other suggestion?
Suggested solution
Add specific support for checking doctest strings with a cell contextual menu.
Alternative
No response
Additional context
No response

python -m doctest notebook.py This doesn't work since the doctest string is hidden inside a __ upper level function and ignored by doctest

We plan to hoist functions out of the upper __ - see https://github.com/marimo-team/marimo/issues/2293

marimo export script notebook.py -o notebook_test.py; python -m doctest notebook_test.py This fails with a "ContextNotInitializedError" on a call to mo.status.progress_bar

I will look into fixing this. This is likely the best solution for now, once we fix this.

Fix for supporting for option 3: https://github.com/marimo-team/marimo/pull/2352

Also have a fix for option 1: #2353


Do not store downloaded files in <>/site-packages/download

Seems like <>/site-packages/download is not the best place for downloaded models.
<>/site-packages/deeppavlov/download or ~/download might be a better decision.
Also <>/site-packages/utils is not the best place for utils specific to deeppavlov library.

relies on #112

Done in #574
moved to ~/.deeppavlov/


Add environment.yml and requirements.txt for Tingley septal conversion

This is to discuss how (or if) we should store the requirements.txt (for pip) and the environment.yml (for conda). I added the frozen version of the state of the repos just after I ran the conversion.
Right now, they just stand as an indicator of what libraries the environment that I used in dandi had. I am not sure this will be sufficient to reproduce the environment as I have not tested it but maybe this should be enough?
Let me know what you think.

In general I would think all we really need is (a) version of nwb-conversion-tools, and (b) version of spikeextractors. Those are the API's that have the most changes related to debugs, function calls, etc.
While the full frozen state would indeed be requires to perfectly (100%) ensure reproducibility of original files, this might be overkill for our purposes.

In general I would think all we really need is (a) version of nwb-conversion-tools, and (b) version of spikeextractors. Those are the API's that have the most changes related to debugs, function calls, etc.
While the full frozen state would indeed be required to perfectly (100%) ensure reproducibility of original files, this might be overkill for our purposes.

@CodyCBakerPhD
The changes that we used in both  nwb-conversion-tools and spikeextractors are not included in any tagged version of them, aren't they? (i.e. the latest additions to cellexplorersortingextractor in spikeextractors).

@h-mayorquin No, probably not; you can pin it to the commit hash of the version state used successfully to run the conversion though (example: https://github.com/catalystneuro/nwb-conversion-tools/blob/main/requirements-full.txt#L13-L14)

@h-mayorquin No, probably not; you can pin it to the commit hash of the version state used successfully to run the conversion though (example: https://github.com/catalystneuro/nwb-conversion-tools/blob/main/requirements-full.txt#L13-L14)

@CodyCBakerPhD
The latest commit follows your suggestion above to only have the pinned versions of nwb-conversion-tools and spikeextractors.

@h-mayorquin Looks great!


Batteryservice example does not work

Hi Adam,
I try to use pybleno on Ubuntu 18.04(using NVIDIA jetson nano).
My goal is  to have a jetson nano(as a peripheral) send an integer to an iPad(as a central) which uses the CoreBluetooth framework.
So, I want to check pybleno's program(on jetson nano) can send an integer to the iPad app like LightBlue.
First, I downloaded the pybleno like this:
$ git clone https://github.com/Adam-Langley/pybleno
Then, I tried batteryservice example like this:
$ sudo python3 main.py
I got this message on jetsons terminal.
Hit  to disconnect
on -> stateChange: poweredOn
Exception in thread HCISocketPoller:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/BluetoothHCI/BluetoothHCI.py", line 197, in _socket_poller
self._socket_on_data_user_callback(bytearray(data))
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Hci.py", line 387, in onSocketData
self.processCmdCompleteEvent(cmd, status, result)
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Hci.py", line 488, in processCmdCompleteEvent
self.emit('leAdvertiseEnableSet', [status])
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Emit.py", line 17, in emit
handler(*arguments)
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Gap.py", line 190, in onHciLeAdvertiseEnableSet
self.emit('advertisingStart', [error])
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Emit.py", line 17, in emit
handler(*arguments)
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Bindings.py", line 115, in onAdvertisingStart
self.emit('advertisingStart', [error])
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Emit.py", line 17, in emit
handler(*arguments)
File "/usr/local/lib/python3.6/dist-packages/pybleno/Bleno.py", line 158, in onAdvertisingStart
self.emit('advertisingStart', [error])
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/Emit.py", line 17, in emit
handler(*arguments)
File "main.py", line 20, in onAdvertisingStart
print('on -> advertisingStart: ' + ('error ' + error if error else 'success'));
TypeError: must be str, not Exception
Thanks!

Hi Adam,
When I use this command,
sudo hciconfig hci0 down
sudo hciconfig hci0 up
sudo python3 main.py
I got this error message.
Hit  to disconnect
on -> stateChange: poweredOn
on -> advertisingStart: success
setServices: success
Exception in thread HCISocketPoller:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/usr/local/lib/python3.6/dist-packages/pybleno/hci_socket/BluetoothHCI/BluetoothHCI.py", line 194, in _socket_poller
data = self._socket.recv(1024)         # blocking
BrokenPipeError: [Errno 32] Broken pipe
In this case, LightBlue on iPad can connect to batteryservice example program on jetson nano only about 30 seconds.
Thanks!


Modify spelling and add more clear description on testnet page

Checklist

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix
As mentioned in #963 by @rdlrt we should change testntets to plural. I've modified some of the spelling on the page. I've also moved test ada section under wallet section to make the guide more clear. And with a cherry on top made some clarifications on what wallets and external sources support what specific testnet networks.
One thing I'm missing is where to get Guild Network test ada, if someone can add that or direct me where to find it I would be happy to add it as well

Good to talk in plural about testnet(s). We might add this to style guide.

I hate dragging everyone through these grammar issues but since https://github.com/cardano-foundation/developer-portal/pull/983/commits/09b64a8c240361cf9c3836bfe133c31df9d7cbfe only corrected the Mainnet spelling once (in response to the example in my review, not every use of uncapitalised "mainnet" in the document), I just changed it everywhere except:

mainnet functionality because it's referring to the type of functionality (vs testnet functionality) rather than the Mainnet itself.
Cardano mainnet because here it means generally a mainnet for the (proper name) Cardano.

The rest of the places Mainnet, being a proper noun, has been capitalised in https://github.com/cardano-foundation/developer-portal/pull/983/commits/8d78095a0011fca43fed5ea6168d9af50156cc65.  @katomm since this directly contradicts what is in the Style Guide I'm not going to merge this unless you concur.
And if you do (as I said in review comment above) there needs be another PR (which I am willing to do) to update the term Mainnet everywhere in the Dev Portal and correct the rule in the Style Guide.  (Not opening up a discussion issue for this because all the information for that deliberation is right here already.)
cc @os11k @fill-the-fill @rdlrt

@rphair the explanation seems to me very good and understandable; we should then make so in the style guide to get consistency. (same would apply to testnet(s) then?)

sure @katomm - in an upcoming PR I'll update the Style Guide for mainnet including a search for mainnet and testnet on the other markdown pages with corrections wherever they need to be capitalised.  Hopefully this job will not be too much bigger than it sounds. 😬
I believe this is important for the same reason as spelling ada correctly in each context is important... the Developer Portal is not just a canonical documentation reference but also a trusted source for correctness in many other documentation sets.  If we fix these things & keep their usage consistent here, it will improve understanding & documentation quality for the Cardano community as a whole. 🧐


help me pls

I just can't see the 'run' option at the place that is supposed to be (at the top right corner, also next to 'more actions' three points option), so now I can't execute my code well. Pls help me, appreciate it

Version: 1.90.2
Commit: 5437499feb04f7a586f677b155b039bc2b3669eb
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

What documentation is telling you to expect that button there?

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


mini.files closes and does not reopen

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.files
Description
I've bound <leader>e to the function if not require("mini.files").close() then require("mini.files").open() end. This opens mini.files when I press these keys. However, if I exit using the q key after opening it for the first time or after resetting with <BS>, it sometimes doesn't open again when I press <leader>e or using commands. This issue doesn't happen every time, but it's not uncommon.
I'm using AstroNvim, but there doesn't seem to be any other plugin interfering with the functionality of mini.files.
The following is my mini-files.lua configuration
return {
  "echasnovski/mini.files",
  dependencies = {
    "nvim-tree/nvim-web-devicons",
    {
      "AstroNvim/astrocore",
      opts = {
        mappings = {
          n = {
            ["<Leader>e"] = {
              function()
                if not require("mini.files").close() then require("mini.files").open() end
              end,
              desc = "Explorer",
            },
          },
        },
      },
    },
  },
  specs = {
    { "neo-tree.nvim", optional = true, enabled = false },
  },
  opts = {},
}

Neovim version
v0.10.1
Steps to reproduce

toggle with if not require("mini.files").close() then require("mini.files").open() end
Open with the key previously bound and close with q.
mini.files just doesn't open again.

Expected behavior
mini.files opens correctly no matter how I close it.
Actual behavior
Frequently, mini.files stops functioning if I exit with the q key. It requires restarting Neovim to regain functionality.

Thanks for the issue!
Unfortunately, I can not reproduce with the following setup:
require('mini.files').setup()
vim.keymap.set('n', '<Leader>et', function()
  if not require('mini.files').close() then require('mini.files').open() end
end)

Executing <Leader>et toggles explorer (closes if shown, opens if not shown). Also opens it if explorer was previously closed with q.

I am pretty confident that this is not an issue with 'mini.files'. If anything, it looks like this config loads 'neo-tree.nvim' which has <Leader>e mapping defined in AstroNvim. So <Leader>e toggles 'mini.files' only once after which it seems to be overridden to toggle Neo-tree.


Potential missing rule: early pick-up of pile

Loving your work so far! It's a really smooth experience and great fun to play around with.
I'm not sure if this is an official rule, but I've always played with the ability to pick up the pile at will, rather than having to play through until you can't play any more cards.

Hi, first of all thanks for the kind words!
The funny thing is, my friend and I recently had a discussion about this exact issue. He pointed the same thing out you did. The times I played with him I always used to play a card whenever I could. I just found out that he didn't and often picked up the pile instead.
Anyways, your comment only further confirms that this should be allowed 😁


reload detail pane when version select changes

Fixes the version select issue mentioned in https://github.com/jenkinsci/job-dsl-plugin/pull/479#issuecomment-139211146

Thanks!


Add IP-based access control

Update graphql_handler to include client IP in requests.
Modify signIn to validate IP against allowAccessFrom.
Update TestSchema to accept test_addr for simulating client IP.

Close: #39

@sophie-cluml Since it is safe to validate the token only under HTTPS, we don't need to validate other things like the IP address after the sign-in process.

@sehkone 네 확인 감사합니다.


Compliment: Kudos!

I am truly surprised there aren't any comments here or on your blog for the amazing work done on this project.  Very well done!  Thank you for taking the time to put all this information together.  It got me to install Visual Studio Code, for one. 😆   Also consider adding Disqus to your blog so people can share their appreciation there as well.
In any case, thanks again. 👍

Closing now as this isn't an issue but more of praise. 👍


Do not allow double RLock() of RWMutex

Do not allow double RLock() of RWMutex. It may lead to deadlock if there is Lock() call from another goroutine in between.
Fix #2208.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

LGTM
Ping @vieux @abronan let's merge this ASAP

Also, we could have similar problems in other parts of the code, I'm looking at tools to auto-detect double locking like this.

I asked a question, but besides this: LGTM


Load style.css after main.css

This is for users who are still afraid to touch a .scss file. So, they can do overrides in style.css without the !important rule.

I've also added a small commit myself, to clean up the $modificated_bootscoreCss if function a bit. You can quickly check that out before the merge.


[Accessibility]A11y_VSCode_ViewSearch_ScreenReader: Screenreader does not read the necessary headings on the screen.

Environment Details:
VSCode Version : 1.24.0
OS Version : Win10
Additional Details:
MAS Violated : MAS 1.3.1
ScreenReader Versions : NVDA2018.1.1 / JAWS18.0.5038
Repro Steps:

Launch VS Code.
Navigate to Activity Bar and select "Search"(Cntrl+Shift+F) button.
Navigate to complementary landmark -"Search" section and read the content "Search" using basic navigation.

Actual:
Screenreader does not read the content "Search" as appopriate heading which is so.
Same issue has been observed with JAWS.
Expected:
Screenreader should read- "Search" as heading with appropriate heading level.
Recommendations:
Add appropriate native heading level tag to the content "Search".
or,
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
The screenreader users will not be able to know the appropriate headings present on the screen.
MAS Reference:
MAS 1.3.1 - https://microsoft.sharepoint.com/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={54f28d1f-a2d1-4dcd-84e1-5c9b87e8aba4}
Attachment For Reference:
A11y_VSCode_ViewSearch_ScreenReader_Headings.pptx
Does this issue occur when all extensions are disabled?: Yes

Navigate to complementary landmark -"Search" section and read the content "Search"

Sorry, I don't know what this means. When I press ctrl+shift+f, it reads "Search". What is the complementary landmark?

Basically the point here is that the headings like "Search", "Explorer" etc. are not identified as headings. They should have proper heading levels. The side bar content - "Search", "Explorer" etc. can be accessed using landmarks (D command in NVDA for navigating to landmarks) and then try to read headings inside that using (H command in NVDA for headings). There are no heading levels found for above mentioned headings.

Hope this explains the needed.

Dup of https://github.com/Microsoft/vscode/issues/51760


Research extensibility

We tried extending TabBarBase to activate tab by its' id as opposed to index. To do this one needs access to _getTabs method, but it's private and call getIndexOfTabById, which is buried in mdcFoundation.adapter_ and is not accessible either

Most of these components are hard to extend. From personal experience, I've gone through so much trouble customizing:
mwc-button
mwc-textfield
mwc-fab
mwc-drawer
mwc-radio
mwc-switch
and probably a couple more I forgot. Most of the times, I simply make the changes locally and simply make sure I remember to use the same machine to build the project.
Here's an example of a simple change I made to the css file for mwc-drawer:
.mdc-drawer-app-content {
  /* Default styles */
  margin-left: 0;
  margin-right: 0;
  position: relative;

  /* Added style */
  width: 100%;
}

On more important client projects, I have unfortunately been forced to either accept the component for what it is, or choose an alternative.

I agree, I don't remember having issues to customize mwc elements on the fly but have been using mwc-tab-bar in few projects now and hell I always spend so much time trying to harmonize its logic with my application. Maybe providing some user-friendly accessibility functions is a good idea ?

For a specific example, it would be nice to be able to use actionItems other than mwc-icon-buttons.

We're currently exploring theming in general in #526.
Are you extending just to modify styling, or do you want to modify behavior as well?
If you just need styling changes, it seems likely we just need to set all private API to protected instead.

Yes, protected API and cleaner way to access adapter methods would be nice

Can you give more details of what you're trying to do? It will help us understand what kind of customization users need. For example, why do you need to call _getTabs?

<my-tab-bar active-tab-id="${this._tab}" @MDCTabBar:activated="${this._tabActivated}">
    <mwc-tab id="tab1">Tab 1</mwc-tab>
    ${this._someCondition ? html`
        <mwc-tab id="tab2">Tab 2</mwc-tab>
    ` : ''}
</my-tab-bar>
class MyTabBar extends TabBarBase {
    @property({ type: String, attribute: 'active-tab-id' })
    _activeTabId;

    firstUpdated() {
        if (this._activeTabId)
            this.setActiveIndex();
    }

    setActiveIndex() {
        //let tabIndex = this.mdcFoundation['adapter_'].getIndexOfTabById(this._tab);
        let tabIndex = this.getIndexOfTabById(this._activeTabId);
        if (tabIndex >= 0 && tabIndex != this.activeIndex) {
            this.activeIndex = tabIndex;
            if (this.mdcFoundation)
                this.mdcFoundation.activateTab(this.activeIndex);
        }
    }

    getIndexOfTabById(id: string) {
        const tabElements = this['_getTabs']();
        for (let i = 0; i < tabElements.length; i++) {
            if (tabElements[i].id === id) {
                return i;
            }
        }
        return -1;
    }
}
customElements.define('my-tab-bar', MyTabBar);

While it would be nice to change the functionality by extending the base class, I think most use cases could be covered by adding properties that customize the behavior of the component. For example, I remember paper-tabs having a property that allowed users to specify what to use as the selected property, instead of the index.
As for styling, most of my use cases have been covered by using the custom properties already available, and when they aren't enough, it has been as easy as adding the relevant custom property into the css file.

Per https://github.com/material-components/material-components-web-components/issues/903#issuecomment-640785717, we should also consider the impact of html in labels.

Obsolete with M3


Add pull request and test info to CONTRIBUTING.md.

Inspired by dart-lang/test#535

@anders-sandholm ping


Update yarn.lock

Some packages can be updated.
However, there are many which cannot, but these will be removed with the stack update.

Tested in local build.


use seperate flag for activating task creation build ops

Context

avoids always printing stats when used for activating task creation capturing in build scan plugin

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@breskeby Are you also using this new flag in our performance tests?


Fix duplication rating on quality gate report

Fix duplication rating on quality gate report

Pull Request Test Coverage Report for Build 335533157

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

Totals

Change from base Build 333490513:
0.7%

Covered Lines:
450

Relevant Lines:
739

💛  - Coveralls


support id derefrencing of nodes. so asString(m.list.feed.games.json.title) will look up child with id "games" in m.list.feed

given this object graph
list:NOde :{
feed: Node {
_children: [
0: ItemNode{
id: "header"
json: { title: "a" }
}
1: ItemNode{
id: "games"
json: { title: "b"}
}
}
asString(m.list.feed[games].json.title)  will return "b" which is equivalent to
asString(m.list.feed[1].json.title)
this needs to be modified in getPath function in Utils.bs.. particularly this bit.. it will do part.find(key), if key is NOT an integer, AND key is a string, and key is not ""

note, this will require [] bracket support at the plugin level, which we do not presently have.

this is a bad idea..


Gemini pro Video support [BUG]

I am trying to use Gemini 1.5 pro and upload a video file  as part of the request but I can't find any input parameter for doing so

am I missing something ? any way to acoomplish that ?

also , similair subject - is there any "loaders" for Video files  as there are for text files (PDF/DOCS etc..)  I couldn't find any - , basically - I would like to create embeddings from the a collection of video files

videos are not supported yet, only images for now

@HenryHengZJ  I have been trying Flowise and results sounds promising. Any update on video support?


Collision shapes are drawn in wrong color and collision contact points are not drawn at all

Ubuntu - Godot version: 3.0 master
Issue description:
Collision display debug options, draw collision shape in wrong color (gray instead one selected in Project Settings) and don't draw contact points at all.

On master f527a6946 , 2D shapes (if correspond to this issue):
Shapes on KinematicBody2D do not draw points at all, on RigidBody2D is ok.
CircleShape2D and CapsuleShape2D do not show even on editor.

On 3D shapes I see black on RigidBody shapes, white/gray on the rest, also lines seem to cast and receive shadows.

is this better now?

Still have the problem

@reduz
Still all are light grey and no collision contact dot. :<

On 2D are fine (the visuals, not the polygon edition) and 3D are still affected by light (black/light color), I can see the collision point but is affected by light too.


What is the Initial Login & Password for the Web Administrator application?

Hi, there!
Thank you for using SoftEther.
Before you submit an issue, please read the following:
Is this a question?
Yes, but I can't log into the users site either?
All I want to know is WHAT is the the initial Login ID and Password required to login to the new SoftEther Web Administrator console.
I've got everything installed okay and can connect to the Web  Console but have no idea what the initial Login ID  and Password are as they are not documented on the SoftEther website as far as I can find although I "think" the initial Login ID is "administrator" I do NOT know what the initial Password is???
Thanks for any help...
Brian

+1

Found this:
user: administrator
pass: (your server manager pass)


cmd/go: module update should drop unused indirect dependencies?

What version of Go are you using (go version)?
Latest go devel: go version devel +d278f09333 Thu Jul 19 05:40:37 2018 +0000 darwin/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/ikorolev/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/ikorolev/.gvm/pkgsets/go1.11beta1/global:/Users/ikorolev/dev/go"
GOPROXY=""
GORACE=""
GOROOT="/Users/ikorolev/.gvm/gos/go1.11beta1"
GOTMPDIR=""
GOTOOLDIR="/Users/ikorolev/.gvm/gos/go1.11beta1/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/_b/d1934m9s587_8t_6ngv3hnc00000gp/T/go-build301560759=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Sorry, can't give you a standalone reproduction - bug is produced only with downloading updated dependencies.
Consider the following flow:
A(v0.0.1) uses B(v0.0.1):
module github.com/mwf/vgo-modules/a

require github.com/mwf/vgo-modules/b v0.0.1

Then A(v0.1.0) drops the dependency.
Here we got a project main which uses A(v0.0.1) and we want to upgrade A to v0.1.0
cd `mktemp -d`
git clone https://github.com/mwf/vgo-modules .
echo "\n$(cat go.mod)"
echo "\n$(cat go.sum)"
go get<EMAIL_ADDRESS>echo "\n$(cat go.mod)"

We get an unexpected indirect dependency github.com/mwf/vgo-modules/b v0.0.1 // indirect in go.mod, it seems to be taken from go.sum as a side-effect.
What did you expect to see?
go.mod should contain updated a dependency
module github.com/mwf/vgo-modules

require github.com/mwf/vgo-modules/a v0.1.0

What did you see instead?
We get an unexpected indirect dependency from b in go.mod, it seems to be taken from go.sum as a side-effect.
module github.com/mwf/vgo-modules

require (
    github.com/mwf/vgo-modules/a v0.1.0
    github.com/mwf/vgo-modules/b v0.0.1 // indirect
)

Currently there is a workaround to run go mod -sync after go get, so maybe the issue is not so critical. But still looks buggy.

This is working as intended. (It's a bit confusing, but arguably less confusing than the alternatives.)
In general, go get tries to make the smallest possible change to the build list in order to get the requested modules at the requested versions.
When you started out, you had a direct requirement on a and an indirect one (via a) on b. Upgrading a did not require any changes to b in the build list, so go get did not make them, but that means that the previously-implicit dependency on b must be made explicit in the build file.
In this specific case, we theoretically have enough information to prune out the dependency. However, in the general case that can get expensive: if the main module depends on some other package c that imports b, and c does not have a go.mod of its own, then we don't actually know whether we still need b unless we process all of the (transitive) imports for c. But you didn't tell us to change anything about c and we don't need to touch it otherwise, so if go get started scanning its imports we'd be doing a lot of extra work that you didn't ask for.
$ cd $(mktemp -d)
$ git clone https://github.com/mwf/vgo-modules .

$ go list -m all
github.com/mwf/vgo-modules
github.com/mwf/vgo-modules/a v0.0.1
github.com/mwf/vgo-modules/b v0.0.1

$ go mod -graph
github.com/mwf/vgo-modules<EMAIL_ADDRESS>github.com/mwf/vgo-modules<EMAIL_ADDRESS># bug in mod -graph: #26489
<EMAIL_ADDRESS><EMAIL_ADDRESS>
$ go get<EMAIL_ADDRESS>
$ go list -m all
github.com/mwf/vgo-modules
github.com/mwf/vgo-modules/a v0.1.0
github.com/mwf/vgo-modules/b v0.0.1

$ go mod -graph
github.com/mwf/vgo-modules<EMAIL_ADDRESS>github.com/mwf/vgo-modules<EMAIL_ADDRESS>

I'm going to close this for now, but we'll keep an eye out for similar issues. Thanks for the great repro steps!

Thanks for a great explanation, Brian!
The logic does make sense now when dealing with dependencies without a go.mod file.
Actually the common usage will be
go get module1
go get module2
...
go mod -sync

Which leads to the minimal case of upgrading a single dependency with go get module1 && go mod-sync.
It's really unobvious and a bit confusing why do you need to run 2 commands instead of a single one, but for a bunch of consequent updates it could be really faster than if go get would always traverse the full tree.
I checked go help module-get - this nuance is not covered even in a minimal way (that one may want to run go mod -sync after get).
Let's reopen the issue to update the docs?
And @bcmills, could you please take a look at https://github.com/golang/go/issues/26048#issuecomment-405635622 about go install behaviour? Maybe I'm missing there some similar nuances, but I can't see them yet :)

Let's reopen the issue to update the docs?

Done.

I don't think this is working as intended, fwiw. Maybe I'm wrong. Will return to this.

I think this is probably incorrect behavior but it's not a showstopper for Go 1.11 because 'go mod tidy' will correct it. Don't want to make changes now but we should investigate for Go 1.12.

This still reproduces as of Go 1.14 RC1.
(It is somewhat related to #26902, which was fixed in 1.13, but the behavior here is “not removing transitive dependencies that were never actually relevant” rather than “upgrading dependencies that are not relevant”.)
$ go version
go version devel +363bcd00 Wed Feb 12 18:22:50 2020 +0000 linux/amd64

$ git clone https://github.com/mwf/vgo-modules
…

$ cd vgo-modules/

vgo-modules$ cat go.mod
module github.com/mwf/vgo-modules

require github.com/mwf/vgo-modules/a v0.0.1

vgo-modules$ cat go.sum
github.com/mwf/vgo-modules/a v0.0.1 h1:BaK3DzgdA4LxVXd42qID7F9G+KFxQ4SL69hm+BNcjA0=
github.com/mwf/vgo-modules/a v0.0.1/go.mod h1:nK31MtN2feKpYgPBywpeTy/Om+1x3OxQHQAsyi95AQ0=
github.com/mwf/vgo-modules/b v0.0.1 h1:aEPPx6pEuQ1/Wu9SzeLkiYltlir2h1fDJmKRv1ckarY=
github.com/mwf/vgo-modules/b v0.0.1/go.mod h1:7kI8RJ5+IPwKWU1MrE/kTA9AdWBNzMmucdpAiWXATOs=

vgo-modules$ go get<EMAIL_ADDRESS>go: downloading github.com/mwf/vgo-modules/a v0.1.0

vgo-modules$ cat go.mod
module github.com/mwf/vgo-modules

go 1.14

require (
        github.com/mwf/vgo-modules/a v0.1.0
        github.com/mwf/vgo-modules/b v0.0.1 // indirect
)

vgo-modules$ cat go.sum
github.com/mwf/vgo-modules/a v0.0.1 h1:BaK3DzgdA4LxVXd42qID7F9G+KFxQ4SL69hm+BNcjA0=
github.com/mwf/vgo-modules/a v0.0.1/go.mod h1:nK31MtN2feKpYgPBywpeTy/Om+1x3OxQHQAsyi95AQ0=
github.com/mwf/vgo-modules/a v0.1.0 h1:QD+ijrXwAYk4CMTOqEAzcogJoF4zLfit2alZVmT80EM=
github.com/mwf/vgo-modules/a v0.1.0/go.mod h1:XGJvSKC62ygHgRNmDf4RqXyp0zngXIJMLc6BaSfyTfI=
github.com/mwf/vgo-modules/b v0.0.1 h1:aEPPx6pEuQ1/Wu9SzeLkiYltlir2h1fDJmKRv1ckarY=
github.com/mwf/vgo-modules/b v0.0.1/go.mod h1:7kI8RJ5+IPwKWU1MrE/kTA9AdWBNzMmucdpAiWXATOs=

vgo-modules$ go mod tidy

vgo-modules$ cat go.mod
module github.com/mwf/vgo-modules

go 1.14

require github.com/mwf/vgo-modules/a v0.1.0

vgo-modules$ cat go.sum
github.com/mwf/vgo-modules/a v0.1.0 h1:QD+ijrXwAYk4CMTOqEAzcogJoF4zLfit2alZVmT80EM=
github.com/mwf/vgo-modules/a v0.1.0/go.mod h1:XGJvSKC62ygHgRNmDf4RqXyp0zngXIJMLc6BaSfyTfI=


REX-Ray Docker Plugin for GCE Persistent Disks

This patch provides a REX-Ray Docker plugin for Google GCE Persistent Disks

Codecov Report

Merging #750 into release/0.8.0 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release/0.8.0    #750   +/-   ##
=============================================
  Coverage           7.43%   7.43%
=============================================
  Files                 21      21
  Lines               2581    2581
=============================================
  Hits                 192     192
  Misses              2372    2372
  Partials              17      17

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 55d2106...0ba4dec. Read the comment docs.


Add game instances list api endpoint

Checklist

[ ] Use a separate branch in your local repo (not master).
[ ] Test coverage is 100% (or you have a story for why it's ok).

Coverage decreased (-0.4%) to 99.584% when pulling 0f8bd1fce799c39628c824e7c0a0a69fd7ea57a7 on francoijs:add-game-instances-list-api-endpoint into 314f8f0ba66e6a58505ac8fd7b41c83604c47b5c on google:master.


crystal bench

Not to be merged yet, only to trigger CI.

Ah, nice! I meant to add --release because that's how I ran it locally and I completely forgot. :joy:

I ran all the benchmarks and it looks like this one is running a lot slower than I expected but is also using way less CPU than the others. Most implementations are being saturated by ghz but for some reason the Crystal implementation is using under 1/4 of that.

I've seen it happen before with the Crystal gRPC library but it wasn't consistently reproducible. So thanks to your work, at least I can reproduce it reliably! :100:
I don't think you'll have to change anything when I fix this since this repo ignores shard.lock so any changes I make to the library should just work here. I at least wanted to let you know about it, though, because I don't want people to think Crystal is slow — the request rate in ratio with CPU consumption is amazing. If it got 6200 at 24% CPU, we could expect it to be ~26k if we can saturate the core, which outperforms everything except C++ and Rust according to the results I got for my run.

Great, good to know these benchmarks prove useful! Not being as performant as expected is the sole reason this repository was created. We've heard it one too many times. ;)
Keep in mind that your implementation may need to be asynchronous (if it's not already the case) to reach (surpass?) e.g. Rust Tonic performance. Please let me know when you manage to do some performance boosts! :)

@jgaskins I'd advise you also to run the benchmarks over longer period, e.g. 5 minutes. We did some tests an a desktop Linux machine and the results were not as good as we expected (for Rust and Crystal).
Results (successful requests per second)
cpp_grpc       : 29644
rust_tonic     : 23406
lua_grpc       : 19156
go_grpc        : 15472
java_grpc      : 13576
java_micronaut : 12949
node_grpc      : 12537
kotlin_grpc    : 8935
dart_grpc      : 8490
swift_grpc     : 4742
php_grpc       : 2721
ruby_grpc      : 1990
scala_akka     : 1887
crystal_grpc   : 1421
python_grpc    : 1255

Crystal details:
Summary:
  Count:    426484
  Total:    300.00 s
  Slowest:    52.04 ms
  Fastest:    0.36 ms
  Average:    35.06 ms
  Requests/sec:    1421.61

Response time histogram:
  0.362 [1]    |
  5.531 [89043]    |∎∎∎∎∎∎∎∎∎∎∎
  10.699 [1367]    |
  15.867 [23]    |
  21.035 [0]    |
  26.204 [0]    |
  31.372 [0]    |
  36.540 [0]    |
  41.708 [1196]    |
  46.876 [327045]    |∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎∎
  52.045 [7759]    |∎

Latency distribution:
  10 % in 1.76 ms
  25 % in 42.79 ms
  50 % in 43.76 ms
  75 % in 43.93 ms
  90 % in 44.17 ms
  95 % in 45.63 ms
  99 % in 47.15 ms

Status code distribution:
  [OK]            426434 responses
  [Unavailable]   50 responses

Error distribution:
  [50]   rpc error: code = Unavailable desc = transport is closing

FYI The test above has been run on a Intel(R) Xeon(R) CPU E5-1650 v3 @ 3.50GHz


findcountours() application has stopped working .

Hi,im using opencv 3.0 and i am using Visual Studio 2013 .Every time i call findcontours() my application crashes.I need some help if someone know what is happening .
Update: Other functions work fine but findcontours crash my aplication.

@zadannu, please provide minimal code example reproducing the crash. Try to build latest master branch OpenCV version ant test your application with it.

I suggest moving this to http://answers.opencv.org/questions. The community will help you debug the problem, since the actual functions are working perfectly fine here.


LBL on CARMENES data

Hello,
I tried to compute the LBL algorithm on various spectra from the CARMENES data release  but they seem to mostly be failing

Below is the latest wrap I used to launch the LBL on GL251 data. I also attached the log of the LBL compute function, everything appears to be working fine on the surface and the LBL converges in less than 3 or 4 iterations.
LOG-2024-11-12-lbl_compute.log
Thank you for your help!
Merwan
`rparams = dict()
rparams['INSTRUMENT'] = 'CARMENES'
rparams['DATA_SOURCE'] = 'None'
rparams['DATA_DIR'] = cwd + '/lbl/'
rparams['DATA_TYPES'] = ['SCIENCE']
rparams['OBJECT_SCIENCE'] = [target]
rparams['OBJECT_TEMPLATE'] = [target]
rparams['OBJECT_TEFF'] = [3420]
rparams['BLAZE_FILE'] = 'carmenes_dummy_blaze.fits'
rparams['RUN_LBL_TELLUCLEAN'] = True
rparams['RUN_LBL_TEMPLATE'] = True
rparams['RUN_LBL_MASK'] = True
rparams['RUN_LBL_COMPUTE'] = True
rparams['RUN_LBL_COMPILE'] = True
rparams['SKIP_LBL_TELLUCLEAN'] = False
rparams['SKIP_LBL_TEMPLATE'] = False
rparams['SKIP_LBL_MASK'] = False
rparams['SKIP_LBL_COMPUTE'] = True
rparams['SKIP_LBL_COMPILE'] = True
lbl_wrap(rparams)`

That log comes from v0.59 (over a year old)
__version__: str = '0.59.0'
__date__: str = '2023-07-04'

Can you try using v0.64 (git pull on the main branch), no one is going to try to debug a version that old!

Hello, thank you for your response.
Thanks for pointing out that my data was outdated. I’ve redone the computations with the latest version, but sadly, the issue seems to remain: the RVs are more scattered and less precise with LBL compared to the CARMENES reduction.

Below is the latest log of LBL_COMPUTE applied on GL251
LOG-2024-11-12-lbl_compute.log
Thank you for your help,
Merwan

Hello, thank you for your response.
Thanks for pointing out that my version was outdated. I’ve redone the computations with the latest version, but sadly, the issue remains: the RVs are more scattered and less precise with LBL compared to the CARMENES reduction.

Below is the latest log of LBL_COMPUTE applied on GL251
LOG-2024-11-16-lbl_compute.log
Thank you for your help,
Merwan

Hi,
I ran the newest version of LBL and it keeps giving spurious results when applied on CARMENES data even though the code itself runs properly

Thank you for your help,
Merwan


Bring the deps to contemporary versions, plus s/poetry/rye

This is a prerequisite of updating bdantic, so I'm starting here :D

FWIW, the package builds and tests pass. I'm making a bold assumption that no other functionality I wasn't aware off got broken :D

Gentlest of pings? :)


Schema file for Policy Assignments

Feature will enhance VS Code experience

@anwather I assigned this to myself and completed it (first version)


Fix crash when using ImGui across DLL boundaries on Windows

Description
You can read the documentation I included in the PR changes to understand what I'm fixing here.
If you're running Windows, previously with CUBOS_ENGINE_SHARED set to ON on CMake you probably got a crash on both Tesseratos and on the ImGui sample. With this change, that should no longer happen, so please test that before approving.
Checklist

[x] Self-review changes.
[x] Updated imgui plugin documentation.
[x] Update sample.
[x] Add entry to the changelog's unreleased section.

If @SrGesus and @jdbaracho are Windows users then I don't mind looking at the code and review it, else I think we should assign Windows users as reviewers no? I have no machine to test this on.
@RiscadoA thoughts?

Is this about CUBOS_ENGINE_SHARED or BUILD_ENGINE_SHARED @RiscadoA ?

The first one, the second one no longer exists, it was the old name of the same option which was renamed in a previous PR


Update version: MedZed.youtubetomp3 version 1.0.0

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115141&drop=dogfoodAlpha

@wingetbot run

/azp run


vulcan module

this project  is for google codein

Can one of the admins please verify this patch?

Please don't just recreate your module from last year. Also, modules are now created in their own repos. (see https://github.com/MovingBlocks/DestinationSol/wiki/Initializing-a-Module and relevant)


[Hexagon] Skip test if "onnx" module not available

Don't fail the test for reasons that are unrelated to the feature being tested.

@tvm-bot rerun

@kparzysz-quic this could silently skip the test which happened before in the CI. Could you please explain why you need this? If you are running this in a different CI and you don't want this test to run, I think you can filter it in the pytest


Constraints on model saving should be clearly documented

See https://github.com/geneontology/minerva/blob/d8f7fc14a0de376103c96404b8ead328540ae2f7/minerva-server/src/main/java/org/geneontology/minerva/server/validation/BeforeSaveModelValidator.java
The logic is hidden in java. Ideally this would be separated into easily configured rules, e.g. SPARQL or Shex, see #40

This would also be great for having things like per-site, or even per-user/org, rules.


net: conn_mgr: documentation and implementation guidelines

Not yet proofread, not yet ready for review

There are still a couple minor crosslink errors, and I need to update codeowners / maintainers files to include the new docs, but otherwise these docs should be ready for review now

Please check the compliance issues which are currently shown at
https://github.com/zephyrproject-rtos/zephyr/actions/runs/6103554259/job/16564154813?pr=62126

All comments by @peknis addressed

I think someone will want to update the API overview page to point to conn_mgr_docs target? Thanks! (and sorry that I always seem to be 15 minutes behind, as of late)

@kartben I've opened a PR with the link update: https://github.com/zephyrproject-rtos/zephyr/pull/63233

Sorry about that @kartben I completely spaced out on updating that. Thanks for pointing it out, and thanks @rlubos for putting a PR in to do it

Sorry about that @kartben I completely spaced out on updating that. Thanks for pointing it out, and thanks @rlubos for putting a PR in to do it

No worries at all!


Split JS blob into smaller parts and load only what's needed

Handled a while ago actually. Cannot recall the PR.


consider different locales

according to https://pineco.de/using-laravels-localization-js/ we get the default locale and cache it, but what happens when u change the locale ?
the lib can take care of this by switching the translation according to the current app locale which can be acquired by using props on the vue comp init and send it as an option to the class constructor.
then we check if the locale is found in the translations keys locales and either fallback to the default or display empty string, which could be another option sent to the class constructor

@ctf0 Thank you for the idea!
Since we need the current language's files, we can solve it only at the back-end side.
You are right, we need to change the default config to the current locale.
So that means we need to change this:
config('app.locale')
to this
app()->getLocale()
That means we don't need to pass any locale argument to the translator constructor.
If I need the current locale anywhere in my JS, I often do this:
<html lang="{{ app()->getLocale() }}">
and then
let lang = document.documentElement.lang
Also, that might would be a good solution, instead of using cache here, we could use view composers. Then we could react immediately to the local changes.
What do you think?

i was thinking more of fetching all the locales and save it to the localeStorage like
'required': {
    en: 'english text',
    fr: 'french text'
}

this way you only prepare the trans once, instead of going through the whole thing each time the user change the locale.
also whether using config('app.locale') or app()->getLocale() its the same as getLocale() basically return config('app.locale')
still, if you prefer to keep the lib as light as possible, we can maybe do all on the backend and display the result as a blade directive like https://github.com/tightenco/ziggy , this way you dont need to use cache as blade take care of this & it will auto-invalidate when new trans gets added.

@ctf0 Hmm, this getLocale() thing is new for me! Thank you!
I think I would keep the JS part like this, this is maintainable for us this way.
But I love the idea about the blade directive!
I think I extract the back-end part to a package with the blade directive part.
I link the package here soon!
Thank you for your ideas! :)

So we created a package, you can find the docs in the readme!
I close this issue, if you think, be brave to open a new issue!
Thank you for the idea!

so i believe it would make more sense if importing the i18n already comes bundled with the translations.
as atm we need to do 3 steps

add window.translations = @translations; to view
import I18n
init the lib let translator = new I18n(window.translations);


Missing Cursor Movements and Whiteboard Actions in MP4 Output

In the output MP4, I have noticed that cursor movements and whiteboard actions are not included, even though these elements are expected to be present according to the documentation.
Could you please investigate or provide guidance on resolving this issue?
You can use the links from Issue #55 to reproduce the problem.
LOG.txt
Environment:
OS: Windows 10 22H2 19045.5011
Python version: 3.12.7
bbb-dl version: 1.0.13
Thank you!

you are funny. If you use the option -sa and -sc it obviously skips the cursor and annotations.

Also if you use the option -kt changeing the options -sa or -sc will have no effect as explained in the readme. Since the already created frames will be reused.

If you want, you can send me the bbb session via mail.

Just for the record. K4KNPR provided me a link to his bbb session. The bbb server seems to be on a new beta version. The cursors are now stored with absolute coordinates instead of relative. Also the shapes are now stored into separate svgs per slide, instead of all slides in one svg.


merge traits

This is a place holder to find a way to best merge a set of traits into one
Possible examples
export const traits = createEntityFeatureFactory(
  {
     products: [
       addLoadEntities<Product>(),
      addSingleSelection<Product>(),
     addSort<Product>()
  ],
   orders: [
       addLoadEntities<Order>(),
      addSingleSelection<Order>(),
     addSort<Order>()
  ]
  }
);

Another possible way
export const traits = createEntityFeatureFactory(
....
   addProperty('products', addLoadEntities<Product>(),
      addSingleSelection<Product>(),
     addSort<Product>()),
addProperty('orders',  addLoadEntities<Order>(),
      addSingleSelection<Order>(),
     addSort<Order>())

First one I think is easier to implement , second one is harder but more flexible so might be more useful, there are state shapes that the second can do but not the first

this was implement in a different way with the combineEntityFeatureFactory, mixEntityFeatureFactory and addEntityFeaturesProperties


Update/Add linux-modules-extra-azure

Tool name
linux-modules-extra-azure
Tool license
Same license as linux-image-azure
Add or update?

[X] Add
[ ] Update

Desired version
Same version as linux-image-azure
Approximate size
125.93 MB
Brief description of tool
This package contains extra kernel modules for the linux-image-azure kernel package that the runner images are using. We're looking to mount erofs filesystems for systemd integration tests which need the erofs kernel module which is available from linux-modules-extra-azure.
We need this to be installed in the regular image because the version of the modules needs to match the version of the running kernel, otherwise the modules can't be loaded by the running kernel. We can't install the version that matches the running kernel because newer versions get released and older versions of the package are removed from the Ubuntu repositories.
Another option is to make sure that a version of the linux-modules-extra-azure package that matches the kernel package that is shipped in the current image version is always available.
Currently, if I do sudo apt-get install linux-modules-extra-$(uname -r), the command fails because the matching version is not available in the mirrors. This requires the GA team to make sure the version is available in the azure apt mirrors until the runner image with the same kernel version is no longer in use.
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[ ] Azure DevOps
[X] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
sudo apt-get install linux-modules-extra-azure

Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hello @DaanDeMeyer . We will take a look.

Hello! We would not like to install the package due to maintenance concerns as well as lack of high demand from other customers. If you have questions left feel free to reach us again.

This is not a problem that we can solve in our GA actions in any way. The images simply get out of sync with the repositories. Please reconsider.

The kernel in the repository is updated, so the old modules for the kernel that is actually running in the images you make available no longer exist and cannot be installed. We are unable to run CI jobs and add coverage for security-related features due to this issue.

Here are some clarifications:
These kernel modules cannot lag behind the actual kernel update when you install both together, as they are all built from the same source package. Canonical choose to split it out in multiple binary packages, for some reasons. When you install the kernel image package and that extra-module package at the same time, they are guaranteed to be compatible, as the unversioned metapackages that you are referencing point to the exact same kernel build. This is guaranteed by how the packages are set up.
The problem arises because the kernel package is installed whenever the image is built, and we are trying to install the extra modules package long after that has happened, on the user script. By installing both at image creation time, they will be in sync.
You can see that linux-image-azure and linux-modules-extra-azure are part of the same source package and thus are built and published together here:
https://packages.ubuntu.com/source/jammy-updates/linux-meta-azure
And if for some weird, strange circumstance that does go out of sync at image creation time, nothing bad happens, there's simply some unusable kernel modules. The installation location is named after its ABI, so it won't affect the actual kernel.
To be extra clear: these are not out-of-tree or 3rd party kernel modules. They have been part of the mainline kernel since ages, and are all built together with the kernel image itself. It's just that Canonical choose to organize them in separate packages for some reason, built at the same time. They cannot go out of sync or be incompatible.

reopened at https://github.com/actions/runner-images/issues/7587


Fuzz server

https://speakerdeck.com/filosottile/automated-testing-with-go-fuzz

https://github.com/dvyukov/go-fuzz/blob/master/examples/httpreq/httpreq.go
https://github.com/dvyukov/go-fuzz/blob/master/examples/httpresp/httpresp.go
https://github.com/dvyukov/go-fuzz/blob/master/examples/httpserver/main.go


Refactor code

Refactor code and functions

Hi @pprathameshmore , I would like to work on this.

@ddazal You really did good job. Thank you for your contribution.

Thank you for let me work on your app :)


Flutter ExpansionPanelList Widget now working both in the flutter example and in my app where I've already implemented it and was working fine few months ago.

https://github.com/flutter/flutter/blob/ead455963c12b453cdb2358cad34969c76daf180/packages/flutter/lib/src/material/expansion_panel.dart#L160
This code is not working in the current flutter version. I implemented this several month on my app and it's working fine but upon revision of my code and upgrade to the new flutter version, it's not working both the example code and the one in my app which was working perfectly before is no more expand as it should. Please can some help me out here??

I just got it fixed by changing the bool expression in the expansioncallback like this
  expansionCallback: (int index, bool isExpanded) {
    setState(() {
      _data[index].isExpanded = isExpanded;
    });
  },

instead of this, which was what is on the flutter sample and was working the last time I worked on it:
expansionCallback: (int index, bool isExpanded) {
setState(() {
_data[index].isExpanded = !isExpanded;
});
},

Hi @SAnetwork2020. Thanks for filing this. As for the issue with the example in the doc, it has been fixed by per https://github.com/flutter/flutter/commit/af79b4c7a3a425814eabc8df32708b1426643f5c. You can check out https://master-api.flutter.dev/flutter/material/ExpansionPanelList-class.html for the updated example.
Thanks!


problem.derivative() example fails

The source code at https://github.com/cvxgrp/cvxpy/blob/master/cvxpy/problems/problem.py has this example:
import cvxpy as cp
import numpy as np
p = cp.Parameter()
x = cp.Variable()
quadratic = cp.square(x - 2 * p)
problem = cp.Problem(cp.Minimize(quadratic), [x >= 0])
p.value = 3.0
problem.solve(requires_grad=True, eps=1e-10)
# derivative() populates the delta attribute of the variables
problem.derivative()
p.delta = 1e-3
# Because x* = 2 * p, dx*/dp = 2, so (dx*/dp)(p.delta) == 2e-3
np.testing.assert_allclose(x.delta, 2e-3)

Is the last step meant to succeed?  I get this output:

Traceback (most recent call last):
File "try_cvxpy_init_00.py", line 13, in
np.testing.assert_allclose(x.delta, 2e-3)
File "/usr/local/lib/python3.6/dist-packages/numpy/testing/_private/utils.py", line 1533, in assert_allclose
verbose=verbose, header=header, equal_nan=equal_nan)
File "/usr/local/lib/python3.6/dist-packages/numpy/testing/_private/utils.py", line 846, in assert_array_compare
raise AssertionError(msg)
AssertionError:
Not equal to tolerance rtol=1e-07, atol=0
Mismatched elements: 1 / 1 (100%)
Max absolute difference: 0.002
Max relative difference: 1.
x: array(-0.)
y: array(0.002)

Thanks for reporting! I believe the issue is that p.delta should be set before problem.derivative() is called, not after. We’ll update the docstring.

Yes, that does fix it.


feat: improve typing on core commands

Some of the core commands are using Dict[...] as an argument for settings data, Dict is invariant and mutable.
Invariance causes issues when passing other types, for example hset requires Dict[str | bytes, str | bytes | float| None], but if you pass Dict[str, float] mypy will complain although Dict[str, float] is a "subset" of the required type.
Mutability is not required for these functions as they don't change the underlying data and so it should be reflected in the typing.

Nice! thank you! This is in fact what the README says the api should have looked like!

@pbabics could you run make templated-sources (You'll have to add the development dependencies by running pip install -r requirements/dev.txt first). This will update the stubs for the pipeline classes in coredis/pipeline.pyi to be consistent with the signature changes in the core commands.

Actually scratch that, I'll do it post merge!