text:
TypeError: Cannot read property 'dismissable' of null

Description
What wanted you to do? what went wrong?
I saw this was closed in a commit but I'm still having this error:
Upon inject command, and error:
/usr/share/atom/resources/app/src/notification.js:96
Hide Stack Trace
TypeError: Cannot read property 'dismissable' of null
at Notification.isDismissable (/usr/share/atom/resources/app/src/notification.js:96:32)
at new Notification (/usr/share/atom/resources/app/src/notification.js:14:20)
at NotificationManager.addWarning (/usr/share/atom/resources/app/src/notification-manager.js:115:39)
at Object.insert (/home/justin.barca/.atom/packages/snippet-injector/lib/snippet-injector.js:275:26)
at HTMLElement.snippetInjectorInsert (/home/justin.barca/.atom/packages/snippet-injector/lib/snippet-injector.js:107:48)
at CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app/src/command-registry.js:388:36)
at Object.didConfirmSelection (/usr/share/atom/resources/app/node_modules/command-palette/lib/command-palette-view.js:143:35)
at SelectListView.confirmSelection (/usr/share/atom/resources/app/node_modules/command-palette/node_modules/atom-select-list/src/select-list-view.js:327:26)
at HTMLDivElement.core:confirm (/usr/share/atom/resources/app/node_modules/command-palette/node_modules/atom-select-list/src/select-list-view.js:84:20)
at CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app/src/command-registry.js:388:36)
at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/usr/share/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:621:22)
at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/usr/share/atom/resources/app/node_modules/atom-keymap/lib/keymap-manager.js:412:28)
at WindowEventHandler.handleDocumentKeyEvent (/usr/share/atom/resources/app/src/window-event-handler.js:108:40)
Steps to reproduce

run 'snippet-injector:insert' through panel

Environment
Atom Version: v1.23.1
Other installed packages:
Built-in Atom Packages (92)
├── atom-dark-syntax@0.28.0
├── atom-dark-ui@0.53.0
├── atom-light-syntax@0.29.0
├── atom-light-ui@0.46.0
├── base16-tomorrow-dark-theme@1.5.0
├── base16-tomorrow-light-theme@1.5.0
├── one-dark-ui@1.10.8
├── one-light-ui@1.10.8
├── one-dark-syntax@1.8.0
├── one-light-syntax@1.8.0
├── solarized-dark-syntax@1.1.2
├── solarized-light-syntax@1.1.2
├── about@1.7.8
├── archive-view@0.64.1
├── autocomplete-atom-api@0.10.5
├── autocomplete-css@0.17.4 (disabled)
├── autocomplete-html@0.8.3 (disabled)
├── autocomplete-plus@2.38.0
├── autocomplete-snippets@1.11.2 (disabled)
├── autoflow@0.29.0
├── autosave@0.24.6
├── background-tips@0.27.1
├── bookmarks@0.44.4
├── bracket-matcher@0.88.0
├── command-palette@0.42.0
├── dalek@0.2.1
├── deprecation-cop@0.56.9
├── dev-live-reload@0.47.1
├── encoding-selector@0.23.7
├── exception-reporting@0.42.0
├── find-and-replace@0.215.0
├── fuzzy-finder@1.7.3
├── github@0.8.3
├── git-diff@1.3.6
├── go-to-line@0.32.1
├── grammar-selector@0.49.8
├── image-view@0.62.4
├── incompatible-packages@0.27.3
├── keybinding-resolver@0.38.1
├── line-ending-selector@0.7.4
├── link@0.31.3
├── markdown-preview@0.159.18
├── metrics@1.2.6
├── notifications@0.69.2
├── open-on-github@1.3.1
├── package-generator@1.1.1
├── settings-view@0.253.0
├── snippets@1.1.9
├── spell-check@0.72.3
├── status-bar@1.8.14
├── styleguide@0.49.8
├── symbols-view@0.118.1
├── tabs@0.109.1
├── timecop@0.36.0
├── tree-view@0.221.3
├── update-package-dependencies@0.12.0
├── welcome@0.36.6
├── whitespace@0.37.5
├── wrap-guide@0.40.2
├── language-c@0.58.1
├── language-clojure@0.22.4
├── language-coffee-script@0.49.3
├── language-csharp@0.14.3
├── language-css@0.42.7
├── language-gfm@0.90.2
├── language-git@0.19.1
├── language-go@0.44.3
├── language-html@0.48.4
├── language-hyperlink@0.16.3
├── language-java@0.27.6
├── language-javascript@0.127.6
├── language-json@0.19.1
├── language-less@0.33.0
├── language-make@0.22.3
├── language-mustache@0.14.4
├── language-objective-c@0.15.1
├── language-perl@0.38.1
├── language-php@0.42.2
├── language-property-list@0.9.1
├── language-python@0.45.5
├── language-ruby@0.71.4
├── language-ruby-on-rails@0.25.2
├── language-sass@0.61.1
├── language-shellscript@0.25.4
├── language-source@0.9.0
├── language-sql@0.25.8
├── language-text@0.7.3
├── language-todo@0.29.3
├── language-toml@0.18.1
├── language-typescript@0.2.2
├── language-xml@0.35.2
└── language-yaml@0.31.1
Community Packages (77) /home/justin.barca/.atom/packages
├── atom-css-unit-converter@1.1.1
├── atom-ide-ui@0.11.2
├── atom-import-js@0.15.0
├── atom-import-sort@4.2.0
├── atom-monokai@0.10.9
├── atom-monokai-dark@0.4.0
├── atom-quokka@1.0.73
├── atom-terminal-panel@4.4.4 (disabled)
├── atom-ternjs@0.18.3
├── autocomplete@0.47.0
├── bottom-dock@0.4.4
├── busy-signal@1.4.3
├── change-case@0.6.5
├── city-lights-ui@1.3.3
├── color-picker@2.3.0
├── css-hyperclick@0.1.5
├── docblockr@0.13.7
├── emmet@2.4.3
├── es6-javascript@1.0.0
├── expand-region@0.4.0 (disabled)
├── expand-selection-to-quotes@0.11.0
├── extract-selection@0.4.0
├── file-type-icons@1.3.4
├── fix-indent-on-paste@0.1.1
├── formatter-json@1.0.1
├── from-and-to-json@1.0.0
├── git-difftool@0.2.7
├── gitlab@0.1.3-7
├── gitlab-integration@0.4.7
├── goto-definition@1.3.4
├── highlight-selected@0.13.1
├── hyperclick@0.1.5
├── ide-css@0.3.0
├── ide-html@0.4.2
├── ide-json@0.2.1
├── ide-typescript@0.7.6
├── intentions@1.1.5
├── js-hyperclick@1.13.0 (disabled)
├── jump-to-import@3.6.5 (disabled)
├── language-babel@2.85.0
├── language-javascript-plus@1.9.3
├── language-vue@0.23.1
├── language-vue-component@0.5.0
├── linter@2.2.0 (disabled)
├── linter-coffeelint@2.0.0
├── linter-eslint@8.4.1
├── linter-flake8@2.3.0
├── linter-jshint@3.1.8
├── linter-json-lint@0.1.3
├── linter-ui-default@1.7.1
├── minimap@4.29.8
├── minimap-highlight-selected@4.6.1
├── minimap-linter@2.1.3
├── monokai-dark@1.5.0
├── monokai-seti@0.9.0
├── monokai-shade@0.4.0
├── monokai-slate@0.6.0
├── numix-ui@0.2.3
├── organize-imports@1.0.0
├── pigments@0.40.2
├── platformio-ide-terminal@2.8.1 (disabled)
├── prettier-atom@0.53.0
├── pretty-json@2.0.0
├── project-manager@3.3.5
├── seti-ui@1.10.0
├── snippet-injector@1.5.3
├── sorter@1.1.0
├── sublime-block-comment@0.5.1
├── swap-selection@0.4.0
├── text-pastry@0.6.1
├── todo-show@2.3.0
├── toggle@0.6.2
├── toggle-quotes@1.1.0
├── uuidgen@1.2.1
├── vue2-autocomplete@0.1.0-alpha.4 (disabled)
├── vuejs2-snippets@1.1.0
└── xml-formatter@0.11.0

I'll have a closer look on this later. But on first sight it seems that something changed in the notification api.

👍 Thanks
On Thu, Jun 21, 2018, 2:17 PM Maximilian Schmidt, notifications@github.com
wrote:

I'll have a closer look on this later. But on first sight it seems that
something changed in the notification api.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MCStreetguy/atom-snippet-injector/issues/18#issuecomment-399196877,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfQTiQEw_GkoYZ_2v7RJMw4YWzo-Y86Bks5t--MwgaJpZM4Uydef
.

Actually I can't reproduce your problem. For me injecting snippets just works fine and all notifications pop up as expected. Could you update to the latest version (v1.29.0) and have a look if that fixes the error?

authors: MCStreetguy justin-barca

repo: MCStreetguy/atom-snippet-injector


text:
[REQUEST] Forge Biome Dictionary support

I have noticed that TerrainControl doesn't use Forge Biome Dictionary and this causes crashes. At least Mo' Creatures and Roguelike Dungeons causes crash and server restart because of a NullPointerException thrown.
Adding support for this would be superior, I love to use Biome Bundle but at this moment I can't use it because of those crashes.
This issue is strongly related with #500
EDIT:
Tried to do this for myself but didn't achieve anything.

@LiskoSlayer63 @Timethor Actually I managed to"fix" (it isn't a fix) this but with a very vague manner. What I did was that I added a try-catch straight in to Forges source code... It isn't crashing anymore but it also doesn't work like it should. Can you please consider adding the BiomeDictionary support?

@bloodmc Is there any further information about this?

For now I am using a work around with the mod BiomeTweaker to add Dictionary support to biomes:
This seems to work, but still need to do more testing! Create a script for biometweaker and especially put in the first line otherwise, you aren't changing anything at all as you terraincontrol only starts adding it's biomes on server start and not sooner. Just start Minecraft client and look at the Biometweaker output files: terraincontrol biomes are missing.
Once you start a world with Terraincontrol active and use the command /btoutput you will see now the Terraincontrol biomes are also having a file.
Configscript e.g.
Tweaker.setStage("SERVER_STARTING") //This line needs to be in!!! Later lines need to tweak what you want! For now I strongly suggest you only change Dic entries and nothing else!
currentBiome = forBiomes(50)
currentBiome.removeDicType("OCEAN")
currentBiome.addDicType("BEACH")
Biometweaker can only handle the lower 256 biomes, so all other Terraincontrol biomes will need to extend those! I hope ForgeDic support is implemented soon, so I can leave out BiomeTweaker as the more mods influencing the same stuff, the more likely something will break.
Guinaro

After playing with terraincontrol settings some more, my previous workaround: doesn't really do anything much. You do have influence on the biomedic entries, but it doesn't prevent any crashes related to things like 'updating neighbours' crashes, .... There is still a serious incompatibility with mods that like to add their own features to the terrain. For now, my only solution will be: implement the features of the mod I want regarding terrain generation myself: plants, BO3s, ... . Most mods offer control over those features, but not all. Biomes O' Plenty is such a mod. In the 1.10.2 version you cannot disable anything via config files: so the moment you enable this mod, you don't need to explore much at all, if you even succeed in generating a new world before crashing.
Terraincontrol is a superb mod: but unfortunately it is seriously handicapped in playing nice with other mods. But let's keep the faith that those will get resolved.

The Forge version is more or less taken over by @PG85 - see his fork called OpenTerrainGen. I don't think this will be fixed by me.

source: gharchive/issue

authors: Guinaro LiskoSlayer63 rutgerkok

repo: MCTCP/TerrainControl

license: mit

license_type: permissive

license_source: bigquery


text:
Added URLs for external hyperlinks in the docs

Generated documentation had broken links. For example, `cookiecutter`_ was visible rather than a proper link.

This won't be fixed until we address #56

@ianmkenney is this superseded by #57?

Closed as stale -- @ianmkenney please reopen if you think it's needed.

source: gharchive/pull-request

authors: ianmkenney lilyminium

repo: MDAnalysis/cookiecutter-mdakit

license: MIT

license_source: github-api


text:
Query regarding the body of STOW-RS request

I was trying to parse the byte array send by store request using this client and I am getting an EOF error of reading. I checked the content from the request body and reading the test dicom file directly and found some difference at the beginning.

content from http post request

content from reading file directly

What is this difference at the begnning? I am trying to pass the POST request body directly to a parsing function like this

The request message has multipart content type. Therefore, the message body contains one or more DICOM data sets and encapsulation boundaries.

See DICOM PS 3.18 Section 8.6

authors: hackermd lafith


text:
Seemingly wrong synchronize scaling handling

In the following code, it seems that when np.random.uniform() < self.p_synchronize_scaling_across_axes, the scaling should be synchronized across axes.
https://github.com/MIC-DKFZ/batchgeneratorsv2/blob/c4f7b8d43bef7e70f30ad7a3de39f2b46c795e56/batchgeneratorsv2/transforms/spatial/spatial.py#L59-L61

Dear @function2-llx ,
good catch! Thank you for the bug report, we have created a Pull Request to fix it :)
Best,
Michael


text:
Implementation of decoder for 'C' instructions

Hello, I tried to add some check for supported base type. This is to understand whether the instruction, for example, is supported by RV32C or not. I want to know if you accept this idea of mine or not. Also I add entries for c_addiw and compressed load/store of 64-bit values.

Please merge your branch with the latest master to start CI procedure.

1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:30: FAILED:
1:   CHECK( RISCVInstr<uint32> (0xb001).get_disasm() == "c_j -2048" )
1: with expansion:
1:   "c_j, -2048" == "c_j -2048"

OK, I agree we need an additional print type for that. Could you please call it RELATIVE_JUMP?
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:31: FAILED:
1:   CHECK( RISCVInstr<uint32> (0x2ffd).get_disasm() == "c_jal 2046" )
1: with expansion:
1:   "c_jal, 2046" == "c_jal 2046"

Same here
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:37: FAILED:
1:   CHECK( RISCVInstr<uint32> (0x6405).get_disasm() == "c_lui $s0, 1" )
1: with expansion:
1:   "c_lui $s0, 0x1" == "c_lui $s0, 1"

The test is not accurate here — we're using hexadecimal prints for LUI of MIPS and RISC-V.
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:38: FAILED:
1:   CHECK( RISCVInstr<uint32> (0x647d).get_disasm() == "c_lui $s0, 31" )
1: with expansion:
1:   "c_lui $s0, 0x1f" == "c_lui $s0, 31"

Same here
1: -------------------------------------------------------------------------------
1: RISCV invalid instruction
1: -------------------------------------------------------------------------------
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:61
1: ...............................................................................
1:
1: C:/projects/mipt-mips/simulator/risc_v/t/unit_test.cpp:63: FAILED:
1:   CHECK( RISCVInstr<uint32>(0x0).get_disasm() == "unknown" )
1: with expansion:
1:   "c_addi4spn $s0, $sp, 0" == "unknown"

That's wrong. In RISC-V ISA, page 79, is stated that:

A 16-bit instruction with all bits zero is permanently reserved as an illegal instruction.

Please pay attention that builds with Clang and MS lead to segmentation faults and weird outputs.

Almost perfect.
These functions are not covered by unit tests:
constexpr uint32 get_C_SQSP_immediate() const noexcept;
constexpr uint32 get_C_LQSP_immediate() const noexcept;

Additionally, your code to check instruction by ISA bit width is not called for RV64 and RV128 cases. Could you please mark some of proper instructions with these identifiers?

File "riscv-opcodes/opcodes-rvc-pseudo" should contain a field for c_lqsp, but I don't have permission to edit this file. That's why I temporary add instr_c_lqsp in "riscv_instr.cpp". What do you think?

File "riscv-opcodes/opcodes-rvc-pseudo" should contain a field for c_lqsp, but I don't have permission to edit this file. That's why I temporary add instr_c_lqsp in "riscv_instr.cpp". What do you think?

Open a pull request to their repository: https://github.com/riscv/riscv-opcodes/blob/master/opcodes-rvc-pseudo. That's a good opportunity to contribute to RISC-V community!

(however, please keep the temporal solution for now)

authors: BEANefiT pavelkryukov

repo: MIPT-ILab/mipt-mips


text:
mimic-iv website

It looks to me the mimiciv website was rolled back to an older version?

Looks like it's fixed!

repo: MIT-LCP/mimic-iv


text:
GncOptimizer.h: No such file or directory

When running "catkin build", it occurs an error about Kimera-RPGO:
catkin_ws/src/Kimera-RPGO/src/RobustSolver.cpp:16:10: fatal error: gtsam/nonlinear/GncOptimizer.h: No such file or directory
It seems to be "gtsam" version problem, but I have no ideas to solve this.
How can I fix this?
My "gtsam" version is 4.1.0, using Ubuntu20.04 and Ros-Noetic.

Hi Jeff,
You need to checkout to a specific version of gtsam, if I remember it correctly, the name of the branch is related with gncimprovements
BR

Hi Kanster,
The gtsam branch relevant to gncimprovements can not been find. Could you please show more infomation for the right gtsam version?
By the way mine is 4.0.3, using Ubuntu18. This problem also occurs when I was installing RPGO.
Could you help me?
Thanks very much.

authors: Z-Jeff kanster zhuoyuan00

repo: MIT-SPARK/Kimera-VIO


text:
Avoid collapsing 'Properties' view when editing child-properties (e.g. "material.something")

With the current logic, QmitkPropertyItemModelwill rebuild itself whenever the
propertylist changes. This will in consequence collapse a tree view
that displays the model.
That behavior is not nice but can be tolerated when something else in
the application triggers a modification of the propertylist. When the
user modifies a child-item via the 'Properties' view then this
collapsing is very annoying, though.
I couldn't find any negative side-effects of this removal.
Signed-off-by: Daniel Maleike daniel.maleike@stryker.com

Only works the first time a property is changed. From the second time on, stuff is collapsing again. :/

Only works the first time a property is changed. From the second time on, stuff is collapsing again. :/

oups, this is bad testing on my part. I was basically checking multiple toggles on the same bool property and that seemed to work. I need to have a deeper look. I'd appreciate if you keep me updated when someone has a fix for this (behavior should bother some people I guess)

See https://phabricator.mitk.org/T26656. :/

authors: kislinsk maleike

repo: MITK/MITK


text:
pkg/diagnostic output for 2D and 3D variables

There have been two recent emails on the support list that stemmed from users including 2D and 3D variables in the same output stream (this one and this one).
pkg/diagnostics doesn't support this (only the upper level of the 3D variable is saved to disk), but the manual does not warn users of this limitation.
The manual should be updated to explicitly warn users against combining 2D and 3D variables in the same output stream.

explicit now in recent baroclinic gyre tutorial. Package Diagnostics section in outp packages chapter is in need of a major overhaul, should be mentioned there too.

authors: edoddridge jrscott

repo: MITgcm/MITgcm


text:
🛑 F&M Transfer is down

In 498862e, F&M Transfer (https://www.fmtransfer.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: F&M Transfer is back up in 55b6a56 after 15 minutes.

repo: MJTSoftware/MJT-Uptime


text:
Transitions

Is your feature request related to a problem? Please describe.
Need to connect the game a little more with the storyline throughout the game.
Describe the solution you'd like
Add transitions blurbs that explain the next quest and enemy.

Finished

repo: MLH-Fellowship/fantasy-game


text:
Enable Smooth Scrolling for the Website

Currently when the navbar links are clicked, the scrolling isn't very neat. It seems very sudden. Adding smooth scrolling could fix that.

I am currently working on this since it's a very minor fix.


text:
Add contributors in the README.md

RT

@all-contributors please add @huangyz0918 for infrastructure, tests and code

@all-contributors please add @HuaizhengZhang for content, tests and doc

@all-contributors please add @YuanmingLeee for tests and code

repo: MLSysOps/ALaaS


text:
custom club tag is not working

the custom club tag is not working. I tried to enter some custom names in the club tag input box but the club tag is still not changing.

You have to enter club data not the actual club name itself

what does it mean by the club data

Contact me on discord

authors: Bowloe MManoah

repo: MManoah/league-profile-tool


text:
Improvements

We should sync updates made on the lanekeeping and GXZ_kin branches.  This would primarily be done by modifying the parse_rosbag scripts to capture the road frame and curvature information into the mat file.  And then by adjusting the messages / main controller code to handle variation of the controller used.
Also, may be a good idea to remove unneeded elements from the analysis folder and/or add a README for future improvement.

Speaking of improvements, I am curious why you switch from julia/JuMP to python/CASADI? I am also curious how the performance stacks up?
Thanks and great work!

So there were a few reasons to move away from JuMP:
(1) Initially I used RobotOS.jl but it was slow (I think it needs to compile messages every time you run a node).
(2) I had precompilation working for the MPC module in Julia but it may still need a couple seconds to start up.
(3) I tried to use pyjulia to remove the requirement of having a main script be in Julia, so only the MPC portion could be written in Julia.  This is tricky since I could only get this working smoothly with specific versions of Julia and Python.
(4) Casadi provides a similar interface to JuMP if you use the opti interface and starts almost immediately.  I think, under the hood, it's basically a wrapper to Ipopt in C++ so the speeds are pretty good.
I found that, with a warm start, Casadi/Ipopt can find solutions at 50 Hz.  Earlier, with JuMP/Ipopt, I could only get max 20 Hz, although it's likely I wasn't properly warm starting (just relied on whatever is cached when you call the solve function in JuMP).  This hasn't been tested though with difficult optimization problems (e.g. collision avoidance); in this repo, usually the warm start is close to optimal since the reference path changes slowly.
So mostly changed due to simplicity, speed of start up with ROS/Python, and the fact that I don't need an additional install/configuration of Julia.  Just need to do pip install casadi and that's it.

Thank you very much for going into the details of this.
I definitely know the struggle of setting things up with Julia/RobotOS/Python and ROS, possible, but not fun.
Wow, there is a huge difference in speed!

to stop printing all the messages, try
		p_opts = {"expand": True, "print_time":False}


text:
Update/ui_v2

Large UI overhaul

removes the right prompts menu and adds it as a tab to the left sidebar
removes all settings on the left sidebar and turns it into a single settings menu
changes Settings gear icon to circle with initials if signed in user has user data
fixes sidebar in light mode theme (no longer black)
moves Temperature to global setting instead of per chat
moves System prompt to global setting instead of per chat (currently disabled since requires backend changes)
adds a reset settings button
makes dark theme slightly darker
Modifies intro on confirmation that app received model data (typewriter effect/fade in)
adds info icon/text box for model info
fixes placeholder text for message input on mobile devices (no wrap - shorter placeholder)
reduces image icon size for faster load times (azure logo)

Notes:

tried to update translations but there's actually a lot of missing translation info already. Maybe tackle that all together in the future.

looks like margins messed up with sidebar buttons when comparing with prod. Will fix that tomorrow but everything else should be ready for testing.

@mitcheman @blaze-burgess-msf so far all primary functionalities are up and running for me; folder creation is smooth and the UI looks clean
there are two things I'll mention:

In the event that a user enters a prompt and stops the prompt generation via the 'Stop Generating' button, sometimes the 'Regenerate response' action is disabled (edge case but this could still happen during regular interactions, video below)

https://github.com/MSF-USA/chatbot-ui/assets/119887378/b875e719-c309-404b-b748-ecb21ebbd21a

If a user wishes to change the model temperature mid conversation, they have to scroll to the top of the page which can be troublesome in long conversations. This should perhaps end up being a pop up modal, or end up having some autoscroll functionality. Nice to have but not necessary- just provides some ease.

authors: Chris-Graham-MSF mitcheman

repo: MSF-USA/chatbot-ui


text:
Should we create a research.md?

It seems like a good idea to create a space for tracking relevant research based on the modules presented.
It may be that the research represents further exploration of the topics that are discussed and might give ideas as to how the technology can be used.
Thus, I'm proposing something like, "Research.MD" where you can add that and relate it in the course content.
I'm currently looking at the IoT module and a few examples of research look good to add.

i feel this might be a good idea!

Hmmm - neat idea! If you're still excited about it, feel free to fork the repo and make a PR 👍

repo: MSFTImagine/computerscience


text:
CMake generated WindowsStore 8.0 ARM core module missing ole32.lib

How to reproduce:
setup_winrt.bat ws 8.0 arm
build OpenCV.sln in bin/WS/8.0/ARM
core module generates the following error because linker input is missing ole32.lib.
1>------ Build started: Project: opencv_core, Configuration: Debug ARM ------
1>LINK : warning LNK4075: ignoring '/INCREMENTAL' due to '/OPT:ICF' specification
1>     Creating library D:/GitHub/opencv/bin/WS/8.0/arm/lib/Debug/opencv_core300d.lib and object D:/GitHub/opencv/bin/WS/8.0/arm/lib/Debug/opencv_core300d.exp
1>system.obj : error LNK2019: unresolved external symbol __imp_CoCreateGuid referenced in function __unwind$4
1>D:\GitHub\opencv\bin\WS\8.0\arm\bin\Debug\opencv_core300d.dll : fatal error LNK1120: 1 unresolved externals
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========

On it. Preliminary investigation indicates that this might be a cmake platfrom configuration issue:
 179   if(MSVC_C_ARCHITECTURE_ID STREQUAL ARM OR MSVC_CXX_ARCHITECTURE_ID STREQUAL ARM)
 180     set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib")
 181   elseif(MSVC_VERSION GREATER 1310)
 182     set(_RTC1 "/RTC1")
 183     set(_FLAGS_CXX " /GR /EHsc")
 184     set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib")
 185   else()
 186     set(_RTC1 "/GZ")
 187     set(_FLAGS_CXX " /GR /GX")
 188     set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib uuid.lib odbc32.lib odbccp32.lib")
 189   endif()

You should probably check with Gilles.
Thanks,
Dale

From: Max Kostin notifications@github.com
Sent: Wednesday, January 21, 2015 6:04 AM
To: MSOpenTech/opencv
Cc: Dale Stammen (MS OPEN TECH)
Subject: Re: [opencv] CMake generated WindowsStore 8.0 ARM core module missing ole32.lib (#12)
On it. Preliminary investigation indicates that this might be a cmake platfrom configurationhttp://www.cmake.org/gitweb?p=cmake.git;a=blob;f=Modules/Platform/Windows-MSVC.cmake;hb=v3.1.0#l179 issue:
179   if(MSVC_C_ARCHITECTURE_ID STREQUAL ARM OR MSVC_CXX_ARCHITECTURE_ID STREQUAL ARM)
180     set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib")
181   elseif(MSVC_VERSION GREATER 1310)
182     set(_RTC1 "/RTC1")
183     set(_FLAGS_CXX " /GR /EHsc")
184     set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib")
185   else()
186     set(_RTC1 "/GZ")
187     set(_FLAGS_CXX " /GR /GX")
188     set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib uuid.lib odbc32.lib odbccp32.lib")
189   endif()

Reply to this email directly or view it on GitHubhttps://github.com/MSOpenTech/opencv/issues/12#issuecomment-70841842.

Adding ole32.lib (line 172) in the Windows-MSVC.cmake configuration file shipped with cmake installations fixes the issue.
<path-to-cmake-installation-dir>\share\cmake-3.1\Modules\Platform\Windows-MSVC.cmake
- set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib")
+ set(CMAKE_C_STANDARD_LIBRARIES_INIT "kernel32.lib user32.lib ole32.lib")

Regenerating OCV projects with
cmake
  -G Visual Studio 12 2013 ARM
  -DCMAKE_SYSTEM_NAME:String=WindowsStore
  -DCMAKE_SYSTEM_VERSION:String=8.0
  <path-to-cmake-src>

or alternatively using wrapper script (ends up with the same commant)
<path-to-cmake-src>/platfroms/winrt/setup_winrt.bat WS 8.0 ARM

makes it build successfully.

CC Gilles @khouzam

I added the missing ole32.lib for WindowsStore 8.0 in modules/core/CMakeLists.txt.

Thanks. I'll see to get it add to CMake

referencing commit: 5f040ca91e013e2db1c65cae31d34c85cf446359

authors: khouzam mkostin stammen

repo: MSOpenTech/opencv


text:
Contribute videoio maintenance update

A few refactoring changes simplifying inner structure of videoio for WinRT bits, comment improvements and overall code cleanup.
Related PR: #4047

Merged, closing.


text:
[BUG] Chunk with startRow skips rows!

Package version, Laravel version
Package version: 2.1.17
Laravel version: 5.4.21
Expected behaviour
Variables:

Excel sheet with 30 rows
Chunk size of 10
import.startRow in the config file set to 3

Expectation: All the rows to be read without skipping anything.
Actual behaviour
12th, 13th, 22nd, 23rd rows are skipped and not read at all!
Code
Config::set('excel.import.startRow', 3);

Excel::filter('chunk')
         ->selectSheets('Sheet 1')
         ->load($filePath)
         ->chunk(
                    10,
                    function ($results)
                    {
                        foreach ($results as $row)
                        {
                            var_dump($row);
                        }
                    },
                    false
                );

Screenshot of Excel file

Steps to reproduce the behaviour

Create an excel sheet with the third row having headers.
Have around 30 rows in the excel sheet.
Set the startRow to 3.
Read the excel sheet using chunks with chunk size of 10.
Notice that 12th, 13th, 22nd and 23rd rows are skipped.

Just bumping this up!
@patrickbrouwers Can you help?

This is not something we will fix in 2.1 anymore. I would suggest handling the row skipping yourself.

@patrickbrouwers how do you skip 2 rows programmatically?

I have this issue. Someone please solve thanks!

authors: StSarc kennblvnp patrickbrouwers

repo: Maatwebsite/Laravel-Excel


text:
[BUG] Data Validation breaks Excel spreadsheet

Hi,
I'm using package v2.1.* (according to the README.md file in my vendor folder) and Laravel 5.1. I'm generating a Excel spreadsheet and I want a drop down list for some columns. So, I have the following code that runs for 1000 cells in the same column. This code works fine when I open the spreadsheet using LibreOffice. However, when I open it on Excel, I get the following error: "We found a problem with some content in 'filename'. Do you want us to try to recover as much as we can?"
Here's a screenshot of the recovered spreadsheet. It doesn't work as expected and the formatting is getting weird.

The code is as follows:
$objValidation = $cell->getDataValidation();
$objValidation->setType(\PHPExcel_Cell_DataValidation::TYPE_LIST);
$objValidation->setErrorStyle(\PHPExcel_Cell_DataValidation::STYLE_INFORMATION);
$objValidation->setShowErrorMessage(true);
$objValidation->setShowDropDown(true);
$objValidation->setErrorTitle($errorTitle);
$objValidation->setError($errorMessage);
$objValidation->setAllowBlank(false);
$objValidation->setShowInputMessage(true);
$objValidation->setPromptTitle($promptTitle);
$objValidation->setPrompt($prompt);
$objValidation->setFormula1($options);

The variable $options is a comma separated string (e.g., '"value 1","value 2","value 3"').

As you are using native PHPExcel methods, you can better open an issue over there.

Done.

authors: demontiejr patrickbrouwers


text:
RAD weird behaviour with program switch and delay/timer functions

RAD board uploads code properly when programming switch is in the non-grey-line position. However, the program will sometimes crash as soon as it reaches a delay function.
RAD board will not appear as a device when uploading with programming switch in the grey-line position.

Resolution: Pinout and configuration > system core > sys > Debug --> set to "Serial Wire". This will allow full program functionality while in programming mode. Do not attempt to program the chip with the switch in grey-line position as it will not upload properly.

repo: MacRover/maxwell


text:
Still receiving ERROR: triton-2.1.0-cp310-cp310-win_amd64.whl is not a supported wheel on this platform.

Sorry to open a new ticket on this, I'm not very familiar with GitHub and wasn't sure if I should have replied to the old thread. Anyway...
I am receiving the "ERROR: triton-2.1.0-cp310-cp310-win_amd64.whl is not a supported wheel on this platform."
I have tried all the fixes from your previous post a few months ago, but nothing seems to have any effect. I am running Windows 11, Python 3.13.1, and Cuda 12.6.3.
I have tried manually downloading the Triton wheel from your GitHub, but that doesn't seem to make any difference, either.
Not sure if it matters, but I also receive the two following errors (but they don't cause the one_click_install to bomb):
ERROR: Could not find a version that satisfies the requirement torch==2.3.0+cu121 (from versions: none)
ERROR: No matching distribution found for torch==2.3.0+cu121

You're running python==3.13.1 & there currently isn't a triton wheel that works w/ that version of python - https://huggingface.co/MackinationsAi/site-packages/tree/main/triton. If you are able to get the gradio up & running, triton is not strictly necessary for this repo to work though, it helps optimize xformers but you should be fine w/o it. If you do run into any issues getting everything going then let me know here, I am going to close this issue ticket on Monday if I don't here back from you by then.

I downgraded my Python to 3.10.11 and BOOM! Everything works like a charm. Cannot thank you enough! Consider this closed.


text:
🛑 Shields.io is down

In 8be6f64, Shields.io (https://shields.io) was down:

HTTP code: 503
Response time: 266 ms

Resolved: Shields.io is back up in 2a89b79.

repo: Magic-Services/upptime


text:
🛑 Libreddit (libreddit.spike.codes) is down

In 4b51c42, Libreddit (libreddit.spike.codes) (https://libreddit.spike.codes) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Libreddit (libreddit.spike.codes) is back up in d7e98a4.


text:
🛑 Libreddit (libreddit.spike.codes) is down

In 1cee75a, Libreddit (libreddit.spike.codes) (https://libreddit.spike.codes) was down:

HTTP code: 502
Response time: 28766 ms

Resolved: Libreddit (libreddit.spike.codes) is back up in 2ddcd6e after 1 hour, 6 minutes.


text:
🛑 Webblate is down

In e346dff, Webblate (https://weblate.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Webblate is back up in f4a16f7 after 10 minutes.


text:
🛑 MagicLike (old) is down

In 2dc961b, MagicLike (old) (https://magiclike.codeberg.page) was down:

HTTP code: 404
Response time: 12826 ms

Resolved: MagicLike (old) is back up in e5f3028 after 31 minutes.


text:
The performance of the code

Recently I have ran the SimulatedExample.m, I found the result is not as good as that in your paper.
Figure 1: the result of this code ; Figure 2 : the result of the paper.
Could your tell me how to set the parameters of this package to improve the performance.
Figure 1:

Figure 2:

@TFLQW  Thanks for your interest. First of all, as I mentioned in the code:
% Example code for multi-output prediction using some simulated data (It is
% not the code for the following paper, there are some slight differences)

Also, you can find I mentioned in the paper:
The ARMSEs for both training and test points in [- 10, 10] are reported in Table 3, and an **example** of
prediction is demonstrated in Fig. 1.

This example code just shows how it works, as similar to the paper's figure. Both of them are examples,
Since one run will highly reply to the random seed setting and other environmental variables, you need to run many times and compute the ARMSE (average root mean square error) or other criteria to evaluate the model performance.

@Magica-Chen Thanks for your reply, what I understand is that, the performance of MV-GPR may depend on the initial parameters setting, like the SE_init，so can you recommand some initialization function which could get a better performance?

@TFLQW Yeah, you're right. Initial hyperparameters may have a huge influence on hyperparameter learning and then the performance of the model.  However, in my previous paper: "How priors of initial hyperparameters affect Gaussian process regression models", I find that even hyperparameter estimation is sensitive to the initial hyperparameters, the model performance is considerable in single GPR model with SE kernel. Here 'considerable performance' means, if you run the model 200 times, and look at the average performance, initial hyperparameters of SE kernel may have less effect on the performance.
But, to be honest, similar research on MV-GPR has not been done, so I cannot draw a conclusion about that, but I think for SE kernel, it is probably fine for the different initials.  Therefore, this is also I just use the common-used way to initialisation.
hyp = rand(n_parameter,1);      % This is the most commonly-used method

For some specific cases, maybe you can take advantage of the some real meanings of your data, like you know something about the  'lengthscale' of data, then you can use more suitable initial prior.

@Magica-Chen Thanks, I got it!

authors: Magica-Chen TFLQW


text:
Removed services duplicated by ports in firewall config

Fixes #108
@bdunne @abellotti

I'd prefer to see the port references go.  Having the named services clearly describes what is open and what service is using it in my opinion.

cc @jrafanie @Fryguy

The services only cover the tcp protocol on the listed ports so I wouldn't be able to just use the services. I could list the services then additionally add the same port with udp if that would be better.

I don't think we actually need the UDP port, do we?

No idea :sweat_smile:

I don't think we actually need the UDP port, do we?

I hope we had a commit message that said why we used it.  I remember needing it for something, but I don't remember

Apache should only be using TCP

Just FYI the ports enabled by a particular service are defined in files in /usr/lib/firewalld/services/

I'd be good with removing UDP from the allowed ports if we run it on an appliance and can browse the UI and hit some rest api endpoints. :fire:

Okay, I'll run a build for this branch.

@carbonin Any more on this PR?

@carbonin Bump?

@carbonin Is this darga yes/no?

@Fryguy it's a no, it definitely isn't a bug.

🎉

authors: Fryguy bdunne carbonin jrafanie

repo: ManageIQ/manageiq-appliance-build


text:
Nightly Upstream builds from 2/25 are empty files for rhevm and rhos.

The nightly upstream builds for 2015/02/25 are empty files for the ovirt and openstack appliance images. The image for vsphere, manageiq-vsphere-master-201502252000-a377dd0f98.ova, is present.

@blrm Thanks.  It looks like the centos OS install failed because it couldn't find:
http://mirror.centos.org/centos/6/os/x86_64/Packages/system-config-network-tui-1.6.0.el6.2-1.el6.noarch.rpm.  This happens every so often where the yum repo is not available.  The next nightly should build since that file exists. I cleaned up the build machine to destroy that failed build.

cc @JPrause @jvlcek @mkanoor

@blrm If it's a problem again, we can re-open.  Thanks for reporting this!

Thanks JoeJoe!
On 2/26/15 4:57 PM, Joe Rafaniello wrote:

@blrm https://github.com/blrm Thanks. It looks like the centos OS
install failed because it couldn't find:
http://mirror.centos.org/centos/6/os/x86_64/Packages/system-config-network-tui-1.6.0.el6.2-1.el6.noarch.rpm.
This happens every so often where the yum repo is not available. The
next nightly should build since that file exists. I cleaned up the
build machine to destroy that failed build.
screen shot 2015-02-26 at 4 53 22 pm
https://cloud.githubusercontent.com/assets/19339/6402745/0455bf42-bdd8-11e4-8c5c-b5ebc6329845.png
cc @JPrause https://github.com/JPrause @jvlcek
https://github.com/jvlcek @mkanoor https://github.com/mkanoor
—
Reply to this email directly or view it on GitHub
https://github.com/ManageIQ/manageiq/issues/1910#issuecomment-76282888.

authors: blrm jrafanie jvlcek

repo: ManageIQ/manageiq


text:
Record Openstack Infra VCRs again

Tried to record Openstack infra VCRs with current master branch.
It might help with travis failures.

This probably won't be needed, as it doesn't seem to fix the issue at hand; I believe @petrblaho may be working on a solution however!

I agree with @tzumainn this PR can be closed most propably.

authors: aufi tzumainn


text:
MiqWidgetSet#find_with_same_order -> with_array_order

@miq-bot assign @kbrock
@miq-bot add_label refactoring

this is a really fun implementation. not sure why we haven't merged it before

authors: kbrock lpichler


text:
Introduce Zone#maintenance?

When I was profiling, I noticed that it was looking up the maintenance Zone when inserting an MiqQueue record.
This is probably a moot point as this will only happen.

But I did like how Zone#maintenance? read.
And I wondered if we wanted to /^__maintenance__/.match?(zone.name) instead of the MiqRegion.my_region.maintenance_zone == zone?
And I did find an issue with the cached Zone#maintenance_zone in one case.

There were a reasonable number of tests on the maintenance_zone concept.

In general I like this PR because I agree it's much cleaner, so let's continue with it.

update:

uncache maintenance_zone (since my_region is cached elsewhere)
s/Zone/described_class/g
updated Zone.maintenance?() to use case

Still wondering if we want to just compare the string with /^__maintenance__/ and not force the maintenance_zone to be loaded at all.

Still wondering if we want to just compare the string with /^maintenance/ and not force the maintenance_zone to be loaded at all.

I recall having this discussion in the original PR, but I can't remember how we got the current code - you should probably check that out and see if there was a reason there.

authors: Fryguy kbrock


text:
Adding OpenShift Enterprise Provider

This is largely copy/paste from adding the Atomic provider (https://github.com/ManageIQ/manageiq/pull/5012).  There were a few things that @simon3z cleaned up in that PR that meant I could avoid touching a few extra files.
@simon3z ... make sure this is correct as far as naming is concerned.  In this case I'm assuming that the current Openshift provider really OpenShift Origin whereas this PR is for OpenShift Enterprise.
If that's the wrong assumption, then it's really not that hard to update this PR to be for OpenShift Origin, but I'll also have to include changes to the current Openshift provider to make it show up as OpenShift Enterprise.
/cc @Fryguy, @chessbyte, @matthewd

@blomquisg any way to rename the current OpenShift provider (only in the combobox) to OpenShift Origin? (I can do it later on if you want)

@serenamarie125 do we have an icon for OpenShift Enterprise?

We want BOTH OpenShift providers (names and icons) in upstream.

@simon3z I figure we can change the description attribute for the existing OpenStack to "OpenShift Origin" ...
There might be some translation files, too?

We want BOTH OpenShift providers (names and icons) in upstream.

That's clear I am asking to rename the current OpenShift to => OpenShift Origin to make clear the distinction with the one of this PR which is OpenShift Enterprise.

@simon3z I figure we can change the description attribute for the existing OpenStack to "OpenShift Origin" ...

@blomquisg I hope you're not going to rename OpenStack :smile:
That should be ok IMO.

@blomquisg I hope you're not going to rename OpenStack ￼

haha, why not?  it'll be sorta like pin the tail on the provider .. which ever provider you accidentally choose is what you get.

LGTM. Do we want to wait for the icon?

For the icon, prefer to copy placeholder.png...then we can merge and @epwinchell can make an icon later.

@blomquisg Looks good...can you update the image to use placeholder.png...you can mark the new commit as [skip ci] if you want to avoid the wrath of Travis.

@Fryguy updated the provider icon to use placeholder

@epwinchell Can you please create an icon?

authors: Fryguy blomquisg chessbyte simon3z


text:
[WIP] Network manager topology fix n plus 1 problem on network port security groups

Fix network_topology n+1 problem
Fix network_topology n+1 problem on network_port -> security_groups

@Fryguy  so will this work for you as a fix of my n+1 problem when using HABTM? I think I will not be able to reproduce and fix the possible rails bug for the release otherwise.
The composite primary_key is not a problem anymore, since @carbonin added the primary key to each table


text:
Update miq_report.rb

Line 60 appears to have a bug that causes the Reports module to not load for ManageIQ. The SQL query should not have a Group By instead it should be a Order by.

@sgpal, @dclarizio @Fryguy What is the status of this old PR?

This looks like a bug. Reason not to just merge?

:+1:

authors: chessbyte kbrock sgpal


text:
MiqOptionParser and all of its classes are no longer used

This code is no longer used and should be removed.
The last callers appear to be removed in:
commit 4cc2f0744f0104dd8150b3429a7e911c58a1466c
Author: Joe Rafaniello jrafanie@redhat.com
Date:   Tue Mar 15 16:40:11 2016 -0400

@jrafanie @Fryguy Please review
@miq-bot add_label technical debt

LGTM

✂ 🔥 ✂ 🔥 ✂ 🔥
🍰  🍪 👏 🙇 😍 🎉

authors: bdunne jrafanie


text:
An alternative way of testing videos

I was just talking to @yoshiask about testing the SVG branch and this came up. I think it's general enough so I'll share with the broader community for comments.
Proposal to test rendering:

Add a shape to the scene, say a circle with radius 1 and centered at the origin. Set the circle's color to pure red (#FF0000)
Render the scene
Get the camer's pixel_array. From that array, get a pixel that is inside the circle.
If the pixel's value is exactly equal to (255, 0, 0), the test passes.

Currently, we are trying to render the scene and output it to disk, and then comparing the output files. This is testing the rendering AS WELL AS the operation of writing the file. In general, we should have separate tests for these two operations.
Note the proposed method can be done with --dry_run, no output files necessary. So we can forget about all of the tests_cache, tests_data, etc.
cc @huguesdevimeux as Tests Czar.

Get the camer's pixel_array. From that array, get a pixel that is inside the circle.
If the pixel's value is exactly equal to (255, 0, 0), the test passes.

Maybe I didn't understood well but this is EXACTLY what is done for the graphical units tests.

Hmmmm ok maybe this is less novel than I thought... but I think there is a major difference: the current tests render a whole scene and they need a cached frame to compare it to. What I'm suggesting is that we also do this on the fly, checking for some properties of some pixels. This doesn't require cached frames.
I guess the real suggestion here is that we should write more tests :sweat_smile: (and not that we came up with a whole new way of doing things). Sorry, I got carried away!

I see lol.
But still, you're suggestion is instersting. Maybe open a new issue :D

authors: huguesdevimeux leotrs

repo: ManimCommunity/manim


text:
Make livestreaming a thing

List of Changes

Change 1

Add a streaming scene that uses its own file writer to override the normal functioning of the original to create a streaming protocol. Then, the original Scene class can use SceneFileWriter as a default, unless a metaclass I created hijacks the original and adds the Streamer as a class attribute, which Scene will check for. And, in order to avoid using a metaclass all the time due to unfamiliarity with most, simply create the base class for all streaming services. You can see there the syntax for the base is kind of odd, meaning you use Stream to acquire the metaclass but the base class is the scene you pass as an argument. I updated streaming_console_banner for the simplified explanation.
class StreamMeta(type):
    """The metaclass kind of 'hijacks' the process of scene instantiation, throwing a
    file writer class which is the adequately created Streamer. I didn't want to have
    any streaming classes inherited from anything other than Scene to have to do this.
    But since not everyone has seen a metaclass before, the Stream class shrouds the
    actual purpose of a Scene that can stream; to use this as a metaclass. The updated
    streaming info should reflect this, if I don't forget.
    """
    def __new__(mcs, name, bases, namespace, scene=Scene, **kwargs):
        namespace["file_writer_class"] = Streamer
        return super().__new__(mcs, name, tuple([scene]), namespace)

class Stream(metaclass=StreamMeta):
    pass

Change 2

Use rtp instead of tcp as the streaming protocol. Mainly for no other reason than it's what I understood and coded in that direction. If you think that means I didn't crack Twitch streaming then you are ... absolutely right. Moving on.

Change 3

There are two versions of the livestreaming I made. Both do the same thing. The one under the function livestream is the main version and opens up an interactive session. The other one, stream, just returns the streamer. Easy to use for casual on the fly testing like:
>>> from manim import stream, stuff, other, stuff
>>> manim = stream()
>>> manim.play(...)

Nevertheless, both are available in the main library, depending on whether you need 500 objects in your global variables or not.

Change 4

Any extras were probably made in order to have this working.
Motivation
Street cred.
Explanation for Changes
I figure that any livestreaming option that was there in the past kind of broke, but the backbone of its capability was continuously carried around like a fossil. So I tried to implement it and it worked.
Testing Status
There's varying amounts of good and bad news here. Let me explain.

Good

Since this Manim can directly influence camera configurations from the command line, this gives you high quality streaming:
python -m manim --livestream
This gives you medium:
python -m manim -m --livestream

Good

It still looks pretty guys. Also, by making file configurations extrinsic you don't create hashed files in the main folder.

Good

Ability to use other base scenes for streaming, granted some weird syntax courtesy of the metaclass.
>>> class AnotherStreamer(Stream, scene=MovingCameraScene):
...     pass
...
>>> manim2 = AnotherStreamer()

Bad

I never really explained how I open the streaming protocol. The protocol needs an sdp file to work so after every stream I refresh and create one in the media\streams folder. To open the sdp in ffplay, along with some other quirks, I run this command in a batch file:
ffplay -x 640 -y 360 -protocol_whitelist "file,rtp,udp" -i "path\to\manim\media\streams\streams.sdp" -reorder_queue_size 0

Bad

ffplay might glitch and show you a lot of red for some reason. Reopening it should fix one of those problems.

Bad

Ain't touched UNIX with this setting so I don't know how it goes over there. I think I tried to use os.path for most of the slashes, but if there's an error just reverse them.

Bad

Was I supposed to update the documentation for this?

Good

If it works though, expect something like this:

Further Comments
Really just syntax errors in space_ops.py and geometry
Acknowledgement

[ ] I have read the Contributing Guidelines*
[ ] I quote these fellas #441 so they know I did this.

*half true

Hi, thanks for the PR !
But I think you messed up some git stuff, as the merge conflicts and the commtis can witness.
Maybe could you hop on discord so we can help you? ~~You won't lose any street cred. ~~

Hi, thanks for the PR !
But I think you messed up some git stuff, as the merge conflicts and the commtis can witness. Thiss makes review a little hard.
Maybe could you hop on discord so we can help you? ~You won't lose any street cred.~

Alrighty then.

There are two main problems here:

You have applied your changes to a relatively old branch (GeometryAdditions instead of master)
If I understand your workflow correctly, you downloaded a ZIP file containing Manim's source code (use https://github.com/ManimCommunity/manim/archive/master.zip for the most recent version, if you dislike working with git directly), then modified these, and then you uploaded the changed files via GitHub's web interface?

The problem with your workflow in this case is that you are probably using a different line ending setting or something like that in your local files: if you look at https://github.com/ManimCommunity/manim/pull/773/commits/24be6fee0c35f5ce7d634df71b76e5133ef6ae3a it seems as if you changed every single line with your upload, which makes it basically impossible for us to determine what you actually changed.
(I see that, while I am writing this, you already hopped onto Discord, we'll help you to figure out stuff over there. And of course: thank you for your efforts so far!)

authors: NeoPlato behackl huguesdevimeux


text:
New feature: change_layout method for Graph mobject

Motivation

Improvement of our implementation of Graphs: allow changing the layout of graphs as well as switching to a different automatic layout.
Overview / Explanation for Changes

This PR factors out the code responsible for generating the graph layout to a private function, allowing to reuse it in a new class method change_layout.
Oneline Summary of Changes

- New feature: change_layout method for Graph mobject (:pr:`PR NUMBER HERE`)

Testing Status

Example added in the documentation, builds locally and looks correct.
Further Comments

Graphs (and updaters in general?) seem to suffer from the problem that there seems to be lag of precisely one frame, which makes the edges move behind the vertices a tiny bit. This should be investigated in general; I've already looked into it a bit but can't really tell where this is coming from.
Acknowledgements

[x] I have read the Contributing Guidelines

Reviewer Checklist

[ ] Newly added functions/classes are either private or have a docstring
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings
[ ] The oneline summary has been included in the wiki

change_layout in action: https://twitter.com/behackl/status/1351315710358790145

change_layout in action: https://twitter.com/behackl/status/1351315710358790145


text:
make idle timeout configurable

[sway]idle timeout is currently hardcoded to 240 seconds, any thoughts yet how to make that configurable?

sure, good idea. should work just like this:
set $idle_seconds 240
set $idle swayidle -w \
    timeout $idle_seconds 'light -G > /tmp/brightness && light -S 10' resume 'light -S $([ -f /tmp/brightness ] && cat /tmp/brightness || echo 100%)' \
    timeout 300 'exec $locking' \
    timeout 600 'swaymsg "output * dpms off"' \
    resume 'swaymsg "output * dpms on"' \
    before-sleep 'playerctl pause' \
    before-sleep 'exec $locking'

so you can put set $idle_seconds 60 into a user config after that. Perhaps, while we're at it, we could do the same for the locking timeout and dmps-timeout?

please install manjaro-sway-settings-git and take this on a test ride.

authors: boredland eterps

repo: Manjaro-Sway/manjaro-sway


text:
Development: Add instructions on how to setup development environment

Description
We need to add instructions on how to setup development environment locally to a separate document.

Fixed in afde5dff9aadc5710f6ae649e0ef7ebf1a312e5a

repo: ManrajGrover/HackerEarth-CLI


text:
Making class code only visible for instructor.

Please make the class code only visible to teacher/instructor
In a class card in class code is accessible by everyone who join the class, please make it visible for only class instructor

In the server, I can add an extra key as a role for the instructor and students, then on the front-end, We can easily validate that one, and as per that, we can render.

Hello, @Rishi-121 are you still working on this issue ?
Please create a pull request if you need any help, or if you are not interested then we can assign this issues to someone else.
This issue will be reassigned to you, whenever you create a pull request regarding this issue.

Hello, @Rishi-121 are you still working on this issue ?
Please create a pull request if you need any help, or if you are not interested then we can assign this issues to someone else.
This issue will be reassigned to you, whenever you create a pull request regarding this issue.

Hey, Can I work on this?

Yes, I am assigning this you @deepthiteki.

authors: Rishi-121 aavishkarmishra deepthiteki


text:
grunt watch

In Readme grunt watch is only for dev mode ?

Yes, today the watch task is only about copying some files and building css and templates. All in the src directory.
But in a close future, Grunt will be replaced by webpack.

It's not clear in the Readme what is for dev or not.

authors: GuillaumeAmat frodrigo

repo: MapContrib/MapContrib


text:
Expose the MapsIndoors Instance so it can be used from the outside

What

Expose the MapsIndoors Instance so it can be used from the outside

How

Refactor the Map.jsx component to be using the useImperativeHandle
Return the mapsIndoorsInstance
Use the ref to assign the mapsIndoorsInstance to the window so it can be accessible from the outside

Ready for re-review @ammapspeople

I get this error when running as React app:

And I get somewhat same error when running the test app for the built React component:

Oops, missed a check 👀 Pushing the fix now @ammapspeople

Oops, missed a check 👀 Pushing the fix now @ammapspeople

How do I get the instance when using the built React component? 🤔

@ammapspeople updated the implementation to expose the MapsIndoors Instance on the window instead.

Hmm, I could add a section to the README file, do you think that would be sufficient? @ammapspeople

Hmm, I could add a section to the README file, do you think that would be sufficient? @ammapspeople

@andreeaceachir142 That might work!

Updated README files @ammapspeople

@ammapspeople The docs on GitBook has been updated with the new example for polygonFillColor as well. ✅

repo: MapsPeople/web-ui


text:
temp key is not created for many-to-many table

i filed this one with the EFCore team, but they are putting it back on Mapster. Here's my setup:
TypeAdapterConfig<ServiceAreaDTO, ServiceArea>
    .NewConfig()
    .Map(dest => dest.ServiceAreaToZipCodes,
            src => src.ZipCodes)
        .Ignore(dest => dest.AffiliateId)
        .Ignore(dest => dest.Affiliate);

TypeAdapterConfig<ZipCodeDTO, ServiceAreaToZipCode>
    .NewConfig()
    .Map(dest => dest.ZipCode.Zip,
        src => src.Zip);

and here's the rest of it
if you want me to paste the code here  as well i can. i don't know what they mean by examine the entity graph to find the difference. I cannot use the mapster debugger in core.

Thx u for the issue. It is bug.
In fact, Mapster doesn't allow Map(dest => dest.ZipCode.Zip. What Maspter expects is just first level property in destination (ex Map(dest => dest.Zip). But because there is no error throw, so after mapping ZipCode will always be null.
In short run, I will throw exception when we pass nested property to Map function.
In long run, I will try to find way to support nested properties.
For now, as workaround, pls use following config
        TypeAdapterConfig<ZipCodeDTO, ServiceAreaToZipCode>
            .NewConfig()
            .Map(dest => dest.ZipCode,
                src => new { src.Zip});

4.0 now support multi-level properties

repo: MapsterMapper/Mapster


text:
Replace T in IProvider with IFeature

Only one cast was needed in the entire code base so the generic IProvider did not provide enough added value.

Good work.

authors: inforithmics pauldendulk

repo: Mapsui/Mapsui


text:
Feature/player select

Some large changes made in order to merge, but successful still.

added player selection, name validation to come but I left it as is for testing purposes
some modifications to the game page
moved script tags to index.html

Gonna close this one out and start on a new branch...


text:
Create GitHub releases on release

Motivations
It would be nice to publish GitHub releases after creating the tag.
You can see an example of a GitHub release created by release-please here.
Ideally, we should implement something like that.
Problem
Usually, you want to add binaries and other artefacts to the release.
How does the user attaches these files if we create the GitHub release?
This makes me wonder if we should implement this feature or not.
Solution
There should be an additional flag to the release command.
The functionality is disabled by default.

Hey, I just discovered release-plz after fighting with release-please for a bit. It seems really cool so far, so thanks!
I'm interested in this issue, since my strategy with release-please was to run a "release binaries" workflow triggered on the GitHub actions release event. That works as long as you don't use the default GITHUB_TOKEN in the release-please action.

Hi Yusef, thanks!

triggered on the GitHub actions release event.

So you can add binaries and file to a release after the release is created?

Yes, exactly. I currently have a release-binaries workflow that runs on the "release created" event. But that event only gets dispatched if you tell release-please to use a non-default github token.

Well, that's cool! We could add a flag to the release-plz release command to also generate a github release.
For example release-plz release --gh-release.
Do you think we can re-use the same github token we are already using in release-plz or we would have issues similar to release-please?
Anyway, if you want to work on this, a PR is welcome! :)

[ ] Gitea also has releases. Before closing this issue, I want to add support for gitea releases, too.

You can now create github releases with:
release-plz release --git-token ${GITHUB_TOKEN} --git-release.
GitHub action
The GitHub action creates releases by default 👍
You can turn off this feature by changing the git_release flag.

authors: MarcoIeni yusefnapora

repo: MarcoIeni/release-plz


text:
Seperate scripts or single script?

PR #45 got me thinking, and I would like to discuss this new idea before acting on it.
Having many loose scripts looks good at first sight, but gets cumbersome soon from a development perspective. If you want two pieces of functionality that look very similar but have a different purpose, either a lot of code needs be duplicated. The other option is that the scripts can be merged with a branch for the two functionalities. However, there is currently not a good way to set options for each script, because GM_setValue/GM_getValue can only set and get data from the script they are running in.
A solution to this problem could be to provide all the current scripts as a single scripts, where you can enable and disable each option. The Settings script provides an example of where you could enable and disable options. This way we can be sure that you either have the default settings or only enable what you want, because the settings script would also be included.
What do you guys think about this? Is a single script better for you as a user if you have in-app settings, or do you prefer separate scripts for each functionality?

One script with settings to turn on/off specific functionality would be my preference. Thanks!

The other thing I think of is some basic settings scripts where general config and values could be set. But creating one script with possibility to switch needed scripts is good idea. It also allow to reduce code duplication such as getting the user platform as it can be the part of interface (utility class)

That basic settings script already exists but has some limitations as I explained in the original post.
I’ll create a separate branch for this soon.

Do you mean there will be a single script that will load the other scripts from urls or all the code will be in a single script?

All code within a single script. Bundled via webpack for example, so we can also use ES6 syntax.


text:
Bulk Mode Missing

One of my favorites was passing bulk data to the older version.
diff --git a/freq.py b/freq.py
index 6466ee0..f01a460 100755
--- a/freq.py
+++ b/freq.py
@@ -160,6 +160,7 @@ if __name__ == "__main__":
     parser.add_argument('-m','--measure',required=False,help='Measure likelihood of a given string',dest='measure')
     parser.add_argument('-n','--normal',required=False,help='Update the table based on the following normal string',dest='normal')
     parser.add_argument('-f','--normalfile',required=False,help='Update the table based on the contents of the normal file',dest='normalfile')
+    parser.add_argument('-b','--bulk',required=False,help='File containing a list of strings to test',dest='bulkfile')
     parser.add_argument('-p','--print',action='store_true',required=False,help='Print a table of the most likely letters in order',dest='printtable')
     parser.add_argument('-c','--create',action='store_true',required=False,help='Create a new empty frequency table',dest='create')
     parser.add_argument('-v','--verbose',action='store_true',required=False,help='show calculation process',dest='verbose')
@@ -199,4 +200,12 @@ if __name__ == "__main__":
             for eachline in filehandle:
                 fc.tally_str(eachline.decode("latin1"))
     if args.measure: print(fc.probability(args.measure))
+    if args.bulkfile:
+        try:
+            with open(args.bulkfile, 'r') as bulkFile:
+                for line in bulkFile:
+                    line = line.strip()
+                    print("%-30s %s" %(line, fc.probability(line)))
+        except FileNotFoundError:
+            sys.stderr.write("Failed to find bulk file: %s\n" %(args.bulkfile))
     fc.save(args.freqtable)

Hi,
I just commited a new tool named freq_sort.py.  I hope to move functionality such as you described here into that tool and keep freq.py limited to just the raw calculations and maintenance of tables.  I think that tool with the -v option will give you what your looking for.  If not please let me know I can tweak it.
Mark

authors: MarkBaggett richluby

repo: MarkBaggett/freq


text:
Vagrant linux image requires cmake for git branch

In order to run bundle install successfully on the git branch, the Vagrant linux image (or any linux install) requires that the cmake library is installed or otherwise the rugged 0.21.4 gem will fail to install.
Perhaps add this library to the markusproject/ubuntu image or just add to the instructions that cmake is required separately... by simply running sudo apt-get install cmake?

@oneohtrix Can you please add cmake to the image?

@oneohtrix @david-yz-liu It's also going to require gitolite.
sudo apt-get install gitolite should get the files required if one chooses to use Git functionality later on. The package will install but won't get setup unless the user chooses to run sudo dpkg-reconfigure gitolite after.. but I'll provide instructions on that somewhere on the Wiki.

@oneohtrix Sorry couple more dependencies: sudo apt-get install libssh2-1 libssh2-1-dev libgpg-error-dev

Oh crap, sorry I didn't see this! I'll add it ASAP.

@oneohtrix Its actually okay! I edited the Vagrant instructions to account for this.

Well if @oneohtrix adds it to the image, that's better in the long run since it removes an installation step.

@oneohtrix @david-yz-liu Okay, Lawrence if you look at this wiki page https://github.com/MarkUsProject/Wiki/blob/master/SettingUpVagrantEnvironment.md (not sure why its not on the main MarkUs wiki yet) you can see the additional setup I had to do to get Gitolite working.

hey @david-yz-liu @SoftwareDev -- can this be closed?

Yes, thanks.

authors: SoftwareDev david-yz-liu oneohtrix

repo: MarkUsProject/Markus


text:
Bug fix and corresponding tests

This is regarding the RT ticket id:7666

Whenever the students did not submit any assignment files the Submission dates field value will be set to nil.
When the Instructor is collecting submissions with no assignment files, it is getting collected (by setting is_collected=true) but it is showing the following errors
undefined method <=' for nil:NilClass undefined method -' for nil:NilClass
Fix:
1.Now the error message wont appear in the page and also the instructor can collect even if the students didn't submit any files.
Added tests for the corresponding changes made.

This is looking good!
I think we need an additional test for each submission rule type (grace period, penalty decay, etc.) that tests the case where the submission is empty and the submission was created during the penalty period.

@ANUSHIYAPRIYA last thing: can you also update the changelog?


text:
repo: change PR template to be more user friendly

Motivation and Context

Previously, I found myself having to move where I put a description of "my changes" to be below the "description" prompt as I kept putting it directly under your "Your changes" where space was provided. Seen here:

Furthermore, when pruning the checklist of things to do before merging the PR, one had to delete the "if applicable" milestones from the middle of the list which was awkward when highlighting. This PR addresses these issues.
Your Changes

Description:

Get rid of the blank line under "Your changes".
Move all the "if applicable" (and as such often deleted) milestones in the checklist to the bottom for easy truncating.
Hide 'if applicable' from public view, and make deletion of the milestone the obvious action in those cases when not applicable.

Type of change (select all that apply):

[x] Refactoring (internal change to codebase, without changing functionality)

Testing

Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[x] I have fixed any Hound bot comments (check after opening pull request).
[x] I have verified that the TravisCI tests have passed (check after opening pull request).
[x] I have reviewed the test coverage changes reported on Coveralls (check after opening pull request).

Required documentation changes (if applicable)

I like it! My only thought is that I'm not sure if the word "milestone" makes the most sense since that word has a specific meaning (on github at least). What about "item", "line", or "checklist item"?

authors: ignaspa mishaschwartz


text:
Change test url

Gopay changed test url
from https://testgw.gopay.cz/ to https://gw.sandbox.gopay.com/
https://doc.gopay.com/cs/#dokumentace-gopay-rest-api

https://doc.gopay.com/cs/#dokumentace-gopay-rest-api

@wamesro Thank you for the PR, but as you can see, our integration tools shows some fails.
https://travis-ci.org/Markette/GopayApi/jobs/223140982

@f3l1x Sorry, but I changed only the addresses, I did not change the annotations.

Thank you.

repo: Markette/GopayApi


text:
Make lyrics available in app itself

So, I looked into an API and apparently you have lyrics available for third-party apps, but i couldn't find any button that would show that lyrics in app itself. And that feature is not on the list (unless Ctrl+F failed me).

So yeah, this is like a realllyyy long story but I'll shorten it for you.
TLDR: The lyrics part of the API was an afterthought and is completely unreliable.  It doesn't work for any song newer than late 2015.  Also I'm rubbish at UI design and couldn't come up with a nice way to show the lyrics.
TLDR2: If you / someone else wants to have a go at this by all means.  I'd be interested to see what people come up with. :+1:

I believe Musixmatch has a API for developers (I think Shuttle uses it internally for lyrics).

Yeah, but do you have like $10,000 a month or something like that :+1: :cry:

Oh! Didn't read the TOS carefully. There is also the Genius API (should have a lot more than wikia).

So I have no real idea how to implement this but I think I have an idea for the UI... I think the lyrics could be displayed like another visualizer, so filling the whole screen, that would give them enough space and they're relatively easily accessible.

Could the lyrics possibly be overlayed when only showing the album art in fullscreen?

I've looked into this and as far as I can tell my lyrics "thing" has about an 80% success rate with finding lyrics.  This is probably too low too implement a solid lyrics feature

What if when the lyrics search fails, it just doesn't give the option to view the lyrics in the application, so if the lyrics are available, its good, but if not, nothing bad happens?
I think the best way would be to have a toggleable overlay over everything but the UI (button could be near the queue button or hidden in the radio/instant mix dropdown) that displays the lyrics (could also scroll based on time of the song).
If this is worth implementing I could give it a try, but it would probably take me a long time.

I don't know if you've looked at this but this may or may not be helpful.

authors: MCManuelLP MarshallOfSound Meegooo SicariusNoctis delold emphaticpuma tehloxely


text:
Sound Quality Poor

This may be related to https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/2570 , however for me everthing works, but the sound quality is very quiet & scratchy compared to listneing through the chrome extension/browser. I am on osx sierra. let me know if you need any other info that might help :)

@brewster1134 Are you on High Sierra or Sierra?  And silly question but is the volume turned up and are you using the default audio device?

I am on 10.12.5 (16F73). I also had checked that the value in the player
was set all the way up and strictly used the media keys for volume, but
good call on the default audio device. i was using wireless headphones.
through the laptop speakers it seems fine (as loud as i could play them at
work ;).  However the bluetooth speakers sound fine using the browser
player still.
On Tue, Jun 20, 2017 at 8:49 AM Samuel Attard notifications@github.com
wrote:

@brewster1134 https://github.com/brewster1134 Are you on High Sierra or
Sierra? And silly question but is the volume turned up and are you using
the default audio device?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/2589#issuecomment-309802319,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANRVAVCH4jeWUyYa6N2RAY3p9xn3SGbks5sF-nzgaJpZM4N_ySb
.

--
-rb

This will be a driver difference between the version of Chromium in GPMDP and the version of Chromium / your browser that you are testing with.  Your best chance for this to get fixed is to either
a) Find a way to replicate in Chromium and report it as a Chromium but
b) Fail to replicate in Chromium and hope that means the next Chromium update Electron gets will fix this issue in the next update
Other than that there isn't much I can I'm afraid to investigate a sound issue super specific to your hardware set.

fair enough. would it be fair to say an upcoming version of GPMDP will have
a new Chromium version included?
On Tue, Jun 20, 2017 at 8:59 AM Samuel Attard notifications@github.com
wrote:

This will be a driver difference between the version of Chromium in GPMDP
and the version of Chromium / your browser that you are testing with. Your
best chance for this to get fixed is to either
a) Find a way to replicate in Chromium and report it as a Chromium but
b) Fail to replicate in Chromium and hope that means the next Chromium
update Electron gets will fix this issue in the next update
Other than that there isn't much I can I'm afraid to investigate a sound
issue super specific to your hardware set.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/2589#issuecomment-309805509,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANRVGgPtgFCRsfM9mvnXjCJh6IuzA4tks5sF-xYgaJpZM4N_ySb
.

--
-rb

👍


text:
Narrow screen width hides some controls

I use my secondary monitor in portrait mode, but it's only 768px wide/high. Apparently Google decided that this screensize isn't important anymore. The global minimum width seems to be around 980px, while the controlbar minimum width is around 850px.
Result: you have to horizontally scroll to see playtimes and the queue and miniplayer buttons are completely hidden. This is the default behaviour on the normal web player as well.
Perhaps you could change this behaviour by removing the album art or scaling the controls?
A simpler option would be to allow us to set a default zoom level.

I'm working on documentation for this but you can actually set a zoom level (woo hoo).  Just use the shortcuts you would use in any other browser
Ctrl + --> Zooms in
Ctrl - --> Zooms out
Ctrl 0 ---> Resets Zoom


text:
Adds Select All functionality

Looks like this was just missing from the edit menu. Added.

#225

Goddammit these Pr's come in before I can find my mac lol...
Thanks for this, this was exactly the change I was going to implement

Ahhh crap, this is what I get for not letting the build finish.
Please run the linter before pushing to a pull request.  The build failed on some quote style issues

Ah crap, sorry! Thought I double-checked my quotes... Also, didn't notice the lint script... I'll slow down :checkered_flag:

authors: MarshallOfSound clatour


text:
Update VVCEasy.bat

check and get the UAC for administrator privilege

@neoOpus, thank you for making pull request.
Is there any that you could do without creating new window cmd in batchfile?

Martin Eesmaa

I found this project (https://github.com/gerardog/gsudo) and it gives the elevated privilege to the same terminal window/tab... so I think you can add it to your project if you think it fits with your requirements...
let me know if you want some help regarding this or some other stuff

@neoOpus, okay, I guess I have no choise of creating new Windows Terminal window, it will be stuck in old cmd, but I can accept your pull request. I hope the UAC Script will help to users... :)

@neoOpus, okay, I guess I have no choise of creating new Windows Terminal window, it will be stuck in old cmd, but I can accept your pull request. I hope the UAC Script will help to users... :)

I send you a second reply to your question, I guess you didn't read it, it is about gsudo which can make what you wanted but it is part of another project that maybe you can include in your distribution...

authors: MartinEesmaa neoOpus

repo: MartinEesmaa/VVCEasy


text:
Switch is not deleted

The switch is not deleted in the Home app after deleting it from the configuration file

@Caban-ru and @oigroigm. Found a bug that made the receiver disappear. Sorry for inconvenience. It should be fixed in 2.1.1.

just tested 2.1.1
receiver now fine) works after restart as well
about switches:
I can see them in HB web ui (and they works), but they are not appearing in HomeKit

Ok good that it is working again. Thank you for testing. I'm not sure why it is not appearing in Homekit but is in the web UI. What I can think of is removing the Homebridge from the Home app and after removing the persist and accesories folder, readding. This is the only thing I can think of. You will have to setup all other plugins too in the home app though :/

@oigroigm Do you still have issues in the latest version?

I installed the latest version but hombrige crashes

@oigroigm Hmm. can you post your Homebridge log when you run it with debug enabled? If you don't know how that works, you can add the "debugToInfo": true, line to your config.json:
"platforms": [{
    "platform": "DenonAVR",
    "debugToInfo": true,
    "devices": [{

[2/22/2020, 10:34:45 AM] [DenonAVR] DEBUG: pollForUpdates: 192.168.1.6
[2/22/2020, 10:34:46 AM] TypeError: Cannot read property '0' of undefined
at /usr/lib/node_modules/homebridge-denon-heos/index.js:376:49
at Parser. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:304:18)
at Parser.emit (events.js:198:13)
at SAXParser.onclosetag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:262:26)
at emit (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:624:35)
at emitNode (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:629:5)
at closeTag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:889:7)
at SAXParser.write (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:1436:13)
at Parser.exports.Parser.Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:323:31)
at Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:5:59)
at exports.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:369:19)
at Request._callback (/usr/lib/node_modules/homebridge-denon-heos/index.js:370:6)
at Request.self.callback (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:185:22)
at Request.emit (events.js:198:13)
at Request. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:1154:10)
at Request.emit (events.js:198:13)
[2/22/2020, 10:34:46 AM] Got SIGTERM, shutting down Homebridge...
[2/22/2020, 10:34:48 AM] [DenonAVR] DEBUG: pollForUpdates: 192.168.1.6
[2/22/2020, 10:34:48 AM] TypeError: Cannot read property '0' of undefined
at /usr/lib/node_modules/homebridge-denon-heos/index.js:376:49
at Parser. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:304:18)
at Parser.emit (events.js:198:13)
at SAXParser.onclosetag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:262:26)
at emit (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:624:35)
at emitNode (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:629:5)
at closeTag (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:889:7)
at SAXParser.write (/usr/lib/node_modules/homebridge-denon-heos/node_modules/sax/lib/sax.js:1436:13)
at Parser.exports.Parser.Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:323:31)
at Parser.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:5:59)
at exports.parseString (/usr/lib/node_modules/homebridge-denon-heos/node_modules/xml2js/lib/parser.js:369:19)
at Request._callback (/usr/lib/node_modules/homebridge-denon-heos/index.js:370:6)
at Request.self.callback (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:185:22)
at Request.emit (events.js:198:13)
at Request. (/usr/lib/node_modules/homebridge-denon-heos/node_modules/request/request.js:1154:10)
at Request.emit (events.js:198:13)

Hmm. It looks like you get a different result from the status page.
Can you try to put the following line in your browser and post the result?
http://192.168.1.6:8080/goform/formMainZone_MainZoneXmlStatusLite.xml or http://192.168.1.6:80/goform/formMainZone_MainZoneXmlStatusLite.xml

STANDBY

AUXD

-60.0

off

Ok, clear. Apparently your receiver doesn't have the option to switch between relative and absolute volume display, so it misses that info too. This means that it was reading data that wasn't available. In 2.3.6 this should be fixed.

i tried the new version and it seems to work.
Thanks

Great thanks for reporting. will close this issue. If there are other issues please let me know!

authors: Caban-ru Martvvliet oigroigm

repo: Martvvliet/homebridge-denon-heos


text:
Allow GET-requests

Allow the user to use GET instead of HEAD requests (e.g. for applications which do not have an implementation for HEAD-requests).

Fixed in those commits:

78bdfd32527de4db7f06e797819a9f5dab97755a
13a6809b57b546292b81ebfa3f616b85a4655fd2
1692154e62ac93e48b1e17ad1c2432479a0ca39f
16c04047670c07719dc22e663fa13820df2f5ee4
8a2f6599b3eb5158148270928a5a4d78437f8ca4
5c9c1b30489ca2c7d8b83f99c7363aa58c590028


text:
component does not re-render when uploading videos

hi, really nice file upload component, but the component does not re-render when uploading videos, when the first video is selected, it shows up as normal, but when replaced with a second video, the first one still persists

I'll try to reproduce this and let you know.

ok, thanks

authors: Marvinified prime-infinity

repo: Marvinified/use-file-upload


text:
add support for OkHttp

http://square.github.io/okhttp/

it seems that OkHttp is already implemented - see https://github.com/Mashape/httpsnippet/blob/master/src/targets/java/okhttp.js

authors: ahmadnassri michalbcz

repo: Mashape/httpsnippet


text:
[BUG] Fix workflows

Description
(Description about the issue)
Fix the github greeting workflows
Use Case
(Use cases and benefits)
Screenshots
(Screenshots of the issue or reference)
Additional Context
(Any other context about the problem)

@MastanSayyad I have created a PR, pls have a look

repo: MastanSayyad/Visual-Sort


text:
🛑 Inventario App is down

In bb265f9, Inventario App (https://inventario.voluntariosgreenpeace.cl/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Inventario App is back up in c5869e1 after 11 hours, 5 minutes.


text:
🛑 Inventario App is down

In c75126c, Inventario App (https://inventario.voluntariosgreenpeace.cl/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Inventario App is back up in ab430ec.


text:
Select box click not registering properly on safari

Not sure if everyone also gets this, but I have to click on a specific section if I want to open the menu (circled in image).

Fixed in https://github.com/Matriz88/hef-website/commit/de58607c44afd234f4c1f60ca521839b435cfaec


text:
Getting with spam on bungee console warnings

What is the output url of /viaversion dump?

ViaBackWards: Latest (2.3.1)
ViaVersion: Latest (1.6.0)

How/when does this error happen? login?:

I don't know

Is there an error in the console? Use pastebin.com. Is there a kick message?:

https://pastebin.com/YFsWz27q

One of your plugins is sending wrong metadata. This could be because a plugin is outdated, or just sends wrong data without reason.
I’d recommend debugging your plugins by removing them one by one until it no longer happens to find out which one is causing this. You can also ignore this warning by changing ‘suppress-metadata-errors’ to ‘true’ in the ViaVersion config.

authors: Matsv NeximaOzi

repo: Matsv/ViaBackwards


text:
Feature Request: Octopus Intelligent

Would love to see support added for Octopus Intelligent! Can be dealt with manually for now having a fixed charging cost - but would be better if TeslamateAgile could pull the schedule directly from the Octopus API to account for any unplanned charges.

Hey, we already have an API integration with Octopus set up for Octopus Agile so this shouldn't be too difficult if they are exposing an endpoint to grab pricing data from, do you know what API calls can be made to fetch either point in time pricing data or the 24 hour schedule similar to the fixed price provider?

Doesn't look like Octopus have updated their public REST API documentation yet, however there is already a Home Assistant module (written in Python) which queries that API. I'm using it at the moment and it works as advertised to retrieve the Intelligent charging schedule each night.
Link here:
https://github.com/megakid/ha_octopus_intelligent/blob/main/custom_components/octopus_intelligent/octopus_intelligent_system.py
I don't write Python, but from what I can make out it seems to centre around the "registeredKrakenflexDevice" type. This lines up with what you see in the Octopus app, as when you're on Intelligent you get a new "Devices" tab that appears inside the app - and is where you can see the proposed charging schedule (after 5pm each evening).
This dynamic schedule is required to capture any charging windows that Octopus allocates outside of the guaranteed Intelligent off-peak window (23:30-05:30), as they are also charged at the off-peak rate.

Thanks for the information
I've had a look but it seems like the data is all designed to be realtime or recent history / future and I couldn't see a way to retrieve pricing data in between two arbitrary points in time which is what TeslaMateAgile needs to work properly unfortunately.
If you are able to find one (or maybe even use the same / similar one) that you can use for Agile I'll definitely look into it, I can see this being the case that it works on any Octopus tariff if you know how to call it correctly 🙂

Hey so little update, I am actually using Intelligent Octopus myself now so this has become of interest to me again!
I've contacted Octopus myself and they said there is no API similar to Agile pricing for it unfortunately.  I could perhaps do some work to allow recent charges to be calculated if data is available and ignore older charges, most of the time TeslaMateAgile is only calculating charges that have just finished so it could definitely be worthwhile.
Funnily enough I'm also now using Home Assistant so that custom integration is also of interest to me - I'll check it out and see if I can work something out here 😄

This may only be if you have the Octopus Mini which reads the smart meter every 10 mins or so, but within HA I have a sensor called: sensor.electricity_[METER_SERIAL_NUMBER]_current_rate
So this sensor is a little custom to each user as it has the meter serial number in it and I've not had a chance to delve into the source code for the integration to see how its getting this data but from the graph below it is varying convincingly.

I've just had a thought that might solve this and provide a lot greater flexibility:
We could simply integrate with Home Assistant directly which can listen to say, an entity state, which will provide an easy rolling window of time / price over I believe 2 weeks by default in HA which for most use cases is plenty.
This way you can just provide the number however you like, via any integration e.g. the Intelligent Octopus one above and you could even mix it in with solar/home battery data if you have that and leave TeslaMateAgile to do what it is good at which is taking a bunch of prices and associated times and calculating your charge cost.
This for me personally would solve 2 problems I'm running into at the moment:

How do I take account of Intelligent Octopus' flexible rates, especially with no API
How do I integrate house solar/battery into the price equation

And means TeslaMateAgile does not need to worry about storing or keeping track of prices from a live feed which has been suggested before and would have to be how it's done for Intelligent Octopus as well.
What do you guys think about that, anything else I could do to make the experience better for you?

I have now done the above and Home Assistant is now a provider with the release of v1.12.0!
I've provided an example in the readme FAQ about using Intelligent Octopus and that is my recommendation for anyone wishing to use IO with TeslaMateAgile, I am using it myself this way in fact!
I am however not planning at this time to provide native support for Intelligent Octopus due to the API not being able to respond to a date range.

authors: MattJeanes dakaix duncanbarnes

repo: MattJeanes/TeslaMateAgile


text:
High cpu usage, sometimes GPU usage.

I am getting relatively very high cpu usage (10-20%) even when the program is idle and not playing music. Furthermore it's sometimes using my dedicated GPU for 5-10% which it shouldn't do in any case I believe.
I have an i5 6600k and a rx vega 56.

I came to post this. I usually run DOTA 2 at around 100FPS on a 120Hz monitor so it's noticeable when I have lower FPS. I had this running in the background and it froze multiple times.
When I turned it off, my FPS probably doubled - not even joking. Extremely heavy on the CPU and possibly memory. Using the browser is much more efficient.

I've seen those issues myself. This shouldn't be an issue in my rewrite. Sorry about the long delay on that.
I suspect I may have a memory leak somewhere obscure.
I expect to be back on this project in the near future.

Hi Martyn, I also experienced this problem recently - even just after loading the app i experienced ~3% gpu usage

Dang, same here.
I love this app, but I've stopped using it for now. I do a lot of animation and compositing, and my computer locks up regularly if I have it open, once I discovered it was the culprit and closed it, no issues.
Note: the same thing happens if I open the official Youtube Music website in chrome, so not sure it's the app that's the issue. But if you have any way of restraining Youtube music that'd be amazing.
Looking forward to this getting fixed.

Dang, same here.
I love this app, but I've stopped using it for now. I do a lot of animation and compositing, and my computer locks up regularly if I have it open, once I discovered it was the culprit and closed it, no issues.
Note: the same thing happens if I open the official Youtube Music website in chrome, so not sure it's the app that's the issue. But if you have any way of restraining Youtube music that'd be amazing.
Looking forward to this getting fixed.

I suspect it might be something to do with a YouTube music bug. When I
installed YouTube music's official PWA (which is nothing more than a
standalone chromium tab) I also had this problem. I do not have it when
running YouTube music as one of my regular tabs though.
So I feel like it's either a YouTube music or chromium bug.
On Sun, 3 Jan 2021, 05:43 kameryn1811, notifications@github.com wrote:

Dang, same here.
I love this app, but I've stopped using it for now. I do a lot of
animation and compositing, and my computer locks up regularly if I have it
open, once I discovered it was the culprit and closed it, no issues.
Note: the same thing happens if I open the official Youtube Music website
in chrome, so not sure it's the app that's the issue. But if you have any
way of restraining Youtube music that'd be amazing.
Looking forward to this getting fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MaverickMartyn/youtube-music-desktop/issues/11#issuecomment-753567102,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIQBQRS5N65KDYXCJNVH3DSX7YWDANCNFSM4I4PKTMA
.

I suspect it might be something to do with a YouTube music bug. When I
installed YouTube music's official PWA (which is nothing more than a
standalone chromium tab) I also had this problem. I do not have it when
running YouTube music as one of my regular tabs though.
So I feel like it's either a YouTube music or chromium bug.
On Sun, 3 Jan 2021, 05:43 kameryn1811, notifications@github.com wrote:

Dang, same here.
I love this app, but I've stopped using it for now. I do a lot of
animation and compositing, and my computer locks up regularly if I have it
open, once I discovered it was the culprit and closed it, no issues.
Note: the same thing happens if I open the official Youtube Music website
in chrome, so not sure it's the app that's the issue. But if you have any
way of restraining Youtube music that'd be amazing.
Looking forward to this getting fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MaverickMartyn/youtube-music-desktop/issues/11#issuecomment-753567102,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIQBQRS5N65KDYXCJNVH3DSX7YWDANCNFSM4I4PKTMA
.

Following on from your point Paul, I don't observe this behaviour when using YTM on firefox

Following on from your point Paul, I don't observe this behaviour when using YTM on firefox

@PauldeKoning Same here, I'm using the YouTube Music PWA on my Chromebook and it uses 100% of my CPU while unfocused and doing nothing but playing music.

Since the problem persists on a browser as well, I collected some observations, hopefully they will be useful:

The problem may happen in both Chrome and Firefox, however on Firefox it happens sporadically, whereas in Chrome it's always reproducible
in Firefox the problem may appear at first, but then go away. From what I've seen it never reappeared again unless I restarted Firefox. Closing/Opening the tab in particular never made it start reproducing again for me.
CPU usage is a little bit higher in Chrome (≈22%), than it is in Firefox (≈16)
In both browsers the process has lots of threads — graphics stuff, audio, media… However, the CPU is always being taken by the main thread whose TID is equal to the PID.
Problem never appears in Chrome nor Firefox if you are logged out. IOW it is only reproducible when you're logged-in.

Found a workaround for Chrome: running it with --no-sandbox makes problem disappear.
Couldn't find something similar for Firefox, because I no longer can reproduce it there.

authors: Hi-Angel MaverickMartyn PauldeKoning SpheeresX kameryn1811 lakimens r-britten

repo: MaverickMartyn/youtube-music-desktop