Set a header, after instancation?

Hello guys,
Sorry for my lack of knowledge of the library. I'm trying to set a header after some actions.
I'm currently working on building a small package for Laravel to make some requests to an API.
Because not all requests will require authentication, I have created a method named login() which I would like to set a header on the GuzzleHttp\Client after it's originally instanciated.
Here's what I currently have, take a closer look at the search() method. This will require authentication and I will need the Authorization header set with a token. So I call the aforementioned login() method before I make my request to set this header.
I'm having trouble finding a way to set a header. Can you help?
<?php

namespace Dawson\TVDB;

use GuzzleHttp\Client;
use GuzzleHttp\Exception\ClientException;

class TVDB
{
    protected $token;

    public function __construct()
    {
        $this->username = config('tvdb.username');
        $this->userkey = config('tvdb.userkey');
        $this->apikey = config('tvdb.apikey');

        $this->client = new Client([
            'base_uri' => 'api.domain.com',
            'headers'  => [
                'Accept' => 'application/json',
                'Content-Type' => 'application/json',
            ]
        ]);
    }

    public function search($name)
    {
        $this->login();

        try {
            return $this->client->get('search/series?name=' . $name)->getBody();
        } catch(ClientException $e) {
            throw new TVDBException($e);
        }
    }

    private function login()
    {
        try {
            $response = $this->client->post('login', ['json' => [
                'apikey'   => $this->apikey,
                'username' => $this->username,
                'userkey'  => $this->userkey
            ]])->getBody();

            $this->token = json_decode($response->getContents())->token;

            // Set a header here?
            $this->client->setHeader('Authorization', 'Bearer' . $this->token);
        } catch(ClientException $e) {
            throw new TVDBException($e);
        }
    }
}

Solved this by using a handler.
/**
 * Handle Authorization Header
 */
private function handleAuthorizationHeader()
{
	return function (callable $handler)
	{
        return function (RequestInterface $request, array $options) use ($handler)
        {
        	if($this->token) {
        		$request = $request->withHeader('Authorization', 'Bearer ' . $this->token);
        	}

            return $handler($request, $options);
        };
    };
}

Then added the handler stack to the Client.
$this->stack = new HandlerStack();
$this->stack->setHandler(new CurlHandler());
$this->stack->push($this->handleAuthorizationHeader());

$this->client = new Client([
	'handler'  => $this->stack,
	'base_uri' => 'https://api.thetvdb.com',
	'headers'  => [
		'Accept' => 'application/json',
		'Content-Type' => 'application/json',
	]
]);

@joedawson thanks, it worked for me.


arch: arm: userspace: add ip and sp to clobber

We should be adding a compiler barrier for IP and SP
registers when we are doing syscall generation on
ARMv6-M architecture. The syscall generation itself
only does an SVC trigger; the execution returns to
thread mode and ARM does not guarantee that IP
register is preserved, when we finally get back to
the point where the syscall was invoked. The SP
also needs to be preserved for syscalls returning
64-bit results. In that case the r0 may hold a
pointer to the stack where the 64-bit result was
pushed, That is, the stack pointer may have been
changed due to the syscall, and C code needs to
know that.
Signed-off-by: Ioannis Glaropoulos<EMAIL_ADDRESS>

@erwango could you please do a sanity check on nucleo_g071rb?  Thanks in advance!

@ioannisg, sanitycheck is passed (vs status on master)

@ioannisg, sanitycheck is passed (vs status on master)

Great, thanks @erwango


(maint) release_notes: move 3.0.0 up; remove conflict marker

Move 3.0.0 to the top of the release notes (above 2.3.4), and remove a
vestigial conflict marker.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1277/

lgtm


Cannot run specs - duplicate factory definition

On creating dummy_app then running rspec:
/home/cjy/.rvm/gems/ruby-2.1.5/gems/factory_girl-4.4.0/lib/factory_girl/decorator.rb:10:in `method_missing': Factory already registered: inquiry (FactoryGirl::DuplicateDefinitionError)

This seems to indicate the problem, if not the best solution in this case:
https://github.com/thoughtbot/factory_girl/issues/638
It's possible this is a bug in factory_girl.

OK, Sorry --- I've just tried this with a clean rvm gemset and I'm not reproducing the issue. So unless someone else can reproduce it, it should be closed.


Add match board in homescreen

Well this match board is added as a component so technically we can put this at anywhere.
Drafting, until all the teaming information gets realeased.

Brackets has been set
Check this if you have time @CloneWith


Fix Chinese translation

Hello @yesmeck, as per readme, translations are updated via our crowdin page.

OK, I've suggested a new translation on crowdin.


Allow image manager to work without any S3. Maybe some daemon/pipeline style workflow.

from @frosty-geek during lean operator coffee

This is already possible when not using the mirror functionality.

Closing than.

I will explain this better in the documentation.

Hope it is better with #325


Documentation improvements

Found some stuff when reading through the docs that I believe we should fix / clarify. Think most of it is pretty straight forward. However, I would like to get input on my changes regarding remoteSelector. From my understanding the remoteSelector is actually returning a new facet and not a selector. I changed the docs to clarify that, but as I said, I would like to get some feedback on that.

Nice updates, Adam! I'm not sure about changing the naming convention of selectors, however. I think remote.md might need to be reverted, but Paulo and Fernando are certainly more qualified than me at making that call.

Regarding the name change of selectors. A call to remoteSelector actually returns a RemoteFacet<V> and not a selector, a selector is instead passed in to the remoteSelector function as the first argument. While reading the documentation I thought that naming the return value from calling remoteSelector a "selector" was confusing and hence the change.
Example:
const userNameSelector = remoteSelector(({ user }) => user.name, [profileFacet])

({ user }) => user.name is the selector and userNameSelector is a RemoteFacet.
Just realised that the order of the parameters in the example in the documentation is incorrect. Will change that in a new commit.

I agree, the result of the remoteSelector call is a RemoteFacet, not a selector. I think the change to userNameFacet is good.
On top of that, we have discussed in the past to rename RemoteFacet to SharedFacet, since it better reflects what it does (memory that is shared with the backend, rather than remote, which might imply a network connection). What do you think? I can do that in a separate PR

On top of that, we have discussed in the past to rename RemoteFacet to SharedFacet, since it better reflects what it does (memory that is shared with the backend, rather than remote, which might imply a network connection). What do you think? I can do that in a separate PR

I think that SharedFacet is a slightly better name than RemoteFacet, since like you say remote implies that there might be a network request. However, if we ought to do the change, then we should also change the name remote everywhere, which means renaming remoteSelector to sharedSelector, RemoteFacetDriverProvider to SharedFacetDriverProvider, etc. It's a pretty large change, so I understand if it might not be worth it though. I'm ok with either (keep it or change it).

Pushed a new commit where I adjusted the documentation in remote.md so example function calls receives correct parameters. It seems like initialValue for both remoteSelector and remoteDynamicSelector has been removed. Is that something we want back? (could be added in a new PR if we want it back)

If we had removed the initialValue, we probably don't need it. Thanks for updating the docs on that.


Frontend: Build the MegaMenu sub component for both Light and Dark mode

Issue Summary
The design for the MegaMenu as part of the Navbar component has being completed and now it needs to be built using our chosen tech stacks.
Figma link
The design for the MegaMenu can be found here. Please make sure what you build follows the design.
Suggested fix

Build the MegaMenu component following the design in the Figma file.
Make sure your codes are maintainable and legible.
After coding the component using React and scss, create storybook for the component.
Test it out on your end and make sure it's working properly.

Coding Rules

Let your codes look neat and readable
No unnecessary spacing and always leave one empty line after each code block
Separate each word with a hyphen - in your scss classes, ids.

PR Rules

Link the issue your PR is for by saying Fixes #PRNumber
Add a description of what your PR is about
Add screenshots or video of the working implementation of your PR

Nobody wants to pick this up?

Please assign me this issue so that I can resolve it under GSSOC-23

Ok sure
You can start working on it @adityavpawar


Problem with tests. Maven doesn't compile

Failed tests:
BrokenDBTest.canDeleteDBOnBrokenLog:92 Can't delete index
Tests in error:
StoreWALTest.discard_corrupted_log:109 » IO java.io.IOException: The requested...
Tests run: 1750, Failures: 1, Errors: 1, Skipped: 4

Fixed in both branches

Thanks!!!
On 19 Jun 2015 19:04, "Jan Kotek"<EMAIL_ADDRESS>wrote:

Fixed in both branches
—
Reply to this email directly or view it on GitHub
https://github.com/jankotek/mapdb/issues/362#issuecomment-113592483.


Release v1.9.1: Add release notes

Please review release notes
@fmcquillan99 @orhankislal @mktal @njayaram2

Looks good, here are my minor suggestions:
new features section:
(1)
+* New function: Sessionization (MADLIB-909, MADLIB-1001)
(2)
+* New function: Pivot (MADLIB-908, MADLIB-1004)

Added a function to do basic OLAP type operations on data stored
in one table and output the summarized data to a second table.

(3)
+* PGXN:  Update PostgreSQL Extension Network for latest release (MADLIB-959)
bug fixes section
(4)
+* User documentation improvements including for hypothesis test docs (MADLIB-1001)
(5)
+* madpack uninstall support for Apache HAWQ 2.0+

I would suggest you also add the class_weight to the SVM part of the release notes, since that is new too.


Add aggregates module

This PR adds a modules called aggregates.py that defined functions used to compute economic aggregates. All these functions were defined before, but contained in firm.py or household.py.  It is now the case that firm.py and household.py contain only functions necessary to solve an individual household or firm problem and any economic aggregates are computed in aggregates.py
A new unit test, test_aggregates.py was added to include unit testing of the functions that used to be testing in test_firm.py but that were aggregate functions.

@jdebacker . This looks ready to me. Are you ready for me to merge this?

@rickecon, yes, this is ready to merge.


Prepare release 3.1.0

[ ] Update changelog
[ ] Update readme

Contributor section
"Stable release" badge
Screen capture
Features: new cheats

[ ] Cut the release, build, prepare the tag and link archives
[ ] Advertise release

Reddit
Steam page
Discord

When I start TQVaultae 3.1.0 a message appears:
"Could not load Text DB".

I have TQAE Atlamntis v.2.7 - current version on Steam.
What is the answer? Why do you publish the wrong version with the description that it allegedly works with TQAE Atlantis 2.7?


[bug] Main window onCloseRequested is not triggered when the window is closed via APIs

Describe the bug
It seems the main window's onCloseRequested is correctly triggered when user closes the app with top right X button or Alt+F4, but not triggered via APIs such as appWindow.close() or exit().
Reproduction

Create a Svelte component or anything equivalent:

<script lang="ts">
    appWindow.onCloseRequested(async () => {
        await invoke('test')
    })
</script>

<button on:click={() => appWindow.close()}>Test</button>

Click the test button
Confirm that the "test" command is not triggered.
However, close the app with Alt+F4. Observe "test" command is triggered.

Expected behavior
Closing the app or main window via API should also trigger onCloseRequested.
Platform and versions
Environment
  › OS: Windows 10.0.22621 X64
  › Webview2: 105.0.1343.42
  › MSVC:
      - Visual Studio Community 2022
  › Node.js: 16.17.1
  › npm: 8.15.0
  › pnpm: 7.12.2
  › yarn: Not installed!
  › rustup: 1.25.1
  › rustc: 1.64.0
  › cargo: 1.64.0
  › Rust toolchain: stable-x86_64-pc-windows-msvc

Packages
  › @tauri-apps/cli [NPM]: 1.1.1
  › @tauri-apps/api [NPM]: 1.1.0
  › tauri [RUST]: 1.1.1,
  › tauri-build [RUST]: 1.1.1,
  › tao [RUST]: 0.14.0,
  › wry [RUST]: 0.21.1,

App
  › build-type: build
  › CSP: unset
  › distDir: ../dist
  › devPath: http://localhost:5173/
  › framework: Svelte

App directory structure
  ├─ .git
  ├─ dist
  ├─ node_modules
  ├─ src
  └─ src-tauri

Stack trace
No response
Additional context
This is needed so that the app can do necessary clean up and save data to file regardless of how user initiates the app termination. Otherwise, we have to duplicate the clean-up logic to all the possible routes.

For exit(), it is a desired behavior since it is meant to close the process as soon as possible. If you want to run code before that, you can just run the code before calling it.
For window.close() This is a known behavior for v1 but is planned to be changed in v2 where there will be window.close() that triggers the event and window.destroy() that will bypass it.

Thanks for clarification. If we can explain this (especially exit() at the moment) in doc that will help clear up the confusion.

in v2 there are now separate close and destroy apis.


Typography: Remove input in favor of subtitle1

Description

Originally introduced in #8754, the input typo was created to consolidate the hardcoded styles found in _input.scss. Now looking back (and checking the material design spec) I recognize that the styles were actually subtitle1 but not hooked up to the overall typography theme provider.
I don't anticipate much backlash considering that the option was never possible before and was only recently introduced. We should maintain parity with the original spec. Now users get the best of both worlds, with the customization opportunities of the Typo property from the earlier PR, and with this going back to subtitle1 (but attached to the theme provider) a more cohesive application style that's more in tune with Material Design expectations.
I consider this a part of the overall effort in #8884,
How Has This Been Tested?

Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

Migration guide:

This removes Typo.input. Use Typo.subtitle1 instead.
This removes all input typo CSS. Use --mud-typography-subtitle1 instead.
This changes the order of the Typo enum

Added to the v8 migration guide at #9953

I don't think the migration guide is complete for this.
Since this removes Input (aka InputTypography in v8) (not mentioned in the migration guide) from Theme.
Are people suppose to use Subtitle1Typography for their inputs?

This changes the order of the Typo enum

I would lean towards that all enums mustset numbers explicitly, and when you remove an enum the old order stays, just the removed gets "reserved" but since it's merged already then ok. But for future it's better to do so for better compatibility.

@ScarletKuro I thought the same but then this was added only recently which means the enum order changed only recently and nobody said anything, and now we are changing it back, so I didn't mention it.

I slightly modified the guide saying that Input class and property was removed, use subtitle1, hope that's correct.

I slightly modified the guide saying that Input class and property was removed, use subtitle1, hope that's correct.

Yes subtitle1 is the replacement for input which is what it should have been originally @ScarletKuro


Support JUnit tests for plugins in flutter/plugins

We really should be writing and requiring unit tests of our plugins.  We had our first "external" (Googler, not on Flutter team) contribution yesterday, and it yielded the comment "We have no tests for this code so please review carefully."
FYI @collinjackson @mravn-google @szakarias

Also filed https://github.com/flutter/flutter/issues/10141 for the ObjC equivalent

I'm closing this based on all of the testability @collinjackson & co have done this year towards testability; it's obsolete at this point.


In Max OS X bundler install raise a error if the memory limit is reached

According to the Docker specified https://github.com/docker/docker/blob/master/image/spec/v1.md#image-json-field-descriptions (more specifically the MemorySwap option), the containers are always executed unsupported the swap.
The solution could be in force for the Docker constigure the azk of containers with swap support.

@nuxlli, the docker documentation says:

The operator can constrain the memory available to a container easily with docker run -m. If the host supports swap memory, then the -m memory setting can be larger than physical RAM.

In https://docs.docker.com/reference/run/#runtime-constraints-on-cpu-and-memory

I managed to enable swap on the fly with a few commands. But you need to add the swap partition to /etc/fstab in debian2docker.
In azk vm ssh:

Login as the root user
$ sudo su

Type following command to create 512MB swap file (1024 * 512MB = 524288 block size):
$ dd if=/dev/zero of=/swapfile1 bs=1024 count=524288

Set up a Linux swap area:
$ mkswap /mnt/sda1/swap2gb

Activate /swapfile1 swap space immediately:
$ swapon /mnt/sda1/swap2gb

List active swaps:
$ swapon -s

How do I verify swap is activated or not?
Simply use free command:
$ free -m

Stress test:
4 VMs each 256M for 30 seconds.
docker run --rm --name stress-test jess/stress --vm 4 --vm-bytes 256M --timeout 30s


Add codespell support (config, workflow to detect/not fix) and make it fix few typos

More about codespell: https://github.com/codespell-project/codespell .
I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback.
CI workflow has 'permissions' set only to 'read' so also should be safe.
TODO

[ ] remove TEMP commit and potentially codespell workflow

thanks a lot @yarikoptic ! no objections on my side, seems like a nice addition but I don't have experience with codespell, maybe @glemaitre has an opinion about adding the workflow or not? IIRC in nilearn we use the precommit hook but not the github workflow

We have codespell correction in scikit-learn but we did not put it as a pre-commit. But it might be OK just to use as a pre-commit hook.


[BUG] Missing dependencies

Describe the bug
Your requirements are not correctly documented, causing them to not be installed, resulting in a broken package. The install_requires parameter in your setup.py needs to include everything in your requirements.txt.
To Reproduce
Steps to reproduce the behavior:
python3.11 -m venv .env
. .env/bin/activate
pip install GoDaddyPy==2.5.1
python -c "from godaddypy import Client, Account"

Expected behavior
Package should be importable after installation. Instead, package throws the error: ModuleNotFoundError: No module named 'configloader'

I'll create a PR to fix this.

Solved in #54, #56

I am encountering this issue with the latest 2.5.1


Missing bracket

On line https://github.com/hed-standard/hed-javascript/blob/e579f8dcf351d850bf6c7bd97600a9be99f86acd/data/HED8.1.0.xml#L5630 the opening bracket ( is missing.

The specification and the schema have been separated.  This issue is transferred to the hed-schemas repository issues.


test-bot: allow colors on Azure

I don't know if it will work, will test here.

Yeh, looks like it's working 👍

Seems to work just fine.

Thanks @dawidd6!


[DEFECT] Create a module for RAVEN Errors

Defect Description
Most of the RAVEN model entities use the Runners.Error exception to check for Errors
from model evaluations. Even if this approach is correct, it causes a slow down of parallelization and importing of such modules because it requires the import of each Runner subclass (and/or pickling) since the Runners are   defined in the init class
What did you expect to see happen?
Fast import and parallel
What did you see instead?
Slow down of import and parallel
Do you have a suggested fix for the development team?
Create a module where the RAVEN errors are defined (Runners Error, Sampler Errors, etc.)
Describe how to Reproduce
Steps to reproduce the behavior:

Import the plugins

Screenshots and Input Files
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.
Platform (please complete the following information):

OS: [e.g. iOS]
Version: [e.g. 22]
Dependencies Installation: [CONDA or PIP]

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[ ] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

@wangcj05 @PaulTalbot-INL @mandd

I like this idea; I have some Sampler errors and Optimizer errors that are local to their init files, and these could easily be relocated.

I tagged this also as a task...because it is not really a "defect" but more a task


copy ppkg from network share then execute during OOBE

Are we not able to copy a provisioning package from a network drive to the local PC and execute the ppkg during OOBE?

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

ID: f207b491-f262-43cf-1002-8f3fb2a052c6
Version Independent ID: 5358ed30-928c-e397-04e7-7f4c602777e2
Content: How provisioning works in Windows 10 (Windows 10)
Content Source: windows/configuration/provisioning-packages/provisioning-how-it-works.md
Product: w10
Technology: windows
GitHub Login: @jdeckerMS
Microsoft Alias: jdecker

@dbdmora No, only at runtime.


[Umbrella][Feature][Connector-V2][Docs] Refactor connector-v2 docs using unified format

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I had searched in the issues and found no similar issues.

Describe the proposal
According to #4544 , we will start refactor connector-v2 docs to improve user experience.
Task list

[x] Jdbc Connector - MySQL @zhilinli123
[ ] Kafka Connector @MonsterChenzhuo
[ ] Http Connector @MonsterChenzhuo
[ ] Mongodb Connector @MonsterChenzhuo
[ ] Clickhouse Connector @MonsterChenzhuo
[ ] My Hours @TaoZex
[ ] Klaviyo @TaoZex
[ ] lemlist @TaoZex
[ ] Onesignal @TaoZex
[ ] HDFS  @lightzhao
[ ] Hive @lightzhao
[ ] PostgreSQL @zhilinli123
[ ] SQL Server Connector @zhangchengming601
[ ] Redshift Connector @zhangchengming601

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

mysql documentation improvements please assign to me

kafka documentation、http documentation、clickhouse documentation improvements please assign to me

kafka sink端的partition参数存在误导，用户会理解成为sink端的并行度，建议修改一下，因为kafka的Connector的sink端的配置存在误导sink {   kafka {       topic = "seatunnel"       bootstrap.servers = "CentOS:9092"       partition = 1       format = json       kafka.request.timeout.ms = 60000       semantics = EXACTLY_ONCE   } }

Please assign me the connector documentation related to the SaaS API

Please assign me the connector documentation related to the SaaS API

Can you explain which specific connectors are they?

Please assign me the connector documentation related to the SaaS API

Can you explain which specific connectors are they?

Thanks. Please assign My Hours、Klaviyo、lemlist、Onesignal connector doc to me.

HDFS documentation、Hive documentation  improvements please assign to me

Hi all, there are any progress on it?

Hi all, there are any progress on it?

Please help me with the final review of Mysql. It is basically finished
@TyrantLucifer

PostgreSQL Connector Assign to me Thanks .

SQL Server documentation improvements please assign to me

Redshift documentation improvements please assign to me

https://github.com/apache/incubator-seatunnel/pull/4590

https://github.com/apache/incubator-seatunnel/pull/4721

Our template:https://github.com/apache/incubator-seatunnel/pull/4590

Code of Conduct
* [x]  I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

Search before asking
* [x]  I had searched in the [issues](https://github.com/apache/incubator-seatunnel/issues?q=is%3Aissue+label%3A%22bug%22) and found no similar issues.

Describe the proposal
According to #4544 , we will start refactor connector-v2 docs to improve user experience.
Task list
* [x]  Jdbc Connector - MySQL @zhilinli123

* [ ]  Kafka Connector @MonsterChenzhuo

* [ ]  Http Connector @MonsterChenzhuo

* [x]  Mongodb Connector @MonsterChenzhuo

* [ ]  Clickhouse Connector @MonsterChenzhuo

* [ ]  My Hours @TaoZex

* [ ]  Klaviyo @TaoZex

* [ ]  lemlist @TaoZex

* [ ]  Onesignal @TaoZex

* [ ]  HDFS  @lightzhao

* [ ]  Hive @lightzhao

* [ ]  PostgreSQL @zhilinli123

* [ ]  SQL Server Connector @zhangchengming601

* [ ]  Redshift Connector @zhangchengming601

Are you willing to submit PR?
* [ ]  Yes I am willing to submit a PR!

I want to try Doris and Hudi

Please assign  Paimon、hbase、Assert、Console to me.

Please assign db2,oceanbase to me.

Please assign db2 to me.

 I have submitted the pr.

oracle:https://github.com/apache/seatunnel/pull/5239 @ic4y

FakeSource Assign me

I want to try redis,iceberg and kudu

When can I test and use it sink to iceberg


use youtube-dl with cookies file in code not from command line

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.03.20. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2018.03.20

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[X ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.03.20
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
from __future__ import unicode_literals
import youtube_dl

ydl_opts = {}
with youtube_dl.YoutubeDL(ydl_opts) as ydl:
    ydl.download(['https://www.youtube.com/watch?v=BaW_jenozKc'])

this for downoalding simple youtube video i need how add the cookies file untill i can downoald from my account on linda im trying to create small downaolder untill help fast the process any idea how add cookies file

https://github.com/rg3/youtube-dl/blob/master/youtube_dl/YoutubeDL.py#L208


Parse times as strings

Ability to parse times as strings
Using default parse time method https://docs.microsoft.com/en-us/dotnet/api/system.timespan.tryparse?view=netcore-3.1

Altizar proposed better time format.. closing so far..


Gradle file changes

Please review the changes and merge ASAP as they are creating problems in Android Studio while compiling.
Making the changes manually in './node_modules' is okay, but everytime you try to delete and again do 'npm install' the problem occurs again.

 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.Danish Ansari seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@mddanishansari what is your build output in Studio? And what is your compile SDK?

My compile SDK version is 27 but soon I'll update it to 28
Following is my build output
WARNING: Configuration 'compile' is obsolete and has been replaced with 'implementation' and 'api'.
It will be removed at the end of 2018. For more information see: http://d.android.com/r/tools/update-dependency-configurations.html
Affected Modules: react-native-android-fragment

WARNING: Configuration 'testCompile' is obsolete and has been replaced with 'testImplementation'.
It will be removed at the end of 2018. For more information see: http://d.android.com/r/tools/update-dependency-configurations.html
Affected Modules: react-native-android-fragment

WARNING: Configuration 'androidTestCompile' is obsolete and has been replaced with 'androidTestImplementation'.
It will be removed at the end of 2018. For more information see: http://d.android.com/r/tools/update-dependency-configurations.html
Affected Modules: react-native-android-fragment

WARNING: The specified Android SDK Build Tools version (26.0.3) is ignored, as it is below the minimum supported version (28.0.3) for Android Gradle Plugin 3.3.1.
Android SDK Build Tools 28.0.3 will be used.
To suppress this warning, remove "buildToolsVersion '26.0.3'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.
Remove Build Tools version and sync project
Affected Modules: react-native-android-fragment

@mddanishansari We'll be updating to sdk 28 soon at Hudl as well. When that happens we'll push to update this repo as well. As of right now those are only Warnings though and the build should still compile.

@jpshelley thanks a lot

@mddanishansari Thanks for suggesting this request. We'll be officially deprecating this repo now that Hudl has upgraded and can take advantage of react-native's official ReactFragment
If you believe the official ReactFragment also needs this support I would suggest opening a PR there! https://github.com/facebook/react-native/pulls


Fix gradle build issues in H2 module

Description:
$Subject

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/14619


[BUG] pdftocairo not executed

Failed to execute pdftocairo. Report bug with postscript file to dev.

I believe you will need to install "poppler" version 0.89.0 to be able to execute pdftocairo correctly. I will update the README to address that. Please let me know if that does not solve your issue.
choco install poppler --version 0.89.0 -y --force

it worked, but nothing is displayed, just an empty page

Hi @MaikBrant, could you share the postscript file or an example one that has the same issue? Thanks!

This is my first job in this programming language, maybe I need to start the file in a special way?

I believe there are multiple ways to define headers for postscript file. However, it will be difficult to debug this issue without a reproducible example file.
Are you able to view the file using GhostScript?

% Фон
newpath
0.0 0.2 0.5 setrgbcolor
0 0 moveto
612 0 lineto
612 792 lineto
0 792 lineto
closepath
fill
% Луна
1.0 1.0 0.6 setrgbcolor
300 700 70 0 360 arc
fill
% Земля
newpath
0.0 0.0 0.0 setrgbcolor
0 0 moveto
612 0 lineto
612 100 lineto
0 100 lineto
closepath
fill
%холм1
newpath
0.1 0.3 0.1 setrgbcolor
0 100 moveto
0 250 rlineto
170 200 rlineto
400 -450 rlineto
closepath
gsave
fill
%холм2
newpath
0.2 0.3 0.1 setrgbcolor
0 100 moveto
400 500 rlineto
400 -500 rlineto
closepath
gsave
fill
%дом1
newpath
10 100 moveto
0 300 rlineto
170 0 rlineto
0 -300 rlineto
closepath
gsave
0.5 0.5 0.5 setrgbcolor
fill
%кусочек дерева нижний
newpath
0.1 0.4 0.0 setrgbcolor
400 210 moveto
100 100 rlineto
100 -100 rlineto
closepath
gsave
fill
%кусочек дерева средний
newpath
0.1 0.5 0.1 setrgbcolor
400 265 moveto
100 100 rlineto
100 -100 rlineto
closepath
gsave
fill
%кусочек дерева верхний

newpath
0.1 0.6 0.0 setrgbcolor
400 320 moveto
100 100 rlineto
100 -100 rlineto
closepath
gsave
fill
%ствол
newpath
490 100 moveto
0 110 rlineto
20 0 rlineto
0 -110 rlineto
closepath
gsave
0.3 0.2 0.1 setrgbcolor
fill
%окно1

newpath
25 200 moveto
0 60 rlineto
60 0 rlineto
0 -60 rlineto
closepath
gsave
1.0 0.9 0.2 setrgbcolor
fill
%окно2

newpath
105 200 moveto
0 60 rlineto
60 0 rlineto
0 -60 rlineto
closepath
gsave
1.0 0.9 0.2 setrgbcolor
fill
%окно3

newpath
105 280 moveto
0 60 rlineto
60 0 rlineto
0 -60 rlineto
closepath
gsave
1.0 0.9 0.2 setrgbcolor
fill
%окно4
newpath
25 280 moveto
0 60 rlineto
60 0 rlineto
0 -60 rlineto
closepath
gsave
1.0 0.9 0.2 setrgbcolor
fill
% дом2
newpath
180 100 moveto
0 250 rlineto
160 0 rlineto
0 -250 rlineto
closepath
gsave
0.3 0.3 0.3 setrgbcolor
fill
%окно1

newpath
200 200 moveto
0 50 rlineto
50 0 rlineto
0 -50 rlineto
closepath
gsave
1.0 0.9 0.2 setrgbcolor
fill
%окно3

newpath
265 260 moveto
0 50 rlineto
50 0 rlineto
0 -50 rlineto
closepath
gsave
1.0 0.9 0.2 setrgbcolor
Fill
Here is example, when I use GhostScript image displayed, but in Visual Studio nothing is displayed

@ahnafnafee, I'm sorry to bother you, but is there any way to solve my problem, or do I need to look for alternative ways to solve the problem?

The problem was in the name of the folder where the file was stored


Flask app installed as egg raises builtins.NotADirectoryError for templates

I've created a MANIFEST.in file and added the include_package_data flag to setup as recommended in #84 and have confirmed that the template files do appear in the egg file installed into my virtualenv:
(test)david@snape:~$ unzip -l .virtualenvs/test/lib/python3.4/site-packages/hxl_proxy-0.3-py3.4.egg | grep error
      154  2015-08-26 16:48   hxl_proxy/templates/error.html

When I run from the source directory, all is fine.  When I try to run from the egg installed in the virtualenv, I get the following error:
Traceback (most recent call last):
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/_compat.py", line 33, in reraise
    raise value
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/app.py", line 1379, in handle_user_exception
    return handler(e)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/hxl_proxy-0.3-py3.4.egg/hxl_proxy/controllers.py", line 44, in error
    return render_template('error.html', message=str(e)), status
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/templating.py", line 127, in render_template
    return _render(ctx.app.jinja_env.get_or_select_template(template_name_or_list),
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Jinja2-2.8-py3.4.egg/jinja2/environment.py", line 851, in get_or_select_template
    return self.get_template(template_name_or_list, parent, globals)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Jinja2-2.8-py3.4.egg/jinja2/environment.py", line 812, in get_template
    return self._load_template(name, self.make_globals(globals))
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Jinja2-2.8-py3.4.egg/jinja2/environment.py", line 774, in _load_template
    cache_key = self.loader.get_source(self, name)[1]
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Flask-0.10.1-py3.4.egg/flask/templating.py", line 60, in get_source
    return loader.get_source(environment, local_name)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Jinja2-2.8-py3.4.egg/jinja2/loaders.py", line 171, in get_source
    f = open_if_exists(filename)
  File "/home/david/.virtualenvs/test/lib/python3.4/site-packages/Jinja2-2.8-py3.4.egg/jinja2/utils.py", line 151, in open_if_exists
    return open(filename, mode)
NotADirectoryError: [Errno 20] Not a directory: '/home/david/.virtualenvs/test/lib/python3.4/site-packages/hxl_proxy-0.3-py3.4.egg/hxl_proxy/templates/error.html'

@davidmegginson, just stumbled across this myself. You can work around it by giving zip_safe=False as a parameter to setup in your setup.py.

Thanks, Denis! That should make life much easier. I'll test when I'm back at my computer, then close.

Confirmed.

isnt the actual problem that a filesystemloader is used instead of a packageloader?

@RonnyPfannschmidt - would that be a configuration error in my app, or a core problem with flask? I'm still relatively new to the world of Python and WSGI.

problem with flask

@RonnyPfannschmidt - would you want to create a new issue, referencing this one? This one has gone a bit off track, and I don't have the background to describe the problem properly.


Edge implementation duplication

Describe the bug
As part of the performance improvements done to handle many edges, we had to introduce a split implementation. There is one for a small number of edges which can have animations, and one for a large number of edges which cannot have animations.
The current high perf implementation can either have animations or allow adding and removing nodes. Right now, the whole mesh is recreated when the list of nodes changes, so animating between the two sets of vertex positions is all over the place. Having both would require changing the implementation so that the mesh vertex indices remain stable as nodes are added/removed.
Steps to Reproduce the Bug or Issue
Look at the code
Expected behavior
One implementation for Egde
Screenshots or Videos
No response
Platform

Reagraph Version: [e.g. 1.0.0]
OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
NodeJS version: ...

Your Example Website or App
No response
Additional context
No response

This could potentially be useful: https://www.npmjs.com/package/@react-three/drei#performance


Check the possibility of creation and population of databases at the beginning of test suite

Description:
Currently a database is created per test class. $subject to reduce the time taken for table/SQL tests.

We have 5 table type test classes and 5 SQL connector test classes ATM. I checked the time taken to create one database which turned out to be 700-900 milliseconds and deletion of the db directory (which already is done after suite) takes ~300 milliseconds. Therefore the total time would be like 8-9 seconds on average. Also if we do this, we will need to write clean-up scripts for each test case to avoid conflicts between the tables/table data/procedures belonging to other test classes, which would be error-prone and also add a bit more time to build time.
Therefore I think the 8-9 seconds gain is not worth effort and complexities it introduces.


Fix medicalwindow

This PR fixes the bug in the medical window if the medicalHistory is too long.
The window now looks like this, with a scroll bar.

Closes #272

Codecov Report

Merging #284 (151552b) into master (91e1b63) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #284   +/-   ##
=========================================
  Coverage     76.94%   76.94%
  Complexity      998      998
=========================================
  Files           140      140
  Lines          2646     2646
  Branches        356      356
=========================================
  Hits           2036     2036
  Misses          530      530
  Partials         80       80

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 91e1b63...151552b. Read the comment docs.


Set up test and CI

Maybe writing tests with Rust would fit the toolchain better.
See https://doc.rust-lang.org/std/process/struct.Command.html

Done in #23


Cannot generate mnist digits with the InfoGAN.py script

I tried to run the script for InfoGAN applied to mnist. However, the generated images at the end of the training phase are not even vaguely similar to digits' shapes.
Is there someone else that tried the script? Do you have my same problem? Is there anything I should try to make it work?
I have already tried to use more training epochs and reducing the learning rate but nothing worked so far....

I checked 01c19e4 and confirmed that the changes have been merged, but the problem still exists.
infogan/infogan.py
@@ -1,4 +1,4 @@
from future import print_function
from future import print_function, division
from keras.datasets import mnist
from keras.layers import Input, Dense, Reshape, Flatten, Dropout, multiply, concatenate


Add X-Plane SDK support

Hi,
This commit is for adding the pre-built X-Plane SDK library into VCPKG. I don't have a Mac to test whether I copy the library into the right place or not, so if I'm making any mistake, please let me know. I will try and correct it as soon as possible.
Also, this is my very first pull request for something like this. Please let me know if I'm doing anything wrong.
Thank you so much for your time.

 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.:x: xdarkradx sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi, @xdarkradx! Thanks for this PR. Things look pretty good, but there are a few issues we should fix.
Firstly, I found this to fail when building on x86 with errors like so:
The following files were built for an incorrect architecture:
    E:/vcpkg-dev2/packages/x-plane_x86-windows/debug/lib/XPLM_64.lib
Expected x86, but was: x64

The x-plane SDK website says that the SDK is only usable on 64-bit architectures, so I recommend that we just block the install on x86 with an error message.
I also noticed that the x-plane SDK sample code on the website expects the headers to be installed at the root of the include prefix. This PR places the headers into x-plane/, which will cause the example projects to not work without amending the path. I suggest we install all the headers to the root in order to make the samples "just work" as much as possible.
I've made the above changes and pushed them to a branch: dev/codym/5408. Feel free to take a look. If you like them, you can reply back here and I'll merge the changes into the PR. 😄
I'm also still investigating what to do about the macOS frameworks. I'm not sure if they're in the correct place or not, so I'm working to find out!

Hi @Codiferous, thank you so much for the fixes. I changed my default triplet so I totally forgot about the x86. The changes look good to me, please feel free to merge it.
Thank you so much for your time and help.

@xdarkradx I haven't found the answer for the framework question, but I don't think it's worth blocking this PR. I tested it out on Windows and was successfully able to build a plugin DLL (though I didn't try using it, as I don't have x-plane and I also don't want to make my coworkers jealous by hanging out with a flight simulator at work). I think even if the frameworks are placed in the wrong place on macOS, that can be addressed in another PR.
Nice work on this PR! This looks like a super cool library to have. 😄 Also, congrats on your first PR. We'd love to have you submit more!


Feature to generate slugs automatically when a pattern is changed

When changing the pattern of the AutoroutePart we could allow the user to generate the slugs for all the existing content items. This would be a checkbox in the part settings.
If the AutoroutePart is newly attached it also counts as a pattern change, and this would automatically generate the urls for the existing items.

When working on the Autoroute module I've been thinking about a way to be able to have multiple Alias for the same content item since we can create multiple patterns on a content type. Should it remove the last Alias created ? Maybe not since it's not recommended SEO wise. So, if we generate url's we should maybe use multiple patterns to keep old Alias created.
So first I would try to implement the multiple patterns generation on a content type.

checkbox : a checkbox on each patterns except the default one named "Generate"
checkbox : a checkbox on the Autoroute settings to allow to generate checked patterns on save/edit of it's content items.

For regenerating all Alias :

checkbox : a checkbox on each patterns except the default one named "Generate" (the same)
button : could be done on saving / creating the content type (save button).

If we don't type any pattern it would generate the default one wich is always "/content-slug"

If the AutoroutePart is newly attached it also counts as a pattern change, and this would automatically generate the urls for the existing items.

I've been thinking about this while implementing it.
Scenario :

I remove the AutoroutePart from a content type that has content items with already defined Alias
I add back the AutoroutePart to the content type "I made an error".

In this scenario maybe the Alias I created are good. If I add the AutoroutePart back right now it will recreate the default Patterns for each culture ; so it will create the Alias from the {Content.Slug} pattern that will maybe generate URL's like : item, item-1, item-2 "because they need to be different".
What I think right now is that we should provide a button for generating Alias once you have set different patterns than the default one else this will add processing for nothing. This is a really rare scenario to want to update all Alias when adding an AutoroutePart. If we want this we should provide a global configuration for enabling/disabling this at least.


Autocomplete: Customize display when no result is returned by the search method

We would like to customize what the autocomplete component displays when the search method returns no result.
At the moment, it just shows nothing (the popover is closed).
The enhancement could be a simple text property (NoResultMessage parameter) or even better, a render fragment (NoResultTemplate parameter). Thus, it could allow us to display a text with an icon or create button.

We usually do a string NoResultText and a RenderFragment NoResultContent where the RF overrides the string if provided.
In Select we have added support for RenderFragments instead of text in the text field but in Autocomplete we have not. So it would have to be added for this, or you just implement NoResultText.

We usually do a string NoResultText and a RenderFragment NoResultContent where the RF overrides the string if provided.
In Select we have added support for RenderFragments instead of text in the text field but in Autocomplete we have not. So it would have to be added for this, or you just implement NoResultText.

I think this can be closed. There is a NoItemsTemplate now.
https://mudblazor.com/components/autocomplete#presentation-extras


⚠ Turnos 24x7 has degraded performance

In 1d3600f, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1303 ms

Resolved: Turnos 24x7 performance has improved in dee95e1.


KAYAK-987 Revert to scalatest for ConsumerAndProducerApiSpec

KAYAK-987 Revert to scalatest for ConsumerAndProducerApiSpec #413

I'll probably be trying a few different things on this branch, along with retrying failed and successful builds, so feel free to ignore for now.

I'm going to merge to get early feedback on whether this does just as well on the master workflow as it does on this branch. Shouldn't make a difference, but who knows. If it does well, we can clean up names and imports to our liking. (Everything's based on whatever was in https://github.com/Banno/kafka4s/blob/4f0122cac334da0de945e2a308f2a8788e19340a/core/src/test/scala/com/banno/kafka/ConsumerAndProducerApiSpec.scala)

This is looking ok on master so far, so I'll clean up some of these odds and ends.


Add CONTRIBUTING.md to make the rules, procedures clearer to new contributors

This is generally regarded as conventional now, so it'd be a good idea to do it. I can do it myself :)

Good idea, and I wasn't aware (I've created a discussion instead). If you still want to do then yes, greatly appreciated.


Create Merge K Sorted Lists.java

Merge K Sorted Lists in java
Constraints:
Space complextiy = O(k)
Time complexity = nlogk
where k is the number of lists and n is number of elements across all lists.
Sample input:
4
5
10 20 30 40 50
7
5 7 9 11 19 55 57
3
1 2 3
2
32 39
Sample output:
1 2 3 5 7 9 10 11 19 20 30 32 39 40 50 55 57

#2

hey what changes do you need because this code works fine on my system.

#2

hey what changes do you need because this code works fine on my system.

Hello sir, you have to change directory, as written here https://github.com/abhpd/hacktoberfest2020/issues/2

Closing PR for #245.


heroku

can you give deploy to heroku bro?

Sorry, there won't be any benefit of me deploying it, as subscribers will need to be manually added to the bot.


Action inside the type attribute Why is called type rather than called name?

Action inside the type attribute Why is called type rather than called name?

@gaearon

See https://github.com/reactjs/redux/issues/541 .


Batch of docs reorganization and renaming

Following up on https://github.com/submariner-io/submariner-website/pull/361, suggest a few additional layout adjustments.
This is going to need a lot of link fixing (on top of the lots of broken links we already have). Marking as a draft for now to get feedback, will fix links if there is agreement on the structure.

I'm not sure about the /development/contributor-roles/ section. Maybe it should be /community/contributor-roles/?

What do folks think about rename "online resources" to "other resources"?

@dfarrell07 LGTM

Cool, thanks @nyechiel. I'm cleaning up the broken links now.

According to find . -name \*.md -not -path "./src/themes/*" -exec markdown-link-check -c ./.markdownlinkcheck.json {} \;, I've fixed all the broken links now.

I did the last bit of link fixing in https://github.com/submariner-io/submariner-website/pull/389 to get the linting passing here.

I'm not sure about the /development/contributor-roles/ section. Maybe it should be /community/contributor-roles/?

After a few days to think about it, I'm leaning towards moving it to the community section. If I was showing up new to the project, I think that's what I'd expect to find.

@dfarrell07 this now conflicts

Updated screenshots:


CDR-1580 Add check for updated EHR_(STATUS/FOLDER) ids and versions

Changes
Updating an EHR_STATUS or FOLDER does not check the If-Match header against the DB. Also it is not checked if the uid contained in the payload matches.
This fix adds additional check as well as 2 sql scripts to validate and fix the incorrect versioned object ids.
Pre-Merge checklist

[x] New code is tested
[x] Present and new tests pass
[x] Documentation is updated
[x] The build is working without errors
[x] No new Sonar issues introduced
[x] Changelog is updated
[ ] Code has been reviewed

@vmueller-vg The checks for EHR_STATUS and FOLDER are placed on the repository level - I'm not entirely sure if this makes sense. I'm also not sure if current script dir is the best location.

@vmueller-vg I don't know why the update of EHR_STATUS and FOLDER was different in the past, but now they are aligned -> https://github.com/ehrbase/ehrbase/pull/1398/commits/3e59f1e8405060ce88479498371cf105fe70c5a3. I've also added unit tests to ensure the VersionedObjectUpdate is the same for COMPOSITION, EHR_STATUS and FOLDER.

Replaced by https://github.com/ehrbase/ehrbase/pull/1398


MockWebServer: have a way to emit chunks of different sizes and with delays

Currently, MockWebServer has limited support of chunked transfer encoding, via the MockResponse.setChunkedBody API, which returns the body in chunked of a fixed size.
It would be useful to extend this support to allow emitting arbitrary sized chunks, and with delays, in order to test streaming cases.
I could find this previously mentioned in this "meta" issue.
As said in this comment, I believe one way to do this could be to change the body of MockResponse to be a Source instead of a Buffer - of course this would be a breaking change on the API.

Our recent answer for MockWebServer feature requests is

To be clear on the status of this, enhancements for MockWebServer isn't a big focus for the OkHttp maintainers, but as this seems like a reasonable request, it's probably something that would be accepted given a nice simple clean PR.

But please consider how to make this change without breaking the API.  Breaking the existing API would probably mean we wouldn't land the PR.

Oooh interesting. I think this is a bad fit for MockWebServer now in terms of general utility vs. complexity. If you’d like it, please consider making your own fork. It’s not a particularly complicated library.


Fixes #2342 : Add NoMethodTagInClassDocblockRule

All checks have passed 🎉 @TomasVotruba it is ready for review.

Thank you


Ignore olm.csv.metadata objects besides olm.bundle.objects

https://github.com/operator-framework/operator-registry/pull/1094 rewires opm render to produce olm.csv.metadata objects. catalogD controller should ignore olm.csv.metadata objects also here

I actually think we should not ignore these. This metadata is much smaller, and is helpful for tools like kubectl operator

Closing this out. We can revisit of olm.csv.metadata becomes a problem.


feat adjust stats derive by delta row

After analyzing, user may insert new rows.
analyzed rows: the rows have been analyzed
delta row: rows inserted after analyze job
if analyzed rows are filtered out, then we try to estimate filter result by delta row with unknown column stats.
Proposed changes
Issue Number: close #xxx

run buildall

run buildall

run buildall

run buildall


Train simple audio recognition model

@petewarden
I trained this model in colab and it is working in arduino for "yes" and "no" command but when I tried for "on" and "off" I am not getting output in my arduino board( I have done the necessary changes in tensorflow example code "micro_speech" ).
Can you please suggest how to get model working for "on" and "off".

@rajaniyadav,
In order to expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here and also the TensorFlow version you are using. Thanks!

You can also share the your Colab gist with us. Select 'File' -> 'Save a copy as Github Gist', and share the link of the new window. Thanks!

I have train the model in colab link https://colab.research.google.com/github/tensorflow/tensorflow/blob/master/tensorflow/lite/micro/examples/micro_speech/train_speech_model.ipynb
with modification -  os.environ["WANTED_WORDS"] = "on,off"
tensorflow version -1.15
github gist https://gist.github.com/rajaniyadav/d031321012472f41d993aa59822dfa47

Why the below link is no longer accessible...
https://colab.research.google.com/github/tensorflow/tensorflow/blob/master/tensorflow/lite/micro/examples/micro_speech/train_speech_model.ipynb

Here is the updated link! https://colab.sandbox.google.com/github/tensorflow/tensorflow/blob/master/tensorflow/lite/micro/examples/micro_speech/train/train_micro_speech_model.ipynb
Refer to the new training document here: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/micro/examples/micro_speech/train

Closing the issue due to inactivity. @rajaniyadav, you can re-open this issue if you're still blocked on this issue.


Text overlap in “Spread Trends” graph [Screenshot added]

Describe the bug.
Name of the Month and date overlaps in the graph x-axis if we toggle 14D option.
To Reproduce
Steps to reproduce the behavior:

Go to home page
Scroll to the graphs and see the option of choosing time frame. [All, 1M, 14D]
Select 14D option
Check the x-axis labels of graphs

Expected behavior
Text should not overlap and name of the month should be slightly below the dates.
Screenshots
Refer the highlight part:

Desktop (please complete the following information):

Not tested on Desktop

Smartphone (please complete the following information):

Device: iPhone 7+
OS: 13.3
Browser: Safari

Should be fixed in #776

Great!


batch tag upserts, remove randint in tag sampling

This PR complements graphite-project/graphite-web#2125 and switches writer to batch tag updates.
The implementation is pretty basic, series created and updated are added to 2 queues which are then processed in batches in a separate thread.  2 queues are used so that new series are prioritized when writing to the TagDB, and so that they don't compete with updates for space in the queue.  Each queue has a maximum length of TAG_QUEUE_SIZE.
The writer will write at most TAG_BATCH_SIZE paths in each request, and will process the queue as quickly as possible until the queue is empty.  I'm not sure if we want to add a delay in between batches to avoid overloading graphite-web.
The other change here is to remove the use of random.randint in selecting which updates to send to the tagdb, instead just using a global counter and sending every TAG_UPDATE_INTERVALth path.  As long as the total number of series being processed isn't an exact multiple or factor of TAG_UPDATE_INTERVAL this will still result in every series getting upserted periodically.
Finally I updated run_twistd_plugin to enable debug logging when passing --debug on the command line.
As always, any and all suggestions for improvement are appreciated.
H/T to @Civil for the suggestion of batching the tagdb updates.

I realized that state.database.tag() was previously set up not to wait for the http call to complete, the latest commit changes that so that state.database.tag() returns a Deferred and writeTags() waits for it to complete before firing off the next request.


add an UNSAFE keyword for CREATE PROPERTY.

add to the CREATE PROPERTY an UNSAFE keyword to skip the persistent data check, this will speedup a lot the property creation, but should be documented properly because can lead to runtime errors.

implemented and linkded in this issue https://github.com/orientechnologies/orientdb/issues/3399 closing

@tglman you forgot the release....changing it


SVG thumbnails feature not working

SVG thumbnails feature not working. in windows 10 20H2 latest

@halcyon101
what version of PowerToys are you running?

Versions 0.31.1 and 0.31.2 the thumbnails as file icons started working for me again, but the preview pane says "This file can't be previewed" for SVGs, and for markdown "This file can't be previewed because of an error in the Markdown Preview Handler."  I even manually installed the latest .net core 3.1 today before updating powertoys to 0.31.2 today.

I've identified a related issue.
The preview relies on the viewBox attribute, but an increasing number of SVGs are optimized so they only have width & height attributes.
Since viewBox="0 0 1024 1024" is the same as width="1024" height="1024", this can be easily fixed.

@nickcombs32
thank you very much for debugging the issue. If you want to open a PR that would be great. Otherwise we will do it. Thanks.

@halcyon101
what version of PowerToys are you running?

Latest version and it's now back to working apparently, so I'm happy ! Thank you all for make it happen, It would have been great if you could add more thumbnails to this app as well so we didn't need to install so many apps, for example for *.ai and *.psd *.pdf and so on .

@halcyon101
thanks for the follow up.
Mode file types will be added in the future.

@halcyon101
thanks for the follow up.
Mode file types will be added in the future.

Much Appreciated, very useful addition app to windows.


Is this project dead?

There have not been any meaningful commits for six years. Is this project still being maintained?
Are there any newer alternatives?

I've been looking around for alternatives, haven't found any yet. I'm currently experimenting with training a custom spaCy NER model using the training data from this repo. You can see my progress in this repo here, if you want to help, please do.

There is python blinding for libpostal library, here it is https://github.com/openvenues/pypostal
Both of them have parsing issues, unfortunately, there is no silver bullet.


Scroll binding option

There should be an option to have the overview scrolling bound to the main editor scrolling, similar to the overview functionality in Sublime Text

i thought this extension was that, but no it's like extra mini editor. I was thinking of it as navigator in photoshop so you have bird eye view of whole file.

@techsin This extension is what you're describing. It was recently broken from a Brackets update. Check out the other issues for the fix.


Add AnimatableTabBarController to support custom transition animation

Same as AnimatableNavigationController with reverse animation when we can using child controller index
Added to the demo too, near the other tab bar controller where the animation is coded, not configured using storyboard

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

  Generated by :no_entry_sign: Danger

The following comments is not directly related to this PR
I'm wondering, should we move the both tabBar examples in the Transition part (by diving in 3 subnavigations) directly?

@phimage Great PR 👍👍👍, can you merge it when you feel it is ready?


[Bug / FR] Support brew install

I installed gtm-enhanced for IntelliJ from the JetBrains store and got gtm from brew brew tap git-time-metric/gtm; brew install gtm.
When starting IntelliJ with the gtm-enhanced plugin I got a window like 'gtm isnt the latest'.
The version-tag for gtm is 1.3.5 so it seems to be latest.
Describe the solution you'd like
I would like to support home-brew for Mac.

brew would be MacOS and I can take a look at it. Just wanted to know if there would be interest.
@kilpkonn

Yeah definitely, I'd love it!
Hit me up if there's any questions etc. and I'll see if I can help as well if needed

What a funny coincidence.
The last days I have thought about gtm and that I'd like it to be added to Homebrew (because I noticed I still had 1.0.2 installed and needed to manually update it), then I wanted to create in Issue here and the first thing I see is exactly that issue.
Since this issue is already almost 3 months out and so far nothing has happened, I researched a bit on how to add a project to Homebrew (/Linuxbrew) and it actually is not very difficult.
I basically did everything, you just need to do some last steps (which should take less than 5 minutes) as the project owner so it can be used by Homebrew.
I'll still explain what I did and how Homebrew works:
Homebrew has different taps, which are basically libraries with projects you can install.
If you use "brew install xxx" it searches the default / main homebrew tap if xxx exists there to install it first. However, you can add other tabs / sources from where you can install projects with "brew tap ". If you tapped them, "brew install xxx" will also search through them.
So to make this project available for Homebrew / Linuxbrew, there are two possibilities:

You create an own tap. Then people will use "brew tap developest/xxx" to add your tap and then they can install gtm via "brew install gtm"
You get gtm into the official default / main tap. Then they just need to use "brew install gtm", however, the requirements for that are higher.

For both of these, you need a "formula", basically a script telling the commando line on how to download.
If you wanna know about how to create a formula for this project, there are these helpful tutorials:
https://docs.brew.sh/Formula-Cookbook
https://medium.com/ballerina-techblog/how-to-create-your-own-homebrew-package-or-formula-8dfbf8e001d3
The first one (official documentation) is especially important if you want to get gtm into the main tap. For that, there are additional requirements (e.g. a test).
However, the original gtm already has a brew formula, which you can use with few adjustments: https://github.com/git-time-metric/homebrew-gtm/blob/master/gtm.rb
I used that formula and adjusted it to this project (homepage, version, sha hashes and url) plus I added the licence information.
Notice that this formula still lacks some requirements to add it to the official tap. You can use this one for an own tap now and add it into the official tap sometimes later though.
class Gtm < Formula
  desc 'Simple, seamless, lightweight time tracking for all your git projects'
  homepage 'https://github.com/DEVELOPEST/gtm-core'
  license "MIT"

  version 'v1.0.3'
  sha256Mac = '3153d874d64e735ffb96538eef16696b5354b9d46192f271325dce1507a683e7'
  sha256Linux = '65a2ac04a5c14b155924df17ce3a910007c18f36c1209e24e506443732cdbf4e'

  os = ''
  if OS.mac? then
      os = 'macOS'
      sha256 "#{sha256Mac}"
  elsif OS.linux? then
      os = 'linux'
      sha256 "#{sha256Linux}"
  end
  url "https://github.com/DEVELOPEST/gtm-core/releases/download/#{Gtm.version}/gtm-#{os}.zip"

  def install
    bin.install 'gtm'
  end

end

There is just one thing you need to adjust:
This formula searches for "gtm-macOS.zip" and "gtm-linux.zip" in the GitHub release of v1.0.3
However, you named them inconsistently:

For this formula to work, you need to rename "gtm-core-macOS.zip" to "gtm-macOS.zip".
I would also strongly recommend to rename "gtm-core_1.0.3_amd64.deb" to "gtm-amd64.deb" and "gtm-1.0.3-64.msi" to "gtm-64.msi" so the naming scheme is more consistent.
If you did that, the formula is correct and should download gtm correctly via brew. The last thing you need to do is upload the formula.
I'll tell you how to do your own tap. If you want to add gtm to the official tap, you need to add more information to it and request it to be merged then. I think that is a good idea and if you want that, I can try to edit the formula and make such an appeal in a few weeks, I just don't have the time right now.
So your own tap is basically just a GitHub repository. If you create the GitHub repository "homebrew-xxx" and just put the file as gtm.rb in it, it is a valid tap you can tap with "brew tap developest/xxx"
So the only thing you need to do is creating the repository and putting the file in it.
If you want to program other things with DEVELOPEST as well and get them into the same tap, you might chose a more general name such as "homebrew-utils" (then the tap command is "brew tap developest/utils").
But if you do not currently have plans for that, I would rather recommend to name the repository "homebrew-gtm" (then the tap command is "brew tap developest/gtm").

As a TL;DR, this is basically a clear to do list on what needs to be done so you can download gtm via Homebrew and Linuxbrew:

Create a repository named "homebrew-gtm".
Put the file gtm.rb into the repository:
(Since GitHub doesn't allow .rb files to be uploaded, it is zipped)
gtm.rb.zip
Rename the macOS releases of v1.0.3 into "gtm-macOS.zip" (preferably also the other releases for a more consistent naming scheme as I wrote in the upper post)

(4. Edit the README.md file of gtm to show how you can download it via Homebrew and Linuxbrew (I can also just do this via pull request if you do the other steps))

Big thanks for detailed guide. I'll do the rename and create homebrew-gtm repo, I can also give you permissions to it as I personally don't use brew so I'm gonna be always out of loop with that...

I've renamed assets in release and you should have invite with Maintainer access to https://github.com/DEVELOPEST/homebrew-gtm so feel free to set it up. I decided I'm not gonna put the script as I wont test it so it would altogether make more sense if someone more familiar with the subject could do it.
Once you have it working there submit a simple PR against this repo with updated instruction to install and I'll merge
Also if there is anything more I should do comment below! :point_right:

Thanks!
I uploaded the script with a readme in homebrew-gtm and tested it early; it worked without any problems :D
I also updated the GTM readme and I'll do a pull request for that now.


Fully Functional Stations & Inventory Tabs in Manager

Assuming you're happy with the code, the only thing that still needs to be done before a potential 1.2.14 release is converting the tick rate for the manager to a configurable mod setting (it's currently hardcoded to tick at every 60 ticks) and doing whatever else you need to clean up from getting the GUI initially functioning.
Still need to bind the handlers for the on-click for the station names as well, but that can be done at a later point.

Only thing missing at this point for a potential 1.2.14 release is an update to changelog.txt, other than that I think everything is ready for release candidate testing to rule out any random crashes before a release.

I'm happy with this at this point for being released for testing, and assuming no other changes need to be made after further testing feedback then a changelog entry can be made it and it can go live.


No file upload with v1.0.4

After upgrading from 1.0.1 to 1.0.4, file uploads are no longer possible. It seems the JS scrambles the form data somehow.
After replacing the JS with the old version, uploads are working again.

Was this a problem after the json object fixes?  I suspect not.


cmd/compile: open code unsafe.Slice

golang.org/cl/352953 used unsafe.Slice in the runtime. I am about to revert that change b/c it causes a 4% performance regression in a stack copying benchmark. At least half of that is due to function call overhead. We might want to open code unsafe.Slice instead of making it a runtime call.
cc @mdempsky

Looking at the current implementation of unsafe.Slice, I see that it gives an error if the overall size of the slice is > maxAlloc, but I don't know why.  One can imagine using unsafe.Slice on a huge chunk of memory that is larger than maxAlloc, and doesn't seem any less safe than unsafe.Slice itself.

I think my rationale was to ensure the slice didn't wrap around the address space, and I just copied that from growslice. But reflecting on it now, it doesn't actually achieve that. Perhaps there's no reason to cap the addressed memory range.
Probably a better check would be mem > -uintptr(ptr) to prevent wrap around.

I think my rationale was to ensure the slice didn't wrap around the address space, and I just copied that from growslice. But reflecting on it now, it doesn't actually achieve that. Perhaps there's no reason to cap the addressed memory range at maxAlloc.
Probably a better check would be mem > -uintptr(ptr) to prevent wrap around.

It's already "unsafe". Can we just omit all checks and have the compiler emit ideal code that assumes its usage is correct, just like unsafe.Pointer casts? Then we can put all sorts of safety checks behind checkptr, again like unsafe.Pointer casts.

Great. I'm thousands of miles from my computer, so I'm afraid I can't benchmark for you now. But the benchmark is checked in—StackCopyWithStkobj.

It seems strange, but I even now can't reproduce on my M1. With and without unsafe.Slice give me the same benchmark result.

Never mind, I can reproduce on linux/amd64.
name                   old time/op    new time/op    delta
StackCopyWithStkobj-8    13.8ms ± 1%    14.5ms ± 3%  +5.09%  (p=0.000 n=10+9)


Individual zone switches are not available for irrigator (ggq)

Describe the bug
Individual zone switches (Eg: zone 1, zone 2, zone 3, zone 4, etc) are not sent by the Tuya API backend for an irrigator.  Only a single switch is given for the Irrigator which reflects the state of all the zones (IE: If any zone is running, then the switch state is true, if no zone is running then the switch state is false).  Setting the switch to true just starts zone 1 of the irrigator.
Expected behavior
Individual zone switch states in addition to the existing "overall" switch state should be provided and also able to be set.
Screenshots
N/A
Home Assistant Version

e.g. 2021.8.8

Device info (please complete the following information, which can be found in log):
{
"active_time":<PHONE_NUMBER>,
"biz_type": 0,
"category": "ggq",
"create_time":<PHONE_NUMBER>,
"icon": "smart/icon/ay1556838860681oKmdO/487fc118acc3a5fc21ace268d4406ecd.png",
"id": "xxx",
"ip": "",
"lat": "",
"local_key": "",
"lon": "",
"model": "WX8",
"name": "WX8",
"online": true,
"owner_id": "xxx",
"product_id": "3c8bxnseywh7radc",
"product_name": "WX8",
"status": [
{
"code": "switch",
"value": false
}
],
"sub": false,
"time_zone": "+10:00",
"uid": "***",
"update_time":<PHONE_NUMBER>,
"uuid": "xxx"
},
Device specifications (please complete the following information, which can be found in log):
2021-09-15 19:28:21 DEBUG (SyncWorker_5) [tuya iot] Request: method = GET, url = https://openapi.tuyaeu.com/v1.0/devices/xxx/specifications, params = None, body = None, t =<PHONE_NUMBER>292
2021-09-15 19:28:21 DEBUG (SyncWorker_5) [tuya iot] Response: {
"result": {
"category": "ggq",
"functions": [
{
"code": "switch",
"type": "Boolean",
"values": "{}"
}
],
"status": [
{
"code": "switch",
"type": "Boolean",
"values": "{}"
}
]
},
"success": true,
"t":<PHONE_NUMBER>674
}
Additional context
N/A

In this case you might need to get your device standardized so that it exposes more zones to the API. You can create a ticket in the Tuya service console with your device specifications.
https://service.console.tuya.com/8/2/create?step=3&id=01030503&source=content_feedback.

I contacted support and they said
"Except for 'switch', the other function points belong to the device manufacturer's custom commands, which do not conform to the Tuya Cloud standard command set. Only supports control on the APP, does not support cloud API control."
I asked if there were any plans to allow "custom" commands to be passed through to the device and they said not currently.
Looks like switch is all we get for the time being :(


Remove PropTypes

Closes #359

Removes PropTypes - following React Native move to move away from dynamic type checking
Updates & cleans up Flow types
Clean up / refactoring

@GantMan @sibelius @dgladkov if any one of you wants to review the code, please do!
Otherwise, I'll wait a few weeks and if no one vetoes, I'll merge and release myself ;)


Implement override fields under registry operator reconcile

Which area/kind this issue is related to?

/area registry
Issue Description

With the addition of the new override fields to the DevfileRegistry CRD #1439, changes would need to be done to the reconcile procedure under the controller to have the registry operator update these fields when changes are applied.
Parent Epic: #1446
Acceptance Criteria

[ ] Implement update mechanism for hostnameOverride field under controller reconcile procedure
[ ] Implement update mechanism for nameOverride field under controller reconcile procedure
[ ] Implement update mechanism for fullnameOverride field under controller reconcile procedure
[ ] Update documentation to include usage of override fields
[ ] Run integration tests to verify changes

Parent issue updated to https://github.com/devfile/api/issues/1446.


Possible issue in parallel builds when two file have the same name

I have multiple files named index.nim in my nimiBook in nimiSlides. And I have gotten two errors in the C-compiler/linker regarding this: link to job And I suspect that it might be due to them using the same filenames in nimcache. This is problematic as you typically have multiple index.nim files in a book (one for each section).
A possible solution to this would be to only allow a single file of each filename to be built at the same time. So if a index.nim file is already being built, and another index.nim file is about to be built as well, it will be put aside until the other file has finished.
I think this could be done using a global HashSet[string] and adding a condition here that checks if the current filename is already in the set or not.

Isn't there some compiler option that allows to override the name of file in cache? If so we could just add the index of the file in the list of file to build or something else to disambiguate

I would say it depends on what -o:outputname does when using nim r as there is no non-temporary binary to save. Hopefully it uses that name as well in the cache but I'm not holding my hopes up 😬 will investigate it further tomorrow, I hope I'm wrong though.

another option would be to override the Nim cache folder and replicate the folder structure of source files (+ a base folder that could be the default one)

well if I understand correctly we have issues with a structure like this:
index.nim
section1/index.nim

because when processed concurrently nim creates c files with the same name in nimcache folder (calling it $nimcache here), like $nimcache/index.c. If instead we ask for the second index file to use as nimcache $nimcache/section1 (so that the c file is $nimcache/index.c/index.c` this should solve the issue, right?

still, I had a brief look at the log and not entirely sure the issue is conflict of filenames:
I see a name like /home/runner/.cache/nim/index_d/index_48315C86F820572F8ADD5336011EE02A9F1B2C7F and it looks like filenames are mangled? maybe it is an issue when filenames have same name and same content?
and to be honest I have only a limited knowledge of nimcache logic...

Yes that sounds like it could work. And we would get the benefit of all the cache being intact 👍

ah but you have also stuff like<EMAIL_ADDRESS>so that might create a conflict anyway

ah but you have also stuff like<EMAIL_ADDRESS>so that might create a conflict anyway

Yes exactly, it's those that are the problem I think because it was the C compiler that complained about not finding references. Which would be because we have overridden some file before it was finished compiling or something like that

I will try to fix this in the coming week. What I've found is that it's as easy as passing --nimcache:path to change the nimcache folder. And we can get the current nimcache folder using this code from forum thread:
import std/compilesettings
const nimcache = querySetting(SingleValueSetting.nimcacheDir)

So we just have to combine this nimcache with the relative path of the current document to get the desired results.


Unknown metric error when loading a model trained with stateful metrics

Hi everyone,
I'm trying to load a saved model which has been trained with a custom stateful metric (object of the class ValidAccuracy using the code
dependencies = {
     'ValidAccuracy': ValidAccuracy
}
model = keras.models.load_model(model_file, custom_objects=dependencies)

and I get the error
ValueError: Unknown metric function: {'config': {'batch_size': 32, 'name': 'valid_accuracy', 'trainable': False}, 'class_name': <class 'spearec.run.metrics.ValidAccuracy'>}.
Is there any solution or workaround?

dependencies = {
    'valid_accuracy': ValidAccuracy
}

should solve the issue.
Stateful metric inherits from layer and forces it to lower_case_underscore. As per the error message, it is looking for valid_accuracy and not ValidAccuracy.

Stateful Metrics are custom layers:
If you really want to load a custom model with custom layers: https://github.com/keras-team/keras/issues/4871.
Or you can just do load_weights.
I'm of the opinion that load_model shouldn't even exist.
You shouldn't need the training metrics, or the optimizer to do predictions.

Or you can just do load_weights.
...
You shouldn't need the training metrics, or the optimizer to do predictions.

That works for me, nice and elegant solution, thx!

load_weights worked for me too :-) thanks.


Fix Live Reload Reliability

This pull request is not ready to merge
I've been playing around with beefy and noticing that the live reloading is pretty unreliable, this is my first crack at fixing it. It works very well now for me, but I still need to fix the tests and add some tests for this functionality.
Before I take that step of adding/fixing tests, I'd like to make sure that this approach looks solid:
I found out that the main problem is that watchify takes a while to build its bundle, but the live reload kicks in immediately, and serves up stale content before it can be rebuilt by watchify.
In order to get around this problem, I created a list of files which are managed by watchify, and ignore these files when watching for static file changes. Also, when watchify is finished with its build, I trigger a reload on the client. This ensures that the client update is only triggered when watchify is complete, and allows static files such as CSS to trigger a reload as well.
This approach seems to work really well for me. If this code looks OK, I'll go ahead and fix the tests and add more unit tests. If not, let me know how I can make it better!

This will fix #49 .

I'm getting two page refreshes on every save. First refresh doesn't show new code, second refresh does. Will try to make a patch, but it might take a while.

What's the progress on this? I also get reloads before watchify has finished.

:+1:

@chrisdickinson When you have a chance, could you check this out? Seeing this very often.

I'm not using beefy much anymore, so I'm closing this because I probably won't be keeping it up to date. If anyone else wants to pick up this issue, feel free to reopen or fork my branch!