Image rotation increase buffer size

Due to a rotation of the image in sharp, I used the rotate() function to fix the problem.
Link of that issue - https://github.com/lovell/sharp/issues/903
Have you searched for similar questions?
--yes
here is the code
Sharp(req.file.buffer)          .resize({ width: 1000 })          .webp()          .rotate()          .toBuffer()          .then((data) => {             console.log("buffer: ", data);             console.log("base64: ", data.toString("base64"));          })          .catch((err) => {             console.log("err: ", err);          });
image upload without rotation-
buffer:  <Buffer 52 49 46 46 86 4d 00 00 57 45 42 50 56 50 38 20 7a 4d 00 00 d0 10 02 9d 01 2a e8 03 c2 01 3e 6d 34 97 48 24 23 2a 25 a4 72 6a b1 40 0d 89 69 6e 50 15 ...
19804 more bytes>
the result for the same image with rotation -
buffer:  <Buffer 52 49 46 46 aa 28 01 00 57 45 42 50 56 50 38 20 9e 28 01 00 f0 29 08 9d 01 2a e8 03 ae 08 3e 6d 36 96 48 24 23 2a 29 a6 d4 99 11 40 0d 89 69 6d e7 99 ...
75904 more bytes>
This is not the case only of a single image others seem to behave like it.
I will be thankful if I get a working solution for the problem.

It looks like this code is both resizing and potentially converting the file format to WebP, so I would expect the output file size to be different.
Please can you provide a sample input image that you would expect not to increase in size after this series of transformations.

I am trying to say when the .rotate() is added to the image (after all other transformations) to display it correctly, which was the solution to https://github.com/lovell/sharp/issues/903 the buffer size is increasing since the file dimensions are the same buffer should not have to be increased and this behaviour seems odd to me.
I am using multer with sharp in nodejs(with express) to upload images.
here is an image I am working with-
https://user-images.githubusercontent.com/65894534/126290906-a66471c2-3bf5-4ac7-811b-6b4c8fa5272f.jpg

Processing the input image provided with the following code:
await sharp(input)
  .resize({ width: 1000 })
  .webp()
  .rotate()
  .toBuffer();

...produces a WebP image that is 1000x2222 and 75954 bytes.
Processing the input image provided with the following code:
await sharp(input)
  .resize({ width: 1000 })
  .webp()
  .toBuffer();

...produces a WebP image that is 1000x450 and 19854 bytes.
This seems like the expected behaviour, as the second image has smaller dimensions.

ok, thanks a lot I understood the mistake in my code, your package has helped a lot, I have just switched from jimp and this is the best support I ever got in a GitHub issue(I was surprised to see the owner of the repository himself answering).


Feature Request: Provide at least one version of the raw data files with UTF-8 encoding

First, I just wanted to say -- thanks so much for the incredible resource! Being able to access the raw database dumps that back ChEBI is fantastic.
As mentioned in the ChEBI FTP site README file:

The data has been exported in an ASCII format.

Due to this conversion, when using this exported data, it becomes difficult to display it in a user friendly way. For example, the definition of the deuterium atom (CHEBI:29237) is shown as follows on the ChEBI website:

The stable isotope of hydrogen with relative atomic mass 2.014102 and a natural abundance of 0.0115 atom percent (from Greek δευτερον, second).

However, in the flat tab separated data file dump, it is stored as:

The stable isotope of hydrogen with relative atomic mass 2.014102 and a natural abundance of 0.0115 atom percent (from Greek deltaepsilonupsilontauepsilonrhoomicronnu, second).

In general, the conversion from the unicode characters to the ASCII format is lossy -- there is no general way to convert back to the greek characters (that won't run into weird corner cases).
This could be solved by providing the flat file (or other dump files) in the UTF-8 format, which removes all ambiguity.
Is this something that could be considered for some future data release?
Thanks,
Nate

Thanks. We are currently redeveloping ChEBI and this is something we can definitely take a look at and fix in the new system.


Semantic label for lidar

Feature

Proposal

Alternatives

Additional context

Thanks for your great work!  Would you release LiDAR labels and 3D bounding box annotation for all scenes in the future?

Hi @ucaszyp
I may not fully understand the context of this issue. Could you elaborate this issue with more context?

What "great work" are you talking about?
When you request release LiDAR labels and 3D bounding box annotation for all scenes, what data are you talking about?

Hi @ucaszyp not sure I understand the issue, are you requesting labels for the DDAD dataset?
cc @RaresAmbrus @VitorGuizilini-TRI


Auditable needs cleanup

What are the schemas that are affected by the issue
/common/auditable
What are examples of products that are impacted by the issue
Anything using profile

From @lrosenthol on Slack:

Then let’s give it a better name and fix the field names

See also #222 – this might be a duplicate, but I can't tell, as @lrosenthol raised both issues.


chore: add variant tests

PR for add test for this variants :

BreakStatement
DebuggerStatement
EmptyStatement
LabeledStatement
Literal
RegExpLiteral
ReturnStatement

Tell me if there are any missing test cases, there may be some I don't know yet.

Thanks @Maestro31 (missed the PR)


Adjust readme requirements

Python3 / the tools are now required and DFU stuff is only needed if you're working with the firmware.
(WSL/Ubuntu VM requirement probably needs adjusting, but there aren't instructions for setting up a native toolchain... yet)

Good catch, thank you!


iOS tv-casting-app: Implement UDC API call

Problem
The iOS tv-casting-app currently does not send User directed commissioning requests to any commissioner
Proposed Solution
TBD

SDK supports this, but moving this to post V1 to implement the tooling.


Verify that addresses reported by identify are correct before inserting them in the DHT

Right now nodes tell us the addresses they are reachable from, and we immediately insert them in the DHT without any verification. This leads to a lot of unreachable addresses in the DHT, such as <IP_ADDRESS> or IPs behind NATs.
Instead, we should first attempt to connect to these addresses to see if they're reachable.
Note that connectivity is not transitive. For example if a node A is listening on a given address, this address might be reachable by B but not by C. If we implement this change, then C will not tell B about that address that A has, even though it would have been desirable. This is considered an acceptable trade-off.
On the implementation side, the good news is that Kademlia should automatically detect us reaching the node and add the address we connected to to the DHT, without having to call any method (although I'm not sure that this is the case, but I think that it should be the right logic).
In other words, all we have to do is call Swarm::dial for each address that a node reports through identity.
However, this is complicated by the "one connection per node" policy. If we connect to a node we're already connected to, we're going to drop the existing connection.
In other words, we first have to land https://github.com/libp2p/rust-libp2p/pull/1440
cc @romanb @arkpar

We have to be careful though not to introduce a DDOS vector by just dialling every reported address. Rate-limiting the dialling attempts may help a bit.

Even after #5099 there is still a problem to solve:
At the moment we insert nodes in the DHT as soon as we connect to them. We should instead only insert in the DHT nodes that belong to the correct chain.
In other words, we should wait until we know that node is on chain A before inserting it in the DHT of chain A.


Update CONTRIBUTING.md

Typo Fix: "at" -> "of"

Hi @vatsalya-vyas,
We really appreciate the time and effort you've put into your suggestion. Your input is valuable to us. However, after careful consideration, we've decided to continue using at in this particular context.
Thanks for your understanding and your contribution to the project. Looking forward to more interactions in the future!

Hi @vatsalya-vyas , check out the https://github.com/configu/configu/issues/266#issuecomment-1826770636 for details regarding the Hacktoberfest rewards


Error during install

--> Starting operation: Install Dovecot
    [chat.dill.moe] Reading package lists...
    [chat.dill.moe] Building dependency tree...
    [chat.dill.moe] Reading state information...
    [chat.dill.moe] Some packages could not be installed. This may mean that you have
    [chat.dill.moe] requested an impossible situation or if you are using the unstable
    [chat.dill.moe] distribution that some required packages have not yet been created
    [chat.dill.moe] or been moved out of Incoming.
    [chat.dill.moe] The following information may help to resolve the situation:
    [chat.dill.moe]
    [chat.dill.moe] The following packages have unmet dependencies:
    [chat.dill.moe]  dovecot-core : Depends: libc6 (>= 2.36) but 2.35-0ubuntu3.8 is to be installed
    [chat.dill.moe]                 Depends: libicu72 (>= 72.1~rc-1~) but it is not installable
    [chat.dill.moe]                 Depends: libzstd1 (>= 1.5.2) but 1.4.8+dfsg-3build1 is to be installed
    [chat.dill.moe]  dovecot-imapd : Depends: libzstd1 (>= 1.5.2) but 1.4.8+dfsg-3build1 is to be installed
    [chat.dill.moe] E: Unable to correct problems, you have held broken packages.
    [chat.dill.moe] Error: executed 0 commands

--> Disconnecting from hosts...
--> pyinfra error: No hosts remaining!
Traceback (most recent call last):
  File "/home/cross/Downloads/chatmail/venv/bin/cmdeploy", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/cross/Downloads/chatmail/cmdeploy/src/cmdeploy/cmdeploy.py", line 348, in main
    res = args.func(args, out, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cross/Downloads/chatmail/cmdeploy/src/cmdeploy/cmdeploy.py", line 89, in run_cmd
    retcode = out.check_call(cmd, env=env)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cross/Downloads/chatmail/cmdeploy/src/cmdeploy/cmdeploy.py", line 257, in check_call
    return subprocess.check_call(arg, shell=True, env=env)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/subprocess.py", line 413, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'pyinfra --ssh-user root chat.dill.moe /home/cross/Downloads/chatmail/cmdeploy/src/cmdeploy/deploy.py -y' returned non-zero exit status 1.

On ubuntu 22.04

Nevermind wrong ubuntu version you need to use 20.04 apparently

Hm, we actually recommend Debian 12 instead of Ubuntu... Isn't 20.04 EOL already? Not sure if using 20.04 is viable.

Please re-open if it doesn't work with Debian 12 :)


Warm transfer make second call

Warm transfer when we make second call the contact subscription not invoked for the onConnected and onConnecting events call back.
Unable to identify the second call is connecting or connected.
Is there any way to find the second call connection state.

We can use onRefresh to identify all the calls connected to the agent with the connection id.

Hi @sivac47123. Sorry this took so long for us to see. A warm transfer is modeled as an additional connection on the same contact, as indicated by the api name: contact.addConnection. Therefore, this new connection will reflect the states you are looking for, like going into the connecting and connected states. contact.onRefresh().getSingleActiveThirdPartyConnection().getState() will get you the state of the active warm transfer connection on each refresh to the contact object (refreshes will occur on connection state changes, as well as other changes to the contact object). I'm going to close this, as I think that answers your question. Please let me know if that is not satisfactory.


Build a custom telemetry and logging platform link is broken

The link specified under the section Azure Event Hubs/Build a custom telemetry and logging platform is broken.
See [Designing and Sizing a Global Scale Telemetry Platform on Azure Event Hubs](https://azure.microsoft.com/documentation/videos/build-2015-designing-and-sizing-a-global-scale-telemetry-platform-on-azure-event-Hubs/) for details.

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

ID: a0409994-bdd8-0c16-fae7-d60bce1b5a4b
Version Independent ID: ed86b02f-035c-d14d-4f22-d48d5aba17f4
Content: Monitoring and performance tuning - Azure SQL Database
Content Source: azure-sql/database/monitor-tune-overview.md
Service: sql-db-mi
Sub-service: performance
GitHub Login: @dimitri-furman
Microsoft Alias: dfurman

Hi @element824 ,
Thanks for the feedback. We will look into this and update as appropriate.
Hi @dimitri-furman,
Could you please look into this document request. Thanks

#reassign:williamDassafMSFT

#please-close


Error 1064 (SQL syntax) when pulling DB

grunt pull_db --target="staging"
Pulling database from 'staging' into Local
>> Creating DUMP of remote database
Password:
>> Database DUMP succesfully exported to:
>> backups/staging/20150625/16-16-06/db_backup.sql
>> Adapt the database: set the correct urls for the destination in the database.
>> Creating DUMP of local database
>> Database DUMP succesfully exported to:
>> backups/local/20150625/16-16-06/db_backup.sql
>> Importing DUMP into local database
ERROR 1064 (42000) at line 1: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'bash: LW9@@T^x: command not found
Usage: mysqldump [OPTIONS] database [tables]
O' at line 1
>> Database imported succesfully

Operations completed

Done, without errors.

.sql file says:
bash: LW9@@T^x: command not found
Usage: mysqldump [OPTIONS] database [tables]
OR     mysqldump [OPTIONS] --databases [OPTIONS] DB1 [DB2 DB3...]
OR     mysqldump [OPTIONS] --all-databases [OPTIONS]
For more options, use mysqldump --help

Same error here. Please help us fix this, there is no better availible DB and files solution for WP out there than yours. The others all want to make profit. Trellis is not as clever as you...
Best, Gabriel

Try my fix described here: https://github.com/webrain/grunt-wordpress-deploy/issues/27

Thanks! It does the job.

Thanks @herbie4 it's works


Workspace and varsub fixes.

I've fixed stuff so that the initially spawned workspace will take on the first defined user workspace name, or default to a predefined value. workspace_next_name has been renamed to workspace_init_name, because(as far as I know) the only time a workspace will be created without a name given explicitly is at startup time. All other workspace commands either specify a name, or refer to an existing workspace.
In addition, I've altered the subsitution so that it applies to all arguments, since there are many i3 commands which use multiple variables in them.

I've fixed stuff so that the initially spawned workspace will take on the first defined user workspace name, or default to a predefined value. workspace_next_name has been renamed to workspace_init_name, because(as far as I know) the only time a workspace will be created without a name given explicitly is at startup time. All other workspace commands either specify a name, or refer to an existing workspace.

That's not the behavior of i3. This also causes problems with multiple outputs. Not merging this one.

In addition, I've altered the subsitution so that it applies to all arguments, since there are many i3 commands which use multiple variables in them.

Can you make this change in a separate pull request?

That's my bad, forgot about the multiple output stuff. It still leaves the issue of initial workspaces being named incorrectly, meaning that if you switch out of your first workspace you can't get back to it. Do you have some idea in mind for a workaround for this?

Add workspace [name] to the end of your sway config to switch to that workspace on startup.

Ok, fair enough. Do you want me to open up a new PR or just add a commit which reverts the workspace changes?

Can you either open a new PR, or rebase so that the workspace changes are never present in the history?

Alright, I'll open up another PR once I've modified and rebased


istio-cni legacy args cleanup

Please provide a description of this PR:
See manifests/charts/istio-cni/templates/configmap-cni.yaml
https://github.com/istio/istio/blob/b30e8f559f07efb62cbccc67f2c0ee2b9a4b1e39/manifests/charts/istio-cni/templates/configmap-cni.yaml#L20-L32

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1569657c43aeef2567f83b3153f608af58d58489
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
1d6c6f97bcb206c57d47b41303875aa962f3546a
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-ambient

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
1d6c6f97bcb206c57d47b41303875aa962f3546a
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Escape prefixes and sufixes in formula widgets

Related issue: https://github.com/CartoDB/cartodb/issues/11726

LGTM!


Feature-wc-store-notice

turns off default store notice, which is not bootstrap, or compadible with bootstrap
add a new store notice function, only if enabled and notice string not empty
changes the navbar to sticky rather than fixed
removes a div between header and nav, easier to target
adds a close button, and removes :before icon from store notice.
NOTE: the child theme will override these changes - need to copy and paste woocommerce header into child theme to test

Thank you for this PR. But we should not do this that way, because the store notice is a temporary thing and touching the header.php is sensitive. We can use the hook <?php bs_after_primary(); ?> for that and add some additional CSS for positioning. Is this an idea?

We can use body class .woocommerce-demo-store .fixed-top to push content down like here https://github.com/bootscore/bootscore/blob/main/scss/bootscore/_admin_bar.scss.

Great I'll look into it I'm just remote for a few days but I'm will get
onto the PRs eventually
On Fri, 7 Oct 2022, 5:35 am Basti, @.***> wrote:

We can use body class .woocommerce-demo-store .fixed-top to push content
down like here
https://github.com/bootscore/bootscore/blob/main/scss/bootscore/_admin_bar.scss
.
—
Reply to this email directly, view it on GitHub
https://github.com/bootscore/bootscore/pull/280#issuecomment-1271420724,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARTWGA6XBTWDLH42DZ6JGGLWB74H7ANCNFSM6AAAAAAQ6GNB5Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @crftwrk, I have had a good look into this, here are the options (they all work but have limitations)

Original idea - inject with wp_body_open, change navbar to sticky-top, notice has a close button so can be removed if in the way. The sticky-top navbar is pushed down by the injected HTML, but goes back to the top of the page if the notice is closed.
Inject with bs_after_primary, or wp_footer (default wc behaviour)
a) Display with fixed-top, inject an extra body class and push fixed elements and off canvas down cannot have close button
b) Display with fixed-bottom
i) has a close button, gets in the way of the footer/body content
ii) has a close button, add padding to footer equal to height of store notice. When store notice is closed, this padding is still present in the footer. default WooCommerce behavior in store front theme
iii) No close button, gets in the way of the footer/body content, although the footer, and off canvas elements could be pushed up. Can have a close button, but not if your pushing items up with CSS

I think for UX considerations a close button is important in case the notice is in the way

I completely agree that alert must have a close button. What's about this:
We can create a new hook inside the fixed-top above the navbar
  <?php wp_body_open(); ?>

        <?php bs_top_nav(); ?>

This is the same result as your first commit, but keeps the fixed-top-class. BTW this hook can be used for any other stuff as well. So, why not?

Sounds great I'll give it a go
On Sat, 8 Oct 2022, 10:41 am Basti, @.***> wrote:

I completely agree that alert must have a close button. What's about this:
We can create a new hook inside the fixed-top above the navbar

    <?php bs_top_nav(); ?>

This is the same result as your first commit, but keeps the
fixed-top-class. BTW this hook can be used for any other stuff as well.
So, why not?
—
Reply to this email directly, view it on GitHub
https://github.com/bootscore/bootscore/pull/280#issuecomment-1272344874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARTWGAZGH74JLGKNMJ4I3K3WCGI25ANCNFSM6AAAAAAQ6GNB5Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Maybe I have a new idea. We had a php function for warníng if user browsed site by Internet Explorer. Now we replaced this function by Javascript. But function hooked below the

  <?php bootscore_ie_alert(); ?>

https://github.com/bootscore/bootscore/blob/652bc3c411b154bf96361b32db689b99f9e9b295/header-woocommerce.php, please scroll down to the bottom. The result was this:

We can use this position for the shop notice. Then we have no trouble with navbar classes or overlapping content.

Great,Is the hook still there, and should I rename it? If yes, what should
I rename to?
On Sat, 8 Oct 2022, 12:57 pm Basti, @.***> wrote:

Maybe I have a new idea. We had a php function for warníng if user browsed
site by Internet Explorer. Now we replaced this function by Javascript. But
function hooked below the

https://github.com/bootscore/bootscore/blob/652bc3c411b154bf96361b32db689b99f9e9b295/header-woocommerce.php,
please scroll down to the bottom. The result was this:
https://camo.githubusercontent.com/42905c708cc42224f8a9fe3e8da528d94798f946195d244ed413af3ffd4bace9/68747470733a2f2f626f6f7473636f72652e6d652f77702d636f6e74656e742f75706c6f6164732f323032322f30362f626f6f7473636f72652d69652d7761726e696e672d616c6572742e77656270
We can use this position for the shop notice. Then we have no trouble with
navbar classes or overlapping content.
—
Reply to this email directly, view it on GitHub
https://github.com/bootscore/bootscore/pull/280#issuecomment-1272368527,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARTWGAYLVITQX44YUX2PNPDWCGYYXANCNFSM6AAAAAAQ6GNB5Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

No, hook is removed. We have to create a new one, but it's simple and upgrade-safe.
header.php

    <?php bs_after_masthead(); ?>

functions.php
// Hook after #masthead
function bs_after_masthead() {
  do_action('bs_after_masthead');
}

Basic usage in child's functions.php
function my_function() {
  echo '

Hello Hook

';
}
add_action('bs_after_masthead', 'my_function', 5);

Result

Now we can use this hook for store notice. What do you think?

Looks great, thanks for the help, I'll have a go at it tonight.
On Sun, 9 Oct 2022, 4:10 am Basti, @.***> wrote:

No, hook is removed. We have to create a new one, but it's simple and
upgrade-safe.
header.php

<?php bs_after_masthead(); ?>

functions.php
// Hook after #mastheadfunction bs_after_masthead() {
do_action('bs_after_masthead');
}
Basic usage in child's functions.php
function my_function() {
echo 'Hello Hook';
}
add_action('bs_after_masthead', 'my_function', 5);
Result
[image: screenshot]
https://user-images.githubusercontent.com/51531217/194745261-05eeb796-a7e2-442f-ba04-18388e707a0d.png
Now we can use this hook for store notice. What do you think?
—
Reply to this email directly, view it on GitHub
https://github.com/bootscore/bootscore/pull/280#issuecomment-1272484573,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARTWGAYDUM5CIE5DPI4TBXDWCJ4WPANCNFSM6AAAAAAQ6GNB5Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you have a look? I had it working, but then I am now having issues with my WP development environment.

It is not working on my site. However, this PR has already conflicts with other (merged) PR's, there are 16 files changed. I suggest to close this PR to not mess up here anything and then start with two fresh PR's. One for the hook and the second one for the store notice. We will release <IP_ADDRESS> this weekend when Justin is ready to review. I would be happy if we can ship at least the hook by this release. Do you agree?

@crftwrk should I wait with the release for this PR?

@justinkruit let's release now and target this PR to next minor release. Bootstrap 5.2.2 has some very useful bugfixes.


Saving settings to local storage

Motivation and context

This PR provide saving user settings to local storage. Resolved #1209
How has this been tested?

Manually
Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG file
- [ ] I have updated the documentation accordingly
- [ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[x] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-0.05%) to 74.851% when pulling 9963bb2c90d50f6b9f7081b5234e27dc54c4e781 on dk/saving-settings into 7f214c92e8bd46ebd360775eaa8877421b9d68d6 on develop.

@TOsmanov, hi, can you adjust documentation according this new feature, please?


Pagination component does not render with a dotsLength < 2

Is this a bug report, a feature request, or a question?
Bug report
Have you followed the required steps before opening a bug report?

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
This is not a React Native bug and specific to this library.
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
The issue can be reproduced on both platforms for any version supported by the library.
Is the bug reproductible in a production environment (not a debug one)?
The bug is always reproducible.
Environment
Environment:
React: 16.9.0
React native: 0.61.5
react-native-snap-carousel: 3.8.4 (also master)
Target Platform:
Android (6.0)
iOS (10.0)
Expected Behavior
I expect the pagination component to render itself if I pass a value of 1 or higher to the dotsLength prop.
Actual Behavior
I noticed that my pagination component suddenly wasn't rendering after changing the number of items that I wanted the pagination component to show for.
After checking the code for the pagination component on master I found out that it does not render if the dotsLength passed is falsey or less than 2 (https://github.com/archriss/react-native-snap-carousel/blob/master/src/pagination/Pagination.js#L145).
Reproducible Demo
<Pagination
              dotsLength={1}
              activeDotIndex={0}
              containerStyle={{ backgroundColor: 'rgba(0, 0, 0, 0.75)' }}
              dotStyle={{
                  width: 10,
                  height: 10,
                  borderRadius: 5,
                  marginHorizontal: 8,
                  backgroundColor: 'rgba(255, 255, 255, 0.92)'
              }}
              inactiveDotOpacity={0.4}
              inactiveDotScale={0.6}
/>

Steps to Reproduce

Install the library.
Take the pagination example code (https://github.com/archriss/react-native-snap-carousel/blob/master/doc/PAGINATION.md)
Change the dotsLength to 1 as above.

@RemyLivewall  Pagination require more than 1 page to work. the source code contain the following check at the start of render method of the component.
  if (!dotsLength || dotsLength < 2) {             return false;         }

As I mentioned in the issue I found this behavior in the code. I understand that pagination does nothing with a single page, but that doesn't mean I don't want to have the option to show it for a single page.

Any solution?

Any solution?

Forking the library and changing the line is an option. In the application where we use the component it's not an issue so we haven't fixed it on our end.

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2

Here's a quick workaround
<Pagination
   ...
   dotsLength={data.length > 1 ? data.length : 3}
   activeDotIndex={data.length > 1 ? activeDot : 1}
   inactiveDotOpacity={data.length > 1 ? 0.5 : 0}
/>

You basically render three dots, set the middle one as active, and set the opacity of the rest to 0.
Hope this helps


Issue #SB-18213 Teachers data: Admin is getting View more and downloa…

…d button even if there is not data available

Verified


Add support for importing code from multiple samples in client lib readme

Closing as we're archiving this repository.


View Config not found for name BVLinearGradient

First try and this happened. I just copy paste your sample code.
"react": "16.3.1",
"react-native": "0.55.3",
"react-native-af-video-player": "^0.1.9",
OS : window 10. Test on Emulator Nexus 5 Api 23

@chanthea I made this mistake here too. What should I do?

@chanthea could u solve it?

@d41z
我这边解决好了.我 react-ntive link,fix it.
I'm done here. I react-ntive link,ok.

+1

react-native link:  it doesn't work for me. do u have another way to fix it.

Not work if you are using expo. Well i able to make this work by install all packages that mention you to link in read me.
Note : don't ever use react-native link randomly. It will link all available native packages in your project.

this issue should still be open, no linking works. not using expo.

Try to rebuild , no need to link for RN > 0.60 on android
on ios just make npx pod install


[otbn] Put instruction coding into documentation

The OTBN instruction encoding was done in https://github.com/lowRISC/opentitan/issues/2391, we now need to add it to the docs which are generated from insn.yml

That's actually a duplicate of https://github.com/lowRISC/opentitan/issues/2799.


[US] - Implement Figma Log in

As a
LiRA-Map User
I want
A nice Login page as designed in Figma
so that
I can log in

(Var allerede implementeret)


 tag with nodeintegration on require('electron') returns undefined

Electron version:0.36.8
Operating system:Windows 7

Acturally, require will return undefined when you require anything

Can you provide a reproducible example?

The  is loading an online page which I'm require in the console, I'll give you a example on Monday, Thank you~

I've found the problem, it's because the hosted page defined a require function, is there any way to change the electron require's name?

You can find a solution in http://electron.atom.io/docs/v0.36.8/faq/electron-faq/#i-can-not-use-jqueryrequirejsmeteorangularjs-in-electron.

Thank you very much~

@zcbenz Sorry to bother, but I have another question, what if the page in the webview navigated and the latter page(likely a logedin page) rewrited a require and I want to run some script to save node's require before the page's any JS starts to run? I've tried the webview's events but none of those can supply my needs.


New rule: requireUseStrict to ensure strict mode is used

Thanks I did forget to update the docs, I'll fix that!

@markelog ready for another review

Updated tests a bit, thanks


New icon?

Something that emphasizes the keyboard. Thoughts?

Hmm. I'd prefer that the dominant text on the icon was not 'cd ~'. What if the vertical split was the other way, with the keys taking 2/3 of the room, and the prompt taking 1/3, and the prompt was just the prompt? The extra row of keys could hold the 48?

Like this?

Hmm.. Still not sold. If I had to choose between the current icon and this one, I would pick the current one. Out of curiosity, what is it about the current icon that motivated you to suggest changing it?

Was just a bit bored and thought of the design, no big deal either way.


[LOC] Invalid link in an article

Repro steps:

Go to https://docs.microsoft.com/en-us/SkypeForBusiness/plan-your-deployment/enterprise-voice-solution/enterprise-voice-solution?bc=/SkypeForBusiness/breadcrumb/toc.json&toc=/SkypeForBusiness/toc.json
Search for the "Plan your Phone System in Office 365 (Cloud PBX) solution" link
click on the link

Expected results:
Link redirect to an existing page
Actual results:
Link redirects to a non existing page
GH article:
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/blob/live/Skype/SfbServer/plan-your-deployment/enterprise-voice-solution/enterprise-voice-solution.md

@OfficeGlobal Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@officedocsbot assign @scanum

@OfficeGlobal
Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


[SuSo-F-010] Update to grid layout with multiple sections (visually distinguishable)

Initialize with 9x9 grid
Separate development functions section (only visible in development mode) - can be hidden
New development function: allow to paste sudokus
Remove sidebar and move language selection into header
Introduce sections (white tiles on top of a grayish background) and split UI into multiple sections which are organized in some kind of grid layout style
Make headings less prominent

before this pull requst:
https://github.com/user-attachments/assets/69f4db38-754c-4f6e-8f1a-79b9aae33df5
after this pull request:
https://github.com/user-attachments/assets/596e235b-52bc-4dad-b8be-af87c23defe7


Jackson dependencies

Coverage decreased (-0.1%) to 53.283% when pulling be3c34bb50baa4031f707bfa2d16e2ac5e79be09 on compae:feature/scala-2.11-jackson into c033fda73f643c48b008ab48dcbe8ee930efc5a0 on Stratio:master.

:shipit:

A job will handle this PR ASAP on Jenkins

Rebase && testing in progress


[XCTestExpectation] Adding expectationForNotification

What's in this pull request?
This pull request adds the XCTestCase convenience constructor for XCTestExpectation expectationForNotification:object:handler:.
It also includes the XCNotificationExpectationHandler typealias file as well as tests to check expectationForNotification:object:handler: behavior.
https://bugs.swift.org/browse/SR-907
However, this pull request does not include the convenience constructor expectationForPredicate:evaluatedWithObject:handler: described in the ticket, as NSPredicate initializers have yet to be implemented.
Why merge this pull request?
This will provide a full Swift implementation of expectationForNotification:object:handler: that allows asynchronous testing for NSNotification.

This is awesome, thanks @yoonapps!! :100: I have a few nitpicks, but other than those this looks great to me!

@yoonapps This is fantastic! :heart_eyes:  Could you squash these three commits into one?
@mike-ferris-apple Could you ask @swift-ci to please test?

Awesome, yes I will address these reviews and squash the commits this evening! Thanks for the reviews.

@swift-ci please test

@modocache addressed issues

@modocache done :heavy_check_mark:

Ok well Github is not showing the extra line... but there is one :confused:

Yup, it's there! If no new line is present at the bottom of a file GitHub shows a symbol that looks like :no_entry_sign:.
Sorry @mike-ferris-apple, but could you ask @swift-ci to please test again? I think the updates to the pull request cancelled the previous CI job.
If CI passes on Linux (OS X is still broken, see #77), then I'll merge. :smile:

@swift-ci please test

:confounded:
@modocache The SwiftXCTestFunctionalTests build is passing for me locally. Although, I've seen the test fail once, and it was because the test expected // CHECK: in the new handler test main.swift to be the // CHECK: in the existing handler test main.swift. But I ran the test again and it was fine. Do you think this could be a file path issue?
// RUN: %{swiftc} %s -o %{built_tests_dir}/Handler
// RUN: %{built_tests_dir}/Handler > %t || true
// RUN: %{xctest_checker} %t %s

@yoonapps Nice catch!! It most certainly is a file path issue, one I've seen before--it's pretty nasty, but we need to use a unique file name for each test. You should be fine if you "namespace" your test file name, like so:
// RUN: %{swiftc} %s -o %{built_tests_dir}/Aynschronous-Notifications-Handler
// RUN: %{built_tests_dir}/Aynschronous-Notifications-Handler > %t || true
// RUN: %{xctest_checker} %t %s

Sorry about that. I'll create a JIRA issue so we can track fixing this.

Ok! I updated the paths for both the Notification Expectations and Handler main.swift files. Please work! :sob:

@mike-ferris-apple Mind asking @swift-ci to please test again? :sweat_smile:
I wonder when CI will be opened up to all committers... :thought_balloon:

Sorry!

Haha, don't be! I'm actually fine with merging this without testing again, but better safe than sorry.

@swift-ci please test

Awesome, thanks @yoonapps! Congratulations on completing your first starter task! :bouquet:

AWWWW YEEEEE
@modocache Thank you so much for all your help! Can't wait to do more. :fire:


Relative links don't format properly

Environment

Git repository URL: https://github.com/RobertsLab/sams-notebook
GitHub Pages hosted (if yes provide URL to site): https://robertslab.github.io/sams-notebook/

Expected behavior
Relative links in markdown files to other markdown files should render a properly formatted URL.
Steps to reproduce the behavior
Here's an example:
Post 1 URL: https://robertslab.github.io//sams-notebook/2015/05/05/rna-isolation-geoduck-gonad-in-paraffin-histology-blocks-3.html
Post 1 markdown: https://raw.githubusercontent.com/RobertsLab/sams-notebook/master/_posts/2015-05-05-rna-isolation-geoduck-gonad-in-paraffin-histology-blocks-3.markdown
Post 2 URL: https://robertslab.github.io/sams-notebook/2015/05/29/bioanalyzer-geoduck-gonad-rna-quality-assessment.html
Incorrect Post 2 URL: https://robertslab.github.io//sams-notebook/2015/05/05/2015-05-29-bioanalyzer-geoduck-gonad-rna-quality-assessment.md
The content of Post 1 URL contains a link (text in post is: 20180528) that should link to Post 2. However, the link ends up as Incorrect Post 2 URL, instead of Post 2 URL.
The link to Post 2 is coded as a markdown relative link: [20150528](2015-05-29-bioanalyzer-geoduck-gonad-rna-quality-assessment.md)
This relative link works perfectly when viewing the markdown on GitHub: https://github.com/RobertsLab/sams-notebook/blob/master/_posts/2015-05-05-rna-isolation-geoduck-gonad-in-paraffin-histology-blocks-3.md
Looking at the URL structure of Incorrect Post 2 URL, it is apparent that the permalink structure specified in my _config.yml file is being inserted when the URL is generated. According to the Relative links for GitHub pages explanation, it seems like the GitHub Pages backend should handle this situation.
That leads me to think there's an issue with the theme and the way it handles relative links.
Thanks for any guidance/suggestions on how to remedy!
Other

Thanks so much for the quick response and the explanation. A bit of a bummer that the GitHub Pages relative links capability can't function in tandem with the Jekyll tagging structure for links.


Custom Shader: Lighting artifacts are produced just outside of the area of effect of Spotlights and Omnilights

Godot version
Godot 4 rc1
System information
Windows 10, Vulkan, NVIDIA GTX 1660
Issue description
It's likely that I configured something incorrectly, but I'm using a custom lighting shader and objects that are outside of the area of effect of spotlights and omnilights get lighting artifacts:

This is the shader code:
shader_type spatial;
render_mode diffuse_toon, specular_toon;

group_uniforms Textures;

	uniform vec4 base_color : source_color = vec4(1.0, 1.0, 1.0, 1.0);

group_uniforms;

group_uniforms LightingAndShadows;

	uniform float attenuation_strength : hint_range(0.0, 1.0, 0.01) = 0.8;
	uniform float shadow_edge : hint_range(0, 2.0, 0.01) = 0;
	uniform float light_edge : hint_range(0.01, 2.0, 0.01) = 0.3;

group_uniforms;

void fragment() {
	ALBEDO = base_color.rgb;
}

void light() {
	// Attenuation
	float attenuation = mix(clamp(ATTENUATION, 0.0, 1.0), 1.0, 1.0 - attenuation_strength);

	// Diffuse
	float NdotL = dot(NORMAL, LIGHT);
	float diffuse = clamp(NdotL, 0.0, 1.0);
	diffuse = smoothstep(shadow_edge, light_edge, diffuse);

	// Light color
	vec3 light_color = normalize(LIGHT_COLOR);

	DIFFUSE_LIGHT += diffuse * attenuation * ALBEDO * light_color;
}

Steps to reproduce

Create an object that uses a material with the toon.gdshader
Add either a spot light or an omni light, place the object just outside of the area of effect of the light; where it shouldn't be light, but is close to the border.
This produces lighting artifacts

Minimal reproduction project
CustomShaderLightingBug.zip

This is due to Forward Clustering.

Related to https://github.com/godotengine/godot/issues/72996.

cc @lyuma

I think the documentation can be improved about light shaders. The issue here is that the shader is trying to get lights to affect objects well outside their respective ranges. You can see in the image in the OP that there is no overlap between the lights and the objects.
The ATTENTUATION variable includes the range attenuation which would naturally make the light's impact fall off as it reached the edge of the range. In the shader in the OP, ATTENTUATION is override and interpolated towards 1.0 making the light not fall off smoothly as it reaches the outer rim of its range. At a certain point the light just falls off as it exceeds the boundary of the cluster it is in and since ATTENUATION was ignored, there is no smooth falloff before that happens.
To be clear, the offending line is float attenuation = mix(clamp(ATTENUATION, 0.0, 1.0), 1.0, 1.0 - attenuation_strength);

Can confirm. Was my mistake.

At the end I just multiply my the default built-in ATTENUATION, and that fixed the issue. This is what I changed.
DIFFUSE_LIGHT += diffuse * ATTENUATION * ALBEDO * light_color;
Should I close this issue?

I would like to leave this open until a note is added to the documentation. We currently don't have enough information about light shaders so it is too easy to make this sort of mistake

Ok.
I was doing some more testing to correct this issue in the shader that I'm working on and found that it still occurs even if I'm not using attenuation. Am I doing something wrong? Is this expected behavior?

This is my new light() function:
void light() {
	float NDotL = dot(NORMAL, LIGHT);
	DIFFUSE_LIGHT += NDotL * ALBEDO;
}

@hidemat The issue will occur if you do not multiply by ATTENUATION, or at the very least ensure that your lighting approaches 0 as ATTENUATION approaches 0 (doesn't need to be linear necessarily).
If ATTENUATION is 0, Godot can choose to include or not include your light arbitrarily, and that arbitrary engine decision is what causes those clustering artifacts. That's why you need to respect ATTENUATION.

@hidemat The issue will occur if you do not multiply by ATTENUATION, or at the very least ensure that your lighting approaches 0 as ATTENUATION approaches 0 (doesn't need to be linear necessarily).
If ATTENUATION is 0, Godot can choose to include or not include your light arbitrarily, and that arbitrary engine decision is what causes those clustering artifacts. That's why you need to respect ATTENUATION.

Interesting. Ok I rewrote it to account for ATTENUATION when it is approaching 0, and that fixes the issue:

@hidemat That's just not going to work the way you describe it. As long as you are using Forward+ (Clustered), you'll need to ensure that the light function has no effect once ATTENUATION reaches 0. If not, you'll need to come up with a different approach, something like...

Perhaps clustered rendering just isn't for you. If you use the Mobile renderer, every light will run the light() function and you can make whatever arbitrary shading logic you want without artifacts.
Keep the clustered renedering, but increase the range for each light far enough that you can ensure that it is not relevant when attenuation hits 0 (or make the range so high that it's globally visible, and calculate the actual light attenuation yourself. This is similar to losing the clustering optimization since every object now sees every light, similar to the Mobile renderer.
Another option perhaps would be to use a spotlight with large angle like 180 so it's sort of a negative spotlight.
Perhaps negate the diffuse light, so you're subtracting the area it hits instead of adding, and then position your light sources so you can subtract them safely. Not sure about this exactly.
Keep using 3.x for now since it has a renderer that may be better suited to your specific needs of colored shadows. As 4.1 and 4.2 are developed, more options should open up to allow you to better customize the rendering pipeline to suit your needs.

Although it seems that simply enabling SDFGI produces some pretty results:

This is with multiplying by the default ATTENUATION value and not attempting to colorize the shadow.

@hidemat 5. Keep using 3.x for now since it has a renderer that may be better suited to your specific needs of colored shadows. As 4.1 and 4.2 are developed, more options should open up to allow you to better customize the rendering pipeline to suit your needs.

Is there something the godot team is already working on that should fix this issue or is it low priority ? Thanks.

Custom texture shadow is really a deal breaker for me using godot.
There is one last thing i want to try but there is nothing in the documentation that help me understand how the light function work.
What kind of caluclation is done because i don't think it's a standard multiply because when i grab my texture and divide it by the albedo and plug it into a mix3Scalar with an dot(light,normal)*attenuation the result is not the texture that i pluged in, there is always the effect of the fragment albedo in there.

Custom texture shadow is really a deal breaker for me using godot.

Are you referring to light projectors? This is already supported in Godot 4 using the Projector property in OmniLight3D and SpotLight3D.

Custom texture shadow is really a deal breaker for me using godot.

Are you referring to light projector textures (aka. light cookies)? This is already supported in Godot 4 using the Projector property in OmniLight3D and SpotLight3D. You must enable shadows on the light for it to work though.

No i updated my post with a picture so you can better understand what is was getting to.

Here is a more flagrant exemple.

Here is the fragment shader with a pure blue as a color constant, on of the quirk of this technique is that you need to add a value to all 3 channel or the division will not work.
And here is the shader graph with the resulting color.

Keep in mind the the color inputed is pure greenand there are blue in it now.


Dynamically disabling Radio does not work.

Description
I am trying to dynamically disable a radio when I change the state but the attribute of the Radio does not update the component.
CodeSandbox/Snack link
https://snack.expo.dev/@dashty/honest-toffee
const [disabled, setDisabled] = React.useState(false);

 <FormControl isInvalid>
        <FormControl.Label _text={{
        fontSize: "lg",
        bold: true
      }}>
          Select Prize
        </FormControl.Label>
        <Radio.Group name="exampleGroup" accessibilityLabel="select prize" defaultValue={groupValue} onChange={value => {
        setDisabled(true)
      }}>
          <Radio value="1" my="1" isDisabled={disabled}>
            First
          </Radio>
        </Radio.Group>
        <FormControl.ErrorMessage leftIcon={<WarningOutlineIcon size="xs" />}>
          You must select a Prize.
        </FormControl.ErrorMessage>
        <Text>Disabled: {disabled ? "True" : "False"} </Text>
      </FormControl>

Steps to reproduce
No response
NativeBase Version
3.3.7
Platform

[ ] Android
[ ] CRA
[ ] Expo
[X] iOS
[ ] Next

@dashty94 Thanks for reporting! We've fixed the issue in pre-release 3.4.0-rc.1. Hope to release 3.4.0 soon.

This issue has been fixed in version 3.4.0 or later.


How to set the cell items size to be equal?

How to set the cell items size to be equal?
How customization the cell size to be equal,I find the RACollectionViewReorderableTripletLayout must set a large size,and the other cell be a half of  large size,can there be equal?

@lin493369 Were you able to create cell items of equal sizes? I am also looking to do the same.


Switch to Android Embedding v2

❯ flutter pub get                                                                                [11:57:00]
Running "flutter pub get" in flutter_clean_calendar...              1,1s
Running "flutter pub get" in example...                             1,0s
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Warning
──────────────────────────────────────────────────────────────────────────────
Your Flutter application is created using an older version of the Android
embedding. It's being deprecated in favor of Android embedding v2. Follow the
steps at

https://flutter.dev/go/android-project-migration

to migrate your project.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

Migration is done. Terminal prints now:
❯ flutter pub get                                                                                [12:05:09]
Running "flutter pub get" in flutter_clean_calendar...              0,3s
Running "flutter pub get" in example...                             0,4s


Tax calculations incorrectly handle quantities

Tax calculations for quantities > 1 are giving unexpected results. The working assumption is that BC already multiplies the price by quantity so this line is unnecessary:
https://github.com/hatertron3000/demo-tax-provider/blob/a83b2565ef6e7408d92e5fa2aaf23b81c7adbcc0/pages/api/tax/estimate.ts#L68

Confirmed, BC is already accounting for quantity in the item price. Consider the following two items:
                {
                    "id": "f0e62a4c72c6575031b5353dc5063a53",
                    "item_code": "ABS",
                    "name": "[Sample] Able Brewing System",
                    "price": {
                        "amount": 675,
                        "tax_inclusive": true
                    },
                    "quantity": 3,
                    "tax_class": {
                        "code": "",
                        "class_id": "0",
                        "name": "Default Tax Class"
                    },
                    "tax_exempt": false,
                    "type": "item",
                    "wrapping": null
                }

                {
    "id": "2f5bd0216c28e1890a9934dab41fc664",
                    "item_code": "ABS",
                    "name": "[Sample] Able Brewing System",
                    "price": {
                        "amount": 450,
                        "tax_inclusive": false
                    },
                    "quantity": 2,
                    "tax_class": {
                        "code": "",
                        "class_id": "0",
                        "name": "Default Tax Class"
                    },
                    "tax_exempt": false,
                    "type": "item",
                    "wrapping": null
                },

Note that the price.amount is quantity * 225 (the product's price). Also note that this behavior is the same whether price.tax_inclusive is true or false.


Add CocoaPods support

Hello!
This PR adds support for installation via CocoaPods.

Thank you Stephen! 👍


Update runtime.py: change<EMAIL_ADDRESS>to<EMAIL_ADDRESS>(merge to my develop to test ci)

My code has been giving me an error in my streamlit.app "Manage app" side panel lately because the email of the user accessing it is not on the approved list of emails (so my app doesn't know how to act). The problematic email accessing it is<EMAIL_ADDRESS>This commit addresses the only instance of non-test code that has<EMAIL_ADDRESS>so the problem must be from this. Presumably due to the script health check https://github.com/streamlit/streamlit/blob/621b802fc07d17a657d9e75bd913303adb335491/lib/streamlit/web/server/server.py#L340
So, anyway, I think we should change this example email to<EMAIL_ADDRESS>for the same reason the regular user email is (see https://github.com/streamlit/streamlit/pull/7219). example.com is a reserved domain for such purposes (https://www.rfc-editor.org/rfc/rfc2606.html#section-3), unlike test.com, which is just a regular domain some random person owns, I guess. If it's somehow useful to distinguish between this "user" and the regular local user, then<EMAIL_ADDRESS>could perhaps be used instead.
This code change may fail ci, since some of the test code seems to still have<EMAIL_ADDRESS>in it, in which case I will notice and then go fix those failures.
Describe your changes
GitHub Issue Link (if applicable)
Testing Plan

Explanation of why no additional tests are needed
Unit Tests (JS and/or Python)
E2E Tests
Any manual testing needed?

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

some of the ci failed to even run?


Failed to set other environments for Azure connector

Describe the bug

Failed to change the environment for Azure connector
Steps to reproduce
https://github.com/seal-io/walrus/assets/26084756/2d62ae47-c74a-4596-bae5-315ee1d5d46b
Expected behavior

Environment

Walrus version: main 0a3779654c43dfb1a5d3299c4862502a909f2878
UI version: dev-c3de3bf

Additional context
Add any other context about the problem here.

Environment

Walrus version: main 30ceb5ac9dcaade04505b64a3f6e6ef3bc1c80b2
UI version: dev-1305100

Test Result: pass


Support for FHIR version R4B

Describe the solution you'd like
Support FHIR version R4B - http://hl7.org/fhir/2021Mar/ballot-intro.html

[ ] Base it off the branch r4/master, name the new branch r4b/master
[ ] Upgrade to firely-net-sdk 3.2 R4B (or later) pre-requisite: #379
[ ] Support the generic search parameter _source

Blocked by https://github.com/FirelyTeam/firely-net-sdk/issues/1746

Any reason why support for R4B has been blocked? :-)

@pietew It's not really blocked anymore except for our integration tests failing (consult PR #508) which I haven't had time to investigate.
One option is to verify the branch in #508 locally, and if it works satisfactory disable the integration tests for the time being for R4B.
If anyone wants to look into verifying R4B before I have the time to get back to it they are welcome to do that, and I will gladly review it.


Add missing Javadoc + update README

Mind taking another look, especially at the new section "Markup Selectors and Out Of Band Swaps".


How can I get *.summary.final.org_sents and *.gold files?

Hi,
I tried your code and got an error after the first epoch:
...
STEP A: Epoch 1 : Covered 83540/83564 : Minibatch CE Loss= 19.836004, Minibatch Accuracy= 0.878014
STEP A: Epoch 1 : Covered 83560/83564 : Minibatch CE Loss= 10.907549, Minibatch Accuracy= 0.878505
STEP A: Epoch 1 : Saving model after epoch completion
STEP A: Epoch 1 : Performance on the validation data
STEP A: Epoch 1 : Validation (1220) accuracy= 0.852109
STEP A: Epoch 1 : Writing final validation summaries
Writing predictions and final summaries ...
Traceback (most recent call last):
  File "document_summarizer_gpu2.py", line 252, in <module>
    tf.app.run()
  File "/s1_md0/v-botsh/anaconda/tf0.10_py36/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 30, in run
    sys.exit(main(sys.argv))
  File "document_summarizer_gpu2.py", line 245, in main
    train()
  File "document_summarizer_gpu2.py", line 169, in train
    validation_data.write_prediction_summaries(validation_logits, "step-a.model.ckpt.epoch-"+str(epoch), session=sess)
  File "/s1_md0/v-botsh/Research/Repo/sidenet/data_utils.py", line 61, in write_prediction_summaries
    self.process_predictions_rankedtopthree(modelname+"."+self.data_type)
  File "/s1_md0/v-botsh/Research/Repo/sidenet/data_utils.py", line 116, in process_predictions_rankedtopthree
    docsents = open(sent_filename).readlines()
IOError: [Errno 2] No such file or directory: 'CNN-DailyMail/JP-Hermann/cnn/validation-sent/8f6b39e6c63b0ae3546cdfeb8209693f292b060e.summary.final.org_sents'

I only have "*.summary.final" files instead of ".summary.final.org_sents"
Where can I get those org_sents files?
besides, I noticed that "*.gold" files are also important for rouge evaluation. How can I get these files?
Thanks!

They can be found in "Dataset with sideinfo: http://kinloch.inf.ed.ac.uk/public/cnn-dm-sideinfo-data.zip". But, I have also updated the Readme page to directly download those preprocessed files.

@shashiongithub  Thank you for your response.
The file you provided only contains "*.summary.final" files
*.summary.final.org_sents files and *.gold files are not included in the zip.
Or is there any script that I can use to generate .gold and .summary.final.org_sent files from these summary.final files?
Thank you :)

Ohh thanks, I have uploaded them as well.
Shashi


mgmt storage, add test for lcm blob actions and bug fix

Description
test current SDK support for LCM actions with premium account.
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

API change check
API changes are not detected in this pull request.


Questions about training

Hi, I read your paper and I have some questions about it.
Question 1) theoretical background of your actor-critic learning
In your paper, you experimented REINFORCE and actor-critic. As far as I know, naive REINFORCE and actor-critic is on-policy RL method. So, it needs to simulate the environment using action sampled from the policy. But, there is no environment that can simulate dialogue. Also, your paper and your code seem that you train your model without simulation. It seems off-policy learning.
I think that, in this setting, policy can not learn the influence derived from the action sampled from the policy
So, my question is what is the theoretical background of training REINFORCE and actor-critic in this way that doesn't use simulation.
Question 2) In equation 10, Why do you use the immediate reward(R) only?
To the best of my knowledge, when you use policy gradient using REINFORCE or Actor-critic, you should use accumulated expected return from a state to the end of the scenario which like an accumulated reward from Monte-Carlo sampling or TD learning (r+v(s_{T+1})) - v(s_{T}).
In equation 10, however, you just use immediate reward R, and also your code doesn't contain accumulation or TD.
Is there any theoretical background on it?
I have little knowledge of RL, so I can be wrong. If my question is wrong, please tell me.

Thanks for your interest to our work.
The theoretical background to answers of your questions is well explained in http://rail.eecs.berkeley.edu/deeprlcourse/
Best,

Thanks for the reply and the link!
I checked your link,  I watched this video and also this slide.
In the lecture, when they use actor-critic, they also use 'rewards to go'. My question 2 is just about it. In equation 10 in your paper, your reward function R is the immediate reward. I wonder that why using only immediate R is OK. Is it just your suggestion or Is there any paper using immediate reward or Am I missing something?


unhandled ZLE widget error since 0.5.0

Hi guys,
since version 0.5.0, and zsh 5.3.1 on Arch 4.8.13,
an error appears everytime I launch zsh (login,terminal..)
This one :
zsh-syntax-highlighting: unhandled ZLE widget 'history-substring-search-up'
zsh-syntax-highlighting: unhandled ZLE widget 'history-substring-search-down'

With an empty zshrc, error still appears.
Any idea ? thx :)

Hi guys,
since version 5.0.0, and zsh 5.3.1 on Arch 4.8.13,

ITYM 0.5.0, but thanks for giving these details up front.

an error appears everytime I launch zsh (login,terminal..)
This one :
zsh-syntax-highlighting: unhandled ZLE widget 'history-substring-search-up'
zsh-syntax-highlighting: unhandled ZLE widget 'history-substring-search-down'

With an empty zshrc, error still appears.

That's be odd.  Launch zsh as zsh -x to see which startup files it
reads (and what they do to these two widgets).  See also #409.

I'll try when i'm back home. Thx !

I searched a little, and found this :
bindkey "$terminfo[kcuu1]" history-substring-search-up
I think that could come from here. Any idea about it ? is it needed ?
(i've this line in both /etc/zsh/zshrc and ~/.zshrc)
Thx

I searched a little, and found this :
bindkey "$terminfo[kcuu1]" history-substring-search-up
I think that could come from here. Any idea about it ?

None at all.  Are you sure you didn't put it there?  It's not part of
upstream and Arch doesn't usually patch its packages.

is it needed ?

It probably breaks your up-arrow key, since it maps it to a nonexistent
widget.  See how that key behaves with/without that binding.  Binding to
undefined-key is also an option.
Unless you install history-substring-search-up after sourcing z-sy-h
in your dotfiles.  In this case, just put z-sy-h last.

(i've this line in both /etc/zsh/zshrc and ~/.zshrc)

Ah, because you emptied your zshrc, which is not the same as zsh -f.

wow, forgot zsh -f
With it, it works perfectly.
So the problem comes from the bind. I'll remove it, since there is no difference with or without it.
You can close issue, Mea Culpa !

I ran into this as well after I installed https://github.com/zsh-users/zsh-history-substring-search. The problem was the order in which I was defining key bindings and sourcing the two plugins.
1. bind keys first, recommended order of sources

history search works properly and has syntax highlighting
syntax highlighting displays errors

bindkey '\e[A' history-substring-search-up
bindkey '\e[B' history-substring-search-down

source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh
source /usr/local/share/zsh-history-substring-search/zsh-history-substring-search.zsh

2. bind keys first, switch order of sources

history search works, but doesn't have syntax highlighting
syntax highlighting doesn't display any errors

bindkey '\e[A' history-substring-search-up
bindkey '\e[B' history-substring-search-down

source /usr/local/share/zsh-history-substring-search/zsh-history-substring-search.zsh
source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

3. bind keys last, recommended order of sources

history search works and has syntax highlighting
syntax highlighting doesn't report any errors

source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh
source /usr/local/share/zsh-history-substring-search/zsh-history-substring-search.zsh

bindkey '\e[A' history-substring-search-up
bindkey '\e[B' history-substring-search-down

So, 3 works, though I'd prefer to have my key bindings first, but it's not that big of a deal.

I ran into this as well after I installed https://github.com/zsh-users/zsh-history-substring-search. The problem was the order in which I was defining key bindings and sourcing the two plugins.

Thanks for the details @katylava!

1. bind keys first, recommended order of sources

history search works properly and has syntax highlighting
syntax highlighting displays errors

bindkey '\e[A' history-substring-search-up
bindkey '\e[B' history-substring-search-down

source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

At this point, history-substring-search-* are partially initialized, they have been referenced by bindkey but not yet created with zle -N.  I remember fixing this, are you using latest master of z-sy-h?

source /usr/local/share/zsh-history-substring-search/zsh-history-substring-search.zsh

**2. bind keys first, switch order of sources**

- history search works, but doesn't have syntax highlighting
- syntax highlighting doesn't display any errors

bindkey '\e[A' history-substring-search-up
bindkey '\e[B' history-substring-search-down
source /usr/local/share/zsh-history-substring-search/zsh-history-substring-search.zsh
source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

Here, by the time z-sy-h runs, all widgets are properly defined.  What version of zsh are you using?  z-sy-h works better with more recent zsh's.

3. bind keys last, recommended order of sources

history search works and has syntax highlighting
syntax highlighting doesn't report any errors

source /usr/local/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh
source /usr/local/share/zsh-history-substring-search/zsh-history-substring-search.zsh

bindkey '\e[A' history-substring-search-up
bindkey '\e[B' history-substring-search-down

So, 3 works, though I'd prefer to have my key bindings first, but it's not that big of a deal.

Try your option (1) but also add zle -N history-substring-search-up; zle -N history-substring-search-down before either source command?

I remember fixing this, are you using latest master of z-sy-h?

Using 0.5.0 installed with homebrew.

What version of zsh are you using?

zsh 5.2 (x86_64-apple-darwin16.0) ... which surprised me as I thought I had installed it with homebrew.

Try your option (1) but also add zle -N history-substring-search-up; zle -N history-substring-search-down before either source command?

That works, thank you!

On Mac OS Catalina with Zsh 5.7.1 everything installed through homebrew, I had to put the bindkeys at the very end as @katylava suggested

@katylava @erdaltoprak thanks - that did the trick on Arch Linux, as well (zsh 5.8).


The vehicle fails to initialize properly in Autoware

After running the launch file, the vehicle is displayed normally in CARLA and the image topics are subscribed correctly, but the vehicle fails to initialize properly in Autoware.

My branch is galactic.


"Giant" Button not Appearing

💬 Question
I'm trying to add a simple button using the Facebook Icon but it's not appearing.
I was using this as a reference for designing a page, but I can't seem to get the social icons setup. I also setup my project using npx react-native init MyApp --template @ui-kitten/template-js so I think most things are setup correctly.
I marked this as a question because I do not think this is a bug, just me being a slow learner.
UI Kitten and Eva version

Package
Version

@eva-design/eva
2.0.0-alpha.1

@ui-kitten/components
5.0.0-alpha.1

You run v5, but the code you're trying to run is v4. See the docs or what's changed


Refactor Idea: Replace usages with const declaration

Replace usages with const declaration, when the cursor is at the const declaration.

Nice idea! For my understanding, how would this be different from inline variable?

It is the opposite direction: I want to replace all duplicate expressions with the const at the cursor.

Ah I see, this would be useful!
In the meantime, you can achieve the same by combining the following refactorings:

Extract the right hand side of the const into a new variable (here: top + height - checkBoxHeight - margin)
Inline the original const (here: bottomBoundary)
Rename the new variable to the name of the original const (here: to bottomBoundary)

Just realized that one variant of the refactoring already exists, you should be able to go over top + height - checkboxHeight - margin and use "Replace with existing variable": https://p42.ai/documentation/code-assist/replace-expression-with-existing-variable


Remove version: Freeplane.Freeplane version 1.11.7

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


🐛 correct detection of boundaries with NaNs

What the pull request does
Fixes #31
Type of change

[x] :bug: Bug fix (non-breaking change which fixes an issue)
[x] :book: This change requires a documentation update

Checklist

[x] The changes are tested
[x] The changes do not modify the behavior of the previously existing functions

Codecov Report

Merging #34 (f9258c5) into main (467091d) will decrease coverage by 0.88%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
- Coverage   95.00%   94.11%   -0.89%
==========================================
  Files           7        7
  Lines         100      102       +2
==========================================
+ Hits           95       96       +1
- Misses          5        6       +1

Flag
Coverage Δ

unittests
94.11% <50.00%> (-0.89%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/lib/boundaries.jl
50.00% <50.00%> (ø)

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 467091d...f9258c5. Read the comment docs.


addEvents without rerender?

Anyway? Sorry if I'm not seeing it.

Can't do that at the moment! It's a perfectly reasonable use case though. We can add an optional second silent argument to the addEvents function that will skip the rendering step.
I can't do this right at the moment but if you want to submit a pull request that would be awesome! Should be a real simple fix, lines 1483 and 1495 will be of interest. If you do a PR you should add the option to setEvents and removeEvents as well :grin:

Ah! No worries and makes sense. Was just trying to animate new events being added to my calendar without the existing ones "blinking" / repeating the CSS animation . Good to know it's an easy fix.
After I wrap everything else up I'll revisit and send a pull

@whatnickcodes This was now added in 1.4.2.


fix appinfo handling

currently there is a appinfo.json file.
this is extra work as i have to maintian / change information at to places -
(package.json contains the version info too...)

some research later:
Handling process.env - Caveats
you can just define something in the quasar.conf.js file:
const packageInfo = require('./package.json')
// ...
  build: {
      // ....
      env: {
          appinfo: {
              name: packageInfo.name,
              version: packageInfo.version,
              productName: packageInfo.productName,
              description: packageInfo.description,
              projectUrl: packageInfo.projectUrl,
              previewUrl: packageInfo.previewUrl,
          },
      },
    },

and use it just this way:
AboutPage.vue
<script>
export default {
    name: 'AboutPage',
    data () {
        // https://quasar.dev/quasar-cli-webpack/handling-process-env#caveats
        // console.log(process.env.TEST)
        return {
            appinfo: process.env.appinfo,
        }
    }
}
</script>


Aging report?

Hi,
Also a fan here.  Also sorry for abusing the issue system.
I'm looking for a report that uses Starting Date, not Due Date, so I can see how old each card is, grouping them into buckets (e.g., 0-30 days old, 31-60 days old, etc.).  I thought I might be able to make the Gantt Chart do it but no luck.  Is this something I'd have to write or is it already in your excellent .pbit somewhere?
Thanks.
Jeff

Hi Jeff,
No problem.  It's not in the .pbit (yet), but I've just written this code which will add a calculated column called Ageing Bucket.  It's dax so the column will be recalculated on each refresh. (the code doesn't need formatting, but better if you do!)
Ageing Bucket =
// Add a calculated column to calculate the age of each task
VAR age = TODAY() - Planner[Start Date]
VAR bucket =  IF (Planner[Start Date],
IF (
age <= 30, "0-30",
IF (
age <= 60, "31-60",
IF (
age <= 90, "61-90",
IF (
age <= 120, "90-120", "120+"
)
)
)
), BLANK()
)
RETURN bucket
I've attached a .pbix file with the code in, and added another page which should get you started with some ideas for visuals.

Planner V7.6.zip

Brilliant.  Perfect.  Exactly.  Thank you!
I'll offer a little syntactic sweetening:  The buckets will align properly if you add leading spaces to the name.  The spaces are crushed out at render-time so it looks smooth.  I also prefer "not started" to BLANK(), and that can probably be internationalized, too:
VAR bucket = IF (Planner[Start Date],
IF (
age <= 30, "  0-30",
IF (
age <= 60, " 31-60",
IF (
age <= 90, " 61-90",
IF (
age <= 120, " 90-120", "120+"
)
)
)
), "not started"
)

When you publish this page I suppose we should close this issue.  For the optics of the thing.  :^)

😊
From: Jeff Grossman @.>
Sent: 03 August 2021 19:42
To: ben-howard/PowerBI-Planner @.>
Cc: Ben Howard (AP) @.>; Comment @.>
Subject: Re: [ben-howard/PowerBI-Planner] Aging report? (#6)
When you publish this page I suppose we should close this issue. For the optics of the thing. :^)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ben-howard/PowerBI-Planner/issues/6#issuecomment-892077064, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AI26AMTFTODSCQ6X6BSRKKTT3A2BFANCNFSM5BNRBE2A.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

Thanks Geoff.  Updated and everything published.
From: Jeff Grossman @.>
Sent: 03 August 2021 19:40
To: ben-howard/PowerBI-Planner @.>
Cc: Ben Howard (AP) @.>; Comment @.>
Subject: Re: [ben-howard/PowerBI-Planner] Aging report? (#6)
Brilliant. Perfect. Exactly. Thank you!
I'll offer a little syntactic sweetening: The buckets will align properly if you add leading spaces to the name. The spaces are crushed out at render-time so it looks smooth. I also prefer "not started" to BLANK(), and that can probably be internationalized, too:
VAR bucket = IF (Planner[Start Date],
IF (
age <= 30, " 0-30",
IF (
age <= 60, " 31-60",
IF (
age <= 90, " 61-90",
IF (
age <= 120, " 90-120", "120+"
)
)
)
), "not started"
)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ben-howard/PowerBI-Planner/issues/6#issuecomment-892075409, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AI26AMVHJJQ5OLWRCDACACDT3AZW3ANCNFSM5BNRBE2A.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.


OpTestOpenBMC: Download dumps to logdir

Download the openbmc dumps to logdir to allow for running
--run testcases.testRestAPI.HostOff.test_obmc_download_dumps
(possibly as an additional component of automation).
Additional issue opened on openbmc:
BMC Capture journal ->
https://github.com/openbmc/phosphor-debug-collector/issues/6
Signed-off-by: Deb McLemore<EMAIL_ADDRESS>

retest this please


Re-loading the Wasm module

I do not get the section about memory usage. Could you please provide an example how can one reload the wasm module to clear the memory? So a complete example of loading the module, computing a hash, and then clearing the memory?

I think I figured it out. But it is not about reloading the module and more about re-instantiatating a module? I made the following code:
const simdModule = await WebAssembly.compileStreaming(fetch(simdURL))
const noSimdModule = await WebAssembly.compileStreaming(fetch(noSimdURL))

async function getSIMD(importObject: WebAssembly.Imports) {
  return { instance: await WebAssembly.instantiate(simdModule, importObject), module: simdModule }
}
 async function getNoSIMD(importObject: WebAssembly.Imports) {
  return { instance: await WebAssembly.instantiate(noSimdModule, importObject), module: noSimdModule }
}

I can then call setupWasm every time I need to hash and my understanding is that after that instance if garbage collected, memory and everything is freed?
From what I see you need only instance, so not sure why you require loaded to return WebAssemblyInstantiatedSource which returns both the instance and the module? Now I have to attach module myself in the functions above, because WebAssembly.instantiate with module as a parameter does not return it. I think this is a bad API design for WebAssembly.instantiate. It would simple to just pass module through and have same return value as when you pass a byte array to WebAssembly.instantiate. Anyway, I do not think you should be requiring WebAssemblyInstantiatedSource, but the instance itself?

Hi, reloading the module simply means calling the default export (loadWasm()) again, and getting a new computeHash instance as a result. You also need to make sure you are not keeping around the reference to another, previously returned computeHash function.


fix: docker entry files are now managed

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Regression: Project does not work in 23.2.3 but does in 23.2.0, osgi error

This is an issue in 23.2.3 apparently at some point after 23.2.1
git clone https://github.com/vaadin/vaadin-flow-karaf-example.git
# check that pom.xml version is 23.2.3
cd vaadin-flow-karaf-example
mvn install
mvn -pl main-ui install -Prun

The error is
org.apache.felix.resolver.reason.ReasonException: Unable to resolve root: missing requirement [root] osgi.identity; osgi.identity=main-ui; type=karaf.feature; version="[1.0.0.SNAPSHOT,1.0.0.SNAPSHOT]"; filter:="(&(osgi.identity=main-ui)(type=karaf.feature)(version>=1.0.0.SNAPSHOT)(version<=1.0.0.SNAPSHOT))" [caused by: Unable to resolve main-ui/1.0.0.SNAPSHOT: missing requirement [main-ui/1.0.0.SNAPSHOT] osgi.identity; osgi.identity=com.vaadin.flow.server; type=osgi.bundle; version="[23.2.3,23.2.3]"; resolution:=mandatory [caused by: Unable to resolve com.vaadin.flow.server/23.2.3: missing requirement [com.vaadin.flow.server/23.2.3] osgi.wiring.package; filter:="(&(osgi.wiring.package=com.vaadin.pro.licensechecker)(version>=1.10.0)(!(version>=2.0.0)))"]]
	at org.apache.felix.resolver.Candidates$MissingRequirementError.toException(Candidates.java:1341)
	at org.apache.felix.resolver.ResolverImpl.doResolve(ResolverImpl.java:433)
	at org.apache.felix.resolver.ResolverImpl.resolve(ResolverImpl.java:420)
	at org.apache.felix.resolver.ResolverImpl.resolve(ResolverImpl.java:374)
	at org.apache.karaf.features.internal.region.SubsystemResolver.resolve(SubsystemResolver.java:256)
	at org.apache.karaf.features.internal.service.Deployer.deploy(Deployer.java:399)
	at org.apache.karaf.features.internal.service.FeaturesServiceImpl.doProvision(FeaturesServiceImpl.java:1069)
	at org.apache.karaf.features.internal.service.FeaturesServiceImpl.lambda$doProvisionInThread$13(FeaturesServiceImpl.java:1004)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: org.apache.felix.resolver.reason.ReasonException: Unable to resolve main-ui/1.0.0.SNAPSHOT: missing requirement [main-ui/1.0.0.SNAPSHOT] osgi.identity; osgi.identity=com.vaadin.flow.server; type=osgi.bundle; version="[23.2.3,23.2.3]"; resolution:=mandatory [caused by: Unable to resolve com.vaadin.flow.server/23.2.3: missing requirement [com.vaadin.flow.server/23.2.3] osgi.wiring.package; filter:="(&(osgi.wiring.package=com.vaadin.pro.licensechecker)(version>=1.10.0)(!(version>=2.0.0)))"]
	at org.apache.felix.resolver.Candidates$MissingRequirementError.toException(Candidates.java:1341)
	... 12 more
Caused by: org.apache.felix.resolver.reason.ReasonException: Unable to resolve com.vaadin.flow.server/23.2.3: missing requirement [com.vaadin.flow.server/23.2.3] osgi.wiring.package; filter:="(&(osgi.wiring.package=com.vaadin.pro.licensechecker)(version>=1.10.0)(!(version>=2.0.0)))"
	at org.apache.felix.resolver.Candidates$MissingRequirementError.toException(Candidates.java:1341)
	... 13 more

This is BTW a regression since it works fine in 23.2.0
mvn -B -q versions:set-property -Dproperty=vaadin.version -DnewVersion=23.2.0
mvn clean install
mvn -pl main-ui install -Prun

And it runs fine

adding the license-checker dependency to the pom.xml under main-ui module can solve the issue.
it looks the license-checker version is the cause..
       <dependency>
            <groupId>com.vaadin</groupId>
            <artifactId>license-checker</artifactId>
            <version>1.10.0</version>
        </dependency>

23.2.1 works fine, in 23.2.2 there is no error, but a different problem in browser
HTTP ERROR 404 Not Found

URI: /
Status: 404
Message: Not Found

Issue with license-checker seems to be that vaadin osgi 8.0.2 references version 1.4.1

Also vaadin-dev-server has a different version of license-checker
NFO] com.example:main-ui:jar:1.0.0-SNAPSHOT
[INFO] +- com.vaadin:flow-osgi:jar:8.0.2:compile
[INFO] |  \- com.vaadin:license-checker:jar:1.9.0:compile (version managed from 1.4.1)
[INFO] \- com.vaadin:vaadin:jar:23.2.3:compile
[INFO]    \- com.vaadin:vaadin-core:jar:23.2.3:compile
[INFO]       \- com.vaadin:vaadin-dev-server:jar:23.2.3:compile
[INFO]          \- (com.vaadin:license-checker:jar:1.9.0:compile - version managed from 1.10.0; omitted for duplicate)

platform has just upgraded license-checker to 1.10.0. probably that is the place causing version difference.

@manolo could you confirm is it still relevant?


Packaging tools

Checklist before submitting

[x] Did you read the contributor guide?
[x] Did you update any affected documentation?
[x] Did you write any tests to validate this change?

Description
Fixes #299 .
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

I've created this as a Draft PR to get the initial feedback on the implementation. I'm yet to do various things like -

Creating build scripts to create pip package
Creating conda recipe
Updating documentation

With the current implementation, I'm able to do pip install git+https://github.com/npanpaliya/open-ce.git@packaging-tools.

@bnemanich , @jayfurmanek and @bethune-bryant - Could you please take a glance on this PR and provide me an early feedback on approach?

After making the above comment, I realized it probably wouldn't be straightforward to pip install the package, since it will already be running as a pip installed package, then I got to wondering how it's working now. In your changes here, it's copying in the open-ce folder where the docker_build file is, which would copy the libs from the module, but not the open-ce script in bin.
https://github.com/npanpaliya/open-ce/blob/8bb678d077cb8639ed9c3b9455685bfe1bd9414f/open_ce/docker_build.py#L164..L165
This shouldn't work, because the open-ce entry point should be in anaconda3/envs/<env>/bin/, not in anaconda3/envs/<env>/lib/python3.6/site-packages/open_ce/.
It turns out the open-ce script is currently being installed in both locations. So we probably need to fix that in the setup.py file.
Also, once that's fixed we'll have to figure out how to execute open-ce inside the container, it will probably have to be copied in separately.
I've tried to find a way to pip install the package in the container, because I think that would be the best option, but I can't find a way to do that.

After making the above comment, I realized it probably wouldn't be straightforward to pip install the package, since it will already be running as a pip installed package, then I got to wondering how it's working now. In your changes here, it's copying in the open-ce folder where the docker_build file is, which would copy the libs from the module, but not the open-ce script in bin.
https://github.com/npanpaliya/open-ce/blob/8bb678d077cb8639ed9c3b9455685bfe1bd9414f/open_ce/docker_build.py#L164..L165
This shouldn't work, because the open-ce entry point should be in anaconda3/envs/<env>/bin/, not in anaconda3/envs/<env>/lib/python3.6/site-packages/open_ce/.
It turns out the open-ce script is currently being installed in both locations. So we probably need to fix that in the setup.py file.
Also, once that's fixed we'll have to figure out how to execute open-ce inside the container, it will probably have to be copied in separately.
I've tried to find a way to pip install the package in the container, because I think that would be the best option, but I can't find a way to do that.

Yes, I had also thought on it earlier but didn't find a way. Unless we know from where (git tag/branch/local code copy) we did pip install on host system programmatically inside docker_build.py , we can't do a pip install inside container.
I agree, open-ce is being copied at two locations currently, which I've fixed now. Now, I need to copy open-ce binary at the right location inside the container.

Codecov Report

Merging #329 (7c4ea6e) into main (6be1994) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #329   +/-   ##
=======================================
  Coverage   96.79%   96.80%
=======================================
  Files          17       18    +1
  Lines        1466     1467    +1
=======================================
+ Hits         1419     1420    +1
  Misses         47       47

Flag
Coverage Δ

unittests
96.80% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

open_ce/errors.py
100.00% <ø> (ø)

open_ce/__init__.py
100.00% <100.00%> (ø)

open_ce/build_env.py
96.49% <100.00%> (ø)

open_ce/build_feedstock.py
100.00% <100.00%> (ø)

open_ce/build_image.py
100.00% <100.00%> (ø)

open_ce/build_tree.py
95.00% <100.00%> (ø)

open_ce/conda_env_file_generator.py
100.00% <100.00%> (ø)

open_ce/conda_utils.py
100.00% <100.00%> (ø)

open_ce/docker_build.py
99.15% <100.00%> (ø)

open_ce/env_config.py
98.11% <100.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #329 (4668af7) into main (6be1994) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #329   +/-   ##
=======================================
  Coverage   96.79%   96.80%
=======================================
  Files          17       18    +1
  Lines        1466     1467    +1
=======================================
+ Hits         1419     1420    +1
  Misses         47       47

Flag
Coverage Δ

unittests
96.80% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

open_ce/errors.py
100.00% <ø> (ø)

open_ce/__init__.py
100.00% <100.00%> (ø)

open_ce/build_env.py
96.49% <100.00%> (ø)

open_ce/build_feedstock.py
100.00% <100.00%> (ø)

open_ce/build_image.py
100.00% <100.00%> (ø)

open_ce/build_tree.py
95.00% <100.00%> (ø)

open_ce/conda_env_file_generator.py
100.00% <100.00%> (ø)

open_ce/conda_utils.py
100.00% <100.00%> (ø)

open_ce/docker_build.py
99.15% <100.00%> (ø)

open_ce/env_config.py
98.11% <100.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #329 (4668af7) into main (6be1994) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #329   +/-   ##
=======================================
  Coverage   96.79%   96.80%
=======================================
  Files          17       18    +1
  Lines        1466     1467    +1
=======================================
+ Hits         1419     1420    +1
  Misses         47       47

Flag
Coverage Δ

unittests
96.80% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

open_ce/errors.py
100.00% <ø> (ø)

open_ce/__init__.py
100.00% <100.00%> (ø)

open_ce/build_env.py
96.49% <100.00%> (ø)

open_ce/build_feedstock.py
100.00% <100.00%> (ø)

open_ce/build_image.py
100.00% <100.00%> (ø)

open_ce/build_tree.py
95.00% <100.00%> (ø)

open_ce/conda_env_file_generator.py
100.00% <100.00%> (ø)

open_ce/conda_utils.py
100.00% <100.00%> (ø)

open_ce/docker_build.py
99.15% <100.00%> (ø)

open_ce/env_config.py
98.11% <100.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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


Update zio-prelude to 1.0.0-RC32

About this PR
📦 Updates dev.zio:zio-prelude from 1.0.0-RC21 to 1.0.0-RC32
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-prelude" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-prelude" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #180.


added function to convert string into integer

Created a function to convert string representing integer into integer type. I think is this valid with respect to problem statement.

Hi @prabodhw96,
This repository is not maintained anymore. The new project is here: https://github.com/adnanaziz/EPIJudge


Updated release?

Hi,
The current version on pypi is lagging behind the builds here. Additionally, the builds in Actions only keep artefacts for a few months. Could it be possible to release a new version with the current code?
Cheers

Hi, yes, I believe this should be possible now (again). The reason we weren't able to release on pypi anymore was because with the switch to C++20 there was simply no manylinux compatible platform for us to use. Now I see that they have updated the compatible images so I will try again.

Hi, I made a test release on https://test.pypi.org/project/pyoperon/0.4.0/#files
Could you please check if it works for you?

Hi! I've tested the python 3.12 wheel and I was able to successfully run this example.
By the way, I've released the code for my paper and I had to include the wheels form the action pages to guarantee reproducibility, you can check here how I've done it.

Cool, in the meanwhile I have figured it out how to build manylinux_2_28 wheels using the official docker images so I can release that version on PyPI as well (it'll take me a while due to busy times but hopefully real soon).

By the way, could you paste the link to the specific action you downloaded the wheels from? Thanks!

Hi,
Sure, I'm pretty sure it was this action https://github.com/heal-research/pyoperon/actions/runs/8570655759
I've updated our README to link to it.


Not at sea and drafts

When a user with drafts changes status to 'Not at Sea' the patrol summary should appear with the addition of a total number of drafts and a dot next to each boarding record that is a draft. The color should be the same orange as the validation exceptions. From that page the user can select a boarding record to see the summary (whether or not it is a draft) or continue as usual or cancel. If a user wants to edit a draft record they will be trained to cancel and go to the home screeen where the draft boardings are. https://mongodb.invisionapp.com/share/M6YN2Y77B8E#/screens/430523456_Home-On_Duty-5_Unsaved

Note that you can find the total number of drafts by filtering on reports that have the draft attribute set and reportingOfficer set to the current user


0.8.8版本编译报错

安装0.8.8版本后项目编译报错 unexpect token ,  指向双问号运算符报错，
降到0.7.2版本就能正常跑起来

We temporarily close this due to the lack of enough information.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required


Component paths get wrong index

Having the same region name in a layout and page controller gets the wrong index
https://github.com/enonic/nextjs-enonic-demo/blob/bea85e9caf4efaa4a0c0fe38c096a5b8d18a9f37/src/_enonicAdapter/guillotine/fetchContent.ts#L381-L384
Log of region index and paths:
index: 0
path : /main/8/main/0
---
index: 0
path: /main/9/main/0
---

I have a main region in my layout and in the page controller.

Suggested change:
if (region) {
        // getting the index of component from string like '/main/0/left/1'
        const cmpIndex = parseInt(cmp.path.split("/")[2])
        region.components.splice(cmpIndex, 0, cmp);
}

@poi33 This piece of code gets the component index inside a region, not a region index.

@pmi sorry my misunderstandig. I hope you still understood the problem.

I assume first region is used for both components ? :)


document indexing

It would be pretty cool to index documents (doc, pdf, xls, docx, rtf, text) using the attachment elasticsearch plugin. https://github.com/elastic/elasticsearch-mapper-attachments

Definitely, would probably melt certain types of clusters.  How would you pick which documents to send to it?  Just email and http attachments with certain mime types?  Configurable list?


Setup CI Test for GitHub

2414185237
I know everyone is different and i just want to hear your opinions. Is it the act of showing how much you love someone? Is it more of exercise and physical stamina? Is it a way to<EMAIL_ADDRESS>pleasure? I imagine it’s different depending on who you sleep with
Personally I don’t think i could have sex with someone i didn’t love, because i view it as a love connection (not judging anyone, you should do whatever you want, as long as it’s consensual) I just want to know how other people view it

Solve build gossip error by adding ${PROJECT_SOURCE_DIR}/third_party/gossip to CMakeList.txt

@zehuanw This MR is ready to merge, the three tests Single node tests, Multi node tests with NCCL and Multi node tests with Gossip work well.

Ways for further optimization:

Support fine-grained control (cancel/restart) to each job: decompose the single github-ci.yml file into multiple files, each file is responsible for one job.
Use artifacts: Coupled with the up-mentioned file-decomposition (https://docs.github.com/en/free-pro-team@latest/actions/guides/storing-workflow-data-as-artifacts)