Add --rackhd-sku-id support

This PR adds support for the --rackhd-sku-id flag. This flag is mutually exclusive with --rackhd-node-id, and one of these is still required.
When passed, the SKU ID is used in the PreCreate() code to get a list of nodes belonging to the SKU, and picking the first one that does not have a dockermachine tag. It then tags the node with dockermachine and currently proceeds as the driver always has.
At this point, we are not adding anything for workflows (e.g. OS installs), that's in a later PR. Nothing is done at this point to untag a node, either. But when you delete a node via docker-machine it removes the node from RackHD's inventory entirely, so it's a non-issue. I'm not sure that approach works long term, but that's a separate issue.
Might be easier to review this PR commit by commit, to see the progression. Some simple tests were added at the very end.
This should close #7 .

 Will take a deeper look later this after noon

LGTM


gazebo_ros_pkgs: 3.2.0-0 in 'crystal/distribution.yaml' [bloom]

Increasing version of package(s) in repository gazebo_ros_pkgs to 3.2.0-0:

upstream repository: https://github.com/ros-simulation/gazebo_ros_pkgs
release repository: https://github.com/ros2-gbp/gazebo_ros_pkgs-release.git
distro file: crystal/distribution.yaml
bloom version: 0.7.2
previous version for package: 3.1.0-0

gazebo_dev

No changes

gazebo_msgs
* [ros2] World plugin to get/set entity state services (#839 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/839>)
  remove status_message
* Contributors: chapulina

gazebo_plugins
* [ros2] World plugin to get/set entity state services (#839 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/839>)
  remove status_message
* [ros2] Fix diff_drive error message (#882 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/882>)
* Fix Windows conflicting macros and missing usleep (#885 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/885>)
  * Fix conflicting Windows macros and missing usleep
  * fix spacing
  * fix spacing again
  * remove lint
* gazebo_plugins: Port the gazebo_ros_p3d plugin (#845 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/845>)
  * Port the gazebo_ros_p3d plugin
  * Address most of the review feedback. A couple items remain
  * Remove the model_ member variable since it was just and alias for _parent
  * Use OnUpdate instead to get the UpdateInfo through the callback parameter
  * demo, test, and a bit more cleaning up
  * linters
* [ros2] ENABLE_DISPLAY_TESTS, and make camera tests more robust (#854 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/854>)
* Contributors: Jonathan Noyola, Michael Jeronimo, Romain Reignier, chapulina

gazebo_ros
* [ros2] Disable test_node to clean CI (#901 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/901>)
  * [ros2] Disable test_node to clean CI
  * uncrustify
* [ros2] Port time commands (pause / reset) (#866 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/866>)
* [ros2] World plugin to get/set entity state services (#839 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/839>)
  remove status_message
* Fix Windows conflicting macros and missing usleep (#885 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/885>)
  * Fix conflicting Windows macros and missing usleep
  * fix spacing
  * fix spacing again
  * remove lint
* Call rclcpp::init() only from gazebo_ros_init (#859 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/859>)
* [ros] Revert sim time test (#853 <https://github.com/ros-simulation/gazebo_ros_pkgs/issues/853>)
* Contributors: Jonathan Noyola, Tamaki Nishino, chapulina

gazebo_ros_pkgs

No changes

Thanks! I'll be sure to always schedule my PRs to right before you have coffee :smile: :coffee:

Thanks! I'll be sure to always schedule my PRs to right before you have coffee smile coffee

Best frenemy :coffee: :black_heart:


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/902 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit c2fb196:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Star surface area is too small

It's too easy to tap something around the star and instead visit the link.  It'd be good to make the tappable area around the star a little larger.  Once again, we may want to explore some other option, such as long pressing.

Should the star also expand or just the clickable area around the star?

I'm not sure, we may instead have some other visual signifier to show the tappable area.  Let's discuss with Scott first.


Updated ReadMe in "updatedREADME" branch.

The new README now includes a table of contents, better organization, images, a list of key features, a logo at the top, and some other information.

Github added a native and automatic TOC feature a while back

I recommend to close this PR and extract things that might still be useful to a separate PR.


add ability to create react query client options based on page context

Adding the ability to create react query client options based on pageContext.
My use case:
All of my backend API endpoints require a user session which is stored in a cookie. On a client side this is not a problem, but on the server side I need to somehow pass cookie value from pageContext to my fetch function. A page could contain several components which require data, and I don't want to manually put all of it to +data hook. Instead, I want to use useSuspendQuery and isolate components.
So, I decided to use meta field in react-query options and store there a request library which would encapsulate user token.
So, there is my +queryClientConfig.ts file:
import { PageContext, Config } from 'vike/types';

export default (pageContext: PageContext) => {
    return {
        defaultOptions: {
            queries: {
                meta: {
                    request: (endpoint: string) => {
                        if (import.meta.env.SSR) {
                            return import('../data/request/server.request').then(r => r.serverRequest(endpoint, pageContext.headers!.cookie));
                        }

                        return import('../data/request/browser.request').then(r => r.browserRequest(endpoint));
                    },
                },
            },
        },
    } satisfies Config['queryClientConfig'];
};

And a usage of it:
    const { data: user } = useSuspenseQuery({
        queryKey: ['me'],
        queryFn: ({ meta }) =>
            meta!.request<IUser>({
                endpoint: 'me',
            }),
    });

It's similar to https://github.com/vikejs/vike/issues/256#issuecomment-1030861810

Makes sense! Released in 0.1.2.


Data Retention on EditProfile when rotating device

Describe the bug
In EditProfileFragment, changes on EditText is not retained when rotating
Screenshots
Before rotating:

After rotating:

Would you like to work on the issue?
Yes

Duplicate on #652


Move epinio/ui-backend into epinio/ui

Summary
Fixes https://github.com/epinio/ui/issues/269

Occurred changes and/or fixed issues

Moved content from epinio/ui-backend to epinio/ui backend
Moved things from root of backend out to where they should be

updated root .gitignore
moved .github/workflows

Updated lots of paths given change of location (scripts) and repo (go deps)

There's two gh workflows that build releases. One nightly, one on tag. We should be able to get the release one pretty solid by fixing the nightly. I've tested what i can, but we'll probably need to iterate
Areas or cases that should be tested

Run backend .. and run dashboard... test together

All docs in PR should be updated with the new instructions. It's basically though run the ui backend from epinio/ui backend rather than epinio/ui-backend

There is too much to check, it's probably easier to merge and iterate/check.
We have just released so we have time to do that.
We should also probably disable the old scheduled releases in the old repo to avoid conflicts.

I've seen also the release drafter action/config, so this will probably fix https://github.com/epinio/ui/issues/281 as well.


Add coverage tracking

Codecov Report

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

@@            Coverage Diff            @@
##             master       #3   +/-   ##
=========================================
  Coverage          ?   84.61%
=========================================
  Files             ?        2
  Lines             ?       26
  Branches          ?        0
=========================================
  Hits              ?       22
  Misses            ?        4
  Partials          ?        0

Continue to review full report at Codecov.

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


Obscured obs location showing code below map:

eg https://www.inaturalist.org/observations/17274031

Closed in 4936f84a


use covid tracking date column

Background
See https://trello.com/c/1HA3Egdm/189-adding-jupyterlab-appears-to-break-build-all-at-the-moment and https://covidactnow.slack.com/archives/C0110QQAPJA/p1591105732043200
Please Check

[ ] Are the JSON schemas updated?
[ ] Are the api docs updated?
[ ] Are tests passing?

 Adding Jupyterlab Appears to Break Build-All at the Moment


Bug: submitting an empty project overview throws error [5]

Describe the Bug:
Probability (how likely the bug is to happen, scored from 1-5): 2
(For example, probability of 5 is something like "it happens to all users every time they log in."
Probability of 1 "only happens to certain users when a really specific and unlikely path is followed.")
Effect (how bad the bug is when it does happen, scored from 1-5): 3
(For example, effect of 5 is "the entire app crashes and makes it unusable for all users" or "the bug causes the wrong data to be saved, with critical information (e.g. payment) being affected."
Effect of 1 is "It makes some styling look a little bit weird.")
Steps to reproduce the behaviour:

in Dev (or start the app with yarn dev --as-role CIF_ADMIN and clean db (make drop_db && make deploy_db_migrations)
click "add a project"
try to submit without filling out any of the form fields
receive an error

Screenshots:
https://www.loom.com/share/c4b3afb74def40579da7388a36a33371
Additional information:

Duplicate of https://app.zenhub.com/workspaces/climate-action-secretariat-60ca4121764d710011481ca2/issues/bcgov/cas-cif/824, I think

Also duplicate of #691, closing


Release

Codecov Report

Merging #154 (0c0736c) into master (43edfed) will increase coverage by 10.40%.
The diff coverage is 13.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@              Coverage Diff              @@
##             master     #154       +/-   ##
=============================================
+ Coverage     12.39%   22.79%   +10.40%
- Complexity       19       43       +24
=============================================
  Files            12       12
  Lines           581      715      +134
  Branches         30       42       +12
=============================================
+ Hits             72      163       +91
- Misses          508      550       +42
- Partials          1        2        +1

Flag
Coverage Δ

IntegrationTests
13.00% <6.77%> (+5.08%)
:arrow_up:

UnitTests
11.32% <6.77%> (+5.82%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/main/java/com/napier/sem/City.java
100.00% <ø> (ø)

src/main/java/com/napier/sem/CityReporting.java
14.58% <0.00%> (-1.44%)
:arrow_down:

src/main/java/com/napier/sem/Country.java
100.00% <ø> (ø)

src/main/java/com/napier/sem/PopulationReport.java
100.00% <ø> (+100.00%)
:arrow_up:

src/main/java/com/napier/sem/CountryReporting.java
22.05% <1.92%> (-4.26%)
:arrow_down:

.../main/java/com/napier/sem/PopulationReporting.java
23.40% <19.23%> (+23.40%)
:arrow_up:

...main/java/com/napier/sem/CapitalCityReporting.java
18.49% <19.35%> (+18.49%)
:arrow_up:

...rc/main/java/com/napier/sem/LanguageReporting.java
69.23% <100.00%> (+69.23%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


add button names

trying to get some perfect light house scores for a site, missing button names for accessibility


Binary caches for Haskell packages with profiling enabled?

Would it be possible to have something like this, so that we can get a binary cache for Haskell packages compiled with profiling enabled?
# NOTE: this is just some mock code so that you get the idea.
# Ideally this expression will be written in whatever way it is acceptable
# within `pkgs/top-level/haskell-packages.nix`
pkgs.haskell.packages.ghc7102_profiling = pkgs.haskell.packages.ghc7102.override {
  overrides = self: super: {
    mkDerivation = args: super.mkDerivation (args // {
      enableLibraryProfiling = true;
      enableExecutableProfiling = true;
    });
  }
}

Or maybe there's a better solution? I don't know, I'm just putting the idea out here :)

@peti should know. Maybe his hydra instance provides them already.

We don't provide any profiled versions of the Haskell package set (neither on hydra.nixos.org nor on hydra.cryp.to). It's trivial to enable those kind of builds, obviously, but I'm not sure that our infrastructure can take another 7,000 builds. One factor is that our machines aren't necessarily powerful enough to build them. The second, probably even more important factor, is that we don't have enough disk space to store them, i.e. hydra.nixos.org is frequently running out of disk space as it is now already.

Hmm, that's understandable. Thanks.
What about simply setting enableLibraryProfiling for the current Haskell package sets? That would ensure that profiling-enabled libraries are packaged along with non-profiling-enabled libraries. It would still take some more space, but only about half the space it would take to keep a whole different package set.
With enableLibraryProfiling enabled for a library foo depended by bar: If bar is a library or executable compiled with profiling enabled, then it will use foo's libraries that were compiled compiled with profiling enabled, otherwise it will use foo's libraries that were compiled without profiling enabled.
Additionally, if somebody needs to compile a particular package with enableExecutableProfiling, they can override just that package (which probably won't lead to a recompilation of the whole package set, as this scenario probably means that you are trying to profile that particular executable, and not any package depending on it).

Also, maybe it is not necessary to enable enableLibraryProfiling for all the current Haskell package sets. Personally, I would appreciate having this enabled for ghc7102 and ghcjs. Perhaps others using some recent lts would appreciate that too. But maybe we don't need to enable this for older package sets not used as much.

If we enable profiling in the default package set, then every user will build or download those extra copies of the library regardless of whether they want profiling or not. Furthermore, build of packages like pandoc and darcs will compile their entire dependency tree with profiling versions included even though they'll never be used in the build of the executable. That's not exactly an ideal situation either.

Hmm… that's a good point. So I guess this will have to wait until those disk space issues are solved.
Thanks anyway @peti.

@peti could we perhaps use multi-output derivations to separate the profiled libraries from the regular libraries?

Yes, I suppose we could that. It's not going to be simple, though, because I don't believe that ghc supports different libdirs for library variants out of the box.

Do the profiled and non-profiled builds share lots of work? If not, I see no reason to always perform the work for both.

@vcunat, there's no shared work. Profiled and non-profiles builds are pretty much separate things.


I want to find html tags from the inner text

Steps to reproduce
Tell us about your environment:

Puppeteer version:
Platform / OS version:
URLs (if applicable):
Node.js version:

What steps will reproduce the problem?
Please include code that reproduces the issue.

What is the expected result?
What happens instead?

the script below can extract h1 to example domain:
const browser = await puppeteer.launch();
const page = await browser.newPage();
await page.goto('https://example.com/');
const title = await page.$x("//h1");
let text = await page.evaluate(h1 => h1.textContent, title[0]);
console.log(text)
await browser.close();
how if I want to reverse, so I get text Example domain then use it to get h1


Consider "distributing" the static checker checks to the VarType hierarchy

E.g., so that you can't add a null to a string.

I'm never going to do this.


document is not defined

Are you server-side rendering? https://github.com/zenoamaro/react-quill/issues/185

OK, #185 should contain everything you need 😄

If you're stumbling upon this, go straight to https://github.com/zenoamaro/react-quill/issues/122 instead of #185. #185 just forwards you to https://github.com/zenoamaro/react-quill/issues/122.

Hi,
Add { ssr: false } to stop server side rendering in dynamic import.

const ReactQuill = typeof window === 'object' ? require('react-quill') : () => false;
// Use ReactQuill as desired

Don't forget to import dynamic from 'next/dynamic' like I did for MuhammadMinhaj's answer to work!

React Quill Rendering Twice ??

how to fix it？

const ReactQuill = dynamic(import('react-quill'), { ssr: false})
Try This.

const ReactQuill = dynamic(import('react-quill'), { ssr: false})

doesn't work

I made a component for the quill editor and render it dynamic , and client side.
"use client";
import dynamic from "next/dynamic";
const ReactQuill = dynamic(() => import("react-quill"), { ssr: false });

import "react-quill/dist/quill.bubble.css";

function TextEditor({ value, name, handleQuillChange, label }) {
	return (

{label}

			<ReactQuill
				className=' mt-1 px-2 rounded-xl border hover:scale-105 duration-300'
				value={value}
				onChange={handleQuillChange(name)}
				theme='bubble'
			/>

	);
}

export default TextEditor;

Then I import it on a form that is rendered also  "use client"
but I have this error:
ReferenceError: document is not defined
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:7656:12)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:31:30)
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:1025:1)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:31:30)
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:5650:14)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:31:30)
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:10040:13)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:319:20)
    at eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:11:3)
    at (sc_client)/./node_modules/quill/dist/quill.js (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\_sc_client_node_modules_react-quill_lib_index_js.js:909:1)
    at __webpack_require__ (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\webpack-runtime.js:33:43)
    at eval (webpack-internal:///(sc_client)/./node_modules/react-quill/lib/index.js:43:31)
    at (sc_client)/./node_modules/react-quill/lib/index.js (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\_sc_client_node_modules_react-quill_lib_index_js.js:920:1)
    at __webpack_require__ (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\webpack-runtime.js:33:43)
    at __webpack_require__.t (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\webpack-runtime.js:73:38)           at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

const ReactQuill = dynamic(import('react-quill'), { ssr: false}) Try This.

Removing the arrow function solved the issue for me.

const ReactQuill = dynamic(import('react-quill'), { ssr: false}) Try This.

Wrapping the import within useMemo solved the issue for me (source: https://stackoverflow.com/a/74096771/12103912)

const ReactQuill = typeof window === 'object' ? require('react-quill') : () => false;

Tks. I work for me

I made a component for the quill editor and render it dynamic , and client side.
"use client";
import dynamic from "next/dynamic";
const ReactQuill = dynamic(() => import("react-quill"), { ssr: false });

import "react-quill/dist/quill.bubble.css";

function TextEditor({ value, name, handleQuillChange, label }) {
	return (

{label}

			<ReactQuill
				className=' mt-1 px-2 rounded-xl border hover:scale-105 duration-300'
				value={value}
				onChange={handleQuillChange(name)}
				theme='bubble'
			/>

	);
}

export default TextEditor;

Then I import it on a form that is rendered also "use client"
but I have this error:
ReferenceError: document is not defined
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:7656:12)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:31:30)
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:1025:1)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:31:30)
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:5650:14)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:31:30)
    at Object.eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:10040:13)
    at __nested_webpack_require_697__ (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:319:20)
    at eval (webpack-internal:///(sc_client)/./node_modules/quill/dist/quill.js:11:3)
    at (sc_client)/./node_modules/quill/dist/quill.js (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\_sc_client_node_modules_react-quill_lib_index_js.js:909:1)
    at __webpack_require__ (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\webpack-runtime.js:33:43)
    at eval (webpack-internal:///(sc_client)/./node_modules/react-quill/lib/index.js:43:31)
    at (sc_client)/./node_modules/react-quill/lib/index.js (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\_sc_client_node_modules_react-quill_lib_index_js.js:920:1)
    at __webpack_require__ (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\webpack-runtime.js:33:43)
    at __webpack_require__.t (D:\Coding\NEXTJS\jjudit_nextjs13\.next\server\webpack-runtime.js:73:38)           at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

I Have the same problem if someone can help

make sure include "use client" at the top of the file

const ReactQuill = useMemo(() => dynamic(() => import('react-quill'), { ssr: false }),[]);
this helped me. don't forget to use 'useMemo' it will store the data otherwise it will render. and obviously, write the code inside of your react function component.

const ReactQuill = dynamic(() => import('react-quill'), { ssr: false });

greats work like a charm

const ReactQuill = typeof window === 'object' ? require('react-quill') : () => false;
// Use ReactQuill as desired

It worked perfectly Thank you

const ReactQuill = typeof window === 'object' ? require('react-quill') : () => false;
// Use ReactQuill as desired

im using this but showing error?


clone --fork does not work?

I'm trying to use the clone --fork feature
Here is what I ran
❯ ghr clone github.com:m4xshen/hardtime.nvim --fork
 INFO Forking from 'https://github.com/m4xshen/hardtime.nvim.git'
ERROR I/O error: No such file or directory (os error 2)

I was kinda expecting that it would fork that into my username yujinyuz/hardtime.nvim.git and clone but nothing seems to happen

It seems that this uses gh under the hood.
It worked after I configured it via gh auth login

Hi, thank you for trying ghr and your feedback!
I will improve the error message so users can notice installing gh is required.


Dependency version pinning discussion

During the release process of the last release, I inadvertently (but harmlessley) dropped the version pinning from the released version. This is not a major issue since we support a very wide range of versions of things, and are up-to-date, so I propose to bring it back in the next release rather than issuing a hurried patch.
One factor in this was having a differences in pyproject.toml between 'main' and 'develop'. An alternative would be to have a separate variant for releases and development. This would require developers to use pip install -e .[dev] when working on the package, which doesn't seem like a difficult change. If they failed to do so, it's also unlikely there would be any real problems as the CI/CD would repeat testing on the latest versions of things, which would still allow the package to be kept up to date easily.
If anyone has any comment on the current release or this change, please feel free to leave it here, and I will take it into account when planning the next release.

A quick suggestion - once an approach is chosen, I would suggest adding a note in the maintainers notes documentation (somewhere in here: https://scores.readthedocs.io/en/stable/maintainer.html#this-section-covers-the-process-for-making-a-release-from-the-develop-branch) about ensuring correct version pinning before the release - simply to reduce the likelihood of this reoccurring.

Fixed by #760
Pinned versions are now set during the hatch build process for the release packaging.


Multiple STOPS_BEFORE constraint issue

The following query results in an error (both in the Python package and the Playground)
sample(temperature=0.8, openai_chunksize=128, max_len=128, chatty_openai=True)
    "The movie review in positive sentiment is: '[OUTPUT]"
FROM
    "openai/text-ada-001"
WHERE
    STOPS_BEFORE(OUTPUT, "\\n") and STOPS_BEFORE(OUTPUT, "n")

AssertionError: The specified set of constraints contains multiple incompatible postprocessing operations for the same variable. The conflicting operations are:  and . Please make sure the used constraints implement postprocess_order for each other, to use them together.
The same query with "STOPS_AT" instead of "STOPS_BEFORE" works.

Thanks, this will be fixed in the next release (see commit).


chore(deps): update module github.com/stretchr/testify to v1.8.4

This PR contains the following updates:

Package
Type
Update
Change

github.com/stretchr/testify
require
patch
v1.8.3 -> v1.8.4

Release Notes

stretchr/testify
v1.8.4
Compare Source

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 6.2.3 :tada:
The release is available on:

GitHub release
v6.2.3

Your semantic-release bot :package::rocket:


Color Contrast Errors

I can't comment in the original issue, but these still seem like valid accessibility problems
https://github.com/jekyll/minima/issues/293
I've attached a screenshot from - https://wave.webaim.org/
And a export of the html file from - https://accessibilityinsights.io/
AutomatedChecks_20200329_YourawesometitleWrit.html.zip

I added a pull request here https://github.com/jekyll/minima/pull/500


[usbdev_swdrive_test] SW pin driving test

Passing Verilator top-level test that exercises software pin driving, using a new bypass mode in the interface to the DPI model.
Software-driven outputs are propagated straight back to the input pins independently, to permit full checking of pins_drive/sense functionality.
Annotated usbdpi.sv logic because there are many signals with very similar names.

Hi @alees24! I'm working through old usbdev PRs and this was the oldest(!). I've just rebased it to the current head of master. Is this still something you're working on?

It got lost in the review process a longer time ago. It still has relevance, since it's still the only test that exercises the sw drive and sense registers/functionality, and it's something that can only be done in Verilator chip-level simulation (as performed here) or block level DV (not yet done).
I have updated the BUILD file to accommodate subsequent changes and re-tested with Verilator,


[Feature]: Possibility to unsticky top task tiles

What's the request? Toggle or option to unsticky top tiles
That imminent tasks that require your attention now jump to the top of "Tasks" is awesome.
 Especially in unrestricted, a lot can come together that I sometimes just don't want to deal with now (or ever).
Having the reminders on top is nice, but them scrolling down with you isn't. I would love to have an option or a toggle to just unsticky them even with the danger of overlooking them because I was scrolled down.
As an example:
I have currently minimized four (Model Train set - won't change it, DMT Dupe - won't do it inrun, Ghost busting - no need, bad ghost, SIT Course enrollment - won't change), and three big ones I want to be reminded (Kramco, Daylight shaving, autumn-aton) and Boombox.
Those take up more than half (!) of the full height of the sidebar already and it can become worse unfortunately. It makes scrolling down and looking at other tasks/Resources quite hard.

Account State
Unrestricted (In-Run)
Your current TourGuide version
313a0d436cad58350f5d3a3e72600818f7a2c1ae

yeah, this is a persistent problem i do not have a great way to deal with -- i agree that the supernag situation is quite problematic. even in standard, i've absolutely had to zoom out or manage my window explicitly to limit the real estate taken up by the supernags. what you describe here makes sense, but i think that sort of management of priority chaining in the minimization struction might not be possible with how it's currently designed, so it will take a bit of poking. wanted to comment to note that this is on my large scale "things we should fix" list but it also is not going to get dealt with imminently. still, thank you for reporting it, it's good to get validation that it isn't just me having these issues lol


fork: private fork

I suspect the API supports private forks and now that private repos are free this functionality makes more sense.

I suspect the API supports private forks and now that private repos are free this functionality makes more sense.

I suspect the API supports private forks and now that private repos are free this functionality makes more sense.


Added logic to render main release note pages in PRs

When a new release note file is pushed in the _includes/releases/vXX.Y/vXX.Y.Z.md, this PR should now display the changes in the associated major version file.

This change is

@ianjevans This is ready for review now. Can you give it your blessing?


add source-map support for the observable transfomer

This is currently

This issue has been moved to dart-lang/observe#29.


Consider using giscus

Hey there,
I see that you're using utterances for comments. As we know, it utilizes GitHub Issues, which (given enough time) would flood your repository's issues with comments.
I've been developing an alternative: giscus, a similar project that utilizes GitHub Discussions instead. It has support for replies and other cool stuff from GitHub Discussions. The big advantage is the fact that it uses your repository's Discussions, which is more suitable for comments. I would really appreciate it if you tried it. Feedback is welcome, the code is open source.
You can convert existing issues into discussions, as described here.
Thanks!

Hi @laymonage thanks for letting me know! Interesting project, and I do like the idea and format of Discussions as opposed to Issues!
I'll put some time in to do the conversion, awesome project! Looking forward to it!


Implement feature that allows to export tabular data suited to a terminal. This solves issue #340.

Hi I implemented a wrapper that helps to export tabular data into a Command Line Interface. All test with different versions of  formats were performed. I appreciate any comment or suggestion. If there is any additional  change or comment, please let me know.

As soon as @hugovk is satisfied with the PR, so am I.


Add an "Anywhere" type

Replacements are currently limited to specific locations ("Request String", "Request Header", etc.). Another replacement type could be added to allow for searching the whole request for replacements, rather than just specific locations.

Hey @danielthatcher ,
I think the "request string" replacement provides this functionality. It just looks for the first instance / every instance of a specific regex or string within the entire contents of the request and replaces it.

Ah, sorry about that. I had taken "Request String" to mean the query string of the request. It might be worth renaming this, but that might just be me.

Yeah, I couldn't think of a better name for it and I agree it's ambiguously named. If can think of something better to call it i'd be down for the change.


slack webhook check whether error repeats

Discussion: https://github.com/Bisonai/orakl/pull/964

@martinkersner this issue is unclear, can I close it?

The way how slack webhook is implemented is not exactly what we wanted, however, it does not make big issue, so we can close it 🙏


FEATURE REQUEST: Private lobby

Is the feature "Join lobby via code" make a private room, where only people with the code can join, or is it just for convenience and everybody can join if they find the lobby in the public list?

Anybody can join any lobby even if its set as friends only if you give them the lobby code.

This may be what you're looking for by the way. https://thunderstore.io/c/lethal-company/p/Dev1A3/LobbyInviteOnly/
I'll be closing this issue

No, no. The use case you described earlier was what I was looking. This lets streamers host lobbies without the need for people to be friends with them. Appreciate your response.


Remove dependency on legacy styling system

Ticket: #389

@MicheleK17 Some of the styles break - especially the drawer transition.  These should be fixed in a new Ticket


extra_solver_state Documentation

Is the documentation for extra_solver_state going to come out at some point?
Alternatively, are there any useful referencez to try and use this functionality? I would be particularly interested in reconstruct the SDE backwards in time. If I understood correctly,  would it be like solving the reverse SDE?
Many thanks

Probably not. Realistically this package is currently in maintenace mode.
I recommend trying Diffrax for any new projects.


Keep using 200 for no data found

There is a lot of debate about the status code for no data found: https://stackoverflow.com/questions/11746894/what-is-the-proper-rest-response-code-for-a-valid-request-but-an-empty-data
The main candidates are 200, 204, 404.
To keep the logic consistent and easy to maintain for both us and our clients. We follow the following basic principles:

For a list of items, it's acceptable to use 200 with an empty list [] to keep the logic simple for both on the server-side and the client-side instead of 204 or 404 to make the problem more complex.
For an item, to keep the status code consistent, we still use 200 with empty content (which means the length of the body is 0) instead of 204 or 404.

BTW: We definitely don't want to use 404 because it's really confusing for clients to distinguish whether the URL is wrong or something else.


Prevent reconciliation errors in hibernated shoots while migrating kube-state-metrics

How to categorize this PR?
/area monitoring
/kind enhancement
What this PR does / why we need it:
In PR #9941 the kube-state-metrics deployment was refactored and in PR #10355 we fixed an issue, but it did not correctly handle hibernated clusters. If the migration is executed for a hibernated shoot, it fails until the shoot is woken up. This PR prevents such reconciliation errors for hibernated shoots.
Special notes for your reviewer:
/cc @istvanballok @plkokanov
Release note:
NONE

/assign


fixed select all for permission-tab

Fixed small bug, which prevented the "Select All" and "Select None" in the matrix to be working.

ACK


MSID: 91359 Version: 2 DOI: 10.1101/2<IP_ADDRESS>7326

MSID: 91359
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>7326
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91359
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>7326

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/91359v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>7326
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/91359/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

error: [journal-ref-article-title] This journal reference (id c9) has no article-title element.


transloco key manager does not detects scopes correctly

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior

The key manager detects TRANSLATION_SCOPE_KEY instead of myTranslationScopeKey while parsing the module definition below.
const TRANSLATION_SCOPE _KEY = 'myTranslationScopeKey';

@NgModule({
  imports: [CommonModule, TranslocoModule],
  providers: [
    {
      provide: TRANSLOCO_SCOPE,
      useValue: {
        scope: TRANSLATION_SCOPE_KEY,
        loader: scopeLoader((language, root) => import(`./${root}/${language}.json`))
      }
    }
  ]
})
export class MyFeatureModule {}

Expected behavior

The key manager detects myTranslationScopeKey while parsing the module definition below.
Minimal reproduction of the problem with instructions
Run transloco-keys-manager extract
What is the motivation / use case for changing the behavior?
In my application the scope key is a constant in a separate file which is imported in different places (the feature module, a transloco testing module...)
Environment

Angular version: 9.0.2

@stijnvn please submit this issue in the keys manager repo;


Add old config format cause the otlp OC handler only supports OAP server 9.2.0

Add old config format cause the otlp OC handler only supports OAP server 9.2.0
It's a supplement for [#9499] [#9521]
Most users have not been updated to OAP server 9.2.0 (such as helm charts), according to the official link to the latest document, we can't successfully deploy the otlp exporter.

I don't think adding comments here would help much.
If it is a helm issue, it's better to polish docs and codrs there.

Such as we could default versions of OAP in helm repo to 9.2.0

In essence, it's not a helm issue.
It's the official document issue.
We can link Provide Linux OS monitoring from the official UI
Then we can link otel-collector-config.yaml from any version 's this page
This issue bothered me for 3 days. I've tried any config change/Env change to use the otlp exporter. until asked my friend @Ax1an to figure out if this commit is any version requirements
This is a very serious misleading if I don't look at the commit history

I see. You mean the UI side links. Then I am afraid we don't have a solution to provide versionalized link for every release.
Could you follow Zhenxu suggestions to update configs in showcase repo?

Could you also update the config in our demo repo? https://github.com/apache/skywalking-showcase/tree/main/deploy/platform/kubernetes

I‘m willing to support open source
But what I'm skilled at k8s/cloud native/architect
OpenTelemetry is a new thing for me, I will do my best


feat: Controller Extension support in fe-fpm-writer

Support for controller extension creation using fe-fpm-writer.
Some special parts to mention:

In manifest.json - for controller extension we can have:
1.1. Single controller using property controllerName - string;
1.2. Multiple controllers using property controllerNames - string array;
In result we need handle cases when controllerName is defined in manifest.json and we try to add new controllerName, then we need apply transition from controllerName to controllerNames.
Also additional property overwrite is available in case user wants to replace existing with new.
There no support for typescript controller version, because did not find valid syntax for it. And there is also issue reported in https://github.com/SAP/ui5-typescript/issues/332

@815are don't forget to also add a typescript template for the controller extension.

@815are don't forget to also add a typescript template for the controller extension.

@tobiasqueck
I checked but did not find valid usage of ui5 override, there is also issue reported for UI5 team regarding Controller extension in typescript - https://github.com/SAP/ui5-typescript/issues/332
In result I have interface as:
export interface ControllerExtension extends CustomElement {
    ...
    /**
     * Typescript generation is not supported for control extensions.
     */
    typescript?: false;
}

The link from @815are is correct, currently there is no official support for this.
You can still write them in TS i think but not with class syntax .


Takes too much time to open

I'm in a Lenovo Y50, using Debian + Gnome Shell. gnome-terminal takes ~.5s to load while hyperterm takes `2s.

How long does it take for chrome to load? Hyperterm uses electron which is based on chrome so chrome is going to be a determining factor in how fast the UI can possibly load.

https://drive.google.com/file/d/0Bz517xiOtwEFY0x6RS1Wd0pwSWs/view?usp=sharing
Recording kind of delays everything a little bit, but you'll see how there's a 1/2 second difference approx between Chrome and Hyperterm

https://github.com/zeit/hyperterm/pull/153#issuecomment-234307376

I see. Ok then. It'd be nice to have it open a bit faster but maybe that's a hard thing to do, I understand. It's just that I tend a lot to open the terminal and cd to my 'projects' directory and hyperterm usually just catches the 'd' in cd, so I just have to wait a little longer :D

That could probably be solved, by starting keyboard capture as soon as the process is started, not when electron is loaded. Then pass the captured keys straight to the actual terminal or trough electron?

I think we should do as much as we can to make it as fast as possible. I like the idea of comparing it to Chrome init!

The idea about capturing keyboard early is great. I'd love this on my Mac as well! I have the same problem as the OP. Often opening and instantly starting to type but it swallows the first or first two characters. Seeing it isn't as critical as being able to type into it right away.

I think we should do as much as we can to make it as fast as possible.

That's especially important in tiling window managers. You often end up just doing super + return to bring up a new terminal, because you forgot that you have already a billions of them open. The startup time of hyper should be improved, as that is a bit of laggy experience right now.

Startup time has been improved. (Especially in v1.1.0 release)
Feel free to reopen.

Still takes 1.8 seconds to open (version 1.3.1) on my MacBook 12". Apple's Terminal.app takes roughly 0.7 seconds.

@hachre and Chrome ?

@chabou: Chrome takes only 1.4 seconds. (Even though I have 16 extensions installed)

We must improve on it but we have non-compressible Electron bootup time 😞

Maybe some good ideas: http://blog.atom.io/2017/04/18/improving-startup-time.html

Another way to achieve this: #525

How about when user "closes" Hyper by clicking x or Alt + F4, Hyper can just hide the BrowserWindow instance but keep the process running? Some people might say it'll be a RAM lockup, but I doubt that Hyper is in any way resource intensive.

Although it is based on the electron, I think the comparison with Chrome is nonsense. We have to improve the startup time. I understand tweaking electrons internal is not possible. But we can re-architect some part. For example, we can start a hyper service and basically run every terminal as a client to the main service. In that scenario, the startup time of every client should drop to 0.1ms.

Bench with http://isitsnappy.com/
From click on dock to shell prompt printed:
v1.4.8: 1.6s
v2.0.0: 1.3s
I think this is ok now :)

I just started my hyper terminal (2.0) and it took 20s on the first try, then 8s on the next 3 tries.  This is unusably slow for me.   My machine is a relatively new z440-  Powershell and Cmd on the same machine come up instantaneously.
I ran Hyper -v and got this output:
C:\Users\erjorge>hyper -v
running in prod mode

electron will open file://C:\Users\erjorge\AppData\Local\hyper\app-2.0.0\resources\app.asar\index.html

[26476:0614/081223.616:WARNING:keyboard_util.cc(165)] Invalid accelerator token: del
[26476:0614/081223.616:WARNING:accelerator_util.cc(62)] ctrl+del doesn't contain a valid key

[26476:0614/081223.620:WARNING:keyboard_util.cc(165)] Invalid accelerator token: del
[26476:0614/081223.620:WARNING:accelerator_util.cc(62)] ctrl+del doesn't contain a valid key

[26476:0614/081225.696:ERROR:cache_util_win.cc(20)] Unable to move the cache: 5

[26476:0614/081225.696:ERROR:cache_util.cc(134)] Unable to move cache folder C:\Users\erjorge\AppData\Roaming\Hyper\GPUCache to C:\Users\erjorge\AppData\Roaming\Hyper\old_GPUCache_000
[26476:0614/081225.696:ERROR:disk_cache.cc(132)] Unable to create cache
[26476:0614/081225.696:ERROR:shader_disk_cache.cc(604)] Shader Cache Creation failed: -2

Checking for notification messages

No matching notification messages

I'm experiencing slow startup from Hyper too but on Windows (Hyper even with the slowness is by far the best terminal emulator here). It seems like the slow startup problem exists across platforms.
The suggestion to run the main process in the background, letting the RAM take the hit, sounds like a great idea to me, though I've got no idea how practical it is to implement. I'd definitely pay for the feature though.

+1 for always on background process if that makes sense

Both on windows and mac my hyper launches in sub 2 seconds, I think that's acceptable. The question is why you're getting so slow start times 🤔 can you attach your config?

Config: https://gist.github.com/jamesyale/474cb69731927fddf515043073663a36

Yes it can be improved @ssabpisa. PR's to improve startup perf is welcome 🙌


Is there an API to get a broadcast list recipients (the contacts of the list)?
I tried with GetGroupMetaData but it didn't return anything.
I know it's possible because it's available on whatsapp web. Maybe we can get this data with a custom query?
Thanks!

I didn't figure out how to capture brodcast recipients, but was able to "gather" this from ack events (delivery and read confirmation).

I didn't figure out how to capture brodcast recipients, but was able to "gather" this from ack events (delivery and read confirmation).

Example JID:<PHONE_NUMBER>@broadcast
func (wac *Conn) GetBroadcastMetaData(jid string) (<-chan string, error) { 	data := []interface{}{"query", "contact", jid} 	return wac.writeJson(data) }


Use cuda base image for final docker image

This is necessary so cuda works at all.

Hi @djmaze – thanks so much for opening a PR! Would it be possible to point out the error you're seeing with the current docker image? Is it failing to detect nvidia-smi or something else?

Okay, I had a deeper look and found out the problem is only with docker swarm mode (which is what I am using). In swarm mode, docker does not respect the NVIDIA_DRIVER_CAPABILITIES setting so nvidia-smi (and the necessary libraries) are not available in the container.
That's why in swarm mode, ollama outputs the following error message:
routes.go:915: warning: gpu support may not be enabled, check that you have installed GPU drivers: nvidia-smi command failed

So in swarm mode, you need to use a base image which already contains the cuda libraries & nvidia-smi.
For people who are interested, I created an issue  for nvidia-container-toolkit.
I guess not many people are using swarm mode with ollama. So we can probably close this as wontfix and I will just maintain my own image.

Okay, I had a deeper look and found out the problem is only with docker swarm mode (which is what I am using). In swarm mode, docker does not respect the NVIDIA_DRIVER_CAPABILITIES setting so nvidia-smi (and the necessary libraries) are not available in the container.
That's why in swarm mode, ollama outputs the following error message:
routes.go:915: warning: gpu support may not be enabled, check that you have installed GPU drivers: nvidia-smi command failed

So in swarm mode, you need to use a base image which already contains the cuda libraries & nvidia-smi.
For people who are interested, I created an issue for nvidia-container-toolkit.
I guess not many people are using swarm mode with ollama. So we can probably close this as wontfix and I will just maintain my own image.
(Update: As ollama seems to be using nvidia-smi only for determining the available VRAM, I tried to find out if there is an alternative way to do that. Turns out that seems not to be easily possible.)

@djmaze I don't think I'm in swarm mode and having the issues as well

@seth100  Well, it seems you have different error message. Did you check if nvidia-smi is available in your running container (using docker exec ..)?

@djmaze yes, it's available

So, then it's working in principle. That must be another problem.

We've made changes over the past few weeks to transition to using the GPU management libraries instead of CLI to discover information about the GPU, iterate through various possible locations, and if we can't find a working management library, we fall back to CPU mode.  When you run a container with GPU access, typically the libraries for the host driver get plumbed through automatically by the container runtime.
We're about to cut a pre-release of 0.1.21, which should be final in a few days.  You can wait for that for the official container image, or if you want to try it now, I've pushed dhiltgen/ollama:0.1.21-rc to docker hub.  You can run with something equivalent to:
docker run --rm -it --gpus all -e OLLAMA_DEBUG=1 dhiltgen/ollama:0.1.21-rc

The latest release (0.1.22) is up and should discover GPUs properly.  I'm going to close this PR for now.  If you still have problems getting the container to detect your GPU, please run with OLLAMA_DEBUG=1 in the environment and share the log in an issue or on Discord so we can understand why it's failing.


Delay is treated as hours

If there is a delay. The card shows it as hours.
E.g.
8 minutes delayed on the hvv website are +480minutes in the card.

duplicate of https://github.com/nilstgmd/hvv-card/issues/7


HOSTAPD plugin

Enable HOSTAPD by default in order to allow Raspberry PI zero get accessible via WIFI. Will allow user to access Raspberry PI using WIFI and SSH connection without using keyboard and monitor.

In case you consider that my proposal is ok with you, please add it to the master branch. I did several tests using PI zero with wifi and PI 3.

Can you move it to disabled and I'll include it for sure.  Just don't want it on by default specially using the 192.168.x.x network as that would cause me tons of grief.

I will put in disabled. Regarding network address, from my standpoint,  anything else would be ok. 192.168.x.x is just a proposal. 10.x.x.x or 172.16.x.x are good as well.

192.168.x.x is perfectly fine, though I think industry standard for it would have been on the 10.x.x.x network.  That being said this is a great addition thank you very much for it!

You’re welcome! And thank you, too.


recent: Don't show "No conversations match your filters." while loading messages.

Fixes: #32503
UI Changes:

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@alya, this PR is up for review, can you please take a look.

@reharsh Make sure to update your commit message to match the commit style guidelines
Adding it to my list to review.

@prakhar1144 updated, please check when you get to this.

@amanagr is probably the right reviewer for this as the recent view maintainer.

Looks good to me. Tested manually for different filters too.

Merged, thanks @reharsh!


add new attributes

what

add option to change port for the cluster, the variable was defined but not used on the cluster resource so the SG was using it but not the cluster.
add two attributes important to be sure no unexpected downtime without being planed: preferred_maintenance_window and auto_minor_version_upgrade

why

we want to have the cluster up to date on the last minor release
we want to be able to change the default port

references

No issue or bug fix, just some additions

/rebuild-readme

/test all

/terraform-fmt

/test all

/test all

thanks for the quick merge, really appreciate it ;)


Cannot load Firebase inside isoalte

I am not sure actually if this is a combine plugin related.
Combine works fine until I add the steps to start FirebaseAuth and Firebasestore.
The error I get is:
W/ProviderInstaller( 4962): Failed to load providerinstaller module: No acceptable module com.google.android.gms.providerinstaller.dynamite found. Local version is 0 and remote version is 0.
F/libc    ( 4962): Fatal signal 6 (SIGABRT), code -1 (SI_QUEUE) in tid 4962 (.cookingjournal), pid 4962 (.cookingjournal)
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
Build fingerprint: 'google/sdk_gphone64_x86_64/emu64xa:14/UPB5.230623.003/10615560:user/release-keys'
Revision: '0'
ABI: 'x86_64'
Timestamp: 2023-08-27 23:03:33.909536880+0300
Process uptime: 29s

The code that configures firebase is
final firebaseApp = await Firebase.initializeApp(
      options: DefaultFirebaseOptions.currentPlatform,
    );
    final FirebaseAuth auth = FirebaseAuth.instanceFor(app: firebaseApp);
    final fireStoreRepo = FirebaseFirestore.instanceFor(app: firebaseApp);
    fireStoreRepo.disableNetwork();

Note: code works when in main isolate.
Can this be related to Combine or maybe there is another problem ?

Hello. Can you please send a repo with minimum reproducible code?
I've tested firebase initialization and it works fine.
Also, I've tried to google this exception and everybody suggests to add network permissions on android. However, as you say, it works in main isolate. So, looks like problem isn't in permissions

I am not sure exactly what changed, but now it looks like it is working. I did not test with google cloud functions and firebase auth, only with crashlitics this time.
I think we can close this one.
Also: The idea of this package is very neat.


Question about this package

We currently have a custom REST api written in Laravel, and I need to publish our own SDK for developers to use to consume our API, is this the intent of. your package, for us to create and publish our own company SDK?

Hey @justageek - sorry I must have missed this notification. Yes this is the aim of the library 😄


Deep Dive episode 5: incremental financial innovation

Acceptance

[ ] Rufus and Stephen have been updated that conversation is ready to be published on YouTube and Anchor.
[ ] Page stub has been made and URL generated
[ ] Conversation published on Youtube & Anchor
[ ] Blog Post containing all content published
[ ] Secondary Content and Direct links shared on Social Media

Tasks

[x] Duplicate this Template and rename as the working title of the conversation
[ ] Create a new folder for the conversation here: TBC we need to link up on file storage
[x] Source the video file and transcript from the conversation and add to folder
[ ] Create a document for the conversation that includes: Title, Conversation Summary, About the person in conversation, links to resources mentioned in the podcast, timestamps for at least two video excerpts and two quotes. Add this to folder.
[ ] Create Youtube video

[x] Create the Youtube Thumbnail from this template.
[ ] Edit the video file in VN Editor:

[x] Trim beginning and end, plus any sections of poor internet connection or other interruptions
[x] Add thumbnail at the beginning of the video with a fade transition to the video
[x] Add graphic overlay "Please Like & Subscribe" at the 5, 10, 20, 40 minute
[x] Add End Screen with a fading transition
[x] Increase the sound volume as much as possible without clipping the sound (you will notice because it will sound bad)

[ ] Export as Video and also Audio only
[ ] Upload to Youtube: Add title, thumbnail, summary, page stub URL, add to Web3 playlist, Add Language, Make License Creative Commons.
[ ] Notes from the conversation are on the webpage for that conversation.

[ ] Notes are complete, comprehensive and tidied

[ ] Inform Rufus and Stephen that youtube video is ready to go.
[ ] Publish
[ ] Ensure Rufus and Stephen tweet about video

[ ] Create Podcast

[ ] In Anchor click create new episode, use the "Intro Jingle" and add the audio only edit
[ ] Add title & summary (incl page URL)
[ ] Create and add thumbnail from this template
[ ] Add season number and episode number

[ ] Create Secondary Content

[ ] Cut the video excerpts from the main video
[ ] For reels, change the video for to 4:5 and the Canvas to 9:16 - Add this as a the background
[ ] Reduce video saturation to -100
[ ] Write subtitles overlay to go above the video on the background
[ ] Create Reel Cover using this template
[ ] For Twitter Keep the video excerpt as 16:9 format and add subtitles to the top of the video
[ ] Create Quote posts using this template
[ ] Add all content to the folder that has been created

[ ] Create Blog Post

[ ] Create a blog post beginning with the summary then embed the video, the podcast and include the transcript
[ ] Add the relevant quotes and video excerpts (Twitter format) as images between the transcript

[ ] Compile all the information in the Web3 Spreadsheet

Once blog is complete we can schedule content on Twitter, FB and IG

@EilidhRoss1 changed title here - note i would refer to these sessions under the general title "deep dives".

FIXED. Deep dive ep 5 fully processed and published

FIXED. Deep dive ep 5 fully processed and published


Make geojson region output compatible with QuPath

Need to fix a list of things in the exported geoJSON format to be compatible with QuPath:

a multipolygon can not have only one element, else it should be saved as polygon.
the color element should be in features.properties.classification.color and not directly in features.properties.color.
polygons should always be closed (this should already be the case with most of our outputs, but check Points2Regions files).

Fixed with https://github.com/TissUUmaps/TissUUmaps/commit/905fb860d6c067cd32f6f517cfc1500603c74aa5. Will wait until this is released to close the issue.


Control flow not working as expected outside of if block

TypeScript Version:  2.0.2 and nightly (2.0.0-dev.20160904)
Code
// A *self-contained* demonstration of the problem follows...
public test(records: IRecord | IRecord[]) {
  if (!Array.isArray(records)) {
    records = Array(records); // or records = [records]
  }
  return records.length;
}

Expected behavior:
Length should return the length of the array since I casted records to an array if it comes in as a single IRecord.
Actual behavior:
I get the error: Property length does not exist on type 'IRecord | Record[]'.
Workaround:
Not ideal
public test(records: IRecord | IRecord[]) {
  let _records: IRecord[];
  if (!Array.isArray(records)) {
    _records = Array(records);
  } else {
    _records = records;
  }
  return _records.length;
}

@iDev0urer your original example compiles fine for me with TS2.0.2. However your repro is not self-contained since IRecord is not defined, and public test ... is invalid syntax. This is what I used:
interface IRecord { someProp }

function test(records: IRecord | IRecord[]) {
  if (!Array.isArray(records)) {
    records = Array(records); // or records = [records]
  }
  return records.length;
}

...and that works as expected.
However if you declared interface IRecord { } (i.e. an empty interface), then the problem occurs as you describe. But the reason in that case is that IRecord[] would be a structural subtype of IRecord, and that affects the narrowing logic.

Here's an equivalent example to the OP, showing how narrowing behaviour depends on whether the types in the union are unrelated or supertype/subtype.
But I don't know why the compiler acts differently at (1) and (2) below, and whether the compiler could do better here, since it seems inconsistent. Someone from the team, probably @ahejlsberg, would be best able to answer that.
class Animal {species}
class Cat extends Animal {meow}
class Dog extends Animal {woof}
declare function isDog(x): x is Dog;

function test1(x: Cat | Dog) {
  if (!isDog(x)) {
    x // x is Cat
    x = new Dog();
    x // x is Dog                                       <===== (1)
  }
  return x.woof; // OK
}

function test2(x: Animal | Dog) {
  if (!isDog(x)) {
    x // x is Animal
    x = new Dog();
    x // x is Animal|Dog  ---  but why not Dog here?    <===== (2)
  }
  return x.woof; // ERROR: property 'woof' does not exist on type 'Animal | Dog'
}

@yortus do you think it might make a difference that my examples were done inside of a class rather than as regular functions?

@iDev0urer I suspect the issue in the OP is that you've declared interface IRecord { } in which case an IRecord[] is also an IRecord (because every type is a subtype of the empty interface).
@yortus The same is true at (2) in your example. A Dog is also an Animal, so when you assign a Dog to Animal | Dog you still have an Animal | Dog. In general, types like Animal | Dog are kind of non-sensical. You really can only expect meaningful results when the types in a union are truly disjoint.

@ahejlsberg yeah that was the intention as I had already pointed out the supertype/subtype problem in the OP. I was more interested in why the compiler can't make the test2 example (which corresponds to the OP) work. I suppose it is because subtype reduction on the (2) line doesn't remove Animal from the union because all dogs are animals?

@iDev0urer the narrowing behaviour is the same in both class methods and regular functions.

@ahejlsberg actually my IRecord looks like this:
export interface IRecord {
  Id?: string;
  type?: string;
  attributes?: IRecordAttributes;
  extIdField?: string;
}

but I have a feeling that I am running into the issue you are describing since all of the params in IRecord are all optional it is basically an empty object. Thanks for the clarification!

types like Animal | Dog are kind of non-sensical.

@ahejlsberg not really. IRecord | IRecord[] is just like Animal | Dog to the compiler, and the OP naturally expected it to work. Unions like this occur regularly in valid code, where the subtype/supertype relationship is incidental and unintentional. #10471 is another case of the same problem with call signatures. Two more examples are here.
I'd rather describe this situation as a limitation of structural typing, rather than a problem with the source code, which is often perfectly acceptable runtime-valid code complete with valid type guards.
Perhaps it's not possible, but it would be great for this to be given further attention to see if there is some way to get narrowing to work the way the programmer expects (and to match runtime behaviour) for unions of unintentionally related types like in this issue.

@yortus thanks for the help!


ATS 10.x XPACK crash in production

I saw this crash 2 times in the last day:
(gdb) bt
#0  0x00007ff7fb532acf in raise () from /lib64/libc.so.6
#1  0x00007ff7fb505ea5 in abort () from /lib64/libc.so.6
#2  0x00007ff7fd0e2a4b in ink_abort (message_format=message_format@entry=0x7ff7fd0f5bef "%s:%d: failed assertion `%s`")
    at /sd/workspace/src/git.ouryahoo.com/Edge/build/_scm/trafficserver10.0/src/tscore/ink_error.cc:99
#3  0x00007ff7fd0df425 in _ink_assert (expression=expression@entry=0x8fd040 "name_len + value_len <= this->_capacity",
    file=file@entry=0x8fcf88 "/sd/workspace/src/git.ouryahoo.com/Edge/build/_scm/trafficserver10.0/src/proxy/hdrs/XPACK.cc", line=line@entry=568)
    at /sd/workspace/src/git.ouryahoo.com/Edge/build/_scm/trafficserver10.0/src/tscore/ink_assert.cc:35
#4  0x00000000006baa1e in XpackDynamicTableStorage::write (this=<optimized out>, name=<optimized out>, name_len=4, value=<optimized out>, value_len=<optimized out>)
    at /sd/workspace/src/git.ouryahoo.com/Edge/build/_scm/trafficserver10.0/src/proxy/hdrs/XPACK.cc:568
#5  0x00000000006bab80 in XpackDynamicTable::insert_entry (this=0x7fed18aacd38, name=0x7ff7efdfb300 "linkent-typerity-policyty\b\212", name_len=4, value=0x7fe871ad8b14 "", value_len=18550)
    at /sd/workspace/src/git.ouryahoo.com/Edge/build/_scm/trafficserver10.0/src/proxy/hdrs/XPACK.cc:356
#6  0x00000000006bad12 in XpackDynamicTable::insert_entry (value_len=<optimized out>, value=<optimized out>, name_len=<optimized out>, name=<optimized out>, this=<optimized out>)
    at /sd/workspace/src/git.ouryahoo.com/Edge/build/_scm/trafficserver10.0/src/proxy/hdrs/XPACK.cc:338

Was this from H2 or H3?

@maskit this was from h2

I think this code is wrong.
https://github.com/apache/trafficserver/blob/94dfd0e0f571a4e91bb0bcaa49af1c039db94e31/src/proxy/hdrs/XPACK.cc#L507-L509
If _entries_head is sizeof(_entries) - 1 (i.e. the max index number), the while loop keeps removing entries until extra_space_needed becomes larger than freed. I'm not sure if this is the cause of the crash, but it explains why the _available is a bit larger than _requrired.
The correct condition is probably this->_entries_head == tail.


Create C. Discrete Acceleration.cpp

Nice solution! Unfortunately, I'm only hosting my own solutions in this repository. You are welcome to fork this repo and add your solutions there.


Define some structure for lifetime values

Here is a description of how to encode lifetime values. We intend to include the types and values defined in this pull request with the same semantics in an upcoming release of the PSA Cryptography API specification.
This proposal is backward-compatible with the current values for volatile keys (0) and persistent keys (1). Platforms with a secure element (i.e. a cryptoprocessor that can process keys which are not accessible to the PSA crypto core) should use the lifetime 0x1ff (whether the key material is stored inside the secure element, or in generic PSA storage in a form wrapped by the secure element).
This pull request defines the lifetime structure, but does not yet implement any new feature. A future pull request will adapt the (currently experimental) secure element support code accordingly.
This stems from an internal discussion (private link). This pull request is a continuation of https://github.com/ARMmbed/mbed-crypto/pull/358. The first commit here is the final state of that mbed-crypto PR (the history there wasn't worth carrying over).

I looked at the reported failure in the pr-merge job and it looks like a temporary glitch in the infrastructure - failed to clone the repo - not a problem with this PR.

Does this need a ChangeLog entry? I'm not sure.

So far new PSA features haven't been getting a changelog entry, but we've started to write changelog entries for behavior change. In any case, this is documentation only so far, the new feature isn't implemented.
The documentation does indicate that certain lifetime values are now considered volatile, but the code doesn't treat them as such. (This is not a behavior change because the volatility of these values was previously unspecified.) So, annoyingly, we shouldn't merge this yet.


Microsoft SQL Server Report Builder

Which version of the "Microsoft SQL Server Report Builder" is required to edit the .rdl files? A link to the Microsoft SQL Server Report Builder download page would be helpful.

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

ID: 87b7f14d-d6ca-7248-7621-47669905bea0
Version Independent ID: c1e289a5-c3cc-a748-4f59-6a56001c4b8d
Content: Creating an RDL Layout Report - Business Central
Content Source: dev-itpro/developer/devenv-howto-rdl-report-layout.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello, editing RDL report layouts requires Report Builder for SQL Server 2016. If you have an on-premises solution, this is available on the installation DVD and installed as part of Setup. Otherwise, you can download it from here.
We also list the Report RDL requirements in the System Requirements. The article has been updated with a link to the requirements. I hope this helps. Thanks for the feedback.


Fix README

There are a few updates in the dev channel that needs to be updated in the dev channel's README file

I've started writing a FAQ in the wiki in order to make the documentation richer and cleaner: https://github.com/h0m3/protonlaunch/wiki/FAQ
Its in its initial draft

I'm working on the Wiki to make sure there's necessary information allow anyone to get started with protonlaunch and also some generic wine / proton information.
https://github.com/h0m3/protonlaunch/wiki
Its still on its first drafts (tons of errors yet) but it should be somewhat ready once the next main release of protonlaunch shows up.


Adding app "Partner Customer Delegated Admin Offline Processor"

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

File
Status
Preview URL
Details

support/azure/active-directory/verify-first-party-apps-sign-in.md
:white_check_mark:Succeeded

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


Updated press resources link to domain relative url

Updated press resources link to domain relative url
Additions
None
Removals
None
Changes

Updated press resources link to domain relative url in single template

Testing
Navigate to the following and click the press resources link after the article

/newsroom/cfpb-takes-action-against-realtysouth-for-mortgage-disclosure-violations/

Review

@jameshupp
@anselmbradford
@sebworks

Preview
None
Notes
None

:1

:1:


Wrong Result Item quality in craft queue after relog

when using "add open recipe" in the craft queue it shows me the right Result Item quality! like this:

but after relogging or swapping chars it thinks R3 isn't possible with the assigned mats and shows the resulting item at R2 instead:

so every time i want to see the average profit for the item i have to remake the whole recipe and add it again!
however when i press craft it still gives me the R3 item!

there seems to be something wrong with the skill requirement in the craft queue in general, bc even when going in and editing the item it still shows the wrong Result!
this should give a R3 sigil:

It only updates to R3 after i add the +40skill reagent

Note: Probably a Serialization Issue


Build(deps-dev): Bump cypress from 10.3.1 to 10.4.0 (#3822)

Pull request opened by the merge tool on behalf of #3822

Merged in 5fbf61057fdda9bcdacd1db09fc563e4483ff127


dependency Issue with installing Petal component

Hello
I am using Elixir 1.14.0, Phoenix 1.7.0 rc2 on OSX
I am tyring to play with Petal but I am having some installation issues
I am following the guide https://petal.build/components
and when just adding at first the latest version. I am getting some conflicts
❯ mix deps.get
Resolving Hex dependencies...
Resolution completed in 0.093s
Because your app depends on petal_components ~> 0.19 which depends on phoenix_live_view ~> 0.18.3, phoenix_live_view ~> 0.18.3 is required.
So, because your app depends on phoenix_live_view ~> 0.17.5, version solving failed.
** (Mix) Hex dependency resolution failed
I was able to resolve that conflict by upgrading
phoenix_live_view 0.17.12 => 0.18.11
phoenix_live_dashboard 0.6.5 => 0.7.2
So that seems that like something is not setup right in terms of dependencies?

Hey @epinault yeah we depend on the latest live view because of the new declarative attributes. I believe it's backwards compatible so upgrading shouldn't hurt your app


ng2-ui-auth.js not found

Hey there,
First of all thanks for converting this library. I'm having some troubles where system.js isn't loading ng2-ui-auth.
In my login.ts I have,
import {Component, View, AfterContentInit, ElementRef, Renderer} from 'angular2/core';
import {Router, ROUTER_DIRECTIVES} from 'angular2/router';
import {FormBuilder, ControlGroup, Validators} from 'angular2/common';

import {Auth} from 'ng2-ui-auth';

and in my bootstrap.ts I have,
import {NG2_UI_AUTH_PROVIDERS, JwtHttp} from 'ng2-ui-auth';

bootstrap(App, [
   ...
    NG2_UI_AUTH_PROVIDERS({
    	providers: {
    		google: {
    			clientId: GOOGLE_CLIENT_ID
    		},
    		facebook: {
    			clientId: FACEBOOK_CLIENT_ID
    		}
    	}
    }),
    JwtHttp,
    ...
]);

It's also in my dependencies,
  "dependencies": {
    "angular2": "2.0.0-beta.1",
   ...
    "systemjs": "0.19.6",
    "ng2-ui-auth": "~1.0.12"
  },

I'm using Typescript as my compiler. Am I suppose to somehow manually include the library in my index.html?
Thanks in advance,
H.

in systemJS in the config file you can just use
SystemJS.config({
    defaultJSExtensions: true,
    paths: {
        './*': './*',
        '../*': '../*',
        '*': './node_modules/*'
    },
    packageConfigPaths: ['./node_modules/*/package.json'],

and it will load everything automatically from your node_modules folder.
you can see https://github.com/ronzeidman/open-ng2-tests for an example

I'll try it out! Thanks. Angular2 is so confusing to start with.


My Feature Request does not appear on the list

I just created a new feature, and I can see it on my items, but it does not appear on the board under its category.
I think that because of that, it also gives me an error every time I edit it, but the edit work.
The error is because after editing, it send me to the list, and the item is not there:

The steps I followed were:

create a new user using sso with ploi
submit new feature
asked to check the email (so I lost my content and my new feature was gone, this could be another issue to fix).
I created a new feature again.
Save it and now it appears on my Items, but not on the Site list, where is supposed to be.
I edit it, and the edit works, but I get the error above every time I save it.

Not sure why it does not appear on the list, maybe it needs to be approved?

So this is intentional, because the administrators can configure it this way. There's always a way to see your items back by pressing "My items" on the left.

Thanks.
I think this is then an UX bug that would need to be addressed.
The user is never notified about this and there is no way to see what is going on (unless you open a support ticket).

To me, it's pretty clear that there's a "My items" link on the left where you see all your submitted items.

To me is not clear.
So at least one person find it confusing.


Feature/go air

Checklist

[x] I have performed a self-review of my code
[x] I have reached out to another developer to review my code
[x] New and existing unit tests pass locally with my changes
[x] All Lint and CI checks are passing

lgtm king :+1:


failed in running train-vgg-decoder.lua

I run the command th train-vgg-decoder.lua --contentDir images/content/ --styleDir images/style/ on ubuntu, the following error occurred:
/home/ubuntu/torch/install/bin/luajit: gesvd : Lapack library not found in compile time
 at /home/ubuntu/torch/pkg/torch/lib/TH/generic/THLapack.c:124
stack traceback:
	[C]: at 0x7f8ac39e0210
	[C]: in function 'svd'
	/home/ubuntu/torch/install/share/lua/5.1/nninit.lua:235: in function 'initialiser'
	/home/ubuntu/torch/install/share/lua/5.1/nninit.lua:67: in function 'init'
	train-vgg-decoder.lua:120: in main chunk
	[C]: in function 'dofile'
	...untu/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
	[C]: at 0x00406670

You need to install lapack.
sudo apt-get install liblapack-dev liblapacke-dev

(Not sure which is actually used by Torch.)

thank you! It solved by reinstall Torch.


strange background color of nm-applet icon

I'm using Hyprland on a notebook, connected to an external 2K monitor with the scale factor set to 1.2.
After merging #2626, I have noticed that the nm-applet icon is not functioning properly on my external 2K monitor. As seen in the provided screenshot, the background color of the WIFI icon appears to be somewhat strange.

But this issue does not happen on built-in screen:

I reverted #2626 and recompiled Waybar, which resolved the issue at hand (lower one in the above screenshots). While I believe this pull
request has valid points, so I decided to open this issue instead of creating a merge request to revert #2626.
BTW, I'm using Faenza-dark icon theme.
Regards.

sorry, seems my icons are broken...


Why can the dollar value of tokens in Leather deviate from SPOT price on an exchange?

I assume there can be temporary changes especially when markets are volatile and prices fluctuate a lot in a short period of time because it takes time for services to update their prices, and for the wallet to check and download the latest details.
In addition, we have one endpoint and the price there can deviate from the spot price on any given exchange or even another aggregator because they use different sources.
The dollar value in Leather is an approximation? should we be clearer on that?
If the service (to check dollar value) can't be reached larger deviations could be possible. Perhaps not showing the dollar value, or a loading error, would be better in that case. related to: https://github.com/leather-wallet/extension/issues/4787

@mica000 @fabric-8 a lot of apps have disclaimers in the app when displaying fiat conversions. Maybe we need one too?

Additional request, to cover value of LP tokens. https://www.notion.so/trustmachines/User-wants-to-see-dollar-value-of-liquidity-provider-LP-tokens-7df5ad95da50486393dd5bfa6ecada27?pvs=4


Constrain AssetMapper 6.4

In order to fix PhpStan error in #23
Run vendor/bin/phpstan analyse
Note: Using configuration file /home/runner/work/sass-bundle/sass-bundle/phpstan.neon.dist.
 0/7 [░░░░░░░░░░░░░░░░░░░░░░░░░░░░]   0%
 7/7 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] [10](https://github.com/SymfonyCasts/sass-bundle/actions/runs/6655476098/job/18179258827?pr=23#step:5:11)0%

Error: Call to an undefined method Symfony\Component\AssetMapper\Path\PublicAssetsPathResolverInterface::getPublicFilesystemPath().
 ------ --------------------------------------------------------------------------------------------------
  Line   AssetMapper/SassPublicPathAssetPathResolver.php
 ------ --------------------------------------------------------------------------------------------------
  33     Call to an undefined method
         Symfony\Component\AssetMapper\Path\PublicAssetsPathResolverInterface::getPublicFilesystemPath().
 ------ --------------------------------------------------------------------------------------------------

Error:  [ERROR] Found 1 error

Error: Process completed with exit code 1.

Seems getPublicFilesystemPath() is gone from the Symfony\Component\AssetMapper\Path\PublicAssetsPathResolverInterface in 6.4. I suppose we should release this fix, and then drop getPublicFilesystemPath() from ours SassPublicPathAssetPathResolver() as well and release it again?
Any thoughts?

I think we should not completly drop the support. You have a working example of the phpstan fix here: src/AssetMapper/SassPublicPathAssetPathResolver.php

Hm, I see now... probably suggest to upgrade sass bundle in the throwing exception when method does not exist?

It should never reach the exception. It's only to fool phpstan

With latest asset mapper but older sass bundle it's pretty possible I think.
OK, closing it in favor of https://github.com/SymfonyCasts/sass-bundle/pull/22


Wrong prices

Wrong CIS dollar prices.
For example, GTA V costs $49.99 in CIS, but in Steamdb (CIS dollars) it costs $39.99

It's 39.99 in Azerbaijan, which is the country we use to check CIS prices.


manage.py celeryctl doesn't appear to work

I've got a Flask app using celery, I've confirmed that manage.py celeryd works just fine and is running.
But when I do any manage.py celeryctl command I get an error:
python manage.py celeryctl status
usage: manage.py celeryctl [-h]
manage.py celeryctl: error: unrecognized arguments: status
python manage.py celeryctl purge
usage: manage.py celeryctl [-h]
manage.py celeryctl: error: unrecognized arguments: purge
python manage.py celeryctl -h
usage: manage.py celeryctl [-h]
optional arguments:
-h, --help  show this help message and exit

I found the reason for the celeryctl problem.
I contributed a patch to Flask-Script so that commands could override how command-line options
was parsed, this commit was accepted and integrated, but in the recent version
it is simply gone.  Very strange.
I'm going to fork Flask-Script and upload a custom version to PyPI so that we can get this working
ASAP.

Fixed now.  We use the 'Flask-Script-fix' distribution instead, which installs itself as flask.ext.script.
You have to update your files to use 'flask.ext.script' instead of 'flaskext.script' (e.g. see updated examples/)

Thanks that's working perfectly!


Validation.js

As a baby programmer, i am using this great repo to learn best practice and how to write clean code .Using the utils/validation.js  in another project, i got the the following error. rule is not a function. Please how can i solve this. Thanks

Sorry for the late reply.
Can you find "src/utils/validation.js" and add console.log(rules) after "export function createValidator(rules, params) {" (line# 57) to make sure all rules are function?


[BEAM-11457] Add option to skip key-value clone

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aromanenko-dev

@JozoVilcek Thanks, I'll take a look asap.

retest this please

retest this please

@aromanenko-dev change is ready for another review round. Apologies for delay.


Supporting for hidden attribute of @ApiParam and @ApiModelProperty annotations.

PR introduces functionality that will support hidden attributes.
Unit test is provided.
Related issue is #1382

I don't know how to make circleci pass the unit test.
The error is:
unable to resolve class springfox.documentation.swagger.readers.parameter.SwaggerExpandedParameterBuilder
It works perfectly on my laptop. Looks like it isn't able to find class from springfox-swagger-common package.

I'll take a look

Thanks for the PR! :metal:

Hi,
Thank you for the PR, i am awaiting for this to work !
When do you think we will get this new version  to be downloaded ?
Thank u

Im hoping to get it out by 1st of Oct

Hi.
@mzaarour
You can build your own snapshot by cloning project and running
./gradlew clean build

from the root directory of the project.

Hi all,
Thank you for your response.  I can wait for the next release.


Hello, i have problem with PermisionEX

Hello, when the server is firing, PermisionsEx is working, but when the server is started it pops up such an error: [11:52:17 ERROR]: [PermissionsEx] Error occurred with PermissionsEx! Please post it to https://github.com/PEXPlugins/PermissionsEx/issues/new?title=&body=%23%23%23+Description+%23%23%23

[Insert+description+of+issue+here]
%23%23%23+Detailed+Information+%23%23%23

[Is+available+here](https%3A%2F%2Fgist.github.com%2Fcdb5c4dd8f99f56821e889d308428ba4)
. Full error:
Please help

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


Code factorization for errors handling

I noticed we use the following piece of code in the FAPI\Boilerplate\Api\Tweet class.
        // Use any valid status code here
        if ($response->getStatusCode() !== 201) {
            switch ($response->getStatusCode()) {
                case 400:
                    throw new DomainExceptions\ValidationException();
                    break;
                default:
                    $this->handleErrors($response);
                    break;
            }
        }

If we assume that other classes like Tweet will be created, do you think that this could be factorized in a function for every request type (get, create, update, delete)?

Different calls can have different errors, but a class level error handler which contains all the common cases should work.


[WIP] KYCAID integration

[x] KYCAID configuration
[x] Add identifier to Document model
[x] Modular KYC switch
[x] Applicant worker
[x] Profile manual / kycaid
[x] Document worker
[x] Document worker Specs
[x] Fog::File to KYCAID Tempfile
[x] Address Worker
[ ] Address Worker Specs
[x] Callback
[ ] Callback worker and specs
[ ] QA

[ ] Let's rename KYCAID to Kycaid
[ ] Create a public module for api and move kyc (copy version, ping, ...)


[AKS] fix: add arguments removed by a previous PR

Description

This PR adds back two arguments that were removed in a previous PR.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AKS


when i use Textarea Component it's return an input

hey
i copy this code from doc to my component
import {
    Box,
    Button,
    Checkbox,
    ControlFeedback,
    FormCheck,
    FormCheckLabel,
    FormGroup,
    Input,
    Label,
    Radio,
    RadioGroup,
    Select,
    Textarea,
    Typography
} from "@smooth-ui/core-sc";

<Textarea size="lg" placeholder="Large" />

and this return an input
<input class="sui-input kgmrda-0 fcwcRG" placeholder="Large">
how can i fix this?

@svahdati can you try to upgrade it?

@neoziro Thanks, finally it's work.

Great 👍


Flags should be set to prevent duplicate initialization.

It is unexpected to initialize the database directly when container restart or application restart. It may be a coding mistake, because the flag createDatabase was designed.
Unauthorized access also will be caused by using admin, 123 after restart, which introduces security risks.
I will submit a pull requests to fix it later.

@dextercai plz make a PR

@dextercai it just injects some example data for first-time use, does it harm your service?

But the service should not injects a full-admin user after each container or application restart.

Otherwise, it should be clearly pointed out in the document that when the createDatabase parameter is not specified, the restart of the container or application will still cause the insertion of sample data, especially a high-level admin user.

Fixed in: https://github.com/casdoor/casdoor/commit/7a95b9c1d53f45fb0a938428c105f3524222c0f1


ImportError: cannot import name 'is_union' from 'pydantic.typing'

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug

import openai
Traceback (most recent call last):
File "", line 1, in
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/init.py", line 8, in
from . import types
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/types/init.py", line 5, in
from .edit import Edit as Edit
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/types/edit.py", line 6, in
from .._models import BaseModel
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/_models.py", line 33, in
from ._utils import (
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/_utils/init.py", line 2, in
from ._utils import flatten as flatten
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/_utils/_utils.py", line 22, in
from .._compat import is_union as _is_union
File "/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/openai/_compat.py", line 55, in
from pydantic.typing import is_union as is_union
ImportError: cannot import name 'is_union' from 'pydantic.typing' (/demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/pydantic/typing.cpython-39-x86_64-linux-gnu.so)

To Reproduce
python -c "import pydantic.utils; print(pydantic.utils.version_info())"
pydantic version: 1.7.4
pydantic compiled: True
install path: /demo/miniconda3/envs/sqlcode/lib/python3.9/site-packages/pydantic
python version: 3.9.18 (main, Sep 11 2023, 13:41:44)  [GCC 11.2.0]
platform: Linux-5.15.0-88-generic-x86_64-with-glibc2.35
optional deps. installed: ['typing-extensions']
import openai
Code snippets
No response
OS
linux
Python version
python 3.9
Library version
openai 1.1.0

Hey @dshwei, we only support Pydantic >= v1.9.0 and it looks like you're using v1.7.4, please upgrade to a later version!


Modify copyright notice

Remove "All materials copyright Sydney Informatics Hub, University of Sydney" or make it much smaller and push it away from the main body of text.

#14


[PDI-19836] Handle file cases and missing destination cases

Cases found in further testing.

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 57m 40s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 4099, Failures: 0, Skipped: 13    Test Results

:information_source:  This is an automatic message


Transfer of ownership

@Flaxbeard Please transfer ownership of this repository to the organization, Esteemed-Innovation.

@Flaxbeard In case I need to add some reasoning:

We may need to add/remove people as contributors in the future.
We may want to do FSP-related projects as well as FSP.
We may want to change the description and URL
We may want to change some settings such as webhooks and services
We may decide to change the name because having someone's name in a mod name is dumb. It's not "Azanor's Thaumcraft" or "CovertJaguar's Railcraft".

@Flaxbeard I'll add you and all the other contributors to the organization as soon as the repo gets transferred over. Don't tell me to fork it, because Git doesn't allow searching forked repos or transferring issues from the original to a fork. There are many other features that normal repositories have but forks don't.

Forks are buggy and annoying :/
I'm against changing the name, just "Steam Power" or "SteamCraft" sounds dumb.

I was thinking "Esteemed Innovation".

That's better. Still though, changing the mod's name is likely to cause much confusion, it's already well-known as FSP.

I dunno. I feel weird working on a project that is labeled as "One other person's project". The name's also changed like twice in the past already. The name would be changed only once we move out of beta and into release.

Done.

Okay, so it turns out you, @Flaxbeard, have to be an owner of the org to transfer ownership. Pls try it again bb <3