Add a sonar tutorial

It would be great to help people to setup nyc with sonar.
Here is my setup:
npm install xunit-file --save-dev

add the following to the mocha command:
--reporter xunit-file --reporter-options file='./report/test-all.xml'

This is what I use (typescript + webpack + react) :
nyc mocha-webpack --webpack-config webpack.coverage.config.js \"(tests|web_modules)/**/*.test.\\?(ts|tsx)\" --require source-map-support/register --reporter mocha-sonar-reporter --reporter-options file='./coverage/TEST-xunit.xml'

webpack config:

  // Enable sourcemaps for debugging webpack's output.
  devtool: "cheap-module-source-map",
  module: {
    loaders: [
      {
        test: /\.tsx{0,1}?$/,
        exclude: [/node_modules/, /json/],
        loaders: ['istanbul-instrumenter', "babel-loader", "ts-loader"]
      }
    ]
  }

  plugins: [
    // Allow to define React as a global variable for JSX/TSX.
    new webpack.ProvidePlugin({"React": "react",}),
  ]

@serut I don't do a ton of front-end coding these days, so I'm not the best person to write this tutorial -- would love the help if you wanted to help contribute:
https://github.com/istanbuljs/istanbuljs.github.io


Update NOTICE for CM API 6.0.0

Director now uses CM API 6.0.0, instead of the previous shifting 6.x.0
version. The legal notice is updated accordingly.

don't forget to pick this to the v2.0.x branch


Config migration steps are obsolete, resubmit as requested as original was ignored for more than a year

This page has old guidance for config migration. That section should be completely replaced with a link to the import/export capabilities https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-import-export-config

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

ID: b05d11c1-26dd-7bfe-a93c-aee83c101298
Version Independent ID: c86c1628-4fb2-7cca-9f04-2d3cddfbe73b
Content: Azure AD Connect: Upgrade from a previous version - Microsoft Entra
Content Source: articles/active-directory/hybrid/connect/how-to-upgrade-previous-version.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@billmath
Can you please check and add your comments on this doc update request as applicable.

@dloder0
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

wow, never even noticed that.  Guess that shows how unfocused the "Move a custom configuration from the active server to the staging server" section is.  I think moving from active to a new staging server should only reference the import/export feature.  Maybe the remaining instructions go under a different title section like "Method to script creation of Sync Rules" rather than being lumped in with an active/staging topic. Once import/export became available I have not had to assist any customer with the old PS method, but the doc spends 90% of the content of that section discussing that old method.

You are correct, the moving the individual rules is kind of buried there.  I have gone ahead and broken that section up into 2 sections.  One for the custom import/export and one for the custom rules.  We do have customers, on occassion that want to export just a few select rules and then import them into testing/upgrade servers.
These changes should be live later today.  I am going to go ahead and close this issue and I apologize for the delay on the preivous one.  Thanks!
#please-close

I mis-spelled previous.  :-)


Change the initial screen so it doesn't offer the USB option.

Following https://github.com/endlessm/kolibri-explore-plugin/issues/723
Drop the Endless Key USB option from welcome screen.

I guess removing the related views like https://github.com/endlessm/kolibri-explore-plugin/blob/master/packages/welcome-screen/src/views/UsbDriveConnection.vue or assets like https://github.com/endlessm/kolibri-explore-plugin/blob/master/packages/welcome-screen/src/assets/usb-connect.jpg will be part of https://github.com/endlessm/kolibri-explore-plugin/issues/733 and this is only about removing the option from the welcome screen, right?

I think so – but I'm not too familiar with this codebase.

Available in Ladybird-632.380

Verified Passed in Chromebook: Ladybird 6.32-380 and Windows: <IP_ADDRESS>


nixos/libvirtd: make it socket-activated

Description of changes
Add support for socket-activating libvirtd (default: off) rather than always running it on boot.
This means we don't start stuff we may not need all the time.
Also manually add user units (both sockets and services) which for unknown reasons isn't part of pkgs.libvirtd to easily allow running the QEMU "session" variant.
Things to consider for the future:

socket activation should default to on
libvirt currently runs in "monolith" mode but upstream will be removing that in favour of modular mode

This PR has a bit of noise as a consequence of some minor cleanups but it makes things easier to read.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

@fpletz , I realized that you had disabled socket activation by default, but how about adding it back as an option? For workstations where you run a VM every now and then, there's not much point in always starting libvirtd.

@peterhoeg Socket activation breaks autostart in libvirtd. Upstream doesn't use socket activation either. If you can solve the autostart issue or add an option to explicitely disable autostart support as a NixOS option this might work though.

Note the the libvirtd NixOS tests fails with your changes and socketActivation enabled due to the autostart issue. I appreciate the style fixes but they make reviewing this PR unnecessarily painful. Two commits would've been nice.


Add a README

Hello, I propose as part of this ticket to add a README.md file at the root of your project, I think it would be interesting to have a presentation of the project on Github
I allow myself to use the file index.md in the folder "docs" to add the README in the presentation of the project ?
If you agree a PR is already prepared :)

Closed per the PR -- the README had been added in the meantime


Add the ability of hiding apps in the app drawer

There's a lot of unnecessary system apps on phones that you can't uninstall via traditional means, and I feal that using solutions like Island or adb is a little too invasive for just hiding from the apps drawer. So, please, add a dialog to hide apps from the Apps menu.

Please organize your requests. Your requests have exceed certain amounts in a short period and without proper documentation and trials and errors first (Although some of them cannot be done by yourself). Sometime it feels more like opinions compare then articulate suggestions. Although some of them are doable but is not in a shape of a good software design pattern and require some knowledge to understand how and why it should or shouldn't. There are some good examples for contributing. For instance, platforms and players, that some users making their own platforms, creating and fixing wallpaper packs by themself. Then did pull requests, merged into the repository.
While this page does open for crowds as a space to submit suggestions. But if having lots of requests without organizing and structural preliminaries. I will suggest to join and initial your opinions in our discord server. It helps to refine opinions, construct the idea and evolve into a more matured state.
If you wish to further engaging. Hers is a good guideline to read.

You are totally right, and I'm sorry. I'm arleady doing a custom flash platform as of now. But I can't modify the app drawer itself as the app is colsed source.

But I can't modify the app drawer itself as the app is closed source.
Yes. And it is actually a proper idea. But you know, opinions are from everywhere. 🙂 I have to organize it a bit.

Requests are welcome. But have reach the level that hard to handle. 🥵 And btw I am taking a break. Some of your request might able to organize into the 1.4 tho.

Oke. Thank you for your collaboration and making this project grow.

Thanks for the understanding. 🙏


Dockerize iSparrowData

add dockerfile

add workflow to build them automatically

Requirements:

make as lightweight as possible

expose command line

done in #21. Currently not testable on linux because of a lack of proper audio device, and general problems with docker device exposure on my mac. must be tested properly on the raspberry.


Optimize performance of slow token table join

No change in behavior, just SQL optimization.

Coverage: 93.967% (-0.006%) from 93.973% when pulling 1e40a2968c67a865595663674519dbcecfe05ed3 on cody-slow-token-query into 8729548b17942614978bf1168766cb720e2eb7ac on main.


Manually add ARM OSX Migrator

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Superseded by https://github.com/conda-forge/bipedal-locomotion-framework-feedstock/pull/32 .


go/compile: corrupted function return value

What version of Go are you using (go version)?
go version go1.9.4 linux/amd64
Does this issue reproduce with the latest release?
Yes (go version devel +2010189407 Tue Feb 13 16:34:46 2018 +0000 linux/amd64)
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/jnml"
GORACE=""
GOROOT="/home/jnml/go"
GOTOOLDIR="/home/jnml/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build334481324=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
// Minimized reproducer for the (incorrect) translation of github.com/gcc-mirror/gcc/gcc/testsuite/gcc.c-torture/execute/pr70429.c

package main

func main() {
	if foo(1) != int32(0x3edae8) {
		panic(42)
	}
}

func foo(_a int32) (r int32) {
	//defer func() { println("r ", r) }() // <- Uncommenting this defer statement makes the panic go away.

	return shr1(int32(shr2(int64(0x14ff6e2207db5d1f), int(_a))), 4)
}

func shr1(n int32, m int) int32 { return n >> uint(m) }

func shr2(n int64, m int) int64 {
	if m < 0 { // <- Commenting out this if statement makes the panic go away.
		m = -m
	}
	if m >= 64 { // <- Commenting out this if statement makes the panic go away.
		return n
	}

	return n >> uint(m)
}

Playground
What did you expect to see?
No panic.
What did you see instead?
Panic.

To @randall77 for codegen.

Possibly worth backporting to 1.9 and 1.10.
Pretty obscure though.

Reopening for possible 1.10 final and 1.9.5.

CL 93716 OK for Go 1.9.5
CL 94215 OK for Go 1.9.5


MAINT: stats: Use explicit keyword parameters instead of **kwds.

The oldest Python version supported by SciPy now accepts function
definitions of the form
def func(*args, param1=value1, param2=value2, ...)

so use that form instead of
def func(*args, **kwds)

This gives a nicer signature, and simplifies the argument validation.

@sethtroisi, these are the only functions that I found where updating the signature made sense.  There are a few other occurrences of def func(*args, **kwds), but those intentionally have a completely generic signature.
But if I missed any and it turns out there are more functions where we can make a similar change, than we should do so! I'm not looking for more, so if you find any, feel free to update them in a follow-up PR.

Thanks @WarrenWeckesser for finding and improving these. The new explicit keyword parameters simplifies the tedious kwds parsing :)


Gravitional waves example

Hello,
I really like the ln library and I've been using it with a drawbot I've built. Thank you!
I am trying to generate the gravitational waves example from the image in the Readme. I've gone through all of the examples and I can't seem to figure out how to generate that. I've successfully generated run all of the other examples. Can you elaborate on how that specifically is done?

I'm guessing I did that with function.go but I don't remember the specific function off hand.

Sorry to bump a really old issue, but @speckone this line might be helpful https://turtletoy.net/turtle/4b8ea1c123
Author ported parts of ln to javascript and used to recreate this example.


fixed, range second value is a copy - use index.

Seems broken: lines 159, 160 use the index within the range onto the actual value.
Plus: when not using a copy (not sure if the go compiler is smart enough to detect that on it's own - should anyways) only the index should be used, not both index and value when the value is taken from the range again. This would also have prevented the typo mentioned above and is less confusing when reading the code.

@dlorch Hey Daniel
I've used dnsserver this week and had to overcome that issue on my own (was so dumb not looking into PRs already created),
@dlorch I hope u could approve this PR and get it merged.
Should be really helpful

Thank you for the contribution and the fixes!


Support to WithIgnoreErr option in the ExportService

What type of PR is this?

/kind feature
/kind api-change
What this PR does / why we need it:
Support to WithIgnoreErr option in the export service.
Eventually, we would like to implement https://github.com/kubernetes-sigs/kube-scheduler-simulator/issues/41 .

When this is passed, export/import method shouldn't return errors and log the error instead.

The simulator may only be able to get some of the resources due to lack of permissions or other reasons.
And some users may want to import just some of the resources from an existing cluster.

Which issue(s) this PR fixes:

Relates #41
Special notes for your reviewer:
/label tide/merge-method-squash

/assign @sanposhiho

/ok-to-test

/remove-kind api-change

@sanposhiho Thanks! Fixed it.


Check whether a domain is formally pending HSTS preload (in queue)

This adds a field, HSTS Preload Pending, which checks whether a domain is in the HSTS preload queue.
Unfortunately, this does mean another HTTPS request made every time pshtt is run. It does this once already for the preload list itself. This is big enough to be cached on disk, but only if opted into with a flag, which I don't think anyone does right now.
For tools which orchestrate parallel uses of pshtt, like domain-scan, it might be helpful to have the orchestrator manage the downloading to disk of the preload list and pending list at the start of a bulk scan, and pass in their locations as flags so that pshtt don't have to re-download it every time during a bulk scan, but also don't rely on a cached version from a previous bulk scan.
What do you think? Should I add a cache flag for the pending list, like we do for the preload list? Is this field even worth capturing here at all?

I think this makes sense and is a nice way to show an organization that a domain is making progress. With our HTTPS reports, we've had multiple teams reach out to say they've submitted their domain for preloading, and this gives them some validation.
Merging. Thanks!


Add 10 year wrap up blog post

@georgiamoon This PR adds the 10 year wrapup post and associated files. Also has one change to the notebook from a previous post where the github folder structure has since been modified.
Note that the github pages preview shows some formatting inconsistencies. Not sure why this is happening but I suspect it's because we're now using Jekyll modules that don't build correctly on github pages. Previewing the changes locally confirms the final site output is formatted fine.

This change is

One quick comment, and then I also wanted to add a lede photo, but we can do that as a follow up post.


Extending fan control with multiple sensors

This version of fan-control can use external temperature sensors - along with the built in one - to turn on/off the fan. Each sensor can have different temperature limit. Some code of dewheater project was reused. New sensor BMP280 control code added.

Thanks for the PR it all looks good :-)
We do not allow PR into master could you please change the PR so that its going into dev.
Thanks
Alex

Hi Alex,
I made the changes you asked for.
Thanks,
Tamas


Update v4.identity.md

Updated some examples to reduce client confusion.
Also looking to bring in latest schema documentation.

:shipit:


Colorbox Popup not coming in center in the iPad

When i am opening color box popup in iPad then popup is not coming in the center of the ipad body.
sometimes when i changes the mode from portrait to landscape or landscape to portrait then it seems fine but that happen once in a while.
Please suggest a solution?

Also noticing this issue on the latest version of IOS. On other browsers/devices the Colorbox appears in the center, on IOS it seems to be at the top of the page so the user is required to scroll up to it. Has anyone found a fix for this?


searchPlaceholder option not working.

searchPlaceholder option not working.

It works on my end, and in the example in the library, so I'll need the exact steps to reproduce your issue, along with any relevant code and table options in order to look into this.

Below are the options I am using for MUIDataTable.All the options other than searchPlaceholder is working for me.
<MUIDataTable
title="Install Base"
columns={columns}
data={dataForTable}
options={{
selectableRows: 'none',
download: false,
print: false,
rowsPerPage: 100,
filter: false,
viewColumns: false,
searchPlaceholder: 'This is a search placeholder',
}}

I tried checking if its working in the following sandbox link - https://codesandbox.io/s/muidatatables-custom-component-example-o8fl3. its not working

Your versions of this library and the material-ui library are too low. If you use material ui >= v3 and the latest version of this library, it will work.

Got it fixed :) you were right with versions. Thank you so much!


Updating to Swift 3.0

I pulled down the develop branch and converted it to Swift 3 (Or attempted to). It looks mostly successful with only a few of the unit tests failing still that I was hoping you could take a look at. I'm having some weirdness in Xcode that is making some of this debugging a challenge on the latest Xcode beta build. Let me know what you think!

Thanks @patrick-lind I'l merge it and will take a look.


Launcher doesn't show any representations

What happened?
Loader doesn't show any published material, and throws an exception in the script editor. It started happening since #418, but doesn't appear related. It may be specific to colorbleed-config, but can't tell for certain. Has anyone else spotted this?
# Traceback (most recent call last):
#   File "C:\Users\marcus\packages\avalon_core\5.3.2\python\avalon\tools\loader\app.py", line 174, in _assetschanged
#     subsets_model.set_asset(document['_id'])
#   File "C:\Users\marcus\packages\avalon_core\5.3.2\python\avalon\tools\loader\model.py", line 37, in set_asset
#     self.refresh()
#   File "C:\Users\marcus\packages\avalon_core\5.3.2\python\avalon\tools\loader\model.py", line 135, in refresh
#     active_groups = lib.get_active_group_config(asset_id)
#   File "C:\Users\marcus\packages\avalon_core\5.3.2\python\avalon\tools\loader\lib.py", line 132, in get_active_group_config
#     default_group_config = predefineds.pop("__default__")
# KeyError: '__default__'

To Reproduce

Launch Maya
Open Loader

Workaround
I found that if I manually call this, it works as expected afterwards.
from avalon.tools.loader import lib
lib.refresh_group_config()

I believe initially somewhere it did a first refresh with lib.refresh_group_config() as the module got loaded or something alike. See wherever it got triggered or initializes the dictionary with __default__.

Hmm, I'll have a check on this, once I got back to office tomorrow.

I found the issue, there was a missing call to that function in the app.py.
https://github.com/getavalon/core/blame/6102ed740e3897516cf4b457d5b495aa202500a5/avalon/tools/cbloader/app.py#L121
It's a result of the refactor from #388, this commit in particular: here

Thanks !

Fixed with https://github.com/getavalon/core/pull/429


make name parameter in HTTP\Header mandatory

if the name is not mandatory, when we create a HTTP\Header and call it as string from __toString(), we will get the ": " value, which seems incorrect as message header.

Hmm. I was thinking there were some headers that only had a single value but I'm not able to find this now, so I"ll merge this for now with the caveat that if I find something along those lines in the future we might have to re-evaluate.


fix #6 Removed Impure Helper and replaced it with a proper Method

Uglify is not able to minify the JS due to the "impure helper" used. I replaced it with a method inside the component.

I can not reproduce this. An uglified version of the module is used on https://franzskuffka.github.io/vue-medium-editor/.

Just for the record, I'm having this same this exact same error with Laravel-mix when it tries to uglify it. I hope @dsampaolo applies your comments and fix this PR, I'm needing this too :-\

@javisperez @dsampaolo I think your problem is not actually this plugin. I do not think it makes sense to structure this plugin for the sake of making uglify happy.
There is a simple solution, called babili. It will make sure your ES6 does not get maimed during minification.
https://github.com/babel/babili


There might be a problem with the dependency declaration for @creit-tech/xbull-wallet-connect

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
when I ran npm ls tslib locally, I saw the following error invalid: "github:Creit-Tech/xBull-Wallet-Connect#0.2.0" from<EMAIL_ADDRESS>which is odd because it seems to be a valid github url with the correct release. But I figured I should let you know

Yeah this is common because not all tools recognize importing libraries from public github repos. Something I plan to do is deploying the xbull connect library to npm so these types of errors don't happen, probably will work on that today or tomorrow. I'll let you know once is updated

Sorry for the big delay on this, the latest version of the kit fixes this because now the xbull-wallet-connect package is loaded from npm instead of github.


Update REST credentials before they expire

Instead of waiting until the credentials expire before refreshing, we should start the refresh cycle a few seconds or minutes before the credentials expire.  That way we minimize the latency impact of a refresh, and we avoid problems when things like the metadata service becomes temporarily unavailable (it seems to happen during VM migrations).

Another idea is to return unexpired credentials if there is an error trying to refresh them.

Yes, we would like to do this.  Probably not until Q1.

Probably involves some background threads for REST clients, which we will be looking at as part of DIREGAPIC. Keep open for now.

Realistically no time to work on this. Closing.


⚠ Bofrost.at has degraded performance

In f65c34e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 941 ms

Resolved: Bofrost.at performance has improved in 7452e61.


Geometry.clip_polygons_2d() is bug

Godot version: 3.2rc4
OS/device including version: any
Issue description: instead of cutting a hole it produce stacked polygon if b completely inside a. the behaviour is right if b only clipped part of a.

Steps to reproduce: -
Minimal reproduction project:
geometry.tar.gz

Closing as discussed above.


[SPARK-50088][SQL] Refactor UnresolvedStarBase.expand

What changes were proposed in this pull request?
Refactoring of UnresolvedStarBase.expand(...)
Why are the changes needed?
Refactoring is needed for the Single-pass Analyzer project (please check link)
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
Github Copilot

+1, LGTM. Merging to master.
Thank you, @mihailoale-db and @vladimirg-db for review.


Docker Compose hubot image file

In the docker compose example file located at https://raw.githubusercontent.com/RocketChat/Rocket.Chat/develop/docker-compose.yml the hubot image used is pulled from "rocketchat/hubot-rocketchat".
The rocketchat github page at https://github.com/RocketChat/hubot-rocketchat states that:

Please DO NOT USE v1.x.x of the adapter. It is experimental, and in anticipation of a performance
oriented sweeping change within the core - which has not happened yet.
Please continue to use Hubot Adapter v0.1.x until further notice.

I suggest changing the docker compose file to use rocketchat/hubot-rocketchat:v0.1.4 as base image for hubot instead.

This was done by @marceloschmidt in commit 6a4608446b7b5bb3fc9c30ef3ac457cc341bb145


Fix false description about how setState works.

The sentence that I'm replacing here says that this.setState merges new values into an existing state object rather than replace the object. This blatantly contradicts the explanations elsewhere in this documentation of how setState works. React treats the state object as immutable.  Accordingly, setState does not change it. Instead, it creates a whole new object that duplicates the original state except that the properties of the object that are passed to setState are used to set the corresponding properties of the new state object. So, it replaces the state object, the exact opposite of what the sentence that I want to fix is claiming.

Hello @Rick-Kirkham, thank you for contributing!
Unfortunately the sentence you modify in this PR no longer exists in the docs, so I will close this PR as outdated.
But we welcome opening PRs for other open issues.
Thank you!


Create marun.txt

The address marun.edu.tr is used for student e-mail addresses.

Marmara University
I am a student in the Department of Actuarial Sciences
Web Site: https://marmara.edu.tr/
Mail Adres: https://posta.marun.edu.tr/
Email uses : @marun.edu.tr

@sevdalooq I am sorry, the domain marun.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Not able to save Ng-model Value into database

The ng-model value is displayed at the same view but when in controller i want to save ng-model value in to other variable. undefined value interested. And also $watch is not working on the ng-model.

did you ever figured it out?

Try this: http://stackoverflow.com/a/22768720/2276198


23 detail pagina layout chris

experimental with layout style
adjusted spacing and line-height
Quote-block styling

need to fix

element specificity

Everything looks good and clear. Maybe try to avoid using !important if possible and remove the commented lines for better readability.


Equilbrium plot

hello group 4. I am confused about what I should expect for my equilibrium plot. Should it look like the original shape or should it look sort of random? What does yours look like?

Yes you should be comparing that to the actual figures in configurations


adding in missing olmconfig file

Issue #, if available:

Fixes: aws-controllers-k8s/community#1105

Description of changes:
adding in missing olmconfig file
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Signed-off-by: Adam D. Cornett<EMAIL_ADDRESS>

/lgtm


Not sure why I'm getting a nil response object in GET method of AFHTTPSessionManager.

Not sure why I'm getting a nil response object in GET method of AFHTTPSessionManager.
By the way, it works for me on some APIs, but not all.
I do have some Chinese and Japanese characters in the return of the API that has the problem.
the version of the library is 3.0.4

By the way, it works for me on some APIs, but not all.
I do have some Chinese and Japanese characters in the return of the API that has the problem.

Could you post an example demonstrating the problem?

It turns out to be a broken JSON returned from the server. I checked with Apple's built-in library and Python.

Thanks!


fabolas example broken?

python version: 3.7.5
emukit version: 0.4.7
I tried running the example.py in emukit/examples/fabolas and got this error message:
File "example.py", line 37, in
res = fmin_fabolas(wrapper, space=space, s_min=s_min, s_max=s_max, n_iters=100, marginalize_hypers=False)
File "/home/anaconda/anaconda3/envs/notebook_2_test/lib/python3.7/site-packages/emukit/examples/fabolas/fmin.py", line 55, in fmin_fabolas
s_max=s_max, marginalize_hypers=marginalize_hypers)
File "/home/anaconda/anaconda3/envs/notebook_2_test/lib/python3.7/site-packages/emukit/examples/fabolas/fabolas_loop.py", line 65, in init
loop_state = create_loop_state(model_objective.X, model_objective.Y, model_cost.Y)
TypeError: create_loop_state() takes 2 positional arguments but 3 were given

Appears so. it should be create_loop_state(..., cost=model_cost.Y). i cannot verify this at the moment, but will do the fix later. or feel free to create a PR in meantime!


Question

Hi is there any way to use this without LiveKit Cloud?

You can use this with any livekit server (including self-hosting). see docs


📝 Update includes in docs/en/docs/tutorial/path-operation-configuration.md

Udate docs/en/docs/tutorial/path-operation-configuration.md to fix #12554

Thank you! @AlexWendland :100:


STAR outputs

The gene count file should be an output for STAR

Gene count file already being outputted as {sample_name}_GDC38.ReadsPerGene.out.tab for the STAR alignment rule


Test cases are not listed in release mode with Visual Studio 2015, Boost 1.59.0

I want to run boost unit tests in release mode with Visual Studio 2015. Unfortunately, the test explorer of Visual Studio 2015 does not display any boost unit test in release mode.
Boost Unit Test Adapter displays the following message:
Could not locate debug symbols for '...'. To make use of '--list_content' discovery, ensure that debug symbols are available or make use of '<ForceListContent>' via a .runsettings file.
However, the release version is built with debug symbols (C7 compatible (/Z7), PDB).
There are no problems in debug mode. All boost unit tests are listed in the test explorer of Visual Studio 2015 and can be executed properly.
Microsoft Visual Studio Enterprise 2015, Version 14.0.25431.01 Update 3
Boost 1.59.0
Boost Unit Test Adapter <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>

@esj1fe think has to be clarified what the issue with release build & debug symbols is.
Have you tried  as proposed in the error message?
https://github.com/etas/vs-boost-unit-test-adapter/wiki/User-Manual#force-list-content-discovery

Hi @esj1fe and thanks for the feedback.
Boost 1.59 is not supported as requested in issue #140.
The fact that discovery is working in debug mode is confusing since, as @guwirth suggested, this might be a product of forcing list_content discovery. Most likely, I think this is happening since the executable 'extension' ends in .test.boostd.exe unless a runsettings file is in use.
We suggest that you check your configuration properties and make sure that both 'Debug' and 'Release' configurations match. Should this be the case, or an equivalent runsettings file is used in both modes of execution, please coordinate with @guwirth should you wish to request the (re-)inclusion of Boost 1.59 support.

@netspiri as far as I understand from https://github.com/etas/vs-boost-unit-test-adapter/issues/197 Boost.Test 1.59 support means searching for one more symbol? If yes I would add it again.

@guwirth It is not as straight-forward as you make it seem.
Due to the introduction of version checking since Boost 1.63 and PR #198 respectively, this complicates things slightly. Since this PR, we now run the executable twice during discovery to acquire the exact Boost version. This allows us to react to version differences in a much more convenient manner should things change in Boost.Test in the future.
To recap, we need to make sure to distinguish between the following:

Version
Debug Symbol
Supports --version
Remarks

Boost 1.59
boost::unit_test::runtime_config::list_content
no
Tests are all marked as disabled by default in DOT output

Boost 1.60
boost::unit_test::runtime_config::LIST_CONTENT
no

Boost 1.61
boost::unit_test::runtime_config::LIST_CONTENT
no

Boost 1.62
boost::unit_test::runtime_config::LIST_CONTENT
no
Requires runsettings workaround

Boost 1.63
boost::unit_test::runtime_config::LIST_CONTENT
yes

Boost 1.64
boost::unit_test::runtime_config::btrt_list_content
yes

Boost 1.65
boost::unit_test::runtime_config::btrt_list_content
yes

Given that Boost 1.59 notes all tests as disabled by default, this adds yet another challenge which needs to be catered for. It is not just a matter of listing them all as enabled since this might conflict with the actual test conditions, causing a UX issue.

@LukaszMendakiewicz maybe you have an idea?

@netspiri is a namespace also a symbol? Is it possible to search only for boost::unit_test?

That's in the same vein as the discussion in #197. @netspiri mentioned there that they will measure the actual cost of running the symbol scan multiple times.
If that's not prohibitively costly, maybe we can do that, probably in the order of decreasing Boost version not to penalize people moving to the latest.
If it turns out costly, then maybe the solution should be to use the marker file / file name if one wishes to remain on an older Boost version?
Either of the above would still however require the complication for additional workarounds for 1.59, which makes them slightly less attractive... On the other extreme (with apologies to @esj1fe), maybe the adapter should choose to support only that last N releases of Boost (say, 5)?

Test cases are not listed in release mode with Visual Studio 2015, Boost 1.59:
problems:

Boost 1.59 is no more supported from BUTA EXE detection
detection in debug mode because of file extension *.test.boostd.exe
not detected in release mode because of *.test.boost.exe

Should not both extension be supported?
Would also remove the first ´.´ to detect also namings like *.AcceptanceTest.boostd.exe

solved with <IP_ADDRESS>:
https://ci.appveyor.com/api/buildjobs/u3w5u4fwdj1f0n3j/artifacts/BoostTestPlugin%2Fbin%2FRelease%2FBoostUnitTestAdapter.vsix
@esj1fe please try and give us feedback: The default 'ForceListContent' extension has been amended to accept file patterns which match:

*test.boostd.exe
*test.boost.exe

There is still no Boost.Test 1.59 support.


[hearthis.at] ExtractorError: An extractor error has occured.

Just attempted to run youtube-dl on a hearthis.at song, and it fails during the Extractor phase. Logfile posted below.
G:\>youtube-dl.exe --verbose https://hearthis.at/djjbrew/never-gonna-shut-up-dance-rough-draft-2/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'https://hearthis.at/djjbrew/never-gonna-shut-up-dance-rough-draft-2/']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2015.09.28
[debug] Python version 2.7.8 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg N-69972-g6c91afe, ffprobe N-69972-g6c91afe
[debug] Proxy map: {}
[HearThisAt] djjbrew - never-gonna-shut-up-dance-rough-draft-2: Downloading webpage
[HearThisAt] 443616: Downloading playlist
WARNING: unable to extract comment count; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[HearThisAt] djjbrew - never-gonna-shut-up-dance-rough-draft-2: Determining extension
ERROR: An extractor error has occured. (caused by KeyError(u'content-disposition',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "youtube_dl\extractor\common.pyo", line 288, in extract
  File "youtube_dl\extractor\hearthisat.pyo", line 94, in _real_extract
  File "youtube_dl\utils.pyo", line 1777, in urlhandle_detect_ext
  File "youtube_dl\utils.pyo", line 1773, in <lambda>
  File "rfc822.pyo", line 388, in __getitem__
KeyError: u'content-disposition'
Traceback (most recent call last):
  File "youtube_dl\YoutubeDL.pyo", line 660, in extract_info
  File "youtube_dl\extractor\common.pyo", line 294, in extract
ExtractorError: An extractor error has occured. (caused by KeyError(u'content-disposition',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Sorry for the late response. HearThis.at will function again in the next version.


Support for streaming to Twitch or Youtube

Would it be possible to stream directly to a streaming service like Twitch or Youtube?
Would it be possible with just wl-screenrec or are there any additional tooling required on the client side?

It may already work with something like
wl-screenrec --ffmpeg-muxer flv rtmp://iad05.contribute.live-video.net/app/{stream_key}

source https://consultwithgriff.com/streaming-mp4-to-twitch-youtube-with-ffmpeg/

Also be careful about bitrate settings

not sure but this doesn't work for me
does it work for y'all?

not sure but this doesn't work for me does it work for y'all?

I could not get it to work. However, I still have my daytime job ;-) so I have not invested the right amount of time into this to say for sure if it works or not.

update: I got it working with ffmpeg

https://files.catbox.moe/mfj3ll.mp4
This isn't a good solution but ayy, it works.
NB: This has been rewritten with AI, because I know no one would understand one thing i'm saying :skull:, aanyways:
Create two aliases (preferably add them to your shell) with names of your choice.
For the first alias, which records the video:
cd $HOME/Videos/; wl-screenrec --audio --audio-device <your audio monitor> --low-power=off --filename live_stream.mkv

Ensure the output format is mkv as mp4 might not work (at least in my experience). If you're unsure about your audio monitor, use this command in the terminal: pactl list short sources.
Now, for the second alias, which streams the [live] recorded video to your RTMP:
ffmpeg -re -i $HOME/Videos/live_stream.mkv -c:v libx264 -preset faster -minrate 3000k -maxrate 4500k -bufsize 2M -pix_fmt yuv420p -g 60 -c:a aac -b:a 128k -ac 2 -f flv <rtmp://blablabla>

Note that these ffmpeg settings are quite basic; make sure to adjust parameters like bitrate and bufsize according to your preferences. After making changes, restart your terminal.
Once your terminal restarts, search for the aliases you just created.

Run the first alias, then open another tab or terminal session and execute the second alias. If you encounter an error about video reading, it's normal. Just run the second alias again, and it should work.

Funny that I did this because I wanted to live stream today without using bloated OBS
(don't start the topic on why OBS isn't bloated and whatnot lmao)

Live streaming right now, and omg the delay the stream has
it's like 10 mins behind :skull:

That won't work on my low-end computers. I cannot afford a live transcode. :-)

That won't work on my low-end computers. I cannot afford a live transcode. :-)

oof, bro i forgor i did this guide

This works fine for me on YouTube with the following command:
wl-screenrec --ffmpeg-muxer flv -f rtmp://a.rtmp.youtube.com/live2/<stream key>

However it seems that flv is not supported by my GPU and so wl-screenrec takes more CPU usage than I'd like.

So flv is a container not a codec. I let ffmpeg choose a default codec for a given container. All you should have to do to make this work with hw Accel is pass --codec avc

https://github.com/russelltg/wl-screenrec/issues/75 known issue unfortunately.


kubernetes-dashboard service fails to start

The kubernetes-dashboard service also fails to start with the following message:
2019/12/21 14:14:56 Storing encryption key in a secret panic: secrets is forbidden: User "system:serviceaccount:go-api-boilerplate:kubernetes-dashboard" cannot create resource "secrets" in API group "" in the namespace "kube-system"
Originally posted by @mar1n3r0 in https://github.com/vardius/go-api-boilerplate/issues/23#issuecomment-568184198

Fixed:
https://github.com/vardius/go-api-boilerplate/pull/27

Reopening this as it's solvable if needed:
kubernetes-dashboard-per-namespace
In the case of minikube there is a built-in kubernetes-dashboard service so I would rather not use it to keep the cluster one pod lighter but in case Docker for Mac doesn't have it as a built-in feature we can fix it.

We do install dashboard for our namespace. Also it can be disabled if needed like this:
kubernetes-dashboard:
+  enabled: false
  enableInsecureLogin: true
  fullnameOverride: 'kubernetes-dashboard'
  extraArgs:
    - '--heapster-host=http://heapster.default:8082'

heapster:
+  enabled: false
  fullnameOverride: 'heapster'

This applies to all charts, they can be disabled or enabled with enabled: false value.


Per-stage (database) config

We would like to use different database settings per stage. There is already some stage-related code in app.js, but it always reads the config from appJson.defaultStage.
What about changing this line to something like this?
_.get(appJson, `stages["${process.env.STAGE || appJson.defaultStage}"].config`));

Or maybe there a better way to solve this? Thanks!

Update: Our current workaround (which is probably good enough) looks like this:
const app = new App();
if (!app.isWebhook()) app.setDynamoDb('users');

So file persistence is used during local development and DynamoDB for stage/live.

Haha awesome, I shoud take a day off to be in sync with your schedule again ;)


Update Getting help section of wiki

Currently looks like

@bobpersing @cbaksik
We could just say

For assistance using the POD Aggregator, please contact<EMAIL_ADDRESS>or post in the POD Slack team’s #pod-support channel

Or we could add more detailed wording

Assistance for data providers, data consumers, or general assistance using the POD Aggregator is available
Please contact<EMAIL_ADDRESS>or post in the POD Slack team’s #pod-support channel


chore(deps): updated openapi-response-validator to ^7.0.1 (was ^4.0.1)

Thanks! Will merge straight after the basePath PR is merged (very soon)

I just remembered that the latest version of openapi-response-validator has a different api, which broke us previously. Let me investigate when I have time

@rwalle61 I have just rebased against latest master (includes the basePath PR). All checks have passed, let me know if you remember or figure out what the problem was.

@mvegter I think our yarn workspace isn't actually paying attention to the package.json of the openapi-validator submodule. For example, the yarn.lock hasn't changed.
I think the fix is for me to make our openapi-validator a proper package listed in packages, so that we can reuse the code in the jest and chai plugins, and fix this package.json behaviour

hey @mvegter, we won't be able to merge this or https://github.com/RuntimeTools/OpenAPIValidators/pull/174 until we:

manually update the yarn.lock
fix our code to work with the latest version of openapi-response-validator, which breaks these tests because the validation error does not include the names of the invalid properties

You may be able to install the latest version of openapi-response-validator by manually updating openapi-validator's package.json, deleting the node_modules, then rerunning yarn install.
(To automate this, I will need to fix our yarn workspace configuration, which I will do at some point).
It'd be great if you could create an issue for this! You're very welcome to PR the fix too, and I'll happily guide you :)
Let me know so I know whether to keep these PRs open.

@rwalle61 I'm going to give it a shot at fixing the packages. but no promises as my yarn knowledge is very limited.


Consumer is not working after some time

After running the consumer for sometime, the program is not receiving any messages posted into the queue.
In GDB, the code is getting blocked at "amqp_simple_wait_frame" method of AMQPQueue.cpp file.

(gdb) bt
#0  0x00007ffff7c81997 in __GI___poll (fds=0x7fffffffd980, nfds=1, timeout=-1) at ../sysdeps/unix/sysv/linux/poll.c:29
#1  0x00007ffff7f69a45 in recv_with_timeout () from /usr/local/lib/x86_64-linux-gnu/librabbitmq.so.4
#2  0x00007ffff7f6a3f9 in wait_frame_inner () from /usr/local/lib/x86_64-linux-gnu/librabbitmq.so.4
#3  0x00007ffff7f6a9af in amqp_simple_wait_frame_noblock () from /usr/local/lib/x86_64-linux-gnu/librabbitmq.so.4
#4  0x00007ffff7fa336f in AMQPQueue::sendConsumeCommand (this=0x5555555ade20) at src/AMQPQueue.cpp:463
#5  0x00007ffff7fa2f5e in AMQPQueue::Consume (this=0x5555555ade20, parms=256) at src/AMQPQueue.cpp:390
#6  0x0000555555556edc in main (argc=3, argv=0x7fffffffe028) at consume.cpp:64

If your "for some time" is a small number of messages, like < 100 of them, make sure you are ACKing your messages by calling AMQPQueue::Ack() for each message received if you have not set the NoAck flag when you are consuming messages.


Add node visitor pattern + implement internal references (the :ref: role)

Fixes #16
As promised, this PR fixes the :ref: role implementation. It is used to link to internal hyperlink targets within the project (i.e. can also be used for explicit hyperlink targets in other documents).
Node transformers (visitor pattern)
I also added the concept of a "node transformer" (or node visitor), which makes it easy to collect all AnchorNodes - so it can be used by the RefResolver.
Node transformers is a pattern that Python Sphinx & docutils (rst parser) use a lot: https://github.com/docutils-mirror/docutils/tree/e88c5fb08d5cdfa8b4ac1020dd6f7177778d5990/docutils/transforms They allow to remove more state awareness from the parser/nodes to visitors, e.g. the whole generation of Metas/meta entries can be removed from Parser  and the nodes into a visitor in the future.
Transformers also allow to add or "collapse" nodes (e.g. create the section nodes based on title nodes, or apply .. class:: directives on the adjacent node). You can see some more ideas in https://github.com/doctrine/rst-parser/issues/145
For the implementation, I've used Twig's node visitor as a base, as this is a proven pattern in Twig.
Metas
I'm not so sure about the role of the Metas object in the current guides, and I see there also is a "TODO" item about this in DocResolver. In my test project, I've defined Metas as a service, which allowed me to directly inject it in the RefResolver. This works great in the test project of the Symfony docs. However, the Parser somehow resets and creates a new Metas instance in some circumstances, breaking this implementation.
Maybe it is a good idea to discuss how Metas have to work (also in relation to node transformers) and refactor this part of the Parser as well. cc @jaapio @mvriel

Hi Wouter,
Thanks for this great change. It's great to see that you can contribute to this project. I didn't have time yet to look in detail at your code. But I wanted to provide some input on your changes to let you know that I'm still here.
Transformers
I like this addition. In phpDocumentor itself, we call this our compiler step. We are separating this from the parsing step as we want to reflect the parsed content as close as possible in the AST. But also to allow some dynamic imports or circular object references, which are harder to handle when you want to cache the AST. In theory you are doing the same thing, as you added the transformers just before the parser returns the document. If we would move this out of the parser completely. We could introduce a compiling step allowing the transformers to iterate over the full documentation set? So you could even extract information from all documents and manipulate the documents together? An example of such a use case: https://github.com/phpDocumentor/phpDocumentor/blob/master/src/phpDocumentor/Compiler/Pass/TableOfContentsBuilder.php
Meta's
I'm not fully aware of all locations meta's are used. But how I understood the logic works is that all meta entries, which are pages, headers, and anchors should be collected in a single meta's object. When running the parser with cache this meta's object is read from the cache. Allowing the parser to process just the files that are not changed. And still, be able to link to all files from the newly parsed files. So basically it works like some kind of table-of-contents for the full documentation.
I think you are right that there are some situations where this logic breaks. I'm not sure in what situations this is as it was already implemented in some way in the original project. And we tried to keep it as is. I think this happens when a fragment or include is processed. Although I think we can fix this by moving the meta's into the parser context. As the context should be passed around when processing a single file including all content imported into that file.
I do like your suggestion to move the meta's into the transformation phase. This makes a lot of sense, as it would allow us to remove some more logic from the parser. Making it a more pure parser.

Hi Wouter,
Thanks for this great change. It's great to see that you can contribute to this project. I didn't have time yet to look in detail at your code. But I wanted to provide some input on your changes to let you know that I'm still here.
Transformers
I like this addition. In phpDocumentor itself, we call this our compiler step. We are separating this from the parsing step as we want to reflect the parsed content as close as possible in the AST. But also to allow some dynamic imports or circular object references, which are harder to handle when you want to cache the AST. In theory you are doing the same thing, as you added the transformers just before the parser returns the document. If we would move this out of the parser completely. We could introduce a compiling step allowing the transformers to iterate over the full documentation set? So you could even extract information from all documents and manipulate the documents together? An example of such a use case: https://github.com/phpDocumentor/phpDocumentor/blob/master/src/phpDocumentor/Compiler/Pass/TableOfContentsBuilder.php
Meta's
I'm not fully aware of all locations meta's are used. But how I understood the logic works is that all meta entries, which are pages, headers, and anchors should be collected in a single meta's object. When running the parser with cache this meta's object is read from the cache. Allowing the parser to process just the files that are not changed. And still, be able to link to all files from the newly parsed files. So basically it works like some kind of table-of-contents for the full documentation.
I think you are right that there are some situations where this logic breaks. I'm not sure in what situations this is as it was already implemented in some way in the original project. And we tried to keep it as is. I think this happens when a fragment or include is processed. Although I think we can fix this by moving the meta's into the parser context. As the context should be passed around when processing a single file including all content imported into that file.
I do like your suggestion to move the meta's into the transformation phase. This makes a lot of sense, as it would allow us to remove some more logic from the parser. Making it a more pure parser.

Hi! No worries about the response time. OSS time comes and goes :) I think we both understand the value of this freedom, so don't feel obligated to respond in time (whatever "in time" means).

In theory you are doing the same thing, as you added the transformers just before the parser returns the document. If we would move this out of the parser completely. We could introduce a compiling step allowing the transformers to iterate over the full documentation set?

At first, I did like your suggestion of moving this outside the parser. However, I'm now doubting it a bit.
I think it comes down to: what is the deliverable of the parser? Do we want it to return an incomplete/messy AST that is closest to the plain text, or do we want it to return a complete AST?
To give an example of what I mean with an incomplete AST, let's take a look at one of the more complex directives in rst: .. class:: It can be used in two ways:
.. with content
.. class:: important

    Some text

.. without content
.. class:: important

Some text

.. the latter can be used in complex scenarios, e.g. adding a class to an image
.. class:: thumbnail
.. image:: images/foo.png

The first two examples are rendered equally (
Some text
) and the last example is ).
The docutils parser fixes this in quite a nice way by producing a PendingNode if the class directive has no content. A transformer then visits all pending nodes, adds the class to the adjacent node and removes the pending node from the tree.
If we move the transformer step from the Parser, the Parser will return an AST with "pending nodes". This means the AST is not ready to be rendered, i.e. "incomplete"/messy. I think I favor a parser that returns a complete AST, where these cases are resolved already.

We could introduce a compiling step allowing the transformers to iterate over the full documentation set? So you could even extract information from all documents and manipulate the documents together?

If I continue my reasoning above, I would say this is a different use-case and it makes sense to both have a "transformer" step inside the parser for each document, and a "compiler" step that does things on the project as a whole (and not individual documents).
Btw, we can't use Sphinx as a reference for e.g. the table of contents feature. In Sphinx, document discovery and TOC go hand in hand: You only tell Sphinx about one index file, e.g. index.rst. This file then contains a .. toctree:: directive, listing sub files for this document. Sphinx then parses these files (which in turn can also contain a .. toctree:: directives). In the end, this automatically creates a full TOC tree like:
index.rst
├─ security.rst
│  ├─ security/custom_authenticators.rst
│  ├─ security/voters.rst
│  └─ security/...
├─ controllers.rst
│  └─ controllers/...
├─ ...

When I find some time for this project again, I'll try to dig a bit deeper in the Metas object and see if it's API can be made more simple and support both global and local reference scopes.

Before anyone continues with this PR, https://github.com/phpDocumentor/guides/pull/132 is improving the metas, I'm still working on this and it's far from finished. But it's getting shape.

With the new meta's structure in place, we might need to revisit this PR; maybe we should extend the Metas class to act like a repository that allows you to find a document by title or even a sub-section by title. As metas is now a Tree of Entries I think it should be flexible enough to cover all cases.

This PR has been split, into multiple pr's. #132  and #233


crouton extension doesn't save settings

The crouton integration extension doesn't save settings.  For example if you enable the HiDPI setting or disable the extension on the next login the HiDPI setting is disabled and the extension is no longer set to "Disable" (I'm not referring to the Chrome OS extension setting, the enable/disable setting in the extension itself).  Is there any way to make the extension remember previous settings on logout?

Yeah, known issue--see #1321


Broadcast updates from model callbacks

EDIT: Still TODO

[x] merge tests https://github.com/leastbad/cable_ready/pull/10
[x] allow to pass html_options https://github.com/leastbad/cable_ready/pull/11
[ ] address focus issues. there's a possibility we could profit from prior art (https://github.com/stimulusreflex/stimulus_reflex/pull/76) here. If that's the case, should we extract and move this up to CR? already done: https://github.com/stimulusreflex/stimulus_reflex/commit/8bed62ebbe5181a289ab5c36f0771f0fefa4071b, but we call morphdom manually. Maybe we can just copy that?
[ ] proposal: introduce data-cable-permanent or data-morph-permanent ? could be done by passing a custom permanentAttributeName: https://github.com/stimulusreflex/cable_ready/blob/master/javascript/morph_callbacks.js#L33

After being really impressed with @andrewculver's exciting Sprinkles video, we decided to see how we could build similar functionality into CableReady. It directly addresses a real blind-spot in the library, which is the case when you need to update the same view for a lot of people with content customized for each person.
The big idea of this feature is that instead of rendering unique content and sending it down the wire, each subscriber gets a ping notification to fetch the current page and morphdom it into the content of the web component. This allows ActionController to do what it does best, which is show the user exactly what they are allowed to see. It's also likely to take advantage of HTTP caching.
The approach taken in this PR builds on the work already done for the stream_from helper, borrowing its secure identifier infrastructure and following the same general structure.
This solution has three significant elements. On the server, there is the CableReady::Broadcastable (broadcastable.rb) which ideally gets included in ApplicationRecord. There's also the broadcast_from view helper (cable_ready_helper.rb) and the web component it emits, broadcast-from (broadcast_from_element.js).
While the naming has changed, the functionality is 1:1 with the original video, with a few upgrades:

web component supports an optional url attribute to instuct the web component to pull updates from a different page
web component supports updating multiple content areas with one fetch call, so long as parameters are identical
model broadcasting is now opt-in on a per-model basis
model broadcasting defaults to "all changes, all of the time" but on allows a Symbol or Array of [:create, :update, :destroy] and if accepts a Symbol or Proc
has_many associations now support a broadcast option, with the simplest value true indicating "all changes, all of the time" but customizable via Array, Symbol, Hash or Proc
thanks to excellent contributions from @erlingur, has_many :through associations are working great

When planning your broadcastable architecture, keep in mind that there are three different possible outcomes:

updates to an individual model instance / resource
creates/updates/deletes on any instance of the model aka "the index view"
creates/updates/deletes on collections (associations) on an instance of the owner

Let's build up an example:
class ApplicationRecord
  include CableReady::Broadcastable
end

class User < ApplicationRecord
  enable_broadcasts on: [:create, :update], if: -> { id.even? }
  has_many :posts, broadcast: ->(resource) { resource.id.odd? }
end

class Post < ApplicationRecord
  belongs_to :user
end

View code I use to test enable_broadcasts method (aka broadcasting from model) is:
<%= broadcast_from current_user do %>

<%= rand(1..1000) %>

<% end %>
<%= broadcast_from current_user do %>

<%= rand(1..1000) %>

<% end %>

This would pick up any update callback broadcast on the User model. I use two because it's important to see that there is only one fetch for two updates.
We can add a third instance of the helper, but instead of a model instance, we can pass a class constant to receive create and destroy callbacks. This is perfect for an index view, as it allows us to update a list of instances without getting stuck in "helper for model instance that doesn't exist yet or has just been destroyed" sinkholes:
<%= broadcast_from User do %>

<%= rand(1..1000) %>

<% end %>

Here's the view code to test association broadcasting. It's the same, except we pass a Symbol representing the name of the model association in addition to the model instance which owns the collection:
<%= broadcast_from current_user, :posts do %>

<%= rand(1..1000) %>

<% end %>

Right now, things are in good shape but there are a few ways to jump in and improve this PR:

tests are sorely needed and highly appreciated
is there a way to do broadcast_collections better; I have concerns that on a busy site or with a big table, running this code for every record could get bad, fast
is there a way to move some or all of this PR out of broadcaster.rb without blowing up all of the self accessors? even just figuring out how to move module ClassMethods out would make this so much lighter ✔
if someone could add some begin/rescue and raise some Argument exceptions if people pass the wrong options
is there a way that the has_many if Procs can receive a reference to the current associated model and not just the owner/originator of the association, that'd be great eg. right now if "User has_many Post" the Procs for the posts association receive a reference to the user, but that's likely only half of the story for most Procs you'd write here
@hopsoft would love to hear feedback on naming of methods and everything

Procs passed to if options on the enable_broadcasts method can access the current model instance as self, so you can do things like enable_broadcasts on: :create, if: -> { id.odd? }.
Remember, if you're testing this in your app and you change one of the if/on options in your app's model, you must refresh the browser to allow ActionCable to reconnect!
Thanks again to @andrewculver @erlingur @ParamagicDev @julianrubisch @fractaledmind and others who have contributed so far. Great team effort.

Awesome to see this coming together.  💯

Yeah agreed. We should point out how to use if conditions on enable_broadcasts effectively

@leastbad you describe like
<%= updates_for current_user do %>

<%= rand(1..1000) %>

<% end %>
<%= updates_for current_user do %>

<%= rand(1..1000) %>

<% end %>

But the correct isn't?
<%= updates_for current_user do %>

<%= rand(1..1000) %>

<% end %>
<%= updates_for Post do %>

<%= rand(1..1000) %>

<% end %>

They are both correct, because they both demonstrate different concepts.
As I explained on Discord, the goal of the example was to demonstrate that it works fine if you have multiple content areas with the same identifier.


local backtest failed-data-request.txt file with wrong entries

Expected Behavior

failed-data-request.txt should list all missing files
Actual Behavior

failed-data-request.txt has wrong list of files, e.g.
/equity/usa/daily/erus.zip
/equity/usa/daily/efnl.zip
/equity/usa/daily/aaxj.zip
/equity/usa/daily/gaf.zip
/equity/usa/daily/aud.zip
/equity/usa/daily/bund.zip
/equity/usa/daily/ewq.zip
/equity/usa/daily/enzl.zip
/equity/usa/daily/ivv.zip
/equity/usa/daily/icol.zip
/equity/usa/daily/eidoewi.zip
/equity/usa/daily/ewj.zip
/equity/usa/daily/eww.zip
/equity/usa/daily/epi.zip
/equity/usa/daily/ewh.zip
/equity/usa/daily/ezu.zip
/equity/usa/daily/and.zip
/equity/usa/daily/ech.zip
/equity/usa/daily/ewz.zip
/equity/usa/daily/chib.zip
/equity/usa/daily/ewp.zip
/equity/usa/daily/ewc.zip
/equity/usa/daily/aia.zip
/equity/usa/daily/ewl.zip
/equity/usa/daily/argt.zip
/equity/usa/daily/gxc.zip
/equity/usa/daily/norwewy.zip
/equity/usa/daily/ewk.zip
/equity/usa/daily/egpt.zip
/equity/usa/daily/ewd.zip
/equity/usa/daily/ewo.zip
/equity/usa/daily/braq.zip
/equity/usa/daily/adru.zip

/equity/usa/daily/eidoewi.zip and /equity/usa/daily/norwewy.zip are obviously wrong, it should be:
/equity/usa/daily/eido.zip
/equity/usa/daily/ewi.zip
/equity/usa/daily/norw.zip
/equity/usa/daily/ewy.zip

Potential Solution

Reproducing the Problem

in local environment with not enough data, backtest strategy from library: BETA FACTOR IN COUNTRY EQUITY INDEXES, then check failed-data-request.txt file from backtest result
System Information

host: ubuntu linux 20, docker: pulled from official repo, LEAN Engine v<IP_ADDRESS>
Checklist

[ * ] I have completely filled out this template
[ * ] I have confirmed that this issue exists on the current master branch
[ * ] I have confirmed that this is not a duplicate issue by searching issues

[ *  ] I have provided detailed steps to reproduce the issue

Hey @cjbd!
Thanks for the report, the algorithm has a typo it's missing 2 ,, we will fix it ASAP

@Martin-Molinero , thanks for the update


Added IBM Validated Open Source List  as data source

PR for #25
Signed-off-by: arshPratap<EMAIL_ADDRESS>

This is a good start, but there is also SLES 12.x and RHEL 7.x and RHEL 8.x that we'll want to parse and provide data files for. It's fine for us to merge as-is, but we want to make clear that it's only including half of the distributions included in the list.
And can you make the code you used to parse these files from the JSON source available? We probably want to add that code to package_build.py and instead of a distro, maybe have it triggered via something like: package_build.py validated_oss

These are all working as expected, well done!


downloading coco dataset the right way

I used the get_coco.sh script from yolov5 repo. But it does not seem to download datasets/coco/annotations/instances_train2017.json. The script only downloads instances_train2017.json. Therefore the training job fails.
Do you recommend any other process to download the coco dataset?

COCO dataset is large and I believe that official web here: https://cocodataset.org/#download might help you. In YOLOX we didn't provide any script for downloading coco dataset.


Triggering ModSec from an application

My hosting provider runs modsec on its web servers.
Is there a way for Apache or PHP to trigger a core modsec rule in the response?
I mean I have a long list of rules in my WordPress WAF but the hoster will surely not add a rule only for me.
Please advise.

Now I read that they have the https://www.imunify360.com/ ruleset.

Hi @szepeviktor,
Not sure what exactly you want to do, but usually the trigger is the user input or the web server output. The rule (or rules set) uses ModSecurity engine to classify if the request or response is harmful or not. After processing the content, ModSecurity can take a disruptive action (deny, redirect..), log the data or even do absolutely nothing :)
What exactly you are trying to aiming?

I am used to using Fail2ban. My WAF calls PHP error_log() which land in Apache's error.log and the IP address get banned by Fail2ban.
Is there a way to get an IP address banned by modsec? Or trigger a very critical error which hopefully ends up by banning that IP by another system running on the hoster's server?

or the web server output

So is there a way to output something that triggers a critical rule?


Won't Install or Work - Windows 10-64 bit, latest version of NodeJS

C:\Users\Gerald\Downloads\PingBot-master>npm install
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of erlpack@hammerandchisel/erlpack but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>No repository field.
added 113 packages in 21.971s
After config was setup and saved tried to run
C:\Users\Gerald\Downloads\PingBot-master>npm start

<EMAIL_ADDRESS>start C:\Users\Gerald\Downloads\PingBot-master
node ./index.js

C:\Users\Gerald\Downloads\PingBot-master\node_modules\ts-node\src\index.ts:307
throw new TSError(formatDiagnostics(diagnosticList, cwd, ts, lineOffset))
^
TSError: ⨯ Unable to compile TypeScript
src\ping-bot.ts (68,18): Argument of type 'Error' is not assignable to parameter of type 'string'. (2345)
src\ping-bot.ts (346,16): Argument of type 'Error' is not assignable to parameter of type 'string'. (2345)
at getOutput (C:\Users\Gerald\Downloads\PingBot-master\node_modules\ts-node\src\index.ts:307:15)
at C:\Users\Gerald\Downloads\PingBot-master\node_modules\ts-node\src\index.ts:336:16
at Object.compile (C:\Users\Gerald\Downloads\PingBot-master\node_modules\ts-node\src\index.ts:498:11)
at Module.m._compile (C:\Users\Gerald\Downloads\PingBot-master\node_modules\ts-node\src\index.ts:392:43)
at Module._extensions..js (module.js:584:10)
at Object.require.extensions.(anonymous function) [as .ts] (C:\Users\Gerald\Downloads\PingBot-master\node_modules\ts-node\src\index.ts:395:12)
at Module.load (module.js:507:32)
at tryModuleLoad (module.js:470:12)
at Function.Module._load (module.js:462:3)
at Module.require (module.js:517:17)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node ./index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Gerald\AppData\Roaming\npm-cache_logs\2017-08-12T06_08_36_092Z-debug.log

Thank you for the error report, it's fixed in the latest version of the master branch, please use git pull, follow the installation instructions and it should work for you.


Helm doc version should be updated in branch-2.1 after release > 2.1

Page
https://docs.alluxio.io/os/user/stable/en/deploy/Running-Alluxio-On-Kubernetes.html
Summary
As of now the documentation in the linked page points to the helm chart version included in the released 2.1 docker image (=0.5.1). However branch-2.1 was updated to helm 0.5.2 (https://github.com/Alluxio/alluxio/pull/10384/files). As docs for the released 2.1 are generated off branch-2.1 we have not updated the helm documentation version in the branch but only in master (https://github.com/Alluxio/alluxio/pull/10405#event-2768885278).
This is a workaround till the time we have documentation at patch release granularity.

@madanadit can this be marked as done?

No


How to view the load of the master when starting in a distributed manner

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
I am currently using local for distributed startup, and I can obtain the workload of the worker from the master. However, I need more master workload information, such as CPU usage and memory usage. Where should I obtain it from? Can you provide me with specific code? The code for obtaining the worker is as follows:
workers_info = [w for w in runner.clients.values()]

Hi! Master logs a high cpu-warning like how workers do, and if it it doesn't you should be fine.
Are you encountering a problem? If not, master stats are kind of boring as they should have zero impact on the actual load (unless it is, for some reason, completely overloaded).


Java Error

When running the jadx-gui.bat on Windows, I got the error:
Error: This Java instance does not support a 64-bit JVM. Please install the desired version.
I do have the both 32bit and 64bit of jdk8u171 installed, and jadx version is 0.7.1

I found out that uninstalling 32-bit jdk, only keep 64-bit jdk will allow jadx-gui to work. Please close the issue.


fix: update amino serialization with fix for regen message types

This PR adds a custom serialization for amino messages that omits the type & value wrapper for Regen specific message types.
It is a workaround until https://github.com/regen-network/regen-ledger/pull/1480 get's released (targeting Regen Ledger v5.0, requiring a network upgrade).

looks like tests are passing w/o $type field? should i remove that field from the amino converters?

@amaurym It would be great if you could take a look at this and @mhagel's PR tomorrow (Monday) so we get some reviews of these pieces and can move to publishing this fork & merging https://github.com/regen-network/regen-js/pull/38.

looks like tests are passing w/o $type field? should i remove that field from the amino converters?

I took care of that, thanks @technicallyty


The "double" Rgroup labels are replaced with R# when user saves the structure in CML-format.

Steps to reproduce:

Create any structure.
With the R-Group Label Tool set any label for any structure atom, for example, R1.
Click the 'Save As' button. Choose the CML-format.

Actual correct result
The structure is saved and opened correctly.

With the R-Group Label Tool set any other label for the same structure atom. Now two labels present on the atom.
Click the 'Save As' button. Choose the CML-format.
Clear the canvas.
Click "Open..." button, open saved in 5th step structure.

Actual result:
The R-Labels are replaced with R#.
Steps to reproduce:

Create any structure.
With the R-Group Label Tool set any label for any structure atom, for example, R1.
Click the 'Save As' button. Choose the CML-format.

Actual correct result
The structure is saved and opened correctly.

With the R-Group Label Tool set any other label for the same structure atom. Now two labels present on the atom.
Click the 'Save As' button. Choose the CML-format.
Clear the canvas.
Click "Open..." button, open saved in 5th step structure.

Actual result:
The R-Labels are replaced with R#.

The check structure contains the message that RLabels present at the canvas:
Nevertheless when user wth R-Group Label Tool clicks this R# atom, no R-Labes are selected in the opened window:
Expected result:
The structure is opened in the same view that before saving - the both RLabels presents at the structure and when users clicks the atom with R-Group Label Tool, these rLabels are selected in the opened window.
The check structure contains the message that RLabels present at the canvas:

Nevertheless when user wth R-Group Label Tool clicks this R# atom, no R-Labes are selected in the opened window:

Expected result:
The structure is opened in the same view that before saving - the both RLabels presents at the structure and when users clicks the atom with R-Group Label Tool, these rLabels are selected in the opened window.

CML format supports only positive integer values according to  http://www.uwosh.edu/faculty_staff/gutow/marvin.1/doc/user/cml-doc.html rgroupRef - R-group reference. Currently, only positive integer values are accepted. So it's not possible for now to store multiple groups to one atom in CML files. Agreed on closing this issue with @AlexanderSavelyev

CML format supports only positive integer values according to  http://www.uwosh.edu/faculty_staff/gutow/marvin.1/doc/user/cml-doc.html rgroupRef - R-group reference. Currently, only positive integer values are accepted. So it's not possible for now to store multiple groups to one atom in CML files. Agreed on closing this issue with @AlexanderSavelyev


DeletingAllS3BucketsIssue116.py

1jc for Issue 116 Sprint 4   " Issue 116 Fix script to delete all s3 buckets even with data"
fix #116

Implementation:
Issue #116 Fix script to delete all s3 buckets even with data

Research

Write python scripts

Create branch in repository
issue_#116_Fix_script_to_delete_all_s3_buckets_even_with_data

Create file
DeletingAllS3BucketsIssue116.py for deleting s3 buckets

Put files into branch made

Do a git pull (to update your local)

Testing to be done see directions for testing script

Need at least two reviewers for two approvals for pull request

TESTING:
Requirements:

AWS CLI configuration
Python Boto3 installed
For testing:
•	Need AWS CLI
•	$ aws –version
•	If your system does not have AWS CLI
•	$ aws configure
•	Need access key id, secret access key, default region name as us-west-2
•	Install Python Boto3 in AWS CLI
•	$ python -m pip install boto3
Import boto3
Client = boto3.client(
‘s3’,
aws_access_key_id = ACCESS_KEY,
aws_secret_access_key = SECRET_KEY,
aws_session_tokens = SESSION_TOKEN
)
•	$ git pull
How do I run a git pull command?
Now go back to the original folder and follow the instructions:
First,
run git status
Git will tell you the repository is clean.
Then,
run git fetch
Next
run git status
Git will say your branch is one commit behind
4.run git pull
updates your local branch
At command prompt navigate to the branch where script file is available in this case branch - :issue_#116_Fix_script_to_delete_all_s3_buckets_even_with_data
type python then the script DeletingAllS3BucketsIssue116.py
enter to run script
verify print command

Activities   Documentation
Date
Activity
Time Spent

Sprint 4   Issue 116
March  4, 5
Research
2.0 hours

Sprint 4   Issue 116
March 13
Research
2.0 hours

Sprint 4   Issue 116
March 14
Documentation
2.0 hours

Sprint 4   Issue 116
March 14
Folder,   Initiate Pull, commits
5.0 hours

Totals

11.0 hours


Is this project dead?

Just wanted to ask the same question. Last commit 6 months ago and the documetation is dead...not good.

It looks very dead indeed. Since AsgardCMS uses this package heavily, I've asgard Gravitano to grant me access to help maintain this package and didn't got a response.
For that reason I re-packaged the package under a new namespace. I've also added a lot of tests which aren't present here.
https://github.com/nWidart/laravel-modules

63% really nice

One thing missing is the migrations, but I'm having trouble testing that / setting it up.

@nWidart  What is the best way to transfer to your package in an existing project?

Hello,
Replace pingpong/modules, but nwidart/laravel-modules, will do. That's what I did on AsgardCMS.

Using your laravel modules package how do i access a model from another module?

@nWidart are you going to keep maintaining your fork because i think i m going to switch to it

I stopped maintaining the fork (https://github.com/nWidart/modules) however I will continue to maintain the new package (https://github.com/nWidart/laravel-modules).

i'm using caffeinated, it's usage is quite similar.

@gravitano can i help you to host documentation ? I have a free space on my server if you like.

I too use this package in 5 projects, and i have interest in maintain this package alive. If i can help in something, reply this issue please.

Yes! Dead!!!!!!!!!

Is there any living HMVC alternative for Laravel?

Switching to nWidart fork too :tada: :)


VSCode warns about active terminal sessions that do not exist

Does this issue occur when all extensions are disabled?: Yes, apart from remote ssh extension to be able to connect to the server

VS Code Version:  1.93.1
OS Version: Windows 11 (Remote VM ubuntu 22)

Steps to Reproduce:

Connect to ssh remote
Reload the window

I am using VSCode Remote Extension to connect to my VM, when I try to reload the window, it always shows this warning:
"Do you want to terminate the 6 active terminal sessions?"
However, I don't have any open terminal sessions. I suspected the port forwardings, but there are only two of them.
How can I find out what are these open terminal sessions?
I tried sudo ss -tnp | grep sshd to see open ssh connections but there was only two of them.

Can you share your settings.json file?


GLSL -> HLSL issue with sampler2D arrays

Hey ;)
I found one small problem with sampler2D arrays.
Here is what I have in my GLSL file:
uniform sampler2D u_textures[2];
layout(location = 0) out vec4 outResult;

void main()
{
    outResult =  texture(u_textures[0], texCoord);
}

Here is what I got in HLSL file:
Texture2D<float4> u_textures;
SamplerState _u_textures_sampler;
static float4 outResult

void frag_main()
{
    outResult = u_textures[0].Sample(_u_textures[0]_sampler, texCoord); //! error here
}

Here is what error message I got:
error X3120: invalid type for index - index must be a scalar, or a vector with the correct number of
 dimensions

Kha, D3D11 target.

It should work now. 🙂

Hoho


bump usps version

Description:
Fixes usps platform. Dependency myusps now leverages Selenium webdriver to overcome login issues. This is a breaking change since the user must now have additional dependencies installed - either google-chrome and chromedriver, or phantomjs. There is a new config option driver that allows the user to specify their preference, though phantomjs is the default. Doc PR forthcoming that will outline choices and make suggestions based on user's OS.
Related issue (if applicable): fixes https://github.com/home-assistant/home-assistant/issues/10325, https://github.com/home-assistant/home-assistant/issues/9143, https://github.com/happyleavesaoc/python-myusps/issues/9
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here> (TBD)
Example entry for configuration.yaml (if applicable):
usps:
  username: !secret myusps_username
  password: !secret myusps_password
  name: 'USPS'
  driver: 'chrome'

Checklist:

[x] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

As someone who uses Selenium at my job I really like what you have done here.  I don't know if any other component uses Selenium but I can see this being useful in other cases as well.   Great work :)

It feels a bit weird to expose the driver via the config. Why would the user need to be concerned with this?
I also think that one should not default to phantomjs, since it's no longer maintained after headless Chrome was launched.

Makes sense.

@dshokouhi The Chevy Bolt component also uses Selenium.  I struggle with the prereqs on it with Docker.. Hopefully Selenium can be added to the Docker image for an easy to use component requirement.
cc: @sdague

@CCOSTAN  thats what I was thinking, its such a great tool to use in situations like this when a scraper will no longer work.

@dshokouhi Or even using remote selenium leveraging a prebuilt selenium docker image (https://github.com/SeleniumHQ/docker-selenium).  Grabbing data from a remote selenium server and storing in MQTT for HA to use would be pretty amaze-balls.

I'm not sure how I add the phantomjs dependency for Hassio?


JSON parsing error

As seen in the screenshot, the UI is not properly detecting urls and renders everything as one big link. However, looking at the error message (No Patient resources processed from bulk data export.) I wonder if this is more than just a presentation problem. The JSON is valid, can be parsed, and does contain a Patient entry.

@vlad-ignatov We have seen this issue. We think this root cause is that the bearer token expired during the test. We have similar ticket in g10 test kit https://github.com/onc-healthit/onc-certification-g10-test-kit/issues/465.
We are working on a solution to refresh bearer token.
At the same time, you can run these test groups manually, for example, run the test group 1, then test group 2.4. This could avoid token expiration issue.


Update package in pub.dev

current version in pub.dev is 0.0.2
Pls update it as per docs.

Thanks for highlighting this @Im-PJ,
I'll push an update up

@Im-PJ,
Updated 👍


⚠ TorrentLeech has degraded performance

In ff85b0e, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 3328 ms

Resolved: TorrentLeech performance has improved in a908076 after 6 minutes.


Don't draw lines outside the chart area.

As discussed in #1779 the ability to set min and max options for the axes might cause some problems when we have data outside of those settings causing it flow outside of the chart area.
As being shown by @Raf2k:

This fixes it.

I'm not really sure about all possible edge cases though or if this even is the best approach to the problem.
There's also probably some tests that could be written along with the fixes. Not sure how though

Oh, I forgot to clean up some stuff in the tests. Will fix right now!

Could you please give us a before and after screenshot.

Is this still an issue? Cause it seems to be working for me on beta-2.

This looks good so far, but I have not reviewed this code thoroughly.

I think this got fixed in a slightly different way in 2.0.0
There is a call to clipPath before the drawing occurs so nothing outside the chart area is visible

Yeah, it looked fixed in beta-2 and I don't think this is an issue on v1 so I guess we can just close this?


Add failedRefresh to the  metadata of a refreshed element.

When a readRefresh fails, trace  (the failed read metadata) is left in the element's [propNames.metadata].failedRefresh

Coverage increased (+0.1%) to 82.594% when pulling f108ec0e1e9f5080e3875a875025159e15451ef0 on fix/refresh-status into 31a0529001bd3afa473b6934b5bad8df179f0687 on master.


Rebase failed

I was trying to update the gh-pages branch of my repository fetching from master

Brackets 1.1.0-15558 (mac), Git 2.2.1 & Brackets-Git 0.14.9
Rebase failed
Error: /usr/local/Cellar/git/2.2.1/libexec/git-core/git-rebase: line 47: git-sh-setup: No such file or directory

call: git rebase --ignore-date master
Error: /usr/local/Cellar/git/2.2.1/libexec/git-core/git-rebase: line 47: git-sh-setup: No such file or directory
    at Object.exports.toError (file:///Users/federicozivolo/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/ErrorHandler.js:162:19)
    at NodeConnection.<anonymous> (file:///Users/federicozivolo/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/Cli.js:176:44)
    at j (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:26860)
    at Object.k.fireWith (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:27673)
    at NodeConnection._receive (/utils/NodeConnection.js:492:34)

see https://github.com/zaggino/brackets-git/issues/816


fix(config): Error when browers option isn't array

I forgot that the browsers option must be an array. This commit catches this case and throws a more descriptive error instead of something a long the lines of TypeError: undefined is not a function
I think I've got everything formatted properly. Happy to make changes.

Thanks, but checks like this should go into lib/config.js, specifically into this function. Also if you add a check like this, please include a unit test for it :)

Rodger. Will do!

@Dignifiedquire Done. The unit test feels brittle. Chai's expect().to.throw is a bit goofy. Thoughts?

Updated with feedback and rebased from upstream.

Thanks, could you fix linting please?

Done. Thank you for your patience!

Thank you for staying with it :octocat:

hi, in which scenarios will the browsers option wont't be array?

Seems like a fancy solution, but i read that Array.isArray(); is actually a slow option. Wouldn't it be better to check it like this?  if (config.browsers && config.browsers.constructor !== Array) {};

@DarthCharles I dont think it's bottle neck :)
Thanks


missing numpy cuda math functions, e.g. matmul

I have a kernel runing with guvectorize on 'cpu' and 'cuda'
Since there is no matmul support, I have to implement this outside of the kernel as a jitted function decorated depending if it is 'cpu' or 'cuda' used.
This swould be a relative "easy" way to generate functions like matmul, transpose. I do not how to add patches, but I could write the functions example.
Not sure if this is related to Docs 6 & 8 but it wold be usefull to have those functions and additional ones when using 'cuda' and basically extend http://numba.pydata.org/numba-doc/latest/cuda/cudapysupported.html#built-in-functions

Thanks for the request. The following is under development and may help?

There is no e.g. matmul support for CUDA directly from Numba.
The reason for 1. is in part that core linear algebra functionality really needs to come from bindings to e.g. the cuBLAS library so that performance optimal BLAS kernels are doing the necessary work.
Numba supports the __cuda_array_interface__ specification, which means it can interoperate with CuPy.
CuPy provides np.dot along with many more NumPy like functions.

Failing this, pyculib provides a cuBLAS wrapper.

CuPy from what I saw is more like C, and I would prefer to use python :)
pyculib has not seen too much activity in the last year or so
I will use my approach with jitted function and I'll wait for the feature to be added

Just to check we are talking about the same thing. When I say CuPy I mean  https://github.com/cupy/cupy

CuPy : NumPy-like API accelerated with CUDA

It is a Python package that has a NumPy like API and targets CUDA GPUs. No C or C-like behaviour involved unless you want to write custom kernels/do lower level things. As you asked for NumPy like functions, they support a lot https://docs-cupy.chainer.org/en/stable/overview.html, which was my reason for suggesting it. The specific matrix product related functions are here https://docs-cupy.chainer.org/en/stable/reference/linalg.html#matrix-and-vector-products.
Hope this helps?

CuPy looks definitely interesting, but it would be nice if numba can integrate natively some of those functions
I would prefer to leave this issue open as a feature request.

CUPy is definitely interesting, especially for numba interoperability:
https://github.com/cupy/cupy/pull/1760
Also I made some test runs using cupy & tensorflow vs numpy
https://github.com/cristipurdel/python/blob/master/linalg_solve_np_cp_tf.py


Printing of Tamil language characters

Output of mplcairo.get_versions()
{'python': '3.9.12 (main, Jun  1 2022, 11:38:51) \n[GCC 7.5.0]', 'mplcairo': '0.5', 'matplotlib': '3.5.3', 'cairo': '1.16.0', 'freetype': '2.11.0', 'pybind11': '2.10.0', 'raqm': None, 'hb': None}

The tamil words in x-axis are not printed correctly.
Eg:- சொல்லுங்க

Will mplcairo resolve this ?

It will, but you need to also have raqm installed, which is currently not the case (get_versions says 'raqm': None).

It will, but you need to also have raqm installed, which is currently not the case (get_versions says 'raqm': None).

I installed raqm using sudo apt-get install libraqm-dev . Still it shows like this . Any pointers ?

What does get_version return?

The error got solved . Needed to update,upgrade and restart the machine . Once I did I got the required output. Thank you so much.


Endless loop of "not having enough farms to deploy storage nodes"

Network: devnet
3bot ID: weynandkuijpers.3bot
When deploying a VDC on devnet and selecting the option to select which farms to deploy the initial VDC in you end up in a loop "of not having enough nodes / farms" to deploy on.

In case of using 1 farm to distribute the 10 zdb on, it won't work as devnet only one farm has 7 nodes, and in case of using 2 farms or more to distribute 10 zdb on, it won't work as well, as we have only one farm on devnet lochristi_dev_lab.


How to predict something?

Sorry, I feel a little bit stupid, but I can't help me other ways.
I'm playing with the reuters-example and it run fine. So I do have now a trained model. Thats where all examples end as far as I can see.
I read about to save the model. So I could load it later to use again.
But, how do I use these models to predict a new text? Do I use models.predict() ?
Do I have to prepare this text in a special way? I tried it with
import keras.preprocessing.text
text = np.array(['this is just some random, stupid text'])
print(text.shape)
tk = keras.preprocessing.text.Tokenizer(nb_words=2000, filters=keras.preprocessing.text.base_filter(), lower=True, split=" ")
tk.fit_on_texts(text)
pred = tk.texts_to_sequences(text)
print(pred)

model.predict(pred)

But I always get
(1L,)
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-80-dbddb6dc0010> in <module>()
      5 tk.fit_on_texts(text)
      6 pred = tk.texts_to_sequences(text)
----> 7 print(pred.shape)
      8 print(pred)
      9

AttributeError: 'list' object has no attribute 'shape'

Have you any reccomondations, how to start or follow up?
Thank you.
Kind regards
Ben

Yes, you should preprocess your training and testing data in the exactly same way.
If your are running the example file reuters_mlp.py, you should look at the line 30 (or line 29), which says:
X_test = tokenizer.sequences_to_matrix(X_test, mode='binary')
Then you know what you should do:
pred = tokenizer.sequences_to_matrix(text, mode='binary')
model.predict(pred)

Warning: I think you should use the same Tokenizer() as it is in the reuters_mlp.py instead of a newly created Tokenizer() object.

Well, I recommend you to learn some basic knowledge about Python objects, say: list, dict, etc.
And some knowledge about useful libraries, say: numpy.
model.predict() accepts a numpy array as its parameter, rather than a list.

Thank you very much for your help!
I'll try to dig deeper into python and kreas.

Hi, @soloice

you should preprocess your training and testing data in the exactly same way.

So I have to use the same tokenizer for fitting model and for prediction for new data. So I wonder what the canonical way for saving tokenizer for future usage? Should I use any third-party libs for that (cPicke e.g.)? I didn't find any save method in Tokenizer class

@feeeper I'm afraid that you'll need to pickle it yourself.

Thanks a lot for the answer!

@soloice @fchollet curious if this is still the case? I would love if it was possible to save the Tokenizer along side my model

agree with michael-erasmus, this feature is must have


[Fix #11998] Fix an error when inspecting blank heredoc delimiter

Fixes #11998.
This PR fixes an error when inspecting blank heredoc delimiter.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

We should have a lint cop for this!

Naming/HeredocDelimiterNaming might detect it by default instead of creating a new Lint cop. WDYT?
https://docs.rubocop.org/rubocop/cops_naming.html#namingheredocdelimiternaming

@koic Yeah, that's a good idea!