Flexible pricing: program-level discount isn't reflected in enrollment pop-up

Steps to Reproduce

Set up a flexible pricing tier that is associated with a program, like this one on RC
Make and approve a flexible pricing request with a course in that program, like 14.100x on RC
Click the Enroll button

or, if you are already enrolled,
Look at the upsell box in the lower right of the about page.
Expected Behavior

The price in the pop-up should reflect the program-level discount
Clicking the "Get Certificate" button takes you to the checkout page, with the program-level discount
The discounts match.

Actual Behavior

The price in the pop-up does not reflect the program-level discount
Clicking the "Get Certificate" button takes you to the checkout page, with the program-level discount
The discounts don't match.

Hi @pdpinch, I am having trouble replicating this issue using the Flexible Price and Course you shared in the description.  I noticed that the discount for the Flexible Price is set to 0% off.
I tested this by submitting a Flexible Price request for the course you referenced with my yearly income being less than $1, which would then trigger the use of this course | program Flexible Price Tier: https://rc.mitxonline.mit.edu/admin/flexiblepricing/flexiblepricetier/38/change/.

I approved my Flexible Price request.
When I clicked 'Enroll', the modal shown displayed the adjusted price (now $250).
If I clicked the modal option to get the certificate, the checkout page also displayed $250.
If I clicked enroll without paying in the modal, and then navigated to the course details page (https://rc.mitxonline.mit.edu/courses/course-v1:MITx+14.100x/), I was also shown $250 in the upsell card shown in the bottom right of the page.

Sorry if I did this incorrectly.


Expose models from rn sdk

I'm very excited to try this out! Just out of interest, is there a specific reason you use enums instead of a literal string union?
export type ParticipantType = "listener" | "user"


ci: add automated release workflow on tag creation

New release workflow that will create GitHub releases when semantic version tags are pushed.

Codecov Report

Merging #387 (6bc619b) into main (184940f) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #387   +/-   ##
=======================================
  Coverage   34.18%   34.18%
=======================================
  Files          19       19
  Lines        1799     1799
=======================================
  Hits          615      615
  Misses       1118     1118
  Partials       66       66


Merge melodic into noetic

this updates the noetic branch with the latest melodic changes

an attempt to bring noetic in line with melodic again


zh localization

Localize the article title

I signed it

/assign @tengqm

@cloris-cc Are you proposing another commit?

No response from the author for over 2 months.
/close


Week 3 Review Ticket

[x] Github Pages with all links

Organize files, directories and menus for the first 3 weeks (Patterns, numbers, math functions). (1/1)
Write Factorial function using classes, providing implementation of call. (1/1)
Factors is written in imperative and OOP form. (1/1)
Team: 2/2
overall: 5/5


Admin Styles: Framework Compatability

I'm facing a few instances where I'm given a site to migrate to Apostrophe.
A site could be using a CSS framework, Bootstrap & Foundation being the mature and popular ones.
Could we consider making the LESS of the site a bit more extrenal-framework-proof?
Right now it's dead easy to integrate foundation or bootstrap, but doing so introduces a few glitches to the admin interface (inputs, paddings, line heights, e.t.c).
A solution could be to introduce a few abstract resets in a LESS that would be applied to the admin classes and their children, to keep the design uniform no matter what framework is being introduced.

We don't have plans to address this any time soon, however it's possible that we might refactor some of our CSS in tandem with some of our backend refactors. I'm putting this in the backlog for now and we'll update if we address any of it.

We've been making some progress on this as a current project requires bootstrap. The biggest offender is that we rely on our reset.less to remove default styling (bullets, margin, and padding) on li tags, which is pretty silly. As I find them, I am adding those specific styles to the classes that need them.

Ben, is reset.less in Apostrophe itself, or just in our sandbox stuff? I
think it's fine in the latter because nobody has to use it... right?
On Mon, Sep 28, 2015 at 5:46 PM, benirose<EMAIL_ADDRESS>wrote:

We've been making some progress on this as a current project requires
bootstrap. The biggest offender is that we rely on our reset.less to remove
default styling (bullets, margin, and padding) on li tags, which is pretty
silly. As I find them, I am adding those specific styles to the classes
that need them.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/101#issuecomment-143883031.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Right, it's in our sandbox, but we have admin ui that depends on it!

OK, I see. So our admin CSS should carry its own correctly scoped reset
rules.
And it will need them.
On Mon, Sep 28, 2015 at 11:48 PM, benirose<EMAIL_ADDRESS>wrote:

Right, it's in our sandbox, but we have admin ui that depends on it!
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/101#issuecomment-143935816.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

We should look at how we use the apos-ui class, which was intended for this
purpose, and how we make sure we don't overuse it, i.e. it shouldn't be on
a container like an apos-area because that would screw up the user content
inside it. It should only be on an inner div containing buttons, etc.
On Tue, Sep 29, 2015 at 10:52 AM, Tom Boutell<EMAIL_ADDRESS>wrote:

OK, I see. So our admin CSS should carry its own correctly scoped reset
rules.
And it will need them.
On Mon, Sep 28, 2015 at 11:48 PM, benirose<EMAIL_ADDRESS>wrote:

Right, it's in our sandbox, but we have admin ui that depends on it!
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/101#issuecomment-143935816
.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Right, I think we're mostly good in that regard.

Actually I'm not sure we use apos-ui anymore; poking around it seems like
our markup has lots of apos-ui-* things which presumably have their own
rules. We're going to need to reintroduce apos-ui I think. We could use a
mixin heavily, but that would produce a nasty-huge CSS file.
On Tue, Sep 29, 2015 at 10:56 AM, benirose<EMAIL_ADDRESS>wrote:

Right, I think we're mostly good in that regard.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/101#issuecomment-144085881.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

We did this for 2.0, we have a reset scoped to .apos-ui. It will not be changed for 0.5 as that would be too disruptive now that it's in maintenance support mode.


Impro 779 fill in horizontally

Addresses IMPRO-779
Description
Testing:

[ ] Ran tests and they passed OK
[ ] Added new tests for the new feature(s)

CLA

[ ] If a new developer, signed up to CLA

A nearly functioning PR
CLI tests failing which I need to look at and the fill in horizontally method is raising warnings. To fix the warnings I need to rearrange the indexing in that function to ignore NaN points first, then do the comparison with max_nbhood_orography.


Added pull request template

Closes #187
Addition of pull request template in .yml format.
@prakhartiwari0 Please review this PR.

PR merged! Well done on your contribution!

It's not working as expected

Hey @gambhirsharma, should I create another PR for resolving this issue.

Hey @gambhirsharma, should I create another PR for resolving this issue.

Yes, make a new PR with a new branch.
Read the official Documentation properly.

@gambhirsharma Sorry for the inconvenience caused. I have created a new PR with a new branch.


chore: update website deploy workflow

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

Requirements

[ ] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[ ] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

Codecov Report

Merging #780 (954e75b) into master (e78f732) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #780   +/-   ##
=======================================
  Coverage   61.65%   61.65%
=======================================
  Files           7        7
  Lines         785      785
=======================================
  Hits          484      484
  Misses        267      267
  Partials       34       34


[BUG] MongoDB Dataintegration ignores authMechanism

Mage version
v0.9.74
Describe the bug
The Source/Destination Configuration within a Data integration pipeline ignores the authMechanism as describes: MongoDB integration
To reproduce

Create a data integration pipeline
Use a MongoDB data source that uses a non default authMechanism like the legacy or Puppet-MongoDB default SCRAM-SHA-1
The data source will not authenticate.

Expected behavior
Adding:
authMechanism: SCRAM-SHA-1
Will get mage connected to a MongoDB SCRAM-SHA-1
Screenshots
No response
Operating system
any
Additional context
No response

Authentication in data integration with MongoDB has some strange hickups, will try to figure out what it is.. For now let's move forward on auth and 'don't care' about old SHA-1


Create a data connector for Apache Kafka

Create a data connector that:

Fetches data from Apache Kafka.
Stores the content locally (at least temporarily for processing), and runs them through unstructured.partition.auto.

See Adding Data Connectors for details on how to get started. Make sure to include a link to this issue when submitting a PR.
Definition of Done

[ ] The checklist has been completed.
[ ] The connector is able to process entries from Notion using the API.
[ ] The connector is able to process a single entry.
[ ] The connector is able to process several entries, with an option to process them recursively.
[ ] The connector can accept any credentials, if necessary.
[ ] The connector should be able of processing documents through unstructured.partition.auto.

Kafka is written in Java


Adds a little smoother installation experience.

Adds a spinner
Fixes the rnpm command out of band
Removes auto-installing of rnpm
Adds a version check of at least 1.7.0 of rnpm instead of just out-of-date
Silences the verbose logging during install
Adds next step instructions at the end

I stared at that gif for too long.

This is what happens when people run ignite now.


Couldn't find preset "es2015" relative to directory

I am getting the following error:
ERROR in (webpack)/buildin/module.js
Module build failed: Error: Couldn't find preset "es2015" relative to directory
after having installed:
$ npm install --save-dev babel-loader babel-core babel-preset-es2015
and ran webpack-dev-server

@WebTerminator Are you using windows or mac or linux. ??

+1 (same problem on mac)

@timcan @WebTerminator Looking into it. Thanks for reporting 👍

Replace
rules: [
    {
      test: /\.js$/, //Check for all js files
      use: [{
        loader: 'babel-loader',
        options: { presets: ['es2015'] }
      }]
    }
  ]

with
rules: [ {
    test: /\.js$/,
    loader: "babel-loader",
    query: {
        presets: [ "babel-preset-es2015" ].map(require.resolve)
    }
} ];

@radiocity Thanks for pointing it out 👍

Closing this issue as the above snippet solved the problem @timcan @WebTerminator Let me know if the issue is still there.


RqLive doens't fail when the input stream is empty

org.takes.rq.RqLive will return an empty request when receive an empty input stream.
Request req = new RqLive(new ByteArrayInputStream(new byte[0]));

I think it should throw an exception.

@hs3180 good catch, thanks

@hs3180 we will find someone to do this task, soon

@hs3180 milestone set to 1.0 (correct me if I am wrong)

@hs3180 thanks for tis bug, I topped your account for 15 mins, transaction AP-2NY50858CU066131T

@hdouss it's yours now, please proceed keeping in mind our principles. Feel free to ask any technical questions right here in the ticket; Budget here is 30 mins (keep this principle in mind)

Is it ok to just throw an IOException with a "Empty request" message in this case ?

@davvd @hs3180 If empty requests should not be accepted, Tests on org.takes.http.FtSecureTest will fail. All requests (com.jcabi.http.request.JdkRequest.JdkRequest requests) sent in org.takes.http.FtSecureTest are empty.

@davvd : waiting on decision whether to accept empty requests, or open an issue to fix tests on org.takes.http.FtSecureTest

@HDouss this ticket is not talking about request with empty header and body but a tcp connection that says nothing.
a request with no content is acceptable. We should reject request that doesn't meet the http specification, including a request with only a EOF. Every request should start with a start-line.

@hs3180 I agree. That's what I meant. JdkRequest is sending a request with an EOF onFtSecureTest

@hs3180 I was wrong. I was confused by the fact that inputStream.available returned 0. The inputStream was not empty, and so was not the JdkRequest.

@HDouss inputStream.available tells you how many bytes in buffer. inputStream.read will return -1 if meets EOF

@hs3180 PR #447  has been merged to master. The issue can be closed.

@HDouss thanks!

@ypshenychka please, help us to evaluate quality here, as per par.24

@davvd Quality is good.

@davvd Quality is good.

@ypshenychka many thanks

@hdouss just added 10 mins to @ypshenychka (for QA), payment ID is 71307443. 30 mins sent to your balance (ID 71307447), many thanks! It took 312 hours and 8 mins.. +30 added to your rating, at the moment it is: +30


snapshot failure

i run the operator in the local k8s cluster environment, k8s version 1.13.5.
es cluster runs normally, but snapshot fails.

the snapshot configuration:

Is there any other configuration required?

You can use this image to fix the issue komljen/elasticsearch-cron:0.0.6.
The image you provided for cron is not up to date.


libtorrent-rasterbar-1_2_x: use python3 by default

Description of changes

All packages that depend on it are already overriding the default python to python3 in all-packages.nix. Let's just make it the default instead.
See https://github.com/NixOS/nixpkgs/pull/201859.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package marked as broken and skipped:

libtorrentRasterbar-1_2_x

1 package built:

libtorrent-rasterbar-1_2_x


Using formula2 instead of formula

I was trying to use the complex formula:
=IF(ISNUMBER(MATCH(1,
(INDIRECT(ADDRESS(ROW(),1))=OFFSET(
Sheet1!$A$1:$D$53,0,MATCH($A$1,OFFSET(
Sheet1!$A$1:$D$53,0,0,1),0)-1,,1))*
(INDIRECT(ADDRESS(ROW(),2))=OFFSET(
Sheet1!$A$1:$D$53,0,MATCH($B$1,OFFSET(
Sheet1!$A$1:$D$53,0,0,1),0)-1,,1))*
1,0)),1,0)
But export-excel appears to add the implicit intersection operator @ to the indirect and offset functions and it appears to be because currently export-excel uses .formula instead of .formula2 in line 328.
Is it possible to a switch to add this functionality?

@darg2012 were you looking at a newer version of EPPlus when you see .Formula2? I don't see that on the version that currently being used.

My bad, I assumed there was an .formula2 equivalent of .formula similar to how .value2 and .value both exists. For now, I'm just doing a simple search and replace but would like to avoid that step during file creation. Will keep looking for possible solutions that preferably avoid using VBA.


README: fix hello world example so it compiles

Appearantly the HTTP_PROTOTYPE macro is needed, otherwise the example
won't compile.
The compile error was:
/usr/local/include/pistache/http.h:758:5: error: static assertion failed: An http handler must be an http prototype, did you forget the HTTP_PROTOTYPE macro ?
     static_assert(details::IsHttpPrototype<H>::value, "An http handler must be an http prototype, did you forget the HTTP_PROTOTYPE macro ?");

The fix was also mentioned in #71 but it seems the README didn't actually get updated.

Anyone there :smile: ?

Hello,
Good catch, thanks for the Pull Request, merging it in.


Segmentation fault when using NCCL in MultiWorkerMirroredStrategy

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): RHEL 7.9
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 2.5.0
Python version: 3.8.6
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): 10.2.0
CUDA/cuDNN version: 11.1.1
GPU model and memory: A100

Describe the current behavior
When running the distributed training with MultiWorkerMirroredStrategy and NCCL the application will crash during a NCCL reduction operation due to a bug in TF.
Standalone code to reproduce the issue
Similar to #50853 run the following on at least 2 nodes with at least 2 GPUs:
import tensorflow as tf
from mpi_cluster_resolver import MPIClusterResolver

resolver = MPIClusterResolver()
communication = tf.distribute.experimental.CollectiveCommunication.NCCL
options = tf.distribute.experimental.CommunicationOptions(implementation=communication)
strategy = tf.distribute.MultiWorkerMirroredStrategy(communication_options=options, cluster_resolver=resolver)

with strategy.scope():
    (x_train, y_train), _ = tf.keras.datasets.mnist.load_data()
    x_train = x_train / 255.0
    train_data = tf.data.Dataset.from_tensor_slices((x_train, y_train)).batch(128)

    model = tf.keras.models.Sequential([
        tf.keras.layers.Flatten(input_shape=(28, 28)),
        tf.keras.layers.Dense(128, activation='relu'),
        tf.keras.layers.Dense(10),
    ])

    model.compile(
        optimizer=tf.keras.optimizers.SGD(),
        loss=tf.keras.losses.SparseCategoricalCrossentropy(from_logits=True),
        metrics=['accuracy'])

is_master = resolver.task_id == 0
verbose = 2 if is_master else 0
model.fit(train_data.repeat(), epochs=1, steps_per_epoch=10, verbose=verbose)

With
mpi_cluster_resolver.py.txt
Other info / logs
I was able to collect the following backtrace:
   #0  tensorflow::NcclCommunicator::Enqueue(std::shared_ptr<tensorflow::CollectiveContext>, std::function<void (tensorflow::Status const&)>) (this=0x1ebf9260,
std::shared_ptr<tensorflow::CollectiveContext> (use count 5, weak count 0) = {...}, done=...) at tensorflow/core/nccl/collective_communicator.cc:78
#1  0x00002aaacc95de30 in tensorflow::NcclReducer::Run(std::function<void (tensorflow::Status const&)>) ()
   from <prefix>/tensorflow/python/_pywrap_tensorflow_internal.so
#2  0x00002aaaeb699dc0 in std::_Function_handler<void (), tensorflow::BaseCollectiveExecutor::ExecuteAsync(tensorflow::OpKernelContext*, tensorflow::CollectiveParams const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::function<void (tensorflow::Status const&)>)::{lambda()#3}>::_M_invoke(std::_Any_data const&) ()
   from <prefix>/tensorflow/python/../libtensorflow_framework.so.2
#3  0x00002aaaeb2cdbf6 in std::function<void ()>::operator()() const ()
   from <prefix>/tensorflow/python/../libtensorflow_framework.so.2
#4  0x00002aaaeb9350a9 in tensorflow::UnboundedWorkQueue::PooledThreadFunc() ()
   from <prefix>/tensorflow/python/../libtensorflow_framework.so.2
#5  0x00002aaaeb935164 in std::_Function_handler<void (), tensorflow::UnboundedWorkQueue::Schedule(std::function<void ()>)::{lambda()#1}>::_M_invoke(std::_Any_data const&) ()
   from <prefix>/tensorflow/python/../libtensorflow_framework.so.2
#6  0x00002aaaeb2cdbf6 in std::function<void ()>::operator()() const ()
   from <prefix>/tensorflow/python/../libtensorflow_framework.so.2
#7  0x00002aaaeb93c799 in tensorflow::(anonymous namespace)::PThread::ThreadFn(void*) ()
   from <prefix>/tensorflow/python/../libtensorflow_framework.so.2
#8  0x00002aaaab7deea5 in start_thread () from /lib64/libpthread.so.0
#9  0x00002aaaabcf496d in clone () from /lib64/libc.so.6

Further debugging led me to https://github.com/tensorflow/tensorflow/blob/98765aab1af708f8663fb983aca75115d70da8df/tensorflow/core/nccl/collective_communicator.cc#L89 which calls effectively col_ctx->op_ctx->device()->tensorflow_gpu_device_info() which returns nullptr which is then returned an dereferenced in col_ctx->op_ctx->op_device_context()->stream() which obviously segfaults.
Some further debugging examining col_ctx->op_ctx->device()->name() shows that in the case of the crash /job:worker/replica:0/task:1/device:CPU:0 or /job:worker/replica:0/task:0/device:CPU:0 is used (both of the 2 nodes crash), so a CPU device ends up there which of course does not have a GPU info set.

The VERBOSE 2 log shows:
2021-07-23 17:56:53.724673: I tensorflow/core/common_runtime/collective_param_resolver_local.cc:688] CompleteInstanceLocal /job:worker/replica:0/task:1/device:CPU:0 instance_key: 101 gr 0x2b35e80016f0
2021-07-23 17:56:53.724702: I tensorflow/core/common_runtime/collective_param_resolver_local.cc:681] AssignCollectiveType NcclReduce
2021-07-23 17:56:53.724717: I tensorflow/core/kernels/collective_ops.cc:579] CollectiveReduceV2 ExecuteAsync start for collective CollectiveReduceV2: ReduceV2(Add,Id) device /job:worker/replica:0/task:1/device:CPU:0 group 2 instance 101
2021-07-23 17:56:53.724732: I tensorflow/core/common_runtime/base_collective_executor.cc:395] CreateCollective type int64 name NcclReduce

I have the same issue using SlurmClusterResolver , with tensorflow-2.5.0 and 2.6.0.
I have a very similar environment : compiled from source, RHEL 8, bazel 3.7.2, gcc 8.3.1, CUDA 11.2, NCCL 2.8.3, V100
Any news?

We investigated a bit and compared the tensorflow logs for tensorflow-2.4.1 (no segfault) and tensorflow-2.5.0 (segfault). We use 1 node with 4 GPUs, 2 workers, and each worker managing 2 GPUs. We also use Keras (model.fit()) and the MultiWorkerMirroredStrategy.
Using tensorflow-2.4.1:
Epoch 1/10
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 1, group_size = 2, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 6 all-reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 1, group_size = 2, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 6 all-reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1
INFO:tensorflow:Collective batch_all_reduce: 1 all-reduces, num_devices = 2, group_size = 4, implementation = AUTO, num_packs = 1

Using tensorflow-2.5.0:
Epoch 1/10
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 1, group_size = 2, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 6 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 1, group_size = 2, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 6 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1
INFO:tensorflow:Collective all_reduce tensors: 1 all_reduces, num_devices = 2, group_size = 4, implementation = NCCL, num_packs = 1

We see that the communication implementation does not switch to AUTO with tensorflow-2.5.0 when all_reduces = 1.
Here is the code line where the switch is supposed to take place:
https://github.com/tensorflow/tensorflow/blob/v2.5.0/tensorflow/python/distribute/cross_device_ops.py#L1124
The value of self._limited_nccl is always False with tensorflow-2.5.0.
Just to see what happens, we set self._limited_nccl = True and the training ends properly.


Project System: VSHost replacement

From @davkean on April 11, 2016 22:56
Currently csproj/msvbproj uses vshost.exe to "speed up" F5 debugging for desktop client apps.It is also used for immediate window evaluation without starting a debugging session. We need to figure out if we are going to bring this forward to CPS or not.
Copied from original issue: dotnet/roslyn#10484

From @tmat on April 12, 2016 3:19
@tmat

ClickOnce partial trust support is also handled by VSHost.

From @KirillOsenkov on April 17, 2016 5:51
Note that using VSHost prevents you from inspecting most optimized variables in Watch/Locals/etc. windows. I always turn it off (and enable COMPLUS_ZapDisable=1) to actually see the variables in the debugger. My understanding is that the vshost process doesn't inherit environment from devenv, so COMPLUS_ZapDisable=1 has no effect.
I've never seen any benefit from VSHost being turned on. No perceptible perf improvement at all.

We gathered the data, there's ~300ms win on F5 with the the hosting process, but we end up paying 3 - 10 seconds on solution load, and stop debugging.

It's a bit more complicated than that. There are other factors. The debugger team is working on some precise measurements to determine why exactly VS hosts helps F5 speedup in some scenarios.

@tmat @ManishJayaswal where did we land with this?

Debugger team had an action item to identify the specific benefits of this process on various project types. I had pinged them last week and they had asked for 2 more weeks time to get back.

This has been removed - there's no work to do here.

Actually, it's still there to support DTEE for class libraries. It's only loaded when DTEE is used.

@davkean @tmat Can you explain a little more what work CPS would need to do in this case?

We fire up VSHost for the immediate window inside csproj, the work, if we supported it, would be to figure out how that works and port it.

I'm not sure it's worth the effort. I think there are more important things to do.


[Bug] missing underline action

Initial checklist

[X] I agree to follow the code of conduct
[X] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected packages and versions
7.0.0
Link to runnable example
No response
Steps to reproduce
In playground use the underline action which actually does a strike-through
Expected behavior
There should be an underline action in commonmark or gfm
Actual behavior
There is no action preset for underline
Runtime
No response
OS
No response
Build and bundle tools
No response

Underline is not part of the commonmark or GFM spec.


fix: mediaLink of stored objects returns invalid path

Fixes issue where if object is put from a different domain than where it is get from. This happens e.g. when integration tests run on localhost but emulator runs inside docker compose.

Oh and thank you for the edits! I really needed your Host changes, this PR just brings that change further forward.


Dotnet pack fable files

Using the latest Fake libs so can use the relevant Fake.Dotnet features. Judging by the dotnet pack and nuget metadata docs I hope this is correct.

I already published it to NuGet (5 hours ago). But it is not reflecting there! Do you have any idea?

@enerqi I guess it could be because of the bin folder location
Target.create "NuGet" (fun _ ->
    let ReleaseNotes = String.toLines release.Notes
    "src/FsToolkit.ErrorHandling"
    |> DotNet.pack(fun p ->
           { p with
               OutputPath = Some "bin"
               Configuration = DotNet.BuildConfiguration.Release
               MSBuildParams = {MSBuild.CliArguments.Create() with
                                    // "/p" (property) arguments to MSBuild.exe
                                    Properties = [("Version", release.NugetVersion)
                                                  ("PackageReleaseNotes", ReleaseNotes)]}})
)

Target.create "PublishNuget" (fun _ ->
    Paket.push(fun p ->
        { p with
            PublishUrl = "https://www.nuget.org"
            WorkingDir = "bin" })
)

Can you verify this as well ?

Yes, it seems that OutputPath is relative to the directory passed to dotnet pack.
OutputPath = Some "../../bin" works. I guess the other approach would be to set the WorkingDir on the "PublishNuget" target to src/FsToolkit.ErrorHandling and leave the pack OutputPath as bin.
I tried passing "." to dotnet pack (starting from the project root dir) but that packages up tests aswell and puts the nupkgs under the project directories, so that doesn't look any prettier.


KOGITO-9470: Update base image versions for Serverless Logic Web Tools

https://issues.redhat.com/browse/KOGITO-9470
Also in this PR

Add an option to show/hide the disclaimer in the dashbuilder-viewer-deployment-webapp package. (default is to hide; kubesmarts always shows it).
Remove restriction for workflow ids since it can be anything now

Known issues:

https://issues.redhat.com/browse/KOGITO-9480
https://issues.redhat.com/browse/KOGITO-9481

@caponetto can we make sure that https://issues.redhat.com/browse/KOGITO-9480 is a blocker for our next release?

FYI @jesuino


fix merge limit should use bottom phase

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run pipelinex_p0


Version example for constraint routing demo

I added the version constraint (Added in 1.2) to the list of constraints included in this demo.

One tiny nit. LGTM otherwise :smile:

:+1:


Update Doc: SvelteKit 1.0

Hey @Steellgold - thanks for the contribution!
@shinokada can you please check this out if it's valid? Thanks!

It is ok to be merged.

I will update the page and create a PR.

Hello @Steellgold @shinokada,
There are two PR's for the Svelte Kit documentation - which one should I merge?
Cheers,
Zoltan

You need to merge, https://github.com/themesberg/flowbite/pull/455.


fix(examples): Fix tests in external-data example so they don't fail in Node20

Description
Fixes the tests of the external-data example so they succeed in Node20.
Also simplifies that package a bit by removing an unnecessary split of the webpack config and removing the unused stream-http dependency.
I also tried to simplify further by replacing node-fetch with Node's native fetch but that led to different test failures. Not urgent so I left it alone.
Reviewer Guidance
The review process is outlined on this wiki page.
I confirmed the tests pass in Node18 and Node20.
AB#20804

@microsoft-github-policy-service rerun


Changing the editor state

Hi!
In this example, we can load a new content in the editor.
Unfortunately, it doesn't preserve history (load a saved content, and do ctrl+z : nothing happens).
This is because it creates a new editor instance when loading saved content.
To preserve history, we can set a transaction that replace the root doc :
const doc = MyNewDoc

// Create a transaction to set the new document content
const tr = editor.state.tr.replaceWith(0, editor.state.doc.content.size, editor.schema.nodeFromJSON(doc))

// Apply the transaction to the current state
const newState = editor.state.apply(tr) as EditorState

// Update the editor view with the new state
editor.view.updateState(newState)

I use nodeFromJSON in this example, but we can use jsonFromHTML to do this.
So, I didn't find a function to do that simply in the prosekit code.
Maybe an helper to do that will be useful ?

prosekit v0.9.13 adds editor.setContent() method, which is an easy way to replace the whole document in the current editor.


How to open specific ballon by click on a button?

I have a list with an address list and when a user clicks on an address balloon must open. How I can implement this? Plus I'm using the cluster feature.
A rough example:

I tried to do an array with refs and then search a ref and open but it's ugly and works only without the cluster feature.
I think much relevant doc article is https://tech.yandex.ru/maps/jsbox/2.1/object_manager_balloon, I didn't try ObjectManager and I will go to try it in a few mins :)
Thanks!

@kotchuprik i think you will understand idea:

thanks! that looks good for me, going to check for my case.

well, I have another question :D how I can zoom to cluster automatically? I do not need two column balloon view I need just open balloon X for button X.

I did it! I just must to add instanceRef={ref => this.map = ref} to my Map and then this.map.setCenter :)


Boiler TargetTemperature limits

Homebridge OTGW sets the Boiler Target Temperature range of valid values to the MAX CH setpoint boundaries (OT message 0x31), but the Control Setpoint (OT message 0x01) can be lower than that.
Following log message in Homebridge v1.3.0-beta.56:
[13/02/2021, 17:15:40] Received warning for the plugin 'homebridge-otgw' from the characteristic 'Target Temperature': characteristic was supplied illegal value: number 10 exceeded minimum of 20.
[13/02/2021, 17:15:40] Above characteristic warning was triggered at: Error:
    at Characteristic.characteristicWarning (/usr/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:2055:79)
    at Characteristic.validateUserInput (/usr/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:2011:14)
    at Characteristic.updateValue (/usr/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1338:20)
    at new CharacteristicDelegate (/usr/lib/node_modules/homebridge-otgw/node_modules/homebridge-lib/lib/CharacteristicDelegate.js:165:28)
    at Boiler.addCharacteristicDelegate (/usr/lib/node_modules/homebridge-otgw/node_modules/homebridge-lib/lib/ServiceDelegate.js:245:36)
    at new Boiler (/usr/lib/node_modules/homebridge-otgw/lib/OtgwService.js:184:14)
    at new OtgwAccessory (/usr/lib/node_modules/homebridge-otgw/lib/OtgwAccessory.js:28:24)
    at OtgwPlatform.init (/usr/lib/node_modules/homebridge-otgw/lib/OtgwPlatform.js:187:31)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)

Solved in v1.1.8


MDX does not render correctly

🐛 Bug Report
MDX does not render correctly.
Have you read the Contributing Guidelines on issues?
yes
To Reproduce

Setup a fresh docusaurus v2 project: npx @docusaurus/init@next init docu-test classic
Replace content of docs/mdx.md with the raw content of the following gist.
npm start
Open the docs page http://localhost:3000/docs/mdx with the browser of your choice.

Expected behavior
The value of my constant myVar should be rendered for all three expressions.
Actual Behavior

Your Environment

Docusaurus version used: "2.0.0-alpha.43"
Environment name and version:

Node.js: v10.16.3

Operating system and version: MacOS 10.15.3

Reproducible Demo
Here you can find a working demo of this issue: https://github.com/maxarndt/docusaurus-v2-broken-mdx-demo

Repro case attached here for easier reference:
export const myVar = 'hello mdx';

{myVar} is the var. MyVar: {myVar}

My var is {myVar}

I don't think this is something to do with Docusaurus. Might want to file on the MDX repo instead. Anyway one easy work-around for you is to wrap the line which doesn't render properly with a
, which "turns on JSX mode" for that line.

My bad! The mdx docs explicitly say:

Any line that start with the < characters starts a JSX block.


Error code 1009: SIG_BAD_CLOCK in method keybase.1.login.login

[write something useful and descriptive here]
my log id: a9175cebc6b25bffc8d4db1c
I have a Mac-mini M1 Apple chip: bad signature: your computer's clock is wrong: signature is dated 10791 seconds in the future => Error code 1009: SIG_BAD_CLOCK in method keybase.1.login.login
More feedback (or press  when done):
▶ INFO Not sending up a UID for logged in user; none found

Success! Your log ID is:  a9175cebc6b25bffc8d4db1c
Here's a URL to submit new bug reports containing this ID:
https://github.com/keybase/client/issues/new?body=[write something useful and descriptive here]

my log id: a9175cebc6b25bffc8d4db1c
Thanks!


【Hackathon No.29】add RFC for PixelUnshuffle

增加paddle.nn. PixelUnshuffle设计文档。

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Your Name seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

already signed

你好，我们发现你的设计文档中部分内容与另一个开发者的pr（#44）有雷同现象，pr（#44） 先于你提交，从遵守开源规则、维护有序开源生态的角度出发，你可以基于其他开发者的pr 进行commit修改，不建议再次提交个人pr，因此我们会优先review pr（#44），感谢你对飞桨的贡献！


update for Public team list not displaying

Workaround is ineffective if you don't know what team you should be searching for, this needs to be communicate by the team owner through some other medium. Not practical for searchers to go through each letter of the alphabet hoping to find a match, for example.

@ChelseaStats - thanks for adding the value on this from your expertise. Maybe change it to: "try searching for it in the top right search box. Also, the team owners should communicate team names to colleagues since many teams could show up in the search results." ?

Thanks for the update. I'm not sure this helps the situation however.
It is not multiple teams might appear in the search (if that is the case it would be easy to see which one they wanted to join), It is the issue is there is no option to see all public teams including those that used to be private (the current bug).
When searching 'a' no teams are shown
When searching 'b' no teams are shown
...
when searching 'm' - MicrosoftBugFixing appears (for example).
that is a lot of searching to find the one team.
Does that make sense?

@ChelseaStats I'm a little confused as well.  Are you saying that when you go to the "Join or create a team" page, you don't see all teams in the organization - both public and private?  Or that the search isn't returning teams when you look for them?

I am not seeing any teams in my org.
We had around 30 developers in my org that couldn't see a team we had set up privately among a small group doing a test - we then wanted to roll this out to all staff but they couldn't see the team. we tried sending the link - but that didn't work either. We then found a work around by asking them to search for the team and then it appeared.
(I believe the original bug was any team that was private and had since been made public was not showing)
I hope this helps.

@ChelseaStats - going to merge this since it will be beneficial even if there is a bug that gets resolved.
@islubin - Sounds like the possible bug is "I believe the original bug was any team that was private and had since been made public was not showing".


chore: Adds CoudEvents when a ScaledObject scales a workload from/to zero or one

Checklist

[x] When introducing a new scaler, I agree with the scaling governance policy
[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Tests have been added
[x] Changelog has been updated and is aligned with our changelog requirements
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Relates to #3527

Can you open a PR for our docs as well please?

@neelanjan00 any update on this please?

eventEmitter.Emit() will emit a k8s event at the same time. Replacing the recorder with eventEmitter and r.Recorder.Event() with eventEmitter.Emit() is better than passing an eventEmitter as a new parameter. Alternatively, we could pass an eventEmitter as a new parameter and delete the recorder once all r.Recorder.Event() calls have been replaced.

We are planning release on Thursday, any chance we can incorporate the feedback?

Hey @tomkerkhove, @zroubalik sorry for not being active on this! I am presently occupied with some office work so not able to put time away for this, not sure if I will be able to add the E2E tests on time for the release.

No worries!

Any update on this @neelanjan00? If you prefer us to work on this then that is fine as well

Hi @tomkerkhove sorry for being inactive on this, I don't think it will be feasible for me to work on the remaining tasks, it will be helpful if someone else can take it up. Thanks!

Closing as discussed on https://github.com/kedacore/keda/issues/3527#issuecomment-2206823001 @SpiritZhou is taking over and will use this one as inspiration - Thanks for working on this anyway @neelanjan00!


Void return type error running postgraphql

I have a postgres database on RDS and I'm running into the following when I run postgraphql
Error: Procedures with a void return type are not allowed in GraphQL queries.
at createPgSingleProcedureQueryGqlFieldEntry  (.../node_modules/postgraphql/src/postgraphql/schema/procedures/createPgProcedureQueryGqlFieldEntry.ts:40:11)
at Object.createPgProcedureQueryGqlFieldEntry [as default] (.../node_modules/postgraphql/src/postgraphql/schema/procedures/createPgProcedureQueryGqlFieldEntry.ts:24:9)
(.../node_modules/postgraphql/src/postgraphql/schema/createPostGraphQLSchema.ts:131:81)
I have not created any functions that return void.
I'm using postgraphql as an express library as done here  https://github.com/postgraphql/postgraphql/blob/607f6629986735139e2b77d5a9b1143846c4f691/docs/library.md
any thoughts as to what may be happening?

My guess would be one of your extensions has installed a function like that into the public schema. To confirm, run "\df" from psql, if this is the issue I strongly recommend using an alternative schema for PostGraphQL.
Do you get the same issue with the v4 alpha? "npm install postgraphql@next"

Hi Benjie - thanks for the quick reply! Think you're correct that a recently installed extension is the cause..
with V4 alpha I get a similar error:
Error: Schema query must be Object Type but got: undefined.
at invariant (.../node_modules/graphql/jsutils/invariant.js:19:11)
at new GraphQLSchema (.../node_modules/graphql/type/schema.js:72:88)
Thanks

Hi Benjie,
I confirm this issue.
In my case PostGIS extension is installed: this is a "legacy" application which uses geospacial features of PostGres and I can't remove extension. Any idea to solve this issue without removing the extension ?

Currently we do not have a workaround; in V4 you could potentially install a replacement for the introspection plugin that filters out the PostGIS extension but that may be a fair amount of work.
It might just be a case of automatically ignoring stable functions that return void (though I'm not sure what the point of a stable function that returns void is... I'm guessing it's not really a stable function?)

What about either reinstalling the extension into a different schema, or using a schema other than public for hosting postgraphql?

Both of those would work; I strongly recommend using an alternative schema for PostGraphQL data/functions because the public schema is often polluted, and you don't really want those extra methods being exposed to your GraphQL clients. Many (but not all) extensions will allow you to set a different namespace to install into, but then you must make sure that new namespace is part of your search_path otherwise you have to give the fully qualified method name each time.

@eric-taix I'd love to know if the latest PostGraphQL v4 solves your issue? npm install postgraphql@next

@graphqlgirl Similarly, I think the issue you listed above should be fixed by now?

yep thanks @benjie, it's been fixed -- if you've updated PostGraphQL v4, haven't checked that out yet

@benjie I tried postgraphql 4.0.0-alpha2.8 and it works with the postgis extension : I'm now able to request my database and Geometry columns with GraphiQL.
Thanks so much for the hard work

Awesome. I'm going to close this then 👍


Quote getarch include paths in cmake

fixes #2576 (embedded spaces in path to OpenBLAS source)

need to redo this from the correct branch...


Content, descriptions and validations for genesets

This PR adds support for content and description upload to models and eryph-packer.
Plus:

updates packages for security fix of Microsoft.Identity.Client
Validation of manifests
limits for yaml sources and markdown content
Client methods to update geneset content
use correct binary prefix for sizes (KiB, MiB, GiB)

This PR requires a updated version of the genepool, that offers new features like customized expand options. The Expand parameter is currently not considered in the dotnet genepool client (as paged responses are currently not implemented at all, see #15).

The command geneset init acme/acme-geneset --description "... too long description ..." with an actual description > 80 characters will fail with the expected error but the geneset is still initialized (without the description). This means the user cannot just repeat the command with a fixed description.

The command geneset init acme/acme-geneset --description "... too long description ..." with an actual description > 80 characters will fail with the expected error but the geneset is still initialized (without the description). This means the user cannot just repeat the command with a fixed description.

Jap, fixed that. However init is the only command where this kind of idempotence should be necessary.
Its by design that other properties like long description, markdown, metadata are not configureable by command as it is expected that in this cases the geneset.json is created directly or modified.

The geneset init command should create a readme.md with placeholder content.

The validation for some commands could be improved further:

eryph-packer geneset-tag init "dbosoft/e2etests-os/invalid%value" does not fail but invalid%value is not a valid tag name.
eryph-packer geneset-tag ref dbosoft/e2etests-fodder/latest "invalid|test" does not fail but the referenced is invalid.

The second case can also not be corrected via the CLI as the validation triggers when the manifest is read.

The validation for some commands could be improved further:

eryph-packer geneset-tag init "dbosoft/e2etests-os/invalid%value" does not fail but invalid%value is not a valid tag name.
eryph-packer geneset-tag ref dbosoft/e2etests-fodder/latest "invalid|test" does not fail but the referenced is invalid.

The second case can also not be corrected via the CLI as the validation triggers when the manifest is read.

moved validation to a new extension that validates arguments and options in command parsing.

The geneset init command should create a readme.md with placeholder content.

done

After eryph-packer geneset init dbosoft/test, eryph-packer geneset info dbosoft/test immediately displays an error:
The operation failed with following error(s):
    The geneset manifest is invalid.
        ShortDescription: The ShortDescription is required.

Could we either make the --description argument mandatory or provide a default value for the short description?

After eryph-packer geneset init dbosoft/test, eryph-packer geneset info dbosoft/test immediately displays an error:
The operation failed with following error(s):
    The geneset manifest is invalid.
        ShortDescription: The ShortDescription is required.

Could we either make the --description argument mandatory or provide a default value for the short description?

set geneset name as default

eryph-packer geneset-tag init dbosoft/test/1.0 can be invoked without invoking eryph-packer geneset init dbosoft/test first. An error only occurs when the user tries to push the gene set tag at the end.

fixed, geneset init is now enforced


Property 'getOwnMetadata' does not exist on type 'typeof Reflect'.

Hi, I am using angular 2 beta 8 and I got this error from the constructor:
Property 'getOwnMetadata' does not exist on type 'typeof Reflect'.
I use gulp-typescript for compiling.
Please help. Thank you.
Rex

By adding this, i do not the compile error anymore:
/// < reference path="../../node_modules/reflect-metadata/reflect-metadata.d.ts"/>


Missing standalone plans

As per MS support, SharePoint Online standalone plans also include Power Automate capabilities.  That means that in this article 2 plans are missing from the list below the phrase "The following Office 365 licenses include Power Automate capabilities."
SharePoint Online Plan 1
SharePoint Online Plan 2
Case number for reference:  30746744
Opening this as an issue, not PR, because perhaps that information should undergo additional verification.

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

ID: 8b98a3d2-eee9-5956-4e6f-65a0240db884
Version Independent ID: 8b98a3d2-eee9-5956-4e6f-65a0240db884
Content: Frequently asked questions about Power Automate licensing - Power Platform
Content Source: power-platform/admin/power-automate-licensing/faqs.md
Service: power-platform
GitHub Login: @PriyaKodukula
Microsoft Alias: prkoduku

Thanks for your comment, @PowershellScripts. This issue has been assigned to the doc author for review.

@PowershellScripts
Thanks for the feedback. We're in the process of updating this content.


Update PushNotificationGcmListener.cs

pass along extras to intent to make them available downstream

awesome!  thanks for a great project, do you have any schedule to publish an update on nuget?

Already did :)
Sent from my iPhone

On 11 nov 2015, at 6:40 p.m., jmeadecvlt<EMAIL_ADDRESS>wrote:
awesome! thanks for a great project, do you have any schedule to publish an update on nuget?
—
Reply to this email directly or view it on GitHub.


Rename 001_installation.md to 001_Installation.md

This PR fixes a broken link in the README caused by a case mismatch in the filename. I've updated the filename to correct the issue.

My bad, thank you.


Normalization for the feature

Hi Brandon,
Just a new question in this repo. For this line,
https://github.com/brandontrabucco/design-baselines/blob/785dbcfa58107bfcc426257a1c2e69d7f71c3c27/design_baselines/coms_cleaned/__init__.py#L323
When I trace back, from top to bottom, when we set normalize_xs=True
if normalize_xs:
        task.map_normalize_x()
x = task.x
initial_x = tf.gather(x, indices, axis=0)
xt = initial_x
solution = xt
# for here 'solution' should be a normalized feature
# should this be  'task.denormalize_x(solution)' instead of 'solution'?
# I guess the 'predict' function is designed for original feature
score = task.predict(solution)

Maybe I have some misunderstanding. Please let me know. Thanks!
Nathan

Hi Nathan-zh,
Thanks for your interest in the code!  The task.predict internally tracks if the task features are normalized, and will automatically denormalize them when evaluating the designs at this line of code:
https://github.com/brandontrabucco/design-bench/blob/27ed0c3d1c28acb43084fac1b317fc7268cd7c9a/design_bench/oracles/oracle_builder.py#L495
-Brandon

That's great. I skim your codes and find the flag expect_normalized_x. Thanks so much!


The name of xRec is not valid. The name of variables and parameters must be suffixed with the type or object name. A (AA0072))

Describe the bug
This CodeCop is NonSense for Event Subscribers.
To Reproduce
Steps and to reproduce the behavior:
Create an Event Subscriber like the following
[EventSubscriber(ObjectType::Table, Database::"Gen. Journal Line", 'OnAfterValidateEvent', 'Description', true, false)]
local procedure GenJnlLineOnAfterValidateEvent(var Rec: Record "Gen. Journal Line"; var xRec: Record "Gen. Journal Line")
begin
end;

Expected behavior
There shouldn't be an info or any CodeCop Action, since the subscriber cannot change the signature of an event.
Screenshots
If applicable, add screenshots to help explain your problem.

5. Versions:

AL Language: 6.0.344420

Business Central:

Another one to add to the litany of errors mainly documented in https://github.com/microsoft/AL/issues/6120
This diagnostic should just be turned off until it's anywhere near ready for use. End-users should not have to act as unwitting alpha-testers.

I am closing this issue because it has been resolved, please check the latest developer preview . Please open a new issue if you believe this has not been resolved and reference the current issue.


One Dimensional Array -> First Missing Integer : Add C++ optimisation; and fix some bugs

Hello @Abhijit2505  !
I am Samrendra, a Participant of SWOC'21. I have found a few issues on the page - First Missing Integer in the section One Dimensional Array and would like to work on them.
Link to the page - https://abhijit2505.github.io/Algorithmic-Treasure-Site/Pages/Array/firstMissingInt.html
Following are the issues:
1. The current C++ code sorts the vector using stl::sort(); this has an amortized time complexity of O(nlogn), but the given algorithm is wrongly marked as an O(n) time complexity. This needs to be corrected.

2. The current C++ code can be further optimised to reach the required O(n) time and O(1) space complexity by using swapping of elements. I would like to provide that code too.
3. There is no explanation of test cases provided. I would like to provide explained sample input and sample output.
Please assign this issue to me, would like to work on them.

@SamrendraS please get started

@SamrendraS please get started

@Abhijit2505  Please accept PR 71
https://github.com/Abhijit2505/Algorithmic-Treasure-Original/pull/71

@Abhijit2505  Please accept PR 71
https://github.com/Abhijit2505/Algorithmic-Treasure-Original/pull/71


What are the qualifications of a "Hot Event"

The @joindin/leadership-team  has been discussing several items and one has brought us to open this discussion:

What are the qualifications of a "Hot Event"

What does an event need to do to qualify to appear on the Joind.in homepage?
Currently Joind.in is working like this:

We're scoring the event based on absolute value of the date difference between the event start and now. (Source)
We take the score and subtract the result of the comment count + attendee acount divided by 5. (Source)

Some of the ideas tossed around:

Required CFP info (or past dates?)
Event image required
at least 1 claimed talk

Thanks to @mcneely for bringing the topic up and @iansltx and @ericpoe for input so far. I'll leave the discussion open for 7 days to get feedback from others (If you can read this we'd appreciate your feedback)

As mentioned elsewhere, I'm +1 on an event image requirement.
Re: the CFP requirement, user groups/meetups that don't have a formal CFP period would get filtered out by that change. Maybe Hot Events shouldn't include those types of events, but that would also filter out curated conferences.
Likewise, a claimed talk requirement would filter out networking-focused events that don't have talks per se. I'm personally fine with doing that for Hot Events but that's just me.

IMHO the aim should be minimal effort on the part of those wanting to leave feedback. Without feedback the site is nothing.
Given the above I think hot events should be events that are in progress, or have finished within the last day or so. Then anyone who is at an event goes to joind.in homepage and can click through the events.
Every PHPSW night we organise we seem to be first or second in the list. Whenever I'm at a conference the event is also 1st or 2nd in the list.  So it seems that the current formula is working. I'd vote for no change.

@DaveLiddament the rationale behind the change is that, while well-attended, well-commented, well-maintained event listings get top billing on the page, everything lower down is a mixed bag, with a bunch of events that consist of nothing more than a description (and one attendee, namely the organizer). The idea of any changes would be to ensure that events that are actually engaged in the platform don't get drowned in the noise. e.g. there are a few events on the Hot page that are ghost towns, which are keeping JestPHP (which has a logo, talks listed, multiple attendees, etc.) from the front page.

Personally I think it should be
Absolute days distance from event. Ie before event compare to start date. After event compare to end date. So 0 for currently running. 1 for either day before or day after.
Then event image yes / no
Then
Amount of comments
So all events running currently are at the top. Followers by events which either start tomorrow or ended yesterday
When two events have the same number ie 0 for currently running. Events with images take precedence. If all events have images amount of comments takes the winning spot.

As joind.in is mainly about feedback I'd filter out events that have no slots assigned (for the Hot Events). As there is also a possibility to create networking slots that's not a drawback.
I would not take CfP infos into account though. Otherwise we would need to verify whether the CfP-site actually existed which migbt not be the case anymore when the hot events are estimated...
Besides that a logo should be a general requirement unrelated to the hot events but making it a part of the algorithm can help. Perhaps we can silently ignore all events for the hot events that have no logo.


Question: Static html page, include only css

Situation
I need to have a few static html sites in my Vue project that can be displayed without JS enabled and am stuck on getting the css imported while running the dev server.
What I have done so far

Added a new HtmlWebpackPlugin entry to webpack.dev.conf.js and webpack.prod.conf.js.

  new HtmlWebpackPlugin({
    filename: 'imprint.html',
    template: 'src/assets/pages/imprint.html',
    inject: false, // inject=false, because I don't want the vue javascript
  }),

Created a new src/assets/pages/imprint.html file that shall be available besides the index.html

<!DOCTYPE html>

    <meta charset="utf-8" />
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <title>Impressum</title>
    <meta name="viewport" content="width=device-width, initial-scale=1">

    <% for (var css in htmlWebpackPlugin.files.css) { %>
    <link href="<%= htmlWebpackPlugin.files.css[css] %>" rel="stylesheet">
    <% } %>

Static page

What works
When running node build/build.js the dist/imprint.html file contains the correct css file and can be served by a webserver.
What doesn't work
When running webpack-dev-server --inline --progress --config build/webpack.dev.conf.js the styles (in index.html) are included inline and imprint.html doesn't contain any styles.
Question
How do I configure the webpack.dev.conf so that the styles are included in imprint.html the same way as in index.html while running the dev server?

Recording my solution here for anyone that might stumble across the same problem:
I was unable to manually include the css inline into the static pages but I extracted them into a separate file (like it is done when you build for production).
In webpack.dev.conf.js I added the ExtractTextPlugin,
const ExtractTextPlugin = require('extract-text-webpack-plugin')

Set the style loader extractoption to true
module: {
    rules: utils.styleLoaders({ sourceMap: config.dev.cssSourceMap, usePostCSS: true, extract: true })
}

Added the ExtractTextPlugin before the HtmlWebPackPlugin of the static site
new ExtractTextPlugin({
  filename: utils.assetsPath('css/[name].[contenthash].css'),
  allChunks: true,
}),

new HtmlWebpackPlugin({
  filename: 'imprint.html',
  template: 'src/assets/pages/imprint.html',
  inject: false
}),

Included the stylesheet file in the imprint.html template's head

  [...]
  <% for (var css in htmlWebpackPlugin.files.css) { %>
    <link href="<%= htmlWebpackPlugin.files.css[css] %>" rel="stylesheet">
  <% } %>

Closing issue.


Follow-up #2751: For Number#toString with a radix other than 10, never switch to exponential display

Follow-up to #2751
See @littledan's comment https://github.com/Microsoft/ChakraCore/issues/2751#issuecomment-326902301
/cc @xiaoyinl @littledan @bterlson

I'd like to work on this issue.

@xiaoyinl thanks!


parse discriminator value from the referenced scheme

Add vendor extension used by swagger-codegen for Swagger 2.0 as an alternative to OpenAPI 3.0 mapping feature.
For example: https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/resources/python/model.mustache

Coverage remained the same at 81.339% when pulling dcc1ff208a30f00dbed6f323c5bb86d826ccb375 on curv85:feature/openapi_2.0_discrimiator_value into d38f2f2a5a0d387e9788b1cc6cc5e96817e8c5df on Rebilly:master.

Thanks for PR 👍


ghcr.io/ko-build/ko:v0.13.0 is not signed

https://github.com/ko-build/ko/actions/runs/4386793516/jobs/7681559191
This is due to a change in cosign v2 that requires the --yes flag to confirm, which was not set on the release workflow. Related: #973
We can fix this in a few ways, none of which are super ideal. In descending preference order:

merge #977 and cut v0.13.1 -- v0.13.0 will still not be signed, but at least latest will be
define a new manual workflow that just signs the image -- it will be signed with this workflow's details, not the normal release workflow
merge #977 and move the v0.13.0 tag to point to it-- this will break Go module users since v0.13.0 is already present in the go mod cache 👎

May I do these ? ☝🆙

I think we need to decide which one we want to do first 😅

I would rather not do 3. as.. as said, it will break go modules. I think cutting a v0.13.1 is fine, or have a manual workflow to sign. So I think 1. is the easiest/quickest to do, but 2. seems fine as well 👼🏼

my vote is also to continue with the first one since I agree with the @vdemeester, this is the easiest way to go with!

as #977 merged, will you @imjasonh cut a new release 0.13.1?

will close this, resolved in v0.14.x


DEV DETECTIVE

Develop a Javascript Dev-detective  app.
Description 📜
Dev Detective is a api based github user stats finder app. It use api which is provided by github.
Requirements 🛠

HTML,CSS , JAVASCRIPT

Example:

The app must have user avatar image, number of repos, number of followers, number of following, stars , bio , social profiles

Bonuses ✨

Add a section  for following and followers.

Any number of people can work on a single issue 👨💻👨💻
This issue is open to all.🌍

Project location 👉🏻 {DEV_DEVTECTIVE}/Ayush-Vish/
👨🏻💻 Happy Coding 👩🏻💻

Hey @Ayush-Vish I wanted to work on this mini project!

@ikunal-04  I am already working on this issue.

I have raised a PR kindly merge it .

@ikunal-04 I am already working on this issue.

okay


NO-JIRA: updating ocp release task notes

What this PR does / why we need it:

It adds docs about what to do when a new release is cut.

Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/approve
/lgtm


Update end-user-pdf.md

"Export to PDF for ArcGIS map visuals from Power BI Desktop is currently not supported." point has been added under 'Considerations and limitations'. This was an action item for: https://hybridbi-csi.visualstudio.com/SR Actions - MD/_workitems/edit/227
Reference case:<PHONE_NUMBER>006768.

Learn Build status updates of commit 7d128ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/consumer/end-user-pdf.md
:white_check_mark:Succeeded

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

@mihart
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @Saurabh0507 ,
I moved your suggestion to the private repo and set it to publish.  No action needed from you :-)  In the future, you can make changes in the private repo powerbi-docs-pr since you are a Microsoft employee.  I've emailed you instructions.  Thanks!


Bug on v2.5 - When we edit and save an image which was built, IDs are lost

even if no changes was made to Build image recipes.

This is a false positive.
But I found an other bug:
When we edit&save an image which is not the latest version, the version is not given in the request.
So the server compares the last version to the version being sent instead of the version that we are editing.

Updated the title to reflect the real issue

This one seems important enough to be promoted to Proposed for sprint.
Opinions?

stale and will be obsoleted by the CIMI module resource


Is there a pure C# implementation of flex layout?

Sorry for creating an issue not directly related to xamarin/flex. But I'm planning to write a pure C# one and wondering if an implementation already exists.

Found one:
https://github.com/NZMob/Yoga.Net
Yoga.Net is a C# port of  facebook/yoga

Any findings so far? I'm looking for one as well.

We have one on Xamarin.Forms
https://github.com/xamarin/Xamarin.Forms/blob/main/Xamarin.Flex/Flex.cs


Known Object Setting

Thanks for your work!
How to train or test the 'Known Object Setting'?

Hi @liuc-v,
Results for the known object setting can be computed using the Matlab code provided by HO-RCNN codebase. You can find the command to generate those Matlab results in the readme.
Fred.


feat: add support for mini.icons

This adds explicit support for mini.icons for providing icons to oil. This generalizes the icon providing into a function that figures out which icon provider is available (preferring mini.icons over lspkind) and returning a common icon providing interface.
This also uses the glyph style in mini.icons to control the nerd_font = "auto" option since mini.icons has a built in style for not using nerd fonts and using plain text.
Lastly, this makes sure that the generalized icon provider function is lazily evaluated when it's necessary rather than on setup and caches the value so that it doesn't have to re-calculate the provider each time.

LGTM thanks!


BASIC is incorrectly labeled as Visual Basic .NET in many repositories

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[x] verified an issue has not already been logged for your issue (linguist issues).

Problem Description

URL of the affected repository:
https://github.com/search?q=10+20+30+GOTO+extension%3Abas&type=Code
While not every BASIC program begins the line numbering at 10, increments the line numbering by 10, and contains the GOTO statement, the search does return numerous BASIC programs that have been incorrectly labeled as Visual Basic .NET.
Of the roughly 13,000 search results, it appears that the majority are BASIC programs that have been incorrectly labeled as Visual Basic .NET. The presence of line numbering is a dead giveaway that these are not Visual Basic .NET programs.
Last modified on:

Today
Expected language:

"BASIC"
It is true that there are numerous dialects of BASIC (e.g. AppleSoft, Commodore, GW-BASIC), but most dialects share similar syntax and reserved words, and all dialects are collectively known as BASIC. At the very least, a file with the extension .bas and line numbering should be labeled as BASIC rather than Visual Basic .NET, which is clearly incorrect.
Detected language:

"Visual Basic .NET"

While not every BASIC program begins the line numbering at 10, increments the line numbering by 10, and contains the GOTO statement, the search does return numerous BASIC programs that have been incorrectly labeled as Visual Basic .NET.
Of the roughly 13,000 search results, it appears that the majority are BASIC programs that have been incorrectly labeled as Visual Basic .NET.

The language you're seeing in the search results is the cached results from an analysis performed prior to https://github.com/github/linguist/pull/4725 being merged and deployed way back in 2019.
Thanks to that PR, the .bas extension is now only associated with VBA:
https://github.com/github/linguist/blob/9eb9472be957108fc48b6c9f725f2100c18b7a5e/lib/linguist/languages.yml#L5837-L5845
The changes in that PR were deployed early last year, however we do not go back and reanalyse every repository that could be affected as this is incredibly resource intensive, so these repositories will continue to report the old language until such time as the files are updated which forces re-analysis and cache invalidation.

At the very least, a file with the extension .bas and line numbering should be labeled as BASIC rather than Visual Basic .NET, which is clearly incorrect.

Linguist doesn't currently know about "BASIC", but it does several other variants, and once https://github.com/github/linguist/pull/4998 has been finished and merged, it'll know about FreeBasic and will then have the .bas extension associated with that language too.
I have no idea how BASIC differs from the other forms Linguist knows about or FreeBasic, but if it's distinctly different and ideally easily distinguishable via a regex for the heuristics, we'd welcome a PR adding support.

While not every BASIC program begins the line numbering at 10, increments the line numbering by 10, and contains the GOTO statement, the search does return numerous BASIC programs that have been incorrectly labeled as Visual Basic .NET.
Of the roughly 13,000 search results, it appears that the majority are BASIC programs that have been incorrectly labeled as Visual Basic .NET.

The language you're seeing in the search results is the cached results from an analysis performed prior to https://github.com/github/linguist/pull/4725 being merged and deployed way back in 2019.
Thanks to that PR, the .bas extension is now only associated with VBA:
https://github.com/github/linguist/blob/9eb9472be957108fc48b6c9f725f2100c18b7a5e/lib/linguist/languages.yml#L5837-L5845
The changes in that PR were deployed early last year, however we do not go back and reanalyse every repository that could be affected as this is incredibly resource intensive, so these repositories will continue to report the old language until such time as the files are updated which forces re-analysis and cache invalidation.

At the very least, a file with the extension .bas and line numbering should be labeled as BASIC rather than Visual Basic .NET, which is clearly incorrect.

Linguist doesn't currently know about "BASIC", but it does several other variants, and once https://github.com/github/linguist/pull/4998 has been finished and merged, it'll know about FreeBasic and will then have the .bas extension associated with that language too.
I have no idea how BASIC differs from the other forms Linguist knows about or FreeBasic, but if it's distinctly different and ideally easily distinguishable via a regex for the heuristics, we'd welcome a PR adding support.

@lildude Thanks for the information. I've opened a pull request at #5166.

@lildude Thanks for the information. I've opened a pull request at #5166.


Bugfix admin redirect fixes

fixes jointakahe/takahe#285
Return to the correct page on a paginated list in the admin section after visiting a detail page it contains.

Set the list page index as a GET parameter in the href used to link to the detail page.
Capture the list page index in the View when creating the context
Set the list page index from the context as a GET parameter on the back link on the detail page

Works for:

[x]  announcements
[x]  federation
[x]  hashtags
[x]  identities
[x]  invites
[x]  reports
[x]  users

Thanks!


Move futures-lite to dev-dependencies

Hi, this an interesting library!
I noticed futures-lite is only used in examples/ and (non-running) documentation tests. This change moves that dependency to dev-dependencies from dependencies since it's not used by the library itself. This means it's not added to the dependency tree of the nusb's users.

Thanks!


Release 0.3 version bumps and changelog

PR checklist:

[x] make succeeds (on *nix)
[x] make reftests succeeds
[x] tested examples with the following backends:
[ ] rustfmt run on changed code

bors r=msiglreith


Add support for IPv6 in service account lookups

Closes #511

Thanks for the quick fix @jacobtomlinson! Would you be willing to publish a release with this fix soon? No worries if not, since I've already implemented a workaround in my own code, but it'd be nice if I could remove that workaround.

Sure thing! I've tagged v0.18.0 which contains this fix.


Add an option to create a randomized name

The name of the load balancer is limited to 32 characters, which can be easily violated, especially when the name is composed of many parts.

Hi, version 1.0.24 contains a variable to use a random name.


[VS Code] Could not connect to the server via. Check that you are using a valid service endpoint [ via Proxy ]

Error:
Could not connect to the server. Check that you are using a valid service endpoint
Steps to Reproduce:
I can't log in to my account behind the corporate proxy.  If i switch to no-proxy connection everything [logging in to account / starting live share etc. ]  work perfectly.
Traceback:
Agent.Rpc.Auth V] > #1 [14] auth.login(<0:>)
[Agent.Http V] < POST https://insiders.liveshare.vsengsaas.visualstudio.com/auth/token
[Agent.Http E] > POST https://insiders.liveshare.vsengsaas.visualstudio.com/auth/token => An error occurred while sending the request.
[Agent V] < #1 telemetry.httpRequestComplete: <65:35e51a44>
[Agent.Rpc.Auth V] < #1 [14] Error: Could not connect to the server. Check that you are using a valid service endpoint
[Client.Rpc.Auth E] > auth.login() error:    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.Cascade.Tracing.TracingHttpMessageHandler.<SendAsync>d__14.MoveNext() in E:\A\_work\12\s\src\Common\Tracing\TracingHttpMessageHandler.cs:line 79
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.Cascade.Agent.ForwardResponseMessageHandler.<SendAsync>d__7.MoveNext() in E:\A\_work\12\s\src\Agent\ForwardResponseMessageHandler.cs:line 55
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime...

Version Data

extensionName
VSLS

extensionVersion
0.2.206

protocolVersion
2.1

applicationName
VSCode

applicationVersion
1.21.1

platformName
MacOS

platformVersion
16.7.0

@alioguzhan Thanks for reporting the issue!  Yes, this appears to be another symptom of #86 as you already identified.  Would you mind if we merged this bug in with #86 to consolidate information in one spot?

Great!  Done.  Thanks for reporting it!!


Sanity checking of RTT values dropped them too easily (if over 60ms, not over 60s)

Fixed in commit 6dec3ca, closing issue


Including a Max Hit

Is your feature request related to a problem? Please describe.
I recently started using this extremely helpful client, one thing that I noticed is that the interface doesn't show you your max hit when you view equipment stats.
 Then the calculated number would be displayed when viewing the equipment stats.
Describe alternatives you've considered
Including the max hit in the top left corner of the client, right under the health bar of the enemy would also be a great solution.
Additional context
http://prntscr.com/kmltgx

@MagicfTail was/is working on a plugin that achieves this (https://github.com/runelite/runelite/pull/2841). I can't tell you when it'll be added to the client though.


Generic support is possible?

Feature request
I tried to use formatcp with generics, which caused an error. Is it possible to formatcp support generics?
    use const_format::formatcp;

    pub trait Named {
        const NAME: &'static str;
    }

    struct Foo;

    impl Named for Foo {
        const NAME: &'static str = "Foo";
    }

    struct Box<T>(T);

    impl<T: Named> Named for Box<T> {
        const NAME: &'static str = formatcp!("{}Box", <T as Named>::NAME);
    }

    #[test]
    fn test() {
        assert_eq!(Foo::NAME, "Foo");
        assert_eq!(Box::<Foo>::NAME, "FooBox");
    }

errors
error[E0401]: can't use generic parameters from outer function
   |
   |     impl<T: Named> Named for Box<T> {
   |          - type parameter from outer function
   |         const NAME: &'static str = formatcp!("{}Box", <T as Named>::NAME);
   |                                                        ^ use of generic parameter from outer function

error[E0080]: evaluation of constant value failed
   |
   |         const NAME: &'static str = formatcp!("{}Box", <T as Named>::NAME);
   |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ referenced constant has errors
   |
   = note: this error originates in the macro `$crate::pmr::__formatcp_impl` which comes from the expansion of the macro `formatcp` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0080]: evaluation of constant value failed
   |
   |         const NAME: &'static str = formatcp!("{}Box", <T as Named>::NAME);
   |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ referenced constant has errors
   |
   = note: this error originates in the macro `__cf_osRcTFl4A::__concatcp_inner` which comes from the expansion of the macro `formatcp` (in Nightly builds, run with -Z macro-backtrace for more info)

This would be really useful. If this worked, it could reduce heap allocations in my crate to almost zero, as opposed to multiple String allocations on every query.

Any updates? It is really useful

Is there an alternative solution, I'm having the same problem😢

It could be done like this:
pub trait Named {
    const NAME: &'static str;
}

struct Foo;

impl Named for Foo {
    const NAME: &'static str = "Foo";
}

struct Box<T>(T);

impl<T: Named> Named for Box<T> {
    const NAME: &'static str = buf_and_len_to_str(&concat_buf(<T as Named>::NAME, "Box"));
}

const fn buf_and_len_to_str(buf_len: &([u8; 60], usize)) -> &'static str {
    let buf = &buf_len.0;
    let len = buf_len.1;
    assert!(len < buf.len(), "buf is too long");
    // I didn't find a way to slice an array in const fn
    let buf = unsafe { core::slice::from_raw_parts(buf.as_ptr(), len) };
    match core::str::from_utf8(buf) {
        Ok(s) => s,
        Err(_) => panic!(),
    }
}

const fn concat_buf(left: &'static str, right: &'static str) -> ([u8; 60], usize) {
    let mut buf = [0u8; 60];
    let mut i = 0;
    while i < left.len() {
        buf[i] = left.as_bytes()[i];
        i += 1;
    }
    while i - left.len() < right.len() {
        buf[i] = right.as_bytes()[i - left.len()];
        i += 1;
    }

    (buf, i)
}

#[test]
fn test() {
    assert_eq!(Foo::NAME, "Foo");
    assert_eq!(Box::<Foo>::NAME, "FooBox");
}

Wow, thats really hacky awesome!  When you said it can be mitigated, you didn't by any chance mean the whole buffer ending up in the binary? Probably not... still many thanks :D

you didn't by any chance mean the whole buffer ending up in the binary? Probably not... still many thanks :D

You can compute the length of the buffer and pass it by generics

Sorry for not responding this entire time.
@Ddystopia  You're right that you can do it with a fixed size buffer, but the issue with this approach is that can leave the concatenated string truncated.
(I believe that the const_generic_exprs unstable feature is necessary to do this properly)
Also, you can subslice a slice in const (since Rust 1.71.0) with split_at.

Sorry for not responding this entire time.
@Ddystopia You're right that you can do it with a fixed size buffer, but the issue with this approach is that it can do any of these:
* leave the concatenated string truncated

* panic because the returned buffer isn't large enough to fit the concatenated strings

* (conditional on truncation) panic because the returned buffer doesn't end in valid utf8

(I believe that the const_generic_exprs unstable feature is necessary to do this properly, so that the resulting string is exactly as long as needed, but I haven't used this feature at all so I don't know its limitations)
Also, you can subslice a slice in const (since Rust 1.71.0) with split_at.

I want to make it clear that I left this here because someone wanted a workaround. Of course, it's too limited to be in this library. It's just that someone said that this feature would help them get rid of allocations, and for the sake of that, this kind of thing is as good as it gets. Thank you for your response

About the incomplete generic_const_exprs feature. It can be done, but it is even more unergonomic than the previous workaround.
#![feature(generic_const_exprs)]

pub trait Named {
    const NAME: &'static str;
}

struct Foo;

impl Named for Foo {
    const NAME: &'static str = "Foo";
}

struct Box<T>(T);

impl<T: Named> Named for Box<T>
where
    [(); <T as Named>::NAME.len() + "Box".len()]:,
{
    const NAME: &'static str = {
        buf_and_len_to_str::<{ <T as Named>::NAME.len() + "Box".len() }>(&concat_buf::<
            { <T as Named>::NAME.len() + "Box".len() },
        >(<T as Named>::NAME, "Box"))
    };
}

const fn buf_and_len_to_str<const LEN: usize>(buf_len: &([u8; LEN], usize)) -> &str {
    let buf = &buf_len.0;
    let len = buf_len.1;
    let buf = buf.split_at(len).0;
    match core::str::from_utf8(buf) {
        Ok(s) => s,
        Err(_) => panic!(),
    }
}

const fn concat_buf<const LEN: usize>(left: &str, right: &str) -> ([u8; LEN], usize) {
    let mut buf = [b'7'; LEN];
    let mut i = 0;
    while i < left.len() {
        buf[i] = left.as_bytes()[i];
        i += 1;
    }
    while i - left.len() < right.len() {
        buf[i] = right.as_bytes()[i - left.len()];
        i += 1;
    }

    (buf, i)
}

or
pub trait Len {
    const LEN: usize;
}

impl Len for Foo {
    const LEN: usize = <Self as Named>::NAME.len();
}

impl<T: Len> Len for Box<T> {
    const LEN: usize = <T as Len>::LEN + "Box".len();
}

impl<T: Named + Len> Named for Box<T>
where
    [(); <Self as Len>::LEN]:,
{
    const NAME: &'static str = {
        buf_and_len_to_str::<{ Self::LEN }>(&concat_buf::<{ Self::LEN }>(<T as Named>::NAME, "Box"))
    };
}


Lucene test integration and working

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1382
Description
Clear and concise code change description.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Builds | Releases | Other)
Screenshots (if applicable)
Checklist

[ ] I have read and acknowledged the Code of conduct.
[ ] I have read the Contributing page.
[ ] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[ ] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[ ] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

thanks maimoona - still a great prototype to refer to in the future. thanks for this!


fix pathJoin on windows

Please provide a description of this PR:
fixes: https://github.com/istio/istio/issues/53847

In response to a cherrypick label: new pull request created: #54256


fix: fix hardhat and ledger init

When we have a deployer configuration DeploymentsManager creates a map of address and protocol. When you read config you can't init ledger provider.
const config: HardhatUserConfig = {
namedAccounts: {
deployer: {
default: 0, // here this will by default take the first account as deployer
"goerli": 'ledger://0x84ded7b35314c823977c5794ed76e2a80f948269',
},
},

Deployed a test package<EMAIL_ADDRESS>

I have tested it with my ledger

fixed in 0.11.30
Thanks!


[RFC][DNM]Pinmux configuration code generated from Device tree

GOAL:
Aim of this PR is to enable generation of pinmux configuration code based on information extracted
from device tree files.
At compilation, board device tree is parsed and a pinmux dedicated header file is generated
under ${PROJECT_BINARY_DIR}/include/generated.
This file holds a pinmux array. It is included by board pinmux code and parsed at pinmux init.
File name and array name are based on pinmux node compatible.
STATUS:

Supports single and mutli pinmux instances
Currently tested on disco_l475_iot1 (single instance pinmux) and frdm_k64f (multi instance pinmux)
Tested on samples/hello_world (both) and samples/sensors/hts221 (disco_l475_iot1)
Limited to UART and I2C nodes (because  only thse nodes contrain pinmux information on above boards dts files)

Preliminary work:
Some preliminary work was needed to enable this:

extract_dts_include.py: generate struct dictionary that store all decive nodes information into structs (@agross-linaro)
yaml: Add yaml pinmux base file
dts: nxp: add label to pinmux nodes
nxp: move PORT_PCR_MUX to pinmux_mcux driver

TODO:

So far, extract_dts_includes flies knows it deals with a pinmux node by checking "pin" string is contained in node yaml description. Instanciate a yaml information to store node type as "pinmux", so this type could be checked against.

CI faces an error in my code when running sample/synchronisation for target sam_e70_xplained.
I don't face this issue in my environment (tested on python3.4.X and 3.5.3)..

@erwango : Try rebasing onto the latest version of master. CI rebases this branch onto master before testing it.

@SebastianBoe : Thanks for the hint! Unfortunately, I'm already on top of arm branch..

recheck

Moved to master branch as arm branch is rebased on master branch by CI before testing and it showed some conflicts at that step

why is this submitted to the ARM branch anyways, this device tree which is cross-arch

Moved to master branch as arm branch is rebased on master branch by CI before testing and it showed some conflicts at that step

hmm, when you submit a PR to the ARM branch, the PR is rebased on top of the branch, not master.

hmm, when you submit a PR to the ARM branch, the PR is rebased on top of the branch, not master.

This is what I was told by @SebastianBoe , but I was facing some issues in CI I could not reproduce on my end (top of arm branch at that time). I've been able to reproduce CI issues only after rebasing on top of master. Hence this comment...
Anyway, here I am on master branch, let's go on here.

@dbkinder: You might have been added during while I was changing branch from arm to master, and hence some commits showed up that were not related to my PR. And then I kept you in reviewers list.
No documentation to review for now. Ill remove you from the list.

Don't know if something changed in CI recently, but I now get following error:
"author email (Erwan Gouriou<EMAIL_ADDRESS>needs to match one of the signed-off-by entries."
Checked all the commits but couldn't find the issue

@nashif pushed something like that recently, yeah.
E.g. this commit https://github.com/zephyrproject-rtos/zephyr/pull/5021/commits/bf4def723a7d62cd4d38729b61d1eee3520dd30f has: Signed-Off-By: Erwan Gouriou<EMAIL_ADDRESS>(count u's in email's surname ;-) ).

Accidentally closed - Re-opening


fix(cli): fix the wrong graph in "gas log" when merging branches

When merging the left branch, the right branch also needs to move to the left

4c564ea (b3) 15
a3f25ce 13
ebb775e 8
| * 2da7d6e (b1) 14
| * aeac003 6
| | * 8c9e82e (main) 12
| | | * 1c22915 (b5) 11
| | |/
| |/|
| * | bec94e0 a
| | | * ef0e4c2 (b2) 10
| | | * dee8986 7
| | |/
| |/|
|/| |
| | d7677e0 4
| | e1d77c4 3
|/|
| | * f478341 (b4) 9
| |/
| * dc2a3cc 5
|/
d0eeb20 2
bce9056 1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 86.762%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7537

Relevant Lines:
8687

💛  - Coveralls

the gas log graph in commit message should be put into a code block


Start local storage plugin

Since implementation is pretty simple included it here.

As discussed we normally use feature branch workflow. Internal developers have write access and can use this workflow. feature branch -> main. Developers can also choose to use a fork but that's only necessary for external contributors.


Option to include/exclude taxes from discount

Similar to how we allow users to choose whether to include/exclude shipping price from discounts, we should also let them do the same for taxes.
For example:
Currently if a seller sets a 100% off discount code for a Printful product, the buyer still has to pay the tax amount. The seller should have the option to cover taxes as well and have the buyer pay $0.

@gullible1 could you help me understand this feature request a bit, please?  Is the desired outcome of this issue to:

Add a checkbox to the Discount page in admin where a shop admin can toggle Exclude Taxes.
When the checkbox is selected, then no taxes are added to the cart total.
When the checkbox is not selected, then taxes are calculated as expected, and added to the cart total.

Is this correct, or have I misunderstood your example?

@theprivileges When the checkbox is selected, the discount should not apply to the taxes. But the taxes should remain in the cart. In some sense it is very similar to the existing option "Exclude Shipping Fees" but for taxes vs shipping.
For example, assuming both the "Exclude taxes" and "Exclude Shipping Fees" are selected. Here is an example of a cart:
Subtotal: $100
Taxes: $8.5
Shipping: $5
Discount 10% off: -$10
Total: $103.5
Notice how the discount only applied to the subtotal and not to the taxes nor the shipping.
Let us know any question! 👍

Hi everyone,
I've implemented the requested feature in separate branch, and am in the process of writing a unit test to confirm that the cart totals are calculated as expected. I do have a concern to bring up at this point:
If a seller chooses to configure Taxes on their UI (from Settings > Checkout), it seems to me like there isn't enough information for them to know how taxes are calculated. For example, how many new sellers would know that in order to generate @franckc's example cart, they would have to set the Tax Rate to 8.5%? In other words, are there places in the world where the norm is that a discount is applied on a product before calculating the tax on that 'final' product price?

@micahalcorn @gullible1 Could you comment on ^^^ ?

All,
I'm submitting a Pull Request for review in relation to this issue. If and when that is approved, all related changes will "go live", and be reflected on the user end.
Best,
Rajath


PHP process disconnected ,and then The PHP Language Server server crashed 5 times in the last 3 minutes.

win10
vscode 1.18.0
node 8.1.0
npm 5.4.2

PHP Fatal error:  Method AdvancedJsonRpc\ErrorResponse::__toString() must not throw an exception, caught AdvancedJsonRpc\Error: Malformed UTF-8 characters, possibly incorrectly encoded in C:\Users\BYENDS.vscode\extensions\felixfbecker.php-intellisense-1.5.4\vendor\felixfbecker\language-server\src\Protocol\Message.php on line 0

Hi there!
Where are we on this? I can confirm the bug still exists. I tried the solution provided by @XiaoXuanBing , but even though the process does not crash anymore, I don't have any intellisense... :(
Thanks guys!