Blurry image quality from cameras

The images from the cameras seems anomaly blurry:

We may need to check the configuration here about exposure and gain:
https://github.com/sbrodeur/ros-icreate-bbb/blob/master/scripts/ros.sh#L19
The cameras also have auto-focus capability, but this is not configured. To disable autofocus:
v4l2-ctl --device=/dev/video6 --set-ctrl focus_auto=0
v4l2-ctl --device=/dev/video7 --set-ctrl focus_auto=0

To set the focus to a fixed value:
v4l2-ctl --device=/dev/video6 --set-ctrl focus_absolute=30
v4l2-ctl --device=/dev/video7 --set-ctrl focus_absolute=30

We may need to play with these options to see if there is any effect on the image quality. The exposure was set to a high value to avoid having blurry images during movement. A high gain may also cause more noise in the images. The quality can also depend on the amount of lighting in the room, which is not particularly great for the image above.

Commands dont seem to be recognized due to the focus_auto and focus_absolute keywords.
Also, the gain and exposure commands, while work in command line seem to have no effect when changed in the ros.sh file

The ros.sh file is called when the ros service is started, so it requires to run the following command for changes to take effect:
service ros restart

Was able to set focus, exposure and manual mode in startup script (ros.sh), but the gain parameter is overwritten somewhere in the code. I suspect capturev4l2.cpp. Will continue to investigate.

We should be able to set exposure from the capture node:
http://stackoverflow.com/questions/26084794/v4l2-help-in-camera-settings

Here is some code example to integrate here:
https://github.com/sbrodeur/ros-icreate-bbb/blob/master/src/camera/src/capturev4l2.cpp#L158

        // Manual exposure control
        // See: https://linuxtv.org/downloads/v4l-dvb-apis/extended-controls.html
        struct v4l2_control stream_control_exposure;
        stream_control_exposure.id = V4L2_CID_EXPOSURE_AUTO;
        stream_control_exposure.value = V4L2_EXPOSURE_MANUAL;
        if(xioctl(_fd, VIDIOC_S_CTRL, &stream_control_exposure) != 0)
        {
        	perror("Couldn't set camera exposure to manual");
        	return 1;
        }

        // Manual exposure absolute value
        struct v4l2_control stream_control_exposure_value;
        stream_control_exposure_value.id = V4L2_CID_EXPOSURE_ABSOLUTE;
        stream_control_exposure_value.value = _exposure;
        if(xioctl(_fd, VIDIOC_S_CTRL, &stream_control_exposure_value) != 0)
        {
        	perror("Couldn't set camera exposure absolute value");
        	return 1;
        }

        // Manual focus
		struct v4l2_control stream_control_focus;
        stream_control_focus.id = V4L2_CID_FOCUS_AUTO;
        stream_control_focus.value = false;
		if(xioctl(_fd, VIDIOC_S_CTRL, &stream_control_focus) != 0)
		{
			perror("Couldn't set autofocus to manual");
			return 1;
		}

        // Manual focus absolute value
		struct v4l2_control stream_control_focus_value;
		stream_control_focus_value.id = V4L2_CID_FOCUS_ABSOLUTE;
		stream_control_focus_value.value = _focus;
		if(xioctl(_fd, VIDIOC_S_CTRL, &stream_control_focus_value) != 0)
		{
			perror("Couldn't set camera focus absolute value");
			return 1;
		}

See commit 92ffa7d for the integration (as above) of exposure and focus control. It needs to be tested on the robot.

I'm not sure if we can flip the image by the camera, since it doesn't seem supported:
root@beaglebone:~# v4l2-ctl --device=/dev/video6 --set-ctrl horizontal_flip=1
unknown control 'horizontal_flip'

So far, gain adjustment isn't working. The culprit seems to be in initMmap(), when we start the capture. If we try to change setting after that it gives errors , and if done before they seem to be reset. So far the best solution seems to be adding a script that runs 30 sec after the end of ros.sh  that will run the settings command line.

See relevant discussion here:
http://www.linuxquestions.org/questions/linux-software-2/camera-settings-in-linux-v4l2-or-uvc-drivers-4175525260/
"I finally solved the issue. The thing is that I had to change the Exposure and the Gain settings after I started the photo capturing not before (all other settings are to be set before). In addition Gain can only be set when Exposure is set to manual."

It is possible to create a cron job to periodically (e.g. every minute) set the camera configuration:
* * * * * /root/work/ros-icreate-bbb/scripts/ros-cron.sh >> /root/work/ros-icreate-bbb/scripts/ros-cron.log 2>&1

This will currently execute every minute automatically on the robot.
Compared to the previous camera image, now the output is much more clear and still not blurry during movement:

The cameras seem also to have similar (albeit not perfectly equal) color balance output under the same configuration (e.g. check the wall color).

See commit 10e4455 for the cron job implementation.


Make pSome cancelable and cancel pending requests

Also version locked xo to <=0.21.0 to ensure compatibility with node 4.

Implements https://github.com/sindresorhus/p-any/issues/3
Would help with https://github.com/sindresorhus/is-online/issues/40
Would help with https://github.com/sindresorhus/is-online/issues/47#issuecomment-370578237 (via pTimeout's cancel() propagating through pAny)

bump @sindresorhus

Nice work. Looks good :)


chore(logging): logging struct and level improvisations.

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


Update export to use latest iceberg

Right now we are using a fixed version (tag pre-multi-remotes).
We should move to the newest version as soon as it is freezed.

I think that we can use the current version and for export use the class IceMetadatalessFileTreeWriter


[Spark][Integration] column level lineage

Signed-off-by: Pawel Leszczynski<EMAIL_ADDRESS>Problem
Turn on the introduced column level lineage mechanism and include it within the integration tests.
Part of: #673
Solution

Modify existing visitors and dataset builders to include column level lineage when applicable.
Modify existing integration tests to verify if column level lineage is included.

Note: All schema changes require discussion. Please link the issue for context.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
Checklist

[X] You've signed-off your work
[X] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)

@pawel-big-lebowski thanks for the effort! do you have a target date for this feature to release?

@xiaoyongzhu Nope but this should not take long.


Make sure kill on a NL triggers future callbacks

Otherwise the parent context might not get cleaned up correctly

ack


useToast (Notification) is not working with Nuxt 3.12.1

Environment

Operating System: Windows_NT
Node Version:     v20.5.1
Nuxt Version:     3.12.1
CLI Version:      3.12.0
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      ssr, devtools, app, css, modules, colorMode, i18n, components, vite, runtimeConfig
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Version
2.16.0
Reproduction
https://stackblitz.com/edit/nuxt-ui-jpwbrh?file=app.vue
Description
I'm build a SPA Nuxt App (ssr:false), Nuxt version was 3.10.0 and it was working fine. After upgrading to Nuxt 3.12.1, notifications component doesn't appear. No errors or warnings.
Additional context
No response
Logs
No response

Duplicate #1867 and #1871. Answered in https://github.com/nuxt/ui/issues/1867#issuecomment-2165516400.


auth_ldap can't read variables

Hi
then i try to enable or disable the ldap_auth via rule.
i have created a map
geo $realm {
default "restricted area";
<IP_ADDRESS>/24 off;
}
auth_ldap $realm;
auth_ldap_servers ldap_srv;
But what i see is the variable $realm as text on login mask.
Then i try this via auth_basic it will work... but i need this via ldap

Anything on this? I have the same issue...

hell, wanted to see if you ever solved this problem ? i'm experiencing the same thing

Hi, as far as I remember I didn't get this working.
Op vrijdag 1 juli 2016 heeft Alexei Smirnov<EMAIL_ADDRESS>het
volgende geschreven:

hell, wanted to see if you ever solved this problem ? i'm experiencing the
same thing
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kvspb/nginx-auth-ldap/issues/77#issuecomment-229978094,
or mute the thread
https://github.com/notifications/unsubscribe/AAHhkCZbxqPUQew0vlCzDU4lDAkcstu7ks5qRTOkgaJpZM4D8U6k
.

--
Dimitry Declercq
<EMAIL_ADDRESS>+32 486 90 98 24

Trying to use an nginx map as well for variable names but looks like this module doesn't honor them. Would love to see this.

All,
Anyone ever find a solution to this?
Would be nice to be able to exclude LDAP authentication from some of our internal / safe networks...
Thanks


How to execute a specified function in a wasm file using cli

Run the command ./bin/wazero run --help, however, there aren't any args for executing a specified function.

we currently don't support arbitrary function invocation. Can you mention an example of what you are trying to invoke? Most of the time we find people are invoking toy applications like calculators, so it will help to know a more realistic use of this feature.

Hey
https://github.com/bots-garden/capsule can do what you want @orangeC23
it’s one of the many projects using Wazero ..


Parse Substrate's Metadata V14

Description
Metadata parsing impl and type-conversion from network json schema to Types Model.
Tasks:

[x] Substrate Metadata definitions: (base and v9 - v14)
[x] Xcm Encode / Decode
[x] Test Cases for verifying chain data
[x] Chain specific schemas definitions
[x] Testing Old Type Registry and Type Exp
[ ] More Test cases to reach code coverage of minimum 99%

Acceptance Criteria:

Minimum 99% Code Coverage Required.
Parsing of all chain blocks and events without any hiccups

Blocked due to test cases for substrate's metadata.


Add example and context to Velociraptor blog post

This PR adds an example to the Velociraptor blog post and adds some context to the description of the "subscribe" use case.
Additionally it removes references to wizardry (🧙).

I pushed a commit with minor linting and touch ups to your branch in order to authorize CI.

Closing this one due to missing CI rights; commits cherry-picked and continued in in #3574.


Flutter web is stuck at the initialization loading

I have provided a loading animation for my flutter web application so that user will not think that the web application is frozen. The problem is when I open my web application for the first time and there is no cache or anything similar on my pc, it displays the loading animation and the animation will stay there forever. Even when there is no http request on flight. Please notice that this problem does not always occur.
Here is my flutter doctor output:
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.0, on macOS 12.1 21C52 darwin-x64, locale en-US)                                                                       Checking Android licenses is taking an unexpectedly long time...[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

• No issues found!

This is my index.html file:
<!DOCTYPE html>

  <base href="$FLUTTER_BASE_HREF">

  <meta charset="UTF-8">
  <meta content="IE=Edge" http-equiv="X-UA-Compatible">
  <meta name="description" content="A new Flutter project.">

  <meta name="apple-mobile-web-app-capable" content="yes">
  <meta name="apple-mobile-web-app-status-bar-style" content="black">
  <meta name="apple-mobile-web-app-title" content="weight_and_balance">
  <link rel="apple-touch-icon" href="icons/Icon-192.png">

  <link rel="icon" type="image/png" href="favicon.png"/>

  <title>weight_and_balance</title>
  <link rel="manifest" href="manifest.json">

remove this line

from index file
its work for me

remove this line
from index file its work for me

What do you mean by "this line"? Can you be more specific?

@armin1440 I tried running the project with your index.html file several times, but can not reproduce this issue.
Which host platform are you running the web on? Please provide the output of flutter doctor -v as well. Thanks!
Also, you may try removing the loading same as in this gallery index.html example to see if it works.

@armin1440 I tried running the project with your index.html file several times, but can not reproduce this issue.
Which host platform are you running the web on? Please provide the output of flutter doctor -v as well. Thanks!
Also, you may try removing the loading same as in this gallery index.html example to see if it works.

Please try it with poor internet connection and with no cached data. It takes about couple of minutes for me to download needed data at the beginning. As I said earlier, It occasionally happens.
This is a screenshot of my browser. After a minute, no progress is made in http requests because they are all done and there is no request on flight.

@armin1440 Can you confirm you are testing on a new Flutter project and only modifying the index.html file as above? That would be appreciated if you could provide a recorded video for this.
Can you also help to check this when using html or canvaskit renderer? It seems that it may be faster if you use html renderer (https://github.com/flutter/flutter/issues/76009#issuecomment-781681343)
May be related/similar to https://github.com/flutter/flutter/issues/76009. Please take a look at this issue and comments to see if it's similar to your case.

@armin1440 Can you confirm you are testing on a new Flutter project and only modifying the index.html file as above? That would be appreciated if you could provide a recorded video for this.
Can you also help to check this when using html or canvaskit renderer? It seems that it may be faster if you use html renderer (#76009 (comment))
May be related/similar to #76009. Please take a look at this issue and comments to see if it's similar to your case.

Yes, I confirm that.
I tried html and canvaskist. The problem occurs when I use canvaskit. The #76009 is similar to my issue but here I am stuck at the loading and I'm emphasizing on that not the start-up time.
Here's a video of openning my website https://drive.google.com/file/d/1RZPayDfngrBkURjoTNS_iTCPTfE07A6b/view?usp=share_link

Thanks for confirming. Can you check if the issue occurs with either the local environment or deployed site? Also, can you try applying the workaround at https://github.com/flutter/flutter/issues/76009#issuecomment-1412708684 to see if the web is still stuck?

Thanks for confirming. Can you check if the issue occurs with either the local environment or deployed site? Also, can you try applying the workaround at #76009 (comment) to see if the web is still stuck?

It only happens in the deployed site. I tried the workaround and it worked. It's surprising because it apparently only speeds up the start up time and does not change anything else.
Anyway shouldn't it be automatically added to the index.html file?

Since this issue is caused by canvaskit when starting on empty browser cache. Also, the workaround is applicable to your case. I would suggest you follow up and leave a comment (your case and your question at https://github.com/flutter/flutter/issues/120347#issuecomment-1427475248) at https://github.com/flutter/flutter/issues/76009 about this.
That issue has been brought up recently with many comments and investigations by the team (see https://github.com/flutter/flutter/issues/76009#issuecomment-1421578481).
Closing this issue as a duplicate. Please follow up on https://github.com/flutter/flutter/issues/76009 for further updates. Once it's fixed but you still have this issue, please write a comment and I will reopen this issue for further discussion. Thanks!

remove this line
from index file its work for me

What do you mean by "this line"? Can you be more specific?
sorry :) this line
<base href="$FLUTTER_BASE_HREF">


Add complex data types

Updating array-api-tests fails because the newer test suite crashes when looking for complex types. This PR adds complex types even though they appear to be totally broken with respect to support in the onnx package itself. I'm sure there is a better way to deal with this situation and I'm happy to hear your opinions!
Related to #29

If complex dtypes are broken in ONNX, I think the test suite shouldn't compel you to add them. There is https://github.com/data-apis/array-api-tests?tab=readme-ov-file#skipping-dtypes, which I think should work for skipping complex dtypes. If not, maybe that requires an upstream fix.

Thanks for dropping in @rgommers ! Unfortunately, the array-api-tests call ndx.asarray(complex()).dtype in the collection phase regardless whether complex types are meant to be skipped which causes a crash (see this branch).

Thanks for this. A few points:

Your test adding two complex numbers together fails due to ONNX shape inference failing when trying to infer the types for the cast operator:
E           RuntimeError: [ShapeInferenceError] (op_type:Cast, node name: _this_): input typestr: T1, has unsupported type: tensor(complex128)

.pixi/envs/default/lib/python3.12/site-packages/onnx/shape_inference.py:46: RuntimeError

As noted in the error, it is clear that the ONNX standard does not support casting of complex numbers - see https://github.com/onnx/onnx/blob/main/docs/Operators.md#type-constraints-21.
The reason any casting is occurring in the first place is because the add implementation for core types ( complex64/complex128 is one since you subclass Floating) first performs type promotion. This is something that will be resolved as part of https://github.com/Quantco/ndonnx/pull/17 systematically. You shouldn't need to support casting to implement addition.

The CoreOperationsImpl that contains implementations of functionality for the core dtypes exposed by ndonnx may not be ready for complex numbers. If we want to add functionality bit by bit, it is worth adding a new implementation block for complex numbers as support in ONNX seems sparse. We will get better error messages. For instance not implementing add gives:

TypeError: Unsupported operand type(s) for add: 'Complex128' and 'Complex128'

This is more readable than a long ONNX stack trace.

I'll close this PR given:

Complex types are not particularly useful given the current state of the ONNX standard
It seems like we are well able to run the array-api-tests without them (#37 )


How to fix "Stack offset of 4848 exceeded max offset" after building

Error: Function ZN161$LT$raydium_cp_swap..instructions..initialize..Initialize$u20$as$u20$anchor_lang..Accounts$LT$raydium_cp_swap..instructions..initialize..InitializeBumps$GT$$GT$12try_accounts17h1a5660f3939c4019E Stack offset of 4848 exceeded max offset of 4096 by 752 bytes, please minimize large stack variables

Same issue here.

If you update dependecy for this commit:
raydium-cp-swap = { git = "https://github.com/raydium-io/raydium-cp-swap", rev = "2562fa8d171ecbdbcaff8503d00fa16ca3c74d26", features = [...] }
Anchor should compile the code, however it will be problematic to use on-chain. Probably adding observation_state to Initialize had overflowed stack size.
https://github.com/raydium-io/raydium-cp-swap/commit/2562fa8d171ecbdbcaff8503d00fa16ca3c74d26

use anchor_spl::associated_token::{create as create_ata, Create as CreateAta};
...
/// creator lp token account
    #[account(
        // init,
        associated_token::mint = lp_mint,
        associated_token::authority = creator,
        // payer = creator,
        token::token_program = token_program,
    )]
    pub creator_lp_token: Box<InterfaceAccount<'info, TokenAccount>>,
 ...

create_ata(CpiContext::new(
        ctx.accounts.token_program.to_account_info(),
        CreateAta {
            associated_token: ctx.accounts.creator_lp_token.to_account_info(),
            authority: ctx.accounts.creator.to_account_info(),
            mint: ctx.accounts.lp_mint.to_account_info(),
            payer: ctx.accounts.creator.to_account_info(),
            system_program: ctx.accounts.system_program.to_account_info(),
            token_program: ctx.accounts.token_program.to_account_info(),
        },
    ))?;

Initializing the lp_token_account inside the program instead of using init fixes this

It's not a problem when you actually invoke the instruction when running test.

Error: Function ZN161$LT$raydium_cp_swap..instructions..initialize..Initialize$u20$as$u20$anchor_lang..Accounts$LT$raydium_cp_swap..instructions..initialize..InitializeBumps$GT$$GT$12try_accounts17h1a5660f3939c4019E Stack offset of 4848 exceeded max offset of 4096 by 752 bytes, please minimize large stack variables

Currently, we recommend depend on versions anchor 0.29 and below solana 1.18. There are optimizations for related issues in anchor 0.31 https://github.com/coral-xyz/anchor/pull/2939
After anchor 0.31 is officially released, we will upgrade the dependencies. Thank you.


Race condition in Microstream example.

Race condition was causing requests against uninitialized Microstream root.

So this now works because everything is correctly initialized before the constructor returns?

yes


cluster_centers not updating

Hi vlukiyanov,
Thank you for your pytorch implement. It has been a big help. However, I noticed that in this version, the cluster_centers are not updating during training.
This is the code I used to test:
        a = list(model.parameters())[-2].clone()  # hidden layer
        b = list(model.parameters())[-1].clone()  # cluster centers
        optimizer.zero_grad()
        loss.backward()
        optimizer.step(closure=None)
        print (torch.equal(a, list(model.parameters())[-2]), torch.equal(b, list(model.parameters())[-1]))

It returns   False, True
I am very new to pytorch, but I think this might be a bug from pytorch. I checked there's gradient
for cluster centers but somehow the parameters are not updating.
Maybe there's something I don't understand about pytorch, but I just wanted to point this out to you.

You're quite right (you could also get it out using model.state_dict()['assignment.cluster_centers'] I think); must be another issue when I converted everything to 1.0.0 from 0.4.0.
I think the cluster centres don't contribute as much to the gradient in practice, so hopefully that explains why the accuracy was still high.
I've pushed the commit above to a branch, but need to test on some real data and make sure its still reproduces the accuracy.
Thanks for spotting.

Ok merged that branch in should be fixed, let me know how it works for you and reopen if it doesn't or you think there's an error.
Thanks again for spotting.

Hi vlukiyanov
Thanks for the prompt response and resolving the issue so quickly! The parameters are updating now!
Thanks again for the implementation :)


By analyzing the blame information on this pull request, we identified @skv-headless and @trave7er to be potential reviewers.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


can't install this project,hear are this error

There is most likely something with your npm installation.

Closing as no updates for two weeks.


Expand & update sections for command line usage

👋 Hello there! Welcome. Please follow the steps below to tell us about your contribution.

Please complete a Checklist
Fill in all sections of the template
Click "Create pull request"

Checklist
<-- Please fill the boxes with [x] before submitting a pull request -->

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix

Bring Cardano CLI documentation to developers.cardano.org as its official home from now onward.
Update Cardano Node documentation
Small reorganization to accommodate previously mentioned changes.

@rphair

from a first read of the source it looks like it will be a "user manual" for the CLI based features and I'm not aware of anything else like this, so it would be very welcome here.

That's exactly right. A few months ago, while discussing documentation topics with @KtorZ, he showed the anual survey where one the main weaknesses or concerns from the community was that documentation was a little bit too scattered.
At the time I was working on having node and cli documentation on a docusaurus website,  changed my mind, because of the survey: it makes it clear that ddevelopers.cardano.org is the most trusted and visited place to look for documentation. So we agreed that instead of having yet another website we would move our stuff to the developerss portal. In that regard, of course Mathias and Nicolas Cerny are well aware of this work.

(source of truth) To your knowledge, does this material have the same scope as any other CLI documentation? If so, how would the target audience be different between this material & any others?

The intention is to make developers.cardano.org  the official place to look for end user documentation for Node and CLI. It has same scope, but extended to cover undocumented features. i.e. RTS flags in node  are not discussed anywhere else. Moreover,  after this is PR merged, I will remove end user documentation from places under my control, i.e. https://github.com/input-output-hk/cardano-node-wiki and most likely will update the course https://cardano-course.gitbook.io/cardano-course to be based on the developers.cardano.org.

(maintenance) Given the new terms that have been added to cardano-cli with each release, do you and/or associates plan to keep this material up to date going forward, or is this a one-time sync with the Conway release of the node & CLI?

That is the intention. We plan to maintain this up to date with future releases. Our goal is to archive or remove  end user documentation from other places. Decentralized network is great, decentralized documentation is not. =)
I'll rebase on staging

@rphair The only merge conflict that shows up here is in yarn.lock.  I need advise on how to resolve this conflicts. mainy because the file starts with
This file is generated by running "yarn install" inside your project.
Manual changes might be lost - proceed with caution!
 DO NOT EDIT THIS FILE DIRECTLY.
yarn lockfile v1
So, since this is an auto generated file, i'm puzzled about the merge conflict.

Then you should be fine if you remove the yarn.lock changes from this branch so it doesn't appear on the /files listing anymore, since you've already ruled out other merge conflicts.
The auto-generated comments above are generated by the yarn install, which contributors never need to run unless yarn.lock gets clobbered in their branch.  Generally all PRs here only submit documentation changes & all PRs are therefore built with the yarn.lock from the repository itself.  If you generate a difference yarn.lock in your local fork, just don't commit it.
Removing yarn.lock from a PR after committing it is a common problem for both contributors & editors so I wrote a blurb about how to fix the problem here: https://developers.cardano.org/docs/portal-contribute#i-committed-yarnlock-to-my-pr-branch-what-do-i-do

Wow this is huge. Thank you @CarlosLopezDeLara. I can help with resolving the conflicts.

@CarlosLopezDeLara you are obv faster than me. 😁 thank you

@rphair @katomm @thenic95
This should be ready for review

@CarlosLopezDeLara thanks for your continuing attention, but what you did was not a rebase in the practical sense: since your PR at this time now contains all the changes to staging since you originally created this branch: as duplicates to the changes that have already been merged.  For a quick verification of this fact, see the /files page where it appears as if you are contributing all the /blog entries since March.
I think you are at the point where you have to create a new branch based on an updated staging in your fork that includes only the changes you are making.  There are ways of reconciling problems like this but I think these are no longer possible once a branch is submitted as a pull request, since the changes can no longer be "unwound" as if they had never happened.  The most widely recommended solution is to recreate the PR.
If other editors contradict this recommendation & allow this PR to continue, it really would be fraught with merge conflicts unless work on every other affected page on the Dev Portal were suspended until this PR was merged... and even then we could face a plague of duplicated or omitted changes if not done perfectly.  cc @katomm @fill-the-fill

@rphair  you are right, looks like I screwed it a bit. I like your suggestion, it will be easier and faster to reconcile. I'll create new branch. Thus closing this PR.

superseded by #1287 1287


Add Oak Hill Academy

Oak Hill Academy emails are<EMAIL_ADDRESS>https://www.oak-hill.net/course-descriptions/
See Computer/Technology with Advanced Engineering and Technology

@CoreyShupe Please rename lib/domains/net/oak-hill to lib/domains/net/oak-hill.txt. Thank you.

@philipto fixed, forgot about the whole .txt thing

@CoreyShupe Pull request merged. Thank you!


Expression is always false

Hello.
Take a look at line 1593 of nanovg_gl.h.
Expression 'ctx != NULL' is always false.

This is actually still valid. The bug report would probably be more telling, if it said that gl will leak, if the context creation failed.


Errors when trying to rebuild

When running PendulumRebuild I'm getting the following errors

Can you send your plugin config?

Can you send your plugin config and what os you are using?

configs:

OS: Arch linux

So this problem is arising partially because of your config and partially because of a mistake I made in the user command creation.
First, in order to use the report generation (Pendulum and PendulumRebuild commands), you need to specifically set the gen_reports=true option in your config as it is disabled by default.
See example config from README.md: (You'll need to translate it to fennel yourself)
require('pendulum').setup({
    log_file = vim.fn.expand("$HOME/Documents/my_custom_log.csv"),
    timeout_len = 300,  -- 5 minutes
    timer_len = 60,     -- 1 minute
    gen_reports = true, -- Enable report generation (requires Go)
    top_n = 10,         -- Include top 10 entries in the report
})

Secondly, your issue arose because the two user commands were created regardless of the value of gen_reports, when they should only have been created when the option is specifically enabled. I have since fixed this in the most recent version.


glfw-minecraft: merge glfw-wayland-minecraft into glfw

Description of changes
Alternative to https://github.com/NixOS/nixpkgs/pull/306264
In addition to updating the derivation to 3.4, this combines the two glfw 3.x derivations into one, saving a few lines.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 310073 run on aarch64-linux 1

2 packages built:

glfw3-minecraft
prismlauncher

Result of nixpkgs-review pr 310073 run on x86_64-linux 1

2 packages built:

glfw3-minecraft
prismlauncher

Result of nixpkgs-review pr 310073 run on aarch64-linux 1

2 packages built:

glfw3-minecraft
prismlauncher

Result of nixpkgs-review pr 310073 run on x86_64-linux 1

2 packages built:

glfw3-minecraft
prismlauncher

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1890

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1936


Updating contributors list for Omnia 1.7

Updating contributors list for Omnia 1.7

Can one of the admins verify this patch?


Bluetooth: host: Fallback to L2CAP for CPUP if rejected by master on LL

Fallback to L2CAP Connection Parameters Update Request if LL Connection
Update Request was rejected by remote device that has this marked as
supported in features. This can happen if procedure is supported only
by remote controller, but not enabled by host. This is connection
parameters update with iOS devices.
Signed-off-by: Szymon Janc<EMAIL_ADDRESS>

Codecov Report

Merging #11140 into master will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11140      +/-   ##
==========================================
- Coverage   53.11%   52.98%   -0.14%
==========================================
  Files         218      218
  Lines       26868    26855      -13
  Branches     5952     5951       -1
==========================================
- Hits        14271    14229      -42
- Misses      10162    10187      +25
- Partials     2435     2439       +4

Impacted Files
Coverage Δ

subsys/bluetooth/host/hci_core.c
2.42% <ø> (ø)
:arrow_up:

subsys/logging/log_output.c
63.9% <0%> (-8.79%)
:arrow_down:

subsys/logging/log_core.c
68.05% <0%> (-3.63%)
:arrow_down:

misc/printk.c
85.45% <0%> (-3.1%)
:arrow_down:

include/logging/log_msg.h
89.09% <0%> (-1.82%)
:arrow_down:

include/logging/log_core.h
100% <0%> (ø)
:arrow_up:

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 d158c07...6239845. Read the comment docs.

Didn't run PTS yet


how to compile the go-id program

I am trying to compile and run the go program to retrieve goroutine ids, i filed this ticket too
Issue #4 on t-yuki/goid
https://github.com/t-yuki/goid/issues/4

@ORESoftware what did you try? What did you see when you tried?

a readme with instructions, even if only for yourself when you try to compile it in 3 years. Last time I tried compiling it said it was missing runtime.h which is a golang related file to compile C programs? i don't know much about C, I just want to get it to work and use it in a Go program. thx.

It's likely this code does not compile any more. Sorry, i'm not going to fix it, you're welcome to fork the repo.


METRON-264: Create Docker container to simplify management of deploy dependencies

Provides a Docker container for running deployment.
Tested with EC2 and on bare metal.

What's your thought process on using a contrib directory?  What else would live in there?
I think metron-deployment desperately needs some reorganization.  Maybe your thought process with contrib can help that reorganization.

@nickwallen - I agree about the reorg. Will likely happen organically as a result of METRON-214.
Contrib is for stuff that's loosely related, but not essential to the project. I should probably put the packer stuff there as well.

+1

+1 great job


Support retry delay (exponential and fixed)

We don't have retry delay for now.
Support retry delay by taking new config parameter(s) for retry_policy.
Changes to uamqp library may also be needed.

I only implemented exponential to be consistent with azure-core
Python azure-core doesn't have fixed retry delay.


package for macos malloc error

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

I want to package for mac run
electron-packager . --platform=darwin --arch=x64 --overwrite  --packageManager=yarn --icon=./imgs/logo.png.hqx

but got error
node(80551,0x7fffcc6873c0) malloc: *** error for object 0x12f3863e0: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
[1]    80551 abort      electron-packager . --platform=darwin --arch=x64 --overwrite

@bimohxh What version of node are you running?

@MarshallOfSound  node version is 8.4.0, but this node version can run in windows with no error

Looks like the new 8.4.0 version of nodejs has some issues, see #714 as well.  Might want to raise this upstream in the node repo

Node 8.5.0 was released today, can you try that? If that doesn't work, can you downgrade to Node 6?

Since the original issue reporter has not responded to the request for additional information in about two weeks, I'm going to assume that they either found the answer in another manner or decided not to pursue this issue. If I'm incorrect, they can respond with answers to the questions posed, and I'll reopen.

This seemed to be an issue with 8.4 and 8.5. I upgraded to 8.8.1 and this error went away.

What are the version numbers referring to? There has never been an Electron Packager 8.8.1.

Node (updated the comment)


After Loading Trace, no prompt 'Trace Processor native acceleration' dialog on ui.perfetto.dev.

I'm following https://perfetto.dev/docs/visualization/large-traces to load large trace with native acceleration on a Macbook.
Terminal prompt shows 'Trace loaded: 3239.81 MB in 82.43s (39.3 MB/s)',
Then '[HTTP] Starting RPC server on localhost:9001'
But after '[HTTP] This server can be used by reloading https://ui.perfetto.dev and clicking on YES on the "Trace Processor native acceleration" dialog',
this dialog does not show up once I visit ui.perfetto.dev on Safari.

the same on a chromium-based browser as well.

Are you waiting that the trace load before showing the ui?
What happens if you curl https://<IP_ADDRESS>:9001 after the trace loads in trace processor?
It definitely works on my macbook.

yeah I'm accessing ui.perfetto.dev after 'Trace loaded' prompt.
curl localhost:9001 shows:

Perfetto Trace Processor RPC Server
This service can be used in two ways:

Open or reload https://ui.perfetto.dev/
It will automatically try to connect and use the server on localhost:9001 when
available. Click YES when prompted to use Trace Processor Native Acceleration
in the UI dialog.
See https://perfetto.dev/docs/visualization/large-traces for more.
Python API.
Example: perfetto.TraceProcessor(addr='localhost:9001')
See https://perfetto.dev/docs/analysis/trace-processor#python-api for more.
For questions:
https://perfetto.dev/docs/contributing/getting-started#community

But then, curl https://<IP_ADDRESS>:9001 shows
curl: (35) OpenSSL/3.2.0: error:0A0000C6:SSL routines::packet length too long

curl https://<IP_ADDRESS>:9001/ will not work since you the port is a plain HTTP port not a HTTPS one.
When you open ui.perfetto.dev.in, can you please Chrome devtools and see if an error is displayed there?

Are you waiting that the trace load before showing the ui? What happens if you curl https://<IP_ADDRESS>:9001 after the trace loads in trace processor?
It definitely works on my macbook.

I have been able to resolve this in a chronium-based brower(Naver Whale), but not in Safari.
It seems like Primiano was correct in the sense that terminal prompt 'Traced Loaded' does not signify the actual completion of trace load. I waited 2~3 hours after load, then ui.perfetto correctly showed the option for native acceleration.


Capture and display topic classification confidence score

Capture and display RAG document snippet's classifier confidence score in Local UI and PDF report
Tasks

[ ] Capture confidence score in backend schema
[ ] Display confidence score in PDF report
[ ] Display confidence score in Local UI on SafeLoader and SafeRetriever pages

Delivered Local UI support in release v0.1.18


Playground: I18n

based on ang 10 update
WIP notes in https://hackmd.io/OgzivKZORc2I5bzoaIHmMQ

Codecov Report

Merging #4726 into master will decrease coverage by 26.17%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #4726       +/-   ##
===========================================
- Coverage   54.44%   28.27%   -26.18%
===========================================
  Files        1120       58     -1062
  Lines       38961     4503    -34458
  Branches     4904        0     -4904
===========================================
- Hits        21211     1273    -19938
+ Misses      17574     3054    -14520
  Partials      176      176


High memory usage by volumes

At 14:01 (UTC time) all works great. At 14:07 sharply increased consumption of memory up to 100% (starting from master server)
System Setup

command line to start "weed master", "weed volume", "weed filer".

/usr/local/bin/weed -logdir=/var/log/seaweedfs/master master -ip=<IP_ADDRESS> -mdir=/data/seaweedfs/master-metadata -defaultReplication=010 -metrics.address=<IP_ADDRESS>:9091
/usr/local/bin/weed -logdir=/var/log/seaweedfs/filer filer -master=<IP_ADDRESS>:9333 -defaultReplicaPlacement=010
/usr/local/bin/weed -logdir=/var/log/seaweedfs/volume_8082 volume -mserver=<IP_ADDRESS>:9333 -ip=<IP_ADDRESS> -port=8082 -dir=/data/seaweedfs/volumes_8082 -max=230 -dataCenter=dc1 -rack=rack1
/usr/local/bin/weed -logdir=/var/log/seaweedfs/volume_8081 volume -mserver=<IP_ADDRESS>:9333 -ip=<IP_ADDRESS> -port=8081 -dir=/data/seaweedfs/volumes_8081 -max=230 -dataCenter=dc1 -rack=rack1
/usr/local/bin/weed -logdir=/var/log/seaweedfs/volume_8083 volume -mserver=<IP_ADDRESS>:9333 -ip=<IP_ADDRESS> -port=8083 -dir=/data/seaweedfs/volumes_8083 -max=230 -dataCenter=dc1 -rack=rack1

OS version

:~$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.5 LTS
Release:	18.04
Codename:	bionic

output of weed version

:~$ weed version
version 30GB 2.31 f5b5e4d linux amd64

content of filer.toml

:~$ cat filer.toml
# A sample TOML config file for SeaweedFS filer store
# Used with "weed filer" or "weed server -filer"
# Put this file to one of the location, with descending priority
#    ./filer.toml
#    $HOME/.seaweedfs/filer.toml
#    /etc/seaweedfs/filer.toml

[memory]
# local in memory, mostly for testing purpose
enabled = false

[leveldb]
# local on disk, mostly for simple single-machine setup, fairly scalable
enabled = false
dir = "."					# directory to store level db files

[leveldb2]
# local on disk, mostly for simple single-machine setup, fairly scalable
# faster than previous leveldb, recommended.
enabled = false
dir = "."					# directory to store level db files

####################################################
# multiple filers on shared storage, fairly scalable
####################################################

[mysql]
# CREATE TABLE IF NOT EXISTS filemeta (
#   dirhash     BIGINT         COMMENT 'first 64 bits of MD5 hash value of directory field',
#   name        VARCHAR(1000)  COMMENT 'directory or file name',
#   directory   TEXT           COMMENT 'full path to parent directory',
#   meta        LONGBLOB,
#   PRIMARY KEY (dirhash, name)
# ) DEFAULT CHARSET=utf8;

enabled = false
hostname = "localhost"
port = 3306
username = ""
password = ""
database = ""              # create or use an existing database
connection_max_idle = 2
connection_max_open = 100

[postgres]
# CREATE TABLE IF NOT EXISTS filemeta (
#   dirhash     BIGINT,
#   name        VARCHAR(65535),
#   directory   VARCHAR(65535),
#   meta        bytea,
#   PRIMARY KEY (dirhash, name)
# );
enabled = true
hostname = ""
port =
username = ""
password = ""
database = ""              # create or use an existing database
sslmode = "disable"
connection_max_idle = 50
connection_max_open = 100

[cassandra]
# CREATE TABLE filemeta (
#    directory varchar,
#    name varchar,
#    meta blob,
#    PRIMARY KEY (directory, name)
# ) WITH CLUSTERING ORDER BY (name ASC);
enabled = false
keyspace="seaweedfs"
hosts=[
	"localhost:9042",
]

[redis]
enabled = false
address  = "localhost:6379"
password = ""
db = 0

[redis_cluster]
enabled = false
addresses = [
    "localhost:30001",
    "localhost:30002",
    "localhost:30003",
    "localhost:30004",
    "localhost:30005",
    "localhost:30006",
]
password = ""

 After that LA skyrocketed. Had problems even to login over ssh (GMT+3 time).

Filer ERROR log (UTC time)
E0419 13:46:43 20059 stream.go:39] read chunk: http://<IP_ADDRESS>:8081/118,1ae7a35d392dbce5?readDeleted=true: 404
E0419 14:02:12 20059 filer_server_handlers_read.go:159] failed to write entry content: <nil>
E0419 14:05:54 20059 stream.go:39] read chunk: read tcp <IP_ADDRESS>:47990-><IP_ADDRESS>:8083: i/o timeout
E0419 14:06:42 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:33382: write: broken pipe
E0419 14:06:53 20059 stream.go:39] read chunk: read tcp <IP_ADDRESS>:55664-><IP_ADDRESS>:8083: i/o timeout
E0419 14:06:55 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:36306: write: broken pipe
E0419 14:06:57 20059 stream.go:39] read chunk: read tcp <IP_ADDRESS>:56186-><IP_ADDRESS>:8083: i/o timeout
E0419 14:07:27 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:37496: write: broken pipe
E0419 14:07:27 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:36430: write: broken pipe
E0419 14:07:34 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:39038: write: broken pipe
E0419 14:08:21 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:36748: write: broken pipe
E0419 14:08:21 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:38494: write: broken pipe
E0419 14:08:23 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:41124: write: broken pipe
E0419 14:08:23 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:41250: write: broken pipe
E0419 14:08:29 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:40050: write: broken pipe
E0419 14:08:50 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:40378: write: broken pipe
E0419 14:09:05 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:37638: write: broken pipe
E0419 14:09:17 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:42160: write: broken pipe
E0419 14:09:26 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:40570: write: broken pipe
E0419 14:09:26 20059 upload_content.go:239] upload 925067de-a111-11eb-b44d-02420a0000c2.xml 302550 bytes to http://<IP_ADDRESS>:8082/1147,20895ab89e6d8684: Post "http://<IP_ADDRESS>:8082/1147,20895ab89e6d8684": write tcp <IP_ADDRESS>:40760-><IP_ADDRESS>:8082: use of closed network connection
E0419 14:09:27 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:40780: write: broken pipe
E0419 14:09:27 20059 upload_content.go:239] upload 438f9a34-7321-11eb-ac58-02420a0000b8.xml 46283 bytes to http://<IP_ADDRESS>:8082/1179,20895ac2eab63d91: Post "http://<IP_ADDRESS>:8082/1179,20895ac2eab63d91": write tcp <IP_ADDRESS>:40868-><IP_ADDRESS>:8082: use of closed network connection
E0419 14:09:27 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:42110: write: broken pipe
E0419 14:09:27 20059 stream.go:44] write chunk: write tcp <IP_ADDRESS>:8888-><IP_ADDRESS>:41502: write: broken pipe

Seaweedfs version is consistent for all nodes, we have 5 nodes, every node is one rack with three disks, three separate volume servers.
Restarting master and volume servers didn't help.
Stopping master server and after that restarting volume servers did help.
We've observed similar behavior over a week ago. During the night everything came in order without our help.
What other data do you need?

Please follow issue template and add missing info.
You can also run with -cpuprofile=xxx -memprofile=yyy to collect cpu and memory profiling.
When the usage is spiking, please run a kill -3 <pid> and collect stack trace from the stdout.

Got another memory spike after enabling task with 150 threads with requests to get files.
Attaching kill -3 stdout of one of the volumes.
kill_volume.txt

you need to run ec.rebuild in weed shell.
there are a lot of threads recoverOneRemoteEcShardInterval, which reads other EC shards to calculate missing ec shards. This basically 10X the amount of volume server traffic.

ec.rebuild and ec.balance don't plan to do anything.
Maybe it's happening because of the load and high LA (during memory spikes) some volumes may fall out of the cluster.

could be. there were many these error messages. However, a bug fixed in ae74d8f showed the nil error instead of the real one.
Apr 21 06:56:20 data2.prophy.science seaweedfs-volume-8083[1011]: I0421 06:56:20  1011 store_ec.go:202] clearing ec shard 245.6 locations: read ec shard 245.6 from <IP_ADDRESS>:8081: receiving ec shard 245.6 from <IP_ADDRESS>:8081: <nil>

We've been able to trigger spikes with this task (with 150 threads) regularly. The last spike took ~15 minutes to happen (GMT+3 time).

rack4cpu_mem_profile.zip
rack5cpu_mem_profile.zip
While we are figuring out the situation what do you think about setting MemoryMaz in systems for seaweed services?

It seems current code will all following requests trying to do the heavy operations to read from recover from remote shards.
Added a fix a8114da02dddd65b0371e7a9e6a5125d085c55d4 for this.


Zernike Noll indexing different from what Anand expects?

Anand says tip and tilt are swapped, and maybe entries on lower rows of the Zernike pyramid as well. Note to self to check this and see if I can reproduce.

Can confirm that at least tip/tilt are swapped: zernike.noll_indices(2) returns (1, 1) when it should be (-1, 1) for the first entry on the second row of the triangle. Everything else looks fine up to Z_15

I am confused.
Both wikipedia (https://en.wikipedia.org/wiki/Zernike_polynomials#Noll.27s_sequential_indices) and the Online Encyclopedia of Integer Sequences (https://oeis.org/A176988) indicate a mapping of j=2 --> (n,m) = (1,1). I.e. those agree with what is currently coded in poppy.
What reference are you using for j=2 --> (1, -1) ?

See also:
In [7]: poppy.zernike.noll_indices(2)
Out[7]: (1, 1)

In [8]: poppy.zernike.str_zernike(1,1)
Out[8]: '\\sqrt{4}* ( 1 r^1  ) * \\cos(1 \\theta)'

This agrees with the definition for $Z_2 = 2 r \cos \theta$ given in Table 1 of Noll 1976. See the following horrible URL:
https://www.osapublishing.org/view_article.cfm?gotourl=https%3A%2F%2Fwww.osapublishing.org%2FDirectPDFAccess%2FCEE5519A-B678-4BF5-371C8DC6335E764C_56041%2Fjosa-66-3-207.pdf%3Fda%3D1%26id%3D56041%26seq%3D0%26mobile%3Dno&org=

Ah, Wikipedia is not in agreement with itself. I was going off what Anand said and this figure: https://upload.wikimedia.org/wikipedia/commons/9/9f/ZernikePolynome6.svg

Argh. Perhaps the appropriate bug fix here is replacing that figure on wikipedia?
Wait, ha, if you click through to the figure page, the caption has an erratum that the Noll indices for tip and tilt are interchanged. See https://en.wikipedia.org/wiki/Zernike_polynomials#/media/File:ZernikePolynome6.svg
Yeah that should just flat out be fixed on wikipedia. Do you want to edit the svg or shall I?

I can do it. Took me a minute to find my login info.

Ugh, uploading proves more complicated than anticipated.

https://commons.wikimedia.org/wiki/Commons:FAQ#How_can_I_upload_a_new_version_of_a_file.3F
Guess I'll check back next week!

Fixed the picture on Wikipedia (and the copy I pinned to my wall)


Doxygen supports darkmode toggle

Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Add this toggle. I'm using a dark mode default but when browsing webpage, I'd prefer light mode.


Adding traditional Chinese from linovelib

Describe your suggested feature
I think adding traditional Chinese version of linovelib would be better for traditional Chinese reader. Currently only simplified Chinese is supported with (https://www.bilinovel.com/).
Other details
Traditional Chinese is also supported in the same website (https://tw.linovelib.com/) , by the ‘繁體’ button at the lowest in the https://www.bilinovel.com/.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version 1.1.19.
[X] I will fill out all of the requested information in this form.

this website seems not to support traditional chinese anymore?

this website seems not to support traditional chinese anymore?

No, it is still supporting

this website seems not to support traditional chinese anymore?

No, it is still supporting

but when i try to click the '繁體' buttom, the web just refresh and still display simplified chinese, also the https://tw.linovelib.com/ will redirect to the www.bilinovel.com

this website seems not to support traditional chinese anymore?

No, it is still supporting

but when i try to click the '繁體' buttom, the web just refresh and still display simplified chinese, also the https://tw.linovelib.com/ will redirect to the www.bilinovel.com

That's weird. I can use the https://tw.linovelib.com normally and not getting redirected back.

oh I found the key
the website would check user's ip address
it will display traditional chinese when i use proxy of hk or tw

cloud you add https://raw.githubusercontent.com/MareDevi/lnreader-plugins/plugins/v3.0.0/.dist/plugins.min.json to your lnreader repo to help me to check if linovelib-tw plugin works well.
due to the network problem i cloudn't test thorough it

Hi, it works perfectly fine, photos are loaded and the novel works well. Thanks!

could you add https://raw.githubusercontent.com/MareDevi/lnreader-plugins/plugins/v3.0.0/.dist/plugins.min.json to your lnreader repo to help me to check if linovelib-tw plugin works well. due to the network problem i cloudn't test thorough it


Getting size data without emitting files

Is it possible use this loader without outputting files?

Why ?
I'm not clear about your use-case. You can simply ignore the file that's outputted if that's an option.
https://github.com/boopathi/image-size-loader/blob/4fa307fd391f0d262f6dfe1c14d88b458081b71b/index.js#L46
This is the code that's responsible for the copying the image. PR welcome :).

Well, I'm new to the custom loader thing, so maybe I'm doing something unusual/wrong. Basically I'm getting the dimensions of images that are sprites in a game. I just check the images so I know how to scale the plane that I'm applying them to.
In my case I'm still just requesting them via ajax from their original folder. Would it to be more normal to read from /src and move them to an out folder from which they can be required?

I need this feature too. The reason is: I'm using url-loader to get data-uri images. So I do not need to emit files, because they are inlined.
P.S. @boopathi Probably it would be cool if this loader will work on top of url-laoder/file-loader to reuse the abilities of that loaders.
There is one more thing. What are you thinking about this hack?
var foo = new String('foo/bar.gif');
Object.assign(foo, {
width: 123
});

console.log(foo + '', foo.width)

@SleepWalker, @rhys-vdw
If you still need this, I've forked this loader to one which doesn't output the file: https://github.com/tremby/image-dimensions-loader (also published on NPM as image-dimensions-loader).

@boopathi: assuming you're not going to add an option to do something similar, a link from your readme to my fork for others looking for this feature would be appreciated.


SortSite flags WCAG 2.0 F24 failure

As reported on Slack:

Hello, we’re using an automated tool to check our design system built pages for accessibility problems and we get an error on a text-area:
If you set any of the colors on the body or a elements you must set all of them.
WCAG 2.0 AA F24Section 508 (2017) AA F24

Which is F24: Failure of Success Criterion 1.4.3, 1.4.6 and 1.4.8 due to specifying foreground colors without specifying background colors or vice versa
The automated tool that flagged this is SortSite.

As far as I can see neither the background colour nor the foreground colour are set on textareas, so I’m not sure why it’s complaining…
We should be careful if we decide to change the background / foreground colours of inputs as it's caused problems in the past – see https://accessibility.blog.gov.uk/2017/03/27/how-users-change-colours-on-websites and https://github.com/alphagov/govuk_elements/pull/397.

The procedure for testing F24 only talks about foreground and background colours, neither of which are set on the textarea component.
The only colour that we do set is the border colour, and technique C25 suggests that this is a totally fine thing to do.
I think this might be a bug in Powermapper's implementation of the F24 test?

I've not yet been able to reproduce this using Powermapper's online demo against http://govuk-frontend-review.herokuapp.com/components/textarea/preview.
(I had to run it locally and remove the robots.txt restriction to get it to run)
I've asked the person that originally raised this if they have any additional styles that might be setting the background or foreground colour on the textarea.
It could be that the desktop version behaves differently, or they're using an older version?

@antimega has been able to reproduce this in SortSite:

If you set any of the colors on the body or a elements you must set all of them.	WCAG 2.1 AA F24	1 pages
In HTML the color attributes are text, bgcolor, link, alink and vlink. In CSS the attributes are color and background-color. Some users have browser defaults set to white text on a black background, so setting one color without setting the others can result in black text on a black background.
The colors used are:
color: rgb(0,0,0); uses browser default background-color: rgb(255,255,255);
http://govuk-frontend-review.herokuapp.com/components/textarea/preview	Line 29

Line 29 is the <body> tag, which does set the background colour but not the foreground colour (which is the exact opposite of what the SiteSort error says).
We're also seeing this error flagged on http://govuk-frontend-review.herokuapp.com/components/back-link/preview, so it looks like this might not be related to the textarea component at all.

Hi, we are getting the same type of issues from our accessibility tests. We use pa11y.

It's actually complaining that the foreground colour needs to have a contrast of 7:1 with the white background colour.
What would you suggest?

Hi @nikola-naydenov-hmcts,
The tool there appears to be testing against WCAG Level AAA, and is reporting that blue links don't have sufficient contrast with the white background.
Public services in the UK are expected to comply with Level AA which only requires a contrast ratio of 4.5:1, and GOV.UK frontend is built to match that.
If you are intending to meet Level AAA, you'll need to customise Frontend to match those requirements. Otherwise, you may have to change the settings on the auditing tool to ignore Level AAA failures.
Hopefully that helps!


Updates openshift-assisted-ui-lib to 2.6.7-cim

Release notes
cc @openshift-assisted/ui-maintainers

Closing without merge, we do not want to revert to 2.6 branch here


Call for volunteers: please comment if you are interested in helping maintain go-ping.

There are a variety of open issues and pull requests, most dating back to early 2019 but some as far as 2017. Additionally there have been no commits in over a year.
Are you still interested in maintaining this, @sparrc? If not, would you be amenable to giving commit rights to others?
What is the plan for go-ping?

I also have the same question. I've been thinking about making a permanent fork, as I use this for one of my projects.

Apologies, I have been considering soliciting for help in maintaining this project. My first child was born in early-2019 and I no longer have enough free time outside of work to maintain this myself, as has clearly been noticed.
I will change the title of this issue as a call for volunteers to help in maintaining this project, as I'd rather not see it abandoned.
@SuperQ I'm assuming you are interested?

Yes, I'm interested.

Ping @sparrc :smile:

I'd also potentially be interested in helping out if you feel more hands are needed 👍

I think it'd be easier to coordinate this if we were able to chat in real-time. Are you both able to join the gophers slack workspace? We can chat there in channel #go-ping, and we can also use that as a place for people to go for support and questions.
Not sure if there's a better workspace than gophers.slack.com, but it still seems fairly active, you can join by going here: https://gophers.slack.com/join/shared_invite/zt-fajz7jh3-2cpkmFU~hQb8d5LmOCnhfQ#/

Ugh, more slacks to sign up for. I'm happy to setup a Zoom/Meet/Jitsi call some time. I'm on EU central time.

I would prefer to have a method of continuous chat communication for now. Gophers slack seems to me to be the most logical place but I'm open to other venues too.

gophers link seems to be no longer active

@pgollangi try this: https://invite.slack.golangbridge.org/

This repo has now been transferred to the go-ping organization


4.1 release

The moment we've all been waiting for. Swift Package Manager support for all services and XCode 12 compatibility

@apaparazzi0329 we need to update .travis.yml to run all of the jobs on XCode 12.

@apaparazzi0329 we need to update .travis.yml to run all of the jobs on XCode 12.

@mediumTaj Unfortunately swift package manager compiles from a single source folder so any source that needs to be compiled for a service has to be in the same respective folder.

@mediumTaj Unfortunately swift package manager compiles from a single source folder so any source that needs to be compiled for a service has to be in the same respective folder.

FYI that this fixes #1044 and #1046

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


Nats integration

I am working on e posing this bad boy over nats.
there is a good rust Nats driver
this then allows the library to be able to be controlled from any language that nats supports which is about 20 I think.
Let me know if your I terraced in this.
I don’t know yet if I should use protobuf buffers as the types and if the rust api is generic enough to even expose the types over NATS.
Nats will mean that thousands of real time actors ( client or servers ) can control unreal

I like this idea. Did you work on it at all?


advertised homepage http://osteele.com/sources/javascript/functional/ is down

The home page is now back up.


Accidently triggering nested snippets in LaTeX math

Example:
Say I wanted to write $\frac{c}{2}$. I would do the following:
$frac|$ -> TAB -> frac snippet expansion: $\frac{|}{}$
now I write c and press TAB to move the cursor to the denumenator:
$frac{c|}{}$ -> TAB ->  $\frac{\cite{|}}{}$
The cite snippet expanded accidently!
This is solely an issue with a single-trigger-letter snippets inside math: pretty match only in math you'd write a single letter and press tab, NOT expecting the letter to expand into a snippet.
Possible workarounds:

Disable nested snippet expansion: inconvinient, they are generally very useful
Add a condition to the single-letter snippets so they don't expand in math envs:
I tried Doing that using (texmathp) from auctex, but it doesn't work while inside a nested snippet; something in yasnippet messes that up. I haven't look into what exacly yet tho.
Change the trigger key of snippets cite, gls... not to be a single letter

I've got it! finally!
In nested expansion, the condition runs in a narrowed buffer, so surrounding (texmathp) with (save-restriction (widen) ...) gives the correct behaviour. This should probably be reported to yasnippet. Meanwhile, I'll add the condition to the short-key snippets.


polymc: mark knownVulnerabilities OVE-20221017-0001

Description of changes
Mark OVE-20221017-0001 as knownVulnerabilities: info, #196460 is tracking this here and has a few more external links.
tl;dr a PolyMC maintainer has removed all other maintainers' access to the repository, while no malicious code changes have been pushed (yet?), PolyMC incorporates functionality which pulls down code repository data which is likely under the control of this maintainer. More notes on potential impact are on the linked issue.

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

cc @starcraft66, missed your comment on the linked issue, sorry!

Waiting on checks to all go green

Should merge ASAP...

Building shows error correctly
error: Package ‘polymc-1.4.2’ in /nix/store/648nsmcj06ap6s7qgqbbgv8nsan0qp5z-source/pkgs/games/polymc/default.nix:64 is marked as insecure, refusing to evaluate.

       Known issues:
        - OVE-20221017-0001
    ...

Note about "needless delays in merging", this wouldn't be published on channels until it anyways evals through hydra, and channels update. So the few minutes "wasted" here are a fraction of the total time it'll take to propagate fully.
Also end-users need to update channels.


Lombok 1.18.2 is not compatible with auto compile on save feature of Apache Netbeans 9.0

Hi,
I have been working in Netbeans using lombok quiet a long time. I found that it is becoming unstable with the new Apache Netbeans 9.0 auto compile features when working in multi module maven projects.
Project structure:
.
├── mod1
│   ├── pom.xml
│   └── src
│       ├── main
│       │   └── java
│       │       └── np
│       │           └── com
│       │               └── ngopal
│       │                   └── mod1
│       │                       └── User.java
│       └── test
│           └── java
├── mod2
│   ├── pom.xml
│   └── src
│       ├── main
│       │   └── java
│       │       └── np
│       │           └── com
│       │               └── ngopal
│       │                   └── mod2
│       └── test
│           └── java
│               └── HelloTest.java
└── pom.xml

User.java
@Data
public class User {
    private String msg;

    public void hello(){
        setMsg("Hello 8");
        System.out.println(msg);
    }
}

HelloTest.java
public class HelloTest {
    @Test
    public void test(){
        User u = new User();
        u.hello();
    }
}

So i have two modules where mod2 depends upon mod1 and I just wrote a HelloTest.java junit test for printing the msg from User class.
Lombok Version 1.16.10 and Apache Netbeans 9.0 (Works good but the Netbeans editor shows red error regardless of successful compile)
Step 1: I changed setMsg("Hello 8") -> setMsg("Hello 9") in User.java
Step 2: I ran Test Project in mod2 It correctly displays Hello 9
Lombok Version 1.16.10 and Apache Netbeans 9.0 (Does not work at all but editor does not show any red error which is good )
Step 1: I changed  setMsg("Hello 9") -> setMsg("Hello 10") in User.java
Step 2: I ran Test Project in mod2 It displays Hello 9
Currently it works only by re-compiling mod1 manually every time I change some code in mod1 module.

Sounds like a duplicate of #1770 – can you give the edge release a try? It contains the fix. https://projectlombok.org/download-edge

I tried the latest build of lombok edge.
mvn install:install-file -DgroupId=org.projectlombok -DartifactId=lombok-edge -Dpackaging=jar -Dversion=1.18.3 -Dfile=lombok-edge.jar -DgeneratePom=true

Updated my dependencies and tried clean build which cleared and installed artifact new one with new lombok. But still Netbeans cannot pickup edits of mod1 in mod2 during the JUnit test . However during the Netbeans editor mode if I edit any files in User.java by adding any new method inside, these methods were quickly taken up in HelloTest.java which is good.
Thanks
Narayan


Mixed Content not shown in Plugin Instance

I have a plugin for a company Outlook-Web-App Instance that runs over https of course. Some
Mails will have content mixed into the mails that come from insecure (http) sources, mostly images.
I think this was fine before the last Franz update but now the images are not displayed anymore.
This is probably the exact same issue that Rambox used to have (see: https://github.com/saenzramiro/rambox/issues/202 )
Is there a way to allow this mixing of content for a plugin?
Or is it something that needs to be fixed in Franz itself?

So clearly reading docs is fundamental, regardless, i had no luck with this.
I added the following bit to the package.json of the plugin:
"webviewOptions": {
      "disablewebsecurity": "on",
      "webpreferences": "allowRunningInsecureContent, allowDisplayingInsecureContent, webSecurity=0"
    },

Yet i still get the mixed content warning. Am i missing something?

I also have the same problem with Pocket Casts.
Did you get a change to get it to work in the mean time?

I also read the docs, but they weren't very clear, specially the Electron pages.
Shouldn't the flag for "disablewebsecurity" be true or false?
"on" seems a little bit odd..

Well i can safely say i tried every single combination and none of them had any effect. :)

That's my problem also.
I'm not sure if the problem is with Pocket Casts or with Franz.
I can use the web app on the browser, with no problems whatsoever.
The website has a https connection, but sometimes uses http for its CDN.

I'm starting to wonder if Franz is dead? Some communication from the dev(s) on any channel would be kinda nice to know if Franz is actually still in active development or atleast so we all know where we stand.


[flow] post-HOC change bug fixes

Various small flow fixes.

@rosskevin Is this me or this PR is making GitHub crash?

Just you.


[Daily] Using Access Token instead of Github Token + Enable Auto merge

For :

Re-Propagate Composer Dependencies to Packages
Re-Apply Rector Rules

This is to handle pending start github action workflow:

Compared to weekly pull request, it is using ACCESS_TOKEN instead of GITHUB_TOKEN
Enable auto merge as well.

rebased.

:+1:


https://trello.com/c/NtoRNptT/636-roles-create-signing-authority-role-3

This role grants access to Sign 1/2 & Sign 2/2 (Propose and Accept) a Credit Transfer Proposal

 Roles | Create Signing Authority role |3|


Avoid loops in s2fft.sampling.reindex functions to reduce compile and run times

The index conversion functions in s2fft.sampling.reindex currently use Python loops (over L iterations) to perform the reindexing operations slice by slice. As these loops are unrolled when JIT compiling this leads to long compile times as L gets bigger.
This PR refactors the reindexing functions to instead construct arrays of indices and use these to gather or scatter the relevant entries in single indexing operations, thus avoiding the loops. For the functions converting between HEALPix and 2D harmonic coefficient indexing, we can use the numpy.triu_indices function to construct the relevant index arrays. For the maps between 1D and 2D layouts we can use numpy.where on a boolean array constructed so that the relevant array elements are True.  In both cases the explicitly constructed index arrays add some memory overhead - as they depend only L which is static they will be constants at compile time, and in all cases are I believe $\mathcal{O}(L^2)$ in size - however this is no larger than the memory requirements of the harmonic coefficient arrays themselves, and the computational graphs of the resulting compiled functions are much smaller (no longer scaling with L) so its unclear if overall the memory requirements are much larger.
I've done some benchmarking of the new implementations here compared to the current implementations and in all cases tested the compile times are significantly lower and the run times comparable or significantly better:
https://gist.github.com/matt-graham/7a2b5b77f51b5301b8910ced176d8a8a
For example for the flm_2d_to_hp_fast function, the following plot shows the compile times of the current implementation ("Current"), the proposed implementation ("triu_indices based") and two other alternatives I tried out ("Slice based" and "Nested fori_loop") for various L values

and the corresponding plot for run times of the compiled functions

On both compile and run times the proposed "triu_indices based" implementation is significantly quicker than both the current implementation and the other alternatives considered.
EDIT: There was a slight bug in the flm_hp_to_2d_fast reimplementation now hopefully fixed. The run times for the new function are now a little slower but the compile time is still much better than current implementation and importantly not quickly growing with L.

@matt-graham nice, this was a temporary loop that I never got around to removing!


feat: Support uploading roles and permssions via xlsx files.

issue #1829
Support uploading roles and permssions as demonstrates by the demo videos below:
https://github.com/casdoor/casdoor/assets/31148569/3e30ac2d-0ed3-4a22-9865-a2a717875c0a
The template xlsx files are also included.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Error out if model spreads itself in its declaration

Fix #2543.

Added the breaking change label, because if a spec has the form that previously just did nothing, now it will error out.


Is there any way to run testcafe suite on Smart TVs or Android TVs?

What is your Test Scenario?

What are you suggesting?

What alternatives have you considered?

Additional context

I have test automation framework developed using TestCafe and working fine on browsers. However, I have new requirement to run test suite on the application installed on Smart TVs. So, I searched for the possible options to re-use my test cases. But I couldn't find any except using Suitest automation library in JS and write new test scripts using the new library that will run specifically on TVs.
I am still looking for an alternative solution which can save my effort by re-using test cafe scripts that use selector which works fine on browsers, but not apps on devices like Smart TVs.
Below are the possible alternatives and please clarify if this can be achieved -

Can we extend the existing Selector capability to identify elements on Apps?
If not, then how can we integrate Testcafe with Suitest automation library or Android Studio even?

Kindly suggest if there is any other alternative solution to the problem.
Thanks
Rajat

TestCafe is intended to test web applications based on web technologies like HTML and JS.
If your application is webview-based, it is technically possible to use TestCafe for testing WebView based apps (built with Cordova/PhoneGap/etc).
Please follow the https://github.com/DevExpress/testcafe/issues/2252 to see the progress.
However, we cannot give any estimate as to when this feature will be implemented, since it requires much time.
As a workaround, I can only recommend you run your application (if it is built with HTML + JS) in the browser of your Smart TV and connect it as remote

Can we extend the existing Selector capability to identify elements on Apps?

If your application is webview-based, you do not need to extend anything.

If not, then how can we integrate Testcafe with Suitest automation library or Android Studio even?

You don't need any integration if your application is webview-based.


Non-atomic project init

I was playing around with the CLI and found the below a bit strange, and I'll explain my thought process here.
So initially, I thought I could start a git + Radicle project from the outset, but it failed due to the directory not existing. Ok, I'll make the directory and init that. Oh, it needs to be a git repository. Alright, let's add some stuff and commit. Hmmm, I originally wanted a main default branch but because I used git init it's actually master, and it says initialisation failed. Alright, I'll initialise with master instead. Wait... what?! It is initialised? I guess I'll rad edit so :)
This kind of failure but not actually failing is a bit confusing. I'm wondering how that can improved. I've said "non-atomic" in the title, but maybe we could have better error hints, like:
== Initialization step failed git: src refspec 'refs/heads/main' does not match any existing object; class=Reference (4)
== The project was created but the 'default-branch' could not be found, perhaps you meant to use a different branch. You can change this by using 'rad edit <id>'

haptop in ~/Developer λ RAD_HOME=/home/haptop/TestRad rad init ./totally --name totally --default-branch main
== Initialization failed No such file or directory (os error 2)
haptop in ~/Developer λ mkdir totally
haptop in ~/Developer λ RAD_HOME=/home/haptop/TestRad rad init ./totally --name totally --default-branch main

Initializing local 🌱 project in /home/haptop/Developer/totally

== Initialization failed could not find repository from '/home/haptop/Developer/totally'; class=Repository (6); code=NotFound (-3)
haptop in ~/Developer λ cd totally
haptop in ~/Developer/totally λ git init
haptop in ~/Developer/totally on master λ RAD_HOME=/home/haptop/TestRad rad init . --name totally --default-branch main

Initializing local 🌱 project in .

ok Passphrase · ********
ok Unsealing key...
== Initialization failed error: repository head does not point to any commits
haptop in ~/Developer/totally on master λ touch README.md
haptop in ~/Developer/totally on master λ git add README.md && git commit -m "Add README.md"
[master (root-commit) 088b331] Add README.md
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 README.md
haptop in ~/Developer/totally on master λ RAD_HOME=/home/haptop/TestRad rad init . --name totally --default-branch main

Initializing local 🌱 project in .

ok Passphrase · ********
ok Unsealing key...
ok Description · Totally testing rad
!! Initializing...

== Initialization failed git: src refspec 'refs/heads/main' does not match any existing object; class=Reference (4)
== src refspec 'refs/heads/main' does not match any existing object; class=Reference (4)

haptop in ~/Developer/totally on master λ RAD_HOME=/home/haptop/TestRad rad init . --name totally --default-branch master

Initializing local 🌱 project in .

== Initialization failed repository is already initialized with remote rad://z4Jn1YW15aEUnjctdHRRTSYMCW9Af/z6MkudE3bnPSZnzwk85VJsid7cxM4qn1SCXH5rMV7rgLHaUe

So I think the correct thing to do here is to fail when you specify a default branch that does not exist in the repo, and not create the project.
Additionally, I think we could take better measures or give better error messages when initialization doesn't fully complete.


docs: Add conda-forge install instructions

Resolves #227

Add instructions for pixi and conda for installing jupytergis from conda-forge.

c.f. https://github.com/conda-forge/jupytergis-packages-feedstock

Use 'python -m pip' pattern as a best practice for ensuring the right pip is used.

Even in virtual environments things can get broken in ways you don't expect.

This is ready for review.


Make contents in list elements tokenized in Document.builder

This patch fixes #638.

Coverage increased (+0.006%) to 89.972% when pulling 12bb4d1cccaaed1b4289fa3f63bb78fd28269311 on takahi-i:redpen-638 into 738bf47a414598d550f35642e1a66a68b6879d20 on redpen-cc:master.


Default cache use wrong cached values

Map<String, String> context = Maps.newHashMap();
context.put("key", "first");
Object value = JsonPath.read(context, "concat(\"/\", $.key)");
assertThat(value).isEqualTo("/first");
context.put("key", "second");
value = JsonPath.read(context, "concat(\"/\", $.key)");
assertThat(value).isEqualTo("/second");

That test fails, the cache seems to do something very odd with concat, if not using concat it does not fail, but using concat the cache uses the same value, even that the map where is reading changed.

I can confirm the issue.
FunctionPathToken and Parameter cache the result from the first evaluation and return it also if the input has changed. @mgreenwood1001, would you like to have a look?

I'll take a look this evening and provide a fix.

Will be fixed in 2.3.0

Hey there ! I'm using 2.4.0 and i'm still having this issue, is that expected ?

Hey there ! I'm using 2.4.0 and i'm still having this issue, is that expected ?


"RecentMessages" not thread safe.

I get a lot of silent exceptions thrown by the MD5 hash algorithm. You can see these exceptions by either enabling "System.Security.Cryptography.ChryptographicException" in debug exception settings or by adding a handler to TaskScheduler.UnobservedTaskException.
It's because the ComputeHash is not thread safe. I suggest to not reuse the created MD5 object and instead create it each time when calculating hash.
So:
using (HashAlgorithm hasher = MD5.Create())
{
return Convert.ToBase64String(hasher.ComputeHash(message));
}

Thanks for the bug report.  You are correct: MD5 is not thread safe.
Do you want to submit a PR?

I tried to push a branch, but I don't have access to the repository.
On Tue, 27 Aug 2019, 11:59 am Richard Schneider<EMAIL_ADDRESS>wrote:

Thanks for the bug report. You are correct: MD5 is not thread safe.
Do you want to submit a PR?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/richardschneider/net-mdns/issues/75?email_source=notifications&email_token=AAVIDL3PWXPJWIVDDGSMLJTQGT3HZA5CNFSM4IQBNDQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5HGNLI#issuecomment-525231789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVIDL5TSYPA3OCFSLO4EPLQGT3HZANCNFSM4IQBNDQQ
.

Read https://help.github.com/en/articles/creating-a-pull-request-from-a-fork

Done. When can it be released?

Fixed in release https://github.com/richardschneider/net-mdns/releases/tag/v0.25.0


[BEAM-1382] Unify encoding of common coders across SDKs

Following Robert's original work in #1764,

move the standard_coders.yaml file to the common/fn_api module in a test resource
add the Java version of tests
make the Python TupleCoder (and hence KVCoder) be more efficient in outer context and update spec so all 3 agree (spec, py, java).

Known issues:

[ ] the python standard_coder_test does not seem to have any tests in it when run via mvn install. However, I was able to make it fail to find the file now.
[ ] Python needs to obtain and use the standard_coders.yaml file from the right common place. @robertwb has ideas for how this is best properly integrated.

R: @robertwb (for py + general)
R: @kennknowles or @tgroh (for Java comments)
Note I'm keeping a list of known issues in the PR desc.

Thanks. LGTM from my side.

Yea LGTM

retest this please

Coverage remained the same at 69.605% when pulling ca3e67aba9b41f8ee243e8ed9d3ca7f3c09ca1c8 on dhalperi:coder-crosssdk-tests into 8c1a577fe52ffd02e57f3e65924aced10e38a82a on apache:master.

The changes to setup.py does not make sense to me. This would not work for people who just want to work from a copy beam/sdks/python/. The location for packaged test data is `tests/data/*', this file not would be included in the tarball. Running the test, after installing the tarball will fail.
Could you just have a symlink instead of copying files? And until we have a solution could we please revert the change to setup.py and include a copy of the file in the tests/data/ location?

I'm fine with putting the copy in tests/data (with an appropriate git ignore). Symlinks may get messy on Windows and with tar, so I think (perhaps to defensively) a copy is safer.

Thank you @robertwb. @sb2nov is making the changes in https://github.com/apache/beam/pull/1964.


Data product identities

Similar to #680 our data products (digital object types) need identities, such as:

Official public facing names

Note that this doesn't necessarily have to be the same as the backend. The titles just need to make sense/add value to anyone who lands on the HRA, similar to how we need this same synthesis for our application products.

Branded logos
Short descriptions (<125 characters)
Long descriptions

Resources

HRA Redesign canvas
HRA-KG canvas
Product Identities

Work continues in the HRA Product Organization spreadsheet and the HRA-KG paper. I am closing this issue.


[Update Request]: Keybase v6.2.4

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Keybase.Keybase
Package Version
6.2.4
Please describe the changes you would like to see
Source release https://github.com/keybase/client/releases/tag/v6.2.4
Binary release https://s3.amazonaws.com/prerelease.keybase.io/windows/Keybase_6.2.4-20231231194644%2Bae7e4a1c15.amd64.msi

Working on it. PR is on the way.

@orgcontrib - Could you please add the following content to https://github.com/microsoft/winget-pkgs/pull/135242#issue-2087138794? This will associate the pull request with this issue and attach https://github.com/microsoft/winget-pkgs/labels/In-PR to this issue. When pull request is approved and merged, this issue will automatically closed.
- Resolve #135227

@denelon is there any chance to document the average review wait time, a contributer may expect?


Copying Used Factory Also Copies Compatible Machines

Problem Description
When shift+right clicking and shift+left clicking on the factory icon to quickly transfer which factory you're using to various lines, normally if the factory you're copying over can't be used for the good (for example, you can't use a foundry to make rocket fuel, but you can use assemblers) it'll throw an error telling you "pasted machine is incompatible".
however, different goods have overlap. for example, both belts, green circuits, rocket fuel, and electric engine units can be made in the assembler 2, but only belts can be made in the foundry, only circuits can be made in the electromagnetic plant, only rocket fuel can be made in the biochamber, and only electric engine units can not be made in the assembler 1. if you were to put in an order to make belts, set it to use the assembler 2, copy that, and then make an order to make rocket fuel, green circuits, and electric engine units, and copy that assembler 2 over to those production lines, you run into this bug.
after you copy the machine over, if you later decide to change the factory building (maybe you've unlocked a more efficient machine), you will be limited by the compatibility group. this not only creates situations where you cannot later change the factory to use a newly unlocked building because the original group you copied from doesn't include the building, but it also means you can set the factory to a building that is incompatible. here's an example of me making engine units in a biochamber, belts in an EM plant, and green circuits and rocket fuel in a foundry. the only exception to this would be the assembler 1, which it seems does not get added over in the case of the electric engine unit. so while I can make a plan to build engine units in an EM plant, I cannot make a plan to build them in assembler 1s (neither building should be possible).
space age factory planner shenanigans.zip
factorio-current.log
Reproduction

create several factory lines that share a common production building (such as assembler 2s)
copy that common building across lines from a production line that can be made from a building the other lines cannot
behold as you can now attempt to make rocket fuel from lava

I've encountered this issue without needing to copy and paste; no diea exactly how but FP will let me set Engine Units as being made in a Foundry when they can't be

Thanks for the reports. When copy/pasting, I need to manually verify what is compatible and what isn't, and it seems I got it wrong in some way here. Will have a look.

Fixed this for the next release, thanks again


Any mechanism for 'overly verbose output'

I have a set of tests running locally on a variety of developer (osx) machines.
Oddly, on the jenkins centos box -- they fail.
Also very oddly, when I run them locally, 250 of them pass.
On the jenkins box, 452 of them fail ;)  (there aren't 452 of them haha)
So, basically, i'm just trying to think of any way I can get bin/rake teaspoon to spit out a heck of alot more output as it is trying to run.. and i'm hoping anyone here has some hints as to how to do that :)
Thanks!

What type of output are you looking for? You can run teaspoon --format documentation to output your spec details, but I assume that's not what you're looking for.

Mike - yeah, not what I was looking for but thanks.  I started making use of Magic Lamp, and despite all the tests running on every developer box etc.. for some reason, on the jenkins centos box, all the tests that use magic lamp now fail.  And, since there is so little debugging/bootstrapping/I don't even know what I am looking for (hah) output, I can't seem to track down why this is the case.

I've never used Magic Lamp but if you can create an app that reproduces the problem I'll happily take a look.


Editor crashes if tool script deletes the scene root

Tested versions

Reproducible in 4.3
Reproducible also in newest 4.4 master, custom build

System information
Godot v4.3.dev3 - Ubuntu 20.04.6 LTS (Focal Fossa) - X11 - Vulkan (Mobile) - dedicated AMD Radeon RX 6600 (RADV NAVI23) () - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
Editor crashes if a scene root deletes itself via tool script
Backtrace below. I tried to turn debug symbols on by compiling with production=yes debug_symbols=yes, but didn't get any extra info in this case:
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.4.dev.custom_build (db05f1dd8441be7fe6dd83097b0732722b0cc1c3)
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x43090) [0x7fa7a4fb8090] (??:0)
[2] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x65b730) [0x564959d5b730] (/home/ultsi/dev/godot/godot-src/core/string/ustring.cpp:4158)
[3] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0xc326a2) [0x56495a3326a2] (/home/ultsi/dev/godot/godot-src/core/string/ustring.cpp:4219)
[4] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x39afef9) [0x56495d0afef9] (/home/ultsi/dev/godot/godot-src/editor/gui/scene_tree_editor.cpp:337)
[5] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x39b2c99) [0x56495d0b2c99] (/home/ultsi/dev/godot/godot-src/editor/gui/scene_tree_editor.cpp:653)
[6] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x3a6c861) [0x56495d16c861] (/home/ultsi/dev/godot/godot-src/./core/variant/binder_common.h:304)
[7] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0xf06525) [0x56495a606525] (/home/ultsi/dev/godot/godot-src/core/variant/callable.cpp:57)
[8] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x654721) [0x564959d54721] (/home/ultsi/dev/godot/godot-src/core/object/message_queue.cpp:221)
[9] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0xcce948) [0x56495a3ce948] (/home/ultsi/dev/godot/godot-src/core/object/message_queue.cpp:268)
[10] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x523ca5f) [0x56495e93ca5f] (/home/ultsi/dev/godot/godot-src/servers/physics_server_3d.cpp:94 (discriminator 2))
[11] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x525ae4a) [0x56495e95ae4a] (/home/ultsi/dev/godot/godot-src/platform/linuxbsd/os_linuxbsd.cpp:962)
[12] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x623f42) [0x564959d23f42] (/home/ultsi/dev/godot/godot-src/platform/linuxbsd/godot_linuxbsd.cpp:85)
[13] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf3) [0x7fa7a4f99083] (??:0)
[14] /home/ultsi/dev/godot/godot-src/bin/godot.linuxbsd.editor.x86_64(+0x6265ce) [0x564959d265ce] (??:?)
-- END OF BACKTRACE --

Steps to reproduce

Create and save a scene
Add a script to the root
call queue_free in _physics_process
Editor crashes when trying to open the scene

Minimal reproduction project (MRP)
tool-script-crash.zip

See: https://github.com/godotengine/godot/issues/90176
If you're interested in a PR that allows you to do this see: https://github.com/godotengine/godot/pull/93726

I read through the closed issue, and I still think this is a valid issue for normal users. In my case I created a scene resembling a bullet, tested with @tool mode that its velocity is set correctly via export variables etc. in the editor. Then while running the game I created the behavior where the bullet deletes itself after it is a long distance away from its starting point.
Because I wrote the auto delete behavior in game with hot reload, the editor didn't crash instantly after adding queue_free(). Only the next day, when I came back and started to edit the bullet scene in the editor, the editor started crashing without any obvious reason. Took me an hour to debug the issue.
I think a fix could be that queue_free() throws an error, if it is being used from a tool script, and if it is freeing the scene root.

Ah older PRs describe that this is something user needs to be aware of: https://github.com/godotengine/godot/issues/26409
Maintainers: you can close this if you think the same still.

Yeah this is sadly a case of "With great power comes great responsibility". Tool scripts are super powerful but that also implies the power of wreaking havoc in the editor and causing crashes.
We're working on a Safe Mode feature which will at least let you recover such crashed projects in a better way (tested with the MRP, it makes it possible to open the project and fix the tool script): #92563.


custom sorting function

Umzug sorts migration filenames by name, thus files named 1, 2, 12 will be sorted 1, 12, 2. Files can also be named using different paterns (major.minor etc.), so it's best to allow to supply custom sorting function.

Hmmm.. I thought umzug sorts the migrations on its own, but I had to add a manual ORDER BY when fetching executed migrations that are returned from the custom executed callback. I would note that in the docs, if that's really the case (?). I can make a PR.

I wonder how do you guys name the migrations, without being affected by this

We're using epoch timestamp. After that, you can add any string you want: <timestamp>-<name>

So what is going on now?
https://github.com/sequelize/umzug/blob/ebb08d65d5365a77d54b68eb87e2cc1249cb727a/src/index.js#L457

Released at v3.0.0-beta.4!