Add dead link tally script

Description

A script we used today to query for the number of dead links in the link validation cache.
I will fill in more details later.
Checklist

[ ] My pull request has a descriptive title (not a vague title like Update index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@WordPress/openverse-api what do we think about this? Does it need to live in the API, and if so do we like the root-level scripts directory? Or should we move it into the WordPress/openverse repository?

Closed in favor of https://github.com/WordPress/openverse/pull/301


ancient upgrade :interactive is updating the wrong dependencies in project.clj

I tried upgrade :interactive like so on my project.clj:
[org.clojure/data.xml "0.0.8"] is available but we use "0.0.6"
Do you want to upgrade? [yes/no] yes

[org.clojure/core.incubator "0.1.3"] is available but we use "0.1.2"
Do you want to upgrade? [yes/no] yes

[hiccup "1.0.5"] is available but we use "1.0.1"
Do you want to upgrade? [yes/no] yes

[hickory "0.5.4"] is available but we use "0.2.0"
Do you want to upgrade? [yes/no] yes

[seesaw "1.4.4"] is available but we use "1.4.2"
Do you want to upgrade? [yes/no] yes

[org.cloudhoist/stevedore "0.8.0-alpha.1"] is available but we use "0.7.2"
Do you want to upgrade? [yes/no] no

[org.cloudhoist/pallet "0.8.0-alpha.9"] is available but we use "0.7.2"
Do you want to upgrade? [yes/no] no

[hiccup-bridge "1.0.1"] is available but we use "1.0.0-SNAPSHOT"
Do you want to upgrade? [yes/no] yes

[prismatic/plumbing "0.3.5"] is available but we use "0.3.3"
Do you want to upgrade? [yes/no] yes

For some of the dependencies it failed to update; for others it swapped version numbers.  Here are some results in the project.clj file:
[org.clojure/core.incubator "0.1.2"]  -- failed to upgrade
[org.cloudhoist/pallet "1.0.1"]  -- swapped with the new hiccup-bridge version number evidently
[hiccup-bridge "0.3.5"] -- swapped with the new prismatic plumbing version number evidently
[prismatic/plumbing "0.3.3"] -- failed to upgrade

The mangled project.clj is below:
(defproject my-project/my-project "0.1.0-SNAPSHOT"
  :license {:name "Eclipse Public License",
            :url "http://www.eclipse.org/legal/epl-v10.html"}
  :dependencies [[org.clojure/clojure "1.6.0"]
                 [org.clojure/data.xml "0.0.8"]
                 [org.clojure/core.incubator "0.1.3"]
                 [clj-tagsoup "0.3.0"]
                 [hiccup "1.0.5"]
                 [hickory "0.5.4"]
                 [seesaw "1.4.4"]
                 [org.clojure/core.incubator "0.1.2"]
                 [org.cloudhoist/stevedore "0.7.2"]
                 [org.cloudhoist/pallet "1.0.1"]
                 [hiccup-bridge "0.3.5"]
                 [prismatic/plumbing "0.3.3"]
                 [utils "1.0.0-SNAPSHOT"]]
  :url "http://example.com/FIXME"
  :repositories {"sonatype"
                 "http://oss.sonatype.org/content/repositories/releases"}
  :min-lein-version "2.0.0"
  :description "FIXME: write description")

This should be fixed. Can you try the latest SNAPSHOT?

This worked great even with the duplicated dependency (I checked out the project as it was when this bug showed up).
(I had to use the :no-tests option as I only had the default testing, which has an intentionally failing test.  The error message from that btw was as follows:
running test task ["test"] ...
(error) tests have failed (use ':no-tests' to disable testing).
(error) #)

The issue with tests should be fixed now. Thanks for noticing!

Thanks for fixing it!
On Tue, Jan 13, 2015 at 1:32 PM, Yannick Scherer<EMAIL_ADDRESS>wrote:

Closed #41 https://github.com/xsc/lein-ancient/issues/41.
—
Reply to this email directly or view it on GitHub
https://github.com/xsc/lein-ancient/issues/41#event-218383118.


Add z-index scale

Fixed as of https://github.com/Impria/ghast/commit/5050545bf999087e714dc97954396757009098c3


[WIP]Trying to apply shelveset on current commit when parent changeset is not...

... found
Need to use the --force option

A really quick fix for #676.
@nogains-nopains @gerich-home if you could test that (because I don't have a test case myself...)

:metal: Looking forward to hearing if it works... :wink:

yes, it is working for me. Thanks  a lot!


Fix TimeResponseData.to_pandas() to handle zero state case

Fixes issue #957.

coverage: 94.991% (+0.002%) from 94.989%
when pulling 6a8779c1bb01da099dcefac5001b74c7ba428842 on murrayrm:to_pandas_bug-03Jan2024
into b389b64749aeae32cf82370f71be08785161cd1f on python-control:main.

LGTM


Basic data type aliases

I'd argue it would greatly improve FTUE if aliases would be added, namely:

int and integer for number
list for array

Currently, the primitive types are aligned with types supported by JSON Schema and JSON in general. Moreover, number is basically a superset of integer and float types as we know them from other languages. I'm afraid from my POV consistency and

There should be one-- and preferably only one --obvious way to do it.

wins the tradeoff over FTUE.

I'd argue that using TLAs or LFLAs saves time to OPs but not readers. :tongue:

Either way, I would keep it simple without aliases. Maybe parser can warn tho. Eg. "int – have you mean integer" (note that we do not have integer :)

I would keep it simple without aliases.

:+1:

For the time we will keep MSON without aliases.


Validate SLES 15 SP6

K3s Versions to be Validated

v1.30.2+k3s1
v1.29.6+k3s1
v1.28.11+k3s1
v1.27.15+k3s1
Testing Considerations

Install and run sonobuoy conformance tests on a hardened cluster
Validate SUC upgrade
Install Rancher Manager
Validate snapshot restore via cluster-reset-restore-path

Additional Information

Validations have been completed.


Clicking on 'Resources' nav button will now collapse the explorer if …

…already active.
=========
Fix for #826
Just deleted case logic and moved above case 2: which performs the same logic for Bot / Resources / Notifications sections

Pull Request Test Coverage Report for Build 314

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.545%

Totals

Change from base Build 305:
0.0%

Covered Lines:
3549

Relevant Lines:
5941

💛  - Coveralls


Changing a users tag color causes the app to crash

Steps to reproduce:

Got to any user profile.
Tag User
Change color of tag.
From the users profile select any comment they’ve posted.
Device: iPhone X iOS 12 developer beta 12.

Good find, fixed for 1.2.1!


Windows [TP4] localhost mitigation for 2-second perf regression

Signed-off-by: John Howard<EMAIL_ADDRESS>OK, this is going to require quite some explanation!! Our server perf team reported a significant regression with the latest build of docker. Myself and @swernli narrowed it down to either a bug in GOLang 1.5.1 and/or a bug in the Windows networking stack in TP4. It is only evident after upgrading from 1.4.3 to 1.5.1. There is a huge comment in the PR which explains what the mitigation is in far more detail. I have deliberately not changed the behaviour of any non-Windows client to avoid any possible other side effects (slim though that might be).
The 2-second delay is because we hit this twice during docker run.

@jhowardmsft wow, did you fill bug for golang?

@LK4D4 Not yet. More investigation still needed - it's not clear if it's golang or Windows yet (or both)

For your entertainment: Redacted, and not the cause although we thought it might have been. Can you believe someone has a host called "localhost" on the Microsoft corporate network?!?!?!?

:sob: LGTM


Typo in 'Naeva Meetup (Evidence)' Datapad

In the datapad, 'Naeva Meetup (Evidence)', there is a typo in Naeva's line beginning "Two things, Alder. And I'm going to make this..." At the start of the line, Adler is incorrectly typed as 'Alder'.

Details
There is a typo in the subtitles of the first datapad collected as part of the SysDef questline, called 'Naeva Meetup (Evidence)' and collected behind Adler in the Broken Spear. At the start of Naeva's line beginning "Two things, Alder. And I'm going to make this..." Adler is incorrectly typed as 'Alder'.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Play game, listen to datapad with subtitles on

Fixed for the next release.


PHP 7.4

Closes #162

@sean-e-dietrich there is a related ticket https://github.com/docksal/docksal/issues/1245.
Makes sense to include it here

@lmakarov @achekulaev made some adjustments to the tests and then commented out the drush 8 commands.


Data Science with Python : Dealing with Categorical Columns

Welcome to 'DSWP' Team, good to see you here
This issue will helps readers in giving all the guidance that one needs to learn about working on Dealing with Categorical Columns. Tutorial to Dealing with Categorical Columns and how it's applied using sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Peehu Saxena / Shivang Dubey
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

Hello, I am Deepthi M with serial number:172, Batch-2. I would like to do documentation on this issue.

Approach: A detailed explanation.

Hello, I am M. Ashish Reddy. I would like to do a VIDEO on this issue.
Batch number: 5
serial number:174.

This is created on the request of a contributor. If it's not claimed by evening, it'll will be assigned on queue. There are many more issues coming up. @ashish-reddy-20-08 @deepthi1107

ok sure.

Ok mam, Looking for new issues.
Thank you @prathimacode-hub

New issues are created. Check on to them. @ashish-reddy-20-08 @deepthi1107

Name: Ayushi Shrivastava
S No: 811
Approach to be followed:

Documentation
Categorical columns might hamper the efficiency of a ML model so to deal with it we can convert categorical columns into numerical and enhance the efficiency of a ML model using a example dataset with proper code and explanation.

Issue assigned to @ayushi424 for documentation

@prathimacode-hub, Mam can i work on the video part for this

Yes, issue assigned to @ashish-reddy-20-08 for documentation

@prathimacode-hub , Can you evaluate the PR mam
Link of PR: https://github.com/girlscript/winter-of-contributing/pull/2906
Name: M. Ashish Reddy
Batch - 5

@prathimacode-hub , Can you evaluate the PR mam
Link of PR: #2906
Name: M. Ashish Reddy
Batch - 5

@prathimacode-hub I would like to work on this issue
Name: Tavneet Kaur Gill
Serial No. 323
Type: Audio
Approach: Explanation how it works, what it means, how to incorporate in different topics of data science

Name: Saloni Thete
Sr.No: 821
Type: Documentation
Approach: Solutions and their detailed explanation to dealing with categorical columns along with resources and examples.
@prathimacode-hub Kindly assign :)

Issue assigned to @tavneetgill for audio contribution
@SaloniThete, for documentation, audio and video it's already assigned. You can try for some other issue.

Sure,Thanks :)


Proof of concept--using Kotlin in Ion Java

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Marking as "ready for review" so that it will run all of the workflows. Not actually ready for review yet.


vue-tsc errors when importing components

Describe the bug
In my project (see https://github.com/it-at-m/refarch-templates/tree/feat/5-add-webcomponent-template) I get errors when trying to import a Web Component in a TypeScript based Vue SFC.
The error is reported by the tool vue-tsc.
I am not 100% sure if its a issue by muc-patternlab-vue or my specific setup of dependencies and configuration files used in the project
To Reproduce
Steps to reproduce the behavior:

Checkout https://github.com/it-at-m/refarch-templates/tree/feat/5-add-webcomponent-template
Open refarch-hello-world-webcomponent.ce.vue
Error appears in IDE and when executing vue-tsc

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Additional context
I am using the following dependencies. This might help in replicating the issue:
  "dependencies": {
    "@muenchen/muc-patternlab-vue": "1.13.0",
    "@vueuse/core": "^10.9.0",
    "vue": "^3.4.21"
  },
  "devDependencies": {
    "@muenchen/prettier-codeformat": "^1.0.1",
    "@rushstack/eslint-patch": "^1.10.2",
    "@tsconfig/node20": "^20.1.4",
    "@types/node": "^20.0.0",
    "@typescript-eslint/eslint-plugin": "^7.6.0",
    "@typescript-eslint/parser": "^7.6.0",
    "@unplugin-vue-ce/sub-style": "^1.0.0-beta.21",
    "@vitejs/plugin-vue": "^5.0.4",
    "@vue/eslint-config-prettier": "^9.0.0",
    "@vue/eslint-config-typescript": "^13.0.0",
    "@vue/test-utils": "^2.4.1",
    "@vue/tsconfig": "^0.5.1",
    "eslint": "^8.57.0",
    "eslint-plugin-vue": "^9.24.1",
    "prettier": "^3.2.5",
    "typescript": "^5.4.5",
    "vite": "^5.2.8",
    "vitest": "^2.0.0",
    "vite-plugin-css-injected-by-js": "^3.5.0",
    "vue-tsc": "^2.0.12"
  }

I got Node 20.14.0 installed on my machine.

There seems to have been a change in how the library was getting published. Previously it only uploaded the contents of the dist-Folder to npm, but currently it uploads everything.
So at the moment a quick fix would be to change the import-Statement from e.g.
import {MucCard} from "@muenchen/muc-patternlab-vue";

to
import {MucCard} from "@muenchen/muc-patternlab-vue/dist";

I'm now trying to find out why it changed the publishing behaviour.

This should be fixed with https://github.com/it-at-m/muc-patternlab-vue/commit/58b7cd367fd33c47c6b7ce5189bebad3e72be9c9
Can you please try out beta-Version 1.13.0-beta.8 and report back if this fixes your problem?

Unfortunately it still does not work with the version above. I still get the same exact error
src/refarch-hello-world-webcomponent.ce.vue:19:28 - error TS2307: Cannot find module '@muenchen/muc-patternlab-vue' or its corresponding type declarations.

19 import { MucCallout } from "@muenchen/muc-patternlab-vue";

by vue-tsc and the IDE still underlines the import as invalid.

It works now. :) Thank you very much. We can close this issue. ;)


My reply posts (Fedi to Bsky) show with something deleted

Bluesky users have pointed out that my replies from Mastodon show something deleted. There isn't anything extra below my posts and the info icon isn't clickable on Bluesky so wondering if this is some kind of bridge issue?
PS: This only occurs on replies to Bsky accounts.
Kind regards

Oops, yes! Looks like this is a bug caused by #1411. Thanks for reporting! Will fix.

Oops, yes! Looks like this is a bug caused by #1411. Thanks for reporting! Will fix.

Many thanks. Always appreciate the time and effort you put into this. Best wishes.

Should be fixed as of last night, thanks again for reporting!


want res.sendError(error, noLog) new "noLog" boolean arg

I like the logging of error responses:
2011-11-24 18:35:00Z WARN: ErrorSent(requestId=095b86af-d734-4ee5-99bf-9156f4eedc92): HttpCode=500, RestCode=InternalError, message=error determining account for 'admin':
InternalError: error determining account for 'admin'
    at EventEmitter.<anonymous> (/Users/trentm/widget.js:292:33)
    at EventEmitter.emit (events.js:64:17)
    ...

and I want to use res.sendError for all my error responses for the common response formatting:
{
  "code": "ResourceNotFound",
  "message": "no such login: 'adminasdf'"
}

However, I'd like to NOT bother logging the response and stack for some error responses, e.g. a simple 404 for some cases. Would you be amenable to a "noLog" boolean to skip logging on a one-off? If so, I can impl. it.

Fine with me!

Will be in<EMAIL_ADDRESS>


Fix blob_get() would timeout. Issue #2070

Tentative fixing. The node is primed with some fixed peers before triggering the blob download.
Please check comments on issue #2070

@cristi-zz nice to see your face here - we've come full circle since, huh!
Thanks for the PR and congrats on the first one, we'll get this reviewed soon.
Can we show you some appreciation?

@tzarebczan thank you for the appreciation!
This bug was on my mind for so long! However the fix is ugly (duplicate code, etc), some guidance is needed to proper fix this. My guess is that peers must be added to the Node object before interacting with Kademlia, and this "priming" should happen at the startup of the library and not be rebooted each time an API call finishes. Now, the peers are added in some API handling routines (download a stream with get ) but not in others (like blob_get). Don't know what is the design intention so I can't dig further.

Ready for review. All tests (int+unit) are green on local machine.

Coverage remained the same at 67.97% when pulling fefd8f49314e1c0979ed9fe05b96418bd64daeec on cristi-zz:fix-blob_get-timeout into 268decd6556619625151c574890eba9f8dc2c90d on lbryio:master.

@jackrobison please review this issue. it's been open for over a month

Hi! Thank you for looking into this! This bug, is bugging me for a while, now.
Overall intention:
Make RPC call blob_get connect to some peers and get the actual blobs. Imho there were some refactoring on the way the DHT operations are handled and this particular method was left out. Fixing it directly would result in duplicate code (true, 2-3 lines only but still, a new source of grief). So I tried to extract the common behavior so future changes in the semantics of DHT will not break these calls.
Mechanics:
In the original code the _delayed_add_fixed_peers was a regular method that was called after fixed_peers_delay. Also, the handle returned by call_later is used to cancel this task [why the delay? why the need to cancel? Not important right now, maybe]. I tried to reproduce the exact behavior , constrained by the facts:
(1) the _delayed_add_fixed_peers must call an async code (so it must be also made async. The call_later does not take a coroutine) and (2) not being an expert on asyncio.
Reading loop.call_later docs I assume that if the task is cancelled before the delay, no exception is thrown. I THINK I tried to mimic the same behavior in prime_node_with_peer_addresses by catching only the Cancelled exception. Looking back at the code, maybe is the wrong place to catch it? Where this exception would propagate? To add_fixed_peers? Is there a more pythonic way to add peers after a delay while being able to cancel the whole operation?
Suggestions are more than welcome wrt to this issue!

Hi! @jackrobison !
Refactored the offending line [see comments on commit] and removed the try/catching the Cancel exception. Also don't know if you get notified at comments or only when "resolve conversation" is pressed. So sorry if I spammed you with tags.
Cheers!

Hm, it worked on my github instance, Maybe retrigger the CI pipeline?
https://github.com/cristi-zz/lbry-sdk/actions/runs/1420543979


Cannot navigate to Controller method from routes

I cannot navigate to controller methods by ctrl+click on the name of controller if controller put inside a folder and modify the namespace in the Routeservice provider.

Same here @karunais13

Same problem.
It's tough。

I resolve this problem success, my ide is phpstorm. I install laravel plugin in Ide's plugins market. then restart ide, the navigation work!!!

I don't see how this technically can be solved?
Wouldn't this mean, for the navigation part, it needs to be supported directly from the IDE, e.g. a plugin like https://plugins.jetbrains.com/plugin/7532-laravel


Discover broken links when building absolute links to documentation sources

What happened:
During Docforge replication of documentation nodes, there are cases where nodes could have references to documentation files (markdowns) that are outside of the scope of the particular build. Such references are transformed to absolute links (depending on the ResourceHandler responsible for the source node) or they are already such links. Sometimes those links are broken and we can add a check if they do actually exist in order to inform the user it might be broken.
What you expected to happen:
To be notified that an absolute link inside the replicated documentation points to a nonexisting source.

/assign


See improved Project Overview page

I don't think data collectors should see forms that don't have a published version. This is probably a separate issue to file and doesn't seem urgent to me but I do want to mention it.

It's not hard to implement, so I'll go ahead and do that!

@lognaturel, this should be ready for a second round of review.

Updates look great to me! We may also want to hide the Latest Submission column from the projects for data collectors but I don't think it's as distracting/misleading.

This might be a little complicated, because Data Collector is not a sitewide role: we'd have to check each project for whether the user is a Data Collector on the project (or something along those lines).

This might be a little complicated, because Data Collector is not a sitewide role

I see. It's going to be very interesting to learn more about how folks segment their work and users across projects.


Add a shebang to the bundled file?

I did expect I could just execute the downloaded javascript file with bun installed on my system, but since there is no shebang, the system doesn't know how to execute it. I would suggest adding the following shebang to the top of the file:
#!/usr/bin/env bun

Cheers for the suggestion. I'll add that in a build step.

Added in 0.26


disabled comments in list_transcript

Hello all,
I have seen a lot of problems because of disabled comments. I ran into the same issue when I wrote a loop that checks for transcript availability. If there is a video with comments disabled, my loop stops. But I want to be able to program that it tells me that there are no subtitles without stopping the loop. Because it raises the special class and tells me the reason why I can't get the list, I cant grab this one with an exception and an error would be easier for me to handle.
So is there any way to catch the warning in an exception, or would it make sense to program an error that can be caught with an exception?
Many thanks in advance,
Aaron
Translated with www.DeepL.com/Translator (free version)

Hi @AaronPhilipp,
I am not 100% sure what you are trying to achieve and what your distinction between exception/error is. Generally speaking, you should be able to catch any exceptions thrown by this module using try/except and ignore them if you don't want the program to stop.
Is this what you are looking for? If not, please post the code you are trying to run.

I am not sure what you mean. This module doesn't provide any functionality that something anything to do with comments 🤔
Please share some code. I can't help you without knowing what you are doing 😄

I mean captions of course. Sorry. Long day.
Basically I have his function. I will
`def get_caption(video_id):
print(video_id)
df = pd.DataFrame(columns=['video_id', 'subtitle'])

transcript_list = YouTubeTranscriptApi.list_transcripts(video_id=video_id)

transcript = transcript_list.find_generated_transcript(['de', 'en'])

if transcript == 'de ("Deutsch (automatisch erzeugt)")[TRANSLATABLE]':

    result = transcript.fetch()

    text = ''

    for i in result:
        text += i['text'] + ' '

elif 'en ("Englisch (automatisch erzeugt)")[TRANSLATABLE]':

    translated_transcript = transcript.translate('de')
    result = translated_transcript.fetch()

    text = ''

    for i in result:
        text += i['text'] + ' '

df = pd.concat([df, pd.DataFrame([{'video_id': video_id, 'subtitle': text}])])

time.sleep(randint(2, 7))  # short break after request

return df`

The I have a list with video_ids and pass this list to a loop:
for i in video_ids_list: tmp = df = get_caption(video_id=i)     print(i + ': ok' + ' (' + datetime.datetime.now().strftime("%H:%M:%S") + ')')     if os.path.isfile(path + 'captions_' + name + '_' + time.strftime("%Y-%m-%d") + '.csv'):         df = pd.read_csv(path + 'captions_' + name + '_' + time.strftime("%Y-%m-%d") + '.csv',                              index_col=0)         df = pd.concat([df, tmp], ignore_index=True)         df.to_csv((path + 'captions_' + name + '_' + time.strftime("%Y-%m-%d") + '.csv'),                       encoding='utf-8-sig')     else:         tmp.to_csv((path + 'captions_' + name + '_' + time.strftime("%Y-%m-%d") + '.csv'),                        encoding='utf-8-sig')
Now I don't want to get interupted with missing captions.

Just wrap get_caption in a try/except and skip that iteration
try:
   tmp = df = get_caption(video_id=i)
except CouldNotRetrieveTranscript:
  continue

Also, you can't compare a Transcript object with a string. So you can't just do transcript == 'de ("Deutsch (automatisch erzeugt)")[TRANSLATABLE]'. You can replace that entire section with:
transcript_list = YouTubeTranscriptApi.list_transcripts(video_id=video_id)
transcript = transcript_list.find_generated_transcript(['de', 'en'])
if transcript.language_code == 'en':
    transcript = transcript.translate('de')
text = ' '.join(snippet['text'] for snippet in transcript.fetch())

Great! This works exactly the way I needed! Thank you for your help and sorry for question, I'm still learning python. :)


Issue with /docs/tutorials/kubernetes-basics/cluster-intro/

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
http://kubernetes.io/...

The description needs to be updated. There is not enough information.


post: ipfs camp deep dives

This is the story of deep dives at IPFS Camp 2019 as told from the perspective of a dweb historical researcher in the far future.
TODO:

[ ] add table with videos

I approve of this message.

Any objections to merging this?

Hey Alan - should have caught this earlier but there's a formatting error in the table for 'IPFS for websites', and it breaks the link to the video. I've fixed it in the read.me table - How do I request a blog post edit?

Send a PR and request a review? 😕

@doctorrobinson I fix it here https://github.com/ipfs/blog/pull/285


Strengthen key derivation of Ed25519 private key?

Hi.  I'm wondering if the key derivation of the Ed25519 private key could be strengthened, so that I'm able to verify that some randomness provided by the host was included in the resulting private key or public key.  Right now key derivation is memcpy:
https://github.com/tillitis/tillitis-key1-apps/blob/f45a70040059b54831ede297f892a65476f103c7/apps/signer/main.c#L66
The CDI takes USS from the user, but from my limited understanding, there is no way for me as a user without hardware knowledge to verify that the USS provided ended up in the CDI in the documented way, or am I missing some method?

Hej Simon! The firmware is part of the FPGA bitstream on your TKey, and if you ran tkey-verification verify, that actually verified that the checksum of the firmware running is the one expected for your TKey revision, as signed by Tillitis. And it is the firmware that receives the device app with the optional USS, and prepares the CDI for device app's use before finally starting it. This is done here: https://github.com/tillitis/tillitis-key1/blob/main/hw/application_fpga/fw/tk1/main.c#L91
I think that answers your question?

And perhaps the comment there would better say Pick up CDI as private key, and derive the public key from it

Thanks for pointer to firmware verification stuff!  Interesting, I'll take a look.  Unless I'm missing something, it doesn't really change a lot -- I still can't confirm (strongly) from the ed25519 public key output that my USS was part of the input, can I?  While I think the current functionality is good -- I have to trust that you manufactured and sent me the correct key -- I think it could be improved in a way that I can confirm that the public key I get from the device was influenced (in a strong, cryptographic, way) by the randomness I sent to the device.  Right now I can't -- I have to trust you on this.  I don't want to suggest solutions, as there may be many different designs that leads to stronger guarantees for the user.

The desired key derivation function I'm thinking about is something like this:

Host sends a random string (say 32 bytes) Nonce to the Tillitis key
The key derives the Ed25519 private key and sends the Public key together with a Proof to the Host
The Host is able to verify that the Nonce was used to generate the private key corresponding to the public key
The private key should be secure (non-guessable) even if the nonce leaks, assuming secrets/entropy from Tillitis are good
The private key should be secure (non-guessable) if the secrets/entropy in Tillitis is compromised, assuming the nonce is not leaked

The Key derivation is thus a form of a Two-Party Verifiable Random Function where one party (the host) may disclose its secret key after derivation.
I don't want to go into solution-space, because I think the desired functionality could be built in many different ways, but one pointer for a relevant construction is the DDH-DVRF:
https://eprint.iacr.org/2020/096

I think this sounds like a really good, cool idea well worth exploring.
We should be able to make a version of the signer app that implements something like this.


remove old go:generate directive which is broken in go1.8

This should unblock
https://github.com/bazelbuild/rules_go/pull/226


Problem with dynamically constructed validation function

Are you submitting a bug report or a feature request?
bug report
What is the current behavior?
I'm trying to use validation function, that is dynamically constructed during render phase. I need it to check, that field value is equal to another field's value. This doesn't work. However static function works perfectly.
What is the expected behavior?
I guess it should work, unless I'm doing somethingg wrong
Sandbox Link
Here is the sandbox link https://codesandbox.io/s/4xvj31y174
What's your environment?
Redux-Form v7.2.0

The value of validate props should not change between renders as it causes the field to be re-registered. In your example you should call createEqualityValidator outside of the component so that you don't create a new instance each time.
If for some reason you can't create the validation function statically, you can still call createEqualityValidator during render as long as you memoize the result.

@ancestorak thx for the explaination


Edit hdinsight-provision-clusters.md

Made changes to reflect updates in Blob terminology.

Hi @ShawnJackson, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-shawja). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


OpenGL wrapper or 3D support in CanvasView

I'll very likely need this to build out some of the augmented reality features. I want a nice wrapper library for working with opengl (building from scratch to my needs, like I did with the CanvasView)
https://developer.android.com/develop/ui/views/graphics/opengl/projection
Before implementing, experiment with 3D in the Canvas (will need to implement my own projection system + camera)

Not needed for now - nothing in the AR features requires a 3D rendering


FAQ Section

We may want to include a new area in the documentation that stores questions we generate ourselves or receive from the outside, along with solutions.

We can start this in the wiki and then put it in the documentation with each release.

Make sure we have a link to the wiki in the official document for users to add questions or see newer questions

Started a list in the wiki:  https://github.com/rstoneback/pysat/wiki/FAQ
Feel free to add or modify.

Closing with merge of #217.


Restic and usage cpu

Output of restic version
0.73
How did you run restic exactly?
nice -n 19 restic -r $myrepository backup $myfolder

What backend/server/service did you use?
Expected behavior
When i check with command top, restic use 185,0% of CPU.

Actual behavior

Steps to reproduce the behavior

Do you have any idea what may have caused this?
Do you have an idea how to solve the issue?

Again, you did not specify what's wrong with the current restic behaviour that you experience, and what would be the expected (correct) behaviour.

@scialo What is the actual problem you are having?

No problem, thanks

So, I'm gonna guess here: You've run restic with nice and saw 185% CPU usage.
That's expected for two reasons:

restic uses multiple CPUs/cores if available, and the percentage is relative to one core. So if you have two cores and a program uses both the whole time, the CPU usage would be 200%.
nice only tells the OS: "if there's another program that can be run at any given moment, prefer the other program". So if there isn't any other program, the OS will just run restic. So you see restic at the top of the list of active processes probably because at that moment there was no other CPU intensive program to run.

That's only as far as we can help you without any in-depth description.

Thanks very much.


Allow custom built-in functions

Custom could be included in the plan if built using a provided capabilities.json file.
We should extend the builtin registry in order to source builtins from disk (or the classpath, or URL, or whatever :))

The Java ServiceLoader would be a nice way to allow for pluggable built-in functions.


Python script of version_data

Fixes #405

Just one very minor change and then I can merge!

LGTM!

Merged! Thanks for contributing!


Add Logging for class instantiations and method calls

logs will go in session objects as well as to local files


dev Nov11

use Channel to partially avoid stuck when client call Commit method
count neuron layer for future debug

notice:

Method ExecuteRevision (server side) now competes with method Commit (client side). Priority is given to client side for complete network update.

Issues:

CI checks failed.
Hot update. If I renew a neuron with new version number or even different return format, how would program react?
How to implement circular reference? Plan this in future versions.


Number utils

Description

Created some functions to do some conversions for the new input field:

number -> exponential
exponential -> number
number -> NIST
NIST -> number
displaying a number (this depends on the length)

I have made the input for all these functions a string, since we can deal with very small or very large numbers it can lead to precision issues if we were to use the Number library to manipulate these inputs

UNIT TESTS!!

With regards to point-2, I am a bit confused if you are referring to floating-point precision, or if you are talking about allowing arbitrary precision.

If the former, string vs number will not solve anything, the problem is inherently baked into binary representation. Short of using a specialized library we just need to be careful and deal with it. e.g. 0.2 * 0.2 != 0.04
if the latter, I don't think we need to worry too much about it. Our problem domains should not have extremely large numbers or exceedingly small numbers.

So overall it would be good if you elaborate a bit more on the design goals.

@mwdchang I've address these comments and am focusing on formatting rather than parsing.
Change made:
removed cleanNumberString
removed numberToExponential, we can just use toExponential
exponentialToNumber now uses toLocaleString to properly display the full number rather than the custom parsing
numberToNist is relatively unchanged aside from using parseFloat to clean the number input
nistToNumber cleans all whitespaces and then parseFloat rather than the custom parsing


Example Error

The example code does not work.
I have tried running the code in the example folder and have received the following error:
Firefox can’t establish a connection to the server at ws://localhost:5000/echo.

However, the following command does work:
wscat --connect ws://localhost:5000/echo

I have installed the following Python packages:
$ pip freeze
click==8.0.0
colorama==0.4.4
Flask==2.0.0
flask-sock==0.3.0
h11==0.12.0
itsdangerous==2.0.0
Jinja2==3.0.0
MarkupSafe==2.0.0
simple-websocket==0.2.0
Werkzeug==2.0.0
wsproto==1.0.0

What did you type in the address bar in Firefox? You should use http://localhost:5000.

Yes, that is what I did, you can see the error here:

Here are the exact steps to reproduce the issue, in the example folder:
$ py -m venv venv
$ py -m pip install flask
$ py -m pip install flask_sock
$ py -m pip install "werkzeug>=2.0.0rc3"

$ FLASK_APP="echo.py" flask run
 * Serving Flask app "echo.py"
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
<IP_ADDRESS> - - [17/May/2021 16:46:08] "GET / HTTP/1.1" 200 -
<IP_ADDRESS> - - [17/May/2021 16:46:08] "GET /echo HTTP/1.1" 400 -

Okay, thanks. I've found the problem, it is a bug in the Werkzeug changes I contributed. See https://github.com/pallets/werkzeug/issues/2131.
I'll work on a fix.

Ok, thanks Miguel.

A fix for this issue has been committed to Werkzeug towards the 2.0.2 release. You can verify the fix by installing Werkzeug's 2.0.x branch (or wait until a new release is made).

Confirmed, installed the 2.0.x branch and it is working, thanks!
py -m pip install -U https://github.com/pallets/werkzeug/archive/2.0.x.tar.gz


Remove development-only pom.xml file

Hurray!

The real versions are still here? https://github.com/useshortcut/spymemcached/blob/master%2Bpatches-on-couchbase-spymemcached/ivy/libraries.properties

@favila That is what Ivy relies on, yes. I also can't get Ivy to resolve the dependencies when I run ant jar, but given our discussion about how spymemcached is consumed by backend, we're basically electing to ignore this repository and cover the security surface area for it in useshortcut/backend.

Also I'm pretty sure the pom in the release artifact doesn't mention any of these: https://github.com/useshortcut/spymemcached/blob/master%2Bpatches-on-couchbase-spymemcached/spymemcached-2.12.3.sc5.pom


craft a webpage in R Markdown for site

index.Rmd with links to the files

I suspect that http://rmarkdown.rstudio.com/rmarkdown_websites.html may be a great starting point (again let's keep it simple).


Apim 5453 - mapi-v2 - get current theme, get default theme + update theme

Issue
https://gravitee.atlassian.net/browse/APIM-5453
Description
Endpoints added in MAPI-V2:

User can get default theme for portal next
User can get current theme for portal next + portal
User can update theme by theme id

Additional context

Will be addressed by other PRs


[kubeapiserver] Allow allow-listing of source component

What does this PR do?
Extends the configuration language to also allow sources. Instead of:
collected_event_types:
  pod:
    - Failed
    - BackOff
    - Unhealthy
    - FailedScheduling
    - FailedMount
    - FailedAttachVolume
  node:
    - TerminatingEvictedPod
    - NodeNotReady
    - Rebooted
    - HostPortConflict

Now we have:
collected_event_types:
  - kind: pod
    reasons:
      - Failed
      - BackOff
      - Unhealthy
      - FailedScheduling
      - FailedMount
      - FailedAttachVolume
  - kind: node
    reasons:
      - TerminatingEvictedPod
      - NodeNotReady
      - Rebooted
      - HostPortConflict
  - source: watermarkpodautoscaler

Describe how to test/QA your changes
Ensure that the different filters work. To ease QA though, I'm applying skip-qa, and updating #13158 to add instructions to test the changes here as well.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

The change in cmd/agent/dist/conf.d/kubernetes_apiserver.d/conf.yaml.example looks like it's an incompatible change. Is the old syntax still supported?

@djmitche it is incompatible, but it has never been released (#13158, where it was introduced, will ship in 7.40 too), so we're not worried about backwards compatibility yet.

should we keep "toLower" to avoid small error if customer do a small typo?

@clamoriniere there's a small performance penalty with these changes, since we no longer cache a normalized version of the config, since it's much more complex. It's easier to add this later than having to revert, so I think we can ship it in 7.40 and see how people use it.


owner of items

Hi, how are you?
I wanted to know if the ALLITEMS function has an additional parameter, or if there is another function that tells me who the owner is of each type of item (e.g., report, report builder, semantic model, notebook, data warehouse, etc.).

Do you mean semantic link's list_items function? The API that this calls does not return the item owner. Currently there is no way to programmatically find the item owner. The only exception is with regard to semantic models. You can do this. This calls the REST API which returns the 'Configured By' column, showing the owner of the semantic model.
python import sempy.fabric as fabric workspace_name = '' # Enter your workspace name fabric.list_datasets(workspace=workspace_name, mode='rest')


Use the nixos-rebuild script

As noted by zimbatm in #25, the deploy script is starting to look like nixos-rebuild.
It seems like my use case from #25 is actually subsumed by nixos-rebuild, unlike my earlier quick interpretation of the docs. Clearly I didn't find the parenthesized bit from --target-host:

(and no build artifacts will be copied to the local machine)

challenges
nixos-rebuild does need some bootstrapping. nixos-rebuild.sh has a build process that substitutes some dependencies into the script. Replicating this is a step back from the current script, which takes dependencies from the environment, allowing it to run without a proper /nix/store store. This is useful when deploying from restrictive environments without root access or mount namespace capability. So instead of substituting dependencies from the nix store into the script, we should allow values from the environment.
Note: static nix is not officially supported yet, but here's a branch that makes it work on x86_64-linux deployer machines. https://github.com/tweag/terraform-nixos/compare/master...hercules-ci:install-static-user-nix?expand=1

Assuming that nix is installed on the system, the deploy script could also build and invoke nixos-rebuild.sh on the fly. I don't think it's going to add too much to the invocation time.
Static nix is interesting but seems a bit orthogonal.

Static nix is what allows us to use nix without a /nix directory.
The script wont work if it's not in /nix/store.
Also you'd have to build it, which requires that the deployer is a Nix trusted-user on multi-user systems.

While it would be nice if the terraform module also worked on machines that didn't have nix installed on them, isn't that an existing requirement because the nixos-instantiate.sh script invokes nix-instantiate?

Not quite. nixos-instantiate has to be available in the environment, but it does not have to be installed into /nix/store. This works with static nix.
Building any derivations is not feasible in such a restricted environment, but instantiation works.

Nice. Does that also work on Darwin or does it depend on having user namespaces available?

No namespaces used. I see no reason why it shouldn't work on Darwin.


ktlint_function_signature_body_expression_wrapping configuration doesn't work via .editorconfig

Expected Behavior
Function declaration should be multiline.
The changing of existing parameters in .editorconfig doesn't lead to expected result.
The debug output of ktlint is here ktlint-output.txt
fun someFunction(
    a: Int,
    b: Int,
    c: Int
): Int {
    return a + b + c
}

Observed Behavior
Function declaration is single line
fun someFunction(a: Int, b: Int, c: Int): Int {
    return a + b + c
}

Steps to Reproduce
1 - install ktlint
2 - create .editorconfig file with output of command: ktlint generateEditorConfig --code-style android_studio
3  - change ktlint_function_signature_body_expression_wrapping to multiline
4 - format the code ktlint --editorconfig=".editorconfig" --format --log-level=debug
Your Environment
ktlint v1.2.1

Your example code does have a block body instead of an expression body. So the .editorconfig property ktlint_function_signature_body_expression_wrapping has no effect on that example.
When using code style android_studio, the function signature can contain any number of parameters in a single line as long as the entire signature fits on the single line. If you always want to wrap signature containing multiple parameters, you will need to set property ktlint_function_signature_rule_force_multiline_when_parameter_count_greater_or_equal_than to 2. See https://pinterest.github.io/ktlint/latest/rules/standard/#function-signature
In case it still does not work as expected, please post your .editorconfig.  Also make sure to explicitly save .editorconfig after changing it, and before running ktlint again.


Add option to set oracle config file path

For #138
Hi, I've added an option to change file path for oracle_config.yaml. Note that the once_cell dependency can be removed once it lands in rust stable, although one of the other crates oracle-core depends on pulls it in anyways

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 7 (28.57%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.08%) to 64.765%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/oracle_config.rs
2
3
66.67%

core/src/main.rs
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/src/main.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
1873

Relevant Lines:
2892

💛  - Coveralls


Date and read-time present in table of contents but missing from post

The example Geekblog site includes the upload date and approximate article duration metadata in both the table of contents view  (in the footer) and the post view (in the header).
My implementation of Geekblog is only showing this metadata on the table of contents and not in the gblog-post__header div which is missing a nested gblog-post__meta div.
Can you help me understand how I can include this metadata on the post itself?
Desired behavior -

My implementation -

Thanks for your report. Can you show me how your content folder looks like?

This is my content folder:

So I just changed that folder name from "post" to "posts" and that appears to have resolved the issue!

Ok you have to keep it named posts  for now as this is hard-coded, see https://github.com/thegeeklab/hugo-geekblog/blob/master/layouts/_default/single.html

Thanks for the quick reply! The theme is fantastic and exactly what I was looking for.

Right. I'll check how to fix it in a future release, so keep this issue open to track it. Thanks again :)

This should be fixed now.


feat(shared-data): add H/S to deck definition, bump deck definition schema

Overview
Adds heater-shaker module compatibility to deck definitions in anticipation of #9642.
Bumps the schema to version 3.
Changelog

added h/s module compatibility to slots for all deck types
removed schema 1 & 2 definitions and fixtures
updated api to use v3 deck, updated tests
updated app, components & PD to use the new deck schema and definitions (by @shlokamin)

Review requests

Diff looks kinda big but it's mostly because of V1 & V2 deck removal. A good place to start reviewing would be shared-data. Everything else is basically switching all systems over to using V3.
Check that no unexpected changes have gone in, like removal of a field in a v3 deck definition
Check that the correct slots on the deck are made compatible for heater-shaker (that's all except the trash). Especially check for short trash and ot3 because I am not familiar with those decks.
All backend + FE/ client changes look good

Risk assessment
Low. V3 is a superset of V2 deck so I don't anticipate things breaking if tests pass

LGTM! Diff'd 3.json with 2.json manually, and the only differences were the addition of heaterShakerModuleType and setting the allowed version to 3.

Thanks for checking!

Should we delete shared-data/deck/schemas/2.json and shared-data/deck/schemas/1.json, now that they're not used?

I prefer keeping them so we have some reference to the updates in schemas. Helps explain where the version '3' is from, since all other references to previous versions are removed. But not too attached to them either so can remove them if necessary.

Going to merge this so it unblocks the H/S in PAPI work. Will revisit if required when we have a decision about the slot 11 incompatibility.


Trezor pairing is broken

During pairing of a Trezor the extension attempts to load
chrome-extension://mgffkfbidihjpoaomajlbgchddlicgpn/trezor-usb-permissions.html
This results in
Your file couldn’t be accessed
It may have been moved, edited, or deleted.
ERR_FILE_NOT_FOUND

pls pass me these details:

os
pali version
trezor firmware version
trezor suite version
screenshot of the error message
steps to reproduce the error (if it is reproducible)
trezor model

it is working fine using pali v1.0.22 with trezor suite latest v22.11 and trezor firmware v1.10.2 or v1.10.5 (latest one) on macos, pls if it is reproducible provide more details so i can find the error
thanks for the report 😄

Having the same issue with Pali Wallet

Will this be fixed?

I still can't connect Trezor model T

Trezor model T
os: Windows 10
pali version: v1.0.24
trezor firmware version: v2.4.3
trezor suite version: v22.2.3

thanks, we will check your problem 😄

Chrome drops this when I try to connect my wallet

Chrome drops this when I try to connect my wallet

After that I closed and opened again and it connected correctly. But the problem persists sometimes

hi @pauserratgutierrez, just so you know, we have this issue solved a while ago, as well as the other reported issues that are already closed, i'll push a new version with all of these issues fixed, sorry for the late reply, i'm closing this issue now and if you have any other problems you can reopen this, thanks for the report 😄


Adds the OrdersPage [frontend + backend]

Description
Closes #200

This PR does the following:

Adds ... the Orders Page[frontend + backend]
Fixes #200
Updates ...

Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.

Placeholder: "Fixes #200 "

Changes
List the detailed changes made in this PR.

Added a new feature to ...
Refactored the ...
Fixed a bug in ...

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Verify that ...

Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.
Additional Context
Any additional context or information that reviewers should be aware of.

This PR is based on the following...

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@Trisha-tech please please merge this...

Resolve these conflicts @arunimaChintu

@Trisha-tech done...

@Trisha-tech I have resolved the conflicts... Please merge this now

@Trisha-tech Please merge this soon... Tomorrow is the last date for gssoc24

@Trisha-tech would you please mind to add the labels... Please


⛏ chore: fix Django user api

Addresses 453
🚧 Changes
- add client/app/utils/enums.ts
- modify client/app/api/auth/[...nextauth]/route.ts
- modify client/app/utils/actions.ts
- modify bc_obps/registration/api/user.py
- modify client/e2e/auth/authentication.spec.ts

🚀 Impact:

Use of enums for improved code quality
Modified client/app/utils/actions.ts for to include GUID in Authorization header for endpoints calls occurring before a NextAuth token is created (new user)
Removed raise_401_if_role_not_authorized check on user routes that do not have a user/user role in the db
Enhanced  client/e2e/auth/authentication.spec.ts to delete created new user allowing test re-run without manual intervention to delete user bc-cas-dev-three

🔬 Testing:

Follow the developer environment setup to set up db, load fixtures and start the API server
From terminal command, start the app development server:

cd client && yarn dev

API tests

Open terminal to run endpoint tests

cd bc_obps && make pythontests

All tests should pass

APP tests
e2e tests
Ensure you have the following configuration

Files exists in folder client/playwright/.auth from 1password

OBPS FE Auth Session cas_analyst
OBPS FE Auth Session cas_pending

File exists: client/e2e/.env.local
client/e2e/.env.local values reflect instructions from file client/e2e/.env.local.example
Ensure the the erc.user table includes rows below without bc-cas-dev-three

SELECT first_name,last_name,app_role_id FROM erc.user;

| FirstName         | LastName          | App Role      |
| ----------------- | ----------------- | ------------- |
| Bcgov             | Cas               | industry_user |
| Bcgov             | Cas SECONDARY     | industry_user |
| Your Firstname    | Your Lastname     | cas_admin      |

From terminal command, run Playwright e2e tests:

cd client && yarn e2e

All tests should pass

Note: You can access the Playwright GUI for tests using command:
cd client && yarn e2e:ui

New User Login

Ensure the API and App servers are running (as detailed above)
In a new incognito window, navigate to http://localhost:3000
You should be directed to the anonymous landing page
Click "Industry Operator Log In"
Log in with Bceid: bc-cas-dev-three, defined in 1password
You should be directed to dashboard profile form with values in login name and email corresponding to the login ID details

Attempt to navigate to Dashboard
You should be directed to Dashboard\Profile form
Complete the profile form and click "Submit"
Attempt to navigate to Dashboard
You should be re-directed to the IO dashboard

This is also going to address this ticket: https://app.zenhub.com/workspaces/team-moose-sprint-board-60d216175a2689000ec774d0/issues/gh/bcgov/cas-registration/456

Great stuff 🤩 One minor observation but not a blocker: Could we implement a redirect for new users to go directly to the dashboard upon completing the user form, instead of reloading the page with updated data?

Related to this comment but not longer valid: https://app.zenhub.com/workspaces/team-moose-sprint-board-60d216175a2689000ec774d0/issues/gh/bcgov/cas-registration/467


BytePrimeCache Static Array Optimization

Implementing a static ByteArray inside PrimeCacheBase has demonstrated a significant performance improvement across all methods of interest (those measured by PrimeCachePerformanceTest).
The proposed changes are a great start to optimization. Further tests and comparisons are recommended before merging, as there is certainly more to gain through careful and patient study.

ArrayDeque appears to be aggressively resizing before reaching capacity, reducing the performance of targetedIndex at indices just before 24.
Graphs will be produced soon, showing what a difference these changes have made.

Processing and formatting test result data in a CSV file friendly format is an essential feature in performance testing automation for achieving greater efficiency.

Data Processing and Formatting
To reach a greater level of testing automation in this area, additional methods involved in data assessment are suggested:

[ ] Outlier detection and removal
[ ] Distribution characteristics
[ ] Uncertainty quantification

The goal of this PR however, is to simply demonstrate that the implementation of a static byte array for the first n primes can improve performance, and to begin using it immediately in the main release.
The way that this will be demonstrated is intended to set a high expectation of quality for future development in this project. When future performance claims are made, they will be easily compared with this existing data. Thus, a sufficient investment in data processing and formatting is in preparation.

Looking forward to wrapping this one up using new statistics ( #24 , #27 ) and performance testing methods ( #22  ).


New button architecture.

Don't inherent from a generic button class.
Checkbox, toggle and radio-button should have two layers. One for the button itself and the other for the button+label.

checkbox (just the checkbox)
radio (just the radio button)
toggle (just the toggle)
button<checkbox> (checkbox with label)
button<radio> (radio button with label)
button<toggle> (toggle with label)
momentary_button

In the mouse_event add the hitbox id, so that it can be matched with the id of the widget instead of doing a contains() check.


Macro: When switching to the previous mode, using the “Undo” button, the “Edit mode” drop-down does not disappear

Steps to Reproduce

Switch to the Macro mode
Open this file  Macro sequence.zip
Switch to the Sequence mode
Switch to the Snake or Flex mode
Switch to the Sequence mode again - the Text-editing mode
Click on the “Undo” button a few times

Actual behavior
The “Edit mode” drop-down does not disappear

Expected behavior
The “Edit mode” drop-down is not displayed on the screen

Screenshots
https://github.com/epam/ketcher/assets/139227567/110c56c6-d59f-44a2-98dc-8e77d13e0f69
Desktop (please complete the following information):

OS: Windows 10 Pro (version 22H2) x64
Browser: Сhrome
Version: (122.0.6261.112) x64

Ketcher version version  2.20.0-rc.1

Moved to Refined Backlog since not fixed over 1 release


Duplicate of #39 for tests

Duplicate of https://github.com/Kong/go-kong/pull/39 to allow Enterprise tests that require GH secrets to proceed. All discussion should continue on #39; nothing should be done with this PR other than view test results.

Codecov Report

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

@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage        ?   37.62%
=======================================
  Files           ?       38
  Lines           ?     2488
  Branches        ?        0
=======================================
  Hits            ?      936
  Misses          ?     1322
  Partials        ?      230

Flag
Coverage Δ

1.1.3
37.62% <0.00%> (?)

1.2.3
37.62% <0.00%> (?)

community
37.62% <0.00%> (?)

integration
37.62% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 cba87b1...d2562e6. Read the comment docs.


config.presets as an array

🚀 Feature Proposal

Allow jest configuration file to include an array of presets, on top of a module/file path.
Motivation

When adding multiple presets, the only solution we have is messy: deep concatenation of properties.
Example
export default {
  preset: ["ts-node", "webdriverio"]
}

Pitch
This deserves to make it into the core because it goes alongside what jest does well in its' configuration: intuitive customisability. We support other properties like projects, so I think that this deserves the same level of concern.
Out of Scope
Jest is awesome, it feels very good to use!

I'm not sure if I agree that this would bring an improvement over requiring and (fully or selectively) merging the objects like you could do today. I know many libraries out there support multiple presets... and some times it makes sense; like with for example adding multiple complex capabilities to something like Babel that needs to handle everything between here and the moon. But in the context of Jest, I wonder if it just makes things unnecessarily complicated?
How would you resolve conflicts / overlapping configuration options between the presets? And how would that handling be less messy (and more user friendly) than the current option?
Could additional/improved documentation be an alternative?

@sigveio  I would say that documentation is the better alternative. I think you're right, it's a bit complicated.
Would you be happy a more polished version of this below?

A preset is a jest configuration, so feel free to import and merge it into to create custom configurations.
This is useful when there are multiple presets, and provides flexibility if there are conflicts between common properties (ie Multiple Reporters or Transformers).
With ts-node and cjs examples.
import { default as TsJest } from "ts-jest"

export default {
  ...TsJest,
  // ... other configuration
}

There would be too many caveats here. There would have to be warnings on things that can't be concatenated together (ie. environment, can't be both jsdom and node).


[tests] diffuser models in model zoo

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[ ] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ ] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

I added diffuser models in model zoo.
💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

Just wanna mark that this PR is part of efforts for #3049


Adds breadcrumb navigation to internal pages

Adds working breadcrumbs to internal pages.
This uses the same repoSchema that the sidebar uses to determine the breadcrumb items to show.

If it's ok @10thfloor I'm going to put up the display name feature as a separate PR.


chore(deps): update common-dev-assets digest to c11b977

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
b31a8c6 -> c11b977

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update Blog “new-beginnings/index-1”

Automatically generated by Netlify CMS

👷 Deploy Preview for tubular-cheesecake-3ca0e1 processing.

Name
Link

🔨 Latest commit
fa4775fe0204579162a4a283a8fe9975b2e93de5

🔍 Latest deploy log
https://app.netlify.com/sites/tubular-cheesecake-3ca0e1/deploys/651dd7b7b0ba490007e32f8c


onDeselectAll event is too early

The "onDeselectAll" event is received before columns had really changed

Awesome fix!!!!!! Thanks a lot!!!


[Tracking] Linting

Tracking decisions and development of our tooling for automated code formatting and validation

We have decided to use just eslint, with our standard configuration in @eternagame/eslint-plugin. At a high level, we use the airbnb config plus the strictest typescript config and recommended vue3 config, in addition to some of our own customizations.
We have opted not to use prettier. While its power and consistency is nice, Thomas and I find some of its opinionated decisions to be undesirable, so we prefer to rely on what eslint provides.


[PLT-8712] Fix opening attachment preview on compact view

Summary
Fix opening attachment preview on compact view
Also, removed unnecessary instance function and props in FilenameOverlay.
Ticket Link
Jira ticket: PLT-8712
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-010bcda53607bf123.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-010bcda53607bf123

Spinmint test server destroyed


[WIP] coprocessor: separate thread for encoding data for batch cop

Signed-off-by: JaySon-Huang<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tiflash/issues/4415
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[x] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

/run-integration-test

TODO: consider using SyncTunnelSender introduced in #5286 ?


AsyncAPI schemas should use their id not ours

Describe the bug
Currently, in templates, all you have access to is the uid() function from the AsyncAPI parser for the schema which returns a different name than what the model is generated with. This should not be the case.
Why? Because I cannot change what I have access to, we should not interpret another id if one is already given by another tool.

@magicmatatjahu I will start on this issue since it is a blocker for me to continue rewriting the template.


Does this work on macOS?

this is a quote from one of the admins of the BTD6 mods discord server
"Mac: No Support. Should've picked a better OS lol" (he was talking about support for BTD6 mods in general)


feat: subset translation

Please answer these questions before submitting a pull request

Why submit this pull request?

[ ] Bugfix

[ ] New feature provided

[ ] Improve performance

[ ] Backport patches

Related issues

Bugfix

Description

How to fix?

New feature or improvement

Describe the details and related test reports.

Backport patches

Why need to backport?

Source branch

Related commits and pull requests

Target branch

Codecov Report

Merging #497 (a69430a) into master (e8eddcc) will decrease coverage by 0.04%.
The diff coverage is 35.45%.

@@            Coverage Diff             @@
##           master     #497      +/-   ##
==========================================
- Coverage   37.24%   37.19%   -0.05%
==========================================
  Files          49       51       +2
  Lines        4030     4132     +102
==========================================
+ Hits         1501     1537      +36
- Misses       2328     2386      +58
- Partials      201      209       +8

Impacted Files
Coverage Δ

pkg/ingress/controller.go
1.35% <0.00%> (+1.35%)
:arrow_up:

pkg/kube/translation/util.go
47.19% <4.76%> (-13.68%)
:arrow_down:

pkg/kube/translation/translator.go
45.67% <10.00%> (-5.71%)
:arrow_down:

pkg/ingress/pod.go
28.88% <28.88%> (ø)

pkg/kube/translation/apisix_route.go
42.71% <33.33%> (ø)

pkg/kube/translation/ingress.go
87.33% <100.00%> (ø)

pkg/kube/translation/plugin.go
100.00% <100.00%> (ø)

pkg/types/pod.go
100.00% <100.00%> (ø)

pkg/ingress/ingress.go
8.58% <0.00%> (ø)

... and 2 more

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 e8eddcc...a69430a. Read the comment docs.


add dataframe.io.to_hdf

Add to_hdf method to DataFrame and Series
In [1]: import dask.dataframe as dd

In [2]: import pandas as pd

In [3]: df = pd.DataFrame({'x': [1, 2, 3]}, index=[1., 2., 3.])

In [4]: df = dd.from_pandas(df, npartitions=2)

In [5]: df.to_hdf('myfile.h5', '/x')

In [6]: pd.read_hdf('myfile.h5', '/x')
Out[6]:
   x
1  1
2  2
3  3

This turned into a "bunch of small changes on dask.dataframe" PR

Changes include

to_hdf
read_hdf
a better, more robust to_castra, particularly with categoricals and index information
Better handling of categoricals in set_index, including fully releasing the old dataset

Merging soon if no comment


Scalar-aggregate subqueries in projection lists

Add some syntax for scalar subqueries, to allow multiple independent aggregates to be projected for the same result entity (or group) without producing a cross-product of input rows between the different aggregates.

I'm not sure if this is  a wishlist thing. It seems like we have been working around it with views instead...?
@hongsudt @carlkesselman @robes @bugacov

Closing this stale issue.


mattdrayer/increment-version: Bump to v0.2.9

@jzoldak @nedbat -- made the new v.0.2.8 tag before updating setup.py (whoops).  Doing things the correct way this time and bumping to v0.2.9

LGTM, deferring to @nedbat for the thumbs.

:+1:


refactor vic_run() rec and dmy usage

removes rec from vic_run()
modifies *dmy to be a single dmy_struct rather than array of dmy_structs
closes #266

The failing tests here are on MacOS which they just switched on for us.  I am will remove those tests for now and will add back in #244.

Reviewing the commit it seems that rec was used extensively to provide info during an error. From RASM we know how hard it is to diagnose errors when you don't know when (rec) or where (in RASM) they occur. Is there another way to provide that info (even if we remove rec from the vic_run call).

Also - have you done a regression test to ensure that the result has not changed after the refactor (funny things can happen if the arguments in va_arg are not aligned for example.

I've been thinking, it may be nice to add a bit of debugging information such as the grid cell, timestep etc. to a driver specific error function.
I have not done a regression test but I will now.

I just ran some tests using the classic driver and got the same results via this branch and the develop branch.
The usefulness of rec for debugging is still to be addressed. This is mostly just an issue for the classic driver because we can print the date and timestep before each vic_run call in the other drivers. @bartnijssen, do you have any thoughts on how to proceed?

I say let's move ahead with the refactor and address the debugging info at a later stage. Probably revisit the debugging together with #261 (even though that may not be exactly how we do it).


typo in geomaterials domain

On page 49 of the provisional document a material is listed as "Debris flow, landslide, and other localized mass-movement sediment" but in the domain it is listed as "Debris flows, landslides, and other localized mass-movement sediment"

These (and similar issues) should be moot with publication of the final specification document and updating of the reference tables.


Placeholder for checking that onResult rules are always MECE

This will need some code as it's inexpressible at the JSON Schema level.

This was completed as part of 0.1.0!
https://github.com/snowplow/factotum/blob/release/0.1.0/src/factotum/fileparser/mod.rs#L116-L124
https://github.com/snowplow/factotum/blob/release/0.1.0/src/factotum/fileparser/tests.rs#L65-L85
Perhaps there's more to it?

Ace, closing!


Feature (Frontend): Move expenses plan api to new file

Codecov Report

Merging #154 (d856efc) into main (12756fc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #154   +/-   ##
=========================================
  Coverage     75.76%   75.76%
  Complexity      128      128
=========================================
  Files            41       41
  Lines           359      359
  Branches          7        7
=========================================
  Hits            272      272
  Misses           82       82
  Partials          5        5

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 12756fc...d856efc. Read the comment docs.


⚠ Panoramas has degraded performance

In 06ad164, Panoramas ($SITE_URL_PANORAMAS) experienced degraded performance:

HTTP code: 200
Response time: 6514 ms

Resolved: Panoramas performance has improved in 81327d4 after 8 minutes.


Fix error messages

Pull Request Test Coverage Report for Build 82

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.045%

Totals

Change from base Build 73:
0.0%

Covered Lines:
1870

Relevant Lines:
1947

💛  - Coveralls


Add missing packet types to switch case

Ok, I understand the fix is for suppressing warnings. I merge it.


Release proposal 1.2.0 / 0.28

See CHANGELOG and experimental/CHANGLOG for release details.
If there are any issues or PRs which should BLOCK the release please comment them here.

The metrics exporters are still using ancient protos (otlpTypes from exporter-trace-otlp-http), think we could move to a newer format, via otlp-transformer package before doing a release? Or if that would delay it too much is there a possibility to cut a new, smaller release after moving to newer protos?

The metrics exporters are still using ancient protos (otlpTypes from exporter-trace-otlp-http), think we could move to a newer format, via otlp-transformer package before doing a release? Or if that would delay it too much is there a possibility to cut a new, smaller release after moving to newer protos?

Certainly we will need to release ASAP since the proto is very out of date but I think we can cut this release and another one next week for that. @pichlermarc was looking into that so maybe he has an opinion.

Thanks, makes perfect sense!

Updating the protos might take a bit as we will have to update many of the tests as well. I expect to have a PR ready for that on Tuesday.
@dyladan's timeline to release the proto update next week seems realistic. 🙂


Create cart component

This will require access to redux store information.
the single product snap shot will be nested within this.


Branding.js link

Updated code samples to include MDS (and non-MDS) site support

Hi @MartinHatch, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Thanks Martin!

Hi Martin,
I've merged this...but somehow it's not yet working correctly. Navigating from for the google maps list to the color list and back to the google maps list causes the google maps list not to render anymore. Do you see this as well in your testing?

Might be a caching issue as the files / paths are the same.
Have you tried deploying to a new Site Collection?
From: Bert Jansen<EMAIL_ADDRESS>Sent: 29 April 2015 13:36
To: OfficeDev/PnP
Cc: Martin Hatch
Subject: Re: [PnP] Branding.js link (#732)
Hi Martin,
I've merged this...but somehow it's not yet working correctly. Navigating from for the google maps list to the color list and back to the google maps list causes the google maps list not to render anymore. Do you see this as well in your testing?
—
Reply to this email directly or view it on GitHubhttps://github.com/OfficeDev/PnP/pull/732#issuecomment-97413338.

@jansenbe In which environment specifically are you seeing this? (SP2013 or SPO?)
I have just deployed the WSP to a brand new Site Collection and am not seeing that issue occur.

@jansenbe ok just seen this on an SPO site. I'll investigate .. not sure whats wrong

I've been testing in SPO only


Test against numpy nightly wheels

This adds a CI run using numpy nightly wheels as described at https://scientific-python.org/specs/spec-0004. It will therefore test against the latest numpy 2.0 build.
Not using matplotlib nightly wheels as they pin to numpy < 2.0 and only really need to test against install dependencies rather than test dependencies.

Confirmation that this works as intended:
Downloading https://pypi.anaconda.org/scientific-python-nightly-wheels/simple/numpy/2.0.0.dev0/numpy-2.0.0.dev0-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (18.0 MB)
...
Run python -m pip list
Package         Version
--------------- ----------
contourpy       1.1.2.dev1
coverage        7.3.1
cycler          0.11.0
fonttools       4.42.1
iniconfig       2.0.0
kiwisolver      1.4.5
matplotlib      3.8.0
numpy           2.0.0.dev0
packaging       23.1
Pillow          10.0.1
pip             23.2.1
pluggy          1.3.0
pyparsing       3.1.1
pytest          7.4.2
pytest-cov      4.1.0
python-dateutil 2.8.2
six             1.[16](https://github.com/contourpy/contourpy/actions/runs/6207431127/job/16852647827#step:13:17).0
wurlitzer       3.0.3


Update to Commanders Act version 5

As a member of the analytics team I want results to be gathered with the new Commanders Act platform requiring version 5 of the SDKs to be running in Play SRG.
Acceptance criteria

The Commanders Act SDK has been updated.

Tasks

[x] Update SRG Letterbox to version supporting Commanders Act version 5 (related to https://github.com/SRGSSR/srgletterbox-apple/issues/283).
[x] Production / debug source keys have been properly set up.

Missing device language property reported. It was set by Commanders Act SDK 4 with TC_LANGUAGE. No more in Commanders Act SDK 5.
Discussed here:
https://github.com/SRGSSR/srganalytics-documentation/issues/8
https://github.com/CommandersAct/iOSV5/issues/9


hovering effect on icons

Is your feature request related to a problem? Please describe.

When we hover we can see a transition but when we remove our mouse there is no transition.
Describe the solution you'd like
I will add a transition at the end.
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@Git21221 any update on this?

please assign me this issue @AshutoshDash1999

@Git21221 any update on this?

yeah updaing


[VS Code] The Visual Studio Live Share service is unavailable at this time. Please try again later.

<!--
For Visual Studio problems/feedback, please use the Report

@zomatree Thanks for reporting this! We’re looking into the issue, and are tracking it via #917. In the meantime, I’ll close this as a duplicate so we can consolidate the conversation.


fix: refactor pull flow to use amplify error handling framework

Description of changes

refactor pull flow to use the new amplify error types and error handling

Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #10974 (8a0905c) into dev (38a8603) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev   #10974      +/-   ##
==========================================
- Coverage   48.19%   48.19%   -0.01%
==========================================
  Files         670      670
  Lines       32446    32436      -10
  Branches     6598     6598
==========================================
- Hits        15638    15633       -5
+ Misses      15188    15183       -5
  Partials     1620     1620

Impacted Files
Coverage Δ

...s/amplify-cli-core/src/errors/amplify-exception.ts
80.00% <ø> (ø)

packages/amplify-cli-core/src/errors/index.ts
65.51% <ø> (-5.92%)
:arrow_down:

packages/amplify-cli/src/initialize-env.ts
14.66% <ø> (+0.74%)
:arrow_up:

...li/src/domain/amplify-usageData/getUsageDataUrl.ts
100.00% <0.00%> (+12.50%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Captive portal not working with iphone hotspots

We had a training session with Fernando Torres in Malmo recently.  We were able to run the WiFi captive portal example when we used the local WiFi network, but it never worked with iphone hotspots.  Multiple people have tried, multiple times, but we can't seem to get it to work on iphone.  This is really important because it would typically be the easiest way to demonstrate the program in the field.
Can you see if you can figure out why it isn't working with iphone?
Thanks,
Steve Brown

Hi @sbro-u-blox,
To understand the situation a bit better:

were you able to configure xplr-hpg-2 through an iphone (xplr has no configuration, you connect to it while it runs in access point mode)?
if first step was ok, then you run iphone as a hotspot and the xplr was configured to connect to it. Did you see it connected? (e.g. via iphone hotspot status, or though the xplr serial log)
If was connected, you were not able to access it's web page ?

A log of xplr serial console would help here

I was able to configure the credentials, and I believe they are correct.  But, I can't see that the device is connecting to the hotspot.
How would I tell from the serial log?  I see this:
D [(5528) xplrWifiStarter|xplrWifiStarterFsm|270|: WiFi init successful!
D [(5559) xplrWifiStarter|xplrWifiStarterFsm|281|: Register handlers successful!
D [(5584) xplrWifiStarter|xplrWifiStarterFsm|300|: Wifi credentials configured:1
D [(5609) xplrWifiStarter|xplrWifiStarterFsm|306|: WiFi mode selected: STATION
D [(5609) xplrWifiStarter|xplrWifiStarterFsm|314|: WiFi set mode successful!
D [(5656) xplrWifiStarter|xplrWifiStarterFsm|354|: WiFi set config successful!
I (5681) phy_init: phy_version 4670,719f9f6,Feb 18 2021,17:07:07
I (5785) wifi:mode : sta (e8:31:cd:da:a0:f4)
I (5786) wifi:enable tsf
D [(5788) xplrWifiWebserver|xplrWifiWebserverStart|228|: Starting server on port: '80'
D [(5791) xplrWifiWebserver|xplrWifiWebserverStart|232|: Registering URI handlers
I [(5798) xplrWifiDns|mDnsInit|339|: mdns hostname set to: [xplr-hpg]
D [(5814) xplrWifiStarter|xplrWifiStarterFsm|382|: WiFi started successful!
D [(5841) xplrWifiStarter|xplrWifiStarterFsm|431|: Call esp_wifi_connect success!
I (5849) wifi:new:<6,0>, old:<1,0>, ap:<255,255>, sta:<6,0>, prof:1
I (5851) wifi:state: init -> auth (b0)
Prior to that I see the correct SSID and Password on the serial interface.
I see GNSS statistics on the serial interface, but I can't ever get PointPerfect to get a high precision fix.
If you let me know how to capture more information for you, I will give it a try.  I'm kind of new at this, so pardon my ignorance.
Steve
From: dbou-ublox @.>
Sent: Wednesday, July 5, 2023 2:56 PM
To: u-blox/XPLR-HPG-software @.>
Cc: Steve Brown @.>; Mention @.>
Subject: Re: [u-blox/XPLR-HPG-software] Captive portal not working with iphone hotspots (Issue #1)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
**** This is an EXTERNAL email. It was sent from outside of u-blox. ****
Hi @sbro-u-bloxhttps://github.com/sbro-u-blox,
To understand the situation a bit better:

were you able to configure xplr-hpg-2 through an iphone (xplr has no configuration, you connect to it while it runs in access point mode)?
if first step was ok, then you run iphone as a hotspot and the xplr was configured to connect to it. Did you see it connected? (e.g. via iphone hotspot status, or though the xplr serial log)
If was connected, you were not able to access it's web page ?

A log of xplr serial console would help here

Reply to this email directly, view it on GitHubhttps://github.com/u-blox/XPLR-HPG-software/issues/1#issuecomment-1622404635, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A27SF4WUCTWW7ZVO2QCN3O3XOXBELANCNFSM6AAAAAAZ7MFXEA.
You are receiving this because you were mentioned.Message ID: @.***>

By the way, if I follow the exact same sequence of steps with my home WiFi network, it works as expected.
I get a FLOAT fix, and I can track my device on the web portal.
It seems to have something to do with using the iphone hotspot.
Steve
From: dbou-ublox @.>
Sent: Wednesday, July 5, 2023 2:56 PM
To: u-blox/XPLR-HPG-software @.>
Cc: Steve Brown @.>; Mention @.>
Subject: Re: [u-blox/XPLR-HPG-software] Captive portal not working with iphone hotspots (Issue #1)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
**** This is an EXTERNAL email. It was sent from outside of u-blox. ****
Hi @sbro-u-bloxhttps://github.com/sbro-u-blox,
To understand the situation a bit better:

were you able to configure xplr-hpg-2 through an iphone (xplr has no configuration, you connect to it while it runs in access point mode)?
if first step was ok, then you run iphone as a hotspot and the xplr was configured to connect to it. Did you see it connected? (e.g. via iphone hotspot status, or though the xplr serial log)
If was connected, you were not able to access it's web page ?

A log of xplr serial console would help here

Reply to this email directly, view it on GitHubhttps://github.com/u-blox/XPLR-HPG-software/issues/1#issuecomment-1622404635, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A27SF4WUCTWW7ZVO2QCN3O3XOXBELANCNFSM6AAAAAAZ7MFXEA.
You are receiving this because you were mentioned.Message ID: @.***>

Just a guess, since I do not have an iphone available.
Is the hotspot running at 5GHz maybe? (I think it n recent iOS you can tune it). If it is 5GHz only, then xplr will not connect, since the NINA module operates at 2.4 only

Hi Steve, It was working for me but I have not tried recently for the new code. Have you enabled compatibility mode and allowed access for others in the hotspot settings?

Hello Steve, just as Michael suggests, have you enabled compatibility mode?
Furthermore, searching in esp forums I see that other people had issues when they were having a backtick/apostrophe in their hotspot name. Can that be the case for you as well?

I can try changing the name and let you know what happens.  I have a “-“ in my hotspot name (SRB-XR).
But, I realized that I have that in my home network too (The-Brown-Network) which does work every time.
My hotspot password has “!” in it.  Any issues with that?
Maybe I’m doing something else wrong.  If it was actually connecting to the hotspot, how do I get to the web portal from my  PC?
I have to connect my laptop to the hotspot, then what address should I use in the browser?  I’m thinking maybe I messed that up.
Steve Brown
u-blox America
Field Applications Engineer
39W716 Buckskin Trail
St. Charles, IL 60175
847-910-6791 (cell)
@.***
On Jul 6, 2023, at 2:45 AM, Al3x @.***> wrote:

You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
**** This is an EXTERNAL email. It was sent from outside of u-blox. ****
Hello Steve, just as Michael suggests, have you enabled compatibility mode?
Furthermore, searching in esp forums I see that other people had issues when they were having a backtick/apostrophe in their hotspot name. Can that be the case for you as well?
—
Reply to this email directly, view it on GitHubhttps://github.com/u-blox/XPLR-HPG-software/issues/1#issuecomment-1623154926, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A27SF4RVEVWYRIPEPA5FHPLXOZULDANCNFSM6AAAAAAZ7MFXEA.
You are receiving this because you were mentioned.Message ID: @.***>

I figured it out.  It was user malfunction on my part.
I realized that my PC was reconnecting to my home WiFi network and not to the hot spot.
Once I forced my PC to join the hot spot, I was able to get it working on my iphone.
Sorry for the false alarm, but thanks a bunch for the really quick responses.
I'll keep playing with the kit and let you know what else I find.
Steve
From: Michael Ammann @.>
Sent: Thursday, July 6, 2023 11:38 AM
To: u-blox/XPLR-HPG-software @.>
Cc: Steve Brown @.>; Mention @.>
Subject: Re: [u-blox/XPLR-HPG-software] Captive portal not working with iphone hotspots (Issue #1)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
**** This is an EXTERNAL email. It was sent from outside of u-blox. ****
I would suggest to monitor the NINA serial port from the laptop to see if the device connects to the hotspot of the phone after you entered the wifi credentials using the captive portal.
Likely you should see there some printfs indicating state changes or progress. Providing a log with this serial output could give you a clue or help the team to know what is going on.

Reply to this email directly, view it on GitHubhttps://github.com/u-blox/XPLR-HPG-software/issues/1#issuecomment-1623982490, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A27SF4TKPGCG6G6H2ADI4L3XO3SUZANCNFSM6AAAAAAZ7MFXEA.
You are receiving this because you were mentioned.Message ID: @.***>

Great! Closing this issue


Add Datamodel spec 2.0.0

To convert the original spec to reStructuredText, I downloaded a .docx of the original and used pandoc:
pandoc --extract-media . spec.docx -o spec.md

And then made some small tweaks by hand.

Does the build error msg mean anything?

Yes, it sort of does.  We're running sphinx-build -W, asking it to treat warnings as errors.  I did not remove a duplicate label from the new spec (the other label is in the previous version of the spec), and sphinx-build is complaining about that.


Revert "Allow combining of short boolean options (e.g. crictl ps -aq)."

This reverts commit 804971138bc9f9f7f51311ab38837e01db7c6aa9.
This has broken "crictl exec" arguments in master (shown here in CRI-O as well https://github.com/kubernetes-incubator/cri-o/pull/1457):
Mar 19 17:28:52 runcom.internal crio[25912]: time="2018-03-19
17:28:52.150599893+01:00" level=debug msg="ExecSyncRequest
&ExecSyncRequest{ContainerId:7520b,Cmd:[-c echo hello world
sh],Timeout:0,}"
You can see above the args are totally wrong. Not sure what caused this
but given the change wasn't needed let's revert and make exec work
again. Tested successfully with CRI-O.
@feiskyer @mrunalp @umohnani8 PTAL
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

LGTM

LGTM

@Random-Liu PTAL

/lgtm

So what was the exec command exactly? As part of https://github.com/kubernetes-incubator/cri-tools/pull/264 I've rebased vendor/github.com/urfave so it's possible that it has changed the behaviour and the breakage is not due to the UseShortOptionHandling: true change per se.
But to investigate I'd need to know how to reproduce.

It's not related to UseShortOptionHandling indeed. Something else. You can reproduce by running cri-o and issuing the following
crictl exec --sync CNTID sh -c "echo hello world"
You'll get the same log as above, which is clearly messing up with arguments order.


Bad interaction of variable and induction

variable (xs : List Nat)

theorem foo : xs = xs := by
  induction xs

    unsolved goals
    case nil
    ⊢ [] = []

    case cons
    head✝ : Nat
    tail✝ : List Nat
     : tail✝ = tail✝ → tail✝ = tail✝
    ⊢ head✝ :: tail✝ = head✝ :: tail✝

Note the type of the (unnamed) IH. Its binder is in fact a BinderInfo.auxDecl, so the recursion support seems to be involved as well.

variable (xs : List Nat)

theorem foo : xs = xs := by
  induction xs <;> rfl

    fail to show termination for
      foo
    with errors
    structural recursion cannot be used

    well founded recursion has not been implemented yet

The following example produces the same error
variable (xs : List Nat)

theorem foo : xs = xs := by
  cases xs with
  | nil        => rfl
  | cons x xs  => rfl

induction and cases use the "revert--intro" idiom. Here, they are reverting the auxDecl foo : xs = xs.
We can "see" what is going on when we use the following options
variable (xs : List Nat)

set_option pp.auxDecls true
set_option trace.Meta.Tactic true
theorem foo : xs = xs := by
  cases xs with
  | nil        => rfl
  | cons x xs  => rfl

Lean 3 has a similar problem, but it is harder to trigger it because these auxiliary declarations are added only when we use equations. Recall that in Lean 4, all definitions are assumed to be potentially recursive. Here is the Lean 3 version
variable (xs : list nat)

def foo : unit -> xs = xs
  | _ := by cases xs; refl. -- `cases` produces an error message saying `foo` cannot be reverted.

I think the Lean 3 "error-message" approach doesn't work for Lean 4 because we always create these auxDecls in Lean4. The best solution I found so far is to clear any auxDecls that depends on variables being reverted.


MB-8924 TOO can request a reweigh for a shipment frontend

Description
Added the Button and modal for reweighs, reconfigured the weight display to be two boxes instead of one.
Reviewer Notes
There are still a few outstanding questions about the behavior of the "Request reweigh" button after the reweigh is requested.
Setup
Make sure data is up to date:
make db_dev_e2e_populate

Build and run the office app:
make server_run
make office_client_run

To check storybook changes locally:
make storybook

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots

Messages

:book:
:link: MB-8924

  Generated by :no_entry_sign: dangerJS against dfa5a03af40c1878d6f149627a005935c2fa8d6e