Pathfinder

ref #157

@miclegr Thank you for contributing, this looks like it will be a valuable contribution to the library. Could you please rebase main onto your branch? When you checked out to create your branch the CI tests were not properly configured and thus are not running.

sure, did the rebasing now. Please note that when it comes to tests there's the problem I've outlined in my comment in #157 :

Unfortunately the L-BFGS optimizer is not working well when optimizing model's (negative) log-lilekihood while working in jax's default float32 mode, usually convergence fails. I've noticed that by simply dividing model's likelihood by number of observations (hence optimizing model's average likelihood) optimization converges. Unfortunately given blackjax design it's quite unnatural to ask for average likelihood. So in the end it's recommended to turn on double precision mode

A consequence of this is that, for running pathfinder tests, double precision mode is needed. Since double precision mode needs to be set at jax initialization time (see here), test suite should support some test in float32 mode and some in float64 mode (e.g. by running them in separate processes). It's feasible (see here for example) but not implemented at the moment. Right know i've just set the float64 mode for that tests: https://github.com/miclegr/blackjax/blob/128ce00bd2b28e06f79d126c9d3c097b6378ccc8/tests/test_pathfinder.py#L2-L3 But this wont work when running the full suite of tests. Happy to spend some time to get the multi float mode tests in place, if that's the solution.

Tentative feature request to have L-BFGS optimization path exposed directly in jax, without the code duplication of this pull request: google/jax#10243

Per discussion in https://github.com/google/jax/issues/10243, seems it is a better practice to use JAXopt - could you update the code and add a JAXopt dependency?

Yep, I'm going to start working on that. Do you prefer JAXopt as a full requirement (i.e. as an entry in requirements.txt) or as an additional one (i.e. import within a try/except)?

let's make it a full requirement - I imagine we will use it in other optimization related inference method

@miclegr is there anything we can do to help?

Hi! I was finally able to complete the refactoring to make the code use jaxopt instead of jax.scipy.optimize.
I've also took some time to make it work as much as possible with float32, by making ad ad-hoc adjustment to the stepsize of the LBFGS during training: https://github.com/miclegr/blackjax/blob/pathfinder/blackjax/vi/pathfinder.py#L290-L295
Now the tests run fine without the need to turn on the float64 mode, also the notebook :) So it should be mergeable.
Happy to do any remaing work
Michele

Great work @miclegr! I made some minor clean up - will merge once test pass.

Thanks! and thanks for all the great work at pymc and tensorflow_probability 😉


Hide context menu if anonymous user clicks on it

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/21129
Tests

Open a public room as an unauthenticated user (https://staging.new.expensify.com/r/8813851443373010)
Right-click on a comment to display the context menu
Click on "Reply in thread"

[x] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

Same stets from Test seciton

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/6829422/4bba4e83-5cf2-4621-971f-0704695daced

Mobile Web - Chrome

https://github.com/Expensify/App/assets/6829422/d0a9037b-d23e-4bc0-8e5e-19963018f029

Mobile Web - Safari

https://github.com/Expensify/App/assets/6829422/31ca5efe-c0a5-473e-baa5-a304229c546a

Desktop

N/A (anonymous users on desktop not supported)

iOS

https://github.com/Expensify/App/assets/6829422/05274a5e-fedb-482e-98b4-fe624ec22caa

Android

https://github.com/Expensify/App/assets/6829422/737f4b1b-cee0-4f1d-8028-c2e0152f9a0b

Hi @fedirjh, yeah most of them. I think with the exception of the emojis which open other context menu. But since those actions are not allowed for anonymous users, all of them at the end should redirect to the sign in page.

Have you found an exception?

Thanks @fedirjh! I will ask internally for a final review

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/mountiny in version: 1.3.30-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.30-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


How to integrate Google Analytics or Vercel Analytics?

I couldn't find any related documentation.

Very interesting. We were just discussing analytics internally.
https://vercel.com/docs/concepts/analytics/audiences/quickstart
Has some info on how to set it up. We have not yet upgraded to Svelte-Kit 1.0 (although we are close!) which means the instructions need to change slightly.
rather than using ˋ+layout.sveltefile, you should use a__layout.sveltefile inside yourpages` directory.
<script>
  import { dev } from '$app/environment';
  import { inject } from '@vercel/analytics';

  inject({ mode: dev ? 'development' : 'production' });

</script>

you will probably want to copy the default ˋ__layout.svelte` file as indicated in the docs here: https://docs.evidence.dev/features/advanced/custom-styles-and-themes#page-layout

@archiewood thanks a lot, I will try it.

@hooopo -  closing this for now - feel free to reopen if you need more guidance


No module named 'PyQt5.sip' error when running executable file.

Version Info:
Nuitka: 0.5.33
Python: 3.6.5 (v3.6.5:f59c0932b4, Mar 28 2018, 17:00:18) [MSC v.1900 64 bit (AMD64)]
Executable: C:\Users\Tearzz\AppData\Local\Programs\Python\Python36\python.exe
OS: Windows
Arch: x86_64
Installation Method:
Downloaded the repo from the developer branch and ran setup.py
This is how I ran Nuitka:
nuitka --standalone MobilePayAlerts.py --plugin-enable=qt-plugins --show-progress --show-scons
compile log.txt
After Nuitka compiled my script MobilePayAlerts.py, I tried running MobilePayAlerts.exe where it gave me the error
Traceback (most recent call last):
  File "D:\Projects\Python Projects\MobilePayAlerts\MPA\Qt\MobilePayAlerts.dist\MobilePayAlerts.py", line 1, in <module>
ModuleNotFoundError: No module named 'PyQt5.sip'

This is what is at the top of my file:
from PyQt5 import QtCore, QtGui, QtWidgets

Hallo Ben,
looks perfect to me. Should totally work. The only thing that strikes is that local path to Python, is  that a virtualenv of normal CPython there, or self compiled, or something?
And then the importing of "PyQt5.sip" right there, which really will not exist normally. But maybe your PyQt5 is newer than what I have seen so far. Can you do with that standard Python interpreter on the prompt, do "import PyQt5.sip" and "print(PyQt5.sip.file)" which would prove that?
I checked, the top level "sip" is included in fact, according to the log, so there is no reason to not find it.
As a workaround, you can also try then --include-module=PyQt5.sip and see if that helps. Looks like a new missing hidden dependency. I would like version information of your PyQt5 there. python -m pip freeze should give you that.
Yours,
Kay

Hi, the local path to Python was the default install directory when launching the executable setup file from their site. "C:\Users\Tearzz\AppData\Local\Programs\Python\Python36\python.exe"
PyQt5.sip
I opened the Python interpreter and ran import PyQt5.sip, no exceptions made. However, when I ran print(PyQt5.sip.file) it returned:
Traceback (most recent call last):
    File "<stdin>", line 1, in <module>
AttributeError: module 'PyQt5.sip' has no attribute 'file'

--include-module=PyQt5.sip
If I try to run Nuitka like so: nuitka --standalone MobilePayAlerts.py --include-module=PyQt5.sip --plugin-enable=qt-plugins --show-progress --show-scons
It returns: Error, 'sip.pyd' is not a proper python module name.
PyQt5 version:
If I run python -m pip freeze it returns these two:
PyQt5==5.11.2
PyQt5-sip==4.19.12

Few clarifications here:

If I recall correctly, long time ago sip used to be shipped inside the PyQt5 pypi package but this was changed recently and now sip has become an external PyQt5 dependency, so if you do for instance pip show PyQt5 you'll see:
...
Requires: PyQt5-sip <------
...

PyQt5-sip package contains basically just the sip extension module, ie:
d:\virtual_envs\py364_32\lib\site-packages\pyqt5\sip.pyd

Here in github if you want to use underscores on normal text, you need to escape them, I've noticed Kay didn't escape them couple of threads ago, he meant to write this "print(PyQt5.sip.__file__)". If you use the markdown code rendering though you don't need to escape the underscores, ie: print(PyQt5.sip.__file__).

@BenTearzz file didn't exists because that's not a module attribute, but if you use __file__ you'll get the path of the sip extension module, ie:
d:\virtual_envs\py364_32\lib\site-packages\pyqt5\sip.pyd

@BenTearzz One last advice, this report is much better than your last duplicated one but it'd definitely improve even more if you'd provided the whole information so we can reproduce these type of issues locally without any effort, that'd help to narrow it down faster. It's not necessary for you to share the whole code but just a minimal snippet of few lines so we can test the nuitka specified version on top of it.
This issue is easy enough to solve without any code snippet though but still... it'll be always very helpful to reproduce these type of things locally on top of some minimal code. That's why the template says:
*  If possible please supply a [Short, Self Contained, Correct, Example](http://sscce.org/)
   that demonstrates the issue i.e a small piece of code which reproduces
   the issue and can be run with out any other (or as few as possible)
   external dependencies.

Yours,
B.

Ooops, in my previous mail I said this issue was easy to solve but I forgot to explain the solution, my bad :/ . So yeah, right now Nuitka isn't copying the new sip.pyd living in site-packages\PyQt5\sip.pyd into your MobilePayAlerts.dist folder.
@BenTearzz Just to check it works, please copy manually the module from your\python\Lib\site-packages\PyQt5\sip.pyd  into your\path\to\MobilePayAlerts.dist\PyQt5 folder, once you've done that it should run without any problems, please let us know after this the bug has gone, of course this should be done by nuitka itself.
Yours,
B

Where it lives in PyPI, what the separation is I don't really care. The on disk situation is all that matters in a living installation, which is the input. But I think I didn't yet come across as sip below that namespace PyQt5 yet, but always top level. So the implicit dependency asks for "sip" and not "PyQt5.sip".
Regarding __file__ what @brupelo said, happens a lot to me, switched to Github as issue tracker for real only relatively recently, I would like to know if it is a real ".pyd" behind that, or just an alias, so I would appreciate your output @BenTearzz
I am not so sure if copying will be enough, as Nuitka has an internal table of modules it feels responsible for, and probably won't even try it. Of course, chances are, the fall back might consider __path__ and use that and happen to work, but bets are off there.
I will try and fix it for 0.6.0 coming this weekend hopefully.
Yours,
Kay

The old sip (ie: 4.19.8) used to live on Lib\site-packages, the newest sip versions (4.19.12) are copied inside Lib\site-packages\PyQt5, which of course, is the correct way to ship it.

@brupelo Dunno, isn't sip used by other things than PyQt5? So does every sip using package now ship its own copy then?
Anyway, got to follow their choices, not question them.

@brupelo @kayhayen running print(PyQt5.sip.__file__) does indeed print the file path which is ...\site-packages\PyQt5\sip.pyd
In the script I tried to compile I added import PyQt5.sip which seems to have fixed the issue.
I ran Nuitka like this: nuitka --standalone MobilePayAlerts.py --plugin-enable=qt-plugins --show-progress --show-scons

@BenTearzz Very good solution. What I normally suggest these days is --include-module=PyQt5.sip as the workaround. However, I appreciate if people not just do that, but also report it, so it can become baseline.
I think that concludes input needed. I will let you know once the fixes are coming.

@brupelo Dunno, isn't sip used by other things than PyQt5? So does every sip using package now ship its own copy then?

Kay, maybe I'm wrong but I recall in the past sip used to ship not only the extension module sip.pyd but also sip.exe and other files, as you probably already know, sip.exe is a similar tool than swig, in fact, if you compile from sources you'll see the whole stuff sip provides. I used sip few times in the past to create extension modules out of custom c++ code but I didn't really like it very much and nowadays I just use the good old swig :)
Although I've heard very nice things about pybind11 or clif, this last one from google... But meh, swig does the job pretty well so I'm alright :)

Anyway, got to follow their choices, not question them.

Oh, yeah, sure sure... Just provided some meta-information more as additional curiosity rather than something necessary to tackle the issue, at this point you already know how to fix this thing, so... ;)
Anyway, all clear so far, so yeah, thanks everyone.

This is now on factory http://nuitka.net/doc/factory.html and will likely be in 0.6.0 which I hope to arrive this weekend.

I've tested latest factory and now nuitka is copying correctly boh sip files site-packages\sip.pyd and site-packages\PyQt5\sip.pyd, i say correct ones cos they match the original checksum, good job.
Btw, I've been able to spot another little bug related to this issue. Take a look to this line PySidePyQtPlugin.py#L59. libraryPaths won't always return values, for instance, it can return an empty list.
If that happens you'll get a nice uncaptured crash here PySidePyQtPlugin.py#L110 because... ValueError: not enough values to unpack (expected 1, got 0). So yeah, consider yourself you've been warned ;)

Sure @brupelo but apparently although this is a list, everybody so far only ever had a single library path for Qt there. The unpacking checks it, and surely we will hear from them if it fails. Or who knows, maybe somebody who has that creates the PR, one never knows.

@kayhayen Kay, actually that's not a bad strategy at all... and no, not trying to be sarcastic or kidding about, it's true. I've seen few companies out there already applying the principle of "don't fix stuff till users/customers complain and don't waste time trying to anticipate the future being too defensive". And I must to say that motto was working pretty well for them... I guess you can add to the 2 existing python schools of "ask for forgiveness" or "ask for permission" the additional of... "wait till your customers bark about it" ;-D

So this is in the new pre-release now, closing now as release is immanent and not hotfixes will happen for 0.5.33 anymore.


Refactor AwsFog and ProvisionWorker from D

https://codeclimate.com/repos/54d4fa64695680762a0024c5/ProvisionWorker
https://codeclimate.com/repos/54d4fa64695680762a0024c5/AwsFog

 Code Climate has analyzed this pull request.

Ping @jntullo since you'd touched these most recently.

Resolves #298 and #299

@calebthompson @chriskacerguis currently resolving issues with provisioning and adding some more refactoring.

This causes provisioning to break, and is a dup ticket of #298 #299

Closing pull request.
No merge.
:-1:


Your Release Notes Seem to Have Error...

You state "If you're a Mac or Windows user, the Docker Toolbox will install Docker Machine v0.12.2 for you, alongside the latest versions of the Docker Engine, Compose and Kitematic."
In fact, when I downloaded docker toolbox from the link you provided, the command:
$ docker-machine -v
docker-machine version 0.11.0, build 5b27455
I'm trying to use the latest version so it's the only reason I caught it :)
I haven't been able to use the digital ocean driver to create a machine with the high-cpu droplets, but the regular droplet and high-memory droplet work. I am referring to the "droplet-size" environment variable. Is there coding that is needed to include this (the high-cpu is a new feature)?

Yep, we'll release an updated toolbox soon.


Automated Package Upgrade

:tada: This PR is included in version 25.6.86 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Migrate to Manifest v3

The timeline when v3 is supported in all browsers (mainly Firefox ESR) and when they plan to deprecate the current version is still not clear.
Start by updating the background script to work as non persistent, which should also improve performance.

Chrome Web Store will not support submissions of new extensions with v2 after January 17, 2022. There is no date for complete deprecation of v2, but it's definitely coming.

The v2 extensions will be deprecated in January 2023 (!), according to https://developer.chrome.com/docs/extensions/mv3/mv2-sunset/.

https://developer.chrome.com/docs/extensions/mv3/intro/
https://developer.chrome.com/docs/extensions/mv2/background_migration/

I opened a blocker on Pontoon side, because manifest v3 will remove webRequest API, that is needed to support containers (aka contextual identities) in Firefox.
However today I found (and sorry for my negligence), that supposedly Firefox will keep the webRequest API for manifest v3. If that's true, there would be no regression, because in Chrome the containers are not available anyway.

https://blog.mozilla.org/addons/2022/10/31/begin-your-mv3-migration-by-implementing-new-features-today/

Firefox is rolling our MV3 support, however it might not be on-par with what will be needed for Chrome. MV2 will still be supported for Firefox, so for now the aim is to have MV3 manifest and code ready for Chromium. https://blog.mozilla.org/addons/2022/11/17/manifest-v3-signing-available-november-21-on-firefox-nightly/
As for Chromium, they pushed back the timeline a little, to June 2023. https://developer.chrome.com/blog/more-mv2-transition/

As of early December, the whole MV2 sunset in Chrome is postponed/under review... https://developer.chrome.com/docs/extensions/mv3/mv2-sunset/

How to work with workers in Webpack - https://github.com/webpack/webpack/blob/main/examples/worker/example.js.

https://developer.chrome.com/docs/extensions/develop/migrate
https://blog.mozilla.org/addons/2024/03/13/manifest-v3-manifest-v2-march-2024-update/

https://developer.chrome.com/docs/extensions/develop/migrate/what-is-mv3


Hover over circle ratings should display the actual rating on the visualization table

Expected behaviour
When a user is viewing their reflections and hover over a circle rating, a small hover popup comes up by the mouse that displays the actual rating logged
Details and resources
Insert screenshot
Checklist

[x] This issue is linked to the appropriate project.
[x] This issue has appropriate labels.

I took this ticket as I can work on this at the same time as #377.
It will not be closed until #377 is merged to main.


files are not downloading by using dio.download on real IOS devices.

New Issue Checklist

[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. flutter / ios

Dio Version                    e.g.  3..0.8

Issue Description and Steps
The issue is that when we download the files form server those files are not getting to save to the local storage in real iOs device, I am using the iPhone 11 with the latest os version. All working fine with android and ios Simulator. problem is with the real ios devices.

Any update?

Any update?

facing same issue. Any Updates?

I think as it was not working I had used something else.
On Thu, 29 Jul, 2021, 7:00 pm Deepashree, @.***> wrote:

facing same issue. Any Updates?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutterchina/dio/issues/656#issuecomment-889145961,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIIL2MKDXWNGZSYUB6PPSTT2FJYBANCNFSM4KIVFMDQ
.

Same issue.

Please create a new issue with a reproducible example.


Post landing page in different forums and news sites

hacker news
lobster
post a medium post
jomofis.com
upwork forum

we should take a screenshot of the app and put it in the landing page to have something similar to https://travis-ci.org/ landing page.

considering this done


[refactor] Remove palo_internal_service.proto and PInternalService from code base

Remove palo_internal_service.proto and PInternalService from code base, because it is not used now
Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[x] Code refactor (Modify the code structure, format the code, etc...)
[ ] Optimization. Including functional usability improvements and performance improvements.
[ ] Dependency. Such as changes related to third-party components.
[ ] Other.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have created an issue on (Fix #ISSUE) and described the bug/feature there in detail
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If these changes need document changes, I have updated the document
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

#6306 will cause BE crash in DEBUG mode, remove this service will fix that bug.


Strange Values of OutputPower at the evening

What happened?
Mostly every evening (at around 8pm-9pm the values of "OutputPower" go high like "OutputPower: 214748364.7 W"... With my 4.5kW peak installation this might not be possible. Even though, TodayGenerateEnergy has a reasonable value of 4.1kWh.
Values have spiked since May 12th
What Stick type are you using?

Shine X

Which Inverter are you using?

 It seems to be a ModBus read error. At the moment I simply discard implausible values directly after reading.

Which firmware are you using? The output power has to be a signed 32 bit value. I think this was fixed after the last official binary firmware release.

Looks like you are using an unsigned value instead of a signed one. Maybe you want to change OUTPUT_POWER to SIZE_32BIT_S in your custom code.


image not showing

Hello ,
i have an ecommerce  website and i'm trying to share my products in facebook using your
package however when i do share a product the thumbnail  doesn't appear on facebook
it juste appears like this on facebook

You need to set og:image meta tag.

https://developers.facebook.com/docs/sharing/webmasters/images.

@nisarr thanks for answering this question.


How are plots so pretty?

Among my main reasons for sticking with Spyder 4 is its SVG-like plot quality. How's this accomplished - can it be reproduced in a Jupyter notebook or JupyterLab? Their notebook backends are nice-ish, but still far from Spyder's inline - and their inline is a smudge. I'm seeing the same graphics backend options in Preferences as in Spyder 3, yet visual quality differs significantly.
Alternatively, can Spyder's IPython console be 'used in' JupyterLab?

How's this accomplished

I don't think we do anything special about it, but @jnsebgosselin is the one who created that plugin, so he knows better.

can it be reproduced in a Jupyter notebook or JupyterLab?

Could you post screenshots of what you see in Spyder 4 and the notebook? I'm curious to see the difference between both outputs.

and their inline is a smudge

Did you customize our inline backend to get better resolution? I say it because you can do the same in the notebook, the only problem is that they don't offer graphical options for that.

I'm seeing the same graphics backend options in Preferences as in Spyder 3, yet visual quality differs significantly.

Again, could you post Spyder 3, 4 and notebook screenshots? Thanks!

Alternatively, can Spyder's IPython console be 'used in' JupyterLab?

Well, you can connect one of our IPython consoles to an already running notebook, but it can't be embedded in JupyterLab (if that's what you want).

@ccordoba12 Strange... Spyder 3's are now identical to 4's - and looking through my old screenshots, nothing particularly distinct. I might be under a false impression.
Spyder definitively stomps Jupyter (and Colab), though - here are some screenshots along them gif'd (quality degraded), and reproducing source code below:

Code:
import numpy as np
import matplotlib.pyplot as plt

def test(scaling=1):
    def _style_and_show(w, h, scaling, noticks=True):
        plt.gcf().set_size_inches(w * scaling, h * scaling)
        if noticks:
            plt.xticks([])
            plt.yticks([])
        plt.show()

    np.random.seed(1)
    x = np.random.randn(400, 400)
    for w, h, noticks in [(4, 4, False), (9, 9, True)]:
        plt.imshow(x, cmap='bwr')
        _style_and_show(w, h, scaling, noticks)

    np.random.seed(1)
    y = np.random.randn(10000)
    plt.hist(y, bins=500)
    _style_and_show(9, 4, scaling)

scaling = (5.17 / 4)  # jupyter-to-spyder
# test()         # jupyter
test(scaling)  # spyder

Note: had to rescale to match # of pixels, maybe per each using a different DPI setting. For Spyder:

And JUST as I pasted the last image, I searched "Jupyter DPI", and StackOverflow spit out the fix:
%config InlineBackend.figure_format = 'retina'

Fun times.

Fun times.

Yeah, we plan to activate that option by default in 4.2 when we detect a HiDPI display. That's the advantage of Spyder being a desktop app: we can very simply detect and set those things on behalf of the users.


feat: close BaseModal on ESC

Fixes https://github.com/snapshot-labs/snapshot/issues/3623

We only need to listen for it when modal is opened, because it is mounted anyway


Focus outside viewport leads to weird behaviour

Describe the Bug
Whenever the DOM focuses an element that is inside the viewport tree but outside of the current view, the DOM jumps there and the viewport breaks, causing zoom, position issues.
Your Example Website or App
https://codesandbox.io/p/sandbox/react-flow-focus-issue-98688l
Steps to Reproduce the Bug or Issue
There are two ways:

In the sandbox, click on the contentEditable div, and write a lot. This will expand the div and make the DOM move to the right, but the viewport doesn't move.
In the sandbox, move the canvas far left getting the node out of sight. Then press "Tab". The existing node will be focused, but the ReactFlow render is broken.

Expected behavior
I am not sure what is the expected behaviour.
Two ways come to my mind:

Make the ReactFlow renderer scroll to the position until the DOM element is in view. Like a fitView when this happens.
Don't allow the DOM to move if this happens.

Probably number 2. is not viable because of accessibility issues.
Screenshots or Videos
Two ways of reproduce this issue:

Writing until getting out of the viewport:

https://github.com/xyflow/xyflow/assets/30879716/c9b32f5e-b8fb-4a8a-a089-28861d7b4ede

Focusing with "Tab" to a node out of the viewport:

https://github.com/xyflow/xyflow/assets/30879716/ef463bad-f632-47ee-b966-c0ee32f65e6a
Platform

OS: macOS
Browser: Arc (Chromium)
Version: <IP_ADDRESS>

Additional context
No response

This is a known issue but unfortunately we haven't found a good solution that works for everyone. There are two things you can do:

set disableKeyboardA11y={false} - this disables the selection via Tab key
if your flow is a fullscreen app, you can set .react-flow__pane { position: fixed !important; } - this prevents the native browser behaviour

In my view the only solution to address this on library side is to implement the Tab selection on our own. We will have another look when v12 is launched!

Thanks!
Effectively my flow is fullscreen, and the second option worked perfectly.
Feel free to close the issue if you want :)


Distance fog isn't world-space

Godot 3.0 alpha1 official
Windows 10 64 bits
In Godot 2.1.3, distance fog was distance based, so when looking around it didn't change the color of a given object seen far away as long as you stay where you are, no matter where you look at.
In 3.0 objects in fog suddenly become more visible depending on where you look at. It looks a bit messy on some fog setups that are usually fine in distance-based versions:
https://www.youtube.com/watch?v=TwkdifkfbyE&feature=youtu.be

this is fixed now, closing


Installation issues on x86_64 Ubuntu 20.04.4

I'm trying to use brainrender to visualize Neuropixel probe tracks, but ran into several issues during installation.
I began by trying a basic installation per the documentation:
conda create -n brainrender python=3.7
conda activate brainrender
pip install brainrender
python
>> from brainrender import Scene; s = Scene()

This leads to issue #197, which is fixed by pinning vedo==2021.0.5. However, this leads to a new issue, which is an unrecognized keyword argument print_authors passed to the Atlas constructor on line 26 of atlas.py. The latest source shows that this should be handled by commit ae92a40c2769f249d8ae7a040a64d43fad949ba0, so I tried installing from source using pip install git+https://github.com/brainglobe/brainrender.git. This leads to a situation where I can import brainrender, but from brainrender import Scene fails because Python can't find the brainrender module. Even though the wheel reports building successfully and pip show brainrender shows the correct path. This issue persists also using pip install https://github.com/brainglobe/brainrender/tarball/master and pip install https://github.com/brainglobe/brainrender.git#egg=brainrender. It also occurs if I clone from source and install using pip install .
FYI, the documentation linked to above references a different, deprecated repository: https://github.com/BrancoLab/BrainRender.git. I wonder if it is also not necessary to use python 3.7 as the documentation suggests, since 3.8 is listed as valid target in both setup.py and pyproject.toml.
Also, It looks like the default PyPI release is version <IP_ADDRESS>, although for some reason this is timestamped January 19, 2022, while version <IP_ADDRESS> actually has a later release date.
Anyways, I eventually discovered with pip show -f and pip install -vvv that dist-info and tests are added to the wheel, but the actual brainrender package is not! I suspect the issue lies in line 90 of setup.py:     packages=find_packages(include=("tests",)),, which should probably include brainrender.
In the end, if I clone from source and do an editable install with pip install -e ., this works and I can finally create a Scene object. Actually, I'm having new issues with some of the provided examples clearly not rendering properly, but that may be a separate issue, and I haven't had time to look into it yet.

Hi! I am installing brainrender in the fresh environment, and cannot import it afterwards. (Couple of months ago everything was ok.)
conda create -n br python=3.8
conda activate br
pip install brainrender

>>> import brainrender
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'brainrender'

Unfortunately, I can neither import brainrender nor from brainrender import Scene when installing <IP_ADDRESS> from PyPI.
conda create -n br_pypi python=3.7
conda activate br_pypi
pip install --no-cache-dir -vvv brainrender &> br_pypi.log

br_pypi.log
However, I can now import brainrender when doing a non-editable pip install from source:
conda create -n br_gh python=3.7
conda activate br_gh
git clone https://github.com/brainglobe/brainrender.git
cd brainrender
pip install --no-cache-dir -vvv . &> br_gh.log

br_gh.log
I get the same result on Windows.

@grahamfindlay if you need it to work right now, the version <IP_ADDRESS> works for me in python 3.8 environment. I had to replace the version of vtk from vtk<=9.0.0 to vtk<=9.0.1 in setup.py, and then pip install .. Source can be downloaded from pypi https://pypi.org/project/brainrender/<IP_ADDRESS>/#files

Thanks @noisysky . I actually got the most recent version working using an editable pip install from source (pip install -e) before opening the issue 😄.  Just wanted to make sure someone was aware of the problem.

Thanks for the updates.
Something seems to have gone wrong with the latest releases. Perhaps the way the package is created is not compatible with the latest pip versions? I'll try to look into this ASAP, in the mean while pip install -e seems to work (thanks @grahamfindlay )

I've been trying to figure out what is going on here.
With the version that is currently on github I can install and use brainrender on windows. I can install it on Mac, and it all looks fine but when I try to use it it fails to find loguru or any other package used. So it finds brainrender, it's not the same as what you had @grahamfindlay  but it doesn't find other package used within brainrender even though they were install correctly and are in the environment.
I have no idea why this is happening now or how to fix it! pip install -e . also doesn't seem to work for me

Any of the above does not fix the problem for me, the error: No module named "brainrender' persists, any advice?


Correct typo in readme.md

The typo is corrected already. Thanks for your contribution. Closed the pr.


chore: update stripe webhook

[x] chore: remove preview webhook
[x] test production

Removed from Stripe dashboard.


plugins should have a description that can be used by interfaces

Added description and short_description params to plugin class
Each has default value of None so we retain backwards-compatibility
Added default values for each in the style of the default values for the other optional params in that class
Added basic unit tests for each in the style of the unit tests I saw for plugins
Added a link to the Markdown README for improved readability

This is in response to issue #81

A general comment: you made your changes on the master branch of your forked repo. This is generally discouraged with git. In the future I recommend creating a feature branch to make your changes in, and then submitting the PR from that branch instead of master. For rationale and details about how to do this, see this guide. Don't worry about it for this PR though.

Yep, understood RE PRs in branches, will do in the future.  Thanks!

Awesome, thanks for the quick updates @maxvonhippel!


feature(spa + seo): implement Events for Google Analytics

We need to make sure we are accurately tracking our traffic to openforge.io via our analytics.google.com account.
For anyone who takes on this task; I'll add them to our analytics account.
https://github.com/angulartics/angulartics2

We (@PaulPaulDevelops @daftclaud Geoff and myself) reviewed google analytics and it looks good. We are tracking the traffic currently and it appears to be displaying the correct information.
Our questions to you @jedihacks would be what specific functions would you want us to track using angularics2 that is not currently being tracked by google analytics on its own?
@PaulPaulDevelops and @daftclaud do not believe we can use that specific plugin but can mimic functionality with the existing Google Analytics tracker within the stenciljs application.

Ok - this sounds good.  I just renamed the ticket to Implement Events for Google Analytics
Can you hook up events for our current analytics implementation?  https://developers.google.com/analytics/devguides/collection/analyticsjs/events
Such as button clicks, application sliders, etc.   Let's make sure they're well named so we can track them easily.
Let's do that and then mark this as done!

@PaulPaulDevelops maybe we have a service that sends all the information to GA, then each item we want to track we just pass the parameters to this function. I believe we are currently listening for button clicks, so we can listen for others things as well. Check with Geoff and Jedi on what they want track specifically if not everything.

This is in branch 'feature/google-events'. We need to specify which events we want to track and with what parameters we send them (category, action, label, value).

The issue is backlogged, I updated ticket conversation to Teamwork Task, Closing the defect till we revisit.


(Mainnet Verification) Stats site mainnet shows an error

Description
A clear description of the bug is:
The stats site mainnet shows an error.

https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2229
Must include any relevant identifiers, like:

Network: Devnet

Verified Mainnet,

Verified


Bugs

[ ] Responsiveness
[ ]  Card Structure

I can work on the responsiveness of the project.

Will you please explain what do u mean by card structure? I would like to work on this issue.

I can work on Responsiveness of the project.

Editor page is not responsive as canvas is called from the js function.

Home Page has cards that can be improved that are not looking good.

I can work on improving card structure.

Can I work on this issue?

Can I work on this issue?

@srishtij2000  Can you please point out the bugs you wish to fix

I can work on improving the card structure

I can work on improving the card structure

It's already assigned to @rutujadhanawade for 2 more days. After that, you can contribute

Okay


⚠ Bofrost.de has degraded performance

In ecd68d3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1251 ms

Resolved: Bofrost.de performance has improved in 9a1717d after 6 minutes.


DRY up the file with reusable blocks.

Refs #79.

Am reviewing @vpetersson - I note we have a root level security: block, but also security sections at operation level - I don't think both are required...

Thanks for catching! Yes, reckless refactoring from me. Thanks for finding that. Fixing.

@vpetersson Line 66 (PATCH /product/{tea_product_identifier} requestBody could use #/schemas/compoennts/base_product_fields

An example of another area we can consider schema consolidation:
GET /product parameters could make use of allOf if we create a Schema Object with the Pagination Parameters - as these are re-used in a few places.
Thoughts @vpetersson ?


Mark failure ok when checking symfony/flex version.

What does this Pull Request do?
Big red error is scary. This lets you know it's ok.
Interested parties
@alxp

Is this section still required now that the whole of Crayfish is using Symfony?

😅 I'll try taking it out. Can you help test?

I can, but I'll also ask @alxp to confirm what the behavior he was experiencing originally, 'cause I think he added this.

Already done (i.e. the whole section remove) by @whikloj . Thanks!


Download spk button doesn't actually set filename

So I noticed, the download spk feature is written to rename the spk to something reasonable, via the download attribute on the a tag: https://github.com/sandstorm-io/sandstorm-app-market/commit/ea4399471198f48ae2f09b397a9103477f6b9bf4
However, interestingly... it doesn't. I am assuming the issue is that we have a link, and then inside it, is a div button thing, and we're hitting the div button, not the original link?

It has nothing to do with the div, if you remove it the behavior is the same... I think the root of it is here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a where it says "download only works for same-origin URLs, or the blob: and data: schemes."
app-index.sandstorm.io and apps.sandstorm.io are different origins, which is why that attribute doesn't work.


Updated documentation and examples

Updated userGuide for new features
Updated AddCommand's example to include website
Updated StatusBarFooter to show persons when starting app (before it shows only after the address book does something to a person)
Updated EditCommandParserTest to include website

Coverage increased (+0.006%) to 89.152% when pulling d7b54f082bf24ed48b6e6c43fdc7820e1298572f on Jemereny:master into a5b97ac9b715295c82a7aee2a71a553eb372a9cb on CS2103AUG2017-W10-B3:master.

Coverage increased (+0.006%) to 89.152% when pulling 4d16cb66e6183725b7fd52927941bfe21e64a775 on Jemereny:master into a5b97ac9b715295c82a7aee2a71a553eb372a9cb on CS2103AUG2017-W10-B3:master.

Coverage increased (+0.006%) to 89.152% when pulling f28da8a90011bebe4f62050071730befcbfb3a53 on Jemereny:master into a5b97ac9b715295c82a7aee2a71a553eb372a9cb on CS2103AUG2017-W10-B3:master.

Coverage increased (+0.006%) to 89.152% when pulling f28da8a90011bebe4f62050071730befcbfb3a53 on Jemereny:master into a5b97ac9b715295c82a7aee2a71a553eb372a9cb on CS2103AUG2017-W10-B3:master.

All comments have been addressed and fixed. I will merge this pr.


Deprovisioning?

The article mentions deprovisioning at the top, but then doesn't have anything down in the tutorial. When exploring the configuration screens for this application, I can't find anything that would indicate deprovisioning is supported... Is the article wrong? Or am I missing something?

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

ID: fe2af6f9-785f-8bcc-0afb-3e7398b1eb56
Version Independent ID: e045f8bd-0ba2-27dd-1a23-3383b2463adf
Content: Tutorial: Configure Zendesk for automatic user provisioning with Azure Active Directory
Content Source: articles/active-directory/saas-apps/zendesk-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @zhchia
Microsoft Alias: jeedes

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

@JamesTran-MSFT I think I answered my own question. Deprovisioning seems to automatically occur as part of removing the user from the application. I suppose this could have been inferred based on the summary at the top of the article. Perhaps just adding a section towards the end stating this would be helpful to others. Thanks!

@bretcowie
Thank you for the quick follow up and I'm glad that you were able to resolve your issue!
I've created a PR to update the documentation and it's currently waiting for the author(@zchia)  to review and sign off.
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!


Yeoman 1.1

CHECKLIST
We will not consider a PR untill all of the following items are checked off--thank you!

[x] There aren't existing pull requests attempting to address the issue mentioned here
[x] Submission developed in a feature branch--not master
[x] Confirmed there are no outstanding npm run eslint issues
[x] Added/updated tests for new/changed code

TIP: You can watch/run a specific test during developemnt with npm run test:watchone test/TEST-FILE-NAME

[x] Appropriate coverage validated with npm run cover:local
[x] All npm test passes without any issues

CONVINCING DESCRIPTION
Currently code is using an older version of the core generator code. Since 1.0 was released and the project is now honoring semver this is a good idea. Also a bunch of deprecated stuff was removed prior to the 1.0 bump, so we need to make sure everything works.
Documentation on upgrading is here:
http://yeoman.io/blog/hello-generator-1.0.html
RELATED INFORMATION
Fixes #157

Codecov Report

Merging #159 into master will decrease coverage by 0.16%.
The diff coverage is 93.02%.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
- Coverage   96.71%   96.55%   -0.17%
==========================================
  Files          15       15
  Lines        1736     1742       +6
  Branches      313      316       +3
==========================================
+ Hits         1679     1682       +3
- Misses         57       60       +3

Impacted Files
Coverage Δ

generators/source-selecting-subgenerator.js
84.33% <ø> (ø)
:arrow_up:

generators/amp-add-local/index.js
97.46% <ø> (ø)
:arrow_up:

generators/model/index.js
100% <ø> (ø)
:arrow_up:

generators/behavior/index.js
100% <ø> (ø)
:arrow_up:

generators/subgenerator.js
85.71% <ø> (-0.5%)
:arrow_down:

generators/action/index.js
100% <ø> (ø)
:arrow_up:

generators/amp-add-common/index.js
83.6% <100%> (+0.27%)
:arrow_up:

generators/app/index.js
97.32% <100%> (ø)
:arrow_up:

generators/base-generator.js
100% <100%> (ø)
:arrow_up:

generators/common/sdk-versions.js
99.2% <83.33%> (-0.8%)
:arrow_down:

... and 1 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 d4d8285...fe50ea6. Read the comment docs.


Cross-site scripting vulnerability

Bug report
Relational columns (select) does not escape html which allows scripts to be executed!
https://github.com/Laravel-Backpack/CRUD/blob/master/src/resources/views/columns/select.blade.php
What I did:
$this->crud->setColumnDetails('team_id', [
    'label' => 'Team',
    'type' => "select",
    'name' => 'team_id',
    'entity' => 'team',
    'attribute' => "name",
    'model' => "App\Team",
]);

What I expected to happen:
The column should escape all html, and render it as text.
What happened:
If the team name contains html and javascript, e.g:

Then the alert will pop up if that column is loaded.
What I've already tried to fix it:
src/resources/views/columns/select.blade.php

    <?php
        $attributes = $crud->getModelAttributeFromRelation($entry, $column['entity'], $column['attribute']);
        if (count($attributes)) {
-            echo implode(', ', $attributes);
+            echo htmlspecialchars(implode(', ', $attributes));
        } else {
            echo '-';
        }
    ?>

Backpack, Laravel, PHP, DB version:
{
    "backpack/base": "^0.9.0",
    "backpack/crud": "^3.4",
    "laravel/framework": "5.5.*"
}

PHP 7.2.3 (cli) (built: Mar  1 2018 16:50:09) ( NTS )

5.7.21 - MySQL Community Server (GPL)

Hi @noh4ck ,
Thanks for revealing this.
I do agree with the small change, since I don't see anybody using the name to store HTML.
But IMHO, the


allow additional pre-stitched image file extensions

can we allow the following file extensions as pre-stitched images in the registration folder to proceed through the pipeline, such as .btf, .nd2, and  .tif? These are straight from the microscope vendor's software (Nikon and Olympus).

and .czi too

Can all three tools (UnMicst, Coreograph and ilastik) read those formats?

#133 added these here: https://github.com/labsyspharm/mcmicro-nf/blob/726e59c649876b41cda6a441072cb273d116c4ab/main.nf#L78-L82
The files still need to go through ASHLAR to be converted to .ome.tif to all downstream modules. Using ASHLAR as a converter in this way is easier than modifying Coreograph + UnMicst + ilastik to be able to work with all the requested formats.


[communication]-[sms]-Fixes mypy type errors.

Description
Fixed mypy type errors. Closes #32842 .
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run python - azure-communication-sms - tests

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-sms


Add a Torque scheduler backend

The current PBS backend targets essentially PBS pro. FYI @ZQyou.

@smoors is working on a Torque backend. @ZQyou, since you have done some local modifications in our PBS backend (which is actually tailored to PBS pro rather than Torque), you two may want to sync.


Object doesn't support property or method 'assign' on IE11

I tried to use CoreUI-Vue PRO 3.0 beta-1 with Internet Explorer 11 but the page doesn't show up.
Even your demo page doesn't show up.
The web console prints out "SCRIPT438: Object doesn't support property or method 'assign'"
`

Hi! thank you for reporting this issue! It was really helpful because ie11 was working before, and I didn't check it after some breaking-ie11 dependencies have been introduced.
The next release will be fixed. For a quick fix:
In package.json
    "@coreui/vue": "coreui/coreui-vue#3.0.0-beta.7",
    "@coreui/vue-chartjs": "coreui/coreui-vue#1.0.3",

then run npm i
in babel.config.js
module.exports = {
  presets: [
    [
      '@babel/preset-env',
      {
        useBuiltIns: 'entry',
        corejs: 3
      }
    ]
  ]
}

in vue.config.js
module.exports = {
  lintOnSave: false,
  runtimeCompiler: true,
  configureWebpack: {
    //Necessary to run npm link https://webpack.js.org/configuration/resolve/#resolve-symlinks
    resolve: {
       symlinks: false
    }
  },
  transpileDependencies: [
    '@coreui/utils'
  ]
}

in main.js add
import 'core-js/stable'


replace owncloud/davclient.js with webdav

Description
replace owncloud/davclient.js with webdav
Related Issue https://github.com/owncloud/owncloud-sdk/issues/712
Related https://github.com/owncloud/owncloud-sdk/issues/705
This PR replaces owncloud/davclient.js with perry-mitchell/webdav-client. This makes owncloud-sdk compatible with nodejs.
Changes -

[x] - Import the code related to making dav request and parsing dav response into new Dav class from https://github.com/owncloud/davclient.js/blob/master/lib/client.js
[x] - Replace the use of https://github.com/owncloud/davclient.js with the newly created Dav class
[x] - Unskip and adjust all the tests so that they pass

TODO

[ ] - It Still doesn't work with web fully, make all the tests pass on web (https://github.com/owncloud/web/pull/4738)
- this will probably require code changes on web
In web folders are redered as files, this may be because the d:resoucetype property used to be an array of xmlNode (eg. [d:collection] where d:collection is xmlNode), but it now returns it as array of string (eg. ["d:collection"]) this needs to be handled in web. There may also be other errors that needs to be checked.

@LukasHirt Looks like everything is working now. I tested it on web https://github.com/owncloud/web/pull/4738
All tests are passing on web. I mostly just copied the code from owncloud/davclient.js you may find some improvements there.
can you review when you have time?

@kulmann I've fixed everything can you review it. Also all tests pass on web https://github.com/owncloud/web/pull/4738

Thanks a lot @kulmann, I've rebased, will merge when CI is green

I thought that if already passing and approved provider tests would start failing CI would fail, but that seems not to be the case

@individual-it I have confirmed that all the failures on oc10 provider tests exist on the previously skipped tests.
There was some failure on public link tests which i have fixed on https://github.com/owncloud/owncloud-sdk/pull/735/commits/c024777722794bd004adfb4e368e15004adfe38b
They are now down to 73 warnings. Can you review and merge now?


Installer isnt opening on windows 10

when attempting to run the installer.exe it doesnt do anything, the installer log says the following
================================================================================
MPPatch version 0.1.3
Revision 8adb8b73, built on May 8, 2021 by jenkins@himawari
Build ID   : 3e5748f5-a000-46d0-87d3-cd86ca811320
Build Date : 2021-05-08T12:40:07Z
Revision   : 8adb8b735e884f26ae8e750339bbf213f8b556a4
Tree Status: clean
[2023-11-29T19:24:45Z/ INFO] Changing patch package...

Never mind i got it to work by running installer.jar instead of the exe

Likely duplicate of one of the many bugs caused by launch4j. Will be fixed in the next version.


Add gulp-retina-sprite to blackList

missing documentation

did you open an issue on their repo for this?

Done. I agree we should do this but for other additions we didn't do that too. https://github.com/gulpjs/plugins/commit/77fecf805a5b61f007755df2837fa4f902fad0fa for example, no issue were opened.


Broadcast not starting (iOS)

I am using iPhone 11 iOS version is 13.5.1
When I try to broadcast, RPSystemBroadcastPickerView is displayed, on tapping 'Start Recording' it doesn't ask for channel name or anything.
I think it's the same issue like #181 . Something related to OS version

Are you still have a problem?

If you want to broadcast you need to use app group for send channel name from main program to extension

`class SampleHandler: RPBroadcastSampleHandler {
var sendAudioChecker = true

override func broadcastStarted(withSetupInfo setupInfo: [String : NSObject]?) {

    let defaults = UserDefaults(suiteName: "group.com.blablabla.Api")
    let IDS = defaults?.string(forKey: "API")

    if let setupInfo = setupInfo, let channel = setupInfo["channelName"] as? String {
        //In-App Screen Capture
        AgoraUploader.startBroadcast(to: channel)
    } else {
        //iOS Screen Record and Broadcast
        print(IDS!)
        print("start broadcast")
    AgoraUploader.startBroadcast(to:IDS!)
    }
}`

Hi thank you for reply.
override func broadcastStarted(withSetupInfo setupInfo: [String : NSObject]?) { }
If I'm not wrong this function is automatically triggered when user taps on start recording, but in my case this function is not triggering. I've added breakpoints in it but it never gets called

when I try the sample code it not work too I use app group and start broadcast by use `  if #available(iOS 12.0, *) {
let broadcastPicker = RPSystemBroadcastPickerView(frame: CGRect(x: 5, y: 5, width: 40, height: 40))
broadcastPicker.preferredExtension = "com.blablabla.Broadcast"
                          ShareScreen.addSubview(broadcastPicker)

                      } else {
                          // Fallback on earlier versions
                      }`

and then use app group to transfer channel name to broadcast extension and start broadcast by
`override func broadcastStarted(withSetupInfo setupInfo: [String : NSObject]?) {
let defaults = UserDefaults(suiteName: "group.com.blablabla.Api")
let IDS = defaults?.string(forKey: "API")

if let setupInfo = setupInfo, let channel = setupInfo["channelName"] as? String {
    //In-App Screen Capture
    AgoraUploader.startBroadcast(to: channel)
} else {
    //iOS Screen Record and Broadcast
    print(IDS!)
    print("start broadcast")
AgoraUploader.startBroadcast(to:IDS!)
}

}`

Ok. Let me give it a try

Ok 👍

I've tried following things:

In viewDidLoad of BroadcastViewController I added:

let userDefaults = UserDefaults(suiteName: "group.com.agora.ios")! userDefaults.set("12345", forKey: "API")  userDefaults.synchronize()

In broadcastStarted of SampleHandler I added:

let defaults = UserDefaults(suiteName: "group.com.agora.ios")         let IDS = defaults?.string(forKey: "API")         print(IDS!)         if let setupInfo = setupInfo, let channel = setupInfo["channelName"] as? String {             //In-App Screen Capture             AgoraUploader.startBroadcast(to: channel)         } else {             //iOS Screen Record and Broadcast             print(IDS!)             print("start broadcast")             AgoraUploader.startBroadcast(to:IDS!)         }
but it's still the same. in this case channel name is 12345 and I tried to view the livestream with this channel name but there was nothing.

Yes I have.

So in ### agoraEngine.JoinChannel token was set to nil, I changed it to the token from KeyCenter but still nothing.
Maybe it has started uploading the broadcast, but in AudienceViewController VideoSession is nil

maybe you can try creating a new project w/o token and see if it's working for you or not? just for experiments

Did that. Still the same

Ok. Apparently creating a project without token worked. Thank you

@saimt to help you understand what is happening
https://github.com/AgoraIO/Advanced-Video/blob/83674ba810c081e71214a91652a7c9eaffdd84b7/iOS%26macOS/Agora-Screen-Sharing/Agora-Screen-Sharing-iOS-Broadcast/AgoraUploader.swift#L59
you will need to provide a token here, and the channel you used to generate the token should match with the name of the channel which you are joining.

Hi, Yes I did that but it didn't work. Earlier on Agora dashboard, I created a project with App ID + App Certificate + Token which wasn't working. Now I created the project with App ID only. So I changed the AppID in KeyCenter and it started working.

if it works with an AppID only project, it can almost be certain that your issue is caused by token authentication.
you can create a event listener and set it as sdk delegate here
https://github.com/AgoraIO/Advanced-Video/blob/83674ba810c081e71214a91652a7c9eaffdd84b7/iOS%26macOS/Agora-Screen-Sharing/Agora-Screen-Sharing-iOS-Broadcast/AgoraUploader.swift#L32
so that you can listen to onError events to check out what kind of error code you are having when joining channel.


I will push another commit here to bump the commit id again.

done - this PR can be merged when CI goes green

done - this PR can be merged when CI goes green

I forced-pushed the last commit again to trigger CI.

I forced-pushed the last commit again to trigger CI.


CSRF cleanup and fixes

Use itsdangerous to sign tokens and check expiration, instead of rolling our own.

All tokens are url safe, essentially reverting #206.  There is no longer a url_safe parameter. cc @twolfson
All tokens store a timstamp, which is checked in validate_csrf.  There is no longer a time_limit parameter to generate_csrf.

Fix anti-pattern of storing app on extension, use current_app when needed.
Add the DELETE method as protected by default.
Generate the same signed token over the lifetime of a request. Exposed as request.csrf_token. (#227)
@csrf.error_handler is deprecated in favor of the standard Flask error system @app.errorhandler(CSRFError).

decorator is "fixed" so that it handles both raising an exception or returning a response.  A returned response is converted into a raised exception, which is handled by Flask.
addresses #200, #209, #243, #252

I'm no longer using this package but /cc @brettlangdon . Thanks for the heads up @davidism =)

Is there any estimate for this one appear in a new release?

Hi, i'm upgrading to the v0.14 from the v0.12, i was using the parameter time_limit on my forms because i need to use different level of csrf protection.
I saw that with this version the parameter is gone from generate_csrf, is there another way to set it?

class MyForm(FlaskForm):
    class Meta:
        csrf_time_limit = 60  # seconds

# or

form = SomeForm(meta={'csrf_time_limit': 60})

For some reason the method  validate_csrf_token of the class _FlaskFormCSRF is not called, validate_csrf  of the module csrf.py is called instead, and the value of time_limit is always None.

RuntimeError
RuntimeError: A secret key is required to use CSRF.
Even though my secret key is set I'm still getting this error.
my code: app.config['SECRET KEY'] = '31b5a797ef10c9327ab35f66d9428ca23bcd9b9db887e47f5658a0b5c6caad4a'


[Backport release-23.05] thunderbirdPackages.thunderbird-115: init at 115.0

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Perhaps wait with switching the thunderbird-bin default on 23.05?  The older one will still supported for more than a month, and we haven't had that much testing so far.


added tests and fix cli css style selector lowercase problem

#341 #434 ...by setting svg style content to commandline css content, if available -- this overwrites cloneCssStyles
didn't change mermaidAPI util cloneCssStyles -- pending refactoring (branch refactor_mermaidAPI_render)

Great! Mermaid is really benefiting form your work.


Bug with verification of users

using command twitch user <not twitch-verified user> shows that this user has verification.
Not for all, seems when streamer is Affilative partner

But

Will fix in latest patch


Color blindness emulation

An initial attempt at adding 'color blindness emulation' to the editor. To use it add a query string param color-blindness-emulation with one of the following values. Note the links point to the branch build of the editor for testing.

protanopia
protanomaly
deuteranopia
deuteranomaly
tritanopia
tritanomaly
achromatopsia
achromatomaly

This is the first stage, if it's correct we'll merge and work out a way to create a UI
Note: Most of the credit goes to https://github.com/hail2u/color-blindness-emulation

First up, here is a collection of tools to test against and reference implementations

https://gist.github.com/Lokno/df7c3bfdc9ad32558bb7
https://github.com/mlaily/NegativeScreen/issues/9
https://gist.github.com/dannyid/c45be2b9d695ac0d9120
https://github.com/MaPePeR/jsColorblindSimulator

To test I installed Color Oracle however taking a screenshot with Color Oracle enabled seemed to mess with the colors. I've manually tested with Color Oracle comparing with the osm-liberty style. This wasn't really a good enough test because it was only a visual (side by side) check. But it seemed correct.
Secondly I tested with the http://mapeper.github.io/jsColorblindSimulator/ uploading a screenshot  of the editor, the results were as follows

achromatomaly - very slight color variation
achromatopsia - very slight color variation
deuteranomaly - very slight color variation
deuteranopia - same
protanomaly - same
protanopia - same
tritanomaly - same
tritanopia - very slight color variation

There was very slight color variations in some modes which could be to do with the generation of the screenshots, or some kind of compression. Either way the variation was barely noticeable in the modes highlighted above.
I've also checked the matrixes against other implementations and they are the same.
I believe this to be good to merge.


ci: configure the 2.22.x protected branch

Configures CI for 2.22.x branch

Both "Kokoro - Test: Downstream" are not required or expected to work for the LTS branch.


Coveralls is ignoring unit test

You need to add configuration to your gradle build to generate report for both androidTests and unit tests.

Fixed in release 0.1.1


Sparse Methods Missing from 0.3.0

What happened + What you expected to happen
I attempted to import the BottomUpSparse and MinTraceSparse methods from hierarchicalforecast.methods
However, upon inspection of the class, I noticed that release 0.3.0 doesn't contain either of these sparse methods:
NAME
hierarchicalforecast.methods - # AUTOGENERATED! DO NOT EDIT! File to edit: ../nbs/methods.ipynb.
CLASSES
HReconciler(builtins.object)
BottomUp
ERM
MiddleOut
MinTrace
OptimalCombination
TopDown
I used the standard pip install command: pip install hierarchicalforecast
Is there a different version that I should install to access these sparse methods?
Versions / Dependencies

dateutil                    2.8.2
hierarchicalforecast        0.3.0
matplotlib                  3.7.1
numpy                       1.23.5
pandas                      2.0.2
session_info                1.0.0
statsforecast               1.6.0

IPython             8.14.0
jupyter_client      8.2.0
jupyter_core        5.3.0
notebook            6.5.4
Python 3.10.11 (main, Apr 20 2023, 19:02:41) [GCC 11.2.0]
Linux-4.18.0-3<IP_ADDRESS>.1.el8_6.x86_64-x86_64-with-glibc2.35
Reproduction script
from hierarchicalforecast.methods import BottomUp, TopDown, MinTrace, ERM, OptimalCombination, MiddleOut, BottomUpSparse, MinTraceSparse
Issue Severity
Low: It annoys or frustrates me.

Hey @tbellamey, we just released 0.4.0. You should be able to use those methods if you upgrade your installation.

Hi @jmoralez , I upgraded my installation to 0.4.0. However, upon running my script (no code changed), I am now getting the below error. This seems to be inducing an error in using StatsForecast AutoETS models now. I also tried StatsForecast HoltWinters and received the same error.
This error did not raise with the same dataset & script when running 0.3.0 - any ideas what might have changed the behavior from 0.3.0 to 0.4.0? I can also open this as a new issue for tracking.

NotImplementedError                       Traceback (most recent call last)
Cell In[8], line 109
98 #valid_agg_reset = valid_agg.reset_index()
100 model = StatsForecast(models=[
102                           AutoETS(season_length=12,model='AAA',alias='AutoETS_AAA')
(...)
107                          ],
108                           freq='MS', n_jobs=1, verbose=True)
--> 109 model.fit(train_agg)
111 p = model.forecast(h=h_months, fitted=True)
112 p_fitted = model.forecast_fitted_values()
File ~/lib/python3.10/site-packages/statsforecast/core.py:880, in StatsForecast.fit(self, df, sort_df, prediction_intervals)
878 self.prepare_fit(df, sort_df)
879 if self.n_jobs == 1:
--> 880     self.fitted = self.ga.fit(models=self.models)
881 else:
882     self.fitted = self._fit_parallel()
File ~/lib/python3.10/site-packages/statsforecast/core.py:77, in GroupedArray.fit(self, models)
75     for i_model, model in enumerate(models):
76         new_model = model.new()
---> 77         fm[i, i_model] = new_model.fit(y=y, X=X)
78 return fm
File ~/lib/python3.10/site-packages/statsforecast/models.py:650, in AutoETS.fit(self, y, X)
628 def fit(
629     self,
630     y: np.ndarray,
631     X: Optional[np.ndarray] = None,
632 ):
633     """Fit the Exponential Smoothing model.
634
635     Fit an Exponential Smoothing model to a time series (numpy array) y
(...)
648         Exponential Smoothing fitted model.
649     """
--> 650     self.model_ = ets_f(
651         y, m=self.season_length, model=self.model, damped=self.damped
652     )
653     self.model_["actual_residuals"] = y - self.model_["fitted"]
654     self._store_cs(y=y, X=X)
File ~/lib/python3.10/site-packages/statsforecast/ets.py:1241, in ets_f(y, m, model, damped, alpha, beta, gamma, phi, additive_only, blambda, biasadj, lower, upper, opt_crit, nmse, bounds, ic, restrict, allow_multiplicative_trend, use_initial_values, maxit)
1238 # ses for non-optimized tiny datasets
1239 if n <= npars + 4:
1240     # we need HoltWintersZZ function
-> 1241     raise NotImplementedError("tiny datasets")
1242 # fit model (assuming only one nonseasonal model)
1243 if errortype == "Z":
NotImplementedError: tiny datasets

Closing since the original issue is solved.


How to disable saving files to disk? (use_file_database=false don't help)

We recently updated our tdlib from version 1.8.4 to 1.8.31.
After this, session files (photos, videos, voices, etc.) began to be saved on the disk.
We pass the setTdlibParameters command with the following params:
setTdlibParameters {
  use_test_dc = false
  database_directory = "/some_directory"
  files_directory = ""
  database_encryption_key = bytes [0] { }
  use_file_database = false
  use_chat_info_database = false
  use_message_database = false
  use_secret_chats = false
  api_id = 111
  api_hash = "our_hash"
  system_language_code = "en"
  device_model = "Compute Cloud"
  system_version = ""
  application_version = "1.0"
}

With these params on v1.8.4 everything worked fine.
But after the update, files started saving to disk.
What could be the reason?

There were no changes in TDLib behavior. Files are never downloaded automatically by TDLib, they can only be downloaded by your app. Some photos can always have their smallest size, which is received along with the photo and saved on the disk. These photos can be ignored by settings the option "ignore_inline_thumbnails" (see https://core.telegram.org/tdlib/options for details).

Thanks for the reply. After that I double checked our code and noticed that we use DownloadFile method in some cases, which downloads files.
We added a periodic launch of optimizeStorage to clean up data.


Create a dummy login to display mock Classes/Finals data

This task is to create a dummy login to display mock Classes/Finals data
Notes:

The dummy User ID/Password will be determined outside of this issue
The mock data used to display Classes and Finals data should be completely scrubbed, and make no reference to any real information such as:

Student information of any kind
Class information of any kind
UCSD information of any kind

@a6wu Assigning this to you until if/when we are instructed to begin work on this. Note: This issue should be tied to the Kadabra milestone if no true AD test accounts w/ PID are to be provided prior to app submission.

Commit 12eb479 adds internal test accounts. Username is studentdemo, and password is the same.

Adds code that swaps between production and demo endpoints.

API Gateway MobileDemoMockData has been setup and verified in dev and prod. v6.0-rc endpoints updated. closing.


scripts: locked pyocd version to 0.12.0 in requirements.txt

Workaround for #11279

Codecov Report

Merging #11280 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11280   +/-   ##
=======================================
  Coverage    53.1%    53.1%
=======================================
  Files         218      218
  Lines       26887    26887
  Branches     5954     5954
=======================================
  Hits        14277    14277
  Misses      10174    10174
  Partials     2436     2436

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 45f0917...bc3714e. Read the comment docs.

1: UC6 Body has no content, should at least have 1 line.


Can not get input layer through getLayerByName()

model.getLayerByName() can not get input layer, to fix this bug, add extra check for input layer.

Do not have this bug, the scene of accident QAQ


Questions about bdd100k data preprocess mentioned in the paper

The lanes of BDD100K are labelled by two lines. Training the networks using the provided labels is tricky. Therefore, based on these two lines, we calculate the center lines as new targets

How to pair disorder lanes in the json file ?

Do you calculate the center lines of the testing set too?

Thanks a lot.

BDD100K just records the coordinates of the lane points as annotations. Hence, to calculate the centerline, you just average the coordinates of two neighboring points and the averaged value is just the coordinate of the lane point for the centerline. For the testing set, I also do the same operation. And I will upload the scripts of processing the annotations of BDD100K soon.

That's great! By the way, 8 pixels dense targets w.r.t 360x640 image size?

We dilate ground-truth lanes of the training set of BDD100K as 8 pixels to provide denser targets while
keeping these of testing set unchanged (2 pixels).

I get it , that would be 4 pixels for training set and 1 pixels for testing set while resize to 360x640.  What's more, I am confused about the difference between L_seg and L_exist loss. Do the L_seg have more num classes than  L_exist such as single dash line, double full line.

I only perform binary segmentation in BDD100K since there are so many lanes in the image. Hence, the lane existence prediction branch is removed for BDD100K.

BDD100K just records the coordinates of the lane points as annotations. Hence, to calculate the centerline, you just average the coordinates of two neighboring points and the averaged value is just the coordinate of the lane point for the centerline. For the testing set, I also do the same operation. And I will upload the scripts of processing the annotations of BDD100K soon.

Can you share the implementation details of preprocessing the lanes as the process of combining two lines to form a centerline is not clear?


Missing file: test_sqlalchemy.py

Readme, make reference file:
test_sqlalchemy.py
There is no such file.

The new way of running tests is with pytest. Feel free to send a PR to update the docs! I would use this opportunity to replace py.test with the non-deprecated version pytest in this line: https://github.com/mitsuhiko/flask-sqlalchemy/blob/master/tox.ini#L5


Can property names, esp userAssignedIdentities, be dynamic?

Using this as an example of assigning userAssignedIdentities to a resource the JSON structure looks like:
  "identity": {
    "type": "userAssigned",
    "userAssignedIdentities": {
      "/subscriptions/01234567-89AB-CDEF-0123-456789ABCDEF/resourceGroups/myResourceGroup/providers/Microsoft.ManagedIdentity/userAssignedIdentities/myID": {}
    }

Using Bicep I can use the literal value of the userAssignedIdentity like:
    identity: {
        type: 'userAssigned'
        userAssignedIdentities: {
          '/subscriptions/01234567-89AB-CDEF-0123-456789ABCDEF/resourcegroups/myResourceGroup/providers/microsoft.managedidentity/userassignedidentities/myID':{}
        }
    }

but given I've created the identity earlier in  the same bicep file using:
resource mi 'Microsoft.ManagedIdentity/userAssignedIdentities@2018-11-30'={
  name: 'myID'
  location: location
}

it seems natural to be able to reference this resource, perhaps like this:
    identity: {
        type: 'userAssigned'
        userAssignedIdentities: {
          mi.id :{}
        }
    }

However this gives a BCP018 error parsing the '.' in the property name.
Of course if I've missed a trick please advise, always learning, but hard coding the identity into my bicep file feels so wrong
Cheers

You need to use string interpolation syntax here:
    identity: {
        type: 'userAssigned'
        userAssignedIdentities: {
          '${mi.id}' :{}
        }
    }

Thanks Alex but while that looks perfect it isn't working for me :(
Using the example I get this warning in VSCode

while attempting a build give me this
/home/trevor/source/bicep/dScript.bicep(27,7) : Error BCP067: String interpolation in not supported in object keys.

I'm running Bicep CLI version 0.1.37-alpha (f88c6d95ad)

ohh, it's because you are on an old version. If you install 0.1.226 you should be good to go

OK, build now works but I'm getting errors in VS Code (launched from within WSL) but that's a different issue and I see it comes with helpful reporting tools

Sounds good - looks like I can close this


test: enable RPC tracing for integration tests

This will facilitate the debugging of flakes by giving visibility
into calls prior to the failing one.
The tracing options are left at their default values of (currently):
single_line_mode=on
use_short_repeated_primitives=on
truncate_string_field_longer_than=128

Tested by injecting an error into an integration test and observing
the error report.
Fixes #4034.

This change is

Later is fine methinks.


T2: VID use VID block commitment

Subtasks

[x] #1757
[x] #1784

Description
Currently we do not use the VID commitment:
https://github.com/EspressoSystems/HotShot/blob/2c5672285edb21fb57b69b40788f9c9e7dce6852/crates/task-impls/src/da.rs#L781-L789
Blocked by https://github.com/EspressoSystems/jellyfish/issues/369
Later we need to switch everything to use the VID commitment. cc @shenkeyao

Done in #1814


Upgrade redux to 4.0.0

Issue: #3436

LGTM, although I reckon we should just get rid of the redux store already ;)


Refactor PackingPlan proto to not serialize instance/ram dist

We should remove the instance and ram distribution from the serialized protobuf packing plan. To do this we first need to refactor the executor to not use it. This can happen once we've cut over to distributing the packing plan to the executor via the state manager.

This refactor has been made on the billg/scaling_proto branch (see this commit), so the instance dist has been removed in #1219.


solved #473   Added literate and illetate states on india map, table

Made this map and added the table
https://github.com/user-attachments/assets/2fa9866b-2dfd-42e1-842d-7cc4b1314fbe

Mam,this pr not added this to leaderboard,can I know the reason


Consider adding a command-line parameter to specify the root config file path

Can you make it so that this can be given a location where the config is.
The example use case is for ci.
given directory structure
.gitlab
markdown-folder

When I create a stage I can specify a image with this cli installed
so the config for this would be in the .gitlab folder
currently I could do:
script:
  - cd .gitlab/configfolder
  - markdownlint-cli2 ../../**

but I would prefer
script:
  - markdownlint-cli2  -c .gitlab/configfolder/conf.yml **

CLI2 explicitly reserves command line arguments for glob patterns, so I do not see how this could be done without changing that. Also, it adds code/complexity to support something that is already possible (your first example). Is there a technical reason to prefer the second form, or is this more a matter of preference?

preference and it also allows more options on how to use this cli. By relying on convention for where the config is it makes devs might have to go through more hoops to make it work in their code bases.

I'll leave this open for now for possible feedback, but it is counter to a principle of the tool and I do not expect to implement it.

Hi, I just wanted to share some feedback because it was requested in this earlier comment:

I'll leave this open for now for possible feedback, but it is counter to a principle of the tool and I do not expect to implement it.

I'm just getting started with markdownlint and I've made my own custom lint rule. Awesome tool by the way, so featureful and well documented. Thank you 👍
Now, I'm trying to figure out how I will run the linter as I go about my day-to-day development. So, I've looked into the two CLIs (markdownlint-cli and markdownlint-cli2). My plan is to run the linter manually, in an ad hoc way, when I feel like it. My plan is to not commit a markdownlint-cli2 config file into my "playground" repos because the repos exist to illustrate a point, and so the more boilerplate items (config files, CI files, boilerplate documentation files, etc) is inside of them, the more out of focus they become.
So I think I'll use markdownlint-cli instead, which from what I understand is equally well endorsed!
This GitHub issue was the piece that made it clear I should probably start with markdownlint-cli and maybe in the future graduate to markdownlint-cli2. Anyway I thought this might be useful feedback from a new markdownlint user even though I don't know what the moral of the story is. Thanks,

Very cool 👍 !
I've now moved to markdownlint-cli2 (from markdownlint-cli) and am using the new markdownlint-cli2-config command: diff.
And now I can use stuff like the pretty formatter, and I'm getting more mileage out of the tools. Works great! I never knew about "clickable links" in the terminal but that works too.

Glad to hear it, welcome to the future! :)
FYI, now that it is JSONC, this simulated comment can become a real one: https://github.com/dgroomes/my-config/blob/9e9ed0d6b37a772b59453216f73f810f4bfcb333/markdownlint/.markdownlint-cli2.jsonc#L18

Another thing to consider, instead of naming the rules like MD123, you can use the text name like "line-length" which is a little more descriptive and might avoid the need to describe each rule in the list with a comment.

Very cool, thanks for the tip and code review! I was probably thinking I would memorize the markdown lint rules numbers or something but that's silly. I'll opt for the titled/named rules instead of the MD... names!


⚠ Bofrost.de has degraded performance

In 5a90a56, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 943 ms

Resolved: Bofrost.de performance has improved in 23214f0.