Face

Is it only results women face?

Currently the pretrained model only supports the women face since the people in the dataset used in our experiments are almost female.

Not working for men faces. And also generated face is not exactly similar to reference image. is it ?
As the faces are not model generated may be we cannot expect 100% copy of original image.

@j-sieger @AIprogrammer I tried the model on Indian model images, it is producing blondes


https://yelp.github.io/pyleus/  404!

hi, all
sorry to find that the document page 404!
pls check that!

I just noticed this myself, but apparently this project is now deprecated.  I would recommend people consider switching to streamparse, since we still actively maintain that (and have developer preview releases that support Storm 1.0).

@dan-blanchard  oops! so cool! and is the streamparse use MessagePack to serializer?

@l3rnao, yes, but it's not super straightforward yet, because that's only possible in the developer preview releases of 3.0.  They're actually quite stable, but we haven't pushed out 3.0 final yet because there are some little things I would like to have in place for the release.  You can install the 3.0 preview with pip install streamparse==3.0.0.dev3.  To use it, you need to follow the steps outlined here.

@dan-blanchard thank you soooo much.  have a nice day : )


Uetate to 1.20.4

Not function properly on 1.20.4 Java and latest bedrock (1.20.60+). Probably need to add some packages from newer version but I can't add new NSM libraries usefully to plugin.

Please do so, we beg of you.

Updated to 1.20.5 in https://github.com/GeyserMC/Hurricane/commit/de9ade84a1a38d7cbbd960cc92fa1f21c8d96ce7


Generation is overriding current PR's

When 2 changes in the generation will happen my commits were lost as automation forcepushes changes to the same branch.
Additionally branch name should include id of the API.
Solution will be to have suffix added to branch with apiID and some timestamp?
Without that we are likely to loose changes.
TL;DR - chore/add-openapi gets overwritten every time changes happen - We have lost update of one api that were overridden by other.

Solution will be to have suffix added to branch with apiID and some timestamp?

yeah, I had tried to include the API ID for this and it did not work for me.
With the create-pull-request action you can do:
with:
  branch-suffix: timestamp

To generate a unique branch.

you can do:

That will solve problem IMHO, but not confident enough to do it myself right now.

Let's see is #134 fixed this.


How to write unit testing / e2e for the services using access token in Angular Project?

Please follow the issue template below. Failure to do so will result in a delay in answering your question.
Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
How to write unit testing for the services using access token in Angular Project?

Can you provide more details about the unit tests that you're trying to write and the things that don't make sense?

I have implemented MSAL Module in app.module.ts
import { BrowserModule } from '@angular/platform-browser';
import { NgModule} from '@angular/core';

import { AppRoutingModule } from './app-routing.module';
import { AppComponent } from './app.component';

import { MsalInterceptor, MsalModule } from "@azure/msal-angular";
import { HTTP_INTERCEPTORS, HttpClientModule } from "@angular/common/http";
import { SharedModule } from './shared/shared.module';

const isIE = window.navigator.userAgent.indexOf('MSIE ') > -1 || window.navigator.userAgent.indexOf('Trident/') > -1;

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    AppRoutingModule,
    HttpClientModule,
    SharedModule,
    MsalModule.forRoot({
      auth: {
        clientId: '<client ID>', // This is your client ID
        authority: 'https://login.microsoftonline.com/<tenant id>', // This is your tenant id
        redirectUri: 'http://localhost:4200/'// This is your redirect URI
      },
      cache: {
        cacheLocation: 'localStorage',
        storeAuthStateInCookie: isIE, // set to true for IE 11
      },
    }, {
      popUp: !isIE,
      consentScopes: [
        'user.read',
        'openid',
        'api://<clientid>/test'
      ],
      unprotectedResources: [],
      protectedResourceMap: [
        ['https://graph.microsoft.com/v1.0/me', ['user.read']],
        ['https://graph.microsoft.com/v1.0/me/drives', ['user.read']],
        ['http://localhost:3000/test', ['api://<clientid>/test']],
      ],
      extraQueryParameters: {}
    }),

  ],
  providers: [
    {
      provide: HTTP_INTERCEPTORS,
      useClass: MsalInterceptor,
      multi: true
  }
  ],
  bootstrap: [AppComponent],

})
export class AppModule { }

I have hosted an express webservice in localhost:3000 ( one of the protectedResourceMap that I mentioned in the MSAL configuration in the app.module.ts). The webservice uses Azure AD passport to validate the Bearer token.
Is there a way to write a unit test case in angular where I can call the azure protected service (http://localhost:3000/test) and check the response ?

Yes, there are many ways to do something like this. I would recommend either of the following:

Unit test the msal functionality by mocking the response using a library like Sinon.
E2E/Integration test the entire app using a library like Puppeteer.

Since everytime you make a request to the server to get a new access token, the actual contents of some of the fields that come back are totally unpredictable by nature. If unit testing, you'll need to account for those fields based on the written test and either mock them or not check against them. Certain things will stay consistent and you will be able to check some of those by looking in localStorage. You can see some examples of mocking out certain function calls in our angular sample.


not show podcast episode list

CPod version:
CPod v1.25.6
i'm using Linux Mint 19 Cinnamon
Description of the problem:
not show podcast episode list in home screen

Steps to reproduce:

Please attach a screenshot of the console log (Settings > scroll to bottom > Open developer tools > Console).

Sorry about this. Please try v1.25.7.

Closed by accident.

it's work now perfect


Using Hg38 with Gemini in Bcbio

In the config files for bcbio I specified tools_on: gemini and also vcfanno: [gemini, somatic]
However, when reading the Gemini documentation now, the documentation says "GEMINI solely supports human genetic variation mapped to build 37 (aka hg19) of the human genome"
How can the pipeline run successfully without reporting an error about the use of GEMINI and Hg38?
Any advice on what to do or how to get around this as I now need to do downstream analysis of the results?

Hi Victoria,
Good question. Did the run not produce the database? We use vcf2db.py to load the GEMINI database, which is compatible with arbitrary genomes. The idea is you annotate the variants with vcfanno and then load them with vcf2db.py and then you can do normal GEMINI queries on them.

@roryk Thanks for clarifying that... I am very new to bioinformatics and am using bcbio and GEMINI for the first time. I will have a look at the output files again and get back to you if I have more questions/trouble.
Thanks so much.
Kind regards,
Victoria

Hi Victoria,
Great! Helping new folks get moving with best practice pipelines is one of our major goals, so welcome aboard!


Make db txs atomic, use Decimals instead of Floats, and replace prints with logs

By making database transactions atomic we can avoid race conditions related to balance updates.
By using Decimals vs. Floats we can avoid imprecise balance calculations.
By replacing prints with proper logging we can save our errors & info messages for later analysis.

Meant to open to firoorg/


Memory Courputing DinkToPdf

I'm trying to run a for loop for 24 HTML files to convert to PDF.
It runs success for first 3 files and then throws this error.

I have the same problem. Have you found any solution?

Yes. Change the library. This library is not actively maintained for years.
This is much better fully HTML5 compatible, Supports parallel conversion, Doesn't need extra dll linking as well as works seamlessly on Windows, Mac, and Linux. The only requirement is Chrome installed on the host system
https://github.com/Sicos1977/ChromiumHtmlToPdf

What you can do is to use this implementation in your loop.
var converter = new BasicConverter(new PdfTools());
var globalSetting = new GlobalSettings
{
ColorMode = ColorMode.Color,
Orientation = Orientation.Portrait,
PaperSize = PaperKind.A4,
};
var pdfBytes = converter.Convert(new HtmlToPdfDocument()
{
GlobalSettings = globalSetting,
Objects = {
new ObjectSettings() {
HtmlContent = template.ToString()
}
}
});


How can i style agm-marker @input - label?

Questions are more suited for Angular Google Maps chatroom on Gitter or Stack Overflow.


adding 'form: auto' to formatCurrency

Description
Adding a new form: 'auto' option to the formatCurrency utility function. auto will heuristically select short or explicit depending on the provided currency option and the pre-assigned defaultCurrency for the I18n instance. The table below describes how the heuristic operates (using USD and CAD as example currencies).

defaultCurrency
currency
result
notes

USD
USD
short
exact match currencies use the short form

USD
CAD
explicit
non-matching currencies use the explicit form

undefined
USD
short
ambiguous matches use the short form

USD
undefined
short
ambiguous matches use the short form

Type of change

[x] @shopify/react-i18n Minor: New feature (non-breaking change which adds functionality)

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above

I'm confused as to why this PR is still open if it got merged already. @patsissons Can this be closed now?

I closed the zombie PR, and now it says merged. 🧟


Make FlatHashTable more memory-efficient

Current implementation stores two indices per bucket, while we can easily get away with one.

Yup, was on my todo list. The bit-packed version already does that.

Addressed in #94 .


How to throw a 500 error from ~gress controllers

Based on the documentation/code, it looks like the right way to throw errors is to use the SCIMMY.Types.Error. This expects a scimType and status which is then handled and converted into a SCIMMY.Messages.Error type here
The problem with using the SCIMMY.Types.Error constructor is that it doesn't support any scimType for statusCode 500.
What is the recommended way to throw a 500 error?

Ok, so this worked for me
import { SCIMMY } from "scimmy-routers";

const scimError = new SCIMMY.Types.Error(
      500,
      undefined as any as string,
      "Internal Server Error. Please try again later."
    )

Hi @always-akshat,
As you pointed out, there's no scimType for a 500 error. This is because the scimType property is primarily used to indicate the type of SCIM protocol issue encountered in a request that resulted in a 400 Bad Request response (as well as 409 Conflict and 413 Payload Too Large responses with values of "uniqueness" and "tooMany" respectively).
The snippet you provided in your follow-up comment should allow you to throw a 500 error that gets translated into the SCIM protocol format, but as far as I'm aware, TypeScript should also let you omit the typing of the scimType parameter:
throw new SCIMMY.Types.Error(500, undefined, "Internal Server Error. Please try again later.");

Thanks,
Sam


Movement (Basic)

[x] stand
[x] jump
[x] air-jump

Forked from https://github.com/SebLague/2DPlatformer-Tutorial/
Good start but not without bugs.
Created #16 to track bug fixing and clean up

[ ] auto-run
[ ] wall-run

converted to #29

[ ] auto-run
[ ] wall-run

converted to #29


start moving padding to standalone module

[ ] Closes #xxxx
[ ] Tests added
[x] Passes pre-commit run --all-files
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

@rabernat do you mind if I check this out and play a bit with it?

Also when did this Readthedocs stuff start originally? It seems to be related to the type hinting being introduced (https://mypy.readthedocs.io/en/stable/runtime_troubles.html), but I am not entirely sure how to fix that.

Please go ahead Julius. I will not be able to touch this again until after the holidays. I need to actually finish my grading today! (See https://github.com/ocean-transport/weekly-standup/issues/20#issuecomment-992548179)


fix emulator appearance

Summary
I found out that some changes I made previously messed up the emulator appearance as can be seen in

I removed the skin param because it is deprecated: https://developer.android.com/studio/run/emulator-commandline#deprecated and that fixed the issue
Test Plan
looks better now

What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Checklist

[x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

@matt-oakes I fixed the invalid links in description, can you please review? Thanks!


Add jupiter.err.ee info extractor

Description of your pull request and other information
Add jupiter.err.ee extractor
Example:

https://jupiter.err.ee/1609145945/impulss',

Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[x] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Copilot Summary

🤖 Generated by Copilot at f1bf8ea
Summary
🌐🎥🆕

Add support for Jupiter website by creating a new extractor class JupiterIE in jupiter.py and importing it in _extractors.py. Extract video information and formats, as well as some metadata fields, from the content API.

JupiterIE class
Extracts video formats
From autumn website

Walkthrough

Add support for downloading videos from the Jupiter website (link, link)

Import the JupiterIE class from jupiter.py in _extractors.py (link)
Define the JupiterIE class in jupiter.py (link)

note. the http headers from chunks are wrong. they all report year 2000 and that is also set for final filename.
http://vod21.err.ee/hls/vod/802840935f9706b97a5113f4ead3cca9/5/v/fragment-58-f2-v1.m4s
Sun, 19 Nov 2000 08:52:00 GMT

how to override this per extractor?

@seproDev, @bashonly this is ready for review

radio show and podcast season and episode are 0:

924eaf334, 9c33a8092

@seproDev have you checked and do you have opinion of saved file dates?

https://github.com/yt-dlp/yt-dlp/pull/8549#issuecomment-1802446009

I don't think there is a way to overwrite the mtime from the extractor. imo. it's fine to leave as is.

I've debugged code, and currently it's not configurable (it just overwrites from headers unconditionally), would need some hacking to make it configurable, or accept from the info extractor. I might look into it if it's possible the change would be accepted.
In such case, what would the timestamp input be? use modified_timestamp? or rather a new keyword: file_timestamp ?

--no-mtime can be used by the user to disable writing the modified time.
Any potential core changes should be made in a separate PR.

thanks @seproDev!


FontAwesome Alternative

Feels like given our relatively specific needs for icons the current FA approach is limiting. We do have a Pro license but given the Open Source nature of this project the usage of that is a bit of a no-no and therefore certain icons (like a Piano roll) are unfortunately out of reach. Ticketing this to check for alternatives before our usage and icon recognition has spread too far to make the change.

Quick searching around led to the following suggestions:

https://pictogrammers.com/ - Nice bc a real big library
https://github.com/Make-Lemonade/iconicicons - seems to cover more than we got with FA, but not as vast as the above


Make manual fully reproducible

While investigating why I can't build exactly the same system configuration on different machines, I've discovered that lib option in manual is causing problems. Here is the picture:

Declared by is invalid and even references a path specific for a local machine.
This patch is a proof-of-concept, when it is applied my system configuration is fully reproducible. Maybe there is a possible proper fix around homeManagerManual / scrubbedEval, but I don't know how to do it.

Wow, nice find! I have completely missed this before. I fixed the problem in a different way, by introducing the lib module into HM instead of pulling it from NixOS. Hopefully it'll work fine for you.

Yes, this works perfectly! Thanks!


Create FHIR Service Request

ServiceRequest is a record of a request for a procedure or diagnostic or other service to be planned, proposed, or performed, as distinguished by the ServiceRequest.intent field value, with or on a patient. The procedure will lead to either a Procedure or DiagnosticReport, which in turn may reference one or more Observations, which summarize the performance of the procedures and associated documentation such as observations, images, findings that are relevant to the treatment/management of the subject.
Peter: How is this created? Simplest might be to create with a StructureMap from the QuestionnaireResponse when a client is registered
Francis: Yes. That's one approach . We can decide on the best approach once we assign a ticket for this incase we need to change the implementation for another alternative

Peter: How is this created? Simplest might be to create with a StructureMap from the QuestionnaireResponse when a client is registered

Francis: Yes. That's one approach . We can decide on the best approach once we assign a ticket for this incase we need to change the implementation for another alternative

I'm still not convinced this is required and that we should do it, marking as blocked.

service request generation is handled in #599, that looks like it covers this issue, closing


Update plain example app to latest version of react native

Updates the React Native plain example app to use the latest version of React Native. The current example app does not build successfully when using XCode 10 due to a variety of issues.
This was achieved by running react-native-git-upgrade and resolving conflicts manually. I have tested this on the Android + iOS apps running in release mode, and confirmed that errors are reported for each available button.
We may also want to create tickets to update the cocoapods and codepush sample projects, as they will be affected by the same issue.

Can we get CI passing?

I've been attempting to get CI passing on a separate test branch - a bit of additional config is required to get Travis working with XCode 10.

I believe this should now pass CI, by changing to an xcode 10 image on Travis and using the legacy build system. See this issue for further detail: https://github.com/facebook/react-native/issues/19573


Feat: BackTOTOp button is not present in about and team secction

Describe the feature
It will help user to grab more number of user engagement.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hey @Damini2004

If you want to work on this issue, let me know and share some idea on discord.

@Damini2004 assigned add this icon to the resourcepage


Consider logger candidate from logging module only

Refer: https://github.com/charliermarsh/ruff/pull/3718#issuecomment-1496105349
cc @charliermarsh The code seems to be a bit messy (lot of branches). If you can take a look, it would be useful :)

I think we found the reason it didn't show up in the ecosystem check. The artifact contained that false positive which when diffed with the updated code still produced that warning, thus cancelling each other out.


(MODULES-661) Allow remote grants

You can now create remote databases, grants, and roles
Not sure about the best way to test this, going to do some beaker tests to run against localhost.
I assume undef's are a bad thing in future parser too? I can change that if needed :)

Hi @petems! undefs should be fine, but yeah, will need test and doc updates

i'm a big fan of undefs, https://github.com/puppetlabs/puppetlabs-concat/pull/270#issuecomment-75937714

@petems ping?

This is dependant on the hash conncection settings by @tomhey, so I'll close this :+1:


Copyright holder is Glasgow University?

Copy'n'paste error? :)  "Copyright 2004, The University Court of the University of Glasgow."
I don't know who the real copyright holder(s) should be, so no pull request.

Well the code of psqueues was based on the IntMap code from the
containers library, which has exactly the same licence:
https://github.com/haskell/containers/blob/07ab0fa052843dc8fd4c874876d03d8a71525f87/LICENSE
.
It seems to be a BSD-style licence so I'd gather that it's OK. Or would you
suggest to change it and remove the Glasgow Reference?
2015-05-11 15:38 GMT+02:00 Thomas Schilling<EMAIL_ADDRESS>
Copy'n'paste error? :) "Copyright 2004, The University Court of the
University of Glasgow."
I don't know who the real copyright holder(s) should be, so no pull
request.
—
Reply to this email directly or view it on GitHub
https://github.com/bttr/psqueues/issues/4.

It may be that Glasgow still has copyright on this code, but probably not exclusively. So the question is: who else holds the copyright?


fix: support len validator

attr

Codecov Report

Merging #188 (e18bdfb) into master (5d06e6e) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
- Coverage   96.79%   96.69%   -0.11%
==========================================
  Files          39       39
  Lines        2963     2964       +1
==========================================
- Hits         2868     2866       -2
- Misses         95       98       +3

Impacted Files
Coverage Δ

src/validator.js
100.00% <100.00%> (ø)

src/adapters/sequelize.js
93.51% <0.00%> (-1.03%)
:arrow_down:

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 5d06e6e...e18bdfb. Read the comment docs.


.NET Portability Analyzer not avalaiable for VS 2022

This page mentions the  .NET Portability Analyzer.
It turns out that this VS extension is not available for Visual Studio 2022.
It does appear to be available for Visual Studio 2019.
You could modify the page to mention that fact or go bug the team who works on that extension to update it.

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

ID: 1bd94618-808b-ebda-96d8-bcf0d41ccfda
Version Independent ID: 1bd94618-808b-ebda-96d8-bcf0d41ccfda
Content: Migrating from .NET Framework to .NET Standard - Business Central
Content Source: dev-itpro/developer/devenv-migrate-from-dotnet-framework-to-dotnet-standard.md
Service: dynamics365-business-central
GitHub Login: @KennieNP
Microsoft Alias: kepontop

Thanks for the feedback. We'll update the topic to mention that. Will be available with the next refresh of the docs. Thanks.


the new updated backtrack

its not that bad but phook headpositions meme is hitting more
https://youtu.be/QnXB9OzO_kw
tested it again : it hits usually when aiming at last tick but when u aim in the middle u can miss like 20 shots
its so inconsistent

It really is way better now. I do not have any problems with it after the newest commit.


Converted CRA examples to Vite

Motivation and Context

Jira Link:
Description

How has this been tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] My code follows the code style of this project. (run lint)
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Check #640


http://api.col.plus/dataset - filtering and sorting

Based on input from @yroskov
We need to be able to

orderBy

created
modified
title
author

api.col.plus/dataset?orderBy=["created", "DESC"]

We also want to be able to filter the dataset list to see only

datasets created after last release of scrutinized cat
datasets modified after last release of scrutinized cat

Since the client has access to the date of the last release, this could simply be
api.col.plus/dataset?modified={"gt": "2018-10-25"}

i.e. allowing json params for greaterthan and lessthan operators

Maybe simpler
api.col.plus/dataset?min-modified=2018-10-25

I'd like to avoid passing json parameters, but the min-modified/created solution looks fine to me.
Sorting seems straight forward. What field exactly should author be, contactPerson?

What field exactly should author be, contactPerson?

authorsAndEditors

ok, then use the first entry of authorsAndEditors which is a list

Yes

Contact Person and Author/Editor have different roles. Author/Editor generates content. Contact Person is quite often an another person who is responsible for export and holds routine communications.

Implemented:
Extended existing sortBy parameter
Extend the search request with modified, created and released props that all act as minimum dates

Also added a reverse parameter that rerverses the sort order:
https://github.com/Sp2000/colplus-backend/commit/afc8ef9377b12c5ac9095bd9afc823a9166e7328


Improve behavior of cfpropertylist extension

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
This extension sometimes causes problems when used with RVM gemsets.
Description
Be careful about removing the #to_plist method in case it was already removed.

@jdee Is this good to merge in your opinion?

Can we get this merged in, I'm sick of copying this code into my local gem everytime it brakes, seems like a no brainer...


Use PHPUnit 10

Closes https://github.com/doctrine/mongodb-odm/issues/2509
Since we are using PHP 8.1, we don't need PHPUnit 9 anymore, I've installed rector locally and replaced annotations with attributes.

I'm surprised so little had to be done :) One thing that is off though is number of tests executed:
This PR's PHP 8.1, Server 6.0, highest executed 2646 tests
2.6.x's counterpart has 2467 tests
Am i looking correctly that there is a test that got lost somewhere?

Actually there's a chance that PHPUnit's deprecation was counted towards number of tests:
There was 1 PHPUnit test runner deprecation:

1) Your XML configuration validates against a deprecated schema. Migrate your XML configuration using "--migrate-configuration"!

but that be weird decision from PHPUnit's team

I'm surprised so little had to be done :) One thing that is off though is number of tests executed:
This PR's PHP 8.1, Server 6.0, highest executed 2646 tests 2.6.x's counterpart has 2467 tests
Am i looking correctly that there is a test that got lost somewhere?

Good catch! it was my fault in https://github.com/doctrine/mongodb-odm/pull/2564/files#diff-62b86ab419dbc39877816e5c1f489b8fa4110e14bb395c2ba9f7cd961720ef60L13-L17
    /**
     * @testWith ["groups"]
     *           ["groupsSimple"]
     */

was changed to
    #[TestWith(['groups', 'groupsSimple'])]

instead of
    #[TestWith(['groups'])]
    #[TestWith(['groupsSimple'])]


Record update syntax with qualified name

I get a compiler error when using a qualified name in a record update statement e.g.
bug = { Counter.myData | value = 4 }
Gives
-- SYNTAX PROBLEM ------------------------------------------ .\.\CounterPair.elm

I ran into something unexpected when parsing your code!

8▒ bug1 = { Counter.myData | value = 4 }
            ^
I am looking for one of the following things:

    a closing bracket '}'
    a lower case name
    whitespace

Detected errors in 1 module.

Similar to https://github.com/elm-lang/elm-compiler/issues/951
Can be reproduced here: https://github.com/mcapodici/elm-architecture-tutorial/tree/master/examples/2

There is already an issue covering this. Please check, then close this duplicate here.

https://github.com/elm-lang/elm-compiler/issues/1068

Correct it is a duplicate. Happy if you close this or i can next time im
logged in. Sorry i didnt find this i did search before i raised.
On 20/02/2016 9:53 PM, "Janis Voigtländer"<EMAIL_ADDRESS>wrote:

There is already an issue covering this. Please check, then close this
duplicate here.
—
Reply to this email directly or view it on GitHub
https://github.com/elm-lang/elm-compiler/issues/1293#issuecomment-186570313
.

Yeah, it's difficult to find, because Evan "hides" it behind the meta issue collecting all record related stuff into one. Please close the issue yourself, I don't have the access rights to do so in this repository.


if I can propose some small features

I would like to propose these small features, partly already feasible in windows but really badly done and managed, I think it would take your implementation of the windows taskbar to a higher level.

As you can see from the small preview in the attachment, the idea would be to be able to position the icons of the open software always in the standard way from left to right, but alternatively to be able to pin fixed icons on the bar, which can be positioned in 3 points of the bar, left (normal method), center or right.
So for example, having in pin the icons of your favorite software positioned on the right and additional software open if necessary on the left.
Currently you have to create a new folder with linked icons to put them on the right, but it's a mess when this thing could be done natively on the taskbar.

Another feature would be to be able to put folders on the right, and going up in mouse hover, the subfolders are cascaded so as to have for example a general folder where you can quickly access other subfolders without these being opened in the explorer . This is already half doable with add to toolbar, but it works poorly.
I would like the icon of the general folder on the bar and going above to open the subfolders menu ... a bit like what happens on Apple's OSX.

these ideas you just don't like?


Error occurred loading flow for integration luxtronik2

I was running 2023.10.20-Alpha (I am 90% sure, it was 10.20) and it was working fine.
Then i updated to 2023.10.31-Beta and the integration did no longer start after restarting home assistant.
Then I downgraded to 2023.10.20-Alpha and rebootet home assistant and I get the same error.
See Home Assistant Core logs:

Error occurred loading flow for integration luxtronik2: Exception importing custom_components.luxtronik2.config_flow
1:55:14 PM – (ERROR) config_entries.py
Unexpected exception importing platform custom_components.luxtronik2.config_flow
1:55:14 PM – (ERROR) loader.py
Setup failed for custom integration luxtronik2: Unable to import component: Exception importing custom_components.luxtronik2
1:54:55 PM – (ERROR) setup.py
Unexpected exception importing component custom_components.luxtronik2
1:54:55 PM – (ERROR) loader.py

Please help

yes you need a backup

Same problem. Downgrade to 2023.10.20 did not resolve the issue.

Use the fix wich benpru provided in const.py
Then it works

Am 01.11.2023 um 19:50 schrieb Ronicon98 @.***>:
Same problem. Downgrade to 2023.10.20 did not resolve the issue.
—
Reply to this email directly, view it on GitHub https://github.com/BenPru/luxtronik/issues/184#issuecomment-1789479654, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSVZLTEELZUWJCI5Q7YFVLYCKKYRAVCNFSM6AAAAAA6ZIH6LSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGQ3TSNRVGQ.
You are receiving this because you commented.

Where do I find this fix?

https://github.com/BenPru/luxtronik/commit/fc7bdf5c063678b3425278b66bb62480d56e5666￼

Fix missing import · @.***
github.com

Am 01.11.2023 um 19:59 schrieb qubabos @.***>:
Where do I find this fix?
—
Reply to this email directly, view it on GitHub https://github.com/BenPru/luxtronik/issues/184#issuecomment-1789491873, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSVZLWPOPJC6OYN6GCZUB3YCKL2FAVCNFSM6AAAAAA6ZIH6LSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGQ4TCOBXGM.
You are receiving this because you commented.

"Access to GitHub was denied" if I click that link

2023.11.1-Beta from HACS fixes this issue.
Thank you <3


VSCode unable to find any tasks when you use a certain regex as problemMatcher pattern.

VSCode Version: 1.33.0-insider / commit ...7f0d8
VSCode Version: 1.32.3 / commit ...eaae4
OS Version: Windows 10 Pro Build 17134

VSCode shows no tasks to run when you use certain regular expressions as regexp in pattern in problemMatcher. It fails to recognize any tasks (even other tasks.)
Terminal > Run Task > No task to run found
Steps to Reproduce:

Create tasks.json
Create this task:

{
"label": "Angular Linting",
"type": "shell",
"command": "ng lint",
"problemMatcher": {
"pattern":[
{
"regexp": "^(ERROR).*?([^\\s]*)[(\\d+).*?(\\d+)(.*)$",
"severity": 1,
"file": 2,
"line": 3,
"column": 4,
"message": 5,
}
]
},
}

Does this issue occur when all extensions are disabled?: Yes (Insider)
Background: I wanted to write a problem matcher for the Angular CLI "ng lint" command, with example output:
ERROR: c:/Code/Project/Test/e2e/app.po.ts[4, 5]: expected call-signature: 'navigateTo' to have a typedef
By the way, the Regexp playground recommended in the problem matcher guide seems to use a different syntax than VSCode.

That's not a valid regular expression. However, tasks shouldn't fail to enumerate because of it.


Define minimal valuable implementation

Summary
Create an operator which synchronizes seccomp profiles from config maps to file system locations.
Use cases and possible issues

A user creates a new profile and the operator syncs them on disk for each node

How to map local paths to the profiles?

A user deletes a profile

TBD

A user updates a profile

Probably create a new profile and do not delete the old one

A user wants to apply a profile to a workload

The operator takes care of adding the right annotation to the workload
We have to define what the user has to do to apply the profile (apply a label, annotation, …)

Implementation details

Operator DaemonSet
Watch ConfigMaps only on operator namespace
Security best practices

Non-root operator!
Non-root agents?
RBAC permissions for the ServiceAccount

Testing

E2E - prow (blocked - sig repo)

Documentation

Security concerns?

Easy deployment method

Single plain file

Events

Persisted files
Event for failures

I was wondering whether is it worth adding this too:

A user wants to ensure the appropriate ACL permissions are set for all profiles across all nodes.

The operator will create directories and profiles with only read & write permissions (0600) to the user running the Kubelet.
The operator will consolidate ACL permissions to ensure that files inside seccomp-root/operator is only read and write by the user running the Kubelet.

I was wondering whether is it worth adding this too:

A user wants to ensure the appropriate ACL permissions are set for all profiles across all nodes.

The operator will create directories and profiles with only read & write permissions (i.e. 0600) to the user running the Kubelet.
The operator will consolidate ACL permissions to ensure that files inside seccomp-root/operator have read and write permissions (i.e. 0600) only to the user running the Kubelet.

Sure, sounds good. :)

On the "delete" function for MVP we could reconcile only when the entire ConfigMap was deleted. It would then check the control plane for containers referring to the profiles that are to be deleted and raise a failure event in case they are still in use. Then the entire physical folder (that represents the given ConfigMap) would be deleted.
Later on, once we have a CRD we could add validation to block profiles in use from being deleted in the first place. That actually would actually resolve some of the scenarios of zombie profiles.

On the "delete" function for MVP we could reconcile only when the entire ConfigMap was deleted. It would then check the control plane for containers referring to the profiles that are to be deleted and raise a failure event in case they are still in use. Then the entire physical folder (that represents the given ConfigMap) would be deleted.
Later on, once we have a CRD we could add validation to block profiles in use from being deleted in the first place. That actually would actually resolve some of the scenarios of zombie profiles.

Sounds like a plan, we could also rethink that we only allow one profile per config map, but this seems very restrictive, right?

I would agree. I think the challenge on trying to allow only one profile per config map is that we won't be able to validate that at creation time, which may impact user experience. Feels like one of those decisions to be deferred until we decide to support customs CRD's.

/kind feature
/priority important-soon

@hasheddan @pjbgf I'd prefer that we cut-out the next necessary steps to achieve this goal and put them into separate issues. WDYT?

Closing this one, we can handle the open bits in dedicated issues.


Add installation instructions for Alpine Linux

@jirutka thank you for the contribution and apologies for the delay!


German translation for powered, support, proton-c and the categories directory

Sup,
I translated a couple more pages for the german translation. It would be nice to know which pages don't need translations due to being auto-generated, I know this is the case for keyboards.md at least;
As always feedback concerning my use of commas and spellings etc is greatly appreciated.
Cheers

To my knowledge, _i18n/keyboards.md is the only page that's auto-updated and should be ignored for translation purposes.

Actually, can you revert that last commit? _i18n/de/categories/keyboards.md is a different file from the auto-generated _i18n/en/keyboards.md and serves a different purpose. _i18n/de/categories/keyboards.md is still purposeful when translating.

Opp, yes I added it back. Since this PR and #85 translated some of the same files I'd suggest we go with #85 since I already reviewed it and it has the proper pronouns. And then just add the still missing files from #82 .
Should I delete:

_i18n/de/powered.md
_i18n/de/support.md
_i18n/de/sidebar.md (covered by #86 )

from this PR or can you cherry pick it?
@john2ksonn can you have a review proton-c.md and the others please? thanks

Oops, yes I added it back. Since this PR and #85 translated some of the same files I'd suggest we go with #85 since I already reviewed it and it has the proper pronouns. And then just add the still missing files from #82.

I like this idea; let's do that.

Should I delete:
* _i18n/de/powered.md
* _i18n/de/support.md
* _i18n/de/sidebar.md (covered by #86 )

from this PR or can you cherry pick it?

It's actually easier on our end if you just delete those files from this branch. I could cherry-pick it but then I'd have to submit those changes in a new PR, which means you wouldn't get credit for those changes, and that doesn't sit right with me.

@kuchosauronad0 Looks like I merged the German PRs in the wrong order. Could you rebase this branch, please?

Hmmm, interesting that https://github.com/qmk/qmk.fm/pull/82/commits/6fbc129c19b0c14bd2022b986129dd6676eb9362 seems to have been made in the future. GitHub weirdness, I'm sure.


Session (opened tabs) is lost after Code update

Whenever I update Code to new release, next time I start the Code (e.g. after system restart) the What's new tab pops up and all the previously opened tabs are gone. Just the What's new tab is opened and I can't see an option to reopen the previous tabs.
I have been using the "window.restoreWindows": "all" option.

VSCode Version: 1.20.0
OS Version: Fedora 27

Steps to Reproduce:

Have one Code window with multiple file tabs opened
Update Code to new version
Close and open Code
The tabs from step 1 are not opened, just What's new tab is

Does this issue occur when all extensions are disabled?: I would need to wait for another code release to see whether the issue occurs without extension but I doubt it would have any effect.

I have reopened the editor without extensions and I'll try to keep it open till the new version is out. By the way I use the following extensions: GitLens, Python, RPM Spec, Sublime Text Keymap.

Likely https://github.com/Microsoft/vscode/issues/12058 and https://github.com/Microsoft/vscode/issues/39125


how to zoom to layer which is invisible at load time?

I have a layer which , its maxResolution is set as 10 while adding on the map. It will only visible at a that zoom level. But when user clicks on a button called zoomto layer against that layer, i want to zoom to that particular layer . But when i tried to take extent of that layer its always getting infinity as its not visible in the map , my code ,
let lay = this.mapService.getLayerByName(layer.layerName);
let resolution = lay.getMaxResolution();
let map = this.mapService.getMap();
let zoom = Math.floor(map.getView().getZoomForResolution(resolution)+1);
map.getView().setZoom(zoom);
let layerExtent = lay.getSource().getExtent();
console.log("layerExtent",layerExtent)
this.mapService.fitExtent(layerExtent);
map.getView().fit(lay.getSource().getExtent(), map.getSize());
map.getView().setZoom(zoom);](url)

is there any way to zoom to a layer which is not visible at that moment ?

Hello, we had the same problem and decided to calculate the extent from the source table and
wrote a custom function on the server which returns the extent on demand (with http request).
If your data is static you can just store the extent value to a variable and use that variable.


fix: upgrade volarjs/services from v0.0.4 to v0.0.5

The published v0.0.4 (f5d4949) of volar-service-typescript causes error because it tries to access ctx.typescript.languageService; in fact it was undefined. After f5d4949, the v0.0.4 continues but no new version
of it was published on npmjs. v0.0.5 has just been released, in which the above problem was fixed. Some API changes were compensated for.
Changes

Version-bumping volar-service-* from v0.0.4 to v0.0.5
Upgrading @volar/* to v1.7.0
Some API changes were compensated.

Testing

Before: It errored, looks like this:
~/.local/state/nvim/lsp.log
[ERROR][2023-06-03 14:42:35] .../vim/lsp/rpc.lua:734    "rpc"   "/home/an7/.local/share/nvim/mason/bin/astro-ls"        "stderr"        "service typescript cr
ashed on file:///home/an7/files/dev/coccoc-astro/src/pages/search/index.astro.tsx TypeError: Cannot read properties of undefined (reading 'languageService')\n
    at /home/an7/.local/share/nvim/mason/packages/astro-language-server/node_modules/@astrojs/language-server/node_modules/volar-service-typescript/out/servic
es/diagnostics.js:12:40\n    at Object.provideSemanticDiagnostics (/home/an7/.local/share/nvim/mason/packages/astro-language-server/node_modules/@astrojs/lang
uage-server/node_modules/volar-service-typescript/out/index.js:278:24)\n    at Object.provideSemanticDiagnostics (/home/an7/.local/share/nvim/mason/packages/a
stro-language-server/node_modules/@astrojs/language-server/dist/plugins/typescript/index.js:53:56)\n    at /home/an7/.local/share/nvim/mason/packages/astro-la
nguage-server/node_modules/@astrojs/language-server/node_modules/@volar/language-service/out/languageFeatures/validation.js:261:52\n    at /home/an7/.local/sh
are/nvim/mason/packages/astro-language-server/node_modules/@astrojs/language-server/node_modules/@volar/language-service/out/utils/featureWorkers.js:23:65\n
  at safeCall (/home/an7/.local/share/nvim/mason/packages/astro-language-server/node_modules/@astrojs/language-server/node_modules/@volar/language-service/out
/utils/featureWorkers.js:133:22)\n    at /home/an7/.local/share/nvim/mason/packages/astro-language-server/node_modules/@astrojs/language-server/node_modules/@
volar/language-service/out/utils/featureWorkers.js:23:50\n    at async visitEmbedded (/home/an7/.local/share/nvim/mason/packages/astro-language-server/node_mo
dules/@astrojs/language-server/node_modules/@volar/language-service/out/utils/definePlugin.js:12:18)\n    at async visitEmbedded (/home/an7/.local/share/nvim/
mason/packages/astro-language-server/node_modules/@astrojs/language-server/node_modules/@volar/language-service/out/utils/definePlugin.js:6:14)\n    at async
languageFeatureWorker (/home/an7/.local/share/nvim/mason/packages/astro-language-server/node_modules/@astrojs/language-server/node_modules/@volar/language-ser
vice/out/utils/featureWorkers.js:20:9)\n"

Steps: Built it with pnpm build, then npm pack-ed the language-server. Went to mason.nvim's astro-language-server installation and load the packed file into the package.json, then installed it.
After: The LSP in Neovim works, albeit with some settings about typescript.tsdk that must be set.
VSCode was not tested (I'm not a VSCode guru), though pnpm test was good.
Docs

Public documentation was not updated and I believe that we don't need to, since this PR only touches internal stuff. The typescript.tsdk configuration was needed to run in Neovim though (since @astrojs/language-server@2.0.0), but that's for the nvim-lspconfig documentation.

Hi maintainers. Something happened in the Windows CI, and frankly I don't understand one bit of it... Looks like a heap memory error or something?

I also found windows CI failed in https://github.com/vuejs/language-tools/pull/3248, probably a regression of v1.7, pin v1.6.9 and services v0.0.4 should temporarily avoid the problem.

I also found windows CI failed in vuejs/language-tools#3248, probably a regression of v1.7, pin v1.6.9 and services v0.0.4 should temporarily avoid the problem.

That would means this PR is on hold. Has you figured out why it happens?

@intagaming How about closing this PR and creating a new PR that adds the PIN?

"@volar/xxxx": "^1.6.9" -> "@volar/xxxx": "1.6.9"
"volar-service-xxxx": "^0.0.4" -> "volar-service-xxxx": "0.0.4"

@intagaming How about closing this PR and creating a new PR that adds the PIN?
* `"@volar/xxxx": "^1.6.9"` -> `"@volar/xxxx": "1.6.9"`

* `"volar-service-xxxx": "^0.0.4"` -> `"volar-service-xxxx": "0.0.4"`

v0.0.4 was broken and will cause the error in the original post, thus can't be used anymore. Is there anything that I missed or something? Do your config works with 0.0.4?

The current issue is that the dependency "@volar/xxxx" is being installed as version 1.7.x, which causes problems. So I tried reinstalling it as version 1.6.9, and the language server started up without any errors.
e.g.:
$ mkdir -p /tmp/astrols-check
$ cd /tmp/astrols-check
$ npm init -y
$ npm i @astrojs/language-server
$ cat<EMAIL_ADDRESS>| grep "version"
	"version": "1.7.1",
$ cat node_modules/volar-service-typescript/package.json | grep "version"
	"version": "0.0.4",
$ # PIN Like
$ npm i<EMAIL_ADDRESS>$ cat<EMAIL_ADDRESS>| grep "version"
	"version": "1.6.9",

I will try it again, but I'm pretty sure the git-head commit hash I checked the other day in the pulled package.json was bang-on v0.0.4, which assuming the npmjs package repository does its job, will definitely cause error. I will try one more time just to be absolutely sure.

I have also tried it in this repository itself. There are no errors in launching the language server or any issues behind the scenes after it starts, and it is working fine.
1. Prepare:
mkdir -p /tmp/astrols-check
cd /tmp/astrols-check
git clone git@github.com:withastro/language-tools.git
cd language-tools

2. Change packages/language-server/package.json:

I will fix it to version 1.6.9.

{
  // ...snip
    "@volar/kit": "1.6.9",
    "@volar/language-core": "1.6.9",
    "@volar/language-server": "1.6.9",
    "@volar/language-service": "1.6.9",
    "@volar/source-map": "1.6.9",
  // ...snip
}

3. Install and build:
pnpm i
pnpm build

cat<EMAIL_ADDRESS>| grep "version"
	"version": "1.6.9",
cat packages/language-server/node_modules/volar-service-typescript/package.json  | grep "version"
	"version": "0.0.4",

That would means this PR is on hold. Have you figured out why it happens?

Not yet, I'm trying to investigate but the error is the same.


actionmenu: should function as a controlled component

Related packages

actionmenu
dropdownmenu (Perhaps?)

Enhancement Request
Desired behavior
There should be a prop on action menu that allows external control over which item is selected. Similar to this:
 <select value={this.state.value} onChange={this.handleChange}>
            <option value="grapefruit">Grapefruit</option>
            <option value="lime">Lime</option>
            <option value="coconut">Coconut</option>
            <option value="mango">Mango</option>
          </select>

Found here.
Today's behavior
The component can't expose its state.
The Value Add
This means that external state cannot affect the displayed option. For example, adjusting another actionmenu used as a filter wouldn't reset this one back to its initial state on change.
The Tradeoffs
More complex API. Otherwise, we don't see any.

Triage: We need to look at the current api and potential futures.  Plan to pair it.

Hi Travis, following up on this stale issue:
Recently, ActionMenu now supports value in ActionMenu.Item and onChange in ActionMenu.
https://github.com/pluralsight/design-system/blob/master/packages/actionmenu/CHANGELOG.md#510-2019-07-30
And Dropdown, which uses ActionMenu does what I think is exactly what you want here, take value and an onChange, making it feel very analogous to native <select />.
https://github.com/pluralsight/design-system/blob/master/packages/dropdown/CHANGELOG.md#210-2019-08-06
I hope this new api feels more natural and usable to you.  Thanks for your input!


Show / Hide control arrows around timepicker

Add an optionnal parameter on timepicker object in order to control the visibility of the controls arrows.

The history needs to be cleaned up on this PR.

is this PR ever going to be accepted?

Not sure if we should. Why not add your own custom template or css to achieve this goal?

It looks like this is already in master in a form with show-spinners.


Create FileSystemPath (Absolute or Relative path)

Alternative approach to #77
VirtualPath is used for Inputs/Outputs, while FileSystemPath is used by the Toolchain for locating tools/resources. I'm not sure if this actually makes sense, but it results in cleaner filesystem access with less fatalErrors and preconditions.

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


do not use dot as keydelimeter

What is actually the issue with using a dot as key delimiter?

We are using natural language statements as message ids, they can contain
dot as end of statement, so fetchtranslation method will fail.
26 дек. 2016 г. 21:45 пользователь "Gerben Meyer"<EMAIL_ADDRESS>написал:

What is actually the issue with using a dot as key delimiter?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zoover/react-i18nify/pull/19#issuecomment-269233931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOzU0ysrHVDnbIm3TIMxmn-nx2B5GYoYks5rMAs7gaJpZM4LVYuh
.

+1

We have a limilar solution with some legacy translation content base.
login.login has a text ["Anmelden"], however login.login.details ["Bitten credentials eingeben"]
The login.login becomes a treenode and the "Anmelden" is lost.


Use pathlib wherever appropriate

Replace all raw path manipulation with pathlib to avoid future platform compatibility issues.

The clone module is the only one left to fix.

Resolved with 2adb91328b39fa51ceb09dad1f0d0188da3d9bf6


Authentication Support

Added support for authentication.
This includes a login page and page guards on every page except the login page.

Closes #2


Not able to add command line parameters

I am trying to create a shortcut to a program I have written that requires a command line parameter like this:
MyProgram.exe Parameter1, Parameter2
SMM works if I leave off the parameters but that's no help.  If include the parameters, SMM complains that the file path does not exist.  Bummer.

To provide command line arguments, make sure the 'Shortcut Type' is set to 'Command' not 'Software'.
Software shortcuts are for single executable files which are executed directly, whereas command shortcuts can accept an entire terminal command (including arguments) which is passed to the command line process.
Hopefully that will solve your problem 🙂

After playing around a bit, I discovered that I had to sign out then sign in again.  I have no clue as to why.


Separate HTM data feed from POST data processing

Because much of the ST data needs to be aggregate or processed in some way before it gets fed into HTM, the best way to do this would be to completely separate the HTM process from the ST data process. The server will still accept POST data from ST the same way, and it will save it to InfluxDB, but there will be no HTM involved. It may be that we want another entirely different program to extract data from InfluxDB, pass it into HTM models, and keep track of their progress.
I'm thinking about separating this project into three parts:
.
├── LICENSE
├── cli
├── data-server
└── htm-processor

Command Line Interface
This currently exists as cli.py and depends upon some of the code in runapp.py, but I want to make it more stand-alone. See the README for a description of what it currently does.
Data Server
This will be an HTTP server with the following responsibilities:

Accept data POSTs from SmartThings and write them to InfluxDB
Provide a web interface for users to see SmartThings data graphed over time (including HTM anomaly values or predictions, if they exist).

Currently this functionaly exists through the runapp.py python program, but it also includes HTM processing, which will be removed and put into the "HTM Processor" described below.
HTM Processor
I imagine this will be a program that can be run at any time, and will inspect the data in InfluxDB and pass any non-processed data into HITC. Models will be created automatically when new data series are found. This processor needs to keep track of the last time each model has seen data, so it knows where to start feeding more data. It will also be responsible for allowing the configuration of models to include aggregation.

@jefffohl Any thoughts?

There are still a few things I am cleaning up...


docs: improve timeout doc

It was a bit of a pain to figure out what would be thrown if a timeout was reached, yet figured out it was HeadersTimeoutError, BodyTimeoutError or a ConnectTimeoutError. I added a description to the timeout method that explains that it throws that error.

@Nytelife26 correct me if I'm wrong but this PR is generally pointless because there are many errors that could occur and these timeout errors would still happen even if you didn't use PetitioRequest#timeout since there are defaults.

these timeout errors would still happen even if you didn't use PetitioRequest#timeout since there are defaults

It is quite well in-scope for this PR to document those too, then.

and as requesting is promise based users should be catching errors anyways

They should be handling errors, yes. It is hard to create specific handlers without knowing the error types in advance, though.

technically when it does throw a timeout it's not from timeout right?

Correct. The timeout throwing documentation should probably go under
PetitioRequest#send, and be linked with @.***onPetitioRequest#timeout` if that's okay.
Thanks in advance.

technically when it does throw a timeout it's not from timeout right?

Correct. The timeout throwing documentation should probably go under
PetitioRequest#send, and be linked with @.***onPetitioRequest#timeout` if that's okay.
Thanks in advance.

Alright just wanted to clear that up, I'll add these when I have time today

@diced Status on this? It's been a while.

@diced Status on this? It's been a while.

Sorry forgot about it since, I wasn't able to commit due to the tests not succeeding because one of the websites used in the tests was down at the time, I'll make a commit by tonight.

Sorry forgot about it since, I wasn't able to commit due to the tests
not succeeding because one of the websites used in the tests was down at
the time, I'll make a commit by tonight.

Ah. Sincerest apologies then, good to hear from you.
As for that, I'm working on internalizing testing by using a local
development server, so that shouldn't be an issue in the future.


Make script compatible with Monterey

In Monterey php is not installed by default.
This PR checks multiple locations for php.
This is the same logic as alfred-laravel-docs uses https://github.com/tillkruss/alfred-laravel-docs/blob/main/info.plist

Thanks, I didn't have the time to look that up yet 👍


training custom dataset problem

gt_masks_ann.append(ann['segmentation'])

KeyError: 'segmentation'

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

Thank you for your prompt response and exciting code. Let me try. Do you have any other suggestions for training custom dataset or how can i set in config?

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

It works, but  loss_cls: nan
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
2020-04-17 13:04:36,691 - mmdet - INFO - Epoch [1][50/4553]     lr: 0.00428, eta: 50 days, 21:20:20, time: 3.219, data_time: 2.560, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
2020-04-17 13:05:04,312 - mmdet - INFO - Epoch [1][100/4553]    lr: 0.00864, eta: 29 days, 19:26:31, time: 0.552, data_time: 0.010, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
@thuyngch

Hi @betterhalfwzm ,
I fixed the issue related to 'segmentation' for you. Please check it here: https://github.com/thuyngch/ATSS-EfficientDet-PyTorch/commit/3f734f5ae328bf0be75aa58a745f498a2500a0ea

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

It works, but loss_cls: nan
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
2020-04-17 13:04:36,691 - mmdet - INFO - Epoch [1][50/4553] lr: 0.00428, eta: 50 days, 21:20:20, time: 3.219, data_time: 2.560, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
2020-04-17 13:05:04,312 - mmdet - INFO - Epoch [1][100/4553] lr: 0.00864, eta: 29 days, 19:26:31, time: 0.552, data_time: 0.010, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
@thuyngch

How many batchsize and GPUs did you set?
If batchsizeGPUs=16n, set lr_start=ne-2.
For example, if batchsizeGPUs=32=16*2, then set lr_start=2e-2.

Hi @betterhalfwzm ,
I fixed the issue related to 'segmentation' field for you. Please check it here: 3f734f5
谢谢！！我现在数据只有一类，num_classes=2, 但是我的类别是1，我需要转为0吗？
@thuyngch

Hi @betterhalfwzm ,
I fixed the issue related to 'segmentation' field for you. Please check it here: 3f734f5
谢谢！！我现在数据只有一类，num_classes=2, 但是我的类别是1，我需要转为0吗？
@thuyngch

Please use English!

Hi @betterhalfwzm ,
I fixed the issue related to 'segmentation' field for you. Please check it here: 3f734f5
谢谢！！我现在数据只有一类，num_classes=2, 但是我的类别是1，我需要转为0吗？
@thuyngch

Please use English!

Thank you!! Now I have only one class of data, num_classes = 2, but my class is id is 1. Do I need to change to 0?

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

It works, but loss_cls: nan
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
2020-04-17 13:04:36,691 - mmdet - INFO - Epoch [1][50/4553] lr: 0.00428, eta: 50 days, 21:20:20, time: 3.219, data_time: 2.560, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
2020-04-17 13:05:04,312 - mmdet - INFO - Epoch [1][100/4553] lr: 0.00864, eta: 29 days, 19:26:31, time: 0.552, data_time: 0.010, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
@thuyngch

How many batchsize and GPUs did you set?
If batchsize_GPUs=16n, set lr_start=ne-2. For example, if batchsize_GPUs=32=16*2, then set lr_start=2e-2.

4 gpus ，imgs_per_gpu = 16

Hi @betterhalfwzm ,
I fixed the issue related to 'segmentation' field for you. Please check it here: 3f734f5
谢谢！！我现在数据只有一类，num_classes=2, 但是我的类别是1，我需要转为0吗？
@thuyngch

Please use English!

Thank you!! Now I have only one class of data, num_classes = 2, but my class is id is 1. Do I need to change to 0?

No need to change ;)

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

It works, but loss_cls: nan
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
2020-04-17 13:04:36,691 - mmdet - INFO - Epoch [1][50/4553] lr: 0.00428, eta: 50 days, 21:20:20, time: 3.219, data_time: 2.560, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
2020-04-17 13:05:04,312 - mmdet - INFO - Epoch [1][100/4553] lr: 0.00864, eta: 29 days, 19:26:31, time: 0.552, data_time: 0.010, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
@thuyngch

How many batchsize and GPUs did you set?
If batchsize_GPUs=16n, set lr_start=ne-2. For example, if batchsize_GPUs=32=16*2, then set lr_start=2e-2.

4 gpus ，imgs_per_gpu = 16

So set lr_start=4e-2

Hi @betterhalfwzm ,
Maybe your custom dataset does not have the field 'segmentation' as in COCO json format.
You can simply remove this field from code.

It works, but loss_cls: nan
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
/home/wangzhaoming/anaconda3/envs/torch1.4/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py:101: UserWarning: torch.distributed.reduce_op is deprecated, please use torch.distributed.ReduceOp instead
warnings.warn("torch.distributed.reduce_op is deprecated, please use "
2020-04-17 13:04:36,691 - mmdet - INFO - Epoch [1][50/4553] lr: 0.00428, eta: 50 days, 21:20:20, time: 3.219, data_time: 2.560, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
2020-04-17 13:05:04,312 - mmdet - INFO - Epoch [1][100/4553] lr: 0.00864, eta: 29 days, 19:26:31, time: 0.552, data_time: 0.010, memory: 8367, loss_cls: nan, loss_bbox: nan, loss_centerness: nan, loss: nan
@thuyngch

How many batchsize and GPUs did you set?
If batchsize_GPUs=16n, set lr_start=ne-2. For example, if batchsize_GPUs=32=16*2, then set lr_start=2e-2.

4 gpus ，imgs_per_gpu = 16

So set lr_start=4e-2

lr_end = ?

You can set lr_end = lr_start/100 as using in the official mmdetection.
However, it could be 1e-4.
Try to test it!

You can set lr_end = lr_start/100 as using in the official mmdetection.
However, it could be 1e-4.
Try to test it!

Thanks again. I've trained successfully. Expected results！


exports is not defined

I created a project with create-react-app, when I build the project and visit it in browser, it throw the error:
Uncaught ReferenceError: exports is not defined
    at 668 (index.js:288:1)
    at n (bootstrap:19:32)
    at 3953 (types.ts:160:13)
    at n (bootstrap:19:32)
    at startup:4:27
    at startup:4:47

This is not the place to ask for CRA support. You may want to try the latest release though: https://github.com/sindresorhus/query-string/releases/tag/v8.0.0


docs: add template documents for connectors

Signed-off-by: Michaelg22<EMAIL_ADDRESS>

pls move the file to vanus-connect/templates


Fix bugs in visualize segmentation

What is the purpose of this PR?
We've received a lot of bugs that have resulted from merging #193 in. This PR will address these.
How did you implement your changes
It seems that the bugs aren't completely related to visualize_segmentation but rather on what's being passed in. data_xr instead of data_xr_summed, for example, was incorrectly passed in, which likely caused @awedwards issue.
There chan_list verification logic was a little buggy, so that was improved as well.

So this method works to fix visualize_segmentation in the notebook for now (which should address at least @awedwards issue), but the conflicts between channels and compartments is definitely a minor headache. Would be helpful to discuss alternatives here.

Looks good. Can you update the description with which of the issues this is the closing?
The thing I don't like about visualize_segmentation is that there are two separate things people will want to do.

Cycle through different example images in the notebook, seeing how the overlays look
Save the segmentation output for all FOVs in their dataset.

Right now, they're combined together into the same function. This is partly what lead to the show=true errors. What do you think the best way to split these up would be?

@ngreenwald I think the best way to do this is to have plot_overlay only have imshow calls and visualize_segmentation only have imsave calls. Right now, it's also possible for plot_overlay to save an image and I agree, it's really confusing. We can accomplish this by simply returning the result of what would normally be saved by plot_overlay, then allow visualize_segmentation to save it for us.
This is great for making the functionality clearer, but in terms of the issues people have been bringing up, I don't think it's actually show=True that's causing the problem anymore, but rather other errors (ex. data_xr and not data_xr_summed, quite possibly user-specific data issues and filesystem errors, etc.) that just happen to be seen by this particular call to plot_overlay first. I'm doing more investigating into these to see what the root problems are, but I do believe something is getting propagated into the call to plot_overlay with show=True (especially for #354).

The get_topmost_subplot error happens when you create two different plots in the same plotting window. This is happening because of the for loop, where each FOV is getting imshowed. The error goes away if only a single FOV is generated.
Okay great, I agree. I think as a further simplification, we don't even need visualize_segmentation to call plot_overlay for generating the RGB overlay. We can just have a separate cell that directly calls plot_overlay to generate the overlay people care about (the channel overlay).
Once they're done with that, then we can have them call visualize_segmentation (which we'll call something else). That function will loop through, and save the output files for all of the fovs in the xarray

@ngreenwald a few questions:

The call to plot_overlay will display only the result for a single fov at a time, as opposed to multiple at a time?
With this new method, looks like we're only going to be saving _segmentation_borders.tiff and _segmentation_labels.tiff files?
Have we already tried using plt.imshow instead of io.imshow? I think plt.imshow could allow us to display multiple channel overlays at once, although that would be a problem if the user passed in 100 fovs.

I think we should move all of the stuff that's in plot_overlay which doesn't involve plotting a multi-color overlay and move it to the visualize_segmentation function, which we can rename to save_segmentation_output or something like that.
Then there will be one cell where the user calls plot_overlay directly, with a single FOV specified, to look at the output. And a separate cell which loops over all of the data and produces the borders.

Even better, we should modifiy plot_overlay to be create_overlay. Then it can return the image to displayed in the notebook. save_segmentation_output can then also call create_overlay, but without displaying the image each time.

@ngreenwald OK, the new channel structure has been added, ready for review.


FullName not working as it did in 2.19.3, seems to have broken in 2.20.0

My urfave/cli version is
2.19.3
Checklist

[yes] Are you running the latest v2 release? The list of releases is here.
[yes] Did you check the manual for your release? The v2 manual is here
[yes] Did you perform a search about this problem? Here's the GitHub guide about searching.

Dependency Management

My project is using go modules.
My project is using vendoring.
My project is automatically downloading the latest version.

Describe the bug
In urfave 2.19.3 when calling FullName() it is constructed from c.commandNamePath.
In 2.19.3 there is code in startApp() that does not seem to be in 2.20.0-2.21.1
To reproduce
c.Command.FullName() (where c is *cli.Context of a subcommand)
Observed behavior
Only the "subcommand" name is printed
Expected behavior
FullName() would emit "command subcommand"
Additional context
Add any other context about the problem here.
Want to fix this yourself?
We'd love to have more contributors on this project! If the fix for
this bug is easily explained and very small, feel free to create a
pull request for it.

Fullname() is returning "subcommand", not "command subcommand".

Greg

@devnut Yes the FullName was actually buggy as you can see from your example. It wasnt consistent so FullName returns the name of the command and HelpName the display name in help.


Feature/idle overlay

Exposes a configuration option to enable a overlay utilising the logout door sprite, with a timer representing MM:SS until logout due to idle.
Also has an option for seconds until logout representing the time at which the plugin should render the overlay.
Revisions to name and description of the configuration options highly appreciated, because I can't word things to save my life apparently.

Will rectify travis CI failure condition's this evening

travis style issues rectified

Going to close this for now and re-evaluate.
The idle plugin has problems out of the scope of this PR, namely that the edge cases mentioned above are present in the base plugin, not just my infobox implementation.
Will take a look at the plugin itself, rather than simply adding to the problem.


Some unicode characters are not available

When trying to access the webui of Transmission 4.0.2, some of the unicode characters are not working. Specifically the Black Up-Pointing Triangle (▲)
Is there a font pack I need to download or anything I can do on  my end to fix this?

Thanks, this will be fixed in next version.

Forgot to update this issue: this is fixed now.


[BUG] Creating library from template when template includes a doc set content type

Reporting an Issue or Missing Feature
Doc sets created in a library provisioned from a template try to use docsethomepage.aspx and are not usable.
Expected behavior
Create a library from a template including a doc set content type and doc sets should work correctly.
Actual behavior
Using $list.SaveAsTemplate().the template is correctly saved in the site.
A library created from it using the UI works as expected.
A library created using $web.Lists.Add() creates the library successfully, but any document sets created from the contained content type try to open using docsethomepage.aspx and are not usable.
Steps to reproduce behavior
Please include complete script or code samples in-line or linked from gists
$listCreation = New-Object Microsoft.SharePoint.Client.ListCreationInformation
$listCreation.Title = 'Title'
$listCreation.Template = 'SavedTemplate'
$web.Lists.Add($listCreation)
Invoke-PnPQuery
What is the version of the Cmdlet module you are running?
1.12.0
Which operating system/environment are you running PnP PowerShell on?

[X] Windows

This is a CSOM SDK issue. We can't do anything here.
Please reach out to Microsoft support for this.


Difficult to tap attackers / Can't attack when over empty chip block

As the title says, if attackers are going through an empty block, they can't be tapped. This makes scenarios where they are on the wire for a short length before going through a block, then more short wire before reaching another block, and so on (e.g. level 21) difficult to tap them in time.

Hmmm ... you're right, if you tap on a slot, then the buy-chip-menu has priority. Which means that attackers effectively can't be inverted when on a slot.
However, I don't know how to mend this. In fact, this used to be the other way round in early versions, and that was even more annoying: It was impossible to buy chips when an attacker was on the slot.

Sure that you mean level 21 and not 22?
Because, in level 22 the lanes are quite short indeed. Perhaps they should be made somewhat longer.

Sure that you mean level 21 and not 22?
Because, in level 22 the lanes are quite short indeed. Perhaps they should be made somewhat longer.

You're right, level 22. I've found it's hard to know what level I'm on and I've made that mistake on a few issues then caught it and changed it. It would be really helpful to have it show the level number while playing.

Hmmm ... you're right, if you tap on a slot, then the buy-chip-menu has priority. Which means that attackers effectively can't be inverted when on a slot.
However, I don't know how to mend this. In fact, this used to be the other way round in early versions, and that was even more annoying: It was impossible to buy chips when an attacker was on the slot.

Yeah, I think I saw that, and I agree, I'm not sure of a good solution. It could use logic like if the box contains a number that would be made smaller by inverting it, invert it, and if it doesn't contain a number or it contains one that would be made larger by inverting it, select the box. But then that could be used to "cheat" and tap numbers inside boxes to auto-invert them only when they'd be decreased. Another option would be to increase the zone around boxes on the sides outside the traces, so tapping above/below/beside them would activate them, and tapping inside them would activate them only when empty, otherwise it would flip any attackers inside them. Still another possibility would be that if they contain an attacker, it would show a popup giving a choice to flip the attacker on the opposite side from the build/upgrade options. That would require another click, but that probably wouldn't be too bad.

increase the zone around boxes on the sides outside the traces

I'm not sure this is ideal, since some levels can get pretty cramped already, but why not, with some testing this could do.

Still another possibility would be that if they contain an attacker, it would show a popup […]. That would require another click, but that probably wouldn't be too bad.

That could work if attackers weren't constantly moving :/ in earlier levels, or with heavy slowed down attackers thanks to recruits, this is a good idea… but in the rush of later levels this is just impossible to handle. Not mentioning the horrendous mistakes you could make, selling your SHR6 chip when you wanted to invert an attacker !
My suggestion would be to have longer wires in-selected areas between chip slots, so that you get more time to tap on attackers between them (whether they are empty or full). As a rule of thumb, a level should then probably contain at least one long wire to confortably tap on the attackers (like level 8 does, since it was modified to allow this).
Then it could even be used as a difficulty slider while procedurally making endless mode levels 😉 #95

I'd like to close this. I think that the rule "buy/sell menu has priority over tapping attackers" is ok most of the time. Any logic behind (trying to be smart) will confuse the player, and there will always be situations in which the decision is felt to be wrong.
Level 22 has a longer entry line now. And what's more, because it comes quite late in the game now, it should be feasible with all the heroes you've got by then. I've just managed to finish it on turbo without loss of life using a combination of ACC, SUB 6, and CLK 4. Plus the Cernettes at level 8, but I have not even upgraded Dorothy Vaughan, so there is still some margin.


MVP / Minimal viable product

Minimal viable product (MVP):

[ ] Small device with a gyro sensor

ESP8266 / WeMOS D1 + GY-521
MicroSD Card adapter to store data locally in first version
The devices should be cheap to build

[ ] Data analysis platform integration; choice need to be made

ElasticSearch + LogStash + Kibana?

Good additions

[ ] Custom WiFiManager implementation (UI) to make it easy to connect a device to a network and setup the device config; ex: remote server address, device ID, etc.

We can embed a React app 🤓

[ ] Other sensor type; ex: temp, heart rate

[ ] Muscle signal sensor EMG Sensor
[ ] Heart rate sensor
[ ] Heart Rate Oximetry Sensor
[ ] EMG || ECG || EOG Sensor
[ ] Resistive Thin Film Pressure Sensor

[ ] Expose a ReactJS App: ESP32 Arduino: Serving a React.js app

Jamboards

Feb 16th 2022 - System draft

@f3rland y'a t'il une méthode pour tester le code à distance? Ou peut être un émulateur ?

Note to self: https://randomnerdtutorials.com/esp32-esp8266-firebase-gauges-charts/

https://www.hackster.io/Hack-star-Arduino/esp32-online-simulator-you-were-looking-for-2022-327d42


Fix a crash on reading MusicXML file with octave shift starting with a rest

Fixes #266.
This fix allows octave shifts start (and end) with ImoNoteRest rather than only ImoNote, and modifies the relevant places in the code (MusicXML reading and octave shifts handling in StaffObjsCursor) correspondingly. After these changes octave shifts starting with rests seem to be loaded and rendered correctly. However, I am not sure whether allowing octave shifts start only with notes was intentional, and whether I could miss something else with my changes.

Thanks for the PR Dmitri,

I am not sure whether allowing octave shifts start only with notes was intentional, and whether I could miss something else with my changes.

I didn't included rests because I though than an octave change will always start and end in a note, not in a rest. And all MusicXML samples I had with octave changes seemed to confirm that guess. Your PR looks OK to me.
I'm going now to build and check your PR to confirm that there are no issues in regression tests, before merging.

All regression tests passed! Thank you.


Use duet for asynchrony, in particular for Collector

We are planning to switch to duet for supporting async code, with one of the reasons being that it is "reentrant" and supports nested invocations of the event loop (duet.run). We don't yet have much async code in cirq, but do use it for Collector and PauliSumCollector and this can lead to problems if running in a context where the asyncio event loop is already running (#4004). The solution proposed there is to patch asyncio to make it reentrant, using the nest-asyncio library, but this seems like something we should not rely on. Instead, I propose that we make the switch to duet.
The main question I have is whether we need to continue to support Collector.collect_async and Sampler.*_async methods being asyncio coroutines, or whether we can just switch them to be duet coroutines. In this PR as it stands I've just switched things, but if we want to support asyncio we would need to name the duet coroutine functions differently and then wrap them into an asyncio-compatible form for the existing methods.

Please fix the failing tests, then re-request review.

Hm, we don't have a version of duet for 3.6, and it'd be quite hard to do because of some asnyc/await language features that landed in 3.7. @balopat, what is the plan for dropping python 3.6 support?

This is blocked on #3574, which is blocked on google3 having their default python runtime upgraded to 3.7.
If we want to move forward with duet for the OSS community (I think that this is great!) then we have to think about workarounds until that upgrade is done which might take a year still. A workaround design might mean to offer the current async flow on 3.6 and duet on 3.7 somehow.

This is blocked on #3574, which is blocked on google3 having their default python runtime upgraded to 3.7.
If we want to move forward with duet for the OSS community (I think that this is great!) then we have to think about workarounds until that upgrade is done which might take a year still. A workaround design might mean to offer the current async flow on 3.6 and duet on 3.7 somehow.

One idea:
We could add a cirq-duet module that would use duet for the collector.
People who have python 3.7+ could use that module, the ones using 3.6 can still use the old collector, on which we could add even a deprecation warning...
WDYT, @maffoo?

Do we have an update on timeline for Python 3.7+?

Actually, @fedimser is working on adding python 3.6 support to duet (https://github.com/google/duet/pull/23) which will let us move forward with these async changes without waiting for 3.7 support in colab etc.

This is ready for review. @balopat, @mpharrigan


Waymo Open Dataset Performance on ICCV Paper

Dear authors,
Many thanks for open-sourcing this great work @Abyssaledge @abbyxxn .
I have noted that there is a performance gap between the reported results on ICCV Paper and this repo's README, which are 72.85 and 70.90 L1 validation set mAP, respectively.
May I check with you what caused the difference between the paper's reported result and this repo's result?
Moreover, could you please advise how I can reproduce the paper-reported results, by modifying this repository?
Thank you!
Regards,
Charles

@Abyssaledge Alright, noted with thanks! :smiley_cat:


python3.8 support

Hi, could you please add support for python 3.8? That would be much helpful for me!
Thanks!

Sorry, this library depends on PySide6, but PySide6 does not support Python 3.8.
Thanks for Issue.


missing v0.1.13 git tag and github releases

can you create github releases and git tag for v0.1.13?
thanks!

would also appreciate this!

I know its a little too late, but 0.1.13 is now properly tagged, as well as all new releases.


Scripts Inherited By Scenes All Share Variables

Hi its me again, sorry to bother! I noticed today that when a script is tied to a scene, when you instance that scene all of those instanced scenes share variables. So for example if I change a variable for one of the scripts it changes for every scene that has that script.
Is this because I am declaring my variables this way?
Is there a different way I should be declaring variables?
**EDIT
Upon doing more reading I realize that this is an issue with godot. I need to make each scene unique from one another. Still researching how to do that.
Thank you and have a wonderful day!
local Test = {}

Test.extends = "KinematicBody2D"

Test.Variable = 60

Turns out it was an issue of using self or not haha. Same as last time I posted an issue. I guess I need to be using self more often.

Hi there! Sorry about the delay.
Yeah, anytime you need to access properties or call methods in instances, use self. If you want to really change the script table itself, only then you use your Test variable to access stuff.
I still have to document this better and create examples of projects/scripts in Lua =/
I'm glad you figured it out ^^
Don't hesitate in asking if you have any more trouble, we'll be happy to help!


Pure-swift hang() and tests.

Hi! I've been writing some commandline tools in Swift and wanted to leverage promises; I need to ensure the command doesn't terminate before the promises resolve. After reading #423, here's a ~port of PMKHang to Swift.
Tested using Xcode 8.3.3 / Swift 3.1 on macOS and Swift 4.0.2 on Ubuntu 14.04.

PMK5 (recently released has wait())
https://github.com/mxcl/PromiseKit/blob/master/Sources/Promise.swift#L73-L98
Now, it operates differently, and I don't know which approach is better.
Also, do you use PMK5 and not 4? I haven't pushed 5 to CocoaPods yet is why I ask. I planned to write a blog post and document it better first.
Anyway, this is much appreciated. What do you think? Do you want this in 4, maybe also in 5? Do you think the approach you have taken with runloops is the more sensible choice?
I'm not sure really, I added wait() to PMK5 as an experiment, so I don't have firm feelings on this matter.

PMK5 (recently released has wait())

Whoops, I forgot to mention that I tried wait() first and it deadlocked the main thread - for example:
func testWait() {
    let ex = expectation(description: "block executed")
    after(seconds: 0.02).done {
        ex.fulfill()
    }.wait()
    waitForExpectations(timeout: 0)
}

This makes sense; it's possible to remove the deadlock by using done(on:, ...) but if any part of your chain hits the main queue you're sunk...and the GCD API doesn't seem to offer a clean solution to this. (dispatch_get_current_queue is missed.)

Also, do you use PMK5 and not 4?

Yes, I'm using PMK5 - figured it'd be best to start acclimating early. :)

Do you want this in 4, maybe also in 5?

Haven't tested this revision against 4 - I had something similar that worked in 4 but was less elegant. Would be happy to have it in 5 and if you think it'd be valuable to backport to 4 I can revisit my original attempt.

Do you think the approach you have taken with runloops is the more sensible choice?

I'm just copying your approach from PMKHang.m - I prefer your approach in wait(). wait() is conceptually cleaner with less overhead; hang() just happens to be useful when invoked from a non-runloop application.

K cool, then I think we need to offer both, so I'll figure out how to merge this.

Tests fail for iOS, tvOS and watchOS.

Will merge, when CI passes.

Sorry, I've been out sick.
Investigating iOS et al - practically, there's no reason to call hang() over wait() as all iOS/tvOS/watchOS apps will have runloops.

Ok, the issue is that Apple-y platforms (that is, not os(Linux)) use a different CFRunLoop setup. After this tweak the ATV and iOS tests pass locally - the watchOS tests won't build with/without my changes for some reason on Xcode 9.2.
Long term this may introduce other issues - other platforms being added to Swift (e.g. Haiku, Fuchsia) will have to be added to this conditional.


Spelling

I think sufix should be suffix.
As long as I was going to report it, I also checked in general with something like the following:
find . -name '*.dart' -print -exec ispell {} \;

I fumbled the keys halfway through the process, and finished it off in Emacs with something like the following:
M-x find-name-dired RET *.dart RET t Q sufix RET suffix RET Y

Similarly for *.md and ' +$' -> ''.
I also checked dart format . --fix and dart test.
Let me know if you prefer prefixes, lowercase or one-liners in the commit title and body. And if you want an extra commit for the dart format change, or the two current commits squashed as one.
Oh, it's an API change, maybe I shouldn't have edited CHANGELOG.md. I reverted the editing of --exclude-sufix in CHANGELOG.md and force pushed.

Hi @bradyt , thank you for the contribution!

I think sufix should be suffix.

Yes, you are correct!

Similarly for *.md and ' +$' -> ''.

I can also add a finder using regular expressions.
e.g. dlcov -c 80 -e **/*.g.dart
It's open to discussion, if you need this feature, let me know.

I also checked dart format . --fix and dart test.

If you are using linux or mac os, there is a makefile script, so you can run make analyze in the project root folder.

Let me know if you prefer prefixes, lowercase or one-liners in the commit title and body.

I follow this convention Conventional Commits.
In addition to this, I will add a PR template, too.

Oh, it's an API change, maybe I shouldn't have edited CHANGELOG.md. I reverted the editing of --exclude-sufix in CHANGELOG.md and force pushed.

Don't worry about that, you can go this way, since we do not have [UNRELEASED] section in this changelog, so I will add this information before generate a new release.

Similarly for *.md and ' +$' -> ''.

I can also add a finder using regular expressions.
e.g. dlcov -c 80 -e **/*.g.dart
It's open to discussion, if you need this feature, let me know.

Oh, I was speaking in code, apologies. What I meant was that I removed trailing whitespace from markdown files in the second commit, with regex that matches ' +$', in other words, remove any positive number of spaces before end-of-line. Just a style change to your README.md.

Let me know if you prefer prefixes, lowercase or one-liners in the commit title and body.

I follow this convention Conventional Commits.

Sounds like the commit messages should be written as follows:
style: remove trailing whitespace
fix!: change spelling of sufix to suffix, repostory to repository

I've rewritten the commit messages and force pushed.

I was using double space before end-of-line, in order to break lines hahaha
I just tested this PR, let's merge it!


replayed notebooks should not default to being editable

Kui does support a client specifying that notebooks should always be readonly. But in the stock Kui client (which does not specify this bit, because we want to allow users to author notebooks), when a user replays someone else's notebook... that user gets all of the decorations (e.g. split header, block delete button) as if the notebook is in authoring mode

closing in favor of #8043


Incorrect pulling of migrations from remote

It seems like the CLI is only pulling database table schemas, not other migrations like computed fields, triggers, etc., during nhost init --remote.
Documentation from Hasura:

https://hasura.io/docs/latest/migrations-metadata-seeds/migrations-metadata-setup/#step-2-set-up-a-project-directory
https://hasura.io/docs/latest/hasura-cli/commands/hasura_migrate_create/#examples

When pulling SQL migrations from the server, we want to pull all migrations except the auth and storage schemas since Hasura Auth and Hasura Storage handle these two schemas.

Thanks for looking into this issue. Any idea on when it could be fixed? Thanks again.

This one has been fixed in recent releases. @seyaobey-dev could you confirm?


fix(ci): update black to rev 22.3.0

Description

Updates black to rev 22.3.0.
Should fix
ImportError: cannot import name '_unicodefun' from 'click' (/home/runner/.cache/pre-commit/repoykquj97p/py_env-python3.10/lib/python3.10/site-packages/click/__init__.py)

See https://github.com/psf/black/issues/2964#issuecomment-1080974737
Related Issue

N/A
Checklist

[X] I've formatted the PR title in accordance with the structure of the conventional commit messages.
[X] I've updated the code style using pre-commit if needed.
[X] I've updated or supplemented the documentation as needed.
[X] I've read the CODE_OF_CONDUCT.md document.
[X] I've read the CONTRIBUTING.md guide.

black works now, but there is still a problem with trinity


⚠ Spryr has degraded performance

In de34e7a, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5043 ms

Resolved: Spryr performance has improved in 1272ff1 after 10 minutes.


feat: add etherscan table

Motivation
Closes #2609
Add standalone etherscan_configs table
That supports different chain_alias = {key = ""} entries.
the --etherscan-api-key is now used to lookup chain_alias, if no alias found it's treated as API key, like before

Solution

renamed to [etherscan]


[speedtest] migrate to common

Signed-off-by: Nicholas Wilde<EMAIL_ADDRESS>Description of the change
Benefits
Possible drawbacks
Applicable issues
Additional information
Checklist

[X] Chart version bumped in Chart.yaml according to semver.
[X] Title of the PR starts with chart name (e.g. [home-assistant])
[X] Variables are documented in the README.md (this can be done with using our helm-docs wrapper ./hack/gen-helm-docs.sh stable <chart>)

@billimek are you still using this chart? Do you think it still has a purpose over speedtest-exporter?

@billimek are you still using this chart? Do you think it still has a purpose over speedtest-exporter? We can deprecate it and point people over to the other chart.

I don't personally run this one anymore.  The only purpose I see over speedtest-exporter would be if someone chooses to use influxdb vs prometheus.

I'd suggest we deprecate it or rename it to speedtest-influxdb

I'd be fine with deprecating as well. We can leave the deprecated version in the repo for some time, and if nobody wants it back we can remove it altogether

@nicholaswilde I'll go ahead and create a PR to deprecate this.

Thanks. I am interested in how you guys do depreciation.

This is pretty much it
https://github.com/k8s-at-home/charts/pull/826


Runaway terminalProcess processes

These processes linger around long after I shutdown Code:

We usually detect whether the parent process is dead to suicide a process.

Good find, thanks for the link :)

@joaomoreno please verify when you get a chance