embed rclone ?

Hi,
I d like to know how you consider a third party should proceed to embed rclone within another GO project.
Should i use the internal API, or am i better suited to parse the cli output ?
Currently i m working on a small gui, using rclone under the hood, its written in go for the server part.
I diged the source a bit, but could not find a quick and easy way to re use rclone as an API.
On the other hand i found i could simply parse the output (with -vv) to get a whole detailed set of information about the progress.
Thus i made a cli output parser to transform the raw text into JSON objects,
and yes that works, but it will give some little trouble that i d like to get ride,

i will have to pack rclone exe file into my app (working, just little weird imho)
i will 1+[all transfers] running process on the target system (some waste in here)

So yeah, just like to know if you any remarks or recommendations.
Thanks!

I haven't designed the rclone API to be re-used, except from within rclone.  It is possible though - I often write little scripts embedding rclone.
What I'd like to do is fix #633 - you can see some code there.
That would enable you to call rclone from within your binary.
With a bit of thought, it would be possible to make an API so you could do what you want...

Thanks! I ll give it a check.


Custom theme types inside a theme not working!

Godot version
4.1.beta2
System information
Ubuntu
Issue description
defining a custom type and it's base inside theme is not working.
Steps to reproduce
create a custom type ie TestButton which inherits from Button, then add a theme for the root node and define the custom type inside the theme and it's base, then add a stylebox for normal and see nothing happens!
Minimal reproduction project
No need

I totally forgot about theme_type_variation which is necessary for custom theme types to work, so am going to close it for now because it works fine, but also it needs to work automatically if i have a class_name TestButton and a custom theme type defined with same name TestButton


May need an update to the dotnet-migrate-package pipeline

Noticed this error in a recent run of the dotnet-migrate-package pipeline.
"##[warning]Task 'NuGet authenticate' version 0 (NuGetAuthenticate@0) is deprecated. This task will be removed. From January 31, 2024, onwards it may no longer be available. Please see https://learn.microsoft.com/azure/devops/pipelines/tasks/reference/nuget-authenticate-v0 for more information about this task."

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

I opened https://github.com/dotnet/dnceng/issues/1738 which covers more repos than just this one so I will close this. Feel free to reopen this one if you feel like it's better to track these individually.


Add bossfight-specific combinations for map10 test

Add bossfight-specific combinations for map10 test

Fix included as part of #99


Change navbar GitHub link to org

closes #29796

@mdo should we backport this to v4-dev for consistency?


state that snowflake_database var does not alter where change history table is created

Minor alteration to the README to make it clear that using a two part name in the CHANGE_HISTORY_TABLE argument will result in the METADATA database being used to store the change history table, and not the value in the SNOWFLAKE_DATABASE argument.

Happy to alter the wording on this. I assumed that the snowflake_database var would set the database being used in the context of the connection, as I'm used to SMO and ADO.NET, and probably other people used to those libraries would make the same assumption.


VSCode prefers Windows Terminal's Cascadia Code install not User's

Issue Type: Bug
For some reason, VSCode prefers to use Windows Terminal's version of the Cascadia Code Font Face, not the one I have installed manually.
As Windows Terminal still ships with the font with the "curvey" (and less readable to many) characters this means that things like comments are, to me, unreadable as well.
You can see from this screenshot (that I can't paste from VSCode) that the font installed does not have the curvey characters, yet does in VSCode. This is not desired behaviour.
I also know it's preferring the one installed from Windows Terminal because if I uninstall Windows Terminal I get readable characters. I like Windows Terminal, I like VS Code, I like Cascadia Code without the curvey characters. Why is VSCode assuming I like them? I don't... Please stop forcing them on me.

VS Code version: Code 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-14T22:10:15.214Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2803)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.71GB (5.48GB free)

Process Argv
--crash-reporter-id 337d750c-de1b-4397-80d5-d7564e2707ba

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 5.4.72-microsoft-standard-WSL2

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2803)

Memory (System)
12.26GB (11.90GB free)

VM
0%

Extensions (35)

Extension
Author (truncated)
Version

vsc-material-theme
Equ
33.2.0

vsc-material-theme-icons
equ
1.2.2

vscode-azurevirtualmachines
ms-
0.4.0

vscode-language-pack-en-GB
MS-
1.54.1

remote-containers
ms-
0.187.0

remote-ssh
ms-
0.65.7

remote-ssh-edit
ms-
0.65.7

remote-wsl
ms-
0.58.2

vscode-remote-extensionpack
ms-
0.21.0

material-product-icons
PKi
1.0.4

vscode-autohotkey
sle
0.2.2

armview
ben
0.4.5

shell-format
fox
7.1.0

vscode-pull-request-github
Git
0.28.0

azure-pipelines
ms-
1.188.1

vscode-azureappservice
ms-
0.22.0

vscode-azurefunctions
ms-
1.4.1

vscode-azureresourcegroups
ms-
0.4.0

vscode-azurestorage
ms-
0.12.1

vscode-azurevirtualmachines
ms-
0.4.0

vscode-bicep
ms-
0.4.412

vscode-cosmosdb
ms-
0.17.0

vscode-docker
ms-
1.14.0

vscode-language-pack-en-GB
MS-
1.54.1

vscode-dotnet-runtime
ms-
1.2.0

python
ms-
2021.6.944021595

vscode-pylance
ms-
2021.7.4

jupyter
ms-
2021.8.1013163132

azure-account
ms-
0.9.8

azurecli
ms-
0.5.0

github-issues-prs
ms-
0.9.3

vscode-node-azure-pack
ms-
0.2.1

azurerm-vscode-tools
msa
0.15.2

bash-debug
rog
0.3.9

markdown-all-in-one
yzh
3.4.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551:30311712
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255:30335481
vscod805:30301674
pythonvspyt200cf:30331938
vscextlang:30333561
binariesv615:30325510
bridge0708:30335490

As I work around I have changed my settings to use Cascadia Code PL, which does not ship with "Cascadia Curve", but it's a work around, not a fix. Other apps don't suffer this problem. For example, when browsing Stack Overflow, who use Cascadia Code Mono for code, on Firefox the "Curve" characters are not used for code in italics.

VS Code does not do explicit font picks, it just passes setting value to CSS styles so question is to Chromium.
Make sure that you have latest version of font(note that Windows will "merge" instalation with same name), don't have ss01 enabled https://github.com/microsoft/cascadia-code#font-features
Having manually installed font with name different from fonts installed with WT is best way to avoid version conflict

VS Code does not do explicit font picks, it just passes setting value to CSS styles so question is to Chromium.
Make sure that you have latest version of font(note that Windows will "merge" instalation with same name), don't have ss01 enabled https://github.com/microsoft/cascadia-code#font-features
Having manually installed font with name different from fonts installed with WT is best way to avoid version conflict

This doesn't really solve the problem @IllusionMH  . As mentioned, the latest font is installed; I explicitly stated it is. ss01 isn't part of the version shipped with WT, so enabling (or explicitly not) as no effect; it's the latest version that uses ss01.
As for using a different name, that would break other applications that use the correct version of the font; that is not a good suggestion in my opinion.

So as I can see - you have 2 conflicting versions of font installed (user and WT) and as I've mentioned - VS Code just sets font-family: 'Cascadia Code'; for text in editor, nothing more.
Is this problem reproducible in Chromium based browser if you set next styles on text?
font-family: 'Cascadia Code';
font-style: italic;

I'm afraid I don't have Chrome installed @IllusionMH  so couldn't comment on if that occurs or not. As mentioned, I use Firefox and the problem does not occur in that application. The only app I have noticed (so far) that uses the wrong font is VSCode.

@IllusionMH is right, font selection is deferred to Chromium. If there's a problem you could try follow up in the electron repo if you can reproduce it there.

@IllusionMH is right, font selection is deferred to Chromium. If there's a problem you could try follow up in the electron repo if you can reproduce it there.

Hi @Tyriar , as a non-programmer, and someone who literally has no experience or interaction with Electron, it would be impossible for me to write an application to reproduce the problem. I'm happy to report the bug with VSCode on the Electron repo though, if that's where you are saying it should be reported. Are you able to point me in the direction of their repo please?
Otherwise, if someone has to make an app, perhaps a member of the VSCode team could log it on their repo, as it is a bug.
Many thanks.

@deepak1556 do you think this is worth pursuing in the Electron repo? We just using font-family CSS and Chromium selects the actual font.


ci: use jest circus runner

Types

[x] 💄 Code Style Changes
[x] 🏗 Build System
[x] ⏱ Tests

Background or solution
the jasmine runner is outdated, circus is the new one
Changelog

CI 时间有点超长的迹象


Images do not appear in slide presentation in the browser

Hi,
everything great but... images do not appear in slide presentation in the browser:
I use wikilinks: ![[files/IMG_3311.JPG]] that in browser is <img src="files/IMG_3311.JPG" alt="" style="object-fit: scale-down">
Obsidian 1.0.3 / Mac OS Ventura 13.0.1 / Safari 16.1

I can reproduce the same bug.

Im not able to reproduce this bug. Can you provide me a sample vault?

Im not able to reproduce this bug. Can you provide me a sample vault?

I tested plugin in a new empty vault and it works so... there might be some plugin that conflicts...

I encountered this problem as well and from what I can tell the issue is specifically with images that are not in the same directory as the markdown file.
In a new vault, I used this markdown file:
# Image Test

---

# PNG

![[images/image.png]]

---

# SVG

![[images/image.svg]]

It renders fine in the Obsidian edit tab and in the Advanced Slides Preview window.  The images are not present in presentation mode in the browser.
Developer tools in the browser show 404 errors for the images in question.
My environment:

Obsidian 1.3.5
MacOS Ventura 13.4
browsers chrome and safari

Wanted to provide some additional observations and a workaround.
It appears that the problem will occur whenever images are not in the root of the vault.
Sample vault file structure that will exhibit the problem:
├── slides
│   ├── slideshow-1
│   │   ├── slideshow.md
│   │   ├── image.png
│   │   └── image.svg
└── templates

$ cat slideshow.md

# Image Test

---

# PNG

![[image.png]]

---

# SVG

![[image.svg]]

Images render in the Obsidian edit tab and the Advanced Slideshow Preview, but not in the browser.  In Chrome, View -> Developer -> Inspect Elements -> Sources tab shows no folder under localhost:/slides/slideshow-1.
Workaround:
Changing the image path to be relative to the vault root results in the images not being visible in the Advanced Slides Preview tab, but they will be visible in the browser, ie from my example above:
$ cat slideshow.md

# Image Test

---

# PNG

![[slides/slideshow-1/image.png]]

---

# SVG

![[slides/slideshow-1/image.svg]]

The images will  appear In Chrome, View -> Developer -> Inspect Elements -> Sources tab under localhost:/slides/slideshow-1.

+1


build for KW40z, KW41z fails to generate isr_tables

Describe the bug
Building an example for a frdm_kw41z results in a failure in generating isr_tables when gpio portb is enabled:
[ 97%] Generating isr_tables.c
gen_isr_tables.py: multiple registrations at table_index 31 for irq 31 (0x1f)
Traceback (most recent call last):
  File "/[..]/zephyr/arch/common/gen_isr_tables.py", line 308, in <module>
    main()
  File "/[..]/zephyr/arch/common/gen_isr_tables.py", line 300, in main
    error("multiple registrations at table_index %d for irq %d (0x%x)" % (table_index, irq, irq))
  File "/[..]/zephyr/arch/common/gen_isr_tables.py", line 35, in error
    raise Exception()
Exception
make[2]: *** [zephyr/CMakeFiles/kernel_elf.dir/build.make:63: zephyr/isr_tables.c] Error 1
make[1]: *** [CMakeFiles/Makefile2:314: zephyr/CMakeFiles/kernel_elf.dir/all] Error 2
make: *** [Makefile:84: all] Error 2

To Reproduce
Steps to reproduce the behavior:

enable GPIO_MCUX_PORTB in boards/arm/frdm_kw41z/Kconfig.defconfig
build e.g. hello_world example
cmake -DBOARD=frdm_kw41z
make
See error

Impact
Examples for frdm_kw41z or boards based on kw40z and kw41z will fail in building when all gpios are activated
Screenshots or console output
in debug mode, I'm getting this:
[ 97%] Generating isr_tables.c
gen_isr_tables.py: (32, 0)
gen_isr_tables.py: Configured interrupt routing
gen_isr_tables.py: handler    irq flags param
gen_isr_tables.py: --------------------------
gen_isr_tables.py: 0x1957     31  0     0x20000ed0
gen_isr_tables.py: 0x1957     31  0     0x20000edc
gen_isr_tables.py: 0x1957     30  0     0x20000ee8
gen_isr_tables.py: offset is 0
gen_isr_tables.py: num_vectors is 32
gen_isr_tables.py: multiple registrations at table_index 31 for irq 31 (0x1f)

Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK
Commit SHA or Version used: 0f3bd0f43bf5bbc559303dc91d4fc3aacb58281e

Additional context
This check was recently introduced by #11273 by @andrewboie, issue is similar to k64 in #11275.
possible fix: Assign a different interrupt number in nxp_kw40z.dtsi and nxp_kw41z.dtsi.
How were the numbers 30/31 selected? Which number to select instead of 31?

I think the change from @andrewboie is just showing an issue in that we need to implement a shared GPIO handler for this case in the mcux gpio driver.

gen_isr_tables.py: 0x1957     31  0     0x20000ed0
gen_isr_tables.py: 0x1957     31  0     0x20000edc

Yeah, before I added the check it was undefined which would be installed; same handler function in both cases but the parameter would be one or the other for all interrupts which is not what you want.

@jremmert-phytec-iot KW41z platform uses IRQ 31 for both Port B & C's GPIOs. Port B is also used for ADC and other peripherals, this is why it's normally disabled.
@galak @andrewboie a proper implementation would be to check on IRQ 31 both PORTB_ISFR and PORTC_ISFR registers to check the source of the interrupt in this particular case. In general, I don't see how to create a generic configuration for peripherals that share the same interrupt.
A fast and middle-ground way to fix this is to remove interrupts on GPIOB.
GPIO on port B is not used by default, no pinmux initialization; let alone pin interrupts.


Bug in UpSampling2D with channels_first and Torch backend

In up_sampling_2d.py https://github.com/keras-team/keras/blob/5bc8488c0ea3f43c70c70ebca919093cd56066eb/keras/layers/reshaping/up_sampling2d.py#L166
When data_format == "channels_first", new_shape is wrong because x has been transposed. This leads to wrong output dimensions during training.
Dirty fix:
Instead x.shape[rows], x.shape[cols] use x.shape[1], x.shape[2].
         # Correct code for lines 165-168
             new_shape = (
                 x.shape[1] * height_factor,
                 x.shape[2] * width_factor,
             )

Otherwise, move the new_shape out of the if clause.
Hope it helps,
Vyacheslav

I observe the same behavior across all backends and in keras 2 as well.
new_shape as highlighted by you does not traverse to that block when it is channels_first.
Below is the sample code
import os
os.environ["KERAS_BACKEND"]="torch"
import keras
import numpy as np

input_shape = (2, 2, 1, 3)
x = np.arange(np.prod(input_shape)).reshape(input_shape)
print(x)

y = keras.layers.UpSampling2D(data_format="channels_first",size=(1, 2))(x)
print(y)

Hi, thank you for looking into this issue. The bug only appears when interpolation is not nearest. You could, e.g., try the following
import os
os.environ["KERAS_BACKEND"]="torch"
import keras
import numpy as np

input_shape = (2, 2, 1, 3)
x = np.arange(np.prod(input_shape)).reshape(input_shape)
print(x)

y1 = keras.layers.UpSampling2D(data_format="channels_first",size=(1, 2))(x)
print(y1.shape)

y2 = keras.layers.UpSampling2D(data_format="channels_first",size=(1, 2), interpolation="bilinear")(x)
print(y2.shape)

I will take a look and try to fix it.


Typo in artisan laracogs:starter

Guys, this repo is bananas, great work. One small thing so far, at the end of running laracogs:starter it says:
Don't forget to run:
composer dump
Then:
artisan migration

It should be artisan migrate

I totally missed that - thank you! and great catch

No problem... oh, and while I'm QA'ing for you:
In https://github.com/YABhq/Laracogs/blob/develop/README.md
where it says
For Laravel 5.2

You will also need to set the location of the email for password reminders. (app/auth.php - at the bottom)

the file location should read
config/auth.php
;)

These will be in the next release


Option to control the relays on large pcb

Is your feature request related to a problem? Please describe.
Cannot control the relays on large Heishamon PCB

Hello,
I think the issue should be opened on https://github.com/Egyras/HeishaMon instead of this repository which focuses on the Home Assistant Integration.
I would also advise to add more details when opening the issue.

I was redirected already, quote:
The relay function is not yet available in the home assistant integration.
The person how created that software needs to add that.
Can you check if the relay option is available in the integration before
removing my request?
Thanks in advance!
W dniu wt., 16.07.2024 o 13:06 Grégoire Seux @.***>
napisał(a):

Hello,
I think the issue should be opened on https://github.com/Egyras/HeishaMon
instead of this repository which focuses on the Home Assistant Integration.
I would also advise to add more details when opening the issue.
—
Reply to this email directly, view it on GitHub
https://github.com/kamaradclimber/heishamon-homeassistant/issues/227#issuecomment-2230622891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BF3EBUB7WLIQOOOIG4HI73DZMT5DJAVCNFSM6AAAAABK6JTXMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZQGYZDEOBZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok thanks for the quote.
Could you give more details/documentation on what are the relays you refer to  please?
At the moment I can’t find details on Heishamon repository.

Controlling the relays on the new large model isn't documented yet, however it is working.
It requires a mqtt publish on the topic panasonic_heat_pump/gpio/relay/one or panasonic_heat_pump/gpio/relay/two with a value of 0 or 1.

I would prefer to implement only features that are documented, it makes it easier to know when features are available for use.

I'll update the documentation soon ;-)

I would prefer to implement only features that are documented, it makes it easier to know when features are available for use.

@kamaradclimber by in safe side, @IgorYbema is main hardware and software developer :) new boards are really fresh ones..
Your doing great job :) this integration becomes more and more important to heishamon community :)

Documented now :) https://github.com/IgorYbema/HeishaMon/tree/main?tab=readme-ov-file#large-board-relay-control

The relays works, thank you 😀


Move all generated files to .xp-codegen directory

Ensure that all the files are generated outside ./src. In projects rootDirs in tsconfig.json can be used to create "overlapping" file systems.

This was implemented in 2.0.0


Introduce SystemZ version identifier and deprecate S390X.

The new 64-bit architecture is called System z. IMHO we should
use this name instead the cryptic S390X.

See https://github.com/D-Programming-Language/dmd/pull/5101 and https://github.com/D-Programming-Language/dlang.org/pull/1100.

Whoops I forgot I don't have merge rights.  Anyway LGTM.

I already changed the druntime source: https://github.com/D-Programming-Language/druntime/pull/1392


fix(js) prevent long series of ///// from freezing

Resolves #2656.

It should be possible this way too:
(/\/[\n]*)

@allejo  https://regular-expressions.mobi/catastrophic.html?wlr=1


Adds automatic error handling for ToolNode

This PR also changes the params of the ToolNode constructor, but the other params were undocumented and unlikely to be used. I think worth fixing now.

CC @vbarda


PI Search box

IF PI selected, disable input. Allow clicking clear button which should re-enable.
Hint text and custom search text saying search by email or kerb.

Just noticed that this was reassigned. The Typeahead control doesn't even display the [clear] button when the control is disabled. Not sure what the best approach would be. Maybe for now just place a [clear] button next to the Typeahead and later see if @cydoval can do some overlay magic with it?


Support lazy loading for nerdcommenter

Some recent vim plugin managers allow lazy loading with a syntax similar to this:
Bundle 'scrooloose/nerdcommenter', {'on': '<Plug>NERDCommenterToggle' }
map <leader>c<space> <Plug>NERDCommenterToggle

Unfortunately due to the way nerdcommenter starts it's not possible to load it lazily, when doing so it fails with something like this:
Error detected while processing function NERDComment..<SNR>43_IsInSexyComment..<SNR>43_FindBoundingLinesOfSexyCom..<SNR>43_Multipart.
.<SNR>43_Right:
line    3:
E121: Undefined variable: b:NERDCommenterDelims
E15: Invalid expression: has_key(params, 'alt') ? b:NERDCommenterDelims['rightAlt'] : b:NERDCommenterDelims['right']
line    5:
E121: Undefined variable: delim
E15: Invalid expression: delim == ''
Error detected while processing function NERDComment..<SNR>43_IsInSexyComment..<SNR>43_FindBoundingLinesOfSexyCom..<SNR>43_Multipart.
.<SNR>43_Right:
line    3:
E121: Undefined variable: b:NERDCommenterDelims
E15: Invalid expression: has_key(params, 'alt') ? b:NERDCommenterDelims['rightAlt'] : b:NERDCommenterDelims['right']
line    5:
E121: Undefined variable: delim
E15: Invalid expression: delim == ''

That's because there hasn't been enough time to call SetUpForNewFiletype, which is called after autocommands.
augroup NERDCommenter

    "if the user enters a buffer or reads a buffer then we gotta set up
    "the comment delimiters for that new filetype
    autocmd BufEnter,BufRead * :call s:SetUpForNewFiletype(&filetype, 0)

    "if the filetype of a buffer changes, force the script to reset the
    "delims for the buffer
    autocmd Filetype * :call s:SetUpForNewFiletype(&filetype, 1)
augroup END

A temporal workaround it to call SetUpForNewFiletype manually just at the end of NERD_commenter.vim
inoremap <silent> <plug>NERDCommenterInsert <SPACE><BS><ESC>:call NERDComment('i', "insert")<CR>
call s:SetUpForNewFiletype(&filetype, 1)

Anyway probably there is a better a way, so I left it here for your consideration.
Thanks for the great plugin!

Thanks for the contribution @chilicuil. I don't know of a better solution and I've hit this issue myself so I've gone ahead and implemented this hack. If anybody comes up with a better solution I'm all ears...


typo fix exmaples

Closes: #23

:robot:
Changes in the patch:

In .github/workflows/ci.yaml, the name of a job was changed from "Exmaple Complete Validate" to "Examples Complete Validate". The directory name was also changed from exmaples/complete to examples/complete.
In .github/workflows/tfsec.yaml, the name of a job was changed from "Exmaple Remote Validate" to "Example Remote Validate". The directory name was also changed from exmaples/remote to examples/remote.
Renamed the file exmaples/README.md to examples/README.md.
Renamed the file exmaples/complete/README.md to examples/complete/README.md.
Renamed the file exmaples/complete/main.tf to examples/complete/main.tf.
Renamed the file exmaples/complete/outputs.tf to examples/complete/outputs.tf.
Renamed the file exmaples/complete/terraform.tfvars to examples/complete/terraform.tfvars.
Renamed the file exmaples/complete/variables.tf to examples/complete/variables.tf.
Renamed the file exmaples/complete/versions.tf to examples/complete/versions.tf.
Renamed the file exmaples/remote/README.md to examples/remote/README.md.
Renamed the file exmaples/remote/main.tf to examples/remote/main.tf.
Renamed the file exmaples/remote/outputs.tf to examples/remote/outputs.tf.
Renamed the file exmaples/remote/variables.tf to examples/remote/variables.tf.
Renamed the file exmaples/remote/versions.tf to examples/remote/versions.tf.

Optimizations:

No optimizations are required as the changes are mostly renaming of files and directories, which does not impact the functionality of the code.

Potential problems:

There are no potential problems identified in the modified lines.

> gpt-3.5-turbo-16k


Stash

This PR adds
⭐ stash Version v0.21.0

Hello, can you upload one new screenshot ? There is the same probelm as last time in the screenshot.

Hello, can you upload one new screenshot ? There is the same probelm as last time in the screenshot.

no way... :D how are these screenshots flagged?
but ya, I will take care of it!
(sorry for the inconvenience)

CasaOS in order to comply with the legal and regulatory requirements of multiple countries worldwide, the Stash application cannot currently be made available on official app stores. You may consider self-hosting this application through third-party app stores. We deeply apologize for any inconvenience this may cause.


Building from Visual Studio triggers a modal dialog

Every build throws a "The solution 'Runner' has been modified outside the environment" modal. We need to pin down which generated file(s) are causing this and make them smarter about only re-writing when something changes, as it's made the IDE-based workflow much more annoying.

cc @loic-sharma
@stuartmorgan  is this still an issue? (Desktop Triage)

Probably; I never worked on fixing it. Although this probably predates the switch to CMake; it may be impossible to fix now in our current design.

@stuartmorgan I wasn't able to reproduce this on VS 17.x. Are these the correct steps:

Open ./build/windows/my_app.sln in Visual Studio
Set my_app as the start up project
Run Build solution
Run Start debugging to launch the app

If so, I think we can close this.

Oh right, I was misremembering and thought it was about command-line builds messing with the IDE.
So this was almost certainly fixed by switching to CMake, where we're no longer rebuilding sub-parts of the project in the re-entrant build.


fix talent expansion

Compendium documents changed the id property to a private property. This code switches to using a CompendiumCollection method for getting the document.
Fixes #982

I considered refactoring this code to use something like await game.packs.filter(pack => pack.documentName == type), but the awaits for game.packs.keys() and game.packs.get(packId) seem unnecessary in FoundryVTT 0.8.9. That makes me think I'm missing something, so I left the code as close to the original version as possible.
The silver lining is I think this improves performance anyways, because there is an await for each fetch of a document in a given pack,


Dockerfile: Go 1.8 & Official latest haproxy 1.5.X

As stated in #200, this upgrades the docker image to Go 1.8 and the latest Haproxy 1.5.X from there.
We have been running this for 1.5 month without any issue.

Coverage remained the same at 39.335% when pulling 69ec46905f540e3f09407b100c4260ba22e04fc2 on mvallerie:master into e21b2587ac6b98cc97fcd1739f9c09a80f4f2994 on QubitProducts:master.

Looks good. Thank you.


Almost there

Hi Santiago
Thanks for sharing this great project! I'm following along all the instructions and I almost have everything working.
I have RasPiOS running on a Raspberry Pi model 3B and have installed all the necessary software including xrdp (my first time using this!).
I can run the AstroCam app and see the preview on the screen in the RDP window.
But when I click the capture button, nothing happens and there are no pictures in the Astrophotography/JPG folder.
I tried running the app from the command line with the same result but after I quit I saw this appear:
pi@raspberrypi:~/AstroCameraApp/PythonScripts $ python3 AstroCam.py
Start
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.7/threading.py", line 865, in run
    self._target(*self._args, **self._kwargs)
  File "AstroCam.py", line 108, in camHandler
    camera.capture(stream, format='jpeg')
  File "/usr/lib/python3/dist-packages/picamera/camera.py", line 1423, in capture
    'Timed out waiting for capture to end')
picamera.exc.PiCameraRuntimeError: Timed out waiting for capture to end

Does any of this help to explain why I can't take pictures yet?
Cheers,
Richard Gain

Hi Richard :),
wow, it's nice to see someone has already tried replicating my project and
from the looks of it you are already practically done with the setup. I'm
happy my project instructions and python code are managing to be of use for
other members of the community!
Regarding the issue you have, it seems by looking at the last error output
"picamera.exc.PiCameraRuntimeError: Timed out waiting for capture to end"
that you may have run into a memory issue with the Picamera module
controlling the Pi. This means in essence that the GPU processing the
camera data is running out of allocated video memory and is unable to
capture images, thus timing out eventually. This is probably due to the HQ
camera modules higher resolution compared to the V1 and V2 camera modules,
which requires a higher memory capacity in order to not run out of it.
As for a possible solution; I stumbled upon this same issue too while
testing the camera initially using the command lines raspistill sequence
capture function. I managed to resolve it by increasing the amount of
allocated memory to the GPU from 128mb to 256mb, although admittedly that's
a ton and you probably won't need as much. Still I assume my AstroCam app
eats up a high amount of VRAM due to the live preview camera stream, so if
you have a GPU memory of, say for the Raspberry Pi 3, 64mb, try increasing
it to 128mb and if it still doesn't work to 256mb. You can do this from the
Raspberry Pi Software Configuration tools advanced settings section in the
Raspbian Settings menu.
I hope I was able to help and that this solution will provide useful for
you. If it works, I would also be thankful if you could let me know, so as
to include the VRAM requirement for the AstroCam script into the Hubble Pi
documentation. Should you have any further questions or if it still doesn't
work, do not hesitate to write back, as I would love to find out and work
on any possible bugs and issues within my code.
With kind regards and best of lucks, Santiago
On Sun, 28 Jun 2020, 14:17 richgain<EMAIL_ADDRESS>wrote:

Hi Santiago
Thanks for sharing this great project! I'm following along all the
instructions and I almost have everything working.
I have RasPiOS running on a Raspberry Pi model 3B and have installed all
the necessary software including xrdp (my first time using this!).
I can run the AstroCam app and see the preview on the screen in the RDP
window.
But when I click the capture button, nothing happens and there are no
pictures in the Astrophotography/JPG folder.
I tried running the app from the command line with the same result but
after I quit I saw this appear:
pi@raspberrypi:~/AstroCameraApp/PythonScripts $ python3 AstroCam.py
Start
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/usr/lib/python3.7/threading.py", line 865, in run
self._target(*self._args, **self._kwargs)
File "AstroCam.py", line 108, in camHandler
camera.capture(stream, format='jpeg')
File "/usr/lib/python3/dist-packages/picamera/camera.py", line 1423, in capture
'Timed out waiting for capture to end')
picamera.exc.PiCameraRuntimeError: Timed out waiting for capture to end
Does any of this help to explain why I can't take pictures yet?
Cheers,
Richard Gain
[image: image]
https://user-images.githubusercontent.com/1440566/85947277-97433b80-b941-11ea-8e6c-9afb7195e813.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RemovedMoney326/Hubble-Pi/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/API4GY7BITOBS6MSJFMFR5DRY4YFFANCNFSM4OKQFJMQ
.

Thanks for the speedy response.  I tried increasing the GPU allocation from 128mb to 256mb and rebooting but it didn't seem to make any difference. Would it help if I tried to run the app in debug mode to try and work out what is happening? I'm not a programmer but I do a lot of software beta testing at work and I don't mind digging into code a little bit.
Incidentally, when I run 'Camera Preview' the preview window displays correctly on the TFT screen but nothing shows up on the xrdp window. Is that the the expected behaviour?
Whilst I remember, I have some feedback for you on the instructions (which are generally superb, by the way). One section of the document that I got stuck on for a while was this paragraph in section 3.

"The other two shortcuts can be set up by pasting the following two text files from the Github repository on the desktop and subsequently making them executable".
Eventually, I worked out that you have to paste them as text files and then change the file extension to ".desktop" which I had not known about before. That might be worth including for anyone else who is not familiar with creating shortcuts in this OS.

Hello again Richard :),
firstlt, thank you for your feedback on the documentation! I will try to
keep it in mind once I make the next version of it with commentary on an
autoguide extension I'm already working on. I'm glad to hear that you found
it easy to follow and well written aside from that detail though :)
As for your issue, that's unfortunate to hear, I really thought VRAM would
be the cause of the issue, judging by the error code. Could you maybe open
the AstroCam.py file on the Thonny Python IDE built into Raspbian and
execute the program from there? Once opened, try taking a capture in
AstroCam and then send me the Python terminal output on the lower part of
the Thonny GUI. If something is wrong, an error should pop up there the
moment you click the capture button and it should probably give a more
detailed output on where exactly the program is failing to properly execute
the Picamera capture command.
Also, try doing a recording with the Record button too and see if the H264
video output works at least. To be honest I'm quite surprised in the case
that it's not a VRAM issue that the preview does work but the actual camera
functions don't, as most bugs I encountered during development had to do
with the camera module not initializing at times, leaving all functions
including the preview off. Maybe it's because of an issue with the output
path for the pictures, but that should show up in the Python console if
that's the case.
If you find out anything more, feel free to let me know, as I would very
much like to know the cause of this issue you have encountered, as well as
to figure out a way to solve it either by either modifying the python code
or the relevant raspbian setting and include it in the documentation.
With kind regards, Santiago
On Sun, 28 Jun 2020, 18:04 richgain<EMAIL_ADDRESS>wrote:

Thanks for the speedy response. I tried increasing the GPU allocation from
128mb to 256mb and rebooting but it didn't seem to make any difference.
Would it help if I tried to run the app in debug mode to try and work out
what is happening? I'm not a programmer but I do a lot of software beta
testing at work and I don't mind digging into code a little bit.
Incidentally, when I run 'Camera Preview' the preview window displays
correctly on the TFT screen but nothing shows up on the xrdp window. Is
that the the expected behaviour?
Whilst I remember, I have some feedback for you on the instructions (which
are generally superb, by the way). One section of the document that I got
stuck on for a while was this paragraph in section 3.
"The other two shortcuts can be set up by pasting the following two text
files from the Github repository on the desktop and subsequently making
them executable".
Eventually, I worked out that you have to paste them as text files and
then change the file extension to ".desktop" which I had not known about
before. That might be worth including for anyone else who is not familiar
with creating shortcuts in this OS.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RemovedMoney326/Hubble-Pi/issues/1#issuecomment-650786410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/API4GY433KYR4WOUPLGQYBLRY5S2PANCNFSM4OKQFJMQ
.

PD: Oh yeah, I almost forgot; the Camera Preview window really is only
supposed to show up in the TFT screen. Unfortunately, this is due to the
way the Raspistill preview was implemented by the Raspberry Pi Foundation
in Raspbian, which makes the GPU output the preview directly as a video
signal instead of to the desktop (which is why you see the preview overlaid
on top of the desktop on the TFT screen instead of in a neat window or
something like that). I think there are ways to work around it, but I found
that it would be more convenient to build the preview into a window using a
camera stream like Eric did in the code I based AstroCam on, as the
raspistill preview also has other limitations such as not being able to
modify the camera settings once it's turned on. Still, it is more stable
and delivers a better quality preview, so I kept it for when one just wants
to find an object in the night sky befire taking pictures.
On Sun, 28 Jun 2020, 18:04 richgain<EMAIL_ADDRESS>wrote:

Thanks for the speedy response. I tried increasing the GPU allocation from
128mb to 256mb and rebooting but it didn't seem to make any difference.
Would it help if I tried to run the app in debug mode to try and work out
what is happening? I'm not a programmer but I do a lot of software beta
testing at work and I don't mind digging into code a little bit.
Incidentally, when I run 'Camera Preview' the preview window displays
correctly on the TFT screen but nothing shows up on the xrdp window. Is
that the the expected behaviour?
Whilst I remember, I have some feedback for you on the instructions (which
are generally superb, by the way). One section of the document that I got
stuck on for a while was this paragraph in section 3.
"The other two shortcuts can be set up by pasting the following two text
files from the Github repository on the desktop and subsequently making
them executable".
Eventually, I worked out that you have to paste them as text files and
then change the file extension to ".desktop" which I had not known about
before. That might be worth including for anyone else who is not familiar
with creating shortcuts in this OS.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RemovedMoney326/Hubble-Pi/issues/1#issuecomment-650786410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/API4GY433KYR4WOUPLGQYBLRY5S2PANCNFSM4OKQFJMQ
.

OK, thanks. Good advice to run it in Thonny. I ran it in debug mode and stepped through the program.
All good until I clicked the Capture button, then I got similar message to before:
Start
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.7/threading.py", line 865, in run
    self._target(*self._args, **self._kwargs)
  File "/home/pi/AstroCameraApp/PythonScripts/AstroCam.py", line 108, in camHandler
    camera.capture(stream, format='jpeg')
  File "/usr/lib/python3/dist-packages/picamera/camera.py", line 1423, in capture
    'Timed out waiting for capture to end')
picamera.exc.PiCameraRuntimeError: Timed out waiting for capture to end

I wonder if it could have anything to do with installing Pillow whilst I was trying to get the scripts working.
Thanks for confirming that the Camera Preview doesn't display in the RDP window.
Best regards,
Richard

I just discovered that I don't have to click Capture. I get the same error after about 1 minute even if I just start the AstroCam program and leave it running.

Hmmm, so it really is running out of memory, even with enough VRAM, that's strange. Also, it seems it has to do with the camera stream solution for the Tkinter preview window... I've tried to reproduce the error in my own Raspberry Pi 4, but as in the screenshot below, it runs flawlessly (the telescope is covered, so the camera isn't picturing anything, but it outputs the JPG and all correctly). As it seems to be something very specific to your setup, I would ask you to check the following list of questions/tests and write me back with the answers/results if you like, so I can narrow down the list of culprits.

Raspbian, Python and Picamera versions (try updating them to the latest ones if they aren't already)

Confirmation of the currently allocated VRAM to the GPU in the info window (I have 256mb, so ideally try allocating the same amount if you haven't already as per the previous advice)

HQ Camera Module test using Raspbian and Picamera; this is to test the camera hardware and whether or not it communicates to the Pi correctly. If you can, make a short Python capture script and test if it is able to capture an image upon running.

I will also try to check the program for performance issues with the camera stream for the preview window, as that seems to be at first glance the source of the problem. Since I still have a Raspberry Pi 2 from an older project lying around, I should also be able to test the AstroCam script using a weaker GPU and check whether older Raspberry Pi models are being taxed excessively by it in combination with the HQ Camera module. I will let you know in case I find out anything more or if I upload an updated version of the Python code.

I agree, I think the stream is the problem. I commented out lines 107-112 and then found I could take a picture with the Capture button.
Raspbian = 2020-05-27-raspios-buster-armhf.img
Python = Python 3.7.3 (/usr/bin/python3)
Picamera = 1.13 (reinstalled it anyway, just in case)
VRAM = 256mb
As it was getting dark, I took three pictures:

using  raspistill -o image.jpg
using  "Picamera 3.1. Capturing to a file" script
using  raspistill -o image.jpg

1 and 3 look OK but 2 seems pretty dark

Ok, the camera stream is the problem then, as it seems to be straining your Raspberry Pi 3 GPU, causing it to time out the capture event . I will try and test the AstroCam script later this day on my Raspberry Pi 2 to look for the same issue there. I will also start looking into ways to do the camera preview in a more performance-friendly fashion, so I will keep you informed once I update the AstroCam script on the repository here.
If you want to try out something in the meantime, you can make a copy of the AstroCam.py file in your Raspberry Pi and play around with the lines for the camera preview to check if you can get it to work properly by modifying something. You can try changing the camera preview resolution to a lower one (the preview resolution is set each time the code has camera.resolution=(350,300) in it) or reducing the stream quality using the various PiCamera module setting for the camera.capture(strean) function (for more info on this, you can check the official PiCamera documentation from the Raspberry Pi foundation)
Lastly, I apologize for my code not correctly working after you went through all the trouble to replicate my project and all the inconveniences this might cause you. I hope I can still somehow fix it, although sadly I can't promise anything yet.
With kind regards, Santiago

Hi Santiago
Nothing to apologise for!  Everything gratefully received.
I am experimenting a bit more today and will let you know if I make progress.
Richard

OK, I've hacked your code about a bit and I have a working version now. The preview is a bit jerky but it works fine. There must be some issue with the stream function that I don't understand, so I'm not using it.

PreviewTest.zip

Hi Richard,
sounds great! I'm glad you managed to work it out then, seems like the issue really was the preview camera stream. If you want to, feel free to also send me the code you implemented to fix the issue (from what I understood, you did manage to implement the preview in a different way right?) so that I can review it and maybe pull it into the AstroCam.py file in the repository in its current or a modified way if works better than my current stream implementation.
With regards, Santiago

Hi Santiago
Did you see the PreviewTest.zip file attached above? I'm not sure it's worth including in your AstroCam app as it simply uses camera.capture to repeatedly save a 480x300 jpeg preview file instead of a stream, but it works for me.
I adjusted the sizes of the windows to give me a bigger preview and a rectangular outline for the app.
I also changed a couple of default settings to the way I prefer them; binning to 2 and ISO to 400.
Cheers,
Richard

Aha! I have now managed to get your script to work on my RasPi 3B by changing just one line.
Line 108 is now:
camera.capture(stream, 'jpeg', use_video_port=True)
It still flickers but it's a much simpler fix.

I see, you used the video port! That's actually quite a good idea, as the video port is better suited for doing sequential captures such as the one needed for a live preview stream. I guess that is why it runs now for you as well. Once I get the chance, I will implement and test out the stream using the video port on my own Hubble Pi and pull it into the AstroCam repository if it also works fine for me (even if my implementation works for me, if your line of code allows the program to run for the both of us, it still is worth implementing so that others don't step on the same issue.)
Good work in any case and you have my thanks for your patience and helpfulness, I hope you can enjoy your setup now that it's up and working :)
With kind regards, Santiago


Keeping the scroll position after reload

First of all, thanks for the awesome project. This is such impressive work and the team is doing great work for people all over the world.
I found that when I update the code on the left panel, the right panel is automatically reloaded and scrolled to top.
Is it possible to keep the scroll position after the reload? For example, if a user is seeing a section from 1000px from the top and update the code, the reloaded page shows the same section (1000px from the top) so that I can find what's changed without scrolling down the right panel.
Currently, I've added
document.addEventListener("DOMContentLoaded", function(event) {
            var scrollpos = localStorage.getItem('scrollpos');
            if (scrollpos) window.scrollTo(0, scrollpos);
        });

        window.onbeforeunload = function(e) {
            localStorage.setItem('scrollpos', window.scrollY);
        };

to the JS code manually to keep the scroll.
Thank you

Thanks for the feedback @hatemhosny

I'm using https://livecodes.io/ but the scroll position is not fixed here as well. Isn't https://livecodes.io/ production mode?

What you said makes sense and I respect the direction. But from a users' point of view, it would be better to find what's updated without scrolling down to the changed section. I think there could be a prop to enable this. (e.g., fixedScrollPosition)

Thanks and looking forward to updates!

I'll note that it's the default behavior of Safari to preserve scroll position on page reload, without preserving form inputs.

@jinhojang6
Now the result page scroll position is maintained after reload.
I do not use localstorage so as not to affect other projects.
The app keeps track of the result page scroll position and when it reloads, the position is sent to the new page.
Scroll position is reset if a new project is loaded.
This behavior only works inside the playground. If the result page is exported, this does not occur.
This behavior can be disabled by the query param:
?scrollPosition=false
You may test this in https://dev.livecodes.io
It is documented in https://dev.livecodes.io/docs/features/result#scroll-position
Thank you for reporting that.
Please let me know if this solves your problem.


Update the copy regarding unsigned add-ons

steps to reproduce:

Load https://addons.allizom.org/en-US/developers/addon/submit/2
Notice the warning copy under the checkbox for unlisted add-ons

observed behavior:
Since we now sign all unlisted add-ons, the copy needs to be updated.
screenshot:
https://www.dropbox.com/s/214tc1ikkcxlaen/Screenshot 2015-12-01 12.12.56.png?dl=0

Would be nice to get in before the tag if that's possible.

Touching this part of the code is tricky, and should be thoroughly QA tested, not sure we want that on such a short notice


Support EconomyAPI-PM5

Hi there
Just curious of youd be able to update this plugin so it would support the EconomyAPI-PM5 by mathchat900, bedrockeconomy is having a few issues of its own atm

Support for EconomyAPI was removed for theRankShop-PM5 because EconomyAPI is no longer under (official) development since PM4 and it's not performant. You can add support for EconomyAPI-PM5 by creating a addon for theRankShop.
If you want to create a addon-plugin:

Take a look at EconomyPluginMgr.php
Use the EconomyPlugin-interface

theRankShop won't add support for EconomyAPI again.


Unblock webpack Heroku deploys

Notes

Unblock webpack Heroku deploys by moving webpack-required dependencies out of  devDependencies
From the Heroku/nodejs docs: "...since you usually don’t want all development dependencies in your production builds, it’s preferable to move only the dependencies you actually need for production builds (bower, grunt, gulp, etc) into dependencies." (https://devcenter.heroku.com/articles/nodejs-support)

@alexsoble Awesome! 👍
Would you be open to trying one other thing first, which is to tell yarn "install both sets of dependencies?"  That keeps the package.json accurate about which things are for development and which things are for the actual app.  I think adding YARN_PRODUCTION=false yarn install --frozen-lockfile before yarn run build in the assets.rake hook might do the trick.
If that doesn't work, it'd be good to also see if moving these to devDependencies doesn't increase the bundle size.  I don't think it will, and even if it does I think we should still ship to unblock it and tackle later, but running a building locally could quickly confirm/disconfirm.

@alexsoble Yeah I think in general we want NODE_ENV=production, and if we want to change the npm/yarn config to behave in a non-production way in production, we should do that in a limited specific way (eg., on the command line rather than as an environment variable in general).
The Heroku docs also apply to when you don't directly run yarn install, but here we do since we're a Ruby dyno and we're manually running yarn install in assets.rake.  Normally folks can't just add this in a more limited way.

@kevinrobinson OK that makes sense! I tried force-pushing the tip of the experiment/webpack-jest branch to the demo site, expecting the deploy to fail since webpack etc. are in devDependencies, but the deploy succeeded! My guess it that something must have been cached from my earlier deploys. 😅

Ah that is scary if deploys are not isolated and repeatable!
I can poke around at this more later this week too then, but will leave it
for last. Would rather ship to clear the large PR and revisit this after.
On Tue, Oct 24, 2017 at 2:07 PM Alex Soble<EMAIL_ADDRESS>wrote:

@kevinrobinson https://github.com/kevinrobinson OK that makes sense! I
tried force-pushing the tip of the experiment/webpack-jest branch to the
demo site, expecting the deploy to fail since webpack etc. are in
devDependencies, but the deploy succeeded! My guess it that something
must have been cached from my earlier deploys. 😅
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/studentinsights/studentinsights/pull/1187#issuecomment-339080391,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAgvUqdi3WduR3qKc9ZPJU-HGuJpvAsks5svidPgaJpZM4QEkdq
.

This is working now in https://github.com/studentinsights/studentinsights/pull/1182#issuecomment-339215449

Thanks @alexsoble!


What is IPFS Cluster

Create a high level description that covers key concepts and vocabulary associated with IPFS Cluster. This would serve as a first entry point on https://cluster.ipfs.io/ to learn about Cluster.

We do need a 'what is Cluster'. The vocab will not likely be on the homepage. Is this issue for the vocab we need somewhere, or about redesigning the homepage? Those should be two separate issues.

Also, there is an existing homepage redesign issue at #56 .

The vocab part of this issue is now in #87 and the homepage redesign issue is in #56 . Closing.


Cache Flow modification message in Antrea Agent

Save OpenFlow modification message in the caches instead of ofctrl.Flow
Signed-off-by: wenyingd<EMAIL_ADDRESS>

/test-all
/test-windows-all

/test-all
/test-windows-all

/test-windows-all

@wenyingd this is a large size change, could you add more details about it for reviewers to understand?

What's the purpose of the change? (I know it but other reviewers don't)
How "caching flow modification message in Antrea Agent" could achieve the goal?
A evaluation of the improvement, for example, on a Node with XXX flows, how much memory it consumes before and now?

/test-all

/test-windows-all

The commit message is not wrapped with the same length limit. I will re-format it when merging. Would suggest to automate the formatting (e.g. set your .vimrc) to avoid efforts spent on this.

/test-all
/test-windows-all
/test-ipv6-all
/test-multicluster-e2e

need rebase

/test-all

/test-ipv6-only-all
/test-ipv6-all
/test-windows-all

/test-all
/test-windows-all
/test-ipv6-all
/test-multicluster-e2e


Improve speed on BB JM QL

Georgios Konstantopoulos:

Speed has been improved. Still slow though
Movement of our ship is now not smooth but "broken"
Not smooth movement of any missiles (ours or from UFOs)

Got speed up to 16 fps on an approx BB machine (via emulator). Throwing back to testers before doing more work on this.


Issue with --auto flags when sending proof proposal and having multiple credentials of the same type in the wallet

I do not know if this is a bug or simply something where aca-py's api behaves a bit unclear.
Given i have the following two credentials in my wallet, and both are based on the same schema and credential definition.
GET /credentials
{
  "results": [
    {
      "referent": "4ebbc6be-8cb2-4c1c-bd3e-77fc45d89647",
      "attrs": {
        "bic": "xxx",
        "iban": "yyy"
      },
      "schema_id": "M6Mbe3qx7vB4wpZF4sBRjt:2:bank_account:1.0",
      "cred_def_id": "M6Mbe3qx7vB4wpZF4sBRjt:3:CL:571:bank_account_no_revoc",
      "rev_reg_id": null,
      "cred_rev_id": null
    },
    {
      "referent": "72301763-D077-4A8F-9BC6-4613A612D4E8",
      "attrs": {
        "bic": "vvv",
        "iban": "zzz"
      },
      "schema_id": "M6Mbe3qx7vB4wpZF4sBRjt:2:bank_account:1.0",
      "cred_def_id": "M6Mbe3qx7vB4wpZF4sBRjt:3:CL:571:bank_account_no_revoc",
      "rev_reg_id": null,
      "cred_rev_id": null
    }
  ]
}

And my agent is configured with all the --auto flags enabled, and I want to send a proof presentation proposal like:
POST /present-proof/send-proposal
{
  "connection_id": "91992d99-c12d-426f-9b9d-de42b6c56b86",
  "presentation_proposal": {
    "@type": "did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/present-proof/1.0/presentation-preview",
    "attributes": [
      {
        "name": "bic",
        "value": "xxx",
        "cred_def_id": "M6Mbe3qx7vB4wpZF4sBRjt:3:CL:571:bank_account_no_revoc",
        "referent": "4ebbc6be-8cb2-4c1c-bd3e-77fc45d89647"
      },
      {
        "name": "iban",
        "value": "yyy",
        "cred_def_id": "M6Mbe3qx7vB4wpZF4sBRjt:3:CL:571:bank_account_no_revoc",
        "referent": "4ebbc6be-8cb2-4c1c-bd3e-77fc45d89647"
      }
    ],
    "predicates": []
  }
}

The exchange fails and I see the following in the logs:
2020-09-02 09:22:34,613 aries_cloudagent.core.conductor ERROR Exception in message handler:
Traceback (most recent call last):
File "/home/indy/.pyenv/versions/3.6.9/lib/python3.6/site-packages/aries_cloudagent/core/dispatcher.py", line 178, in handle_message
await handler(context, responder)
File "/home/indy/.pyenv/versions/3.6.9/lib/python3.6/site-packages/aries_cloudagent/protocols/present_proof/v1_0/handlers/presentation_request_handler.py", line 97, in handle
holder=await context.inject(BaseHolder),
File "/home/indy/.pyenv/versions/3.6.9/lib/python3.6/site-packages/aries_cloudagent/protocols/present_proof/v1_0/util/indy.py", line 54, in indy_proof_req_preview2indy_requested_creds
name = indy_proof_request["requested_attributes"][referent]["name"]
KeyError: 'name'

If either credential is removed the call succeeds. The problem seems to be that the referent information is lost in the exchange between the agents and aca-py does not know which credential to pick and hence fails (as this happens in the request_received state).

I'll get deeper into this late in the week. It should be fixable.

In this context, the referent in the proposal does not mean the cred id in the wallet: https://github.com/sklump/aries-rfcs/tree/master/features/0037-present-proof#referent
The two referents here, if both the same, would specify to use the same single credential to provide both attributes. In this case, since both are the same, it looks like it should have found the matching credential. I'll keep looking into it.

Yep, it's a bug, to do with what the proof req looks like when it specifies "names" on a single credential rather rather than "name". I should be able to fix it in the next day or so.


1026-market-header

Coverage remained the same at 74.218% when pulling 624cf059996ace6ebfa4c3f1f7083213f204ac58 on 1026-market-header into bd2f1f668f474b42d15d430c8e5e9b4943c23652 on master.


Dw biomarker refactor

Overhaul of Biomarker structure

So you're splitting this up into separate components. It's a good thig to do but difficult to properly review from a code perspective (I appreciate that it's normally me that sends you this kind of review :-)
Easier to test for changes then have a look through the code for anything suspect.
When I used the interface to populate it all the map seemed to work fine, but when I reloaded the page it didn't.
https://preview534--micronutrientsupport-tool.netlify.app/quick-maps/biomarker?country-id=MWI&mnd-id=I&measure=biomarker&data-level=household&age-gender-group=all
There's also a js error (on dev too https://develop--micronutrientsupport-tool.netlify.app/quick-maps/biomarker?country-id=MWI&mnd-id=I&measure=biomarker&data-level=household&age-gender-group=all) which might mask any problems.

Hi @bgsandan . I've done a bit more work and resolved the issues that @jon571 Jon highlighted. I've added you as a reviewer as @jon571 is on AL


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 943 ms

Resolved: Bofrost.fr performance has improved in ff56505 after 6 minutes.


LightGBM hyperparameter

Description
Use optuna to find LightGBM hyperparameter.
But it doesn't seem to have a significant effect. 🥺
Motivation and Context
https://github.com/microsoft/qlib/issues/269
How Has This Been Tested?

[X] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[ ] Fix bugs
[X] Add new feature
[ ] Update documentation

@ChengYen-Tang  Thanks!
It looks so great!


Releases Doc Update

Re: Releases
Issue Description
confusion on "Latest": as of today, the latest release is either v20.2.3, or v20.1.10, depending on what you mean by latest: the former in terms of product features, the latter in chronological order.
Suggested Resolution
Group the releases by major version, example below, as it is more likely that customers might want to check the latest in both absolute terms, and the latest of their current version. Then it will be immediately obvious which version is the latest in terms of prod features (the top item on the top group), and the latest for each major version.
v20.2
v20.2.3 | Dec 14, 2020 | Precompiled 64-bit Binary
v20.2.2 | Nov 25, 2020 | Precompiled 64-bit Binary
v20.2.1 | Nov 20, 2020 | Precompiled 64-bit Binary
v20.2.0 | Nov 10, 2020 | Precompiled 64-bit Binary
v20.1
v20.1.10 | Dec 21, 2020 | Precompiled 64-bit Binary
v20.1.9 | Dec 1, 2020 | Precompiled 64-bit Binary
v20.1.8 | Oct 21, 2020 | Precompiled 64-bit Binary

Thanks for opening this, @fabiog1901. I just fixed the ordering: https://github.com/cockroachdb/docs/pull/9295. There's some automation to the way the page is built, so restructuring the page to be more clearly grouped by release will be a larger effort. I'll put this on the backlog for our product ops engineer.

Thanks for opening this, @fabiog1901. I just fixed the ordering: https://github.com/cockroachdb/docs/pull/9295. There's some automation to the way the page is built, so restructuring the page to be more clearly grouped by release will be a larger effort. I'll put this on the backlog for our product ops engineer.


How to cite drawio if I use diagrams.net to plot diagrams

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
Not really a problem. I created various diagrams using the drawio desktop. However, I could not find a bibliography entry for drawio to put in acknowledgement and bibliography.
Describe the solution you'd like
Would you please consider make this repo citable? https://guides.github.com/activities/citable-code/
or even better a comprehensive bibtex entry like:
@Article{         harris2020array,
 title         = {Array programming with {NumPy}},
 author        = {Charles R. Harris and K. Jarrod Millman and St{\'{e}}fan J.
                 van der Walt and Ralf Gommers and Pauli Virtanen and David
                 Cournapeau and Eric Wieser and Julian Taylor and Sebastian
                 Berg and Nathaniel J. Smith and Robert Kern and Matti Picus
                 and Stephan Hoyer and Marten H. van Kerkwijk and Matthew
                 Brett and Allan Haldane and Jaime Fern{\'{a}}ndez del
                 R{\'{i}}o and Mark Wiebe and Pearu Peterson and Pierre
                 G{\'{e}}rard-Marchant and Kevin Sheppard and Tyler Reddy and
                 Warren Weckesser and Hameer Abbasi and Christoph Gohlke and
                 Travis E. Oliphant},
 year          = {2020},
 month         = sep,
 journal       = {Nature},
 volume        = {585},
 number        = {7825},
 pages         = {357--362},
 doi           = {10.1038/s41586-020-2649-2},
 publisher     = {Springer Science and Business Media {LLC}},
 url           = {https://doi.org/10.1038/s41586-020-2649-2}
}

Describe alternatives you've considered
Or a simple bib entry like

T. A. Driscoll, N. Hale, and L. N. Trefethen, editors, Chebfun Guide, Pafnuty Publications, Oxford, 2014.

If someone could submit a PR for this please.

Sorry if I missed something obvious. I searched through the repo, however, still don't see how to cite draw.io.

Sorry if I missed something obvious. I searched through the repo, however, still don't see how to cite draw.io.

I've got the same problem. I can see the commit where it was added, but it seems to have vanished without record. In case you're still looking, here's the link to the most updated version of the file I found in the commit history: https://github.com/jgraph/drawio/blob/5fbd103a2834d490f460c8b5d18028b8bd584a56/CITATION.cff

@davidjgraph hey, would you accept a PR that put back this citation file? For others here is the bibtex I got using the .cff done by scaomath
original cff:
cff-version: 1.2.0
message: "To cite drawio or www.diagrams.net in publications please use:"
type: software
license: Apache-2.0
abstract: "diagrams.net, previously draw.io, is an online diagramming web site that delivers the source in this project."
authors:
- name: "JGraph"
  website: "http://www.diagrams.net"
title: "diagrams.net, draw.io"
version: 15.5.2
date-released: 2021-10-14
repository-code: "https://github.com/jgraph/drawio"
url: "https://www.diagrams.net/"

derived bibtex:
@misc{drawio,
author = {JGraph},
month = {10},
title = {diagrams.net, draw.io},
url = {https://www.diagrams.net/},
year = {2021}
}


Remove unused binaries from image

antrea-octant-plugin can be removed from image antrea-ubuntu
and other binaries can be removed from image octant-antrea-ubuntu.
This patch adds Makefile target per binary and only includes
the necessary binaries in the docker image.
Fixes: #708

How about grouping antrea binaries except for the ocatant to the bin target? Thus make bin looks easier to use.

@weiqiangt I have discussed this with Antonin in the issue#708. Since "make antrea-agent antrea-controller antrea-cni antctl-ubuntu" doesn't seem too long at this time, I am OK with not adding an extra target for now.

/test-all


Use php attributes to configure commands

As a side note, origami:database:details command is missing on the available commands wiki page.

Codecov Report

Merging #379 (1bc9d67) into master (30ea8e2) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #379      +/-   ##
============================================
- Coverage     97.81%   97.80%   -0.02%
+ Complexity      335      334       -1
============================================
  Files            41       41
  Lines          1098     1092       -6
============================================
- Hits           1074     1068       -6
  Misses           24       24

Impacted Files
Coverage Δ

src/Command/Configure/PhpstormCommand.php
96.15% <ø> (ø)

src/Command/DataCommand.php
100.00% <ø> (ø)

src/Command/Database/DetailsCommand.php
100.00% <ø> (ø)

src/Command/Database/DumpCommand.php
92.50% <ø> (ø)

src/Command/Database/ResetCommand.php
100.00% <ø> (ø)

src/Command/Database/RestoreCommand.php
90.47% <ø> (ø)

src/Command/DebugCommand.php
89.28% <ø> (ø)

src/Command/DefaultCommand.php
100.00% <ø> (ø)

src/Command/InstallCommand.php
100.00% <ø> (ø)

src/Command/LogsCommand.php
100.00% <ø> (ø)

... and 11 more

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 30ea8e2...1bc9d67. Read the comment docs.


fix for getting the beat scheduler working again in ODL edx deployments

Closes #836
How Has This Been Tested?
Built xpro ami locally with packer, started instance from AMI, checked file, executed file (didn't actually run because it wasn't an instance part of a real env... but celery beat attempted to run, which is all I'm looking for...)
Types of changes

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

Checklist:

[X] My code follows the code style of this project. (Did you install and run the pre-commit hooks?)
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Related upstream PR to openedx which uses single-beat.
https://github.com/openedx/configuration/pull/6764

:+1:


[feat]: shadcn-ui in angular

Feature description
just like you guys have shadcn in react make this in angular
Affected component/components
all component
Additional Context
No response
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

checkout Spartan UI, this is a project that is specifically doing the same.

checkout Spartan UI, this is a project that is specifically doing the same.

Damn, pretty cool actually good on them!


Connection close received (certificate unknown)

Hi, I'm currently try to deploy QUIC server, where client is from android with cronet.
When I type following command to activate QUIC server as a http3-server in the example,
python server.py -c certs/test.pem -k certs/test.key --port 7443
I find that QUIC server responses to client as follows:
2021-09-03 21:37:57,765 INFO quic [bb545e91d4e68f20] Connection close received (code 0x12E, reason 199:TLS handshake failure (ENCRYPTION_HANDSHAKE) 46: certificate unknown)
It seems that certificate does not have problem since I ran https nginx server with same pem and key files..
Hope there's one can answer my problem. Thanks in advance.

@lgs96 did you find a solution for this? Nothing related to QUIC seems to work locally...

@litvinav My problem was from the incorrect reference of certificate file. After I refer the correct certificate file, I could solve the problem.


Autoformatter?

What is the problem?
Does this turn on some sort of autoformatter? And if so, how can I disable it? Not seeing an option at https://github.com/neilsustc/vscode-markdown#supported-settings
I cloned down https://github.com/StackExchange/blackbox/ to make a small PR to the README.md but when I hit save, it autoformatted the file.
When I disabled this extension, it didn't happen.
I also have prettier installed, but when I disabled that it still happened. Also, I have this in my user settings (among other things):
"[markdown]": {
  // per https://github.com/prettier/prettier-vscode/issues/275
        "editor.formatOnSave": false,
        "editor.wordWrap": "bounded",
        "editor.wordWrapColumn": 100,
    },

How can I reproduce it?
Install this extension, clone down above repo, try to edit README.md

What do you mean about "autoformatting" (the toc, or GFM table)?
There is an option markdown.extension.toc.updateOnSave defaults to true, which controls whether to update toc on file save.

Looks like that's it. Thanks!


Explicitly detach from the console on exit

Without this cmd won't redraw it's prompt when alacritty exits
Fixes https://github.com/jwilm/alacritty/issues/1698

Should be resolved, I changed the attach console call as well.


[Feat] Integrate AuthClient with VerifyAPI into react-auth-dapp

Context

Our sample auth dapp provides a testing ground for wallets integrating Auth.
Once Verify API is part of the AuthClient, we should ensure it's exposed and configured properly to help the entire web3wallet team dogfood their implementations of Verify API.

Dependencies

Depends on:

#72
Publishing updated version of @walletconnect/auth-client containing the Verify API changes

Need to publish updated @walletconnect/auth-client and will be ready for PR

Published auth client canary, ran into issue where the verify server is blocking our domain. Gancho is handling this

Waiting for new auth package to be published tonight to update then merge


Fixed issue #483

Title and Issue number
Title :About the newsletter section
Issue No. : #483
Code Stack : html
Close #483
Video/Screenshots (mandatory)

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[ ] I have commented my code, particularly in hard-to-understand areas
[X] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
I fixed the typo error as discussed in the issue and I also removed another typo error which I found in the FAQ section.
Are you contributing under any Open-source programme?
Yes , GSSOC'24.

Ok...I will make a new PR.


Install a custom library with data_files in pipenv fails

Install a custom library in pipenv fails
I have a setup.py file with data_files for config files.
When I run pipenv run python setup.py install it generates a egg file in
virtual environment but without the data files set in setup.py
Expected result
Create a egg folder with data_files setting in setup.py file
Actual result
An egg file in site-packages folder without the data files setting in setup.py

If you create a virtual environment, do you have the same issue? The problem may be interactions between pip/setuptools rather than anything Pipenv.
python -m venv testvenv
testvenv/bin/python setup.py install
cd testvenv/lib/python?.?/site-packages/  # are they in here?

Other things to check:

Build an sdist and see if the data files are packaged in the tar.gz
Move to a MANIFEST.in approach. Example from a package I have:

https://github.com/nicktimko/claptrap/blob/6ab439f/MANIFEST.in
https://github.com/nicktimko/claptrap/blob/6ab439f/setup.py#L21

Thanks for the feedback, yes I tested in another virtualenv and it works fine.
I also tested with sdist command and with a properly MANIFEST file everything is ok.
The problem is when I do pipenv run python setup.py install and then check the virtualenv managed by pipenv. In site-packages there is not my_lib.egg as a folder with all my config files, instead it is a file.
That's why I created this issue everything seems to work until I try to install with pipenv,  just in case would be a strange bug in pipenv.

Sounds good then. I'd try to create a minimal, complete example that you could point to though. Your report doesn't have any way to recreate the issue as there is no setup.py pointed to.

My problem is with a custom library which use cython. I will try to create a minimal to try to replicate.
Sorry for not sharing anything, I check this 'possible bug' in my company

This should be resolved on pipenv==2022.8.19


Have you tried running CLI on Bun?

Have you tried running on Bun? That is pretty much my go-to these days. The CLI does not run on Bun - and it may be a Bun bug which I can report to them, but I can't diagnose what's wrong to make a meaningful report.
bunx zzapi-cli tests.zzb -e local # runs in Node
bunx --bun zzapi-cli tests.zzb -e local # runs in Bun

When I run the second one with Bun, the CLI outputs this:
5/21/2024, 8:15:28 PM [ERROR] GET List Test Error executing request: Attempted to assign to readonly property.)

Any idea where to start looking?

Oh. We're sorry. We have no clue about bun. The error message may be a clue. At the moment, we don't have enough bandwidth to support bun.

That's reasonable. I'll try to look into this over the weekend and see if I can figure out from where the message originates.

I cloned the CLI & the core library.  Got the CLI building and was able to run it from a pure build with Bun using tests-bundle.zzb from the zzapi examples. 🤔
I then went back to my app repo and started playing around. TL;DR it looks like the latest release of Bun (1.1.9) fixed some Node compatibility bugs and this now works perfectly. 🎉
As they hype, code runs a lot faster & yields the same results. Just sharing this in case it interests you.
time node ./dist ../zzapi/examples/tests-bundle.zzb
# Node 18.17 (old, I know) output omitted
real 0m5.736s

time bun ./dist ../zzapi/examples/tests-bundle.zzb
# Bun 1.1.9 (latest) output omitted
real 0m1.494s


Expose back-end port

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


Fix: Keep packages sorted

This PR

[x] keeps packages in composer.json sorted without having to specify --sort-packages when requiring packages

Also not directly related to #433.
💁♂ For reference, see https://getcomposer.org/doc/06-config.md#sort-packages.

@localheinz There's a conflict with the composer.lock file that needs to be resolved...

@chartjes
Let me sort this out once #434 has been merged!

Rebased!

Thanks for reviewing all the PRs, @chartjes!


Automatically trim recordings

Is your feature request related to a problem? Please describe.
I'm always frustrated when I watch a recorded lecture and need to skip multiple minutes of silence/lecture setup to the actual beginning.
Describe the solution you'd like
Hello everyone,
I'm a big fan of the new streaming site and use it for many of my lectures.
One thing that I've noticed is that most recordings seem to be turned on and off automatically which leads to big sections of silence at the beginning and end of the recorded lectures.
I think it should be possible to auto detect these sections and remove them using ffmpeg.
Maybe both the full and trimmed version could be hosted by the site and switched somehow.
What do you think?
Describe alternatives you've considered
Downloading the videos and doing it myself locally on my computer :wink:

I'm a big fan of the new streaming site and use it for many of my lectures.

Yay! :partying_face: :heart:
What we currently do is automatically detect silences in recordings and offer a button to skip them (for example here: https://live.rbg.tum.de/w/FDS/12051)
I didn't start automatically cutting recordings yet because I'm not confident about false positives (e.g. here: https://live.rbg.tum.de/w/ccppprog/15036 we detected silences where there are none)
Addtitionally, some lectures overlap with other lectures, so the video doesn't start with silence but with some other lecture instead... If you have any ideas on how to enhance the feature, feel free to share them :)

Oh, then it's my bad, I didn't see that the feature is already implemented!
Great Job! :) I'm gonna close the issue, because the current apporach fixes the problem.


Compatibility for Quality Colors

It would be great if u Can add support for Quality Colors

I asked Lego about it.
Apparently, this is a commissioned mod and as such the source code is not available to support it.

yeah i saw your converstion on discord
since u dont touch vanilla code cant u just add colour to the text for quality items ?


move request pooling into hoodie.js core

I made withSingleRequest & withPreviousRequestsAborted for request pooling in the  hoodie.account pooling.
I think withSingleRequest should be the default behavior of GETs / DELETEs and withPreviousRequestsAborted the default behavior for PUTs / POSTs in  hoodie.request.
That means if I call hoodie.request('GET', '/_session') several times before the first request finishes, the subsequent calls don't send another request, but simply get the promise of the first.

closing in favor of https://github.com/hoodiehq/hoodie-client/issues/81


Getting error message on adding rental product in cart if rental booking is not available for that day.

Bug report
Title
Getting error message on adding rental product in cart if rental booking is not available for that day.
Issue Description
Getting error message on adding rental product in cart if rental booking is not available for that day.
Preconditions
Please provide as detailed information about your environment as possible.
1. framework Version.-master

Steps to reproduce
1. step1- Add a product for date on which rental is not available

Expected result
Display a correct erorr message
Actual result

fixed


Fix #187: ModuleNotFoundError: No module named 'django' when using SQLAlchemy

Fix for #187.  I'm sure that there is a more elegant way to fix this, but it seems to have gotten the job done.

I just issued a PR for rest framework that fixes the issue, the same method is used here that is breaking and we could probably do the same solution

@audiolion  That sounds good.  I had not looked into actually fixing the requests issue.  😄   Regardless of my bandaid (which did get around the requests issue), it still looks like Travis CI builds are still failing.  I'm still new to all of this, but looks like the tests pass but the script exits with a nonzero status.

No worries, your CI is almost passing! Just a few lint errors
apistar/backends/__init__.py:4:5: F401 'apistar.backends.django_backend.DjangoBackend' imported but unused
apistar/backends/__init__.py:10:5: F401 'apistar.backends.sqlalchemy_backend.SQLAlchemy' imported but unused
apistar/backends/__init__.py:14:1: W391 blank line at end of file

You can run scripts/lint before pushing and it will tell you of these issues. In the case of unused imports you remove them, and you have an extra blank line at the end of the init file

Thanks @audiolion

I'm wondering in the file that I was trying to fix whether I should be passing # noqa for the imports similar to this SO

We no longer need the django/sqlalchemy part of this.
I'm good with us switching to using urllib3 directly. We could list a minimum version for requests in our setup.py rather than using compat.py.


feat(build system): Added step to build process that creates index.js in...

... the root folder and exports all the main classes for CommonJS support
Change-Id: I53e0bcf483f6e3d46e9d9a28a4881fad7b6be13c

Since index.js is generated code, would you please add it to .gitignore?

Sorry for the delay, I missed the update.  Thanks for adding this!

No worries thanks!  Any idea when these latest changes will be published to NPM?

With our next release version, 1.3.0, which I'm currently expecting by the end of the month.
Oh, and one more thing I forgot to ask you to add to the pull request before I merged it.  Can you please add yourself to CONTRIBUTORS and AUTHORS?

Ok cool thanks!  I've submitted a separate PR for the CONTRIBUTORS and AUTHORS update.  https://github.com/google/shaka-player/pull/37

@palmerj3
Do you know if it's straightforward to take your build and use it as an AMD module? Ideally I'd like to import shaka using RequireJS. I've attempted to take the output of this build task and wrap it in a requireJS define() which works to a point. I can import it but then when I try to play back content I get an error: cannot read property addEventListener of undefined.
For some reason I'm having a hard time finding out exactly where it's failing but from what I can tell it seems like there might be a problem when it hits FakeEventTarget and tries to create shaka.util.MultiMap. shaka seems to be undefined at this point.
Do you have any insight here? Or at the very least, could you explain how you are using it? This might give me a better idea of what I have to do to get it working how I want.

@sanbornhnewyyz I think instead of us supporting all possible async dependency resolution solutions you could utilize webpack or something like this within your project which would allow you to utilize CommonJS, RequireJS, etc dependencies interchangeably.
Sounds similar to what you've already done but perhaps take a look at:
http://requirejs.org/docs/api.html#cjsmodule
For me the way I'm using it is:

@sanbornhnewyyz I spoke too soon... so I was able to get it working but I'm not sure it would be wise to include in the main build... @joeyparrish would like your thoughts.
It's not ideal because it's tying the build process to specific implementation details (e.g. shaka.player, shaka.polyfill, etc). But it works... perhaps it's useful to you.
If you add this to build.sh:
(library_sources_0; closure_sources_0) | compile_0 \   --output_wrapper='%output% define({player: shaka.player, polyfill: shaka.polyfill, util: shaka.util});' \   --js_output_file "$dir"/index2.js
In this case "index2.js" will define all the CommonJS stuff needed so you can do something like this:
(function() {
  'use strict';

  require(['node_modules/shaka-player/index'], function(shaka) {
    shaka.polyfill.Fullscreen.install();
    shaka.polyfill.MediaKeys.install();
    shaka.polyfill.VideoPlaybackQuality.install();

    var player = new shaka.player.Player(document.getElementById('foo'));
    var playerSource = new shaka.player.HttpVideoSource('http://video.webmfiles.org/elephants-dream.webm');
    player.load(playerSource).then(function() {
      player.play();
    });
  });
}());

@joeyparrish @sanbornhnewyyz I created this - let me know your thoughts.  https://github.com/google/shaka-player/pull/55

@palmerj3 maybe I spoke too soon. At least for requireJS purposes, your commonJS build works fine if you wrap the whole thing with:
define(function (require, exports, module) {...});

I tried doing this with your example above and it playsback fine. But when I try to require this module in our actual project I run into these undefined errors when I try to playback content. So it looks like something is not playing nice with our project
I agree with not supporting all module flavours though for the big ones you could add a flag that let's you set which module type you want.

I wouldn't mind supporting both commonJS and requireJS by default, however I'd prefer it greatly if we could combine all of this into one wrapper.  It cuts down compilation time, which makes a big difference for us trying to iterate rapidly on new features.  Let's continue this discussion on #55.


pymongo.errors.OperationFailure: unknown top level operator: $expr

I have an aggregate query that includes this $match action in the pipeline:
{'$match': {'$or': [{'$expr': {'$gt': ["$latest", "$thing.analysisMeta.timestamp"]}},
                            {'thing.analysisMeta': None}]}},

In the real mongo database, this aggregate query works. With MongoMock >= 3.15.0, I get the error:
pymongo.errors.OperationFailure: unknown top level operator: $expr

In MongoMock 3.12.0 and 3.14.0, I do not get the error. I think it might be related to the fix for this issue: https://github.com/mongomock/mongomock/issues/495
For now, I can stay on 3.14.0, but would be nice to fix this in future releases so that we are not stuck.

Hi @valerierose , thanks for reporting. It actually never worked because it was never implemented in mongomock. However it should not raise an OperationFailure but a NotImplementedError. I'll fix that ASAP, and then try an implementation of the feature but that might take more time.


Validation Breaking With Multiple Items with same aa-form-group name

This may be the same bug as 125, but I am unsure so it's getting submitted as a new one.
I am running into issues where I have a list of objects and each of those is being edited within the same form.  To do this, I am passing each object into a directive which has aa-form-group inputs.  It appears that validation is only disappearing when the last bound item satisfies the validation, yet the form will not submit until all items pass validation.
The README.md has instructions as to how to repeat the issue.
Plnkr

Hey! Yeah it's getting confused because you have the same field names in the same form multiple times. This should fix it for you (just need to make multiple forms, angular and AA have no problem with multiple nested forms: http://plnkr.co/edit/sdHXdBj5FxNnvysMaMZ1?p=preview
This fix it for you?

Yup.  That did the trick.  I didn't see the advanced tab for the form extensions demo.  I do now :/


Dynamically rate limit the number of Graph API calls to fetch item data

Microsoft implements some rate limits as a number of requests per unit time. This doesn't fit well with the current Corso architecture because we have multiple collections that can be fetching items at the same time. Our strategy to speed up backups has been to fetch items in a collection in parallel, but this makes it such that items may fail rate limiting retries because other collections use the refreshed rate limit credits. Currently our only knob to control things is to adjust the parallelism per collection. However, reducing the fetch parallelism may increase the runtime of backups with few items as they will fetch items more slowly even though they don't need to
A more coordinated approach to controlling parallelism for item data fetch would be useful. That would allow smaller backups to run with minimal limits while slowing things down so larger backups don't fail

closing as we have a token bucket rate limiter now


fix: Pass sketch to plot instead of polydata

Pass sketch to plot instead of polydata, to allow drawing the faces of the sketch with a bit of transparency.

Did you try rendering the docs and see the outcome? =)

Did you try rendering the docs and see the outcome? =)

Nope, I will try on my local

Looks perfect now - thanks @AlejandroFernandezLuces

You can merge whenever you want


Add fab

Codecov Report

Merging #192 into master will decrease coverage by 0.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
- Coverage   99.21%   98.93%   -0.28%
==========================================
  Files          24       20       -4
  Lines         127       94      -33
  Branches        7        6       -1
==========================================
- Hits          126       93      -33
  Partials        1        1

Impacted Files
Coverage Δ

src/Fab.vue
100% <100%> (ø)

src/Drawer/DrawerNavItem.vue
100% <0%> (ø)
:arrow_up:

src/Drawer/Drawer.vue
85.71% <0%> (ø)
:arrow_up:

src/index.js
100% <0%> (ø)
:arrow_up:

src/Toolbar/ToolbarRow.js

src/Dialog/DialogHeaderTitle.js

src/Dialog/Dialog.vue

src/Drawer/DrawerNav.js

src/Toolbar/ToolbarSection.js

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 bd72338...2edc933. Read the comment docs.

Thanks! There're some adjustments I'll like to make to the api. I'll merge this and add them afterwards 🙂


Fix PHPUnit test cases and configuration

This PR fixes issue #46
On PHP7 and PHPUnit 6, this repo was having trouble running the tests, mainly because PHPUnit wasn't able to find any tests because the TestCase class was extending an old version of TestCase
Bumped the PHPUnit version and extended the correct class. Since this repo aims to support +5.6 (based on the travis configuration file) it doesn't make sense to use an old version of PHPUnit

Just checked the Travis results and I can confirm that tests are running now on all the supported PHP versions. :)

Bump @marcqualie

Apologies for the delay @andreskrey
Thanks for the contribution

No worries, just wanted to check if the project was still alive :)


Fix from_lsb_to_nsec() API overflow

Same issue as reported in https://github.com/STMicroelectronics/stm32-ism330dhcx/issues/1

Fixed by 9e0534a7


Wrong map coordinates for Landing Pads

The Landing Pads endpoint, returns a few pads with wrong coordinates. To be more precise, landing pads: OCISLY, JRTI-1 and ASOG, should have "longitude": -80.6188, instead of "longitude": 80.6188.
Thank you!

Updated 👍
Thanks!


Feat(compiler-base): reuse kclvm bug macro.

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

[ ] N
[x] Y

issue #115
2. What is the scope of this PR (e.g. component or file name):
compiler_base/macros/bug.rs
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

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

init crate 'macros' in compiler_base.
reuse bug macro 'bug!()' in compiler_base/macros/src/bug.rs.
add test cases for 'bug!()' in compiler_base/macros/src/tests.rs.
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

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

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

compiler_base/macros/tests.rs
6. Release note

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.579%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
22391

Relevant Lines:
37582

💛  - Coveralls


read 'docker.follow' from System.properties only

fixes issue #249
Signed-off-by: Jae Gangemi<EMAIL_ADDRESS>

Why is removing a configuration point a good thing?

b/c it's not a valid configuration setting, see #249.

Thanks !
I merged it, but could you please do me a favour and let's stick to the traditional way of PRs of sending them from your repo ? Probably accidentally you created the branch on this repo (not on your rebased fork).
This would help me in keeping the main repo more focussed.

that's weird - not sure how i would have manged to do that but i'll be sure to pay attention in the future.

No problem, btw :)