Examples of shallow rendering tests?

Are there any examples of this, especially for apps which use the scss approach shown in https://github.com/shakacode/react-webpack-rails-tutorial?
The current testing approach (added in https://github.com/shakacode/react_on_rails/issues/84) makes it difficult to make unit-level component tests that are not coupled to the rendering/behavior of children components (this article discusses the tradeoffs of various React testing approaches)
I've tried, but my imported css object is always an empty object.  It would be good to see a working example...
Thanks,
-- Chad

@thewoolleyman, @alexfedoseev is working on a solution to this, FYI.

@justin808 Cool - is the work @alexfedoseev doing on any open source repo or branch that I can follow?

@thewoolleyman Previously I came up with this solution, later on I found this one, but haven't tried it yet.

@alexfedoseev looks good, but your webpack-fu is impressive, much greater than mine.  I recognize these as some of the same issues I ran into, but I don't have the context or experience yet to dig down to the root causes.
But I'm looking forward to someone smart like you figuring it out and making it part of the react_on_rails conventions.
And ideally, I'd love for it to be test-framework agnostic, and (fairly) easily support usage of tools like Karma and Jasmine.  But that's a separate discussion.
Thanks!
-- Chad

And ideally, I'd love for it to be test-framework agnostic, and (fairly) easily support usage of tools like Karma and Jasmine. But that's a separate discussion.

This is exactly why I'm not releasing my first solution as module. Babel plugin makes it test-framework agnostic and I want to try out it first. Hope I'll get to this point soon, b/c currently I have a few important things to do before I can get back to testing issue.

@thewoolleyman We're starting to get on the critical path to LAUNCH of our product, so fixing some of these is going to rely on some community help. Thanks for your patience!
BTW, I'm probably old school, but I still believe in mostly Capybara tests of the UI! Test the whole flow. You need that regardless, so if you duplicate the tests in shallow rendering, you have to fix 2 places when you change the logic. Many many years of doing this has given me much more comfort in SLOW capybara tests to ensure I can just hit merge and deploy!

If you want to talk old-school ;), this is still my go-to article when I explain how I like to think about categorizing tests:  http://groboutils.sourceforge.net/testing-junit/art_naming.html
But yes, application integration tests via Capybara of all the happy-path flows are indispensable, and with cloud-based parallelized CI not as much of a pain as in the past, but I still like to have some unit-level coverage, especially for error-handling or edge cases that are difficult or impossible to set up via Capybara.  Which means I like to have at least one unit test file for every real file, even if it just does a dumb render with no assertions, just to make sure there's no syntax errors in the happy path.
But, I'm still trying to figure out the right mix of these in React/Redux functional-world.  It does seem like unit tests of React components aren't as critical as the building blocks of other ecosystems, but I would still like to be able to do it via loosely-coupled shallow rendering when it does seem warranted.

@thewoolleyman if you're simply trying to do shallow rendering tests and get css modules imports to work, we do already have that working. The thing Alex is trying to do is completely leverage webpack, which is useful if you are using other loaders, lots of aliases, etc.
scripts/test
#!/bin/sh

NODE_PATH=.:./app \
$(npm bin)/mocha \
  --require app/libs/tests/setup.js \
  --compilers js:babel-register \
  --recursive 'app/**/*.spec.@(js|jsx)'

app/libs/tests/setup.js
import hook from 'css-modules-require-hook';
import sass from 'node-sass';

hook({
  extensions: ['.scss'],
  preprocessCss: data => sass.renderSync({ data }).css,
  generateScopedName: '[name]__[local]__[hash:base64:5]',
});

Oh just realized the title of the issue was an example of a shallow rendering test:
import { expect, wrapperFuncs } from 'app/libs/tests/utils';
import MyComponent from './MyComponent';

// SETUP
const defaultProps = {
  user: {
    avatarUrl: 'foobar.com',
    profilePath: '/profile',
  },
};

const { shallowWrapper, staticWrapper } = wrapperFuncs(MyComponent, defaultProps);

// TESTS
describe('<MyComponent />', () => {
  context('with required props', () => {
    it('renders an avatar', () => {
      const actual = staticWrapper().find('img').first();
      const expected = defaultProps.avatarUrl;
      expect(actual).to.have.attr('src', expected);
    });

    it('renders a link to the user profile', () => {
      const actual = shallowWrapper().find('a').first();
      const expected = { href: defaultProps.profilePath };
      expect(actual).to.have.descendants(expected);
    });
  });
});

app/libs/tests/utils.js
import chai from 'chai';
import React from 'react';
import chaiEnzyme from 'chai-enzyme';
import _ from 'lodash';
import { shallow as shallowRender, render as staticRender } from 'enzyme';

chai.use(chaiEnzyme());

export { shallow as shallowRender } from 'enzyme';
export { render as staticRender } from 'enzyme';
export const { expect } = chai;

export function wrapperFuncs(Component, defaultProps) {
  const createProps = (overrides = {}) => _.merge({}, defaultProps, overrides);
  return {
    staticWrapper: (overrides = {}) => staticRender(<Component {...createProps(overrides)} />),
    shallowWrapper: (overrides = {}) => shallowRender(<Component {...createProps(overrides)} />),
  };
}

@robwise Thanks a lot, those examples worked perfectly :)  I'll close this out.  Not sure if we have/want example of this in any of the example apps, if so we can open a separate issue...


Torii: show server address on init

Resolves: #1003
Added

Logging messages at server startup with server address

Almost feel bad to push 4 characters but it is how it is, let me know if I can miraculously add more lines. 😎

Great contribution thank you!


[TextServer] Fix SVG emoji placement.

Fix SVG emoji placement, which seems to be broken by ThorVG update.

Does this fix https://github.com/godotengine/godot/issues/80766? I wanted to add a bugsquad edit but I'm not sure.

No, Apple Emoji font is not SVG and will need a separate fix (I'm not sure what's wrong in this case).

Thanks!


Add note about port of X-Forwarded-Host to nginx.conf example

tusd will use X-Forwarded-Host as the host if -behind-proxy is enabled. However, the example Nginx config doesn't put the port number into X-Forwarded-Host, but only the address. This makes the URL returned by tusd to be on port 80/443 instead of what Nginx is actually listening on.
Changing proxy_set_header X-Forwarded-Host $host to proxy_set_header X-Forwarded-Host $host:$server_port can fix this problem. But this leads to the occurrence of :80/:443 in the returned URL even the client doesn't put :80/:443 in the Host header, which can be a little weird. So I just add a comment about this.
Related issue: #136

Thanks for the PR! We can also consider adding a logic in tusd's URL generation to drop the default ports for HTTP(S) if they are used.

I've stumbled upon the same problem. I've used the fix mentioned above but I think the absFileURL from unrouted_handler.go.
https://github.com/tus/tusd/blob/fb5b6209a7e1c807c9de8f081fa39ba8386b91d0/pkg/handler/unrouted_handler.go#L403-L404

Thank you for raising this issue. I updated our Nginx example and the URL generation code to exclude default ports.


Ping Spamming in some scenarios

Scenario 1: Adding a device was online, but goes offline then comes online again. But earlier bug means double ping!

It doesn't anymore. Or at least I think it doesnt


CICD is failing when run on Node 6

CICD is breaking when run on Node 6 due to ES6 usage in parrot-server, we should transpile parrot-server down to ES5.

Closed with 90f9e6d after internal PR and review.

Reopening since the issue still exists after transpiling.


Add create function CA for var type decleration and select-expr

Purpose
Fixes #40614

Shall we add a test case for the following as well
type MyType record {|
    string value;
|};

function func() {
    stream<MyType> streamName = stream from string letter in "JohnSnow"
        select createMyType(letter);
}

Added a test


Add new Spec

This PR creates a new Spec (v6) to change the current use of  to a pinned nixpkgs.

What does that mean?

This needs a changelog

@madeline-os tested by:
$ nix-thunk create https://github.com/plt-amy/1lab.git 1lab
$ cat 1lab/thunk.nix
# DO NOT HAND-EDIT THIS FILE
let fetch = { private ? false, fetchSubmodules ? false, owner, repo, rev, sha256, ... }:
  if !fetchSubmodules && !private then builtins.fetchTarball {
    url = "https://github.com/${owner}/${repo}/archive/${rev}.tar.gz"; inherit sha256;
  } else (builtins.fetchTarball {
  url = "https://github.com/NixOS/nixpkgs/archive/3aad50c30c826430b0270fcf8264c8c41b005403.tar.gz";
  sha256 = "0xwqsf08sywd23x0xvw4c4ghq0l28w2ki22h0bdn766i16z9q2gr";
}).fetchFromGitHub {
    inherit owner repo rev sha256 fetchSubmodules private;
  };
  json = builtins.fromJSON (builtins.readFile ./github.json);
in fetch json⏎
$ nix-thunk unpack 1lab
Cloning into './1lab-7dbbebbff4b4db50/.'...
HEAD is now at 6d51d2ea defn: start work on synthetic homotopy theory; n-truncations
✔ Fetched thunk 1lab

then poking around the files to make sure everything was in order, which it was.


Sticky Headers issue with three adapters when switching between them through check boxes.

I am trying to achieve the sticky headers with 3 adapters with 3 recyclerview and 3 check boxes. When you choose one check box, it will be checked and that recyclerview will be visible whereas the other two recyclerview visibility will be gone.
The issue that I am facing is that when I scroll down the list and then switch to the other recyclerview through the check box then the header item for the previous recycler view is shown. I am also using one sticky_header_layout for all three recyclerview. Your help will be really appreciated here. Thank you.

@aaru25kumar, that is a particular situation.
Since I reworked the logic of the sticky header in the last month, could you please check the new Snapshot that I just released.
If doesn't resolve your situation, then I can think some solutions, but maybe it is not necessary anymore.

Pay attention that, this snapshot brings a lot of changes to the whole library, it includes my last 2 months of development. Please read the release notes for this snapshot in issue #39.

I am actually using the version 5.0.0-b8 and I was looking into migrations where you have mentioned that we can use setStickyHeaders(boolean) instead of enableStickyHeaders(boolean) and not to use the sticky_header_layout. But I am haven't seen those changes in the library at all.

I can't change b8 content. That is published as it is for ever. In this phase of development, please use the snapshot version that receives continuous update from my dev.

I just switched from b8 to the snapshot and I am using the setStickyHeaders(true) to all the three adapters and also removed the include sitcky_header_layout in my xml file. But the issue is still present.

@aaru25kumar, depends what you show and hide when you switch RV, probably you have all in one Activity, I suggest to disable the sticky header and enable again for the new one RV. This cannot be done by the library because you are not detaching anything, just showing and hide something.
Most probably you don't need at all, to triplicate the RV widget, you only need to switch adapter!

I got it working as per your suggestion. I apologize for not leaving a comment sooner.


merge in seedlabeling vignette on readthedocs

as in the title, make sure this gets in the readthedocs

already handled on master of scvi-tools


wifi-2649 Fix Captive portal vifC vifS mismatch

Mismatch in vifC and vifS for captive portal is
leading to opennds restarts and memory leakage.
Fix by making sure the vifC and vifS are in parity.
uci_blob not being freed, fix by making uci_blob
as global so that it need not be freed and can
be reused instead.
Signed-off-by: Chaitanya Godavarthi<EMAIL_ADDRESS>Cleaning up captive portal code to improve stability and reduce memory leaks
Signed-off-by: Nagendrababu<EMAIL_ADDRESS>

Merged 31abbf39, thanks.


Add latitudes and longitudes to emissions/monthly/aust10km/2023/*/emissions.nc

The emissions files are tricky to work with, because there are no latitudes and longitudes in the files.  This makes it difficult view the data in a netCDF viewer and to overlay other geospatial data.
The solution could be to provide one of the following

New files with latitude and longitudes included for example CH4(lat,lon)

A python script that merges the latitudes and longitudes from domain_v1.d01.nc with the emissions files

Provide a custom projection string in the header of the emissions files - so the netCDF files can be reprojected using gdal (something like ‘+proj=lcc +lat_1=x +lat_2=y +lat_0=i +lon_0=j +x_0=0 +y_0=0 +datum=WGS84 +units=m’ )

If the new netCDF files can be viewed using an off-the shelf netCDF viewer like Panoply - then this would be really useful.

Hi @sarah-shannon , thanks for raising this issue. Apologies for the delayed response.
I don't have a current ETA for a fix, but I am looking into what it would take to restructure our results files to make them easier to work with in panoply.

I think this is addressed in #117 but we've not distributed updated
files yet. There could be a delay on that with Christmas approaching.

Sorry about that, GitHub automation closed the issue by mistake. Re-opening and I'll give an update when we've uploaded fixed datasets.

@sarah-shannon and @prayner , I've made some updates to our results post-processing to add lat/lon, projection details and coordinates, and a time dimension among other things.
I've uploaded a sample of the new file based on our 2023-01 month result: https://openmethane.s3.amazonaws.com/develop/monthly/2023/01/emissions.nc
@sarah-shannon if you get a chance, could you grab a copy and let me know if you have any feedback? Does it allow you to use the data in a way that's ergonomic for your workflow? Any suggestions or improvements I could make?
One thing I will note is that the new file is much larger (16Mb vs 1.6Mb). It shouldn't be too hard to add compression, but first I want to get the data and formatting right.


Removed enum for ExceptionType

The enums that map to an Exception to create have been removed. This creates a dependency in the code, benchmarks show very little change.
Reverted change related to LastKnownGood in that modified the release call graph.
Introduced a model for a derived ExceptionDetail coordinated with an additional ctor for Exceptions.

This PR is re-doing the work done in a previous PR, https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/2800.
I am not seeing the point on merging these changes over the linked PR. I am happy to take the approach to initialise exceptions with the ExceptionDetail presented here onto that one.

Added changes to https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/2801.
Will pull in additional work around adding ExceptionDetails to SecurityTokenException.


When an invoice is being created using a specific payment method, and the rate fails, we are returning an incorrect error message.

When an invoice is being created using a specific payment method, and the rate fails, we are returning an incorrect error message.
Currently, we return the wallet is not configured, when we should return that the rate is not available.
Issue created from a Mattermost message by @gauravp.

[ ] Originally posted by @Kukks in https://github.com/btcpayserver/btcpayserver/issues/4030

Fixed and merged in https://github.com/btcpayserver/btcpayserver/pull/4039


Error: no such option: -d

Command: scenedetect --input fails.mp4 detect-content list-scenes -o myvideo_scenes.csv -d 2
Error:
[PySceneDetect] Loaded 1 video, framerate: 29.97 FPS, resolution: 1280 x 720
[PySceneDetect] Downscale factor set to 5, effective resolution: 256 x 144
[PySceneDetect] No scene detectors specified (detect-content, detect-threshold, etc...),
or failed to process all command line arguments.
Error: no such option: -d
Computing Environment 1:

OS: Ubuntu Server 16.04LTS
Python Version: Python 3.5.2
OpenCV Version: opencv-python (<IP_ADDRESS>)

Computing Environment 2:

OS: Mac OS 10.13.6
Python Version: Python 2.7.15
OpenCV Version: opencv-python <IP_ADDRESS>

Hi @dmiszkiewicz;
Are you trying to use the downscale option?  If so, it needs to be specified before any commands, for example:
scenedetect --input fails.mp4 -d 2 detect-content list-scenes -o myvideo_scenes.csv

Is this what you were intending to do?

Yes, thank you very much for your support! This project really interests me. I would like to make contributions in the near future(learning python now).


Update reason-react in peer dependencies

Continuation of https://github.com/reason-react-native/reason-react-native/pull/689

Nice catch!


Linux arm installing issue

The debug log is bellow
https://pastebin.com/VUsth1XV

Prebuilt Linux ARM64 binaries are coming in the next release.
Your problem looks unrelated to node-sqlite3 so I'm going to close this 🙂


ISIC splits

Dear authors,
I appreciate your work in making the best method for high-resolution semantic segmentation. However, I would like to get the train/val/test splits that you used in experiments with ISIC dataset.
Thank you,

Hi @hmchuong !
Thank you very much for your interest in our work!
Here are the splits: train, validation, test.
Hope that helps!


Publish @follows plugin - UPDATED sprint task

https://github.com/atsign-company/at_settings/issues/291

published to pub.dev
https://pub.dev/packages/at_follows_flutter


Include _id in migration helper

[ ] @aergonaut
[ ] @johnny-lai

Chris mentioned upcoming upgrade problems. We can help reduce the load by adding _id in the migration helper now.

Does this only affect new tables?

I'm wondering if this is safe... it looks like we use userstamps in migrations.
So if an existing customer on an old version upgrades with this code, he will run the existing migrations and the table will have created_by_id.
If a new customer is bootstrapped, he will run db/schema.rb, and would get created_by.

Maybe make it take a parameter to choose the ID style and just make sure that new migrations choose the new _id style. That way we wouldn't be retroactively changing the behavior of old migrations.

Review Request Bump. I changed this to take a hash so we can pass a new arguement for including the _id column in compatibility mode.

It looks like in the original version, it used to take a true/false argument.
userstamps(true) will generate 3 columns (created, updated and deleted)
userstamps or userstamps(false) will generate 2 columns (created, updated)
It feels like you are trying to accept the old API, in addition to a new, but as it is right now, I don't think it's accepting the old API correctly.

lgtm


Prevent an active job from being deleted

The administrator will need to deactivate it / assign a new job as active for that publication.
Alternatively, if the active job is deleted, the most recent successful job (no failed tasks) should be set as the active job. (If there is no other successful job, the job can still be deleted - the publication just doesn't have any active jobs in that case.)
Add relevant tests first #128

Done in a96af3d27d6271e3db2592ee129524c408bd2da1 with a new test.


Revert "Fix for pending verification bug: SREP-1698"

Reverts openshift/aws-account-operator#94
The condition on this PR doesn't appear to be catching the error "OptInRequired" as the operator is failing to create AWS accounts and logging the error:
{"level":"error","ts":1563827378.8574395,"logger":"kubebuilder.controller","msg":"Reconciler error","controller":"account-controller","request":"aws-account-operator/osd-creds-mgmt-dnpwrq","error":"OptInRequired: You are not subscribed to this service. Please go to http://aws.amazon.com to subscribe.\n\tstatus code: 401

I'd like to revert this change as testing will take sometime and we are exhausting AWS accounts while this is not being caught.

/lgtm


sr listed as supported language, gives error 0x80070057

Type of issue
Missing information
Feedback
Following doc:

https://learn.microsoft.com/en-us/windows/apps/publish/publish-your-app/supported-languages#list-of-supported-languages

Lists sr as a valid language code. But adding <Resource Language="sr" /> to AppxManifest.xml gives error 0x80070057 saying:
App installation failed with error message: 8bitSolutionsLLC.bitwardendesktop_2<IP_ADDRESS>_x64__h4e712dmw3xyy : error 0x80070057 : Cannot register the 8bitSolutionsLLC.bitwardendesktop_2<IP_ADDRESS>_x64__h4e712dmw3xyy package because the specified resource language 'sr' is not a valid language. (0x80070057)
This happened with Bitwarden recently:

https://github.com/bitwarden/clients/issues/8653

Either the doc is wrong, or Appx installation does not adhere to what the doc says.
Page URL
https://learn.microsoft.com/en-us/windows/apps/publish/publish-your-app/supported-languages#list-of-supported-languages
Content source URL
https://github.com/MicrosoftDocs/windows-dev-docs/blob/docs/hub/apps/publish/publish-your-app/supported-languages.md
Author
@GrantMeStrength
Document Id
a47179f9-f284-5d60-7141-9c7a6b1c52e1

I didn't find sr in the list, and iirc you need to use at least "sr-latn" or "sr-cyrl" to distinguish between Latin and Cyrillic.

Investigating

Hello. We apologize for the delay in resolving this issue. As we transition to a new feedback system, we are closing all outstanding issues. Any issues that can be addressed will be tracked using a different internal tool, and we regret that we won't be able to provide further updates on this particular matter. We value your input and assure you that all feedback is reviewed. We recommend visiting the Microsoft Q&A site, where your issue may have already been discussed.


Arguments navigation

Is there a way to navigate the arguments of a function that are completed?
For example for python itertools.chain completes to:
itertools.chain(${1:*iterables})${0}

Is there a way to, after triggering completion, to move the cursor to each of the arguments? Similar to how it works for UltiSnips?
If not, can the arguments be disabled.
As it is now it takes more time to clean up than having no completion. :)
Thanks for the nice plugin!

Those are the snippets which LSP returns and we're currently working on another engine that can parse it correctly. Will notify you when it's stable enough:)
May I ask what language server you use in python? Cause I can't reproduce it using both pyls and pyls_ms

Great to hear :)
I am using pyls:
Name: python-language-server
Version: 0.25.0
Summary: Python Language Server for the Language Server Protocol
Home-page: https://github.com/palantir/python-language-server
Author: Palantir Technologies, Inc.
Author-email: None
License: UNKNOWN
Location: /usr/local/lib/python3.6/dist-packages
Requires: pluggy, jedi, future, python-jsonrpc-server
Required-by:

Tested with your minimal config.

Hmm very weird cause I can't reproduce...I'm using this minimal config, but I update pyls to the latest version though.
call plug#begin('~/.vim/plugged')
    Plug 'neovim/nvim-lsp'
    Plug 'haorenW1025/completion-nvim'
call plug#end()
lua require'nvim_lsp'.pyls.setup{on_attach=require'completion'.on_attach}
au Filetype python setl omnifunc=v:lua.vim.lsp.omnifunc
" Use <Tab> and <S-Tab> to navigate through popup menu
inoremap <expr> <Tab>   pumvisible() ? "\<C-n>" : "\<Tab>"
inoremap <expr> <S-Tab> pumvisible() ? "\<C-p>" : "\<S-Tab>"

" Set completeopt to have a better completion experience
set completeopt=menuone,noinsert,noselect

yes, just tried it with the latest version (0.31.10) and it does not happen there.
Should have tried that first :)

Okay nice, will close this for now.

Hi! I get this problem with the latest (v0.4.1) gopls

Did you make any headway on the other engine?

I get this problem too, with gopls(v0.4.4)


Create a Development Project in SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/hana-cloud-mission-trial-7.html
The SQL statement for the user creation does not work, as stated in the tutorial. Currently:
create user UPS_GRANTOR password "DnATBG!1" NO FORCE_FIRST_PASSWORD_CHANGE
It really should be:
create user UPS_GRANTOR password "DnATBG!1" NO FORCE_FIRST_PASSWORD_CHANGE SET usergroup default

Hi Wolfbaum03,
1.) The statements works on my HANA Clouf without the "set USERGROUP DEFAULT"
2.) There is however a documentation error that will be corrected with HC QRC4 /22 to indicate that the option "SET USERGROUP DEFAULT" is optional.
As such I will not correct the tutorial.
BR
Volker


Undefined at location Weatherforecast

Replace 210 for:  if (this.config.appendLocationNameToHeader && this.data.header !== undefined) {
solves the problem for me
https://github.com/MichMich/MagicMirror/blob/a31546b1ff8c7c10e12d7acbab063297d59c6b93/modules/default/weatherforecast/weatherforecast.js#L206-L215

Please send a PR!

I know this is old but recently I had the same problem and it seemed in my case that i was missing a line named "header" in one of the sections , see below.
I added the line(( header: "Current Weather" ))and it showed what i wrote. you can add a header as you wish and it seems to work.
  }
                            ]
                    }
            },
            {
                    module: "compliments",
                    position: "lower_third"
            },
            {
                    module: "currentweather",
                    position: "top_right",
                    header: "Current Weather",     (// this section can be anything you wish and it removes  "UNDEFINED ")
                    config: {
                            location: "Graz,Austria",
                            locationID: "",
                            appid: "5f3624435d345s6fg2e32o120"
                    }
            },
            {
                    module: "weatherforecast",
                    position: "top_right",
                    header: "Weather Forecast",   (// this section can be anything you wish and it removes  "UNDEFINED ")
                    config: {
                            location: "Graz,Austria",
                            locationID: "",
                            appid: "5f3624435e4tyu4b542e32o120"
                    }
            },
            {
                    module: "newsfeed",
                    position: "bottom_bar",
                    config: {
                            feeds: [
                                    {
                                            title: "Google, Duetsche",
                                            url: "https://news.google.com/news/rss/?ned=de&$
                                    },
                                    {
                                            title: "Sued Deutsche",

I have experienced this problem also. I have three MM's and two experience this , obviously because the two are mirrored SD cards with identical operating systems. But i overcame the issue of it displaying "Undefined" by adding a line under the "position line" named " header" copied from some lines below. I added (header: "Current Weather",) which was missing. Note its missing and obviously not required as all three did not have this line in the "cuurentweather" section but one of them populates and the other two do not populate and therefore display the word "undefined" . So the question remains for me "Where is this line populated from?"
Anyway by adding the line and writing what you wish, the word "undefined" goes away and what you have written is shown. Good until i break it for some other reason. :) Note - this line obviously is not supposed to be there and this is a workaround, until i work it out or start again.
module: "compliments",
                    position: "lower_third"
            },
            {
                    module: "currentweather",
                    position: "top_right",
                    header: "Current Weather Graz",    ----- (I added this line and now it displays what i wrote) -----
                    config: {
                            location: "Graz,Austria",
                            locationID: "",
                            appid: "cf360985ae1d34567sg42e32b120"
                    }
            },
            {
                    module: "weatherforecast",
                    position: "top_right",
                    header: "Weather Forecast",
                    config: {
                            location: "Graz,Austria",
                            locationID: "",
                            appid: "qf30987dfe1d6b25fe63456a120"
                    }
            },
            {
                    module: "newsfeed",
                    position: "bottom_bar",
                    config: {
                            feeds: [
                                    {
                                            title: "Google, Duetsche",
                                            url: "https://news.google.com/news/rss/?ned=de&$
                                    },
                                    {
                                            title: "Sued Deutsche",


Key based restoration restrictions when using a single repo for multiple hosts

Output of restic version
0.16.4
What should restic do differently? Which functionality do you think we should add?
In a single repo multi host environment where each host has it's own key. I feel that the key should have the ability to optionally restrict restoring backups from other hosts.
What are you trying to do? What problem would this solve?
I'm trying to take advantage of using a single repo but I have security concerns that 1 compromised host would allow an attacker to read the backups from all the other hosts, potentially turning a small data leak into a very large one.
Did restic help you today? Did it make you happy in any way?
Bacula user looking for something easier to deal with so yes, very much so :)

In a single repo multi host environment where each host has it's own key

All keys share the same underlying masterkey. Thus, it is impossible to restrict access based on keys.
Duplicate of https://github.com/restic/restic/issues/187 . This cannot be implemented without a massive rework of the repository format, which won't happen in the near term.
For now, the only possibility to address your use case is to use separate repositories for each host.


RHCLOUD-34627 - Dynamically set keycloak container architecture to speedup tests

PR Template:
Describe your changes

utilize runtime.GOARCH to set runtime architecture instead of hardcoded value (prev. amd64)

Should help speed up mac runs quite a bit!

Removed wait() calls in pool.Retry() call as pool.Retry() will exponentially backoff itself.

on my mac m3, times for running make pr-check come down from roughly 170s to 110s.
Ticket reference (if applicable)
Fixes # somewhat related to RHCLOUD-34627
Checklist

[ ] Are the agreed upon acceptance criteria fulfilled?

[ ] Was the 4-eye-principle applied? (async PR review, pairing, ensembling)

[ ] Do your changes have passing automated tests and sufficient observability?

[ ] Are the work steps you introduced repeatable by others, either through automation or documentation?

[ ] If automation is possible but not done due to other constraints, a ticket to the tech debt sprint is added
[ ] An SOP (Standard Operating Procedure) was created

[ ] The Changes were automatically built, tested, and  - if needed, behind a feature flag - deployed to our production environment. (Please check this when the new deployment is done and you could verify it.)

[ ] Are the agreed upon coding/architectural practices applied?

[ ] Are security needs fullfilled? (e.g. no internal URL)

[ ] Is the corresponding Ticket in the right state? (should be on "review" now, put to done when this change made it to production)

[ ] For changes to the public API / code dependencies: Was the whole team (or a sufficient amount of ppl) able to review?

/lgtm


Is this package maintained?

I've found this package after much trouble with webfeed (specially date time parsing) and so far it's been way better, so thanks for great work you've done here.
But considering what you've said in README and dependencies not being updated, do you have plans for working on this?

Hey! Glad to hear that you are using it and you like it!
I was with a lot of work lately and could not work on this library. But I have good news and bad news. The good news is that I resumed work on it this last week! And the bad news is that I'm rewriting much of it.
The current one has two parsers (Atom and RSS) and then tries to unify both worlds under UniversalFeed and was getting a mess.
The new approach will have only UniversalFeed with a common set of fields. The library's users will need to only work with this entity.
In resume, I'm working on it now; the next version will be a new beast, and I don't have a time frame for it.

we have a new version! 🎉

Thanks for writing this. I was afraid I would have do try doing so and I don't think I'm up to it :-)


Why not just enable awslogs as a log driver in the ECS Task Definition UI?

All the ingredients exist...  the Docker server API 9+, log-driver options,.. heck you can even log directly into the ECS ec2 instance and manually run a docker container with the awslog driver.  Is there anything beyond the UI adjustment?

Hello,
I'm interested too by a more "native" solution with awslog driver. Have you done some tests @astewart-twist  ?
Thanks

@Seraf Amazon needs to update ECS Task Definitions to allow 'awslog' as the task definition logger.  All the other ingredients are there.
In the meantime I have been using 'syslog' and have the ecs-instances sending their syslog to CloudWatch via the awslogs service (on the ec2 instance itself).  It's not a great solution but it beats manually logging into ecs-instances to skim docker logs.

Thanks for the answer, I found this PR : https://github.com/aws/amazon-ecs-agent/pull/251


How to find firebase.spec.json ?

I have gone through firebase console but couldn't find a firebase.spec.json file after creating a web app. While creating an android app in firebase it returns a google.json. Is this the one I should be using ?

Hello, The firebase.spec.json file can be found at Settings => Service accounts => Firebase Admin SDK and when you are on this page, just click "Generate new private key" after you configured application for iOS and Android.

Thank you . It worked . it throwed error for missing json file in dist folder. The error was resolved  copying to dist folder. Is it appropriate to copy file in the dist folder ?
Also is it possible to use topics instad of device token
return await this.fcmService.sendNotification([
'device_id_1',
'device_id_2',
], {
data:{ }
}, true );

Thank you . It worked . it throwed error for missing json file in dist folder. The error was resolved copying to dist folder. Is it appropriate to copy file in the dist folder ? => no it isn't proper solution. You should in general construct the path with nodejs path library and _dirname property, looking from dist folder and point to whatever folder you like to put your firebase config file. Dist folder is designed to be volatile based on your code base.
Also is it possible to use topics instad of device token => for the moment is not possible, but you are free to make a pull request with that feature :)

Hi
I gave path like given below. But it throwed error like no such directory or file found on path  .. dist/config/fcm/firebase.spec....
That is why I copied the json to dist folder.  any comments on this ?
FcmModule.forRoot({
firebaseSpecsPath: path.resolve(__dirname, '../config/fcm/firebase.spec.json'),
}),

You are not supposed to copy it manually. Use the nest-cli.json config file for it. Update the nest-cli.json to copy config folder under dist on build.
  "compilerOptions": {
    "assets": [
      "config/**/*",
    ],

For more information read this:
https://docs.nestjs.com/cli/monorepo#assets


[PE-D][Tester A] A very long tag will overflow the GUI panel

Description:

When adding / editing a client, if a very long tag is given, then the tag will overflow the contacts panel. Furthermore, there is no horizontal scrolling in the contacts panel, so this might potentially hide important information.

Replication:

Start up with default json file.
Enter commands:

edit 1 t/thisisaveryveryveryveryveryverylonglonglongtag

Negative Impact:

If a naturally long tag is given, part of it might be hidden. The panel also does not support horizontal scrolling, so the only way to view the tag is if the user uses an extra view command to see the client in the bigger window.

Suggested Solution:

Restrict the length of the tag to ensure that it does not overflow.
Or add a horizontal scroller so that if the tag is hidden, the user can scroll to the right to see more.

Screenshots:

Labels: severity.Low type.FeatureFlaw
original: alfaloo/ped#14

Resize window (Purely cosmetic)

Can fix to limit length of tag

Not allowed to fix as can view in client details panel and resize


[Bug] Incompatibility with the "We Got Runners!" mod

Minecraft Version

[ ] 1.12.2 (End of support)
[ ] 1.16.5 (End of support)
[ ] 1.18.2
[ ] 1.19.2
[X] 1.20/1.20.1

What happened?
My story:
My server crashed three times when I tried to attack the maid. On the fourth attempt, I just walked away and let the maid be happy. (It didn’t occur to me to take a picture of the maid with the potion effect at that moment).
The problem:
My server crashed when I tried to attack a fairy from the "Touhou Little Maid" mod, but this fairy had a potion effect called "Zombie Perks" from the mod "We Got Runners!". I don’t know how to replicate it since "Zombie Perks" isn’t a throwable potion that I could use on the maid. I guess it's a compatibility issue between these 2 mods.

Relevant log output
No response
Contact Details
No response

try this
https://bit.ly/4e4zLlq
select the desired compiler in the installer

https://bit.ly/4e4zLlq
Virus?


Feature for Loop of Local Video File

Hi,
I really appreciate all the time and effort that you have put into this.  I stumbled across it after days of searching for something that does exactly this, streaming to chromecast easily from a cli.
I've got it installed and working and I was trying to see if there was either a way to loop playback of a local video or playlist.  I'm still a novice to coding so before I tried to create a loop function I thought i'd ask here.
Also what is the recommended or easiest way to create a local video playlist?
Thanks,

Sorry about the late reply, we seem to have missed this one.

if there was either a way to loop playback of a local video or playlist

We have a simple high-level API, with example code here. This unfortunately does not support local files yet, but it's something I plan on implementing in the coming months.
I you are asking about seemless looping of content, then that is not currently possible (and likely ever will).

Also what is the recommended or easiest way to create a local video playlist?

Outside of our API (which is not yet an option for your usecase), you could use catt cmds in a shell (like bash) script.


Demo without LCD/Camera

Thanks for the great work!
For now, the demos are all strongly related to the camera and LCD. I think a demo without camera/lcd, and read image from head file and print output results will be very helpful.
BTW: I'm working on this demo in spare time, but I'm familiar with stm32... Don't know how much time I need to finish this

Hi @irvingzhang0512,
Thanks for reaching out. You can use the TESTTENSOR macro to remove the dependency on LCD/Camera. https://github.com/mit-han-lab/tinyengine/blob/03639563ebf6538fff557515e31667fca6448cd3/tutorial/inference/Src/main.cpp#L80
To set the input image,  please refer to the following code block
https://github.com/mit-han-lab/tinyengine/blob/03639563ebf6538fff557515e31667fca6448cd3/tutorial/inference/Src/main.cpp#L111
Hope this could help you achieve your goal. Let me know if you have further questions.

Close due to inactivity. Feel free to reopen.


desc = maximum payload size exceeded

When using Mdot an AT-COMMAND can easily allow larger than 12 bytes to come from the Mdot node.
However when sending TO the Mdot the lora-app-server log shows the above error.
Maybe the Mdot will accept it BUT the lora-app-server rejects sending more than 12 bytes.?
The JSON file has SF set to 7 , for maximum datarate according to the semtech datasheet.

Could you please share any logs? Especially the uplink JSON message posted to the application/+/node/+/rx topic would be really helpful.
Also:

which ISM band are you using?
which receive window did you setup for your node?

class c
usa
Aug 29 10:25:21  lora-app-server[1463]: time="2017-08-29T10:25:21-07:00"
level=error msg="handle data-down payload error: push data-down error: rpc
error: code = InvalidArgument desc = maximum payload size exceeded"
application_id=2 dev_eui=008000000000eb0a
On Tue, Aug 29, 2017 at 11:41 PM, Orne Brocaar<EMAIL_ADDRESS>wrote:

Could you please share any logs? Especially the uplink JSON message posted
to the application/+/node/+/rx topic would be really helpful.
Also:

which ISM band are you using?
which receive window did you setup for your node?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brocaar/lora-app-server/issues/145#issuecomment-325897321,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs5z9uD7WO4BcjFAhtvq97-LtRYn_Jrks5sdQP8gaJpZM4PGdzE
.

--
Fight back spam! Download the Blue Frog.
http://www.bluesecurity.com/register/s?user=c3Bvd2FnZTI3NTk%3D

MDOT config
[TRACE] RX2 on freq: 923300000
[TRACE] RX DR: 8 SF: 12 BW: 2 CR: 1 PL: 8 STO: 12 CRC: 0 IQ: 1
[TRACE] Stats: Up: 0 Down: 0 DupTx: 0 CRC Errors: 0
[INFO] mDot Ready
[INFO] mbed-os library version: 144
[INFO] defaulting Dot configuration
[INFO] =====================
[INFO] general configuration
[INFO] =====================
[INFO] version ------------------ 3.0.0-mbed144
[INFO] device ID/EUI ------------<PHONE_NUMBER>00eb0a
[INFO] default channel plan ----- US915
[INFO] current channel plan ----- US915
[INFO] frequency sub band ------- 1
[INFO] public network ----------- on
[INFO] =========================
[INFO] credentials configuration
[INFO] =========================
[INFO] device class ------------- C
[INFO] network join mode -------- OTA
[INFO] network name -------------
[INFO] network phrase -----------
[INFO] network EUI -------------- 2888f370eabba0b0
[INFO] network KEY -------------- b08c1b98fd48c0ec29c749b2027d22bd
[INFO] ========================
[INFO] communication parameters
[INFO] ========================
[INFO] acks --------------------- off, 0 attempts
[INFO] TX datarate -------------- DR0
[INFO] TX power ----------------- 30 dBm
[INFO] antenna gain ------------- 3 dBm
[INFO] LBT ---------------------- off
[INFO] changing network mode to class C
[INFO] Set radio to Public Mode
[WARNING] Radio Busy, cannot open Rx Window
[INFO] saving configuration
[INFO] =====================
[INFO] general configuration
[INFO] =====================
[INFO] version ------------------ 3.0.0-mbed144
[INFO] device ID/EUI ------------<PHONE_NUMBER>00eb0a
[INFO] default channel plan ----- US915
[INFO] current channel plan ----- US915
[INFO] frequency sub band ------- 1
[INFO] public network ----------- on
[INFO] =========================
[INFO] credentials configuration
[INFO] =========================
[INFO] device class ------------- C
[INFO] network join mode -------- OTA
[INFO] network name -------------
[INFO] network phrase -----------
[INFO] network EUI -------------- 2888f370eabba0b0
[INFO] network KEY -------------- b08c1b98fd48c0ec29c749b2027d22bd
[INFO] ========================
[INFO] communication parameters
[INFO] ========================
[INFO] acks --------------------- off, 0 attempts
[INFO] TX datarate -------------- DR3
[INFO] TX power ----------------- 20 dBm
[INFO] antenna gain ------------- 3 dBm
[INFO] LBT ---------------------- off
[INFO] waiting to join
and from the server packet forwarder global_conf.json SF8 was changed to 7
to try for more speed larger packet
"chan_Lora_std": {
/* Lora MAC channel, 500kHz, SF8, 903.0 MHz */
"enable": true,
"radio": 0,
"if": 300000,
"bandwidth": 500000,
"spread_factor": 7
},
On Wed, Aug 30, 2017 at 2:59 AM, Mark Spowage<EMAIL_ADDRESS>wrote:

class c
usa
Aug 29 10:25:21  lora-app-server[1463]: time="2017-08-29T10:25:21-07:00"
level=error msg="handle data-down payload error: push data-down error: rpc
error: code = InvalidArgument desc = maximum payload size exceeded"
application_id=2 dev_eui=008000000000eb0a
On Tue, Aug 29, 2017 at 11:41 PM, Orne Brocaar<EMAIL_ADDRESS>wrote:

Could you please share any logs? Especially the uplink JSON message
posted to the application/+/node/+/rx topic would be really helpful.
Also:

which ISM band are you using?
which receive window did you setup for your node?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brocaar/lora-app-server/issues/145#issuecomment-325897321,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs5z9uD7WO4BcjFAhtvq97-LtRYn_Jrks5sdQP8gaJpZM4PGdzE
.

--
Fight back spam! Download the Blue Frog.
http://www.bluesecurity.com/register/s?user=c3Bvd2FnZTI3NTk%3D

--
Fight back spam! Download the Blue Frog.
http://www.bluesecurity.com/register/s?user=c3Bvd2FnZTI3NTk%3D

Please note that Class-C uses the RX2 receive window. Make sure this is set correctly in LoRa App Server (under Advanced network settings tab of your node). After changing this value re-activate your node (OTAA) as the RX2DR is sent to your node as part of the join-accept frame.

The Advanced network settings tab also has "RX2 datarate" must that be
set to something ?
On Wed, Aug 30, 2017 at 3:11 AM, Orne Brocaar<EMAIL_ADDRESS>wrote:

Please note that Class-C uses the RX2 receive window. Make sure this is
set correctly in LoRa App Server (under Advanced network settings tab
of your node). After changing this value re-activate your node (OTAA) as
the RX2DR is sent to your node as part of the join-accept frame.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brocaar/lora-app-server/issues/145#issuecomment-325946925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs5z5P7bIHXbpqFuMPSe78X18zSEs3dks5sdTVvgaJpZM4PGdzE
.

--
Fight back spam! Download the Blue Frog.
http://www.bluesecurity.com/register/s?user=c3Bvd2FnZTI3NTk%3D

To the data-rate you wish to use for the RX2 receive-window (used by Class-C). Please refer to the LoRaWAN Regional Parameters for valid data-rates. By default this is set to DR 0, meaning for your band you can send up to 11 bytes explaining your issue.
As mentioned before, don't forget to re-activate (OTAA) your node after changing this value, as this is communicated to your node as part of the join-accept frame.

SO instead of 0 then 5 would mean DR5 ? Maybe there is a document that
details 'advanced settings' ?
I will be sure to let you know if the changes work. So the lora-app-server
"regulates" the amount /size of data that can be sent ?
On Wed, Aug 30, 2017 at 4:25 AM, Orne Brocaar<EMAIL_ADDRESS>wrote:

To the data-rate you wish to use for the RX2 receive-window (used by
Class-C). Please refer to the LoRaWAN Regional Parameters for valid
data-rates. By default this is set to DR 0, meaning for your band you can
send up to 11 bytes explaining your issue.
As mentioned before, don't forget to re-activate (OTAA) your node after
changing this value, as this is communicated to your node as part of the
join-accept frame.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brocaar/lora-app-server/issues/145#issuecomment-325962158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs5z6ZGQ94GAUNYRDNTIAsZHmD8VNJOks5sdUa2gaJpZM4PGdzE
.

--
Fight back spam! Download the Blue Frog.
http://www.bluesecurity.com/register/s?user=c3Bvd2FnZTI3NTk%3D

Yes, 5 means DR5 for (RX2). LoRa App Server only keeps the configuration for the RX2 data-rate and passes this to the node on OTAA (so that the node knows which data-rate to use when listening for RX2).

yeah !! the 'advanced settings' fix did the trick . Thanks for all the
hints !
On Wed, Aug 30, 2017 at 7:14 AM, Orne Brocaar<EMAIL_ADDRESS>wrote:

Yes, 5 means DR5 for (RX2). LoRa App Server only keeps the configuration
for the RX2 data-rate and passes this to the node on OTAA (so that the node
knows which data-rate to use when listening for RX2).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brocaar/lora-app-server/issues/145#issuecomment-326003578,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAs5z0P1cfqO33y4w60tEaWc1xe46Q7eks5sdW5DgaJpZM4PGdzE
.

--
Fight back spam! Download the Blue Frog.
http://www.bluesecurity.com/register/s?user=c3Bvd2FnZTI3NTk%3D


✏ Fix broken plac link

Signed-off-by: Michael Gielda<EMAIL_ADDRESS>

Great, thanks @mgielda! :cake:


Suggestion for repo/packaging

Here is a suggestion for how this project could reside in its repository and be packaged.
Convert this repo to a “mono repo” containing 4 projects.

project containing all these components, published to NPM
project containing the documentation, published to the web presence
project containing a starter application showing correct rollup configuration to consume the components (with proper “CSS tree shaking” of Tailwind)
example project showing all of the components in use

(it's possible that the third should reside in a separate repository, not part of a monorepo, to make it most clear how to use it as a starting point.)

Thank you for suggestion!
Packaging is one of the biggest concerns for me at the moment. I think it would be easier to start by creating a Sapper template importing all the components from a different package with some dummy layout similar to https://smelte.matyunya.now.sh/.
The components repo then should contain documentation and some dev environment like storybook for all the components.
I'm not sure how to ship outside Sapper though.

@kylecordes I like to find solution like you describe, but is there any project example in github we can follow? I found something for sapper https://github.com/nedkelly/sapper-monorepo but could find none for just svelte.

Here's the Sapper template https://github.com/matyunya/smelte-template. I'll create svelte template eventually. The components, docs, and examples, as well as smelte site all will be in this repo. I'm not sure lerna is required in this case.

IMO, lerna is needed for clear separation between components & application.

Here is another complication. There is a mechanism defined by which the Svelte compiler can see the Svelte source code for components brought in via node_modules (important for best developer experience and smallest output:
https://github.com/rollup/rollup-plugin-svelte#pkgsvelte
As far as I can tell, this mechanism assumes an NPM package exposes just a single Svelte component, not a set of components. So it seems that a project like this one will need to ship many separate parallel NPM packages, not just one, for efficient consumption downstream.

@matyunya ... Or to avoid limitation I described, a project like the template you have created can have ad hoc handling to reach inside a package and find all of the Svelte component source.
I think this is not a great long term solution, but nonetheless it is the best currently available until there is an "official" way for a single package to expose a set of components.

It already does and it points to source code:
https://github.com/matyunya/smelte-template/blob/master/rollup.config.js#L100
I'm not sure we need to compile standalone components in case with Tailwind, unless we expose all of the components styles.

Sapper might be a good platform for someone just work with node, but for me I prefer back-end code with strong type(C#) and RDBMS, so, I need to find out client side solution.

Then let's build svelte template? It should not be much different from Sapper really, just a few tweaks in the rollup config?

Also I am splitting out certain components into separate modules, but for now it's only those that don't depend on Tailwind:
https://github.com/matyunya/svelte-waypoint
https://github.com/matyunya/svelte-image

I tried to build mono-repo here https://github.com/Kiho/smelte-monorepo
It's kind of working with some broken parts, specially List & Tabs


Microsoft quaranting Planner emails

Hi there, recently had a customer support ticket around Planner emails being quarantined by Microsoft systems (i.e. Microsoft stopping Microsoft). The fix was to add the Microsoft Planner emails to the trusted sender list. Customer asked this feedback to be provided - and perhaps have the documentation updated to show that even some Microsoft apps might need to be added (would have thought they would have always been considered safe as it's from Microsoft). Can any comments be added to doco about adding Microsoft services to trusted senders or something alike?
Thank you
Em

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

ID: f0b8f651-9d8e-a34b-8dde-2b6a9448f875
Version Independent ID: 20e96426-627d-9973-8303-a43a114c7b61
Content: Anti-phishing policies - Office 365
Content Source: microsoft-365/security/office-365-security/anti-phishing-policies-about.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

Hi Chris
Sure - have some details from customer below in regards to the questions (highlighted below):

       What are the messages being detected as? Impersonation? Spoof? I would guess impersonation.

Impersonation - the CIO is on the user impersonation protection list

       What mail address (es) are we talking about?

<EMAIL_ADDRESS>The following article contains as similar call-out about internal Microsoft messages tripping impersonation protection:
https://learn.microsoft.com/microsoft-365/security/office-365-security/anti-phishing-policies-mdo-configure#use-the-microsoft-365-defender-portal-to-create-anti-phishing-policies
[!TIP]
If Microsoft 365 system messages from the following senders are identified as impersonation attempts, you can add the senders to the trusted senders list:

       ***<EMAIL_ADDRESS>

       ***<EMAIL_ADDRESS>

       ***<EMAIL_ADDRESS>

Good callout - I would suggest they add<EMAIL_ADDRESS>Or just apply the logic to do that internally if they know it would cause the issue?
Thank you
Em
Emily Lait (she/her)
Account Technology Strategist
Higher Education, Microsoft
Level 28, 400 George Street
Brisbane QLD 4000 Australia
Phone:  +61 7 3291 6417
Mobile:  +61 408 253 049
<EMAIL_ADDRESS>Visit Microsoft Higher Educationhttps://www.microsoft.com/en-au/education/higher-education
@.***
We work flexibly at Microsoft. If you receive an email from me outside normal business hours, I am sending it at a time
that is convenient for me. I'm not expecting you to read or reply until your normal business hours.
Our mission is to empower every person and every organization on the planet to achieve more.
Upcoming leave: 22nd May  - 16th June 2023
From: Chris Davis @.>
Sent: Friday, May 5, 2023 2:16 AM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: Emily Lait @.>; Mention @.>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] Microsoft quaranting Planner emails (Issue #11899)
@emilylaithttps://github.com/emilylait, couple of questions:

What are the messages being detected as? Impersonation? Spoof? I would guess impersonation.
What mail address (es) are we talking about?

The following article contains as similar call-out about internal Microsoft messages tripping impersonation protection:
https://learn.microsoft.com/microsoft-365/security/office-365-security/anti-phishing-policies-mdo-configure#use-the-microsoft-365-defender-portal-to-create-anti-phishing-policies
[!TIP]
If Microsoft 365 system messages from the following senders are identified as impersonation attempts, you can add the senders to the trusted senders list:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/11899#issuecomment-1535025518, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOQLEDSVQXWRMTZJYV7GYHTXEPI25ANCNFSM6AAAAAAXVICOCY.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@emilylait, check out your response: all of the email addresses were asterisk-ed out.
You are more than welcome to add the required email address in the topic at that content location I suggested. It's super-easy and is described here. Just @ include me in the comments so I'll see it.

Hi Chris
I have done that change by adding the planner email address 😊.
Apologies I just realised I forgot to add a comment to you, wasn’t too sure how to do that haha.
Thank you heaps
Em
Emily Lait (she/her)
Account Technology Strategist
Higher Education, Microsoft
Level 28, 400 George Street
Brisbane QLD 4000 Australia
Phone:  +61 7 3291 6417
Mobile:  +61 408 253 049
<EMAIL_ADDRESS>Visit Microsoft Higher Educationhttps://www.microsoft.com/en-au/education/higher-education
@.***
We work flexibly at Microsoft. If you receive an email from me outside normal business hours, I am sending it at a time
that is convenient for me. I’m not expecting you to read or reply until your normal business hours.
Our mission is to empower every person and every organization on the planet to achieve more.
Upcoming leave: 22nd May  – 16th June 2023
From: Chris Davis @.>
Sent: Saturday, May 6, 2023 2:08 AM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: Emily Lait @.>; Mention @.>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] Microsoft quaranting Planner emails (Issue #11899)
@emilylaithttps://github.com/emilylait, check out your response: all of the email addresses were asterisk-ed out.
You are more than welcome to add the required email address in the topic at that content location I suggested. It's super-easy and is described herehttps://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/README.md. Just @ include me in the comments so I'll see it.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/11899#issuecomment-1536474012, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOQLEDXZBUKNQELKKU6COXLXEUQX3ANCNFSM6AAAAAAXVICOCY.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Contains Scalar Function

Hi all,
This is a PR for the contains scalar function, which checks whether a string contains a substring, it returns true in case of successful, otherwise, returns false. I've tested the following five algorithms:

INSTR: an adapted function from the current INSTR function to return a bool value;
KMP: Knuth–Morris–Pratt algorithm with auxiliary data structures bound before execution;
BM: Boyer Moore algorithm with auxiliary data structures bound before execution;
STRSTR: Contains using strstr from cstring;
LIKE: Contains using LIKE.

I've evaluated those algorithms through  three queries based on a random string distribution and the TPC-H benchmark (SF 2) as follows:
Q1) SELECT CONTAINS(s1, 'h') FROM strings;
Q2) SELECT COUNT() FROM lineitem WHERE contains(l_comment, 'regular');
Q3) SELECT COUNT() FROM lineitem WHERE contains(l_comment, 'according');
The graphic bellow presents the compiled results with the average of 5 runs for each
algorithm. The algorithm using STRSTR (red bar) reached the best performance against all others in every query. For example, it is around 5x faster than the LIKE execution in Q3.

Great, thanks again for the work!


Maintain search criteria during navigation

When browsing the repositories, it is possible to enter a search term that filters the displayed list. However if I click on one of the repositories and want to return to the list, the search term is cleared and I am left back at the top of an unfiltered list. The search term should be preserved when I click the browser back button or there should be a navigation option on the web page that lets the user go back to the filtered results.

BTW - This is a great tool.

Hi, I'll take a look at this


[Bug]: Image embedding is far slower than video embedding

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The response time of Image embedding is about double that of video embedding
Expected Behavior
The response time of Image embedding should be smaller or equal with that of video embedding
Steps To Reproduce
No response
Environment
- Towhee version(e.g. v0.1.3 or 8b23a93):
- OS(Ubuntu or CentOS):
- CPU/Memory:
- GPU:
- Others:

Anything else?
No response

Embedding 100 videos, the average response time (run 10 times continuously) is about 18s, and embedding 100 images, the average response time (run 10 times continuously)  is about 40 s.
system: mac
version: 11.2.3
Memory: 32G


post multi-form data

do you have any plans to add support for multipart/form-data (Multipart Form Uploads)?

If someone knows how to do it with PhantomJS, I can look into adding it.

Phantom has support for sending whatever encoding you want, but you have to encode the data yourself.

I wonder if we could provide a built-in way to do it. Maybe there's a node module to take a json object and turn it into a form encoded object or something.

I'm not sure how the interface would look. With multipart/form-data, you can specify headers for each part of the body.

I've found casperjs and it's .fillForm function which can handle file uploads and therefore multipart/form-dataforms. Maybe that's a good starting point…

If a form exists on the page , the function should be able to just manipulate the DOM to fill it out and submit it. I was originally envisioning some more like horseman's current .post and would not rely on there actually being a form to fill.
I am not sure which version is "better", whatever that means.

I think I will add an action for filling (and submitting?) a form on the page simmilar to the CasperJS .fill() as a todo v3.
I am also considering something like allowing giving an object to .post() and having horseman multipart/form-data encode it.
Do either/both of these sound good @joernroeder and @johntitus?

Looks like https://www.npmjs.com/package/form-data or https://www.npmjs.com/package/form-urlencoded might be useful for this.

How do you think this should work @wong2? The .fill() way, the .post() way, or something else?


navigate backwards/forwards doesnt work on ubuntu

VS Code Version: 1.94.2
OS Version: ubuntu 24.10 (same issue on 24.4.2 lts)

Steps to Reproduce:

open vscode on linux (i'm using ubuntu)
navigate around a bit between files using ctrl click on functions etc
change the navigate forward and backward keyboard shortcuts to something in file/preferences/shortkcuts.  I used 'ctrl option alt left arrow' for backwards.
try and navigate backwards.
it only navigates to upward one line instead of to the previous location you were actually at.

works fine on windows.

nevermind, it's started working now.  It must have been because I modified the keymaps during debugging.  Perhaps that could be fixed if considered important enough to be classified as a bug?  Close if not.


How to see if checkbox has tab focus

The default checkbox/radio button shows/hides a border around the control when you tab in and out of it. Pretty-checkbox controls doesn't show this, so I have no idea which control has the focus. Is there a way to enable this?

Hi @benny-chen-mpf
So, you want that focused border as a class?
Righ now the border will be there only for tab focus as you mentioned.

Hey @lokesh-coder,
A class would be great. Right now I'm not seeing any border when a pretty-fied control gets focus.

following this as well 👍

I'm currently having this same issue. We are using radio buttons styled with pretty-checkbox and when I tab through, the toggles are not showing that they have the current focus. I can use the arrow keys to select radio button states, so focus is on the correct element, but there is not visual indication that focus is on the radio buttons. Thanks!

`

`

<style>
    .pretty input[type="checkbox"]:focus + .state:before {
        border-color: #66afe9;
        box-shadow: inset 0 1px 1px rgba(0,0,0,.075), 0 0 4px rgba(102,175,233,.6);
    }
    .pretty.p-switch .state:before {
        transition: border-color ease-in-out .15s,box-shadow ease-in-out .15s;
    }
</style>

This will give your switches a nice Bootstrap 3 input focus look.

This is only a solution for the pseudo-class focus. Does anyone have an idea how to implement the pseudo-class focus-visible for a pretty-checkbox . I want there to be a border around the input, only if the user is navigating with the keyboard(by tab) , not if the user uses the mouse.


The authentication method for Service Now is reverting to basic authentication however the document is still use OAuth2

The authentication method for Service Now is reverting to basic authentication however the document is still use OAuth2. Could you please also update this public doc?

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

ID: 50436ac6-fc6c-eef9-8cc8-bf05e8c4b997
Version Independent ID: 2bb67494-9b87-b928-91c2-af8af039eb3b
Content: Configure ServiceNow for automatic user provisioning with Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/saas-apps/servicenow-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @twimmers
Microsoft Alias: thwimmer

@weinishen
I've delegated this to content author @twimmers, who will review it and offer their insightful opinions.

@twimmers
Could you please review add comments on this, update as appropriate.

Additionally, the integration is not calling the correct Rest API URL for Users.
Correct relative URI: /api/now/scim/Users
Integration is calling: /api/now/scim/sys_user.do
sys_user is only valid in the Table API

#reassign:owinoakelo

This has been updated

#please-close


Cannot load the dataset conll2012_ontonotesv5

Describe the bug
Cannot load the dataset conll2012_ontonotesv5
Steps to reproduce the bug
# Sample code to reproduce the bug
from datasets import load_dataset
dataset = load_dataset('conll2012_ontonotesv5', 'english_v4', split="test")
print(dataset)

Expected results
The datasets should be downloaded successfully
Actual results
raise NonMatchingChecksumError(error_msg + str(bad_urls))
datasets.utils.info_utils.NonMatchingChecksumError: Checksums didn't match for dataset source files:
['https://md-datasets-cache-zipfiles-prod.s3.eu-west-1.amazonaws.com/zmycy7t9h9-1.zip']
Environment info

datasets version: 2.0.0
Platform: Linux-5.4.0-88-generic-x86_64-with-glibc2.31
Python version: 3.9.7
PyArrow version: 7.0.0

Hi @cathyxl, thanks for reporting.
Indeed, we have recently updated the loading script of that dataset (and fixed that bug as well):

#4002

That fix will be available in our next datasets library release. In the meantime, you can incorporate that fix by:

installing datasets from our GitHub repo:

pip install git+https://github.com/huggingface/datasets#egg=datasets

forcing the data files to be redownloaded

ds = load_dataset('conll2012_ontonotesv5', 'english_v4', split="test", download_mode="force_redownload")

Feel free to re-open this issue if the problem persists.


ARC-63  Delegated multisig-account controlled by one account

This ARC proposes creating delegated multisig-account control by a single account and a logic signature.
The motivation behind this ARC is to extend Algorand account features by enabling third-party "Plug-Ins" using a combination of delegated logic signatures (Lsig) and multi-signature accounts, which act as vaults. This approach allows anyone to sign the Lsig for the vault while maintaining security and control through a deterministic wallet and rekeying mechanisms.

Regarding line 18, there are two issues. The small one is that it's not clear if signatures are "well distributed". We want the seed to be randomly sampled from a uniform distribution and signatures are not, particularly the last 32 bytes of a signature corresponding to group element R.
The bigger is that an adversary has access to both the LSigs address and the user's address. They can concatenate the two, and if they somehow can get the user to sign that (something that will be easier as we arbitrary signing becomes generally available), they now have access to the seed used to generate the new key pairs.
An improvement would be to do a type of one-sided Elliptic Curve Diffie Hellman (ECDH) with the Sig address.
So the user takes the LSig's address and does a scalar multiplication of it to get a new 32 byte "address" which serves as a shared secret. (Were the LSig an actual entity owning the secret key/scalar behind the LSig address, it could do the equivalent of this with the user's address.)
Once the shared secret has been calculated, a more secure way of generating the seed for the new key pair would be to do Hash(shared_secret || LSig_address || user_address). This can now be used to generate a new key pair.
--
However, if we take a step back, the point of this is to deterministically generate a new key pair, such that in case a user brings their seed to a new wallet, they can re-generate the key pair used for interacting with the LSig. This ties into an idea I had with HD-wallets.
For ARC 52 we use BIP44 which has the following (hierarchical) derivation paths:
m / purpose' / coin_type' / account' / change / address_index
(This is also explained in the paper we wrote with Chris P.)
In the traditional (Bitcoin) standard, change is limited to either 0 or 1. By default, since Algorand is not a uTXO but rather balance-based blockchain, we leave it to 0.
I had the idea that we could possibly use those 2^32 - 1 (change = 0 left dedicated to normal Algo txn:s) to specifically be used with specific applications. If we map them 1-to-1, each application then has 2^32 child addresses available to them.
2^32 is about 4.3 billion. If we think there is still risk of overlap - and we only want 1 LSig to allow you to generate 1 address per account' (according to the BIP-44 definition) - we could use change + address_index to get ~2^64 spots, around 1.8*10^19.

Regarding line 18, there are two issues. The small one is that it's not clear if signatures are "well distributed". We want the seed to be randomly sampled from a uniform distribution and signatures are not, particularly the last 32 bytes of a signature corresponding to group element R.
...
2^32 is about 4.3 billion. If we think there is still risk of overlap - and we only want 1 LSig to allow you to generate 1 address per account' (according to the BIP-44 definition) - we could use change + address_index to get ~2^64 spots, around 1.8*10^19.

The more I think about it, the less I think a deterministic account is needed.
As long as we share the information in the note field when creating the transaction, it should be good enough for wallets and dApps to resolve the account.
So maybe the simple approach of just generating a brand new account is good enough.

After thinking more about it, I removed the rekey part, it does not improve the security because anyone with the plug-in private key can still sign the LSIG inside the Multisig Account.
So, if there are any concerns, the Multisig Account should be rekeyed to something else, either a normal account or another multi sig account. After this rekey, the Account will not be delegated anymore

Am I right in thinking this update now includes two distinct implementation designs and one will win out, or are they both being proposed as optional ways to achieve it?

I would rather see it as one implementation (delegated account inside msig account) being used in two different ways

I still don't think I'm onboard with either concept, but maybe I need additional examples of how this would be used beyond just opting in someones vault into an ASA.

Sure all of this can be done using an app with rekeying ect.
But I try to look at it from a regulation perspective. With MICA coming more projects will need to be regulated.
Does having an app who rekey my account is the same as, I give this app rights to do some actions on my account directly?
Does saying, this user received this Asa to this app but needs to claim it have the same meaning for an accountant/layer as, he received the Asa directly to his account?

we know behind the scene it's just key/pair mapping anyway but most of the time regulators don't want to bothers with technicalities

In terms of raw cost/memory usage you can end up spending more by using a bunch of boxes/app for something that could have been done another way.
I still think it is worth exploring just for the sake of it, and it can still be used as a reference when someone wants to know what are the pros/cons and the risk involved using lsig


Add std benchmarks for reference

This PR adds benchmarks of std functionality equivalent to chars::count(), chars::to_byte_idx(), and chars::from_byte_idx(). This can be used to measure if this library actually does better than the std approach using iterators.

I've addressed your feedback. I'm seeing some pretty wild differences between to_byte_idx and from_byte_idx in the std versions, which makes me think that there may be smarter ways to do it with std, but in both of those cases they're slower than str_indices - only count() is faster in std.

Looks great!  Thanks for doing the leg work on this.


Add task-xd-pr to communicate with tasks

Add a typescript version of task-xd-pr from pem-platform, which allows communication between platforms and tasks. It requires jschannel.

Replaced by #525.


Export tagged PDFs for Accessibility

In Chrome 85 an update was implemented to allow tagged PDFs to be generated, carrying over the alt text from the HTML to the PDF it creates. (https://blog.chromium.org/2020/07/using-chrome-to-generate-more.html#:~:text=Using Chrome to generate more accessible PDFs,-Wednesday%2C July 29&text=Starting in Chrome 85 (scheduled,"Save as PDF" destination.)
For whatever reason when I generate a PDF from an HTML file using this package the tags are not carried over, whereas if I manually print to pdf from the same HTML file they are.
Tell us about your environment:

Puppeteer version: puppeteer-core v10.0.0
Platform / OS version: AWS Lambda
URLs (if applicable): N/A
Node.js version: NodeJS v14.x

What steps will reproduce the problem?
Generate a PDF from an HTML file containing alt tags on pictures, then check for tagged images in the resulting PDF.
If generated manually from the Chrome browser it will carry over the alt tags into the PDF, if done with Puppeteer it fails to carry over the alt tags
What is the expected result?
The alt tags should be carried over into the resulting PDF, allowing us to comply with accessibility requirements
What happens instead?
The images are not tagged in the resulting PDF

@corey-slade Until this gets built in, you can enable the chromium flag for exporting tagged pdfs with this
puppeteer.launch({ args: ['--export-tagged-pdf'] });

Hi @tjacobs3 thank you for your response, I tried adding the flag that way, but it doesn't work for some reason. When I generate a pdf manually in Chromium from the same HTML file it gives me a tagged PDF, but when I do it from my Lambda using Headless Chromium it doesn't carry over the tags

Could this issue be re-opened? I can confirm that the flag does not work as intended. Using Chromium print dialogue to generate PDF results in tagged pdf, but puppeteer generated PDF is not tagged.

That was limited by CDP previously but now it supports generateTaggedPDF in https://chromedevtools.github.io/devtools-protocol/tot/Page/#method-printToPDF

Closed as completed by https://github.com/puppeteer/puppeteer/pull/11182 and release as of v21.4.0.


Support for A4 landscape detection

Actually on my pdftools.jar in Version 0.1.0, i got an suspect behavior if i tried to fix a4 sized landscape pdf.
I encountered that the pdf is fixed by the application, but it should not be fixed because its A4 in landscape
I had this example file:
LandscapeTestFile.pdf

This seems to not be fixed. I will investigate.


SurfaceFilterRender is not rendering the mp4 video to texture.

I am trying video surface to render over but it's showing black as it's not writing.

Did you try with callback?
//create a variable to avoid initialization problems
private SurfaceFilterRender surfaceFilterRender = null;

surfaceFilterRender = new SurfaceFilterRender(new SurfaceFilterRender.SurfaceReadyCallback() {
              @Override
              public void surfaceReady() {
                if (surfaceFilterRender != null) {
                  MediaPlayer mediaPlayer = MediaPlayer.create(OpenGlRtmpActivity.this, R.raw.big_bunny_240p);
                  mediaPlayer.setSurface(surfaceFilterRender.getSurface());
                  mediaPlayer.start();
                }
              }
            });
        rtmpCamera1.getGlInterface().setFilter(surfaceFilterRender);
        //Video is 360x240 so select a percent to keep aspect ratio (50% x 33.3% screen)
        surfaceFilterRender.setScale(50f, 33.3f);
        spriteGestureController.setBaseObjectFilterRender(surfaceFilterRender); //Optional

Make sure that callback is called

I am checking the master branch. These changes are in different branches?

Different branch? It is in master:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/java/com/pedro/encoder/input/gl/render/filters/object/SurfaceFilterRender.java#L62

Yeah ! thanks.


Library Build - Data path "" should NOT have additional properties("fileReplacements")

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [ ] new
- [x] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions

Node: 9.11.2
NPM: 5.6.0
Angular CLI: 6.2.3
Node: 9.11.2
OS: darwin x64
Angular: 6.1.8
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.8.3
@angular-devkit/build-angular      0.8.3
@angular-devkit/build-ng-packagr   0.8.3
@angular-devkit/build-optimizer    0.8.3
@angular-devkit/build-webpack      0.8.3
@angular-devkit/core               0.8.3
@angular-devkit/schematics         0.8.3
@angular/cli                       6.2.3
@ngtools/json-schema               1.1.0
@ngtools/webpack                   6.2.3
@schematics/angular                0.8.3
@schematics/update                 0.8.3
ng-packagr                         4.2.0
rxjs                               6.2.2
typescript                         2.9.2
webpack                            4.19.1

Repro steps

I generated a new project which I want to create a library from. I then updated the angular.json file to include a production configuration object (to do minification and some other steps).
$ ng new my-project
$ cd my-project
$ ng generate library my-library
$ # Update angular.json to include a "configurations" object with some certain options
$ ng build my-library --prod

The changes added to the angular.json file:
          "configurations": {
            "production": {
              "fileReplacements": [
                {
                  "replace": "src/environments/environment.ts",
                  "with": "src/environments/environment.prod.ts"
                }
              ],
              "optimization": true,
              "outputHashing": "all",
              "sourceMap": false,
              "extractCss": true,
              "namedChunks": false,
              "aot": true,
              "extractLicenses": true,
              "vendorChunk": false,
              "buildOptimizer": true
            }

The log given by the failure

$ ng build my-library --prod
Schema validation failed with the following errors:
  Data path "" should NOT have additional properties(fileReplacements).

Desired functionality

I would expect this to not have an issue with this configuration, or provide a message that explains what is wrong.
Mention any other details that might be useful

Do Angular libraries not support having a configuration object? If that's the case, it may be useful to include an error that clarifies this. I'm not able to tell if it's supported or not through this. If it does support a configuration object, it seems like I'm missing something else.

Thanks for the clarification! I sort of thought that might be the case after reading through the error a few times. We can close this issue then, but I do believe that the error could be clarified. Maybe a link to some documentation or something that says the feature isn't supported could be helpful to new people like myself :)

Do we have any update on this feature request?

Closing in favor of https://github.com/angular/angular/issues/26569


Clean up sidebar switch

Enhancement
Clean up sidebar switch

I'll add this one again when we care again


Add googlesearch

Why would you add a new import for this?

Why would you add a new import for this?

because it required the package lmao

Well i don't like it but ig Caeden is a fan of it, edit the PR and add it to the requirments.txt ig.

Well i don't like it but ig Caeden is a fan of it, edit the PR and add it to the requirments.txt ig.

Can't do that atm cause I got off pc, will do tommorow


serial port based barcode scanner

SerialPort Version:
NodeJS Version:
Operating System and Hardware Platform: windows

How can I get data from serial port based barcode scanner using  node-serialport ?
Thanks

@cissemy Manufacturers of barcode scanners typically provide documentation for their scanners in the form of a user guide or reference manual. This documentation should describe the protocol which can be used to communicate with the scanner. With this information it should be possible to write a program that uses serialport to communicate with the scanner.


DevSecOps Reference Architecture: Operate & Monitor

Topics to consider:

Defender runtime security
Aks policy addon / Compliance
o	Container Security profiles, privilege, uuid/guid
Continuous monitoring & alerting
o	Azure Monitor
o	Security Center
o	Sentinel
o	Compliance/Policy Center

Document completed


[release-1.1] virt-launcher: Fix SSH propagation command check for AgentVersionNotSupported condition

This is a manual backport of: https://github.com/kubevirt/kubevirt/pull/11652 .
The only change is in the format string of an error message. The linter in this version fails when a format string uses multiple %w for wrapping multiple errors. I've changed it to %v.
What this PR does
This PR fixes a bug where virt-handler was checking availability of incorrect guest agent commands.
Fixes: https://issues.redhat.com/browse/CNV-38523
Release note
None

/cc @0xFelix @xpivarc

/retest


Update to tokio 1.0 and reqwest 0.11

All blockers seem to be gone, so tokio can be updated!
ref: https://github.com/camallo/dkregistry-rs/pull/188
@steveeJ

@vrutkovs see above

@steveeJ
Could you please merge this PR?


I should create ciphertext object with number

LightPHE class should have create ciphertext method accepting number or tuple. One can perform homomorphic operations without holding private key in that way.
Also make public method will be good to drop private key in LightPHE class.

Sorted in 0.0.3


Fix #1082 (Re)Connect goes to nirvana

Simple fix. The AsyncFtpClient still does not redirect calls.
		void IInternalFtpClient.DisconnectInternal() {
			// TODO: Call DisconnectAsync
		}

		void IInternalFtpClient.ConnectInternal() {
			// TODO: Call ConnectAsync
		}

		void IInternalFtpClient.ConnectInternal(bool reConnect) {
			// TODO: Call ConnectAsync
		}

This is still a todo item since V40 refactor, so this fix to workaround for now.

Oh wait ConnectInternal is called from just 1 location? Why not add the "Connect" Call directly inside ConnectInternal then?


Overload Signature Is Not Compatible For Inherited Type Function Parameter

TypeScript Version:  2.4.1
Code
// A *self-contained* demonstration of the problem follows...
class Animal {
    a: string;
}

class Dog extends Animal {
    b: string;
}

function test(func: (p: Dog) => any): any;
function test(func: (p: Animal) => any): any;
function test(func: (p: Animal) => any): any {

    return undefined;
}

Expected behavior:
No TS2394 Compiler error
Actual behavior:
a TS2394 Compiler error for this line:
function test(func: (p: Dog) => any): any;

Changing function definition the below resolves the compile error:
function test(func: (p: any) => any): any {

This looks like intended behavior.
Since the func parameter is a callback, TS 2.4 is checking it more strictly than earlier versions.
The type of func in each overload needs to be assignable to the type of the func in the implementation.  Or, from the other direction: the type of func in the implementation should be a supertype of all the types declared for func in the overloads.
Let's call (p: Dog) => any  a Dog-consumer, and (p: Animal) => any an Animal-consumer.  Note that every Animal-consumer is a Dog-consumer (because something that consumes animals will accept a dog), but not every Dog-consumer is an Animal-consumer (because something that consumes dogs might not accept, say, a cat).  That means that Dog-consumer is a supertype of Animal-consumer.  Since one overload takes a Dog-consumer, and the other takes an Animal-consumer, the implementation should take a supertype of these: Dog-consumer.
That is:
function test(func: (p: Dog) => any): any;
function test(func: (p: Animal) => any): any;
function test(func: (p: Dog) => any): any {  // no error now

Ahh, so I confused a "callback parameter" with the return type.
I would expect this to be valid but it's not, why?
function test(func: (p: Dog) => any): any;
function test(func: (p: Animal) => any): any;
function test(func: (p: Animal|Dog) => any): any {

Note that func: ((p: Animal) => any) | ((p:Dog) => any) is essentially equivalent to func: ((p:Dog) => any).  If you use the union, you'll find that you can't actually call func() on anything directly (see #14107), so the latter is probably preferable.

@jcalz I noticed that.  Ended up with something like this:
function test<T,A extends Dog   >(animal:A, func: (p: A) => T): T;
function test<T,A extends Animal>(animal:A, func: (p: A) => Foo<T>): Foo<T>;
function test<T,A extends Animal>(animal:A, func: (p: A) => T|Foo<T>): T|Foo<T> {
    return func(animal);
}

I've included Foo<T> just so it would make sense as to why the overload would be defined
I'm guessing that is the best way?

Crosspost reference https://stackoverflow.com/questions/45087264/how-to-cleanly-write-a-typescript-function-with-different-callback-overload-para for anyone who ends up here


Error with running stress

Hi there,
I have install the package by the mentioned instructions. But when a I am  going to run the "make stress" the following error has appeared.
Error: Failed to launch the browser process!
[0510/195146.876103:ERROR:zygote_host_impl_linux.cc(90)] Running as root without --no-sandbox is not supported. See https://crbug.com/638180.
I will be so grateful if you guide me how I can resolve this error.
thank you.

Hi,
You have to install and run the "make stress" not as root

Hi,
@Fry-4TF1V , I ran "make stress" not as root,but still I face above mentioned issue,could you please elaborate the instructions to be followed,thank you.

Hi @06dilipg,
You can rely on some part of script I did few months ago which worked for me (did not test it recently) :

          "cd /tmp",
          "git clone https://github.com/openfun/bbb-stress-test.git",
          "sudo apt install make -y",
          "cd /tmp/bbb-stress-test/",
          "sudo add-apt-repository universe -y",
          "sudo apt update",
          "sudo apt install docker-compose -y",
          "make bootstrap",
          "sed -i 's/BBB_URL=.*/BBB_URL=https:\\/\\/${var.bbb_fqdn}\\/bigbluebutton\\//g' .env",
          "sed -i 's/BBB_SECRET=.*/BBB_SECRET=${var.bbb_secret}/g' .env",
          "sed -i 's/BBB_MEETING_ID=.*/BBB_MEETING_ID='$(sudo make list-meetings | grep participants | head -n 1 | awk '{print $3}')'/g' .env",
          "sed -i 's/BBB_TEST_DURATION=.*/BBB_TEST_DURATION=${var.params.duration}/g' .env",
          "sed -i 's/BBB_CLIENTS_LISTEN_ONLY=.*/BBB_CLIENTS_LISTEN_ONLY=${var.params.listen_only}/g' .env",
          "sed -i 's/BBB_CLIENTS_MIC=.*/BBB_CLIENTS_MIC=${var.params.micro}/g' .env",
          "sed -i 's/BBB_CLIENTS_WEBCAM=.*/BBB_CLIENTS_WEBCAM=${var.params.webcam}/g' .env",
          "make stress",

This script is based on this file

Hi @06dilipg,
You can rely on some part of script I did few months ago which worked for me (did not test it recently) :
          "cd /tmp",
          "git clone https://github.com/openfun/bbb-stress-test.git",
          "sudo apt install make -y",
          "cd /tmp/bbb-stress-test/",
          "sudo add-apt-repository universe -y",
          "sudo apt update",
          "sudo apt install docker-compose -y",
          "make bootstrap",
          "sed -i 's/BBB_URL=.*/BBB_URL=https:\\/\\/${var.bbb_fqdn}\\/bigbluebutton\\//g' .env",
          "sed -i 's/BBB_SECRET=.*/BBB_SECRET=${var.bbb_secret}/g' .env",
          "sed -i 's/BBB_MEETING_ID=.*/BBB_MEETING_ID='$(sudo make list-meetings | grep participants | head -n 1 | awk '{print $3}')'/g' .env",
          "sed -i 's/BBB_TEST_DURATION=.*/BBB_TEST_DURATION=${var.params.duration}/g' .env",
          "sed -i 's/BBB_CLIENTS_LISTEN_ONLY=.*/BBB_CLIENTS_LISTEN_ONLY=${var.params.listen_only}/g' .env",
          "sed -i 's/BBB_CLIENTS_MIC=.*/BBB_CLIENTS_MIC=${var.params.micro}/g' .env",
          "sed -i 's/BBB_CLIENTS_WEBCAM=.*/BBB_CLIENTS_WEBCAM=${var.params.webcam}/g' .env",
          "make stress",

This script is based on this file

Hi @Fry-4TF1V,
All the above configurations are done. At the end, when i give the command "make stress" i'm getting below error:
ERROR: Couldn't connect to Docker daemon at http+docker://localhost - is it running?
If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.
make: *** [Makefile:47: stress] Error 1
FYI: I'm installing the bbb-stress-test in different server and bigbluebutton is installed in different server.

Hi,
You have to install and run the "make stress" not as root

How I can run this?

Had the same issue, I edited the file lib/stress-test.js, added de arg "--no-sandbox" in the array declared on line 80:
args: [           "--use-fake-device-for-media-stream",           "--use-fake-ui-for-media-stream",           "--mute-audio",           "--no-sandbox"         ],
Not sure if that was the best but it work, now I'm facing the error mentioned here: https://github.com/openfun/bbb-stress-test/issues/11

I had the same problem, if you follow this steps to make it possible to run docker as non-root, and then go back to the very beginning to build the image and run the tests, everything works fine


Add CI coverage for 1.28

Tell us about your request
Add a ci-test (1.28.x) job to validate unit tests against k8s 1.28.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
No response
Are you currently working around this issue?
No response
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Yes, we should add 1.28, but first should fix the timeout condition that happens with 1.25, 1.26, 1.27 that causes these ci pipelines to fail.
then after we fix that timeout condition we can add 1.28 to the matrix

https://github.com/Azure/karpenter/actions/runs/6790866753/job/18461231128?pr=9 referring to these types of runs we see a ton of failing after 10 minutes in our runs, might be worth investigating further
PASS
panic: Log in goroutine after TestAzure has completed: 2023-11-07T22:18:44.558Z	ERROR	pricing	pricing/pricing.go:166	error updating on-demand pricing for region , no on-demand pricing found, using existing pricing data from 2023-10-27T06:44:32Z

goroutine 27 [running]:
testing.(*common).logDepth(0xc0007e64e0, {0xc0004a0300, 0xbd}, 0x3)
	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1022 +0x674
testing.(*common).log(...)
	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1004
testing.(*common).Logf(0xc0007e64e0, {0x21e7df6, 0x2}, {0xc0002a01b0, 0x1, 0x1})
	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1055 +0x9a
go.uber.org/zap/zaptest.testingWriter.Write({{0x2520148?, 0xc0007e64e0?}, 0x1?}, {0xc00090e800, 0xbe, 0x400})
	/home/runner/go/pkg/mod/go.uber.org/zap@v1.26.0/zaptest/logger.go:130 +0x11e
go.uber.org/zap/zapcore.(*ioCore).Write(0xc0007c9a70, {0x2, {0xc14acbd12144e29a, 0x82b9dc4, 0x3275560}, {0x21ea1d2, 0x7}, {0xc000814000, 0x7f}, {0x1, ...}, ...}, ...)
	/home/runner/go/pkg/mod/go.uber.org/zap@v1.26.0/zapcore/core.go:99 +0x193
go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc0005aa1a0, {0x0, 0x0, 0x0})
	/home/runner/go/pkg/mod/go.uber.org/zap@v1.26.0/zapcore/entry.go:253 +0x2b0
go.uber.org/zap.(*SugaredLogger).log(0xc0003540a8, 0x2, {0x225c39c, 0x57}, {0xc0008eefa0, 0x3, 0x3}, {0x0, 0x0, 0x0})
	/home/runner/go/pkg/mod/go.uber.org/zap@v1.26.0/sugar.go:316 +0x130
go.uber.org/zap.(*SugaredLogger).Errorf(0xc00004ae28?, {0x225c39c, 0x57}, {0xc0008eefa0, 0x3, 0x3})
	/home/runner/go/pkg/mod/go.uber.org/zap@v1.26.0/sugar.go:189 +0x85
github.com/Azure/karpenter/pkg/providers/pricing.(*Provider).updatePricing.func1()
	/home/runner/work/karpenter/karpenter/pkg/providers/pricing/pricing.go:166 +0x26d
created by github.com/Azure/karpenter/pkg/providers/pricing.(*Provider).updatePricing in goroutine 173
	/home/runner/work/karpenter/karpenter/pkg/providers/pricing/pricing.go:163 +0x1a5
ginkgo run failed
  could not finalize profiles:
  Unable to read coverage file /home/runner/work/karpenter/karpenter/pkg_providers_pricing_coverage.out:
  open /home/runner/work/karpenter/karpenter/pkg_providers_pricing_coverage.out: no such file or directory
make: *** [Makefile:31: battletest] Error 1
Error: Process completed with exit code 2.

Fixed in https://github.com/Azure/karpenter/pull/31


Presentation: Add ability to use selection storage from @itwin/unified-selection

Added ability to pass selection storage created using @itwin/unified-selection. This allows to decouple unified selection storage from presentation frontend and allows to store different kinds of selectables instead of only instance keys and node keys.

need an entry in NextVersion.md

still missing this


make clippy reports lots of warning

clippy issue
What version of Rust are you using?
rustc 1.28.0-nightly (b907d9665 2018-06-13)
clippy 0.0.207
What did you do?
cargo clean && cargo clippy
What did you expect to see?
No warning should be reported.
What did you see instead?
warning: long literal lacking separators
  --> src/util/codec/number.rs:20:24
   |
20 | const SIGN_MARK: u64 = 0x8000000000000000;
   |                        ^^^^^^^^^^^^^^^^^^ help: consider: `0x8000_0000_0000_0000`
   |
   = note: #[warn(unreadable_literal)] on by default
   = help: for further information visit https://rust-lang-nursery.github.io/rust-clippy/v0.0.207/index.html#unreadable_literal

warning: module has the same name as its containing module
  --> src/util/jemalloc.rs:15:1
   |
15 | / mod jemalloc {
16 | |     use libc::{self, c_char, c_void};
17 | |     use std::{ptr, slice};
18 | |
...  |
56 | |     }
57 | | }
   | |_^
   |
   = note: #[warn(module_inception)] on by default
   = help: for further information visit https://rust-lang-nursery.github.io/rust-clippy/v0.0.207/index.html#module_inception

warning: module has the same name as its containing module
  --> src/coprocessor/codec/chunk/mod.rs:14:1
   |
14 | mod chunk;
   | ^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang-nursery.github.io/rust-clippy/v0.0.207/index.html#module_inception

And lots of other warnings.
@Hoverbear is there a same result in your environment? If so, I can help to fix those warnings and add the clippy check in ci and circle-ci.

You should not invoke cargo clippy, use make clippy instead because we added some globally allowed rules (see https://github.com/pingcap/tikv/blob/master/Makefile#L40)