tf.image.resize_images() expects statically known rank

It looks like the op tf.image.resize_images() has no effect on the image shape when the parameters new_height and new_width are dynamically computed (this is possible since the fix of issue #1001).
For example, this piece of code
shape = tf.shape(image)
height = shape[0]
width = shape[1]
new_shorter_edge = tf.constant(400, dtype=tf.int32)

height_smaller_than_width = tf.less_equal(height, width)
new_height_and_width = tf.cond(
    height_smaller_than_width,
    lambda: (new_shorter_edge, _compute_longer_edge(height, width, new_shorter_edge)),
    lambda: (_compute_longer_edge(width, height, new_shorter_edge), new_shorter_edge)
)

image = tf.image.resize_images(image, new_height_and_width[0], new_height_and_width[1])
image = tf.Print(image, [tf.shape(image), height, width, new_height_and_width[0], new_height_and_width[1]])
return tf.image.random_crop(image, [224, 224])

throws the following exception:
I tensorflow/core/kernels/logging_ops.cc:79] [122 160 3][122][160][400][524]
W tensorflow/core/common_runtime/executor.cc:1096] 0x7f9c08008490 Compute status: Failed precondition: width must be >= target_width: width = 160, target_width = 224

So as you can see in the first line of the printout, the shape of the image is not affected by tf.image.resize_images. Is this a bug or am I doing something wrong?

I also made up a workaround for this:
image = tf.expand_dims(image, 0)
image = tf.image.resize_bilinear(image, tf.pack(new_height_and_width))
image = tf.squeeze(image, [0])
image = tf.Print(image, [tf.shape(image), height, width, new_height_and_width[0], new_height_and_width[1]])
return tf.image.random_crop(image, [224, 224])

for example prints
I tensorflow/core/kernels/logging_ops.cc:79] [400 711 3][281][500][400][711]

@shlens: Do you know the correct way to fix this?

resize_images is light wrapper around several resizing kernels, including resize_bilinear, resize_bicubic, etc. The light wrapper functionality is intended to "assist" with the shape information and 3d and 4d images seemlessly. If you do not need all of these "assists" for shape information and 3d/4d images, could you call the underlying functions directly, e.g. resize_bilinear?

I renamed the issue.  Ideally resize_image wouldn't require shapes to be known statically; certainly there's nothing fundamental about the underlying ops that requires it.

I'll work on this

@fayeshine: Thank you!  Let me know if you have questions.

@girving , tf.image.random_crop has changed to tf.random_crop.
And now this bug seems to be solved naturally.
I can use resize_images to correctly resize images using the following codes.
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets('MNIST_data', one_hot=True)
import matplotlib.pyplot as plt
array = mnist.train.images[:10].reshape([10,28,28,1])
plt.imshow(x[5,:,:,0])
images = tf.convert_to_tensor(x, name='images')
def _compute_longer_edge(height, width, new_shorter_edge):
return width*10
shape = tf.shape(images)
height = shape[1]
width = shape[2]
new_shorter_edge = tf.constant(400, dtype=tf.int32)
height_smaller_than_width = tf.less_equal(height, width)
new_height_and_width = tf.cond(
height_smaller_than_width,
lambda: (new_shorter_edge, _compute_longer_edge(height, width, new_shorter_edge)),
lambda: (_compute_longer_edge(width, height, new_shorter_edge), new_shorter_edge)
)
images_ = tf.image.resize_images(images, new_height_and_width[0], new_height_and_width[1])
#image = tf.image.resize_bilinear(xxx, tf.pack(new_height_and_width))
images_ = tf.Print(images_, [tf.shape(images_), height, width, new_height_and_width[0], new_height_and_width[1]])
res = sess.run(tf.random_crop(images_, [10,224, 224,1]))
plt.imshow(res[5,:,:,0])

Aside: When you paste code format it by indenting four spaces; otherwise the code's indentation will go away.
In the code you've shown the sizes are statically known since images comes from convert_to_tensor.  To exhibit the bug you'll have to make a tf.placeholder with no shape so that shape inference doesn't kick in.

@girving I tried to use tf.placeholder with no shape, however I still cannot reproduce this bug.
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets('MNIST_data', one_hot=True)
import tensorflow as tf
sess = tf.InteractiveSession()
import matplotlib.pyplot as plt
arr = mnist.train.images[:10].reshape([10,28,28,1])
plt.imshow(arr[5,:,:,0])
#images = tf.convert_to_tensor(arr, name='images')
images = tf.placeholder(tf.float32, shape=[None,None,None,None])
#%%
def _compute_longer_edge(height, width, new_shorter_edge):
return tf.cast(width*new_shorter_edge/height, tf.int32)
shape = tf.shape(images)
height = shape[1]
width = shape[2]
new_shorter_edge = tf.constant(400, dtype=tf.int32)
height_smaller_than_width = tf.less_equal(height, width)
new_height_and_width = tf.cond(
height_smaller_than_width,
lambda: (new_shorter_edge, _compute_longer_edge(height, width, new_shorter_edge)),
lambda: (_compute_longer_edge(width, height, new_shorter_edge), new_shorter_edge)
)
images_ = tf.image.resize_images(images, new_height_and_width[0], new_height_and_width[1])
#image = tf.image.resize_bilinear(images, tf.pack(new_height_and_width))
images_ = tf.Print(images_, [tf.shape(images_), height, width, new_height_and_width[0], new_height_and_width[1]])
res = tf.random_crop(images_, [10,224,224,1]).eval(feed_dict={images: arr})
plt.imshow(res[5,:,:,0])
#%%
arr2 = mnist.train.images[10:20].reshape([10,56,14,1])
res2 = tf.random_crop(images_, [10,224,224,1]).eval(feed_dict={images: arr2})
plt.imshow(res2[5,:,:,0])

Please reformat the code as I mentioned so that it's easier to read.

@girving , sorry, I've formatted it by indenting four spaces. I was previously thinking that you mean use 4 spaces to replace tab. :smile:

@fayeshine: So close!  It looks like resize_images goes out of its way to break only if the rank of the input tensor is unknown.  Try using shape=None in your placeholder instead of shape=[None, None, None, None] and you should see a problem.  However, I'm not sure if this is the same problem that  @mackcmillion was experiencing.

@girving Yes, you are right, in this case resize_bilinear can work but resize_images throw an exception. I'll see what cause the problem and try to solve it.

@fayeshine: Yep, the cleanest way is probably to make all the C++ resize ops accept any number of dimensions >= 3 and treat the initial dims-3 dimensions as batch.  There are a lot of other ops that behave this way: mixing together the last few dimensions and broadcasting over the first few.

As noted in #2286, resize_image_with_crop_or_pad has the same problem.  It should be fixed in the same way.

I saw that resize_image_with_crop_or_pad was fixed, does this apply to the other cropping image ops or only this one? For instance crop_to_bouding_box.

It applies to anything that calls  _Check3DImage with require_static=True (the default).  The require_static option should go away entirely.  Looks like crop_to_bounding_box uses require_static=False, so it's clean.

Are there any functions for 3D Nearest neighbour upsampling in tensorflow? Any workaround for this?

Automatically closing due to lack of recent activity. Since this issue is old at this point, please reopen the issue if it still occurs when tried with the latest version of Tensorflow. Thank you.

I think this is still a problem that should be addressed. Is a simple operation and it could be useful if you don't want to depend on other image libraries to do simple image transformations.
Here is an example that shows the problem:
import tensorflow as tf

encoded_image_data = tf.placeholder(dtype=tf.string)
decoded_image = tf.image.decode_image(encoded_image_data, channels=3)
resized_image = tf.image.resize_images(decode_image, (800, 600))
encoded_image = tf.image.encode_jpeg(resized_image)

import urllib2
fd = urllib2.urlopen('https://c8.staticflickr.com/6/5753/20965045223_b809050285_o.jpg')
data = fd.read()

with tf.Session() as sess:
    sess.run([encoded_image], { encoded_image_data: data})

output:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-41-bbd519ee1c20> in <module>()
      4 encoded_image_data = tf.placeholder(dtype=tf.string)
      5 decoded_image = tf.image.decode_image(encoded_image_data, channels=3)
----> 6 resized_image = tf.image.resize_images(decode_image, (800, 600))
      7 encoded_image = tf.image.encode_jpeg(resized_image)
      8

/home/fmilo/anaconda2/envs/tensorflow_built/lib/python2.7/site-packages/tensorflow/python/ops/image_ops_impl.pyc in resize_images(images, size, method, align_corners)
    730   images = ops.convert_to_tensor(images, name='images')
    731   if images.get_shape().ndims is None:
--> 732     raise ValueError('\'images\' contains no shape.')
    733   # TODO(shlens): Migrate this functionality to the underlying Op's.
    734   is_batch = True

ValueError: 'images' contains no shape.

For anyone else who runs into this problem, using the example above with the assumption of 3 Dims:
decoded_image = tf.image.decode_image(encoded_image_data, channels=3)
decoded_image.set_shape([None, None, None])
resized_image = tf.image.resize_images(decode_image, (800, 600))

This allows ndims to be returned whilst allowing flexibility of resizing differently sized images.

Had similar problems and following the last comment seems to have fixed it.
Basically I just added .set_shape([None, None, None]) to my image before the resize.  Does feel like a workaround to a bug.

Based on https://github.com/tensorflow/tensorflow/issues/9356, using decode_png or decode_jpeg to decode the image (which seems to be the most popular source of this issue) can also resolve this issue. This also seems like a workaround to a bug.

@dcunhas you are right. We can decode images using decode_png and decode_jpeg like this:
    image_string = tf.read_file(filename)
    image_decoded = tf.cond(
        tf.image.is_jpeg(image_string),
        lambda: tf.image.decode_jpeg(image_string, channels=3),
        lambda: tf.image.decode_png(image_string, channels=3))

@Alex-zhai You don't need to go that far. You can directly do
image_string = tf.read_file(filename)
image_decoded = tf.image.decode_jpeg(image_string, channels=3)

Based on the documentation (and my own usage), "This op also supports decoding PNGs and non-animated GIFs since the interface is the same, though it is cleaner to use tf.image.decode_image." tf.image.decode_png is the same.

I found the solution
tf.enable_eager_execution()

write this to the top of your add.
or
img_tensor = tf.image.decode_jpeg(img_raw,channels=3)
img_final = tf.image.resize_images(img_tensor, [192, 192])
img_final = img_final/255.0
print(img_final.shape)


HIVE-28292: Optimize SHOW TABLES|VIEWS statements

What changes were proposed in this pull request?
Make some improvements in SHOW TABLES statement:

Push down table pattern to metastore
Request table names instead of table objects
Change index UNIQUETABLE(TBL_NAME, DB_ID) to UNIQUETABLE (DB_ID, TBL_NAME) and drop index TBLS_N49(DB_ID)

Why are the changes needed?
Improve the performance of SHOW TABLES|VIEWS statements.
Does this PR introduce any user-facing change?
No.
Is the change a dependency upgrade?
No.
How was this patch tested?
Passing existing tests and do benchmark tests.
 java -jar hmsbench.jar -H localhost --savedata /tmp/benchdata --sanitize -N 100 -N 10000 -o bench_results.csv -C -d testbench_http --params=100 -M 'listTables.*'

Before this patch

Operation       Mean    Med     Min     Max     Err%
listTables      1.52074 1.50483 1.39134 1.90837 4.07209
listTables.100  1.68746 1.68214 1.57137 1.87610 2.93073
listTables.10000        16.9628 16.9202 16.4026 19.1795 1.77736

After this patch

Operation       Mean    Med     Min     Max     Err%
listTables      1.50856 1.49130 1.39420 1.76155 4.03957
listTables.100  1.66239 1.65599 1.52580 1.96438 3.59206
listTables.10000        10.5922 10.2885 9.74368 15.3253 9.76028

@dengzhhu653 @deniskuzZ @zhangbutao Could you please help review this patch?

@deniskuzZ could you take a look again?


Use Response.StatusCode not .Status

Fixes #12

lgtm fix it @fdr fix it fix it!

Also, how do we bundle this into transferatu ?

Or more generally, @fdr, what's the best way to build htcat for the platform?

"go build"?  There are no special requirements.  I used to use gobot when that existed.

@tt, @naaman, @ryanbrainard would know best

And @gudmundur.

curl -O http://gobot-builder.herokuapp.com/build/github.com/htcat/htcat/cmd/htcat

I take back my lgtm. This is missing similar changes elsewhere: #14 is based on this and handles those as well.

@tt thanks, but gobot seems to have bit rotted and the current deployment is on go1.1--I think we'd like something more modern. I've built locally on my Ubuntu laptop for now.

godist sort of replaces gobot, but you're probably better off just setting up a travis job that uploads the binary to s3. Building locally is probably fine for anything but prod.

This application will be able to read and write all public repo data. This includes the following:

Noping out of there. I don't see any docs in the repo for godist, so not sure if there's a way to use it without that. But thanks.
This is for a vendored binary, so I'd love to avoid dealing with uploading to S3 as an intermediate step (since we'll just have to download right back to vendor it). I'll probably deal with one-off builds for now until I figure out either a Travis setup or get gobot working again.

Closing in favor of #14.


arch_interface: Remove duplicate irq_offload_routine_t typedef.

This commit removes duplicate 'irq_offload_routine_t' typedef
declaration in sys/arch_interface.h.
This typedef is provided by irq_offload.h and, since this header file
is included at the top of sys/arch_interface.h, it is guaranteed to be
defined for arch_irq_offload definition.
While this does not cause a compilation error when compiling with GCC
4.6 and above, GCC 4.5 and below strictly enforce the C99 standard and
do not allow redeclaration of the same typedef in the same scope.
Signed-off-by: Stephanos Ioannidis<EMAIL_ADDRESS>

Similar to the issue #20605.


Use uiSchema to customize oneOf with different types

Prerequisites

[x] I have read the documentation;
[x] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.

Description
I would like to use oneOf with different types. (another way of solving the problem in issue #282)
Steps to Reproduce
Use this schema:
{
  "type": "object",
  "properties": {
    "name": {
      "oneOf": [
        {
          "type": "string"
        },
        {
          "type": "number"
        }
      ]
    }
  }
}

Playground link
Expected behavior
Want to be able to select different types.
Actual behavior
I get this:

Version
1.2.0

I have encountered nearly the same issue in that I would like to use oneOf with an object and a basic type. I also want to use this oneOf inside an array. The oneOf works differently inside and outside of an array.
Here is the oneOf schema by itself:
{
  "oneOf": [
    {
      "type": "integer"
    },
    {
      "type": "object",
      "properties": {
        "elt1": {
          "type": "integer"
        },
        "elt2": {
          "type": "string"
        }
      },
      "required": [
        "elt1",
        "elt2"
      ]
    }
  ]
}

With a playground link.
This works fairly well although it is a little glitchy.
And the same schema within an array:
{
  "type": "object",
  "properties": {
    "myArray": {
      "type": "array",
      "items": {
        "oneOf": [
          {
            "type": "integer"
          },
          {
            "type": "object",
            "properties": {
              "elt1": {
                "type": "integer"
              },
              "elt2": {
                "type": "string"
              }
            },
            "required": [
              "elt1",
              "elt2"
            ]
          }
        ]
      }
    }
  }
}

With a playground link
This displays completely different behavior from the previous schema and does not allow input for the second option.

@Rasstrelnikov Since you have reported a separate problem, I've opened a new issue to track it https://github.com/mozilla-services/react-jsonschema-form/issues/1168


Fix typo that was causing an error on drawing a memo

Fix a typo in the js/app.js file

PS - I don't need it to have hacktoberfest-accepted. I already see a lot more than enough on my hacktoberfest dashboard.


More localization

Ok, different localization stuff. :-)
This fixes issue #250.
Also, there were two functions that handled localization stuff. I refactored them into one.
NSBundle (BLTRExtensions) now handles the localization stuff. QSLocalization is still there, because there is still some weirdness going on. But less than before. :-)

Ouvrir should be spelled Ouvier in the French localization.

except it's in reverse ;) Apparently it's only misspelled in the triggers prefpane.

Can you guys open a new support ticket for this? :)
Where exactly is the typo?
Thanks!

done
https://github.com/quicksilver/Quicksilver/issues/554


Logging option

This PR adds an option to toggle console logging on and off: enableLogging. It closes #10

followed code style
compact scope
does not affect console.error
packed and tested in example app

For the static methods on the WebAuthn class, I added an argument enableLogging but defaulted it to false so any existing implementations won't break.

@Terrahop @EternalDeiwos @christiansmith was hoping one of you could take a look, it's a quick one

Hey @inorganik, could you please roll back your change to the version and I'll merge this (and publish shortly afterward)

@inorganik I've merged and published 0.2.0 but that being said, this package is still unstable and you shouldn't use it for production code.

Thanks! I'm using it in a separate repo that's meant to demo webauthn. I'll be hosting it and didn't want huge server logs.
If I'm not mistaken this lib is based on this repo by Yuriy Ackermann? This is a nice packaged adaptation, and it has a nice fix for self-signed certs (which I PR'd into that repo as well). Although you have the disclaimer, I think after you add a build, I don't see why this couldn't be used in production.

some tests might be good too 🙂


fix: added warning for dangling referrers index deletion

This PR tries to resolve #615.

Codecov Report

Merging #619 (4ae0f85) into main (0ec3b3d) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #619      +/-   ##
==========================================
- Coverage   34.43%   34.30%   -0.14%
==========================================
  Files          32       32
  Lines        1844     1851       +7
==========================================
  Hits          635      635
- Misses       1188     1195       +7
  Partials       21       21

Impacted Files
Coverage Δ

cmd/notation/sign.go
37.71% <0.00%> (-2.47%)
:arrow_down:

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

Should the exit code be zero? /cc @toddysm

@sajayantony I believe so and avoid braking any scripts.
Also, is this the only message we will show:
Warning: failed to delete dangling referrers index sha256:xxx for referrers tag sha256-1ba1561: DELETE "https://ghcr.io/v2/demo-user/python/manifests/sha256:e1b7d6": response status code 405: unsupported: The operation is unsupported.

I believe there should still be a confirmation that the operation succeeded and the signature got attached to the artifact. We should show the standard message we show when signature is attached + the warning.

Should the exit code be zero? /cc @toddysm

Yes, the exit code should be 0. Once the warning is printed out, Successfully signed... is printed out and finally return nil.

@sajayantony I believe so and avoid braking any scripts.
Also, is this the only message we will show:
Warning: failed to delete dangling referrers index sha256:xxx for referrers tag sha256-1ba1561: DELETE "https://ghcr.io/v2/demo-user/python/manifests/sha256:e1b7d6": response status code 405: unsupported: The operation is unsupported.

I believe there should still be a confirmation that the operation succeeded and the signature got attached to the artifact. We should show the standard message we show when signature is attached + the warning.

This warning message is an addition to the original result. Can refer this in the code.

Closing this PR due to suggestion here: https://github.com/notaryproject/notation/issues/615#issuecomment-1500053692.
IMO, we should think through this process again before implementation.

Reopened based on discussion.


check user can modify collection before advertising inbox service

Don't show Inbox service for users with read-only collection-level access permissions. Based on the discussion in https://github.com/eclecticiq/OpenTAXII/issues/152

@traut can you review?


Composer not setting default service in Try It

 Issue by anupama-pathirage
Thursday Nov 30, 2017 at 08:17 GMT
Originally opened as https://github.com/ballerina-lang/composer/issues/4765

Description:
Refer https://github.com/ballerinalang/ballerina/issues/4173 for more details.

Implementation has differed


[CI] Ensure that htmltest-config warnings fail GH check

Ensures that when scripts/htmltest-config.pl reports a warning, that it gets caught by the GH link-check workflow
As it stands, this PR should fail until I fit the pt docs index. Let's see if the check catches the warning now.

Good, the warning is being caught, see https://github.com/open-telemetry/opentelemetry.io/actions/runs/11858217525/job/33048535443?pr=5612:
Run scripts/check-build-log.sh
WARNINGs or ERRORs found in build log:
WARNING: Unrecognized htmltest IgnoreDirs config from front matter in file 'content/pt/docs/_index.md': IgnoreDirs: {}
npm warn exec The following package was not found and will be installed<EMAIL_ADDRESS>


Fix typos in setup guide for ingestion client.

Purpose
Fixes a couple of typos in the setup guide.
Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

/azurepipelines run


additional vocabularies

Is it possible to add additional vocabularies to the @context? I understand this library is intended for use with schema.org, but in the case where we may need to add additional vocabulary terms that don't exist in schema.org, it might be a useful feature.

You can create as many custom types as you want to.

Thank you, but how do I add the additional vocabulary to the @context field?
Should it be something like this?
Schema::context(["https:\/\/schema.org", ["nc" => "http://release.niem.gov/niem/niem-core/4.0/#"}]);

The BaseType allows you to return your own context for the type.
https://github.com/spatie/schema-org/blob/2adeec32d1afb4aaf3299eccb9aa151b1b98868b/src/BaseType.php#L17-L20


React Native Flux Navigation Discussion.

React Native Flux Navigation Discussion
Repo for Test Code
Branch: Navigation
react-native version: 0.7.1

@jsierles

This looks like a fine example combining flux and Navigator, but issues should be used for bug reports or questions. Is there an issue here? Cheers!

Agree with @jsierles Issues should be kept for issues.
@brentvatne close

Thank you so much for looking into this! @piyushchauhan2011 - can you repost this on https://discuss.reactjs.org please? I'd love to chat about it there, I think there is a lot of work that needs to be done in thinking about how we can manage the Navigator routes in a clean way for larger apps, or those with more complicated nested navigation structures.

Yep will do :smile: Thanks :+1:

@brentvatne https://discuss.reactjs.org/t/react-native-navigator-best-way-to-model-in-a-large-app/888?u=piyushchauhan2011


Zoma/simplify condition v2

Fixes #2648
Proposed Changes

Testing

A few minor things, otherwise lets get this checked-in.  I want to integrate with the new LUIS model stuff which is working now.  Thanks for pushing through all the issues on this!  The next simplification we should probably do is to separate the current Ask trigger into two parts, one for missing and one for change.  That is connected to the new model I have though so I will do that.  What was the fix for the scope of exported functions issue?


csvgrep -m with non-ascii pattern gives UnicodeDecodeError

This works ok:
echo -e 'name\nsome Вася' | csvgrep -c name -m some

But this gives UnicodeDecodeError:
echo -e 'name\nsome Вася' | csvgrep -c name -m ася

Possibly pattern is converted to unicode with ascii by default? Use utf-8 maybe.

I can't reproduce on master. Likely fixed by #486, for same reason as #477.


install released dem-sticher, not dev version

This will ensure more reproducible builds, at the expense of needed to release dem-sticher to incorporate changes

Closing as this is already done, but better:
https://github.com/ACCESS-Cloud-Based-InSAR/DockerizedTopsApp/blob/dev/environment.yml#L40


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 16:57:29 on 2023.08.04
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 34618
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=34618

Problem has been resolved in 3m 4s at 17:00:33 on 2023.08.04
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 34618


Fix race condition in TestNotifierSendsUserIDHeader

As seen here and here
The initialization of the set of Alert Managers by service discovery is async, so the Send() can happen before there are any to deliver to.

@jml I didn't spot such a way.


Request to add new exchanges

Hi, first of all, your app is great, thank you very much! It would be awesome if you could add YouHodler and Nexo in your database! Thanks again

Thanks for using the app.
Regarding the new exchanges (YouHodler and Nexo).
Unfortunately, I could not find any information about these exchanges provide a public API. This will not allow them to be added, because the app receives price information directly from exchanges through the API.
Let me know if you have more information.


Elasticlient ACID Transaction Support

Can we achieve ACID transactional support using Elasticlient  (i.e.,  commit and rollback operations) .?
For eg :- when a document is added into a index which is in a transaction, we need to add/update the subset of that in the index for real time search. In case any error happens in table persist or elastic search index persist, the entire transaction should roll back. Is that case possible with Elasticlient .? If not, any other way to achieve this .?

I don't believe Elasticsearch supports any transactions, as that would require distributing locks across many nodes in the cluster. If you need to synchronize across several indices, you need to handle this in your application.


Correction in README.md

Wrong :
Return true to keep the keyboard open else return false to close the keyboard
Correct :
Return false to keep the keyboard open else return true to close the keyboard

Thanks for the PR Kunal :)


Resize the size of dragged nodes on canvas

How can one resize the dragged nodes on the canvas pane? Currently with default theme an option for resizing the nodes and for updating it is not present. Basically on click of the node, an outer line can be displayed and then using the mouse how can one resize the nodes?  Does the library have functionality to resize the nodes in work area of screen? Where can one find documentation related to the library?

Hello,
No, the library does not have support for resizing nodes.
Jero

Thanks Jero.
Do we have any documentation for drawflow library?

Only the readme file.
View simple example

https://github.com/jerosoler/Drawflow/issues/420#issuecomment-1125092595

Is there an option to have file upload functionality for that node that can be dropped to the canvas area and get details of the file upload in export of graph?


[WIP] One frame and multiple frames - Inverse kinematics

[x] Linked issue related to your pull request using '#XXX'
[x] Used the tag [WIP] or [RTR] for on-going changes, or for ready to review
[x] Did you write new tests to cover your changes?
[x] Did you write a proper documentation (docstrings and ReadMe)

#49 #21

This change is

Codecov Report
Base: 82.12% // Head: 82.49% // Increases project coverage by +0.36% :tada:

Coverage data is based on head (0e71c46) compared to base (7ed3d88).
Patch coverage: 96.69% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   82.12%   82.49%   +0.36%
==========================================
  Files          51       52       +1
  Lines        3424     3496      +72
==========================================
+ Hits         2812     2884      +72
  Misses        612      612

Impacted Files
Coverage Δ

bionc/bionc_casadi/joint.py
91.42% <ø> (ø)

bionc/bionc_casadi/natural_segment.py
81.04% <ø> (ø)

bionc/bionc_numpy/natural_marker.py
51.63% <ø> (ø)

bionc/model_creation/generic_data.py
100.00% <ø> (+12.50%)
:arrow_up:

bionc/model_creation/marker_template.py
97.29% <ø> (ø)

bionc/protocols/biomechanical_model.py
95.31% <ø> (+8.16%)
:arrow_up:

bionc/protocols/natural_segment.py
70.50% <ø> (+7.68%)
:arrow_up:

bionc/bionc_casadi/natural_axis.py
54.54% <50.00%> (ø)

bionc/bionc_numpy/inverse_kinematics.py
96.62% <96.62%> (ø)

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

... and 10 more

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

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

@pariterre, I can't run my online tests because of these lines on bioviz. Would you happen to know why?
https://github.com/pyomeca/bioviz/blob/9bce9ec561ea137ab5e346984a0bc2f27e9dd008/bioviz/biorbd_vtk.py#L44-L48


[Bug]: src prop has the wrong type

What happened?
I was trying to get the Cubemap adapter to work with the component, but TS was protesting passing the configuration. TS expects the src to be string, despite the core library allowing also array and object.
**PS: this caused me to try to use the panorama prop, which caused even more problems
What should have happened?
TS should not have raised an error for a valid input.
Solution
I propose allowing the src prop to be of type string | string[] | object and depracating the panorama prop.
Code

// TS raises error with src (expected string)
<ReactPhotoSphereViewer
  src={{
    left: './cubemap/_nx.jpg',
    front: './cubemap/_pz.jpg',
    right: './cubemap/_px.jpg',
    back: './cubemap/_nz.jpg',
    top: './cubemap/_py.jpg',
    bottom: './cubemap/_ny.jpg',
  }}
  height='80svh'
  width='60%'
  adapter={CubemapAdapter}
></ReactPhotoSphereViewer>

Sandbox Link
https://codesandbox.io/p/sandbox/htqxyv
Library Version
6.0.0
What operating system are you using?
Windows
What browser are you using?
Chrome
Logs
couldn't get it to log anything
Interest to fix the bug

[ ] I would like to fix this bug!

Hi, The sandbox url is not working, please fix

Changed the link, it should be working now.

Not exactly, it's the same problem, just a more specifically identified the exact cause


Set up unstable EC2 instance with Terraform

This will require:

a separate AWS account
some credentials
a bucket where to store Terraform state

We don't necessarily need a big instance to start with, as we will be running just a few services for a while and we should be able to recreate/reboot with a bigger size as needed.

Close when https://github.com/libero/infrastructure/pull/2 is completed


linux编译yara出现错误

command-line-arguments
<EMAIL_ADDRESS>running gcc failed: exit status 1
/usr/bin/ld: /usr/lib/gcc/x86_64-linux-gnu/13/crtbeginT.o: relocation R_X86_64_32 against hidden symbol `TMC_END' can not be used when making a shared object
/usr/bin/ld: failed to set dynamic section sizes: bad value
collect2: error: ld returned 1 exit status

Sorry, I don't speak Chinese.

ok, this question is resolved，flag value should be added during program compilation


Feature/show session info in title

Adds current session/lapse to title

Ads tests for title

closes #15
Before:

After:

eslint is not happy about emoji a11y: Emojis should be wrapped in <span>, have role="img", and have an accessible description with aria-label or aria-labelledby
I think since they are in title it's not a concern... we could // eslint-disable-next-line it or use something like 25:00 | Pomodoro

I really liked the emojis on the title, I think we can try to use Unicode for the emojis instead of placing them directly, this we also show the emoji type for the system the user is using.
Not sure if it works, but I think it's worth the try.
We can try to search for the emojis Unicode here.

Codecov Report

Merging #21 into master will decrease coverage by 1.44%.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master   #21      +/-   ##
=======================================
- Coverage   51.44%   50%   -1.45%
=======================================
  Files          29    30       +1
  Lines         381   392      +11
  Branches       41    41
=======================================
  Hits          196   196
- Misses        167   178      +11
  Partials       18    18

Impacted Files
Coverage Δ

src/pomodoro/components/PomodoroPage.js
0% <ø> (ø)
:arrow_up:

src/pomodoro/containers/PomodoroTitle.js
0% <0%> (ø)

src/pomodoro/containers/index.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update user-facing-diagnostics.md

Replace undefined property, with one that works for the latest codebase
Ref: #98044

@tophpalmer,
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

@tophpalmer - Can you review this old PR and determine whether it needs to be closed or merged?
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.

#sign-off


Tests broken (?)

Input:
git clone git@github.com:KyleAMathews/superagent-bluebird-promise.git
cd superagent-bluebird-promise
npm install
npm run test

Output:
  ✗ test/test.coffee
     ✗ #84: Line exceeds maximum allowed length. Length is 81, max is 80.

✗ Lint! » 1 error and 0 warnings in 1 file

-> running 1 suite

   superagent-promise
      should exist.. ok
      should resolve a res object when the returned statusCode is < 400.. ok
      should reject an error object when the returned statusCode is > 400.. ok
      should reject an error object when requesting non-existent page.. ok
      should reject an error object when there is an http error.. ok
      request.SuperagentPromiseError
         should have a originalError property if a child error is provided.. ok
         should have a stack trace that includes the originalError stacktrace.. ok
      cancelling promises
         cancel without reason
            should abort the request when the promise is cancelled.. fail
1:             expected false to be true
            should throw a bluebird CancellationError when the promise is cancelled without a reason.. fail
2:             expected false to be true
         cancel with reason that subclasses CancellationError
            should abort the request when the promise is cancelled.. fail
3:             expected false to be true
            should throw a custom error when the promise is cancelled with a reason.. fail
4:             expected false to be true

-> reporting 4 failures

1:    superagent-promise cancelling promises cancel without reason should abort the request when the promise is cancelled

      expected false to be true
      <no lines in stack>

2:    superagent-promise cancelling promises cancel without reason should throw a bluebird CancellationError when the promise is cancelled without a reason

      expected false to be true
      <no lines in stack>

3:    superagent-promise cancelling promises cancel with reason that subclasses CancellationError should abort the request when the promise is cancelled

      expected false to be true
      <no lines in stack>

4:    superagent-promise cancelling promises cancel with reason that subclasses CancellationError should throw a custom error when the promise is cancelled with a reason

      expected false to be true
      <no lines in stack>

-> failed 4 and passed 7 of 11 tests (112ms)

Hrmmm... looks like it :( Could you make a quick PR to fix that?
On Tue, Aug 25, 2015 at 1:01 PM Trey Davis<EMAIL_ADDRESS>wrote:

Input:
git clone git@github.com:KyleAMathews/superagent-bluebird-promise.git
cd superagent-bluebird-promise
npm install
npm run test
Output:
✗ test/test.coffee
✗ #84: Line exceeds maximum allowed length. Length is 81, max is 80.
✗ Lint! » 1 error and 0 warnings in 1 file
-> running 1 suite
superagent-promise
should exist.. ok
should resolve a res object when the returned statusCode is < 400.. ok
should reject an error object when the returned statusCode is > 400.. ok
should reject an error object when requesting non-existent page.. ok
should reject an error object when there is an http error.. ok
request.SuperagentPromiseError
should have a originalError property if a child error is provided.. ok
should have a stack trace that includes the originalError stacktrace.. ok
cancelling promises
cancel without reason
should abort the request when the promise is cancelled.. fail
1:             expected false to be true
should throw a bluebird CancellationError when the promise is cancelled without a reason.. fail
2:             expected false to be true
cancel with reason that subclasses CancellationError
should abort the request when the promise is cancelled.. fail
3:             expected false to be true
should throw a custom error when the promise is cancelled with a reason.. fail
4:             expected false to be true
-> reporting 4 failures
1:    superagent-promise cancelling promises cancel without reason should abort the request when the promise is cancelled
  expected false to be true
  <no lines in stack>

2:    superagent-promise cancelling promises cancel without reason should throw a bluebird CancellationError when the promise is cancelled without a reason
  expected false to be true
  <no lines in stack>

3:    superagent-promise cancelling promises cancel with reason that subclasses CancellationError should abort the request when the promise is cancelled
  expected false to be true
  <no lines in stack>

4:    superagent-promise cancelling promises cancel with reason that subclasses CancellationError should throw a custom error when the promise is cancelled with a reason
  expected false to be true
  <no lines in stack>

-> failed 4 and passed 7 of 11 tests (112ms)
—
Reply to this email directly or view it on GitHub
https://github.com/KyleAMathews/superagent-bluebird-promise/issues/29.


"Small Deploy with Network Filesystem" crashes Editor when exported end exits

Godot version:

3.2.4.rc (built earlier today)
OS/device including version:

Editor: Windows 10
Target: Oculus Quest 2 (most likely affects any Android device)
Issue description:

Exiting an Android-exported game with "Small Deploy with Network Filesystem" enabled, causes the Editor to crash and print:
WARNING: Thread::~Thread: A Thread object has been destroyed without wait_to_finish() having been called on it. Please do so to ensure correct cleanup of the thread.
     At: core\os\thread.cpp:117
ERROR: EditorFileServer::_subthread_start: Condition "err != OK" is true.
   At: editor\fileserver\editor_file_server.cpp:115

This also breaks using One-Click Deploy more than once, as it intentionally closes the game on the exported end.
Steps to reproduce:

With a very recently-built 3.2.x Godot, create a project with an Android export preset
Enable "Small Deploy with Network Filesystem"
One-Click Deploy twice

Tested with 3.2.4 RC 3 on Linux, exporting to a Xiaomi Pocophone F1, and I can reproduce the issue. It's likely caused by Thread changes in #45618.
I could also reproduce it with a build from latest 3.2 (36bec6677e13775749a49d25bd933a682cce00d4):
ARNING: ~Thread: A Thread object has been destroyed without wait_to_finish() having been called on it. Please do so to ensure correct cleanup of the thread.
   At: core/os/thread.cpp:117.
ERROR: _subthread_start: Condition "err != OK" is true.
   At: editor/fileserver/editor_file_server.cpp:115.
[Thread 0x7fff82ffe640 (LWP 450049) exited]
terminate called after throwing an instance of 'std::system_error'
  what():  Invalid argument

Thread 26 "godot-3.2" received signal SIGABRT, Aborted.
[Switching to Thread 0x7fff8da8a640 (LWP 449251)]
0x00007ffff7aa04b0 in raise () from /lib64/libc.so.6
(gdb) bt
#0  0x00007ffff7aa04b0 in raise () from /lib64/libc.so.6
#1  0x00007ffff7a8b526 in abort () from /lib64/libc.so.6
#2  0x0000000001702e2b in __gnu_cxx::__verbose_terminate_handler () at ../../../../libstdc++-v3/libsupc++/vterminate.cc:95
#3  0x00000000046eb316 in __cxxabiv1::__terminate (handler=<optimized out>) at ../../../../libstdc++-v3/libsupc++/eh_terminate.cc:48
#4  0x00000000046eb381 in std::terminate () at ../../../../libstdc++-v3/libsupc++/eh_terminate.cc:58
#5  0x00000000046eb4d4 in __cxxabiv1::__cxa_throw (obj=obj@entry=0x7fff78000c60, tinfo=tinfo@entry=0x60ca4e8 <typeinfo for std::system_error>, dest=dest@entry=0x47644a0 <std::system_error::~system_error()>)
    at ../../../../libstdc++-v3/libsupc++/eh_throw.cc:95
#6  0x0000000001704c2f in std::__throw_system_error (__i=22) at ../../../../../libstdc++-v3/src/c++11/system_error.cc:338
#7  0x000000000476498c in std::thread::join (this=0x7fff78000b98) at ../../../../../libstdc++-v3/src/c++11/thread.cc:113
#8  0x00000000043ca454 in Thread::wait_to_finish (this=0x7fff78000b90) at core/os/thread.cpp:96
#9  0x0000000002d5ffc6 in EditorFileServer::_thread_start (s=0x6ab3180) at editor/fileserver/editor_file_server.cpp:303
#10 0x00000000043ca28a in Thread::callback (p_self=0x6ab32c0, p_settings=..., p_callback=0x2d5fd34 <EditorFileServer::_thread_start(void*)>, p_userdata=0x6ab3180) at core/os/thread.cpp:68
#11 0x00000000043cb00b in std::__invoke_impl<void, void (*)(Thread*, Thread::Settings const&, void (*)(void*), void*), Thread*, Thread::Settings, void (*)(void*), void*> (
    __f=@0xa44f388: 0x43ca21e <Thread::callback(Thread*, Thread::Settings const&, void (*)(void*), void*)>) at /usr/include/c++/10/bits/invoke.h:60
#12 0x00000000043cae8e in std::__invoke<void (*)(Thread*, Thread::Settings const&, void (*)(void*), void*), Thread*, Thread::Settings, void (*)(void*), void*> (
    __fn=@0xa44f388: 0x43ca21e <Thread::callback(Thread*, Thread::Settings const&, void (*)(void*), void*)>) at /usr/include/c++/10/bits/invoke.h:95
#13 0x00000000043cad49 in std::thread::_Invoker<std::tuple<void (*)(Thread*, Thread::Settings const&, void (*)(void*), void*), Thread*, Thread::Settings, void (*)(void*), void*> >::_M_invoke<0ul, 1ul, 2ul, 3ul, 4ul> (this=0xa44f368) at /usr/include/c++/10/thread:264
#14 0x00000000043cacae in std::thread::_Invoker<std::tuple<void (*)(Thread*, Thread::Settings const&, void (*)(void*), void*), Thread*, Thread::Settings, void (*)(void*), void*> >::operator() (this=0xa44f368)
    at /usr/include/c++/10/thread:271
#15 0x00000000043cac92 in std::thread::_State_impl<std::thread::_Invoker<std::tuple<void (*)(Thread*, Thread::Settings const&, void (*)(void*), void*), Thread*, Thread::Settings, void (*)(void*), void*> > >::_M_run (this=0xa44f360) at /usr/include/c++/10/thread:215
#16 0x0000000004764740 in std::execute_native_thread_routine (__p=0xa44f360) at ../../../../../libstdc++-v3/src/c++11/thread.cc:80
#17 0x00007ffff7d6fdea in start_thread () from /lib64/libpthread.so.0
#18 0x00007ffff7b5afff in clone () from /lib64/libc.so.6


Automatic line wrapping in the fullscreen editor

Scenario:
You are writing a message and decide you need to change something say ten lines up which makes the line longer.
Currently, you first have to manually wrap/break the actual line to be able to enter the text and then you have to manually wrap all the following lines if you want the message to look tidy.
It would be very nice if the editor did the wrapping for you instead and it would save a lot of time for people like me who have a hard time writing a text without multiple edits.

I agree that it's a bit annoying that you most of the time need to "make room" on a line before you can add text to it. But I'm not sure what the best solution would be. Should it push the rest of the text in front of it and re-wrapping all the lines below as needed? For the entire text or just the current paragraph. (However a paragraph is defined.)
What about when you delete text, should it do the reverse and always pull text up from lines below as room becomes available?
There are probably cases where this is not desireable. E.g. when editing code snippets.

I think it would be most useful if it pushed the text in front of it, rewrapping the text in the current paragraph to make room. And for delete it should pull up text from below lines of the current paragraph.
The definition of a paragraph would depend on how NiKom stores the text, I guess. For example if it was possible to define a paragraph as a very long line, ending with the first intentional line break, the definition would be clear.
If the definition what a paragraph is is not clear and the code has to make an educated guess, I can see that there could be occasional issues while editing code snippets. However, I think it would be a big convenience in most cases.

Reflow should definitely only happen within the current paragraph. I think paragraphs should be delimited by two newlines in a row, i.e.
This is the first paragraph.
This is the second paragraph.
Still still the second.
This is the third paragraph.

Newline followed by whitespace (another newline, space, tab) may be a good general rule for paragraph separators. That will also prevent reflow into indented code lines.


set iot wifi

atlantis apply

Ran Apply for project: local-lhr-unifi dir: infrastructure/local/lhr/unifi workspace: terragrunt
Show Output
23:43:14.330 INFO   Downloading Terraform configurations from ../../.. into ./.terragrunt-cache/KsJh_Rruafs8TccdnxhYK5d5tS0/ZjZcA_NaGopneC0TkPLWc57haHY
23:43:15.291 STDOUT tofu: Initializing the backend...
23:43:30.125 STDOUT tofu: Initializing provider plugins...
23:43:30.125 STDOUT tofu: - Reusing previous version of vexxhost/uptimerobot from the dependency lock file
23:43:30.263 STDOUT tofu: - Reusing previous version of paultyng/unifi from the dependency lock file
23:43:30.331 STDOUT tofu: - Reusing previous version of hashicorp/aws from the dependency lock file
23:43:30.417 STDOUT tofu: - Reusing previous version of backblaze/b2 from the dependency lock file
23:43:30.476 STDOUT tofu: - Reusing previous version of cloudflare/cloudflare from the dependency lock file
23:43:30.545 STDOUT tofu: - Reusing previous version of integrations/github from the dependency lock file
23:43:31.496 STDOUT tofu: - Using previously-installed backblaze/b2 v0.8.12
23:43:32.328 STDOUT tofu: - Using previously-installed cloudflare/cloudflare v4.43.0
23:43:32.689 STDOUT tofu: - Using previously-installed integrations/github v6.2.2
23:43:32.954 STDOUT tofu: - Using previously-installed vexxhost/uptimerobot v0.8.2
23:43:33.245 STDOUT tofu: - Using previously-installed paultyng/unifi v0.41.0
23:43:44.903 STDOUT tofu: - Using previously-installed hashicorp/aws v5.70.0
23:43:44.903 STDOUT tofu: OpenTofu has been successfully initialized!
23:43:44.903 STDOUT tofu:
23:43:44.903 STDOUT tofu: You may now begin working with OpenTofu. Try running "tofu plan" to see
23:43:44.903 STDOUT tofu: any changes that are required for your infrastructure. All OpenTofu commands
23:43:44.903 STDOUT tofu: should now work.
23:43:44.903 STDOUT tofu: If you ever set or change modules or backend configuration for OpenTofu,
23:43:44.904 STDOUT tofu: rerun this command to reinitialize your working directory. If you forget, other
23:43:44.904 STDOUT tofu: commands will detect it and remind you to do so if necessary.
23:44:10.882 STDOUT tofu: unifi_wlan.wlan["wlan03"]: Modifying... [id=66f876be7f2bc7414cb0b575]
23:44:12.141 STDOUT tofu: unifi_wlan.wlan["wlan03"]: Modifications complete after 1s [id=66f876be7f2bc7414cb0b575]
23:44:12.337 STDOUT tofu:
23:44:12.337 STDOUT tofu: Apply complete! Resources: 0 added, 1 changed, 0 destroyed.
23:44:12.337 STDOUT tofu:

Automatically merging because all plans have been successfully applied.

Locks and plans deleted for the projects and workspaces modified in this pull request:

dir: infrastructure/local/lhr/unifi workspace: terragrunt


deps vs. exported_deps for c++?

Semantically I understand the difference, but I'm curious what the thinking is about best practices?  Consider the following example.  We'll assume for this example that b/b.h looks like this:
// b.h
#include "a/a.h"

cxx_library(
    name="a",
    public_include_directories=["a/include"],
    headers=["a/include/a/a.h"],
cxx_library(
    name="b",
    public_include_directories=["b/include"],
    headers=["b/include/b/b.h"],
    deps=["//:a"],
#    Is this preferable?
#    exported_deps=["//:a"]
cxx_library(
    name="c",
    public_include_directories=["c/include"],
    headers=["c/include/c/c.h"],
    deps=[
        "//:b",

        # This is required unless we uncomment the exported dep in b
        "//:a",
    ]

So this is more of a philisophical question.  If we use exported_deps, targets carry their dependencies with them.  It makes maintenance a lot easier.  Adding a dependency from c to b requires doing exactly that: Add a dependency to b.  If we use deps instead requires a potentially never-ending cycle of "build / interpret-error-message / add-dependency", which is difficult to scale.  Worse though is that if we aren't using exported_deps, removing the dependency from b to a requires you to remember that you need to also remove it from c.  And I don't think buck2 has the ability to warn you about superfluous dependencies (I think this isn't a solvable problem in general).
But I know there's argument for trying to avoid exported_deps, and this methodology is used at scale (I think google internally operates like this?), the thinking being that requireing it to be explicit leads to better code hygiene and more awareness of your dependency chains.
Curious to hear others' thoughts and what your experience has been at scale.  My gut tells me you want to be using exported_deps because of the maintenance on large scale projects.

My recommendation has been that if you are exposing something from your dependencies in a way that your dependents are going to need it, then you should export that dependency. I think trying to avoid exported_deps entirely would be too painful for the small value it gives you. I could see that calculus being different if you had a system that could automatically keep your deps up-to-date based on the contents of the sources.

Do you think buck2 could gain the ability to report unneeded dependencies?  It seems possible in limited scenarios.  For example, if the only thing a dependency exports is public include directories, and buck2's internal dependency analysis determines that no file from that path is ever loaded, it could warn (or even error) that there is an unnecessary dependency.
From here, there may be opt-in ways to take this further.  For example, maybe there could be something like buck2 check-deps <target-pattern>.  Maybe it looks for exported preprocessor definitions that don't alter the generated output by compiling both ways and checking the hash.
Just thinking out loud though, it might trigger too infrequently to be worth the development effort.

Another kind of warning / error that might be useful.  If B exports a dep A, and C links to B and also exports dep A, then C doesn't need to explicitly export that dep to make it work.  Is there any way to detect this during analysis and require that C explicitly export the dep?  Example:
cxx_library(
    name = "a",
    visibility = ["PUBLIC"],
    public_include_directories=["a"],
    headers=["a/a/a.h"]
)

cxx_library(
    name = "b",
    visibility = ["PUBLIC"],
    public_include_directories=["b"],
    headers=["b/b/b.h"],
    exported_deps=["//:a"]
)

cxx_library(
    name = "c",
    visibility = ["PUBLIC"],
    public_include_directories=["c"],
    headers=["c/c/c.h"],
    exported_deps=["//:b"]
)

cxx_binary(
    name = "d",
    visibility = ["PUBLIC"],
    include_directories=["d"],
    headers=["d/d/d.h"],
    srcs=["d/d.cpp"],
    deps=["//:c"]
)

And here are a, b, c, and d source files:
// a.h
using MyType = int;

// b.h
#include "a/a.h"

// c.h
#include "b/b.h"

// C should be required to explicitly export a, in case the dependency on b
// is ever refactored out.
#include "a/a.h"

// d.h
#include "c/c.h"

// d.cpp
#include "d/d.h"

// d should be required to explicitly depend on a, since it directly includes a header.
#include "a/a.h"

int main(MyType argc, char **argv)
{
}

In theory, I think buck2 has enough information to be able to enforce this.  But I tested this out, and if I run buck2 build //:d right now, everything works fine.  Do you see any value in making buck2 error in this example (is it even possible)?

It's quite easy for BXL to traverse the set of targets and introspect them.
It would be entirely feasible to implement a sort of linter atop that.


python310Packages.dvc-data: 0.28.4 -> 0.35.1, python310Packages.dvc-objects: 0.14.0 -> 0.19.0

Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
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

[ ] Fits CONTRIBUTING.md.

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

2 packages failed to build:

dvc
dvc-with-remotes

6 packages built:

python310Packages.dvc-data
python310Packages.dvc-objects
python310Packages.sqltrie
python311Packages.dvc-data
python311Packages.dvc-objects
python311Packages.sqltrie


@include adding extra space/line break to content

Laravel Version: 5.8.7
PHP Version: 7.2.5, 7.3.3
Database Driver & Version:

Description:
Blade @includes are appending extra white space to the end
Steps To Reproduce:
test/include.blade.php
this is only a test
parent.blade.php

@include('test.include')

expected html output:

this is only a test

actual html output:

this is only a test

In most circumstances, this isn't really an issue, but in cases where you need to prevent white-space between html elements, it becomes an issue as the extra whitespace creates a gap between them.
In Laravel 5.7.x, the same code returned the expected behavior.  Potentially even within an earlier version of 5.8, but I'm not sure.

Heya, I think this might be related to https://github.com/laravel/framework/pull/27544 which adds an extra linebreak at the end. This got fixed in https://github.com/laravel/framework/pull/27976 which will be released tomorrow. Feel free to report back if tomorrow's release doesn't fixes your problem.

I just updated to Laravel 5.8.8 and this issue does not appear to be resolved.

I'm getting the same. Here are some copy-and-paste steps to reproduce:
composer create-project --prefer-dist laravel/laravel=5.8.3 issue-27996

echo -n "
@include('partial')
" > issue-27996/resources/views/parent.blade.php
echo -n "contents" > issue-27996/resources/views/partial.blade.php
echo "Artisan::command('issue-27996', function () {
    \$this->comment(view('parent'));
});" >> issue-27996/routes/console.php

php issue-27996/artisan issue-27996

Okay, re-opening.

Ping @bzixilu. Do you maybe have an idea if this is related to the changes you made?

It appears they may be related.
The cached view code for parent.blade.php:

<?php echo $__env->make('partial', \Illuminate\Support\Arr::except(get_defined_vars(), ['__data', '__path']))->render(); ?>

<?php /* /path/to/issue-27996/resources/views/parent.blade.php */ ?>

The cached view code for partial.blade.php:
contents
<?php /* /path/to/issue-27996/resources/views/partial.blade.php */ ?>

So when partial.blade.php is included, it has that new line.

A fix may be to leave out the \n when appending the path to the compiled view.
diff --git a/src/Illuminate/View/Compilers/BladeCompiler.php b/src/Illuminate/View/Compilers/BladeCompiler.php
index 0f084b06f..c5a51c36f 100644
--- a/src/Illuminate/View/Compilers/BladeCompiler.php
+++ b/src/Illuminate/View/Compilers/BladeCompiler.php
@@ -123,7 +123,7 @@ class BladeCompiler extends Compiler implements CompilerInterface
             );

             if (! empty($this->getPath())) {
-                $contents .= "\n<?php /* {$this->getPath()} */ ?>";
+                $contents .= "<?php /* {$this->getPath()} */ ?>";
             }

             $this->files->put(

@fitztrev it's the base of the whole reason why the feature was added: so the path line would exist at the very last line so PHPStorm could pick it up.

Ok, I don't know the inner workings of PHPStorm. But maybe they would be able to parse it out when it's not on its own line.
But now that you mention that, I'm curious why this was added. https://github.com/laravel/framework/pull/27963#issuecomment-475618587 Maybe this should belong in a package too.

But now that you mention that, I'm curious why this was added. #27963 (comment) Maybe this should belong in a package too.

@fitztrev touché and you make an excellent point here. Ultimately this wasn't my call though for this feature. And I do believe that the impact of this particularly feature (the path in the view) is worth adding because it empowers such a useful feature.
However, I fully agree that we should look at how we can look at a way to keep the feature while solving this bug.

Is this only a problem when you use an include between html tags? Or also in other ways/usages?

However, I fully agree that we should look at how we can look at a way to keep the feature while solving this bug.

👍 We'll wait to see what @bzixilu says, if there's another way PHPStorm could parse it.

Is this only a problem when you use an include between html tags? Or also in other ways/usages?

Other ways/usages too. A blade @include() is always appending a newline when rendered. The surrounding
 tag was only used in the example to better illustrate the problem.

This was touted online by JetBrains, Taylor, Laravel News, etc. Though not specifically mentioned, it seems like it would work out-of-the-box. So including it in the core makes sense. I think we should keep trying to make this work without going the third-party route.

Hi all, sorry for one more breakage of Blade usage scenario. I'll start an investigation today and will do my best to come back soon with the possible solution.

Closing this because we've decided to revert the feature for now. Please see https://github.com/laravel/framework/pull/28099#issuecomment-479900212 for more info.


Javadoc linking

Two things:

This copies the way that Scala links against Javadoc
This shows an example of this actually working to link to a Java exception

Example:

This then links to:

Related issue: None

Type of change: documentation

Impact: no functional change

Development Phase: implementation
Release Notes

Scaladoc now can link against Javadoc

Err... Slightly better way to do this with a plugin... Will rebase.


METRON-410: mysql_server's MySQL install causes mutually assured dest…

…ruction when installed on the same machine as the Ambari Hive MySQL
Allowed the user to indicate their using an existing MySQL instance:
a) Refactored the required database and index settings to a separate property sheet using enhanced configs.
b) Made the MySQL start/stop logic from the add user, remove user and geo ip database load conditional.
c) Auto-populated the Storm UI URL and MySQL Host Address from the Add Service Wizard Master Install Selection page.
d) Cleaned up some lingering metainfo.xml items since I was in the file anyway to add the theme.
Tested on Docker cluster following procedure here: Metron MPack
Test case 1: Default install (Ambari installed MySQL)l
Test case 2: Pre-existing MySQL install.

Bump?

+1.  This is really great.

Oh Travis. I just removed a comment.


Update virtual-network-tcpip-performance-tuning.md

Removed some redundancies and added clarity about impacts of fragmented packets in Azure.

Closing due to incorrect link.


Motors

Hi to everybody,
were can I find documentation about " TargetVelocityMotorHingeJoints: " ?
This feature is used in MotorScene2.json example and I wish like to know more about the command.
Many thanks, bye
. . . Piero

Sorry, but this is not documented yet. In fact you can use any joint which is supported by our library PositionBasedDynamics in the scene file. The motor definition looks like this:
"RigidBodies": [
		{
			"id": 0,
			"geometryFile": "../models/UnitBox.obj",
			"translation": [0,1,0],
			"rotationAxis": [1, 0, 0],
			"rotationAngle": 0,
			"scale": [2, 2, 2],
			"color": [0.1, 0.4, 0.6, 1.0],
			"isDynamic": false,
			"isWall": true,
			"mapInvert": true,
			"mapThickness": 0.0,
			"mapResolution": [25,25,25]
		},
		{
			"id": 1,
			"geometryFile": "../models/UnitBox.obj",
			"translation": [0,0.1,0],
			"rotationAxis": [1, 0, 0],
			"rotationAngle": 0,
			"scale": [1.5,0.1,0.2],
			"color": [0.1, 0.4, 0.6, 1.0],
			"density": 10000,
			"isDynamic": true,
			"isWall": false,
			"mapInvert": false,
			"mapThickness": 0.0,
			"mapResolution": [20,20,20]
		}
	],
	"TargetVelocityMotorHingeJoints": [
		{
			"bodyID1": 1,
			"bodyID2": 0,
			"position": [0,0,0],
            "axis": [0,1,0],
            "repeatSequence": 1,
            "targetSequence": [0,-5,1,5,2,-5]
		}
	],

This motor links the two rigid bodies with the user-defined ids 0 and 1. The hinge axis is defined by the world space position (0,0,0) and the direction vector (0,1,0). So it points in the direction of the y-axis and lies in the origin. "repeatSequence" means that the motor sequence is repeated infinitely. And the "targetSequence": [0,-5,1,5,2,-5] is a definition of keyframes. This is always a pair of time and target velocity. So at time 0 the motor has a velocity of -5, at time 1 it changes to 5 and at time 2 it changes to -5 and then the sequence starts again.
Hope that helps!

Yes, this helps. Many thanks.
Piero


[Snyk] Upgrade @nestjs/axios from 3.0.1 to 3.0.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @nestjs/axios from 3.0.1 to 3.0.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2024-02-07.

Release notes

Package name: @nestjs/axios

3.0.2 - 2024-02-07 Read more

3.0.1 - 2023-10-30 Read more

    from @nestjs/axios GitHub release notes

Commit messages

Package name: @nestjs/axios

a3f98bf Merge pull request #1016 from nestjs/renovate/nest-monorepo
c8be5e5 Merge pull request #910 from nestjs/renovate/cimg-node-21.x
c8c5f81 chore(deps): update nest monorepo to v10.3.2
71341b1 chore: remove reflect-metadata from peer deps
47977ed chore(deps): update typescript-eslint monorepo to v6.21.0
c019cd0 chore(deps): update dependency lint-staged to v15.2.2
6fb30e4 chore(deps): update dependency prettier to v3.2.5
c289ef1 chore(deps): update dependency husky to v9.0.10
a1c0256 chore(deps): update dependency @ types/node to v20.11.16
ae6c6ae chore(deps): update dependency @ types/jest to v29.5.12
c43eb41 chore(deps): update dependency @ types/node to v20.11.15
e326c14 chore(deps): update dependency @ types/node to v20.11.14
cc3ebe7 chore(deps): update dependency lint-staged to v15.2.1
e5f5930 chore(deps): update dependency @ types/node to v20.11.13
b81b772 chore(deps): update typescript-eslint monorepo to v6.20.0
920e5e5 chore(deps): update dependency husky to v9.0.7
29365c6 chore(deps): update dependency @ types/node to v20.11.10
91d18cb chore(deps): update dependency @ types/node to v20.11.9
3fbca54 chore(deps): update dependency @ types/node to v20.11.8
225b32d chore(deps): update dependency @ types/node to v20.11.7
3c3f7b0 chore(deps): update dependency husky to v9.0.6
377383f chore(deps): update dependency axios to v1.6.7
173bb71 chore(deps): update dependency husky to v9.0.5
e1f2d93 chore(deps): update commitlint monorepo to v18.6.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Rebase


FIX: Fix set surf

@christianbrodbeck can you confirm this fixes the bug you pointed out in https://github.com/mne-tools/mne-python/pull/5012 ?

Yes this fixes it. But it's not necessary for older versions. Must be that the dev version of mayavi makes a copy where the earlier version used a view (hence in-place update of the array in load_geometry()).

Ahh -- it would actually make sense for them to do this, it should be faster to only upload things that are known to have changed on update, which you can't for in-place array changes. It might have been luck that it worked before. That is my suspicion anyway based on my limitied graphics optimization knowledge. But feel free to open a Mayavi bug if you're curious.

Codecov Report

Merging #232 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #232      +/-   ##
=========================================
+ Coverage   74.29%   74.3%   +0.01%
=========================================
  Files           7       7
  Lines        2439    2440       +1
  Branches      487     487
=========================================
+ Hits         1812    1813       +1
  Misses        454     454
  Partials      173     173

Yeah that's plausible that that's behind the change. Glad you caught it so quickly!


Two errors produced from DelayedJob; one with stack trace, one without.

Hello, we're noticing some peculiar behavior where an uncaught exception from a DelayedJob produces two different errors in our Rollbar logs; one has a stack trace, one does not. The Rollbar support team directed me here to raise an issue on the repo.
We have two examples that we could link to, please message me privately at<EMAIL_ADDRESS>for the links. I'll try to show as much detail here though.
With the stack trace

Note that this correctly includes the ActiveRecord::RecordInvalid error.
Stack traces do show up in the occurrences:

Without the stack trace

Note how there's no interpretation of project vs. gem stack traces, etc. Also there's no inclusion of the error name.
No stack trace is provided in the occurrences tab either:

Application job definition
Here's our error handling within our jobs:
class ApplicationJob < ActiveJob::Base
  def initialize(*args)
    super
    @context = {}
  end

  rescue_from(StandardError) do |e|
    context[:job] = self.class.name
    Rollbar.scope! data: context
    raise e
  end
  ...
end

Let us know if you need any more info!

@AndrewSouthpaw I sent you an email to take a look at your examples.

@AndrewSouthpaw I tried but wasn't able to reproduce the problem on my own. I'll wait to get the examples from you.

Closing as we are not able to reproduce the issue.

@rivkahstandig3636 I did send Artur the examples, and didn't hear back after that. Was the repro unsuccessful?

Ah I appear to have misunderstood from @ArturMoczulski's comment. @ArturMoczulski should this be reopened?

@AndrewSouthpaw @rivkahstandig3636 sorry for confusion. yes, this needs to be reopened and still addressed. I'm putting it into the v2.17.0 milestone.

Hey @AndrewSouthpaw, sorry for the delay. We were wondering if you are using the rails logger integration we have, something like this in one initializer:
Rails.logger.extend(ActiveSupport::Logger.broadcast(Rollbar::Logger.new))

Also, these items are happening in development or a production environment?

I'm going to close this pending feedback from @AndrewSouthpaw.


add working steps for bcd to binary

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
i wud like to work on this

Go ahead
Deadline 20 May,2021


CHAOS-5650 update fault permissions

update fault permissions

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://665db47b288c9ee8074b20e1--harness-developer.netlify.app


MioMio youtube-dl failing with HTTPError()

H@KitchenComputer /cygdrive/c/Users/H
$ ./youtube-dl.exe --verbose http://www.miomio.tv/watch/cc247337/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://www.miomio.tv/watch/cc247337/']
[debug] Encodings: locale cp1252, fs mbcs, out None, pref cp1252
[debug] youtube-dl version 2015.11.02
[debug] Python version 2.7.10 - Windows-Vista-6.0.6002-SP2
[debug] exe versions: none
[debug] Proxy map: {}
[miomio.tv] 247337: Downloading webpage
[miomio.tv] 247337: Downloading webpage
[miomio.tv] 247337: Downloading XML
ERROR: Unable to download XML: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "youtube_dl\extractor\common.pyo", line 329, in _request_webpage
File "youtube_dl\YoutubeDL.pyo", line 1873, in urlopen
File "urllib2.pyo", line 437, in open
File "urllib2.pyo", line 550, in http_response
File "urllib2.pyo", line 469, in error
File "urllib2.pyo", line 409, in _call_chain
File "urllib2.pyo", line 656, in http_error_302
File "urllib2.pyo", line 437, in open
File "urllib2.pyo", line 550, in http_response
File "urllib2.pyo", line 475, in error
File "urllib2.pyo", line 409, in _call_chain
File "urllib2.pyo", line 558, in http_error_default

Why would you open an issue if theres another one with almost exactly the same title? #7366


Displaying the projects related to each QF round in the admin

Since we can see only the list of current round's projects in the dApp, It is necessary to see the list of projects of each  QF round in the admin panel
We can also display the amount of donation  and estimated matching

is it p1 or p2? @laurenluz
you can use labels

added

Qf round history menu could cover this requirement approximately( after a round),
But it would be good if admin users could see all projects of a round in a page and be able to  add or remove some projects to  the round
like below now video we could see QF round's project  and it would be great if :

There is pagination for a round that it has many project
-Adding or removing could be done on this page or a new page on admin

https://github.com/Giveth/impact-graph/assets/111529185/163473ad-a173-47b1-a0fd-4aac9839b4be

sounds great yeah. I think this is a good p2. Thanks @maryjaf !

@RamRamez pls pick this post your p0 and p1 issues.

@RamRamez pls guide @Meriem-BM on this

Hey @maryjaf from what I understood this is already implemented (we can see projects list on QF round Tab) but what's missing is the pagination, right?
For adding/removing projects we already have an issue #1125 it will be reviewed then merged.

@maryjaf can you check the comment before this one.

There is pagination for a round that it has many project
Adding or removing could be done on this page or a new page on admin

Yeah I had  mentioned the problem of previous view of qf round in admin in this comment
and now the issue related to  adding/removing project has been solved
and the only thing that it would be good to have is this :
-There is pagination for a round that it has many project
@Meriem-BM

Hey @maryjaf, you can test this now

Thanks @Meriem-BM it has been fixed


seperate_words function based on \W+ re instead?

@fabianvf Is it my imagination or could we replace the entire separate_words function better with a \W+ or \W regex instead?

@fabianvf bump

@fabianvf thoughts on this?

Honestly I'm not sure, does it make a difference? Wonder if there are any edge cases related to punctuation or something that will bite us.

It'll be important for nonwestern languages.
On Aug 31, 2017 4:18 PM, "Fabian von Feilitzsch"<EMAIL_ADDRESS>wrote:

Honestly I'm not sure, does it make a difference? Wonder if there are any
edge cases related to punctuation or something that will bite us.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/fabianvf/python-rake/issues/23#issuecomment-326409010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AShd7CGb6Mvox5WWWtd2GjTIZhujfB7rks5sdxT8gaJpZM4OwesY
.

@fabianvf Also doing it that way is faster, easier to understand and more maintainable/pythonic


Fixed missed and new items on CATS retest 7-11-19

Per User Story https://mseng.visualstudio.com/TechnicalContent/_queries/edit/1562764

Nevermind on that one "versus" suggestion. I deleted it. That PR just merged, this should be fine with it as "vs" now.

This is a good test ...
Place extension methods in the <xref:Microsoft.Extensions.DependencyInjection> namespace

... because idk if the full name for namespaces becomes the link text. Others here may know tho. It's a quick patch if it doesn't work out. I'll take a look after this goes live and see what happened.

@guardrex - It appears to me that the xref is broken, however I will try coding it the other way you suggested.

Oh! ...
so ... <xref:Microsoft.Extensions.DependencyInjection> still broke?
That's bad. We need to take it to engineering then. I'll open an issue with them to take a look.
You can go back to the original way that you had it.


Build one-up title component

Wireframe tbc.

Straight-up title-block in a one-up-block, accepting only a 'title' entity. Essentially used to separate out consecutive backgrounded templates in a page.

Wireframe: https://makersacademy.mybalsamiq.com/projects/newsitewireframes/one-up title

Notes:

Component is always max.960px
Component height is flexible and expands as needed.
Acceptable modifiers: --backgrounded, --left, --right
Acceptable locals: modifiers, parent, title, background_image


Creating TXT record through REST API fails with a 5005 Missing type property data

JSON object to add:
{    "class":"IN",    "type":"TXT",    "ttl":3600,    "active":true,    "data":{       "data":"text-string"    },    "is_conditional":false,    "conditional_count":0,    "conditional_limit":0,    "conditional_reset":false,    "conditional_data":{    } }
Error:
{     "code": 5005,     "details": [         "Missing type property data"     ],     "message": "Invalid type property fields",     "success": false }

curl -k -X 'POST' 'https://privatedomain/api/v1/zones/70chars.johnneerdael.dataexfiltration.privatedomain/records' -H 'accept: application/json' -H 'X-SnitchDNS-Auth: APIKEY' -H 'Content-Type: application/json'  -d '{"class":"IN","type":"TXT","ttl":3600,"active":true,"data":{"data":"text-string"},"is_conditional":false,"conditional_count":0,"conditional_limit":0,"conditional_reset":false,"conditional_data":{}}'

Thanks for reporting this, I've now pushed a fix - please pull the latest and greatest and give it another go.
The issue was that it was looking for conditional data although the is_conditional parameter was set to false.
I'll close the ticket now - if there are any other issues feel free to open it again.


How to mirror axes?

Question: To mirror a plot’s vertical axis on the right side of a chart, we currently create the plottable twice. Is there a direct way to achieve this?
ScottPlot Version: ScottPlot Version 5
ScottPlot.Plottables.Scatter s1 = myPlot.Add.Scatter(dataX, dataY);
ScottPlot.Plottables.Scatter s2 = myPlot.Add.Scatter(dataX, dataY);
s2.IsVisible = false;
s2.Axes.YAxis = myPlot.Axes.Right;

We assume, that the ScottPlot.AxisPanels.MirroredXAxis and ScottPlot.AxisPanels.MirroredYAxis  classes exist for precisely for this purpose, but could not figure out how to use them.

Currently, the way to add your own axis objects is in the code but not yet released. It will be in the next release, which Scott has said he hopes to have out this weekend (see https://github.com/ScottPlot/ScottPlot/issues/3457#issuecomment-2154991555).
Once that release comes out you should be able to just do this:
var leftAxis = myPlot.Plot.Axes.Left;
myPlot.Plot.Axes.Remove(Edge.Right); // Remove the (invisible) default right axis, otherwise it will look weird
myPlot.Plot.Axes.AddRightAxis(new MirroredYAxis(leftAxis, Edge.Right));

This replaces the invisible default right axis with a visible one which mirrors whatever left axis you already have.


Add support for long metric types

Currently, the metric exporter only reads double-based metrics. This adds support for long-based metrics.

@smbecker - Thanks for your contribution.


Add custom coordinates instead of current position

In demo-app new points occur at your current position while tapping on the screen. I can find lines in code responsible for this.
I want to change the current location while tapping for another location and don't understand which variables are responsible for this
Do anybody know how to solve this issue?

the lines you are looking for are here: https://github.com/ProjectDent/ARKit-CoreLocation/blob/master/ARKit%2BCoreLocation/ViewController.swift#L236 you can then pass a location to LocationAnnotationNode(location:image:) and change addLocationNodeForCurrentPosition to addLocationNodeWithConfirmedLocation

Thank you so much!
On Sun, Sep 17, 2017 at 10:15 AM Vihan<EMAIL_ADDRESS>wrote:

the lines you are looking for are here:
https://github.com/ProjectDent/ARKit-CoreLocation/blob/master/ARKit%2BCoreLocation/ViewController.swift#L236
you can then pass a location to LocationAnnotationNode(location:image:)
and change addLocationNodeForCurrentPosition to
addLocationNodeWithConfirmedLocation
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ProjectDent/ARKit-CoreLocation/issues/40#issuecomment-330063403,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV7CRUAdWnUIDbG_tVsEG-f4ATGasVHAks5sjVOzgaJpZM4PaDRK
.


Ask A Question option always included in the submit dropdown.

Maybe this is the intended behavior or perhaps its because I am an admin? The Ask A Question option is always available in the submit dropdown even for creating normal posts. Shouldnt it only show in the menu when a new topic is being created?

Fixed in 0b3931209066c960107bb734a4266d5c61a229cc


Update the ACS Chat release date and common package version

Packages impacted by this PR
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check for @azure/communication-chat
API changes are not detected in this pull request for @azure/communication-chat

there were some issues with package-name in README.md. I'd recommend unifying it with other packages that have been migrated to @azure/communication-chat
sorry, I am not sure where is the package-name you mentioning

there were some issues with package-name in README.md. I'd recommend unifying it with other packages that have been migrated to @azure/communication-chat
sorry, I am not sure where is the package-name you mentioning

sorry, I mean the communication-chat/swagger/README.md