App Service Probing using a dependant Service

The above diagram showing that an App Service instance should be removed from routing or restarted due to a failure in DB is bad practice. Probing should ONLY check the health of the app alone and NOT any dependencies (same as for Kubernetes). In the above case the DB could be down and then ALL instances would return a HTTP 500 - this is also a cascade error and the circuit breaker should be used.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 20c42a13-f6cb-1e60-0438-f7f32b86a336
Version Independent ID: 79e7dd15-24aa-2e6d-cee7-7714e1940c11
Content: Monitor the health of App Service instances - Azure App Service
Content Source: articles/app-service/monitor-instances-health-check.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@wardlen Thank you for the feedback.
You are correct that the App Service health check does not distinguish as to why the app instance is offline. The diagram is showcasing that there are a number of items that can make your instance appear offline.
App Service Health check only cares about ensuring that customers are receiving healthy instances so it will take instance out of routing as deemed necessary. And yes, if your DB is offline then it's likely all of your instances will be offline and what occurs to these instances beyond this is not going to change that there is likely human intervention required to restore the DB. Once the DB is restored, the instances should start working again. App Service Health check will continue pinging the instances every minute even when removed from rotation. Once they report as healthy, they are placed back into the rotation.
The App Service Health check is not meant to be a full blow health probe and customers should build in automatic recoveries on their other services such as their DB.
For now, we will leave the article as is as this document was approved by the product group and our document team. I also have review it and do not find anything that warrants changing at this time. If you have concerns, please reply here.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[add-resource] New Skeleton

For your consideration: Perlkonig/grav-skeleton-knowledge-base
This is my first skeleton package. Let me know if I blew something up. I just copied and edited the .dependencies file. I don't actually understand how it works 😄

I've added this, but it won't work with just a .dependency file. You need to actually build the whole ZIP package and upload it and add it to the release for the skeleton.
http://diblas.net/blog/how-to-build-a-grav-skeleton

Ahh! Thanks for the link. Will do that ASAP.

Like that? I couldn't run the script linked because I don't have access to composer CLI. I just walked through the script manually.

I think you have some missing plugins (simplesearch for example).  The package should basically unzip and just 'work', as you can see below, the current one does not.

I see. So that composer create-project line is what runs the the .dependencies file? I don't have access to composer CLI on my host. I'll have to do it manually.
I'll try again tonight when I'm at my home setup and can properly test.

bin/grav install should install stuff from .dependencies

OK. So manually I need to do the following:

Download and extract the latest Grav.
Delete user folder and replace with sample content.
Run bin/grav install to install all plugins (and theme?).
Zip it up.
Extract somewhere accessible by your webserver and test.
If it's good, then upload zip file to GitHub.

When a later Grav version is released, do I just update the ZIP file of my current release? Or should I bump the version number and create a new release?

If your skeleton is not actually changing, i would just rebuild with newer versions of things and re-upload to current release

Now I'm getting parsing errors with my .dependencies file. Is it being parsed as YAML? Also, do I have to also list all the dependencies in the theme blueprint file as well as in the .dependencies file?

I'm not motivated enough to make this work right now.  Closing this for now. I'll try again another day, perhaps.

I hear ya, we've had near 100 kids come by tonight!

Btw I really don't want to add some support to this in the devtools plugin. We have a shell script we use in our builds, but a simple version than can be run from the cli would be handy right ?

FYI we have devtools (not to be confused with the plugin) which is a very close version to what we use for building up our skeletons automatically.
I haven't kept an eye on it in a while but it should still work and you should be able to create a skeleton in a more conventional and easy way with it.
https://github.com/getgrav/devtools

It would. Better would be some documentation. Again, it may just be me. I just don't understand how all the parts are fitting together. If the theme is set up properly (and I see I need to go back and add the dependencies to mine), and a git repository containing the user folder is up (as is mine), the rest should be eminently automatible (that's not even a word). Anyway, brain fried, and I don't plan on doing any front-end development in the near future, so I'll stick to the plugins for now. I'll come back to this later on. Thanks, though!

Yah i have it on my list to document better.. but here's the gist of it anyway:

install fresh grav into 'some directory'
copy / clone your skeleton repo as 'user' folder in this site
this skeleton should have a .dependencies file at the root of it, ie user/.dependencies after copy/cloning.  Your skeleton should contain config, pages, etc, but nothing in user/plugins or user/themes.  Your .dependencies file should have references for this.

Now from the root of your site, run:

$ bin/grav install

This will use the .dependencies file it finds in user/ to install all that stuff you reference (plugins, theme).

At this point, your site should be fully functioning.  Just zip it up and attach that to your release.

Well I have a working demo site, so theoretically I should just be able to zip that up, but nobody else will be able to rebuild it because the .dependencies file is not parsing properly (something with count-views; is it parsed as YAML?). I also need to add the dependencies to the theme's blueprints.yaml. Finally turning off the lights and going to bed. Maybe this will all be clear in the morning 😄

Finally figured it out. I'm just blind. Fixed the .dependencies file and released. It seems to be working now. Let me know.

You apparently added it already. Thanks.


trimFilePaths feature

I'm using this package to display something like :
select * from `foo` limit 100; 0.35ms [edit in /var/www/vhosts/jack/app/Http/Controllers/FooController.php:15
I'm having to trim the file property to remove the application path, ie in the above case /var/www/vhosts/jack
I thought it would be worth trying to add it to the package itself, so that you could do something along the lines of: Backtrace::create()->applicationPath(__DIR__)->trimFilePaths()->frames()) - then you can access the trimmedFile property to access it. and if no application path is provided, it will just be null.
I wanted to do this directly on the file property, but it was being used here - hence why I added a new property.
There might be a better way - so open to suggestions,  If you don't think it's worth it no problem.

Some of the tests are failing. Could you please take a look?

Thanks!


Pilot integration tests

I want to be able to quickly test that:

Pilot runs
logs info and error messages
creates Events.
Waits for PilotResource
Identifies the correct pilot resource
Ignores unrelated PilotResource
Updates PilotResource status
Starts a sub-process database
Logs output of its sub-process
Detects problems in the database
Syncs the configuration of the database with the PilotResource configuration
responds to SIGTERM
Drains the database before it shuts down.
Shuts down.

I think it should be possible to do this all in a single test process.

Start an Kubernetes API server in a goroutine or start a kubectl proxy sub process or launch the tests in a pod.
Start the Pilot in a goroutine
Wait for the pilot to shutdown before exiting the test.

/kind bug

So I'm not sure if this should all be a single huge e2e test. It'll make it difficult to determine what is actually failing without digging through log messages. Ideally, log messages should help us 'drill down' once we know what is failing (i.e. through a particular scenario failing).
We've got some prior work on integration tests in cert-manager already that it'd be good to use as a 'template' (for consistency between projects).
For example, 'detects problems in the database' is a huge umbrella, and really needs to be broken out into multiple subtests e.g.: "Elasticsearch process correctly identifies that xyz occurred and logs an Event to the apiserver".
"Logs output of it's subprocess" can be deferred to a unit test where we mock the io.Reader and io.Writer etc.
Unless that's not what you meant by a "single test process" 😄


make schema content.source.git.url stick to format git@host:org/repo.git

enforce CI to check errors like this https://github.com/openshift/ocp-build-data/pull/526

Codecov Report

Merging #17 into master will increase coverage by 0.19%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   77.63%   77.82%   +0.19%
==========================================
  Files          11       11
  Lines         228      230       +2
  Branches       29       29
==========================================
+ Hits          177      179       +2
  Misses         50       50
  Partials        1        1

Impacted Files
Coverage Δ

validator/schema/image_schema.py
100.00% <100.00%> (ø)

validator/schema/rpm_schema.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fix Helm 2.8.1 linting issues

Fixes a few linting issues I ran into when running make chart with Helm 2.8.1.

nameOverride wasn't specified in values.yaml
daemonset.yaml using wrong/non-existing podLabels and podAnnotations
keepalived.useUnicast wasn't specified in values.yaml

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

/lgtm

@steven-sheehy thanks!

Coverage remained the same at 17.005% when pulling 10d31bf5dd7991c0dd023767083affd3ffa9812c on steven-sheehy:master into a6830dd722e85ef271f01c464c784eea42222aab on aledbf:master.


Connecting to Windows 2012/2012R2/2016/10 with -fast-path and non-RGX/GFX color depth gives a black screen

I'm not sure this is a bug.
As said in the title, if I connect to a windows server from version 2012 to 2016, with disabled fast-path (-fast-path) and a color depth which is not RFX or GFX, I can only get a black screen.
xfreerdp /v:_host_ /d:_domain_ /u:_user_ /bpp:32 -fast-path

Connecting to older OSes (i.e: 2008R2) works.

Sounds like deprecation to me, need to check though.

Ok, this is a deprecation issue.
closing.

Can you explain what you mean by "this is a deprecation issue"? Does that mean disabling fast path is deprecated?
I'd really like to run a session without fast path to help debug #4934 .

@chrislong then good luck with finding a (recent) server that still supports that.
Anyhow, what does debugging the referenced issue have to do with this?
That is independent of the transport but an application issue (local and remote sizes changes going into a loop)

Oh, I didn't know recent servers required fast path. Thanks.
The issue is that I'm looking at a packet capture to help understand (and explain to someone at Microsoft) the weird behavior from #4934. Wireshark understands some RDP, but it fails to parse a lot of the traffic and I also had problems figuring it out by hand, and I think it's because it doesn't understand fast path. I was hoping to disable fast path so it could parse the packets better.


Not able to include Content-Type: text/calendar in multipart/alternative

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
5.600
Where did you get the app from?
F-Droid
Android version
8 (Lineage OS)
Device model
Xiaomi Mi5 Plus
Steps to reproduce
As discussed at https://gitlab.com/techbeeat1/jtx/-/issues/277, an Android application sends Intends and creates an email in K9, which has this structure:
Content-Type: multipart/mixed
  Content-Type: multipart/alternative
    Content-Type: text/plain; charset=utf-8
    Content-Type: text/html; charset=utf-8
  Content-Type: text/calendar

The structure shall be instead:
Content-Type: multipart/alternative
  Content-Type: text/plain; charset=utf-8
  Content-Type: text/html; charset=utf-8
  Content-Type: text/calendar

The author of the application says he uses the Android API correctly, and it is the Email application, which does not create the correct MIME structure.
Expected behavior
Not applicable.
Actual behavior
Not applicable.
Logs
Not applicable.

There is no Android Intent that supports the case you're asking for.

I see.  Is there an Android intent, which can create email with Content-Type: text/calendar; method=REQUEST; component=VTODO?
According to the author of an Android application this in the responsibility of the email application — https://gitlab.com/techbeeat1/jtx/-/issues/276.


poptracker: init at 0.25.7

Description of changes
Adds PopTracker, a tracker for randomized games. https://github.com/black-sliver/PopTracker
This is my first contribution to nixpkgs beyond simple package updates. Feedback and guidance would be appreciated.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[x] Fits CONTRIBUTING.md.

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

@MikaelFangel thank you for the helpful comments. I have addressed them in my amendment
I think pkgs/README.md should probably be amended to make a note of how GPL licenses should be specified. I might get around to that at some point

@a-n-n-a-l-e-e Thanks for your review. I have addressed your comments as best I could. enableParallelBuilding is quite a nice thing to have.

 The makefile has a todo comment about implementing actual tests. This implies that the tests might not actually be meaningful, which is why I didn't bother adding them in the first place.

  Message ID<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>"EmailMessage","potentialAction"<EMAIL_ADDRESS>"ViewAction","target": "https://github.com/NixOS/nixpkgs/pull/280039#pullrequestreview-1826445132","url": "https://github.com/NixOS/nixpkgs/pull/280039#pullrequestreview-1826445132","name": "View Pull Request"},"description": "View this Pull Request on GitHub","publisher"<EMAIL_ADDRESS>"Organization","name": "GitHub","url": "https://github.com"}}]

I have fixed the patch and cleaned up unnecessary code. I decided to remove the test for now because I think that it doesn't add anything meaningful.


pass identifier from build to submit

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
Pass applicationId/bundleIdentifier from build process to submit when aoutosubmit.
How

value from build take precedence over values defined in submit profile in eas.json
extend build context with platform-specific values

Test Plan
Run build with autosubmit (make sure that value passed  from build is used)

/changelog-entry bug-fix Override applicationId/bundleIdentfier when auto-submitting after build.


Adds attributes to operands_and_partials member functions

Summary
I'm slowly working on fixing up operands_and_partials. This PR just adds inline, const, noexcept attributes to the methods of the edges, and since std::tuple<>'s default constructor in C++14 is constexpr it makes all the container_operands etc. functions that return back empty tuples into static constexpr member functions
Tests
No new tests since this just changes attributes of existing member functions
Side Effects
We need to check this works on rtools gcc since they only have partial support for c++14 and std::tuple's default constructor is not constexpr till 14
Release notes
Cleanup attributes on operands_and_partials
Checklist

[ ] Math issue #(issue number)

[x] Copyright holder: (fill in copyright holder information)
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@SteveBronder is this ready for review?

@SteveBronder is this ready for review?

Actually I think this is superseded by #2343

Actually I think this is superseded by #2343


502 when attempting to complete upload

🐛 Bug Report

[x] 📁 I've Included a ZIP file containing my librephotos log files
[x] ❌ I have looked for similar issues (including closed ones)
[ ] 🎬 (If applicable) I've provided pictures or links to videos that clearly demonstrate the issue

📝 Description of issue:
Making some changing to the upload functonality in UhuruPhotos, I keep bumping across this issue.
Performing the /complete operation after uploading all chunks results in a 502 which internally looks to be a worker timeout which causes the service to restart.
I believe i made a mistake in the chunking of the new process, perhaps causing wrong chunks to be uploaded (missing a byte perhaps) so the complete operation can't process the item.
If that's what's happening. I feel the error in that scenario should be something meaningful and not a 5xx
Please provide additional information:

💻 Operating system:  docker running on Ubuntu
⚙ Architecture (x86 or ARM):  x86
🔢 Librephotos version: latest dev
📸 Librephotos installation method (Docker, Kubernetes, .deb, etc.):  docker

🐋 If Docker or Kubernets, provide docker-compose image tag: latest

📁 How is you picture library mounted (Local file system (Type), NFS, SMB, etc.): local file system
☁ If you are virtualizing librephotos, Virtualization platform (Proxmox, Xen, HyperV, etc.):

log.zip

I link this issue here, for the other 502 error you encountered: https://github.com/LibrePhotos/librephotos-docker/issues/104


Need some Lua magic to get reliable messaging

There must be a simple way to get messages in order and reliably, e.g. simple ever increasing sequence number that is assigned from withing Lua, then messages are persisted in a log (ZSET with int53 key) for a period of time - Kafka style. Clients use pub/sub and just query the zset for a missing sequence number. Everything is atomic inside Lua scripts in Redis. Really want to avoid using Kafka or another additional moving part for persistent messaging. Need to google or ask SO if there is already a messaging framework on top of Redis, leveraging Lua. It must be even simpler and more reliable than zmq! Then we could plug Spreads into Ractor.

Note that on a cluster pub/sub is poorly scalable, but the key notifications are OK. There is an issue about this in Redis repo.

Do not need this.


Noir Closures Implementation Progress

I'm opening this as central place to track progress on closures and related features as well as to discuss them.
I'm looking for input on the two big points at the bottom (arrays of closures & handling of incompatible closure types), as there are a few question marks that we should get sorted out.
Basic

[x] Rewrite closures to regular function accepting environment structs - https://github.com/noir-lang/noir/pull/1959

[x] Closures can be used as return values. - https://github.com/noir-lang/noir/pull/2357

[x] Support mutable captures.

[ ] Handle returning closures that use internal mutable variables, e.g:

fn make_counter() -> fn() -> Field {
    let mut x = 0;
    || {
         x = x + 1;
         x
    }
}

I have implemented this locally, but in some cases (including the one above) it generates code that triggers this bug: https://github.com/noir-lang/noir/issues/2316

[ ] stdlib - The signatures of higher-order functions in the stdlib will need to be changed to work with closures. We can do a lot of that with the environment capture syntax we added fn[env](args) -> ret, or we can wait until we have Fn traits.

[ ] Introduce traits for callable objects such as closures (i.e. Fn).
Blocked on missing traits features

Arrays of Incompatible Closure Closures
We could transparently handle arrays of closures with different environments by rewriting them to tuples. Iterating over them can be implemented by unrolling the loop during monomorphization. One problem with this idea is calling a function at an index not known at compile time, we may have to introduce hidden branching to handle it.

[ ] Support arrays of closures by treating them as tuples.

[ ] Unroll loops over closure arrays in the monomorphization pass in order to be able to instantiate the callee for the correct environment types.

Handling of Incompatible Closure Types

[ ] Settle the language design question regarding expressions involving incompatible closure types. This problem arises in two situations:

if/match statements where the different branches return closures with different environment types.
functions returning closures where different exit paths may return closures with different environment types.

The current implementation treats these as compile-time errors, but this may be surprising for many users. As a minimum, we must document this limitation in the Noir manual.
Alternatively, we can rewrite such expressions to hidden enum types than can capture all alternative closure environment types. Later, when the closure is used, Noir will emit hidden branching code to handle every possible case.
This functionality will be complex to implement as the hidden branching must also be applied at call-sites that accept closures as parameters. In the presence of multiple closure parameters, we’ll need nested branching. Performing such transformations behind the scenes may be highly controversial because it can lead to an explosion in the number of required constraints for otherwise simple and innocently looking programs, so perhaps the users will benefit from doing the harder work of rewriting their closure-based code to avoid the use of incompatible closure types.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

  fn make_counter() -> fn() -> Field {
    let mut x = 0;
    || {
         x = x + 1;
         x
    }
}

I think this could should produce an error that x was captured by copy and not by reference, so it is immutable. If we wanted to capture a mutable variable we'd need to do:
fn make_counter() -> fn() -> Field {
    let x = &mut 0;
    || {
         x = x + 1;
         x
    }
}

stdlib - The signatures of higher-order functions in the stdlib will need to be changed to work with closures. We can do a lot of that with the environment capture syntax we added fnenv -> ret, or we can wait until we have Fn traits.

I'm all for doing this sooner rather than later. It's a small change so let's do this after #2383
As for both Arrays of Incompatible Closure Closures and Handling of Incompatible Closure Types, I'd say these are unnecessary for your work on implementing closures. However, these could be useful in the long term. I just don't see them as being implemented soon since it entails a lot of implicit generic managing and corner cases.

Is the work ongoing? Should we break the remaining tasks into standalone Issues?

@Savio-Sou we can mark this as complete with

Introduce traits for callable objects such as closures (i.e. Fn).
Blocked on missing traits features

Being the only remaining feature to break out into a separate issue. I'm not exactly sure we need it though considering we have syntax for closure types already: fn[Env](Params) -> Ret.


kindle

Will not detect mansions of madness app
any way of fixing it

See also https://github.com/NPBruce/valkyrie/issues/779
If you need a file manager to copy the OBB, download the app Total Commander

I will close that for now. You can still comment if you still have issues and we can reopen it.


forceCleanWorker should allow wildcards

If you have multi worker that dies an untimely death, there may be multiple workers that need to be cleaned. For example, you have a server that has 11 jobs processing on a multiworker. If that server dies because ec2 networking issues, you need to clean up redis. (this happened to us  today) You can't simply pass the IP of the dead server into queue.forceCleanWorker You need to pass IP+PID:. It'd be nice if we could pass wildcards or a filter function into forceCleanFilter to determine if the worker should be forced cleaned.
Thoughts? Will you take a PR?

I think I would prefer to keep queue.forceCleanWorkerexplicit (because it's so dangerous) by requiring a single worker name.  However, creating another wrapper around it like prototype.cleanOldWorkers which could accept a base ip would be good!
Send in a PR!

Bump.  Any progress?

Nope. Totally forgot about it!

closing due to inactivity.


Refactor multiple functions to one main in getRandomThing

Due to the complexity of this problem, the required solution will be released in version 1.2.0 (As well as #17)

Many functions was deleted and only 'шип' command left, so current module was refactored to single file for shipping logic, which is easier to maintain


AttributeError: 'DataFrame' object has no attribute 'as_matrix'

My problem occurs in File "crowdcount-mcnn-master\src\data_loader.py", line 38, in init
den = pd.read_csv(os.path.join(self.gt_path,os.path.splitext(fname)[0] + '.csv'), sep=',',header=None).as_matrix()
Did anyone solve this problem?  :D

Now I found use 'value' instead of 'as_matrix()' is useful.  :D

To be precisely, use 'dataframe.values' instead.


add animation

please add animation on a new branch and call it animation

sorry ! I couldn't find a solution to this problem

@faslm These links scale-animation, entrance-animation are shown that materialize uses jQuery to make animation though. So this link of animation login pages with jQuery would help. BTW this google search query is helpful, google.com/search?q=jquery+animated+login+register.

this is also a beautiful background using css 😍 https://codepen.io/davinci/pen/YxwwWd


Detect unknown jrd keys

This fixes an issue where a JSON error message is returned.

Does the spec say there can't be any other entries?

This PR addresses an issue in requires_resource_uri. This test has the assertion.
assert_that(calling( lambda: interpret_payload_as_jrd(response_without.payload_as_string())), raises(RuntimeError))

The interpret_payload_as_jrd tries to create a ClaimedJrd object and validate it. Some servers return a JSON error status for this case. For example, a server might return something like {"error": "no resource query parameter"}. The current code thinks this is a JRD document and fails the test.
The spec say:

The JSON Resource Descriptor (JRD), originally introduced in RFC 6415
[16] and based on the Extensible Resource Descriptor (XRD) format
[17], is a JSON object that comprises the following name/value pairs:
    o subject
    o aliases
    o properties
    o links

This doesn't explicitly state that no other keys can be in the JRD, but that seems to be the intent.
I'd prefer removing the non-JRD assertion from the test instead. I don't think it's relevant to the requirement being tested and it's debatable whether it's a valid assertion. There's nothing in the spec that states what content types can be returned for a 400 error so a JRD is as valid as anything else.


Compact quantiles sort fix

also added reset() to sampling classes

Coverage increased (+0.003%) to 98.71% when pulling 5d55c5b863a7477738dca5ddf3b2a8272652eace on compact_sort_fix into aaf12b3aa52009a532f11868c7be484a1fd8f6d8 on master.


Can't login from Owncloud

I cant login from within Owncloud App.
Log in works with https://myserver/rainloop perfektly, all goes well.
But from inside Owncloud rainloop says "Authentifizierung fehlgeschlagen" auth failure.
What happend?
Owncloud 7.0.4 and Rainloop App 4.3 with rainloop 1.9.1

In the latest version of RainLoop you should use other urls.
Webmail
https://owncloudserver/index.php/apps/rainloop/app/
AdminPanel
https://owncloudserver/index.php/apps/rainloop/app/?admin

Now it is a part of owncloud package.

I don't understand.
In owncloud I click on the rainloop Icon and the login dialog is shown.

In owncloud I click on the rainloop Icon and the login dialog is shown.

What do you expect to see?

I cant log in, as I say in first post.
In what way the URLs you give me should solve the Login failure?

/ ownCloud / Admin / RainLoop Webmail /

Use this link to enter RainLoop admin panel.
After that you should to configure you mail domain.
http://www.rainloop.net/docs/configuration/

OK, thanks, now I understand. Sorry for my misunderstanding.

no problem :)

Hi, RainLoop
I am unable to authenticate with the admin back panel i am using the shared hosting version of owncloud are you able to help me?

http://www.rainloop.net/docs/configuration/
Default login is "admin", password is "12345".

i also have the same problem, i can log into my Rainloop admin panel, but get 'authentication failed' every time i want to log into my Rainloop email account through Owncloud

Try to disable plugins. When I disabled the plugins, I was able to login to all configured accounts.
I used whitelist/blacklist/fullscreen video on logon

@RainLoop You can close the issue. I don't have it on ownCloud 8.2.2 and the last Rainloop package.


⚠ Bofrost.be has degraded performance

In 730d89b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1012 ms

Resolved: Bofrost.be performance has improved in 3f56cd1 after 13 minutes.


Add RPM building ability

zypper install --allow-vendor-change Chatterino-ubuntu-22.04-x86_64-5.15.2.rpm
nothing provides 'libboost_filesystem.so.1.74.0()(64bit)' needed by the to be installed chatterino-0.0.0-0.0.0.x86_64


How to combine LSTM layers with varying input-data shapes (word + character)?

Hi,
I have a NN in which I train a seq-2-seq model.
I want to combine word-level and character-level information. Currently I get an exception saying setting an array element with a sequence which probably origins in the way I provide word and char information
The words are represented like this
[
[1 2 3 4]  # sequence A
[3 2 1 2] # sequence B
...
]

For the characters I have an additional dimension e.g. one word has several characters
[
  [
    [1 2 0] # word 1 in sequence A
    [1 2 1 ] # word 2 in sequence A
   ....
  ]
]

The modelling of the input data seems, thus, to be wrong. How would you correctly train models, one on word and one on char to concatenate them? I attached my code as minimal (not-)working example which demonstrates this error. The error messages changes if using Tensorflow but the error occurs for both frameworks.
Code:
import numpy as np
from keras.preprocessing import sequence
from keras.models import Sequential
from keras.layers import Dense, Activation, Embedding, TimeDistributed, Bidirectional
from keras.layers import LSTM, Merge, Dropout
from keras.utils import np_utils

np.set_printoptions(threshold=np.nan)

def mapValuesCharLevel(sequences, map):

    all = []
    for s in sequences:
        sout=[]
        for w in s:
            wout=[]
            for c in w:
                wout.append(map[c])
            sout.append(wout)
        all.append(sout)
    return all

def buildIntMap(input):
    startIdx=1
    map = {}
    for s in input:
        for w in s:
            if w not in map:
               map[w]=startIdx
               startIdx+=1
    return map

def mapValues(input, map):
    out = []
    for s in input:
        out_s = []
        for w in s:
            out_s.append(map[w])
        out.append(out_s)
    return out

def padWordLevel(data, max,longest_sequence):
    aout=[]
    for s in data:
        sout = []
        for w in s:
            zeros = [0] * max
            zeros[:-len(w)] = w
            sout.append(np.array(zeros))
        while len(sout) < longest_sequence:
            sout.append(np.array(0))
        aout.append(sout)
    return np.array(aout)

#TRAIN = "en_train.txt"
#TEST = "en_test.txt"

#word_train_token, word_train_label = nnu.loadCorpus(TRAIN)
#word_test_token, word_test_label = nnu.loadCorpus(TEST)

word_train_token = word_test_token = [
    ['Great', 'Western', 'said', 'it', 'had', 'a', 'sharp', 'increase', 'in', 'margins', 'in', 'the', 'recent', 'third', 'quarter', '.'],
    ['Margins', 'are', 'the', 'difference', 'between', 'the', 'yield', 'on', 'the', 'company', 's', 'earning', 'assets', 'and', 'its', 'own', 'cost', 'of', 'funds', '.'],
    ['But', 'a', 'reduction', 'in', 'one-time', 'gains', 'on', 'the', 'sale', 'of', 'various', 'assets', 'and', 'an', 'increase', 'in', 'the', 'company', 's', 'provision', 'for ', 'loan', 'losses', 'held', 'down', 'the', 'earnings', 'gain', ', ', 'the', 'company', 'said', '.']
]

word_train_label = word_test_label = [
    ['NNP', 'NNP', 'VBD', 'PRP', 'VBD', 'DT', 'JJ', 'NN', 'IN', 'NNS', 'IN', 'DT', 'JJ', 'JJ', 'NN', '.'],
    ['NNS', 'VBP', 'DT', 'NN', 'IN', 'DT', 'NN', 'IN', 'DT', 'NN', 'POS', 'VBG', 'NNS', 'CC', 'PRP$', 'JJ', 'NN', 'IN', 'NNS', '.'],
    ['CC', 'DT', 'NN', 'IN', 'JJ', 'NNS', 'IN', 'DT', 'NN', 'IN', 'JJ', 'NNS', 'CC', 'DT', 'NN', 'IN', 'DT', 'NN', 'POS', 'NN', 'IN', 'NN', 'NNS', 'VBD', 'RP', 'DT', 'NNS', 'NN', ',', 'DT', 'NN', 'VBD', '.']
]

longest_sequence = max(len(s) for s in (word_train_token+word_test_token))
print("Longest seq: ", longest_sequence)

word_tokenMap = buildIntMap(word_train_token+word_test_token)
word_labelMap = buildIntMap(word_train_label+word_test_label)

word_trainTokenInt = mapValues(word_train_token, word_tokenMap)
word_trainLabelInt = mapValues(word_train_label, word_labelMap)

word_testTokenInt = mapValues(word_test_token, word_tokenMap)
word_testLabelInt = mapValues(word_test_label, word_labelMap)

word_vocabSize=len(word_tokenMap)

word_trainTokenIntPad = sequence.pad_sequences(word_trainTokenInt, maxlen=longest_sequence)
word_trainLabelIntPad = sequence.pad_sequences(word_trainLabelInt, maxlen=longest_sequence)
word_testTokenIntPad = sequence.pad_sequences(word_testTokenInt, maxlen=longest_sequence)
word_testLabelIntPad = sequence.pad_sequences(word_testLabelInt, maxlen=longest_sequence)

word_maximal_value_A = max([ys for sent in word_trainLabelInt for ys in sent])
word_maximal_value_B = max([ys for sent in word_testLabelInt for ys in sent])
word_maximal_value = max([word_maximal_value_A, word_maximal_value_B]) + 1

word_train_label = np.array([np_utils.to_categorical(seq, word_maximal_value) for seq in word_trainLabelIntPad])
word_test_label = np.array([np_utils.to_categorical(seq, word_maximal_value) for seq in word_testLabelIntPad])

longest_word = max(len(w) for s in (word_train_token+word_test_token)for w in s)
print("Longest word: ", longest_word)
char_list = list(set(''.join([w for s in word_train_token for w in s])))

char_indices = dict((c, i) for i, c in enumerate(char_list))
indices_char = dict((i, c) for i, c in enumerate(char_list))

char_trainTokenInt = mapValuesCharLevel(word_train_token, char_indices)
char_testTokenInt = mapValuesCharLevel(word_test_token, char_indices)

char_trainTokenIntPad = padWordLevel(char_trainTokenInt, longest_word, longest_sequence)
char_testTokenIntPad = padWordLevel(char_testTokenInt, longest_word, longest_sequence)

#print("Word data: ", word_trainTokenIntPad[0:2])
#print("Char data: ", char_trainTokenIntPad[0:2])

word_maximal_value_A = max([ys for sent in word_trainLabelInt for ys in sent])
word_maximal_value_B = max([ys for sent in word_testLabelInt for ys in sent])
word_maximal_value = max([word_maximal_value_A, word_maximal_value_B]) + 1

word_train_label = np.array([np_utils.to_categorical(seq, word_maximal_value) for seq in word_trainLabelIntPad])
word_test_label = np.array([np_utils.to_categorical(seq, word_maximal_value) for seq in word_testLabelIntPad])

print(word_trainTokenIntPad.shape)
print(char_trainTokenIntPad.shape)
print(word_train_label.shape)

EMBEDDING_DIM=64
LSTM_DIM=64

word = Sequential()
word.add(Embedding(input_dim=len(word_tokenMap), output_dim=64, input_length=word_trainTokenIntPad.shape[1]))
word.add(Dropout(0.2))

char = Sequential()
print("Num chars: ", len(char_list))
char.add(Embedding(input_dim=len(char_list), output_dim=64, input_length=word_trainTokenIntPad.shape[1]))
char.add(Dropout(0.2))
char.add(LSTM(LSTM_DIM, return_sequences=True))
char.add(TimeDistributed(Dense(word_maximal_value)))

merge = Sequential()
merge.add(Merge([word, char], mode="concat"))
merge.add(Bidirectional(LSTM(LSTM_DIM, return_sequences=True)))
merge.add(TimeDistributed(Dense(word_maximal_value)))
merge.add(Activation('softmax'))

# try using different optimizers and different optimizer configs
merge.compile(loss='categorical_crossentropy',
                 optimizer='sgd',
                 metrics=['accuracy'])

for i in range(0,20):
    print("===== EPOCHE "+str(i+1)+" =====")
    merge.fit([word_trainTokenIntPad,char_trainTokenIntPad], word_train_label, nb_epoch=1, shuffle=True, verbose=1)

Hello!
Actually I'm facing the same problem
have you solved the issue?

No, sorry. Still unsolved :(

def padWordLevel(data, max,longest_sequence):
aout=[]
for s in data:
sout = []
for w in s:
zeros = [0] * max
zeros[:-len(w)] = w
sout.append(np.array(zeros))
while len(sout) < longest_sequence:
sout.append(np.array(0))
aout.append(sout)
return np.array(aout)
change line sout.append(np.array(0)) to sout.append([]), it works.
Maybe you should check whether the padding works as aspected.
The padding result should be an (3, 33, 10) array. but not  (3, 33), with the lower level element have both vector and int.
My implement of the char seq padding function.

def padWordLevel(char_TokenInt, maxlen, longest_seq):
    char_TokenIntPad = []
    for char_in_token in char_TokenInt:
        char_in_token = list(char_in_token)
        for j in range(longest_seq-len(char_in_token)):
            char_in_token.append([])
        char_TokenIntPad.append(pad_sequences(char_in_token, maxlen=maxlen, padding='post'))
    return np.asarray(char_TokenIntPad)

Hope it helps.

Hi,
I'm also trying to find a solution for that problem. As far as I understand, the embedding layer in Keras only accepts a two-dimensional array. For the character-level embeddings above, a three-dimensional array is defined (num_sentences x num_words_per_sentence x num_characters_per_word). Therefore, I get the following error when running the code above:
ValueError: Error when checking input: expected embedding_2_input to have 2 dimensions, but got array with shape (3, 33, 10)
Is there a nice way to solve this?

@mrmutator any updates?
I met the same issue, my embedding inputs have 3 dimensions, and seems keras embedding does not support this.

Is there any update??
I'm having the same issue. Keras seems to accept only 2-dimensional array!!

I changed it by using tf.nn.embedding_lookup, it can support 3-dimensional inputs.
embedding = tf.Variable(tf.random_uniform((features_total_size, n_embedding), -1, 1))
deep_inputs = tf.placeholder(tf.int32, (batch_size, time_step, features), name='deep_inputs')
embed_x = tf.nn.embedding_lookup(embedding, deep_inputs)

@Neway6655 Can you give a complete model with the shape of your data please ? I'm struggling to combine word+character embedding with Keras..
Many thanks :)

@RA-Danny I don't know how to do this using Keras, I am using tensorflow's embedding api directly.

Wondering if the problem was solved properly.
On a side-note, how did you decide the architecture of the network? Are you following a specific publication/logic/heuristic?

I find this website here: https://guillaumegenthial.github.io/sequence-tagging-with-tensorflow.html
It uses Tensorflow 1.x not Keras but the bottom line is that you run a neural network which generates LSTM outputs that you just append to the word embedding lookup table of the actual sequence tagging task you are trying to solve.
Its solved with pre-processing essentially. Doing both, char and word, in a single network does seem to work with Tensorflow/Keras.
Hopefully, this helps someone..


Writing a message into an OracleAQJms Topic throws NPE if a trace context already exists

I get a NPE (java.lang.NullPointerException: value of jms.queue == null) when trying to enqueue a message to an Oracle AQJms Topic. It works with an Oracle Queue.
The Exception atually is thrown in MutableSpan.java:266 when we try to tag a span with the destination name.
The issue is caused when trying to read the channelName from the Destination in MessageParser.java
AQjmsDestination implements Queue and Topic so in this method we always treat any destination as queue. Which results in null when reading the queueName if it's actually a Topic.

will fix the bug, but the rationale is curious.. if what you are given implements both queue and topic, queue should return...
  @Nullable static String channelKind(@Nullable Destination destination) {
    if (destination instanceof Queue) return "queue";
    if (destination instanceof Topic) return "topic";
    return null;
  }

oh sorry I understand now. ps does either of the name methods return for you?

ex do both getQueueName() and getTopicName() return null?

Hi @adriancole
One of them always returns null. For a Topic getQueueName() returns null. For a Queue getTopicName() returns null. So maybe the instance of is not even required. I currently do not know any other way other than returning the method that does not return null.
Thanks a lot.

https://github.com/openzipkin/brave/pull/1099 thanks for helping find the bug. excellent work!


Problem with Device mapping when Geo Data received by Traccar Server

Hi,
I am facing the following problem while sending Geo positions data from device or OsmAndProtocol.
Case 1.
Device not registered with Traccar Server, sending Geo Lat Lon data
Result : 400 Bad Request, because device unknown
Summary : Valid Result as expected
Case 2.
Device registered with Traccar Server, sending Geo Lat Lon data
Result : Empty
Summary : Valid Result as expected
Case 3.
Device not registered with Traccar Server, sending Geo Lat Lon data
Result : Empty
Summary : Invalid Result not expected
Here in Case 3. the Geo Lat Lan data is mapped with previously device in Case 2 and updated this will happen to all unknown device received by the server until it again gets valid registered device.
I have fixed this from My end. I advice it is a very serious error in case if Server receives Geo Data from any unregistered device.
Regards.
Aripaka V

It works as intended. Traccar has internal cache that's synced with database every 5 minutes by default.

May be it is connected with #2372

That's absolutely correct the issue is related to that
Now all my positions are mapped to different device id which is creating problem in tracking devices
Is there is any fix related to this Issue

I have implemented a solution for #2372.


major slowdowns

hey there,
after a few hours running the client always becomes unresponssive and cpu hogging.
i guess either the timers did overflow or due to floating accumulation error inside the code (havent checked)
feasible reproduction steps (maybe) is to open the client, and advance the timers epoch a few days during runtime :s
anyways, great viewer : )
keep up the good work

r-lyeh

Thanks for reporting! I wasn't yet aware of this and I'll have a look.


feature/267-platform-plugin

In order to use SIERRA with something other than ARGoS, I need to be able to
specify what platform  the robots are running on, and things like that. This
will only effect stage 1/2.

Probably introduce some bugs with this huge commit, but they should be minor...


Cannot resolve bootstrap path in scss

This is a help request.
I use coreui as composer extension.
The problem is to resolve the node_modules path.
I use gulp-sass to build my css.
Here my file structure:

And here is my theme directory in my application from where I want to build my css:

My style scss:

My gulpfile.js:

To get it work I must change the bootstrap path as follows:

Is there a way to use gulp and not to change the path ?

Operating system and version (Windows 10)
Browser and version (Chrome latest)

Right now I'm facing a similar issue. It's impossible to use coreui packages in monorepo with lerna or yarn workspaces. The node_modules folder can be several levels up with all dependencies hoisted. So there shouldn't be any direct references to the node_modules folder.

I'm facing this same issue with Yarn. CoreUI is importing bootstrap from the node_modules directory and it's not finding it. I think CoreUI should not be referencing bootstrap, it should be manually included by us because when deploying to production the node_modules directory gets overwritten and that change is lost.

I found a PR to fix this issue https://github.com/coreui/coreui/pull/4 but the original author seems to have deleted his fork.

We don't want to duplicate Bootstrap's css styles, so we overwrite bootstrap variables and generate our own version of Bootstrap to keep CoreUI's styles as light as possible. That's why we're referencing to Bootstrap files.

That's why we're referencing to Bootstrap files.

Yes, but the question is not about 'why' it's about 'how'. The PR mentioned about seems to be a solution. Anyway, on my side it works properly in the lerna/yarn environment without any extra dev dependencies. I just remove node_modules from the import path.

Yep, I also tried removing the node_modules reference from the import statements as a workaround. But if I deploy to Heroku for example, this change will get lost because Heroku will install a new copy. That's why I forked the repo, changed those imports there and then I installed with Yarn using my own copy.

For anyone else with the same problem, I fixed it by modifying my root package.json and setting coreui and bootstrap to nohoist.
"workspaces":` {     "packages": [       "packages/*"     ],     "nohoist": [       "**/@coreui/**",       "**/bootstrap/**"     ]   },

I found a PR to fix this issue coreui/coreui#4 but the original author seems to have deleted his fork.

I totally forgot about this merge request and thought that I could remove the original repo. I will try to restore it for what it's worth.

I have the same issue:
in my local style.scss:
// If you want to override variables do it here
@import "variables";

// Import styles
@import "../node_modules/@coreui/coreui/scss/coreui";

// If you want to add something do it here
@import "custom";

and can't find node_modules


use openssl to strip bag attributes as well

fixes https://bugzilla.redhat.com/show_bug.cgi?id=1548619
@bparees this will actually strip the gorp preceding the -----BEGIN CERTFICATE ------ line
@bostrt fyi ... with the discussion in the bugzilla, the last change in flight got list and merged before I could update

/lgtm

hmm ... everything completed including the deprovision phase, but the test run is still considered running ...
will check again in the AM, and poke as needed

yeah i'm seeing this in PRs all over the place..prow says the job is running, but it's complete.  and it won't rerun it either.

/retest

/refresh

/refresh

I was just about to do that. LOL

@kargakis nothing other than client debug and info in the jenkins-operator logs either ... weird.

On Thu, Mar 22, 2018 at 8:05 AM Michalis Kargakis<EMAIL_ADDRESS>wrote:

/refresh
I was just about to do that. LOL

I need to remember that command - thanks

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/jenkins/pull/556#issuecomment-375281122,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADbadJhBPjuYfSlLR-c6EqGu_sTMxVXdks5tg5OJgaJpZM4S1hUF
.


Add a method to bulk load resources

The idea is to avoid sending hundreds of requests to the API at the same time. Suggestion:
GwentAPI.map(cards, GwentAPI.cards.one);

Released with 2.0.0


Bot treats invalid word as valid guess

Example: word taxe is not present in en/combined but the bot treats it as a valid guess.

This is intentional, and the checking mechanism is still present, just disabled by default.
It is part of the Settings struct, specifically the property "checkWords" controls this. The reason I disabled this (by default) is that the lists in Japanese and English are too small atm and would not permit a lot of fairly standard words. Two things can and should be done here:

Expose this option to the user, so that they can choose if they want the checking.
Add more lists, maybe also add an option to mark a list as "lookup" (like the "default" flag) only, and then possible re-activate the option by default. The "lookup only" thing requires changes in the ListManager, but they should be very doable overall.


Add Dasam Patsahi Ka Granth

As controversial it would be, I believe that access to information can't be denied.

Moved to #39


Add properties to authentication token

We are migrating from a different authentication system, and it would be helpful to have a simple way to map the previous user ID to a property within the JWT, such as external_user_id. This approach would allow us to avoid creating an additional table in our application database and eliminate the need for frequent database calls.

Similarly to this, I need to be able to set aud within the AuthKit JWTs, which does not seem possible.


repo name idea

termo-mas-talvez-eu-saiba-programar - I was hoping that the new version would be the one that maybe you knew how to program. As happy as I am about your personal development, I expected a gradual ascendancy. anyway, congratulations!!!!

RLOF. I was not expecting a message like that. Anyway, I going to show you a new version of the project that you, maybe, see the "gradual ascendancy".  Just wait.

right on! I can't wait to see the project improvements :-)


Milestone 3 Tasks

Setup your EMR cluster

[ ] Attach the screen shot.

Setup your browser, jupyter environment and connect to the master node

[ ] Attach the screen shot.

Develop a Ml model

[ ] Complete Milestone3-Task3.ipynb
[ ] Upload to JupyterHub.

Obtain best hyperparameter settings using spark's MLlib

[ ] Complete Milestone3-Task4.ipynb
[ ] Upload to JupyterHub.

Submission instructions

[ ] Python 3 notebook, with the code for ML model in scikit-learn (Milestone3-Task3.ipynb).
[ ] PySpark notebook, with the code for obtaining the best hyperparameter settings (Milestone3-Task4.ipynb).
[ ] Screenshots:
Setup for EMR cluster.
Setup your browser, jupyter environment & connect to the master node.
Your S3 bucket showing model.joblib file (From Task 3 Develop a ML model using scikit-learn)


Server.license: No such file or directory

Can't start container:
/sbin/entrypoint.sh: line 74: /usr/local/WowzaStreamingEngine/conf/Server.license: No such file or directory
How to fix this?

Hi @vavoluk, Please note that Wowza Media Systems now packages and publishes an official docker image. Instructions for which are well documented in https://www.wowza.com/forums/content.php?867-How-to-set-up-Wowza-Streaming-Engine-using-Docker
I suggest you to evaluate the wowzamedia/wowza-streaming-engine-linux docker image for your usage.


Update to Appium 1.21.0 on iOS 12 & 13 to fix flake

Goal
Fix flake in features/thermal_state.feature:45
For yet-unknown reasons, the UIApplicationDidEnterBackgroundNotification isn't reliably delivered to the fixture on iOS 13 (or 14) when backgrounded by Appium 1.17.0
Changeset
Updates iOS 13 E2E tests to Appium 1.21.0
Testing
Verified by manually running scenario on BrowserStack and with full run on CI

...and now it's just happened on iOS 12 😞

...and now it's just happened on iOS 12 😞

I'll update this PR with the same change and see what happens.

Superseded by #1320


Adjust the number of launch files shown in the launch_files tab

Would it be possible to adjust the number of launch files shown in the launch_files tab? Consider the following example:

The most recent five launch files are shown. However, I usually have around 10 launch files (and different combinations of them) that I regularly use. Would it be possible to show more? Otherwise, I always have to manually dig into the package launch files and this is time-consuming. Thank you very much!

you can change the Launch history length parameter in the Settings tab
http://fkie.github.io/multimaster_fkie/chapter_usage/settings.html

Thank you very much! I will close the issue.


parquet.py does not include filters method for ParquetDataset through pip

The file included in the distribution on pip does not include the latest parquet.py filters method for ParquetDataset found in this repository.

This will be part of the 0.10 release. I'm not sure on timeline as we are blocked on some release process / packaging improvements. Hopefully by sometime in July


Lint for default flag configurations

Depends on: https://github.com/cilium/cilium/pull/34254, https://github.com/cilium/cilium/pull/34255, https://github.com/cilium/cilium/pull/34256
Introduce exhaustruct into the tree specifically to lint that all configuration flags provided between the user and Cilium, or between modules within Cilium, are explicitly populated.
The intent here is twofold:

Encourage developers and reviewers to consider what sane defaults should look like by forcing commits with new flags to explicitly initialize the default configuration

Make it easier to grep for default configurations within the codebase, even when those configurations are defined for a specific Cell.

Historically this was always the case because the flags library expects a default setting to be provided. However, with the move towards defining configuration in the Hive/Cell framework, those references often directly pointed to the Configuration object inside the Cell module, which may or may not be explicitly initialized. This PR extends this property to Cell-provided configuration.

/test

/test


Sort plugin evaluates insert index for batched events

can.batch.start();
listB.push({ id: 'a' });
listB.push({ id: 'a' });
listB.push({ id: 'a' });
can.batch.stop();

"// BATCHED; SORTED..."
"    Sorted..." // <-- Upon creating the list
"    Sorted..."
["type:", "length", "batchNum:", undefined]
["type:", "length", "batchNum:", 9] // <-- This event caused the sort
"    Swapped? 1, 2"
["type:", "length", "batchNum:", undefined]
["type:", "length", "batchNum:", 9] // <-- This event caused the swap
["type:", "length", "batchNum:", 9]

The first two sort's make sense. One for calling new the, next for having batched changes. The swap happens as a result of _change being called subsequently with the same batchNum.
http://jsbin.com/poduxa/6/edit?js,console

Closed via #1718


Preventing the fail of the 'ETIMEDOUT' code in 500 'Internal server error'

Currently ETIMEDOUT code is handled by the defaultErrorHandler as a [500] Internal Server Error.
It should be handled as the regular ECONNRESET || ENOTFOUND || ECONNREFUSED || ETIMEDOUT codes.

Coverage remained the same at 88.301% when pulling e4961b1d18e95a4e8793a40e393eb4e2f02f1f4c on aremishevsky:master into f212a047ef45898ce6053b622cada4b0617b9ddf on chimurai:master.

@chimurai could you please review this PR?

Can someone review and merge this PR?

Can someone review and merge this PR?

Can you add ETIMEDOUT to this unit test with the 504 response?
https://github.com/chimurai/http-proxy-middleware/blob/master/test/unit/handlers.spec.ts#L101

Can you add ETIMEDOUT to this unit test with the 504 response?
https://github.com/chimurai/http-proxy-middleware/blob/master/test/unit/handlers.spec.ts#L101

@chimurai I did that https://github.com/chimurai/http-proxy-middleware/pull/480/files#diff-5dce20e137b0cf4ccabefadbb48baf14ef3c1d68b2dad6aa2b45d4bfbfbdf2e3R107

@chimurai I did that https://github.com/chimurai/http-proxy-middleware/pull/480/files#diff-5dce20e137b0cf4ccabefadbb48baf14ef3c1d68b2dad6aa2b45d4bfbfbdf2e3R107

Oops, missed that. 🙈

Oops, missed that. 🙈

Thanks for this PR!

Thanks for this PR!

Could you please let me know when the new version would be available on the npm?

Could you please let me know when the new version would be available on the npm?

Published in v1.1.0


LDAP connection never closed

Hi, I noticed that the LDAP connection was never closed in ldap.go (Login method), thus resulting in connections not being performed after the TCP connection timed out on the LDAP server.
Adding a "defer s.Close()" worked for me.

Thanks for the issue! I'll fix it shortly.


Have to setup skywalking for each nginx location

I am trying to integrate skywalking with nginx, what bothering me is that it seem I have to setup skywlking for each location.

can I just setup global skywalking for all nginx locations?

That is how things work. You need to intercept all requests.


Feature suggestion: Add a data-modal-id attribute

I came across an issue where I wanted a component that had hyperlinks inside of it to be clickable and bring up a modal. Wrapping the whole component in an anchor tag doesn't work because of the anchor tags inside the component itself don't close properly or consistently in different browsers.
My solution to this is to instead wrap the component in a div and add a data-modal-id attribute which is interchangeable with the href attribute you would use on the anchor tag. Then in the _getElementContext function I do a check for the existence of data-modal-id and select the element based on that. So if the property passed to _getElementContext isn't a string or a .hash it checks for the data attribute.
Your markup would look something like
.
Does this sound like something worth adding?

You can just attach a click listener to the element and open() manually.
HTML:

JS:
const modal = new VanillaModal;
// ...
document.querySelector("div.component-wrapper").addEventListener("click", function() {
  modal.open("#whatever");
}, false);```

This issue can be closed.

I'd say this goes against convention a little though by having to add the listener to a class rather than data attribute. Classes should be reserved for styling.

Could very easily delegate handling a data attribute to a global listener like so:
document.addEventListener("click", function({ target }) {
  if (
    !target.hasAttribute("data-modal-open") ||
    !target.hasAttribute("data-modal-id")
  ) return;
  modal.open(`#${target.getAttribute("data-modal-id")}`);
}, { pasive: true });

This library is extremely lightweight, but very easy to work with.


Config not working

Hello! Tried to run the config setup as provided and got this error; not sure how to proceed from here. It seems a lot has changed about MongoDB's Windows installation since this video was created - perhaps a refresh is needed?

It looks like the installation defaulted to setting up the service and using the location as provided, without using the config files given...? So, perhaps not a problem.

Hi @mathieum906 They must have updated the installer (doesn't surprise me!). Glad they got it all running. Are you good? Can we close the issue?


fix editMod bug

Codecov Report

Merging #211 into master will increase coverage by 4.63%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #211      +/-   ##
============================================
+ Coverage     61.08%   65.72%   +4.63%
- Complexity      685      738      +53
============================================
  Files           130      130
  Lines          2503     2500       -3
  Branches        297      297
============================================
+ Hits           1529     1643     +114
+ Misses          794      702      -92
+ Partials        180      155      -25

Impacted Files
Coverage Δ
Complexity Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...in/java/seedu/address/commons/util/StringUtil.java
94.73% <ø> (ø)
8.00 <0.00> (ø)

...eedu/address/logic/commands/ListModuleCommand.java
80.00% <ø> (ø)
2.00 <0.00> (ø)

...main/java/seedu/address/model/module/ModuleId.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...c/main/java/seedu/address/model/student/Email.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...rc/main/java/seedu/address/model/student/Name.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...c/main/java/seedu/address/model/student/Phone.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...in/java/seedu/address/model/student/StudentId.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...a/seedu/address/model/tutorialgroup/DayOfWeek.java
72.72% <ø> (ø)
5.00 <0.00> (ø)

...a/seedu/address/model/tutorialgroup/TimeOfDay.java
82.35% <ø> (ø)
7.00 <0.00> (ø)

... and 28 more

Continue to review full report at Codecov.

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


why there is no getExpiresTime() method in CacheItemInterface

When I implement save() and saveDeferred() method, I have to get the expires time of the cache item, so I think there should be a getExpiresTime() method in CacheItemInterface.
By the way, when I create a new cache item ready to store into the cache pool, what should isHit() return ?

Too late to add it, unfortunately.

@aequasi I'm sorry I don't understand why too late ?

PSR-6 has already passed, this repository is tied to the PSR.


Wizard for VC Widgets

I created a copy of the Function Wizard which creates widgets for all channels of the selected Fixtures.
Depending on the channel group, the widgets are configured differently (XY Pad, RGB, Click & Go, Intensity, ...).

i wanted this feature for myself, because i often work with different setups and wanted a quick way to setup widgets for groups of fixtures to directly control the channels.
Features i want to add in the Future:

options for widget sizes
option to split the selected features to frame pages
input mapping

Function wizard creates VC widgets too!
Why not adapting the existing tool?

coverage: 31.869% (-0.2%) from 32.066%
when pulling 6b7f5861fab313973c7bd87bab6c0306b9fcd8d3 on 8-Lambda-8:master
into d00a493e1d51647dd83b0ba777e830fa752494c1 on mcallegari:master.

Yes i know, but those widgets are to control the created functions and not to controll the fixtures directly. Because of that i thought it would be better to split them.
But of course i can implement my stuff into the existing one if you like it better like that.
if i do that, should i move the frames of the function widgets into the frame of the "channel widgets" or should i keep them seperate?

Yes, I prefer to have one tool that does everything.
You can add a "Skip" button in the function creation tab.

if i do that, should i move the frames of the function widgets into the frame of the "channel widgets" or should i keep them seperate?

What are "function widgets"?

Now everything is in one Wizard

Now everything is in one Wizard

Thanks for the contribution and Merry Christmas!

OK, looks good. Can you please explain what are the numbers displayed beside the widget descriptions?

The numbers are the channel numbers of the fixture on which the widgets will act

Can we add the wizard shortcut to the VC page as well? I liked the look of that.
This would make sense since it can be used for vc widgets.

Here's another usage case: add a RGB panel (all defaults will do)
Start the wizard and select a few RGB panel rows.
This is the result

I would expect one slider for all the heads, but still someone would argue they want to select specific heads.
Also, listing involved channels in this case would be messy, so I would remove it or change it to "(N channels)" rather than the whole list which (at least for me) is irrelevant

Yes for big RGB Pannels it could be to much widgets, but maybe some users also want this
and yes one Slider to controll all Pixels would be good

maybe like this?
For multihead fixtures (and also groups of fixtures) maybe it would make sense to have multiple Pages of the Frame with the first page to controll ALL and the next pages for each head seperate. Maybe with a limit of pages?

I think the channel numers are a sometimes a helpful info. but yes for longer lists it may be better to write something like "(2-6)" instead of "(2, 3, 4, 5, 6)" or the start channel and the count "(5 channels starting with 2)"
should i put the wizard button also on the VC Page like @yestalgia sugested?

@8-Lambda-8 Is it ready for merge yet?

@ETE-Design no its not ready yet.
like @mcallegari said i have to change the behaviour a bit for RGB Pannels and Multihead fixtures.

Hi, it took a while, but
The Wizard button is back on the VC Page, like @yestalgia wanted:

Can we add the wizard shortcut to the VC page as well? I liked the look of that.

for the Multihead problem i solved it with Frame Pages.
with the 3 checkboxes you can select what pages should be created.
if the "All" page is created, the faders of all other pages have the "Monitor selected channels" option enabled
example 8x60 RGB Matrix:

example 2x Chauvet Intimidator Spot duo

@8-Lambda-8 Can you please merge master into your branch and push? Then I'll be able to test it.

i merged master again into my branch for testing.
is there anything i should change or do you think it is ready to be merged?


SnowPlow installing via CocoaPods

Hi,
I am trying to install Snowplow using cocoa pods, while building its giving me error:
" include of non-modular header inside framework module 'FMDB.FMDatabase' [-Werror,-Wnon-modular-include-in-framework-module]
#import "sqlite3.h""
My project is in Swift.
Thanks,

Please use our Discourse for support: http://discourse.snowplowanalytics.com/


Specify Stream in Mult-Geo 365 capabilites

Stream is not listed in the Microsoft 365 Multi-Geo capabilities.  Stream either does or does not have Multi-Geo capabilities.  Please indicate this in the Multi-Geo documentation.

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

ID: b890c9af-0bab-84c6-8b9d-1ee93cb4683f
Version Independent ID: 98fddfad-754b-ef65-7f5f-5f8bd8f59004
Content: Administering a multi-geo environment - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/administering-a-multi-geo-environment.md
Service: o365-solutions
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@djtrustgod Thank you for your feedback. We have Added the information as M365 Video portal - https://docs.microsoft.com/en-us/microsoft-365/enterprise/administering-a-multi-geo-environment?view=o365-worldwide#video-portal
In a multi-geo tenant, the O365 Video Portal is served only from default geo and all users will be redirected to that central portal url. Hence, the Remote Media Service (RMS) for that region will be used, as follows based on your central location.
Stream is currently available in the following regions:
North America, hosted in the United States
Europe
Asia Pacific
However, Stream is not yet available in the following regions that are currently supported for Microsoft 365 Video, therefore for these local instances, we will use the RMS that is in the closest supported region.
Australia
Canada
India
United Kingdom
Hope this Helps!
@yogkumgit There is no changes required in the documentation. Please proceed with closure.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @djtrustgod Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Add presto-worker-* principals

Store keys for those principals in common "presto-server" keytab
We need multiple principals that include hostname to be able to verify that _HOST placeholder works fine.

Related PR to Presto: https://github.com/Teradata/presto/pull/541


Image Size of Training Data Set

Hi Simon,
The data set you suggested is of size 19 * 19 , (it works) ,but if I want to use images of size 64*64, it does not.
Loading faces..
Traceback (most recent call last):
  File "C:\Users\GM\Desktop\Viola-Jones-master\example.py", line 25, in <module>
    faces_ii_training = list(map(ii.to_integral_image, faces_training))
  File "C:\Users\GM\Desktop\Viola-Jones-master\violajones\IntegralImage.py", line 35, in to_integral_image
    integral_image_arr[y+1, x+1] = integral_image_arr[y+1, x-1+1] + row_sum[y, x]
ValueError: setting an array element with a sequence.

where do I need to change ?
Thanks.

Hi Jim,
I think this is not an issue with the code. I guess you are using color images, are you? However the code was written for monochrome images, which is why it fails.
Regards,
Simon

Dear Simon, You are correct, image type was the issue, that has been resolved, but a new issue  arose, that is MemoryError . I initially tried to train with 10,000 images of size 200*200 and to test with another 10,000 images of same size but failed to do so, then I decrease the number of training images to 2000, but still I get the bellow error  -
Loading faces..
..done. 1000 faces loaded.

Loading non faces..
..done. 1000 non faces loaded.

Creating haar-like features..
..done. 668938 features created.

Calculating scores for images..
Traceback (most recent call last):
  File "C:\Users\GM\Desktop\Viola-Jones-master\example.py", line 31, in <module>
    classifiers = ab.learn(faces_ii_training, non_faces_ii_training, num_classifiers, min_feature_height, max_feature_height, min_feature_width, max_feature_width)
  File "C:\Users\GM\Desktop\Viola-Jones-master\violajones\AdaBoost.py", line 54, in learn
    votes = np.zeros((num_imgs, num_features))
MemoryError

How can I solve this issue?
Also,would you please inform the number of feature values(i.e. length of feature vector) generated for each training image (which are also used to test)?
Is it 668938 (for the above case where I tried to train with images of size 200*200)?
Have you used this implementation in any paper/thesis ?

would you please inform the number of feature values(equivalent to the length of feature vector) generated for each training image (which are also used to test)?

Hi,
sorry for replying late. I am not sure what you mean with "the number of feature values for each image". What the algorithm does is to generate an exhaustive number of Haar-Like features and then subsequently select the "best" features which can then be used as an ensamble, e.g. cascade.
Regards,
Simon


Add heavy snow icons

Codecov Report

Merging #98 (c1f242b) into main (6969abc) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #98   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          637       639    +2
=========================================
+ Hits           637       639    +2

Impacted Files
Coverage Δ

weather_command/_weather.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update manualData.json

Pull Request Test Coverage Report for Build 943928008

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

Totals

Change from base Build 936269061:
0.0%

Covered Lines:
95

Relevant Lines:
1914

💛  - Coveralls


fix: redirect old sdk links to new sdk

links from google or elsewhere can land you on the deprecated js docs
this moves the urls for the deprecated ones so we can still have them referenced, but redirects people so they land on the supported ones

🚀 Deployed on https://66f70f7fc572d2dd4e7cecd8--cozy-fox-0defec.netlify.app


UsbSpecifier for fine-grained device target selection.

Continuing discussion from #60 .  There is a desire to be able to specify the micropython board by attributes like pid/vid/serial_number. This PR aims to address that.
This PR introduces the class UsbSpecifier that takes in a variety of optional keyword arguments. The resulting object can be passed into belay.Device. If an attached board matches all of the specified keyword arguments (like serial_number, pid, etc), then it will be used. If no device is found, or if multiple devices match the provided specifier, then an exception is raised.
@raveslave, this PR is not yet complete, but so far it's in a working state. Would like feedback on naming/usage, and if it satisfies your needs. I'm not a huge fan of the current name UsbSpecifier and am open to suggestions. Aside from feedback, items that need to be completed before merging this PR:

Update docs
Maybe add new CLI features for making this easier; maybe modify the existing belay identify command.

Example usage:
from belay import Device, UsbSpecifier
device = Device(
    UsbSpecifier(
        serial_number="abc123",
    )
)

Codecov Report

Merging #122 (dad598d) into main (a8b5912) will increase coverage by 0.26%.
The diff coverage is 89.47%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   76.63%   76.90%   +0.26%
==========================================
  Files          32       33       +1
  Lines        1318     1355      +37
  Branches      213      221       +8
==========================================
+ Hits         1010     1042      +32
- Misses        240      243       +3
- Partials       68       70       +2

Flag
Coverage Δ

unittests
76.90% <89.47%> (+0.34%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

belay/usb_specifier.py
88.23% <88.23%> (ø)

belay/__init__.py
100.00% <100.00%> (ø)

belay/exceptions.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@raveslave any thoughts?
Also additional notes for related features: I recently just learned that you can have .gitignore in subdirectories. So I would be good with having:
.belay/
    user/
        .gitignore  # contents: "*"
        files-that-shoud-not-be-committed
    dependencies/
        stuff-that-already-exists-and-should-be-comitted

In doing so, we can store user device-configuration data in .belay/user/

checking now, sorry for the delay. looks really promising!

maybe multiple device matches should have its own error type (instead of DeviceNotFoundError)
UsbSpecifier should offer to pass-in bus and address to allow you to pick a specific hub and/or port in a usb-hub
Same as above, those properties should be populated when searching by any other parameter so you can conveniently access bus/address (and include it if printed out)
I like the idea of the CLI having a convenient list/identify feature, if this list is interactive, selecting one could offer to set a local environment var to keep it sticky throught the session?

re:

In doing so, we can store user device-configuration data in .belay/user/

yes, you rarely use ignores in subfolders, but a good way to keep status clean for "local cache", I prefer it in repo rather than ~/ as in many other projects. so if you feel like adding this, I think its a great idea.
ps. we should just make sure it's clear when connecting (log? and print in cli) that it loads from that file, i.e. like ssh -v to see if your keys are being used or not... just so you dont get lost why its connecting to something automagically

Thanks for all the good feedback!

maybe multiple device matches should have its own error type (instead of DeviceNotFoundError)

✅ I added a new exception InsufficientSpecifierError if multiple devices match the provided UsbSpecifier. I'm also fine to change the name if you can think of a better one.

UsbSpecifier should offer to pass-in bus and address to allow you to pick a specific hub and/or port in a usb-hub

I'm trying to not introduce an unnecessary dependency; is location from pyserial equivalent? Example output (copied from your previous post, but just including here for completeness):
{'description': 'Board in FS mode',
 'device': '/dev/cu.usbmodem147201',
 'hwid': 'USB VID:PID=1B4F:0026 SER=e46030f803311824 LOCATION=20-7.2',
 'interface': None,
 'location': '20-7.2',
 'manufacturer': 'MicroPython',
 'name': 'cu.usbmodem147201',
 'pid': 38,
 'product': 'Board in FS mode',
 'serial_number': 'e46030f803311824',
 'vid': 6991}

If it is the same, would we want to keep the name location? If it's not, we can add a dependency like pyusb.

Same as above, those properties should be populated when searching by any other parameter so you can conveniently access bus/address (and include it if printed out)
I like the idea of the CLI having a convenient list/identify feature, if this list is interactive, selecting one could offer to set a local environment var to keep it sticky throught the session?

So lets talk about the interactive CLI bit. Here's what I'm thinking.

There's the existing belay identify command that basically blinks a specified LED (assuming your board has an LED). This makes it easy to identify which board you're talking to. I'm just mentioning this here because it should probably be rolled into this feature.
For the sake of argument, lets call this new command belay select. We can use questionary to do the interactive heavy lifting. Maybe after selecting, promt the user if they want to blink an led and then either confirm or go back to the menu?
Once a device is selected, we have 2 options of saving this info: Save it to a file in .belay/user, or save it to an env var. The issue with saving it to .belay/user is that the script might be "just a script" and not really in a good project structure. So for this, I believe the env var is the way to go. Unfortunately, a child process can't update it's parents environment, so this might be a bit of a dead end. An alternative is for belay select to print out for the user to run export BELAY_DEVICE=some_serialization_of_specified_device.
Assuming (3) gets solved, we could make the port/device-specifier argument to Device as optional, and have it fall back to the env var.

ps. we should just make sure it's clear when connecting (log? and print in cli) that it loads from that file, i.e. like ssh -v to see if your keys are being used or not... just so you dont get lost why its connecting to something automagically

I think the best long-term solution is to add a proper logger, and then allow a verbosity level to be set. I think this would be useful to belay-users and belay-library-developers alike. For the sake of scope, we'll save this for another PR.
I think the action plan is going to look like this:

Lets iron out any UsbSpecifier details in this PR. Things like the bus/address stuff you mentioned, and then also ways of serializing/deserializing the data in prep for step (2).
The interactive cli belay select will be next PR. This PR will also contain the ability to specify a belay.Device() without any arguments (falling back to env var or whatever).
A proper logger will come after that in another PR.

great re: InsufficientSpecifierError, I think the name works!
actually never used 'location' in pyserial but seems to represent the bus and address topology seen in pyusb, but in its own way. so location will be perfect to pin two identical devices connected at the same time.
not sure why my print became 20-7.2 when the format seems to be <bus>-<port>[-<port>], but the docs at least claim its unique so we're good!

regarding the CLI

belay identify is great and I think it should include the led blink

belay select sounds good. no real opinion about prompt libs vs native/homebrew. since the user will be in the cli at this point, I like the idea of the extra blink step as a confirmation step.

true regarding the scope/current working dir.. maybe best to just drop this idea and let the user manage this via source .env or similar

for a while I was thinking pyserial's hwid would be the unique string we want (in setups where bus/hub & port are part of the unique device identifier). but it's simple to just explicitly pass-in all args. drawback is that it gets less obvious what to store in your env variable, or did you have any ideas here?

great stuff! plan 1-2 check, and agree 3 would be its own PR

All sounds good; for this PR I'll do the remaining:

Add the location field.
Add UsbSpecifier serialization/deserialization in preparation for the other features.

so I've been a bit slow on this (I just had my first baby 🥳), but I added location field to UsbSpecifier.

@raveslave I'd still like to add serialization/deserialization of UsbSpecifier in this PR. I guess we need to kinda converge on how we'd like to store the config. There's no generically good way of setting the parent process's environment variable from a python script. An alternative is to store it in some file, but I'm hesitent on this because the user's script might not be in a project. Maybe it's best to just print out the UsbSpecifier representation so the user can copy/paste it into their script? However, It wouldn't directly allow for a default belay.Device() with no args to "just work." So if that's the goal, we need some more ideas.

@roaldarbol In commit c6463eb, I added the list_usb_specifiers. It's usefulness overlaps a lot with list_devices. What are your thoughts on if we were to replace list_devices with list_usb_specifiers completely? And if so, which name to keep.

wow! congrats to you+fam+kid!!
re serialization, I think the easiest approach now is to offer a way to retrieve a specifier that can be easily copied to code and/or an env variable. to me the nicest feature here is to completely remove the needd to ever do ls /dev/tty [tab tab] again, just run the CLI, see whats there, then pass-in the attributes needed in the code and you're done.
documentation-wise, I think its either a matter of locking down on vid+pid and possibly serial. and in the more rare case of you having two or more with an ambiguous serial, you make use of the usb-bus port/hub configuration (which can sometimes be nice to read-back if you require a certain device to be connected in a specific port on a powered hub)
about default Device(), this one is of course tricky, if there is only one device I think it should return it, but how does it know. I know that 'mpremote list' tries to do this, but on my mac it shows the build-in bluetooth modem as well.. without doing some fuzzy micropy detection logic, I think its too risky to simply return the first CDC-ACM device on the bus (unless there is a solid way of detecting the repl)
sounds like a good idea to replace list_devices as its now part of the more powerful/flexible approach

cool! These will be my action items/thoughts:

THIS PR: Add serialization/deserialization of UsbSpecifier to a dict. I think pydantic offers this, so this is super easy. The intent will be that this dictionary-string representation could be stored in an env var like BELAY_DEVICE='{"vid": "0123", "pid": "4567"}'.
THIS PR: replace the body of list_devices with list_usb_specifiers. @roaldarbol let me know if you have any objections.
NEXT PR: In belay.Device, if no device is specified, attempt to access env var BELAY_DEVICE and see if it decodes to a meaningful UsbSpecifier. If not, error our. Later we could add an additional fallback to try and find an available device with fuzzy logic, but it'd have the shortcomings you mentioned.
NEXT PR: I'm in favor of belay select outputing both easy-to-copy-and-paste python code, as well as env var export snippet.

Sorry, have been away on holiday. :-) All sounds good to me! Sounds like a great upgrade - good work guys!

awesome work brian!


Removes sleeps and use Eventually construct in integration tests

Codecov Report
Base: 84.32% // Head: 84.32% // No change to project coverage :thumbsup:

Coverage data is based on head (13752ab) compared to base (1a3d0c3).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #75   +/-   ##
=======================================
  Coverage   84.32%   84.32%
=======================================
  Files          13       13
  Lines        1933     1933
=======================================
  Hits         1630     1630
  Misses        219      219
  Partials       84       84

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

https://github.com/armadaproject/armada-operator/issues/74

My only comment is that are we sure 2s is enough of a timeout for all these operations?


Make helper images linked as well as text

https://docs.google.com/spreadsheets/d/1Fa5e8OniZXCPy3RdMR7B4i5zQpIOzcjhIMh9hfnJ_ZQ/edit#gid=0


feat: Property Editor Header when botname is selected in Project tree

This PR adds a header to the property  editor when the botname is clicked in the project tree and no dialogs are selected. Default routing to main dialog is stopped.

@zhixzhan tests dont pass on ur branch. I think its safe to take this PR into ur branch when u feel its comfortable as i made sure no new tests fail because of this branch.


loading coreml model (SentimentPolarity.mlmodel)

c:\program files (x86)\microsoft visual studio\shared\python36_64\lib\site-packages\coremltools\models\utils.py:92: RuntimeWarning: Unexpected end-group tag: Not all data was converted
spec.ParseFromString(contents)
https://github.com/SwiftBrain/awesome-CoreML-models

Hi @ssgwos, this repo is for WinML documentation. Do you have a specific issue on one of our docs? If not, I will close this issue.


Multiple Hubs

This introduces support for multiple harmony hubs.
To support multiple hubs, the entire API has changed. Both MQTT topics and HTTP endpoints are now namespaces with /hubs/:hub_slug/ to accommodate commands and state querying of each hub on your network.
Endpoints and topics are still namespaced, even if you only have a single hub.
For more info on the changes, read the README on this branch.

This branch also fixes the long standing crash that was being caused by power cycled hubs. 🎉


Draft of lm20 database with parsed html form data (8 new tables)

I am hoping all the sed commands run on ubuntu (i'm on a mac, but tried to be mindful). Let me know if I can edit the table structures or contents, or refactor anything to be more consistent with your other databases.

@ethangalebach, this is beautiful!  I did a little hacking on this tonight, but will definitely prefer your branch.
unfortunately, i found a problem with json-to-multicsv
https://github.com/jsnell/json-to-multicsv/issues/3
this is causing at least some missing data.
we are either going to need to address that, or take another approach of parsing out the data.

I'm happy it worked out! It was a fun exercise. If any other issues like this one arise where I could usefully contribute to the labordata project, let me know.

for sure!


Custom Type for wallet addresses

Passing wallet addresses around as plain string variables seems a bit odd.  A custom address type would be a great improvement.

duplicate of https://github.com/decent-dao/fractal-interface/issues/623


resty.hmac (lua-resty-jwt) returns nil signature

I just wanted to bring to you attention. Should resty.string/sha1 be used?
https://github.com/jkeys089/lua-resty-hmac/issues/2

This issues is caused by older version of openssl.


RuntimeError: DataLoader worker (pid(s)

Hello @kalyanvasudev ,
Thank you for your great job. I have a question for you, please?
I have set the configuration like that:
dm = make_video_datamodule(batch_size=8, num_workers=8)
def make_slowr50_finetuner(num_labels, freeze_backbone=True):
'''Init Pretrained Model, freeze its backbone, and replace its classification head'''
model = torch.hub.load("facebookresearch/pytorchvideo", "slow_r50", pretrained=True)
model.blocks[:-1].requires_grad_(not freeze_backbone)
model.blocks[-1] = create_res_basic_head(in_features=2048, out_features=num_labels)
return model
model = make_slowr50_finetuner(dm.num_labels)
classifier = Classifier(model, lr=2e-4)
trainer = pl.Trainer(gpus=1, precision=16, max_epochs=5, callbacks=[checkpoint_callback])
trainer.fit(classifier, dm)
But after several epochs (about 5 epochs), there is always error like that:
_File "/home/miniconda3/lib/python3.9/site-packages/torch/utils/data/_utils/signal_handling.py", line 66, in handler
_error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 3155823) is killed by signal: Killed.
File "/home/miniconda3/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1003, in try_get_data
raise RuntimeError('DataLoader worker (pid(s) {}) exited unexpectedly'.format(pids_str)) from e
RuntimeError: DataLoader worker (pid(s) 3155823) exited unexpectedly
How to we can fix this error?
When I set worker = 0, there is no problem but time of training is so long.
Thank you very much?

Hi @wenjun90 , it could be also your systems memory constraints not being able to accommodate multiple workers.
To pin point the problem, and weed-out that shared memory is not the problem, could you try with with a smaller batch size and set worker >0 but a small number ?

Closing the issue due to inactivity. Please refer to our later PyTorchLightning VIdeo trainer for more details on how to train video models using Lightning  - https://github.com/facebookresearch/pytorchvideo/pull/158


Community Meeting June 14, 2022

At 11am Eastern, 8am Pacific, 3pm UTC; find your time
Video call link: https://meet.google.com/ohf-kwvd-mrp
Or dial: (US) +1<PHONE_NUMBER> PIN: 936<PHONE_NUMBER>#
More phone numbers: https://tel.meet/ohf-kwvd-mrp?pin=9361820877398
Or join via SIP<EMAIL_ADDRESS>Meeting recordings available on YouTube at https://www.youtube.com/channel/UCfP_yS5uYix0ppSbm2ltS5Q
This issue will collect prospective agenda topics. Add topics you'd like to discuss below!

sharded-test-server demo

Call for ClusterWorkspaceType feedback

Recording https://www.youtube.com/watch?v=r-7DB0MzoDw


Image + Audio + Text input using Llama 3.2 [DO NOT MERGE]

This PR is not in a state to be merged, but it shows how Llama 3.2 can be used to combine image, text, and audio inputs together and get the correct response.
Take a look at llama32_script.py to see how this is done:

llama 3.2 11B vision instruct model is loaded
weights from ultravox-v0_4 (trained on llama 3.1 8B) are loaded without modification
input consists of image and audio

Note: before using the script, a few lines in the transformers library need to be manually commented out to allow for this approach:
https://github.com/huggingface/transformers/blob/main/src/transformers/models/mllama/modeling_mllama.py#L2152-L2155
These lines don't allow you to specify inputs_embeds when vision input is present. Hopefully we can upstream this change in the future.

@farzadab , Are there any updates on this issue?

@kadirnar this was simply a proof of concept. Unfortunately, combining vision into Ultravox is not part of our roadmap.
What is your use-case?

@kadirnar this was simply a proof of concept. Unfortunately, combining vision into Ultravox is not part of our roadmap.
What is your use-case?

I want to create a model like this. However, the gpt-omni repository doesn't share training code and its vocoder isn't good. I want to use Ultravox. However, the model must have image support. Is this possible?
https://github.com/gpt-omni/mini-omni2

The model in this PR will only be able to output text (not speech), but on the input side, yes it does allow for a combination of all three modalities: image, audio, and, text without further training.
Ultravox itself combines audio and text and Llama 3.2 combines text and image, and since they both do this without touching the base LLM, they're changes seem to be compatible (as this PR shows).
Keep in mind that I've only tested the combined model on a handful of samples, so the 3 modalities.


the port option added requires it to be string as all options are using the join which fails

./src/range_repair.py", line 241 called from get_host_tokens uses the self.options.port which is a int type and not string hence the " ".join(command) commands fails with port being int rather than str

Duplicate. Already reported in #37. #42 proposes a fix. Feel free to close this.

Also, workaround is to specify the port at command. That will make it a string.

well i fixed it in code and got it running

@anishek FYI, this bug is now fixed in the master branch.


rate_limit_quota: Quota based rate limiting, part 1

First part of quota-based rate limiting feature. The majority of this PR is to lay out the basic framework and shape of major components with unit test coverage, other functionalities (e.g., periodically usage report,  thread local cache, quota assignment, etc) will be added in next PRs. This PR includes:

Rate limit quota filter basic framework:
Filter factory, filter creation and registration, filter configuration, etc.
Bidirectional GRPC client basic framework :
Grpc stream start and close, message send and receive, callback for the client and filter interaction, etc
Request matching:

matcher tree creation
perform request matching with request header and exact value_match field from the config
on_no_match support (empty bucketId with noassignmentBehavior for now for illustration purpose, will be
modified in the next PRs)

BucketId generation:
Generates the BucketId via either static method (string_value) or dynamic method (custom_value with
typed_config).
Quota assignment caching:
UseBucketId as the key with customized hash/equal function. Will change to thread local storage
Unit tests for the config, filter and grpc client,
Boilerplate doc (mostly as the placeholder, I will finish the doc in another PR #22539 later as development goes )

Note to reviewers:
In sprit of small PRs and making review easier, I plan to divide the whole implementation into several parts.  Hence, I

Removed some refer link in the proto for the ease of a boilerplate doc generation (this feature is WIP).  Also, I have left
some TODOs that will be addressed in next PRs
Temporarily reduced the code coverage requirement for this extension as this is first part of the several PR. Currently, it is ~75% coverage.

Signed-off-by: Tianyu Xia<EMAIL_ADDRESS>

/unassign @mattklein123
Initial commit that just adds back the docs as part of the implementation. Not ready for review yet.

/unassign @mattklein123
Initial commit that just adds back the docs as part of the implementation. Not ready for review yet.

/retest

/retest

/assign @yanavlasov
Could you take a pass on this? Thanks!

friendly ping @yanavlasov

@yanavlasov  PTAL. Thanks!
I have addressed the review comments. The CI doc failure in this PR will be resolved once PR #22539 is merged (That PR is also under review now).

/retest
The CI failure should has nothing to do with this PR

@tyxia i think the tsan problem has been fixed on main here #24252
the other problem is coverage, which is saying:
Code coverage 96.0 is lower than limit of 96.1

is it possible this fail is genuine ?


Use issue forms in the swanky repos

Could you elaborate more about this?
We have issue templates.

Could you elaborate more about this?
We have issue templates.

Using the form to report issues seems like the best way to enforce the standardised format.
(changed the title)
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository