Unable to Add Bot Icon in Custom Web-Chat.

Hi Guys,
I have created my customized version of web-chat from the botchat.css and botchat.js provided in the description.
https://unpkg.com/botframework-webchat/botchat.css
https://unpkg.com/botframework-webchat/botchat.js
I just copied those two files and saved it my project.
I have noticed that botchat.js is minified version. Can I get a developer version for it to add bot icon ?
Is there something I am missing out for adding bot icon.
Thanks,
Manish Chitre.

Closing as duplicate of #43 - please follow that issue for progress on this feature


Boilerplate/ToC for posts

People can arrive at a given post from anywhere, and so it would be good to have a short intro on each post to put things into perspective and provide a pointer to the other content–it's nice to keep this intro fairly consistent across the series. Here's a draft but obviously open to edits.
Your Title Here
The Tabula Muris project is built on a large set of single-cell mRNA sequencing data, representing around 100000 cells from 20 organs and tissues from four mice. We are hopeful that this dataset will be broadly useful to the community as a place to validate findings, formulate new biological hypotheses, and benchmark novel methods and tools. To get the party started, we're publishing a series of posts on our blog that highlight different aspects of this dataset and the different research questions that it led us to explore.
Posts in this series:

Intro - introduction and overview
Another post - a post
Something else - another post

Staring at this a little, I don't like the idea of putting this text right below a post-specific title. I think either it should go at the end of each post, or as a preface before the main text.

I prefer with links because having unnecessary text just for the sake of context in a blog post is not appealing to me

To clarify after some discussion: this is just a suggestion for this set of 4-5 posts, to tie them together. The future organization of the blog as whole can be something different, but I'd like to have the debut hang together in a coherent way.


Add basic structure of green's function

Basic Structure

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cc34165). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   90.90%
=========================================
  Files             ?        1
  Lines             ?       11
  Branches          ?        0
=========================================
  Hits              ?       10
  Misses            ?        1
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc34165...f29b585. Read the comment docs.


Send a message on starting

It's better to send a message to a slack channel on starting instead of updating.
Starting is the first part of the deploying process.
I'll try to explain the changes.
We have many developers and we don't want to conflict on deploy. First steps in our process deployment are:

Send TSTP signal to sidekick-server
Graceful shutdown for other services

This causes to us delay for 1.5 minutes before starting deploy in the console and the message in the channel.

@krim So it's been awhile since I've thought about this, but I think the reason I went with updating and not starting is that the latter includes various pre-checks which could fail (bad ssh, etc..) leaving others potentially confused by a starting message without any finished/failed message.  My memory is there's no hook to tie into then post a "pre deploy checks failed" sort of message.
I can totally see your side of the coin though.  Curious if you have thoughts on the above?

@phallstrom I see your point. When something went wrong(wrong ssh key etc.) we will have a failed message after a starting message. I think that's really that we need to have.
E.g. why we have failed message without starting? It looks like strange 'cause something has to be started before failed.
Some examples:

Puma(it writes Booting Puma on starting when failed if something went wrong)
Sidekiq(it writes Booting Sidekiq on starting when failed if something went wrong)

@krim - Oh... you get a failure message if deploy:check fails (sadly, I don't use capistrano nearly as much as I used to so may have missed this)? I don't think it used to do it that way, but if it does... then this seems like a good solution.  I'll try and give it a serious look soon.

More details here.

@phallstrom Hello, have you got any news?

@krim - Sorry, got busy.  So... playing around with this the problem is that if deploy:check fails (ie, bad ssh host, can't make directories, etc) then we'll have already sent out a message to Slack about it starting, but there's no way to send a "it failed" as capistrano exits (at least it does if the ssh host is bad, on 3.8.1).
That makes me hesitant to make this the default behavior, but maybe I'm too worried about the setup and once it's fine it would make more sense to have it sooner?
Alternatively we could take what you'd done and provide the hook, but not enable it and you could use a custom class to send a starting message.
Thoughts?

@krim scratch that. It does send a failure message. Well that's a nice addition!  I'll look at getting this in place and shipped.

Went with a slightly different implemention. https://github.com/phallstrom/slackistrano/commit/ca040cbf3705c9ee65f75546331e59d083e67aee

@krim 4.0.1 was released just now and should do what you want. Note it will include both a starting and updating message, but you could silence the second via a custom messaging class. Occurs to me you may want it to know both when things started and when you're really doing the update and not the pre-check stuff.
Thanks!


TestFlight integration

Better integration for TestFlight, e.g. auto-submit new builds to TestFlight.

+1


Missing error code

When a user reports fraud during an azure sign in for example to portal.azure.com, an error code shows up on their page which looks like this
Error Code: 500121
Request Id: 07hf610f-249-46sd-a1d7-3ccbdc5d6b00
Correlation Id: fde7a069-5cdf-4439-83a4-0899695ea5d2
Timestamp: 2023-12-01T01:01:11Z
This error code 500121 is not listed on this page.

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

ID: a826ee6f-f457-7109-9f77-67c57977d4e7
Version Independent ID: 11766f27-492e-a579-dced-8a42528cbb47
Content: Microsoft Entra authentication & authorization error codes - Microsoft identity platform
Content Source: docs/identity-platform/reference-error-codes.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@tvjust
Thanks for your feedback! We will investigate and update as appropriate.

Hi@tvjust
For your information The Error Code: 500121 indicates Authentication failed during strong authentication request. The user didn't complete the MFA prompt or they may have decided not to authenticate or timed out while doing other work, or has an issue with their authentication setup.
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@tvjust
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation


catch wrong enabledqmharvest parameter in reqmgr

The workflow dictionary parameter enabledqmharvest recently changed to enableharvesting, but the request manager does not catch workflows where the old parameter is used, it simply ignores that parameter. Please implement a check that the correct parameter used and give a useful error message in case the wrong parameter is used.
Thanks.
Andrew

I guess we do not have any workflows using the old parameter anymore.
So instead of adding more code into WMCore, I would suggest to simply close this issue.
@ticoann what do you think?

@AndrewLevin, Andrew, can we close this?

Can you make a list of allowed parameters and check all of the parameters against this list?

Yes, we can. All the parameters are base parameters defined here.
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMSpec/StdSpecs/StdBase.py#L874
But specific parameters are defined for each spec
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMSpec/StdSpecs/MonteCarloFromGEN.py#L98
We can check whether some parameter is passed which is not defined in there, through the validation error. Is that what you want?

Yes, that is what I want.

Hmm, I agree that we should only allow expected parameters.
However, we sometimes need to provide another parameter that is not expected and
I would like not to have requests failing during injection, e.g., RequestSubType, Comments (that I'm using in my battery of requests).
This issue is basically 2 years old, why would we care about enabledqmharvest that is no longer used?!

Hi Alan,
If you want to have Comments and RequestSubType, can't you just add them to the list?
Do you need to use so many new parameters that you can't add them to the list before you use them?
Andrew

StdBase has only mandatory arguments for all the requests. These parameters are not mandatory, especially "Comments".
I insist, do you see any harvesting problems for the last months due to this retired parameter?

No, I don't see any harvesting problems in the last months. But I still think we should check the workflow parameters. Are we doing any checking of the workflow parameters now?

On May 9, 2015, at 3:29 PM, AndrewLevin <notifications<EMAIL_ADDRESS>wrote:
No, I don't see any harvesting problems in the last months. But I still think we should check the workflow parameters. Are we doing any checking of the workflow parameters now?
—
Reply to this email directly or view it on GitHubhttps://github.com/dmwm/WMCore/issues/4798#issuecomment-100542394.

All the parameters are validated (If it is not optional user should enter the value. If it is optional it will take the default value specified in case a user didn't specify the value) But we are not checking the values it is not defined. (Simply gets ignored). It is good to check whether parameter is defined or not. However, as Alan said if the parameter is newly defined between the deployment it will cause the problem. - Although I don't think you should add the parameters on the fly.

Yes, I think we should have a list of optional parameters and of required parameters, and we should check that all of the parameters that are received are in one of these lists. Alan, do you agree?

Provided we keep:

"SubRequestType" (for TaskChain only - optional)
"Comments" (for all workflows - optional)
It's fine with me. Even so, I'd mark it as low priority and/or improvement ...

Someone requested a workflow with EnableDQMHarvest again:
woodson_HLT_newconditions0_RelVal_245005_150529_211459_9221
Can you please change this issue to be high priority?

Just to clarify, this is for reqmgr2.

Hi Seangchan,
Now there is both a low priority label and a medium priority label. Can you please remove the low priority label?
Thanks.
Andrew

I'd remove the medium... it does not affect any operations, it's simply a mistake from PPD side!

It does affect operations.When they make this mistake we need to debug what happened and then rerun workflows. I really think you should add the medium priority label back.

Hi Andrew, the level of priority relative and we had to coordinate with other requirement. Actually, if there is alternative solution to handle the situation if goes lower level. We have to evaluate the issues for this month to define the priority. I know we have bad consequences when ops make mistake, however since this need to be done after reqmgr2 fixes are in I think we can set low for now, and move up higher when this can be applied.  Besides, Alan is development priority manager, so convince him if you really need something. :smile:

OK, I agree this should have low priority until request manager 2 is in production.

Where was the change of parameter name advertised to requesters ?

Hi Jean-Roch,
This happened so long ago I don't remember. Maybe Alan remembers.
Andrew

And this kind of problems won't happen again since we have a stricter argument validation now.


repo sync

Hello @eswar3! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file corex.py:

Line 513:38: E261 at least two spaces before inline comment
Line 513:80: E501 line too long (88 > 79 characters)
Line 514:1: W293 blank line contains whitespace


The behavior of json.Unmarshal when encountering field type inconsistencies.

func TestJson(t *testing.T) {
    type info struct {
       Id int64 `json:"id"`
    }
    type test struct {
       Result []info `json:"result"`
    }
    r := `{"result":[{"id":"1"},{"id":"2"}]}`

    var res1 test
    var res2 test

    // use encoding/json
    json2.Unmarshal([]byte(r), &res1)

    // use json-iterator/go
    jsoniter.ConfigCompatibleWithStandardLibrary.Unmarshal([]byte(r), &res2)

    len1 := len(res1.Result)      // 2
    len2 := len(res2.Result)      // 1

    fmt.Println(len1, len2)
}

I want to know which handling behavior is more appropriate.

The appropriateness of the behavior depends on your specific use case and requirements:
Strict Type Checking (Standard encoding/json):
Pros: Ensures that the data conforms exactly to the expected types, which can prevent subtle bugs and data inconsistencies.
Cons: Requires the input data to be strictly formatted, which might not always be the case, especially when dealing with external APIs or user-generated content.
Lenient Type Conversion (json-iterator/go):
Pros: More flexible in handling input data, which can be useful when dealing with inconsistent or loosely typed JSON data.
Cons: May mask data issues and lead to unexpected behavior if the type conversion does not work as intended.

Both Unmarshal functions return Error. You should handle errors first. Otherwise results are unpredictable.


Add fluxcd packages new version

What this PR does / why we need it

add fluxcd packages new version

kustomize-controller package v0.32.0
helm-controller package v0.28.1
source-controller  package v0.33.0

Which issue(s) this PR fixes

Fixes: #
Describe testing done for PR

Manually installed package on kind cluster
Special notes for your reviewer

LGTM

LGTM


i18n request

This plugin currently works well, and providing internationalized text will make it more popular I think.
My native language is Chinese and I would very much like to contribute to this project of internationalization.
It seems to helpful for first step in i18n: https://plugins.jetbrains.com/docs/intellij/internationalization.html

Hi,
Thanks for the interrest :)
What part(s) are you thinking of?
If you are thinking of the code / comment names, then (sorry, I forgot to document it :( ), I have actually added the ability to specify your "own" negation words, in a file named (placed at the root of the project)
.namesToHighlight

with one word on each line, like so
Test

resulting in

Oh, sorry for missing adequate description. I means text on settings, in other words user interface.

Thx :)

@aixcyi You are welcome :) however, due to the shared lib not being released yet,you might not be able to build it. If you want me to include the translations you come up, either just make a pr or let me know. Just glad you liked the plugin :)

Thanks to the small amount of text, I think it is possible to ensure translation accuracy manually, therefore, we can temporarily ignore problem that the plugin cannot build.
After all text that needs to be translated was extracted into a message bundle, i would be happy to translate them asap.
>w<


Help using this specific command on Windows to get 60fps video with audio via ffmpeg

Please follow the guide below

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.06.14. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.06.14

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Description of your issue, suggested solution and other information
A pretty simple question/help thing. I'm trying to get 60fps video with audio using youtube-dl and ffmpeg by using the following command "youtube-dl -f 'bestvideo[height<=720]+bestaudio/best[height<=720]' URLHERE". The URL is a Youtube link that does work perfectly fine, and does has a 60fps option
I get a 'The system cannot find the file specified' as output in command prompt. I do have ffmpeg installed.
Youtube-dl does work fine for more basic downloads.
I am using the executable.
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Post the URL.

From "FORMAT SELECTION" part of the readme:

Format selection examples
Note that on Windows you may need to use double quotes instead of single.


[Run] Turning off PT Run in the settings doesn't trigger the action in the runner

I'm running PowerToys (preview) v0.25.0 and ALT+SPACE is "eaten" whether running PowerToys as Admin or not, and whether PowerToys Run is disabled or not. If PowerToys Run is enabled then the search box pops up, if it is disabled nothing happens (no character passed to app). Changing the shortcut key to CTRL+ALT+SPACE does not help (ALT+SPACE is still "eaten").
I want to pass ALT+SPACE through to emacs as it is used as a command therein.
Originally posted by @adk3141 in https://github.com/microsoft/PowerToys/issues/4766#issuecomment-720525717

Good catch.  @enricogior, this is a must fix for 0.27

Workaround: change shortcut key to WinSpace or something.

If you mean change the PowerToys Run shortcut to Win+Space then, sadly, it does not help as Alt+Space is still "eaten". If you mean rchange my emacs keybinding for M-space to Win-space then that gets caught by window's language preference first.

I cannot reproduce it, for me it works as expected in both user & admin mode.
When PT Run is turned off, ALT + SPACE is not captured by PowerToys:

@adk3141
if you quit PowerToys, ALT+SPACE works in Emacs?
If you restart PowerToys, turn PowerToys Run off, quit and restart PowerToys, ALT+SPACE is still not working in Emacs?

I was able to reproduce it. It is hard to say what is the exact steps to reproduce.
The problem is that PowerLauncher.exe sometimes is not terminated when PowerToys run is disabled. After that, 'Enable PowerToys Run' switch has no effect. Probably it is a race condition.

Although previously the behaviour was always reproducible, it now seems intermittent. I also suspect it may be a race condition.

In 0.25 we introduced a drastic way of turning PT Run off: we kill the process.
In theory it should work reliably.

@adk3141
How often does it happen? How many times a day?
Once it happened does it happen always after that?
Is it enough to restart PowerToys to fix it?

It happened reproducibly when I first reported it. The problem doesn't seem to appear today, although I don't know what may have changed (no reboots, etc.). Restarting PowerToys didn't fix it when it wasn't working. The idea that a rogue PowerLauncher.exe process was around which has now gone away may explain the behaviour. I did not explicitly kill any such process.

weird, i was able to repro this yesterday but now it doesn't seem to.

@adk3141 runner should be grabbing the keystroke.

I find it weird too. It now works fine even if the PowerToysRunner process is present. What has changed?

nothing, this is why i think we have some type of bug here.  could be the running not getting an instant update from settings

Changed the title to reflect the actual problem.

We have decided to add logs to the runner process #7958. It potentially can add more context to such kind of issues.

@adk3141 we'll be adding in additional logging in for 0.29 which then once we get some data from you, we'll be able to hopefully resolve the issue in 0.31 (January release)

Marking this as resolved for 0.31.

Marking this as resolved for 0.31.

I haven't seem this (or any other) problem with Release v0.29.3.

I haven't seem this (or any other) problem with Release v0.29.3.

@adk3141
thanks for the follow up.

@adk3141
thanks for the follow up.


Fix iRegs wayfinder JS errors on odd paragraphs

Certain paragraphs in iRegs have odd data-label values, e.g., the first five paragraphs on
https://www.consumerfinance.gov/policy-compliance/rulemaking/regulations/1002/Interp-9/. This commit puts certain statements containing methods that were assuming other formats for that attribute inside conditionals to prevent those methods from erroring on the unexpected input.
/cc @contolini, @cfarm
Testing

Visit https://www.consumerfinance.gov/policy-compliance/rulemaking/regulations/1002/Interp-9/
Open the dev tools console and start scrolling down
Observe errors popping in, e.g., Uncaught TypeError: Cannot read property 'split' of undefined
Pull this branch
yarn run gulp scripts:apps
Visit http://localhost:8000/policy-compliance/rulemaking/regulations/1002/Interp-9/
Open the dev tools console and start scrolling down
Observe no errors!

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the CFPB development guidelines
[ ] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[x] Visually tested in supported browsers and devices (see checklist below :point_down:)
[x] Project documentation has been updated
[x] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[x] Chrome on desktop
[ ] Firefox
[ ] Safari on macOS
[ ] Edge
[ ] Internet Explorer 9, 10, and 11
[ ] Safari on iOS
[ ] Chrome on Android

@Scotchester: Whew, this one turned out to be tricker than I thought. I pushed an update that I think will handle the issue. Basically, interpretation paragraphs should have a data-label with "Interp" in it to be valid; the examples you saw here didn't have that in the data-label, so the wayfinder code that transforms the data-label into the marker text in the wayfinder was choking. I added a guard clause that makes the wayfinder just show "Comment" as the marker for any interpretation paragraphs with invalid data-labels now. That seems to be working for all of the commentary sections I spot checked.
At first I tried to get fancy and modify the code to figure out what the correct maker should be for paragraphs with invalid IDs like this. I can definitely imagine other IDs out there that are invalid in different ways, though, so that seemed like more work than it was worth. Instead I think the best way is to show the generic "Comment" as the marker in the wayfinder and then change the invalid ID to a valid one in the Regdown when we see cases like this.
Also, I know there are other ways than a guard clause to do this, so feel free to change to another method if you have any other ideas. I didn't look at any JS unit tests to see if we need to cover this case yet, either.

@Scotchester: OK, I just added a unit test for the guard clause, but I have no idea if I did it right, so definitely feel free to redo that, too.

Hahah, wow, I really didn't test this PR very thoroughly, did I? 😆 Sorry, @niqjohnson, and thanks for picking up the slack. Your changes are working great for me!


[wts-pack] Update webTestService packaging script and rename to pack_wts.sh

1. Add common and resources folders in package.
2. Add version.json file in package.

BUG=XWALK-4557

LGTM

@haoyunfeix @junli Pls create a new WTS pkg with this PR fix, and @junli can verify if the new pkg meet the request :)


Can not setup and configure with Vite / React / TS+SWC

Current behavior
Unable to configure and run after creating new project with vite.
Desired behavior
The configuration should work for new projects created with vite.
Test code to reproduce
$ npm create vite
-> test
-> React
-> Typescript + SWC

$ npm i

$ npm i -D cypress # running, working, all good with the demo tests

$ npm i -D @badeball/cypress-cucumber-preprocessor

$ npm i -D @bahmutov/cypress-esbuild-preprocessor

Cypress configuration according to docs:
import { defineConfig } from "cypress";
import createBundler from "@bahmutov/cypress-esbuild-preprocessor";
import { addCucumberPreprocessorPlugin } from "@badeball/cypress-cucumber-preprocessor";
import createEsbuildPlugin from "@badeball/cypress-cucumber-preprocessor/esbuild";

async function setupNodeEvents(
    on: Cypress.PluginEvents,
    config: Cypress.PluginConfigOptions
): Promise<Cypress.PluginConfigOptions> {
    // This is required for the preprocessor to be able to generate JSON reports after each run, and more,
    await addCucumberPreprocessorPlugin(on, config);

    on(
        "file:preprocessor",
        createBundler({
            plugins: [createEsbuildPlugin(config)],
        })
    );

    // Make sure to return the config object as it might have been modified by the plugin.
    return config;
}

export default defineConfig({
    e2e: {
        baseUrl: "https://duckduckgo.com",
        specPattern: "**/*.feature",
        setupNodeEvents,
    },
});

When I start Cypress, there are two errors:
import { addCucumberPreprocessorPlugin } from "@badeball/cypress-cucumber-preprocessor";
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: Named export 'addCucumberPreprocessorPlugin' not found.

This can be fixed easily by modifying the import as follows:
import preprocessor from "@badeball/cypress-cucumber-preprocessor";
const { addCucumberPreprocessorPlugin } = preprocessor

Once fixed, we move to the next error:
TypeError: createEsbuildPlugin is not a function
    at setupNodeEvents

Link to repository to reproduce: https://github.com/mpavel/vite-react-ts-swc-cypress-bdd
tsconfig has been updated to contain node16 and it didn't work.
    "module": "node16",
    "moduleResolution": "node16",

The original values coming from vite are:
    "module": "ESNext",
    "moduleResolution": "bundler",

I've added the "paths" to tsconfig and it didn't work:
    "paths": {
      "@badeball/cypress-cucumber-preprocessor/*": [
        "./node_modules/@badeball/cypress-cucumber-preprocessor/dist/subpath-entrypoints/*"
      ]
    }

I've renamed cypress.config.ts to cypress.config.js and it still didn't work.
Versions

Cypress version: ^13.1.0
Preprocessor version: ^18.0.5
Node version: v18.16.1

Checklist

[x] I've read the FAQ.
[x] I've read instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

The configuration should work for new projects created with vite.

This is not even remotely true, nor something I consider a requirement. Vite can create whatever TS configuration they want and it's not necessarily going to fit all. Your example works by applying the following patch. If you want to retain type: module, then you also need to understand the implications of it. However, I won't be your guide through this.
diff --git a/cypress.config.js b/cypress.config.ts
similarity index 86%
rename from cypress.config.js
rename to cypress.config.ts
index 24fa4f1..8d41b54 100644
--- a/cypress.config.js
+++ b/cypress.config.ts
@@ -1,9 +1,7 @@
 import { defineConfig } from 'cypress';
 import createBundler from '@bahmutov/cypress-esbuild-preprocessor';
 import createEsbuildPlugin from '@badeball/cypress-cucumber-preprocessor/esbuild';
-import preprocessor from '@badeball/cypress-cucumber-preprocessor';
-
-const { addCucumberPreprocessorPlugin } = preprocessor
+import { addCucumberPreprocessorPlugin }  from '@badeball/cypress-cucumber-preprocessor';

 async function setupNodeEvents(on, config) {
     // This is required for the preprocessor to be able to generate JSON reports after each run, and more,
diff --git a/package.json b/package.json
index dc0044b..797e9ec 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,6 @@
   "name": "learn-vite-react-cypress",
   "private": true,
   "version": "0.0.0",
-  "type": "module",
   "scripts": {
     "dev": "vite",
     "build": "tsc && vite build",
diff --git a/tsconfig.json b/tsconfig.json
index 04139d1..03977b6 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -3,11 +3,11 @@
     "target": "ES2020",
     "useDefineForClassFields": true,
     "lib": ["ES2020", "DOM", "DOM.Iterable"],
-    "module": "ESNext",
+    "module": "nodenext",
+    "esModuleInterop": true,
     "skipLibCheck": true,

     /* Bundler mode */
-    "moduleResolution": "bundler",
     "allowImportingTsExtensions": true,
     "resolveJsonModule": true,
     "isolatedModules": true,

For the extremely curious - TS's output of named exports is sometimes not detected by Node's internal lexer (https://github.com/nodejs/cjs-module-lexer), ref. https://github.com/nodejs/cjs-module-lexer/issues/76. This is (one of the reasons) why using type: module can sometimes be difficult.

Thank you very much for looking into this!
I agree that there are so many TS configurations out there and it's impossible to keep track of everything.
Since vite is gaining traction I was thinking it would be nice to have a more seamless DX when it comes to installing Cypress and setting up this plugin. That's the reason I opened up this issue. Also with the idea that it will help some future developer debug things if needed 😊
I will put below, for reference, the changes that made things work for me.
// cypress.config.ts
import { addCucumberPreprocessorPlugin } from "@badeball/cypress-cucumber-preprocessor";
import createBundler from "@bahmutov/cypress-esbuild-preprocessor";
import { defineConfig } from "cypress";

// note: import had to be adjusted, compared to the ones in examples
const { createEsbuildPlugin } = require("@badeball/cypress-cucumber-preprocessor/esbuild");

async function setupNodeEvents(
  on: Cypress.PluginEvents,
  config: Cypress.PluginConfigOptions
): Promise<Cypress.PluginConfigOptions> {
  // This is required for the preprocessor to be able to generate JSON reports after each run, and more,
  await addCucumberPreprocessorPlugin(on, config);

  on(
    "file:preprocessor",
    createBundler({
      plugins: [createEsbuildPlugin(config)],
    })
  );

  // Make sure to return the config object as it might have been modified by the plugin.
  return config;
}

export default defineConfig({
  e2e: {
    baseUrl: "https://duckduckgo.com",
    specPattern: "**/*.feature",
    setupNodeEvents,
  },
});

In package.json, remove "type": "module" (which basically defaults to commonjs)
In tsconfig.json and tsconfig.node.js, remove "moduleResolution": "bundler" and add "esModuleInterop": true.
@badeball , should I open a PR to update the vite example? I'm especially looking at the import in the cypress.config.ts, but maybe also to include references to the other required changes.

For reference - I've updated the default exports to play nicer with ESM (834b65b).

@badeball , should I open a PR to update the vite example? I'm especially looking at the import in the cypress.config.ts, but maybe also to include references to the other required changes.

I'm not sure what you think needs to be changed in the example.

@badeball , as written above in the code, the import doesn't work as it's in the example, but must be adjusted in the following way:
// note: import had to be adjusted, compared to the ones in examples
const { createEsbuildPlugin } = require("@badeball/cypress-cucumber-preprocessor/esbuild");

This happens after making the configuration changes I mentioned:

In package.json, remove "type": "module" (which basically defaults to commonjs)
In tsconfig.json and tsconfig.node.js, remove "moduleResolution": "bundler" and add "esModuleInterop": true.

This is something I wanted to highlight that might be worth changing in the documentation. Or in the source code 😄 This is the VSCode error:

From my side it's fine, I figured it out now and I know how to work around it, by changing the import. 😊

What example are you referring to?

The import above works, once "paths" are added like mentioned in the documentation, but it's just TS complaining about the imports.
I also found that this can be solved with creating a declarations file at root level:
// global.d.ts
declare module '@badeball/cypress-cucumber-preprocessor/esbuild';

I'm referring to the Example setup: https://github.com/badeball/cypress-cucumber-preprocessor/blob/master/docs/quick-start.md

Well, createEsbuildPlugin doesn't originate from @bahmutov/cypress-esbuild-preprocessor, nor does the example suggest so. So, what's the problem here?

The main problem I had was trying to find a configuration that works, especially with Vite. This is solved and your first comment helped with this. Thank you!

I ran into the same problem. "type":"module" in package.json was needed to run the project with Vite, but needed to be removed to run Cypress with the cucumber-preprocessor.
If you rename the vite.config.ts file to vite.config.mjs. You don't need to add "type":"module" to your package.json file and the project runs, and so does Cypress. (at least in my case, hopefully this helps someone!)


Maintaining Cross Domain session with passport.js

I am using nodeJs for server and with express and passport as authentication middleware. My front end app is running on angular on localhost:9000 and node app on localhost:3000 . I found that the passport is not able to save session in cross domain . on every page request on my client app , passport create a new session for me instead of maintaining a single session . Also after login on login page it create a session in mongo with user detail available in it , but on second request it never resolve the same session cookie for me  .

I'm having the same issue, did you find a solution?, if not I would really appreciate any help, the login works but the session is not maintained, I have already made sure all headers are fine and that the cookie actually gets to the backend, but it still doesn't work

Have you been able to find a solution for this? I'm facing the same problem.

+1

I have tried
var cors = require('cors');
var app = express();
app.use(cors());

and
app.all('*', function(req, res, next) {
       res.header("Access-Control-Allow-Origin", "*");
       res.header("Access-Control-Allow-Headers", "X-Requested-With");
       res.header('Access-Control-Allow-Headers', 'Content-Type');
       next();
});

But neither of them worked.

did anyone solve this ?

+1

Also +1

You'll need SSO (single sign on) implementation for that using tokens exchange,
however if you use subdomains and not two different domains it is possible to get a shared session by setting up a global session store (e.g with redis) and setting a cookie to the root domain like this :
const expressSession = require('express-session');
const redisStore = require('connect-redis')(expressSession);
...
app.use(
  expressSession({
    store: new redisStore({ host: 'localhost', port: 6379 }),
    secret: 'keyboard cat',
    cookie: { domain: 'domain.com' },
    resave: false,
    saveUninitialized: false
  })
);
app.use(passport.initialize());
app.use(passport.session());

This way you'll get shared session for both app1.domain.com and app2.domain.com
However for cross domains you'll need SSO implementation with another authorization server (e.g sso.otherdomain.com) which will exchange tokens for each domain after authenticating to to the authentication server. I'm trying to find an example/documentation for passport with SSO for myself as well but couldn't find anything so far (closest I've found is this)

For making cross-domain requests we are using the proxy, to avoid of CORS issue.
TL;DR
I think that issue is in the proxy. And we need to set up some "reverse proxy" on NodeJS API side.
I'm also using separate apps:
For server API: http:// localhost:3000/
For client SPA: http://localhost:8080/
When I'm sending request from http://localhost:8080/ I'm using /api-local/ prefix for http-proxy-middleware, so my config looks like:
'/api-local/': {
  target: http:// localhost:3000/,
  pathRewrite: { '^/api-local/': '/' },
  changeOrigin: true,
}

So the next request in SPA fetch('/auth') will be interpreted as fech('http:// localhost:3000/auth').
The trick is that authentication at NodeJS side works very well, all data from LinkedIn are returns and the passportjs is invoke serializeuser method.
When I'm sending the second request after authentication (it does not matter if it is protected or unprotected API request) - API after receiving a request, invoking deserializeUser passportjs method.
Only when I'm disabling passport session with { session: false } option - it works ok.
So there are few conclusions:

passportjs make its job with authentication
Next request after authentication works only with user wast authenticated without a session
there is something wrong with the session OR during next requests, information about user session is missing in request


Format JSON in container-registry-delete.md

@microsoft-github-policy-service agree

@tejaswikolli-web
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hello @tejaswikolli-web, sorry for pinging but I don't know how to remove inactive tag. This is ready for review.

@tejaswikolli-web
Can you review this old PR and determine whether it needs to be closed or merged?
@MicrosoftDocs/public-repo-pr-review-team


Is F*'s prop type the same as Coq's Prop type?

Not exactly even though they are intended for the same kind of use. Coq's Prop is an impredicative universe whereas F*'s prop is defined as a subtype of a predicative universe :
type prop = a:Type0{ forall (x:a). x === () }
The === is (extensional) heterogeneous equality which means that an F* prop type is a subsingleton (it has at most one element) . Typical F* prop are True (aka unit), False or a squashed type (a refinement of unit).

Don't see how True and False defined as inductives could be props with respect to this definition though. On the catalin_prop branch I have already switched to your better definition of prop that would allow this (while also not relying on heterogenous equality):
https://github.com/FStarLang/FStar/blob/catalin_prop/ulib/prims.fst#L127


create-SuperAdmin and Admin UX

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[ ] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

short video description regarding this pull request https://www.loom.com/share/a5832396e42843189ca77ade50d571d9

Nice design. Great look and feel. Good job!


Unable to upgrade to chart version 0.5.4 (on Helm2 client)

Steps to reproduce

Have a timescale v0.5.3 on your cluster (see "Environment" below):
$ helm list
NAME                    	REVISION	UPDATED                 	STATUS  	CHART                       	APP VERSION	NAMESPACE
...
timescale               	9       	Tue Mar 3 19:03:37 2020	DEPLOYED	timescaledb-single-0.5.3    	           	<redacted>

Try to upgrade to v0.5.4:
$ helm upgrade --install timescale  timescaledb/timescaledb-single --set  ... --version=0.5.4

Results in:
Error: parse error in "timescaledb-single/templates/statefulset-timescaledb.yaml": template: timescaledb-single/templates/statefulset-timescaledb.yaml:225: function "concat" not defined

Workaround available
Stick to 0.5.3; we're unable to upgrade our helm client/server to v3 due to a deployment system that currently requires v2 (as v3 is not backwards compatible)
Environment
Kubernetes version 1.14.9
Helm Client/Server: 2.14.2

This is not a Helm 2 vs Helm 3 issue, I can install just fine with Helm 2.16.1.

I'll be closing this, with the solution being: Upgrade to Helm 2.16 or Helm 3.


Save cursor location when editor.action.selectAll (Ctrl+a)

When <Ctrl+a> is pressed (editor.action.selectAll) the cursor goes to the end of the file. I would expect to return to the previous position if the selection is cancelled with <ESC>.
As an example, let's say you are in the middle of file, if you want to select all (and copy it) but later continue where you left, by pressing <ESC> the selection disappear but the cursor will be at the end of the file. I would expect the following behavior: <Ctrl+A> (select All), <Ctrl+C> (Copy), <ESC> and continue where I left.
Also it should be consistent with the Up/Down/Left/Right keys (and the rest of the navigation keys).
Can the position of the cursor be on the middle of a selection, instead of only at the beginning/end?

There is a special action, Cursor Undo (I believe bound to ctrl+u on Windows by default) which undoes the last cursor operation (up to the last edit), so you can use ctrl+a, ctrl+c, and ctrl+u. Cursor Undo is a helpful command to know about, since it works the same in all cases, not just for ctrl+a. For example, I use it a lot in combination with ctrl+d if I go one too many words, etc.

thanks for the recomendation!


udta MUID as identifier for clips in a recording session?

I've asked before regarding identifying clips in the same recording session, and recently bumped into the udta field MUID. Since this is supposed to represent "MediaUniqueID", and seems to have values that are identical for clips in the same session (inclusive array indices 0-18), could this be used to identify clips in the same recording session? The remaining part seems unique for each individual clip.
Would anyone have any hints as to what data types MUID should be interpreted as?

Actually, in the GPMF chunk in the udta atom, MUID is repeated as a GPMF-stream with FourCC MUID and type L/32-bit unsigned (repeats 8, but the final four values are all 0). From what I can see, these four values are identical to the first 16 "raw" udta MUID "field" interpreted as uint32:s. Perhaps this is it, as far as ID:ing clips belonging to the same session goes?

Looking at the samples in this repo, Hero 6 and up (for the Hero series) all have a GPMF stream in the udta atom, whereas the Hero 5 sample only has this as a "regular" MP4 data chunk in the udta atom.

MUID is a 256-bit identifier based on the version 1 UUID from RFC 4122. All chapters with from one capture will share the same MUID.  It is annoying the chapter number does not appear in today's metadata, and MUID + chapter_num I would be more useful (limited to Omni and Fusion 360 products.)

@dnewman-gpsw That's good to know, thanks! Yes, a sequence number of sorts would be good to have. Since the GPS will be turned on at all times in our case, I could probably use the first half of the MUID as "session ID", then use GPSU to derive the sequence for the clips automatically.
Though, UUID1 has a time component. Perhaps that could be used? Assuming the cameras' clocks are properly set and that those are somehow used for the UUID time stamping, that is.

@dnewman-gpsw Hmm, it seems GPMF MUID and udta CAME are the same if you truncate the zeroes in GPMF UDTA...? I.e. is MUID more of a device ID? I compared with GoPro Max footage from DC Rainmaker's site, none chaptered, and they all share the same udat CAME and the first half of the values in MUID (udta and GPMF). For Hero 7 Black footage it's the same, but if I'm right it means I can't match clips from the same recording session based on MUID, since clips from a different recording session may have the same GPMF MUID.
Or is Max different to the Hero series in this regard?
Here are the values from three independent (not chaptered) GoPro Max clips from DC Rainmaker:
GS010002.mp4
GPMF MUID uint32: [618974091,<PHONE_NUMBER>, 369129341, 998971701, 0, 0, 0, 0]
udta MUID uint32: [618974091,<PHONE_NUMBER>, 369129341, 998971701, 245687047,<PHONE_NUMBER>, 21853, 0]
udta CAME uint8:  [139, 203, 228, 36, 172, 197, 179, 125, 125, 119, 0, 22, 53, 25, 139, 59]
udta MUID uint8:  [139, 203, 228, 36, 172, 197, 179, 125, 125, 119, 0, 22, 53, 25, 139, 59, 7, 227, 164, 14, 0, 0, 1, 69, 93, 85, 0, 0, 0, 0, 0, 0]

GS010010.mp4
GPMF MUID uint32: [618974091,<PHONE_NUMBER>, 369129341, 998971701, 0, 0, 0, 0]
udta MUID uint32: [618974091,<PHONE_NUMBER>, 369129341, 998971701, 245687047,<PHONE_NUMBER>, 50305, 0]
udta CAME uint8:  [139, 203, 228, 36, 172, 197, 179, 125, 125, 119, 0, 22, 53, 25, 139, 59]
udta MUID uint8:  [139, 203, 228, 36, 172, 197, 179, 125, 125, 119, 0, 22, 53, 25, 139, 59, 7, 227, 164, 14, 0, 0, 13, 172, 129, 196, 0, 0, 0, 0, 0, 0]

GS010018.mp4
GPMF MUID uint32: [618974091,<PHONE_NUMBER>, 369129341, 998971701, 0, 0, 0, 0]
udta MUID uint32: [618974091,<PHONE_NUMBER>, 369129341, 998971701,<PHONE_NUMBER>,<PHONE_NUMBER>, 52902, 0]
udta CAME uint8:  [139, 203, 228, 36, 172, 197, 179, 125, 125, 119, 0, 22, 53, 25, 139, 59]
udta MUID uint8:  [139, 203, 228, 36, 172, 197, 179, 125, 125, 119, 0, 22, 53, 25, 139, 59, 7, 227, 164, 75, 0, 4, 13, 183, 166, 206, 0, 0, 0, 0, 0, 0]

Three chaptered clips (same recording session) from a Hero 7 Black:
GH010026.MP4
GPMF MUID uint32: [1719790048,<PHONE_NUMBER>, 853951305,<PHONE_NUMBER>, 0, 0, 0, 0]
udta MUID uint32: [1719790048,<PHONE_NUMBER>, 853951305,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>, 57022, 0]
udta CAME uint8:  [224, 233, 129, 102, 227, 71, 251, 110, 73, 67, 230, 50, 245, 94, 67, 191]
udta MUID uint8:  [224, 233, 129, 102, 227, 71, 251, 110, 73, 67, 230, 50, 245, 94, 67, 191, 7, 226, 161, 204, 0, 1, 9, 109, 190, 222, 0, 0, 0, 0, 0, 0]

GH020026.MP4
GPMF MUID uint32: [1719790048,<PHONE_NUMBER>, 853951305,<PHONE_NUMBER>, 0, 0, 0, 0]
udta MUID uint32: [1719790048,<PHONE_NUMBER>, 853951305,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>, 490, 0]
udta CAME uint8:  [224, 233, 129, 102, 227, 71, 251, 110, 73, 67, 230, 50, 245, 94, 67, 191]
udta MUID uint8:  [224, 233, 129, 102, 227, 71, 251, 110, 73, 67, 230, 50, 245, 94, 67, 191, 7, 226, 161, 204, 0, 17, 13, 216, 234, 1, 0, 0, 0, 0, 0, 0]

GH030026.MP4
GPMF MUID uint32: [1719790048,<PHONE_NUMBER>, 853951305,<PHONE_NUMBER>, 0, 0, 0, 0]
udta MUID uint32: [1719790048,<PHONE_NUMBER>, 853951305,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>, 36111, 0]
udta CAME uint8:  [224, 233, 129, 102, 227, 71, 251, 110, 73, 67, 230, 50, 245, 94, 67, 191]
udta MUID uint8:  [224, 233, 129, 102, 227, 71, 251, 110, 73, 67, 230, 50, 245, 94, 67, 191, 7, 226, 161, 205, 0, 34, 3, 66, 15, 141, 0, 0, 0, 0, 0, 0]

Seem I'm back to square one and identifying clips in the same recording session via filename. :( I'll try to get more Hero test footage.

@dnewman-gpsw, we just received a Hero11 Black and I realise I have follow up questions.
The main question is: is it possible to let us know how MUID and GUMI are used between models?
I realise this is outside of GPMF (first half of udta MUID is available as part of the GPMF section in the udta for Hero6 and up, but GUMI only exists as a normal udta field from what I can see).
For Hero11 it seems all clips that belong to the same recording session will get the same MUID - no sequential info, but I can at least identify recording sessions this way. However, this is not true for e.g. Hero7 (only the first half matches but that is more of a unique device ID?). However, GUMI instead seems to be the same between clips for the same recording session for the Hero7.
Other than a Hero11, Hero7, and a Hero5, I don't have access to much in terms of cameras and test footage (which in this case need to be multi-clip recordings). I'm trying to automate most of the process (matching clips, merging GPMF etc) for researchers who use GoPros in the field with a tool I'm building (https://gitlab.com/rwaai/geoelan - though GoPro file/clip matching is broken/insufficient for this version, but geoelan i -g GOPROVIDEO.MP4 --verbose will print all GPMF data in "raw" form for that file for example).
Do you have any insight here, or information you are able to share? Especially if any of the above is true for models Hero6, and Hero8 - 10 as well. E.g. do Hero5-10 use the same GUMI for clips in the same session?

Sorry I don't have this info per camera model.

That's ok, thank anyway. But if you happen to come across something in the future I think it would help others as well. There are a lot of GoPros out there, that are used in many different ways. :)
I'll edit my code if I find more multi-clip recordings from previous models. So far I'm bundling clips from Hero11 recoding sessions on MUID and others on GUMI. I want/need to keep track of the low-resolution clips as well when available, so I match LRV with MP4 by comparing a hash of the full GPMF stream in raw form (just an array of uint8, perhaps just comparing the first DEVC stream will do), then order clips on GPS time stamps. Works so far and is quick enough, but I'll add ordering on file name as a fall back (often the researchers who use the cameras want to rename files to something a little more descriptive).


Feature Request: configurable SMTP servers

I already have an account with MailJet and would prefer to keep using it. It would be nice to have pop configurable for it.

Same here, self-hosted mail server. Such a lovely UI, would be great to be able to use it with any SMTP server! <3

Same! I have an account with migadu so Id love to continue using that and have all my sent in one place. Other than that this looks fantastic

Pop SMTP support is now included!
https://github.com/charmbracelet/pop/releases/tag/v0.2.0

@maaslalani thank you but it's still asks for a RESEND API:
`─# pop
ERROR   RESEND_API_KEY  environment variable is required.
You can grab one at https://resend.com/api-keys`


Confidence Interval Define Inside MultiSeriesArimaModel.predict_timeseries

Inside:
class MultiSeriesArimaModel(AbstractArimaModel):
There is:
    def predict_timeseries(
        self,
        horizon: int = None,
        include_history: bool = True,
        df: Optional[pd.DataFrame] = None) -> pd.DataFrame:
        """
        Predict target column for given horizon_timedelta and history data.
        :param horizon: Int number of periods to forecast forward.
        :param include_history: Boolean to include the historical dates in the data
            frame for predictions.
        :param df: A pd.Dataframe containing regressors (exogenous variables), if they were used to train the model.
        :return: A pd.DataFrame with the forecast components.
        """
        horizon = horizon or self._horizon
        ids = self._pickled_models.keys()
        preds_dfs = list(map(lambda id_: self._predict_timeseries_single_id(id_, horizon, include_history, df), ids))
        return pd.concat(preds_dfs).reset_index(drop=True)

Which calls:
self._predict_timeseries_single_id()
Then calls the original class:
ArimaModel()
Which has multiple function calls and eventually:
def _forecast(
    self,
    horizon: int = None,
    X: pd.DataFrame = None) -> pd.DataFrame:
    horizon = horizon or self._horizon
    preds, conf = self.model().predict(
        horizon,
        X=X,
        return_conf_int=True)
    ds_indices = self._get_ds_indices(start_ds=self._end_ds, periods=horizon + 1, frequency=self._frequency)[1:]
    preds_pd = pd.DataFrame({'ds': ds_indices, 'yhat': preds})
    preds_pd[["yhat_lower", "yhat_upper"]] = conf
    return preds_pd

How can I input my own customer Confidence interval? Essentially the conf for the forecast? Why does calling,
MultiSeriesArimaModel.predict_timeseries()
Not have a confidence input? Why can't I input 90%? or 70%?

All of this is inside model.py at path:
automl/blob/main/runtime/databricks/automl_runtime/forecast/pmdarima/model.py


xss_scanner.py issue

hi
in the function test_query_injection the body is converted to lower case (line 153) but in the get_SQLi_data function it is compared with upper case words (line 187) that results in no match
System information

Thanks for the report! Would you mind sending a pull request that fixes this? 😃

sure, also there are issuse with "decode" in log_SQLi_data function(line 176-179) that causes to error.
Im working on a new sqli example that also test post data for sqli and save vulnerable hosts to file.

@mhils can I submit a PR to fix this issue?

@surajt97: yes, please!


Fixes the plugin to support pyQuil 2.16

This PR updates the PennyLane-Forest plugin to support breaking API changes in pyQuil 2.16. In particular:

The import pyquil.gate_matrices.QUANTUM_GATES no longer works, and has been changed to _matrix_dictionary = pyquil.simulation.matrices.QUANTUM_GATES.

pyquil.numpy_simulator has been changed to pyquil.NumpyWavefunctionSimulator

TomographyExperiment has been changed to Experiment.

There are still quite a few deprecation notices, specifically related to tomography:

FutureWarning: n_shots has been deprecated; if you want to set the number of shots for this run of measure_observables please provide the number to Program.wrap_in_numshots_loop() for the Quil program that you provide when creating your TomographyExperiment object. It looks like your TomographyExperiment object has shots = 1, so for now we will change that to 10000, which was the previous default value.

FutureWarning: active_reset has been deprecated; if you want to enable active qubit reset please provide a Quil program that has a RESET instruction in it when creating your TomographyExperiment object. For now, this value will override that in the TomographyExperiment, but eventually this keyword argument will be removed.

FutureWarning: symmetrize_readout has been deprecated; please provide the symmetrization level when creating your TomographyExperiment object. For now, this value will override that in the TomographyExperiment, but eventually this keyword argument will be removed.

@msohaibalam, I'm a bit unsure the best approach to fix these three (I assume the only high priority one is n_shots?). Furthermore, is active_reset deprecated only for tomography, or for generic QPU runs as well? We do not get a deprecation notice there.
Finally, there is an additional warning I have not noticed before:
UserWarning: SIMPLE-WARNING: Chip specification contained fidelity 1.0d0 > 0.99999d0. Truncating to 0.99999d0

I have just checked running on QPU and this also fixes the error we were seeing there. Thanks @josh146 !


cleanup

It is a simple clean up.
I have a new pragma based system on Stash to select the accessors and i remove the old one (method based)


bpmodify: Do not sort automatically

bpmodify automatically sorts touched list when it is originally sorted.
Even when a user doesn't want this behavior there's no way to avoid
sorting. For example, lists with version numbers.
Now bpmodify sorts the list only if -s is given.
Test: bpmodify_test.go
Change-Id: If2fe9bc871a463a47dd3c0b52982b34c9fde05f0

I updated the build actions in #333 but this seems to rerun the actions as they were defined in your commit, not after the merge.  Can you rebase your change?

Added -n option to sort numerically.
btw, I'm thinking this should be automatic, just like "wasSorted". We can examine if the list is numeric before sorting or checking sortedness. But there's a corner case like the list which is alphabetically sorted but not numerically.

Instead of adding a new option (-n), I changed the default sorting behavior to support numerical sort if all elements are numbers(technically, it can be converted using strconv.Atoi).

Rebased and changed the behavior to sort numerically. Can you please take a look?

@colincross please take a look.


【Hackathon 4th No.214】Arm 虚拟硬件上完成 PP-OCR 文本检测模型的部署与优化方案.

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

Hello 同学, 感谢你报名参加黑客松比赛并选择本题目。
你的RFC文档我正在reivew, 稍后反馈需要调整的问题哈～


Image library How does it work?

I'm sorry. I'm just learning. I need to know how it works.
I'm sorry for bothering you with such a simple question.
also sorry for my bad english.

Please ask questions like this on the golang-nuts mailing list.
Regarding your question, these two blog posts might help:
http://blog.golang.org/go-image-package
http://blog.golang.org/go-imagedraw-package


Make it possible for AbstractChannel subclasses to drive the write-loop

Motivation:
For asynchronous transports, we need to be able to drive the write-loop on our own schedule.
Modification:
Pull the write-loop out of WriteSink into a protected method on AbstractChannel, and offer methods to step and end the write-loop.
Result:
Asynchronous transports can now drive the write-loop forward in response to IO completions.

@chrisvest let me think a bit more about the API... I think we need to fix this for the reader and writer side. What I did with the Write and ReadSink was at the end not a good idea :/

Let us merge this one so we can make progress. I am not 100 % happy with the API but we can fix this once we have something that works.


fix(ux): add focus UI to the select dropdown

Summary

Add outline on dropdown selector when focused. This will make the element focus state obvious when using keyboard navigation
Always use a <button> as button for the dropdown selector, to ensure element is keyboard navigation friendly

How to test

Go to the explore page
Use the keyboard to navigate
When focused on the dropdown, it should have an outline
Go to http://localhost:8080/#/sn:0x07c251045154318a2376a3bb65be47d3c90df1740d8e35c9b9d943aa3f240e50/proposal/1/votes
Use the keyboard to navigate
The "CHOICE" element in the table header should be focusable

@wa0x6e needs rebase.


[FR] Ignore missing entities

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
If an entity is missing (e.g. device is temporarily not connected to HA due to maintenance), the card stops working because it fails to load that entity.
Only way of getting the card working again is to remove all entities that are currently unavailable to HA. That, of course, is a lot of work because usually they are children of at least one parent.
But the alternative right now is a broken card that of course throws errors and messes up the dashboard :(
Describe the solution you'd like
A clear and concise description of what you want to happen.
Ignore missing entities. This is also implemented in the energy dashboard, probably for the exact same reason.
If an entity is no longer available, HA Energy integration ignores that entity for periods where it is not available but shows it for past periods where it was available.
This allows the user to see old devices and their contribution even if they are no longer available now.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

It is really annoying when the chart is not rendering only because one socket is missing, because someone unplugged it

added in v3.6.0

I am still getting entity not found errors.


clientv3: document context to "Watch" API

Address https://github.com/coreos/etcd/issues/8993.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@014c375). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9016   +/-   ##
=========================================
  Coverage          ?   75.99%
=========================================
  Files             ?      359
  Lines             ?    29837
  Branches          ?        0
=========================================
  Hits              ?    22675
  Misses            ?     5585
  Partials          ?     1577

Impacted Files
Coverage Δ

clientv3/watch.go
95.29% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 014c375...1692ef1. Read the comment docs.

@xiang90 PTAL.

lgtm


The link to the fuel chain emission sector does not work.

The link to the fuel chain emission sector does not work.
https://beta-pro.energytransitionmodel.com/scenario/supply/fce/coal
It directs to user to the supply electricity section.
Have any changes been made to this sector recently? @marliekeverweij @antw ?

The FCE sidebar items has been set to dependent_on: _always_hidden; it is effectively disabled. Therefore the ETM redirects you to the first slide in the supply section.
The links in Supply → Fuel production should probably be taken out, and if FCE is being removed permanently, it might be best to delete the sidebar item, slides, and sliders from the interface YAML files.

The links in Supply → Fuel production should probably be taken out, and if FCE is being removed permanently, it might be best to delete the sidebar item, slides, and sliders from the interface YAML files.

I agree!
I opened this issue because we received a questions about this from a user.
If FCE is not active anymore we should remove references and documentation and (preferably) also code and information.
Renaming this issue to make a decision about this

Notifying
@jorisberkhout @AlexanderWirtz @michieldenhaan @MartLubben @noracato @redekok @ChaelKruip

Related issue: https://github.com/quintel/etmodel/issues/2986

Closed: https://github.com/quintel/etmodel/pull/3475


ARROW-8733: [Python][Dataset] Expose statistics of ParquetFileFragment::RowGroupInfo

Not a polished PR, just a quick try (in cython, since that's faster for me) to expose the RowGroupInfo statistics in Python + convert the expression into min/max information. More as food for discussion for now.
What this enables:
In [1]: import pyarrow.dataset as ds
   ...: dataset = ds.parquet_dataset("test_parquet_dask/_metadata", partitioning="hive")
   ...: fragment = list(dataset.get_fragments())[0]
   ...: rg = fragment.row_groups[0]

In [2]: ds.get_min_max_statistics(rg.statistics)
Out[2]:
[{'col': {'min': -1.43563008497128}},
 {'col': {'max': 1.2929964609736964}},
 {'index': {'min': 335}},
 {'index': {'max': 359}}]

Thanks for working on this @jorisvandenbossche !
This does seem like the functionality needed by Dask.  To test my understanding (and for the sake of discussion), I am imagining something (roughly) like the following in Dask to collect row-group statistics (note that I am using pyarrow-0.17.1 from conda, so the get_row_group_fragments call would be replaced):
from collections import defaultdict
import json
import pandas as pd
import pyarrow.dataset as ds
from string import ascii_lowercase as letters

path = "simple.pdf"
df0 = pd.DataFrame(
    {"x": range(26), "myindex": list(letters)}
).set_index("myindex")
df0.to_parquet(path, engine="pyarrow", row_group_size=10)
ds = pds.dataset(path)

# Need index_cols to be specified by user or encoded in
# the "pandas" metadata. Otherwise, we will not bother
# to infer an index column (and wont need statistics).
index_cols = json.loads(
    ds.schema.metadata[b"pandas"].decode("utf8")
)["index_columns"]
filter = None # Some user-defined filter

# Collect path and statistics for each row-group
metadata = defaultdict(list)
for file_frag in ds.get_fragments(filter=filter):
    for rg_frag in file_frag.get_row_group_fragments():
        for rg in rg_frag.row_groups:
            stats = ds.get_min_max_statistics(rg.statistics)
            metadata[rg_frag.path].append((<rg-index>, stats))

In this case, the resulting metadata object would be something like:
defaultdict(list,
            {'simple.pdf': [(0, <stats-for-rg-0>),
              (1, <stats-for-rg-1>),
              (2, <stats-for-rg-2>)]})

@rjzamora indeed something like that. I am not sure that you need to keep track of the path as well, unless maybe to have it working with existing functions to determine sorted columns out of this (but that's more something to discuss on the dask issue/PR)
Now, in the meantime, @bkietz did the more proper implementation -> https://github.com/apache/arrow/pull/7546


[11.x] Added transform function to \Illuminate\Database\Eloquent\Mode…

Added transform function in  \Illuminate\Database\Eloquent\Model class that works like the transform helper function
Eg.
$name = $user->transform("name", fn($name) => $name, "No Name");
$name = $user->transform("contactInfo->address", fn($address) => $address, "Address not filled");

You van do that already using mutators and accesors
https://laravel.com/docs/11.x/eloquent-mutators#accessors-and-mutators

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!


error about Swin & Vit

there is something wrong when i run usage_example:viT & swin
Vit:
it shows a problem：
File "vit_example.py", line 6, in
from pytorch_grad_cam import GradCAM,
ImportError: cannot import name 'FullGrad' from 'pytorch_grad_cam' (/opt/conda/envs/py38_torch1.7.1/lib/python3.8/site-packages/pytorch_grad_cam/init.py)
Swin:
it shows this:
Exception ignored in: <function BaseCAM.del at 0x7f5450fbc290>
Traceback (most recent call last):
File "/home/lsz/pytorch-grad-cam-master/pytorch_grad_cam/base_cam.py", line 191, in del
self.activations_and_grads.release()
AttributeError: 'ScoreCAM' object has no attribute 'activations_and_grads'
Traceback (most recent call last):
File "/home/lsz/pytorch-grad-cam-master/usage_examples/swinT_example.py", line 100, in
reshape_transform=reshape_transform)
TypeError: init() got an unexpected keyword argument 'target_layer'
Thanks~

Thanks, yes it should be target_layers. I pushed a fix to that
For the vit_example Fullgrad error - wasn't able to reproduce that.
Are you using the latest version?

I tried with "target_layers" and the code runs successfully.

your swin can run successfully？Please tell me your versions you are using

Thanks, yes it should be target_layers. I pushed a fix to that
For the vit_example Fullgrad error - wasn't able to reproduce that. Are you using the latest version?
I am using the latest version,and found that py37+torch1.8 can run vit well ,but py3.7+torch1.7 show me the error.
and for swin, it still show me the error about ' target_layers', i have seen the 'target_layer = model.layers[-1].blocks[-1].norm2' in the code yet.
it same as #158
thanks

Can you please retry with the latest code from this rep?
target_layer should be fully replaced in favor of target_layers


Updated to support Laravel 5.4

Not sure if this is backwards compatible though so may need to be a separate branch?

Hi @REBELinBLUE ,
I merged the PR in master and tagged a new v0.4 release. I checked the code, there shouldn't be BCs.
However, v0.3 is still available in a separated branch.
Thanks for your contribution!

Thanks :)


smartrply and calatalogos functions

Hi,
I have about 2 questions about whatsapp functionalities

it is possible to implement the functionality of 'smartreply' or implies a new feature that should be implemented to this library
https://faq.whatsapp.com/en/android/26000101/?category=5245246
The following query has to do with the new functionality that WhatsApp launched recently linked to 'catalogs'
https://faq.whatsapp.com/en/general/26000275/
Is it planned to add this functionality to this library if it is possible to implement it?

thank you

I have implemented 1st one in my repo https://github.com/vasani-arpit/WBOT and you can also configure it with simple json to achieve 2nd one. coding standards is not as rich as this repo but it does the job.


Add CloudCV organization and link to proposal

This PR --

[x] Add CloudCV as an interested participating organization in Google Season of Docs 2021.

@googlebot I signed it!

@emckean Please review and merge this PR.

hi, b/c of the merge conflict, I added your org manually. Please check the file to be sure it is correct. Thank you!

Hi @emckean, Thank you for helping me with it. I checked it and it looks good to me.


ConvexHull does not support masked arrays

Using masked arrays with scipy.spatial.ConvexHull() results in simplices that are masked in the array.  Example:
import scipy as sp
import numpy as np

X_m = np.ma.array(X,mask=m) # returns X array with mask defined by m
hull = sp.spatial.ConvexHull(X_m) # calculates convex hull on masked points

for simplex in hull.simplices:
      print X_m[simplex,1],X_m[simplex,0]

prints:
[-- -122.052021] [-- 37.324186]
[-- --] [-- --]
[-122.499217 -122.503319] [37.572598 37.595792]
[-122.263957 --] [37.831964 --]
If you remove the masked rows:
X_m = np.ma.compress_rows(X_m)

It prints the correct simplices:
[-121.471407 -121.464504] [ 38.408566  38.40513 ]
[-121.471407 -121.471454] [ 38.408566  38.408614]
[-121.464129 -121.371867] [ 38.429389  38.448629]
[-121.420056 -121.464504] [ 38.41     38.40513]

Is the issue fixed?

@souravsingh @pv
Yes, this appears to be fixed. Testing in the 0.19 dev branch with Delaunay, ConvexHull, and Voronoi raises ValueError: Input points cannot be a masked array when a masked array is provided for the points argument.
As an added bonus, because SphericalVoronoi passes through ConvexHull to generate the Delaunay triangulation on a sphere, it will also raise a ValueError when provided with masked input. Nikolai & I hadn't thought of adding this in though, so if the algorithm is ever modified in such a way that it no longer depends on the ConvexHull, the input checking will be lost -- so I guess that's a bit less robust, but it will likely be quite some time (if ever) before an algorithm that skips the convex hull is implemented.

Thanks, closing then.


Using LevelDB with cayley: Error, could not open! file does not exist

Configuration file:
$ cat cayley.cfg
{
"database": "leveldb",
"db_path": "/cayley_0.4.1_darwin_386/db/",
"read_only": false
}

Command:
./cayley repl -ignoredup -dbpath=/wikidata-simple-statements.nt -config=/cayley_0.4.1_darwin_386/cayley.cfg

Error:
E0726 22:56:50.286284 81645 quadstore.go:118] Error, could not open!  file does not exist
E0726 22:56:50.288093 81645 cayley.go:251] file does not exist

The config file takes priority over command line, I don't think the -dbpath switch will do anything in your case since the config file has db_path specified.
From the error, I assumed "/cayley_0.4.1_darwin_386/db/" has not been initialized. Was ./cayley init -config=/cayley_0.4.1_darwin_386/cayley.cfg executed prior to running cayley repl?

I had the same issue (for both bolt and leveldb), and running init resolved it. But it would be great if the error message was a bit more helpful! The leveldb message above doesn't give any clue about the issue, bolt just panics with the message:
panic: runtime error: invalid memory address or nil pointer dereference

I am having this exact same issue with Level, I don't have a config file created either just using command line parameters:
./cayley http --db="leveldb" --dbpath=/tmp/leveldb

Since e849da94022cc444921a11b1baabef13ebd7cb10 you need to init database first:
./cayley init --db="leveldb" --dbpath=/tmp/leveldb
or
./cayley http --db="leveldb" --dbpath=/tmp/leveldb --init
This behavior should be documented better. Ping @robertmeta.

I think this also needs to be fixed if command line doesn't override config, as I think that is unexpected behavior.

Hi, I am a newcomer and I met this error. It is better I think if you make clear at the beginning of your Quickstart guide that in order to play with any backend you must run these three commands in sequence (initialization, loading, backend), e.g.

./cayley init --db=bolt --dbpath=/tmp/test
./cayley load --db=bolt --dbpath=/tmp/test --quads=testdata.nq --alsologtostderr
./cayley repl --db=bolt --dbpath=/tmp/test

Just submitted a pull request that hopefully makes this a bit more clear. Although I thought it was already quite clear in the current docs. But maybe it was already changed since this ticket was created. It seems to be old.

@dennwc @jtorvald  can this be closed?


Return metric names to OptHistory

Earlier, histories stored the objective, which contained information about metrics used during the optimization. FEDOT.Web relied on this field to get metric names.
Thus, metric names should be placed in a separate named field of OptHistory.

Seems reasonable to have metric names in OptHistory,, so that it had self-sufficient information for understanding


Fix dependency for workflow

Switch node-formidable to install from the npm registry instead of their GitHub repo. This was holding up workflow checks on the npm ci step.

See #35

@matt8100 Should we try to get the CI tests to pass?

@harsimrat99 This PR simply resolves an issue preventing the workflow from even beginning the tests. It will need to be merged for future PRs to correctly check test results.

Thanks Matt. I will merge it.


[Device Specific] UI of whole application get disturbed & too much overlapping happens when user change font size from Settings

Frequency: 100%
Repro on build version:  Test flight build V1.17.0
Does not repro build version  Android internal build V1.17.0
Repro on devices: iPhone SE (14.2), iPhone 6+(12.4.5), iPhone 12 (14.4.1)
Does not repro devices Vivo V11 pro (10), Redmi Note 8 (9.0),Google Pixel 4a (11.0), Samsung Galaxy S4 (4.4.2)
Pre-condition: Make sure that user should change application font size to largest from Settings
Repro Steps:

Launch the application
Select Language
Observed that, Language name gets truncated on top RHS
Current Behavior:  Language name displayed as truncated at top RHS
Expected Behavior:  Language name should be displayed as it is with larger text
Impact: Bad user experience
Investigation: 1. Also it is observed that, UI get disturbed while creating , restoring account & Setting PIN
Also, UI get disturbed for Home screen & Hamburger Menu , Settings, etc
Attachment: UIDisturbedForLargestFont.wmv

We have observed same related below issue

Earn button truncated on hamburger menu when user increases text size EarnButton.PNG

We have verified issue on Testflight Build V:1.19.0 and observed that issue is still repro
Observation

We have observed UI disturbance issue throughout the app
Also we have observed text truncation is seen after clicking on learn more (CTA's) on timer page

Attachment
Repro on device : iPhone 6+ (12.5.4)

Hey @MuckT ,We have verified issue on Testflight Build V:1.19.0 and observed that issue is still repro and hence we are reopening this ticket
Observation:

We have observed UI disturbance issue throughout the app
- Also we have observed text truncation is seen after clicking on learn more (CTA's) on timer page

Attachment
Repro on device: iPhone 6+ (12.5.4)
Does not repro on device : iPhone 12 (14.7.1)
FYI : We are reducing the priority to P3 because it is replicating only in one device

https://valora.testquality.com requirement R89.

https://valora.testquality.com requirement R90.

https://valora.testquality.com requirement R91.

https://valora.testquality.com requirement R92.

https://valora.testquality.com requirement R93.

https://valora.testquality.com requirement R94.

https://valora.testquality.com requirement R95.

Hey @MuckT ,We have verified issue on Testflight Build V:1.19.0 and observed that issue is still repro and hence we are reopening this ticket
Observation:
We have observed UI disturbance issue throughout the app
Also we have observed text truncation is seen after clicking on learn more (CTA's) on timer page
Attachments :
Confirmationscreen_SE.jp
Repro on Devices: iPhone SE (13.6.1)
Createpin_SE.jpeg
Repro on device: iPhone SE (13.6.1)
hamburgerscreen_SE.jpeg
Repro on Devices: iPhone SE (13.6.1)
LearmoreCTA_SE.jpeg
Repro on Devices: iPhone SE (13.6.1)
phonenumerscreen_SE.jpeg
Repro on Devices: iPhone SE (13.6.1)

@ValoraQA can we breakout the following into their own tickets.

Confirmation Screen.
Learn More CTA.
Phone Number screen.

Hey @MuckT  we have created separate tickets for the components as instructed in the above comment
1] Phone number verification page : Link
2] Homepage : Link
3] CELO page : Link
Please let us know if anything required from our end.
Thanks..!

https://valora.testquality.com defect D897.


Removing untagged images can damage tagged images

Running action like so
      - name: Delete old untagged containers
        id: untagged
        uses: snok/container-retention-policy@96e897805acf21aa2ebc21fdf8e04c879e7daf9d # v2.0.1
        with:
          image-names: image
          account-type: org
          org-name: ${{ github.repository_owner }}
          cut-off: 2 weeks ago UTC
          untagged-only: true
          token: ***

Can damage tagged images:
$ docker pull ghcr.io/org/imag:tag
tag: Pulling from org/image
manifest unknown

GitHub Support response:

The way Docker publishes images changed recently. Instead of publishing images directly, it now publishes a hidden image alongside a manifest. You can see this below where it says OS / Arch (2). The (2) represents the hidden image and manifest.
This has the unfortunate consequence of breaking tools that detect untagged images (because the hidden image isn't tagged directly). When the hidden image is deleted, it leads to this manifest unknown error. Unfortunately there isn't an obvious fix for this.

provenance is enabled by default since docker/build-push-action@v4
My current workaround is to disable it (haven't tested if it actually helps yet)
 - uses: docker/build-push-action@v4
   with:
     provenance: false
     ...

Solution for already broken images: republish them

Do you propose we update the README of this action to notify users about this behaviour, or do you think this is something we can fix by changing the implementation?

Do you propose we update the README of this action to notify users about this behaviour, or do you think this is something we can fix by changing the implementation?

My understanding is there's no fix currently, but that's based on GH support answers, didn't verify it myself.
So I propose adding a warning in readme and pinning this issue for the time being.

I have confirmed that disabling provenance (provenance: false) fixes the issue. Updated issue description.
Workaround:
 - uses: docker/build-push-action@v4
   with:
     provenance: false
     ...

That's very helpful. Would you be interested in creating a PR too @lukasz-mitka?

No, sorry.

This can be fixed by explicitly excluding untagged images referred in manifests of tagged images similar to https://github.com/Chizkiyahu/delete-untagged-ghcr-action/blob/278ac5c5ae16914324ba447591af23312af6c075/clean_ghcr.py#L137-L138.

I fixed this in a new project. https://github.com/dataaxiom/ghcr-cleanup-action, without the providence workaround above. It requires uploading a temporary manifest to unlink the tag, then that can be deleted.

See https://github.com/snok/container-retention-policy/issues/43#issuecomment-2106346750 @rohanmars. Any reason you don't think that would work?

Yes skipping sha's is what I did essentially not to not delete the platform specific images linked from the manifest. It gets more complicated when you want to support multiple tags to the same multiarch image and when you want to actually delete the multiarch image. In the delete case you would want to include these digests.

For when you actually want to delete the multi-arch image, I would have thought that you'd be fine as long as you always make sure not to delete the SHAs associated with the current tag. Any old untagged images can then be deleted safely - assuming you're passing in the SHAs of all multi-arch images. Is there anything else that needs to be taken into account?


add Flop.bindings/2

resolves #165

Coverage increased (+0.09%) to 96.278% when pulling a83ab74031573ee6f388f7c2fa95ecfdf50e27ef on used-bindings into 9fc1603672db0f29edb77cc0319db5b3e485d14a on main.


Fix error when removing files

The following error happens for some users:
Error: ENOENT: no such file or directory, unlink '/opt/build/repo/public/assets/webfonts/fa-brands-400.eot'

It seems that when deleting files, those files sometimes do not exist. This makes fs.unlink() crash. This PR fixes this by checking if the file exists first.

Thank you!


Curl requests > 1023 characters fail owing to strings being pushed to multi line

When a curl request has more than 1023 characters, VBA inserts a new line into the string web_Curl within WebClient. This can cause the WebHelpers.ExecuteInShell(web_curl) function to fail as the curl request is incomplete on a single line.
This can be fixed by creating a multi-line curl command instead and allowing the request to go over multiple lines.

Making the following modification within the PrepareCurlRequest function does the trick
    ' Set headers and cookies     For Each web_KeyValue In request.Headers         web_Curl = web_Curl & "\" & Chr(10) & " -H '" & web_KeyValue("Key") & ": " & web_KeyValue("Value") & "'"     Next web_KeyValue

Cool, thanks for raising this issue @apatel-rms I'll make the fix soon


feat: adds enctype within the form component

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

5 of 6 (83.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 81.469%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Tempest/View/src/Components/Form.php
5
6
83.33%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.001%

Covered Lines:
5935

Relevant Lines:
7285

💛  - Coveralls

Thanks Yassine


Could not finish step 0 on macOS

Hello, I am facing an issue when trying to run recipes on my local device (macOS):
(nnsvs-dev) XXXXX:svs-world-conv sdercolin$ bash run.sh --stage 0 --stop-stage 0
stage 0: Data preparation
Convert musicxml to label files.
100%|███████████████████████████████████████████████████████████████████████████████████████| 51/51 [00:01<00:00, 46.24it/s]
Copy original label files.
100%|██████████████████████████████████████████████████████████████████████████████████████| 51/51 [00:00<00:00, 923.08it/s]
Round label files.
100%|█████████████████████████████████████████████████████████████████████████████████████| 51/51 [00:00<00:00, 1006.12it/s]
100%|██████████████████████████████████████████████████████████████████████████████████████| 51/51 [00:00<00:00, 541.17it/s]
100%|██████████████████████████████████████████████████████████████████████████████████████| 51/51 [00:00<00:00, 745.91it/s]
0it [00:00, ?it/s]end_time 65050000 of the phoneme ch and start_time 66900000 of the phoneme u is not the same. There seems to be a missing phoneme in sinsy_mono_round.
end_time 68750000 of the phoneme u and start_time 66900000 of the phoneme u is not the same. There seems to be a missing phoneme in sinsy_mono_round.
0it [00:02, ?it/s]
Traceback (most recent call last):
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/align_lab.py", line 57, in <module>
    lab_sinsy = fix_mono_lab_after_align(lab_sinsy, config["spk"])
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/util.py", line 213, in fix_mono_lab_after_align
    return _fix_mono_lab_after_align_natsume_singing(lab)
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/util.py", line 241, in _fix_mono_lab_after_align_natsume_singing
    f.append((f.end_times[-1], lab.end_times[i], lab.contexts[i]))
  File "/Users/sdercolin/dev/nnsvs-dev/lib/python3.8/site-packages/nnmnkwii/io/hts.py", line 159, in append
    raise ValueError(
ValueError: end_time (68750000) must be larger than start_time (68750000).
51it [00:00, 435.75it/s]
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/perf_segmentation.py", line 50, in <module>
    base_segments, start_indices, end_indices = segment_labels(
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/util.py", line 117, in segment_labels
    seg.append((s, e, l), strict)
  File "/Users/sdercolin/dev/nnsvs-dev/lib/python3.8/site-packages/nnmnkwii/io/hts.py", line 159, in append
    raise ValueError(
ValueError: end_time (68750000) must be larger than start_time (68750000).

I have succeed with everything on colab (https://colab.research.google.com/gist/taroushirani/3e54d01e9e85674dbb8eaa7e0e457acd/nnsvs_ofuton_p_utagoe_db_official_recipe.ipynb), and followed the same installation steps on macOS, except:

Installed everything in /usr/local/ instead of /usr/
Using python virtual env
Probably important: Did export ARCHFLAGS="-arch x86_64" before installing pysinsy, because without that I get a lot of unknown type name '__int64_t'. Ref: https://github.com/giampaolo/psutil/issues/1832#issuecomment-704596756

My Environment:

OS: macOS Catalina 10.15.7 (Intel)
Python: 3.8.2

Thanks in advance!

Logs for recipe ofuton_p_utagoe_db
(nnsvs-dev) XXXXX:svs-world-conv sdercolin$ bash run.sh --stage 0 --stop-stage 0
stage 0: Data preparation
Convert musicxml to label files.
100%|███████████████████████████████████████████████████████████████████████████████████████| 57/57 [00:00<00:00, 57.12it/s]
Copy original label files.
100%|█████████████████████████████████████████████████████████████████████████████████████| 57/57 [00:00<00:00, 1191.23it/s]
Round label files.
100%|█████████████████████████████████████████████████████████████████████████████████████| 57/57 [00:00<00:00, 1269.17it/s]
100%|█████████████████████████████████████████████████████████████████████████████████████| 57/57 [00:00<00:00, 1013.24it/s]
100%|█████████████████████████████████████████████████████████████████████████████████████| 57/57 [00:00<00:00, 1227.34it/s]
0it [00:00, ?it/s]akai_kutsu.lab 0.0
akatombo.lab 0.0
2it [00:00, 13.61it/s]amefuri.lab 0.0
antagata_dokosa.lab 0.0
4it [00:00, 11.71it/s]aogeba_tootoshi.lab 0.0
arupusu_itimanjaku.lab 0.0
6it [00:00,  7.72it/s]chatsumi.lab 1.0
chouchou.lab 0.0
8it [00:00,  8.35it/s]daikokusama.lab 0.0
9it [00:01,  7.63it/s]donguri_korokoro.lab 0.0
furusato.lab 0.0
11it [00:01,  8.10it/s]genkotsuyamano_tanukisan.lab 0.0
haruga_kita.lab 0.0
haruno_ogawa.lab 0.0
14it [00:01, 10.09it/s]haruyo_koi.lab 0.0
hato.lab 0.0
16it [00:01, 11.80it/s]hato_low.lab 0.0
hiraita_hiraita.lab 0.0
18it [00:01, 12.95it/s]hotaruno_hikari.lab 0.0
issunboushi.lab 0.0
20it [00:02,  9.76it/s]kagome_kagome.lab 0.0
kagome_kagome_low.lab 0.0
kamomeno_suiheisan.lab 0.0
23it [00:02, 11.02it/s]katatsumuri.lab 0.0
kintaro.lab 0.0
25it [00:02, 12.03it/s]koganemushi.lab 0.0
koinobori.lab 0.0
momiji.lab 0.0
28it [00:02, 12.11it/s]momotarou.lab 0.0
morobito_kozorite.lab 0.0
30it [00:02,  9.39it/s]mushino_koe.lab 0.0
my_grandfathers_clock_1_1.lab 0.0
32it [00:03,  8.73it/s]my_grandfathers_clock_1_2.lab 0.0
my_grandfathers_clock_1_3.lab 0.0
34it [00:03,  8.82it/s]my_grandfathers_clock_2_1.lab 0.0
my_grandfathers_clock_2_2.lab 0.0
36it [00:03,  9.29it/s]my_grandfathers_clock_2_3.lab 0.0
37it [00:03,  9.24it/s]my_grandfathers_clock_3_1.lab 0.0
38it [00:03,  9.24it/s]my_grandfathers_clock_3_2.lab 0.0
39it [00:04,  9.04it/s]my_grandfathers_clock_3_3.lab 0.0
40it [00:04,  8.76it/s]nanatsunoko.lab 0.0
oborodukiyo.lab 0.0
42it [00:04,  9.44it/s]sakura_sakura.lab 0.0
seikurabe.lab 0.0
44it [00:04,  8.59it/s]shabondama.lab 0.0
shoujoujino_tanukibayashi.lab 0.0
46it [00:04,  9.24it/s]tetsudou_shouka.lab 0.0
47it [00:04,  8.33it/s]tombi.lab 0.0
48it [00:05,  8.55it/s]tooryanse.lab 0.0
urashima_taro.lab 0.0
50it [00:05,  8.19it/s]usagi.lab 0.0
usagito_kame.lab 0.0
52it [00:05,  9.22it/s]warewa_uminoko.lab 0.0
53it [00:05,  8.16it/s]yuki.lab 0.0
54it [00:05,  8.50it/s]yurikagono_uta.lab 0.0
zuizuizukkorobashi.lab 0.0
56it [00:05, 10.06it/s]zuizuizukkorobashi_low.lab 0.0
57it [00:05,  9.54it/s]
57it [00:00, 697.88it/s]
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/perf_segmentation.py", line 50, in <module>
    base_segments, start_indices, end_indices = segment_labels(
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/util.py", line 117, in segment_labels
    seg.append((s, e, l), strict)
  File "/Users/sdercolin/dev/nnsvs-dev/lib/python3.8/site-packages/nnmnkwii/io/hts.py", line 163, in append
    raise ValueError(
ValueError: start_time (197850000) must be equal to the last end_time (191750000).
Prepare data for time-lag models
  0%|                                                                                | 0/57 [00:00<?, ?it/s]akai_kutsu: Global offset (in sec): -0.024999999999999998
  0%|                                                                                | 0/57 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "/Users/sdercolin/dev/nnsvs-dev/nnsvs/egs/_common/no2/utils/finalize_lab.py", line 89, in <module>
    assert seg_idx > 0 or exists(lab_align_path)
AssertionError

Hello.  This looks like fastdtw-related problem for me.
The recipes of ofuton_p_utagoe_db, natsume_singing, oniku_kurumi_utagoe_db make use of fastdtw to guarantee the consistency between mono-phone label files included in singing voice databases and full-context label files converted from musicxml.   There are two implementation of fastdtw, c++ version and pure python version, and their output differs with a specific input.
For detail, please see https://github.com/slaypni/fastdtw/issues/46.

Please try the following code.
import numpy as np
from fastdtw import fastdtw

x = [1, 6, 6, 33, 35]
y = [1, 6, 6, 33, 35]

d, path = fastdtw(x, y, radius=len(x))

print(f"d: {d}")
print(f"path: {path}")

The output of pure python version will be:
d: 0.0
path: [(0, 0), (1, 1), (1, 2), (2, 2), (3, 3), (4, 4)]

The output of c++ version will be:
d: 0.0
path: [(0, 0), (1, 1), (2, 2), (3, 3), (4, 4)]

If you installed fastdtw with pip, installation from source may help you.

I tried to build fastdtw from source code, even edited setup.py to ensure Cython is used, but the result still contains (1,2).
Finally I changed my Python to 3.7.3, and reinstalled everything, and it works.... don't know why though..
Thanks for your help!!

I had same error, and I solved this error using past version of fastdtw.
pip install fastdtw==0.3.2

I have the same issue, installing from source, downgrading to 3.7.3, using fastdtw 0.3.2, none of them work for me. Please help.

I think there is something wrong with the compilation from the source cde.   Could you please execute the following commands and paste the content of build.log here?
git clone https://github.com/slaypni/fastdtw.git
cd fastdtw && python setup.py build --verbose  > build.log 2>&1

The other day I met this problem too.  "tp_print" is removed from python 3.9 and re-cythonizing was needed at that time.

@taroushirani Okay! It works now! Thank you very much! 😊


CSS changes

Great skin! I may be able to contribute more if time allows.

Thank you so much for the kind words and your hard work here! I have some bad news, though: the CSS files in dist are generated, so changes to them get overwritten by the build process (gulp build). If you can stomach editing the SASS in src I'd be happy to merge that PR :)

Apologies. I completely missed the SASS files and gulpfile.coffee. I'm getting some errors setting up gulp on my end but the latest commit should suffice while I work that out. Thanks!


display annotation extensions for protein features

now we are displaying residues, can we also display annotaton extensions for protein features? (I might have requested this before....seems like quite a straight forward thing to do?)

I'll give it a go and see if it works and looks OK.

Do we have any protein feature annotations with extensions?  Can you give me an example?

cdc2
On Thu, Jul 25, 2019 at 8:59 AM Kim Rutherford <notifications<EMAIL_ADDRESS>wrote:
Do we have any protein feature annotations with extensions? Can you give me an example?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pombase/website/issues/1377?email_source=notifications&email_token=ADBDJUQXYLOS45ZLR6NTYIDQBFMNHA5CNFSM4IGXZ2Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2YV3XI#issuecomment-514940381, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADBDJUURAR77QPQU27NLL3DQBFMNHANCNFSM4IGXZ2QQ.
--
Antonia Lock, PhD
PomBase Biocurator, http://www.pombase.org
Department of Genetics, Evolution and Environment,
The Darwin Building,
University College London
London WC1E 6BT, UK

cdc2

I can't see any protein feature annotations in the contig file or in Chado for cdc2.  Could you help me out?

ah, I'm on about the protein modification section, not features
On Thu, Jul 25, 2019 at 10:08 AM Kim Rutherford <notifications<EMAIL_ADDRESS>wrote:
cdc2
I can't see any protein feature annotations in the contig file or in Chado for cdc2. Could you help me out?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pombase/website/issues/1377?email_source=notifications&email_token=ADBDJURBM6X34VUVZDIAU6TQBFUPBA5CNFSM4IGXZ2Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2Y3SCQ#issuecomment-514963722, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADBDJUVNIHF5OUDVY5QPSNDQBFUPBANCNFSM4IGXZ2QQ.
--
Antonia Lock, PhD
PomBase Biocurator, http://www.pombase.org
Department of Genetics, Evolution and Environment,
The Darwin Building,
University College London
London WC1E 6BT, UK

ah, I'm on about the protein modification section, not features

Now I'm confused.  We do show the extensions for modifications?  Some of them are hidden in the summary:

yes, so I would like them to be shown in the  summary

like they are in other sections e.g. GO annotations

discussed previously in #180 and #861 (and #943 but that one was just a display bug)

yes so I'd like to see extensions, with the modified residues
(e.g.
T38 added by yfg1
T42 added by yfg1, removed during G1

yes so I'd like to see extensions, with the modified residues

Is that in the summary or just when you show details?

T38 added by yfg1

So remove the residue column and put the residue before the extension?  Will that look OK when there isn't an extension?  It would look like:
 MOD:00047    O-phospho-L-threonine
                   T14     IDA     Kettenbach AN et al. (2015)

yes in summary view.
personally I'd prefer to use the prefix "modified residue" instead of having a separate col
eg
MOD:00047    O-phospho-L-threonine
modified residue  T14     IDA     Kettenbach AN et al. (2015)

Would you like the residue displayed like that in the detailed view too?  So we could remove the Residue column?

I think this would be a nice change.

I'm not too fussed about the detailed view.....although I think it would be nice to have it looking more consistent with the other views, perhaps it's not worth spending too long on it since I guess it will be overhauled eventually when we get around to implementing a  protein sequence schmanzy viewer
I would like the extension shown in the summary view though because it's really useful information and I think it is too hidden at the moment

I'm not too fussed about the detailed view.....although I think it would be nice to have
it looking more consistent with the other views, perhaps it's not worth spending too long on

I turned out to be easier to change the summary if I changed the detailed view to match.
Does this look OK?  (SPCC4E9.02)

Much better!

spot on imo, thanks!

guess I shouldn't close until live

It's all set for tonight's update.  I'll check in the morning.

That's live now.  Please open new issues if you spot any problems.


Updated signed build to 1.5 as well.

Forcing the merge, as required to restore signed build.

Forcing the merge, as required to restore signed build.


rtp pt epg guide isnt working?

Description
https://iptv-org.github.io/epg/guides/pt/rtp.pt.xml is no longer working. Remove ?

The EPG had issues for 2 days for but it is working now.


How do we install packages like troposphere[policy]?

Hey @nikolay, you can specify this in the format:
troposphere = { extras = ['policy'] }

Great! If possible, add this to your examples!

@nateprewitt Sorry, but what if I want to specify a version?

For a version, you can specify additional values inside the table.
troposphere = { extras = ['policy'], version = '1.5.6' }
You can find more information on the Pipfile syntax in the official Pipfile repository.