Specify local file paths outside the project folder

Hello,
Everything works fine when I'm importing tracks from files located in the current working directory (where the ipynb is located) or in subfolders. I'm specifying tracks like so:
b.load_track(
{
    "name": "mybed",
    "type":"annotation",
    "url": "files/my.bed",
    "format":"bed"
}
)

I would like to be able to load tracks from any place in my local disk (using relative or absolute paths) but I did not succeed so far. I tried:
"url":"files/../my.bed"
"url":"files/absolute/path/to/my.bed"
"url":"localhost:8888/absolute/path/to/my.bed"
After checking, https://github.com/igvteam/igv.js/wiki/Tracks-2.0, I don't see how to specify local files outside the working directory. Is there a dedicated syntax for this ?
Thanks for the help and for a great jupyter extension !

Hi @khourhin .  Technically you can't load local files via a path into any javascript browser, its not allowed.  What's happening with jupyter is those paths are being translated to URLs and loaded from the Jupyter web server running on your machine.     I'm not intimately familar with the Jupyter path magic, but you should be able to get it to work via symbolic links from a Jupyter directory.
The last URL you posted should have worked (localhost:8888...).   If you paste that URL into a web browser address bar does it load you bed?

Thanks for the help ! Indeed, I could use symbolic links.
Pasted directly in the web browser, this last URL returns a jupyter 404.
I'll look a little bit more into this and let you know if I find something else.

A 404 means "not found".    You need to construct a URL that works in the web browser address bar, after that it should work as a url in IGV.    I'm not a jupyter expert, but I would expect the URL to start with something like  localhost:8888/files/....      If you want to use arbitrary directories you could try adding symbolic links to them from the directory that is working, i.e. the directory where  "my.bed" is located.   If the directory link  name is "foo"  then  localhost:8888/files/foo.bed.
Do some reading about hosting static files from a Jupyter server, that is really what you are trying to do.

Yes, sorry for the late reply. I intended to dig a little more on file serving with Jupyter and put some guidance here but so far I had no time for this. Hopefully I will at some point ... Thanks again for the help !

I was getting the 404 GET error with the latest igv-jupyter version, which doesn't require a URL but works with the file paths, and solved this by putting all my files downstream of the working directory specified when loading JupyterLab.


Graph zoom

Sometimes, the graph zoom seems to jump. Exact circumstances not found yet.

This bug appears to be a graphic lag rather than a logic error. Will reopen if observed again.


Polling for tasks not working

Hi,
I have a single node set up of conductor back-end such as Dynomite, Redis and Elasticsearch on an Amazon EC2 instance. I have conductor server (jar file) running on another EC2 instance. Both instances are in the same availability zone. Creation of task definition, workflow definition and retrieval of in_progress tasks (/api/tasks/in_progress/) are working. However, polling for tasks is not working, None of the /api/tasks/poll/ and /api/tasks/poll/batch/ is working. They return success without any response body.
I have looked into existing posts on this matter but none of those have helped me yet while those resolved the same issue faced by others. I am not sure if I am missing something in my configurations. Below are the configuration for each of Dynomite, Elastic Search and Conductor server.
Redis
Redis is running on port 6379.
Dynomite Config YAML file
dyn_o_mite:
dyn_listen: <IP_ADDRESS>:8101
data_store: 0
listen: <IP_ADDRESS>:8102
dyn_seed_provider: simple_provider
rack: us-east-1b
servers:

<IP_ADDRESS>:6379:1
tokens: 437425602
stats_listen: <IP_ADDRESS>:22222

Elastic Search
cluster.name: cosmos-elasticsearch-cluster
node.name: cosmos-elasticsearch-node-1
network.host: <IP_ADDRESS>
Conductor server properties
db=dynomite
workflow.dynomite.cluster.hosts=<IP_ADDRESS>:8102:us-east-1b
workflow.dynomite.cluster.name=dyn_o_mite
workflow.dynomite.connection.maxConnsPerHost=31
workflow.namespace.prefix=conductor
workflow.namespace.queue.prefix=conductor_queues
queues.dynomite.threads=10
queues.dynomite.nonQuorum.port=6379
workflow.elasticsearch.url=<IP_ADDRESS>:9300
workflow.elasticsearch.index.name=cosmos-elasticsearch-cluster
EC2_AVAILABILTY_ZONE=us-east-1b

Do you have any logs from the server that you can share?

The issue is resolved. The EC2_AVAILABILTY_ZONE property has a typo. I corrected it in my conductor server properties file and the problem was gone. I should have closed the issue by now. Closing it now. Thanks.

Polling for tasks is not working when i am running server in In-Memory.

@parmeshdayal Try opening a new ticket with more info with regards to your issue.

@parmeshdayal Try opening a new ticket with more info with regards to your issue.


W10 MS support in Azure but outside of WVD

[Enter feedback here]
Can someone add a statement or clarification in this page if W10 MS running in Azure VM is supported -assuming Licensing entitlement is covered- IF not used with Virtual Desktop Service?
It is technically possible to use W10 MS as a standalone IaaS VM in Azure and let multiple users connect directly outside of WVD control plane.
Thanks
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7f267d00-6800-2bf9-122d-402b60672782
Version Independent ID: 8ba82e23-f5ec-2b4d-d30b-c5968609246b
Content: Windows 10 Enterprise multi-session FAQ - Azure
Content Source: articles/virtual-desktop/windows-10-multisession-faq.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@xelizondo
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

@xelizondo
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

@xelizondo Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.We will update you accordingly.

@xelizondo Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.We will update you accordingly.

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

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

It's not for an Azure Virtual Desktop to say whether this scenario is allowed outside of Azure Virtual Desktop. The articles for Azure Virtual Desktop have that focus only.  It would be better to reach out to your licensing representative.
#please-close

This took 1+ year to turn around. By now W10/11 multi-session is supported outside of Azure particularly or specifically to run on Azure Stack HCI for AVD.
Can this be updated in this page?
Can I run Windows 10 Enterprise multi-session on-premises?
Windows 10 Enterprise multi-session can't run in on-premises production environments because it's optimized for the Azure Virtual Desktop service for Azure. It's against the licensing agreement to run Windows 10 Enterprise multi-session outside of Azure for production purposes. Windows 10 Enterprise multi-session won't activate against on-premises Key Management Services (KMS).


Faulty builds for browser in pragmatist 2.3.0

When building for browser with pragmatist build --browser, in the dist/index.js files it's adding this line:
var _register = require('/Users/channi/Documents/Work/applaudience/showtimes-components/react-showtimes-calendar/node_modules/source-map-support/register.js');

Which is breaking the published packages.

Using mocha -r option instead of babel plugin as suggested in https://github.com/knpwrs/gulp-spawn-mocha/issues/25.

Version 2.3.1 published to NPM.


Migrating to the pugjs org

Would you be interested in moving this repo into the pugjs org?  Still with you maintaining it, but it might give this more visibility as the "official" gulp pug implementation.

Sure! I attempted to transfer, but it said I do not have permission for the pugjs organization.

Oops, you should now have a pending invitation to join the pugjs team.  Once you've accepted that you should be able to transfer the repo.

@ForbesLindesay I have moved the package. :)


Initial implementation of USB-based eye tracker using esp32s3 boards.

Moving the #44 PR to feature/usb_support branch as instructed.

LGTM Thank you so much for getting USB to work, very much appreciated!


Improve the description of "limits" augment.

Turns out "limits" augment doesn't mean it will download exactly "limits" amount of pictures. (because errors happened.)
It would also be nice if you add an augments that make counter ignores errors.
(For example I failed to download 10th picture, instead of counting the next picture as 11th, make it count as 10th picture.)

Thanks! I have added a note in the documentation within the limit argument for transparancy. This is only the case where the limit is set to 100 and there are errors.
If you set a limit to less than 100 or more than 100, it will get you the exact number of images (even if there are errors).
Closing this for now.


chore(dev/bench): add a simple python benchmark script for ADBC

Just a starting point to hopefully continue building up some benchmarking suite stuff.

Does it not like that I'm adding the directory that I'm referring to in the title?

Yeah, I haven't gotten it to get the right checkout in that action...


iSic Editing 11 June

18 Inscriptions

@JonPrag changes now made, and my first attempt at adding metadata for a handful of ILipara inscriptions now added.


Lang he il

Pull Request Test Coverage Report for Build 476

1 of 22 (4.55%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 45.794%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/component/src/Localization/Localize.js
1
3
33.33%

packages/component/src/Localization/he-IL.js
0
19
0.0%

Totals

Change from base Build 468:
-0.7%

Covered Lines:
712

Relevant Lines:
1388

💛  - Coveralls


_WEBPACK_IMPORTED_MODULE xxxxx is not a function

(Was originally opened here: https://github.com/angular/angular/issues/15866)
I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question

Current behavior
Webpack (or Typescript) seems to strip static methods on inheritance. I'm posting here because the problem manifests in the context of Angular.
We're using angularfire2, and more specifically FirebaseListObservable.
FirebaseListObservable extends Observable, so FirebaseListObservable should have the static function of.
However, when I do ng serve I get the following error:
__WEBPACK_IMPORTED_MODULE_1_angularfire2__.a.of is not a function
    at AppComponent.webpackJsonp.132.AppComponent.ngOnInit

Expected behavior
Looking through angularfire2's code for FirebaseListObservable, they extend Observable like expected, and thus of should be available. So I'm not thinking it's their problem?
Minimal reproduction of the problem with instructions
ng new bug
npm install angularfire2 --save
npm install firebase --save
npm install rxjs --save

app.component.ts:
import { Component, OnInit } from '@angular/core';

import { FirebaseListObservable } from 'angularfire2';
import { Observable } from 'rxjs';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css']
})
export class AppComponent implements OnInit {
  title = 'app works!';

  ngOnInit() {
    FirebaseListObservable.of({}).subscribe();
  }
}

ng serve --open
Please tell us about your environment:

Angular version: 4.0.1

Browser: [all]

Language: TypeScript 2.2.2

Node (for AoT issues): node --version =   7.8.0

You need to import any operators that will be used.
For of:
import 'rxjs/add/observable/of';

We already tried that, didn't do anything, sadly.

Looking at your code again, using FirebaseListObservable in that way really doesn't make sense.  You're creating a custom observable with a particular value.  Observable itself would be the appropriate class to use.

Hm, well we need the custom methods (push etc.) that only FirebaseListObservable provides. Since its heavily promise-based, we need to do certain tricks to make it work in Observable chains.

Those methods require the presence of an AngularFire instance.  They would not function in isolation.
See the class implementation;  The AngularFire library would need to create a custom of function that dummies out the underlying infrastructure for that to work.

Closing as above ☝

I don't think its intuitive that the of method is not inherited. Also, typescript allows me to transpile it without errors.

If you look at how the function is added via the import (see here), the type information is updated but the function is patched onto Observable at runtime.
Even if it did execute, it would not return a FirebaseListObservable.

@clydin  well that worked for me solving importing for plugin scripts for leafletjs but i don't understand why ? in angular 7 the exact same setup was working just fine, in upgrade to 8 i had to look for a solution for this problem !


Red things

Hello, I uptade my windows 11 on last update (20/11/2021). And after i finish it. It shows a red things in my program.

Is this setting checked for you (the default)?

If yes, do the "red things" disappear?
If they do, it looks like your program file contains control characters.

I have it checked, but red things not disappear. And it started doing random after update windows to newest for windows11

I meant to write "do they disappear if you uncheck it?"

When i uncheck, it disappear. But there is some bugy things

If you load your program file into a different editor (maybe Notepad) does it look OK? If not, the problem isn't specific to VS Code.

Oh, its same in notepad.
Can it be because i install new update to windows 11  ?

You have some control characters in this file. Maybe it got corrupted somehow. Or they were added by some other program.


Release v4.0.7 (with Sigma v4.0.5 and ergo-wallet v4.0.23)

In this release:

#123
#126
#128
#129
#133

#132 is also included


Won't Open

Scott, 1.04b(1) application won't run after extraction.

Resolved.  Re-loaded a second time.


infinity booster card

hi, the infinity booster card doesn't work in the nether? Mod Pack AQM2,

It should work in any Dimension.
Make sure your me system is loaded, otherwise it won't work.

It works for me


[FIX] double registration of the default namespace

Fixes: https://github.com/SAP/openui5/issues/2294
Replaced spaces with tabs for indentation according to the coding guideline
Removed controllerName="..." as it's irrelevant to the topic

@codeworrior ; Can you please check? Thank you!

I just squashed several update commits into a single one to make it easier to merge. Let me know if there is anything else I could / should do.
PS: the "Removed double quotes (...)" in the commit message in https://github.com/SAP/openui5-docs/pull/4/commits/66f45aba946fa667e32017fc8ea433d2b0d15575 refers to https://github.com/SAP/openui5-docs/pull/4#discussion_r314974906. Missed to add the link.

I just squashed several update commits into a single one to make it easier to merge. Let me know if there is anything else I could / should do.

Thanks! No need to to anything else. I'll transfer the changes to the documentation and then merge the pull request.

Changes done in source; should be visible as of tomorrow.

Done with https://github.com/SAP/openui5-docs/commit/fc33505965f8fcf901c891da5c1b8c8458fc92d4#diff-1401d04a516fd983dc461f72943c059a
Guess I just close this PR then..


doppelgreener: Blacklist CouponLocate.com

doppelgreener requests the blacklist of the website CouponLocate.com. See the Metasmoke search here

!!/approve


Use copy-mode -q to leave copy mode before running a command

Fixes #230
Four years ago tmux introduced a -q flag to copy-mode: https://github.com/tmux/tmux/commit/06c3079d66929e0c71575e877098fc533ae5f4a5
Using copy-mode -q on the runner instead of relying on the q mapping, which could be remapped, should be more reliable.
This updates the default value of VimuxResetSequence from "q C-u" to simply "C-u".
This will pose problems for users on a tmux version older than the introduction of the -q flag (before version 3.2a). We could perhaps catch the unknown flag error and try using q in those cases.

I thought it would be prudent to put in a fallback for older versions so the runner prompt is still cleared. I haven't found the precise cause of this issue in tmux. Since we haven't changed our logic it seems likely that this stems from a recent tmux release and that the q C-u reset sequence used to work properly.

@alerque Looks like I need a fresh approval, sorry about that!

No worries, tweaking things in PRs to get them as right as we know how before merging is the right thing to do. That being said I've watched your contributions for a while and don't think there is any need for the friction of waiting for a rubber stamp from me. I just upgraded your access to this repo to Admin level so you should be able to skip the 1-review-needed requirement. Have fun.
Everybody else: please welcome your new overlord!


Eat Your Own Dog Food

It would be good have Retrial using Retrial itself to verify its dependencies :).

Makes sense. Assuming the dependencies aren't already compromised, the threat is shifted to GitHub which would make it harder for an adversary. I'll look into this soon.


imake should be rebottled against gcc-6

https://github.com/Homebrew/homebrew-science/issues/3735#issuecomment-225682417

@217

Fixed via https://github.com/Homebrew/homebrew-x11/commit/f5872d123607fdcc296c9a2161350b10f8611a61.


Use the API : [parse(File in, String charsetName, String baseUri)]

hi,
i don't think it is a issue, but i don't know that why:

input html file.

<!DOCTYPE html>

        <title>JsoupInputTest</title>
        <meta charset="UTF-8">

test parsing input file by jsoup

        s1 test

code

    public static void inputTest() throws IOException{

        String fileName = "../inputTest.html";
        File in = new File(fileName);
        Document doc = Jsoup.parse(in, "UTF-8", "http://example.com/input/");

        System.out.println(doc.select("img").first().absUrl("src"));
        System.out.println(doc.select("a[href]").first().absUrl("href"));

        System.out.println("====================================");

        System.out.println(doc.html());
    }

output

http://example.com/img.jpg
http://example.com/a.jpg
====================================
<!doctype html>

  <title>JsoupInputTest</title>
  <meta charset="UTF-8">

   test parsing input file by jsoup

  s1 test

Q1: i think it will output http://example.com/input/img.jpg, but actually it outputd http://example.com/img.jpg
Q2: the output html is   , it 's not

Q1: both of those are absolute URLs with implicit domains -- the URL starts with a single slash. So only the hostname from the base HREF (the third argument to Jsoup.parse) is used. The output is correct.
See https://developer.mozilla.org/en-US/docs/Learn/Common_questions/What_is_a_URL#Absolute_URLs_vs_relative_URLs for some more details on how URLs are made absolute.
Q2: calling absUrl() doesn't change the value in the DOM; it's calculating the absolute URL, not updating it.
Also please use Stack Overflow for questions, it's a better forum.

@jhy
thanks a lot,i got it


Generated kubeadm docs for 1.11 are inconsistently formatted during release cycle

This is a...

[ ] Feature Request
[X] Bug Report

Problem:
kubeadm docs generated during the 1.11 release cycle by anyone other than @tengqm do not include the HTML formatting that was added for 1.10. This is because https://github.com/kubernetes-incubator/reference-docs/pull/47 is not merged.
Proposed Solution:
Life with mismatched formatting until 1.11 release. sig-docs regenerates all generated docs per release. At release time, if the PR is merged we'll have pretty tables for everything. If it isn't merged, we'll have less pretty lists for everything. But the inconsistencies will be gone.
Page to Update:
includes for siblings of https://kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm/
(source at https://github.com/kubernetes/website/tree/master/content/en/docs/reference/setup-tools/kubeadm/generated)

@Bradamant3 Was this addressed by #8617? If so, could you close this issue?


Use async_forward_entry_setups function

Utilize async_forward_entry_setups instead of async_setup_platforms

Starting with Home Assistant 2023.3, any integration using the old function will fail to start.

Thanks @iMicknl. Just a heads up: Starting with Home Assistant 2023.2, support for Python 3.9 is dropped so it may be worth while dropping building 3.9 in the tests / increasing the minimum requirement of HA to 2023.2 for the next release.

Makes sense, should also add Python 3.11 to CI/CD. Unfortunately I will be still very busy for 1 a 2 weeks, so it will be a bit delayed.

No rush at all. The integration won't break until 2023.3, so you have time.

Please rebase on main for fixed CI/CD.


It keeps telling me to fix my config...and I fixed it twice.

Context
I just wanted to see how far I can go with docker on my dedicated server, and wanted to install a mailserver, and came across this one. So I followed the very steps of the readme, but I just keep getting the same message... I even went as far as to use env_file within the docker-compose.yml file to explicitly point at the .env file...but that did not help, at all.
Expected Behavior
I expected the mailserver to just boot and probably greet me with a nice "You now have a mailserver running" message.
Actual Behavior
root@ingwie-io:~/mailserver# docker-compose up
WARNING: The POSTGREY_AUTO_WHITELIST_CLIENTS variable is not set. Defaulting to a blank string.
mail is up-to-date
Attaching to mail
mail    | 2018-11-05 21:51:44,691 CRIT Supervisor running as root (no user in config file)
mail    | 2018-11-05 21:51:44,691 INFO Included extra file "/etc/supervisor/conf.d/saslauth.conf" during parsing
mail    | 2018-11-05 21:51:44,691 INFO Included extra file "/etc/supervisor/conf.d/supervisor-app.conf" during parsing
mail    | 2018-11-05 21:51:44,698 INFO RPC interface 'supervisor' initialized
mail    | 2018-11-05 21:51:44,698 CRIT Server 'unix_http_server' running without any HTTP authentication checking
mail    | 2018-11-05 21:51:44,698 INFO supervisord started with pid 1
mail    | 2018-11-05 21:51:45,701 INFO spawned: 'mailserver' with pid 10
mail    |
mail    | #
mail    | 2018-11-05 21:51:45,726 INFO success: mailserver entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
mail    | #
mail    | # docker-mailserver
mail    | #
mail    | #
mail    |
mail    | Initializing setup
mail    | Checking configuration
mail    |   * Setting hostname/domainname is required
mail    | 2018-11-05 21:51:45,741 WARN received SIGTERM indicating exit request
mail    | 2018-11-05 21:51:45,742 INFO waiting for mailserver to die
mail    | Error: Please fix your configuration. Exiting...
mail    | 2018-11-05 21:51:45,742 INFO stopped: mailserver (exit status 1)

This actually shows up ~5 times in total before the process actually exits.
Possible Fix
Clarify in the README the exact steps to not run into this. I am very certain I did all the steps mentioned in there.
Steps to Reproduce
Follow the very steps in the README:

Let docker get the image.
Download the tools (setup.sh, .env, docker-compose.yml)
Modify .env
Run docker-compuse up and see what happens.

Your Environment

Amount of RAM available:

root@ingwie-io:~/mailserver# free -m
              total        used        free      shared  buff/cache   available
Mem:           7881        4255         247         139        3379        3191
Swap:           510          74         436

Mailserver version used: None?
Docker version used:

root@ingwie-io:~/mailserver# docker --version
Docker version 18.06.1-ce, build e68fc7a
root@ingwie-io:~/mailserver# docker-compose --version
docker-compose version 1.8.0, build unknown

Environment settings relevant to the config: None that I am aware of.
Any relevant stack traces ("Full trace" preferred): See above.

What are your HOSTNAME, DOMAINNAME and OVERRIDE_HOSTNAME settings?
If you did not set OVERRIDE_HOSTNAME, what is the output of:
docker-compose run --rm mail hostname -f

This should return a FQDN.

I don't think .env files support variable resolution at all. Try
OVERRIDE_HOSTNAME=ingwie.io

in the .env file.

root@ingwie-io:~/mailserver# docker-compose up
WARNING: The POSTGREY_AUTO_WHITELIST_CLIENTS variable is not set. Defaulting to a blank string.
Recreating mail
Attaching to mail
mail    | 2018-11-07 17:11:04,748 CRIT Supervisor running as root (no user in config file)
mail    | 2018-11-07 17:11:04,748 INFO Included extra file "/etc/supervisor/conf.d/saslauth.conf" during parsing
mail    | 2018-11-07 17:11:04,748 INFO Included extra file "/etc/supervisor/conf.d/supervisor-app.conf" during parsing
mail    | 2018-11-07 17:11:04,765 INFO RPC interface 'supervisor' initialized
mail    | 2018-11-07 17:11:04,765 CRIT Server 'unix_http_server' running without any HTTP authentication checking
mail    | 2018-11-07 17:11:04,766 INFO supervisord started with pid 1
mail    | 2018-11-07 17:11:05,768 INFO spawned: 'mailserver' with pid 10
mail    |
mail    | 2018-11-07 17:11:05,957 INFO success: mailserver entered RUNNING state, process has stayed up for > than 0 seconds (startsecs)
mail    | #
mail    | #
mail    | # docker-mailserver
mail    | #
mail    | #
mail    |
mail    | Initializing setup
mail    | Checking configuration
mail    | Configuring mail server
mail    | grep: /tmp/docker-mailserver/postfix-accounts.cf: No such file or directory
mail    | Error: Unless using LDAP, you need at least 1 email account to start the server.
mail    | Error: Please fix your configuration. Exiting...
mail    | 2018-11-07 17:11:06,475 INFO exited: mailserver (exit status 1; not expected)

Well at least it passes the first check now. I guess I use setup.sh to add my account now, right?

Yes, it's not very intuitive, but that's how it works right now

Also make setting up SSL more obvious. that is what I am running into now at the moment… but, should not be too hard. I’ll figure it out, somehow.

Am 07.11.2018 um 18:26 schrieb Daniel Panteleit<EMAIL_ADDRESS>@johansmitsnl https://github.com/johansmitsnl I think about streamlining setup.sh to support only docker-compose.yml. That way the setup would be more intuitive:
We can easily create accounts before starting a failing server
We don't need to search awkwardly for the correct image/container/volume configuration in the script
What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tomav/docker-mailserver/issues/1075#issuecomment-436706805, or mute the thread https://github.com/notifications/unsubscribe-auth/ACwnlF9SpPzM1cRv-lyjJJgg05TORwXwks5usxesgaJpZM4YPWZc.

Also… this is kinda odd.
I tried sending myself an email through GMail and it returned this:
550 5.1.1<EMAIL_ADDRESS>Recipient address rejected: User unknown in local recipient table
This E-Mail absolutely exists, I am logging into this from the same program from which I am sending… So, why is this address unknown?

Am 07.11.2018 um 18:31 schrieb Kevin Ingwie Phoenix Ingwersen<EMAIL_ADDRESS>Also make setting up SSL more obvious. that is what I am running into now at the moment… but, should not be too hard. I’ll figure it out, somehow.

Am 07.11.2018 um 18:26 schrieb Daniel Panteleit<EMAIL_ADDRESS>@johansmitsnl I think about streamlining setup.sh to support only docker-compose.yml. That way the setup would be more intuitive:
• We can easily create accounts before starting a failing server
• We don't need to search awkwardly for the correct image/container/volume configuration in the script
What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Are you sure you created the account<EMAIL_ADDRESS>and not just "o.o"?
What does the mailserver log say?

I did: ./setup.sh email add<EMAIL_ADDRESS>****
The mail log reports the same error as the mailer daemon response I get... But, where exactly do I look for the email creation messages in the log?

There is no output of the account creation in the log. You can check postfix-accounts.cf in your config directory. It should contain a line starting with<EMAIL_ADDRESS>Did you restart the container after creating the account?

I did just now, and I still get the same error.

Am 07.11.2018 um 19:44 schrieb Daniel Panteleit<EMAIL_ADDRESS>There is no output of the account creation in the log. You can check postfix-accounts.cf in your config directory. It should contain a line starting with<EMAIL_ADDRESS>Did you restart the container after creating the account?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tomav/docker-mailserver/issues/1075#issuecomment-436733727, or mute the thread https://github.com/notifications/unsubscribe-auth/ACwnlDRECVgYc87rPG-DrR9MLwWVwzpyks5usyopgaJpZM4YPWZc.

This warnings may be the problem:
postfix/trivial-rewrite[1294]: warning: do not list domain ingwie.io in BOTH mydestination and virtual_mailbox_domains

But I don't know postifx that good. You could post your .env and docker-compose.yml file just in case, but I guess someone else needs to take over.

Here you are:
https://gist.github.com/IngwiePhoenix/e7cf04275b08a5657cdaa955ea5d20ca
Hopefuly someone can help... I am pretty close to having this fixed, so I'd prefer to not having to drop this all now ^^

I think you may have to set up a subdomain for you mail server (like mail.ingwie.io) then set HOSTNAME to mail and DOMAINNAME to ingwie.io (then you won't have to set OVERRIDE_HOSTNAME at all). Your MX record should then point to mail.ingwie.io.
At least that's how I did it. I guess postfix doesn't like that your HOSTNAME and DOMAINNAME is the same.

The postfix issue is a duplicate of what was experienced in #968.
@danielpanteleit I understand that it makes the setup.sh script more complicated, but please don't depend on docker-compose exclusively. I deploy the mailserver container using ansible and it works wonderfully, I'd have a harder time integrating it into my configuration management structure if it did rely on docker-compose.


last.FM API discovery/access (Matt)

Got most of the API backend completed.  The app will search via artist, display artwork and have access to artist pages, playlists and bio page.


[otbn] Keeping ISS/RTL in sync

https://github.com/lowRISC/opentitan/pull/3577 and https://github.com/lowRISC/opentitan/pull/3471 have raised the issue of what requirements we want to place on keeping otbnsim and the RTL in sync in terms of what they implement.
I think there's two main issues:

Do you allow something to be implemented in RTL and merged in without corresponding simulator support?

Allowing a feature to added in RTL without it being implemented in the simulator isn't something that you want long-term though I'd argue it's reasonable in the short term in some cases. Where adding support is an easy change then it can be done as part of the RTL PR but if something more complex crops up it's not unreasonable to do them separately. You could of course say the simulator PR must come first, allowing the simulator to implement functionality RTL doesn't yet have but not allowing the opposite.

What is the smoke test for?

Up til recently it's purely been testing RTL along with some of the tooling. Now the model is integrated it partially tests that too (though in a limited fashion as we don't have much comparison between the model and RTL yet, once that is complete it will test the model to the same extent it tests the RTL). I think it's a good idea for it to be testing both model and RTL though if your answer to 1. is 'yes' you will have a situation where there's an RTL feature you might like to test in the smoke but can't because it'll fail against the model. As anytime we have an RTL feature that isn't supported in the model we would intend to update the simulator quickly I think saying smoke test must always be run with model included/checking enabled is reasonable and just leave any RTL only feature as un-tested in CI.

Copying and pasting from my comments in 3577:

First comment
I think the question about whether we take RTL with no model is above my pay grade :-) But I think it's a pretty strange thing to do. We've presumably specced the design. Updating the simulator is way easier than updating the RTL, so I think we should do that first and add simple tests (smoke or otherwise) as the RTL is updated.

Second comment
Forcing the model to be enabled will only break CI if we both

add / modify an instruction that doesn't match the simulator
add lines to the smoke test that use the instruction

I think we just shouldn't do that.

I should add: From my point of view, "is the RTL correct?" is shorthand for "does the RTL do the same thing as the model?".
Obviously, when the ISS is super-shonky or has massive missing features, there's an argument for running ahead. But it's not all that bad now: we should be able to fix up corner cases in a couple of hours, so I don't see a problem with gating RTL changes that long.

My 2c:

We should work hard to keep simulation and RTL in sync. (There's only limited value to a model that models something different than the RTL.) Typically that means get the ISS changes in first, or together with the RTL.
Life isn't black and white. There are situations where we RTL and model might diverge for a good reason. We should retain a (reasonably easy) way to let that happen, e.g. by disabling a CI job, or setting a parameter. Doing so as an explicit action can unblock us while avoiding unintended diversion between model and RTL.
I don't expect us to run into that situation very often. Once we're through with an initial implementation of the spec we shouldn't have ISS-visible diversions very often.

Just probing some assumptions: do we regard "spec as golden reference" and both ISS & RTL as derived implementations?    Is the "preferred" simulation target the ISS or the RTL?
I am used to having the ISS & RTL sort of cross-check each other for implementation errors (especially when implemented by different people who might interpret ~ambiguous spec language differently) and also cross-check the spec for architectural/construction errors.
I've also seen a situation where the ISS is more of an architectural reference model, perhaps cycle-accurate or even transactional in nature, and there is another verification-team-created, very detailed design reference model allowing for checking of internal design state/behavior against the RTL.  This is also fine, it just affects how we all think about ensuring correctness at the different levels of description/implementation.

I think we're aiming for "spec + ISS" as golden reference. We can't snip the ISS out completely because we're planning to auto-generate the pseudo-code operation for each instruction from the ISS, so the contents of the ISS definitely end up in the spec. As you said, this also has the advantage that you can run it.
Yes, about the ISS/RTL cross check: my take is that we're implementing the two separately (obviously! They are in different languages!), so I'd expect the RTL implementation to find some more bugs in the ISS and lots more corners in the spec.
I've also just pushed up some code (#3613) to add simple testing for the ISS. My plan is that each instruction will have an associated smoke test that gives examples of what happens in each of the corner cases.

From the RTL side I've been implementing things independently of the ISS, in that I read the spec and implement the instruction as it specifies. My bring-up testing for any feature involves manually working out what the results of an operation should be based upon the spec and then checking the RTL does indeed do that. This is what has resulted in the RTL smoke test (a short program that runs various instructions and has expected register values at the end, those values were calculated by me, not just by running the smoke test in the ISS).
Now that the ISS is (almost) at the point we can run the ISS and RTL in parallel and cross check register values at every clock that will be run as the final stage of my RTL bring-up. At that point if I encounter any differences I won't just assume the ISS is correct and alter RTL to match but rather go back to the spec and look at the ISS code to understand what the difference is and where it's come from. For anything that isn't a clear error on either the RTL or ISS side I'll open an issue to discuss, otherwise a PR to fix (which itself can be used for discussion).
So I think between @rswarbrick and me writing the ISS and RTL respectively, @felixmiller writing software and @imphil reviewing as well as doings parts of the ISS and RTL work we have a good degree of cross checking.
Long-term, as @rswarbrick says, spec + ISS is golden reference in that the spec will be partially auto-generated from ISS source. For now with everything under development (those the ISA itself is pretty stable, various small things still being discussed, potential for further change with CFI and if we need anything new for performance reasons) I don't think anything is really regarded as the golden reference (in that if it says its the way something is done, everything must match it). This is mostly because the spec doesn't delve into all the corner cases yet (e.g. see https://github.com/lowRISC/opentitan/issues/3587), these are being found and specified as we come across them as we write implementations.

I wonder if once we think we're at a point the spec can be considered our golden reference we should do a 'corner case review' where go through all the instruction descriptions one by one and think about corner cases just to double check we've not missed anything.

Thanks for the comments @GregAC and @rswarbrick - this helps me to understand the strategy you are going for.  Spec + ISS as golden seems pretty reasonable, especially with the ISS smoke testing layered on top.
I think the idea of a 'corner case review' is a good one - I'm happy to participate if you think you would find another pair of eyes helpful.  But I'll defer to you collectively to decide when you've reached that golden reference point for the spec.

I guess this question has been answered in favor of keeping RTL and ISS always in sync, and that's what we test for. If there's a need to "de-sync" the two for an intermediate period, the recommended solution is to disable the relevant tests.

I guess this question has been answered in favor of keeping RTL and ISS always in sync, and that's what we test for. If there's a need to "de-sync" the two for an intermediate period, the recommended solution is to disable the relevant tests.


[Develop] [Part 2 of 2] Constants loader and Config loader

This is a twin of #16187 against develop.
Conflict with develop originated in differences of dune file formatting.

!ci-build-me

https://github.com/MinaProtocol/mina/pull/16187#pullrequestreview-2398898908


If a version already exist add ability to bump to the next version instead of panicking out.

I'm not sure if this have been discussed before but i wished this package provided a way to bump package's version the next version if the present already exist on the registry. Some packages does it very well like this one  for javascript community.
If you like this idea. I'm willing to open a pull request addressing it.

In what scenarios is this happening?
Is this just wanting to run cargo release -x?  Or is it that the registry and the repo are out of sync with what versions are released?

@epage I was suggesting, instead of it erroring out when package already exists in the registry like the following ...

To add a flag to enable bumping to the next version automatically.

Wouldn't that be cargo release -x patch?

I didn't know that was a thing. Thanks!!


aria-label & aria-labelledby

For the sake of accessibility, would be great to be able to pass in props for aria-label and aria-labelledby.

All non-reserved props are passed through to the underlaying textarea, so you can just use any textarea specific props for this component, including aria.


[UI] list of instance types in the Rancher UI for RKE1 EC2 is stale

similar to https://github.com/rancher/rancher/issues/22907 from 2019 I'd like to use newer instance types in the "Instance Type" field when using the Node Group template.
I've forked the UI repo and have a PR at https://github.com/rancher/ui/pull/5178.
The TLDR is I've tried to update Intel/AMD instance types but have ignored including Graviton instances because Rancher docs say ARM64 support is "experimental".

This list is up-to-date in EKS 2.9, but not ec2 RKE1  node templates:

Validated on Rancher version: v2.10-5d66d49d7a72b4c439a33043007e9bb0223c8693-head
UI: master cf8c6d1
Confirmed the instances types are now in RKE1 templates. AWS EC2 RKE2 and EKS cluster types.


Fixed: random campaign selector, added support to null bidRequest node values and send siteId to win

change "test = config.campaignsList.get(kount);" to "test =
list.get(kount);" and BidRequest:setup():error when request with
“pmp”: null

OKs, I changed SiteId to tha last position at nurl. I put SiteId there before for no one reason. I do not care the position but I think it is an important value to put in nurl.
Thanks :-)

I fixed some other issues I found :-)

I think this commit will fix cache2k update from 0.23.1 to 1.0.0.CR3


Expand README.md

@davidvgalbraith see suggested additions to the README based on the questions we got from new users of the adapter. Please validate the optimizations list.

@jadbox anything else you think the README should cover? fresh eyes are best at detecting what "obvious" things we may have failed to explain.

+1


Added Online Restaurant Website

Description
Added Online Restaurant Website- Issue #376
Tech Stack Used

HTML
CSS
JavaScript

Screenshot

@selemondev can you pls review it.

@DhanushNehru pls review it!

Please resolve merge conflicts @rittikadeb

@DhanushNehru, I have resolved the conflicts. Please review it.

Thanks for your contribution @rittikadeb


Add more explanation/introduction to each protocol

Suggestion by @ksilnoaa - one page of background at the beginning of protocol

@sammyharding The text you are adding to the protocols on Google Docs would address this issue. When you are happy with those, let's review them and add that text to the Markdown versions of the protocols.


Incorrect Report that an Object may be null

Bug Report

🔎 Search Terms

Object is possibly 'null'
undefined

🕗 Version & Regression Information
This is the behavior in every version I tried, and I reviewed the FAQ for entries about unexpected warnings about possible 'null's and 'undefined's.
I tried versions 3.3.3333 and 4.3.0-beta, though I did not try any in the middle.
⏯ Playground Link
Playground link with relevant code
💻 Code

const refs: {current: {value: number} | null}[] = [{current: { value: 2}}];

if (refs.every((ref) => ref.current != null)) {
    refs[0].current.value = 1;
}

🙁 Actual behavior
"Object is possibly 'null'" warning on refs[0].current
🙂 Expected behavior
No warnings, as they cannot be null since the refs.every(...) is checking that.

Similar to https://github.com/microsoft/TypeScript/issues/16069, maybe https://github.com/microsoft/TypeScript/issues/18562.
One of the problems with this is that if something mutates the underlying array, your assumptions would be invalidated.

I think we can call it a duplicate of / encompassed by https://github.com/microsoft/TypeScript/issues/38390

Yes we can!


Old Patcher Does Not List All Current Patches, and New Patcher Does Not Run At All in Windows XP

The patcher does not list all patches it views with a JSON error on many servers when run in Windows XP for an older version and it does not run at all when trying the latest version with a KERNEL32 error when ran. So, is the patcher only going to work now on Vista and later? I am asking because I would like to play the games in English in Windows XP, but I cannot at all. I can play the patched games on Windows 7 for all games (mainly shooting games, but have tried VD also).

This should be fixed with the new version (2021-01-17)

This should be fixed with the new version (2021-01-17)


bacon: 2.2.7 -> 2.2.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Canop/bacon/releases.
meta.description for bacon is: "Background rust code checker"
meta.homepage for bacon is: "https://github.com/Canop/bacon"
meta.changelog for bacon is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.2.8 with grep in /nix/store/1nand3x7g2bfa17qzi9n2y6dqmb6lp9k-bacon-2.2.8
found 2.2.8 in filename of file in /nix/store/1nand3x7g2bfa17qzi9n2y6dqmb6lp9k-bacon-2.2.8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
bacon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1nand3x7g2bfa17qzi9n2y6dqmb6lp9k-bacon-2.2.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A bacon https://github.com/r-ryantm/nixpkgs/archive/f885004d3a35a82d33df84f12669f745d74332d9.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1nand3x7g2bfa17qzi9n2y6dqmb6lp9k-bacon-2.2.8
ls -la /nix/store/1nand3x7g2bfa17qzi9n2y6dqmb6lp9k-bacon-2.2.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

bacon

Maintainer pings
cc @FlorianFranzen for testing.

@ofborg eval


[BUG] AWS Access Key Id in example causes GitHub to block pushes

API Spec link
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/security/resource-manager/Microsoft.Security/preview/2020-01-01-preview/examples/Connectors/CreateUpdateAwsCredConnectorSubscription_example.json#L23
API Spec version
2020-01-01
Describe the bug
This example contains the line
          "awsAccessKeyId": "AKIARPZCNODDNAEQFSOE",

It seems that GitHub has expanded their secret scanning. At Pulumi, we generate a JSON schema and code from the Azure examples. Our generated code is now blocked from being pushed to GitHub, without extra workarounds, due to this key.
Expected behavior
The Azure spec examples don't contain real-looking secrets including AWS access key ids, so that pushing derived schemas or code to GitHub succeeds.
Actual behavior
remote: - GITHUB PUSH PROTECTION
remote:   ——————————————————————————————————————————————————————
remote:    Resolve the following secrets before pushing again.
remote:
remote:    (?) Learn how to resolve a blocked push
remote:    https://docs.github.com/code-security/secret-scanning/pushing-a-branch-blocked-by-push-protection
remote:
remote:
remote:   —— Amazon AWS Access Key ID ——————————————————————————
remote:    locations:
remote:      - commit: f1b05bccccfa232413de3e74d1c34c189e536117
remote:        path: provider/cmd/pulumi-resource-azure-native/schema.json:652276
remote:      - commit: f1b05bccccfa232413de3e74d1c34c189e536117
remote:        path: provider/cmd/pulumi-resource-azure-native/schema.json:652276
remote:      - commit: f1b05bccccfa232413de3e74d1c34c189e536117
remote:        path: provider/cmd/pulumi-resource-azure-native/schema.json:652276
remote:      - commit: f1b05bccccfa232413de3e74d1c34c189e536117
remote:        path: provider/cmd/pulumi-resource-azure-native/schema.json:652276
remote:      - commit: f1b05bccccfa232413de3e74d1c34c189e536117
remote:        path: provider/cmd/pulumi-resource-azure-native/schema.json:652276

Reproduction Steps
I don't have a concrete repro step but here's one of our workflows failing due to this.
Environment
No response

@aymanjarrousms Please help have a look, thank you.

I moved to different project.
@Ameer-az


Add missing CallbackReturn definition

Fixes
/ros2_workspace/src/husky/husky_base/src/husky_hardware.cpp:169:1: error: ‘CallbackReturn’ does not name a type
  169 | CallbackReturn HuskyHardware::on_init(const hardware_interface::HardwareInfo & info)
      | ^~~~~~~~~~~~~~
/ros2_workspace/src/husky/husky_base/src/husky_hardware.cpp:277:1: error: ‘CallbackReturn’ does not name a type
  277 | CallbackReturn HuskyHardware::on_activate(const rclcpp_lifecycle::State & /*previous_state*/)
      | ^~~~~~~~~~~~~~
/ros2_workspace/src/husky/husky_base/src/husky_hardware.cpp:296:1: error: ‘CallbackReturn’ does not name a type
  296 | CallbackReturn HuskyHardware::on_deactivate(const rclcpp_lifecycle::State & /*previous_state*/)
      | ^~~~~~~~~~~~~~

Closed as only needed when building against rolling ros2_control

Duplicate of https://github.com/husky/husky/pull/222


Move back to Java 7 + update to latest Jetty

For websockets we need the new Jetty 9 version, which is dependent on Java 7. Since the PowerMatcher without remoting is virtually useless (except for small demo scenarios), we need to update the repository to Java 7.

See pull request #189


[ff-2397] Allow attributes to be specified as dicts

Fixes: #ff-2397
Motivation and Context
Currently, users need to specify numerical and categorical attributes explicitly. This adds an option to supply a dictionary and we do the conversion to numerical and categorical attributes within the SDK based on value types
Description

Allow users to give subject and action attributes as dicts
If needed, convert these dicts to attributes
Changed the types for this pattern to look clean
Turn off graceful mode for tests to catch errors explicitly

How has this been tested?
All tests pass

lgtm 👍


Outline a Table of Contents

Plan a table of contents to work towards.

The first draft of the Table of Contents has is available online. However, we are looking for suggestions for other topics/areas to cover. Please comment on this issue and we can discuss ideas.


Flutter web application shows just a blank screen

I'm using Windows 10.
Steps to Reproduce

Install flutter sdk, make a new project using vscode, then follow here's steps to run the web application.
The chrome browser window shows up but the page content is empty.

By using git reset --hard command several times in flutter sdk, I confirmed it's working properly in the commit 2a36ad31facc735edceb887819931840599e0777 .
But not working in the commit e026bd3d33c79c8e68ea0065815dcae9981464f5

Logs
Chrome's log:

browser_location.dart:94 Uncaught (in promise) TypeError: Cannot read property 'hash' of undefined
    at HashLocationStrategy.get path [as path] (browser_location.dart:94)
    at _engine.BrowserHistory.new.get currentPath [as currentPath] (history.dart:56)
    at _engine.BrowserHistory.new.[_setupStrategy] (history.dart:164)
    at _engine.BrowserHistory.new.set locationStrategy [as locationStrategy] (history.dart:51)
    at _engine.EngineWindow.new.set locationStrategy [as locationStrategy] (window.dart:81)
    at webOnlyInitializePlatform (initialization.dart:12)
    at webOnlyInitializePlatform.next (<anonymous>)
    at runBody (async_patch.dart:84)
    at Object._async [as async] (async_patch.dart:123)
    at Object.webOnlyInitializePlatform (initialization.dart:8)
    at main$ (main_web_entrypoint.dart:7)
    at main$.next (<anonymous>)
    at runBody (async_patch.dart:84)
    at Object._async [as async] (async_patch.dart:123)
    at main$ (main_web_entrypoint.dart:5)
    at Function.call$0 (client.js:27433)
    at client.js:26055
    at _wrapJsFunctionForAsync_closure.$protected (client.js:3110)
    at _wrapJsFunctionForAsync_closure.call$2 (client.js:11704)
    at Object._asyncStartSync (client.js:3075)
    at main__closure3.$call$body$main__closure (client.js:26063)
    at main__closure3.call$1 (client.js:25993)
    at StaticClosure._rootRunUnary [as call$2$5] (client.js:3393)
    at _CustomZone.runUnary$2$2 (client.js:13083)
    at _CustomZone.runUnaryGuarded$1$2 (client.js:13011)
    at _ControllerSubscription._sendData$1 (client.js:12564)
    at _DelayedData.perform$1 (client.js:12771)
    at _StreamImplEvents.handleNext$1 (client.js:12845)
    at _PendingEvents_schedule_closure.call$0 (client.js:12817)
    at Object._microtaskLoop (client.js:3253)
    at StaticClosure._startMicrotaskLoop (client.js:3259)
    at _AsyncRun__initializeScheduleImmediate_internalCallback.call$1 (client.js:11580)
    at invokeClosure (client.js:1206)
    at MutationObserver.<anonymous> (client.js:1225)

Hundreds of thousands errors from flutter analyze. I don't copy them all...
311497 issues found. (ran in 12.9s; 4343 public members lack documentation)
Analyzing flutter...

  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_run_test\flutter_run_test.dart:5:8 - uri_does_not_exist
  error - The function 'group' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:8:3 - undefined_function
  error - The function 'test' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:9:5 - undefined_function
  error - The function 'expect' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:10:7 - undefined_function
  error - The function 'test' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:12:5 - undefined_function
  error - The function 'expect' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:13:7 - undefined_function
  error - The function 'test' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:15:5 - undefined_function
  error - The function 'expect' isn't defined - dev\automated_tests\flutter_run_test\flutter_run_test.dart:16:7 - undefined_function
  error - Target of URI doesn't exist: 'package:test/test.dart' - dev\automated_tests\flutter_test\child_directory\second_trivial_test.dart:5:8 - uri_does_not_exist
  error - The function 'test' isn't defined - dev\automated_tests\flutter_test\child_directory\second_trivial_test.dart:8:3 - undefined_function  error - Target of URI doesn't exist: 'package:test/test.dart' - dev\automated_tests\flutter_test\child_directory\trivial_test.dart:5:8 - uri_does_not_exist
  error - The function 'test' isn't defined - dev\automated_tests\flutter_test\child_directory\trivial_test.dart:8:3 - undefined_function
  error - Target of URI doesn't exist: 'package:flutter/foundation.dart' - dev\automated_tests\flutter_test\exception_handling_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\exception_handling_test.dart:6:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\exception_handling_test.dart:9:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\exception_handling_test.dart:9:63 - undefined_class
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\exception_handling_test.dart:12:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\exception_handling_test.dart:12:69 - undefined_class
  error - The function 'FlutterError' isn't defined - dev\automated_tests\flutter_test\exception_handling_test.dart:13:11 - undefined_function
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\exception_handling_test.dart:15:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\exception_handling_test.dart:15:78 - undefined_class
  error - Target of URI doesn't exist: 'package:test/test.dart' - dev\automated_tests\flutter_test\filtering_test.dart:5:8 - uri_does_not_exist
  error - The function 'test' isn't defined - dev\automated_tests\flutter_test\filtering_test.dart:8:3 - undefined_function
  error - The function 'expect' isn't defined - dev\automated_tests\flutter_test\filtering_test.dart:9:5 - undefined_function
  error - The function 'test' isn't defined - dev\automated_tests\flutter_test\filtering_test.dart:11:3 - undefined_function
  error - Target of URI doesn't exist: 'package:flutter/widgets.dart' - dev\automated_tests\flutter_test\package_assets_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\package_assets_test.dart:6:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\package_assets_test.dart:9:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\package_assets_test.dart:9:35 - undefined_class
  error - Undefined name 'Image' - dev\automated_tests\flutter_test\package_assets_test.dart:12:7 - undefined_identifier
  error - Undefined name 'BoxFit' - dev\automated_tests\flutter_test\package_assets_test.dart:15:14 - undefined_identifier
  error - Target of URI doesn't exist: 'package:flutter/widgets.dart' - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:6:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:9:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:9:35 - undefined_class  error - The function 'CustomScrollView' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:12:7 - undefined_function
  error - The name 'Widget' isn't a type so it can't be used as a type argument - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:13:19 - non_type_as_type_argument
  error - The function 'SliverToBoxAdapter' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:14:11 - undefined_function
  error - The function 'Container' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_no_flag_test.dart:14:37 - undefined_function
  error - Target of URI doesn't exist: 'package:flutter/widgets.dart' - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:6:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:9:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:9:35 - undefined_class
  error - The function 'CustomScrollView' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:12:7 - undefined_function
  error - The name 'Widget' isn't a type so it can't be used as a type argument - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:13:19 - non_type_as_type_argument
  error - The function 'SliverToBoxAdapter' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:14:11 - undefined_function
  error - The function 'Container' isn't defined - dev\automated_tests\flutter_test\print_user_created_ancestor_test.dart:14:37 - undefined_function
  error - Target of URI doesn't exist: 'package:flutter/foundation.dart' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter/widgets.dart' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:6:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:7:8 - uri_does_not_exist
  error - Classes can only extend other classes - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:9:31 - extends_non_class
  error - Undefined class 'AutomatedTestWidgetsFlutterBinding' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:9:31 - undefined_class
  error - Undefined class 'DebugPrintCallback' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:11:3 - undefined_class
   info - Getter doesn't override an inherited getter - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:11:26 - override_on_non_overriding_getter
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:15:28 - undefined_class
  error - Undefined name 'TestAsyncUtils' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:16:10 - undefined_identifier
  error - Undefined class 'Text' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:17:35 - undefined_class
  error - Undefined name 'TextDirection' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:17:64 - undefined_identifier
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:23:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:23:60 - undefined_class
  error - The function 'Container' isn't defined - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:24:29 - undefined_function
  error - The function 'expect' isn't defined - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:25:5 - undefined_function
  error - Undefined name 'find' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:25:12 - undefined_identifier
  error - Undefined name 'Container' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:25:31 - undefined_identifier
  error - Undefined name 'isNotNull' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:25:43 - undefined_identifier
  error - The function 'expect' isn't defined - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:27:5 - undefined_function
  error - Undefined name 'find' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:27:12 - undefined_identifier
  error - Undefined name 'Container' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:27:31 - undefined_identifier
  error - Undefined name 'isNull' - dev\automated_tests\flutter_test\test_async_utils_guarded_test.dart:27:43 - undefined_identifier
  error - Target of URI doesn't exist: 'package:flutter/foundation.dart' - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:6:8 - uri_does_not_exist
  error - Classes can only extend other classes - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:8:31 - extends_non_class  error - Undefined class 'AutomatedTestWidgetsFlutterBinding' - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:8:31 - undefined_class
  error - Undefined class 'DebugPrintCallback' - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:10:3 - undefined_class
   info - Getter doesn't override an inherited getter - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:10:26 - override_on_non_overriding_getter
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:14:29 - undefined_class
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:20:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\test_async_utils_unguarded_test.dart:20:78 - undefined_class
  error - Target of URI doesn't exist: 'package:flutter/scheduler.dart' - dev\automated_tests\flutter_test\ticker_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter/services.dart' - dev\automated_tests\flutter_test\ticker_test.dart:6:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\ticker_test.dart:7:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\ticker_test.dart:10:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\ticker_test.dart:10:60 - undefined_class
  error - The function 'Ticker' isn't defined - dev\automated_tests\flutter_test\ticker_test.dart:11:5 - undefined_function
  error - Undefined class 'ByteData' - dev\automated_tests\flutter_test\ticker_test.dart:13:11 - undefined_class
  error - Undefined class 'StringCodec' - dev\automated_tests\flutter_test\ticker_test.dart:13:36 - undefined_class
  error - Undefined name 'ServicesBinding' - dev\automated_tests\flutter_test\ticker_test.dart:14:11 - undefined_identifier
  error - Missing parameter type for '_' - dev\automated_tests\flutter_test\ticker_test.dart:14:112 - implicit_dynamic_parameter
  error - Target of URI doesn't exist: 'package:test/test.dart' - dev\automated_tests\flutter_test\trivial_test.dart:5:8 - uri_does_not_exist
  error - The function 'test' isn't defined - dev\automated_tests\flutter_test\trivial_test.dart:8:3 - undefined_function
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\flutter_test\trivial_widget_test.dart:5:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\flutter_test\trivial_widget_test.dart:8:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\flutter_test\trivial_widget_test.dart:8:41 - undefined_class
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\test_smoke_test\crash1_test.dart:7:8 - uri_does_not_exist
  error - The function 'test' isn't defined - dev\automated_tests\test_smoke_test\crash1_test.dart:13:3 - undefined_function
  error - Target of URI doesn't exist: 'package:flutter/foundation.dart' - dev\automated_tests\test_smoke_test\disallow_error_reporter_modification_test.dart:5:8 - uri_does_not_exist
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\test_smoke_test\disallow_error_reporter_modification_test.dart:6:8 - uri_does_not_exist
  error - The function 'testWidgets' isn't defined - dev\automated_tests\test_smoke_test\disallow_error_reporter_modification_test.dart:9:3 - undefined_function
  error - Undefined class 'WidgetTester' - dev\automated_tests\test_smoke_test\disallow_error_reporter_modification_test.dart:9:71 - undefined_class
  error - Undefined name 'reportTestException' - dev\automated_tests\test_smoke_test\disallow_error_reporter_modification_test.dart:11:5 - undefined_identifier
  error - Undefined class 'FlutterErrorDetails' - dev\automated_tests\test_smoke_test\disallow_error_reporter_modification_test.dart:11:28 - undefined_class
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\test_smoke_test\fail_test.dart:5:8 - uri_does_not_exist
  error - The function 'test' isn't defined - dev\automated_tests\test_smoke_test\fail_test.dart:11:3 - undefined_function
  error - The function 'expect' isn't defined - dev\automated_tests\test_smoke_test\fail_test.dart:12:5 - undefined_function
  error - Undefined name 'isTrue' - dev\automated_tests\test_smoke_test\fail_test.dart:12:19 - undefined_identifier
  error - Target of URI doesn't exist: 'package:flutter_test/flutter_test.dart' - dev\automated_tests\test_smoke_test\pass_test.dart:5:8 - uri_does_not_exist

'flutter doctor -v'.
[√] Flutter (Channel master, v1.10.5-pre.23, on Microsoft Windows [Version 10.0.18362.356], locale ja-JP)
    • Flutter version 1.10.5-pre.23 at C:\flutter
    • Framework revision e026bd3d33 (7 hours ago), 2019-09-19 07:29:41 -0400
    • Engine revision 1cada0a6dc
    • Dart version 2.6.0 (build 2.6.0-dev.1.0 ea969c358e)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\shohei\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.4)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 35.0.1
    • Dart plugin version 183.5901
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[√] VS Code (version 1.38.1)
    • VS Code at C:\Users\shohei\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.4.1

[√] VS Code, 32-bit edition (version 1.33.1)
    • VS Code at C:\Program Files (x86)\Microsoft VS Code
    • Flutter extension version 3.4.1

[√] Connected device (2 available)
    • Chrome • chrome • web-javascript • Google Chrome 76.0.3809.132
    • Server • web    • web-javascript • Flutter Tools

• No issues found!

Oops duplicate of #40876 .

Same error on the master channel. Error does not appear on the dev channel.
you can do this as fellows.
flutter channel dev
flutter upgrade
flutter run -d chrome

FIXED IT
switched to beta then back to dev using
flutter channel dev
flutter upgrade
then run via web
flutter run -d web


nixos/libreswan: add missing runtime dependencies

Motivation for this change
Module was not usable (service failed to start due to missing paths).
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@afranchuk and @zimbatm you have made modifications to this module, any thoughts?

The change should be fine. I'm guessing those are new run-time dependencies? It certainly worked with older versions.

I just needed the module, and in the logs it complained. I basically added everything that was missing so that the service successfully starts. One thing I haven't added is the loading of the af_key module, apparently required for netkey. It should have been autoloaded, but there's no note in the logs.


eslint: File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Current behavior
Ignored files in .eslintrc.js are ignored successfully on the initial build. After that if they are changed and a watch build picks them up you get the error:

WARNING in path/file.ts
[unknown]: File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

The files are successfully ignored still - it's just a distracting warning.
Expected behavior
No warnings
Steps to reproduce the issue
// .eslintrc.js
module.exports = {
    ...
    ignorePatterns: ['*/*.ignoreme.ts']
};

// webpack.config.js
modules.exports = {
    plugins: [
        new ForkTSCheckerWebpackPlugin({
            eslint: {
                files: '.',
            },
        }),
    ]
}

Run a watched webpack build. The initial build will run with no warnings. Now update and save test.ignoreme.ts.
You see the warning referred to above.
Environment

fork-ts-checker-webpack-plugin: 5.0.6
typescript: 3.9.2
eslint: 6.8.0
webpack: 4.43.0
os: Windows 10

Notes
This could possibly be worked around by not including the eslint ignored files in the include patterns for fork-ts-checker-webpack-plugin (or at least ignoring the warnings from the file as per #450).
This seems to be the same issue as #316 which was previously fixed but seems to have been partially reverted in the recent restructuring. If I update the eslint reporter locally like so, I no longer see the error
const changedAndIncludedFiles = changedFiles.filter(
  (changedFile) =>
    includedFilesPatterns.some((includedFilesPattern) =>
      minimatch(changedFile, includedFilesPattern)
    ) &&
    (configuration.options.extensions || []).some((extension) =>
      changedFile.endsWith(extension)
-   )
+   ) &&
+  (!engine.isPathIgnored(changedFile )
);

Happy to drop in a PR if you think this looks reasonable.

Thanks for reporting this issue :) Would be great if you could create a PR :)

:tada: This issue has been resolved in version 5.0.13 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add RBAC for new CRDs

Note that having the ClusterRole reference something that doesn't exist should be okay.
Noticed when I applied the new CRDs to my GKE cluster.
Signed-off-by: Nick Young<EMAIL_ADDRESS>

Also, you need to run the ./examples/render.sh step


210 ownedpositionidsrequest

I need to add the POSITIONS table to indexed-db and add saving the position data to block-processor, but it makes sense to do it after the refactoring in #357 is complete

0.65.0-pre.1 published to npm with the new compute_position_id() available to use

It's weird, broken tests that aren't related to this PR

a lot of files changed just because of lint fix from #388


URP support

Hey there,
Congrats for your work! I love this shader, but unfortunatelly it's not supported with the URP.
Do you have any plans to port it soon to Shader Graph?
Thanks!

Please check https://github.com/keijiro/Smrvfx

Wow, impressive! Thanks a lot for sharing this! It would be nice to upload it to Unity asset store so we can buy it and support your work :)


[WIP] Improve mobile compatibility and add touchscreen support to professor tooltips

WIP. Currently a PR to test preview build on mobile device.

Doesn't build b/c I'm relying on window. Will look into other options or adding it to component effect so it is not needed at build time.


[KCC] Add the test case for owned-project solution

This test case still adds a new env var for the new project. Eventually users will only need to specify a NEW_PROJECT_ID_PREFIX, but this improvement will be done after all the test cases are backfilled.


[Docs] Fixes variable width on white panel

That's bizarre, I'm unable to reproduce. What version of Safari?
.Container should have a max-width of 1000px, which should take care of constraining the panel.

Actually Google Chrome Version 48.0.2564.41 beta (64-bit) on this page.

Can you inspect and see why .Container is not constraining the width?

I'm seeing this in Canary, it has to do with the <pre> tag. We shouldn't just stick a max-width on the white panel, I think. If you'd like to figure out with the flexbox stuff isn't working I'd be happy to merge.

Tried a few things but couldn't figure out how to fix it without the the max-width on the white panel. Feel free to close it.

Ok. Thanks for your help, hopefully this is a bug in canary since we're not seeing it anywhere else.


Create accurate timer for daemon startup

bonus: output elapsed time in json so we can include it in tests

@lyoshenka Is this still valid? Need additional information


Pass proper talk ID to e-mail handler

The ORM returns 1 on successful updates which was used as the talk ID (sic!). I've fixed the bug for updates and unified the code for new talk creations.
Fixes #481

Greatly appreciated @sgrodzicki


The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher.

A version bump release is badly needed. With the most recent gradle version 7.4 this is what I get trying to build my Flutter app:
Running Gradle task 'assembleDebug'...

FAILURE: Build failed with an exception.

* What went wrong:
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher.
The following dependencies do not satisfy the required version:
project ':flutter_native_timezone' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50

Same issue here

Same issue here

Due to lack of maintenance in this repo, I have just published a fork on pub.dev.
There is an open PR in this repository that I should solve your issue and I have cherry-picked that into my fork.

Oh great. I've got the same issue. thanks


BABELFISHPG_COMMON: MS SQL DATA TYPES

In summary, we are providing implementation of T-SQL datatypes.
I am confused by first paragraph. I am not sure what information we want to deliver through it.
I suggest that this section to convey following information:

purpose of this extension
reference to each individual type  link between our datatype and T-SQL document
( I assume SQL server users can easily what the types are, don't need any description on what the type is )
other small issues
3.1 BBF_BINARY is implementation behind sys.BINARY
3.2 sys.BPCHAR is also an internal type to support NCHAR.

@zli236 - Please review @macwoody's comments and provide suggested text to @postgresql007 on how this needs to be updated in the documentation.

@macwoody Could you provide the name of the documentation page this issue refers to?

I think that the section that @macwoody is referring to is at _internals/software-architecture.md. cc/ @laurenz

@3manuek Thanks, that must be it.
Addressing the pints in the issue:

I am confused by first paragraph. I am not sure what information we want to deliver through it.

The paragraph is an introduction to the material that follows. It says that the server had to be modified to make Babelfish happen, through outright hacking it up, through hooks and through extensions. I can try to make that somewhat clearer, but details are given in what follows.

I suggest that this section to convey following information:

purpose of this extension

That's stated right there: "Babelfish implements Microsoft SQL Server behavior on top of PostgreSQL"

reference to each individual type link between our datatype and T-SQL document
( I assume SQL server users can easily what the types are, don't need any description on what the type is )

That is a good idea. I'll try to add that.

other small issues
3.1 BBF_BINARY is implementation behind sys.BINARY
3.2 sys.BPCHAR is also an internal type to support NCHAR.

I don't know enough about Babelfish and SQL Server to do that. I'll have to rely on technical input from your side.

@macwoody - could you please respond to Laurenz' questions above? Please be as descriptive as possible to provide enough clarity for Laurenz to complete the documentation.

Ok, I have removed the two internal data types from the list.
This is good to go.


connection type preview form generation

https://issues.redhat.com/browse/RHOAIENG-10319
Description

Generates a preview form from the connection type resource and field descriptors.
Uses the existing mock which encompasses all variations of fields.
Form fields are rendered in their default states but do not react to changes so they act as read only.
Provides a drawer component to render the content preview given the connection type resource.
There are still several UX discussions on going but any changes should be minimal to incorporate.
There's also a quirk with dropdowns that is causing the drawer to scroll back to the top whenever it is opened. This can be dealt with later.

How Has This Been Tested?

Manually. Test pages.
Added unit tests.
Test Impact

Unit tests.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

cc @simrandhaliw

@emilys314 @jeff-phillips-18 rebased this PR

Looks good to me I think

/lgtm


Error budget percentage for an SLO gets set incorrectly in generated reliably.yaml

Describe the bug
I performed the following reliably slo init command sequence:
$ reliably slo init
? Are you building something that will be provided to customers 'as a service'? Yes
? What percentage of requests to your service is it ok to have fail? This will be your 'error budget'. 0.1
? What is the maximum request-response latency you want from this service (in milliseconds)? 100
? Do you want to add a service resource? Yes
? What is the name of the resource provider? aws
? What is the ARN of the resource? sadsfsdfsadsa
? Do you want to add another dependency? No

I got the following reliably.yaml file generated:
slo:
  objective:
    latency: 100ms
    error_budget_percent: 0.10000000149011612
  resources:
  - id: sadsfsdfsadsa
    provider: aws
dependencies: []

The error budget percent is wrong.
Expected behavior
The error_budget_percentage should be 0.1.
Additional context
Using version Reliably CLI version 0.7.0 (2021-04-02)

I guess this is about Golang float precision @xaah


Anyone try Scalable Broadcasting?

An issue with Scaleable Broadcasting. After broadcaster starts, next viewer is ok, 2nd viewer joins ok, but then has wrong roomid stored. He has userid stored instead of roomid, so 3rd viewer cannot join? I am using Chrome and new tab for each viewer. So roomid should be the same.
Anyone else see this?

To test, use Chrome browser: Start with new default broadcast. Random roomid assigned.
Open another tab and browse to page. Same roomid used. Join broadcast.
Open another tab and browse to page. Same roomid is used. Join broadcast.
Open another tab. Different roomid is used. Cannot join broadcast, but can start new broadcast.
This viewer should have received same roomid as other viewers!!
How to fix???

I tested it. It is a mistake in the Local Storage related code, not an issue in the RTCMultiConnection API. Perhaps the result you wanted was to be given the same loomid every time you open a new tab. If you change the example code as shown below, you will get the result you want.

remove this line
localStorage.setItem(connection.socketMessageEvent, connection.sessionid);

change this line
txtBroadcastId.onkeyup = txtBroadcastId.oninput = txtBroadcastId.onpaste = function() {  **localStorage.setItem(connection.socketMessageEvent, connection.sessionid);** };

So the only way I can test is using multiple tabs in Chrome on my PC.   Eventually I will need multiple people from multiple locations for proper testing.
I have not tried your fix yet,  but hopefully it fixes this problem.
I will let you know how it turns out.
Thanks for taking the time for me.
Ray
From: wnsvy223<EMAIL_ADDRESS>Sent: Monday, June 08, 2020 2:15 AM
To: muaz-khan/RTCMultiConnection<EMAIL_ADDRESS>Cc: rayj00<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [muaz-khan/RTCMultiConnection] Anyone try Scalable Broadcasting? (#927)
I tested it. It is a mistake in the Local Storage related code, not an issue in the RTCMultiConnection API. Perhaps the result you wanted was to be given the same loomid every time you open a new tab. If you change the example code as shown below, you will get the result you want.

 remove this line

localStorage.setItem(connection.socketMessageEvent, connection.sessionid);

 change this line

txtBroadcastId.onkeyup = txtBroadcastId.oninput = txtBroadcastId.onpaste = function() { localStorage.setItem(connection.socketMessageEvent, connection.sessionid); };
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/muaz-khan/RTCMultiConnection/issues/927#issuecomment-640392072 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSNU45TDQRPFRRNNN6LLRLRVR6W7ANCNFSM4NXI4UMQ .  https://github.com/notifications/beacon/ABSNU4YCLG4W77D46QRL2T3RVR6W7A5CNFSM4NXI4UM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEYVZXCA.gif

I am getting an error using your #2 below:
Uncaught SyntaxError: Unexpected token '**'
From: wnsvy223<EMAIL_ADDRESS>Sent: Monday, June 08, 2020 2:15 AM
To: muaz-khan/RTCMultiConnection<EMAIL_ADDRESS>Cc: rayj00<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [muaz-khan/RTCMultiConnection] Anyone try Scalable Broadcasting? (#927)
I tested it. It is a mistake in the Local Storage related code, not an issue in the RTCMultiConnection API. Perhaps the result you wanted was to be given the same loomid every time you open a new tab. If you change the example code as shown below, you will get the result you want.

 remove this line

localStorage.setItem(connection.socketMessageEvent, connection.sessionid);

 change this line

txtBroadcastId.onkeyup = txtBroadcastId.oninput = txtBroadcastId.onpaste = function() { localStorage.setItem(connection.socketMessageEvent, connection.sessionid); };
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/muaz-khan/RTCMultiConnection/issues/927#issuecomment-640392072 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSNU45TDQRPFRRNNN6LLRLRVR6W7ANCNFSM4NXI4UMQ .  https://github.com/notifications/beacon/ABSNU4YCLG4W77D46QRL2T3RVR6W7A5CNFSM4NXI4UM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEYVZXCA.gif

I removed both ** and now it's working fine!  You fixed the problem!  Gracias!!  Thank You!  Danke!

Next issue is when there are multiple viewers, if the first viewer leaves (not the broadcaster), all other viewers lose the broadcast.  I thought the connections were supposed to re-establish?

Test with Chrome: Broadcaster and 3 viewers.  Broadcaster --> Viewer1 --> Viewer2 --> Viewer3
If Viewer1 drops out, Viewer2 gets black video with rotating icon.  Viewer3 Is still getting good video from Viewer2.  So video is still being scaled and passed through from broadcaster to Viewer2 --> Viewer3.  Just viewer2 not showing the video, only black video w/rotating icon.  Not sure what you call that...it's like the buffering icon.

I'm sorry. I'm a junior developer and still have difficulty coding, so I don't know the exact reason.
But if you remove this line, connection.maxRelayLimitPerUser = 1; will operate normally.
If you look at the RTCMulticonnection client side code, you can see that the maxRelayLimitPerUser value is set to default 3. I think Muaz Khan cannot manage this api so we should use default values.

I'm sorry. I'm a junior developer and still have difficulty coding, so I don't know the exact reason.
But if you remove this line, connection.maxRelayLimitPerUser = 1; will operate normally.
If you look at the RTCMulticonnection client side code, you can see that the maxRelayLimitPerUser value is set to default 3. I think Muaz Khan cannot manage this api so we should use default values.

Unfortunately removing that line defeats the purpose of scalability.  The broadcaster will then service all of the viewers! Changing to a value of 2 seems to make things better but still testing.

I'm sorry. I'm a junior developer and still have difficulty coding, so I don't know the exact reason.
But if you remove this line, connection.maxRelayLimitPerUser = 1; will operate normally.
If you look at the RTCMulticonnection client side code, you can see that the maxRelayLimitPerUser value is set to default 3. I think Muaz Khan cannot manage this api so we should use default values.

The problem is if you comment out the line in index.html, then the broadcaster will serve the video to all viewers.

I see. I didn't know because I haven't tested a lot of bradcast samples. Thank you for letting me know.

Test with PC Chrome: Broadcaster and 4 viewers. Broadcaster --> Viewer1 --> Viewer2 --> Viewer3 --> Viewer4. If Viewer1 drops out, Viewer2/Viewer3/Viewer3 all gets black video with rotating icon.
Who can find a solution?

Test with PC Chrome: Broadcaster and 4 viewers. Broadcaster --> Viewer1 --> Viewer2 --> Viewer3 --> Viewer4. If Viewer1 drops out, Viewer2/Viewer3/Viewer3 all gets black video with rotating icon.
Who can find a solution?

I am not seeing this issue with Windows 10 Chrome.

I tried replacing PC Chrome with 'replaceTrack' as well and it worked. E.g.
       connection.getSocket(function(socket) {
            socket.emit('can-relay-broadcast');

			connection.getAllParticipants().forEach(function(p) {
				if (p + '' != event.userid + '') {
					connection.replaceTrack(event.stream, p);
				}
			});
        });


gradio problems

[Errno 2] No such file or directory: '/usr/local/lib/python3.8/dist-packages/gradio/blocks.py' -> '/usr/local/lib/python3.8/dist-packages/gradio/blocks.py.bak'

use the latest colab and try again


Prints error message encountered in cleanup-java

Description:
Updates the failure message in cleanup-java to include details of the error encountered.
Related issue:
https://github.com/actions/setup-java/issues/200
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

Hello @bbranan. Thank you for your pull request. Could you please run npm run build to regenerate dist directory?

Hello @bbranan. Could you please try to run npm ci or switch to node 12 with npm 6.* to revert changes in package-lock.json

@dmitry-shibanov I could, but it seems that we're going around in circles for a really minor change. It may be easier for you to cherry pick my initial change and build it in your environment so that it matches the expected project setup. I'd be happy to either give you write access to my repo or close this PR to let you create another with the same initial commit.

Hello @bbranan. Pull request with adding error message is merged, that is why I'm closing your pull request.


Report revamp

Added parameters and calculated volumes in report output and notification box.
Fixed surface map format to avoid issues with some versions of pymol and chimera.

In my build no report file is generated

This is troublesome. Since it works on windows 10 for me but not in macOS for you, I suppose it could be a problem with how directories are handled.
I'll see if I can use wxWidgets to make a general export solution across different OS.

There are still some conflicting files, likely because of the previous merge, that you'd have to fix.
As for the file creation I have several issues. On my machine each calculation creates a new folder inside output/invalid\ MoloVol/. accessing individual folders is a huge pain, because when I run two calculations in quick succession, I need to manually check which one's second counter is larger.
However, accessing the folders is not necessary anyway, because despite a new folder being created each calculation, the surface map and the report are placed in the root folder. This is a problem, because you haven't updated the .gitignore file. This means that once I run a calculation on this branch, I cannot switch branches until I do a hard reset or manually delete the files. Also, this has previously caused me grief, when I accidentally committed an output file that was too large for GitHub.


Volume mounts need to use "Binds" API field

Swarm was putting volume type mounts into the container config's
"Volumes" field, but really these need to go into "Binds".
"Volumes" is only for normal "-v /foo" volumes, not named volumes or
anything else.
I was looking at adding an integration test for this, however it dawned on me I was testing implementation detail as there is not really an easy way to actually get at the underlying container object to have a look at it, or really any good way to dive into this service.

LGTM

Swarm was putting volume type mounts into the container config's
"Volumes" field, but really these need to go into "Binds".

😢
Does swarmkit need this fix, as well?

Is it possible to use the mount structures directly in the engine code?

@stevvooe That would be nice, and ultimately what #22373 is trying to do. That PR should be adapted to whatever we end up with in the service mounts.

@cpuguy83 bump.

Does swarmkit need this fix, as well?

I ask this because swarmkit tries to put source-less volumes in the Volumes field and the rest in binds, whereas this PR doesn't populate Volumes.

@stevvooe By swarmkit do you mean swarmd?

@cpuguy83 I mean github.com/docker/swarmkit.

@stevvooe This is updated.
Will have to look at swarmkit.

LGTM

LGTM

❤


Connections Defaulting to Upper Most Server Group

Issue Type: Bug
Creating New Connections in a server group places the connection in the correct server group but upon reconnecting the server group for that connection defaults to the upper most server group.
Azure Data Studio version: azuredatastudio 1.12.0 (f0b6180e15f455ccf9e8112b407a6aa8c5399fe9, 2019-10-01T18:01:40.565Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.88GB (4.80GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

powershell
ms-
2019.9.0

query-history
Mic
0.1.0

poor-sql-formatter
WSR
0.1.0

Details
Creating New Connections in a server group places the connection in the correct server group but upon reconnecting the server group for that connection defaults to the upper most server group.
This means that I have to remember to change the server group every time I make a connection or go
back
and tidy the connections out of the first group periodically.
Looked through the settings and the groupID's etc all seemed correct.
datasource.connectionGroups
{
    "name": "testinggroup",
    "id": "6ce1ac6e-10f7-4956-a68d-223246eaa637",
    "parentId": "C777F06B-202E-4480-B475-FA416154D458",
    "color": "#98AFC7",
    "description": ""
},
{
    "name": "subgroup",
    "id": "ca5c6e10-7266-49ce-98a7-ae78670515d3",
    "parentId": "6ce1ac6e-10f7-4956-a68d-223246eaa637",
    "color": "#98AC96",
    "description": ""
}

datasource.connections
{
    "options": {
        "connectionName": "",
        "server": "Identifier.database.windows.net",
        "database": "master",
        "authenticationType": "SqlLogin",
        "user": "read-only",
        "password": "",
        "connectTimeout": 30,
        "applicationName": "azdata",
        "groupId": "ca5c6e10-7266-49ce-98a7-ae78670515d3",
        "databaseDisplayName": "Mydatabasae"
    },
    "groupId": "ca5c6e10-7266-49ce-98a7-ae78670515d3",
    "providerName": "MSSQL",
    "savePassword": false,
    "id": "1b619535-dbc9-4409-9aaf-c9c1f47c75ba"
}

So these appear to be correctly grouped within the settings.

@brettmillerb It appears to be working fine for me - reconnecting a server in a group correctly connects the server under the specified group. Could you include more detailed repro steps? And/or a gif showing the issue?

Test group created with a sub group

Closed the connection

Click on the connection and it opens in the upper most group
The datasource connections and connectiongroups still have it linked to the correct groupId,Parent etc

Ah got it - yeah there's an issue there where we're not correctly populating the server group with nested groups. I'll investigate this more - thanks for the report!


Override identifier in Disqus component

It seems like the  component always uses the same identifier:
(this.$route.path || window.location.pathname)

Our project uses localization via nuxt-i18n/vue-i18n so the path will differ for different languages. We'd like to have the same comment thread for any locale.
It would be nice to be able to override identifier in the  component

Sorry for delay.
You can use the pageConfig prop, using the location property, for example:
<Disqus shortname='your_shortname_disqus' :pageConfig="{ identifier: 'yout_unique_identifier' }" />

I am getting

There was a problem with the Disqus configuration. TypeError: Cannot read property 'path' of undefined

near this.$route.path

Thanks @ktquez that seems to work fine