AttributeError: module 'tensorflow.python.keras' has no attribute 'Model'

Running code from object Detection
not a custom code
I am using Window 10
Tensorflow 1.4 CPU
I am getting this error, while running below code.
i want to rectify the error, without updating Tensorflow,
import tensorflow as tf
BOX_ENCODINGS = 'box_encodings'
CLASS_PREDICTIONS_WITH_BACKGROUND = 'class_predictions_with_background'
MASK_PREDICTIONS = 'mask_predictions'
class BoxPredictor(object):
"""BoxPredictor."""
def init(self, is_training, num_classes):
"""Constructor.
Args:
  is_training: Indicates whether the BoxPredictor is in training mode.
  num_classes: number of classes.  Note that num_classes *does not*
    include the background category, so if groundtruth labels take values
    in {0, 1, .., K-1}, num_classes=K (and not K+1, even though the
    assigned classification targets can range from {0,... K}).
"""
self._is_training = is_training
self._num_classes = num_classes

@property
def is_keras_model(self):
return False
@property
def num_classes(self):
return self._num_classes
def predict(self, image_features, num_predictions_per_location,
scope=None, **params):
"""Computes encoded object locations and corresponding confidences.
Takes a list of high level image feature maps as input and produces a list
of box encodings and a list of class scores where each element in the output
lists correspond to the feature maps in the input list.

Args:
  image_features: A list of float tensors of shape [batch_size, height_i,
  width_i, channels_i] containing features for a batch of images.
  num_predictions_per_location: A list of integers representing the number
    of box predictions to be made per spatial location for each feature map.
  scope: Variable and Op scope name.
  **params: Additional keyword arguments for specific implementations of
          BoxPredictor.

Returns:
  A dictionary containing at least the following tensors.
    box_encodings: A list of float tensors. Each entry in the list
      corresponds to a feature map in the input `image_features` list. All
      tensors in the list have one of the two following shapes:
      a. [batch_size, num_anchors_i, q, code_size] representing the location
        of the objects, where q is 1 or the number of classes.
      b. [batch_size, num_anchors_i, code_size].
    class_predictions_with_background: A list of float tensors of shape
      [batch_size, num_anchors_i, num_classes + 1] representing the class
      predictions for the proposals. Each entry in the list corresponds to a
      feature map in the input `image_features` list.

Raises:
  ValueError: If length of `image_features` is not equal to length of
    `num_predictions_per_location`.
"""
if len(image_features) != len(num_predictions_per_location):
  raise ValueError('image_feature and num_predictions_per_location must '
                   'be of same length, found: {} vs {}'.
                   format(len(image_features),
                          len(num_predictions_per_location)))
if scope is not None:
  with tf.variable_scope(scope):
    return self._predict(image_features, num_predictions_per_location,
                         **params)
return self._predict(image_features, num_predictions_per_location,
                     **params)

TODO(rathodv): num_predictions_per_location could be moved to constructor.
This is currently only used by ConvolutionalBoxPredictor.
@abstractmethod
def _predict(self, image_features, num_predictions_per_location, **params):
"""Implementations must override this method.
Args:
  image_features: A list of float tensors of shape [batch_size, height_i,
    width_i, channels_i] containing features for a batch of images.
  num_predictions_per_location: A list of integers representing the number
    of box predictions to be made per spatial location for each feature map.
  **params: Additional keyword arguments for specific implementations of
          BoxPredictor.

Returns:
  A dictionary containing at least the following tensors.
    box_encodings: A list of float tensors. Each entry in the list
      corresponds to a feature map in the input `image_features` list. All
      tensors in the list have one of the two following shapes:
      a. [batch_size, num_anchors_i, q, code_size] representing the location
        of the objects, where q is 1 or the number of classes.
      b. [batch_size, num_anchors_i, code_size].
    class_predictions_with_background: A list of float tensors of shape
      [batch_size, num_anchors_i, num_classes + 1] representing the class
      predictions for the proposals. Each entry in the list corresponds to a
      feature map in the input `image_features` list.
"""
pass

class KerasBoxPredictor(tf.keras.Model):
"""Keras-based BoxPredictor."""
def init(self, is_training, num_classes, freeze_batchnorm,
inplace_batchnorm_update, name=None):
"""Constructor.
Args:
  is_training: Indicates whether the BoxPredictor is in training mode.
  num_classes: number of classes.  Note that num_classes *does not*
    include the background category, so if groundtruth labels take values
    in {0, 1, .., K-1}, num_classes=K (and not K+1, even though the
    assigned classification targets can range from {0,... K}).
  freeze_batchnorm: Whether to freeze batch norm parameters during
    training or not. When training with a small batch size (e.g. 1), it is
    desirable to freeze batch norm update and use pretrained batch norm
    params.
  inplace_batchnorm_update: Whether to update batch norm moving average
    values inplace. When this is false train op must add a control
    dependency on tf.graphkeys.UPDATE_OPS collection in order to update
    batch norm statistics.
  name: A string name scope to assign to the model. If `None`, Keras
    will auto-generate one from the class name.
"""
super(KerasBoxPredictor, self).__init__(name=name)

self._is_training = is_training
self._num_classes = num_classes
self._freeze_batchnorm = freeze_batchnorm
self._inplace_batchnorm_update = inplace_batchnorm_update

@property
def is_keras_model(self):
return True
@property
def num_classes(self):
return self._num_classes
def call(self, image_features, **kwargs):
"""Computes encoded object locations and corresponding confidences.
Takes a list of high level image feature maps as input and produces a list
of box encodings and a list of class scores where each element in the output
lists correspond to the feature maps in the input list.

Args:
  image_features: A list of float tensors of shape [batch_size, height_i,
  width_i, channels_i] containing features for a batch of images.
  **kwargs: Additional keyword arguments for specific implementations of
        BoxPredictor.

Returns:
  A dictionary containing at least the following tensors.
    box_encodings: A list of float tensors. Each entry in the list
      corresponds to a feature map in the input `image_features` list. All
      tensors in the list have one of the two following shapes:
      a. [batch_size, num_anchors_i, q, code_size] representing the location
        of the objects, where q is 1 or the number of classes.
      b. [batch_size, num_anchors_i, code_size].
    class_predictions_with_background: A list of float tensors of shape
      [batch_size, num_anchors_i, num_classes + 1] representing the class
      predictions for the proposals. Each entry in the list corresponds to a
      feature map in the input `image_features` list.
"""
return self._predict(image_features, **kwargs)

@abstractmethod
def _predict(self, image_features, **kwargs):
"""Implementations must override this method.
Args:
  image_features: A list of float tensors of shape [batch_size, height_i,
    width_i, channels_i] containing features for a batch of images.
  **kwargs: Additional keyword arguments for specific implementations of
          BoxPredictor.

Returns:
  A dictionary containing at least the following tensors.
    box_encodings: A list of float tensors. Each entry in the list
      corresponds to a feature map in the input `image_features` list. All
      tensors in the list have one of the two following shapes:
      a. [batch_size, num_anchors_i, q, code_size] representing the location
        of the objects, where q is 1 or the number of classes.
      b. [batch_size, num_anchors_i, code_size].
    class_predictions_with_background: A list of float tensors of shape
      [batch_size, num_anchors_i, num_classes + 1] representing the class
      predictions for the proposals. Each entry in the list corresponds to a
      feature map in the input `image_features` list.
"""
raise NotImplementedError

@esakkirajask, Please provide the link of the object detection code. Thanks!

@esakkirajask, Is this still issue!

No it closed.
On Wed, 6 Nov 2019, 4:44 pm gadagashwini<EMAIL_ADDRESS>wrote:

@esakkirajask https://github.com/esakkirajask, Is this still issue!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/33600?email_source=notifications&email_token=ANQNUGZ3Y2GAABDK3N2VDIDQSKRINA5CNFSM4JDNL47KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDGFLRY#issuecomment-550262215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANQNUG4OEAITSIZKPWV5EK3QSKRINANCNFSM4JDNL47A
.

Closing this issue since its resolved. Thanks!


Textalignment of nodes in filebrowser

Text of nodes in filebrowser is not vertically centered

Will be fixed with https://github.com/danielperna84/hass-poc-configurator/pull/11


Provide information about how to run the full Conformance Suite

I see that it's controlled by this line, but please document somewhere visible so it's easier to find for others.

Gah, totally missed that doc in my morning run of this. LGTM
Of course this can be kept as "document various things better" as that is always possible, but you did have what I was looking for.

I'll modify the main README to surface this info more visibly

Thank you @abiogenesis-now!

Should be updated now.


How to remove regex from :id and :name parameters?

I like the idea of this router, and I'm planning on using it for an upcoming project. However the automatic regex matching for :id and :name is bothering me. I don't want to use it, and if I do, I'll provide my own regex. How do I prevent any regex matching from happening for those parameters?
I don't think it's a good idea to assume the regex that people want to use for random parameters, it can cause bugs and other problems. But that's just my opinion.

@awulkan thanks your question.
you mean that you don't want to use automatic regex matching for :id and :name ? right?
you can custom it  like this:
	mux := gorouter.New()

	mux.GET("/user/{id:[a-zA-Z0-9]+}", func(w http.ResponseWriter, r *http.Request) {
		//fmt.Println(gorouter.GetParam(r,"id")) //get `id` parameter value from url
		w.Write([]byte("/user/{id:[a-zA-Z0-9]+}"))
	})

	mux.GET("/name/{name:[a-zA-Z0-9]+}", func(w http.ResponseWriter, r *http.Request) {
		//fmt.Println(gorouter.GetParam(r,"name")) //get `name` parameter value from url
		w.Write([]byte("/name/{id:[a-zA-Z0-9]+}"))
	})

I mean I don't want any regex checking at all for any parameter by default. I want to opt-in manually to any regex checks used.

@awulkan I see. Thanks your good advise. Actually the default way is designed to provide convenience (maybe only for someones) and compatible with the grammar (like :id ),so if you don't want to use it , i recommend use this grammar，
like this：{id:[a-zA-Z0-9]+}  ,it will avoid the questions you mentioned.

if use default regex checks unless you know what you're doing

Yes, I figured that I can override it. Maybe I will have to fork this repo and remove it myself though. I don't want the to be any default regex at all for my paths since I can sense it will cause problems down the line. And I don't want to override it everywhere in my app, that's very error prone. :/

i also prefer to remove the default regex  of :id and :name.
example sometime the id param could be a string not number, maybe it is an over-design i think.
anyway i will try to use it in my project, thanks!

I don't understand the meaning of question, but I am interesting this.Don't use regex function or rich document about regex function? Can this method can solve this question?
haha.


Some Helm releases are incorrectly packaged

If you compare the following packaged Helm charts with other Helm charts, you will notice that an incorrect Windows-style directory separator (\ instead of /) has been used, which can cause problems when unpacking or using the chart:

https://kedacore.github.io/charts/keda-1.4.0.tgz
https://kedacore.github.io/charts/keda-1.3.2.tgz
https://kedacore.github.io/charts/keda-1.3.0.tgz

Could these be repackaged and re-uploaded?

We don't support KEDA 1 anymore, is there a reason why you cannot upgrade to 2?

We are not using these old KEDA versions ourselves either. We just noticed the problem when using charts-syncer which, by default, synchronizes all charts, but has trouble syncing these specific ones, because they are broken. The quick fix would probably be excluding these versions from the index.yaml file.

I think we can do that, WDYT @kedacore/keda-helm-contributors ?

Everything is possible :D Just wondering if it's worth it.
Are you open to contribute this, @MShekow?

IMO it's much simpler to just fix the 3 tar.gz files. It's really just extracting them (with a tool that can handle Windows-style backslashes, e.g. 7-zip on Windows) and re-tarring it (on Linux/WSL with standard tar).
This way, the index can still contain all charts.

Certainly, that would be awesome, thank you!


Material-ui component Grid throws invariant warning when hot reloading

Material-ui component Grid throws invariant warning when hot reloading.
keyword: alpha, hooks, hmr, rhl, react, component, grid, v4, invariant

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
I expect to hot-reload material-ui components just fine at all newest versions.
Current Behavior 😯
RHL looks like working, although throws Invariant Violation: "Invalid hook call... warning.
Steps to Reproduce 🕹
I can't make codesandbox.io example, because this is related to RHL. Code below:
// test.tsx
import * as React from 'react'  // v16.8.6
import { hot } from 'react-hot-loader/root'  // v4.8.4
import { Grid } from '@material-ui/core'  // v4.0.0-alpha.8

export default hot<React.FC<IProps>>((props) => (<Grid>change me</Grid>)

// index.tsx
import * as React from 'react'  // v16.8.6
import { render } from 'react-dom'  // v16.8.6, aliased to<EMAIL_ADDRESS>
render(<Test />, document.getElementById('root'))

Steps:

webpack-dev-server --mode development --hot
in browser check console and see no warnings yet
in test.tsx file edit "change me"
check console that RHL managed to put update, but throws Invariant Violation: "Invalid hook call... warning. Also,[HMR] Updated modules: log.js:24
[HMR]  - ./src/test.tsx log.js:24
[HMR]  -<EMAIL_ADDRESS>log.js:24

in test.tsx file change "Grid" to "div"
check console that RHL managed to put update, but throws Invariant Violation: "Invalid hook call... warning. Also[HMR] Updated modules: log.js:24
[HMR]  - ./src/test.tsx log.js:24

in test.tsx file change text again and leave it to "div"
check console that RHL managed to put update, and no warnings anymore.

Context 🔦
I'm trying to setup Material-UI React hook-only app in Typescript.
Your Environment 🌎
Using<EMAIL_ADDRESS>with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>. I'm using only babel to transpile TS code.

@Akuukis I evangelize this everywhere I can, I discourage the usage of react-hot-loader. In the current state of affairs, it doesn't worth it: https://github.com/zeit/next.js/issues/4494. Anyway, I doubt the core issue is in Material-UI. I think that you can move the bug report to https://github.com/gaearon/react-hot-loader.

I followed your advice to remove react-hot-loader. While gradually doing so, it turned out that by removing "react-hot-loader/babel" from webpack config file, the hot from import { hot } from 'react-hot-loader/root' started to work as expected. It now reloads only the last component and doesn't throw any errors about material-ui components.

@Akuukis Does it keep the components stats? If it doesn't that you are happy with it, you can remove react-hot-loader completely, hot reloading will keep working.

@oliviertassinari it does maintain state, so it's worth keeping.


reduce astacus memory usage [DDB-853]

This is not the optimal solution (splitting files, changing the manifest format, etc.)
But profiling shows significant reduction in memory usage for large manifests with these changes, without breaking backward compatibility:

use str instead of Path wherever possible. The Path API is nice, but  the abstraction is too expensive when we have a large number of objects.
only convert to pydantic models the subset of data that we care about, for instance if we want a single snapshot in an entire manifest, don't convert the entire manifest, just the single snapshot.
list_backups: stay completely in json-land, but with type-safe accessors to read the data.

Tested on a large 800MiB manifest, peak usage during list_backups was reduced from ~8GiB to ~4.1GiB and lasted for a much shorter time.

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

Comparison is base (7ad60da) 89.74% compared to head (6e1e977) 90.01%.

Files
Patch %
Lines

...dinator/plugins/clickhouse/async_object_storage.py
72.72%
3 Missing :warning:

astacus/node/sqlite_snapshot.py
86.66%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #178      +/-   ##
==========================================
+ Coverage   89.74%   90.01%   +0.27%
==========================================
  Files         142      144       +2
  Lines        9933    10194     +261
==========================================
+ Hits         8914     9176     +262
+ Misses       1019     1018       -1

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

There's a typo, I will undraft once fixed, but this is still reviewable.


Remove cli flag

After chatting with @chosak, it was decided the cli-flag.sh script adds too much overhead for little gain. This PR merges this script into the frontend and backend scripts, respectively, and strips out the irrelevant parts so that bewds and fewds can adjust their scripts without the danger of impacting the other's workflow.
This also adds a NODE_ENV to the .env sample file, which can be set so that NODE_ENV is set to either development or production without running ./frontend.sh.
Removals

Removes cli-flag.sh script.

Changes

Fixes issue where setup.sh was not passing production cli flag.
Adds documentation about NODE_ENV usage.

Testing
new testing instructions:

./setup.sh should run frontend and backend scripts in development mode.
./setup.sh production should run frontend in production mode and backend scripts in development mode.
./frontend.sh should run frontend script in development mode.
./frontend.sh production should run frontend script in production mode.
./backend.sh should run backend script in development mode.
./backend.sh production should run backend script in development mode.

old testing instructions:

./setup.sh should run frontend and backend scripts in development mode.
./setup.sh production should run frontend and backend scripts in production mode.
./frontend.sh should run frontend script in development mode.
./frontend.sh production should run frontend script in production mode.
./backend.sh should run backend script in development mode.
./backend.sh production should run backend script in production mode.

Notes

There exists the not-great workflow of having NODE_ENV=development in your .env file and running ./frontend.sh production and then running gulp test and having it fail because you've gone from the development to production to development environments and have ended up with the production set of dependencies but the gulp set of tasks. Only way around that currently is to manually set your NODE_ENV the same as the frontend.sh script that was run (./frontend.sh or ./frontend.sh production).

You'll see this warning, which is existing behavior:

WARNING in configuration
The 'mode' option has not been set, webpack will fallback to 'production' for this value. Set 'mode' option to 'development' or 'production' to enable defaults for each environment.

@chosak yes. AFAIK, backend.sh is only ever used locally.

@willbarton @chosak @Scotchester okay, pushed up a change that stripped out the production environment setting.

I may be misunderstanding this, but it's not strange that ./setup.sh production is running the frontend in production mode, but the backend is in development (/requirements/local.txt)?


ثاندر سكانز

Source name
ثاندر سكانز-Thunderscans
Source link
https://ar-thunderscans.com
Source language
arabic
Other details
نعم
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

did you chek all.thunderscans
this Source already exist


Feature/topaz update

Update to topaz, zeus, and magnus/galaxy yamls.

Hi @pelahi , apologies, I only got to this one today. Is there any specific aspect where you would like feedback for this PR?

Not really. Just wanted to make sure you are happy with the overall edits. There are a lot of new files but most are just small modifications to point to specific paths.

Thanks for this work Pascal!
The only general comment that I have is around the scope.
For all clusters that will stick around also with Setonix for a while (Topaz, Askap-Ingest?, Garrawarla?), let's define the goal for using Spack, so that we can in turn define a "support level" for users.
My suggested minimal level of goal/support for these clusters would be to use Spack to complement installations on a basis of necessity/convenience, that is, to install packages/applications where Spack is more convenient for whatever reason. In this context, unlike Setonix, we can aim to have a Spack configuration that just "does the job", without spending too much time, for instance, on system externals. Likewise for what regards users, I suggest we do not commit to Spack support for them; they can use it, but we won't help them. The only exception here is probably PaCER/ASKAP/MWA, where we might occasionally need to help them out.
Finally, for these longer-lived clusters, once you're happy with a working configuration, I suggest to move them from the "examples" directory to the root directory of this repo, so they're easier to find, and it's more obvious that these configs are being actively used (we can "archive" them later down the track, when the clusters are retired).

I'll adjust this when I have time, moving stuff from examples.

Sure. Meanwhile, if the current changes are finalised on your end, feel free to merge!


Add support for ZoneInfo

Change Summary

Related issue number
fix #4710
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[ ] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

hey @Youssefares correct me if i'm wrong but i guess its similar to this one we have in Pydantic-extra https://github.com/pydantic/pydantic-extra-types/pull/193

hey @Youssefares correct me if i'm wrong but i guess its similar to this one we have in Pydantic-extra pydantic/pydantic-extra-types#193

This does seem very similar, zoneinfo.ZoneInfo is part of the python std library though. I maybe wrong but my understanding of pydantic-extra-types is that it defines additional convenience types, while pydantic itself is where we offer support for standard types. I am not 100% sure about the need for an extra-type in this scenario given zoneinfo now exists since 3.9.
I'll leave to maintainers to decide whether the extra type using pytz is still required, but I believe supporting zoneinfo is very much worthwhile eitherway.

hey @Youssefares correct me if i'm wrong but i guess its similar to this one we have in Pydantic-extra pydantic/pydantic-extra-types#193

This does seem very similar, zoneinfo.ZoneInfo is part of the python std library though. I maybe wrong but my understanding of pydantic-extra-types is that it defines additional convenience types, while pydantic itself is where we offer support for standard types. I am not 100% sure about the need for an extra-type in this scenario given zoneinfo now exists since 3.9. I also think (in my experience at least), most people try to drop the pytz dependency from their project nowadays if zoneinfo does the job.
I'll leave to maintainers to decide whether the extra type using pytz is still required, but I believe supporting zoneinfo is very much worthwhile eitherway.

Yes as you said the best answer we can have is from maintainers themselves.
cc @sydney-runkle @samuelcolvin

please review

Exciting stuff here - this is next in my queue to review :). Should be able to get to it this evening or first thing tomorrow :).

I'll leave to maintainers to decide whether the extra type using pytz is still required, but I believe supporting zoneinfo is very much worthwhile eitherway.

I think we should definitely just stick to std lib support, so the zoneinfo addition here is great!

CI's failing because this issue #9761 been resurrected somehow it seems? Looking into it..

That look better.. 🟢 ✅


Corrected license in package.json to SPDX

As caught in https://github.com/gmusic-utils/gmusic-theme.js/issues/4, we have been using a deprecated key for our licenses (should be license not licenses) as well as not the proper SPDX format. This should correct the license link on our npm page.
In this PR:

Moved from licenses to license key in package.json
Moved to proper license name (Unlicense not UNLICENSE in package.json)

http://spdx.org/licenses/

/cc @gmusic-utils/gmusic-js

LGTM

:+1:

This has been landed/released in 4.0.3. Hooray for proper licensing on npm :tada:
https://www.npmjs.com/package/gmusic.js


DDRx drivers inside

HI
I see there is not only scope inside but also open source
DDRx RAM driver.
Can there be some quick info about DDR driver, what
works what doesn't, list of memory chips known to work
and would DDR driver work for a small 32bit soft core CPU

I works with DDR1/2/3 I've tested it on spartan3e, spartan3a, virtex-5, artix and latticesemi ecp3.
You can check this release https://github.com/hdl4fpga/hdl4fpga/releases/tag/v0.0


scrollableParent prop type fix

When scrollableParent prop is set to a DOM element, waypoint writes a warning Warning: Failed propType: Invalid prop scrollableParent supplied to Waypoint, expected a ReactNode. to the browser console
This is a fix for that warning according to what README says about scrollableParent prop type

Thanks for taking the time to prepare a PR @alex-ionochkin!
There's probably a better proptype than any here, but I'm not sure which one it is. I'm looking at the PropTypes documentation and there's no type for DOM elements. Which makes sense because A) you should be using React components, and B) in order to gain access to a DOM element in a React tree you need to render it at least once first. However I think the most common thing to pass in as scrollableParent is window or perhaps document.body.
We could perhaps get away with using PropTypes.object, but I think what you have in this PR is fine.


Inference without CUDA

I was getting a CUDA related error when trying to run inference without CUDA: https://github.com/yassouali/pytorch_segmentation/blob/master/inference.py#L116
If I change this line to: checkpoint = torch.load(args.model, map_location=device) it seems to work, the device automatically detects the presence of a gpu.

Thank you for this feedback, I just corrected the error.


Typo Fix

Fixes #7150
Category

[x] Content fix

Related issues
Fixes #7150
What's in this Pull Request?
Typo Fix in Documentation

Docs Build status updates of commit 5fa9f98:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/transform/modernize-userinterface-site-pages-powershell.md
:white_check_mark:Succeeded
View

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 54cbbbf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/transform/modernize-userinterface-site-pages-powershell.md
:white_check_mark:Succeeded
View

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


Module no found

Having trouble loading files and think this is webpack related.
I'm using the react-native-navigation module and here is an example error:
ERROR in ./~/react-native-navigation/src/Navigation.js
Module not found: Error: Cannot resolve 'file' or 'directory' ./platformSpecific in
.../node_modules/react-native-navigation/src
@ ./~/react-native-navigation/src/Navigation.js 17:24-53

Also loading images is baffling. I'm using a loader (see config below)
but still have trouble:
ERROR in ./src/screens/screen.js
Module not found: Error: Cannot resolve 'file' or 'directory' ../../img/colors.png in /[rootpath]/src/screens
 @ ./src/screens/screen.js 51:110-144

thought I had wrong path but tried everything with no luck.
Am I on the right path in regards to this being webpack related? Below is my webpack.config.js
(function() {
  'use strict'

  const path = require('path')

  module.exports = {
    debug: true,
    devtool: 'source-map',
    entry: {
      'index.ios': ['./src/main.ios.js'],
      'index.android': ['./src/main.android.js'],
    },
    output: {
      path: path.resolve(__dirname, 'build'),
      filename: '[name].js',
    },
    stats: {
      colors: true,
      modules: true,
      reasons: true,
      errorDetails: true,
    },
    module: {
      preLoaders: [
        {
          test: /\.(js|jsx|es6)$/,
          include: path.resolve(__dirname, 'src'),
          loader: 'eslint-loader',
        },
      ],
      loaders: [
        {
          test: /\.js$/,
          include: /node_modules\/react-native/,
          loader: 'babel',
          query: {
            cacheDirectory: true,
            presets: ['es2015', 'stage-1', 'react'],
          },
        },
        {
          test: /\.(js|jsx|es6)$/,
          exclude: /node_modules/,
          loader: 'babel',
          query: {
            cacheDirectory: true,
            presets: ['es2015', 'stage-1', 'react'],
          },
        },
        {
          test: /\.(jpe?g|png|gif|svg)$/i,
          loaders: [
            'file?hash=sha512&digest=hex&name=[hash].[ext]',
            'image-webpack?bypassOnDebug&optimizationLevel=7&interlaced=false',
          ],
        },
      ],
    },
  }
}())

Thanks for the detail. Take a peek in the morning

Unless you're preloading images or using a technique like LQIP you may want to consider using something like Image Sets (as in the case for iOS) to store the app assets in the bundled app. Other than that I'm not too sure to be honest. Webpack can be a complex beast at times. Here's a list of other projects for inspiration (both React and React Native, most of which use Webpack). Good luck!


Delete event

The delete event does not working

According to our tests deletion logging should work just fine. What are you doing differently from the tests?
Without any extra info regarding the issue you're having it's impossible for me to help you. How can I recreate the problem?

Hi @freekmurze,
I'm using on the module like this:
class ContractorBenefit extends Model {     use LogsActivity; }

What am I doing wrong?

It's impossible to say with so little code to go on.
My suggestion is that you examine the tests that go along with this package. They prove the deletion logging is working. Try to find how your code differs from the tests.


Question about points  in preprocess/sfm_face_coordinates.npy

In preprocess/sfm_face_coordinates.npy, there are 14 3d points which i don't know their relations to face landmark points. To be specific ,if i want to get the 3d coordinates of the center of left eye(or right eye), which row it is in "face_model_3d_coordinates" array? Thanks for your great work , looking forward to your reply.

Hullo. Sorry, I do not know how I missed this...
You can see in a new script that I just released for our NeurIPS paper, that we calculate the individual gaze for the left and right eyes (we do not use this actually).
The relevant lines of code are at: https://github.com/swook/faze_preprocess/blob/master/create_hdf_files_for_sted.py#L208


Overridden core class does not work when in artisan

Hi
In a fresh Laravel build, I cannot get overridden IoC bindings to work everywhere in the application.
Suppose a service provider that overrides a core class, e.g. cache:
class NewServiceProvider extends ServiceProvider
{
    protected $defer = true;

    public function register()
    {
        $this->app->bind('cache', function($app) {
        	return new \stdClass; // demo purpose
        });
    }

    public function provides()
    {
    	return ['cache'];
    }
}

The provider is then added at the bottom of app.providers config.
Now modify routes.php to the following and go check the result:
Route::get('/', function () {
    dd(app('cache'));
});

// Results in an empty stdClass being shown. It works!

However, fire up artisan tinker and do the same:
$ php artisan tinker
>>> app('cache')
=> Illuminate\Cache\CacheManager

Suddenly the override isn't working anymore...
The same behavior is encountered when processing event listeners...
Is this normal behavior and am I overlooking something? Or is this some kind of bug?

Please ask on the fourms.

Well, since it could be a bug I'd thought to post it over here..
Anyhow:
http://laravel.io/forum/10-01-2015-overridden-core-class-does-not-work-when-in-artisan

I'm sorry to reopen this issue @GrahamCampbell but after some more digging I'm quite convinced this is a bug in the framework...


Anvil fork is blocking the terminal

Component
Anvil
Have you ensured that all of these are up to date?

[x] Foundry
[x] Foundryup

What version of Foundry are you on?
0.2.0 (57180fc 2023-11-09T01:52:52.702931742Z)
What command(s) is the bug in?
anvill --fork-url xxxx
Operating System
Linux
Describe the bug
After I fork the blockchain using the anvil fork command, I am willing to send some transactions that I have regrouped into init_wallet.sh. But I am unable to do so as the the fork command has renderedmy terminal unresponsive.
My use case is that I am launching foundry fom a docker compose file and I am willing to send a few tx after forking in the following way:
image: ghcr.io/foundry-rs/foundry:latest
    restart: unless-stopped
    volumes:
      - ./init_wallet.sh:/init_wallet.sh
    entrypoint: anvil --fork-url "https://mainnet.infura.io/v3/xxx" --host "<IP_ADDRESS>" --silent
    command: ["sh", "init_wallet.sh"]

Thanks for looking at this

could you perhaps try with another api key?

So as it was suggested by @DaniPopes I added the "&" to run the entrypoint command in the background. But this didnt solve the issue:
    entrypoint: anvil --fork-url "https://mainnet.infura.io/v3/xxxx" --host "<IP_ADDRESS>" &     command: ["sh", "init_wallet.sh"]
The error message I am getting is:
   For more information, try '--help'.
error: unrecognized subcommand 'sh'

Usage: anvil [OPTIONS] [COMMAND]

I am a bit confused by the error message, because if I log into the instance, I can launch the command in the terminal and it works fine. I have also try to run the entrypoint command AND the init_wallet.sh command in the background without joy.
I am not sure if the api key is relevant here but will try still. Thanks

so entrypoint is a command that is appended on every command. It was causing this erratic behavior.


RTL WEB SUPPORT

hey , please add web support for RTL usin textdield inputs .
web rtl dosent work as it should .
mouse cursor keeps bouncing to the right , no option to mark a single letter or single word to edit.

@kfirbracha
can you provide your flutter doctor -v and a minimal reproducible code sample

Hi @kfirbracha
It is not fully supported https://github.com/flutter/flutter/issues/47745
Can you please provide your flutter doctor -v your flutter run --verbose and a complete reproducible minimal code sample
Thank you

flutter Team Please solve the text field problem that it does not support RTL , Writing pointer is not working properly I am losing clients Because of this problem

@nizar-khan
as for the comment above there's an issue open addressing this or a similar case,
if you believe that there's some element that said issue is not addressing,
please provide your flutter doctor -v your flutter run --verbose and a complete reproducible minimal code sample

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


SPM Version tool

This PR adds the missing swift tool version in Package.swift.
Otherwise I can not install the library with Xcode

I released new version 1.0.0-rc.4.


docs: 文档修改,移除图例废弃的 api & tooltipStyles 类型定义修整

Checklist

[ ] npm test passes
[ ] tests and/or benchmarks are included
[ ] commit message follows commit guidelines

Description of change

Pull Request Test Coverage Report for Build 500360571

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

Files with Coverage Reduction
New Missed Lines
%

src/chart/controller/gesture.ts
7
82.14%

Totals

Change from base Build 500333952:
-0.06%

Covered Lines:
10820

Relevant Lines:
11226

💛  - Coveralls

Pull Request Test Coverage Report for Build 500360571

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

Files with Coverage Reduction
New Missed Lines
%

src/chart/controller/gesture.ts
7
82.14%

Totals

Change from base Build 500333952:
-0.06%

Covered Lines:
10820

Relevant Lines:
11226

💛  - Coveralls


Despawn entity rather than removing component

Removing the component from the entity would leave a dangling entity with no components attached to it, perhaps bevy optimizes this away internally but I think this looks cleaner.
Objective

Make example cleaner.

Solution

By despawning the entity rather than removing the component.

Yes, I read the code poorly, above it attaches Components to the entity and then removes the task part.


Update Drupal 9.5.x

What does this PR do?

Updating Drupal plugin for version 9.5.x

Relevant logs and/or screenshots
Testing performed using sandbox
INSTALL PLUGIN

ADD TO CART

CHECKOUT

PAYMENT

SUCCESS ORDER

PENDING ORDER

FAILED ORDER

Hello, I have tried the 9.5.0 version and getting this:
when putting the module to my website, I am getting this error and cannot access my website

the error log is as attach:
error_wsod.txt

adding in "core_version_requirement: ^9" in file commerce_razermerchantservices_normal.yml fix the issue

Getting this error when trying to pay:

this are the error log:
error_payment.txt

adding in "core_version_requirement: ^9" in file commerce_razermerchantservices_normal.yml fix the issue

Thank you for letting us know. Will update suggested fix.

Getting this error when trying to pay:  this are the error log: error_payment.txt

Please make sure amount, orderID, verify key and merchantID is correct. For further discussion, you may join RMS Developer Forum Telegram Group https://github.com/RazerMS/Telegram-RMSDeveloperForum


Use AChoreographer_registerRefreshRateCallback to get actual refresh rate

Today, the Android embedding calls Display#getRefreshRate once at initialization to get the refresh rate, and then assumes that stays stable when talking to the VSync waiter.
This is actually incorrect, since modern devices can dynamically change refresh rates. However, repeatedly calling getRefreshRate can be slow on some phones (2.5-5ms measured on a Huawei Nova 3), and calling it every frame is expensive.
We should use the AChoreographer_registerRefreshRateCallback where it's available to more efficiently get the FPS rate. This will unblock using the vsync target time for https://github.com/flutter/engine/pull/29727
@blasten @ds84182 FYI

makes sense. Do we know how fast AChoreographer_registerRefreshRateCallback  dispatches the first callback?

It's documented as dispatching as soon as you register, unless you manage to unregister before it dispatches :)
I'm playing with an implementation of this on a Pixel 4 and it seems like (for Flutter Gallery at least), the Pixel 4 basically runs it at 90hz ... constantly. I'd like to see a case where this actually varies to make sure this even matters...


Alevtina Brown

Please, accept my pull request. Russian Translation.

Great job, I will check it. :-)
Thank you.

@alev7ina Thank you! :)


CardTypeRegister doesn't have to be a singleton

Hi,
CardTypeRegister doesn't have to be a singleton, it could just live with the instance of the text field. You can make the init public, and also create a constant for the default credit card types array that you use as a public variable.

Hi, thanks for the suggestion! The changes you propose certainly make sense! I added this in #61


minimal example

I'd really appreciate a minimal running example how to send data P2P using peer-data and peer-data-server libraries (basic use case). The README.md mentions how to use the library with different frame works. The website shows an example without explanation what it does or what to expect: Should I get output on the console if I open the page twice? The same applies to the Basic Example in the documentation.

Besides, it would be easier to get startet if there was no assumption that a specific build tool (e.g. Webpack) was being used. You might consider using
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Basic PeerData Example</title>
</head>
<body>
    <script type="module">
      import PeerData from<EMAIL_ADDRESS>
      console.log(PeerData);
    </script>
</body>
</html>

If you combine this with a simple peer-data-server example, you would only need 3 files:
├── index.html
├── server.js
└── package.json

No build script is required. You only need peer-data-server and socket.io as dependencies.

take a look at this example: https://github.com/vardius/peer-data/tree/master/examples
this example does not use socket channel as a signalling server
its simple local webrtc connection with custom signalling channel simply forwarding signalling event from one instance to another

I created one with socket channel as signalling server
hope this helps https://github.com/vardius/peer-data/tree/master/examples

Thanks :+1:
local-connection works, but I get errors when I try to connect in socket-channel
Uncaught DOMException: Failed to execute 'addTrack' on 'RTCPeerConnection': A sender already exists for the track.
    at RTCPeerConnection.addTrack (https://webrtc.github.io/adapter/adapter-latest.js:628:31)
    at https://unpkg.com/peer-data@3.2.2/dist/index.es.js:193:73
    at Array.map (<anonymous>)
    at Participant.addStream (https://unpkg.com/peer-data@3.2.2/dist/index.es.js:193:32)
    at http://localhost:3000/remote.js:38:21
    at https://unpkg.com/peer-data@3.2.2/dist/index.es.js:109:70
    at Array.forEach (<anonymous>)
    at EventDispatcher.dispatch (https://unpkg.com/peer-data@3.2.2/dist/index.es.js:109:38)
    at Participant.dispatchRemoteStream (https://unpkg.com/peer-data@3.2.2/dist/index.es.js:337:30)
    at RTCPeerConnection.wrappedCallback (https://webrtc.github.io/adapter/adapter-latest.js:2692:9)

participant:error DOMException: Failed to execute 'setLocalDescription' on 'RTCPeerConnection': Failed to set local offer sdp: Called in wrong state: kHaveRemoteOffer
(anonymous) @ remote.js:24
(anonymous) @ EventDispatcher.ts:25
EventDispatcher.dispatch @ EventDispatcher.ts:25
(anonymous) @ Participant.ts:147
Promise.catch (async)
Participant.renegotiate @ Participant.ts:147
(anonymous) @ remote.js:39
(anonymous) @ EventDispatcher.ts:25
EventDispatcher.dispatch @ EventDispatcher.ts:25
Participant.dispatchRemoteStream @ Participant.ts:221
wrappedCallback @ adapter-latest.js:2692

I guess there exists some code from local-connection in socket-channel that causes this issue, since title and description of socket-channel are

local connection with custom signaling channel

Hey man, i updated reconnect flow to use rollback. there are some still errors in console, for both socket and local examples.
I suspect Failed to execute 'addTrack' on 'RTCPeerConnection': A sender already exists for the track. appears due to even ontrack being spawn twice which is most likely adapter issue. Correct me if i am wrong, but i did not find any other explanation now.
There is new error about remote description, its due to new even being handled by peer-data. https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/onnegotiationneeded
Seems like onnegotiationneeded is called also during initial connection to room before any offer/answer is generated. Will investigate more next day.

Fix the other issue related to onnegotiationneeded https://github.com/vardius/peer-data/releases/tag/v3.2.4

I suspect Failed to execute 'addTrack' on 'RTCPeerConnection': A sender already exists for the track. appears due to even ontrack being spawn twice which is most likely adapter issue. Correct me if i am wrong, but i did not find any other explanation now.

remote:track is printed twice. That means
participant.addStream(event.streams[0]);

is called twice. Should there not be only one remote participant? :thinking:


Update APIs to use MobileCoinRng instead of RngSeed

Soundtrack of this PR: A 'random' InSync song lol
Motivation
Sync up iOS and Android's implementation of idempotence for cross-platform compatibility
In this PR

Deprecate the API calls for prepareTransaction that use RngSeed
De-Deprecate the API calls that use MobileCoinRng
Deprecated APIs will generate a new RNG from the seed, and then call the de-deprecated APIs with that RNG (all that part is done by the API client code on Android).  Then for equivalency, when we receive an Rng via the de-deprecated APIs (that match Android in that they accept an Rng), we will generate a seed used to create a new ChaCha20Rng for the building of the output transaction (which will match Android once Android is updated to generate the seed before any other use).

would be great to have a test for this, @dolanbernard can you provide a test data from Android to cross-test?


Improve ChipInput component

Add Autosuggest for existing tags
Support themed Redo styles for underlined variant the same way they work for underlined TextInput (currently only outlined works because of the class name differences)
Allow ChipInput to adapt to various widths easily

https://mui.wertarbyte.com/


Pgp error

Describe the bug
The installer cant import any pgp key, so no packages are installed

Part of the installer process it updates the archlinux-keyring. But, sometimes you might need to do that manually before running it.

Thanks, also happy easter :)


update license header

In #169, @psifertex asks

Can you clarify how the "All Rights Reserved" on that interacts with the Apache license on the whole repo? I assume it's an artifact from before the apache license was added?
I'd like to create a derivative of that script but obviously I can't do that if it's under full copyright and not Apache. :-)

Originally posted by @psifertex in https://github.com/fireeye/capa/issues/169#issuecomment-660282048
and continues:

Thanks! If you don't mind a suggestion, maybe:
Copyright (C) 2020 FireEye, Inc. Released under an Apache License Version 2.0.
is worth running by the legal team. If they've already agreed to the license I can't imagine they'd mind as you point-out.
Normally having the repo license be Apache 2.0 is enough to be sure, but it's not uncommon for some repositories to be generally released under a specific license but to have some files inside of it released under a different license and my understanding of "All Rights Reserved" is that it's explicitly full copyright that does not allow any other adaptations like those encouraged by open source licenses.

Originally posted by @psifertex in https://github.com/fireeye/capa/issues/169#issuecomment-660716440
I've run this past the legal team and they've reiterated that capa is released under Apache 2.0 with no modifications. The existing header was intended to specify copyright claims rather than licensing. They've recommended the following (more detailed) header for source files:
* Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License.
* You may obtain a copy of the License at: [package root]/LICENSE.txt
* Unless required by applicable law or agreed to in writing, software distributed under the License
* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and limitations under the License.

We should update our source files with this header so that there is no confusion about capa being released as Apache 2.0 and that we are welcome of any and all uses.

That updated header looks great, thanks for running that down.


P2P: Don't add peer to maintained connections if local node is not ready

PR description

Fixes an issue due to which multiple peer connections are setup over time to the local node if it is present in the static nodes list. The code was not handling the case where local node is not yet ready (encountered at startup). https://github.com/hyperledger/besu/pull/1703 added logic to skip adding local node to the set of maintained peer connections. However, the check fails to take into account the case where the local node is not marked ready (occurs at startup since sanitizePeers is called before the call is made to setup the networkRunner at which time the local node is setup and marked ready.

Also adds a one time call to initiate connection to nodes in static nodes list after local node is ready. In the absence of this call, the connection attempt is made after the configured interval (default 60 seconds) to check maintained connections. This is to make sure the node doesn't have to wait for almost a minute before it can start interacting with peers (and sync blocks etc.).

Fixed Issue(s)
Fixes https://github.com/hyperledger/besu/issues/1702
Documentation

[ ] ~I thought about documentation and added the doc-change-required label to this PR if
updates are required.~

No doc updates necessary.
Changelog

[x] I thought about the changelog and included a changelog update if required.

Need some hand holding to fix the acceptance test https://github.com/hyperledger/besu/blob/main/acceptance-tests/tests/src/test/java/org/hyperledger/besu/tests/acceptance/bootstrap/StaticNodesAcceptanceTest.java#L38 . I see there is a separate ProcessBesuNodeRunner() setup but not sure how/where the static nodes are being processed in the acceptance test. Thanks in advance!

Need some hand holding to fix the acceptance test https://github.com/hyperledger/besu/blob/main/acceptance-tests/tests/src/test/java/org/hyperledger/besu/tests/acceptance/bootstrap/StaticNodesAcceptanceTest.java#L38 . I see there is a separate ProcessBesuNodeRunner() setup but not sure how/where the static nodes are being processed in the acceptance test. Thanks in advance!

I'm looking at this @vdamle

@vdamle the change that breaks the AT isn't required - so then you just need to verify that the additional call to checkMaintainedPeers is actually helpful (run up the code on a besu node and see?) and we can review

closing this one - I couldn't push to your branch so I made a new PR fixing the changelog so we could get it in the RC #4543


rvm::system does not install default ruby

The rvm::system recipe does not actually install the default Ruby listed in the attributes and rvm is not accessible as its path has not been added to the PATH environment variable

I can confirm that #285 isn't resolving the issue. I have to explicitly set a node attribute
{"rvm": {"install_rubies" : true }}

for chef run installs ruby. Maybe the attribute should be setted to true by default in attributes default file.

I just send this PR #294 to set the default value to install_rubies attribute.

Closing due to inactivity.
If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help!
Thanks,
Sous-Chefs


The variable type rule has an issue when the variable type is a non-null type.

Describe the bug
The variable type rule has an issue when the variable type is a non-null type.
To Reproduce
The following query is rejected by the server:
query f($x: Episode!){
  hero(episode:$x)
  {
    name
  }
}

with the following error:
{
  "errors": [
    {
      "VariableName": "x",
      "Message": "The variable value is not of the correct type."
    }
  ]
}

The hero signature looks like the following hero(episode: Episode!): Character.
Expected behavior
The query should have not been rejected.

private void CheckForInvalidValueType(in Variable variable)
        {
            if (!variable.Type.IsInstanceOfType(variable.Value))
            {
                throw new QueryException(new VariableError(
                    "The variable value is not of the correct type.",
                    variable.Name));
            }
        }

This issue is fixed with PR #274


@deck.gl/mapbox depends on mapbox css

Explanation
I am building an app that needs to be injected in an arbitrary page, meaning I have no control over how the page is constructed and can't guarantee that I'll be able to inject mapbox-gl's CSS code. When the CSS in not in the page inserting a deck gl layer sets the wrong size for the canvas and causes visual artifacts.
Actual Result
deck.gl resizes the canvas and gl contexts causing visual bugs:

the rendered map ends up outside the div assigned to it (it should be rendered where the red rectangle is)
the data on the deck.gl layer is not rendered at the correct location
a scroll bar is added

Expected Result
deck.gl respects the boundaries and size set by mapbox-gl and renders data at the correct location:

Reproduce Steps

Create HTML that has a div where the map should be rendered that is NOT the full size of the viewport
Import mapbox-gl and deck.gl but NOT mapbox's CSS
Create and insert a deck.gl MapboxLayer

The issue can be fixed by either importing getDeckInstance and passing a custom deck instance to the layer:
const deck = getDeckInstance({
    map,
    gl: map.painter.context.gl,
});
deck.setProps({
    autoResizeDrawingBuffer: false,
    width: false,
    height: false,
});
const myDeckLayer = new MapboxLayer({
    id: 'my-scatterplot',
    type: ScatterplotLayer,
    data: [
        {position: [-95.712891, 37.090240], size: 100000}
    ],
    getPosition: d => d.position,
    getRadius: d => d.size,
    getColor: [255, 0, 0],
    deck,
});

or setting the deck instance properties right after the layer has been added to the map:
map.addLayer(myDeckLayer);
myDeckLayer.deck.setProps({
    autoResizeDrawingBuffer: false,
    width: false,
    height: false,
});

either way seems hacky though.
Code Example
https://codepen.io/anon/pen/oQMXQe

A fix is published in 6.3.1


Need way to call clustered node methods in one go

Currently, clustered nodes are accessible through the blitz.node[id].workers . Having a blitz.node[id][method] - method would heavily simplify usage.

Note: Add base class to blitz.js-utils which handles blitz object transactions. Same Class will be expanded for exposed methods via global object.


Add PathFind, PathNode, and PathNodeLink classes

Summary

Add PathFind, PathNode, PathNodeLink, and 2 flag enums
Add SHVDN.NativeMemory.FVector3 to avoid unnecessary GC pressure when you just get or set simple vector3 values.
Replace direct 2 Native.Call calls in World.GetNextPositionOnStreet with methods of PathFind, keeping how to determine the closest node intact.

The natives for vehicle path node handles can easily crash the game for using one which lower bit is not zero if the corresponding CPathRegion instance is not loaded, so I do not adopt any of them for PathNode because custom existing check is necessary to avoid cheap death of the game process.
I hope this PR will make devs who contribute to CodeWalker and/or those who plays around with it happier.

I'm creating a nested static class in NativeMemory because separating methods would help avoid seeing irrelevant ones when you type something to use IntelliSense. I wouldn't mind new methods for internal use getting separated in different ways tbh, though.


readOnly behaviour is not correct

Observed Behaviour
https://jsfiddle.net/3taojbyk/3/
textarea tag is not mutable with any provided readOnly property
Expected Current Behaviour
textarea should be mutable when rendered with readOnly={false}

How do you use it? Rendering <textarea readonly/> does not seem to work
https://jsfiddle.net/3taojbyk/4/

@wasd171 ah I understand now. Would you mind make a PR for this please? There should be a readonly string in constants.ts that needs to be readOnly.

Closed by https://github.com/trueadm/inferno/pull/576


feat(shield): withShield optional string path

This is https://github.com/omar-dulaimi/prisma-trpc-generator/pull/65 but addressing @omar-dulaimi's comments of consistency with string | boolean pattern.
cc: @lottamus
Description
When withShield is true,
generator trpc {
  ...
  withShield = true
}

We generate a boilerplate configuration using allow for all procedures and we overwrite this file on each generation.
It looks something like this:
import { shield, allow } from 'trpc-shield';
import { Context } from '../../context';

export const permissions = shield<Context>({
  query: {
    aggregate: allow,
    ...
  },
  mutation: {
    create: allow,
    ...
  },
});

This pull request updates the option withShield to allow a string path to your own tRPC Shield to use as the permission layer.
Similarly to withMiddleware, this will extend withShield to allow either a boolean or a string path.
When withShield is a string path:
generator trpc {
  ...
  withShield = "../../libs/shield"
}

The routers/helpers/createRouter.ts will use the path instead:
import { permissions } from "../../libs/shield";

Hey @imsanchez
Thanks for the PR, I'll take a look very soon

Ready for your approval @omar-dulaimi

Released in https://github.com/omar-dulaimi/prisma-trpc-generator/releases/tag/1.4.0
Thank you @imsanchez
Keep them coming!


Provider doesn't seem to honor retainOnDelete

In looking into https://github.com/pulumi/pulumi-synced-folder/issues/44, I was assuming that by setting retainOnDelete: true , I'd be instructing Pulumi (as the docs suggest) "not [to] call through to the resource provider’s delete method". The Command provider, however, doesn't seem to honor this setting.
Given an S3 bucket and this TypeScript program (which uses the managedObjects property to delegate control over the bucket's files to the AWS CLI via the Command provider, I'd expect that after a pulumi up and pulumi destroy, the bucket would contain the files that'd been synced from my-folder:
import * as synced from "@pulumi/synced-folder";

const folder = new synced.S3BucketFolder("my-folder", {
    path: "./my-folder",
    bucketName: "my-bucket",
    acl: aws.s3.PrivateAcl,
    managedObjects: false,
}, { retainOnDelete: true });

Instead, those files are deleted:
$ pulumi up

Updating (dev)
...
     Type                                   Name                    Status
 +   pulumi:pulumi:Stack                    synced-folder-dev       created (0.95s)
 +   └─ synced-folder:index:S3BucketFolder  my-folder               created (0.60s)
 +      └─ command:local:Command            my-folder-sync-command  created (1s)

Resources:
    + 3 created

Duration: 4s

$ aws s3 ls s3://my-bucket
2023-07-31 11:24:52        112 index.html

$ pulumi destroy

Destroying (dev)
...

     Type                                   Name                    Status
 -   pulumi:pulumi:Stack                    synced-folder-dev       deleted
 -   └─ synced-folder:index:S3BucketFolder  my-folder               deleted
 -      └─ command:local:Command            my-folder-sync-command  deleted (3s)

$ aws s3 ls s3://my-bucket
[no results]

Here's a snip from the logs showing the delete step being run:
I0731 11:53:51.206716   22698 log.go:81] SnapshotManager: Beginning mutation for step `delete` on resource `urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder`
I0731 11:53:51.206743   22698 log.go:81] SnapshotManager.doDelete(urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder)
I0731 11:53:51.206754   22698 log.go:81] SnapshotManager.markPendingOperation(urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder, deleting)
I0731 11:53:51.374517   22698 log.go:81] StepExecutor worker(1): applying step delete on urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder (preview false)
I0731 11:53:51.374577   22698 log.go:81] SnapshotManager: deleteSnapshotMutation.End(..., true)
I0731 11:53:51.374595   22698 log.go:81] SnapshotManager.markOperationComplete(urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder)
I0731 11:53:51.374605   22698 log.go:81] Marked old state snapshot as done: urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder
I0731 11:53:51.501183   22698 log.go:81] StepExecutor worker(1): step delete on urn:pulumi:dev::synced-folder::synced-folder:index:S3BucketFolder::my-folder retired

@cnunciato I believe that pulumi-synced-folder is a Pulumi component so the retainOnDelete setting shouldn't apply? Reference documentation link

The retainOnDelete resource option does not apply to component resources, and will not have the intended effect.

Closing this as a no-op.
My recommendation would be to work with provider-level resources for this.

Although, just to be thorough (and for the docs, as I'd like to make sure they're complete): Are there any plans to support this for components also? Or is there some reason that can't be done?


Offset drag instantiated scenes that result in a collision by its bounds to prevent overlap

Let me know if this needs a proposal, but to me this is more of a bug because I can't think of a use-case where the current behavior is the desired one.
Before:
https://github.com/godotengine/godot/assets/105675984/abf1e70b-790e-4257-b4dc-5dcf2e59a7c4
After:
https://github.com/godotengine/godot/assets/105675984/d1713012-70f4-4fbf-a9d3-8da8cb4e85a4

This gets a little more complicated when there are multiple detached visual instances in a scene. I'll keep investigating. That could be part of a separate PR or included in this one if necessary. I would argue this PR at least builds towards the right direction.

Could have a proposal since this is added functionality, if so I would also suggest adding this functionality to the move and/or select mode tools because this could be quite useful for them as well and isn't easily recreated by any other existing functionality afaik.
Regarding the implementation, how does it interact with non-axis aligned surfaces?

Regarding the implementation, how does it interact with non-axis aligned surfaces?

I need to get back into this, but I believe that preview nodes are packaged into temporary node that is not rotated, so the bounds are always axis aligned. My current issue is that the scenes origin isn't always the center of the AABB, so that needs to be factored into the offset, if that even makes sense.

I would also suggest adding this functionality to the move and/or select mode tools because this could be quite useful for them as well and isn't easily recreated by any other existing functionality afaik.

Would be cool.

I need to get back into this, but I believe that preview nodes are packaged into a temporary node that is not rotated, so the bounds are always axis aligned. My current issue is that the scenes origin isn't always the center of the AABB, so that needs to be factored into the offset, if that even makes sense.

 Does it stick the center of the closest aabb side to the surface or something?

Thanks!


Investigate using GitHub actions for publishing python-client to PyPI

This was a failure. I'm not sure why that workflow didn't trigger when we released version 3.7. The GitHub UI doesn't show any sort of logs for actions that didn't actually get triggered, so my gut feeling is that this is too expensive to investigate to be worth implementing.


Increase pkg/proof test coverage

Candidates for testing

unit test the RowProof struct
unit test the ShareProof struct

Will close in favor of https://github.com/celestiaorg/celestia-core/issues/953 it's not immediately obvious where these unit tests should live b/c although the structs are defined in celestia-core, they're not really used outside celestia-app


feat: general compression for value page buffer

This PR introduces general compression for value page buffers, starting with zstd, to reduce the on-disk size of all types of value arrays.
Here are the key details:

After some code exploration, I implemented this as a buffer encoder for ValueEncoder instead of as an independent physical encoder. Please let me know if this approach is suitable.
Enhancements to ValuePageScheduler.schedule_ranges allow it to read the entire buffer range for compressed buffers. To support this, I added a new buffer_size metadata to several buffer structs, populating these variables using metadata from Lance.
Currently, only zstd compression with the default level (level 3) is implemented. If this approach is deemed suitable, we can consider adding more general compression methods in the future.
In a specific test case, the original Lance file was 13MB. After applying zstd compression, its size was reduced to 7.4MB.

I have updated the code to address one remaining test issue.
I ran the tests locally, but one test case, stepWithCompactEncoding - invalid compact unwind encoding, failed and stopped the test suite from completing. I believe this failure is unrelated to my changes. Please let me know if you think otherwise.
Thanks.

Codecov Report
Attention: Patch coverage is 87.15953% with 33 lines in your changes are missing coverage. Please review.

Project coverage is 80.01%. Comparing base (8c1ee00) to head (b7503b8).

Files
Patch %
Lines

...t/lance-encoding/src/encodings/physical/buffers.rs
73.68%
6 Missing and 4 partials :warning:

rust/lance-encoding/src/encoder.rs
0.00%
6 Missing :warning:

...ust/lance-encoding/src/encodings/physical/value.rs
96.52%
1 Missing and 3 partials :warning:

rust/lance-file/src/v2/reader.rs
86.66%
2 Missing and 2 partials :warning:

rust/lance-encoding/src/encodings/physical.rs
85.00%
3 Missing :warning:

...ust/lance-encoding/src/encodings/physical/basic.rs
72.72%
0 Missing and 3 partials :warning:

.../lance-encoding/src/encodings/logical/primitive.rs
71.42%
0 Missing and 2 partials :warning:

...encoding/src/encodings/physical/fixed_size_list.rs
87.50%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2368      +/-   ##
==========================================
+ Coverage   79.99%   80.01%   +0.02%
==========================================
  Files         200      200
  Lines       54519    54700     +181
  Branches    54519    54700     +181
==========================================
+ Hits        43612    43770     +158
- Misses       8389     8401      +12
- Partials     2518     2529      +11

Flag
Coverage Δ

unittests
80.01% <87.15%> (+0.02%)
:arrow_up:

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

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


dotUltimate 2020.3.4

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

already covered in #8904.


Unused identifier warning

unused identifier b (delete or rename to wildcard _ or _b)
For some reason this code is valid and will return 'b' while it shouldn't ?
It will also show a warning it's unused.

That's actually not a bug. The value of a declaration used as the last expression of a block expression is the value of its rhs, unless the block has type ().
The variable b is not referenced, hence the warning.

For example, this function returns (3,4), even though it bind nothing in the let.

func f() : (Nat,Nat) {let _ = (3,4)};
let f : () -> (Nat, Nat) =
f();
(3, 4) : (Nat, Nat)


add clippy support

Hi,
I tried to add Clippy support.
I don't know what lint can display, I just used other lints as examples to make this one.

[x] Clone the repository and install its dependencies with yarn install.
[x] Create a new class for the linter, e.g. src/linters/my-linter.js. Have a look at the other files in that directory to see what functions the class needs to implement.
[x] Import your class in the src/linters/index.js file.
[x] Provide a sample project for the linter under test/linters/projects/my-linter/. It should be simple and contain a few linting errors which your tests will detect.
[x] Provide the expected linting output for your sample project in a test/linters/params/my-linter.js file. Import this file in test/linters/linters.test.js. You can run the tests with yarn test.
[x] Update the action.yml file with the options provided by the new linter.
[ ] Mention your linter in the README.md file.
[ ] Update the test workflow file.

Not sure how to fix Windows tests, ${dir} translate as a path like this one :
D:\a\lint-action\lint-action\test\tmp\clippy/src/main.rs

And clippy output the path like this :
D:\\a\\lint-action\\lint-action\\test\\tmp\\clippy\\src\\main.rs

I will try to fix the windows behavior in the next days

Can be closed as #586 got merged and fixed what I left unfixed, thanks @nick-mobilecoin !


Error signing bitcoin transaction with P2SH: Missing redeem script

Describe the bug
I followed the example for bitcoin transaction signing from https://developer.trustwallet.com/wallet-core/integration-guide/wallet-core-usage.
Here is my Swift code:
Getting key
let hd_wallet = try HDWallet(mnemonic: mnemonic, passphrase: "")
//get private key
let path = DerivationPath(purpose: .bip49, coinType: bitcoin, account: 0, change: 0, address: 0)
let privateKey = hd_wallet.getKey(derivationPath: path.description)

Utxo
let utxo = BitcoinUnspentTransaction.with {
        $0.outPoint.hash = Data(utxoTxId.reversed()) // reverse of UTXO tx id, Bitcoin internal expects network byte order
        $0.outPoint.index = txIdx         // outpoint index of this this UTXO
        $0.outPoint.sequence = UINT32_MAX
        $0.amount = value // value of this UTXO
        $0.script = Data(hexString: scriptHex)!
                                    }
utxos.append(utxo)

Signing Input
let input = BitcoinSigningInput.with {
                                    $0.hashType = BitcoinSigHashType.all.rawValue
                                    $0.amount = satoshi
                                    $0.byteFee = byteFee
                                    $0.toAddress = targetAddress!
                                    $0.changeAddress = changeAddress
                                    $0.utxo = utxos
                                    $0.privateKey = [privateKey.data]
                                }

I can successfully sign a transaction of an address with  BECH32 (P2WPKH) format., but when signing a transaction with BASE58 (P2SH) I get the error message: "Missing Redeem script"
To Reproduce
Sign bitcoin transaction with utxos of a P2SH address like
[{"_id":"5e0e8bfd02435b4daff5b4b4","chain":"BTC","network":"mainnet","coinbase":false,"mintIndex":0,"spentTxid":"","mintTxid":"43d83f9008a85f4c9f6413f566c9cbfcd2f42af73395b1f482debe3e9c2617d7","mintHeight":611011,"spentHeight":-2,"address":"3L4SRknyDsWA7qmxk5gqp7RFRFK4CDwgZp","script":"a914c97f7434a63e8cbdf268f5acad847089302d2b8387","value":14000,"confirmations":-1}]

Utxo input that can successfully be signed
[{"_id":"5e142e5002435b4daf82afc6","chain":"BTC","network":"mainnet","coinbase":false,"mintIndex":1,"spentTxid":"","mintTxid":"c4bdd44b9a541c66b1b7211a0e3cd60d246e6739c9a16e6f2e7d22e81a467367","mintHeight":611694,"spentHeight":-2,"address":"bc1qj7w9lzlgaz79nlq7y585qvrwp75aa73ajqxlzw","script":"0014979c5f8be8e8bc59fc1e250f40306e0fa9defa3d","value":1870,"confirmations":-1}]

Additional context
I am using the 'TrustWalletCore' pod on ios 11.0

What is scriptHex in your example?  In the example guide following script is used:
$0.script = BitcoinScript.buildForAddress(address: address, coin: .bitcoinCash).data // Build lock script from address or public key hash

scriptHex is "a914c97f7434a63e8cbdf268f5acad847089302d2b8387"
I did a check and if I do

with senderAddress = "3L4SRknyDsWA7qmxk5gqp7RFRFK4CDwgZp" it returns the same hex string.
so both
$.script = BitcoinScript.buildForAddress(address: senderAddress, coin: .bitcoin).data

and
$0.script = Data(hexString: scriptHex)!

return the same error "Missing redeem script"

Are you using p2sh-p2wpkh? if so, you have to build script by calling buildPayToWitnessPubkeyHash

I followed your instructions and was able to successfully sign a transaction by adding this code:
//get public key hash
   let ypub = hd_wallet.getExtendedPublicKey(purpose: .bip49, coin: .bitcoin, version: .ypub)
   let ypubAddress = HDWallet.derive(from: ypub, at: path)!
   let ypubHash = ypubAddress.bitcoinKeyHash

   let p2sh = BitcoinScript.buildPayToWitnessPubkeyHash(hash: ypubHash)
   $0.script = p2sh.data

But when broadcasting I get the error: "Code: -26, Error: bad-txns-nonstandard-inputs (code 64)"
What am I missing?
signed transaction output:
0100000000010188a0e7c7c6b385f02f52d0b392b9fda3a5a5e76de047c769b75ca790e84f9e200000000000ffffffff02881300000000000016001472ab1430f60f6cc2826b3908003d9eca7e442755c41100000000000017a914ba5c8d91d100b7ff0b2741708fef1f426e8be70c8702473044022026db6e22a1e4e670ec44797160345378bb5d5c988d722057576d971b7463fe5c02207fd3d9de878a4759186e4a57e102348ddb985907253ec5d5c4c37c17e2d694360121026ec9ce0cdbc5b051e3637830c1f12e6048cca4845790f1fe0a28b979e6b10eee00000000

decoded:
{
	"version": 1,
	"locktime": 0,
	"ins": [
		{
			"n": 0,
			"script": {
				"asm": "",
				"hex": ""
			},
			"sequence":<PHONE_NUMBER>,
			"txid": "209e4fe890a75cb769c747e06de7a5a5a3fdb992b3d0522ff085b3c6c7e7a088",
			"witness": [
				"3044022026db6e22a1e4e670ec44797160345378bb5d5c988d722057576d971b7463fe5c02207fd3d9de878a4759186e4a57e102348ddb985907253ec5d5c4c37c17e2d6943601",
				"026ec9ce0cdbc5b051e3637830c1f12e6048cca4845790f1fe0a28b979e6b10eee"
			]
		}
	],
	"outs": [
		{
			"script": {
				"addresses": [
					"bc1qw243gv8kpakv9qnt8yyqq0v7eflygf647c68ec"
				],
				"asm": "OP_0 72ab1430f60f6cc2826b3908003d9eca7e442755",
				"hex": "001472ab1430f60f6cc2826b3908003d9eca7e442755"
			},
			"value": 5000
		},
		{
			"script": {
				"addresses": [
					"3JgQUvJY51skxXaY1i6ZDAvMQVBYBPw8ub"
				],
				"asm": "OP_HASH160 ba5c8d91d100b7ff0b2741708fef1f426e8be70c OP_EQUAL",
				"hex": "a914ba5c8d91d100b7ff0b2741708fef1f426e8be70c87"
			},
			"value": 4548
		}
	],
	"hash": "83542b3d6b972632f01cc96808702f6153d1ed20022adc7fbcce266ff0c1f18f",
	"txid": "83542b3d6b972632f01cc96808702f6153d1ed20022adc7fbcce266ff0c1f18f"
}

There is a scripts property in BitcoinSigningInput, so you build the redeem script with buildPayToWitnessPubkeyHash
scripts[<lock script>(your original scriptHex)] = <redeem script>

Thanks for your help. I managed to sign and broadcast a transaction with this additional code:
let redeemScript = BitcoinScript.buildPayToWitnessPubkeyHash(hash: ypubHash).data
let redeemScriptHashString = Hash.sha256RIPEMD(data: redeemScript).hexString
$0.scripts[redeemScriptHashString] = redeemScript

now we added a P2SH-P2WPKH test here: https://github.com/trustwallet/wallet-core/blob/9c5927e5ac45fb57339f4d91efbc71154aaa2c70/swift/Tests/Blockchains/BItcoinTests.swift#L84

let scriptHash = lockScript.matchPayToScriptHash()!
This code getting crash can you check ?


Correct wording of the WSL recommendation.

The WSL recommendation says Windows Linux Subsystembut should say “Windows Subsystem for Linux” (WSL)

I believe this was merged in, thus closing


java.lang.ClassCastException in ExpandColumnNamesWildcardQuickFix.kt:22

Error in SQL Delight IntelliJ Plugin
java.lang.ClassCastException in ExpandColumnNamesWildcardQuickFix.kt:22
org.jetbrains.plugins.groovy.lang.psi.impl.GroovyFileImpl cannot be cast to com.squareup.sqldelight.core.lang.SqlDelightFile
View on Bugsnag
Stacktrace
ExpandColumnNamesWildcardQuickFix.kt:22 - com.squareup.sqldelight.intellij.intentions.ExpandColumnNamesWildcardQuickFix.isAvailable

View full stacktrace
Created automatically via Bugsnag

@Egorand
looks like the quickfix is somehow appearing in groovy files?

I think this is coming from me. It happens when I open a file and that click somewhere in that file. The groovy file was a build.gradle file. It also happens to regular Kotlin and Java files.

got it. We probably just need to ignore non-sq files.


[Android] add event key validation to content

You are right, I also thought that but I still implemented to be extra sure. However, I also agree with you about that situation


feat: add option to interrupt test runner

At this moment there's no way to programmatically stop a test run in process with replay, which simply loops until a failure or all configured cases pass
https://github.com/proptest-rs/proptest/blob/24412f5f10a365c8f12a71e3b3a053677ced91b0/proptest/src/test_runner/runner.rs#L609
It would be nice to be able to interrupt this loop at will so in a setup with multiple tests started at the same time (not using fork feature but rather parallelized using rayon for example) following scenarios could be achieved:

fail fast: when one of these test fails then the other tests are interrupted
timeout (not using rusty fork / fork timeout feature): interrupt all running tests when timer (started outside tests) expires

made a PR with proposed implementation https://github.com/proptest-rs/proptest/pull/461 @matthew-russo please have a look when time permits. thank you!

@matthew-russo any thoughts re this issue and associated PR really appreciated, thank you!

ping @matthew-russo any insight re this issue and PR https://github.com/proptest-rs/proptest/pull/461. thank you!


Fetch EULA from the web

Description:
This PR fetches EULA from a mock web server. Locales are also supported.
This SHOULD NOT be merged before we have a real webserver to fetch EULA from.
Screenshots:

@zhaccc There was a change made to the build checks so you'll have to merge the latest from develop and push to this branch so the build_debug_android passes.  Sorry for the hassle!


invalid DSSE envelope payload: invalid character 'i' looking for beginning of value

Trying to verify provenance metadata generated by GCB via Github trigger (but I also encounter the same error if I upload the build package to GCB myself).
$ IMAGE='us-west2-docker.pkg.dev/imre-test/quickstart-docker-repo/quickstart-image:v41@sha256:41cb4b5e32e417b86c2b2229d0581b72f7dffd1cc6b0e586ab2cefdb7a527529'
$ ./slsa-verifier-linux-amd64 verify-image "$IMAGE"   --provenance-path provenance-github.yaml --builder-id=https://cloudbuild.googleapis.com/GoogleHostedWorker --source-uri github.com/irsl/gcb-tests
invalid DSSE envelope payload: invalid character 'i' looking for beginning of value

slsa-verifier is v2.0.1.
The provenance file: https://gist.github.com/irsl/2eaca58f22e0a2d5e9a675f9cc4aef70
The issue seems similar to https://github.com/slsa-framework/slsa-verifier/issues/267
Did I make a mistake somewhere or has the structure changed? Is there a workaround?

Hi @irsl! Thanks for the report.
Currently I think we only support the provenance in json format so that's the error you're getting I think. You can use the gcloud command shown in the README.md to download the provenance in JSON format.

Ah, works like a charm :) Thanks for the hint!


Custom provider use is misleading in the docs

https://github.com/jxnblk/mdx-deck/blob/master/docs/advanced.md#custom-provider-component
export default {
  font: 'Georgia, serif',
  Provider,
}

It is not a valid JSON, and it is somewhat hard to understand what theming engine is looking for.
It looks like export const themes=[{Provider: slideNumbers}] works for mdx-deck 2 (slideNumbers is a function similar to https://github.com/jxnblk/mdx-deck/issues/180#issuecomment-460546242 )

The Provider is a React component, which is stated in the text accompanying the example. Do you have any suggestions on how to make that more clear?

Well I tried again, and it works now. No idea what is was. Sorry for the noise.


linter: linter should go through source code and populate constraint set according to the theory

TODO: copy-paste theory

Yeah.... so it's actually a bit more complex hehe

The linter will maintain a distribution which is a set of constraint_sets, so the API will need to go across aaall constraint_sets and do heavy computations for each of them.
But the good news are, that new constraint sets are generated only when we load some values from some addresses, which means until we meet this operation, we will deal with only one constraint set.
So, for this task, let's assume we deal with only one constraint set.

For this task, let the linter correctly handle empty and double_dispose examples.

Seems like, if it handles these two correctly, it will handle other dispose examples too...


Extending modules with diagnostic settings

Currently, there are no modules that extend with diagnostic settings. For example, Azure SQL with diagnostic settings to send events to Event Hub.
Understandably, these modules should be focused to a particular Azure resource but where should modules demonstrating diagnostic settings be?

For the most part, we enable Log Analytics on every service that has logging features enabled. We also send raw log information to a Diagnostics Storage Account. This Diagnostics Storage Account is locked down using Service Endpoint, the problem we faced in Azure SQL specifically is that it does not allow to send raw logs to a Storage Account that has Service Endpoint enabled. We are sending this feedback to the PG. For now, I label this request as an enhancement, we'll add Log Analytics to Azure SQL.
If you have an example about sending these logs to an Event Hub, please feel free to create a PR.
Thanks!


File renaming

@Nicholas-Schaub Old Pull request #210
Reopened the PR for file renaming plugin as the original branch got deleted and this plugin wasn't merged to the master branch

plugin.json is not updated to include all the I/O arguments

@Nicholas-Schaub This PR should be closed. Updated one is opened
https://github.com/PolusAI/polus-plugins/pull/432


⚠ Bofrost.at has degraded performance

In b3dff5b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1860 ms

Resolved: Bofrost.at performance has improved in 63066da after 6 minutes.


When I tried to reconnect a registered phone number using the registered API, I found that the store displayed a status indicating that it was already connected, but in reality, it was not connected and could not send messages

can you show your code?

Whatsapp.mobileBuilder()
.newConnection(UUID.fromString(myWhatsapp.getSessionId()))
.registered()
.get()
.connectAndAwait()
.join();
}

might want to try 3.4.5

OK

I tried reconnecting, but it still doesn't work. I'm constantly loading. v3.4.8

Whatsapp.mobileBuilder() .newConnection(UUID.fromString(myWhatsapp.getSessionId())) .registered() .get() .connectAndAwait() .join(); }

@topskymonk Hello, may I know how the "myWhatsapp.getSessionId()" smile data is persisted in your local environment? Could you provide some complete examples of your use of mobileBuilder? I'm extremely grateful for your assistance, thank you!


Benchmarking v1.0.0 against v0.9

Out of curiosity I ran a dead-simple benchmark which loads a pre-fetched list of dataflows from estat:
flows = Request('estat').dataflow(tofile='estat_flows.xml')

The file sizte is about 4.1MB.
Benchmark results:
In [34]: %timeit data_resp=sdmx.read_sdmx('estat_flows.xml')
4.13 s ± 21.6 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [35]: exit

(sdmx37) C:\Users\a3305\documents\github>
(sdmx37) C:\Users\a3305\documents\github>deactivate
DeprecationWarning: 'deactivate' is deprecated. Use 'conda deactivate'.

(sdmx37) C:\Users\a3305\documents\github>conda.bat deactivate

(base) C:\Users\a3305\documents\github>
(base) C:\Users\a3305\documents\github>activate pandasdmx09

(pandasdmx09) C:\Users\a3305\documents\github>ipython
Python 3.8.1 (default, Jan  8 2020, 15:55:49) [MSC v.1916 64 bit (AMD64)]
Type 'copyright', 'credits' or 'license' for more information
IPython 7.11.1 -- An enhanced Interactive Python. Type '?' for help.

# v0.9:
In [1]: import pandasdmx as sdmx

In [4]: estat=sdmx.Request('estat')

In [5]: flows=estat.get(fromfile='estat_flows.xml')

In [6]: %timeit flows=estat.get(fromfile='estat_flows.xml')
122 ms ± 902 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

In [7]:

So v0.9 is about 34 times faster than v1.0dev when parsing a 4MB structure message.
Sure, v1.0 has quite a few advantages. Still, the performance gap is significant.
I wonder if v1.0 would be faster when skipping pydantic validation by default using .construct().
Other proposals to make v1.0 faster are welcome.

Excellent, thanks for taking this step into benchmarking the code.

I wonder if v1.0 would be faster when skipping pydantic validation by default using .construct().

That's an interesting suggestion. There could be a flag like API_MODE or something like that:

For ordinary use, leave it off as suggested.
For development purposes, e.g. running the test suite, turn it on.

This helps (a lot!) to identify coding errors in the message readers, the main purpose for which it was used in the first place.
Could also show users how to turn it on, in order to debug when they are encountering errors.

Two things that would need investigation (thus time):

How extensive the code changes would be to enable this switch. Objects are instantiated in many places.

Whether the reader code relies on pydantic's type checking. For example if there's code like:
try:
    obj.attr = value
except Exception:
    # pydantic itself has raised an Exception because *value* was the wrong type
    obj.attr = some_other_value

The code should not rely on this, and I do avoid writing code this way out of habit. But it still requires a thorough check.

I'll gladly do these for a 1.1 follow-up release, but don't expect I can find time to do them before 1.0.

Can be discussed in #144


[Nim] Use ref to fill related

@jinyus
If I can ask to bench it before merge. Locally it shows about -1ms
Thank you

Not merging since it makes the code less concise with no perf benefits.

@inv2004 in case it interests you and you'd like to experiment with it:
I tried to get creative and made some changes in my fork to the single-threaded nim so that posts seq is mutated directly instead of having another seq[RelatedPosts]:
michaelsbradleyjr:related_post_gen:main...michaelsbradleyjr:related_post_gen:nim_rev_10
It's just a bit faster for the 5k runs, but then slower than current main branch times for 20k and 60k runs.
I tried quite a few variations for that approach, but that's the fastest I could make it. I experimented with var mutation vs. return for findTop5, seq vs. array for related:, etc., etc. Note: I've turned on mm:arc aware that it doesn't deal with cycles and my nim_rev_10 does involve cycles, but in this case it doesn't leak because of the way posts is self-contained.
clang, plus the inlining hints, plus mm:arc, minus xxhash gives the best times. But maybe you can think of a way to tweak it further.
If you want, you can hit me up in the Nim discord and I can get you access to my free-tier Azure VM that has the same setup as the official benchmarking VM. I'll run out of free-tier credits in less than a month and plan to delete it at that time, so might as well get the most out of it.


PDFOcr Used by default, creates unwanted behaviour

Hey!
The pdf's I want to scan, contains characters in Hebrew, which are not supported by PDFOcr's backbone (Returns characters as question marks)
Therefore, I wanted to use GCP's VisionOCR that handles hebrew perfectly.
The following code will still use the default PDFOcr and ignore the VisionOCR instance I passed.
from img2table.document import PDF
from img2table.ocr import VisionOCR

ocr = VisionOCR(timeout=25)

doc = PDF("test.pdf", [0])
tables = doc.extract_tables(ocr=ocr, min_confidence=70)

Suggested Fix
Add to PDF.extract_tables  argument force_ocr=False
Then, add a check before using the default PdfOCR
Then, in Document.extract_tables add a check for force_ocr and ocr is None and throw exception if needed.
Let me know if this is acceptable, I can create the PR.
Thanks

Hello,
Yep, it seems that PyMuPDF (the PDF library used) has some issues with non latin characters. I also had some feedback about some Indian dialects not being processed properly.
I will implement an option to bypass text extraction from the OCR and will include it in the next release.

I have added a pdf_text_extraction parameter to the PDF class that allows you to bypass text extraction from the PDF file.
It is available in the latest release that I just published


Another empty string returned (Regular Expression has PREG_BACKTRACK_LIMIT_ERROR)

What version of Minify are you using?
2.0.1

PHP version?
5.2.5

What steps will reproduce the problem?
1. pcre.backtrack_limit set in the php.ini to the default (100,000 I think).
2. A single line in the HTML needs to be too long (148426 characters in my
case).
3. Minify the HTML.

What is the expected output?
Some HTML.

What do you see instead?
Empty string.

Please provide any additional information below.
I think this is very similar to Issue 41 (too many backtracks). The Regular
Expression in this case is:
$html = preg_replace('/>(\\S[\\s\\S]*?)?\\s+</', ">$1 <", $html);

Although my original html did not have any long lines, the long line was
created by minify.

This time though I think I have 2 solutions, the first is to fix the
expression. Here is my new expression:
$html = preg_replace('/>(\\S[\\s\\S]*?)?\\s+</U', ">$1 <", $html);
Note that the only change is the /U modifier. I tested this new expression
and it looks like it works to me with a 128.9 kB file.

My second solution is as fall-back for failing regular expressions. As
preg_replace returns an empty string ("") when it fails, this should be
caught. Each regular expression should be checked like the following:

$newhtml = preg_replace('/>(\\S[\\s\\S]*?)?\\s+</U', ">$1 <", $html);
if ($newhtml != ""){
    $html = $newhtml;
}

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 12 Aug 2008 at 2:13

I think in general the minifiers need to limit lines lengths if it means using
newlines instead of spaces in some cases.

I'll try out that code. Something like this might be fast (incredibly simple
expression + native trim()):

$html = preg_replace_callback('/>([^<]+)</', array('Minify_HTML', '_cbTrim'),
$html);

function _cbTrim($m) {
  return '>' . trim($m[1]) . ' <';
}

Original comment by<EMAIL_ADDRESS>on 13 Aug 2008 at 2:22

Changed state: Accepted

I've got same bug with long html
During tests I've found that count of mathes is 0 and it do not enter callback
function.

No errors, no notices, no info to help resolve this bug =(

Original comment by<EMAIL_ADDRESS>on 15 Jan 2010 at 8:32

Can you try Minify_HTML in R168?

Rather than running 3 patterns on the whole string, there's now one trip across
the
string with two simple patterns.
http://code.google.com/p/minify/source/diff?old=163&r=168&format=unidiff&path=
%2Ftrunk%2Flib%2FMinify%2FHTML.php

I'll consider adding the fallback. But then would these bugs get noticed? ;)

Original comment by<EMAIL_ADDRESS>on 13 Aug 2008 at 2:15

Changed state: Fixed

@swaylex: can you tell which is the first preg_replace_callback to fail? Does
your
input have really long lines, or just really long? Ideally attach a failing
HTML file.

Original comment by<EMAIL_ADDRESS>on 15 Jan 2010 at 9:56

Changed state: NeedInfo


Comparison with other modules?

How would this work in comparison with Greensock or Animejs? I'm considering this one and Animejs (which is slightly bigger) for use in various projects.

I'd say the three of them are solid choices, so it really depends on what you prioritize. GreenSock is the most powerful one but also the heaviest. Animate Plus is the most straightforward and the lightest. Anime is somewhere in the middle.
Happy animating!


Cleanup & streamlining pass of Backtrace Source Map doc

Aligned structure to the sequence of steps for integration
Directed the users to API Tokens rather than the other soon-to-be-deprecated token pages
Streamlined text for acquiring token & url
Removed documentation for the .backtraceclirc subdomin and token parameters - we only need to offer one path
Fixed the tabs

@LoryApiFortress  @rick-bt I fixed the conflicts. Should I land this?

Thank you.
My vote: Yes.

@LoryApiFortress @rick-bt I fixed the conflicts. Should I land this?

yes!


What are the most basic requirements for running ISLE?

The ISLE-DC documentation written in #1635 is meant to be a way for non-technical users to install a simple and most importantly, update-able, sandbox on their local machines.
But I was only able to document MY experience, and I already had many techy things installed on my Mac: Perl, Mac Development Tools (which require and Apple account), Composer, Ansible, Git, PHP, VirtualBox... I was also running as a user with sudo powers and who knows what else.
We need someone with a stock laptop to walk through the installation process - mac or windows - and document the actual requirements!

Thanks @dflitner - Windows doesn't have make available by default.

On windows everything has to be run using WSL2, so we definitely need to highlight that as an essential requirement.
Been a while, so YMMV, but I went through it when my laptop died and I needed git, php, composer, and a handful of php packages.  php-mysql and php-mbstring are the two i can recall specifically.

@dannylamb Is that because ISLE will not work with Docker for Windows using Hyper-V? I haven't tested that part yet, and I understand Docker Desktop uses WSL2 behind the scenes.
I can in any case document a walkthrough for non-technical users using a relatively mint Windows with WSL2 and Ubuntu or Debian (both worked).

It'd be wicked helpful if there was a link or something to how to install ubuntu on a windows machine.
(Hi! I have a newish desktop and this is the first techie thing I've tried installing locally. I'll post what pain points I encounter as I encounter them.)


Split crates

I consider we could move kvdb in a separate crate. What about removing the avl or moving to another repo firstly, because it is written base on the trie, it's not well designed.

For kvdb, should we move hashdb.rs too?
If we don't plan to revisit current avl code in the future then it's ok to remove it.

I just check parity-common, i think our common contains too many crates. Their common only contains crates that are related storage. We need a big plan? Move out hash crates, pubsub crates, jsonrpc crates,
protobuf crates?5

Totally agree with you.

About merkel:

Add a TreeBuilder in cita-merklehash
Rename merklehash to static-merkle
fix the test, merkelhash should build without any features.
remove null

https://github.com/cryptape/cita-common/pull/182

Do we consider to publish crate?

That depends, we have published some before, like rust-numext and static-merkle-tree.

What is the progress ?

Have finished partially, but I think we can do better, it's a long term improvement.

All the independent crate in this repo except cita-web3 and logger.


[Event Request] Codeunit 5496 Graph Mgt - Sales Order Buffer

The following events are required for this object:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeCheckValidRecord(var SalesHeader: Record "Sales Header"; var Result: Boolean; var IsHandled: Boolean)
    begin
    end;

    local procedure CheckValidRecord(var SalesHeader: Record "Sales Header"): Boolean
    var
        IsHandled: Boolean;
        Result: Boolean;
    begin
        //EVENT REQUEST
        IsHandled := false;
        OnBeforeCheckValidRecord(SalesHeader, Result, IsHandled);
        if IsHandled then
            exit(Result);
        //EVENT REQUEST

        if SalesHeader.IsTemporary then
            exit(false);

        if SalesHeader."Document Type" <> SalesHeader."Document Type"::Order then
            exit(false);

        exit(true);
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 12439.


Hide columns header row

I have table without header of columns. How to hide empty header div in my table?

This looks to be a question of implemention. Please repost this question on our support community: