Awesome Book JavaScript classes

In this branch, my partner and I :

Created class methods to add and remove books.
Added CSS styles to the application

Hi LionRouge1
I dont have a partner again he just left the program and have not been issued a new partner

Hi,
I don't have a coding partner at the moment he left this morning and have
not been given a new partner.
On Thu, 8 Jun 2023 at 12:07, AVLESSI Matchoudi @.***>
wrote:

@.**** commented on this pull request.
Hi @Shabiblac https://github.com/Shabiblac,
[image: done]
https://camo.githubusercontent.com/61d20e43d55f9593796d98c4d8aaa8127e0d94f1ca919dc5a3f11a7ad3ce492e/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f676e453446466874466f4c4b4d2f67697068792e676966
While you made a great effort in this project, unfortunately, I cannot
proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is supposed to be a pair-programming
activity but you are the only one who contributed to the project. Kindly
work with your partner to submit this project.
Your Code Review Request will be marked as invalid in your Dashboard, so
please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
—
Reply to this email directly, view it on GitHub
https://github.com/ntimanu/awesome-books/pull/2#pullrequestreview-1469667673,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYHCVOUNKLSUV4SKT76SXGLXKGW7FANCNFSM6AAAAAAY5VP4IA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry @Shabiblac for the delay. Please write to student success about your partner. So they can allow you to work alone


fixed resetting start time bug

What would you like to be added:
Update status using client.Status().Update(..)
Why is this needed:
we reset StartTime to allow updating the nodeSelectors on the job. However StartTime is in status, which we can update via a status update call only
Issue Number:
#157
/kind bug

/ok-to-test

/lgtm

/lgtm

/label tide/merge-method-squash


LoginPop method is opening a microsoft login page in new tab on macbook pro (google chrome- Version 91.0.4472.114))

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.0.0
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.0.0
Description
I would like to have pop up on my existing application when I click on the login button(  login method is called )which is inside one of the dialog. But currently its opening up in new tab.
I am using macbook pro and Google chrome version - Version 91.0.4472.114
and my login() method is as follows
public login(): void {
this.authService.loginPopup({ scopes: authConfig.resourceScopes }).subscribe();
}
kindly help me to get a pop up instead of moving to a new tab.
Error Message
No response
Msal Logs
No response
MSAL Configuration
export const MSALInstanceFactory = () =>
  new PublicClientApplication({
    auth: {
      clientId: authConfig.clientId,
      authority: 'https://login.microsoftonline.com/' + authConfig.tenantId,
      redirectUri: authConfig.redirectUri,
    },
    cache: {
      cacheLocation: BrowserCacheLocation.LocalStorage,
    },
  });

Relevant Code Snippets
public login(): void {
    this.authService.loginPopup({ scopes: authConfig.resourceScopes }).subscribe();
 }

Reproduction Steps
Invoke the login() method.
Expected Behavior
microsoft login page should pop up on the same tab where the application is opened instead of opening up in a new tab.
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
Chrome, Safari
Regression
No response
Source
External (Customer)

@veenagadag We were able to reproduce this. Given this is happening on all browsers on Mac we're inclined to believe there is some system setting that is causing this but have, as of yet, been unable to find it. We're looking into this and will get back to you when we have a solution or workaround.

@tnorling - Hi , Can we customize the login pop up from msal . For example if I don't want to have 'can't access your account'? in the login screen then is there a way to get rid off . Please let me know .

@veenagadag I don't believe so. You can customize the branding of the page for your organization but I don't think you can remove individual elements. We are, however, not responsible for the sign-in pages so there may be certain features we are unaware of. These types of questions should be directed at Azure Support and they can give you a definitive answer.


[TASK] Deprecate DeclarationBlock::expandBorderShorthand

The expandShorthands/createShorthands Functions are deprecated and will be removed without substitution in version 10.0. Expanding and creating the shorthand notation is out of the scope of this library. If you want to include this functionality in your project or build it into a separate package, get the code from the v8.5.1 version of this library.
Helps with fixing #512

Rebased it.

Rebased it.


Listbox is required among UI controls

Hello
I want to use the listbox controller because I want to show previously printed messages in chat, etc.
There is no listBox among UI controls, are there any plans to create in the future?
How do I create a listbox control with the current functionality?

OK, I will make an example code in a few days.

リストボックス UIToolkit::ListBox
https://zenn.dev/reputeless/articles/73f6e98e1201e4bad42e

Thank you very much


Automated cherry pick of #5075: Fix ClusterClaim deletion webhook bug

Cherry pick of #5075 on release-1.12.
#5075: Fix ClusterClaim deletion webhook bug
For details on the cherry pick process, see the cherry pick requests page.

/test-multicluster-e2e

"Kind / E2e tests on a Kind cluster on Linux for Flow Visibility (pull_request) " keeps failing.

Hi @jianjuns I think it's because Kind test can't be rerun since the job depends on previous image tar which might be cleaned up.
The rerun error log is like
 Starting download for antrea-ubuntu-cov
Error: Unable to find an artifact with the name: antrea-ubuntu-cov

The original error is:
2023/06/13 03:07:02 Checking CoreDNS deployment
    fixtures.go:263: Deploying ClickHouse
    flowaggregator_test.go:166: Error when setting up test: services "clickhouse-clickhouse" not found
--- FAIL: TestFlowAggregator (35.64s)

I don't think this change will impact any Kind tests, maybe we can ignore it.

@luolanzone I merged this PR, but in general we should try to pass all default tests. I would not force merging a PR with failed tests.


Add pessimistic version constraint to instrumentation and overnight Canary Build

spawning off some discussion here https://github.com/open-telemetry/opentelemetry-ruby/issues/947, there are two areas that ought to be addressed in instrumentation

Be more defensive with version compatibility
Have a way to safely test against latest versions

Currently our instrumentation has the concept of a compatible block, which is just an arbitrary proc an instrumentation author can use to define logic for whether the instrumentation is compatible with the specific Gem version that has been installed. It could also be used to check arbitrary things like, for example, whether it's a jruby environment, or whether specific permissions are available, etc.
In practice, afaik, this is only used to do a MIN_VERSION comparison, like in action_pack, for example.
Over time, as new major versions come out, it becomes harder for us to say with confidence that our instrumentation is "compatible" with them, and so it might be worthwhile to make a pass through the instrumentation and add a MAX_VERSION comparison as well. In between the outcomes of "This instrumentation unexpectedly doesn't install and logs a warning to the user" and "This instrumentation causes an application crash during runtime", the former seems like a more reasonable posture for us to take.
With that posture we would also need to have some sort of mechanism for reliably testing latest major version releases. One solution here could be to include a latest or canary nightly build in CI, that pulls in the latest major+minor versions of everything , runs the test suite, and informs us of what fails. In this way we could establish some confidence when we want to bump the MAX_VERSION compatibility, or have some context to point to if contributors/users open issues asking for a major version bump.
lastly, as @arielvalentin suggests in this issue comment, https://github.com/open-telemetry/opentelemetry-ruby/issues/947#issue-1000098067 , a DSL for declaring min/max version may make it easier to maintain this approach long term. Perhaps some semantic conventions around what constants to use for MIN_VERSION and MAX_VERSION combined  with a modification to the base compatible() method could accomplish this (and also make it easier for a canary build to inform us when the latest gem version pulled down has a delta with the instrumentation's MAX version).

@robertlaurin Following up on our conversation from last week re: https://github.com/open-telemetry/opentelemetry-ruby/pull/1025.
Please correct me if I am misrepresenting your concerns and suggestions.
Some of concerns you brought up were:

Constraining max versions in gem specs or code make it difficult to test newer versions of instrumented gems
You are concerned about complexity of using multiple strategies for looking up the instrumented gems version
Giving developers the option to set an instrumented name adds complexity

You had suggested the following:

Have each instrumentation provide the version number of the instrumented gem using the constant instead of using Gem.loaded_specs or looking up the installed gem via the Gem API e.g. Kafka::VERSION

You had mentioned that you would run through this when you had time and would update the existing instrumentations. 👍🏼 that is totally fine

Require each instrumentation to declare an instrumented_library_name instead of making this field optional or trying to infer it from the instrumentation gem name.

Fine with me. I will put together a new PR that does that next week.

Keep the version constrains in code as constants instead of using the gemspec

This one I think is still up in the air. I pointed out during out chat that this would not alleviate the issue with ensuring we have a known/tested version as well as supporting an open ended version constraints you want to achieve with the appraisal gem.
I still find using RubyGems specifications to be the most idiomatic way to define compatibility constraints and compare the Requirement to the constant version of instrumented gem.
I recommend considering creating an open-ended appraisal and generating a special gemspec with an open-ended dependency for testing against the latest version of a library. I would also recommend that we make this check optional and potentially a nightly build for most cases that way if the "latest" version fails we do not block folks from releasing working versions.
We have a gemspec with the supported instrumented gems as a development dependency:
Gem::Specification.new do |spec|
  # lots of code...
  spec.add_development_dependency 'trilogy', '>= 2.0', '< 3.0'
end

We add an open-ended appraisal for latest
# frozen_string_literal: true

# Copyright The OpenTelemetry Authors
#
# SPDX-License-Identifier: Apache-2.0

appraise 'trilogy-2.0' do
  gem 'trilogy', '~> 2.0'
end

appraise 'trilogy-latest' do
  gem 'trilogy' # maybe a git repo for edge versions
end

We either maintain multiple gemspecs, generate a new one removing constraints, or possibly add an environment variable that leaves off the constraints
Gem::Specification.new do |spec|
  # lots of code...
  spec.add_development_dependency 'trilogy'
end

Thoughts?


NextMsg() on SyncSubscription may not return until Timeout when max is reached

Setup:

Create sync subscription
call AutoUnsubscribe
Have multiple go routines sitting on NextMsg(Timeout)
Send enough messages to reach the max
All but one NextMsg() will wait until Timeout before exiting, although the subscription is being removed.

(created for tracking purposes - fix coming up)

Closed by #153


update demo to use vite, simplify config.

Update demo to vite, remove previous webpack content.
Set src of react-tiny-popover to dist folder of parent.  Makes it easy to verify demo works on new code.
update to react 18

EDIT: I appreciate that the demo runs standalone on this website, so this may not be a wanted change.
It could be amended to NOT point to the dist folder and revert back to the npm package of react-tiny-popover, but keep the remaining changes?  let me know. I can amend.

Thank you for this. I was actually in the middle of removing this demo entirely, and creating a more modern demo in its own repo.


fix #126 main_window_controller refactor

Refactor into 4 classes

main window related stuff
main window jobs related stuff (MainWindowJobsController?)
main window files related stuff (MainWindowFilesController?)
journal processing (UpdateCycleProcessor?)
bring up test coverage to 80% for the refactored modules

Codecov Report
Attention: 425 lines in your changes are missing coverage. Please review.

Comparison is base (f92a556) 23.42% compared to head (9137f75) 29.52%.
Report is 23 commits behind head on main.

Files
Patch %
Lines

aoget/controller/file_model_controller.py
30.93%
163 Missing :warning:

aoget/controller/update_cycle.py
16.66%
110 Missing :warning:

aoget/controller/downloads.py
44.82%
32 Missing :warning:

aoget/controller/job_controller.py
81.88%
25 Missing :warning:

aoget/view/main_window_jobs.py
0.00%
22 Missing :warning:

aoget/controller/job_task_controller.py
25.00%
21 Missing :warning:

aoget/controller/main_window_controller.py
0.00%
19 Missing :warning:

aoget/view/main_window_files.py
0.00%
16 Missing :warning:

aoget/controller/app_cache.py
70.00%
9 Missing :warning:

aoget/controller/job_editor_controller.py
0.00%
8 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
+ Coverage   23.42%   29.52%   +6.10%
==========================================
  Files          48       59      +11
  Lines        3915     4240     +325
==========================================
+ Hits          917     1252     +335
+ Misses       2998     2988      -10

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


toster crash report

submitting as requested. uploading as a .txt file as github won't let me upload .toml files
report-32be1c8a-731f-486f-b090-e886bbfaaf31.txt

This seems to be a problem in Rust versions >=1.81, as the test name sorting library I use doesn't work on those versions anymore. I recommend temporarily downgrading Rust to 1.80 or below, a fix will be out soon.

Fixed in Toster 1.2.2.


Add label and disabled form for each checkbox type

Description

Storybook of other component such as [RadioGroup] has label for each type. [Checkbox] needs it too.
Some might curious, "What does disabled button looks like for each type?" So it's better to show them on storybook.

Related Issues
none.
Tests
none.
Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] Run yarn test or yarn test -u if you need to update snapshot.
[x] Run yarn lint
[x] I am willing to follow-up on review comments in a timely manner.

Could you kindly share the changes you've made in the screenshot? It'd be convenient for other reviewers to just watch the code instead of pulling it and building it if this is not a heavy updates :)

@hyochan  I understand your point. I edited my screenshoots. Thanks for your insight.

@hyochan I understand your point. I edited my screenshoots. Thanks for your insight.
By the way, is refactor label intentional? I think this PR is not related to refactor.

Thanks for the update. When I look into the screenshot, I feel like giving another row for disabled is bit much.
If the disabled color for each type has different styles, it would be nice but since disabled has a single style, I think it should stand next to the danger column instead of another row.

How do you think?

@hyochan  Thinking about it for a while, I realized I was stuck thinking of each column as a type, not a form. I totally agree with you. I updated code, description and screenshots.

@yujong-lee Oh, these changes are not made on the docs. Hope you make changes for the published one in our new document page which is in the docs directory.

@hyochan Oh. I got it. I made change in docs to.
I think it's quiet confusing to have two 'kinds' of docs.  old, new.
It seems like it's process of migrating old one to new one. Right?
If you explain more about it, I will add it to issue.
(I think there should be more issues so that anyone can easily start contribution.)


Error: Missing required config value at 'app.baseUrl' after fresh npx @backstage/create-app and plugin-create

Hello!
Followed docs to create fresh instance of backstage and to create fresh plugin.
According to docs, I can start plugin in isolation.
Expected Behavior
Plugin start in Isolation and I can enjoy hot reload and fast iteration time
Current Behavior
Error: Missing required config value at 'app.baseUrl' after yarn start
Possible Solution
I tried placing app-config.yaml in plugin directory, tried using env var APP_CONFIG_app_baseUrl - still getting this error.
May be it is suggested behavior, cause I managed to run it in backstage/backstage cloned repo. If so, It would be nice to have a note in docs about it.
Steps to Reproduce
npx @backstage/create-app
yarn create-plugin
cd plugins/...
yarn start

Context
Backstage is pretty complex environment for a newcomer, I enjoy exploring it in my way, and my way is start to build something fast. So hot reloading is pretty crucial. I could use main repo, but it's too heavy and hardware intensive :<
Your Environment

NodeJS Version (v12):LTS 14
Operating System and Version (e.g. Ubuntu 14.04):Ubuntu 20.04 LTS
Browser Information: Chrome

Thanks for reporting @dimkk - I was able to reproduce the error. I believe one should be able to start a standalone plugin even on a newly created Backstage app.
TL;DR of problem
I did some debugging and found out why this is happening. TL;DR app-config.yaml file is loaded correctly, but the Backstage app needs a config.d.ts to set the visibility of app.baseUrl to frontend. Read more about visibility of configs here. This is already being done in the backstage/backstage mono-repo, that is why you didn't encounter this problem.
Quick fix
@dimkk Quick fix to unblock yourself, add a config.d.ts file in packages/app or packages/backend like this one which increases the visibility of app.baseUrl to the frontend. And then update the relevant package.json to load the file, and set it as configSchema. You can read more here.
Details
The backstage-cli plugin:serve command loads all the app config files and tries to serve the bundle for the plugin. If we inspect the loaded config at this point, it looks like this
Loaded config from app-config.yaml
{
  schema: { process: [Function: process], serialize: [Function: serialize] },
  appConfigs: [ { data: [Object], context: 'app-config.yaml' } ],
  frontendConfig: ConfigReader {
    data: {},
    context: 'app-config.yaml',
    fallback: undefined,
    prefix: ''
  },
  frontendAppConfigs: [ { context: 'app-config.yaml', data: {} } ]
}

Note that the frontendConfig.data is empty since there is no config schema file like config.d.ts so far in the new Backstage app, hence all the configs are by default set to be visible to backend only.
Later on, the serve bundle logic is unable to get the URL where to serve the bundle at.
https://github.com/backstage/backstage/blob/81a6c95b0ab08df35da72b3e518630cdcabafdcc/packages/cli/src/lib/bundler/config.ts#L95
Proposed fix
The skeleton used by npx @backstage/create-app should have a simple config.d.ts file in packages/app which sets the visibility of app.baseUrl to frontend.
cc @backstage/reviewers

Actually worked! Thanks!

This looks like the default behaviour now and we can close this ticket right? https://github.com/backstage/backstage/blob/4045eaab952fec349e443990330bd1ac5b2c9c18/packages/core-app-api/config.d.ts#L21-L26

Going to close and then re-open if anyone is running into this.

@benjdlambert Ive created a frontend plugin and want to use property from app.config.yaml file but getting the error

Missing required config value at 'env.secret' in 'app-config.yaml'

below is my app-config.yaml
`yaml
app:
title: XYZ Backstage
baseUrl: http://<IP_ADDRESS>:3000
organization:
name: XYZ Org
backend:
platformRegion: 'us-east-1'
auth:
keys:
- secret: xxxxxxxxxxx
baseUrl: http://<IP_ADDRESS>:7007
listen:
port: 7007
csp:
connect-src: ["'self'", "http:", "https:"]
cors:
origin: http://<IP_ADDRESS>:3000
methods: [GET, HEAD, PATCH, POST, PUT, DELETE]
credentials: true
database:
client: pg
connection:
host: <IP_ADDRESS>
port: 5432
user: postgres
password: postgres
env:
secret: xxxxxxxxxxxxxxx
`
Steps I've done:

add config.d.ts file in plugin folder
added the properties of env and make it visible
include config.d.ts file in package.json

Goal is to read env.secret value in frontend plugin/react app, but no success.

@benjdlambert Thanks for your response The above issue is fixed now. I was also trying to set visibility on AWS secrets but it wasn't working but worked on custom secrets.


removed a sentence

zip files not supported
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ x] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.atlassian.net/browse/DOC-2517

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-508


How should third-party/Cargo.lock in release tarballs be interpreted by downstream users?

In release tarballs up to 1.0.83, the third-party deps lockfile at third-party/Cargo.lock included the cxx version, and was included in crate release version bumps:
https://github.com/dtolnay/cxx/commit/340f0ba52ac4086ee1fdd0c251dc3d4874043a53#diff-3d71a20e095c48e62d8dbc74d1a3dc4da4ddd6e2a9a9c3fea4f910243ff3e363
From 1.0.84, third-party/Cargo.lock is no longer generated in the context of building cxx, due to changes in https://github.com/dtolnay/cxx/pull/1141. This means downstream users of cxx cannot use  third-party/Cargo.lock to build with cargo build --locked.
How should third-party/Cargo.lock be interpreted? Is it meant to be the committed lockfile for the application components of cxx such as cxxbridge? And if so, can we ensure that it is usable as such?

It is not meant to be the committed lockfile for the application components of cxx such as cxxbridge.

To elaborate slightly more: I said this already in #1157 but the file you are fixated on is something used for generating the build targets which are part of this repo's Buck and Bazel-based test suite. I don't know of any documentation that would have directed you to use it for a different purpose.
Separately, the thing you are calling "release tarball" is not a release tarball but a snapshot of the git repository generated automatically by GitHub. You can obtain a "release tarball" of any arbitrary commit of any repo; it is not particular to releases. Those tars reflects whatever contents are in the repo. It is not correct to expect that whatever can be found in a tarball must have been placed there with the intention of having some intended purpose outside of the source repo. Test suites, websites, graphics, etc would all be found in there. If it's in the repo, it's in GitHub's generated tars of the repo. Notably, this is a different thing than rust-lang/rust's rust-src rustup component, for example, which is a release tarball.
In general the norm for lockfiles in Rust is to provide them for applications only. The cxxbridge-cmd crate is not an application. It is a dependency which you would use at build time for building some application. Dependencies which are used at build time for building an application (including libraries, build scripts, proc macros, etc) are not supposed to have each their own unique lockfile. For example the serde crate, or serde_derive, or cxx, or cxxbridge-cmd, would be governed by the lockfile of whatever application whose build they are a part of. This is important because you want sharing of the common dependency graph between different dependencies of an application's build. For example if serde_derive and cxx and cxxbridge-cmd all depend on some crate like syn, the one version of syn in that application's lockfile is the one that all of those dependencies would optimally be built against. It does not make sense for each such crate to have a different lockfile all out of sync with one another.

The cxxbridge-cmd crate is not an application.

This confuses me, because we are literally building a cxxbridge binary? And it is not possible (without very invasive and customised hooking that would likely break across cxx releases) to build this binary using the same Cargo.lock file as the Rust components of our hybrid application, because the binary is defined in the cxxbridge crate, not our workspace.

It is a dependency which you would use at build time for building some application. Dependencies which are used at build time for building an application (including libraries, build scripts, proc macros, etc) are not supposed to have each their own unique lockfile. For example the serde crate, or serde_derive, or cxx, or cxxbridge-cmd, would be governed by the lockfile of whatever application whose build they are a part of.

Indeed, when cxx-build is used inside the build.rs file of a Cargo-led build process, then it is used as a library, and its dependencies are pinned to the application's Cargo.lock. But an automake-led build process needs to use the cxxbridge binary as documented here, and that binary needs to be built with its own Cargo.lock file.
In any case, this answer sufficiently clarifies your position:

It is not meant to be the committed lockfile for the application components of cxx such as cxxbridge.

and tells me that we need to go back to our original approach of checking in two separate Cargo.lock files (one for our workspace, and one specifically for cxxbridge), and updating both of them every time we update cxx in any way.

I believe you are conflating "application" with --crate-type=bin. The latter is something more general / a superset of the former. Something being built as a binary (crate type) during a build does not make it what is ordinarily meant by application.
For example serde's or cxx's build script are also binaries. You can confirm this by running a cargo build --verbose on anything which depends on those libraries. The build scripts do not come with their own lockfile. They are nevertheless built using --crate-type=bin, which is no different from how a proper build system would be expected to integrate cxxbridge-cmd and any other code generators relevant to a build.

The build scripts do not come with their own lockfile.

Indeed, because a Cargo build script is not "standalone"; it is a sub-component of the Rust crate being built, that happens to be compiled as a binary and run as a sub-task by cargo. That is an artifact of how Cargo works, and is how it can share a Cargo.lock file. This is not possible for a binary built from a different Cargo.toml, as described below.

The cxx documentation, which is what I referenced when integrating it into our automake build system (about as proper as one gets for a legacy C++ application), states (emphasis added):

Producing the generated code
CXX's Rust code generation automatically happens when the #[cxx::bridge] procedural macro is expanded during the normal Rust compilation process, so no special build steps are required there.
But the C++ side of the bindings needs to be generated. Your options are:

Use the cxxbridge command, which is a standalone command line interface to the CXX C++ code generator. Wire up your build system to compile and invoke this tool. [... It] can be built from the gen/cmd/ directory of the CXX GitHub repo.

This is precisely what we are doing. The build system runs:
[make -C depends, which runs]
cd gen/cmd && cargo build --locked --offline --release --package=cxxbridge-cmd --bin=cxxbridge

[remainder of dependencies are built]

[./configure]

[make, which runs]
...
path/to/cxxbridge $(CXXBRIDGE_OPTS) rust/src/$(basename $(@F)).rs -o $@
...
cargo build --manifest-path path/to/our/Cargo.toml
...

cxxbridge is being built by a different cargo build invocation than our Rust library, using a different Cargo.toml file (specifically, the one from this repository at gen/cmd/Cargo.toml). And (I know you are a long-time core contributor to the Rust ecosystem, so you already know this, but stating it for completeness) a Cargo.lock file is always defined relative to a given Cargo.toml file(-state). This means that:
cxxbridge does not, and cannot, use the same Cargo.lock file as our Rust library.
If I attempt to do so, I get the following error:
$ tar xzf cxx-1.0.83.tar.gz
$ cp path/to/our/repository/Cargo.lock cxx-1.0.83/Cargo.lock
$ cd cxx-1.0.83/gen/cmd
$ cargo build --locked --offline --release --package=cxxbridge-cmd
error: the lock file path/to/cxx-1.0.83/Cargo.lock needs to be updated but --locked was passed to prevent this
If you want to try to generate the lock file without accessing the network, remove the --locked flag and use --offline instead.

As noted in #1157, we cannot build without --locked for reproducibility reasons; specifically the build needs to work in an offline deterministic VM environment with vendored dependencies. And because cxxbridge is built by a different cargo build invocation and using a different Cargo.toml, it has a different set of dependencies, only a subset of which are in common with our Rust library.
There are hacks I could use to get around this, but (in addition to breaking several boundary abstractions within our build system) they are semantically eqivalent to "just ship our own Cargo.lock file for cxxbridge that we bump on updates, instead of depending on one provided by the cxx repo".

Hopefully this makes it clear that there exists an impedance mismatch between your intentions for how cxxbridge should be used, and how downstream people like myself actually use it based on reading the cxx documentation and looking at the repository contents. Hopefully you can use this information to improve the cxx documentation for future users. But I have now spent over an hour of my life in this issue thread, have run out of spoons trying to convey my downstream user experience to you, and am unable to continue this conversation.

One last follow-up for the benefit of others who might find this issue: I just now learned about RFC 3028: cargo binary dependencies which would AFAICT allow a binary build tool to be tracked as a dependency in a project's Cargo.toml and built by it, therefore sharing its Cargo.lock file. This further supporting my understanding that right now this is not possible.


Updating helia-coord

:tada: This PR is included in version 3.2.3 :tada:
Your semantic-release bot :package::rocket:


Py3

Additional changes:

xrange
help by default
fix output to hex dump

In the meanwhile - this is what I have to add at this point:
https://github.com/frenzymadness/officeparser/pull/1
I know that --export-macros is not working in python3.
Tested like this:

download malware sample xls with macros from hybrid-analysis.com
https://www.hybrid-analysis.com/sample/8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f?environmentId=100

gunzip the file

gunzip 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin.gz

try with python2

python2 officeparser.py --extract-macros 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
Traceback (most recent call last):
  File "officeparser.py", line 1235, in <module>
    _main()
  File "officeparser.py", line 836, in _main
    buffer = StringIO()
NameError: global name 'StringIO' is not defined

try with python3

$ python3 $(which officeparser.py) --extract-macros 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
Traceback (most recent call last):
  File "/usr/bin/officeparser.py", line 1234, in <module>
    _main()
  File "/usr/bin/officeparser.py", line 835, in _main
    buffer = StringIO()
NameError: name 'StringIO' is not defined

Even including "from io import StringIO" is not directly fixing the situation:
5) try with python2 and io.StringIO
python2 officeparser.py --extract-macros 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
Traceback (most recent call last):
  File "officeparser.py", line 1235, in <module>
    _main()
  File "officeparser.py", line 837, in _main
    buffer.write(ofdoc.get_stream(project.index))
TypeError: unicode argument expected, got 'str'

try with python3 and io.StringIO

python3 officeparser.py --extract-macros 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
Traceback (most recent call last):
  File "officeparser.py", line 1235, in <module>
    _main()
  File "officeparser.py", line 837, in _main
    buffer.write(ofdoc.get_stream(project.index))
TypeError: string argument expected, got 'bytes'

The original (cStringIO.StringIO) gives this:
$ python2 officeparser.py.orig --extract-macros 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
INFO: Saving VBA code to ./Sem_1.cls
INFO: Saving VBA code to ./Page1_1.cls
INFO: Saving VBA code to ./Module1_1.bas
INFO: Saving VBA code to ./UserForm1_1.frm
INFO: Saving VBA code to ./Module2_1.bas
INFO: Saving VBA code to ./Module3_1.bas
INFO: Saving VBA code to ./UserForm6_1.frm
INFO: Saving VBA code to ./Page11_1.cls
INFO: Saving VBA code to ./Module6_1.bas
INFO: Saving VBA code to ./Module5_1.bas
INFO: Saving VBA code to ./Module4_1.bas
INFO: Saving VBA code to ./Class1_1.cls
INFO: Saving VBA code to ./Sheet1_1.cls

I'll test it tomorrow but I think that you can try to use BytesIO from io module.

This PR looks good to me. Unfortunately, I cannot download the binary sample because I don't have an account there. Could you please test it with BytesIO from io module?

Changing to BytesIO makes that working with python2, still fails with python3.
$ python2 officeparser.py --extract-macros 8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
INFO: Saving VBA code to ./Sem_2.cls
INFO: Saving VBA code to ./Page1_2.cls
INFO: Saving VBA code to ./Module1_2.bas
INFO: Saving VBA code to ./UserForm1_2.frm
INFO: Saving VBA code to ./Module2_2.bas
INFO: Saving VBA code to ./Module3_2.bas
INFO: Saving VBA code to ./UserForm6_2.frm
INFO: Saving VBA code to ./Page11_2.cls
INFO: Saving VBA code to ./Module6_2.bas
INFO: Saving VBA code to ./Module5_2.bas
INFO: Saving VBA code to ./Module4_2.bas
INFO: Saving VBA code to ./Class1_2.cls
INFO: Saving VBA code to ./Sheet1_2.cls
python3 officeparser.py --extract-macros ~/wrk/officeparser/8db9495dcd5b9ed6a8f1844ffc496f3eb282eb323a00e6d4aa92c58710c5890f.bin
Traceback (most recent call last):
File "officeparser.py", line 1235, in
_main()
File "officeparser.py", line 856, in _main
m = re_keyval.match(line)
TypeError: cannot use a string pattern on a bytes-like object

I cannot download the binary sample because I don't have an account there.
the registration on hybrid-analysis is free of charge, but no problem.

For example this file would download without registration and should be relatively safe to handle for tests:
https://www.activebarcode.com/download/demo/word_form.doc
It works in the unpatched python2:
$ python2 officeparser.py.orig --extract-macros word_form.doc
INFO: Saving VBA code to ./ThisDocument_1.cls
INFO: Saving VBA code to ./NewMacros_1.bas

Patched version works in python2 using the io.BytesIO:
$ python2 officeparser.py --extract-macros /home/mambroz/tmp/macros/word_form.doc
INFO: Saving VBA code to ./ThisDocument.cls
INFO: Saving VBA code to ./NewMacros.bas

But gives some more errors with python3:
$ python3 officeparser.py --extract-macros /home/mambroz/tmp/macros/word_form.doc
Traceback (most recent call last):
  File "officeparser.py", line 1235, in <module>
    _main()
  File "officeparser.py", line 579, in _main
    ofdoc = CompoundBinaryFile(args[0], parser_options)
  File "officeparser.py", line 178, in __init__
    self.header = Header(self.f.read(512), parser_options)
ValueError: read of closed file

It actually fails also with the "--print-header" and "--print-streams" with python3.

I've

included your commits into my branch
Fixed a problem with ValueError: read of closed file (the very last commit)
Investigated the problem with macros extraction

The very first question I need an answer for is whether PROJECT stream in document should be handled as bytes or Unicode. Because now it's mixed and that's the reason why it does not work in Python 3. Do I understand it correctly that it contains some code in VB script so it should be handled as Unicode?

Hello,
sorry I thought I already answered, but my answer is not here.
Yes PROJECT stream in Office documents seems to hold metadata about the macros in the plaintext form in the INI format.  For example:
$ python2 officeparser.py.orig --dump-stream-by-name PROJECT word_form.doc
ID="{F71D9A8C-3763-458D-A309-7E5E41C49A1A}"
Document=ThisDocument/&H00000000
Module=NewMacros
Name="Project"
HelpContextID="0"
VersionCompatible32="393222000"
CMG="C1C327AD2BAD2BAD2BAD2B"
DPB="828064A724A824A824"
GC="4341A5E667E767E798"

[Host Extender Info]
&H00000001={3832D640-CF90-11CF-8E43-00A0C911005A};VBE;&H00000000
&H00000002={000209F2-0000-0000-C000-000000000046};Word8.0;&H00000000

[Workspace]
ThisDocument=46, 46, 678, 454,
NewMacros=69, 69, 678, 506, Z


Initial maintenance work

Closes #1 by doing a lot of misc initial maintenance work. I could break this apart into many PRs but I think it isn't worth the additional effort at this point in time for this project.

Related issue: https://github.com/jupyterhub/team-compass/issues/453


Add basic auth support when connecting to mesos

Blocked by https://issues.apache.org/jira/browse/MESOS-3923


WIP: nixos/luks: Allow for many key files

Description of changes
Would be great to have a series of key files available to try for a device.
TODO: get nixosTests.luks to not hang (not sure if this was related to my 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

jon ringer, back in nixpkgs???

jon ringer, back in nixpkgs???

Aye, took a break. Been reviewing some PRs too.
Should be of use for ZHF as well.


Support logical types with arguments in Python SDK

Fixes: https://github.com/apache/beam/issues/23373
Possible (partial) implementation of logical types with arguments. Currently, only map and atomic values are supported.
Adds an instance method _language_type() to LogicalType that returns the language type for a specific instance of a logical type, i.e. a logical type with a specific argument. For logical types without arguments, this should just return the existing language_type().
Several downsides:

Types need to be hashable and in practice should define a hash function that is a combination of the class and the argument. This implies the argument needs to be hashable as well.
PassThroughLogicalTypes cannot have an argument, as it's impossible for the language type to match the representation type while also acting as a container for the argument that is excluded from the representation (by definition).

I'm not sure if there's a better way to approach this? I can also remove the EnumerationType from schemas.py and use a test specific schema defined/registered in schemas_test.py.

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

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Thanks. Taking a look.

Run Python3.11 PostCommit

Run Python 3.11 PostCommit

This breaks jdbc integration test:
a = (<apache_beam.io.external.xlang_jdbcio_it_test.CrossLanguageJdbcIOTest testMethod=test_xlang_jdbc_write_read_0_postgres>,)
kw = {}

    @wraps(func)
    def standalone_func(*a, **kw):
>       return func(*(a + p.args), **p.kwargs, **kw)
...
apache_beam/typehints/schemas.py:601: in named_tuple_from_schema
    schema_registry=schema_registry).named_tuple_from_schema(schema)
apache_beam/typehints/schemas.py:558: in named_tuple_from_schema
    field_py_type = self.typing_from_runner_api(field.type)
apache_beam/typehints/schemas.py:543: in typing_from_runner_api
    return LogicalType.from_runner_api(
apache_beam/typehints/schemas.py:814: in from_runner_api
    logical_type, logical_type_instance._language_type())
...
cls = <apache_beam.io.jdbc.JdbcDateType object at 0x7f6eac4ff4d0>

    def _language_type(cls):
...
>     raise NotImplementedError()
E     NotImplementedError

apache_beam/typehints/schemas.py:714: NotImplementedError

Run Python 3.11 PostCommit

Run Python 3.11 PostCommit

I am OOO at the moment. @Abacn @riteshghorse can you help review this PR for me? thanks

Seems like this is breaking the jdbc test -

https://ci-beam.apache.org/job/beam_PostCommit_Python311_PR/152/consoleText

@riteshghorse I believe the test failures are because I only implemented value_from_runner_api for map types (all other non-atomic types, including the row type used in that test, remain ValueErrors). I can fix this, but it doesn't answer the larger questions around the type registry.

Hey @bsannicolas ! sorry for the delay here!
You can run postcommit with ./gradlew :sdks:python:test-suites:portable:py311:postCommitPy311IT from the beam directory

waiting on author


[FEATURE] Add Longhorn and Rancher dashboard links in Support section

For troubleshooting and observability, it is necessary sometimes to go into the Longhorn or Rancher Dashboards.
The request is to add a link to both within the Support section or anywhere else it makes sense.

Will add those two dashboard page links on the support page, and more importantly,  it will only be used for debugging and troubling shooting purposes.
Note: the URL link for single and multiple dashboards would be different cc @WuJun2016

Verified fixed on master-1646cb66-head. (11/18). Close this issue.
Result

Rancher link and longhorn link display on the support page after enable the dev mode

Click the Rancher Access Embedded Rancher UI will open a new page and direct to the content

Click the Access Embedded Longhorn UI will open a new page and direct to the content

Test Information

Test Environment: 1 node local kvm machine
Harvester version: master-1646cb66-head. (11/18)

Verify Steps

Go to support page
Both of rancher link and longhorn link will not be visible.
Go to user preference page
Enable DEV mode
Back to support page
Both of rancher link and longhorn link will be visible.
Click links will open a new tab with rancher/longhorn embedded UI.


Share screen functionality not working while building a client application using livekit

I have using electron JS to build a client application for virtual meeting and collaboration. I have used react livekit frame work build the application.
On using control bar component which provide video, audio, leave and share-screen functionality by default. Share screen feature is not working inside client application. And some how I not be able to customise the logic behind share screen.
In browser share screen is working fine not on client appication its not.

This repo contains livekit server and it sounds like the issue you are reporting is related with the client SDK.
For Electron, you will need to use desktopCapturer APIs to accomplish this.


CIDR functions for ARM templates are unrecognized.

When building an arm template that uses any of the CIDR functions, the functions such as parseCidr, cidrHost, etc show as unrecognized. The template does deploy fine and the output is expected though.

here is an example template that you can use to see what I mean:
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
    },
    "functions": [],
    "variables": {
        "sqlServerName": "[concat('exampleServer1','-',uniqueString(resourceGroup().id))]",
        "allowedIpAddresses": [
            {
                "name": "example network 1",
                "CIDR": "<IP_ADDRESS>/24"
            },
            {
                "name": "example network 2",
                "CIDR": "<IP_ADDRESS>/8"
            }
        ]
    },
    "resources": [
        {
            "name": "[variables('sqlServername')]",
            "type": "Microsoft.Sql/servers",
            "location": "[resourceGroup().location]",
            "apiVersion": "2021-11-01",
            "dependsOn": [],
            "properties": {
                "version": "12.0",
                "minimalTlsVersion": "1.2",
                "administratorLogin": "exampleUser",
                "administratorLoginPassword": "badPassword123@@"
            },
            "resources": [
            ]
        },
        {
            "copy": {
                "name": "sqlFirewallAllowedIPs'",
                "count": "[length(variables('allowedIpAddresses'))]"
            },
            "name": "[concat(variables('sqlServername'),'/',variables('allowedIpAddresses')[copyIndex()].name)]",
            "type": "Microsoft.Sql/servers/firewallRules",
            "location": "[resourceGroup().location]",
            "tags": {
                "displayName": "SqlFirewallRules"
            },
            "apiVersion": "2021-11-01",
            "dependsOn": [
                "[resourceId('Microsoft.Sql/servers', variables('sqlServername'))]"
            ],
            "properties": {
                "startIpAddress": "[parseCidr(variables('allowedIpAddresses')[copyIndex()].CIDR).firstUsable]",
                "endIpAddress": "[parseCidr(variables('allowedIpAddresses')[copyIndex()].CIDR).lastUsable]"
            }
        }
    ],
    "outputs": {}
}

This template deploys fine, but parseCidr shows as an unrecognized function in VSCode with the ARM template extension in use.

@kyle-ntx Thanks for letting us know, and providing sample code is a big help.


upper case first letter of title in PR

i think people like it, i know i never upper case commit messages but maybe it looks more elegant after all

You mean the merge message? I followed the Symfony casing :)
I actually want (when time comes) add a bit more flexibility in how the merge message is rendered, but not using the current template System, but more something like Twig (only lighter, as Twig is a little to big :smiley: but maybe we strip Twig down to what we need).

I think the default title is set from the first commit message, however if this commit message starts with lower case then it will not upper case the first letter of the phrase/sentence by default which i think would be desirable.

:+1:

addressed in #507

@GrahamCampbell there must be some setting on git to also ucfirst the individual commit messages, do you know? or anyone else? :blush:

It is possible to force a specific format of the commit message
https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks

The commit-msg hook takes one parameter, which again is the path to a temporary file that contains the commit message written by the developer. If this script exits non-zero, Git aborts the commit process, so you can use it to validate your project state or commit message before allowing a commit to go through. In the last section of this chapter, We’ll demonstrate using this hook to check that your commit message is conformant to a required pattern.

You can (ab)use this to change the commit message but I'm not sure if this will also fire for a merge.
http://stackoverflow.com/a/5824876
Also you need to apply this hook locally, meaning very one of the developers/contributers need to install this.


Game Page API calls for Google Maps and Street View

https://github.com/sopra-fs24-group-12/PeekSeek-client/commit/a1e806f14dbf7edd008e1b6e37e7f27d37205b08


Provide example for batches and updates

A couple of examples would be helpful.
qb package appears deprecated, but there is no other apparent support for batches in currently latest version (2.8.0).
As there is no evident way to extract values from gocqlx.Queryx for gocql.Batch.Query  function either, it is not clear how gocql.Batch is supposed to be used with gocqlx now.
Similarly with updates, there is only Insert/Select examples, but an update example would be also appreciated.

In the master branch you can bindstruct to a batch (https://github.com/scylladb/gocqlx/blob/master/batchx_test.go) but it's not released yet. I forked the repo and released it to my own private branch.


Duplicates of linked collection not exported.

Describe the bug
When you create a collection in A.blend, then create B.blend and link this collection, then create a duplicate of the instance, the duplicated instance is not exported.
To Reproduce
See describe the bug
Expected behavior
Each instance should be exported as node in GLTF.
Version

OS: Arch Linux 5.1.4
Blender Version: 2.8, 2019-05-25

Additional context
Might be relevant: https://github.com/KhronosGroup/glTF-Blender-IO/issues/484

For anyone who runs into this issue, I found a workaround for the time being. 1) Exprort to FBX, then use 2) https://github.com/facebookincubator/FBX2glTF

I started to have a look on it.
Some thoughts about it (for myself, to not forgot)

instances are used for dupli instance and for linking instances
Need to invalidate the cache to get a new node, to avoid having same node multiple time in scene tree
Have to check how it will work for armature instances (proxy)
Check that nodes are not in scene root
instance offset seems to not work as expected

Thanks @julienduroure awesome you're looking into this.
I dove into Blender/Python a bit and got a very simplistic exporter working that fits my needs. I'm sure everything in that code is familiar to you, but maybe there are some parts that might be interesting. Especially because it works with linked files/collections/etc.
https://gist.github.com/roxlu/a3ec74d3e214526c026bdc06adc24117#file-exporter-py-L848

I am getting the same issue.
monkeys.blend contains one Monkey head, with two instances using Blender Collection Instances:
https://docs.blender.org/manual/en/latest/scene_layout/object/properties/instancing/collection.html

Exported directly to gltf format only one shows:

monkeys-blender.gltf 2KB
monkeys-blender.bin 69KB
meshes:
{
    "name" : "Suzanne.004",
    "primitives" : [
        {
            "attributes" : {
                "POSITION" : 0,
                "NORMAL" : 1,
                "TEXCOORD_0" : 2
            },
            "indices" : 3
        }
    ]
}

nodes:
{
    "mesh" : 0,
    "name" : "Suzanne"
},
{
    "children" : [
        0
    ],
    "name" : "Duplication_Offset_Monkeys2"
},
{
    "name" : "Duplication_Offset_Monkeys3"
}

However export to FBX, and then to GLTF using: https://github.com/facebookincubator/FBX2glTF

monkeys-fbx.gltf 8KB
monkeys-fbx.bin 137KB
meshes:
{
    "name": "Suzanne",
    "primitives": [
        {
            "material": 0,
            "mode": 4,
            "attributes": {
                "NORMAL": 2,
                "POSITION": 1,
                "TEXCOORD_0": 3
            },
            "indices": 0
        }
    ]
},
{
    "name": "Monkeys3|Suzanne.001|Dupli|",
    "primitives": [
        {
            "material": 0,
            "mode": 4,
            "attributes": {
                "NORMAL": 6,
                "POSITION": 5,
                "TEXCOORD_0": 7
            },
            "indices": 4
        }
    ]
}

nodes:
{
    "name": "RootNode",
    "translation": [
        0.0,
        0.0,
        0.0
    ],
    "rotation": [
        0.0,
        0.0,
        0.0,
        1.0
    ],
    "scale": [
        1.0,
        1.0,
        1.0
    ],
    "children": [
        1,
        2,
        4
    ]
},
{
    "name": "Suzanne",
    "translation": [
        0.0,
        0.0,
        0.0
    ],
    "rotation": [
        -0.707106828689575,
        0.0,
        0.0,
        0.707106709480286
    ],
    "scale": [
        100.0,
        100.0,
        100.0
    ],
    "mesh": 0
},
{
    "name": "Monkeys2",
    "translation": [
        0.0,
        0.0,
        -3.93487048149109
    ],
    "rotation": [
        -0.707106828689575,
        0.0,
        0.0,
        0.707106709480286
    ],
    "scale": [
        100.0,
        100.0,
        100.0
    ],
    "children": [
        3
    ]
},
{
    "name": "Monkeys2|Suzanne|Dupli|",
    "translation": [
        0.0,
        0.0,
        0.0
    ],
    "rotation": [
        -1.32348898008484e-23,
        -0.0,
        0.0,
        1.0
    ],
    "scale": [
        1.0,
        1.0,
        1.0
    ],
    "mesh": 0
},
{
    "name": "Monkeys3",
    "translation": [
        -3.78579711914062,
        0.0,
        0.200917467474937
    ],
    "rotation": [
        -0.707106828689575,
        0.0,
        0.0,
        0.707106709480286
    ],
    "scale": [
        100.0,
        100.0,
        100.0
    ],
    "children": [
        5
    ]
},
{
    "name": "Monkeys3|Suzanne.001|Dupli|",
    "translation": [
        0.0,
        0.0,
        0.0
    ],
    "rotation": [
        -1.32348898008484e-23,
        -0.0,
        0.0,
        1.0
    ],
    "scale": [
        1.0,
        1.0,
        1.0
    ],
    "mesh": 1
}

This feature is essential for creating small compressed 3D models for the web. It results in smaller Network requests which is helpful especially for Mobile devices..

@kmturley This is solved by PR #950.
Target is to merge it soon (and will be available in blender 2.83)

Thanks, he work for me but he don't work if you have A link to B and link to C.

@StidOfficial I opened  #1049 to track this subject

For me it still isnt working in version 1.5.1 of the addon.
When I link in a collection and export everything works fine, if I duplicate that collection the gltf viewer gives me the following errors

I exported with "selected objects" turned off.

Only workaround for me is to use make library override form relations menu (not Outliner, that does nothing)


BUG:

Modin version checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest released version of Modin.

[X] I have confirmed this bug exists on the main branch of Modin. (In order to do this you can follow this guide.)

Reproducible Example
import numpy  as np

import ray
ray.shutdown()
ray.init()

# pdm: MODIN pandas
import modin.pandas as pdm
# pdo: ORIGINAL pandas
import pandas as pdo

a = np.random.randint(0, high=100, size=(50000000, 4))
# do: ORIGINAL pandas DataFrame
do = pdo.DataFrame(a, columns=['x'+str(i+1) for i in range(4)])
# dm: MODIN pandas DataFrame
dm = pdm.DataFrame(a, columns=['x'+str(i+1) for i in range(4)])

# This works as expected:
%time ho = do.groupby('x1').apply(lambda g: (g.x1 + g.x2).mean())

# This produces only error messages but no output:
%time hm = dm.groupby('x1').apply(lambda g: (g.x1 + g.x2).mean())

# speed issues:
%time do['s'] = do.groupby(['x1']).x2.transform('sum')
# This is considerably SLOWER than the standard pandas version above
# on a windows server with 20 CPUs!!!
%time dm['s'] = dm.groupby(['x1']).x2.transform('sum')

%time ho = do[['x1', 'x2', 'x3']].sum()

%time hm = dm[['x1', 'x2', 'x3']].sum()

Issue Description
The following simple apply operation on a modin DataFrame
dm.groupby('x1').apply(lambda g: (g.x1 + g.x2).mean())
breaks with error messages. This must be a bug since the same method produces sensible results for a standard pandas DataFrame
Furthermore I perfo
Expected Behavior
the apply() above sould yield the same result for a modin DataFrame as for a pandas DataFrame. It breaks
Error Logs
2023-03-30 09:40:02,073	WARNING worker.py:1866 -- Traceback (most recent call last):
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\import_thread.py", line 204, in fetch_and_execute_function_to_run
function({"worker": self.worker})
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\modin\engines\ray\utils.py", line 84, in import_pandas
import pandas  # noqa F401
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas_init.py", line 50, in
from pandas.core.api import (
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\api.py", line 48, in
from pandas.core.groupby import (
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\groupby_init_.py", line 1, in
from pandas.core.groupby.generic import (
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\groupby\generic.py", line 73, in
from pandas.core.frame import DataFrame
File "", line 1004, in _find_and_load
File "", line 158, in enter
File "", line 103, in acquire
_frozen_importlib._DeadlockError: deadlock detected by _ModuleLock('pandas.core.frame') at<PHONE_NUMBER>104
(apply_func pid=21260) 2023-03-30 09:40:02,104	ERROR serialization.py:371 -- cannot import name 'DataFrame' from partially initialized module 'pandas.core.frame' (most likely due to a circular import) (C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\frame.py)
(apply_func pid=21260) Traceback (most recent call last):
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 369, in deserialize_objects
(apply_func pid=21260)     obj = self._deserialize_object(data, metadata, object_ref)
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 252, in _deserialize_object
(apply_func pid=21260)     return self._deserialize_msgpack_data(data, metadata_fields)
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 207, in _deserialize_msgpack_data
(apply_func pid=21260)     python_objects = self.deserialize_pickle5_data(pickle5_data)
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 195, in deserialize_pickle5_data
(apply_func pid=21260)     obj = pickle.loads(in_band, buffers=buffers)
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\frame.py", line 129, in
(apply_func pid=21260)     from pandas.core import (
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\generic.py", line 108, in
(apply_func pid=21260)     from pandas.core import (
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\indexing.py", line 46, in
(apply_func pid=21260)     import pandas.core.common as com
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas_init.py", line 50, in
(apply_func pid=21260)     from pandas.core.api import (
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\api.py", line 48, in
(apply_func pid=21260)     from pandas.core.groupby import (
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\groupby_init.py", line 1, in
(apply_func pid=21260)     from pandas.core.groupby.generic import (
(apply_func pid=21260)   File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\groupby\generic.py", line 73, in
(apply_func pid=21260)     from pandas.core.frame import DataFrame
(apply_func pid=21260) ImportError: cannot import name 'DataFrame' from partially initialized module 'pandas.core.frame' (most likely due to a circular import) (C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\frame.py)
2023-03-30 09:40:03,120	ERROR worker.py:399 -- Unhandled error (suppress with 'RAY_IGNORE_UNHANDLED_ERRORS=1'): ray::apply_func() (pid=28736, ip=<IP_ADDRESS>)
File "python\ray_raylet.pyx", line 528, in ray._raylet.raise_if_dependency_failed
ray.exceptions.RayTaskError: ray::deploy_ray_func() (pid=28736, ip=<IP_ADDRESS>)
File "python\ray_raylet.pyx", line 528, in ray._raylet.raise_if_dependency_failed
ray.exceptions.RayTaskError: ray::apply_func() (pid=21260, ip=<IP_ADDRESS>)
File "python\ray_raylet.pyx", line 528, in ray._raylet.raise_if_dependency_failed
ray.exceptions.RaySystemError: System error: cannot import name 'DataFrame' from partially initialized module 'pandas.core.frame' (most likely due to a circular import) (C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\frame.py)
traceback: Traceback (most recent call last):
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 369, in deserialize_objects
obj = self._deserialize_object(data, metadata, object_ref)
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 252, in _deserialize_object
return self._deserialize_msgpack_data(data, metadata_fields)
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 207, in _deserialize_msgpack_data
python_objects = self._deserialize_pickle5_data(pickle5_data)
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\ray_private\serialization.py", line 195, in _deserialize_pickle5_data
obj = pickle.loads(in_band, buffers=buffers)
File "C:\Anwendungen\Anaconda\envs\hans\lib\site-packages\pandas\core\frame.py", line 129, in
from pandas.core import (
...
 I get a different ray error ending with ValueError: An application is trying to access a Ray object whose owner is unknown(00ffffffffffffffffffffffffffffffffffffff0100000001000000). Please make sure that all Ray objects you are trying to access are part of the current Ray session[...]
I believe the ray errors you are seeing come from a bug in ray that Modin added a workaround for in #4603. Modin will automatically initialize ray with runtime_env={'env_vars': {'__MODIN_AUTOIMPORT_PANDAS__': '1'}} to work around this bug. If I remove the ray.init() from your script so that Modin initializes ray for you, it works. Will that work for you?
About the performance, let's follow up on #5905.

Dear Mahesh,
thank you for the prompt reply. Your solution works.
May I proceed here with another issue? It concerns speed.
Original pandas is considerably faster for the apply:
%time ho = do.groupby('x1').apply(lambda g: (g.x1 + g.x2).mean())
Wall time: 13.1 s
%time hm = dm.groupby('x1').apply(lambda g: (g.x1 + g.x2).mean())
Wall time: 23.2 s
This is strange since I work (alone!) on a virtual windows server with 20 (sic!) CPUs
Look at the next simple transform command which should be quite well suited for parallelization.
Here is original pandas pandas at work, takinng 4.11 seconds
%time do['s'] = do.groupby(['x1']).x2.transform('sum')
Wall time: 4.11 s
And modin consumes 6.48 seconds
%time dm['s'] = dm.groupby(['x1']).x2.transform('sum')
Wall time: 6.48 s
It would be extremely helpful to have benchmarks fort he most important transformations like
groupby(), apply(), transform(), aggregate(), sum() etc.
Thank you and best regards,
Johannes
Von: Mahesh Vashishtha @.>
Gesendet: Donnerstag, 30. März 2023 15:07
An: modin-project/modin @.>
Cc: Ludsteck Johannes @.>; Mention @.>
Betreff: Re: [modin-project/modin] BUG: (Issue #5904)
@LudsteckJhttps://github.com/LudsteckJ thank you for reporting this issue. I get a different ray error ending with ValueError: An application is trying to access a Ray object whose owner is unknown(00ffffffffffffffffffffffffffffffffffffff0100000001000000). Please make sure that all Ray objects you are trying to access are part of the current Ray session[...]
I believe the ray errors you are seeing come from a bug in ray that Modin added a workaround for in #4603https://github.com/modin-project/modin/pull/4603. Modin will automatically initialize ray with runtime_env={'env_vars': {'MODIN_AUTOIMPORT_PANDAS': '1'}} to work around this bug. If I remove the ray.init() from your script so that Modin initializes ray for you, it works. Will that work for you?
About the performance, let's follow up on #5905https://github.com/modin-project/modin/issues/5905.
—
Reply to this email directly, view it on GitHubhttps://github.com/modin-project/modin/issues/5904#issuecomment-1490274119, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFO7AXVLPOMKWLGHKUQWWWDW6WARBANCNFSM6AAAAAAWM4RRJU.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

I'm happy to hear that the fix worked, @LudsteckJ.
Let's follow up in #5905 about the performance. Unfortunately Modin can be slower than pandas in some scenarios. Luckily for #5905 we have some promising work in progress that may help this scenario quite a lot.

It would be extremely helpful to have benchmarks fort he most important transformations like
groupby(), apply(), transform(), aggregate(), sum() etc.

Thank you for this feedback. We do have some benchmarks vs pandas here and documented here, but I don't see benchmarks for groupby's apply and transform. It would be good to add benchmarks for those as well.
I'll close this issue for now because we've found a fix for your bug.


Make InstructionView usable in custom activity

I would like to use the InstructionView (https://github.com/maplibre/maplibre-navigation-android/blob/main/libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/instruction/InstructionView.java)  in a custom Activity. I think its a quiet good component to start with when having a custom ui.
The problem is that it is tightly bound to the NavigationViewModel via the subscribe function. I think it would be a good idea to de-couple it, so it can be used in other view models.
The easiest way that comes to my mind would be having an interface which provides all the needed live data values to update the InstructionView. The subscribe function could now use the interface instead of the NavigationViewModel. This would allow any view model implementing the interface and then subscribing the the InstructionView.
Does anybody have an opinion on this? I would be willing to do a pull request on this.

The problem is that it is tightly bound to the NavigationViewModel via the subscribe function

You can use this also without the view model.
Add the view to your layout, and update the instruction by the route progress callback & milestone event callback.
    <com.mapbox.services.android.navigation.ui.v5.instruction.InstructionView
        android:id="@+id/mapInstruction"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_margin="8dp" />

// Activity / Fragment fields
private val progressChangeListener = ProgressChangeListener { location, routeProgress ->
    binding.mapInstruction.updateDistance(routeProgress)
}

private val milestoneEventListener = MilestoneEventListener { _, _, milestone ->
    binding.mapInstruction.updateInstructions(milestone)
}

// onCreated & mapReady:
MapboxNavigation(
            requireContext(),
            MapboxNavigationOptions.builder()
                .enableOffRouteDetection(true)
                .snapToRoute(true)
                .build()
        )
            .apply {
                addProgressChangeListener(progressChangeListener)
                addMilestoneEventListener(milestoneEventListener)
            }

The easiest way that comes to my mind would be having an interface which provides all the needed live data values to update the InstructionView. The subscribe function could now use the interface instead of the NavigationViewModel. This would allow any view model implementing the interface and then subscribing the the InstructionView.
Does anybody have an opinion on this? I would be willing to do a pull request on this.

Feel free to adapt and improve it anyway, if you don't like the current solution. If the above described solution is enough, a updated class description would maybe help.

Hug

Huge thanks for the quick reply!
I just gave it a quick try and it did work. I think subscribing to the view is a nice way, but its not worth introducing another interface if there is a simple way of updating the instruction view via the mentioned functions.
An example activity in the example app would've helped me. Maybe I'll find some time to add one and do a PR on it, but I'm quiet busy right now. I'll close this one for now. Thanks!


Minimize pom.xml complexity

https://stackoverflow.com/questions/75211691/populate-a-testcontainer-database-in-pom-xml

https://github.com/jabrena/spring-boot-relational-database-poc/pull/47


chore: DRAFT sample migration v10 test

Description of changes
do not merge, these are just samples for v10 migration testing

Issue #, if available

Description of how you validated changes
Checklist

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

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

Codecov Report

Merging #11605 (f5433d4) into cdkv2 (37f08a6) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            cdkv2   #11605   +/-   ##
=======================================
  Coverage   50.17%   50.18%
=======================================
  Files         702      702
  Lines       33958    33958
  Branches     6973     6973
=======================================
+ Hits        17040    17041    +1
+ Misses      15438    15437    -1
  Partials     1480     1480

Impacted Files
Coverage Δ

packages/amplify-cli/src/index.ts
66.10% <0.00%> (+0.84%)
:arrow_up:

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


Create mrepol742.is-a-fullstack.dev.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description

Link to Website

https://mrepol742.github.io

I have read the CLA Document and I hereby sign the CLA

recheck


I Need help with PermissionsEX

If players on my server break something it will say "You cant use (The block)"
I cant figure out. i have my ranks all set up:

There's probably a plugin blocking block breaks. Run /pex toggle debug and see what's being checked.


storage: Mark storage as dirty if indexing fails

@mattbostock This should at least flag the problem you have encountered (and it will lead to crashrecovery after the next restart, which should rebuild the inconsistent indices).

Thanks @beorn7, much appreciated.


Changing bcrypt-nodejs to bcrypt

Description of changes
Since the package bcrypt-nodejs has been deprecated, all occurences of the package have been replaced by bcrypt package.
Related issues (if any)
Testing

[ No UI changes were made ] List browser version(s) any admin UI changes were tested in:
[ Yes ] Please confirm you've added (or verified) test coverage for this change.
[ Yes ] Please confirm npm run test-all ran successfully.

@JedWatson Can you please review the PR

I shall add reviewers.


Handling of differences between Zod and Json Schema

Hi!
When looking through the json schema specs on emails and how zod handles them I realised there's a fundamental difference: while the json schema spec uses RFC 5321 for email and RFC 6531 for idn-email, zod uses a custom RegExp which it deems "reasonable".
This is where I wondered if matching zod.string().email() to "format": "email" actually makes sense, or if we should maybe use "pattern": "[regex from zod]" instead. Otherwise one could run into issues where an email is valid for the json schema, but invalid when using zod.
This issue extends to #40, where I'm currently adding RegExps for the aforementioned RFCs - which wouldn't be in line with how zod handles things and leads to enormous patterns being ejected.
I think this deserves a general answer, with options being

try to generate a json schema which best contains the information given to the zod schema (use "format": "idn-email")
generate a json schema which matches with the validations zod does (use "pattern": "[regex from zod]")

@StefanTerdell

Fair. Check #74

😍
Lovely! Didn't expect you to directly address this in code to be honest 🚀

Well, I agree with the problem description and it was a fairly quick fix. I'll see about getting it released in the coming days.

Fix released in 3.21.2


IndexError: list index out of range

I am currently experiencing IndexError: list index out of range  in GPS.py for Quectel Module: Quectel EG25-G.
Connecting Port..
Receiving GPS data

$GPGSA,A,2,08,10,18,23,24,27,32,,,,,,1.2,0.9,0.8*32
$GPGSV,2,1,08,08,33,295,20,10,68,004,28,18,27,120,24,21,09,309,18*77
$GPGSV,2,2,08,23,42,057,29,24,19,066,32,27,47,255,29,32,55,194,25*75
$GPGGA,180205.00,*removed*,N,*removed*,W,1,06,1.1,1426.6,M,-7.0,M,,*51
$GPVTG,163.5,T,148.1,M,0.0,N,0.0,K,A*2E
$GPRMC,180205.00,A,*removed*,N,*removed*,W,0.0,163.5,141021,15.5,E,A*14
-----Parsing GPRMC-----
Traceback (most recent call last):
  File "GPS.py", line 66, in <module>
    parseGPS(data)
  File "GPS.py", line 28, in parseGPS
    degreeChecksum = sdata[13] #Checksum
IndexError: list index out of range

When I print out sdata: ['$GPRMC', '180205.00', 'A', '*removed*', 'N', '*removed*', 'W', '0.0', '163.5', '141021', '15.5', 'E', 'A*14\r\n']

Please change line 28 by
degreeChecksum = sdata[12]
and try again.


fix add manual new variable in FS

Proposed Changes

Fix problem when add new variable in manual mode on Feature Set.

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests
[ ] Build locally
[ ] Code format / lint
[ ] Docs have been reviewed and added

Pull request type
Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A

Fixes #352
What is the new behavior?

Fix problem when add new variable in manual mode on Feature Set.

Does this introduce a breaking change?

[ ] Yes
[x] No

Codecov Report

Merging #354 (932d275) into master (f0db551) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master    #354      +/-   ##
=========================================
- Coverage    3.35%   3.34%   -0.02%
=========================================
  Files          32      32
  Lines        1132    1136       +4
  Branches      112     113       +1
=========================================
  Hits           38      38
- Misses       1094    1098       +4

Impacted Files
Coverage Δ

...w-variable-dialog/new-variable-dialog.component.ts
3.63% <0.00%> (-0.29%)
:arrow_down:

Continue to review full report at Codecov.

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


45 in page navigation

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #45
Test URLs:

Before: https://main--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/laura/inpage-navigation
After: https://45-in-page-navigation--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/laura/inpage-navigation

One issue with alignment lies in the header. An <li> is rendered, with a hidden button. Because of display: flex; we have this gap. Something we can fix easily by hiding the list item, right?

fixed


Algorithm to Check if the graph is Bipartite(bicolourable)

Detailed description
Bipartite graph has various application in modern coding theory.The implementation of algorithm to check if the graph is Bipartite is crucial.
https://en.m.wikipedia.org/wiki/Bipartite_graph
Context
This feature is a base for many other application oriented algorithms to be implementation.
Possible implementation
The graph is categorized to be bipartite if it doesn't contain odd nodes cycle. So by looking for odd node cycle the graph could be categorized to Bipartite or not.
Additional information
No response

It's already present, right? is_graph_bipartite.cpp

It's already present, right? is_graph_bipartite.cpp

nice catch, will close this as resolved!


[1.9.1] better exception message

I had a pice of Builder code like  this:
public class Build extends BaseProject {
    ...
    public Build() {
        r//epositories = List.of(MAVEN_LOCAL, MAVEN_CENTRAL);
        ..
    }

It turns out that I had to explicitly configure the repositories, despite that I had this in my bld-wrpapper.properties:
bld.repositories=MAVEN_LOCAL,MAVEN_CENTRAL

Is this my mistake, assuming that the config repos will flow in my Build object? What happens  when I do not set the repositories properly, is shown below. However, it would have been much better if the error also spelled out the repos it was given - I would have notice immediately that the repositories config was empty.
So, please improve the eror message to include the repositories list.
hristo:~$./bld -s helidon-download
rife.bld.dependencies.exceptions.ArtifactNotFoundException: Couldn't find artifact for dependency 'io.helidon.http:helidon-http:4.0.8' at
        at rife.bld.dependencies.DependencyResolver.getMavenPom(DependencyResolver.java:417)
        at rife.bld.dependencies.DependencyResolver.getAllDependencies(DependencyResolver.java:126)
        at rife.bld.dependencies.DependencyScopes.resolveScopedDependencies(DependencyScopes.java:147)
        at rife.bld.dependencies.DependencyScopes.resolveCompileDependencies(DependencyScopes.java:75)
        at rife.bld.operations.DownloadOperation.executeDownloadCompileDependencies(DownloadOperation.java:60)
        at rife.bld.operations.DownloadOperation.execute(DownloadOperation.java:44)
        at rife.bld.operations.AbstractOperation.executeOnce(AbstractOperation.java:75)
        at rife.bld.BaseProject.download(BaseProject.java:499)
        at peruncs.shared.Build.downloadHelidon(Build.java:81)
        at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
        at java.base/java.lang.reflect.Method.invoke(Method.java:580)
        at rife.bld.CommandAnnotated.execute(CommandAnnotated.java:27)
        at rife.bld.BuildExecutor.executeCommand(BuildExecutor.java:454)
        at rife.bld.BuildExecutor.execute(BuildExecutor.java:230)
        at rife.bld.BaseProject.execute(BaseProject.java:1679)
        at rife.bld.BuildExecutor.start(BuildExecutor.java:274)
        at peruncs.shared.Build.main(Build.java:19)

The repositories defined in the wrapper's properties are for bld own use (to download extensions, etc.)
The repositories needed by your projects should always be defined in the build file.

That's why the bld dependencies are placed in the lib/bld directory, and not shared with the project.
How to set up dependencies for your project is clearly explained in the documentation.
The wrapper's properties file usage is explained here.
Contrary to Maven or Gradle, bld does not use project configuration files, everything is done in the build file.
Hope this helps.

The locations were already listed in the exception message, however if they were empty they were just blank. I improved the message to take that into account.


Release Google.Cloud.Firestore.Admin.V1 version 3.4.0

Changes in this release:
New features

Expose Firestore PITR fields in Database to stable (commit 164e151)
Expose Firestore snapshot_time field in export API to stable (commit 164e151)
Expose Firestore namespace ID fields in import/export APIs to stable (commit 164e151)

Documentation improvements

Assorted typo fixes and whitespace updates (commit 164e151)

Created release for Google.Cloud.Firestore.Admin.V1-3.4.0


Extra config yaml request

Hello, thank you for your wonderful work!
In order to reproduce the results in your paper, I wonder if you will provide the config of three data sets in the near future

We're glad you like our work! If you look under the sweeps folder you should be able to reproduce the results for the 3 datasets :)
Let us know if you have any more questions!

Thank you for your prompt reply! I will close this issue

Excuse me, in order to get the results of the application in the paper, I guess it is necessary to first download Pictures in official website, including Lamen and Imagenet v1v2 (if correct, Imagenet v1 should be downloaded at https://www.image-net.org/download.php. And where v2 can I download the original picture?)
As for clip_vs_resnet application, I would like to ask where is the imagenet-v2.csv file and whether the results of the paper can be obtained by just running main.py file?


Protected call crashes on protected function result destructor

Not sure if this is happening because of something I'm doing, but in some rare instances I'm getting an access violation exception inside the destructor of soll:protected_function_result, after calling a sol::protected_function like so:
sol::protected_function_result result = someFunction(...);

if (!result.valid())
{
	std::string error = s_Result;
	std::cout << error << std::endl;
}

It should be noted that the corresponding lua function does not return a value.

std::string is not the error type. In debug mode that code will throw an error. Use sol::error to inspect the error instead.
Do you have any other information about this? Is this a statically held function? A statically held result value? Do you have a stack trace? Does this happen at application runtime, or at exit?  Also, s_Result is not the same as result. Asides from that, no repro:
int main(int, char*[]) {
	sol::state lua;
	lua.script("function f () if (a == 1) then error('fug') end end");
	sol::protected_function some_function = lua["f"];

	std::size_t correct_count = 0;
	std::size_t error_count = 0;
	for (std::size_t i = 0; i < 1000000; ++i) {
		lua["a"] = i % 2;
		sol::protected_function_result result = some_function();
		if (!result.valid()) {
			if (i < 2) {
				sol::error error = result;
				std::cout << "an error occured (as expected)" << error.what() << std::endl;
			}
			++error_count;
		}
		else {
			if (i < 2) {
				std::cout << "all good :+1:" << std::endl;
			}
			++correct_count;
		}
	}

	std::cout << "correct_count: " << correct_count << std::endl;
	std::cout << "error_count  : " << error_count << std::endl;

	return 0;
}

Sorry, s_Result is a typo, it was supposed to be result. I changed my code to use sol::error instead, but I'm still encountering this. It seems to trigger a few random errors, in no particular order, some with empty "what" causes and some with "call", and then crashes at the same point.
I should note that this is under x86.
Is there a possibility that using sol across module and thread boundaries (though in a synchronized manner) could be causing this behavior?

I'm using x64.
If you're throwing exceptions and expecting those to be caught by sol2 and propagated properly, you're treading dangerous, swampy lands (especially in x86). This sounds like an exception is crossing boundaries, you're trying to catch it and then continue quietly, and it's blowing up in your face. What Lua version are you using, are you on LuaJIT? These are all important questions to answer.
I don't know about module implementations, those were supposed to still be experimental.
sol2 provides no threading safety; neither does Lua. People have used sol2 in threaded environments before with synchronized access and everything worked out fine. If you're having problems with coroutines in particular, see the threading page for information about it: http://sol2.readthedocs.io/en/latest/threading.html#working-with-multiple-lua-threads
This is the most I can do for you, since I have no other information for you.

The problem is, I don't think there's any exceptions being thrown inside the lua code. The lua function that I'm calling (someFunction) is empty.

I ran the above code on x86, nothing happened either. I removed the function body and left it empty, nothing happened on x86 or x64. What Lua version, what platform, is this function called inside of a DLL, from multiple threads at the same time? Multiple calls into a single Lua state is an absolutely bad idea because the Lua stack is essentially a shared global variable: using sol2 types or Lua C API functions from multiple threads without locking every operation around the ENTIRE state will result in you screwing up the stack with data races or -- worse -- things working just in time and leaving the expectations for Lua's stack subtly changed.
If you have multiple threads that need to share a Lua state, you need to lock every single operation for the entire state, including interacting with the results of function calls (because they are, like everythnig else, based on the stack).

Lua 5.3.4, MSVC140, x86 target using the x64 toolset.
I do call these functions multiple times from multiple threads and from within a DLL, but all calls are protected by a mutex (CRITICAL_SECTION in this case, I'll try using std::mutex to see if it makes a difference). I noticed that I had an empty if statement in one of those functions. Removing that if statement just makes my app freeze but I haven't had much time to debug that yet.

You can't just protect the function call. You need to protect the entire block:
std::lock_guard guard(my_mutex);
{
     sol::protected_function_result result = someFunction();
     if (!result.valid()) {
          sol::error err = result;
          std::cout << "error occurred: " << err.what() << std::endl;
     }
}

If you protect just the call, then access using protected_function_result and its destrutor will not be synchronized (the stack is accessed in the object's destructor).

I have protected the entire block. Anything calling into or out of lua is inside a function that can only be exclusively accessed.

I still have not been able to repro this, and I can't get it to die on x86, x64 with various kinds of functions and setups. I haven't tried putting the code in a DLL and tossing things.

I've started to believe that this is indeed a concurrency issue. It seems that there's an edge case where the execution path calls into lua without proper synchronization. Currently working on figuring out what that code path is. Will update if I have any updates.

Let me know if anything else goes awry. 🤷♂


Very low coverage

Hi. I  tested snap7 and UDP-based protocols with AFLNET, and both showed very low coverage(map density), staying at 0.02%. Can you provide solutions? Thank you.

Hi. Snap7 seems to be a very interesting target. Unfortunately, I haven't tested it, and it is very unlikely that I have time to do that, so I cannot say your results are normal or not.
Maybe you can try to make sure that:

You captured good seeds
You managed to disable things that introduce randomnesses in the codebase (eg., random session id, nonce). To check that, you can use aflnet-replay to replay the first test in the replayable-queue folder, which should be the replayable version of your first seed, and see if aflnet-replay successfully reach your expected states.

I hope it helps.

Thank you very much.


newsbeuter - ncurses 6.0

after last ncurses Update 6.0 newsbeuter can't start.
newsbeuter: error while loading shared libraries: libncursesw.so.5: cannot open shared object file: No such file or directory

duplicate of #198

yes sure, i thinek, with Arch/Manjaro we need a new git-Version in the packages

Arch already has a patch for the issue

yes :+1:  Manjaro 1 hour later :-)


Ignore specific folders code analysis (or at least report errors from them).

I think this was discussed in a different issue https://github.com/microsoft/vscode/issues/78291
But I have a project that has different libraries, each with their own coding style.. They do not have .editorconfig files associated with them (nor does multiple .editorconfig files even work with VSCode currently). It'd be nice to ignore errors (without seeing the badge count either) from these folders so I don't have to see 100+ errors, and none of them from my project. :)
Edit: My project is a C# unity project that has a few other open source projects inside the folders.

Please file this as a feature request against the c# extension


make ch trip request

Make a trip request in the SBB region

Estimated Story points: 6

Story Points: 7


How to I financially support this project?

Not sure if the maintainers needs it, but just want to show my apperciation, even without documentation, the examples are comprehensive, the library is high quality.
Would love to setup some small amount of recurring donation.

cc @dominicstop

hello, thank you for the kind words; i recently made a gh sponsors page: https://github.com/sponsors/dominicstop
since i’ve only created it yesterday, i won’t be able to receive a payout until october (90 days from now)
until then, i appreciate any donations ppl can spare my way


fix: [nuxt3] consola as a plugin

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Related #7375

Did your issue had any of the "$" label on it?

[ ] My DOT address: Payout

Screenshot 📸

[ ] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at b5af1fd
Removed unnecessary plugins and refactored consola plugin in nuxt.config.js. This improved the web app's performance and followed the best practices for Nuxt 3.

🤖 Generated by Copilot at b5af1fd

Plugins removed
nuxt.config.js changed
Winter of refactor

closes #5314


add defaults to the available global params

What is the purpose of this change? What does it change?
add defaults to the global preferences and clearer debug log
Was the change discussed in an issue?
fixes #1936

How to test changes?
odo preference --help
Modifies odo specific configuration settings within the global preference file.

Available Parameters:
NamePrefix - Use this value to set a default name prefix (Default: current directory name)
Timeout - Timeout (in seconds) for OpenShift server connection check (Default: 1)
UpdateNotification - Flag to control if an update notification is shown or not (Default: true)

Make sense, Thanks for the pr
/approve

CI is green
/lgtm


Fix environment variables for controller create

Per https://docs.microsoft.com/en-us/azure/azure-arc/data/upload-metrics-and-logs-to-azure-monitor?pivots=client-operating-system-powershell#create-service-principal

Pull Request Test Coverage Report for Build 409311015

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 41.422%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
73.81%

Totals

Change from base Build 409252993:
-0.01%

Covered Lines:
22065

Relevant Lines:
48860

💛  - Coveralls


utilities: add reaction bookmark command

This commit includes a custom command to send a small bookmark to the
user's DM upon reacting to a specified message.
It is an almost direct copy from my own, old and now private repository,
as I intend to move the custom commands hosted there to here.
Signed-off-by: Luca Zeuch<EMAIL_ADDRESS>Status

[x] Code changes have been tested on an instance of YAGPDB, or there are no code changes
[x] I have read and followed the contribution guide
[x] I have written documentation for my changes, or there is no need to

See it in action:

Updated the docs to use the template now.

There we go, magic :D


Add Cursor Tail Effector done ! #289

Hello, PR
Add new Feature - #289
Done -
https://github.com/user-attachments/assets/60729ef6-c134-4602-9a27-bfa4b9dcf411
✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[x] I have followed all the guidlines mentioned in CONTRIBUTING.md.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation (if applicable).
[x] I have tested it locally and it works fine.
[x] Any dependent changes have been merged and published in downstream modules.

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

[ ] Bug fix (non-breaking change which fixes an issue) 🐛
[x] New feature (non-breaking change which adds functionality) ✨
[ ] UI enhancement (non-breaking change which enhances UI) 🎨
[ ] Documentation update 📚

Keep the styling code into the style.css file all the code for the index.html is there!! Also leave a comment for which this styling is added

Ok wait I will give new PR

okky. but is there any issue in my solution , you didn't merge it.. could you please tell me the reason??


NIFI-5384 FlowFile's queued in batches should all have the same Queue…

… time
While building some batching unit tests I started making the assumption that FlowFile's sent to session.Transfer in batches should all have the same Queued time. Only to find out that wasn't true.
Updating batch queuing of FlowFile's so that FlowFile's transferred together all have the same queue time.
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Session_Test.xml.txt
Test flow I used.


can't modify a mutable document get from db

When I am using the C  API, I try to get a MutableDocument I previously saved from the database, modify it and save it back to db. However, When I try to modify the MutableDocument, it "terminate called after throwing an instance of 'fleece::FleeceException'
what():  incorrect use of persistent shared keys: not in transaction
Aborted (core dumped)".

I haved the same problem in my project https://github.com/jeromebenois/couchbase-lite-c-rs
I resolved this one in calling CBLDatabase_BeginBatch and CBLDatabase_EndBatch in order to encapsulate update in transaction. cf. https://github.com/jeromebenois/couchbase-lite-c-rs/blob/811ce7770e471e7d6f4c8093c0266f3696bb684e/couchbase-lite-c/src/database.rs#L102

@jeromebenois I think officials should fix this, not letting us users to resolve this ourselves.

Yes, this is definitely a bug; sorry about that. Also reported here with reproducible case:
cbl::Database db(“information-state”, {"/tmp", kCBLDatabase_Create});
cbl::MutableDocument doc = db.getMutableDocument(“foo”); // document “foo” is previously saved
doc.set(“new”,10); // crash here
db.saveDocument(doc);

Nasty design issue in the mutable-Fleece layer (HeapDict class) when used with persistent shared keys. HeapDict adds a new key to the SharedKeys as soon as it's added, but the PersistentSharedKeys can only be modified in a transaction, to ensure it's assigned a unique numeric value.
Having to use a transaction to modify a document sucks, so this needs to be fixed. HeapDict should leave the key as a string until writing to an Encoder. Is that feasible?