Make ServiceStructure tail recursive

This fixes https://github.com/Microsoft/visualfsharp/issues/2980

Excellent!

@vasily-kirichenko Do you think it's feasible to add the large test case as a test?  Capturing a few large tests is invaluable to prevent regressions.

@vasily-kirichenko Fantastic work by the way - thank you so much for getting on top of this as soon as you did

@dsyme conflicts fixed. Interesting thing about that TestCrash.zip solution is that Dictionary.fs being a small file, is colored in about 20 seconds. We should investigate what's so unique in it.

It's 5K lines file, so 20 seconds is OK to colorize it
It has an error on each line, so suggestions take a lot of time:

/cc @forki

It has an error on each line, so suggestions take a lot of time

Is that a realistic scenario?

@vasily-kirichenko Yeah, a very long sequence of typically "linear" declarations.  We often need to be careful with these and should stress test more often with such files.  Examples of "linear" decalarations are

expressions which are long lists of "let x  = " followed by a result expression
long lists of sequential execution "a; b; c; d..." (whether single line or multiple lines)
long combinations of these

There are other cases of things that could be considered linear but where we don't in practice hit code that uses it, e.g.

long sequences of matches "match a1 with pat1 -> b1 | _ -> match a2 with pat2 -> b2 | Some _ -> ...." where the last case of the match is another match etc.
also if a1 then b1 elif a2 then b2 elif ...
also combinations of these

Often processing linear expressions drops out as tail-calls when passing an accumulator, e.g. see the case for Expr.Sequential in accFreeInExpr.  But sometimes extra work needs to be postponed, e.g. see accFreeInExprLinear or TcLinearExprs where we push the extra work into a continuation function.
Most of these cases in the codebase are processing the TAST, but in some cases such as TcLinearExprs the processing is over the AST.  I know most of your processing is over the AST too, so all the cases above may also be cases where you're vulnerable to stack overflows


Update to Feature Graduation Template to include docs updates for new features.

Checklist to be completed when graduating an experimental feature

[ ] Updates for Semantic Kernel Docs updates are complete and checked in here: https://github.com/MicrosoftDocs/semantic-kernel-pr
- [ ] Existing Documentation needs to be updated by creating a PR here:  https://github.com/MicrosoftDocs/semantic-kernel-pr
- [ ] New Features need a page created in the Semantic Kernel Learn Docs here: https://github.com/MicrosoftDocs/semantic-kernel-pr
[ ] Notify PM's and EM's that feature is ready for graduation
[ ] Contact PM for list of sample use cases
[ ] Verify there are sample implementations for each of the use cases
[ ] Verify telemetry and logging are complete
[ ] Verify API docs are complete and arrange to have them published
[ ] Make appropriate updates to Learn docs
[ ] Make appropriate updates to Concept samples
[ ] Make appropriate updates to Blog posts
[ ] Verify there are no serious open Issues
[ ] Update table in EXPERIMENTS.md
[ ] Remove SKEXP flag from the experimental code

First checklist item updated below to include docs update

Closing as stale


[Feature]: Geocoded location for person card

Is your feature request related to a problem?
Currently the person card only shows Home or Away.
When someone is away it does not give a quick view of where someone is.
Describe the solution you'd like
It would be great if we could select an entity that contains a geocoded location so we can show the actual address where someone is, if not at home.
In the Android app sensor settings you can enable this sensor so that an entity is automatically available in Home Assistant, for example: sensor.pixel_6_pro_geocoded_location.
Describe alternatives you've considered
No response
Additional context
No response

It does show where a person is when it's added as a zone in HA (not just Home and Away), so you can add zones like: Office, Shop etc.
iPhones also have the field *_pro_geocoded_location, so i could add an optional entity filter for that.
The problem is that a full size address doesn't really fit into the minimal design of the mushroom card.
What do you think @piitaya ?

Yeah, I noticed that later as well. I guess you're right. It's a nice feature that you can see how far someone is when they're traveling home (instead of 'Away'), but you can always tap the tile to see the map of course.
But if you still would like to add the feature then letting the address scroll (like a ticker) would be very cool. I understand that's not that easy so a truncated address would suffice as well. Just seeing 'Grand Central Av...' for example is more helpful than 'Away' I think.

For now, person entity only support zone, not_home and home state so Mushroom must only support this. You can use template if you want your own rules or mix entities.

In case anyone stumbles across this - this is the template card I set up.
One thing I would like to add is to get the badge working as in the person card, but I don't think badge_icon is template aware (also would need to get the current zone's icon somehow)
type: custom:mushroom-template-card
primary: >-
  {% if states('person.phill') == "home" %}

  Home

  {% elif states('person.phill') == "not_home" %}

  {% if state_attr('sensor.phillip_s_iphone_geocoded_location',"Areas Of
  Interest") != "N/A" %}

  {{ state_attr('sensor.phillip_s_iphone_geocoded_location','Areas Of Interest')
  }}

  {% else %}

  {{ state_attr('sensor.phillip_s_iphone_geocoded_location','Sub Locality') }}

  {% endif %}

  {% else %}

  {{ states('person.phill')}}

  {% endif %}
secondary: |

  {{ state_attr('sensor.phillip_s_iphone_geocoded_location','Locality') }}
icon: mdi:home
picture: '{{state_attr(''person.phill'',''entity_picture'')}}'


Prepare release 0.1.1

Summary
Preparing release 0.1.1
Background, Problem or Goal of the patch
N/A.
Design of the fix, or a new feature
N/A.
Related Issue, Pull Request or Code
N/A.
Wanted reviewer
N/A.

BUILD SUCCESS. <jenkins-prb-asakusafw-dag-25>


PERF-#5554: Implement drop_duplicates via new duplicated

Signed-off-by: Igoshev, Iaroslav<EMAIL_ADDRESS>
What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #5554
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

PERF on 8 cores with Ray engine
DataFrame.duplicated()

pandas
Modin master
PR 5587

0.07 s
11.84 s
0.03 s

DataFrame.drop_duplicates()

pandas
Modin master
PR 5587

0.07 s
11.90 s
0.05 s

btw, why it didn't work with high-level .apply()? if the difference is THAT significant maybe we should give a closer look at the .apply() itself and speed up it instead of avoiding its usage

There are many places in the high-level .apply() where we access the metadata, which might be None so we materialize it and get slowdown. Also, when doing apply with a callable we don't know in advance any of the metadata in most cases. With the new implementation of duplicated we know all of the metadata for the resultant frame. That said, of course, we can look into apply in terms of perf improvements as part of a separate issue.

@YarShev Can you show performance with triggering execution? (since you not only make the operation asynchronous, but also change the implementation)

@dchigarev @anmyachev any more questions here?

time for modin: 13.61 s. # master
time for modin: 5.23 s. # PR5587

Just called print("modin_res =", modin_res) right after modin_res = modin_df.drop_duplicates() in the example to trigger execution.

Minding that the first operation in ray is usually slower we could even "improve" the result by adding a warm-up:
NRUNS = 1
execute(md_df.sum()) # warmup
md_res = timeit.timeit(lambda: execute(md_df.drop_duplicates()), number=NRUNS)

So the actual results for this branch are:
without warmup: 4.11s
with warmup: 1.13s.

Complete script
import modin.pandas as pd
import pandas
import numpy as np
import timeit
from asv_bench.benchmarks.utils.common import execute

# def exetuce(df):
#     pass

NROWS = 1_000_000
RAND_LOW = 0
RAND_HIGH = 10_000_000

data = {
    "col0": np.tile(
        np.concatenate([np.random.randint(RAND_LOW, RAND_HIGH, NROWS // 100 - 1), [1]]),
        100,
    ),
    "col1": np.tile(
        np.concatenate([np.random.randint(RAND_LOW, RAND_HIGH, NROWS // 100 - 1), [1]]),
        100,
    ),
}

md_df = pd.DataFrame(data)
pd_df = pandas.DataFrame(data)

# drop_duplicate:
#   input_shape = (1_000_000, 2)
#   output_shape = (10_000, 2)

NRUNS = 1
execute(md_df.sum()) # warmup
md_res = timeit.timeit(lambda: execute(md_df.drop_duplicates()), number=NRUNS)
pd_res = timeit.timeit(lambda: pd_df.drop_duplicates(), number=NRUNS)

print(f"Cumulative time of {NRUNS} runs for modin: {md_res}s.") # ~10s.
print(f"Cumulative time of {NRUNS} runs for pandas: {pd_res}s.") # ~0.3s


Execute Hosting Startups to Lookup by Type Instead of an Attribute

Looking at the source code of the GenericWebHostBuilder I can see it says:
foreach (var attribute in assembly.GetCustomAttributes<HostingStartupAttribute>())
{
    var hostingStartup = (IHostingStartup)Activator.CreateInstance(attribute.HostingStartupType);
    hostingStartup.Configure(_hostingStartupWebHostBuilder);
}

Why can't it simply say?:
foreach (var type in assembly.ExportedTypes.Where(t => t.IsDefined(typeof(IHostingStartup), false))) {
    var hostingStartup = (IHostingStartup)Activator.CreateInstance(type);
    hostingStartup.Configure(_hostingStartupWebHostBuilder);
}

This way it no longer has to have a HostingStartupAttribute and also allows you to have multiple IHostingStartup files in the same assembly.
I considered whether the reason was for performance but after doing a few tests, I did not notice much difference. Although I'm sure you have done more thorough research.

Scanning all types in the assembly can slow down startup time so doing it all the time means everyone has to pay that cost and it's not pay for play.
It's no longer opt-in. If there's an IHostingStartup type defined in the assembly, it's going to be discovered and activated. That means we now need a way to turn it off.

Thanks I assumed that was the case.
When you say it's "no longer opt-in", is this a change in v3?

That was about describing changes that would need to be made to the current model if we went that path (right now it's opt-in you suggestion makes it opt-out).

Thanks for clearing this up, I'll close this and my current issue.


WithMaxBlockingTasks doesn't seem to work as the docs indicate

Describe the bug
WithMaxBlockingTasks doesn't seem to do what I think it should be doing based on the docs.
To Reproduce
I expect this to actually error:
p, _ := ants.NewPool(1, ants.WithMaxBlockingTasks(1))
var wg sync.WaitGroup
for i := 0; i < 5; i++ {
  err := p.Submit(func() {
   wg.Add(1)
   time.Sleep(time.Second)
   wg.Done()
  })
  if err != nil {
     // I expect this to have an error if i >= 2
  }
}
wg.Wait()
p.Release()

Expected behavior
I expect err to return ants.ErrPoolOverload when i >= 2.
Error messages/Trace logs
No error, it processes everything 1 at a time.
System info (please complete the following information):

OS: macOS
Go Version: 1.15.1
ants version: v2.4.2

You should call pool.Submit in different goroutines to get the expected result of WithMaxBlockingTasks, otherwise if you call it serially as you did with a for loop in one goroutine, it just process one by one.

Oh indeed, simply changing my example fixed it:
go func() {
  err := p.Submit(func() {
   wg.Add(1)
   time.Sleep(time.Second)
   wg.Done()
  })
  if err != nil {
     // I expect this to have an error if i >= 2
  }
}()

Thank you.


Migrated to pyproject.toml

Pull Request Test Coverage Report for Build test-60
Details

5 of 8 (62.5%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 24.061%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

zhmc_log_forwarder/version.py
5
8
62.5%

Totals

Change from base Build test-58:
1.5%

Covered Lines:
141

Relevant Lines:
586

💛  - Coveralls


⚠ Chatwoot has degraded performance

In 1218c15, Chatwoot (https://app.chatwoot.com) experienced degraded performance:

HTTP code: 200
Response time: 4979 ms

Resolved: Chatwoot performance has improved in 72b741d.


Results do not load in iOS Chrome or Safari

I'm not sure if this is a known issue or not, but lunr.js is working fine in desktop browsers, but the results do not load in Chrome or Safari in iOS.Is there some way to fix this?

Just some follow up: I tested this on a few devices, and it only seems to be related to alternate keyboards (like Swype), which must be doing something different with their input that's not being picked by the script.
This is obviously a bit of an edge case, but any thoughts on ways around this would be welcome.


Implement tags properly #121

Structure for tags page, visible tags and working link to tags page

Do we dynamically create all the tag pages?

@vpetersson Well, as I understand it, this is the main option, maybe I will try to improve it so that the pages are automatically generated for all existing tags. The theory should be ready-made solutions.


Vagrant DMG installer fails silently on clean 10.10.4

#Vagrant DMG installer fails silently on a brand new MBP running 10.10.4.#
To Reproduce

Download latest OSX DMG from vagrantup.com
Double-click PKG

Expected Behavior

Expects Vagrant to install cleanly
Or Expects Vagrant to fail loudly

Observed Behaviour

Vagrant shows sucessful install screen
vagrant CLI does not run
no /usr/local/bin tree created

Notes
Commit 22a3f46@mitchellh move to /usr/local [GH-54] break this as on a default 10.10.4 install there is no /usr/local/bin only /usr/bin.

Just a heads up, I had this issue today on a new 10.10.5.  Creating /usr/local/bin fixed the issue


Try to add other website

Type
Other
Bug description
I try to add https://unsplash.com/ but i couldn't, I'm kinda noob, i try to fill other queries but it didn't run, any idea if it's possible to add this website, if yes could you tell me how?
And is there a way to setup query for other website?
Steps to reproduce
i try https://api.unsplash.com/ in api section, it didn't work.
Android version
12
App version
1.2.9
Screenshots or videos
No response
Solution
No response
Additional context
No response
Acknowledgements

[x] I have searched the existing issues and this is a new and no duplicate or related to another open issue.
[x] I have written a short but informative title.
[x] I filled out all of the requested information in this issue properly.

Unsplash is not a booru sites, hence it's not possible.
This app only support sites that use booru imageboards such as danbooru or moebooru


MAINT - Replace flake8 with ruff

It was recently brought up that flake8 could be replaced with Ruff and other projects in the ecosystem have started migrating to it.
See original post: https://github.com/nebari-dev/nebari/pull/1545#issuecomment-1326021429

Makes sense. I did see it has a drop in replacement for flake8 settings as well and no need for us to adopt the other format options for now in ruff.


Revamp add reviewer page

NOTE: do not deploy as is on RR, as text for #RecommenderAddReviewersText would be overwritten
(the deployment db update script would overwrite the text that has been tuned on RR)


Is it possible to align the Resource shortnames with MS abbreviations

Can the shortnames be aligned to the MS recommended shortnames/abbreviations where possible:
https://docs.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/resource-abbreviations
I understand this can be done manually by editing the resources.csv, but I would be nice to have everything aligned and consistent where possible.

@jag-sandhu I assume your issue is with the Azure Naming Tool since you are referencing the resources.csv file.  We recently updated the resource short names to align with Enterprise Scale Landing.  The short names on the link you shared will be updated to align with the Azure Naming Tool here soon.  We are waiting on the CAF team to make our changes public.

Here's the PR for your reference: https://github.com/MicrosoftDocs/cloud-adoption-framework/pull/615

@jamasten Thats great!
Thanks for clarifying and pointing me to the PR for visibility.


Build NuGet packages from source

Hi,
I have built Newtonsoft.Json.12.0.3.nupkg using your build.cmd script.
However I do not manage to reference this package in my .Net Core project.
In my startup.cs code, the JsonSerializerSettings is not recognized.
"Error CS0012 The type 'JsonSerializer' is defined in an assembly that is not referenced. You must add a reference to assembly 'Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed'."
Seems like its a known issue but it is not yet figured out: https://stackoverflow.com/questions/38484633/cant-compile-project-when-replacing-a-reference-to-json-net-with-its-source-cod
Please tell me how can it be solved ?

The reason you get that error message is because one of your dependencies must have been built against the NuGet provided Json.NET which is signed but since the signing key is not public you cannot use a source replacement unless you rebuild all your dependencies to be built against it as well.

If I get things right, I guess you mean for the package Microsoft.AspNetCore.Mvc.NewtonsoftJson which references Newtonsoft.Json, but I cannot build this one against the source since this project does not allow this replacement.
How can I replace the Newtonsoft.Json package with its source code with no compilation errors ? I have done this replacement with all other NuGet packages such as Microsoft.EntityFrameworkCore, Microsoft.Extensions, Microsoft.IdentityModel.Tokens and everything compiles fine.

What are you trying to change that you are doing this? Json.NET has many extensibility points and for the most part I have always been able to modify its execution through these.


getting 404 on cds/aspects-inheritance

getting 404 on cds/aspects-inheritance
https://github.com/cap-js/docs/blob/692f45ba7867af90c0d72185f7ffa301e4197140/cds/aspects-inheritance
is this repo not live version of https://cap.cloud.sap/docs/cds ? Is there any central resource to pull in latest docs?

Hi @Muhtasham ,
what you can see on cap.cloud.sap is a mix of two repositories, as we do have this open source repository here but an internal repository as well. So in your case the topic is in the internal repository. That's why you can't find it here or edit it.
Do you have any feedback for that specific topic you want to share?

Do you have any feedback for that specific topic you want to share?

Yes would be nice to have PDF version of whole documentation too :)

Providing a PDF is not planned at this point in time. Anyway thanks for opening the issue and engaging with us :)


Cost Report - Tag Gitea shared service

PR for issue
What is being addressed
Tag Gitea shared service with required tags for cost reporting.
How is this addressed

Add tre_shared_service_id and tre_id tags on Gitea TF scritpts

/test

/test

/test


Korean texts do not render correctly in custom fonts

Version

Phaser Version: 3.60

Operating system: Windows Firefox, Android 13 WebView, etc.

Description

There are many methods (using invisible DOM objects on the HTML file, attempting to load fonts via various scripts like Google's webfontloader, using css and @font-face, etc.) to load custom/local fonts.
No matter what, Korean texts do not render correctly in specified font (we tried .ttf). It would look like either one or two letter is applied while other letter looks like it's falling back to the default font in the same text object. This usually happens when you load the page for the first time (or Ctrl+F5 on Firefox for clean refresh), or all the time on WebView via Android simulators and real phones. On WebView specifically, since they do not cache fonts, they do not load the font at all and falls back to default font every boot.
Reports we got from our Korean customers vary, but it simply does work well.
We've used loading fonts in the preload() function, using CSS and hiding DOM using those fonts off the screen, or using various font loading libraries to pre-load them, but Korean letters only seem to have issues with it on Phaser.
We don't have any Japanese or Chinese users yet, but they also may be suffering from the same issue. We were only able to verify Korean at the moment.
English fonts RARELY have this issue when we spam Ctrl+F5 refresh as well, but it's much harder to reproduce.
Example Test Code

Additional Information

I'm going to need to see the actual code for this because I've worked on a number of massive Korean language games with a Korean game studio that obviously uses Korean text everywhere without issue in Phaser CE, 3.55, and 3.60, so this issue has to do with the implementation I'm afraid.


Auto-add node import to code samples if we detect a node import?

E.g. /// <reference types="node" /> to the Twoslash code ahead of time if we detect any known import for a node project?
Maybe too magical.

Could we do something for both this and the react example? #20

I'm not sure about this one. Like, how would you properly detect node imports? For example, fs, path could as well be the user's absolute file paths when the user's on any other runtime instead of node, for example, deno.
Otherwise, this could be easily implemented as how it's done with React. Right?

We can use ts. preProcessFile to get a list of external modules - if that includes any of the known node ones ( which we can grab via  require("module").builtinModules - then the same concept can apply where we jam in /// <reference types="node" /> at the top

Think I've changed my mind on this - for the Jest website, I basically want to be able to be sure that "jest" is in types  for the TS compiler - the same thing should happen for node IMO.

Great! So no need for this feature, and a compilerOptions > types > 'node' would suffice?

Yeah, we have defaultCompilerOptions in there already so I think this might just need documenting

I think we recommend types: node somewhere


Fixed compile error when ros-indigo-opencv3 is installed (#307) (Jade)

Porting the changes in MR #307 to Jade.

I fixed other issues that were preventing Travis from compiling the jade-devel branch, so this should be all good now.


test nb engine on RPi0

reported by user: troubles installing NB Engine in RPi 0

works well on a VM with similar specs

it is an architecture issue.
the base images we use for the NB image (namely the official python:3-alpine), do not work on the armv6 arch.
it is fixable, but it would imply either changing all the existing NB engine images to build python from source, or add a separate build of each component, just for arm32v6


Update Linux installer to 10.1

First, the Linux installer is kind of broken, we need to add a new
option in order for all of the libraries to be installed. This is
discussed on the nvidia forums here.
Second, the URL for the installer has changed. The file now has a 'run'
extension.

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I have no idea why the linux version is failing to install. It works on my computer.
Maybe python should be added to the run requirements? Because the post-install script requires python.

@andersy005 @xmnlab this can be closed.

Superseded by #16

thanks for working on that @carterbox !
We just merged another PR that upgrades to 10.1
thanks very much for your help!


fix: make sure we resolve cluster only once

Explanation
Make sure we resolve cluster only once.

/cherry-pick release-0.2


<EMAIL_ADDRESS>Module not found: Can't resolve 'async_hooks'

Bug description

Hello,  I am new to Prisma. I have been trying to access Prisma database with nextjs. And every time I try to use PrismaClient from @prisma/client in a nextjs page, I have been seeing this error.
./node_modules/@prisma/client/runtime/index.js:26236:39
Module not found: Can't resolve 'async_hooks'
null

How to reproduce
Steps to reproduce the behavior:
My Schema file
datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model User {
  id      Int      @default(autoincrement()) @id
  email   String   @unique
  name    String
  password String
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
}

I created a migration and ran with the following commands
npx prisma migrate save --experimental
npx prisma migrate up --experimental

Then I also run npx prisma generate
This is my register.tsx file where I am accessing the prisma client
import { useState } from "react";
import { useRouter } from "next/router";
import { PrismaClient } from "@prisma/client";

const prisma = new PrismaClient();

interface RegisterProps {}

const Register: React.FC<RegisterProps> = ({}) => {
  const router = useRouter();
  const [email, setEmail] = useState<string>("");
  const [password, setPassword] = useState<string>("");
  const [name, setName] = useState<string>("");

  const submitHandler = async (e) => {
    e.preventDefault();
    try {
      const user = await prisma.user.create({
        data: {
          email,
          password,
          name,
        },
      });
      console.log(user);
      router.push("/");
    } catch (e) {
      console.log(e);
    }
  };

  return (
     // my template goes here
  );
};

export default Register;

After running this, I am getting this following error
<EMAIL_ADDRESS> Module not found: Can't resolve 'async_hooks'
null

Expected behavior

I expect this code to build my page first. But it fails to do so and shows me the error in the title.
Prisma information

All of these information was included in the steps to reproduce part
Environment & setup

OS: Ubuntu
Database: MySQL
Node.js version: 14.05
Prisma version: 2.11.0

I am closing this issue as I didn't get a reply from anyone and it turned out I can't use PrismaClient inside the front end code. I have to use it in the api or in the fetching methods. Thank you for nothing.

It is quite normal for an answer to take a few days.
But good you found the answer to your question yourself 👍


Mococular

Brainstorm on monocular pipeline:
Questions:

One scaled model per motion?

Expected hurdles:

Have a container with tensorflow (for augmenter) and pytorch (monocular), but I'm sure workable

1st approach:

Frontend: When hitting new session, get option to go single-camera or multi-camera (default) route

This passes flag at session level: monocular

Core:

Pose detection: Another loop waiting for monocular jobs (ie, same as OpenPose and HRNet)
Add big if statement in main.py to support Monocular code / Have main_monocular.py

Core:

I'm not sure we want core and monocular to be in the same repo. The dependencies in mono are already a mess. We could definitely try, and it would help keep maintenance easier (e.g., adding osim models), but even the python versioning is not the same.

2nd approach

frontend: same as 1st
core:

another docker for mono (like mmpose/openpose), app.py saves a video in a folder and the mono docker runs it. Pros: easier for dependency conflicts, cons: harder to pass error information, more maintenance of redundant codes

I think we are talking about the same thing. Most of the moco stuff would happen in the mono docker, so the conflicts would be limited with the OpenCap docker.


Adding user agent string as an available option

We have adaptive css, ie you get different styles based on whether you're on desktop/tablet/mobile/whatever, so I was hoping you could pull this guy to let me configure the user agent that jsdom uses.

Coverage remained the same at 97.059% when pulling 02339d1c3675f6f751898ebcc383c65aeb4d35e2 on bbeesley:useragent into 4cb60b884b2693d0f9b222a0fbe772e4a8d5995e on giakki:develop.

Looks pretty good, can you add an entry in the docs below the command line usage block?
I guess changing the default user-agent won't break existing usage?

Just noting that this is based against the develop branch (which is correct).

Sure, I'll update the docs in the morning. I can't see it causing any issues changing the default user agent string, but I can change it to whatever the default jsdom one is if you prefer (or just not pass the option if it's unset)

I can't see it causing any issues changing the default user agent string, but I can change it to whatever the default jsdom one is if you prefer (or just not pass the option if it's unset)

I don't really forsee issues either, but I wanted the other collaborators opinions on it.

I don't get why there are the other patches in this PR?

Coverage remained the same at 97.059% when pulling f61b6e29826aad68b8fab69aacab68ffd9ac2f11 on bbeesley:useragent into 555993ba6b5eb3a567abbdd765b5034e69a97076 on giakki:develop.

Not sure what you mean by that @XhmikosR
I branched from develop, and the PR is against develop. Looks like a bunch of stuff was merged last night so I've had to rebase and push again, but there's nothing in the diff here that isn't related to this feature.
@RyanZim I've updated the commit with a description of the option on the README. I haven't touched the default string, lmk if you want me to change it.

Well now it's OK after the rebase.
On Nov 7, 2017 10:18, "Bill Beesley"<EMAIL_ADDRESS>wrote:

Not sure what you mean by that @XhmikosR https://github.com/xhmikosr
I branched from develop, and the PR is against develop. Looks like a bunch
of stuff was merged last night so I've had to rebase and push again, but
there's nothing in the diff here that isn't related to this feature.
@RyanZim https://github.com/ryanzim I've updated the commit with a
description of the option on the README. I haven't touched the default
string, lmk if you want me to change it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/giakki/uncss/pull/343#issuecomment-342407717, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVVtWzWtiCg7q3GQlXeHe-Esf9rxAhTks5s0BJTgaJpZM4QTdJU
.

Yeah, sorry; I forgot about this PR and rebased develop onto master. My bad.


Add Collapsible Test output

This PR adds a collapsible test output pane to better display the log. The log is now located a the bottom of the test details instead of a column in the test details table.
Using an accordion for the collapsible object.

This is not important, but now the first table is not using the full available width:

Also, another improvement suggestion: that table is showing two duplicated fields that I think they could be removed, since they're already part of the header:

Fixed the table formatting and removed the test ID. I kept the test state since it also displays the skip reason for skipped test.


How to download photos?

Is there a straight-forward way of downloading the photos received on this? If not, are they encoded on any of the files in C:\Users\name\AppData\Local\Fuckr ?

you can simple drag it to your google chrome and download it. If u want to have it downloaded you can put a link tag or download the photos via code.


Reset navigation mounting all intermediate screens, but then destroyed

Structure
const Tab = TabNavigator(
  Stack: { screen: Stack }
)

const Stack = StackNavigator(
  Home: { screen: Home },
  SubView1: { screen: Subview1},
  SubView2: { screen: Subview2}
)

Reset Action (from SubView2)
const resetToHome = NavigationActions.reset({
  index: 0,
  key: null,
  actions: [
    NavigationActions.navigate({routeName: 'Home'}),
  ],
});
this.props.navigation.dispatch(resetToHome);

Current Behavior
Successfully reset to Home. But as I have some componentWillMount() / componentDidMount() logics, I found that these functions in Home and SubView1 are triggered, which I expect not.
It also seems that the component is destroyed immediately, so my async function produces some warning as below.
Sample warning message

software
version

react-navigation
1.0.0-beta.12

react-native
expo sdk 19.0.0 (supposingly 0.46)

A demo of this issue would aid in working through this issue - thanks!

const App = StackNavigator({
 Screen1: { screen: Screen1 },
 MainTab: { screen: SomeTabNavigator },
 Screen2: { screen: Screen2 },
 Screen3: { screen: Screen3 },
})

My stack is now (Screen1 is not in stack because I have reset to MainTab)
MainTab -> Screen2 -> Screen3 (current screen)
then I reset using dispatch
 this.props.navigation.dispatch(NavigationActions.reset({
    index: 0,
    actions: [
    NavigationActions.navigate({ routeName: 'Screen1'})
   ]
 })

The reset happens and I can go to 'Screen1' but 'Screen2' componentDidMount() is called. Why?

Thanks @spencercarli for looking into here. My situation is just the same as @ravirajn22 . The problem is reset will invoke all the intermidiate screen to mount and unmount themselves.
I think this should be unexpected for reset action and it causes quite a trouble.
For example I have some network call before mount:
// SubView1
componentWillMount() {
  axios.get('xxx').then(response => {
    this.setState({ response: response });
  });
}

This is invoked when I reset from SubView2 to Home (unexpected already). But when it want to setState(), it prompts a warning saying that SubView1 (this) does not exist anymore.

For anyone wants a temp workaround for this warning, I can only think stopping the setState, but not the network call.
constructor(props) {
    super(props);
    this.state = {
      unmounting: false,
    }
  }
componentWillUnmount() {
    this.state = {
      unmounting: true, // can't use setState because it is async
    };
  }
componentWillMount() {
  axios.get('xxx').then(response => {
    if (!this.state.unmounting) { this.setState(...) }
  })
  }

WARNING: this is an anti-pattern in react.
Or you may use isMounted() method to check but it's deprecated now.
See: https://facebook.github.io/react/blog/2015/12/16/ismounted-antipattern.html

@carsonwah I feel this library is no longer maintained by core developers they are all gone. Have u got any efficient work around, the above doesn't seems to be right.

@carsonwah ... i think @spencercarli is breathing new life into this lib, which we can all be grateful for and hopeful of (Spencer is a big advocate for RN and has a highly reputable blog)! In order for issue resolution though, Spencer has been asking for demos. It's unclear whether he is accepting inline code snippets as demos, or if he is requiring more Expo-like demos with accompanying repo. I do hope that copy-and-pasted example code pasted directly into these github issue trackers will suffice for his demo requests, so react-navigation can expedite issue resolution.

@quadsurf Sorry that my project is quite large so it's difficult to share it here. I just made a demo on expo snack but I surprisingly I found no bug there. Maybe there is some setting in my project that causes it. At this moment I haven't found out what exactly is the cause, but anyone see this and having the same issue is welcome to use this as the template to investigate more.
https://snack.expo.io/SkH0z7kYW

@carsonwah some of these issues are being discussed here #723 and some potential solutions in these PRs:
idempotent pushes #2334
queued action dispatch #1313
queued transitions in Transitioner #2400

Pinging OP @carsonwah since this issue has not been active for a while, and it's related to an old version of the lib.
Please let me know if you want this to remain open; if I get no answer in the next 7 days I will close it.
Moreover, we merged the queued action dispatch PR so your issue should be fixable now.

@kelset I have implemented some workaround to fix this, so I can't test it for now. You may close it first. I can come back to this later if I found this issue not fixed yet. Thanks for the work.

somebody fix this problem? or is there any workaround ?

@Pabloitto Have you tried the latest version? Mine (beta-21) seems not having this problem anymore. I can log componentWillUnmount, but not componentWillMount.

I also have this problem when using NavigationActions.reset with more than 1 action, when only 1 action provided the warning is not shown.
I am also using beta-21, don't know if it has been fixed on beta-22.

I also have this problem on. Mine also on beta-21. But in my problem it only happen when there is no extra params.
Example code:
This will trigger the warning
const navigationAction = NavigationActions.reset({
      index: 0,
      actions: [
        NavigationActions.navigate({
          routeName: 'Tabs',
        }),
        NavigationActions.navigate({
          routeName: 'Article',
          params: { uuid: 'kawasan-0001', type: 'history' },
        }),
      ],
    })

This will not trigger the warning
const navigationAction = NavigationActions.reset({
      index: 0,
      actions: [
        NavigationActions.navigate({
          routeName: 'Tabs',
        }),
        NavigationActions.navigate({
          routeName: 'Article',
        }),
      ],
    })

Just upgraded to 1.0.0-beta.30 and still got this issue. I'm not storing my navigation stack in redux, so I still get no way to fix this.
There is a new popToTop() action, but I cannot make it work. It keeps throwing Error: Cannot get config because the route does not have a routeName when I use it.

Ok I'll reopen this.

@kelset I have successfully reproduce the issue!
https://snack.expo.io/S1-VzXDLz
The bug happens when there is setParams() inside componentWillMount() or componentDidMount() methods, of "screens that are being navigated in the reset action".
For example, this reset action causes the bug:
class SomeComponent extends Component {
  componentDidMount() {
    this.props.navigation.setParams({ title: 'some title' });
  }
}
// somewhere else...
const resetToHome = NavigationActions.reset({
    index: 0,
    key: null,
    actions: [
      NavigationActions.navigate({routeName: 'Home'}),
      NavigationActions.navigate({routeName: 'SomeComponent'}),  // Appearing here
    ],
  });

I cannot go deeper into react-navigation source code to debug as I had a hard time understanding the code. Hope this observation can help resolving the bug soon. Thanks for the great works, this library is definitely getting better and better.

@mssio I cannot reproduce your case. Do you have any setParams() in your components?

cc @ericvicenti

@kelset @ericvicenti I believe one of my PRs #2400 was fixing this problem. The transitioner is leaking state between transitions. Some actions like set params or reset don't really have a transition attached but they must pass the newly determined state further down the line.
const nextScenes = NavigationScenesReducer(
  **this.state.scenes**,
  nextProps.navigation.state,
  this.props.navigation.state

);
Currently, only actual state is used to determine the next scene, which can be an issue if componentWillReceiveProps is called multiple times during a transition. This is why I came up the simple solution to queue all potential transitions in order to do not put the transitioner in invalid state.
Same problem with  const indexHasChanged =
nextProps.navigation.state.index !== this.props.navigation.state.index;
indexHasChanged should be calculated against pevious props at every step and not only against current props.
From my PR:
const indexHasChanged = nextProps.navigation.state.index !== prevProps.navigation.state.index;
In a way, this problem is similar to the one solved in #1313

Apparently @brentvatne had an attempt to fix this in https://github.com/react-navigation/react-navigation/commit/f8426bef7ebe39722e155c8ca774a96255867036

hey folks we'll be getting to reset issues shortly. reset works very strangely at the moment and we need to rethink it, so we'll discuss and write up a RFC soon

I've tried v1.0.3 and looks like the issue is still there

Definitely experiencing this and it's a show stopper! Causing all kinds of crashes when doing a reset. Additional snack: https://snack.expo.io/HJoDK8uPM

Anyone else who stumbles here and is a beginner like me, I've instead used the popToTop action and it provides what looks like the same result as Reset.

Some solution for this, when I execute reset, the componentWillMount of all previous screens is executed
Dependencies:
"react-navigation": "^1.2.1",
"react-native": "0.52.1",

In my case in each event I had a call to an api, create a global variable that stopped the calls. But the arrangement seemed too far-fetched. Then I found that the rendering of each screen is also executed.
I chose to leave the library, it's very good but this bug scared me a little.
sorry for my English

Aldo executes the render

Having this issue in 1.2.1 and now in 1.5 aswel. As soon as I reset, all componentWillMount hooks will be run again. componentWillUnmount is never called anywhere.

Seems like this issue is being worked on actively. I just wanted to give an FYI, it seems like the replace function is also causing the same issue?

i have the same issue with 1.5, everything is working fine with 1.0.2.
My application: a 'home' component which has fetch data function inside componentDidMount, and based on the data it will redirect to another 'details' component. Whenever I do a reset to 'home' inside 'details', the 'home' component's method is called again which triggers the fetch function and then redirects to 'details' again and again

can someone who is having this issue please share a mcve (https://stackoverflow.com/help/mcve)? that will help a lot for us to be able to test fixes

@brentvatne Here: https://snack.expo.io/S1-VzXDLz

@tetreault - even better than sharing the full project is to share a MCVE: https://stackoverflow.com/help/mcve - this is actually what we ask for when reporting bugs. please create one and share it here otherwise you will probably not get any answers

Closing since this is a very old issue. Please open a new issue if you're still facing the issue with the latest version.

Is there a better solution than the setTimeout in componentDidMount yet? I am using react-navigation v4 (upgraded from v1) and I still have this issue.

Here is my solution if anybody interested:
Basic idea is listen to the onTransitionStart and onTransitionEnd, do some tricky work.
// StackNavigator Example
const RootStackNavigator = StackNavigator(reactNavigationRouters, {
navigationOptions: ({ navigation }) => { },
initialRouteName: 'LaunchScene',
headerMode: 'screen',
onTransitionStart: (newState: any) => {
transitionManager.transitionStarted(newState)
},
onTransitionEnd: (newState: any) => {
transitionManager.transitionEnded(newState)
}
})
// TransitionManager.ts
class TransitionManager {
private _currentScene?: string
private _listeners: { [key: string]: () => void } = {}
public transitionStarted(newState: any) {
const routeName = newState.scene.route.routeName
if (!routeName) {
return
}
this._currentScene = routeName
}
public transitionEnded(newState: any) {
const routeName = newState.scene.route.routeName
if (!routeName) {
return
}
this._currentScene = undefined
if (this._listeners[routeName]) {
try {
this._listenersrouteName
} catch (error) {
console.warn('TRANSITION END Listener Error: ', error)
}
}
this._listeners = {}
}
public setTransitionListener(routeName: string, listener: () => void) {
if (this._currentScene !== routeName) {
listener()
return
}
this._listeners[routeName] = listener
}
}
const transitionManager = new TransitionManager()
export { transitionManager }
After that, in your Scene listen to the didFocus on props.navigation something like following:
// Scene.ts
componentDidMount() {
this.props.navigation.addListener('didFocus', ()=>{
this.props.navigation.setParams({...})
})
}


Fixes a typo

This PR fixes a small typo:

Installing Installing the Microsoft Dynamics 365 Business Central App
Installing the Microsoft Dynamics 365 Business Central App

Best regards,

Thanks for your contribution!


I have the windows problem

the os is windows and error like this
How to solve it. I reviewed the question18, but seems not the same question?

command line: evaluate_functional_correctness samples.jsonl

Same issue here.


inference on single image

It is possible to do inference on single image to check accuracy? like infer.py ?

That's a good idea. I will update it soon.


The search results were not the same when enter a space into search box of "services" and "subscriptions"

Date: 2016/06/02
[Test category]: creative Test
Severity：Minor
[Steps to reproduce]

Login in marketplace as customer;
Click on "ACCOUNT->Subscriptions";
Enter a space into the "Search subscriptions", then click on search button;
Enter a space into the "Search services", then click on search button.

[Expected result]
In step 3, no subscription should be list;
In step 4, no services should be list.
[Observed result]
In step 3, no subscription was list;
In step 4, no service was list, but the message "Please specify a search term" was displayed in the screen.

The search must behave uniquely. But in any case no error message should be displayed.
If no or a blank search term is given, I suggest for

services, to return the "browse all services" list, by forwarding to this page "Home > All services".
subscriptions, to display all available subscriptions (same as now, when searching with empty string as search term)

@mdehn, do you agree?
[2016-05-19 10:19:30] Reply from @mdehn:
fine for me.

Verified by FNST
[verified-version] V16.0 fix6
[verified-details]Search blank in service and subscription pages will return all of the results. so verified.


'changeset version' fails if there is package without 'version' (prerelease mode)

Affected Packages
@changesets/cli
Problem
If there are packages without 'version' field changeset version fails with error in assemble-release-plan and, later in apply-release-plan. Most likely this situation will happen with private packages only.
Proposed solution
Filter out private packages completely in version command.
packages.packages = packages.packages.filter(pkg => !pkg.packageJson.private);
I'll submit the pull request if you're ok with suggested behavior.

Private packages are meant to be versioned the same way - in a sufficiently big monorepos etc it might make a lot of sense to keep changelogs of those packages. That being said changesets shouldn't fail like this. I think those are possible options to go about this:

ignore only if they are version-less
add a config option to ignore all private packages
enforce version to be added, changesets would have to validate & warn appropriately

I think the first option makes the most sense because it gives the best of both worlds - it allows for both use cases. You can opt-into versioning of those packages by giving them an explicit version, but you can also skip it entirely by not including it at all.
One thing to think about - dependencies of those private packages should be always updated, no matter if they have their changelogs generated for them or not.

enforce version to be added, changesets would have to validate & warn appropriately

I think this is the behavior that we should have(and currently implicitly do).
I don't really see what the problem with adding a version to a package is?
Though if you wanted to avoid updating the version of a package, https://github.com/atlassian/changesets/issues/242 should solve that.

I don't really see what the problem with adding a version to a package is?

There are so many ways a package may be used. Versioning doesn't make sense in all cases. It makes sense to version a private package, which is a server that is published through docker images. But I am not sure about all cases. For instance, if the private package is in a subdirectory of another package and contains just some utils, or it installs an alternative node_modules for experimentations (with Webpack you may resolve modules from custom locations).
Also, some packages are not even private but they don't have a name and a version. I would allow them as well. That would make changesets a zero config tool.

Yea, similarly Emotion itself has some internal “packages” with test utils etc for which versioning doesnt make sense.

Let me then check for existence of version field explicitly. Maybe, print warning "Next packages do not  have a version field and will be ignored" and filter them out.


docs: changed marketplace description

Changed marketplace description
Deleted unused gif files

:tada: This PR is included in version 1.6.1 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


bugfix: fix check kernel version is wrong

Ⅰ. Describe what this PR did
check kernel version when setting disk quota driver, it should check
kernel > 4, instead of major > 4.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
For example，the version of kernel is 3.10, and it supports group quota. you can test like this:
# cd pouch/test
# go test -check.f TestAliKernelDiskQuotaWorks

test will be succeed.
Ⅴ. Special notes for reviews
Signed-off-by: Rudy Zhang<EMAIL_ADDRESS>

LGTM

Codecov Report

Merging #946 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #946   +/-   ##
=======================================
  Coverage   13.21%   13.21%
=======================================
  Files         123      123
  Lines        8145     8145
=======================================
  Hits         1076     1076
  Misses       6973     6973
  Partials       96       96

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 079a92d...0b8cbf9. Read the comment docs.


Add new comment mode: summary

Hello,
Today, GitHub have announced the possibility to display markdown in job summary (by the way, their example is the display of test results).
I think it's truly awesome, and I would appreciate if this action could offer the mode "summary" to report the unit test results instead of decorating pull request. Or maybe in complement?
Anyways, I'd be glad to contribute if you agree with this suggestion.

Thanks for sharing. I agree, it would be great if the action would also publish the markdown as this new summary. Please go for it.

This has been released, enjoy!


"At code:1" error message

platform: mac
code

func TestName(t *testing.T) {
	const code = `
var name = "test_hello";

function main() {
}

main();
`
	runtime := quickjs.NewRuntime()

	context := runtime.NewContext()
	defer context.Free()

	for {
		val, err := context.Eval(code)
		if err != nil {
			fmt.Printf("%#v\n", err)
			time.Sleep(time.Second)
			continue
		}

		g:=context.Globals()
		fmt.Println(g.Get("name"),val)
		time.Sleep(time.Millisecond*10)
		val.Free()
	}
}

output

test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
test_hello undefined
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
&quickjs.Error{Cause:"", Stack:"    at code:1\n"}
test_hello undefined

Hi,
Same results here (linux/amd64/go 1.15.8).
I find the culprit, before parsing code quickjs check the heap size against the memory allocated with:
static inline BOOL js_check_stack_overflow(JSRuntime *rt, size_t alloca_size)
{
    size_t size;
    size = rt->stack_top - js_get_stack_pointer();
    return unlikely((size + alloca_size) > rt->stack_size);
}

with
static inline uint8_t *js_get_stack_pointer(void)
{
    return __builtin_frame_address(0);
}

I have printed value of __builtin_frame_address(0) while calling context.Eval(code):
0x7ffd781aabd0
0x7ffd781aad60
0x7ffd781aa880

The stack value isn't the same but values are close.
Then for no apparent reason stack make a big jump !
0x7f27a0556cd0

Then quickjs think something really bad happen (stack overflow) and throw an error.
If I replace time.Sleep(time.Millisecond) by time.Sleep(0) which does nothing as the doc says, the problem disappear.
What is the relation between time.Sleep() and __builtin_frame_address(0) ?
I check the time.Sleep() implementation in runtime, but this is related to runtime and scheduler which are too complicated to me.
Maybe cgo has an explanation but I have not find it.
As I can't fix the heap size check, I find this workaround: disable the stack check of quickjs by removing the preprocessor CONFIG_STACK_CHECK in quickjs.c line 77:
#if !defined(EMSCRIPTEN)
/* enable stack limitation */
//#define CONFIG_STACK_CHECK /* bug when called after time.Sleep() in Go */
#endif

Maybe someone will fix the js_check_stack_overflow callers to return a real error message as "stack overflow detected" or someone will fix the bug (cgo ? Does the check need to be embedded in Go with runtime's function and not in C part ?).

Some issue in jvm，once call it in ScheduledExecutorService scheduleWithFixedDelay method，it will [exception]，no any error info....


[bug] circular reference

Search before asking

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

Environment
https://laf.dev
What happened
Circular references in cloud functions have no errors and are difficult to troubleshoot
How to reproduce

how to fix it?
how to  easy to find Circular references ?

Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

@juukee I will add detection to indicate circular dependencies.

when can we use it ? Urgent need, more and more functions

when can we use it ? Urgent need, more and more functions

waiting #1388 merge


User story missing on the confluence page?

Problem description
Probably user story is missing in the Readiness checklist per API version table on Confluence.
Expected action

Additional context

Fixed ! thanks Ludovic

Thanks Tanja


Color.to_html() Produces odd results.

Any RGB value >0 turns into FF. So for example Color(255,100,0).to_html() produces ffffff00.
I'm pretty sure the problem is in String _to_hex(float p_val) in core/color.cpp. I'd mess with it myself, but I can't get godot to compile :(
Tested in 2.1.4 x64, Ubuntu Linux

Color needs args in 0 to 1 range.

Oh oops!


fix: add banner_color to User model

Added banner_color to User model (slots and docstring)
About
This pull request is about the undocumented banner_color user attribute, which is for user banners..
Checklist

[ ] I've ran pre-commit to format and lint the change(s) made.
[x] I've checked to make sure the change(s) work on 3.8.6 and higher.
[x] This fixes/solves an Issue.

(If existent): https://github.com/interactions-py/library/issues/607

[ ] I've made this pull request for/as: (check all that apply)

[ ] Documentation
[ ] Breaking change
[ ] New feature/enhancement
[x] Bugfix

frick i opened on accident


Workaround for reversed right click paste

Fixes #496
Not sure if this is the correct way to detect right click, but I found the same method in https://github.com/lzybkr/PSReadLine/blob/9d592b875ebea83d6385b9cc6b6899584b08277a/PSReadLine/BasicEditing.cs#L246-L249

Thanks for the PR.
There is no reliable way I know of to detect paste - we must use a heuristic.
I'm not sure this is the best approach. I think the right fix to this issue will also fix #37 and #77, maybe #245 and possibly others.
What I'd planned was to implement a stronger heuristic that measures the time between keys and makes assumptions about how fast a human can type - and if that rate is greatly exceeded, collect all of the keys and do a single paste operation, hopefully the same as though the user used Ctrl+V.
I guess there are a few patterns where a key is obviously not something pasted - e.g. when Alt is specified, and probably some with Ctrl, though some keys with Ctrl set could be pasted, so it's not a clear cut.

Ah the pleasures of conhost :) Looking at how fast the keys are coming in might actually be a workable solution. Alternatively, have you thought about getting a hold of the mouse events? It seems to be possible
Let's close this in favour of a more robust solution.

I hadn't considered reading mouse events for this issue - but reading those does require not using Console.ReadKey, which means duplicating all of that code. I know this because I investigated detecting resize events.
That said, it's not clear to me that the right click event will be available - and even if it is, that doesn't help if the user types Alt+Space,e,p to paste.

I opened #579 to track all of the right click paste issues - and I think they are all addressable without this PR so I'm closing.


MWPW-139879: Fixed widget cache preloading issue

Resolves: MWPW-139879
Before: https://mwpw-139879-fix-lcp--dc--adobecom.hlx.live/acrobat/online/pdf-to-word
After: https://stage--dc--adobecom.hlx.live/acrobat/online/pdf-to-word

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (26f56a9) 97.90% compared to head (9c2a28e) 97.90%.

Additional details and impacted files
@@           Coverage Diff           @@
##            stage     #510   +/-   ##
=======================================
  Coverage   97.90%   97.90%
=======================================
  Files          26       26
  Lines        2907     2907
=======================================
  Hits         2846     2846
  Misses         61       61

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Load or store to an unaligned address issue when running on Wemos D1 mini

I am trying to run this software on a Wemos D1 mini with ESP8266. The same used in the x-clock software which is working with this IKEA LED panel.
I build the software in environment d1_mini_pro-OTA with platformIO on VS code. I first flashed the software on the ESP and checked this before opening the LED panel. I got an error but thought that makes sense because nothing it connected to the ESP. Then I opened the panel, soldered all wires as described in the README and tested.
Now I got the same error as before:
Exception (9):
epc1=0x4020fc56 epc2=0x00000000 epc3=0x00000000 excvaddr=0x3fff0869 depc=0x00000000

LoadStoreAlignmentCause: Load or store to an unaligned address
  epc1=0x4020fc56 in SPIClass::writeBytes_(unsigned char const*, unsigned char) at ??:?

After this the ESP resets and starts again connecting to WiFi (which is successful) and then crashed again with the same error.
First I was confused by the SPIClass part but I now understand that the LED panel works with shift registers. Each panel has 4 of then each controlling 16 LEDs, bringing the total up to 256 LEDs. I checked all my connections, no short circuits and good connection from LED panel to the ESP board. Tried another identical Wemos D1 mini with the same result.
Just to check the hardware I flashed  X-Clock WEB FOR OBEGRÄNSAD (MINI PRO) V0.1.09 for here: https://o-clock.eu/ and that worked right away! More on this application can be found here (I also found it in another issue here): http://blog.digital-image.de/2023/05/31/x-clock/
But I don't want only a simple clock I want all functionality of this awesome repository! What can I do to diagnose my system further?

If you are still interested try the patch from #105

Yes, I am surely am! Thanks!
But with my new laptop I have some issues setting everything up again. Without your fix I can monitor it running but the screen stays off. No Exception is printed only some jiberish.
I will try to make this working first and then test and review your PR #105


⚠ Secret Website 3 has degraded performance

In 65511e2, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3101 ms

Resolved: Secret Website 3 performance has improved in a469bbc.


fix: relative css position for preventing bug in annotations #461

For this fix to work I had to add a wrapper div to canvas and annotations. Without this wrapper, we'd have to directly change the css positioning for the container provided by the user and this could lead to side effects for the user (e.g. if the user sets another positioning style for its container).
This fix doesn't solve the issue of annotations displayed outside of their container element.

Added a new commit to the PR with a fix for the annotations outside of the viewport.

last fix for a small annotation overflow at the bottom of the viewport. The overflow styling is now on the user provided container (not the wrapper), as the wrapper has a few extra pixels in height.

thanks, will test tomorrow

Hi, reverted it. It does not work when the users does not supply an Element or an element id.

Sorry about that, I'll investigate further and make the relevant test cases.
By the way, to improve code clarity, I could create an extra object in stage: there is already this.container, but I would like to add this.wrapper as an internal reference to the additional div created for positioning the annotations.

there is already this.container, but I would like to add this.wrapper as an internal reference to the additional div created for positioning the annotations.

sounds good, I like being very explicit, especially if it does not matter performance wise

Hi, what is the expected behaviour if no HTML Element or element id is provided?
I've checked with latest version and nothing is displayed on screen (but the stage object is created and includes an actual reference to a container element).
Additional question: what shall be done regarding tooltips? They are displayed on top of everything else and can overflow the viewer. I think this behaviour is nice as a cropped tooltip would not be informative, so I would favour keeping tooltips as they are.

Hi, what is the expected behaviour if no HTML Element or element id is provided?
I've checked with latest version and nothing is displayed on screen (but the stage object is created and includes an actual reference to a container element).

stage.viewer.container should be a plain div whose style can be changed by the user

Additional question: what shall be done regarding tooltips? They are displayed on top of everything else and can overflow the viewer. I think this behaviour is nice as a cropped tooltip would not be informative, so I would favour keeping tooltips as they are.

totally agree, no cropped tooltip

stage.viewer.container should be a plain div whose style can be changed by the user
Meaning this div should be appended to the DOM at some stage? At present there is no code for handling that in viewer.ts (a div is created but not appended to the body of the page)


refactor(extension): change styles for side panel button component

Checklist

[ ] JIRA - <link>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Restyle the side panel button component
Screenshots
https://github.com/input-output-hk/wallet-world/assets/65184652/2c9dcb84-f3f9-49ec-ab71-9acac8ad6491
https://github.com/input-output-hk/wallet-world/assets/65184652/c375b29a-2a8e-443e-8e53-ac55b7253b4b

Allure report
allure-report-publisher generated test report!

smokeTests: ✅ test report for 71774005

passed
failed
skipped
flaky
total
result

Total
36
0
0
0
36
✅

Sorry, it could be possible to change the arrow animation for something more smooth? maybe just an intro animation (repeated once)
I will prepare a video/HTML based on the latest figma layout to better illustrate: https://www.figma.com/file/RPN3V4Et7hMbXkV5ZIF75i/Lace-UI---Browser?type=design&node-id=17762-321424&t=SKQfnOEbAj0PzTlO-4

I'm adding an updated version of the button animation -- gif/video attached

"bouncing" effect removed & replaced by a "slide-in effect (no-repeat)"
arrow size adjustment (to make the prototype match figma)
inner borders removed during transition (hover/in-out)

https://github.com/input-output-hk/lace/assets/8201486/3ada6d88-4194-483a-9bec-7dfae77e83a3

I'm adding an updated version of the button animation -- gif/video attached

"bouncing" effect removed & replaced by a "slide-in effect (no-repeat)"
arrow size adjustment (to make the prototype match figma)
inner borders removed during transition (hover/in-out)

2023-06-21.11-53-52.mp4

@allanleone I've addressed these issues
https://github.com/input-output-hk/lace/assets/65184652/1f776b5d-f27f-4ade-809c-9fcfd4c26506

@jplorek-atix 'Normal' button in dark mode isn't correct. The right section is gray when it should be black:
Actual:

Expected:

Looking good :) I'll approve once @milagi comments are resolved :)


Stepping down from Inexor

Fellow Inexor friends, the wider community, and whom else it may concern.
I am stepping down with my affiliation with the Inexor project.
It's been 5 years with you! What a ride! What an amazing time. As they say: "all good things must come to an end.", and so must my commitment with the Inexor project.
In the last months my contributions to Inexor have been rarer than ever mainly because:

I see the project moves in a good overall direction and I think it no longer needs my help
I have different views than some members, which is okay. However by now, I would just like Inexor to be a playable game instead of a sandbox, and I am unwilling to contribute further to a sandbox engine :/
Currently I am in the process of making FLOSS my living, with a state-subsidized non profit organization. That occupies 99% of my free time.

This decision didn't come light-heartedly for me. It is in no way personal.
I wish you all the best with the project and am delighted to see what Inexor may become in the future.
PS: I will need to migrate all the administrative privileges to someone. We should have a talk about that soon.

As already talked in private: It is unfortunate to see you go, but I wish you all the best for your future :)
As for the administrative stuff:

Netlify decided to discontinue free organizational teams, but continues existing teams.

Which means we either
A: Abandon the current team and create one account that takes care of everything (maybe a shared inexor account everyone has access to) or
B: Keep the existing team with @Fohlen having still permissions (we trust you <3). Our team will continue to only have access to the inexor webpage and not the other websites you have in that team.

These are the members right now:

Since Nooby is also still part of the management team there, I'd go for option A.
@Fohlen can you please check, whether you can add and/or remove further people?
What other administrative things are open for you?
How is Twitter and other social media handled? @a-teammate

@MartinMuzatko I can still add / remove people.
However I'd say it would be better to have a dedicated Netlify because, mainly because the Inexor DNS is also handled via Netlify. This in term means, that for DNS changes to be made, I would be needed to reach out. It is not a problem per-se, but it sure complicates things for you.
The social media handles should be well in the hands of @IAmNotHanni


fix: insert close-paran with weird indent and comments

Insert after last NEWLINE, no need to overwrite

I'm PR or your PR ;)

Thanks, although I'm not really happy with this approach. Especially in code with a lot of indentation, there will be a ton of spaces between the ) and the . even in the common case. It seems relatively rare that there will be comments here, so hopefully it's not as big of a deal as in other cases.
I filed https://github.com/decaffeinate/decaffeinate/issues/782 to think more about how to avoid destroying comments in a more general way.


Adding title sending when allying

Allowing a 'title' to be sent when allying/de-allying. This is something I saw in another plugin and is a very nice to have feature.
If you don't like the text to pop up, you can always just leave the new ally.add_title and ally.remove_title messages empty. (which might be a requested change to keep backwards compatibility?)
When being working on this, I saw a bit of duplicated code between the sendMessage/sendMessageF/getMessage/getMessageF and the new title-methods. So I cleaned this up in a way that shouldn't impact the performance in the slightest bit, but improves the readability a lot.
Please do let me know if I need to change anything.
Screenshots:
Ally:

Neutral:

Thanks for making this, it may also be worth making the title message configurable unique to the chat message in case the user wants to only provide a short title message.

Done. It wasn't much effort.


docs: add Configuration section

Add section explaining the necessary configuration steps tu run this app.

Any news on the docs section? Thanks!


Join queue interop group

That would be great if you could join the queue interop group.  Let's build greater stuff together.

sent an invitation.


Having Issues in IE 11

Hello,
Hope this message finds you well. I have extensively used the datatables in my Vue JS projects but I have browser issues. The page is not getting loaded in IE because of issues while in other browsers it is working fine. Please can you provide your feedback. Thanks

more details?

@kenberkeley - I am also experiencing the same issue. I am using laravel-mix to compile assets with a vue frontend implementing this package.
The rendered code looks like this:

The error is that IE11 doesn't support the => notation.
This appears to come from this file:
ROOT/node_modules/vue2-datatable-component/dist/min.js
I think that since the file is already minified, laravel-mix can't handle the babel conversion to make it IE11 compatible.
Suggestions to work around it?

shown as above, the min.js is fine

@kenberkeley Thanks. I was able to update laravel-mix with the following rule that corrected the output to match what you posted.
In case it helps anyone else, add this to webpack.mix.js
`if (mix.inProduction()) {
mix.version();

mix.webpackConfig({
    module: {
        rules: [{
            test: /\.jsx?$/,
            exclude: /node_modules(?!\/vue2-datatable-component)|dist/,
            use: [
                {
                    loader: 'babel-loader',
                    options: Config.babel()
                }
            ]
        }]
    }
});

}`

i havent used laravel so might not help u, sorry for that


fix: specify prettier parser

fixes #270

Codecov Report

Merging #271 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
+ Coverage   98.21%   98.21%   +<.01%
==========================================
  Files          13       13
  Lines         729      730       +1
  Branches       32       32
==========================================
+ Hits          716      717       +1
  Misses         13       13

Impacted Files
Coverage Δ

src/format.ts
100% <100%> (ø)
:arrow_up:

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 75fb461...8650a39. Read the comment docs.

I would prefer #273.

Better fix :)


lssues of generic type in apimodel

I'm using springfox-swagger2 , version 2.6.1.
Here is a sample:
@ApiModel(description = "this is a result")
public class PageResult<T> {

	@ApiParam(hidden = true)
	private List<T> list;
	private int pageNo = 1;
	@ApiParam(hidden = true)
	private int pageCount;

	//getters and setters
}

and here is a spring-mvc http request:
        @PostMapping("/testModelT")
	@ApiOperation("/TestModel")
	public PageResult<Cat> createResultCat(@ModelAttribute PageResult<Cat> result) {
		List<Cat> list = new LinkedList<Cat>();
		list.add(new Cat("kitty", 1));
		result.setList(list);
		return result;

Maybe I hide two param (list and pageCount) . Infact pageCount is hidden but list is not.

Model attributes don't support generics as of 2.6.1. Also as long as the PagedResult doesnt have getter/setter pairs the property will be ignored.


Added max_tokens alias to length in FinishReason

yes i pushed 5 commits for one line change

@DuckyBlender Why was this added? Please explain the reason.

redundancy and openrouter Gemini api, feel free to close if it doesn't align with your preferences

See https://github.com/dongri/openai-api-rs/pull/137


New: ThunderBadgeQuest

start from vermilion city (after SSAnne) and end on route 11
Quest: Get Thunder Badge and exp on Route 6

TESTED.
It and worked perfectly.


Traffic Portal library typings

I'm trying to add some typings to Traffic Portal things, but every PR that does that will need to add typings for JS libraries we use. So, this adds them in its own PR, to help keep the others cleaner.
This doesn't add type imports to every place they are used, but it does some just because the PRs I cherry-picked originally modified some files that I had to touch anyway for other reasons in the full changeset for which they were originally written.
This will help find bugs, and in some cases ease the transition of code from TPv1 into TPv2.

Which Traffic Control components are affected by this PR?

Traffic Portal

What is the best way to verify this PR?
There should be no behavior change, so existing tests should all still pass. To check the typings, use a TypeScript compiler capable of static JavaScript analysis via JSDoc typings, or some editor/IDE with that functionality built in.
PR submission checklist

[x] This PR uses existing tests
[x] This PR is documentation of a sort
[x] This PR doesn't need a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

This PR now depends on #7242


Jumping when uploading

When uploading to the ESP32, the motors will spin momentarily, as if it is running the last program for a short period of time.

is this while the upload is writing, or before that?
Generally its a good practice to always set the motors to disabled while programming.

Right around when it starts to upload. It’s not realistic to have them plug the wire in everytime (and not possible with the shim board). We need a failsafe.

On Aug 7, 2020, at 3:36 PM, Kevin Harrington<EMAIL_ADDRESS>wrote:
is this while the upload is writing, or before that?
Generally its a good practice to always set the motors to disabled while programming.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/WPIRoboticsEngineering/RBE1001Lib/issues/27#issuecomment-670679974, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5JK5EU5UN3Z7IT3WUSCWLR7RJVNANCNFSM4PX5WUAQ.

 Is it that your program has it run right off the boot? What INO is causing this behavior?
FYI: the shim has a jumper for motor disable right near the user sensor headers and it it easy to get to.

I think I discovered what the issue is: when it first starts to upload, it does something to the BOOT pin. But I use the BOOT button for gatekeeping:
while(!digitialRead(0)) {}
So when it gets ready to upload, a button signal will get sent, the code will run for, say, half a second, then the upload will start.

On Aug 17, 2020, at 3:45 PM, Kevin Harrington<EMAIL_ADDRESS>wrote:
Closed #27 https://github.com/WPIRoboticsEngineering/RBE1001Lib/issues/27.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/WPIRoboticsEngineering/RBE1001Lib/issues/27#event-3663355154, or unsubscribe https://github.com/notifications/unsubscribe-auth/AF5JK5HOCLHYX3TQOVVHRWDSBGCFVANCNFSM4PX5WUAQ.

AH, are you explicitly setting the button to INPUT_PULLUP? There is no external pull up on that pin, so it may be that the reset on program clears the Pullup from the internal bootloader use of the pin, causing it to move.


feat: listed all movies

added the code to display all the movies inside of the movies.txt file When the user clicks the first option they will be displayed all of the movies

Looks good!


Example project to deploy to AWS failed to load container: config.v2.json no such file or directory

First, I want to say thanks for those who make this excellent course!
I was following this guide for deploying a model on AWS Lambda until the part where I had to test the lambda function locally for what I used the next command:
sam local invoke PyTorchFunction -n env.json -e event.json

After some time the command fails with the message:
...
2019-05-21 17:39:23 Invoking app.lambda_handler (python3.6)
2019-05-21 17:39:23 arn:aws:lambda:us-east-1:934676248949:layer:pytorchv1-py36:2 is already cached. Skipping download
...
2019-05-21 17:39:23 Mounting /.../aws-lambda/pytorch as /var/task:ro,delegated inside runtime container
2019-05-21 17:39:53 Function 'PyTorchFunction' timed out after 30 seconds

Enabling verbose logging I realize it was a problem with the Docker container. So, I decide to dig a little more and found the next messages on Docker logs:
... level=error msg="Failed to load container 67ab2499b983f83a29294995013775f0a3810e53f66a6504c63411db4cacdbb6: open /var/lib/docker/containers/67ab2499b983f83a29294995013775f0a3810e53f66a6504c63411db4cacdbb6/config.v2.json: no such file or directory"

The message is Failed to load container config.v2.json: no such file or directory and I think as the SAM example project was updated recently  https://github.com/fastai/course-v3/pull/290 (only 4 days ago) ...
PD: I tested the application on Windows 10 and Linux Ubuntu 18.04 with the same result.

Many thanks! The lambda docs and toolkit isn't maintained by us - so could you please either add this info to the forum, or else provide a PR that fixes the issue?


GPU OOM problem

Hi，When I use the defult parameters which you shared for training, I get the problem of OOM. My GPU is single 3090.  I also tried d6 model, still OOM, is it right?

You should reduce the batch size, I use NVIDIA TESLA V100 32G GPU.


separate smoke and CI tests

This separates the Smoke subset (a small subset useful to run quickly) from the CI subset (the ones we run during appveyor CI).
Also remove some dead code from tests/Convert-NUnit2Args-to-NUnit3Where.fsx
Also added the call needed to invoke the IDE unittests (TEST_VS) though that's not actually invoked yet (we were never previously running these during CI)

I also fixed some failing tests in master an enabled some more CI
Some failing tests remain documented in #915 .  Depending on run times it looks like we should be bale to enable all of FSHARPQA in appveyor, and nearly all of FSHARP

I'll merge this - there was one failing test fixed by the last commit.


mysql no working?

After setting the database information, it is stuck in the loading configuration file when restarting

servers:paperspigot 1.8.8

https://github.com/lucko/LuckPerms/wiki/Storage-system-errors


Notes

This issue is used for collecting notes and random thoughts

Some database ideas similar to time series database
immutable database

https://github.com/EventStore/EventStore
rethinkdb seems used to have it http://www.xaprb.com/blog/2013/12/28/immutability-mvcc-and-garbage-collection/
datomic

streaming database, real time database

https://en.wikipedia.org/wiki/Real-time_database


When creating DB do not use --run-sync-db

With --run-sync-db:
$ sudo PYTHONPATH=/opt/graphite/webapp django-admin.py migrate --settings=graphite.settings --run-syncdb
Operations to perform:
  Synchronize unmigrated apps: account, render, staticfiles, whitelist, metrics, url_shortener, dashboard, composer, events, browser
  Apply all migrations: admin, contenttypes, tagging, auth, sessions
Synchronizing apps without migrations:
  Creating tables...
    Creating table account_profile
    Creating table account_variable
    Creating table account_view
    Creating table account_window
    Creating table account_mygraph
    Creating table dashboard_dashboard
    Creating table dashboard_template
    Creating table events_event
    Creating table url_shortener_link
    Running deferred SQL...
Traceback (most recent call last):
  File "/usr/local/bin/django-admin.py", line 5, in <module>
    management.execute_from_command_line()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 345, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/commands/migrate.py", line 172, in handle
    self.sync_apps(connection, executor.loader.unmigrated_apps)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/commands/migrate.py", line 293, in sync_apps
    cursor.execute(statement)
  File "/usr/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "/usr/local/lib/python2.7/site-packages/django/db/utils.py", line 95, in __exit__
    six.reraise(dj_exc_type, dj_exc_value, traceback)
  File "/usr/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 62, in execute
    return self.cursor.execute(sql)
django.db.utils.ProgrammingError: relation "auth_user" does not exist

Without --run-sync-db:
$ sudo PYTHONPATH=/opt/graphite/webapp django-admin.py migrate --settings=graphite.settings
Operations to perform:
  Apply all migrations: admin, contenttypes, tagging, auth, sessions
Running migrations:
  Rendering model states... DONE
  Applying contenttypes.0001_initial... OK
  Applying auth.0001_initial... OK
  Applying admin.0001_initial... OK
  Applying admin.0002_logentry_remove_auto_add... OK
  Applying contenttypes.0002_remove_content_type_name... OK
  Applying auth.0002_alter_permission_name_max_length... OK
  Applying auth.0003_alter_user_email_max_length... OK
  Applying auth.0004_alter_user_username_opts... OK
  Applying auth.0005_alter_user_last_login_null... OK
  Applying auth.0006_require_contenttypes_0002... OK
  Applying auth.0007_alter_validators_add_error_messages... OK
  Applying sessions.0001_initial... OK
  Applying tagging.0001_initial... OK
  Applying tagging.0002_on_delete... OK

I think you need to run python manage.py migrate auth
PYTHONPATH=$GRAPHITE_ROOT/webapp django-admin.py migrate auth --settings=graphite.settings
first - it will create auth relations, then --run-syncdb will work well.

When i tried it myself without auth (and without --run-syncdb) it seemed to have created all the tables (including auth relations), since when i tried --run-syncdb right after, there were no issues.
At this point, i'm not sure what's the difference between:
PYTHONPATH=$GRAPHITE_ROOT/webapp django-admin.py migrate auth --settings=graphite.settings

and
PYTHONPATH=$GRAPHITE_ROOT/webapp django-admin.py migrate --settings=graphite.settings

but if you say that's the right way of creating the DB for the first time, i can make the change in the PR. I'll await confirmation.

Well... I'm not a big Django expert, unfortunately. If migrate doing the same as migrate auth and migrate --run-syncdb then it's simpler - because it's a single command.
Maybe @brutasse knows?

Codecov Report

Merging #2126 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2126   +/-   ##
=======================================
  Coverage   77.63%   77.63%
=======================================
  Files          79       79
  Lines        8195     8195
  Branches     1684     1684
=======================================
  Hits         6362     6362
  Misses       1575     1575
  Partials      258      258

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 b73cc91...3ea588f. Read the comment docs.

I spent some time with python3 and django 1.11. Looks like migrate auth and migrate behaves little better.

Looks like migrate auth and migrate behaves little better.

Sorry, i'm not sure i understand what you mean. Can you elaborate?

@AlejandroRivera:
No, disregard that. What's your Django version?
Looks like migrate auth and migrate --run-syncdb works fine for 1.8.x, but cause troubles for 1.11

My version is 1.9.13

This can be closed because #2425 was merged


How to Calibrate T265 Camera?

As the title suggests， thanks~

The documentation for the T265 states that it does not require calibration, as it is done on the production line.  The documentation has information on wheel odometry calibration though.
https://github.com/IntelRealSense/librealsense/blob/master/doc/t265.md#wheel-odometry-calibration-file-format
You may also be interested in an example program for using the camera intrinsics and extrinsics API for the T265 that was introduced in RealSense SDK version 2,.20.0
https://github.com/IntelRealSense/librealsense/tree/master/examples/ar-basic

The T265 comes precalibrated and does not require any user recalibration. If you want to integrate it with 3rd-party wheel-odometry you need to provide relevant extrnsics.

However, I have found that the pre-calibration parameter is incorrect.  This did not affect the quality of navigation, but I need to use other calibration parameters to use the camera.

@GigaFlopsis

However, I have found that the pre-calibration parameter is incorrect. This did not affect the quality of navigation

Would you mind to share, what problems you see with the current calibration? I'm also thinking the calibration is not optimal, since the drift is is pretty high at large scale, especially outdoors.

I'm only talking about calibrating the camera. Surprisingly, the navigation in the room works perfectly. I also compared the navigation with halens and it is similar. If we talk about navigation on the street, I have also had problems, I suspect that this is due to the small number of features at a short distance.

I also want to know how to calibrate the t265 imu. I have 2 t265, one is work well and one not . the not work one when I go ahead 1m, it walked forward 1 meter to the left (30 degrees from my positive direction，another good one shows right . how can I calibrate the imu ??

Hi every-one,
After my T265 had performed a few drops the results of the slam algorithm wasn't accurate. I think the imu calibration is not valid anymore.
@dorodnic  is there way to re-calibrate the imu ?

anyone calibrate the camera parameters,is it accurate?

I have tried to calibrate the T265 camera focusing on single fisheye,  with the realsense python camera rectify tuorial to verify my calibration results. However, no matter how many times I tried, the computed parameters and matrix cannot fully meet the product pre-inserted ones, which means I cannot undistort and rectify the captured images.
This is the captured image, and the following is the rectified image using my own calibrated parameters.


Adding spinner or searching

Is there on option to display a spinner or "Searching..." text till the refresh method finished?
I'm loading the data remotely and it can take 2-3 seconds till I get the data.

I need this functionality as well.  I need it to go one step further as well.  I'm using a remote data source to ajax in search results.  I need a way to display what amounts to a "no records found" message when the search returns no results.

@MBentley1976 can you provide an example?  Thinking this should be included in base.

Any updates?

Heres my solution....
/**
	 * Get Data products...will be Injected into products
	 */
	$scope.products = [];
	$scope.isProductFound  = false;
	$scope.getProducts = function(search) {

		if(!search) return;

		$scope.isProductLoaded = true; //for loading info when a search for product is conducted
		var params = {search: search, warehouse_id : $scope.warehouse.selectedWarehouseId};

	    return $http.get(SITE_URL + 'transaction/getProducts', {params: params} ).then(function(response) {

	    	$scope.products = response.data.data
	    	if (response.data.data.length < 1) {

	    		$scope.isProductFound  = true;
	    	} else {

	    		$scope.isProductFound  = false;
	    	}

	    	$scope.isProductLoaded = false;
	    });
    };

 <ui-select name="product" ng-model="product.no_int_product" on-select=onSelectProduct($item) theme="bootstrap" reset-search-input="true" style="width:500px">
    <ui-select-match placeholder="Search product by Name, Code..">{{$select.selected.product_name}}</ui-select-match>
    <ui-select-choices repeat="product.no_int_product as product in products"
                   refresh="getProducts($select.search)"
                    refresh-delay="500" style="width:500px">

      </ui-select-choices>
</ui-select>

Not found...

FWIW I think this is now here: https://github.com/angular-ui/ui-select/pull/1779

Closing as resolved by #1779


Add a testing section

As asked in https://github.com/straydogstudio/axlsx_rails/issues/31#issuecomment-371884437 :smile:

@straydogstudio Here is it. Obviously, any suggestions are welcomed.

Coverage remained the same at 98.161% when pulling 332261f9fadae96740f69b42057641c0b0a4eec2 on sly7-7:patch-2 into fe5ca6c98e3890887eef7549eeaccd43a95c4563 on straydogstudio:master.

@sly7-7 Thank you very much! I think it looks great. At some point I'll add some simple testing examples outside of RSpec also. No need for now though.

You're welcome :)

@straydogstudio Was it reopened on purpose ?

Ha. I closed it before I merged it. I didn't realize it would remove the merge button if I closed it. :-0

Ha, ok, so nvm, I thought it was still open. I just forgot that merging automatically close the PR.


Add R examples in "design" episode

this was beautifully solved in #108 but keeping this issue as a reminder to also add optional exercises in R (see #117 )

exercises in test-design are now available in python, julia, fortran and c++, but need to translate to R

I think apart from the end-to-end test examples, which are still Python only,
all tests design exercises now have an R version since #212 was merged


Default route of /fish is undesirable

https://github.com/canonical/canonicalwebteam.flask-base/blob/27370a7acdd72eec6716f3c8bf73250b7f8e5adf/canonicalwebteam/flask_base/app.py#L270
This ends up exposing a /fish route to every website / webapp that uses flask-base. It's undocumented, not easily removed.
As such it appears on (at least):

https://maas.io/fish
https://ubuntu.com/fish
https://canonical.com/fish
https://juju.is/fish
https://multipass.run/fish
https://anbox-cloud.io/fish
https://snapcraft.io/fish

What might be ideal here, is a route back to / from the TLD, if the route itself being asked for in the path, is not found, or alternately a page that suggests a search for the page they were looking for (eg: ?search=term=fish or similar)


Detail the error types in createMetric and batchCreateMetrics.

This change is

Current dependencies on/for this PR:

main

PR #1005   👈

PR #1004

This comment was auto-generated by Graphite.


fix use of x25519 private keys

This underscore is a typo and stops x25519 private keys from being used.

Hi! Actually, this was hacky use of undocumented private methods, API has been updated in new version of Cryptography. I'll be updating the master to use Cryptography >= 2.5 today, along with new release of noiseprotocol package. Thanks for PR!


NEW FEATURE:  Add theme option for automatic theme based on light or dark mode

Is your feature request related to a problem? Please describe.
not a problem
Describe the solution you'd like
would love to have support for themes to have automatic switching based on light or dark mode.
Describe alternatives you've considered
Idk
Additional context
idg if there's any.

Can't build a feature around this. But here is the hack.

Activity Graph 📈

just replace your username with mine.

Thank you. :)


⚠ Hartman Foundation has degraded performance

In 4fc525f, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 3464 ms

Resolved: Hartman Foundation performance has improved in 28b8db2.


Updated So that cursor changes when on Green full tick

Description
changed cursor when hovering over the full green arrow in
This PR fixes #615
Notes for Reviewers

[x] Yes, I signed my commits.

yay, @Reedham20! 🎈
Feel free to open an issue on the "undefined" item, if interested.

Thanks @leecalcote, I will surely do that


Documentation how to sign a LND/LNCLI release by a developer

PGP key file renamed otherwise it will not work with the verify-install.sh script.
Moreover added documentation how to actually sign LND releases.

@dstadulis great comments! Thanks for taking the time.


New Line Comma Indent Copies Comment Indent

The following code:
data Foo = Foo
  {
 -- Comment with wrong indent
, bar :: Int
  }

Does not reformat to this:
data Foo = Foo
  {
 -- Comment with wrong indent
  , bar :: Int
  }

Is this a bug or feature?

This should now be fixed.


comment out CI typescript cache

let's see how it impacts CI runtime
baseline:

before:
type-check takes ~35s
after:
type-check takes 1m 31s


practice lab vm extract needs password

Hello Sir, in this book author gave a for link for vm to practice that zip file needs a password to extract. Author didn't provide the password to extract in drive link and also in the book. So please provide the password to extract that file and also please rectify this issues. Thank you.

Hello,
Please use the password- MasteringNetworkForensics#2024


Field.regex will allow an invalid string to pass validation

Bug
The regex used to validate a string in pydantic.Field will allow a string that does not match the given pattern to pass without raising a validation error.
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.5.1
            pydantic compiled: True
                 install path: /home/kieran/projects/bimacademy/buildstream/prototypes/backend-admin/.venv/lib/python3.8/site-packages/pydantic
               python version: 3.8.2 (default, Feb 26 2020, 02:56:10)  [GCC 7.4.0]
                     platform: Linux-4.15.0-99-generic-x86_64-with-glibc2.27
     optional deps. installed: []

This simple regex checks for any number of lowercase letters ("a - z") and spaces.
In the example below, the test="this Should Fail" should raise a validation error. It does not.  I think this is because it is using re.match instead if re.fullmatch.
import re
import pydantic

class Test(pydantic.BaseModel):
    test: str = pydantic.Field(regex="[a-z ]+", description="Allow lower case letters and spaces")

try:
    Test(test="this should pass")
except pydantic.ValidationError:
    print("This should have passed!")

try:
    Test(test="this Should Fail")
except pydantic.ValidationError:
    print("This should have failed!")

assert bool(re.fullmatch("[a-z ]+", "this should pass"))
assert not bool(re.fullmatch("[a-z ]+", "this Should Fail"))

Happy to submit a PR! (Looks like a nice easy one)

Not a bug, duplicate of #1396.


Error when installing on M1 Mac

I get the following error when attempting to install on my Mac (2020 Pro M1):
OSError: dlopen(./libbass.so, 0x000A): tried: '/opt/homebrew/Caskroom/miniforge/base/lib/./libbass.so' (no such file), '/opt/homebrew/Caskroom/miniforge/base/lib/./libbass.so' (no such file), '/opt/homebrew/Caskroom/miniforge/base/lib/python3.10/lib-dynload/../.././libbass.so' (no such file), '/opt/homebrew/Caskroom/miniforge/base/lib/./libbass.so' (no such file), '/opt/homebrew/Caskroom/miniforge/base/bin/../lib/./libbass.so' (no such file), './libbass.so' (no such file), '/Users/sysadmin/shellrhythm/libbass.so' (no such file)

Any idea how to fix it?

This is something related to libbass.so (the BASS library) not being found in any of the given folders in the error, however I have never tried installing shellrhythm on Mac yet (due to not owning a Mac myself)
I guess you could try following ./docs/PYBASS3_LINUX.md, and if you did try copying that libbass.so to any of the folders

Got it to work! I had to go into the PyBass3 source code and change it to try to load the .dylib file instead of the .so file (downloaded the dynamic libs from the un4seen site).
Maybe I should open an issue or PR on the PyBass3 repo?

Probably, I should also open a PR for the Linux part


Patching an older change shows above head in xl

Take an old CL which is synced to a previous point in the repository's history.
Patching it into a local repository up to date with origin/main causes a node to be created in the commit tree that appears newer than head but is actually older.
Expected: Head is moved to the parent of the patched change and an unowned branch is created for the commit head used to point to.

Fixed with a30f856cb5437ffb4a0d2dddae3a7f96e10cf5b6