Minor bugs fixed and code improvement

Hi. Thanks for you guys for the great works!
Lately I been trying to apply GLNet for real world problem using you guys code and having little bugs here and there. After a month trying to resolve a bunch of them, I finally mange to get GLNet working for my binary segmentation problem.
This PR contain some minor bugs fix and improvement while I working with you guys code.
If this PR don't meet the quality for a merge. It's totally fine but please told me what can I do to complete my contribution to this repo :D
Regards and thanks, Linus ;)

This pull request introduces 1 alert when merging 46ebe82c334bd7eba4b4efa9439f1d10388c80b2 into 867d2a644b88fc9131d250d589129211279fd0b7 - view on LGTM.com
new alerts:

1 for Unused import

This project has automated code review enabled, but doesn't use the LGTM GitHub App. Migrate over by installing the app. Read about the benefits of migrating to GitHub Apps in the blog.

Comment posted by LGTM.com

This pull request fixes 5 alerts when merging c9d6b4dc28d0c9f61b737b43d9bb019bfe5e0374 into 867d2a644b88fc9131d250d589129211279fd0b7 - view on LGTM.com
fixed alerts:

4 for Unused import
1 for NotImplemented is not an Exception

This project has automated code review enabled, but doesn't use the LGTM GitHub App. Migrate over by installing the app. Read about the benefits of migrating to GitHub Apps in the blog.

Comment posted by LGTM.com

Hi Hoàng Tùng Lâm! Thank you very much for your interest and great effort!

Thanks @chenwydj ;)

This pull request introduces 1 alert and fixes 5 when merging 0dc47fac116d760bda5e848fee44d69a133bce83 into 867d2a644b88fc9131d250d589129211279fd0b7 - view on LGTM.com
new alerts:

1 for Unused import

fixed alerts:

4 for Unused import
1 for NotImplemented is not an Exception

This project has automated code review enabled, but doesn't use the LGTM GitHub App. Migrate over by installing the app. Read about the benefits of migrating to GitHub Apps in the blog.

Comment posted by LGTM.com

Since everything from training script and pertained model was uploaded by the author. I will close this PR :P


Ditch the "application name" part of the StorageClient construction

Just use "gcloud-dotnet" as a hard-coded name.

This was done in #111.

Would we not want folks to allow customization of the User-Agent, which is what this is used for? And even if they do, wouldn't we want to append gcloud-dotnet/0.3 (or whatever the version is) anyway, to mark what version of the SDK they use?

It still allow the customization of this - it's just somewhat hidden (StorageClientImpl.ApplicationName) because it isn't useful to most people.
For the gRPC-based libraries, we include more detailed version information, but at the moment I'm only including "gcloud-dotnet" as the application name. We could put the version number in if required though.


Procedures with same name and different variables are treated as the same

DBeaver doesn't show the correct procedure details when there are more than one procedure with the same name but different variables: E.g. two functions getdimcells() and getdimcells(name) are defined.
This is in a Postgres database, using the General Postgres driver supplied with DBeaver.
In DBeaver:

The procedure tree list shows two getdimcells entries, but both without the variable names.

When double-clicking on each entry, only one of the definitions are shown, and the same one each time, getdimcells(name) in my case.

The same happens when ctrl-click on the code select * from getdimcells(name), I am shown the code for getdimcells() not getdimcells(name). When I hover over the function, I am shown getdimcells twice, but irrespective of which one I choose, I'm shown the code for getdimcells().

For reference, these show correctly in SQL Workbench and PGadmin as the two entries with the variable definitions. I.e. it's not a problem with the database.
From SQL Workbench

case 2 is still actual

Fixed

verified


ZEVA - History Display on Older Model Year Reports

Describe the Bug
In the MYR history, inaccuracies arise where internal users are erroneously marked as report submitters, and duplicated entries suggest a MYR was returned by an analyst twice. This issue was identified and resolved with card #2066 for the 2022 MYR, however persist with older MYR.
Expected Behaviour
The MYR history should accurately display the true actions completed.
Actual Behaviour
Despite addressing the bug for the current MY report (2022), older MYR history retains inaccuracies, misidentifying internal users as submitters and displaying duplicated analyst return entries.
Implications
These inaccuracies may lead to user confusion and compromise the accuracy of historical MYR data.
Steps To Reproduce
Steps to reproduce the behaviour:
User/Role:

As a government analyst, access the production environment.
Navigate to the assessment tab of various MYR records (Tesla 2021 and Toyota 2021.)
Observe instances where internal users are misidentified as submitters or where duplicated entries for MYR returns by an analyst are present.

Hey team! Please add your planning poker estimate with Zenhub @JulianForeman @tim738745

Code changes were made that should fix the Tesla 2021 report history, but the Toyota 2021 report history accurately reflects the history data in the database. So it's the data that's incorrect, and we would have to go into the database to correct that. As for why the data is incorrect: I think it's because
(1) requests are sent to the server asynchronously, and
(2) the analyst clicked the "Save" button and then immediately clicked the "Recommend" button (or the other way around)
We can address this issue by making it so that other buttons are disabled when a user clicks on "save" or "recommend" (this can be another card).
@shayjeff we can do the data change at the same time as the next release; please let me know if this works for you!

@tim738745, that works for me.


Backfill reference tokens and create a new one when refreshing sign in token

Context
Currently, we have a single field hashed_sign_in_token on a Reference. This means we can only associate a single sign in token with a reference at a time. This causes problems when we send a chaser reference request email as it overwrites the saved token of the first reference request email, rendering the initial email useless.
We therefore want to be able to associate multiple tokens with a reference and created a migration to add a new table called ReferenceToken in https://github.com/DFE-Digital/apply-for-postgraduate-teacher-training/pull/1617.
Changes proposed in this pull request
This PR:

adds a migration to create new ReferenceTokens for current references that have a hashed_sign_in_token
updates ApplicationReference#refresh_feedback_token! to create a ReferenceToken with the hashed token for the reference

This will enable us to keep ReferenceToken up to date and have backward capability for when we want to solely use ReferenceToken.
Guidance to review

Should the backfilling migration be in a separate PR?
Does the tests make sense?
Should the update! and create! be in a Transaction?

Link to Trello card
https://trello.com/c/Io6avau8/1119-improve-user-journey-when-referees-click-on-their-first-email-after-a-chaser-has-been-sent
Things to check

[ ] This code doesn't rely on migrations in the same Pull Request (🤷♀)
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

Should the update! and create! be in a Transaction?

That's not a bad idea, yes!

@chubberlisk this code does not rely on migrations in the same PR! If you remove the migration, nothing breaks, because nothing relies on the new behaviour. 😉


jit(arm64): support for select, swap, unreachable, globals

part of #187

[x] unreachable
[x] select
[x] swap
[x] global.get
[x] global.set

Done. So I will polish comments tomorrow then ready

thanks for the look!


514 - bring back mailbox size metric

Closes #514

Summary:
(please input the summary)

@adamgadomski I created an issue for a potential improvement of this metric: https://github.com/ScalaConsultants/mesmer/issues/615
For now merging as is.


LED switch does nothing.

The LED switch does not work for my RWP24 by Mangotek (Firmware: 3.0.0).
I am trying to upload the diagnostics but GitHub says that they aren't a supported file type.

Does the switch work in the native ewelink app?

Closed without response


How to Make it's EXE fully Undetectable?

I gave it to my VM and it's Norton detected it easily.???

Probably because of the UPX packer.
Try to work with these tools:
https://github.com/trustedsec/unicorn
https://null-byte.wonderhowto.com/how-to/bypass-antivirus-using-powershell-and-metasploit-kali-tutorial-0167601/

@PulkitSingh256 I should add that getting it to go FUD is very difficult to impossible for a public-repository RAT but I'm setting the help wanted tag if anyone has any suggestions on the topic.

How did the shell = Dispatch
Dispatch not defined error solved


OneHotEncoder feature mismatch on training/test samples

Unsure if this is just a problem with how I handle things, but let's say that I train my estimator and have a OneHotEncoder transformer.
If one of my categorical features has 4 distinct values, (let's use colors for sake of simplicity),
I will get
Red  n_1
Green n_2
Yellow n_3
Blue n_4
So, my training data will have sample values which match these additional columns because all of those colors exist in some place in my training data.
However, if I have test samples which does not have those columns, there is no way to "fit" the columns to the training data. Additionally, the methods I would need to get the categories from the transformers from within my pipeline are protected, so I can't actually know how the columns were built.
Is this normal, and if so, am I suposed to do some pre-processing/detection before I usethe OneHot on my learner so that I can also fit manually myself?

Hi @Kalyse thanks for the question ... I'm assuming you got it working since the issue is closed now but just in case ...
The One Hot Encoded can be fit to whatever dataset you want suing the fit() method ... if you are using a training/testing split for cross validation purposes, and for some reason your testing set contains a categorical features not present in the training set, you can simply fit the One Hot Encoder to the entire dataset before splitting.
Example
use Rubix\ML\Datasets\Labaled;
use Rubix\ML\Transformers\OneHotEncoder;

$dataset = new Labeled($samples, $labels);

$transformer = new OneHotEncoder();

$transformer->fit($dataset);

[$training, $testing] = $dataset->split(0.5);

or, you can fit and transform in one go using the apply() method on the dataset object ...
...

$dataset->apply(new OneHotEncoder());

[$training, $testing] = $dataset->split(0.5);

Let me know if that help, and thank you again for the question


Add CSS Framework

Add CSS Framework / Group choose Bulma as the CSS Framework (using LiveShare ext. in VScode to colab with Kyle Sunday and Chris Williams)

Added Bulma found CDN when looking up link. Minor corrections to css.


Can it support kotlin multiple platforms?

Hi. I dont like the KMP approach. So feel free to fork this library and build on top of it.


CI failing with "We've detected multiple sites inside your repository"

Describe the bug
I have a CI deployment that has been working fine until it started using v16 of the CLI.
The command I have within my deployment script looks like this
npx netlify-cli deploy --prod --dir=./webclient/dist

But when that command now runs, and pulls down v16, the deployment fails with
We've detected multiple sites inside your repository
? Select the site you want to work with (Use arrow keys or type to search)
❯ @shiftsync/apiclient  apiclient  --filter @shiftsync/apiclient
  @shiftsync/core  core  --filter @shiftsync/core
Error: Process completed with exit code 130.

I specified the version to be 15 and deployment ran through ok, letting it pull 16 again results in the same error, every time.
My repo has the following structure:

The command is run from the root of the repo, passing in the webclient/dist folder once the build has finished earlier in the CI process.
Steps to reproduce
I'm unsure how to repro this outside of a CI environment. If I try and run the same command locally, after building the web app, the cli client ends up publishing a blank site.
Configuration
No response
Environment
System:
OS: Linux 5.15 Ubuntu 20.04 LTS (Focal Fossa)
CPU: (2) x64 Intel(R) Core(TM) i9-9900 CPU @ 3.10GHz
Memory: 22.16 GB / 31.35 GB
Container: Yes
Shell: 5.0.16 - /bin/bash
Binaries:
Node: 18.16.0 - ~/.local/state/fnm_multishells/22934_1693641170757/bin/node
npm: 9.5.1 - ~/.local/state/fnm_multishells/22934_1693641170757/bin/npm
npmGlobalPackages:
netlify-cli: 15.8.1

Noticing this too on v16. Downgraded and pinned to v15.11.0 to get our CI/CD pipeline for deployments working again.

Hey, @scottlovegrove, thanks for reaching out.
First of all, multiple things are happening here; let me maybe explain to you:
We've released a new major (breaking) version that added better monorepo support. We rely on Semantic Versioning, where a new major version indicates that this contains breaking changes and might break existing workflows.
here are the docs for it: https://docs.netlify.com/cli/get-started/#work-with-monorepos
and here is the changelog post: https://www.netlify.com/blog/better-monorepos-on-netlify/
Sadly, we could not introduce this new feature without breaking existing workflows.
The significant change is that if you run from the root of a monorepo, we need to know which part you want to work on. A command is always tied to serve one app or deploy one app so we need to know which app is that -> therefore you can see the prompt. This is mainly required for new ways to store and read up configuration files (see docs).
You can bypass this prompt by changing to this directory before or using the --filter flag in CI/CD environments.
@South-Paw the reason on why the CLI deploy cannot omit the --filter flag when an explicit dir is specified is that a dir does not match a site. We are still missing which configuration file to pick up or about the siteId, which is stored in the .netlify/state.json. As a monorepo can have 1 to n sites stored in it we cannot have this information stored on the repository root.
A good example where the publish directory differs from the package location is Nx (a popular monorepository build system). There the publish directory would be dist/apps/app-1 and the configuration files would be stored at: apps/app-1/{netlify.toml,|netlify/state.json}
/
├─ apps/
│  ├─ app-1/
│  │  ├─ .netlify/state.json
│  │  ├─ package.json
│  │  ├─ netlify.toml
│  │  ├─ ...app-1-source-files
│  ├─ app-2/
│  │  ├─ .netlify/state.json
│  │  ├─ package.json
│  │  ├─ netlify.toml
│  │  ├─ ...app-2-source-files
│
├─ other-packages/
│  ├─ component-library/
│  │  ├─ package.json
│  │  ├─ ...component-library-source-files
│
├─ package.json

@scottlovegrove I hope this helps troubleshoot this issue for you. If there is still an issue or something does not work I'm more than happy to help you get it sorted out if you provide more information (framework, folder tree, etc.) or maybe a simple reproducible example.

@lukasholzer 👋
Our monorepo doesn't actually use .netlify/state.json files in any projects, we just pass the env variable NETLIFY_SITE_ID with the deploy command in GitHub action workflows.
We don't use NX, just npm workspaces and dist folders end up inside the app directories - not the root. I've also raised #6000 as somewhere along the line the CLI now manages to deploy our entire project to the Netlify site rather than the specified dir... which was fantastic to discover.
It just seems like Netlify CLI is making a lot of assumptions about project toolchain and setups that are then being built into the CLI commands (e.g. filter and now the deploy cwds)

Sorry for the frustration @South-Paw.
Regarding the .netlify folder, this is something internal netlify is creating and relying on for storing internal state and build outputs like bundled functions etc.
Now if you have multiple applications, we cannot store them in the root as this would collide.
If you don't need those things, then it's one point, but others might do. I was not thinking that you use Nx just wanted to point out that it is not that easy to omit the --filter flag when a dir is provided as it might fit your use case, but we have to consider way more ways to develop in a monorepo.
If you could maybe tell us some of the hiccups and things that frustrated you on debugging this, this would be so helpful so we can try to make it more obvious and better in the future.
There will be always bugs, so anything we can do to make troubleshooting/issue creation easier is something I would love to do!

Hi @lukasholzer ,
We are experiencing a similar issue after upgrading netlify-cli from 15.11.0 to 16.4.2. We too are now getting the "We've detected multiple sites inside your repository" message when running netlify dev. Note that we do use npm workspaces and have two package.json files: One at the root of the repo, and one in a subfolder of root. The problem is that when running netlify dev, the "We've detected multiple sites inside your repository" list prompt only displays one site (the one in the subfolder) and not the one on the root level. Do you know if there is a way to have it pickup the root level?
Note: We've tried netlify dev --filter [root package.json name property value] but it still prompts us to only select the one in the subfolder.
Thanks!

@jgeringer I would love to understand your use case more on having a workspace, but it seems that the root is a package as well?
So while implementing the workspace feature, we've tried to stick to the implementation logic of pnpm workspaces, yarn workspaces and npm workspaces where the goal is to have everything in sub packages distributed to the workspace.
There is indeed a way how to bypass the --filter flag and fall back to the old behavior by providing the --cwd . flag (when running from the root).
In this case, you tell the CLI that you know where the current working directory is, and we don't have to perform heuristics about where we can detect it in a workspace.
Nevertheless, I would love to understand your use case or maybe you can share some details about your workspace structure!

Hi @lukasholzer , thanks for the quick reply!
Yes, that's correct, in our case, the root is a package as well. Great to know about the --cwd flag, haven't seen that one in the cli docs. Is that page, and the getting started page, the most up to date place to find info about the Netlify CLI?
In our case, we would like to be able to run a script in one of the packages in a sub directory first, and then when that is done, run a script in the root package. If they were broken out into sibling directories, they wouldn't be able to communicate in that order. Sounds like we would have to re-work our directory structure later on or use the --cwd flag. Thank you!

@lukasholzer rather than relying on an undocumented --cwd flag, would it be better if netlify-cli were updated to use something like is-interactive to determine whether or not to prompt the user for monorepo sites?

This was incredibly frustrating. I just want to upload a directory as a netlify website. Everything's already built and in the correct folder.
I tried:

netlify deploy --dir=packages/website/out causes the We've detected multiple sites inside your repository error (https://github.com/ForbesLindesay/atdatabases/actions/runs/7505463095/job/20434753256)
cd packages/website && netlify deploy --dir=out causes the Site not found. Please rerun "netlify link" error (https://github.com/ForbesLindesay/atdatabases/actions/runs/7505707413/job/20435547129)
netlify deploy --filter @databases/website --dir=packages/website/out also causes the Site not found. Please rerun "netlify link" error (https://github.com/ForbesLindesay/atdatabases/actions/runs/7505811436/job/20435883558)
netlify deploy --cwd . --dir=packages/website/out is back to the We've detected multiple sites inside your repository error (https://github.com/ForbesLindesay/atdatabases/actions/runs/7506008454/job/20436482067)

The magic incantation that finally worked (https://github.com/ForbesLindesay/atdatabases/actions/runs/7506096220/job/20436748850) is:
netlify deploy --cwd . --filter @databases/website --dir=packages/website/out

Could you maybe add a simple command for uploading/deploying a directory? If I wanted netlify to magically figure out how to install dependencies & build the site for me, I'd be using netlify's CI service. The whole reason I'm using the CLI is because I want to just upload an already built directory to an existing site.

@ForbesLindesay for that there is the --cwd command this should not need the --filter.
So, in your case, netlify deploy --cwd packages/website/out should be enough.
I understand your request, but the CLI must work for complex scenarios where we detect a framework inside a monorepo, as well as a lot of customers facing this challenge. Using the CLI just to upload a directory is not really common.

Instead of using the "non-public" --cwd flag you could just set the cwd of your github actions task or CI runnter to the cwd of the monorepo like apps/my-app this should work as well

Is there documentation around the recommended command for deploying a monorepo app with the netlify cli? Particularly, which options should be present in addition to --filter?


Two addition ops from DiHT

text_action_filter: Filter to keep texts those contain actions in the text.
text_object_dependency_filter: Identify the objects in the text which are independent with other token, and filter them.
The text containing no objects will be omitted.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: root:x: BeachWangroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add support to replay failed Data Warehouse Events

I have a number of failed Data Warehouse Events due to an authentication issue which I've now addressed. However, Jitsu does not seem to be able to replay/resend events to the Data Warehouse.
It would be great if there was an ability in Jitsu to "replay" events to the Data Warehouse.

We have a multiple replay mechanisms. If it didn't work for your case, could you be more specific? What data warehouse you used? Was it streaming or batch mode? How long authentification issue persisted? What is the version of Jitsu you've been using

Yes, Jitsu replays events automatically. That being said, there's a certain time window when Jitsu tries to do so. 2 days may be over this limit. @absorbb should know better, so I'd leave it for him


is this discontinued?

the default Jellyfin integ in HA is lacking, this one looks more capable, but I can't get it to work in HA 2023.5.x. Are there any updates planned? Thanks!

same feeling i have, i recently moved from plex to jellyfin and the official integration is lacking functionality, this one seems better but it is clashing with the official somehow..

Official is uninstalled and yet this cannot be set up in 2023.8 (and earlier) at all. "Invalid flow specified"

With the update to Jellyfin 10.9 the official integration is not working anymore and it doesn't look like my issue is getting any attention, what is the state on this integration, is there any development here planned?

Guess this one is dead.
Official integration isn't really helpful, and creates abandoned media_player entities here and there....
I'm curious if MQTT can help. There's Websocket plugin for Jellyfin, that can post info into MQTT. It would be cool, as this would be push instead of poll.

Guess this one is dead.
Official integration isn't really helpful, and creates abandoned media_player entities here and there....
I'm curious if MQTT can help. There's Websocket plugin for Jellyfin, that can post info into MQTT. It would be cool, as this would be push instead of poll.

Yeah, that's what I ended up doing, using the Webhook plugin in Jellyfin I post messages to MQTT when playback starts, pauses or stops, I only use this for the Jellyfin client on my Nvidia Shield so it wasn't too much of a hassle to set up and works instantly, even faster than with the official integration when it was working.


Enable ignition UI for deployments via hardhat scripts

Resolves https://github.com/NomicFoundation/hardhat-ignition/issues/839
Tested whatever situations I could think of and everything looked okay. Below is a sample output of a run on a local non-ephemeral network with an immediate follow up run that deploys nothing:

Zoey and I reviewed this, and decided we should add a boolean flag to control whether the UI should be displayed or not, which should default to false. We'll document this in the scripts guide, but it will work out of the box for test use cases (i.e. no unexpected output in tests).


MODELIX-628: play remote changes from modelix into MPS

https://issues.modelix.org/issue/MODELIX-628

@nkoester branch is ready for review.


Create bundle without minification (need for Salesforce LWC support)?

I am trying to get this library to work within Salesforce LWC environment under something called Salesforce Locker.
Essentially all standard browser APIs are monkeypatched and many are not available (e.g. replaceChildren). Also, there is no npm, no bundling and all you can do is provide a dist js file and load it.
Because of that I grabbed the prebuilt dist and for some calls I was able to patch them. However other methods are minified such that it is extremely difficult for me to understand which specific browser API is failing.
Is there a command I can run to generate the browser build without minification? e.g. just bundle but not minify.

Please use this instruction to run the project locally on your computer. You can actually stop after the npm install step, because all the other steps are needed to run the dev version.

The resulting files will be in the packages/build directory.

I hope I can close this issue.


Can't find view when running published release build

Hello
The following code works in dev, but not on production (deployed to IIS)
     var viewAsPdf = new Wkhtmltopdf.NetCore.ViewAsPdf(_hostingEnvironment.ContentRootPath)
            {
                PageMargins = new Margins(0, 0, 0, 0),
                PageSize = Size.A4,
                IsLowQuality = false
            };

            return  await viewAsPdf.GetByteArray("Views/Contract.cshtml", sale);

It apparently can't find the view, which makes sense, since the views are compiled into the dll.
I don't know however how to get it running.

Hi, which version of the library are you using? and what OS?

I am on 1.1.3, I see there is a 2.0.1, I'll update to that version and report back to you. OS is Windows Server with IIS.

2.0.1 resolved all my issues, thanks.

nice, thanks!


Tables from MSSQL not showing on Table View

Expected results
MSSQL database tables showing on Table View
Actual results
No MSSQL tables showed
Screenshots

Database configuration

Tables detected

Table view

Environment

superset version: 0.28.1
python version: 3.6
node.js version: 10.16.0
npm version: 6.9.0

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

You need to add each table, they are not automatically added (click the button on the upper right hand corner on the tables page). There has been a few important fixes for MSSQL after version 0.28, so you might want to try one of the more recent versions if you run into trouble.

I tried with more recent version which is 0.33.0rc1, but still got the same output.

Every table has to be manually added to Superset; on the tables page, click the plus sign on the to right hand corner of the page to create a datasource referencing the table in your database.

Alright thanks.

@syazwan0913 was there still something that required help/fixing, or can this issue be closed?

I already add manually the table from MSSQL database
Schema: dbo
Table: Main

But when I tried to refresh metadata, this error appeared,

@villebro @mistercrunch So any way to fix that?

Same error here

Solution here https://github.com/apache/incubator-superset/issues/4271 thanks to @googlerobot
Run query below at the Superset Metabase
ALTER TABLE public.table_columns
ALTER COLUMN "type" TYPE varchar(128);

@vitorbellini So I need to run the query on SQL Server database right?

@vitorbellini I already can add the sql server tables. Thank you so much

@vitorbellini I can already add the sql server tables. But for the column "name" there is "COLLATE SQL_LATIN1_GENERAL_CP1_CI_AS" ?

@vitorbellini Thanks. I already can add sql server table. Btw, is there any way to fix this bug by changing the source code?

@syazwan0913 are you able to test #7963 which should fix this problem?

I already tested it with master branch and it is okay.


Update common files

Manal changes were done to one of the shared files without removing the notice. I'm closing to avoid overwriting and removing notice to disable the sync.


Upgrade angular framework from 6.x to 8.x

Upgraded angular frame to latest stable 8.x version
Fixed the regular expressions Cross-Site Scripting (XSS) vulnerability [CVE-2019-16769] due to indirect serialize-javascript dependency
Tested all the make commands

this resolves #18 and resolves #19

thank you @akhilpb001 for the fix


Project file cleanup

Just removing a bunch of files I think we don't need. @INCENDE please verify

need collate tui for collating our final finals

@INCENDE Yes we do but we don't need it to my part of our own codebase. We don't own Collate, it lives in it's own project here. Whoever runs collate can just download it from source, and run it themselves. Those binaries don't need to be part of our repo

Alright, so let's remove this after we've finalised our code and docs

I have my own copy of Collate.jar so I'll help to generate collation even after removing these.
We might be changing docs/code even at the last 1hr to submission so waiting until Monday 11pm to removing many files in the repo might be dangerous.


How to enable TLS in paid setup

I am trying to enable TLS by flipping flags and setting CORE_PEER_TLS_SERVERHOSTOVERRIDE in every peer, still I am not able to instantiate chaincode, when I looked up in peer logs it says "userChaincodeStreamGetter -> ERRO 001 Error trying to connect to local peer: x509: certificate signed by unknown authority"

"userChaincodeStreamGetter -> ERRO 001 Error trying to connect to local peer: x509: certificate signed by unknown authority" is this coming from the chaincode container? Looks like the chaincode container is not liking the certificate from the peer?


Add admin to OnBan event on RA bans

RA bans sends null as the admin object in the OnBan event.

This should be fixed since Jan 13th


[TestGen] Support api-scenario

Things to do:

Introduce or modify the models related with api-scenario in the code model (oav 3.0 introduces a breaking change)
Determine which solution we want to use for the extra dependencies of the test cases (Azure.Core.TestFramework and Azure.ResourceManager.Resources for arm template)
Design some data structures for api-scenario in MgmtTest namespace (parses the things in code model into the things generator could recognize)
Write writers to write the new tests

Depend on the plan for API Scenario


Add builder-bid compare to the proposer-config

Description
Right now, the https://docs.teku.consensys.net/reference/cli#builder-bid-compare-factor applies across all validators that are connected to a particular deployment of a beacon client.
We would like this to be available at the validator level (like mev-boost relays) using the proposer config file that already exists. This would allow different validators to set different factors that reflect their individual purposes / perspectives

Since we have completed the blockV3 implementation, we now have the backbone to implement this on VC configuration side. What's the appetite for this @beetrootkid ?


Fixes leak with creating editor settings

Fixes #41020
When cherrypicked may help #40994

Thanks!

Cherry-picked for 3.2.3.


Replication agents provisioning / shorthands

see usage: https://github.com/Cognifide/aem-project-archetype/blob/master/src/main/archetype/build.gradle.kts#L118
aem {

    instance {
        provisioner { // https://github.com/Cognifide/gradle-aem-plugin/blob/master/docs/instance-plugin.md#task-instanceprovision
            enableReplicationAgent("author", "publish", publishInstance)
            deployPackage("com.neva.felix:search-webconsole-plugin:1.3.0")
        }
    }

}

todo create replication agent / configure should enforce that proper page is created


Installation

Hi,
could you please add a word in the README on how to install.
I have tried
micro -plugin install micro-run and
micro -plugin install run without any luck...
Thank you in advance!
Ilya

This is outdated, see https://github.com/terokarvinen/micro-run/issues/2#issuecomment-2200202839


fix: non shell based socks scan...

Not most efficient way to handle socket entries method, but it works
gomod upgrade to fix all CVEs

Using some foresight on this change, I changed the base image to scratch for the second part of the build, and I'm unable to get this to work. Any chance you'd like to merge these 2 pull requests you have open into something that you've verified works on scratch and update one PR and close the other? Or just open another?
Your contributions look super cool and I want to get them pulled in, but I am unable to get this working on my machine.

Merged latest into my master. Tried running #7, It's working fine, Also tried web socket as well.
Can you try this https://github.com/vmorganp/Lazytainer/pull/6#issuecomment-1236277637
I've implemented a new version of Dockerfile which is more cache efficient, faster, and you can build multi arch in your local as well. You can say it's a new generation of docker image builder. GitHub actions use that as well.


signature created using nostril is not working with branle.netlify.app when its used to sign an event created on branle

Using throwaway keys
private key
9d00d99c8dfad84534d3b395280ca3b3e81be5361d69dc0abf8e0fdf5a9d52f9
public key
e8caa2028a7090ffa85f1afee67451b309ba2f9dee655ec8f7e0a02c29388180
For text created on branle.netlify.app as mentioned in noscl video:
{
  "pubkey": "e8caa2028a7090ffa85f1afee67451b309ba2f9dee655ec8f7e0a02c29388180",
  "created_at":<PHONE_NUMBER>,
  "kind": 1,
  "tags": [],
  "content": "test12",
  "id": "b9accb8f240b02dafffd3f8fdb70df4b6db0f0c3b71087a79bcb5b842a1a4441"
}

when this is run through nostril to gets its signature, we get
./nostril --sec  9d00d99c8dfad84534d3b395280ca3b3e81be5361d69dc0abf8e0fdf5a9d52f9 --content '{
>   "pubkey": "e8caa2028a7090ffa85f1afee67451b309ba2f9dee655ec8f7e0a02c29388180",
>   "created_at":<PHONE_NUMBER>,
>   "kind": 1,
>   "tags": [],
>   "content": "test12",
>   "id": "b9accb8f240b02dafffd3f8fdb70df4b6db0f0c3b71087a79bcb5b842a1a4441"
> }'
{"id": "2013ac8971d1a2fa2986849badf211cc37c7e82883f8180897857aba1109d8f5","pubkey": "e8caa2028a7090ffa85f1afee67451b309ba2f9dee655ec8f7e0a02c29388180","created_at":<PHONE_NUMBER>,"kind": 1,"tags": [],"content": "{\n  \"pubkey\": \"e8caa2028a7090ffa85f1afee67451b309ba2f9dee655ec8f7e0a02c29388180\",\n  \"created_at\":<PHONE_NUMBER>,\n  \"kind\": 1,\n  \"tags\": [],\n  \"content\": \"test12\",\n  \"id\": \"b9accb8f240b02dafffd3f8fdb70df4b6db0f0c3b71087a79bcb5b842a1a4441\"\n}","sig": "ad8ea6c9f24f692b6d49494ee14a016ee364358c21eb6bdeb762f5215e8a6b4ab4f3a6df5b1fddacae6ec42ce381579ee5796f8756dd7becdfa8bc32320543e1"}

And when that given signature ( ad8ea...) is put for that string (test12), branle says that its not a valid signature.
More info on issue here

I think you want --content test12

Ok, got it, needed to give that way, and use the --created-at arg
 ./nostril --sec  9d00d99c8dfad84534d3b395280ca3b3e81be5361d69dc0abf8e0fdf5a9d52f9 --content  test12 --created-at<PHONE_NUMBER>

Where created at was the one mentioned in the branle created string.
Then am able to post. Cool, will close the issue, thanks.


add .nvmrc file

Need
sane-reports works best with node 10.24.1, so we added .nvmrc to make this useable with nvm

@idanizi-panw great idea.
Question regarding the node version, I'm using version v12.20.1 (no special reason).
How did you decide to select v10.24.1 as the default?

@idanizi-panw great idea. Question regarding the node version, I'm using version v12.20.1 (no special reason). How did you decide to select v10.24.1 as the default?

@yonatan-roth by running all npm scripts I had to fall back to different node versions until I got to this one that doesn't fail at every npm script we have on package.json

@Bars92 can you approve 🙏


Actors behaving like locks when they should not

I'm getting the old bug sometimes, when actors start behave like locks (opening popup menu on right click and try to open on left click).
It's not consistent and I can't really tell what is making this happen, but when it does, I have to delete the token (reload won't change anything) and drag the actor in the scene to fix it.
I wish I could tell you more so you can reproduce it but it just happen randomly or I didn't get the pattern yet.

I will look into it.

Assuming the "Show all lock interactions" setting is turned off, which interaction options show up in the popup menu?

I'll tell as soon as I get the bug again

It seems like when "Show all lock interactions" setting is turned off, the popup menu is not showing up at all, but when I left clic on the actor, it acts as a lock "Actor is locked", and playing the lock sound.

Could you enter this `canvas.tokens.controlled[0].document.flags.LocknKey' into the console(F12) while having the token in question selected and send me a screenshot of the result?

Sure, as soon as the bug is back i'll do it.

Having disabled the "show everything" on the popup option, I don't see the bug anymore (at least for now).

Yes, it seems like the popup menu "show all" could be the problem.

That narrows down to possible causes, i will look into it

I have looked further into the issue, is it possible, that the token(s) to which the bug applies were locked when the previous bug allowed some unintended tokens to be locked and never unlocked again?

I'll put the option back on and try with fresh tokens

I'm not seeing any bug so far so you're probably right ! If I find anything i'll let you know

I will write a macro that will remove the faulty data from the invalid prototype tokens, just to make sure

I have added debug/clean up code with v2.0.8. The code should remove faulty data entries from all prototype tokens. You can execute the code by either typing game.modules.get("LocknKey").api.CheckActorPTokensCompatibility() into the console or executing it as a macro. Make sure that Item Piles is active during that, since the debug code will not recognize item piles actors/tokens as compatible otherwise

Perfect, thanks man.

For now the problem should be counteracted by using the "Always open owned" word setting. Could you send me the console output when having one of this bugged tokens selected and entering canvas.tokens.controlled[0].document.flags.LocknKey into the console? Also, are you using rideable and if so, do you have the rideable-lock & key integration activated?

Yes, the "always open owned" resolved the issue (idk why I didn't do that before).

Yes, I use rideable and I have the rideable-lock & key integration activated. But my token is not "ridable" and I tried turning off ridable (and the integration) but the bug is still there.

I have fixed a small error with v2.1.1 that may have been responsible for this behaviour.

So, when testing rideable, I found out about this bug ! It was not really a but, actually the actor of the token must have been a pile or a rideable BEFORE (and set as locked) then been back to not rideable + not a pile, but stayed locked (and the setting to lock/unlock was not available anymore since its not a rideable nor a pile)... I fixed it by puting it as rideable, unlocking then puting it back as non rideable.
Do I make sens?

That makes sense, would explain all the strangeness surrounding this behaviour. I will think about a way to prevent this from happening in the future.

Yes, I should have figured this out before... But I dont remember locking the acotr that way (but I was testing so much that it's possible). I'll let you know if I find something else but I don't think so.

Are there still problems related to this issue or is everything solved? If so i would close this issue.


expand variables in DSL file path

Our project currently has a very complex DSL that we would like to move to a file. Unfortunately we need to be able to use variable in the DSL file path so we know where to load it from, which is something that the build-flow-plugin doesn't currently support.
This is a simple change to allow variable expansion in the DSL file path.

The jenkins failure seems like a spurious test problem; is there a way to re-run?

@carej
Since I am currently looking for someone else to take over as a maintainer; I will not be the one merging this change. Please check http://jenkins-ci.org/pull-request-greeting.
Side note: I think introducing new features in this plugin is risky and takes away resources that could be better spent on the pipeline plugin. However, please feel free to merge if you are interested in carrying on the flag.

@carej , I don't know how to trigger the tests again manually other than amending the changes (e.g. whitespace change / commit message)


React Support?

I was wondering if Lozad works with react? I'm currently having trouble getting it work, I was trying to utilize lifecycle methods and have it not start observing until after the component is rendered. However, it seems like based on the nature of react and vdom, it doesn't actually work because my component is rendered before Lozad is initialized.
Any ideas? I was trying something along the lines of:
    observer = lozad('.lozad', {
        load: function() { console.log('loaded'); }
    });

    componentDidMount() {
        this.observer.observe();
    }

Thanks in advanced!

@ApoorvSaxena Does lozad accept a dom element instead of a selector?
If so, you could do something like the following:
class LozadWrapper extends React.Component {
  componentDidMount() {
    const observer = lozad(this.img, {
      rootMargin: '10px 0px', // syntax similar to that of CSS Margin
      threshold: 0.1 // ratio of element convergence
    });
    observer.observe();
  }

  render() {
    const  { src, srcset } = this.props;
    return (
       this.img = el} data-src={src} data-srcset={srcset} />
    );
  }
}

ReactDOM.render(<LozadWrapper src="image.png" srcset="image.png 1000w, image-2x.png 2000w" />, someRootElement);

I would like to work on this.
I can think of two approaches to implement it:

Either accept two different types in first parameter i.e either a dom node or a selector which I think is not a great idea for a variable to have two completely different type of values.
Another way is to pass the dom node in options as the second parameter but It'll be confusing to determine what will happen with both first & second parameters being present.

Any help on how to go about doing this ?

Either accept two different types in first parameter i.e either a dom node or a selector which I think is not a great idea for a variable to have two completely different type of values.

Why not? This is what jQuery does. You can say $('.some-class') or you can say $(document.getElementById('some-id')). This is commonly used inside an event handler like so:
$('.btn').click(function() {
  $(this).css('color', 'purple'); // make the clicked button purple
});

@PezCoder I agree with @styfle, you can go ahead with approach 1, of using either a dom node or a selector.
Regarding:

Another way is to pass the dom node in options as the second parameter but It'll be confusing to determine what will happen with both selector & DOM node present.

Options is not the right place to pass selector or DOM element as it's not a configuration setting.

To accept just the selector but somehow keep track of dynamically inserted selector, something like what jquery's $.on() does. That should keep the library usage the same but should make everything work with frameworks/libraries that inject the elements later in the page i.e after lozad's initialisation.

This is performance intensive task, may even bloat the library size if using a dependency.

Agree, @ApoorvSaxena @styfle The first approach seems optimal, I'll create a PR right away.

Thanks guys! Then, we can add a reference on the image within your react component and pass that to lozad as opposed to a class?
@PezCoder @ApoorvSaxena
Let me know if I can help in any way 😃

I think i went in wrong order of doing things.. 😅
@pixelcure I setup a quick react application with create-react-app
& did this:
  constructor() {
    super();
    this.observer = lozad();
  }

  componentDidMount() {
    this.observer.observe();
  }

  render() {
    return (

    );
  }

This seems to be working fine, I got this DOM node inside my html

i.e with src & data-loaded = true..
Can you create a plunker/jsbin with minimal set of instructions that reproduces your specific issue ?

@PezCoder do you mind sharing the react implementation in plunker/jsbin yourself?

Codepen:
https://codepen.io/PezCoder/pen/mBaOer
Lesson learnt: To not dive in for a solution without verifying the issue first 😄

@pixelcure can you verify the solution as proposed by @PezCoder?

closing this issue for now

Can't get rootMargin option work with the @PezCoder example.

@blainemuri Here's a custom hook I made that should do the trick.
import { useEffect } from "react";
import lozad from "lozad";

function useLozad(selector, options) {
  useEffect(() => {
    const observer = lozad(selector, options);
    observer.observe();
  }, [selector, options]);
}

And here's a CodeSandbox demo: https://codesandbox.io/s/suspicious-wilson-dj2oq

Remember to clean up the subscription when using it in a useEffect:
useEffect(() => {
  const { observe, observer } = lozad()
  observe()
  return () => observer.disconnect()
}, [])


Toggle component does not propagate :hover to label node

I've use an SVG for my label prop on the Toggle component, it displays fine. However the Toggle component does not propagate :hover / hoverColor to the label node. If I place an SvgIcon component next to Toggle it receives the hover event fine and changes color, not so when used as the label node for Toggle.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

I expect the color of my SVG icon to change when hovering over it.
Current Behavior

Hover event not received, no change in SVG style.
Steps to Reproduce (for bugs)
https://codesandbox.io/s/42j70ln7m9

View Code Sandbox example
Mouseover both SVG images, Toggle component doesn't change SVG color on :hover.

Context
UI polish, not essential but would be nice to have it working.
Your Environment
Some details below but issue reproduced in Code Sandbox.

Tech
Version

Material-UI
0.20.0

React
15.6.2

Chrome
63.0.3239.132

Firefox
57.0.4

@gknapp We have been porting the component on the v1-beta branch. We have reimplemented it from the ground-up.
We encourage people starting with Material-UI to go with v1-beta. The master branch is in a low maintenance mode. We don't plan any work on it. However, we still do our best to review and merge pull requests that fix bugs on this branch.


e2e test TestL7NetworkPolicy failed frequently

Describe the bug

=== RUN   TestL7NetworkPolicy/HTTP
=== RUN   TestL7NetworkPolicy/HTTP/Ingress
    l7networkpolicy_test.go:117: Creating ANP test-l7-http-allow-path-hostname
    l7networkpolicy_test.go:117: Creating ANP test-l7-http-allow-any-path
=== CONT  TestL7NetworkPolicy/HTTP
    l7networkpolicy_test.go:177:
        	Error Trace:	/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:177
        	            				/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:226
        	Error:      	Received unexpected error:
        	            	command terminated with exit code 1
        	Test:       	TestL7NetworkPolicy/HTTP
    l7networkpolicy_test.go:178:
        	Error Trace:	/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:178
        	            				/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:226
        	Error:      	Not equal:
        	            	expected: "test-l7-http-server"
        	            	actual  : ""

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-test-l7-http-server
        	            	+
        	Test:       	TestL7NetworkPolicy/HTTP
    l7networkpolicy_test.go:186:
        	Error Trace:	/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:186
        	            				/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:226
        	Error:      	Received unexpected error:
        	            	command terminated with exit code 1
        	Test:       	TestL7NetworkPolicy/HTTP
=== RUN   TestL7NetworkPolicy/HTTP/Egress
    l7networkpolicy_test.go:117: Creating ANP test-l7-http-allow-path-hostname
    l7networkpolicy_test.go:117: Creating ANP test-l7-http-allow-any-path
=== CONT  TestL7NetworkPolicy/HTTP
    l7networkpolicy_test.go:177:
        	Error Trace:	/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:177
        	            				/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:253
        	Error:      	Received unexpected error:
        	            	command terminated with exit code 1
        	Test:       	TestL7NetworkPolicy/HTTP
    l7networkpolicy_test.go:178:
        	Error Trace:	/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:178
        	            				/home/runner/work/antrea/antrea/test/e2e/l7networkpolicy_test.go:253
        	Error:      	Not equal:
        	            	expected: "test-l7-http-server"
        	            	actual  : ""

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-test-l7-http-server
        	            	+
        	Test:       	TestL7NetworkPolicy/HTTP
I0316 18:15:33.285951   18872 framework.go:2380] Sending SIGINT to 'antrea-agent-coverage'
I0316 18:15:33.521355   18872 framework.go:2386] Copying coverage files from Pod 'antrea-agent-9x9m5'
I0316 18:15:34.166477   18872 framework.go:2380] Sending SIGINT to 'antrea-agent-coverage'
I0316 18:15:34.334870   18872 framework.go:2386] Copying coverage files from Pod 'antrea-agent-bvhm7'
I0316 18:15:34.854880   18872 framework.go:2380] Sending SIGINT to 'antrea-agent-coverage'
I0316 18:15:35.051624   18872 framework.go:2386] Copying coverage files from Pod 'antrea-agent-sncxc'
=== CONT  TestL7NetworkPolicy
    fixtures.go:294: Exporting test logs to '/home/runner/work/antrea/antrea/log/TestL7NetworkPolicy/beforeTeardown.Mar16-18-15-48'
    fixtures.go:465: Deleting 'testl7networkpolicy-c0n353cp' K8s Namespace
I0316 18:15:53.219788   18872 framework.go:673] Deleting Namespace testl7networkpolicy-c0n353cp took 9.398443ms
--- FAIL: TestL7NetworkPolicy (123.05s)
    --- FAIL: TestL7NetworkPolicy/HTTP (85.70s)
        --- PASS: TestL7NetworkPolicy/HTTP/Ingress (35.50s)
        --- PASS: TestL7NetworkPolicy/HTTP/Egress (42.17s)

https://github.com/antrea-io/antrea/actions/runs/4439739159/jobs/7792942498?pr=4713
Except the failure itself, it was also reported to wrong testing.T, leading to unclear failed case like above.

Thanks for adding this issue, I'm debugging now.

I'm not sure why the test fails frequently in test-e2e-encap-all-features-enabled, but I think the root cause should be that interfaces for L7 NetworkPolicy are not set with no-flood  correctly.
When I reproduced the issue on my local testbed, I found the counter of the flow (the third one in the following) for L7 NetworkPolicy is 0 in L2ForwardingOutTable
 cookie=0x52010000000000, duration=677.587s, table=Output, n_packets=0, n_bytes=0, priority=211,reg0=0x100/0x100,reg4=0x400000/0xc00000 actions=output:NXM_NX_REG1[],output:NXM_NX_REG9[]
 cookie=0x52010000000000, duration=677.587s, table=Output, n_packets=0, n_bytes=0, priority=211,reg0=0x100/0x100,reg4=0x800000/0xc00000 actions=output:NXM_NX_REG9[]
 cookie=0x52020000000000, duration=677.584s, table=Output, n_packets=0, n_bytes=0, priority=211,ct_mark=0x80/0x80,reg0=0x100/0x100 actions=move:NXM_NX_CT_LABEL[64..75]->OXM_OF_VLAN_VID[],output:"antrea-l7-tap0"
 cookie=0x52030000000000, duration=677.585s, table=Output, n_packets=3052, n_bytes=1031068, priority=210,ct_mark=0x40/0x40 actions=IN_PORT
 cookie=0x52010000000000, duration=677.587s, table=Output, n_packets=3079, n_bytes=287714, priority=200,reg0=0x100/0x100 actions=output:NXM_NX_REG1[]
 cookie=0x52000000000000, duration=677.599s, table=Output, n_packets=0, n_bytes=0, priority=0 actions=drop

I also found that the there are plenty of ARP packets in the flows for the Pods that are used to test L7 NetworkPolicy like the following:
 cookie=0x52010000000000, duration=909.010s, table=ARPSpoofGuard, n_packets=3, n_bytes=126, priority=200,arp,in_port="antrea-gw0",arp_spa=<IP_ADDRESS>,arp_sha=96:18:60:e9:62:73 actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=908.994s, table=ARPSpoofGuard, n_packets=0, n_bytes=0, priority=200,arp,in_port="coredns--70e4ae",arp_spa=<IP_ADDRESS>,arp_sha=32:f8:32:ad:cd:58 actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=908.993s, table=ARPSpoofGuard, n_packets=0, n_bytes=0, priority=200,arp,in_port="coredns--d02e00",arp_spa=<IP_ADDRESS>,arp_sha=12:4d:cb:85:a1:1a actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=106.233s, table=ARPSpoofGuard, n_packets=3, n_bytes=126, priority=200,arp,in_port="x-9c3krw-5c9d49",arp_spa=<IP_ADDRESS>,arp_sha=fa:54:af:eb:db:99 actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=106.128s, table=ARPSpoofGuard, n_packets=3, n_bytes=126, priority=200,arp,in_port="y-9c3krw-8769fb",arp_spa=<IP_ADDRESS>,arp_sha=c2:ab:69:ee:cb:6f actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=106.125s, table=ARPSpoofGuard, n_packets=3, n_bytes=126, priority=200,arp,in_port="y-9c3krw-a6faab",arp_spa=<IP_ADDRESS>,arp_sha=82:80:70:d6:46:71 actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=105.406s, table=ARPSpoofGuard, n_packets=3, n_bytes=126, priority=200,arp,in_port="z-9c3krw-ce2ac8",arp_spa=<IP_ADDRESS>,arp_sha=b2:9c:82:72:cc:31 actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=85.214s, table=ARPSpoofGuard, n_packets=1647079, n_bytes=69177318, priority=200,arp,in_port="test-l7--bd9a4a",arp_spa=<IP_ADDRESS>,arp_sha=a6:e0:e9:b9:93:4d actions=resubmit(,ARPResponder)
 cookie=0x52010000000000, duration=82.200s, table=ARPSpoofGuard, n_packets=1650323, n_bytes=69313566, priority=200,arp,in_port="test-l7--a23e2d",arp_spa=<IP_ADDRESS>,arp_sha=4a:72:29:90:04:da actions=resubmit(,ARPResponder)
 cookie=0x52000000000000, duration=909.022s, table=ARPSpoofGuard, n_packets=22362502, n_bytes=939225084, priority=0 actions=resubmit(,ARPResponder)

There is a chance that the test client Pod cannot send the packets because it cannot get the ARP response from the target server due to the ARP storm.
After debugging, I found that NO_FLOOD is set as expect. Since #4517, ovs-ofctl mod-port br-int 6 no-flood -O Openflow15 is used to set NO_FLOOD for a port, and this doesn't work as expected.


Update array compatibility tests

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

Project coverage is 100.00%. Comparing base (93befb8) to head (ac42d86).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #644   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        32
  Lines         2749      2757    +8
  Branches       349       357    +8
=========================================
+ Hits          2749      2757    +8

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


Unable to download package

[InvalidArgumentException]
Could not find a matching version of package coolsam/laravel-mongodb-transactions. Check the package spelling, your version constraint and that the packa
ge is available in a stability which matches your minimum-stability (dev).
require [--dev] [--prefer-source] [--prefer-dist] [--no-progress] [--no-suggest] [--no-update] [--no-scripts] [--update-no-dev] [--update-with-dependencies] [--update-with-all-dependencies] [--ignore-platform-reqs] [--prefer-stable] [--prefer-lowest] [--sort-packages] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--] []...

Any solution yet?


[bitrich-info/xchange-stream] [Huobi] Websocket MarketData (#579)

Take 2

Dependent on
[Gemini][Huobi][BitStamp] OpenOrdersParams and TradeHistoryParams and bug fixes

Dependent on
[Gemini][Huobi][BitStamp] OpenOrdersParams and TradeHistoryParams and bug fixes

Is this another PR, @mdvx ? I can't find it.

After several attempts, I got it all in this single PR.  Thats why there a couple Gemini and Bitstamp updates mixed in to the 22 file updates.


Add Support for Laravel 5.5

Coverage increased (+81.3%) to 81.25% when pulling 53906f4b3ebea470e9383b4c9c8439874ab12ad8 on support-laravel55 into 703b40af80e4e6b114a78ffcdfa13414fc3ef171 on master.


Playwright comparison

Clear and concise description of the problem
As a developer using Vitest, I want to be able to read a Playwright comparison so that I understand which tool to use in which situation.
Suggested solution
Playwright is a fairly new testing and automation framework from Microsoft, developed by the same team that created Puppeteer. It supports various programming languages and testing types such as end-to-end, API, UI component and unit tests all within a single tool and a single API.
TODO: add comparison

Uses Vite
Similar API to Vitest and Jest
Simular feature/tool set as Cypress
Fast, executes optionally in a real browser
VS Code integration like Jest and Vitest
Parallel tests optionally over multiple machines with sharding
No smart & instant watch mode yet
No out-of-the-box code coverage yet

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

What is the status of this?

I use Playwright for end-to-end testing and vitest for unit and integration testing. The reasoning here is that vitest is much better suited for unit and integration testing as it can leverage testing library to render ui components outside of a browser environment. It also featuers watch functionality which is essential in when doing test-driven development. As a bonus it is much faster than Playwright.

And now Vitest has "browser mode" which launches Playwright (or webdriverio). What are the interests of launching the tests via Vitest and not directly via Playwright ?


chore: use same prettier setting as Kibana's

closes #151
Copied over .prettierrc from Kibana so that we align the lint rule across the team within the organisation.
No need to review every single file that has changed, but take a look at .prettierrc file.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-04-05T12:42:38.577+0000

Duration: 14 min 26 sec

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

closing in favor of #197.
Figured it is better to start from a clean state rather than resolving conflicts.


feat(p-board): selectable option

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[ ] Not that difficult

Type of Change

[ ] New feature
[ ] Bug fixes
[x] Feature improvement

[ ] These changes require a usage change

[ ] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[x] Updated Storybook documents for component
[ ] Wrote test codes for composable, utils and etc.
[ ] Tested with console(if usages are changed)

Description
SSIA
PBoard selectable

PBoardItem selected

Things to Talk About

:tada: This PR is included in version 1.1.0-beta.499 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


make install fails on Alpine Linux 3.13

The 'make install' command shall most probably fail on a typical Alpine Linux setup, which does not have bash shell by default.
If the 'graphviz' package is already installed (if not just do '# apk add graphviz'). On executing 'make install' one sees the following error messages:
make: /bin/bash: No such file or directory make: /bin/bash: No such file or directory make: /bin/bash: No such file or directory go install -tags "" -ldflags "-X main.commit=" make: /bin/bash: No such file or directory make: *** [Makefile:22: install] Error 127

Solution is extremely trivial. Open the Makefile in an editor and modify the line:
SHELL = /bin/bash
to
SHELL = /bin/ash
After this change, 'make install' should succeed.


Create a New Tab landing page

Create an HTML page to redirect new tabs to.
Future Version: Allow users to inject their own page.
May need to find a way to disable extensions on just this page.

Done in #7


File Structure & Scripts Updated

File Hierarchy :

Created testbench, design and common dirs
Made the tb module, core module and top integrated module seperate
Filelists added for each sub block of the repo ( tb, design and top )
Added include files for easier pkg inclusion ( not using the verilog package construct here )

Makefile :

Created labels for running different sub blocks individually without affecting the others
Cleaned up the makfile and the sim path
Added a requirement to set the CORE_REPO_PATH env variable to run the Makefile

We need my changed to be committed first. I think there will be a merge conflict with your PR and the unmerged changes.


rules_go improvement to externalize the nogo fix

What type of PR is this?

Feature

What does this PR do? Why is it needed?

Problem
nogo linters may produce fixes as analysis.Diagnostic.
However, the fixes are not externalized properly.

Solution
In this PR, we externalize the fixes if any by (1) declaring a fix path for each build target, (2) propagates the fix to the output group accessible externally.

Note
Also note that, during the externalization, we need to get rid of fileset since it is transient, instead, we use per-file offset.

Test
Tested below (some info hidden for privacy)
bazel build  --output_groups=nogo_fix --norun_validations  //... --override_repository=io_bazel_rules_go=~/Uber/rules_go
Target //src/code.uber.internal/infra/progsys/renovate/change-exporter/subjecttargets:uselessloggerwith_example up-to-date:
bazel-bin/src/.../uselessloggerwith_example.nogo.fix

Other notes for review

thanks for the comments, i am in the process of addressing them.

this shows the log of the latest version:
https://gist.github.com/peng3141/bdaefac333434cf2ecbef4edfd8d0200

@fmeum @linzhp the PR is ready for review.
could you focus on the high-level design, once it is agreed upon, we can have one pass to address the readability nits.

has been a while, back to this full time.

addressed comments from @linzhp.
made the following design-level change:

move most logic out of rules_go into a third_party patcher tool (@linzhp let us discuss where we can host this, initially we can put it in my personal github repo, but in the long run, we may want to host it more properly).
rules_go does the minimal work now to export the []edits for each file and analyzer.

This way, we make sure we do not impact rules_go's performance.
Accordingly, we shall change the expectation of this PR as follows:
this PR does not provide e2e patching support.  It only exports the fixes.
ready for review, @linzhp and @fmeum , thanks!

@linzhp
let me address your high-level feedback first.
your proposal does not really work:
$ patch -p1 < bazel-out/k8-fastbuild/bin/src/example.com/foo/go_default_library.nogo.assign.patch
$ patch -p1 < bazel-out/k8-fastbuild/bin/src/example.com/foo/go_default_library.nogo.composite.patch
patch is highly unreliable tool that is based on the surrounding context. After you apply the first patch, the 2nd patching may get the context messed up and is not able to proceed.
I do not like such unreliable patching. The only reliable way is to perform the precise offset-based editing. This imposes a strict requirement though: the edits users want to apply should not overlap. This means we have to have some tool that is better than patch.
Also I think this PR is only for exporting edits, right? this is already an improvement over existing status that no fixes can be exported.

addressed the comments, tested with locally with

multiple files have errors of different analyzers.

one outstanding question is about output_group vs nogo_validation, I am not sure about advantage of output_group.
also i think nogo_validation offers more flexibility, e.g., it can print patch message immediately after error message.

discussed offline and made these improvements:

also will add the printing of patch in validation action immediately after the errors.

Also add the nogo_fix to output_group support the case of not running validations.
- tested via $ bazel build --output_groups=nogo_fix --norun_validations

there is no need to have two fix files, one for run_nogo and one for validation action. We just need one file. Actually validation does not need to write to it at all.

half way, will handle the rest tmr.


Profiling and speeding up Lakeroad's usage of Rosette

We need to make Lakeroad faster. I think we can speed it up significantly, if we can figure out how to use Rosette more intelligently.

Some profiling on a specific test case.
Ran the following:
raco symprofile --racket -- $LAKEROAD_DIR/bin/main.rkt   --verilog-module-filepath /Users/gus/lakeroad/integration_tests/lakeroad/three_stage_multiplier_lattice.v   --architecture lattice-ecp5   --template dsp   --out-format verilog   --top-module-name three_stage_multiplier   --verilog-module-out-signal p:16   --initiation-interval 3   --clock-name clk   --module-name out   --input-signal a:16   --input-signal b:16

Had to apply the following change to get any useful info (symprofile doesn't like call-with-limits):
diff --git a/bin/main.rkt b/bin/main.rkt
index b21ea3a..56d10b2 100755
--- a/bin/main.rkt
+++ b/bin/main.rkt
@@ -375,17 +375,18 @@
                                             (apply set (map (compose1 string->keyword car) env))))
                       (current-error-port))))

-       (call-with-limits
-        (timeout)
-        #f
-        (thunk (rosette-synthesize
+      ;;;  (call-with-limits
+      ;;;   (timeout)
+      ;;;   #f
+        ;;; (thunk
+        (rosette-synthesize
                 (compose (lambda (out) (assoc-ref out (string->symbol (verilog-module-out-signal))))
                          bv-expr)
                 sketch
                 input-symbolic-constants
                 #:bv-sequential envs
                 #:lr-sequential envs
-                #:module-semantics module-semantics))))]
+                #:module-semantics module-semantics))]

     [else
      ;;; If initiation interval is #f, then do normal combinational synthesis.

Got these results:

I'm surprised to see that it's actually the multiplier semantics that are so slow -- and they're significantly slower than anything else. I don't know why they are so slow. The only thing that looks off about these numbers (when comparing them to ALU row) is the union size.

One thing to look into: Perhaps generating merged-input-state-hash through many appends is a bad idea; we could do it all at once.

It is weird that the major slowdown only shows up when we have two modules in sequence. We aren't nearly as slow when we're just working with a single DSP48E2.

Oak took a look at this during ASPLOS push and couldn't find much beyond some obvious cleanup we need to do.


Question: Why are no codecs registered when creating a Subscriber?

Hey all, Ive been playing around with Ion and ran into an issue where the offer generated by the server for a subscriber was not valid on Safari. After doing some poking around i realized no codecs are registered for the subscriber media engine when it's created. I'm curious why this is the case.
I was able to solve my problem by registering default codecs here https://github.com/pion/ion-sfu/blob/d1c505133a34ad3e16c820b8bc521c580801ba71/pkg/sfu/subscriber.go#L35-L39

My thought around this is that you only want to register client supported codecs on the subscriber side?

This is strange, subscribers does not register codecs because it registers the codec send by other publishers:
https://github.com/pion/ion-sfu/blob/d1c505133a34ad3e16c820b8bc521c580801ba71/pkg/sfu/router.go#L247
Maybe the issue is that publisher codec was not supported by Safari?

After taking a brief look it seems that this registers the codec that the media is currently being sent on. Wouldn't it make more sense to register all the codecs that the publisher is capable of sending?

SFU will not transcode, and and the moment will not renegotiate codecs in case is not compatible, if we do this, if Chrome publish VP9, Safari will try to select Vp8 or H264 supported codecs, which will not make sense since stream is VP9.

SFU will not transcode, and and the moment will not renegotiate codecs in case is not compatible, if we do this, if Chrome publish VP9, Safari will try to select Vp8 or H264 supported codecs, which will not make sense since stream is VP9.

Got it, after doing some SDP research and getting a better idea of how negotiating codecs works I figured out what we need to do. Thank you!


Changed default string type to be str

This impacts the following APIs:
Metadata: Key is always a str, Value is bytes for binary metadata,
str otherwise
Call Details: str type
gRPC method: str type
hostname/target: str type
I've also moved the byte/string conversion to right below the API surface.  The only reason it was ever at the Cython level was because we had tests for multiple implementations.  Now that all implementations pass through the same GA API, it makes more sense to do the byte/str conversion there.
Also note that we no longer support byte or str input.  All inputs should use the str type.

Will take the line-by-line content of this pull request to breakfast but in the meantime: uh, you deleted an entire unit test?

I've opened a separate PR for that: https://github.com/grpc/grpc/pull/7086
_connectivity_channel.py is completely unused except for this unit test, and because the test exposes the _low.* API, it would require making the legacy API Python3.4 compatible.

Looks sound to me overall; I'd like @thunderboltsid to also review.

Addressed comments

@nathanielmanistaatgoogle
All relevant tests are passing.

I don't like the performance smoke test result; it's the first time I've seen that test fail with only Python test elements failing and it doesn't correspond to failures on master. How closely have you looked at that as possibly indicating a problem with this change?

test this please

Seems fine by me. :)

Looks mostly good to me; just those small items and one question.

Made suggested changes

One last nit; sorry.

Are the performance tests for this still an open mystery?

No, the issue with the performance tests was that the channel args would not encode a python2.7 unicode argument.  We had a channel argument derived from a proto, and I guess the proto was returning a python2.7 unicode string value.
The smoke tests passed about an hour ago (after @thunderboltsid's comment)

Made suggested change.

Luhguhtum.

@jtattermusch This looks ready to merge, the test failures are jenkins flakes.


Make more styles configurable w/ variables

I make use of a lot of bootstrap variable overrides in my app and my goal is to share them all in a partial between my web & email layouts. This is an amazing start towards achieving that, but I noticed a lot of things that are configurable with variables in bootstrap 4 aren't in this gem, so I started converting everything hardcoded that I currently use into a variable. There is obviously still a lot of work to do here, which I'll try to keep submitting PRs for as I add more overrides to my app.
Some notes:

.btn td a had default border color #e9703e which is orange ... seemed strange, so I changed it to transparent which is the bootstrap 4 default. This doesn't seem to come into play anyway since this will be mixed with the btn-x styles but I still thought it worth pointing out the change.
I'm not sure why border-radius has to be specified at .btn, .btn td and .btn td a but I'm leaving as-is as I assume there's a good reason for it
I actually have $headings-margin-bottom: $font-size-base / 2; in my app (same as bootstrap 4), but I saw that line was commented out in the code, so I removed it and left the default 0 so as not to mess with anyone else
body had both margin and Margin properties, so I removed the redundant capitalized one
It's unclear to me why things like the spacing utilities are duplicated between the head.scss & _spacing.scss files. Why not just @import sass/spacing in head.scss?

@stuyam just pinging you to make sure you've seen my PRs

@stuyam hate to keep nagging you as I know you're probably quite busy, but I'd love your thoughts on the rest of the points above

@stuyam any chance you'll have time for this? If not, I'd like to discuss taking over maintenance here.

@stuyam what's it going to take to move this forward?

Sorry life gets busy, merged!


Service inject problem

Hi,
My Service is not working in component. Am I doing something wrong? I did everything exactly like https://github.com/AngularClass/NG6-todomvc-starter
My repository:
https://github.com/IdaszakDaniel/MRTask

https://github.com/IdaszakDaniel/MRTask/blob/master/client/app/components/about/about.controller.js#L13
Your service returns promise.


Features 1266 1267 1278 add transport networks and components

Summary of the discussion
This PR implements the concepts from issues 1266, 1267, and 1278 together, because they are closely related.
Type of change (CHANGELOG.md)
Added
Added new classes with axioms:

transport network: A transport network is an object aggregate of transport network components that enables the transport of people and/or goods.

Axiom: 'has part' some 'road'
rail network: A rail network is a transport network that enables transport on rails.
Axioms: 'has part' some 'railway'
'has part' some 'train station'
waterway network: A waterway network is a transport network that enables transport on water.
Axiom: 'has part' some 'port'
aviation network: An aviation network is a transport network that enables air transport.
Axiom: 'has part' some 'airport'

transport network component: A transport network component is an artificial object that is part of a transport network.

As comment: It can hold other transport network components such as rails and roads.
tunnel: A tunnel is a transport network component that is built through a certain environment (e.g. a mountain or water) and allows to pass through that environment.
As comment: It can contain other transport network components such as rails and roads.
railway: A railway is a transport network component that can only be used by rail vehicles.
canal: A canal is a transport network component that is an artificially created waterway.
transport hub: A transport hub is a transport network component that allows the exchange of people and/or goods.

train station: A train station is a transport hub for trains.
   - freight train station: A freight train station is a train station for the exchange of goods.
   - passenger train station: A passenger train station is a train station for the exchange of passengers.
bus station: A bus station is a transport hub for busses.
port: A port is a transport hub for ships.
   - freight port: A freight port is a port for the exchange of goods.
   - passenger port: A passenger port is a port for the exchange of passengers.
airport: An airport is a transport hub for airplanes.

Workflow checklist
Automation
Closes #1266
Closes #1267
Closes #1278
PR-Assignee

[x] 🐙 Follow the Pull Request Workflow
[x] 📝 Update the CHANGELOG.md
[x] 📙 Add #'s to term tracker item

Reviewer

[ ] 🐙 Follow the Reviewer Guide
[ ] 🐙 Provided feedback and show sufficient appreciation for the work done

It looks fine so far, but I think we can improve axiomatise the relation between the transport network components and the vehicles introduced in PR #1293. Could you please add the following axioms:

railway 'is used by' some train
road 'is used by' some 'road vehicle'
airport 'is used by' some 'aircraft'
'bus station' 'is used by' some bus'
port 'is used by' some ship
'freight port' 'is used by' some 'cargo ship'
'passenger port' 'is used by' some 'passenger ship'
'train station' 'is used by' some train
'freight train station' 'is used by' some 'freight train'
'passenger train station' 'is used by' some 'passenger train'

Good idea. I made the changes.

Thanks for reviewing.


Register (HTML/CSS Only)

You will be working on the register.html page
Requirements:

[ ] Write the logo, "DevSpace" in a similar position to the Wireframe
[ ] Create a label with the text, "Email" and have it in a similar position to the wireframe
[ ] Create an input with a placeholder text, "Write your email here" and have it in a similar position to the wireframe
[ ] Create a label with the text, "Password" and have it in a similar position to the wireframe
[ ] Create an input with a placeholder text, "Write your password here" and have it in a similar position to the wireframe
[ ] Create a button with the text "Register"
[ ] The color patter/design can be of your choice.

You create this in a format of Form if preferred. Make sure it has an email, password, and submit to make the form work.
Wireframe:

In the ticket: Register Screen (JavaScript) please write in the comments that this is ready to be worked on once this is completed and in the main branch.


Add staking and multisigcontrol to checkpoint utility

A few more fields were added to the checkpoint, so they need adding to vegatools checkpoint because funnily enough it sometimes can really help debugging snapshot issues.

Ah it was already done between me sitting on the changes locally for a week, and now :D


update best practices to include process failed suggestion

Docs Build status updates of commit e60f5f6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Ready for docs team review

Docs Build status updates of commit 217f172:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3135a4b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9282eb6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f002210:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3a49fe4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/concepts/user-data-folder.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 722cb48:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/concepts/user-data-folder.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 41d5b37:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/toc.yml
:white_check_mark:Succeeded
View

microsoft-edge/webview2/concepts/developer-guide.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/concepts/user-data-folder.md
:white_check_mark:Succeeded
View

microsoft-edge/webview2/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Is this project still active?

It seems this is not meshing with the latest versions of Foundry well.  For example, I cannot seem to pull in data from an .HDC file; the character remains with default stats and no visible powers.
I'm trying to coach my expectations and get a feel for whether this will be picked up again, or if I should get my big-boy britches and try to fork this and update it.  (Background but no current expertise... it would be tough.  No one expect results if I do that, please.)

Hey there, you should probably just fork it. I got burnt out on Hero and
have stopped running it, which has largely caused me to lose interest in
developing what was really just a tool to make my life easier.
On Fri, Aug 5, 2022, 9:28 PM albijoe @.***> wrote:

It seems this is not meshing with the latest versions of Foundry well. For
example, I cannot seem to pull in data from an .HDC file; the character
remains with default stats and no visible powers.
I'm trying to coach my expectations and get a feel for whether this will
be picked up again, or if I should get my big-boy britches and try to fork
this and update it. (Background but no current expertise... it would be
tough. No one expect results if I do that, please.)
—
Reply to this email directly, view it on GitHub
https://github.com/jared-l-vine/hero6e-foundryvtt/issues/4, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDYRES7ZT45U2LKAWRABZTVXXETPANCNFSM55XYU4LA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Check out https://github.com/dmdorman/hero6e-foundryvtt, an updated fork of this repo.


fix:(tabs) add call for rechecking tab scroll distance

Add call to scroll the tabs header after initialising header pagination, to accurately calculate the scroll distance.
Fixes #12889

Hi, @andrewseguin sorry but I've forgotten all about this PR.
I've created a new PR here for this issue with the modifications suggested by Kristiyan, because I may have deleted the fork from my account and the repo name changed.


Numba decorators

I was thinking of trying to implement pycpd using numba decorators, and I was wondering if you had tried it yourself and whether you think it might speed up calculations significantly. Since pycpd is straight numpy, it sounds to me like it would, but I wanted to make sure I wouldn't waste time on something you had tried. Any thoughts?

I implemented some parts of the pycpd algorithm in cython to speed up those parts which are iterative and published it as cycpd: https://github.com/gattia/cycpd
I also added some of the other original methods from the CPD paper (low rank non-rigid/deformable transform) into cycpd. I have since incorporated them back into pycpd as well - though only on development branch for now. See pull request/merge here: https://github.com/siavashk/pycpd/pull/37.
From what I played around with it seemed the next thing to do to speed things up was to add the Fast Gauss Transform (FGT) from the original CPD paper. I have written about this on the readme/intro page of the cycpd repo. I haven't looked into it much, but it seems like FGT should speed up all of the methods (rigid, affine, deformable).

Hi Anthony. Thanks for the link. It looks awesome. I will definitely try it out and compare. Just out of curiosity, have you tried numba decorators out? If so, how do they compare with your cython compilation?

Arthur, I dont have any experience with Numba. Though, it could be interesting due to the supposedly easy leveraging of GPUs. The below blog makes a few comparisons between Cython/Numba. I used the Cython version as a chance to learn some more Cython code as well as to speed up the python (cython) cpd version.
https://lewiscoleblog.com/cython-numba
Sounds like testing Numba is worth a try. The only thing I've read is that Numba might be more finicky to distribute than Cython.... but that might be an old problem that is solved now.
As a different note, @siavashk originally intended this to be a numpy version for teaching purposes. Though, Numba doesn't seem like it will change the code much, so maybe he's open to adding Numba, or maybe a Numba branch to keep things a little separate. Alternatively, you could create a new repo based that builds off of pycpd like I did with cycpd.

Sorry I am kind of swamped with work. I haven't tried numba but I have heard good things about it. If you fork the project and make changes, I would be happy to help you. It would be a learning experience for me as well. I believe that the code would still be numpy readable even with numba, so people can still use it for educational purposes.
It would be really nice if you could make an example with numba decorators and show that it is faster. Something like run 1000 registrations, one with numba and one without numba and see if it is faster. This would make a good case for the integration.

@siavashk @gattia  thank you both very much for the feedback. It seems that this might be an experiment worth doing. I will fork the repo and progressively work on it over the coming months. At first glance, it seems (from the numba docs) that it would be an easy conversion. I will keep you both posted. Thanks again.


Is it security- and privacy-wise safe to test Sensitive info types with real data?

Due to #870 we were asked to upload real world data to test a sensitive info type provided by Microsoft. Is it safe to do so? Like what are the security and privacy-wise aspects of uploading test data to the sensitive info types test function?
Is it the same level of privacy as if the function/feature would test data on e. g. a SharePoint Online? Does Microsoft uses this uploaded test data by their own for improving their own services/tests?

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

ID: dcb7953e-5257-2578-4607-2d93940d6845
Version Independent ID: 8e59d041-619e-d54f-f28e-faac2bc18ef2
Content: Create a custom sensitive information type in the Security & Compliance Center - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/create-a-custom-sensitive-information-type.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@GiantCrocodile - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP.


Can custom password settings override Azure AD password age setting or not?

I got to this page because I was warned about having AADDS passwords expire after 90 days even if my Azure AD passwords have a longer maximum age. I don't want my users prematurely locked out of AADDS. But there is a sentence on this page that says "Settings like password complexity, age, or expiration time [apply?] only to users manually created in a managed domain." So I don't know if I'm able to simply uncheck the box to enforce maximum password age and have my Azure AD age limits control the experience for users in the AADDS environment.

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

ID: 2eb4c941-c464-75cb-7a98-39c3a58be1b5
Version Independent ID: e93b1a8f-3de2-309e-5a0b-923e2fa15e0b
Content: Create and use password policies in Azure AD Domain Services
Content Source: articles/active-directory-domain-services/password-policy.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @Justinha
Microsoft Alias: justinha

@DavidSchrag Thank you for your query . We will investigate ands update the thread .

@DavidSchrag sorry for the confusion. I'm going to remove the first bullet you cited because it doesn't seem accurate or make sense, let alone the lack of a verb. I think this para earlier in the topic is most relevant for your concern:
If you have an Azure AD password policy that specifies a maximum password age greater than 90 days, that password age is applied to the default policy in Azure AD DS. You can configure a custom password policy to define a different maximum password age in Azure AD DS.  In that scenario, a user's password may expire in Azure AD DS before they're prompted to change in Azure AD or an on-premises AD DS environment.


⚠ Spryr has degraded performance

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

HTTP code: 200
Response time: 5576 ms

Resolved: Spryr performance has improved in 00fbed6 after 9 minutes.


Is it possible to use a proportional font?

I haven't used it yet (my bad) but it would nice to mention in the README if this is possible.

Thanks for the issue report.
Proportional font is not available at this time.
Maybe we can implement such a feature in the future.
I'm going to mention this point in README.


Error HTTP/500: Object reference not set to an instance of an object.

Provide a detailed explanation of the issue in English.

Error HTTP/500: Internal Server Error.
Object reference not set to an instance of an object.
Trace Id: 00-f9633128e12fd24dbe88f4513e37392b-1059fe64c9cf8d4d-00
Version: 0.9.0 (0.9.8121.26944)

Thanks for reporting this. I'm closing this issue as a duplicate of #213.

Fixed in 0.9.1


SideProxy Crash

Unkown error

Actually the @Sidedproxy needs to be used on a variable, which is a commonproxy, just like


[Cryptography]Hill Cipher

It is a polygraphic substitution cipher.
Programming language

[ ] C
[x] C++
[ ] Java
[ ] Python
@Amit366 Please assign this issue to me I want to work on it

Please Assign me this issue. I will be solving it in JAVA. I am participating in LGM-SOC.


Cannot Collect All Fields for a Given Scope

https://github.com/baianat/vee-validate/blob/fac309bc77f4157c56fc263e7e3fbdbb30f29a48/src/errorBag.js#L101
The way the collect() method is written, it ignores a specified scope if no field name is provided. A more useful way to write this would be to collect all fields for the specified scope if the scope is specified but the field name is not.

I think this can use the same syntax as the validate function:
collect('field'); // collects for a field.
collect('scope.field'); // collects for a scoped field
collect('scope.*'); // collects for a scope.
collect('*'); // collects all scopeless fields.
collect(); // collects for all fields with/without scopes.


HDDS-4622. Use single ratis server in MiniOzoneCluster tests for SCM

What changes were proposed in this pull request?
The intention is to replace MockRatisServer by production Ratis server in integration tests (tests that use  MiniOzoneClusters)
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4622
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

cc: @GlenGeng


Adapt code to autocomplete module changes

This is to adjust the usage for the changes in https://github.com/Nexucis/codemirror-next-autocomplete/pull/1
Signed-off-by: Julius Volz<EMAIL_ADDRESS>

Note that this depends on the linked PR in the autocomplete module and will need a package.json dependency update for that before working here. But then it should look like this:


Question Extended Data Fig 4h

Hello,
I have a question about the recently published paper "The single-cell pathology landscape of breast cancer". I'm very nterested in Extended Data Fig 4h, could you explain to me how you processed the data?
If I understand this correctly, you used CellProfiler to create masks for tumor cells based on tumor cell markers, right? How and with which program did you then define the boundary and how did you determine the distance between the cell types and this boundary? I have found the Matlab code in the linked data on Github for this purpose but maybe I do not understand it correctly.
Thank you for your help!
With kind regards
Tim

Hi,
sorry for the late response, I somehow missed this issue.
Segmentation for tumor/stroma regions was done exactly like segmentation for single cells by first training a pixel classifier in ilastik and then segmenting the probability masks using CellProfiler. In the ilastik step a combination of all markers was used to train the classifier to distinguish tumor bulk from stromal regions. The resulting probability masks were then segmented in CellProfiler into masks defining for every pixel whether it is part of the tumor or the stromal region. The tumor/stroma mask can then be "overlaid" onto the single-cell mask and the distance of every single-cell object to the boundary of the tumor object can be calculated. We did this calculation using Matlab scripts but it can also be done directly in CellProfiler.
How to make tissue feature masks using the ImcSegmentationPipeline was discussed in more detail
here.

Thank you very much, your answer was very helpful!


the menu bar icon is not visible when dark menu bar is enabled

System Preferences > General

Dark menu bar disabled:

Dark menu bar enabled:

See solution https://github.com/SimonRemote/simon-mac/issues/9 for a workaround


Fix Database Table Overflowing on /companies

The table view in /companies is overflowing, due to the table having too many fields.
Some alternatives include:

adding a hyperlink to a popup containing all the company data, and only showing some small portion of data
adjusting the UI to allow for horizontal scrolling
make each entry a 2 line row

Johnny decided on creating a Show More button, which will open a modal and show all columns, but only render the following columns by default:

Entity Name
Status
Industrial/Chemical
Grower
Description
Stakeholder Category
Stakeholder Group
Development Stage
Product Group
Solution


Move nedoyyep folder one directory down.

Because of how the scaffolding worked we've got an extra folder in this repo. We'd like to have the setup.py and similar files in the root directory.

This was done in #4