[TS migration] Migrate 'WorkspaceList' page to TypeScript

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/31994
PROPOSAL: NA
Tests

Test case 1:

Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Click on "New workspace"
Click on the back button to go back to Workspace list screen
Verify that the new workspace in step 3 is displayed and all the workspaces are sorted in alphabetical order

Test case 2:

Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Click on any workspace
Click on three dot => delete workspace
Verify that the deleted workspace in step 3 disappeared

Test case 3:

Sign in with account that does not have any workspace
Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Verify that the empty workspace list screen is displayed (that contains "Manage business expenses, issue cards, send invoices, and more." )

Test case 4:

Open offline mode
Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Click on "New workspace"
Click on the back button to go back to Workspace list screen
Verify that the new workspace in step 4 is displayed and all the workspaces are sorted in alphabetical order
Turn off offline mode
Verify that the new workspace in step 4 is displayed and all the workspaces are sorted in alphabetical order

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

Offline tests

QA Steps

Test case 1:

Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Click on "New workspace"
Click on the back button to go back to Workspace list screen
Verify that the new workspace in step 3 is displayed and all the workspaces are sorted in alphabetical order

Test case 2:

Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Click on any workspace
Click on three dot => delete workspace
Verify that the deleted workspace in step 3 disappeared

Test case 3:

Sign in with account that does not have any workspace
Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Verify that the empty workspace list screen is displayed (that contains "Manage business expenses, issue cards, send invoices, and more." )

Test case 4:

Open offline mode
Go to the settings screen
Click on "Workspaces" options to go to Workspace list screen
Click on "New workspace"
Click on the back button to go back to Workspace list screen
Verify that the new workspace in step 4 is displayed and all the workspaces are sorted in alphabetical order
Turn off offline mode
Verify that the new workspace in step 4 is displayed and all the workspaces are sorted in alphabetical order

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

PR Author Checklist

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

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

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

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

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

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

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

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Screencast from 20-01-2024 18:43:23.webm

Android: mWeb Chrome

Screencast from 20-01-2024 17:59:30.webm

iOS: Native

iOS: mWeb Safari

Screencast from 20-01-2024 18:02:33.webm

MacOS: Chrome / Safari

Screencast from 17-01-2024 14:43:40.webm

MacOS: Desktop

Screencast from 20-01-2024 18:10:23.webm

Please change the PR title to something more explanatory

Can you please fix the lint / typescripts errors and let me know when this is ready for another review

I will ping you soon

@s77rt I`ve fixed all your comments. Please help review again

Please rename the PR to: [TS migration] Migrate 'WorkspaceList' page to TypeScript

@blazejkustra I fixed all your comments

@s77rt Thanks for your suggestions. All are updated

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[X] I verified proper code patterns were followed (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

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

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

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If the PR modifies the form input styles:

[X] I verified that all the inputs inside a form are aligned with each other.
[X] I added Design label so the design team can review the changes.

[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@DylanDylann Please add the missing screenshots and write a clear testing steps

@s77rt Added the missing screenshots and test cases. Also, I have a problem with ios native so can not test on it. Please help me on it

@francoisl

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

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.4.30-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.30-1 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.30-1 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Cannot read property 'call' of undefined using react-popper

[x] I have searched the issues of this repository and believe that this is not a duplicate.

First let me thank you guys for this wonderful library, I know the effort you are putting here. thanks
Expected Behavior

I'm using react-popper library for tooltips positioning.
it works in dev env, and it should work in prod env using next build command.
Current Behavior

it breaks in build mode when importing the above library
Steps to Reproduce (for bugs)

yarn add react-popper
integrate the sample code from the readme
yarn run build alias for next build

Context

I want to use react-popper for tooltip positioning
Your Environment

Tech
Version

next
5.1.0

node
v8.5.0

OS
macOS High Sierra v10.13.4 (17E199)

browser
Chrome Version 66

It's probably something related to how webpack bundles this module.

@timneutkens can you elaborate more, how can I debug more on the webpack bundling side ?


[FEATURE] Current Player box can change color depending on current user

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
Current Player box at the top remains the same color despite the turn being of either player X or O which may make it a little harder to notice.
Proposed Solution
Like in the tic-tac-toe board, it can change color to cyan or reddish depending on current player. It will be more visually appealing as well as noticeable.
Screenshots

Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this (issue will not be assigned if this is skipped)
I would like to modify the css properties and maybe a little bit of script in order to make sure that Current Player box changes color/glow depending on current user.

Please assign the required labels to the issue and assign it to me. Thank you.

I would like to implement something like this :


Cleanup of unneeded files?

After creating about 5 new functions projects and deleting the assets/ folder every time, I took a look in the src for this plugin to see if it would be an easy addition. To my surprise I found that the code already exists but is never called.
Is there a reason why it is never called?

Wow! Great spot. Indeed, that code should be called but isn't.
Ok let's get that fixed for the next release then 👍

Looking into this again, I believe I left assets in the function node app, just in case anyone wanted to override environment vars per function. Not sure how useful that is tbh. 🤔


feat: wireframe-to-code

Some corners were cut. Will be refactored later.

@dwelle, thanks for this. Should we be able to enable it in our host (package) apps? If so, is there a guide? The source files for the package example appear to be private so I'm not able to view them.


Update sbt-tpolecat to 0.2.3

Updates io.github.davidgregory084:sbt-tpolecat from 0.1.22 to 0.2.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.davidgregory084", artifactId = "sbt-tpolecat" } ]

labels: sbt-plugin-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #59.


Update CI to use C++14 version of XL Compiler

[x] Update to radiuss_configs one PR is merged there.  Then this will be ready for merge.

@davidbeckingsale - I needed to bring in the newly merged main branch of radiuss-spack-config, kindly please re-review.
thanks,


Use creator venv when creating virtual environments

I did not realize how broken pipenv is out of the box on the latest ubuntu until I upgraded.   It has to do with it defaulting to posix_local when creating a virtualenv, which has different paths for the python binaries.  I initially tried preferring posix_local for the paths but it had the side effect of trying to install everything to local site packages.
Risks:  The venv creator should be available on all systems, but the documentation points out:

venv - this delegates the creation process towards the venv module, as described in PEP 405. This is only available on Python interpreters having version 3.5 or later, and also has the downside that virtualenv must create a process to invoke that module (unless virtualenv is installed in the system python), which can be an expensive operation (especially true on Windows).

I was hoping this would count towards my Hacktoberfest PRS :-/


Issue with "add-domain-admin.sh"

Hello, are the scripts compatible with iredmail 1.3.2 ?
I'm using the add-domain-admin.sh
After using the sql commands i can't log in with the account i tried to change to a domain admin.
The url shows https://domain/iredadmin/domains?msg=PERMISSION_DENIED

Hello, I apologise for the delay. I will take a look at this

I can replicate this by using both iRedAdmin and my script, however the issue seems to be with iRedAdmin. If you mark a user as a Domain Admin (not global) they receive this error logging in. However if you then add Global Admin permission they are able to login successfully.
While its not ideal to have more permission than necessary you can work around this for now by making the user Global Admin:
sh new-domain-admin.sh<EMAIL_ADDRESS>ALL


UAP13:AutoUpdate child element is missing from the Package level Extension element.

The uap13:AutoUpdate element documented here: https://docs.microsoft.com/en-us/uwp/schemas/appxpackage/uapmanifestschema/element-uap13-autoupdate indicates that the parent element should be the package level Extension element.  The AutoUpdate is not listed as a child element on this page.
This is particularly confusing as I have seen information that AutoUpdate should be a child of the Properties element (it is not listed in the documentation under that element either).
Please confirm correct location and update appropriate parent element.
[Enter feedback here]

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

ID: 62762880-6dfc-c9a0-f040-92a359de913e
Version Independent ID: 47f6f79b-4c35-9075-c566-3eeda412ba1f
Content: Extension (in Package/Extensions) (Windows 10) - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-extension.md
Product: uwp
Technology: winrt-reference
GitHub Login: @drewbatgit
Microsoft Alias: drewbat

Update:  I found a reference in the FoundationManifest schema that shows CT_Properties as having uap13:AutoUpdate as a child.
If you confirm that Properties is the correct parent, then the PackageExtension page is correct and both Properties, uap13:AutoUpdate, and uap13:AppInstaller pages need updating instead.

@TimMangan Good catch. uap13:AutoUpdate is a child of Package/Properties. I updated the element hierarchies for uap13:AutoUpdate and uap13:AppInstaller. It looks like uap13:AutoUpdate had already been added to the Package/Properties page in the long span between when you opened the issue and today, so it didn't need to be updated.


Add prefix to all MrVector attributes

Bit of an ugly fix, and it breaks compatibility with http://ozodrukh.github.io/Svg2MrVector/, but does allow inclusion with other attr declarations.

Fixes #14

Thanks @dambrisco! do you mind using mv_attribute instead of mvAttribute so one just have to prepend mv_ and not go nuts over camel case?

Done. Definitely makes more sense with mv_

Awesome, will deploy later today, thanks again @dambrisco!

I've also taken the liberty of re-creating the necessary MrVector support change off of a true fork of svg2android. It can be found here: http://dambrisco.github.io/svg2android/
I have a PR into svg2android, but I wouldn't be surprised if they turned it down. I'm happy to maintain it separately if they don't as it does have a few fixes that the version you currently link to doesn't.

no, they will be glad to accept your PR, they already wrote to make PR with MrVector support, but i forgot  :(

@ozodrukh Yeah, I saw that and that's what actually prompted me to put that together. Hope you don't mind! Reading their updated README it sounded like they were looking to just support canonical VectorDrawable, but it'll be great if they do accept it.


Spliting context to a separate file doubles the result

Below are two versions of my code. When the context is not split into a separate file, the result is only value, working as expected. However when it is split, then the result is double as valuevalue. I can't reasoning why this is the case. Do you have any idea?
Case 1: Not splitting context to another file
routes/index.tsx
import { User } from "../../fresh-project/islands/User.tsx";
import { createContext } from 'preact'

export const Username = createContext()

function ContextProvider({children}) {
  return (
    <Username.Provider value="value">{children}</Username.Provider>
  );
}
export default function App() {
  return (
    <ContextProvider>
      <User />
    </ContextProvider>
  )
}

islands/User.tsx
import { useContext } from 'preact/hooks';
import { Username } from '../routes/index.tsx'

export function User() {
  const username = useContext(Username);
  return <>{username}</>;
}

The result of this is value.
Case 2: Splitting context to another file
routes/index.tsx
import ContextProvider from '../islands/Context.tsx'
import { User } from "../../fresh-project/islands/User.tsx";

export default function App() {
  return (
    <ContextProvider>
      A<User />a
    </ContextProvider>
  )
}

islands/Context.tsx
import { createContext } from 'preact'

export const Username = createContext()

export default function ContextProvider({children}) {
  return (
    <Username.Provider value="value">{children}</Username.Provider>
  );
}

islands/User.tsx
import { useContext } from 'preact/hooks';
import { Username } from './Context.tsx'

export function User() {
  const username = useContext(Username);
  return <>{username}</>;
}

Then at first the result is Avaluea as expected, but quickly rendered to Avalueavaluea.
Case 3: Adding a character after {children} on Context.tsx
import { createContext } from 'preact'

export const Username = createContext()

export default function ContextProvider({children}) {
  return (
    <Username.Provider value="value">{children}c</Username.Provider>
  );
}

Then at first the result is Avaluea as expected, but quickly rendered to Avalueacac.

The issue is that using context to pass data to island is not supported in Fresh.
Closing as duplicate of https://github.com/denoland/fresh/issues/983 .


Add full update_metadata package URL for clarity

Just a quick change so that others won't fall into the same navigational trap that I did while trying to find the original source.

Thanks !


Add more contractions for FirstPerson rule

This PR adds the contractions I'd, I'll and I've to the FirstPerson rule.

Thanks!


python311Packages.boschshcpy: 0.2.84 -> 0.2.88

Diff: https://github.com/tschamm/boschshcpy/compare/0.2.84...0.2.88
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 278274 run on aarch64-darwin 1

4 packages built:

python310Packages.boschshcpy
python310Packages.boschshcpy.dist
python311Packages.boschshcpy
python311Packages.boschshcpy.dist


Add TestGetEntries

Simple UT for GetEntries
Better error handling instead of panics in storage package
Run tests on windows platform as well in github actions

@plutov Thank you! I recommend afero for testing the filesystem


feat(persons): use person uid as primary key in indexedDb

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


Not constant map that looks like a statement is not parsed

f() {
  {'a' : 0, 'b' : 1}.length;
}

library;
import self as self;

static method #main() → dynamic {
  throw "file:///test.dart:2:8: Error: Expected ';' before this.\n  {'a' : 0, 'b' : 1};\n       ^";
}

Another instance of deprecated_InputError. Please keep reporting these issues, I really appreciate the reproduction tests. Let me know if this is blocking you.

This issue might be impacted by https://dart-review.googlesource.com/c/sdk/+/12040.

Yes, also no failing tests in Analyzer marked with this issue.


Add Control events: on_did_mount and on_will_unmount

What will they be used for?

@FeodorFitsner what's the plan over here?


Remove enterprise code from core

This removes enterprise commands that didn't work in core, as well as APIs that weren't available.
Outstanding work:

Document how to get the enterprise commands back.

This PR is stale, closing.


feat(hummingbird): option to override campaign form submit function

We could focus on making onSubmit more flexible by letting it do the regular submit logic only when the onSubmit prop:

resolves (so we await its result, then the handler can do async work)
doesn't throw (wrap it in try/catch)
returns true (like how native JS events work)

Then we don't need another prop at all.
Do you see any downsides to this approach, considering the different campaign types?


Added Python tutorial at Tutorialspoint

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

we've been adding tutorials point content in courses and in books. See #4573 and comment if you think they should go into interactive tutorials.

based on your comments, add them in books instead.

based on your comments, add them in books instead.
Hi @eshellman, as you said I added them to books. I hope it's good now.

Hi @eshellman thank you for merging this PR.
Can you add a 'hacktoberfest-accepted' label to it, please?

Hi @eshellman thank you for merging this PR.
Can you add a 'hacktoberfest-accepted' label to it, please?

There is no need for the label.
PRs only need to be merged, approved OR get the label.
Plus Hacktoberfest should be over by now.

Hi @eshellman thank you for merging this PR.
Can you add a 'hacktoberfest-accepted' label to it, please?

There is no need for the label.
PRs only need to be merged, approved OR get the label.
Plus Hacktoberfest should be over by now.

Okay, thank you for clearing my doubt.
And yes it's over now.


Questions about Silent Renewal

First of all, thank you so much for this library. Wonderful work!
I have some questions about Silent Renewal for Implicit Flow:

How long is the default lifetime of the id token?
Can we configure the id token lifetime?
Can we configure so that auto silent renewal only triggered when the user is active? When the user is inactive, auto silent renewal should not be triggered.

Many thanks

Hi and many thanks for the kind words.
About your questions:

The token lifetime is dependent on the token server, not the client. You won't find any configuration options for that here, since this is a library for the client.
This depends on whether or not you have control about the configuration of the token server.
This is an interesting question I don't know the answer to. Please direct any questions about this to the maintainer of oidc-client.

Kind regards


Test Issue By Bytepay: befitting bomb

This issue is for bytepay test

/bytepay task 0.001DOT

Bytepay: Task has been assigned to @Emptycity-tear.

@Emptycity-tear:
Bind Success.
Now Your Own Polka Address Is: 5FU6HZLVj74hF1sBxAFQ7PkS6HQbVF3gAFDLPLV37YFNKNjV
Previous Is: 5FU6HZLVj74hF1sBxAFQ7PkS6HQbVF3gAFDLPLV37YFNKNjV

/bytepay pay

@Emptycity-tear:
sulnong have paid for this task, visit https://westend.subscan.io/extrinsic/0xad14a0ea61ac0c50bf309b7a0117f419326cb394758d8bd9d1ecdc3528ff7d87 for detail


name="(It is not a test)" in jUnit xml

hello, after running test via sbt in jUnit xml i have all test names like
                 <testcase classname="websocket.SomeTestObj" name="(It is not a test)" time="14.199">

any issues here?

Update guys!
This is sbt bug and fixed with 1.1.0 sbt version, now its fine!


Parse the test cases results log by json

Generate a json file to collect the test results.
Locate easily the failed tests in result file when using the results file
Quickly generate results to match the specified format that QA required
JSON dose not need to take up extra workspace.

Signed-off-by: Wang Hongjuan<EMAIL_ADDRESS>

@wanghongjuan please fix this syntax error:
https://travis-ci.org/otcshare/iotivity-node/jobs/119462483#L1478

@gabrielschulhof I have been fixed the syntax error, but My code failed to access the Travis CI build.


Doc: Add links to matches.

Extracted from #2399. I'm trying to tidy it a bit, and it'll makes things easier to have this submitted separately.
cc @captbaritone

Coverage remained the same at 96.763% when pulling b24561128280f8b789e7bf52a4e6a8ae49f7a39e on Florian-R:matches-link into c2e2eff828f73eaf4490353f83724c357acb7fdf on jashkenas:master.

Coverage remained the same at 96.763% when pulling b24561128280f8b789e7bf52a4e6a8ae49f7a39e on Florian-R:matches-link into c2e2eff828f73eaf4490353f83724c357acb7fdf on jashkenas:master.

Thanks!


pystorm.log.path must be set?

I've been running my topology on 344ce8cbae7e3202876227e3942741d62d767250 and am trying to move to 3.0.0. However a8861e4af53a5236605424324927e59ce7711ca7 results in this exception on sparse submit:
Traceback (most recent call last):
  File "/Users/tdhopper/miniconda/envs/dir/bin/sparse", line 11, in <module>
    sys.exit(main())
  File "/Users/tdhopper/miniconda/envs/dir/lib/python2.7/site-packages/streamparse/cli/sparse.py", line 56, in main
    args.func(args)
  File "/Users/tdhopper/miniconda/envs/dir/lib/python2.7/site-packages/streamparse/cli/submit.py", line 243, in main
    override_name=args.override_name)
  File "/Users/tdhopper/miniconda/envs/dir/lib/python2.7/site-packages/streamparse/cli/submit.py", line 208, in submit_topology
    env_config, nimbus_client, options=options)
  File "/Users/tdhopper/miniconda/envs/dir/lib/python2.7/site-packages/streamparse/cli/submit.py", line 72, in _submit_topology
    print("Routing Python logging to {}.".format(options['pystorm.log.path']))
KeyError: u'pystorm.log.path'

It goes away if I set the log path for my environment in config.json. However, I have it unset so python logs go to worker.log.

This is what I get for sneaking in last minute changes.  Will fix and push out 3.0.1 very soon.

Fixed and streamparse 3.0.1 is up on PyPI

Thanks @dan-blanchard. You do great work!


App fails to deploy with Amplify 1-click

I followed the directions for the Amplify 1-click deployment. The Backend build failed.
The relevant parts of the Amplify Build console show:
2022-06-30T17:12:58.779Z [INFO]: CREATE_IN_PROGRESS AmplifyResourcesPolicy AWS::IAM::Policy      Thu Jun 30 2022 17:12:54 GMT+0000 (Coordinated Universal Time)
  CREATE_IN_PROGRESS LambdaFunction         AWS::Lambda::Function Thu Jun 30 2022 17:12:54 GMT+0000 (Coordinated Universal Time)
  CREATE_IN_PROGRESS AmplifyResourcesPolicy AWS::IAM::Policy      Thu Jun 30 2022 17:12:55 GMT+0000 (Coordinated Universal Time) Resource creation Initiated
  CREATE_IN_PROGRESS LambdaFunction         AWS::Lambda::Function Thu Jun 30 2022 17:12:56 GMT+0000 (Coordinated Universal Time) Resource creation Initiated
2022-06-30T17:13:03.775Z [INFO]:
2022-06-30T17:13:03.776Z [INFO]: CREATE_FAILED LambdaFunction AWS::Lambda::Function Thu Jun 30 2022 17:13:00 GMT+0000 (Coordinated Universal Time)
    Resource handler returned message: "User: arn:aws:sts::111111111111:assumed-role/amplifyconsole-backend-role/BuildSession is not authorized to perform: lambda:PutFunctionConcurrency on resource:
    arn:aws:lambda:us-west-2:111111111111:function:S3Trigger984fb593-devb because no identity-based policy allows the lambda:PutFunctionConcurrency
    action (Service: Lambda, Status Code: 403, Request ID: 3a417a78-3416-4ab9-a589-0e6945fe3947)" (RequestToken: 1a65c44a-93cb-86c6-b612-4a48af0dfe0f, HandlerErrorCode: AccessDenied)
2022-06-30T17:13:18.778Z [INFO]:
2022-06-30T17:13:18.781Z [INFO]: CREATE_COMPLETE AmplifyResourcesPolicy                                                 AWS::IAM::Policy           Thu Jun 30 2022 17:13:13 GMT+0000 (Coordinated Universal Time)
                                 CREATE_FAILED   amplify-photoshare-devb-170520-functionS3Trigger984fb593-19MV1K7RNRV4I AWS::CloudFormation::Stack Thu Jun 30 2022 17:13:14 GMT+0000 (Coordinated Universal Time) The following resource(s) failed to create: [LambdaFunction].
2022-06-30T17:13:28.887Z [INFO]:
2022-06-30T17:13:28.888Z [INFO]: CREATE_FAILED functionS3Trigger984fb593 AWS::CloudFormation::Stack Thu Jun 30 2022 17:13:28 GMT+0000 (Coordinated Universal Time)
    Embedded stack arn:aws:cloudformation:us-west-2:111111111111:stack/amplify-photoshare-devb-170520-functionS3Trigger984fb593-19MV1K7RNRV4I/d0a04fc0-f897-11ec-abb8-02cf258d1d13
   was not successfully created: The following resource(s) failed to create: [LambdaFunction].

I also have this issue. The workaround is to update the amplifyconsole-backend-role with an additional policy allowing "lambda:PutFunctionConcurrency". Would be good to get the guide updated to include this step.


Enable CD for Cyberchief Security Scanner Plugin

Link to GitHub repository
https://github.com/jenkinsci/cyberchief-security-scanner-plugin
Link to the PR enabling CD in plugin
https://github.com/jenkinsci/cyberchief-security-scanner-plugin/pull/2
When modifying release permission
If you are modifying the release permission of your plugin or component, fill out the following checklist:

### Release permission checklist (for submitters)
- [ ] The usernames of the users added to the "developers" section in the .yml file are the same the users use to log in to [accounts.jenkins.io](https://accounts.jenkins.io/).
- [ ] All users added have logged in to [Artifactory](https://repo.jenkins-ci.org/) and [Jira](https://issues.jenkins.io/) once.
- [ ] I have mentioned an [existing team member](https://github.com/orgs/jenkinsci/teams) of the plugin or component team to approve this request.

When enabling automated releases (cd: true)
Follow the documentation to ensure, your pull request is set up properly. Don't merge it yet.
In case of changes requested by the hosting team, an open PR facilitates future reviews, without derailing work across multiple PRs.
Link to the PR enabling CD in your plugin

### CD checklist (for submitters)
- [x] I have provided a link to the pull request in my plugin, which enables CD according to the documentation.

### Reviewer checklist
- [ ] Check that the `$pluginId Developers` team has `Admin` permissions while granting the access.
- [ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
- [ ] If security contacts are changed (this includes add/remove), ping the security officer (currently `@Wadeck`) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it.

You forgot to incrementalify the plugin: https://www.jenkins.io/doc/developer/publishing/releasing-cd/#update-maven-pom-and-config

@NotMyFault Fixed :+1:

Hi @NotMyFault Thanks for merging the PR! May I know what steps I need to take next in order to release the plugin? I tried to follow the steps in this document, but Im getting an error regarding "not-setup-for-mrp"

Does it trigger the release if I added the label after I merged a PR? For example, this PR

Does it trigger the release if I added the label after I merged a PR? For example, this PR

If you put an interesting label on it, for sure :)

Does it trigger the release if I added the label after I merged a PR? For example, this PR

If you put an interesting label on it, for sure :)

Ahh there it is. Thank you so much! :raised_hands:  I can see it now in repo.jenkins-ci but is that all? Or are there other steps to take to have the plugin on https://plugins.jenkins.io/


Add is_taskset and load_taskfile functions

Useful for wrappers.

This PR has had no activity for a long time and has conflicts, so I'm closing it. Please reopen or contact me as necessary.


Add Terra Wallet

Terra uses default curve as implemented getPublicKeySecp256k1 (compressed).
Reference: https://github.com/trustwallet/wallet-core/blob/master/android/app/src/androidTest/java/com/trustwallet/core/app/blockchains/terra/TestTerraTransactions.kt

@diopisemou Terra doesn't seem to support signJson from the trustwallet library. But this implementation seems to depend on that. I checked from here
https://github.com/trustwallet/wallet-core/blob/master/android/app/src/androidTest/java/com/trustwallet/core/app/blockchains/terra/TestTerraTransactions.kt
https://github.com/trustwallet/wallet-core/blob/master/swift/Tests/Blockchains/TerraTests.swift


Initial Saturn support

This gets some basic Saturn support working. I needed to update to Ubuntu 22.04 to get binutils-sh-elf, which doesn't seem to be in 20.04. An example to test with is:
Compile with -O0
mov.l   r14,@-r15
sts.l   pr,@-r15
add     #-4,r15
mov     r15,r14
mov.l   r4,@r14
mov.l   @r14,r0
cmp/eq  #1,r0
bf      16
mov     #0,r0
bra     38
nop
mov.l   @r14,r7
mov     r7,r4
mov     #10,r5
mov.l   44,r6
jsr     @r6
nop
mov     r0,r2
mov     r2,r1
shll2   r1
add     r0,r1
mov     r1,r2
add     r2,r2
mov     r7,r1
sub     r2,r1
mov     r1,r0
bra     38
nop
add     #4,r14
mov     r14,r15
lds.l   @r15+,pr
mov.l   @r15+,r14
rts
nop
.word 0x0000

int test(int arg0)
{
    if (arg0 == 1)
    {
        return 0;
    }

    return arg0 % 10;
}

This example doesn't totally match since I think I had the compiler settings different for the generated assembly. We don't have splitting tools for Saturn yet so this work is still somewhat speculative. I suspect this is the compiler used for Saturn SOTN but I'm not sure.
I needed to wrap the executables in batch files because they expect a particular PATH environment to work.

Exciting! Sucks we need dosbox though :/ and wondering why you bumped python to 3.10 - is that just the default for the newer ubuntu?

As far as I can tell there is no Python3.9 package in 22.04. https://packages.ubuntu.com/search?suite=jammy&section=all&arch=any&keywords=python3.9&searchon=names
I just tried running the compiler in Wine and get this:
winevdm: Z:\GCCSH\BIN\CC1.EXE is a DOS application, you need to install DOSBox.

I went with dosemu since it's the solution that that PS1 DOS compilers are using.

Makes sense. Could you fix the black errors / non-docker build as well (.github/workfflows/ci.yml)? And it looks like the docker test is failing too...something with the sandbox. have you run it locally with the sandbox?

So running the full CI requires your approval? I haven't tried running tests locally, I haven't been able to run this outside docker due to issues with poetry being unable to finish an install.

It requires my approval since you're a new contributor, yes. What's the issue with poetry?

the setup python thing failed trying "3.1"??? I guess just keep it at 3.9 for now

On my Linux VM poetry gets stuck forever trying to install. I think it's this issue: https://github.com/python-poetry/poetry/issues/3352
I've tried using the old installer and removing the cache but it still gets stuck.
On my Mac I run into this
  Hash for rapidfuzz (2.13.7) from archive rapidfuzz-2.13.7-cp310-cp310-macosx_10_9_x86_64.whl not found in known hashes (was: sha256:3d3843b66a57d566b2a4c0ed85c4d87b6d46aef8e5116fb08a89f0c6fb737ab5)

I think the issue with the docker tests is an invalid argument: 'sh-elf-objdump --disassemble --disassemble-zeroes --line-numbers --reloc --start-address=0 -m mips:4300 /tmp/out.s'. The mips part shouldn't be there. I'll see if I can figure out a workflow to debug this.

could it be because of this ? https://github.com/sozud/decomp.me/blob/saturn/backend/coreapp/diff_wrapper.py#L231

I think I figured out the issue, somehow I lost the change with the asm-differ hash update. Could someone approve the workflow?

It looks like everything is passing now, are any other changes needed?


Cannot find module "react-addons-shallow-compare"

I use yarn to install 4.2.0.
After that I installed “react-addons-shallow-compare”，but  the problem still exists.

Please make sure you have all the proper peerDependencies installed with the proper versions, and that all of the react dependencies have identical versions.

The problem is solved.Thanks!


Search results display Template page

When using the search bar, the results will lead to the example page instead of the expected results.
Repro steps:

Open web page
In search bar, search a term that shows at least one result
Click any of the results
 Notice you will be redirected to the example web site instead of the expected page

Expected Behavior
When a result is clicked, the user should be redirected to the respective webpage.
Screenshots

OS: Windows 11
Browser: Edge
Version: Version 122.0.2365.59 (Official build) (64-bit)

Kindly check you site configurations

Hi @gurusabarish Thanks for the answer. Sorry not reply sooner.
What configurations do you mean?  This is the hugo file.
Thanks!

Ok I fixed it on my end. My issue was that I was being sent to the template page when pressed the search results.
My config was fine, the problem was that on the baseUrl parameter at the beginning of the hugo file, I hadn't applied my site URL. and still had the template as base Url


Standardize CI 20200203

cleanup .gitignore
implement Travis like in skel (separate phpstan job, --prefer-lowest in matrix)
apply php-cs-fixer changes for tests folder
move tests into tests/Cache and adjust composer autoload-dev section to fix phpstan:

composer phpstan
> @php vendor/bin/phpstan analyse lib tests
Note: Using configuration file /home/phil/git/sabre-io/cache/phpstan.neon.
 10/10 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ -------------------------------------------------------------------
  Line   tests/Cache/AbstractCacheTest.php
 ------ -------------------------------------------------------------------
         Class Sabre\Cache\AbstractCacheTest was not found while trying to
         analyse it - autoloading is probably not configured properly.
 ------ -------------------------------------------------------------------

 ------ -----------------------------------------------------------------------
  Line   tests/Cache/ApcuTest.php
 ------ -----------------------------------------------------------------------
         Class Sabre\Cache\ApcuTest was not found while trying to analyse it -
         autoloading is probably not configured properly.
 ------ -----------------------------------------------------------------------

 ------ -----------------------------------------------------------------------
  Line   tests/Cache/MemcachedTest.php
 ------ -----------------------------------------------------------------------
         Class Sabre\Cache\MemcachedTest was not found while trying to analyse
         it - autoloading is probably not configured properly.
 ------ -----------------------------------------------------------------------

 ------ -----------------------------------------------------------------
  Line   tests/Cache/MemoryCacheTest.php
 ------ -----------------------------------------------------------------
         Class Sabre\Cache\MemoryCacheTest was not found while trying to
         analyse it - autoloading is probably not configured properly.
 ------ -----------------------------------------------------------------

 [ERROR] Found 4 errors

Script @php vendor/bin/phpstan analyse lib tests handling the phpstan event returned with error code 1

(I made it the similar pattern to what happens in vobject )

We should have this included https://github.com/sabre-io/dav/pull/1237 (also in the skel repo)

skel repo has this stuff, except that it does not actually have any tests, so it does not have an example of autoload-dev things.

My comment was not about autoloading but about not building unnecessary branches


Godot don't compile on Linux

Can't compile 1bc91848e3cee91eccaf2150a74deaf1cd84be13 build on Linux
log:
kubeczek@kubeczek-K52JT:~/programs/godot/20151221/godot-master$ scons platform=x11 target=release_debug -j 4
scons: Reading SConscript files ...
Package libevdev was not found in the pkg-config search path.
Perhaps you should add the directory containing `libevdev.pc'
to the PKG_CONFIG_PATH environment variable
No package 'libevdev' found
evdev not found.. x11 disabled.
sh: 1: i586-mingw32msvc-gcc: not found
sh: 1: i686-w64-mingw32-gcc: not found
No valid target platform selected.
The following were detected:
    haiku
    server
    windows

Please run scons again with argument: platform=<string>
scons: done reading SConscript files.
scons: Building targets ...
scons: `.' is up to date.
scons: done building targets.

Have you installed the package libevdev / libevdev-dev / libevdev-devel (depending on your distro)?
This dependency has recently been introduced by af633c79 , see #3108
I think most distros allready ship with the runtime lib (the libevdev package), so most of the time you just need the headers. If this really is a problem, we could get rid of that library by using raw ioctls on the device files, it'll make the code look a bit messier but shouldn't be hard to do.
Another solution would be condititional compilation, so the gamepad code will only get enabled when libevdev has been found.

Hinsbart, no it's not a problem for me, didn't know this, thanks a lot for info! I assume there should be an update in godot docs page with Ubuntu compilation dependencies, but will open new issue for that. Once more time, thanks! :+1:

Hmm actually it still don't compile in the end  on Ubuntu. The output is different however:
kubeczek@kubeczek-K52JT:~/programs/godot/20151221/godot-master$ scons platform=x11 target=release_debug -j 4
scons: Reading SConscript files ...
sh: 1: i586-mingw32msvc-gcc: not found
sh: 1: i686-w64-mingw32-gcc: not found
PulseAudio development libraries not found, disabling driver
scons: done reading SConscript files.
scons: Building targets ...
g++ -o platform/x11/joystick_linux.x11.opt.tools.64.o -c -O2 -ffast-math -DDEBUG_ENABLED -DFREETYPE_ENABLED -DOC_X86_ASM -DOPENGL_ENABLED -DGLEW_ENABLED -DALSA_ENABLED -DX11_ENABLED -DUNIX_ENABLED -DGLES2_ENABLED -DGLES_OVER_GL -DMUSEPACK_ENABLED -DOPENSSL_ENABLED -DSQUISH_ENABLED -DVORBIS_ENABLED -DOPUS_ENABLED -DTHEORA_ENABLED -DTHEORALIB_ENABLED -DPNG_ENABLED -DDDS_ENABLED -DPVR_ENABLED -DJPG_ENABLED -DWEBP_ENABLED -DSPEEX_ENABLED -DTOOLS_ENABLED -DGDSCRIPT_ENABLED -DMINIZIP_ENABLED -DXML_ENABLED -DETC1_ENABLED -Icore -Icore/math -Itools -Idrivers -I. -Iplatform/x11 -I/usr/include/libevdev-1.0 -I/usr/include/freetype2 -Iplatform/x11/vorbis platform/x11/joystick_linux.cpp
platform/x11/joystick_linux.cpp:37:21: fatal error: libudev.h: No such file or directory
compilation terminated.
g++ -o modules/gridmap/register_types.x11.opt.tools.64.o -c -O2 -ffast-math -DDEBUG_ENABLED -DFREETYPE_ENABLED -DOC_X86_ASM -DOPENGL_ENABLED -DGLEW_ENABLED -DALSA_ENABLED -DX11_ENABLED -DUNIX_ENABLED -DGLES2_ENABLED -DGLES_OVER_GL -DMUSEPACK_ENABLED -DOPENSSL_ENABLED -DSQUISH_ENABLED -DVORBIS_ENABLED -DOPUS_ENABLED -DTHEORA_ENABLED -DTHEORALIB_ENABLED -DPNG_ENABLED -DDDS_ENABLED -DPVR_ENABLED -DJPG_ENABLED -DWEBP_ENABLED -DSPEEX_ENABLED -DTOOLS_ENABLED -DGDSCRIPT_ENABLED -DMINIZIP_ENABLED -DXML_ENABLED -DETC1_ENABLED -Icore -Icore/math -Itools -Idrivers -I. -Iplatform/x11 -I/usr/include/libevdev-1.0 -I/usr/include/freetype2 -Imodules/gridmap/vorbis modules/gridmap/register_types.cpp
g++ -o modules/gridmap/grid_map.x11.opt.tools.64.o -c -O2 -ffast-math -DDEBUG_ENABLED -DFREETYPE_ENABLED -DOC_X86_ASM -DOPENGL_ENABLED -DGLEW_ENABLED -DALSA_ENABLED -DX11_ENABLED -DUNIX_ENABLED -DGLES2_ENABLED -DGLES_OVER_GL -DMUSEPACK_ENABLED -DOPENSSL_ENABLED -DSQUISH_ENABLED -DVORBIS_ENABLED -DOPUS_ENABLED -DTHEORA_ENABLED -DTHEORALIB_ENABLED -DPNG_ENABLED -DDDS_ENABLED -DPVR_ENABLED -DJPG_ENABLED -DWEBP_ENABLED -DSPEEX_ENABLED -DTOOLS_ENABLED -DGDSCRIPT_ENABLED -DMINIZIP_ENABLED -DXML_ENABLED -DETC1_ENABLED -Icore -Icore/math -Itools -Idrivers -I. -Iplatform/x11 -I/usr/include/libevdev-1.0 -I/usr/include/freetype2 -Imodules/gridmap/vorbis modules/gridmap/grid_map.cpp
scons: *** [platform/x11/joystick_linux.x11.opt.tools.64.o] Error 1
g++ -o modules/gdscript/gd_editor.x11.opt.tools.64.o -c -O2 -ffast-math -DDEBUG_ENABLED -DFREETYPE_ENABLED -DOC_X86_ASM -DOPENGL_ENABLED -DGLEW_ENABLED -DALSA_ENABLED -DX11_ENABLED -DUNIX_ENABLED -DGLES2_ENABLED -DGLES_OVER_GL -DMUSEPACK_ENABLED -DOPENSSL_ENABLED -DSQUISH_ENABLED -DVORBIS_ENABLED -DOPUS_ENABLED -DTHEORA_ENABLED -DTHEORALIB_ENABLED -DPNG_ENABLED -DDDS_ENABLED -DPVR_ENABLED -DJPG_ENABLED -DWEBP_ENABLED -DSPEEX_ENABLED -DTOOLS_ENABLED -DGDSCRIPT_ENABLED -DMINIZIP_ENABLED -DXML_ENABLED -DETC1_ENABLED -Icore -Icore/math -Itools -Idrivers -I. -Iplatform/x11 -I/usr/include/libevdev-1.0 -I/usr/include/freetype2 -Imodules/gdscript/vorbis modules/gdscript/gd_editor.cpp
scons: building terminated because of errors.

libudev is needed too? :|
On Mon, Dec 21, 2015 at 3:00 PM, kubecz3k<EMAIL_ADDRESS>wrote:

Reopened #3111 https://github.com/godotengine/godot/issues/3111.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3111#event-497687420.

yeah, seems libudev-dev is also needed. Sorry, I'm using arch which combines runtime and devel files in the same package, so I didn't think this through properly..
Let me know if anything needs to change

I'm on ubuntu15.10 and I also had to install libudev-dev in order to compile.

So to summarize this, on Ubuntu you now need to install libevdev-dev as well as "libudev-dev"

Also could anyone from @godotengine/documentation update http://godotengine.org/projects/godot-engine/wiki/Compiling_for_Linux#Requirements page and add "libevdev-dev" and "libudev-dev" dependencies to  "For Ubuntu users" section?

With PR #3113 godot will build fine even if those packages are not present. Tho we still might want to add a note to the docs that you need these in order to have gamepad support.


HUE-7186 bugfix for variables cannot be commented in editor

When we comment a line with variable value, the variables still have this variable
SELECT * FROM table
-- WHERE col=${col}

@jdesjean, what do you think of this one?

I think the idea is good, but the implementation could be a bit more efficient. Let me take a look.


When using custom edit component muiTableBodyCellEditTextFieldProps not firing in table edit mode

material-react-table version
v1.13.0
react & react-dom versions
react: 18.2.12, react dom: 18.2.5
Describe the bug and the steps to reproduce it

Set editingMode to table
Use any custom edit component like DatePicker, Select
muiTableBodyCellEditTextFieldProps not firing with onBlur or onChange

Minimal, Reproducible Example - (Optional, but Recommended)
https://codesandbox.io/p/sandbox/dazzling-shadow-cvvzmf?file=/package.json:25,27-25,33
Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
Yes, I exactly don't know how to fix it, but I have some idea and open to discuss it in the comments of this issue
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Hi @KevinVandy, if we already have any solutions please let me know, otherwise let me know your thoughts on this.
In the MRT_EditCellTextField component on line 90, it's not exposing any onChange/onBlur/onSelect handler for custom edit component.
And the current onBlur or onChange is tightly coupled with the TextField component, so we need to pass the respected event, So instead I think we can expose some custom generic handler that will accept a key value, and internally update the value and also the cache value.  In this way when we are using the row edit mode. We don't need to update it like this
row._valuesCache.key = value and also it will reflect the updated value on the component, this will solve this issue 599 also.
Also for table edit mode, we can introduce another generic prop like muiTableBodyCellEditProps that will fire for every edit component regardless of custom or internal.


Add anax

I am adding this to support https://github.com/miguelmartin75/anax as a port. At the moment there is a problem with building the dynamic library (missing .lib file) as per https://github.com/miguelmartin75/anax/issues/78. When try to build a dynamic lib at the moment the static lib is built in place of the dynamic one.

Hi @mattiascibien, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@alexkaratarakis can you please also take a look at #362? it seems that the bot does not detect that I have signed the CLA.

Thanks for the solid PR!
vcpkg automatically passes the BUILD_SHARED_LIBS variable. Therefore, libraries that use this CMake-build-in variable are easily portable to vcpkg. I have taken the liberty to remove passing this variable in anax's portfile.

@alexkaratarakis ok... thank you.


node: Make nvm lookup mechanism more efficient in homebrewed environment

Proposed Changes
In homebrewed environment, avoid using brew --prefix nvm which is ruby based and is super slow. Instead, rely on homebrew standard behavior wherein all installed packages are available in canonical path $(brew --prefix)/opt/<package> (for nvm it would obviously be $(brew --prefix)/opt/nvm).
NB: $(brew --prefix) (without additional argument) is a simple shell shortcut and doesn't have the same performance impact.

Also see #1611 for related discussion.


[Tracing Framework]Strict check Span Processor implementation

Is your feature request related to a problem? Please describe.
Strict check Span Processor implementation to catch the instances where span are not closed properly.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@Gaganjuneja would you mind please editing the issue and replacing "A clear and concise desc ..." with the actual description :)


feat: add basic regression test

Description

Remove dummy contents in lib file
Move basic regression test out from the main test-runner file to its own file, which introduces a path for how future tests should be structured
Use log format from config instead of setting it again in test-runner.rs
Reduce logs from SDK and Radio

Issue link (if applicable)
Helps with https://github.com/graphops/poi-radio/issues/175

will put up a new one with improvements over this work on Monday


TimeSeriesScalerMeanVariance got warning of dividing zero when the time series is 0

Hi,
Thanks for your brilliant works of building this library!
I am trying to use tslearn.preprocessing.TimeSeriesScalerMeanVariance for standardization, but it failed when one of my data samples is pure 0. Here is the example:
t = np.array([[0,0,0],[1,2,3]])
TimeSeriesScalerMeanVariance().fit_transform(t)

the output for the above code is:
/lib/python3.6/site-packages/tslearn/preprocessing.py:153: RuntimeWarning: invalid value encountered in true_divide
  X_[i, :, d] = (X_[i, :, d] - cur_mean) * self.std_ / cur_std + self.mu_
array([[[        nan],
        [        nan],
        [        nan]],

       [[-1.22474487],
        [ 0.        ],
        [ 1.22474487]]])

The issue is the standard deviation of a [0,0,0] is 0, it there any good methods for working around?

Hi and thanks for this bug report.
I guess for this, the best idea would be to take inspiration from sklearn (cf here)
This means setting variance to 1. (arbitrarily) as soon as it is zero.
I'll try to do it asap. Do not hesitate to propose a PR on this if you have time before I do :)

OK, It was pretty straight-forward, so I fixed it directly.
The bug should not exist anymore starting from version tslearn-<IP_ADDRESS>

Just when I was about to contribute a bit!! You're too fast @rtavenar ..
I ported the code to numpy though, I expect it to be faster.
X_ = to_time_series_dataset(X)
means, stds = np.mean(X_, axis=1), np.std(X_, axis=1)
stds[stds == 0] = 1
return (X_ - means) * self.std_ / stds + self.mu_

Here's a few stats:
TimeSeriesScalerMeanVariance(mu=0., std=1.).fit_transform([np.random.rand(10000000)]) takes 0.6294596195220947 seconds for the numpy implementation and  0.646794319152832 with the original implementation.
For vectors of length 100000000 the numpy implementation is even slower... Strange stuff...

tks for such a fast fixing


Reusing same image twice in a view controller with different content modes

I'm using hnk_setImageFromURL to set an image property of a UIImageView with content mode "Fill"
After the user clicks a button, I want to add the same image again to a different UIImageView with content mode "Aspect Fill".
If I use the first ImageView's image property the image is not scaled well. The only workaround I've been able to use is calling hnk_setImageFromURL on the second UIImageView again but since this may be a heavy gif file, maybe the cache isn't written yet by the time the user clicks the button and I see two network requests for the image.
What is the best method of achieving this?
Thanks

By default Haneke applies a format to downloaded images which crops them accordingly to the content mode of the view. That's why when you try to reuse it it may not scale as you would expect. If you want a different behaviour you can provide your own format (or nil perhaps).
You can also use cache directly: Fetch the image and set it in the two views.

Although I think it would be nice if two same network fetches could be automatically merged into one.


Weight Normalization

Usually during training add a normalization to the weights used in CNN is a good way to improve the generalization performance.
If I did not misread the code, at the moment it seems that it is not trivial to get the weights during learning.
It seems to me that it might be a good idea to store also the weights in the Network class for easy retrieval during learning.

The weights and biases are stored in regular tensorflow variables (see the load function for instance). You can access these just like any other tensorflow variables (which you can also use for adding regularization). For more details, see here.


Please invite me to the GitHub Community Organization

Name
Swastika
Discord Username (if applicable)
Swasty#1957
Additional Context
No response

Welcome to this community @Swastyy 🎉🎉🎊

Welcome to this community @Swastyy 🎉🎉🎊

Thanks! 🎉

Oh yay! Welcome to the great EddieHub community, @Swastyy, where we believe Collaboration 1st, Code 2nd! 😄👍🎉

Thank you so much @Panquesito7 🙂


Add Gradle Wrapper to exclude from extension build

Gradle Wrapper:
https://docs.gradle.org/current/userguide/gradle_wrapper.html
Is "recommended way to execute any Gradle build" so more and more
extensions have the wrapper files in the repository.  They serve no
purpose once built, so ignore it from the zip.

Just out of curiosity, how did they get there?  Did you put them there?

Yes, I ran gradle init

Closes #1763


Errors when building SimpleITK on aarch64

Hi,
I've been trying to build SimpleITK on aarch64 following this: https://simpleitk.readthedocs.io/en/master/building.html
But I have these errors when I build with Superbuild. How can I solve this issue ?
It also happens when I run: pip install git+https://github.com/SimpleITK/SimpleITKPythonPackage.git -v --no-use-pep517
from https://github.com/SimpleITK/SimpleITKPythonPackage
../../../../lib/libitkpng-5.2.a(pngrtran.c.o):pngrtran.c:function itk_png_do_read_transformations: error: undefined reference to 'itk_png_do_expand_palette_rgba8_neon'
../../../../lib/libitkpng-5.2.a(pngrtran.c.o):pngrtran.c:function itk_png_do_read_transformations: error: undefined reference to 'itk_png_riffle_palette_neon'
../../../../lib/libitkpng-5.2.a(pngrtran.c.o):pngrtran.c:function itk_png_do_read_transformations: error: undefined reference to 'itk_png_do_expand_palette_rgb8_neon'
../../../../lib/libitkpng-5.2.a(pngrutil.c.o):pngrutil.c:function itk_png_read_filter_row: error: undefined reference to 'itk_png_init_filter_functions_neon'
collect2: error: ld returned 1 exit status
Modules/Core/TestKernel/src/CMakeFiles/itkTestDriver.dir/build.make:163: recipe for target 'bin/itkTestDriver' failed
make[5]: *** [bin/itkTestDriver] Error 1
CMakeFiles/Makefile2:16052: recipe for target 'Modules/Core/TestKernel/src/CMakeFiles/itkTestDriver.dir/all' failed
make[4]: *** [Modules/Core/TestKernel/src/CMakeFiles/itkTestDriver.dir/all] Error 2
Makefile:151: recipe for target 'all' failed
make[3]: *** [all] Error 2
CMakeFiles/ITK.dir/build.make:111: recipe for target 'ITK-prefix/src/ITK-stamp/ITK-build' failed
make[2]: *** [ITK-prefix/src/ITK-stamp/ITK-build] Error 2
CMakeFiles/Makefile2:360: recipe for target 'CMakeFiles/ITK.dir/all' failed
make[1]: *** [CMakeFiles/ITK.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2
Thanks

Hi @aletru01 I was facing a similar issue on ppc64le. running cmake with
cmake -DITK_USE_SYSTEM_PNG:BOOL=ON -DBUILD_TESTING:BOOL=ON ../SimpleITK/SuperBuild/
helped but I am seeing two test failures which I just created an issue for, those seem unrelated to the use of SYSTEM_PNG though

Hi @seth-priya, the build is successful on aarch64 with your command.
Thanks


Negative Int! arguments doesn't work

Hi, thanks for your great library. I found a bug during my exploration.
Async-graphql version: 1.11.3
Schema:
#[Object]
impl QueryRoot {
    #[field(desc = "Returns the sum of a and b")]
    async fn add(&self, a: i32, b: i32) -> i32 {
        a + b
    }
}

Execute:
#[actix_rt::main]
async fn main() -> std::io::Result<()> {
    let schema = Schema::new(QueryRoot, EmptyMutation, EmptySubscription);
    let res = schema.execute("{ add(a: 10, b: -20) }").await;
    println!("{:?}", res);
...
}

Result:
Err(Parse(Error { pos: Pos(1:17), message: " --> 1:17\n  |\n1 | { add(a: 10, b: -20) }\n  |                 ^---\n  |\n  = expected value" }))

That's an invalid query, mutations have to select fields to return.

I think it mostly missing the query keyword?

No, this problem caused by the new parser.

@hapcode  Upgrade async-graphql to 1.12.1. 😁

Same thing after rebuild. Also I see async-graphql-parser-0.3.0 in target/debug/deps.
   Compiling async-graphql v1.12.2
   Compiling actix-connect v1.0.2
   Compiling actix-http v1.0.1
   Compiling awc v1.0.1
   Compiling actix v0.9.0
   Compiling actix-web v2.0.0
   Compiling actix-web-actors v2.0.0
   Compiling async-graphql-actix-web v1.4.2

I'm really sorry, I forgot to update the parser version. 😭

Please upgrade async-graphql to 1.12.3.

It works. Thank you. Closed.


Tracing error when using cumsum inside xla.lower_fun

Repro:
from jax import api
from jax import core
from jax import abstract_arrays
import jax.ops as jops
from jax import xla

v = jnp.reshape(jnp.array([1, 0, -1], dtype=jnp.float32), [3, 1])

a = jnp.array([0, 2, 2, 0, 1, 2], dtype=jnp.int32)
b = jnp.array([0, 2, 3, 6], dtype=jnp.int32)

repro_p = core.Primitive("repro")

def repro_prim(v, a, b):
  return repro_p.bind(v, a=a, b=b)

def repro_abstract_eval(v, a, b):
  out_m = a.shape[0]
  return abstract_arrays.ShapedArray( (out_m, ), dtype=v.dtype)

repro_p.def_abstract_eval(repro_abstract_eval)

def _get_foo(a, b):
  foo = jops.index_update(jnp.zeros_like(a), b, jnp.ones(b.shape[0]))
  return jnp.cumsum(foo)

def repro_translation(c, v, a, b):
  foo = xla.lower_fun(_get_foo)(c, a=a, b=b)
  return foo

xla.translations[repro_p] = repro_translation

api.jit(repro_prim)(v, a, b)

Error is:
The functions being transformed should not save traced values to global state.
Details: Tracer from a higher level: Traced<AbstractUnit():JaxprTrace(level=-1/2)> in trace JaxprTrace(level=-1/2).

The issue is setting up a and b as primitive parameters here (by passing them as kwargs, which is our convention for separating value-level inputs from metadata-like parameters):
  return repro_p.bind(v, a=a, b=b)

That's inconsistent with treating a and b as values, e.g. when you write api.jit(repro_prim)(v, a, b).
This runs:
import jax.numpy as jnp
from jax import api
from jax import core
from jax import abstract_arrays
import jax.ops as jops
from jax import xla

v = jnp.reshape(jnp.array([1, 0, -1], dtype=jnp.float32), [3, 1])

a = jnp.array([0, 2, 2, 0, 1, 2], dtype=jnp.int32)
b = jnp.array([0, 2, 3, 6], dtype=jnp.int32)

repro_p = core.Primitive("repro")

def repro_prim(v, a, b):
  return repro_p.bind(v, a, b)  # changed!

def repro_abstract_eval(v, a, b):
  out_m = a.shape[0]
  return abstract_arrays.ShapedArray( (out_m, ), dtype=v.dtype)

repro_p.def_abstract_eval(repro_abstract_eval)

def _get_foo(a, b):
  foo = jops.index_update(jnp.zeros_like(a), b, jnp.ones(b.shape[0]))
  return jnp.cumsum(foo)

def repro_translation(c, v, a, b):
  foo = xla.lower_fun(_get_foo)(c, a, b)  # changed!
  return foo

xla.translations[repro_p] = repro_translation

api.jit(repro_prim)(v, a, b)

I think we've got this sorted, so in the spirit of minimizing our active issues, I'm going to close this.

You should think of primitive params, which are identified by being passed via keyword arguments to Primitive.bind, as things like strings (e.g. "VALID" or "SAME") or shapes (e.g. (2, 3)) which can be construed of as part of the primitive's name or in some cases type-level arguments. Things that are array-like tend to be value-level arguments and must be passed to Primitive.bind as positional arguments (by the same convention).


Allow expansion of disks on a templated VM

Assuming a template VM with two disks, allow us to increase the HD size.
skytap.hd1 = 20GB
skytap.hd2 = 100GB

Fail with an error if the value is smaller than the value in the template.
We could also specify based on dev identifier:
skytap.sda = 20GB
skytap.sdb = 100GB

Added to Skytap's internal Jira as ENG-22737

Any update on this?


Initializing Log4j1 bridge with properties object blocks Log4j2 logging since 2.17.2

Description
We have a legacy class that loads Log4J using PropertyConfigurator.configure(Properties) via the bridge libraries.  Since 2.17.2, once this line is hit our Log4j2 statements will stop logging.  I have been able to replicate through 2.20.  Changing between 2.17.1 and 2.17.2 in the attached Maven project will replicate the before and after scenarios.
Configuration
Version: 2.17.2
Operating system: Windows Server 2012
JDK: 1.8 - 11
Reproduction
Project attached
Log4J2Issue.zip

PropertyConfigurator#configure() was a no-op before version 2.17.2 (cf. source code).
The call must provide a broken configuration (you can debug it by running -Dlog4j2.debug=true), hence the logs vanish.

@ppkarwasz
Thank you for the reply, it filled in the piece I was missing.  It wasn't a broken configuration, rather an incorrect mindset on my part.  Previously this application was using Log4j2 and Log4j1 for independent purposes.  In 2.17.1 we were able to get the proper result based on the configure() option not doing anything and having a logger that matched in our log4j2.xml.  As soon as it was actually performing an initialization it is now overwriting our log4j2 configuration with our legacy Log4j1 configuration, and our Log4j2 usage in essence no longer has loggers to work with.  We will have to look into potential solutions either through context or creating a custom configuration factory, etc.

There is an open JIRA issue (LOG4J2-3467) to provide users a way to disable the improvement in log4j-1.2-api introduced in version 2.17.2. We should probably consider it solving it.


[tensorflow] Remove String tensor limitation for model output

Fixes #2053
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Base: 72.08% // Head: 69.70% // Decreases project coverage by -2.38% :warning:

Coverage data is based on head (41a8811) compared to base (bb5073f).
Patch coverage: 68.95% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2056      +/-   ##
============================================
- Coverage     72.08%   69.70%   -2.39%
- Complexity     5126     5988     +862
============================================
  Files           473      600     +127
  Lines         21970    26570    +4600
  Branches       2351     2888     +537
============================================
+ Hits          15838    18520    +2682
- Misses         4925     6665    +1740
- Partials       1207     1385     +178

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

.../cv/translator/InstanceSegmentationTranslator.java
0.00% <0.00%> (-86.59%)
:arrow_down:

...nslator/InstanceSegmentationTranslatorFactory.java
7.14% <0.00%> (-11.04%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

... and 515 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Added      "currency_decimals":"x", to all countries

Some countries have zero decimal places for their currencies while others may have 3 or 2
#40
Source: google and  http://www.currency-iso.org/dam/downloads/lists/list_one.xml

Thanks for the PR.


Action page response table

This was overlooked in our design and although we made an updated schema, we were not able to change the schema and implement the changes in time for the demo.


Catalan flag does not follow ISO-639 language code

The Catalan flag has been added in https://github.com/lipis/flag-icon-css/pull/374 with the extension of 'ct'. The ISO-639-1 Language code for Catalan is 'ca'.
The class name should be changed 'flag-icon-es-ct' to 'flag-icon-es-ca'. We could also keep it backward compatible by only adding the 'flag-icon-es-ca'.

source?

https://www.loc.gov/standards/iso639-2/php/code_list.php

these are the language codes.. we store country codes

and ca is already taken by Canada

Catalan is not a country. It's a specific region inside of Spain. If you only want to store country codes it makes no sense to add Catalan to this list anyway.
The current usage is with es-ct. If we change it to es-ca it will not replace Canada which is ca.
Since this package is also following ISO 3166-1-alpha-2 code of a country with some region support, it only makes sense that it also uses the ISO 639-1 to give a proper key to those region flags and not some randomly thought up one.

you might have some points here..

Do you mind opening a PR? no need for backwards compatibility..

Sure I'll write one right now

I consider this not being an appropriate change. As pointed previously ISO 639-1 codes are used for languages (and using flags to identify a language is always a bad idea) while the ISO 3166-2 are the codes used to identify country subdivisions. In the case of Catalunya the ISO 3166-2:ES code happens to be es-ct.


Add more width to protocols dropdown

This adds a little more spacing to the protocol column by increasing the width to 90px. This also adds the col class to the labeled selects in order to satisfy the requirements for the style that defines minimum height.

Before

After

Adding the col class to an element that's not an immediate child of row feels a bit like a bad practice to me... but I chose this approach because it was the simplest way to apply the following style for consistency
.col {
  > .labeled-select:not(.taggable),
  > .unlabeled-select:not(.taggable) {
    min-height: $input-height;
    padding-bottom: calc(#{$input-padding-sm}/2);
  }
}

Adding the col class to an element that's not an immediate child of row feels a bit like a bad practice to me... but I chose this approach because it was the simplest way to apply the following style for consistency
.col {
  > .labeled-select:not(.taggable),
  > .unlabeled-select:not(.taggable) {
    min-height: $input-height;
    padding-bottom: calc(#{$input-padding-sm}/2);
  }
}

It's a little out of the norm, but I think your justification is solid -- we do want these inputs to match the selects that are in .rows and .cols on the same page -- and visually it's good, so I think it's fine.


Add acceptance criteria to PR template

Cool! 👏
Will we roll this out to other projects as well? Zeebe Modeler, bpmn-js, ...

I think so, but first I want to have a template to copy from.

Looks good, however "Passes CI checks" step is sometimes blocked by codecov checks. This happens pretty often actually. What'd be our approach there?

I think it is similar to

a single commit closes the issue via Closes #issuenr

In most of the cases it works like this, but sometimes we intentionally deviate from the standard, if it's reasonable.

Have we shipped the merge-me feature yet? Maybe @andreasgeier needs to approve this as well 😛

it works!!


[GenericPoseGraph] Add IMU Calibration and Pose3D node types.

@wally-the-cartographer merge

@wally-the-cartographer merge

Merge requested by authorized user pifon2a. Merge queue now has a length of 1.


Add example using EXPECT from a function called from a CASE

Add an example showing how to define a function or lambda that uses a lest assertion and that can be called from a CASE().
Add reference to example in Readme.
See also issue #23.

Done


M3-1742 Refresh volumes list on volume_clone event.

Purpose
It was found that the volumes landing was not being updated after a volume clone requests completed.

Coverage remained the same at 32.114% when pulling 09108fdbefea4bb2e40445cbe8e85381bbdef6f9 on rjstires:M3-1742 into 76aae46aa959b44fb73fdc68dc94ff6ce7bee6f4 on linode:develop.


runtime: failed to create new OS thread (have 15 already; errno=11)

Describe the bug
We are using the official gitleaks image for version 7.5.0. We are scanning thousands of repositories and sometimes we get an error like this
runtime: failed to create new OS thread (have 15 already; errno=11)
To Reproduce
Not sure how to reproduce it. Maybe it happens when a repository with a lot of commits is scanned. We are running this command
gitleaks --access-token=$token --config-path=custom.toml --leaks-exit-code=2 --format=json --report=${repo}.json -r https://mygitrepo/${repo} --threads=2
Expected behavior
No error is thrown
Screenshots
If applicable, add screenshots to help explain your problem.
Basic Info (please complete the following information):

OS: Gitleaks docker image
Gitleaks Version: 7.5.0

Additional context
Add any other context about the problem here.
cc @zricethezav

Hello @GolanTrevize10 !
It seems you are using an old version of gitleaks.
Currently gitleaks is on version v7.6.1:
https://github.com/zricethezav/gitleaks/releases/tag/v7.6.1
And version v7.6.0 seems to fix some issues with the --threads option (which you are using).
Can you try using that newer version to see if the error still pops up?
Another thing that can be tested is to either modify the value passed to the --threads option or not use it at all?
Cheers!

Hi, we solved already the issue. It has nothing to do with gitleaks but with some reused environment variables running several gitleaks scans in parallel


update AutoFitSurfaceView to avoid stretched previews

probably investigate more on this and create a new PR for that?


doc: Update container package overview

Description:

Updates incorrect overview for container package
issue

Checklist:

[ ] I have formatted my code using  goimport  and  golangci-lint.
[ ] All new code is covered by unit tests.
[ ] This PR does not decrease the overall code coverage.
[ ] I have reviewed the code comments and documentation for clarity.

I dropped the link idea because it seemed too complicated. For example:

If I want to use the doc link I face this issue:

And with Links, it's hard to maintain them. What if cassandra package location change

// [Cassandra]: https://pkg.go.dev/gofr.dev/pkg/gofr/datasource/cassandra

Either way, please correct me if I'm wrong. Go is new to me and don't know how idiomatic documentation should look like in go 😄

@JonasBakys0 the tests and the pipeline is failing due to these errors:

Please fix them.


Broken link causes a leak

Hi,
It seems that if the url is broken the connection leaks.
When that happens I see a message like this on the console:

W/OkHttp: A connection to http://feeds-tmp.soundcloud.com/ was leaked. Did you forget to close a response body?

Which version did you test on? I have tested this url, but I didn't meet this error.

Thanks for answering @Coolerfall.
I'm using the last master commit.
You can test on any invalid/broken url. The url showed above is not complete. Please try this one:
http://podcast.canaltech.com.br/1735x.mp3
You'll see something like this on the log:

W/OkHttp  (25435): A connection to http://podcast.canaltech.com.br/ was leaked. Did you forget to close a response body?

I still didn't get this message, just get 404 error however.

Yeah. That's right. This happens to me too.
But my app tries to download the same file several times, after a short delay. Can you please test that situation?

1.Try to download serveral times, with different DownloadRequest or same DownloadRequest?
2.After a short delay, what does this mean?

1 - With different instances of "DownloadRequest".
2 - 15 seconds

Ok, I got this message. I'v changed some code, and I didn't get this message anymore. U can checkout on develop branch.

Thanks @Coolerfall.
I'll test it in the next days. ;)


FormInnerComponent unify Bootstrap and Material UI API

I've noticed differences between FormComponentInner in Bootstrap and Material UI. This lead to issues in Storybook stories and code redundancy.
I created a vulcan:ui-react package to share reusable UI code, I am not sure if it's the best pattern.
I try to progressively reduce the codebase size so UI becomes more manageable.
This PR needs a bit of testing, mostly to check that it does not break apps using Bootstrap UI.
Isn't the whitelisting of inputProperties a bit weird though? What if I want to add specific accessibility parameters (I am not knowledgeable on this so I don't have much examples) ?

I don't know if we need one more package? Couldn't we add those helpers to vulcan:lib or vulcan:core instead?
Also the reason for the whitelisting is that if you allow any random foo property, that can sometimes be applied directly as attribute on an <input> element and React complains. You make a good point though, I'm not sure what the workaround is…

Ok I'll move this back to vulcan:lib then, I think it's the most appropriate place indeed. I'll try to find a list of all valid props, I think they should be listed somewhere on official docs

I've merged, it works as expected with Bootstrap in the Starter, I've put the ui utils in vulcan:lib


useUser not returning created_at property, although type says it does

Bug report

[ V] I confirm this is a bug with Supabase, not with my own application.
[ V] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
The context useUser from @supabase/auth-helpers-react returns an object that is not aligned with the documented type (and also different from what supebase js client actually returns)

but if I use supabase.auth.getUser(), I do get the right type
To Reproduce
Import and use const user = useUser(); from "@supabase/auth-helpers-react"; and print the attributes of the not-null user object
Expected behavior
Expected behavior is to have the same attributes as the documented type
Screenshots
See above
System information

OS: macOS

Browser : chrome

Version of supabase-js:
"@supabase/auth-helpers-nextjs": "^0.7.1",
"@supabase/auth-helpers-react": "^0.4.0",
"@supabase/supabase-js": "^2.24.0",

Version of Node.js: node 18

I can work on this one @adamcohenhillel  please assign it to me.

hey @IsomerX - I can't seem to be able to assign the issue..

Hey, going to close this issue as we've moved from Auth-helpers to SSR. If needed, please bring discussion to #744
Let me know if there are any concerns though


Is it possible to slide main viewcontroller along with side menu to left side?

do you have two menus? how do you do it?

yes i have it


Automated cherry pick of #5579

Cherry pick of #5579 on release-1.46.
/cc  @amyblais

/release-note-none


NameError: during preprocessing

when i want to give below command as per your manual at here..
Preprocess the training and dev sets in the distractor setting:
python main.py --mode prepro --data_file hotpot_train_v1.1.json --para_limit 2250 --data_split train
python main.py --mode prepro --data_file hotpot_dev_distractor_v1.json --para_limit 2250 --data_split dev
Preprocess the dev set in the full wiki setting:
python main.py --mode prepro --data_file hotpot_dev_fullwiki_v1.json --data_split dev --fullwiki --para_limit
.......
i got error in prepro.py at
96
97     def _process(sent, is_sup_fact, is_title=False):
---> 98         nonlocal [text_context, context_tokens, context_chars, offsets, start_end_facts, flat_offsets]
text_context = undefined
99         N_chars = len(text_context)
NameError: global name 'nonlocal' is not defined
how may i solve it?

nonlocal is part of Python 3. Are you using Python 3 to run our code (see also the README)?


Plotting fixes and improvements

This PR includes a few fixes and improvements to the plotting code including:

Disabling legends on PathPlot by default (Issue #460)
PointPlot and Scatter3DPlot now set facecolors rather than 'c' allowing control over edgecolor (Issue https://github.com/ioam/holoviews/issues/459) (Postponed to 1.5)
Plots now support supplying color_index and size_index as dimension name (Issue #391).

This was a bit of a pain because of various transients due to edgecolors, should be fixed now though. We should update to matplotlib 1.5 to avoid these problems as soon as we can figure out how to get it working on Travis. Let's merge this now and release!

Looks good and the tests are passing. Merging.


Unable to build rstudio connect docker image

Hi,
I am using docker destop to download rstudio connect and I can see below path successfully.
root@565ad3ca8b23:/opt/rstudio-connect/bin# ls
configdigest  connect  feature-usage  license-manager  migrate  rscadmin  usermanager
But when I am going to create my own dockerfile with your provided dockerfile. Where I am unable to see the path like above one. I can see only /opt/rstudio-connect-runtime only

It appears as if you are using the "connect-content-init" directory, which defines an init container and not a general-purpose RStudio Connect image.
Please use the "connect" directory as an example when building your own Dockerfile and Docker image.
https://github.com/rstudio/rstudio-docker-products/tree/main/connect
The "connect" directory is used to build the rstudio/rstudio-connect images, hosted on Docker Hub here: https://hub.docker.com/repository/docker/rstudio/rstudio-connect

Hi Aronatkins,
Could you please help me to resolve the below error.
=> => exporting layers                                                                                15.1s
=> => writing image sha256:e4e6394a3eef053fa00f5eedc88192a822b05d90c17566b01f68084ecf4fefa1                                 0.0s
=> => naming to docker.io/rstudio/rstudio-server-pro:1.4.1717-3                                                             0.0s
Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them
WARNING: Image for service rstudio-server-pro was built because it did not already exist. To rebuild this image you must use docker-compose build or docker-compose up --build.
Building rstudio-connect
[+] Building 6.5s (3/3) FINISHED
=> [internal] load build definition from Dockerfile                                                                         0.1s
=> => transferring dockerfile: 2.58kB                                                                                0.0s
=> [internal] load .dockerignore                                                                                0.2s
=> => transferring context: 2B                                                                                0.0s
=> ERROR [internal] load metadata for docker.io/rstudio/r-base:3.6.2-bionic                                                 6.0s

[internal] load metadata for docker.io/rstudio/r-base:3.6.2-bionic:

failed to solve with frontend dockerfile.v0: failed to create LLB definition: failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/rstudio/r-base/blobs/sha256:c437c12769cd0f9d2b80a030b2184fcb11eb46d4969fc4f1441d505f30160514: 502 Bad Gateway
ERROR: Service 'rstudio-connect' failed to build : Build failed
PS C:\Users\Devivaraprasad_Varap\Documents\argomlops_iris\rstudio-docker-products-main>

Is that error repeatable?
The 502 Bad Gateway HTTP error usually indicates that some gateway or proxy server is having trouble talking with some upstream server. In this case, it is probably either a service disruption with the Docker registry or a corporate proxy.
As of right now, the Docker status page is not showing indication of service problems: https://status.docker.com
Explanation of the HTTP 502 status code: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/502

@devivaraprasad901 - please reach out to<EMAIL_ADDRESS>if you need additional assistance getting started with custom Docker images.


#741: hoist GAE/GCD detection into gcloud._helpers

Enables future reuse (e.g., for storage / pubsub project ID).

Coverage remained the same at 100.0% when pulling 14a7415e646b7a180c116488e8a0528cc0a95755 on tseaver:741-hoist_gae_gcd_detection into 8948a491871ca91f8860290816c963f1b3998514 on GoogleCloudPlatform:master.

LGTM


Does not work

UseWebSockets does not work with nuget package 2.1.1, can't find the declration, even when including the namespaces at top of file...?

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

ID: a3b0cac1-adc7-434c-d867-6e28e39bdae7
Version Independent ID: 762efeed-010e-f422-03d0-22d6681b2851
Content: WebSockets support in ASP.NET Core
Content Source: aspnetcore/fundamentals/websockets.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: tdykstra

Yikes, re-install VS and now it works. Can close this I guess...


include dict.cc entries with subject labels

I used this script to turn a DE-ES dict.cc file into a quickdic compatible with my Tolino. From the original 45k entries more than 20k were dropped because they had a subject label:

WARNING: Malformed line: Atomphysik {f} física {f} atómica noun [phys.]

This change allows lines to have 4 fields/columns: language1, language2, word class,  subject labels.

see also https://github.com/natowi/quickdic-dictionary.dictionarypc/issues/1

I guess you see no need to try and include these extra fields somehow?

Btw since I finally added back support for generating Tolino-compatible files (my original hope was that they'd add support for the newer, much more efficient format), I'd be happy to accept patches to integrate the work from Toligen.
It'll need some updates/polishing though, and using the native binaries a tutorial might actually be more useful than scripts. Unless it's nicer scripts that e.g. just let you select the languages you want and then automates everything...

I guess you see no need to try and include these extra fields somehow?

I did not feel confident enough to make bigger changes and focused first on fixing this major bug.

I'd be happy to accept patches to integrate the work from Toligen.

I could have a look on it, could you guide me a bit? For me it seems Toligen just compiled a jar and created two sample bash scripts. Is there more behind? @rdoeffinger

I did not feel confident enough to make bigger changes and focused first on fixing this major bug.

No worries, for me it's more the problem that I am not sure how to add it in a way that doesn't make it look bad than the actual code changes.
As for the Toligen part, things that would need to be looked at to make it work nicely with this code is:

That code was using an old version of the program that created old v6 format dictionaries. I don't have the code to directly create them any more, but instead wrote a conversion tool. So the script would need to add a step to convert to old format, since the command that is there would now create v7 dictionaries.
Running a .jar file is a bit of a pain, so it would be good to support the binaries I provide in the releases section, they just run natively without any need for a Java installation. The other scripts in this repo show how supporting both is possible. For Windows, a batch file might also be interesting...


[feat] Old version of Cargo fails silently

Describe the problem
I'm new to Tauri and was attempting to run the template (yarn + react).
Because I was running an older version of Cargo, the build failed, but because the rust build output gets replaced by the yarn build, the issue wasn't easy to work out.
Describe the solution you'd like
Ideally, if the 'edition' in Cargo.toml is too new, the yarn build wouldn't get run, so that you could see the error in the output.
Alternatives considered
No response
Additional context
No response

What do you mean with gets replaced by the yarn build? Can you give us an example?

I'm on Windows, in vscode (so a Powershell console), and immediately after this issue happens, the yarn/react build starts, which clears the cargo build output, including the error message. If I kill the build at just the right time I can read the message, but it's like a 0.3s window or something.
So this is the important output that gets cleared:
error: failed to parse manifest at `C:\_projects\Lab\storage-view\src-tauri\Cargo.toml`

Caused by:
  failed to parse the `edition` key

Caused by:
  this version of Cargo is older than the `2021` edition, and only supports `2015` and `2018` editions.

And this is what replaces it:
Compiled successfully!

You can now view storage-view in the browser.

  http://localhost:3000

Note that the development build is not optimized.
To create a production build, use yarn build.

webpack compiled successfully
Files successfully emitted, waiting for typecheck results...
Issues checking in progress...
No issues found.

Thanks for the report. Also you should update your Rust :)


Logging & Config-Seed Docker

Newbie to edgex. Trying to get it running following using docker
https://docs.edgexfoundry.org/Ch-WalkthroughRunning.html
but constantly errors out. Any help appreciated
edgex-support-logging
level=ERROR ts=2019-10-13T04:50:08.8215509Z app=edgex-support-logging source=logger.go:105 msg="logTarget cannot be blank, using stdout only"
level=ERROR ts=2019-10-13T04:50:08.8216213Z app=edgex-support-logging source=main.go:72 msg="registry is not available"
edgex-config-seed
level=ERROR ts=2019-10-13T04:49:54.4588928Z app=edgex-config-seed source=main.go:79 msg="registry is not available"
level=ERROR ts=2019-10-13T04:49:55.469439Z app=edgex-config-seed source=logger.go:105 msg="logTarget cannot be blank, using stdout only"
level=ERROR ts=2019-10-13T04:49:55.4695122Z app=edgex-config-seed source=main.go:92 msg="edgex-config-seed: Service bootstrap failed!"

Please take questions like this to the #help channel on edgexfoundry.slack.com. And be sure to specify which release you're using.