No module named 'regions.core'; 'regions' is not a package

Hi, I got the following message when I tried to import regions like that:
from regions.core import PixCoord
from regions.shapes.circle import CirclePixelRegion
center = PixCoord(x, y)
reg = CirclePixelRegion(center, 1.0)
Traceback (most recent call last):   File "/"...".py", line 11, in <module> from regions.core import PixCoord ImportError: No module named 'regions.core'; 'regions' is not a package
and
    from regions.shapes.circle import CirclePixelRegion                   ImportError: No module named 'regions.shapes'; 'regions' is not a package
I ran the test:
running test
installing to temporary directory: /tmp/regions-test-avppgsct
warning: no files found matching 'README.rst'
warning: no files found matching 'ez_setup.py'
warning: no files found matching '' under directory 'licenses'
warning: no files found matching '' under directory 'cextern'
warning: no files found matching '' under directory 'scripts'
no previously-included directories found matching 'build'
no previously-included directories found matching 'docs/_build'
no previously-included directories found matching 'docs/api'
warning: no previously-included files matching '.o' found anywhere in distribution
=================== test session starts =======================
platform linux -- Python 3.5.2, pytest-4.1.1, py-1.7.0, pluggy-0.8.1
Running tests with regions version 0.4.dev882.
Running tests in regions docs.
Full Python Version:
3.5.2 (default, Nov 12 2018, 13:43:14)
[GCC 5.4.0 20160609]
encodings: sys: utf-8, locale: UTF-8, filesystem: utf-8
byteorder: little
float info: dig: 15, mant_dig: 15
Numpy: 1.15.4
Scipy: 1.2.0
Matplotlib: 3.0.2
Cython: 0.29.6
Astropy: 3.1.1
Astropy-healpix: 0.4
astropy_helpers: 2.0.9
Using Astropy options: remote_data: none.
rootdir: /tmp/regions-test-avppgsct/lib/python3.5/site-packages, inifile: setup.cfg
plugins: remotedata-0.3.1, openfiles-0.3.2, doctestplus-0.2.0, arraydiff-0.3
collected 945 items
regions/_geometry/tests/test_circular_overlap_grid.py ......................................................                                                                                [  5%]
regions/_geometry/tests/test_elliptical_overlap_grid.py ............................................................................................................................................ [ 20%]
.................................................................................................................................................................................................... [ 41%]
......................................................................................................................................................                                               [ 57%]
regions/_geometry/tests/test_rectangular_overlap_grid.py .................................................................................                                                           [ 65%]
regions/_utils/tests/test_examples.py .....                                                                                [ 66%]
regions/core/tests/test_bounding_box.py ..........                                                                                [ 67%]
regions/core/tests/test_compound.py ..                                                                                [ 67%]
regions/core/tests/test_mask.py ............                                                                                [ 68%]
regions/core/tests/test_pixcoord.py ...........                                                                                [ 69%]
regions/io/crtf/tests/test_crtf_language.py ........                                                                                [ 70%]
regions/io/ds9/tests/test_ds9_language.py ........................                                                                                [ 73%]
regions/io/fits/tests/test_fits_region.py ....                                                                                [ 73%]
regions/io/tests/test_shape_object.py .....                                                                                [ 74%]
regions/shapes/tests/test_annulus.py .....s.............s.............s........                                                                                [ 78%]
regions/shapes/tests/test_api.py ............s...............xx....xx.x..x..xxxxx........................................                                                                            [ 88%]
regions/shapes/tests/test_circle.py ..............                                                                                [ 89%]
regions/shapes/tests/test_ellipse.py ..............                                                                                [ 91%]
regions/shapes/tests/test_line.py ...s.........                                                                                [ 92%]
regions/shapes/tests/test_masks.py ...........x....x...                                                                                [ 94%]
regions/shapes/tests/test_point.py ...s........                                                                                [ 95%]
regions/shapes/tests/test_polygon.py ..s............                                                                                [ 97%]
regions/shapes/tests/test_rectangle.py ..............                                                                                [ 98%]
regions/shapes/tests/test_text.py ...s.......                                                                                [100%]
========================== 924 passed, 8 skipped, 13 xfailed in 59.11 seconds ==============
======= warnings summary ===============================
../lib/python3.5/site-packages/regions/io/ds9/tests/test_ds9_language.py::test_read
../lib/python3.5/site-packages/regions/io/ds9/read.py:283: DS9RegionParserWarning: Region type 'panda' was identified, but it is not one of the known region types.
warn(msg, DS9RegionParserWarning)
../lib/python3.5/site-packages/regions/io/ds9/read.py:283: DS9RegionParserWarning: Region type 'panda' was identified, but it is not one of the known region types.
warn(msg, DS9RegionParserWarning)
..v/lib/python3.5/site-packages/regions/io/ds9/read.py:283: DS9RegionParserWarning: Region type 'panda' was identified, but it is not one of the known region types.
warn(msg, DS9RegionParserWarning)
..v/lib/python3.5/site-packages/regions/io/ds9/read.py:283: DS9RegionParserWarning: Region type 'panda' was identified, but it is not one of the known region types.
warn(msg, DS9RegionParserWarning)
../lib/python3.5/site-packages/regions/io/ds9/read.py:283: DS9RegionParserWarning: Region type 'panda' was identified, but it is not one of the known region types.
warn(msg, DS9RegionParserWarning)
../lib/python3.5/site-packages/regions/io/core.py:92: DS9RegionParserWarning: Skipping elliptical annulus Shape
Type : reg
Coord sys : fk5
Region type : ellipse
Meta: OrderedDict([('color', 'red'), ('dashlist', '8 3'), ('width', '3'), ('font', 'helvetica'), ('select', '1'), ('highlite', '1'), ('dash', '0'), ('fixed', '0'), ('edit', '1'), ('move', '1'), ('delete', '1'), ('include', True), ('source', '1'), ('text', 'Ellipse Annulus'), ('fontsize', '10'), ('fontstyle', 'bold'), ('fontweight', 'roman')])
Composite: False
Include: True
warn(msg, DS9RegionParserWarning)

...............etc........................
What is that about?? How can I fix this?
Regions is already installed in Pycharm
Thanks in advance

I don't know what to make of this; I can't reproduce the error and no one else has reported it.
How are you running the tests?  It looks like they were successful

@rasputin1917 - You have an error "regions is not a package", i.e. you have an installation issue.
You should just pip install or conda install the regions package as mentioned here:
https://astropy-regions.readthedocs.io/en/latest/installation.html
Then importing regions should work:
$ python
...
>>> import regions

Did you git clone or download the development version of regions?
That's only advisable when you want to hack on this package, not when you want to use it.

I'll close this issue for now.
But if you still have questions, please comment here again or go on the Astropy Slack ( https://www.astropy.org/help.html ) where you get a reply more quickly concerning installation issues.


Start float doesn't work and balloon does not appear since 0.9.11

I have a block like this:
- {{video https://youtu.be/ZRtdQ81jPUQ}}

The video doesn't have a balloon like it used to, and If I try to use the /Start Float slash command I get the following error in the development view:
caught (in promise) TypeError: Cannot read properties of null (reading 'id')
    at index.fca72f42.js:1:94513

Uninstalling and reinstalling for a third time worked??!


update wayland 1.18.0 -> 1.19.0

Checklist

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name: wayland

current version: 1.18.0
desired version: 1.19.0
Notify maintainers
maintainers:
@primeos
@codyopel
Note for maintainers
Please tag this issue in your PR.

I tried it myself, but it seems that something with the applied patches goes wront?
building '/nix/store/g8wrdrlqfnpm36wx7dqawk73aax553iz-wayland-1.19.0.drv'...

trying https://github.com/swaywm/wlroots/archive/5e19e0053a5800252a27ce127fd015d641ee2e9e.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
unpacking sources
unpacking source archive /nix/store/f2xgdv25jkv2rd2i00z2c1iza3n0jac2-wayland-1.19.0.tar.xz
source root is wayland-1.19.0
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file wayland-1.19.0/src/wayland-version.h
patching sources
applying patch /nix/store/4m12lw220plcb5wnk86j6pjs9qp6rnlc-e53e0edf0f892670f3e8c5dd527b3bb22335d32d.patch
patching file doc/publican/meson.build
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file doc/publican/meson.build.rej
builder for '/nix/store/g8wrdrlqfnpm36wx7dqawk73aax553iz-wayland-1.19.0.drv' failed with exit code 1
cannot build derivation '/nix/store/0p6ls1gpni8lnw8sz7w3wvaqpka7b690-wlroots-0.12.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/0p6ls1gpni8lnw8sz7w3wvaqpka7b690-wlroots-0.12.0.drv' failed

The update to wayland 1.19 is in staging-next right now: https://github.com/NixOS/nixpkgs/commit/106ca78aff3395c5074dd3714f9c3a569469c842

Oh - sorry. I loocked for the PR but I didn't find it.

Solved via  #107324


feat: implement new tile APIs

BREAKING CHANGE: HARP engine is now always used, whether vector tiles are enabled or not. Additionally, some props are now completely removed.

lg: use language instead with an Alpha-2 code (e.g. en or `de)
incidentsLayer: now always included with the traffic layer
useLegacyTruckLayer: no longer supported by HERE
useLegacyTrafficLayer: no longer supported by HERE

Note: this uses the logistics map styles as a base layer as it fits our needs the most, this however can be made configurable in the future.
What the PR Includes

[x] add toggles for different layer options in the testbench.
[x] implement new tile APIs.
[x] allow hotswapping vector and raster tiles.
[x] always use HARP engine.

Checklist

[x] Tests are added.
[x] Manual testing instructions are added.
[ ] Configs are added/adapted if applicable.
[x] Issue is properly linked if applicable.

Manual Testing Instructions

Needs to be tested extensively with the CargoApps. Use https://github.com/impargo/impargo-apps/pull/9572 as a starting point.
You can try out the different layers (vector and raster) using the testbench by running yarn dev.

/publish


Cnx 9502 agis object tracking change detection

Currently the change detection only works for the change of the map elements, their properties and names, but NOT the change in the data source (actual features /data rows in the table). This requires dynamic subscription to every new layer on the map, and it is tricky to extract the layer id from the feature-changed event.

extra step added on RunExpirationCheck to update ModelCard ids, otherwise it could have tried to send non-existing layers and pass silently through no-layers-to-send check.


Build images for MongoDB 3.0

The 3.0 packages require adduser, but don't automatically install it,
hence the slight tweaks in the Dockerfile.
We also need to adjust tests, because MongoDB 3.0 has log messages that
are different from both 2.6 and 3.2.

cc @fancyremarker

Awesome! Thanks!


Update common.json

Updated translation after website review

Your content is merged in the following commit!
https://github.com/signum-nft/signum-swap-translations/commit/391c82453d6805eee4e2bfdd5afe5aa018f81a7b


Localized links

Describe the bug
Links in a localized documentation require manual editing when translating.
To Reproduce Steps to reproduce the behavior:

Create a localized documentation with atleast two languages.
Create two pages for one language.
Link from one page to the other with md links
The language code won't get added dynamically, you have to manually do this.

Expected behavior A clear and concise description of what you expected to
happen.
I want that the language in a markdown link gets inferred by next, so that I don't have to keep this change in mind when translating.

please share a visual example, I am not sure I get it

Taking a guess here.
Previously in v2:

We had next i18n
When you wrote [Home](/home) inside mdx, it would automatically create a url to  localhost:3000/en/home if you were on the English version, and localhost:3000/de/home, if you were on the German version.

In v3

We don't have the same i18n capability, but workaround it with optional middleware.
If you don't have this middleware, you would have to create [Home](/en/home) for the English mdx page and [Home](/de/home) for the German page.
When ideally, you'd only want to use the relative path without having to include the locale.

I'm sorry for leaving this hanging.
In my context I export the next app statically to github pages, so the middleware is not an option for me.
I remembered this issue, as I was writing on the documentation, used a link and it worked as expected. I will have to test it again more in depth, but I don't have the time for that right now. I'll try to do it this weekend.


Repo

What article on docs.github.com is affected?

What part(s) of the article would you like to see updated?

Additional information

#2967

#2967

#2967

#2967

https://github.com/K2bok/K2bok-/commit/64ef578ef92432a71c1b53df1e87f19b179e18d0#commitcomment-46059454

https://github.com/K2bok/K2bok-/commit/64ef578ef92432a71c1b53df1e87f19b179e18d0#commitcomment-46059454

K2bok/K2bok-@64ef578#commitcomment-46059454

K2bok/K2bok-@64ef578#commitcomment-46059454


feat: OSX universal support

If a universal version is available and published, it will be returned instead of x64 or arm64, even if those are available.
Adds new option: preferUniversal (default true) to choose whether universal version is delivered for x64/arm64 request when universal version is present

ack. I'm going to roll this back... It was against main and not next

ack. I'm going to roll this back... It was against main and not next

Oop! Sorry. I just did the GitHub default when I made the PR.

No. I'm wrong. That's not what this is. Yeah. Should have been on Next. Ack!


Print on  split_train_val

on  split_train_val function we have unwanted print, I think it's better to not show it by default

Ah thanks! I was looking for where those prints were coming from since weeks (not very actively I admit...) ^^

The correction is on master, it will be on pip for the next version ;)
Thanks again :)


Initial connection creds stuff

Added the ability to add some credentials as websocket protocols.

This is not needed anymore right? This library works fine for me without this change

No longer relevant.


Feat update doctor extension

升级 @iceworks/doctor 更新为3个维度产出检测报告

Codecov Report

Merging #669 (3fe6355) into release/0.14.0 (f5a619b) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release/0.14.0     #669   +/-   ##
===============================================
  Coverage           57.14%   57.14%
===============================================
  Files                   1        1
  Lines                  70       70
  Branches               15       15
===============================================
  Hits                   40       40
  Misses                 30       30

Continue to review full report at Codecov.

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


OTEL BatchSpanProcessor should not spawn a thread during Puma/Unicorn boot

When using a multi-process webserver such as Puma or Unicorn, no threads should be started during the boot process. These threads won't survive to the child workers.
Here's my config:
# /config/initializers/opentelemetry.rb

  OpenTelemetry::SDK.configure do |c|
    exporter = OpenTelemetry::Exporter::OTLP::Exporter.new(insecure: true)
    processor = OpenTelemetry::SDK::Trace::Export::BatchSpanProcessor.new(exporter: exporter)
    c.add_span_processor(processor)
    c.use_all
  end

Puma complains WARNING: Detected 1 Thread(s) started in app boot when I use the OTEL ruby agent. See example log below from latest Puma:
$ WEB_CONCURRENCY=2 bundle exec puma

[19991] Puma starting in cluster mode...
[19991] * Version 5.0.2 (ruby 2.6.6-p146), codename: Spoony Bard
[19991] * Min threads: 5, max threads: 5
[19991] * Environment: development
[19991] * Process workers: 2
[19991] * Preloading application
I, [2020-11-01T00:14:33.022327 #19991]  INFO -- : Instrumentation: OpenTelemetry::Instrumentation::ConcurrentRuby was successfully installed
I, [2020-11-01T00:14:33.024353 #19991]  INFO -- : Instrumentation: OpenTelemetry::Instrumentation::Dalli was successfully installed
I, [2020-11-01T00:14:33.025486 #19991]  INFO -- : Instrumentation: OpenTelemetry::Instrumentation::Excon was successfully installed
[19991] * Listening on http://<IP_ADDRESS>:9292
[19991] ! WARNING: Detected 1 Thread(s) started in app boot:
[19991] ! #<Thread:0x00007ff7dfb90080@/home/shields/.rvm/gems/ruby-2.6.6/gems/opentelemetry-sdk-0.7.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:147 sleep> - /home/shields/.rvm/gems/ruby-2.6.6/gems/opentelemetry-sdk-0.7.0/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb:131:in `sleep'
[19991] Use Ctrl-C to stop
[19991] * Starting control server on http://<IP_ADDRESS>:9293
[19991] - Worker 0 (pid: 20003) booted, phase: 0
[19991] - Worker 1 (pid: 20007) booted, phase: 0

This issue has been fixed in other telemetry libs, see for example here:
https://github.com/newrelic/newrelic-ruby-agent/commit/57cd9711003456288d6fe81aeda637173416d075
Here's a bit more info:
https://github.com/puma/puma/issues/614

We already handle restarting the thread in https://github.com/open-telemetry/opentelemetry-ruby/blob/e3771684cca4ff674c968761fed838719fade459/sdk/lib/opentelemetry/sdk/trace/export/batch_span_processor.rb#L144-L151
We have not found it particularly important to not start the thread in before fork, despite warnings.


Not following 304 redirect with POST

If I do this http --all -F POST "http://<IP_ADDRESS>:37375 @data.csv
I get:
HTTP/1.1 304 Not Modified
Connection: close
Date: Wed, 14 Aug 2019 13:43:09 GMT
Location: http://<IP_ADDRESS>:27375/

Server (:37375) logs says that redirect sent, but nothing appeared on :27375
If I do this using curl -L - works as expected, data sends to :27375

Could you please give more info about the environment you run on? windows? I can't reproduce this issue on macos.
I ran this server:
from flask import Flask, redirect

app = Flask(__name__)

@app.route('/', methods=['POST'])
def hello():
    return redirect("http://www.example.com", code=304)

locally, and executing http --all -F POST http://<IP_ADDRESS>:5000/ hello=World gives me:

HTTP/1.0 304 NOT MODIFIED
Date: Tue, 10 Sep 2019 07:39:40 GMT
Location: http://www.example.com
Server: Werkzeug/0.15.5 Python/3.7.4

I also couldn't reproduce the problem, @struckoff would you mind sharing a reproducer URL?


[RBD Async DR] Fix resize/Delete volume when it moved to another cluster

In the case of the Async DR, the volumeID will not be the same if the clusterID or the PoolID is different, With Earlier implementation, it is expected that the new volumeID mapping is stored in the rados omap pool. In the case of the   ControllerExpand or the DeleteVolume Request, the only volumeID will be sent it's not possible to find the corresponding poolID in the new cluster.
With This Change, it works as below
The csi-rbdplugin-controller will watch for the PV objects, when there are any PV objects created it will check the omap already exists, If the omap doesn't exist it will generate the new volumeID and it checks for the volumeID mapping entry in the new configmap, if the mapping does not exist, it will add the new entry to the configmap. the controller also does a periodic cleanup to remove the stale volumeID mappings from the configmap.
The configmap will be mounted as a volume to the csi-rbdplugin provisioner, when cephcsi receives a request for the volume it will check for the volumeID mapping in the mounted file, if the mapping exists it will
use the new volumeID for further operations.
fixes: #1936
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@nixpanic @ShyamsundarR @humblec can you please review this one priority?

Is is not much simpler to set the volumeID/handle in the metadata of the RBD image? What prevents doing that?
I'm not really in favour of having a ConfigMap that basically acts as a database of volumes...

Is is not much simpler to set the volumeID/handle in the metadata of the RBD image? What prevents doing that?

Not that much Easy, to go to the rbd image you need the image name and the pool name which is not possible. In most cases, cephcsi will get a request with volumeID. we decode the volumeId and extract clusterID, pool ID, and retrieves the rbd image name from the omap.

I'm not really in favour of having a ConfigMap that basically acts as a database of volumes...

Any other suggestions to solve this issue?

Is is not much simpler to set the volumeID/handle in the metadata of the RBD image? What prevents doing that?
I'm not really in favour of having a ConfigMap that basically acts as a database of volumes...

Agree to the above.  The csi-rbdplugin-controller watching for PVs, updating configmap and also doing stale cleanup looks very tedious/weird from CSI driver stand point.  imo, these kind of tasks should be outside CSI driver itself( which I am repeating :) from https://github.com/ceph/ceph-csi/pull/1877#issuecomment-795281859) . It seems that,  we are adding more hacks/tweaks here and there to get this async story tackled somehow.

Is is not much simpler to set the volumeID/handle in the metadata of the RBD image? What prevents doing that?
I'm not really in favour of having a ConfigMap that basically acts as a database of volumes...

Agree to the above. The csi-rbdplugin-controller watching for PVs, updating configmap and also doing stale cleanup looks very tedious/weird from CSI driver stand point. imo, these kind of tasks should be outside CSI driver itself( which I am repeating :) from #1877 (comment)) . It seems that, we are adding more hacks/tweaks here and there to get this async story tackled somehow.

csi-rbdplugin-controller is a sidecar. in future, the csidriver will talk to a new sidecar to get the corresponding volumeID with the new RPC spec (this is not there yet)
@humblec am looking for a suggestion as I mention in https://github.com/ceph/ceph-csi/issues/1936 a few days ago.

@Madhu-1 Will this take care of existing volumes? Meaning, lets say we enable DR on an existing cluster which already has many volumes.
Also, can there be a case when the data in the new config map is corrupted.  Situations like Split brain etc.?

@Madhu-1 Will this take care of existing volumes? Meaning, lets say we enable DR on an existing cluster which already has many volumes.
Also, can there be a case when the data in the new config map is corrupted. Situations like Split brain etc.?

This is only for the volumes which are moved from one cluster to another cluster, not for the new volume (During failover when the PVC, PV are created for the existing images the entry will get populated in the configmap as cephcsi won't know the pool Name during few operations like Expand and Delete).
chances are less (not sure we can get split-brain in CM) If the configmap is corrupted i assume it will be same as omap corruption either the read from the mounted volume will fail and we are not able to perform any operations.

I'm not really in favour of having a ConfigMap that basically acts as a database of volumes...

Any other suggestions to solve this issue?

Can the volume-id be set in a label on the PV, or in some other attribute? It can be a task of the (currently person) operator that creates the static PVs on the secondary site.
Maybe it would be easier if there is a ReplicatedPersistentVolume CR that points to a PV and some of the other options? That way, a StorageReplicationController can watch for the ReplicatedPersistentVolume objects and store/fetch information there.

I'm not really in favour of having a ConfigMap that basically acts as a database of volumes...

Any other suggestions to solve this issue?

Can the volume-id be set in a label on the PV, or in some other attribute? It can be a task of the (currently person) operator that creates the static PVs on the secondary site.

This becomes a cephcsi specific fix as the other drivers might work with the same volumeID.

Maybe it would be easier if there is a ReplicatedPersistentVolume CR that points to a PV and some of the other options? That way, a StorageReplicationController can watch for the ReplicatedPersistentVolume objects and store/fetch information there.

@nixpanic the problem really exists at the ExpandVolume and DeleteVolume which will get trigger from the external-provisioner.

I understand its a sidecar, the point I am making here is, the concern due to the dependency/references requires for CSI driver to operate due to this change or set of changes in the volumeID handling.

@humblec We heard you and no one is denying what you are saying but I guess what Madhu has been asking is for an alternative solution/idea. Will really appreciate if you can help in figuring out the same.

@agarwal-mudit @Madhu-1 sure, do we have a design doc which can be referred or reflect this async work ? the available design doc here in this repo is very cryptic and also does not cover most of the integration part, may be its missing the implementation changes ( if any) from initial plan..etc.

I favour the solution in #1946 much more than this approach with the ConfigMap.
@humblec could you share your opinion too?

I favour the solution in #1946 much more than this approach with the ConfigMap.
@humblec could you share your opinion too?

Yeah , definitely configmap approach is bit ugly. I will review the solution in 1946 soon. Thanks !


Texture Region dock doesn't resize when the windows resizes

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0 master https://github.com/godotengine/godot/commit/9a78efc7c270211e49fd7b2f071b61c706febffc
Issue description:

Yes this is caused because of the toolbar, wich fixes the minimum size.
Maybe it would be possible to move the snap options to the inspector on a bottom control? (like the mesh previews)


Script bundle does not include sourcemaps

🐞 Bug report
Command (mark with an x)

- [ ] new
- [x ] build
- [x ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?

No
Description
When you create a script bundle with the "scripts" config in angular.json, the generated sourcemap does not take in account the existing sourcemap in those scripts.
🔬 Minimal Reproduction
ng new myapp
create a folder lib with a dummy mylib.ts file

namespace Lib {
    export class App {
        init() {
            setTimeout(() => console.log('Hello'));
        }
    }
}

And a tsconfig.json file, enabling TypeScript sourcemap :

{
    "compileOnSave": false,
    "compilerOptions": {
      "outFile": "./build/mylib.js",
      "sourceMap": true,
      "declaration": true,
      "emitDecoratorMetadata": true,
      "experimentalDecorators": true,
      "moduleResolution": "node",
      "target": "es5",
      "typeRoots": [
        "node_modules/@types"
      ],
      "lib": [
        "es6",
        "dom"
      ]
    }
  }

Build your dummy lib with a tsc command in lib folder.
Then update angular.json config to enable sourcemap and embed the script :

...
           "scripts": [
              "lib/build/mylib.js"
            ],
            "sourceMap": true
...

🔥 Exception or Error
When you launch the app with ng serve or ng build, the sourcemap available in devtools is mylib.js, the built code, whereas I would have expected to retrieve my initial lib TS file, which is available if I include directly the library in index.html.
Scripts.js :

var Lib;
(function (Lib) {
    var App = /** @class */ (function () {
        function App() {
        }
        App.prototype.init = function () {
            setTimeout(function () { return console.log('Hello'); });
        };
        return App;
    }());
    Lib.App = App;
})(Lib || (Lib = {}));
//# sourceMappingURL=mylib.js.map
;
//# sourceMappingURL=scripts.js.map

🌍 Your Environment

Angular CLI: 7.3.6
Node: 8.11.1
OS: win32 x64
Angular: 7.2.10
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.13.6
@angular-devkit/build-angular     0.13.6
@angular-devkit/build-optimizer   0.13.6
@angular-devkit/build-webpack     0.13.6
@angular-devkit/core              7.3.6
@angular-devkit/schematics        7.3.6
@angular/cli                      7.3.6
@ngtools/webpack                  7.3.6
@schematics/angular               7.3.6
@schematics/update                0.13.6
rxjs                              6.3.3
typescript                        3.2.4
webpack                           4.29.0

Anything else relevant?
Maybe this can be fixed with some webpack config ?

Hi, this is expected and to have sourcemaps for libraries you need to enable vendor in angular.json under the serve options.
"serve": {
	"builder": "@angular-devkit/build-angular:dev-server",
	"options": {
	"browserTarget": "myproj:build",
	"sourceMap": {
		"scripts": true,
		"styles": true,
		"vendor": true
	}
},

Hi, thank you for your answer, I didn't know there was such options.
It added vendors sourcemap, but scripts sourcemap are still the same. (it was true by default https://github.com/angular/angular-cli/blob/master/packages/angular_devkit/build_angular/src/browser/schema.json ).
So it does not solve my issue, TypeScript sourcemap for external script is still not there...

hi, any updates on this?
experiencing the same issue in my project  👀
having a hybrid Angular app (AngularJS + Angular 8) and desire to see the legacy part sourcemaps, however, cannot find an appropriate solution
would appreciate any further comment on this issue
thanks in advance

@yardobr @bviale
Hey guys. I faced the same problem.
The solution is to use 'ng serve' with  --vendorSourceMap
The flag is deprecated but it's the only solution right now, or at least the only one that I found.
Here is the solution's origin:
https://github.com/angular/angular-cli/issues/15064


Moved meshery adapter for osm

Signed-off-by: hikky08<EMAIL_ADDRESS>Description
This PR fixes #1451
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Thank you, @hikky08 and @PluckyPrecious 👍

Yippie! 🥳

Yay, @hikky08!

Welldone @hikky08 nice job!

Looking good! @hikky08

Thank you


remove Check-in and Phone triage needs from triage page

 (misc) Don't let users add Phone triage or Check-in need types on Triage form


Using DB::raw in where() with Eloquent adds 'is null' to the SQL query

I'm trying to user DB::raw in the where() method of my query using Eloquent but for some reason is adding 'is null' after my where clause.
This is my code:
People::has('user')->where(DB::raw('first_name = \'Niko\''))->orderBy($sort, $order)->paginate($perPage););

It  execute these SQL's:
0.67ms  select count(*) as aggregate from `people` where (select count(*) from `users` where `users`.`people_id` = `people`.`id`) >= '1' and first_name = 'Niko' is null
0.69ms  select * from `people` where (select count(*) from `users` where `users`.`people_id` = `people`.`id`) >= '1' and first_name = 'Niko' is null order by `id` desc limit 1 offset 0

But if I try this way:
People::has('user')->where('first_name', 'Niko')->orderBy($sort, $order)->paginate($perPage);

Generate these SQL's:
0.94ms  select count(*) as aggregate from `people` where (select count(*) from `users` where `users`.`people_id` = `people`.`id`) >= '1' and `first_name` = 'Niko'
0.72ms  select * from `people` where (select count(*) from `users` where `users`.`people_id` = `people`.`id`) >= '1' and `first_name` = 'Niko' order by `id` desc limit 1 offset 0

As you can see are the same SQL's but without 'is null' after WHERE. This is only an example, I know that I can use the second method but I need to use DB::raw because I will generate complex clauses.
I have the lastest version of Laravel.

This was a big help to me, thanks @taylorotwell!

 Thanks @taylorotwell.
For anyone who comes later and may be needing to use some kind of sql function like to_char or extract (I'm on Oracle) I'm also doing a whereHas. Here's what I had:
$request = Request::all();
$interface_type_id = $request['interface_type_id'];
$month             = $request['month'];
$request = Request::all();

$interface_type_id = $request['interface_type_id'];
$month             = $request['month'];

$data = CommissionActivityDetail::with('CommissionActivity.Type',
									   'AcctInfo.employee'
	    )->whereHas('CommissionActivity.Type', function($q) use ($interface_type_id, $month) {
		    $q->where('interface_type_id', '=', $interface_type_id)
		      ->whereRaw("extract(month from commission.commission_activity_detail.created_date) = $month");
})->get();

@taylorotwell that seems to work. But is it a bug with DB::raw()?

Hi Guys, this is an odd one and I too got stumped by it for a while...
where(DB:raw('blar'))
vs
whereRaw('blar')
But would would DB::raw() work this way and add the extra is null to the end?

@caleywoods your query is not secure.
You must use prepared statements:
->whereRaw('extract(month from commission.commission_activity_detail.created_date) = ?', [$month]);

@gabrielkoerich We're on oracle database, all input gets bound by the database. Thank you though for the code I'll be sure and use it.

public function retrieveAppuserAnswers(Request $request)
{
$answers = DB::table('answers')->whereIn('question_id', function($query){
$query->select('id')->from('questions')->where('appuser_id','=', $request->appuserId);
})->get();
return $answers;
}
i am getting undefined variable request error in this case. How can i pass variable in where clause in above code.

@jkunwar , add "use ($request)" in your callback.
public function retrieveAppuserAnswers(Request $request)
{
    $answers = DB::table('answers')->whereIn('question_id', function ($query) use ($request) {
        $query->select('id')->from('questions')->where('appuser_id', '=', $request->appuserId);
    })->get();

    return $answers;
}

https://stackoverflow.com/questions/35035355/how-to-pass-argument-in-callback-function-in-php

@gabrielkoerich - Great comment preaching on safe programming practices. Also, congrats on your victory against Mexico today!

@moebaca thanks bro!

Use whereRaw and just pass it a string.
Wow
Amazing ans
Thanks owner

"Use whereRaw and just pass it a string."
It's works Perfect

But is it a bug with DB::raw()?
But why would DB::raw() work this way ...?

@taylorotwell, @nikoskip
This is still a bug in DB::raw(). The 'is_null' appended at the end of the query is just plain wrong. A bug should not be closed just because whereRaw() exists as a workaround.
Please re-open this ticket so it can get fixed.

Use whereRaw and just pass it a string.

It'helps me too thanks Taylor


Provide a way to pause / resume the barcode scanner while leaving the camera preview open

A user requested a way to be able to stop the scanner, while the camera preview is still open and providing video input.
This issue serves as a feature request for that.

Hi,
@navaronbracke Thanks.
My current implementaion is like that
But yes, it could be nice to do this via controller or something like that.
  void startListenToScannerCapture() {
    streamSubscription = controller.barcodes.listen(scannerlistener);
  }

  void scannerlistener(BarcodeCapture barcodeCapture) async {
    Some actions
    .
    .
    .
    .
    await streamSubscription.cancel();
  }

  // And when button pressed
        onPressed: () {
          startListenToScannerCapture();
        },

@navaronbracke
Note:
By the way, there is something weird regarding the StreamSubscription<BarcodeCapture>,
streamSubscriptionScanner = controller.barcodes.listen(scannerlistenerCallback);
When I call: streamSubscription.pause()
And move the scanner on a barcode, it doesn't read it, which is ok.
But when I call streamSubscription.resume(),
the stream listener callback is called with the scanned value of the barcode that was scanned while I was on pause.

@navaronbracke
Hi,
Any update on the issue here?
Thanks.

No, not yet. Although there is an open PR, but that is to pause the camera preview, not the barcode scanner.

No, not yet. Although there is an open PR, but that is to pause the camera preview, not the barcode scanner.

Ok thanks.
But when you are going to work on it, pay attention to my additional comment:
https://github.com/juliansteenbakker/mobile_scanner/issues/1006#issuecomment-2049613983

We should clear the scanned barcodes in the scanner itself when resuming. We only do that for the start method currently.


Installation Issue

i get this error when booting into ventura from fresh install

Did you follow the install instructions for Ventura?
https://github.com/5T33Z0/Lenovo-T530-Hackintosh-OpenCore/blob/main/macOS_Install/OpenCore/OC_macOS13.md

Did you follow the install instructions for Ventura?
https://github.com/5T33Z0/Lenovo-T530-Hackintosh-OpenCore/blob/main/macOS_Install/OpenCore/OC_macOS13.md

yes i did created the installer with opencore legacy patcher my t530 has the same specs

But I can see macOS running in the screenshot. Or is this is a differnt installation?
BTW: you can disable the camera in BIOS. No need to put cover the lens.

But I can see macOS running in the screenshot. Or is this is a differnt installation?
BTW: you can disable the camera in BIOS. No need to put cover the lens.

i have monterey installed on my msata. i tried installing ventura to an external hard drive via usb and get that error in verbose mode when i load up monterey the error i got from the ventura installer shows up

Okay. Try installing on a new internal volume instead. Enable the AppleCpuPmCfgLock Qurik if it's not enabled already.

it worked im finishing the rest of the process.


drill-4531: add a test for the fix

@kfaraaz or @krystaln can you please review this PR? add a test for DRILL-4531.
http://<IP_ADDRESS>:8080/job/Framework-Regression/494/console

Looks good.  +1


Focal Point?

Pinch to zoom here only zooms in at the center of your screen, have you interacted with focal points to zoom at the position you pinch at?

Hi
Good point, I'll check how it could be implemented
If you're interested & have time, submit a PR please

I think event.focalX is the way to do this. I did some initial experiments here

I think event.focalX is the way to do this. I did some initial experiments here

Thanks for the hint


Support Java modules (JPMS)

It is currently not possible to use pgpainless in a Java modular application. There are two problems:

The lack of module-info.java included in the jar, or even Automatic-Module-Name in the jar manifest. This is relatively easily solved.
The use of a split package for org.bouncycastle to implement CachingBcPublicKeyDataDecryptorFactory. It looks like this was done to get access to package-private BcImplProvider,  but it is a blocker for those who want to use this library in a modular application.

Although Java modules are not typically in high use on the server side, for a desktop client application (eg Swing or JavaFX) modules are crucial to get jlink/jpackage to produce nice small installers. From, say 100mb to ~30mb for a Hello World app.

Hey! Thanks for the bug report.
I haven't yet played around with modular applications at all, so it's good that you bring up these blockers.
I'll see how easy it is to fix the first point.
Regarding the second point, maybe it is worthwhile trying to get CachingBcPublicKeyDataDecryptorFactory upstreamed to BC?

Regarding the second point, maybe it is worthwhile trying to get CachingBcPublicKeyDataDecryptorFactory upstreamed to BC?

Yes, could be. Unfortunately it's not possible to have classes with the same package in two different jars with JPMS, so it will have to be moved out for support to be possible.

Do you think https://github.com/pgpainless/pgpainless/pull/435 would together with #429 would fix your issue?

From a quick review it looks like it would yes. Thanks!

Closed with PGPainless 1.6.7

Thanks - I'm now using 1.6.7 and can confirm that basic modules support is working. Applications using jlink will need to set reads on org.slf4j, org.bouncycastle.provider and org.bouncycastle.pg as these modules are all required.


Attach the route summarization policy under L3Out Ext EPG

Adding support to attach the route summarization policy under L3Out External EPG Subnet

This appears to be a breaking change. The original schema should be retained, and continue to apply the default policy from the common tenant if no policy is specified in the new bgp_route_summarization_policy attribute.

This appears to be a breaking change. The original schema should be retained, and continue to apply the default policy from the common tenant if no policy is specified in the new bgp_route_summarization_policy attribute.

Sounds good, I am working on this.

This appears to be a breaking change. The original schema should be retained, and continue to apply the default policy from the common tenant if no policy is specified in the new bgp_route_summarization_policy attribute.

I have pushed the updates in this PR. As per the new logic:

If "bgp_route_summarization" is set to 'true' and "bgp_route_summarization_policy" is not defined, 'default' route summarization policy will be added.
If "bgp_route_summarization" is set to 'true' and "bgp_route_summarization_policy" is defined, custom policy name will be added.
If "bgp_route_summarization" is set to 'false' and "bgp_route_summarization_policy" is defined, no policy will be added.
If "bgp_route_summarization" is set to 'false' and "bgp_route_summarization_policy" is not defined, no policy will be added.

This appears to be a breaking change. The original schema should be retained, and continue to apply the default policy from the common tenant if no policy is specified in the new bgp_route_summarization_policy attribute.

done

please also run pre-commit

please also run pre-commit

Done


Put tests execution out of Excel Application Scope

Hello,
I had some issues using the tests features with workflows using Excel applications (for example testing a KillAllProcesses workflow that kills Excel processes)
I edited the workflow to only put "Read Range" and "Write Range" activity into "Excel Application Scope" and it seems to work perfectly.
Does the executions of the tests needs to be in the Excel Application Scope?

Helloread range and write range have two types1. uses excel application scope as a container2. system that does not use excel application scope as containerThe second type can run on environments without excel application installedOn 28/06/2021, 18:44 jestinga @.***> wrote:
Hello,
I had some issues using the tests features with workflows using Excel applications (for example testing a KillAllProcesses workflow that kills Excel processes)
I edited the workflow to only put "Read Range" and "Write Range" activity into "Excel Application Scope" and it seems to work perfectly.
Does the executions of the tests needs to be in the Excel Application Scope?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks for your answer.
I may have not been very clear.
The issue I encountered was that all the executions of tests are done within an Excel Application Scope (by default). So during the execution of tests the file "Tests.xlsx" remains open. This can produce errors if - for example - the workflows I am testing kills all Excel processes (as a KillAllProcesses workflow would do).
As the Excel Application Scope are only used to read workflows to test at the begining and write tests result at the end: Wouldn't it be better to close the file "Tests.xlsx" during the execution of tests to prevent this kind of errors ?
Here is how I think the workflow "RunAllTests" should look like
(instead of wrapping the whole tests execution within one "Excel Application Scope")

Thanks for detailed screenshots and clarification. I agree it is not good practice to include unrelated activities of excel inside excel application scope. I would as also like to remove visible property of excel application scope to allow background execution.On 28/06/2021, 21:29 jestinga @.***> wrote:
Thanks for your answer.
I may have not been very clear.
The issue I encountered was that all the executions of tests are done within an Excel Application Scope (by default). So during the execution of tests the file "Tests.xlsx" remains open. This can produce errors if - for example - the workflows I am testing kills all Excel processes (as a KillAllProcesses workflow would do).
As the Excel Application Scope are only used to read workflows to test at the begining and write tests result at the end: Wouldn't it be better to close the file "Tests.xlsx" during the execution of tests to prevent this kind of errors ?
Here is how I think the workflow "RunAllTests" should look like (instead of wrapping the whole tests execution within one "Excel Application Scope")
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.


Cannot open new tab/pane

[ ] Your Hyper.app version is 2.1.2. Please verify your using the latest Hyper.app version
[ ] I have searched the issues of this repo and believe that this is not a duplicate

Any relevant information from devtools? (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):

Is the issue reproducible in vanilla Hyper.app?

Issue

Sometimes when trying to open a new tab or new pane I get the following error message, and I'm unable to open said tab or pane. However, I can continue to use hyper in the currently opened instances. This especially happens when an ongoing process is already running, or if the computer goes to sleep and I come back to it.
Update:
The error message that pops up (see below) is
hyper unable to start terminal process CreateProcess failed at new WindowsPtyAgent
I think this might be a duplicate of issues #2747

Hyper version: undefined "2.1.2"

OS ARCH VERSION: win32 x64 10.0.17134

Electron: 1.8.8  LANG: undefined

SHELL: undefined   TERM: undefined

 ~/.hyper.js contents

          {
  "updateChannel": "stable",
  "fontSize": 13,
  "fontFamily": "\"Ubuntu Mono\", \"Roboto Mono\", Menlo, \"DejaVu Sans Mono\", Consolas, \"Lucida Console\", monospace",
  "cursorColor": "rgba(248,28,229,0.8)",
  "cursorShape": "BLOCK",
  "cursorBlink": false,
  "foregroundColor": "#fff",
  "backgroundColor": "#000",
  "borderColor": "#333",
  "css": "",
  "termCSS": "",
  "showHamburgerMenu": "",
  "showWindowControls": "",
  "padding": "6px 7px",
  "colors": {
    "black": "#000000",
    "red": "#ff0000",
    "green": "#33ff00",
    "yellow": "#ffff00",
    "blue": "#0066ff",
    "magenta": "#cc00ff",
    "cyan": "#00ffff",
    "white": "#d0d0d0",
    "lightBlack": "#808080",
    "lightRed": "#ff0000",
    "lightGreen": "#33ff00",
    "lightYellow": "#ffff00",
    "lightBlue": "#0066ff",
    "lightMagenta": "#cc00ff",
    "lightCyan": "#00ffff",
    "lightWhite": "#ffffff"
  },
  "MaterialTheme": {
    "theme": "",
    "backgroundOpacity": "1",
    "accentColor": "#64FFDA",
    "vibrancy": "dark"
  },
  "shell": "C:\\Program Files\\Git\\git-cmd.exe",
  "shellArgs": [
    "--command=usr/bin/bash.exe",
    "-l",
    "-i"
  ],
  "env": {
    "TERM": "msys"
  },
  "bell": "SOUND",
  "copyOnSelect": false
}
      {

"plugins": [
"hyper-alt-click",
"hyper-search",
"hyperdocs",
"hyper-font-smoothing",
"hypercwd"
],
"localPlugins": []
}

I have the same problem

try del plugin hypercwd .

I've switched from hypercwd to https://github.com/LeanderFS/hypersamewd and it solved this issue for me (and performance seems to be better)

Plugin related. Closing


Stub translation files for Unmanaged metadata in the Trial Org Config

Add translation file stubs for the following metadata that is included in the Trial Org Configuration:

Opportunity.RecordType Labels and Description
Opportunity.Type Pick List Values
Opportunity.StageName Pick List Value
Opportunity Contact Role Pick List Values

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

**lurch: attach W-038417

Tracking W-038417


Add WebFont feature

This PR added a function to automatically download font in vcode. The font is available even in an environment where the font of the fontFamily is not downloaded.

And to use the size-adjust of the font-face, we updated it to<EMAIL_ADDRESS>that uses more than 92 versions of chrome. The reason for using size-addjust was to size other ASCII code characters and UNICODE characters.
For example, if you write it as follows, you can match the size of all UNICODE with English characters.

This seems to be for the editor.

Actually maybe not, is this for all workbench?

The purpose was a function for editor and terminal, but it seems to be available for all workbench.

Thanks, but it is unlikely that we would accept this PR when it changes the font for the workbench. I also think it is somewhat strange to use a web font in a desktop installed application? Wouldn't a user install the font locally to use it? Or is the purpose here for VSCode for Web?

In fact, when I made this function, I was working in codespace. Since it was a public computer, it was a computer that was initialized upon reboot, so I worked using code-server or codespace. So there was a hassle of re-installing the font for powerline-font and editor. Also, when using a personal computer, I wondered if I could save the trouble of re-installing the font when using more than one computer.

As I had stated, in this form I would not accept the PR. If you want to provide a PR where the editor or terminal font specifically can be set to a web font, please go ahead and then I would assign it to respective component owners. But then I think the feature needs to be scoped to those components that explicitly allow for changing the font (editor, terminal).
For the workbench we have not yet decided how to best support font settings.
Thanks for understanding 🙏


feat: add BitKeep Wallet

Add BitKeep Wallet to the ecosystem project

Thank you, I have fixed it. Can you authorize it?


doc: release-notes-3.7: Power management notes

This adds some bits about additions and improvements on Power Management.

@ceolin  please fix the doc issues

@ceolin please fix the doc issues

done


Learning of gulp added in pt-br

Added more one option of learning of one automator of tasks in pt-br

Thanks! Could you please also add a "menu" item for this new category near the top of the file?

Added, please make sure it is correct :smile:

Thanks, that's perfect.

Thanks, I will try to always be contributing to the repository. :smiley:


Update README.md

Summary of Changes
Include a summary of changes then remove this line
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #64   +/-   ##
=======================================
  Coverage   80.65%   80.65%
=======================================
  Files          14       14
  Lines         517      517
  Branches       47       47
=======================================
  Hits          417      417
  Misses         83       83
  Partials       17       17

Continue to review full report at Codecov.

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

@ttannis updated


Add support for io.js

At this moment Ghost works only with Node v0.10.x.
"engines": {
    "node": "~0.10.0"
},

What is the status of Travis CI supporting io.js?  I'd vote for waiting to put it in until it was able to be tested with the same rigor as the node engine.

What is the status of Travis CI supporting io.js

Support for io.js needs to land in nvm before it can be used on Travis-CI.
Extending what @jgable said, io.js is (obviously) really new, and there's going to have to be a bit of lag while tooling catches up and the dust settles.  At the point when we can easily install it we'll be able to give it a shakedown and see what kind of issues we have with our dependencies, ES6 feature conflicts, etc.

As it stands right now the sqlite3 dependency won't build with io.js. Unless the dependencies work properly with io.js there's barely anything we on Ghost can do to support io.js.

@halfdan You're right, but it seems it'll be fixed soon mapbox/node-sqlite3#385.

:+1:

+1

Supposedly travis is going to ship a new version of nvm with io.js support on Tuesday. Happy to pick this up and add io.js to the builds. Which, I think, for now should be an allowed failure. Suggestions?

@halfdan it seems travis is ok now: http://docs.travis-ci.com/user/build-environment-updates/2015-02-03/#io.js-support.
What about node-sqlite3? It works with io.js fine, but travis and appveyor needs to be updated and of course they need to merge pull request etc.

+1

Sadly they rescheduled the update. Got the PR ready just waiting for Travis to push the change again.

:+1:


latest error..

node:internal/process/esm_loader:108
internalBinding('errors').triggerUncaughtException(
^
Error: Transform failed with 1 error:
/Users/macpro/Downloads/api-key-nodejs-master/coba.ts:5:12: ERROR: Top-level await is currently not supported with the "cjs" output format
at failureErrorWithLog (/Users/macpro/.npm/_npx/fd45a72a545557e9/node_modules/esbuild/lib/main.js:1636:15)
at /Users/macpro/.npm/_npx/fd45a72a545557e9/node_modules/esbuild/lib/main.js:837:29
its use command /imagine with latest version

I do not have the file named coba.ts  can  not help.

thats file same code from ur example and frontpage this source app

Please try to ask this question to chatGPT, It will solve your issue, I can not teach you how to code.
"top-level await" is a modern javascript feature. you really need to be able to solve that alone.

give sample after imagine can get progress in log

check the example like:
https://github.com/UrielCh/midjourney-client/blob/main/samples/variantLast.ts
depending on
https://github.com/UrielCh/midjourney-client/blob/main/samples/progressLogger.ts


Find missing API-Features

Let's find missing API-Features. Please add comments to this issues, which features are missing

Hi! Cannot find an option to migrate variables. According to POST /migration/generate endpoint documentation there should be a variables field in request body that is missing in MigrationPlanGeneration DTO
  public class MigrationPlanGeneration
  {
    /// <summary>
    /// The id of the source process definition for the migration.
    /// </summary>
    [JsonProperty("sourceProcessDefinitionId")]
    public string SourceProcessDefinitionId;

    /// <summary>
    /// The id of the target process definition for the migration.
    /// </summary>
    [JsonProperty("targetProcessDefinitionId")]
    public string TargetProcessDefinitionId;

    /// <summary>
    /// A boolean flag indicating whether instructions between events should be configured to update the event triggers.
    /// </summary>
    [JsonProperty("updateEventTriggers")]
    public bool UpdateEventTriggers;
  }

Hi @AlexanderSysoev,
I'll take a look.

On Jobs... recalculate due date is missing.

Someone added ExternalTasks.Lock, but missed a spot, PR is open.  https://github.com/komsa-ag/Camunda.Api.Client/pull/16

Someone added ExternalTasks.Lock, but missed a spot, PR is open. #16

I will have a look

The post calls to get tasks are missing, and thus the feature to fetch tasks by multiple Ids, and the orQueries:
https://docs.camunda.org/rest/camunda-bpm-platform/7.20-SNAPSHOT/#tag/Task/operation/queryTasks


feat(material/datepicker): Allow user to jump between start and end d…

…ates with arrow keys
relates to internal ticket 239561710

Integration test failures appear to be unrelated


⚠ Bofrost.nl has degraded performance

In 6e6eb58, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2771 ms

Resolved: Bofrost.nl performance has improved in 08aa5bb after 6 minutes.


Publish and Restore implemented

New commands to support bicep publish and bicep restore. Also added NoRestore parameter to Build-BicepNetFile Wrapper module now looks like the following:
Build-BicepNetFile -Path <string> [-NoRestore] [<CommonParameters>]

ConvertTo-BicepNetFile -Path <string> [<CommonParameters>]

Get-BicepNetVersion [<CommonParameters>]

Publish-BicepNetFile -Path <string> -Target <string> [<CommonParameters>]

Restore-BicepNetFile -Path <string> [<CommonParameters>]

Also refactored a majority of the setup code that is shared to be defined in BicepWrapper.cs and utilized in the partial class "command" files.

Tested restore and publish and they work as expected.

Can you change the version number in .bicepVersion to v0.4.1008 as while we are at it?

LGTM! I think we should merge and release a new version of BicepNet. ping @SimonWahlin

Great!
I'll get on that.

Versions bumped to v0.4.1008 for Bicep and 1.0.6 for BicepNet.


Allow for JobSubmitter to give priority to older workflows and tasks

Impact of the new feature
T0 WMAgent, WMAgent
Is your feature request related to a problem? Please describe.
T0 requires to finish workflows as soon as possible in order to delete data fast. Currently, JobSubmitter gives priority to start processing new workflows instead of finishing old ones:
https://github.com/dmwm/WMCore/blob/a55235c93cf788d2f9dad1fd9180a9bb7897b1f0/src/python/WMCore/WMBS/MySQL/Jobs/ListForSubmitter.py#L37
This is problematic, specially when T0 accumulates a job backlog greater than the cache size.
Describe the solution you'd like
Add a JobSubmitter configuration parameter to define whether or not to give priority to new workflows.
Describe alternatives you've considered
None
Additional context

@germanfgv Hi German, are you suggesting to replace that wmbs_workflow.id DESC by wmbs_workflow.id ASC?
I think it makes total sense and there is no reason why central production should have it differently. So i would say go ahead and change it directly in the DAO, no need to make it configurable.
Can you please provide a pull request for this? Otherwise please let us know and we will take it.


job retry failed with unknown reason

The detailed reason of this issue is that if there is no history data in elastic search (i.e. if no gc happens in framework launcher), there will no ‘framework’ index in elastic search. This will cause error response when requesting elastic search api.
When there  is history data in es, the index will be initialized, and the error be gone.
Work around:

Hide link and warning icon when retry is 0
regard as healthy if  'index not found'

@debuggy, do you have a fix for 2 ?


Import Directory.Build.props from parent if present

Resolve: #151
Add check if parent folder contains a Directory.Build.props file and import it if present.

@dthaler I am not understanding how the proposed solution in the linked issue would work. My understanding is that msbuild would only load the inner most directory.build.props when processing the vcxproj.

@Alan-Jowett please rebase

@Alan-Jowett please rebase

Rebased.

github reports "Merging can be performed automatically with 1 approving review." even though I approved, so I may not have permissions to be counted.


[Docs] Improve readme

Closes #13

Add dashboard and APM images
Explain motivation
Add comprehensive features list
Explain some inner workings

Codecov Report

Merging #17 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #17   +/-   ##
======================================
  Coverage    83.1%   83.1%
======================================
  Files          18      18
  Lines        1154    1154
======================================
  Hits          959     959
  Misses        133     133
  Partials       62      62

Continue to review full report at Codecov.

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


Forbid adding WAL to the repository after advancing last record LSN.

When you advance last record LSN, all changes up to that LSN should be
imported into repository. We have been a bit sloppy about that when it
comes to the checkpoint information that we also store in the repository.
In WAL receiver, for example, we would receive a WAL record, advance
last record LSN, and only then update the checkpoint relish at the same
LSN. Reorder that so that you advance the last record LSN only after
updating the checkpoint relish. It hasn't apparently caused any problems
so far, but let's be tidy.
Tighten the check for that, so that it checks for 'lsn > last_record_lsn'
rather than 'lsn >= last_record_lsn'.

Is this comment on last_record_lsn inaccurate? I'd think it would be pointing to the first byte of the last processed WAL record
    // last_record_lsn.load().last points to the end of last processed WAL record.

https://github.com/zenithdb/zenith/blob/main/pageserver/src/layered_repository.rs#L527

Is this comment on last_record_lsn inaccurate? I'd think it would be pointing to the first byte of the last processed WAL record
https://github.com/zenithdb/zenith/blob/7c62a57e543c47d656bbade775aeb964e9c6b0b7/pageserver/src/layered_repository.rs#L527

Yeah, it is accurate. It's a bit surprising, I know. And when it says "points to the end of the last processed WAL record", it's exclusive, i.e. it means the last byte that's part of the WAL record + 1. Or in other words, the location where the next WAL record would start.
(A bit more detail on that; WAL records always start at an 8-byte alignment boundary, and we always align the end-LSN to that. And if the WAL record ends at the end of a WAL page, there's going to be a page header at the end-LSN before the next record.)
We use end-LSNs like that in many places:

The LSN stored on every page is the end of the WAL record that last modified it.
The LSN we store with each page version in the repository is also an end-LSN (same as that stored in the page header)


Add 'To Base91' and 'From Base91' Operations

Add 'To Base91' and 'From Base91' Operations

I wanted this feature too, please consider this feature!

I’d like to throw my hat in for base94. I submitted a pull request several months back and would appreciate it having a second look.  Or first look as the case may be.
Best,
Sebastian

From: Alexander Craggs @.>
Sent: Monday, June 28, 2021 10:24:48 AM
To: gchq/CyberChef @.>
Cc: Subscribed @.***>
Subject: Re: [gchq/CyberChef] Add 'To Base91' and 'From Base91' Operations (#1081)
@popey456963 commented on this pull request.
After this PR we would have Base 32, 58, 62, 64, 85 and 91. I wonder if we could combine all of these operations into a 'From Base N' or 'To Base N' operator. The default alphabet for each base could be selected from a dropdown.

In src/core/operations/FromBase91.mjshttps://github.com/gchq/CyberChef/pull/1081#discussion_r659832458:

+import Operation from "../Operation.mjs";
+import { ALPHABET_OPTIONS, fromBase91 } from "../lib/Base91.mjs";
+/**

From Base91 operation

*/
+class FromBase91 extends Operation {

/**

* FromBase91 constructor

*/

constructor() {

   super();

   this.name = "From Base91";

   this.module = "Default";

   this.description = "BasE91 is an encoding method that uses ASCII characters. Similar to base64, it has the advantage of limiting the size of the encoded data and to be used as a cipher.";

Capitalised 'E' on 'BasE91'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/gchq/CyberChef/pull/1081#pullrequestreview-694019262, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKWFL3P55M3PCG367AOXK33TVCA3BANCNFSM4PWHWYWA.

Hey! This pull request has gotten rather old and that's our fault. This project stopped being actively maintained for a while and it looks like your pull request has started to gather dust. Would you be able to update your branch to the latest version of CyberChef and we'll give it a review?


Feature request: MigrateStruct

Backstory
From my experience writing the raw sql statements and then making them execute from a migrate function is not really a good way to manage tables, because when you modify the tables with the migrator function you have to change also your structs and sql queries which sometimes can be quite challenging.
Feature
Since sqlx maps queries to structs it would also make sense to have an mapper that maps structs into create table commands.
This would improve developer experience quite alot and make it more easier to work with.

I think this would be out of scope for sqlx; but you can still write your own library to do this.

I think this would be out of scope for sqlx; but you can still write your own library to do this.


(For David) Pass config down from open

Passes down the config in open to the instance. Also makes any non-null open config take precedence over the archive. Addresses #55

lgtm


Hotfixes

This contains fixes from kuggenhoffen/socketIO-client -fork merged to original socketIO master..
Related to: https://github.com/ARMmbed/websocket-client/pull/1

@kuggenhoffen could you review

@kuggenhoffen could you update this branch against review comments ?

@RomanSaveljev @kuggenhoffen could you review as well.


I made a Raspberry Pi Pico port of your library!

Hello, I wanted to use a 74HC595 display on one of my Raspberry Pi Pico projects and found this Arduino library. I have adapted it to work with it and have uploaded to GitHub at https://github.com/Devnol/ShiftRegister74HC595-Pico. I always try to show the original sources of my projects and I did as best as I could in showing that my library is a port of yours. However, I am not sure whether I have handled licensing correctly. Therefore, I would like you to please verify that my library correctly shows that it's a port of yours both inside LICENSE and the source files. Would it be easy for you to check out whether I have done so properly and maybe comment what I need to change so that my port abides by the MIT LICENSE correctly?
Thanks in advance, Devnol.

Looks good to me! Thanks for your comment and for giving me credit in the fork.
On a separate note: I think it would be helpful to name the differences between this repo and your fork in your repo's README.md. Please close this issue, unless you have further questions. Cheers! :-)

Thank you so much for your time. I don't think there's any difference between my version of the library and yours in terms of end user functions, iirc it's a 1:1 map, only changing arduino.h api calls internally to pico stdlib functions. The readme is definitely a little rough on the edges and I do plan to work on it at one point. If there is any difference I have missed feel free to point it out. For now I  think closing the issue seems like the thing to do.
Cheers again and have a great summer break!


Back-button in the lecture components

Is your feature request related to a problem?
When you're on a lecture, e.g. for downloading the slides, you can't easily get back to the overview of this course.
Describe the solution you'd like
When you're on a programming exercise, then there's an arrow pointing to the left, which brings you back to the overview of the course. Just copy this idea (and the implementation lul) also for the lecture content.
Thanks
Describe alternatives you've considered
No response
Additional context
No response

You can use the back button of the browser. We want to get rid of all Artemis back buttons in the future.


Erase unneeded data when tailcalling

This should fix #53 by fill part of the stack with nil that cannot be actually used by the tailcalled function (because it does not have varargs and it was not passed in as an argument).

Nope, current "fix" breaks:
local b = {}
function b.inner(a, bk)
  print(a, bk, b == bk)
end
setmetatable(b, {
  __call = function(_, ...)
    b.inner(...)
    return b.inner(...)
  end
})

local a = b("a")

function a(b, c)
  print(b, c, b==a)
end
function d(_, ...)
  a(...)
  return a(...)
end
d("s")

let me do more digging


weex debug main.we 涉及多个文件依赖404

weex debug src -V -e src/main.we
有没遇到相同问题的？

This PR/issue doesn't received response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


fix issue in base backup deletion logic where scheduled base backups are disabled

disabled.

Thanks for this PR !


testing-library blur test fails after update to react 17

Describe the bug
I am getting unit test failures from the version of the React testing-library installed with create-react-app when testing blur events.
Did you try recovering your dependencies?
Yes.
Which terms did you search for in User Guide?
“testing library” “blur"
Environment
$ npx create-react-app --info
npx: installed 97 in 21.591s
Environment Info:
current version of create-react-app: 3.4.1
running from /Users/davidg/.npm/_npx/43979/lib/node_modules/create-react-app
System:
OS: macOS Mojave 10.14.6
CPU: (4) x64 Intel(R) Core(TM) i5-4260U CPU @ 1.40GHz
Binaries:
Node: 10.17.0 - ~/.nvm/versions/node/v10.17.0/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.11.3 - ~/.nvm/versions/node/v10.17.0/bin/npm
Browsers:
Chrome: 86.0.4240.111
Firefox: 81.0.2
Safari: 14.0
npmPackages:
react: ^17.0.1 => 17.0.1
react-dom: ^17.0.1 => 17.0.1
react-scripts: 3.4.4 => 3.4.4
npmGlobalPackages:
create-react-app: Not Found
Steps to reproduce
I create an app like this:
npx create-react-app example1

Then this src/Example.js component:
import React, {useState} from "react";
export default () => {
    const [text, setText] = useState();
    return <>

{text}

        Field
        <input id='field1' onBlur={() => setText('Blur has happened')}/>
        </>;
}

And this src/Example.test.js test:
import Example from "./Example";
import React from "react";
import {fireEvent, render} from "@testing-library/react";
describe('input blur', () => {
    it('should display text when blurring from the field', () => {
        const {getByLabelText, queryByText} = render(<Example/>);
        const input = getByLabelText(/Field/);
        fireEvent.blur(input);
        expect(queryByText('Blur has happened')).toBeInTheDocument();
    });
});

When I run yarn run test I get this output:
 FAIL  src/Example.test.js
  input blur
    ✕ should display text when blurring from the field (55ms)
  ● input blur › should display text when blurring from the field
        expect(received).toBeInTheDocument()
    received value must be an HTMLElement or an SVGElement.
    Received has value: null
       8 |         const input = getByLabelText(/Field/);
       9 |         fireEvent.blur(input);
    > 10 |         expect(queryByText('Blur has happened')).toBeInTheDocument();
         |                                                  ^
      11 |     });
      12 | });
      at Object.toBeInTheDocument (node_modules/expect/build/index.js:342:33)
      at Object.it (src/Example.test.js:10:50)

If I then change
"@testing-library/react": "^9.3.2”,

in package.json to
"@testing-library/react": "11.1.0”,

Run yarn install and then run yarn run test I get:
 PASS  src/Example.test.js
  input blur
    ✓ should display text when blurring from the field (59ms)
Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        3.441s
Ran all test suites related to changed files.

I have only noticed this behaviour since create-react-app has moved to React 17. Am I missing something? Or is a newer version of the testing-library required for react 17?
Expected behavior
I expect the test to pass.
Actual behavior
The test fails with an error, unless the @testing-library/react dependency is updated.
Reproducible demo
https://github.com/dogriffiths/reacttestingissue

I have the same issue. My (simplified) component:
const Input: FunctionComponent<InputProps> = ({
  errorMessage,
 checked,
  ...nativeInputProps
}) => {
  return (
    <Container>
      <TextInput
        {...nativeInputProps}
      />
      {errorMessage && (
        <ErrorMessage onClick={handleOnClick}>{errorMessage}</ErrorMessage>
      )}
      {checked && <CheckedIcon src={CheckGreenIcon} alt="Green Check Icon" />}
    </Container>
  );
};

(All other components are just styled-components.
My failing test:
test('Should trigger correctly each event', async () => {
  const onBlur = jest.fn();
  const onChange = jest.fn();
  const onFocus = jest.fn();
  const props = {
    onBlur,
    onChange,
    onFocus,
  };
  render(withTestTheme(<Input {...props} />));

  const input = screen.getByTestId(inputTextTestId);
  fireEvent.change(input, { target: { value: 'TEST' } });
  fireEvent.blur(input);
  fireEvent.focus(input);
  expect(onChange).toBeCalledTimes(1);
  expect(onBlur).toBeCalledTimes(1); // <-- fails here
  expect(onFocus).toBeCalledTimes(1);
});

I’ve closed this issue as create-react-app now appears to use v11.1.0 of the testing library.
I suspect the underlying change in React to use the focusout instead of noblur was the change that the later testing-library version needed an update for.


ftplugin/systemd.vim should include a finish guard before using dosini scripts

See https://vi.stackexchange.com/q/43216/10604. I wrote to<EMAIL_ADDRESS>on the 4th of October and have received no response.

Cc @konfekt
Can you please create a PR for this?
Thanks
Chris
Am 16. Oktober 2023, um 20:04, schrieb "D. Ben Knoble" @.***>:
See https://vi.stackexchange.com/q/43216/10604. I wrote to @.*** on the 4th of October and have received no response.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.￼Message ID: @.***>
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/vim/vim/issues/13357", "url": "https://github.com/vim/vim/issues/13357", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Dear @benknoble, I checked for your mail and am sorry to say that I found it indeed in the spam folder. This never happened before.
The mentioned error
Error detected while processing BufRead Autocommands for "*/etc/systemd/*.conf"..FileType Autocommands for "*"..function <SNR>1
4_LoadFTPlugin[18]..script /usr/share/vim/vim90/ftplugin/systemd.vim:
line   12:
E174: Command already exists: add ! to replace it: Sman silent exe '!' . KeywordLookup_systemd(<q-args>) | redraw!

should be fixed by https://github.com/vim/vim/pull/13297.
Regarding the wrong b:did_ftplugin check, how about replacing
" runtime/ftplugin/systemd.vim (lines 6-9)
if !exists('b:did_ftplugin')
  " Looks a lot like dosini files.
  runtime! ftplugin/dosini.vim
endif

by
if exists("b:did_ftplugin")
  finish
endif
let b:did_ftplugin = 1

let b:undo_ftplugin = "setl com< cms< fo<"
setlocal comments=:; commentstring=;\ %s formatoptions-=t formatoptions+=croql

?

Ah, glad to hear back @Konfekt; hope my original post wasn't too brusque 😅
I think what we imagined was adding the if exists | finish guard just before
" runtime/ftplugin/systemd.vim (lines 6-9)
if !exists('b:did_ftplugin')
  " Looks a lot like dosini files.
  runtime! ftplugin/dosini.vim
endif

Okay, so b:did_ftplugin is set by dosini.vim. Seems to work as well. I have no strong opinions and would then go with your options if noon sees any red flags.


Do not disable overloads for XMVECTOR for GNUC/clang when building with no-intrinsics

In my experience, these work completely fine on a modern GCC.
This also adds a force macro to disable this functionality.

I think the more robust test is actually __GXX_ABI_VERSION compare to 1004.
Try out:
#if !defined(_XM_NO_XMVECTOR_OVERLOADS_) && (defined(__clang__) || (defined(__GNUC__) && (__GXX_ABI_VERSION < 1004))

I'm trying GNUC 9.3.0 and I'm getting:
/home/walbourn/DirectXMath/Tests/../Inc/DirectXMath.h:477:33: error: ‘DirectX::XMVECTOR DirectX::operator+(DirectX::FXMVECTOR)’ must have an argument of class or enumerated type
477 |     XMVECTOR    XM_CALLCONV     operator+ (FXMVECTOR V) noexcept;

Which is the error that indicates it doesn't support overloading of __m128.

So as far as I can tell, GCC doesn't support this for intrinsics types like __m128 which is what _XM_NO_XMVECTOR_OVERLOADS_ is there for.
Perhaps you were building with _XM_NO_INTRINSICS_ which defines it's own struct XMVECTOR which would work.

Ah yes, we define _XM_NO_INTRINSICS_. Perhaps when this is defined we can always enable the overloads?

OK, that explains it.
Fixed in this commit.

Sorry to revive this. I'm on GCC which disables overloads. I am not defining _XM_NO_INTRINSICS_. Yet, I can use operators. Expressions such as
min + scale * (XMVectorSet(x, y, z, 0) + 0.5)
work just fine (with min, scale being XMVECTORs). What's going on?

LOL. It seems that's a GCC feature (or quirk). The following program compiles:
#include <pmmintrin.h>

int main() {
  __m128 x = _mm_set1_ps(1), y = _mm_set1_ps(2);
  auto z = x + y * 0.5;
  return 0;
}

Have you looked at SimpleMath?


Naming convention issues with arm-batch swagger spec

Wrong names are being used in arm-batch swagger specs [https://github.com/Azure/azure-rest-api-specs/blob/b91ba60a90b712d57331762e68829d2666600e4b/arm-batch/2015-12-01/swagger/BatchManagement.json]
Existing Names ->   Should be
SynchronizeAutoStorageKeys   -> SyncAutoStorageKeys [As this is in Rest APIs, or change rest apis]
RegenerateKey ->    RegenerateKeys [As this is in Rest APIs]
Application_AddApplication -> Application_Create
Application_DeleteApplication ->    Application_Delete
Application_GetApplication -> Application_Get
Application_UpdateApplication ->    Application _Update
Subscription_GetSubscriptionQuotas -> Quota_Get
Account -> BatchAccount (as Resource Name is Batch account that should be used in SDK rather than just Account)
Application_AddApplicationPackage -> Version_Create
Application_DeleteApplicationPackage ->Version_Delete
Application_ActivateApplicationPackage ->   Version_Activate
Application_GetApplicationPackage ->    Version_Get
GetApplicationPackageResult & AddApplicationPackageResult ->Both should be Version
AddApplicationParameters -> Application [Should be same as Application resource which we get with applications/{applicationName}/get]
AccountResource & BatchAccountCreateParameters -> BatchAccount [Should be named as per resource type]
/batchAccounts/{accountName}/applications/{id}/versions/{version}/activate -> batchAccounts/{accountName}/applications/{applicationName}/versions/{versionName}/activate -- ID in full resource is which starts from Subscriptions
batchAccounts/{accountName}/applications/{applicationId}    -> batchAccounts/{accountName}/applications/{applicationName}

This issue is quite old and we already went through this exercise with @anudeepsharma.
Responses inline:

SynchronizeAutoStorageKeys -> SyncAutoStorageKeys [As this is in Rest APIs, or change rest apis]
a. We don't feel the need to match exactly the REST API verbiage especially when it is abbreviated**
RegenerateKey -> RegenerateKeys [As this is in Rest APIs]
a. We don't feel the need to match exactly the REST API verbiage especially when it is abbreviated
Application_AddApplication -> Application_Create
a. This is fixed
Application_DeleteApplication -> Application_Delete
a. This is fixed
Application_GetApplication -> Application_Get
a. This is fixed
Application_UpdateApplication -> Application _Update
a. This is fixed
Subscription_GetSubscriptionQuotas -> Quota_Get
a. This was changed to Location_GetQuotas
Account -> BatchAccount (as Resource Name is Batch account that should be used in SDK rather than just Account)
a. This is fixed.
Application_AddApplicationPackage -> Version_Create
a. This is now called ApplicationPackage_Create
Application_DeleteApplicationPackage ->Version_Delete
a. This is now ApplicationPackage_Delete
Application_ActivateApplicationPackage -> Version_Activate
a. This is now ApplicationPackage_Activate
Application_GetApplicationPackage -> Version_Get
a. This is now ApplicationPackage_Get
GetApplicationPackageResult & AddApplicationPackageResult ->Both should be Version
a. Response is now ApplicationPackage
AddApplicationParameters -> Application [Should be same as Application resource which we get with applications/{applicationName}/get]
a. Not fixed, because the way the C# code generator would generate this is confusing (all properties would be settable via the constructor which implies get-only properties might be transited to the server)
AccountResource & BatchAccountCreateParameters -> BatchAccount [Should be named as per resource type]
a. AccountResource was changed to BatchAccount. BatchAccountCreateParameters was left as is for the same reasoning as 14 above.
/batchAccounts/{accountName}/applications/{id}/versions/{version}/activate -> batchAccounts/{accountName}/applications/{applicationName}/versions/{versionName}/activate -- ID in full resource is which starts from Subscriptions
a. Changed id to applicationId to clarify.
batchAccounts/{accountName}/applications/{applicationId} -> batchAccounts/{accountName}/applications/{applicationName}
a. Discussed and didn't change this because applicationId is not id

@kirthik I think we can close this, as all of the issues were discussed and either resolved or deferred.


dhcpcd fails to detect link-local IPv6 address if it has a 128-bit prefix

This bug was reported by a debian user here
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1019100
Report as following

Package: dhcpcd5
Version: 9.4.1-6
Severity: normal
Tags: ipv6
X-Debbugs-Cc<EMAIL_ADDRESS>Dear Maintainer,
I use dhcpcd on two systems acting as routers to obtain IPv6 prefixes from ISPs and assign them to LAN interfaces.
Both systems use pppd (2.4.9-1+1.1+b1) to create PPPoE connections. Pppd uses a 128-bit prefix length for the link-local address assigned to the PPPoE interface.
dhcpcd 9.4.1-6 hangs at "delaying DHCPv6 for LL address" indefinitely. 9.4.1-5 works fine.
After trying other things, I manually added a /64 (rather than /128) link-local address to the PPPoE interface. Dhcpcd picked this up immediately and proceeded with DHCPv6 prefix delegation.
-- System Information:
Debian Release: bookworm/sid
APT prefers unstable
APT policy: (500, 'unstable'), (500, 'testing')
merged-usr: no
Architecture: amd64 (x86_64)
Kernel: Linux 5.19.0-1-amd64 (SMP w/2 CPU threads; PREEMPT)
Locale: LANG=el_GR.UTF-8, LC_CTYPE=el_GR.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled
Versions of packages dhcpcd5 depends on:
ii  dhcpcd-base  9.4.1-6
ii  lsb-base     11.2
dhcpcd5 recommends no packages.
Versions of packages dhcpcd5 suggests:
pn  dhcpcd-gtk
pn  openresolv | resolvconf
-- no debconf information

Debian maintainer's insight

The only change between 9.4.1-5 and 9.4.1-6 indeed is IPv6-related:

Merge patch from upstream Git: Use netlink to workout address flags.

This patch was taken straight from upstream Git and will be included
in the upcoming 9.4.2 release. If that change has any adverse effect,
I suggest opening a bug upstream
(https://github.com/NetworkConfiguration/dhcpcd/issues) and linking
this Debian bug to it.

The mentioned patch is commit 064767e3610b7f8df974592f340dbbf45f8e44ca
I am not the reporter but I was hit by this issue as well. Per debian maintainer's suggestion, I repost the bug here.

I'm struggling to replicate this - with the latest dhcpcd-9 code base an ethernet interface works fine on Debian-11 with it has a /128 LL address.
What does the output of ip a look like on the machine with the error?

I am not the reporter but I was hit by this issue as well. Per debian maintainer's suggestion, I repost the bug here.

@lark Can you please post the output of the command as requested?

Here is the output of ip a for the PPPoE interface:
8: wan_nova: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1492 qdisc fq_codel state UNKNOWN group default qlen 3
link/ppp
inet 130.43.X.X peer <IP_ADDRESS>/32 scope global wan_nova
   valid_lft forever preferred_lft forever
inet6 fe80::a590:4cf5:1da1:5ab2 peer fe80::ae7a:56ff:fe12:3ef2/128 scope link
   valid_lft forever preferred_lft forever

@lark ping?

@rsmarples Did the above provide sufficient information?

see output of ip a on the pppoe link
8: ppp0: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1492 qdisc cake state UNKNOWN group default qlen 3
    link/ppp
    inet 114.x.x.x peer 114.x.x.x/32 scope global ppp0
       valid_lft forever preferred_lft forever
    inet6 fe80::414c:7433:5a2:364b peer fe80::727b:e8ff:fec2:c673/128 scope link
       valid_lft forever preferred_lft forever

This should now be fixed in the master branch. @lark can you test please?

I'm a (German) Telekom customer and thus "blessed" with PPPoE, too.
Archlinux dhcpcd 9.4.1:
Jun 11 06:42:00 hadante systemd[1]: Starting dhcpcd on ppp0...
Jun 11 06:42:00 hadante dhcpcd[690414]: dhcpcd-9.4.1 starting
Jun 11 06:42:00 hadante dhcpcd[690419]: DUID 00:01:03:04:22:1e:e3:3b
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: IAID 00:00:00:01
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: soliciting a DHCPv6 lease
Jun 11 06:42:00 hadante dhcpcd[690419]: int: activating for delegation
Jun 11 06:42:00 hadante dhcpcd[690419]: int: IAID cb:9c:62:c4
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: soliciting an IPv6 router
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: Router Advertisement from fe80::de38:e1ff:fe10:e21d
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: adding address 2003:ce:6fff:40e5:58e3:e642:d0a0:f30d/64
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: adding route to 2003:ce:6fff:40e5::/64
Jun 11 06:42:00 hadante dhcpcd[690419]: ppp0: adding default route via fe80::de38:e1ff:fe10:e21d
Jun 11 06:42:00 hadante systemd[1]: Started dhcpcd on ppp0.
Jun 11 06:42:02 hadante dhcpcd[690419]: ppp0: ADV 2003:ce:6f27:7300::/56 from fe80::de38:e1ff:fe10:e21d
Jun 11 06:42:02 hadante dhcpcd[690419]: ppp0: REPLY6 received from fe80::de38:e1ff:fe10:e21d
Jun 11 06:42:02 hadante dhcpcd[690419]: ppp0: renew in 43200, rebind in 69120, expire in 86400 seconds
Jun 11 06:42:02 hadante dhcpcd[690419]: lo: adding reject route to 2003:ce:6f27:7300::/56
Jun 11 06:42:02 hadante dhcpcd[690419]: ppp0: delegated prefix 2003:ce:6f27:7300::/56
Jun 11 06:42:02 hadante dhcpcd[690419]: int: adding address 2003:ce:6f27:7300::1/64
Jun 11 06:42:02 hadante dhcpcd[690419]: int: adding route to 2003:ce:6f27:7300::/64
Jun 11 07:40:08 hadante dhcpcd[690419]: lo: pid 1 deleted reject route to 2003:ce:6f27:7300::/56
Jun 11 07:46:11 hadante dhcpcd[690419]: lo: adding reject route to 2003:ce:6f27:7300::/56

Archlinux dhcpcd 10.0.1:
Jun 11 06:40:00 hadante systemd[1]: Starting dhcpcd on ppp0...
Jun 11 06:40:00 hadante dhcpcd[690023]: dhcpcd-10.0.1 starting
Jun 11 06:40:00 hadante dhcpcd[690027]: DUID 00:01:03:04:22:1e:e3:3b
Jun 11 06:40:00 hadante dhcpcd[690027]: if_learnaddrs: if_addrflags6: fe80::58e3:e642:d0a0:f30d%ppp0: No such process
Jun 11 06:40:00 hadante dhcpcd[690023]: if_learnaddrs: if_addrflags6: fe80::58e3:e642:d0a0:f30d%ppp0: No such process
Jun 11 06:40:00 hadante dhcpcd[690027]: ppp0: IAID 00:00:00:01
Jun 11 06:40:01 hadante dhcpcd[690027]: ppp0: soliciting an IPv6 router
Jun 11 06:40:30 hadante dhcpcd[690027]: timed out
Jun 11 06:40:30 hadante dhcpcd[690023]: timed out
Jun 11 06:40:30 hadante dhcpcd[690027]: main: control_stop: No such file or directory
Jun 11 06:40:30 hadante dhcpcd[690023]: main: control_stop: No such file or directory
Jun 11 06:40:30 hadante dhcpcd[690027]: dhcpcd exited
Jun 11 06:40:30 hadante dhcpcd[690023]: dhcpcd_fork_cb: truncated read 0 (expected 4)

Archlinux current dhcpcd master source=("git+https://github.com/rsmarples/dhcpcd.git"), from AUR:
Jun 11 19:56:00 hadante dhcpcd[924134]: dhcpcd-10.0.1 starting
Jun 11 19:56:00 hadante dhcpcd[924138]: DUID 00:01:03:04:22:1e:e3:3b
Jun 11 19:56:00 hadante dhcpcd[924138]: ppp0: IAID 00:00:00:01
Jun 11 19:56:00 hadante dhcpcd[924138]: ppp0: rebinding prior DHCPv6 lease
Jun 11 19:56:00 hadante dhcpcd[924138]: int: activating for delegation
Jun 11 19:56:00 hadante dhcpcd[924138]: int: IAID cb:9c:62:c4
Jun 11 19:56:01 hadante dhcpcd[924138]: ppp0: soliciting an IPv6 router
Jun 11 19:56:01 hadante dhcpcd[924138]: ppp0: Router Advertisement from fe80::de38:e1ff:fe10:e21d
Jun 11 19:56:01 hadante dhcpcd[924138]: ppp0: adding address 2003:ce:6fff:32d4:c9bf:d5cd:ed02:e105/64
Jun 11 19:56:01 hadante dhcpcd[924138]: ppp0: adding route to 2003:ce:6fff:32d4::/64
Jun 11 19:56:01 hadante dhcpcd[924138]: ppp0: adding default route via fe80::de38:e1ff:fe10:e21d
Jun 11 19:56:01 hadante systemd[1]: Started dhcpcd on ppp0.
Jun 11 19:56:10 hadante dhcpcd[924138]: ppp0: failed to rebind prior DHCPv6 delegation
Jun 11 19:56:13 hadante dhcpcd[924138]: ppp0: ADV 2003:ce:6f31:1a00::/56 from fe80::de38:e1ff:fe10:e21d
Jun 11 19:56:13 hadante dhcpcd[924138]: ppp0: REPLY6 received from fe80::de38:e1ff:fe10:e21d
Jun 11 19:56:13 hadante dhcpcd[924138]: ppp0: renew in 43200, rebind in 69120, expire in 86400 seconds
Jun 11 19:56:13 hadante dhcpcd[924138]: lo: adding reject route to 2003:ce:6f31:1a00::/56
Jun 11 19:56:13 hadante dhcpcd[924138]: ppp0: delegated prefix 2003:ce:6f31:1a00::/56
Jun 11 19:56:13 hadante dhcpcd[924138]: int: adding address 2003:ce:6f31:1a00::1/64
Jun 11 19:56:13 hadante dhcpcd[924138]: int: adding route to 2003:ce:6f31:1a00::/64

So for me the issue is fixed, HTH!


Req approvals

Pull Request Functional Test Report for #3635 / e1dc2c9cdd9f5a821af544c2f2c1c2ef932c5114
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community
4dac3ead
Log

Cisco 8000E

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community
542d3a75
Log

Cisco XRd

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community
9b148f84
Log

Juniper ncPTX

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community
9476ddaa
Log

Nokia SR Linux

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community
4129cd2b
Log

Openconfig Lemming

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community
c98e69ad
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community

Cisco 8808

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community

Juniper PTX10008

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community

Nokia 7250 IXR-10e

RT-7.5: BGP Policy - Match and Set Link Bandwidth Community

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

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

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls


Fix: logging dependencies for slf4j

Ref: https://github.com/datakaveri/iudx-aaa-server/pull/122

Can one of the admins verify this patch?

hey @pranavrd  can u also this fix stated here, in cleaning the pom along with the above  https://github.com/datakaveri/iudx-aaa-server/pull/124. Basically bom of log4j is used and no need to define seperate log4j versions . Also add log4j-api package

test this please


Update background bar default color

Update default to gray not blue.
https://github.cerner.com/Geneva/carbon-graphs/blob/master/docs/controls/Gantt.md#activities

Should be pretty straightforward change.
@laurunning Wont this impact current consumers who are using Gantt background bars with Default as Blue? I have marked this as breaking change for now.

I think the default should be gray. Are there any consumers taking this right now?

We don't have a list of all the consumers that use Carbon right now. I was hoping you would have that info 😞

Removing the label breaking since this is not a API change but a visual change approved by our standards.
@laurunning

To clarify are the changes only valid to the duration? What about task and activity?

@laurunning Just confirming, which of these do you want changed to gray by default: https://engineering.cerner.com/carbon-graphs/#/gantt/bar-types

I think we should hide the event lines from the other examples and make a separate example page for them so that they aren't competing
You mean the gray Datelines with Triangles? Do you want to make those Green with Triangles in the separate example we would create instead?

I am not sure I follow, can you clarify the following:

All the default values for Gantt - Tasks, Activities to be Gray color
What do you mean by I do think we should pick different colors for activities vs events?
Are we changing the terminology for Datelines to Event lines?
Would this be applicable for Bar graphs? Default is Blue as well.

Based on the UX sync meeting we will keep the default color as blue and not change it to gray. Converting to gray might make it appear as if it was disabled so we are moving away from this idea.


Show "Unlock profile" button if profile available

If you decide to store the keys in your local browser profile, it then shows you a popup to enter password and unlock your profile each time you open the web wallet. This is fine, but if you click "Continue without the profile" button, there is no way to Unlock it afterwards without reloading the page.
In place of the (pointless?) "Close wallet" button the "Unlock profile" button should be shown on the left.
 Then the "Close wallet" is also shown which does not make sense.

I don't understand this part:

The same issue occurs, if you decide to unlock your profile and then lock it back using the "Lock profile" button. Then the "Close wallet" is also shown which does not make sense.

After using Lock profile there's Unlock, and after that Lock profile again.

I don't understand this part:

The same issue occurs, if you decide to unlock your profile and then lock it back using the "Lock profile" button. Then the "Close wallet" is also shown which does not make sense.

After using Lock profile there's Unlock, and after that Lock profile again.

I'm getting Close wallet after using Lock profile. See below.


Get all features endpoint

GET endpoint to retrieve all the features

Two comments:
1 - Do we need to name the endpoint "get-all", can't we just name it get?
2 - There are some unused imports, let's create a separate PR to remove them later.
Otherwise, the code looks good to me.

@pthomas76

Yeah, we need to name it get-all, because we will have a separate endpoint to get only one feature.
Sounds good, will do once completed with endpoints


v0.8: Prepare for the next release

Changes:

crossbeam-channel 0.5.6 -> 0.5.7

Improve handling of very large timeout. (#953)

crossbeam-deque 0.8.2 -> 0.8.3

Add Stealer::{steal_batch_with_limit, steal_batch_with_limit_and_pop} methods. (#903)
Add Injector::{steal_batch_with_limit, steal_batch_with_limit_and_pop} methods. (#903)

crossbeam-epoch 0.9.13 -> 0.9.14

Update memoffset to 0.8. (#955)

crossbeam-utils 0.8.14 -> 0.8.15

Add #[clippy::has_significant_drop] to ShardedLock{Read,Write}Guard. (#958)
Improve handling of very large timeout. (#953)
Soft-deprecate thread::scope() in favor of the more efficient std::thread::scope that stabilized on Rust 1.63. (#954)

Backports:

https://github.com/crossbeam-rs/crossbeam/pull/903
https://github.com/crossbeam-rs/crossbeam/pull/936
https://github.com/crossbeam-rs/crossbeam/pull/953
https://github.com/crossbeam-rs/crossbeam/pull/954
https://github.com/crossbeam-rs/crossbeam/pull/955
https://github.com/crossbeam-rs/crossbeam/pull/958

bors r+

bors r+

bors r+