Tile background replaced with white in some cases

Context
Caching a mostly transparent service from Arcgis, the background of some tiles gets replaced with white.
Custom grid and source bbox are given as  [1787873, 6058795, 1796055, 6068937], which matches the arcgis extent given. I could observe the behavior both with mapproxy-seed as well as on-demand tile seeding.
Image returned by arcgis (sorry for gimp screenshot, but otherwise transparency would not be visible of course).

Request for this image is: http://.../arcgis/rest/services/[...]/MapServer/export?transparent=True&format=png&bbox=1795726.904656302,6060280.506546704,1796055.0,6060777.347230557&size=275,416&bboxSR=3857&imageSR=3857&f=image
Mapproxy makes this out of it (i.e. everything white):

The tile to the left looks fine though, as we can see here on the demo service:

Expected Behavior
Whole background should remain transparent.
Actual Behavior
Part of the background, mainly on the right and lower part, are replaced with white. This seems to affect tiles that are partially outside of the extent of the map itself.
Context
Stops me from shipping this map ...
Your Environment

Version used: Both Mapproxy 1.14.1 and 1.16 show this bug
Environment name and version (e.g. Python 2.7.5 with mod_wsgi 4.5.9): Python 3.9
Operating System and version: OpenSuse 15.3

We have had a similar problem since switching from 1.13.0 to 1.15.1:
Our service with the following configuration:
#-------------------------------------------------------------
#     Caches
#-------------------------------------------------------------

caches:
  webatlasde_bebb_halbton_compressed:
    format: image/png
    image:
      mode: P
      colors: 256
      opacity: 1.0
      transparent: true
      encoding_options:
      # The algorithm used to quantize (reduce) the image colors
        quantizer: fastoctree
    grids: [grid_webatlas_25833_512]
    sources: [webatlas_bebb_wms_farbe]
    link_single_color_images: true
    minimize_meta_requests: true

  topplus_farbe_compressed:
    format: image/png
    image:
      mode: P
      colors: 256
      opacity: 0.2
      transparent: true
      encoding_options:
        quantizer: fastoctree
    grids: [grid_topplus_25833]
    sources: [topplus_open_farbe]
    link_single_color_images: true
    minimize_meta_requests: true

  topplus_farbe_opak:
    disable_storage: true
    format: mixed
    request_format: image/png
    grids: [grid_topplus_25833]
    sources: [topplus_farbe]

  webatlasde_bebb_halbton:
    grids: [grid_webatlas_25833_256]
    sources: [webatlas_bebb_wms_farbe]
    link_single_color_images: true
    minimize_meta_requests: false
    image:
      mode: RGBA
      colors: 0
      transparent: true

  topplus_farbe:
    format: mixed
    request_format: image/png
    grids: [grid_topplus_25833]
    sources: [topplus_open_farbe]
    link_single_color_images: true
    minimize_meta_requests: true
    image:
      mode: RGBA
      colors: 0
      transparent: true
      encoding_options:
        jpeg_quality: 85
      opacity: 0.2

  webatlasde_bebb_grau:
    grids: [grid_webatlas_25833_256]
    sources: [webatlas_bebb_wms_grau]
    link_single_color_images: true
    minimize_meta_requests: false
    image:
      mode: RGBA
      colors: 0
      transparent: true

  # Topplus Graustufen Cache ist nicht zusaetzlich physisch abgelegt, vorhandener topplus_farbe ist eingefaerbt
  topplus_grau:
    disable_storage: true
    format: mixed
    request_format: image/png
    grids: [grid_topplus_25833]
    link_single_color_images: true
    minimize_meta_requests: true
    image:
      mode: RGBA
      colors: 0
      transparent: true
      opacity: 0.2
    sources:
      l: [{source: topplus_farbe, band: 0, factor: 0.21},
          {source: topplus_farbe, band: 1, factor: 0.72},
          {source: topplus_farbe, band: 2, factor: 0.07}
      ]

#-------------------------------------------------------------
#     Sources
#-------------------------------------------------------------

sources:
  webatlas_bebb_wms_farbe:
    type: wms
    seed_only: true
    concurrent_requests: 16
    wms_opts:
      version: 1.3.0
      featureinfo: false
      legendurl: https://isk.lgb.lvnbb.de/ows/legends/Legende_WebAtlasDE_halbton.png
    image:
      format: image/png
      opacity: 1.0
    req:
      url: https://aaa-dienste-test.lgb.local/aaa-suite/cgi-bin/webatlas/wms?
      layers: WebAtlasDE
      styles: palette_rgb_halbton
      transparent: true

  webatlas_bebb_wms_grau:
    type: wms
    seed_only: true
    concurrent_requests: 16
    wms_opts:
      version: 1.3.0
      featureinfo: false
      legendurl: https://isk.lgb.lvnbb.de/ows/legends/Legende_WebAtlasDE_grau.png
    image:
      format: image/png
      opacity: 1.0
    req:
      url: https://aaa-dienste-test.lgb.local/aaa-suite/cgi-bin/webatlas/wms?
      layers: WebAtlasDE
      styles: palette_grau
      transparent: true

  topplus_open_farbe:
    type: wms
    seed_only: true
    concurrent_requests: 16
    wms_opts:
      version: 1.3.0
      featureinfo: false
      legendgraphic: true
    image:
      format: image/png
      opacity: 0.35
    req:
      url: https://sg.geodatenzentrum.de/wms_topplus_open
      layers: web_scale
      transparent: true

#-------------------------------------------------------------
#     Grids
#-------------------------------------------------------------

grids:
  default_grid:
    tile_size: [256, 256]
    origin: ul

  grid_webatlas_25833_256:
    base: default_grid
    srs: 'EPSG:25833'
    bbox_srs: 'EPSG:25833'
    bbox: [245149, 5685471, 500600, 5940137]
    res: [
       #Res                    # AdV-Level  LGB-Level  Scale (96 dpi)
       #4891.96981025128000000,     # 0            17.471.320.7508974
       2445.98490512564000000,      # 1      0      8.735.660.37544871
       1222.99245256282000000,      # 2      1      4.367.830.18772436
<PHONE_NUMBER>8141000000,      # 3      2      2.183.915.09386218
<PHONE_NUMBER>4070500000,      # 4      3      1.091.957.54693109
<PHONE_NUMBER>7035300000,      # 5      4<PHONE_NUMBER>65545
         76.43702828517630000,      # 6      5<PHONE_NUMBER>32772
         38.21851414258810000,      # 7      6<PHONE_NUMBER>66386
         28,                        #        7       100.000
         19.10925707129410000,      # 8      8        68.247.3466831931
         14,                        #        9        50.000
          9.55462853564703000,      # 9      10       34.123.6733415964
          7,                        #        11       25.000
          4.77731426782352000,      # 10     12       17.061.8366707983
          2.8,                      #        13       10.000
          2.38865713391176000,      # 11     14        8.530.91833539913
          1.96,                     #        15        7.000
          1.4,                      #        16        5.000
          1.19432856695588000,      # 12     17        4.265.45916769957
          0.84,                     #        18        3.000
          0.59716428347793900,      # 13     19        2.132.72958384978
          0.42,                     #        20        1.500
          0.28,                     #        21        1.000
          0.14                      #        22         500
    ]

  grid_webatlas_25833_512:
    tile_size: [512, 512]
    origin: ul
    srs: 'EPSG:25833'
    bbox_srs: 'EPSG:25833'
    bbox: [245149, 5685471, 500600, 5940137]
    res: [
       # Res                   # AdV-Level  LGB-Level  Scale (96 dpi)
       # 4891.96981025128000000,    # 0            17471320.7508974
       2445.98490512564000000,      # 1      0      8735660.37544871
       1222.99245256282000000,      # 2      1      4367830.18772436
<PHONE_NUMBER>8141000000,      # 3      2      2183915.09386218
<PHONE_NUMBER>4070500000,      # 4      3      1091957.54693109
<PHONE_NUMBER>7035300000,      # 5      4<PHONE_NUMBER>65545
         76.43702828517630000,      # 6      5<PHONE_NUMBER>32772
         38.21851414258810000,      # 7      6<PHONE_NUMBER>66386
         28,                        #        7       100000
         19.10925707129410000,      # 8      8        68247.3466831931
         14,                        #        9        50000
          9.55462853564703000,      # 9      10       34123.6733415964
          7,                        #        11       25000
          4.77731426782352000,      # 10     12       17061.8366707983
          2.8,                      #        13       10000
          2.38865713391176000,      # 11     14        8530.91833539913
          1.96,                     #        15        7000
          1.4,                      #        16        5000
          1.19432856695588000,      # 12     17        4265.45916769957
          0.84,                     #        18        3000
          0.59716428347793900,      # 13     19        2132.72958384978
          0.42,                     #        20        1500
          0.28,                     #        21        1000
          0.14                      #        22         500
          ]

  grid_topplus_25833:
    tile_size: [512, 512]
    origin: ul
    srs: 'EPSG:25833'
    bbox_srs: 'EPSG:25833'
    bbox: [-392421, 5099820, 835247, 6286040]
    res: [
    #   Res                   # AdV-Level  LGB-Level  Scale (96 dpi)
    #  4891.96981025128000000,      # 0             17471320.7508974
       2445.98490512564000000,      # 1      0      8.735.660.37544871
       1222.99245256282000000,      # 2      1      4.367.830.18772436
<PHONE_NUMBER>8141000000,      # 3      2      2.183.915.09386218
<PHONE_NUMBER>4070500000,      # 4      3      1.091.957.54693109
<PHONE_NUMBER>7035300000,      # 5      4<PHONE_NUMBER>65545
         76.43702828517630000,      # 6      5<PHONE_NUMBER>32772
         38.21851414258810000,      # 7      6<PHONE_NUMBER>66386
         28,                        #        7       100.000
         19.10925707129410000,      # 8      8        68247.3466831931
         14,                        #        9        50000
          9.55462853564703000,      # 9      10       34123.6733415964
          7,                        #        11       25000
          4.77731426782352000,      # 10     12       17061.8366707983
          2.8                        #        13       10000
    ]

#-------------------------------------------------------------
#     Globals
#-------------------------------------------------------------

globals:
  srs:
    proj_data_dir: '/etc/mapproxy/conf/epsg'
    axis_order_ne: ['EPSG:4839', 'EPSG:3045', 'EPSG:3035', 'EPSG:3034']
    axis_order_en: []
  cache:
    concurrent_tile_creators: 8
    meta_size: [8, 8]
    meta_buffer: 340
  http:
    client_timeout: 1000

shows the following behavior with the different versions:
v1.13.0

v1.15.1

Here is the request (v1.15.1)
https://isk.geobasis-bb.de/mapproxy/webatlasde_topplus/service/wms?SERVICE=WMS&VERSION=1.1.1&REQUEST=GetMap&FORMAT=image%2Fpng&TRANSPARENT=false&CACHEID=4827514&LAYERS=topplus_farbe%2Cwebatlas_farbe&WIDTH=512&HEIGHT=512&SRS=EPSG%3A25833&STYLES=&BBOX=442800%2C5809000%2C657840%2C6024040
However, the problem does not occur at every zoom level.
Environment
Version used: Mapproxy 1.15.1 -> https://packages.debian.org/bookworm/mapproxy
Environment name and version: Python 3.11.2 & mod_wsgi 4.9.4
Operating System and version: Debian 12 (Bookworm)

For file caches this problem is fixed with #828

closed by #828


Add a default css class

Also a class by size, so you can define attributes by cluster size

Great idea, looks good.
Thank you for contributing!


App breaks on iOS when I import a

Any time I import other packages on iOS the app breaks with this error message
CONSOLE LOG: {NSVue (Vue: 2.6.12 | NSVue: 2.9.0)} -> RemoveChild(TabContentItem(16), Placeholder(17))
***** Fatal JavaScript exception - application has been terminated. *****
NativeScript encountered a fatal error: Uncaught TypeError: Cannot read property 'ios' of undefined
at
getViewController(file: app/webpack:/mysoulcenter/node_modules/@nativescript-community/ui-material-bottom-navigation/index.ios.js:431:0)
at (file: app/webpack:/mysoulcenter/node_modules/@nativescript-community/ui-material-bottom-navigation/index.ios.js:459:0)
at setViewControllers(file: app/webpack:/mysoulcenter/node_modules/@nativescript-community/ui-material-bottom-navigation/index.ios.js:458:0)
at [tabStrip:setNative](file: app/webpack:/mysoulcenter/node_modules/@nativescript-community/ui-material-bottom-navigation/index.ios.js:606:0)
at applyAllNativeSetters(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/properties/index.js:1136:0)
at initNativeView(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/properties/index.js:1079:0)
at onResumeNativeUpdates(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:737:22)
at _resumeNativeUpdates(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:280:0)
at onLoaded(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:235:0)
at onLoaded(file: app/webpack:/mysoulcenter/node_modules/@nativescript-community/ui-material-bottom-navigation/index.ios.js:258:0)
at (file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:318:0)
at callFunctionWithSuper(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:312:0)
at callLoaded(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:318:0)
at loadView(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:458:0)
at (file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view-base/index.js:237:0)
at eachChildView(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/content-view/index.js:65:0)
at eachChildView(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/page/page-common.js:106:0)
at eachChild(file: app/webpack:/mysoulcenter/node_modules/@nativescript/core/ui/core/view/view-common.js:759:0)
at onLoaded(file:///app/vendor.js<…>

@ovaltechnologiesinc you need to give a little bit more details like versions and so on...


Fix #276

This fix will remove TypeError: Cannot use 'in' operator to search for '__testdeck_slow' in undefined error.
Like other hooks, theTestUI.getSettings() call should get settings from itself.

@leejh3224
Nice.
Perhaps move the test to a separate describe, see
    describe("regression #248: getters and setters are invoked during initialization of the suite", function() {

where we already have one of these regression tests.
Sorry for not making this clear in the first place. I thought that you would have figured it out by yourself.
But you can leave the test as is and I will move it into the correct place after the merge.
Your call.

@silkentrance
I'm not sure about the test convention for this repository. So it would be better for you to move things around and find best place to put those tests by yourself.
Why not merge this as-is for now?
Thanks again for your quick response.


Support subjectAltName for Chrome

@CynthiaRutledge Great work once again! Do you have Discord or some social media? I'd love to get in touch with you :)


sim-se: syscall emulation of newfstatat returns invalid data in stat struct

Describe the bug
The syscall emulation of newfstatat, implemented in newfstatatFunc() in src/sim/syscall_emul.hh, treats the output stat buffer as the old the old OS::tgt_stat struct, not the new OS::tgt_stat64 struct as it should. Here's a program to demonstrate the bug:
#include <unistd.h>
#include <stdio.h>
#include <sys/stat.h>
#include <syscall.h>
#include <fcntl.h>
#include <stddef.h>

#define PRINT(name, type)			\
  printf(#name "=" type "\n", st.st_##name)

int main(int argc, char *argv[]) {
  struct stat st;
  if (stat(argv[0], &st) < 0) {
    perror("stat");
    return 1;
  }
  PRINT(dev, "%lu");
  PRINT(ino, "%lu");
  PRINT(mode, "0x%x");
  PRINT(nlink, "%lu");
  PRINT(uid, "%u");
  PRINT(gid, "%u");
  PRINT(rdev, "%lu");
  PRINT(size, "%zu");
  PRINT(blksize, "%zu");
  PRINT(blocks, "%zu");
}

Running this program on the host and in gem5 SE should print the same output; however, the output of the gem5 SE run indicates that the stat buffer is corrupt (e.g., on my machine, the mode is 0 when it should be 0x81ed).
Affects version
develop @ 48a40cf
gem5 Modifications
None
To Reproduce

Compile the attached C program,
stat.c.zip: cc -static -o stat stat.c
Compile gem5: scons build/X86/gem5.opt
Run on the host to see expected output: ./stat.
Run in gem5 SE to see buggy output: ./build/X86/gem5.opt configs/deprecated/example/se.py --cmd=./stat

Expected behavior
Both the host and gem5 SE runs should produce (roughly) identical output; the st.st_mode field under gem5 SE should definitely not be zero.
Host Operating System
Ubuntu 22.04
Host ISA
X86
Compiler used
GCC 11.4.0
Additional information
None

Fixed by #283.


ADDED TO-DO List

TO-do list to organize, add and delete your completed task .Made use of JQuery and DOM manipulation @sandipan898
Please review and add hacktoberfest-accepted label to this request

Nice work, give a star to the repository


Cleanup hardcoded cipher suite codepoints in s_server

The hardcoded code points for TLSv1.3 cipher suites are used in the TLS
PSK server callback. However, they seem to have been refactored a while
ago to use tls13_aes128gcmsha256_id, so these defines are not necessary
within the s_server code anymore.

Checklist

ping

Travis red cross is not relevant.

Pushed. Thanks.


5.0 update of intro to MVC

Internal review URL

@wadepickett  please test this for show stopper bugs only. Keep in mind we'll fix  the following as soon as it merges:

@Rick-Anderson:
Adding-controller.md:     Section:  Add a controller:   VS tab:
Instruction:
"In Solution Explorer, right-click Controllers > Add > Controller"
However, the screenshot highlights "New Item..." menu instead.  Screenshot needs to be fixed.

@wadepickett  none of those are show stoppers. Fix them when you go through this series  later in the week. We need to get the 5.0 version out.

Thanks.  I respectfully disagree.  Those are incorrect instructions that can't be followed.   I'm am checking through the rest of it of course, times 3 IDE's.  I am ignoring all the issues in your doc that I am finding that are not completely wrong or blocking, or just don't work.  I am just sticking to the high level scope of blockers.

I've only updated the first 3, don't review beyond that.

hose are incorrect instructions that can't be followed.

Folks are following them now without reporting that. You'll have a chance to fix them soon.

@Rick-Anderson, I stuck just blockers, ignored any other problems,  for all 3 IDE's.  Let me know when you want me to look beyond tutorial 3.  I had also checked "adding-model.md" since you hand submitted it.

In order for scaffolding to run we need to make sure the audience has the following package added first:
Microsoft.VisualStudio.Web.CodeGenerators.Mvc" Version="5.0.0-rc.2.20516.1"

Can you add that as a suggest comment and I'll accept?

Since the moniker range above this 5.0 specific range also includes 5.0, the markup we ask the audience to follow is repeated twice, showing both the 3.x and the 5.0 version.

Good one, check that my last commit fixed that.

However, the screenshot highlights "New Item..." menu instead. Screenshot needs to be fixed, or the instruction. The screenshot following this one assumes "New Item.." was selected.

See https://github.com/dotnet/AspNetCore.Docs/tree/master/aspnetcore/tutorials/first-mvc-app/adding-controller/_static
@wadepickett  the images in aspnetcore\tutorials\first-mvc-app\adding-controller_static are correct, both of them. When you go through this you can figure out why they're not being used.

@Rick-Anderson, yes, fix the text so it is correct. and not mismatched and wrong instruction.

@Rick-Anderson, yes, fix the text so it is correct. and not mismatched and wrong instruction.

Did you look at the images? The text is correct, the images are correct. I don't see why the wrong images are rendering.

Let me know when you want me to look beyond tutorial 3.

You mean 4 right?

@Rick-Anderson "I don't see why the wrong images are rendering."
If you are not seeing the updated images in the browser, but you know they are in the build.  Dump the browser cache.  I have noticed Edge holding on to old images since the last Edge update even after refresh.

@Rick-Anderson "Let me know when you want me to look beyond tutorial 3.
You mean 4 right?"
Yes, by "beyond tutorial 3", I mean any tutorials you want me to review that come sequentially after 3.

@wadepickett  you reviewed Part 4, add a model to an ASP.NET Core MVC app which is beyond 3

I will create a new issue to track any broken instruction called out here that will not be fixed.   Given we are short on time, I will approve, but that is assuming you fix the missing package issue. Don't use the full preview version number of course since this is pretty much going out with the update.

In order for scaffolding to run we need to make sure the audience has the following package added first:
Microsoft.VisualStudio.Web.CodeGenerators.Mvc" Version="5.0.0-rc.2.20516.1"

According to @deepchoudhery  all the scaffolding issues are fixed in RTM 5.0
VS doesn't require adding packages before scaffolding.
I didn't hit that issue when I went through this, I did however, hit these two

Scaffolding.  That is great it is fixed in RTM 5.0.  It seemed weird to required the additional package after the install.

@Rick-Anderson:
Adding-controller.md: Section: Add a controller: VS tab:
Instruction:
"In Solution Explorer, right-click Controllers > Add > Controller"
However, the screenshot highlights "New Item..." menu instead. Screenshot needs to be fixed, or the instruction. The screenshot following this one assumes "New Item.." was selected.

Also, "MVC Controller Class" has been renamed in the latest to just MVC Controller- Empty hasn't it? The instruction and the screenshot for "Add New Item" looks like it needs to be updated, otherwise folks are looking for something that no longer exists:
This is what the updated screen would contain:

The images were correct. I sent you links to show that. For some reason it kept rendering the incorrect images. I made copies of each year old image and reference the copy image, now it's rendering correctly.  Not sure how it could cache an image more than a year old.


Validation fails for files with only ---

What happened?
Validating a file containing only --- will cause validation to fail.
❯ kubectl-validate has3hyphen.yaml

has3hyphen.yaml...ERROR
GVK cannot be empty
Error: validation failed

What did you expect to happen?
This file is essentially an empty file, and I expect the validation to succeed as well as a file containing only comments.
How can we reproduce it (as minimally and precisely as possible)?
has3hyphen.yaml
---

Validate the above file.
❯ kubectl-validate has3hyphen.yaml

has3hyphen.yaml...ERROR
GVK cannot be empty
Error: validation failed

Anything else we need to know?
No response
Kubernetes version

❯ kubectl version
Client Version: v1.28.3
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.2

I agree this is a bug. Lots of files by convention contain empty doccuments. We should ignore them
/label good-first-issue
/assign

Hi @alexzielenski, i would like to take a look at this if its ok?

/assign @ah8ad3


fix thick_fn upper bounds

Fixes #112

Codecov Report
Patch has no changes to coverable lines.

Files Changed
Coverage

...spex/sunxspex_fitting/photon_models_for_fitting.py
0.00%

:loudspeaker: Thoughts on this report? Let us know!.

pre-commit.ci autofix

pre-commit.ci autofix

@samaloney i ran autopep8 --aggressive --in-place on the photon models file but pre-commit.ci is saying it's still failing. maybe it means a different file is failing the check? not sure.
also added a changelog.
@KriSun95 can you look at this when you have a chance?

pre-commit.ci autofix


mention nearest neighbors in readme

Adding a reference to nearest neighbors in the readme would improve the discoverability of the crate.

updated in latest release


Add basic support for the EVM and hw-app-eth.

This enables retrieving an EVM-type public key hash from the app and signing a transaction, both via hw-app-eth. This should (once EIP155 support with chainID > 255 works with metamask) allow metamask to operate for the C chain with the Avalanche app.

Superseded by #44


Filtering Docker container without port

I'm using overlay networks for my backends on Carina (Docker Swarm), so there is no globally mapped port (by design) as I only want the backends exposed to Traefik. Setting the traefik.port label would do what I want (I think), but unfortunately this does not override the "no ports" filter. Could this be changed?

@mithrandi are your container exposing ports in its Dockerfile ?

There are no ports "exposed" by the container at all (either via the Dockerfile / image or when creating the container). If I expose a random fake port with --expose 12345 then that's enough to pass the first "filtering" check, and after that traefik.port takes effect.
At this point, I guess I'm not actually clear on what the expected relationship between Docker networks and EXPOSE / --expose is meant to be. I definitely don't want to publish the backend port; if I just expose it without ever creating any container links or using -P, it'll still be there in the container metadata for traefik to see so that should work, but I'm not sure if there's any other effects I need to worry about?

Reading further at https://docs.docker.com/engine/userguide/networking/work-with-networks/#linking-containers-in-user-defined-networks it seems like EXPOSEing the ports is in fact a sensible/expected thing, so I'm going to close this bug.


Pull Request review Workflow seems to be failing

Issue
Pull Request review Workflow seems to be failing because error: Build input file cannot be found: '/CustomDynamicFontTests/Sources/Mocks/Sourcery/AutoMockable.generated.swift'

Expected
When push, test should successfully build.
Steps to reproduce

Use the template
run sh make.sh
Push to a Github Repo.
Trigger the test workflow.

Note
This seems to be because the order of the lane build_and_test. I modified TestManager.rb to
# TestManager.rb
# frozen_string_literal: true

class TestManager
  def initialize(fastlane:, device:, output_directory:)
    @fastlane = fastlane
    @device = device
    @output_directory = output_directory
  end

  def test(scheme:, targets:)
    @fastlane.scan(
      scheme: scheme,
      device: @device,
      output_directory: @output_directory,
      code_coverage: true,
      result_bundle: true,
      only_testing: targets,
      fail_build: false
    )
  end
end

and
# fastfile

  desc 'Build and Test'
  lane :build_and_test do
    test_manager.test(
      scheme: Constants.TESTS_SCHEME,
      targets: [Constants.TESTS_TARGET, Constants.UI_TESTS_TARGET]
    )
  end

To make it work.

@suho @llleyelll any reason we should distinct build and test in TestManager?

@blyscuit I think we don't need build_for_testing anymore, in the past, some projects need to run tests with different targets. Btw, the root cause of this issue is related to the build_and_test lane and its order?

The issue is with when we use test_without_building: true, the file autogenerated is not created then it would fail. If you run the command locally, it works fine as long and autogenerated is already exited.

@blyscuit Currently, we need to run all tests targets at the same time to export the test result bundle for the xcov, so I prefer to update the TestManager


Fix 1.27 certified kubernetes image to correct file type

Issue #, if available:
Description of changes:
Fix 1.27 certified kubernetes image to correct file type
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


reproduce the results

Hi, since the panda-gym cannot set the random seed as gym, where you can use env.set(seed) to reproduce the results.
When I use env.reset(seed=10), if the random seed is the same, I get the same return value, such as desired_goal. But if I don't set the seed, I can't guarantee the consistency of the state of the subsequent env.resets. Is there a way to guarantee the consistency of the state of each subsequent reset just like gym?

closing In favour of #94


Remove load balancer IP from outputs if klipper is enabled

See https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/issues/221

Awesome @phaer! Merging to staging will give one quick test ASAP before releasing a patch version. You can do it yourself if you feel like it! :)


snippet fix: need 'ref' passing Utf8JsonReader

The existing snippet code is obviously incorrect (since Utf8JsonReader is a ref struct), but in the example as shown it doesn't matter because GetMoreBytesFromStream is never called, since the json string in the demo is less than 4096 bytes.
This PR would fix the C# version, but note that I did not test it.
I assume there is a similar error in the VB version of this code but I don't know how to fix it.
Please feel free to process this PR as desired without my further participation.

@tdykstra @eiriktsarpalis Can you review this?

Thank you @glenn-slayden!


[Map-Pack] The Wall Series: labeled as UT when it's for U1

https://unrealarchive.org/mappacks/unreal-tournament/single-player/T/the-wall-series_8f84bce1.html
Any way to move it from UT to Unreal mappacks section?

Perhaps there needs to be a slightly more nuanced approach with an indication that maps will
(1) run only in UnReal;
(2) run in both UnReal and UT;
(3) run only in UT.
There is the added complication with UnReal maps (that applies to the maps in this pack) that some will only run in specific versions - the maps in this pack won't run in 225 for example.
But, yes, you are right - this map pack is intended to run in UnReal as evidenced by the Readme - it is a 'bonus' that 3 of the 4 run in UT as well.

For simplicity, I've updated it to Unreal. Also closes #3244


Rule proposal: displayName disabled for stateless components

It would be useful to turn off the displayName rule for stateless components. While I want all components declared using a ES6 class syntax or createClass to have a name, there's instances when it's useful to anonymously wrap a component using a quick arrow function syntax. For instance:
function wrapCustomComponent(Component, extendedProps) {
    return (props) => {
        let mergedProps = Object.assign({}, extendedProps || {}, props.rowData);
        return <Component {...mergedProps} />;
    };
}

The component could be assigned to a variable to give it a name, but that requires creating extra variables just to hold it temporarily before it gets returned from a function.

The proper solution here is to use a named function instead of an arrow function. There's no need to use an arrow here - they're not just a cure-all for "short functions".

That's one school of thought, to name everything to improve debugging. But that also reduces readability. Part of why arrow functions and anonymous functions were created was to improve readability. In cases where names add little to no meaning, I would rather trade-off the slight advantage in debugging for improved readability.
Perhaps that wasn't the best example. An even shorter example might be:
return (props) =>
    <DataLoader {...dataProps}>
        <Component {...props}} />
    </DataLoader>;

Yes, this can be written as:
return function WrappedComponent(props) {
   return <DataLoader {...dataProps}>
              <Component {...props}} />
          </DataLoader>;
};

But that's less readable.
The difference is the same between the school of thought that you should never have anonymous functions and should always use names, even if they are arbitrary, such as using iteratee as the name of functions passed to map/reduce, and the school of thought that says to make names have meaning and use anonymous functions instead of arbitrary named ones.
My proposal is simply to create a less opinionated linter rule that can apply to both schools of thought. For now, I've simply disabled the rule each time I use that idiom.

I find those examples much more readable than the arrows. A React component is very different from a HOF callback, and having a name is very important. Arrow functions were created to replace callbacks - not to replace normal functions.
If you insist on ignoring the decades of best practices that say to always name meaningful functions (like React components) then disabling the rule at the usage site (or overall) is the correct approach.


Update README.md

A clean code generator with smart migrations, table relations, multiple language support.

Please read here.

Sorry. Not enough stars: https://github.com/chiraggude/awesome-laravel/blob/master/CONTRIBUTING.md


Provide Requirements.txt

Could you please provide requirements file because the results are not reproducible and getting lots of error.

Please use this repo https://github.com/metabolize/chumpy/tree/python3-2 for chumpy installation on python3

In file setup.py file, make following changes:

from pip.req import parse_requirements
Changed to:
from pip._internal.req import parse_requirements

Run below command to install

python setup.py install

Please find below requirements file for used to run the project.
requirements.txt


Bulk fiscal event push

Don't merge this to develop branch yet. We will have to make the same changes to the adapter. Only then will we merge both the changes together to the develop branch.


Auto slack response

What's included?
Closes Auto Slack Response #54.

[x] Create a Slack App JACbot that can use in the Slack channels.
[x] Customise the response messages if the incoming messages meet some conditions.

If a message contains the keyword exercise and there is no exercise link, a response message will be sent in the thread once.

How to test

Install Slack App JACbot in the channel.
Type in some messages and check if the response message appears in the thread.

Development plan

Create a Slack App: https://api.slack.com/apps

Add credentials to Firebase
firebase functions:config:set slack.signing_secret=xxx
firebase functions:config:set slack.bot_token=xoxb-111-111-xxx

Set up oath & permissions

Bot Token Scopes: channels:history, chat:write, groups:history, im:history, incoming-webhook

Set up event subscriptions

Enable Events: On
Request URL: https://{your domain}.cloudfunctions.net/slack/events
Subscribe to bot events: message.channels, message.groups, message.im

Install the App to the channel in the workspace
https://github.com/jac-uk/digital-platform/assets/79906532/bd7c3187-4244-4bb7-bf0c-15236cb2b0e8

Create a callable function to listen to messages

Use @slack/bolt
Reply to message in the thread only once by checking thread_ts
Customise the response according to the message content

Go to the channel in Slack and add App

Need discussion

Since @slack/bolt requires at least version 12.13.0 of Node, could we upgrade the node version in functions/package.json? I couldn't find a better way to set up to deploy specific functions using specific node versions but I could deploy only this function manually using node 12+.
Set up Slack Apps for development and production so we could test it before going live.
Consider response messages in different scenarios.

@warrensearle Please can you catch up with Ryan on this

@warrensearle Please can you catch up with Ryan on this

Sure thing. @HalcyonJAC I have sent you a message, please let me know when you are free to catch up

With the new approach to ticketing, further work on this ticket is no longer required. Descoped.


Allow usage of googleauth ~> 0.6 in google-cloud-core

Would it be possible to allow google-cloud-core to use googleauth ~> 0.6?
This currently results in an unresolvable gemfile if there is a need to use > 2.0 of the jwt gem

Unfortunately it is not as easy as updating the dependency set in the google-cloud-core gem. We are matching the dependency set in the grpc and google-gax gems. So in order to update our dependency we will need new versions of those gems that also update the dependency.
I hope to see a grpc 1.7.0 and google-gax 0.9.0 release that update their dependency on googleauth.

This will be addressed by #1804.


desktop drop package new version pub.dev

Hi
In git package desktop_drop has version in pubspec with web: 1.0.0.  Can  you realease new version in pub.dev?
Thx

sorry for later.
https://pub.dev/packages/desktop_drop  published now.


Clean up Buffer

I've started looking into this and this turned out to be quite easy. Will file a PR once this has been fleshed out.

Postponing this to the next release in order to get the outstanding stream changes out.


Stuck at "attaching to airflow_init_1"

Thank you  for the help! I figured out on my own some issues and also cant find updated instructions on setting up the environments. Here is a screenshot of my issue. I also am getting that the username and password  for airflow is wrong. I also dont understand where and when to enter that. I would appreciate the help!

Hi @rab241  It looks like the postgres container is not running. Can you

tear down infra with ./tear_down_infra.sh {your-bucket-name} and then
Increase docker size as shown here https://github.com/apache/airflow/issues/14365#issuecomment-785267821, restart docker and ./setup_infra.sh {your-bucket-name}

If the issue still exists can you copy paste the logs ?

Thank you that seemed to work! However it is still stuck at attaching to airflow-init_1 and nothing seems to happen

@rab241 It takes a few min(1-2), can you lmk how long you waited ?

I waited around 10 minutes
Get Outlook for iOShttps://aka.ms/o0ukef

From: startdataengineering @.>
Sent: Wednesday, June 23, 2021 6:47:26 AM
To: josephmachado/beginner_de_project @.>
Cc: Biaz, Rim A @.>; Mention @.>
Subject: Re: [josephmachado/beginner_de_project] Stuck at "attaching to airflow_init_1" (#7)
@rab241https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frab241&data=04|01|rab241%40pitt.edu|b8a3087d35a74180380708d9363cac83|9ef9f489e0a04eeb87cc3a526112fd0d|1|0|637600456484817577|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=j3UWbPk828Vig2tt9Xnr%2FDq6luKLTd%2FaGUMOAxKDk0U%3D&reserved=0 It takes a few min(1-2), can you lmk how long you waited ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjosephmachado%2Fbeginner_de_project%2Fissues%2F7%23issuecomment-866767535&data=04|01|rab241%40pitt.edu|b8a3087d35a74180380708d9363cac83|9ef9f489e0a04eeb87cc3a526112fd0d|1|0|637600456484817577|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=r61bix3fJdrMqjLUZA1UFhCFT7%2F8a5AckfOyHYAcgAI%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUBEAP5QOULC55GVALBLE3TTUHCU5ANCNFSM47EWFZUQ&data=04|01|rab241%40pitt.edu|b8a3087d35a74180380708d9363cac83|9ef9f489e0a04eeb87cc3a526112fd0d|1|0|637600456484827578|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lMiED7dHok3%2B2uA8D19%2FDl%2BfQTviVGfAoNy5fxxegsQ%3D&reserved=0.

@rab241 I had this same issue when rerunning ./setup_infra.sh. Clearing the Docker cache and rerunning the script solved it for me: docker system prune --volumes
Alternatively, maybe adding the --force-recreate flag to the docker compose command will fix this but I haven't tried it out yet.

@josephmachado I was able to recreate the issue by running the ./setup_infra.sh script, canceling it after the image builds, and then re-running the script. Modifying docker compose up airflow-init --force-recreate with this flag fixed the issue without having to manually clear my docker cache. If this solves the problem, I can submit a PR if you'd like.

Hi, thanks for all the help! However, this is the issue that comes up when I ran the airflow recreate

Thank you. That would be amazing @dennisgsmith

@rab241 Think the Postgres container is still running(hence the 5432 port in use already), can you restart the docker container and try again using this https://github.com/josephmachado/beginner_de_project/issues/7#issuecomment-866790709

I had done that and redid it and the same 5432 port in use is still the error

@rab241 If you execute lsof -n -i:5432 | grep LISTEN you can see whatever is taking up that port.

So while I was setting up again, it got stuck again at attaching to airflow. I went into the docker and manually ran the different containers which started to run the airflow init and then reached to points where my  s3 bucket needed more permissions. I created the  permissions and  now it is looping on this error: An error occurred (ClusterNotFound) when calling the DescribeClusters operation: Cluster sde-batch-de-project not found.
it  keeps sleeping  for 60  seconds and checking

I cancelled and  tore down the infrastructure and this error occurred:  An error occurred (NoSuchEntity) when calling the DetachRolePolicy operation: The role with name sde-spectrum-redshift cannot be found.
Dissociating AWSGlueConsoleFullAccess policy from sde-spectrum-redshift role

with manually running airflow-init, and postgres this happens before what I just mentioned

It successfully set up  containers and airflow after completely trying again

Then I ran this and the airflow password worked but then I dont know what to do now
pgcli -h localhost -p 5432 -U airflow

I realized it is working now lol! Thanks for all your help!


Displaying license warnings when installing

For fonts that have non-open licenses, such as https://github.com/fontist/formulas/blob/master/Formulas/ms_truetype_fonts.rb , the user must accept the licenses.
When an installation of such Formula is encountered, we should prompt the user:
In order to install the {font name} fonts you must accept its licensing conditions
as shown below:
----------
... (font license content) ...
----------
To indicate your acceptance, please type "Yes". (Y/N?) Y

This is done, thanks @abunashir !


TWW Beta: Research/Implement Crafting Buffs

General

Blacksmithing

Everburning Forge (Baked into spec info..)

Maybe this is also possible to be mapped from db2 tables..


need support for optional prop types

currently, all the fields specified in a type are considered as required and 'isRequired' is added to each proptype generated, even though the field is optional.
Current Behaviour
The following type
type Props = {
  alignBottom?: boolean
  alignCenter: boolean
  alignTop: boolean
}

is generated as
{
  alignBottom: PropTypes.bool.isRequired,
  alignCenter: PropTypes.bool.isRequired,
  alignTop: PropTypes.bool.isRequired,
}

Expected Behaviour
The following type
type Props = {
  alignBottom?: boolean
  alignCenter: boolean
  alignTop: boolean
}

should be generated as
{
  alignBottom: PropTypes.bool,
  alignCenter: PropTypes.bool.isRequired,
  alignTop: PropTypes.bool.isRequired,
}

It already supports optional types, ref the fixture here https://github.com/merceyz/typescript-to-proptypes/blob/c53bfa8f5f90da13e3fc39ccd44dacee73c492e3/test/boolean-values/optional. I'm guessing you haven't enabled strict mode so typescript doesn't actually detect undefined

@merceyz In strict mode, it works perfect. You can now close this issue.


fix some typo with How_to_use_model_parallel_in_LiBai

fix some typo with How_to_use_model_parallel_in_LiBai
modified:   docs/source/notes/How_to_use_model_parallel_in_LiBai.md

#514  includ change


Establish API for defining macros

Macros in dry-validation 1.0.0 will be based on rule API, for a good start we'll add confirmation because it exists in 0.x:
API ideas
class NewUserContract < Dry::Validation::Contract
  schema do
    required(:email).filled(:string)
    required(:password).filled(:string)
    required(:password_confirmation).filled(:string)
  end

  confirmation :password
end

@solnic what kind of API do you have in mind beyond something like def self.confirmation in a base class?

@flash-gordon probably just that, yes. It’s only a matter of providing shortcuts to common rule types, which does boil down to contract class methods

Closing since there isn't anything special we need to do - these are just class methods, ie:
class MyApp::Contract < Dry::Validation::Contract
  def self.confirmation(name)
    rule(name) do
      key.failure("doesn't match confirmation") if values[name] != values[:"#{name}_confirmation"]
    end
  end
end


fixed G304

You also have to have an entry in the meson.build file.

Looks like the G304 is the same as the G305. We already have a device file for this ID, is it not showing in your machine?

it shows up as g305 rather than g304

The devices have the same ID, and seem the same. We can only have one name.


[sdl1] Build failure in x64-linux

When trying to install gdal on Linux (Ubuntu 16.04), there are some errors:
./vcpkg install gdal
The following packages will be built and installed:
  * curl[core,openssl,ssl]:x64-linux
  * expat[core]:x64-linux
  * freeglut[core]:x64-linux
    gdal[core]:x64-linux
  * geos[core]:x64-linux
  * giflib[core]:x64-linux
  * hdf5[core]:x64-linux
  * libiconv[core]:x64-linux
  * libjpeg-turbo[core]:x64-linux
  * liblzma[core]:x64-linux
  * libpng[core]:x64-linux
  * libpq[core]:x64-linux
  * libwebp[all,core]:x64-linux
  * libxml2[core]:x64-linux
  * netcdf-c[core]:x64-linux
  * opengl[core]:x64-linux
  * openjpeg[core]:x64-linux
  * openssl[core]:x64-linux
  * openssl-unix[core]:x64-linux
  * proj[core]:x64-linux
  * proj4[core]:x64-linux
  * sdl1[core]:x64-linux
  * sqlite3[core]:x64-linux
  * szip[core]:x64-linux
  * tiff[core]:x64-linux
  * zlib[core]:x64-linux
Additional packages (*) will be modified to complete this operation.
Starting package 1/26: sdl1:x64-linux
Building package sdl1[core]:x64-linux...
-- Note: sdl1 only supports dynamic library linkage. Building dynamic library.
-- Downloading https://github.com/SDL-Mirror/SDL/archive/release-1.2.15.tar.gz...
-- Extracting source /home/vruil/vcpkg/vcpkg/downloads/SDL-Mirror-SDL-release-1.2.15.tar.gz
-- Applying patch export-symbols-only-in-shared-build.patch
-- Using source at /home/vruil/vcpkg/vcpkg/buildtrees/sdl1/src/ase-1.2.15-2ca6426de6
-- Building VisualC/SDL1_2017.sln for Release
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
    Command failed: msbuild;/home/vruil/vcpkg/vcpkg/buildtrees/sdl1/x64-linux-rel/ase-1.2.15-2ca6426de6/VisualC/SDL1_2017.sln;/p:Configuration=Release;/t:Rebuild;/p:Platform=x64;/p:PlatformToolset=external;/p:VCPkgLocalAppDataDisabled=true;/p:UseIntelMKL=No;/p:WindowsTargetPlatformVersion=;/m
    Working Directory: /home/vruil/vcpkg/vcpkg/buildtrees/sdl1/x64-linux-rel/ase-1.2.15-2ca6426de6
    Error code: No such file or directory
    See logs for more information:

Call Stack (most recent call first):
  scripts/cmake/vcpkg_install_msbuild.cmake:157 (vcpkg_execute_required_process)
  ports/sdl1/portfile.cmake:23 (vcpkg_install_msbuild)
  scripts/ports.cmake:71 (include)

Error: Building package sdl1:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: sdl1:x64-linux
  Vcpkg version: 2018.11.23-unknownhash

Additionally, attach any relevant sections from the log files above.

also fails on macOS 10.14.3

linux ? project .sln
-- Building VisualC/SDL1_2017.sln for Release
failed msbuild, port windows

also fails on windows 10

@xianshunw For build failures in Windows, please see #7535.

PR merged. Close this issue.


姚苗苗-邸洁-张浩楠——全国主要城市第一产业产值研究

队员
姚苗苗，邸洁，张浩楠
作品名称
全国主要城市第一产业产值研究
作品网址
网站：https://qzgjt3td85.mobirisesite.com/
地图：https://miaomiaoyao2021.github.io/
其他说明
无

Hi all,
You three have a great idea to visualize the GDP of primary industry. However, it seems the work has not finished yet, right?

Hi all,
You three have a great idea to visualize the GDP of primary industry. However, it seems the work has not finished yet, right?

感谢老师提出的建议，我们已经根据您的要求进行完善和补充。


Show image in admin page

Hi, I saw issue #75, but on Django 1.10 with Python 3.4 it doesn't work. The issue is the same, image doesn't appears at the admin page, but is saved on the database.
What do I need to do?

What do you mean by "image doesn't appear on the admin page"? Why should it be?  Is issue can be reproduced at http://forms.viewflow.io/admin/? Do you have code examples to reproduce the issue?

kmmbvnr, not the image itself, but just the name of the image.
If I reopen my previously saved model, it will appear. But the first time I add an image to my model, it doesn't appear on the field.
Just added an image, doesn't appear:

Re-opened the model, it appears:

What's the type of your images field?

I'm closing this since no activity from OP.
The screenshots refer to the ManyToMany field default widget, while description blames about ImageField.
And can't reproduce the issue with a ManyToMany field on django 1.10
Ex: sites field for a flatpage - http://<IP_ADDRESS>:8000/admin/flatpages/flatpage/1/change/


Innocence Ex Fr update

This is a first try as I did not run this instance yet, so triggers may be incorrect.

Thanks for all the continued translation work.  Definitely let me know if there's anything I could do to make life easier for you.


Compiler fixes again #1308

This pr should fix the #1308 bug. A lot of the commits are just docs I wrote or some small refactorings. The bug appears to have been that it made a vreg the leader of another as long as they had the same register class. But they also need to have the same representation size.

Wow, awesome!!!
Merged!!!

@bjourne saw this test failure on macosx/x86.64
==== resource:basis/compiler/cfg/linear-scan/allocation/state/state-tests.factor

resource:basis/compiler/cfg/linear-scan/allocation/state/state-tests.factor: 112

Unit Test: {
    { t }
    [
        3 { } insns>cfg [ align-spill-area ] keep
        spill-area-align>> cell =
    ]
}

=== Expected:
t
=== Got:
f
[Traceback]

Oh I see. I fix I fix


Consider updating the redux-saga docs

Should we consider updating the redux-saga docs?

Forget it...

My plan is to update the docs in order to make Typescript a first class citizen as well as bring some modern ideas into the redux-saga world.  Closing this issue for now but thanks for posting!


Uncaught ReferemceError: __decorate is not defined

Hello,
I ran in some problems i'm trying to fix for the last hours but cant figure it out.
I hope someone here can help me out or confirm it is a bug/issue
here are the error's i'm getting:
https://s14.postimg.org/ggo9hmq1r/Screen_Shot_2016_10_10_at_11_29_04.png

I have the same problem

It's because the library has been built with the TypeScript "noEmitHelpers" flag.  To fix it, add the 'ts-helpers' library and set this flag to false in your own tsconfig.json.
See here for details:  https://github.com/ngParty/ts-helpers

@jinder thank you! Yes indeed it was.
I have been going back/forth on which is best.
@stefanfr @RedEars  I went ahead and published 2.0.9 with the emithelpers back on so pull the latest patch and see if that helps you get past that issue.

@NathanWalker  No, this problem still persists on version 2.0.9.
But with the fix from @jinder it actually works

I still can't get this to work, using angular-cli tried @jinder solution and using the latest commit.

Just saw this one after posting #20. I've downloaded the latest version (2.0.9) with no success.

I'll try to get this working this week (busy back after holidays), if anyone knows a fix, a PR would be greatly appreciated 👍

This issue is closed but persists in 2.0.9. Any updates? This info at the link @jinder provided doesn't appear to resolve the issue:

NOTE: Starting Typescript 2.1 this package won't be needed anymore
Instead you can provide --importHelpers together with --noEmitHelpers which will import emit helpers from Typescript "tslib"

I don't use this library any more, but it should be built with noEmitHelpers set as true, and when you import the library make sure you've included the tslib polyfill (included by default with an angular-cli project).

Thanks @jinder. That's what I've got. Now it complains that __extends is not defined instead.


Ethernet configuration questions.

I see how you can specify an IP address for grblHAL to the sender. There does not appear to be a mechanism to find the IP address. In grblHAL there are extensions to set all the network parameters but I don't see them in TSender.  Do they become available under some circumstance? How does one make the initial connection when grblHAL is configured to use an Ethernet protocol. I am thinking that maybe an initial connection could be made via USB to configure?  Is it possible to connect via USB and then switch to an Ethernet protocol?
I've looked at other motion controllers and they seem user unfriendly. Lots of comments on Ethernet config problems on CNCZone. Some involve directly connecting the PC to the controller via an Ethernet cable. Then changing the IP address of the PC to match the default IP address on the controller to connect and change the controller to a local router IP address. A number of them suggest a second Ethernet NIC and to stay directly connect to the controller. Given that most machinists are pretty far from network gurus, I think that is a poor approach.  Having USB can make it pretty simple to set up.
Anyway, I think making this super easy for a user would be a huge win for grblHAL.

In grblHAL there are extensions to set all the network parameters but I don't see them in TSender.

They are available in the Settings: Grbl tab.

Do they become available under some circumstance?

When the networking plugin is added to the build.

I am thinking that maybe an initial connection could be made via USB to configure?

grblHAL always starts up in serial communication mode (UART or USB) and switches protocol on connect/disconnect.

I am thinking that maybe an initial connection could be made via USB to configure? Is it possible to connect via USB and then switch to an Ethernet protocol?

Switching is automatic and happens on connect.

I have not yet added a way to input the connection parameters in tsender, the config file has to be modified manually. Another todo item.

A number of them suggest a second Ethernet NIC and to stay directly connect to the controller.

I have my SmoothStepper controller for Mach3 connected that way, it ensures no other traffic on the cable and is the optimal configuration.

Thanks. Sometimes I am a bit slow but I think you are saying something similar to what I was thinking.  It makes sense.
To recap what I think you said: You connect to a newly flashed grblHAL system via USB with TSender. You enter the network params (which are visible because of the network plugin) and then disconnect TSender.  Then you should be able to ping the grblHAL system (assuming network connected) and with the right ip address, mask and gateway in Tsender, it should connect.
What defines a connection to grblHAL - physical presence of USB or an actual sender or terminal connecting?  i.e. Do I need to remove the USB cable to force it to switch? (er move the cable to just a power source?)
Also, I think this means that a static IP address is required as there is no discovery mechanism for a DHCP allocated IP addr.

What defines a connection to grblHAL - physical presence of USB or an actual sender or terminal connecting?

The actual sender connecting, when it disconnects communication falls back to serial. There is no need to remove the USB cable. In fact some of the reporting still appears on the serial output.

Do I need to remove the USB cable to force it to switch?

A reboot of the controller may be needed after networking parameters are changed.

Also, I think this means that a static IP address is required as there is no discovery mechanism for a DHCP allocated IP addr.

It is a configuration option - DHCP is supported. When enabled then the settings for IP address etc. are no longer relevant/available. From settings.h:
...
    // Normally used for Ethernet or WiFi Station
    Setting_Hostname = 300,
    Setting_IpMode = 301,
    Setting_IpAddress = 302,
    Setting_Gateway = 303,
    Setting_NetMask = 304,
    Setting_TelnetPort = 305,
    Setting_HttpPort = 306,
    Setting_WebSocketPort = 307,
...

typedef enum {
    IpMode_Static = 0,
    IpMode_DHCP,
    IpMode_AutoIP
} ip_mode_t;

Switching to/from MPG mode and SD card streaming is still possible when network comms is in use so IMO a pretty flexible solution. It is all achieved by swapping HAL function pointers...

Great! It seems you have thought it all through.
By the way, I understood that DHCP is possible. What I meant was that it's likely to be hard to use because you will have to  figure out the new IP addr when it changes (or not ever power down the grblHAL machine).

What I meant was that it's likely to be hard to use because you will have to figure out the new IP addr when it changes (or not ever power down the grblHAL machine).

Two (three) options:

Assign a unique hostname and use that to connect.
Bind the controller MAC address to a fixed IP address in the router. I've used this during testing.
Use SSDP?

Not sure about the last one, I have not yet added that protocol to the networking plugin.

Hi, now that we are making progress on the Teensy 4.1 Ethernet support, I am seeing somewhat different behavior than described above.  I'm using Beta-1, if that matters.
$302-$304 (IP Addr, gateway and mask) are not showing up in the Settings: Grbl tab when I am connected via websocket or telnet.
Also, with Ethernet running, I am unable to connect via USB at all.  Even with invoking from a cmd.exe window "GCode Sender.exe"-port COM23:115200,N,8,1 (com23 is the correct port).  Or changing App.config to portparams line:

COM23:115200,N,8,1

Even after power cycling the teensy. In each case it sits with:

Waiting for controller (COM23:115200,N,8,1)...

in the lower status bar.

$302-$304 (IP Addr, gateway and mask) are not showing up in the Settings: Grbl tab when I am connected via websocket or telnet.

Is this with DHCP enabled? If so that is intentional as these settings does not make sense then, see enet.c - ethernet_settings_report() function.

Also, with Ethernet running, I am unable to connect via USB at all.

Strange, it works for me. Are you sure you are using the latest driver.c?

Ah, that makes sense.  Is there are way to turn DHCP off from the sender or console?

Is there are way to turn DHCP off from the sender or console?

$301 can be used for that. There is an error in the value assignments in my sender: 0 = static, 1 = DHCP, 2 = AutoIp is the correct ones. I'll have to fix that. Some sensible values should be set as default when DHCP is enabled in the config file - more todo...
After changing to static mode settings should be reloaded to make $302-$304 show up. Another todo item is to somehow declare settings dependencies in the definition file used by senders in order to avoid that, my long term plan is to switch to xml or json to add such information (and grouping for settings too...).

Playing with it now.  Looking pretty good with some minor issues.
There seems to be a bug in dealing with $301.  Sometimes it doesn't update the value shown (after clicking on save) and a couple of times TSender crashed when saving (could be unrelated).
If I click on autoIP to get 0 for static, $302 - $304 aren't displayed.  I have to exit TSender and restart to get it to show.
I got $301 set to 0, entered a static IP addr and GW, Mask was there.  Saved.  Power cycled teensy. Restart with -port <static addr> and it works.  Nice.
One issue. if you enter an error IP addr or GW or Mask, it seems to brick the Teensy.  I'll report this in the grblHAL issues section.
A minor annoyance - Editing the octal fields (correct term?) xxx.xxx.xxx.xxx looks like it's just text. As such if you just start typing you run into the field width limit - need clear it out first.  It would be nice to have a dot or tab move to the next octal field.
Minor visual issue - the Value column in Settings: Grbl tab isn't wide enough to show all 4 fields and no way to resize columns.

I don't know if I'd call this a bug but in order to get App.config/PortParams to update when you change the IP address, you have to go into Settings: App and press Save Settings.  Would be nice to have any change that needs to be reflected in the PortParams field to be automatically done.

It would be nice to have a dot or tab move to the next octal field.

Well, this is not easy to code - I believe Windows does this with overlapping input fields looking like a single one. It was hard enough to get numerical fields working properly, this would add another layer of complexity. For a rainy week...? Or use a 3rd party library? I try to avoid that.

Would be nice to have any change that needs to be reflected in the PortParams field to be automatically done.

I need to think about this - I am not sure it is a good idea.
Anyway, more testing is needed.

I need to think about this - I am not sure it is a good idea.

Here is the sequence of events that currently needs to be done:

change ip/gw in Settings: Grbl
hard reset
start sender with -port <new ipaddr in specific protocol form> from a cmd window
go into Settings: App and press save
from then on they can run TSender without -port

Now, they can just start sender and it will connect via what ever protocol was selected.  Note, it has been my experience that the average windows user hates/fears running a cmd.exe window.
Do you agree it would be a lot more friendly to have this sequence?

change ip/gw in Settings: Grbl (app.config Params field gets updated) with a warning that they need to do a hard reset
hard reset
Start TSender without -port...

Maybe in #1, give them the option of making it the default.
As it currently works, there is no indication that network changes need a hard reset to become active.
And as long as I am at it, the only way to make a -port value "permanent" in app.config is to go into Settings: App and press Save Settings. (They could also edit app.config. Not sure that is desirable.) It would be nice to have that behavior noted/shown in the tab. Maybe show both the current default connection and the current active one with a button to update.

How likely is it that most users will use a static IP vs. DHCP? What kind of users will want to use a static IP? Will they typically set the static addresses in driver.h? What about other senders? more?
As for my sender the configuration UI has currently no knowledge of what the settings means, it is completely driven by a text file (hal_setting_codes_en_US.txt). I would like that any further UI-"intelligence" should be added there and not hardcoded in the sender. What I have on my todo list is to add IP connection options in the connection dialog, and possibly a menu item for bringing that up. So IMO a bit of thinking is required. I agree it needs to be improved, an a first step to get us going is perhaps a Wiki page? An optional wiki link could also be added to hal_setting_codes_en_US.txt and presented in the UI as well? This would be nice for other features as well, e.g. the new tool change funtionality.

I agree - a wiki entry is a good start. And I really like the idea of an optional wiki link. This would make it easy to solve problems and quickly provide support.
I would like to hear more about your IP connection dialog. It sounds like the right direction.
The problem with a dynamic IP is that it is, well, dynamic. Consider a machine with a dhcp ip address and the user doesn't use it for a week or two.  He powers up the machine and tries to connect.  If a different IP addr gets assigned, it won't connect to the sender. He will have to do something to fix it.  A static IP address eliminates this issue. How likely is it? That depends on what we tell them to do. I would prefer to tell my users to set a static IP address. I just worry that these really great features will be a source of frustration for the users. Networking always seems to cause problems for people. There are a number of complaining threads about setting up networking for various commercial motion controllers.
By the way, I do appreciate your points on this. It is a tricky area to get right.

The problem with a dynamic IP is that it is, well, dynamic.

It does not have to be. IMO it is a safer bet to connect with DHCP and then reserve the address in the router. If assigning a static IP and then someone connects with a device (a mobile phone?) to the network and get your static IP then what?
In my router it is a matter of checking a box and I get a static lease, nobody else can then get the assigned IP:

change ip/gw in Settings: Grbl (app.config Params field gets updated) with a warning that they need to do a hard reset

I have added notes in the hal_setting_codes_en_US.txt file about this. I have commited this to the subversion repo, you can find it in the Resources folder. Replace the one in your sender folder to see the changes.
I have commited an update for the T4 driver as well, network settings values are copied at stack init so that any later changes does not mess it up. ESC followed by CTRL+T will now hard reset the controller, currently available in all connections modes - but should perhaps be restricted to UART mode as connection will be lost for others. For USB I need to disconnect/reconnect the USB cable before I can connect again (Win7). This is perhaps a reason for the lack of a reset button on the T4?


adding cassandra utils

Adds

Cassandra utils, allowing for native client

Fixes

Typo in factory name

@jacoblee93 I think this is all the more we need,
I got stuck on
https://github.com/langchain-ai/langchainjs/blob/main/.github/contributing/INTEGRATIONS.md
is not really up-to-date perhaps?

Instead, you should add a separate entrypoint for your integration in libs/langchain-community/scripts/create-entrypoints.js within the entrypoints object

I'm not sure where I can add a doc page for the utils as there is no docs/core_docs/docs/integrations/utils directory
But this can be added later on - more useful to have utils/cassandra exposed sooner rather than later.

Haha whoops - thank you!

Thank you!


Update dependencies

TODO: update eper

https://github.com/basho/velvet/pull/19 This is another dep.

And this: https://github.com/basho/riak_cs_multibag/pull/27/files

...and maybe this one: https://github.com/basho/eper/tree/ku-fresh-upstream-0.92

Please review #1197 prior to this.

Updated ＼(^o^)／

All r_t passed :confetti_ball:

@borshop merge

For release note:
Update cuttlefish to 2.0.4, node_package to 2.0.3, riak-erlang-client to 2.1.1, lager to 2.1.1, lager_syslog to 2.1.1, eper to 0.92 (Basho patched),  cluster_info to 2.0.3, riak_repl_pb_api to 2.1.1 and riak_cs_multibag to 2.1.0.

For release note:
Update cuttlefish to 2.0.4, node_package to 2.0.3, riak-erlang-client to 2.1.1, lager to 2.1.1, lager_syslog to 2.1.1, eper to 0.92 (Basho patched), cluster_info to 2.0.3, riak_repl_pb_api to 2.1.1 and riak_cs_multibag to 2.1.0.
[posted via JIRA by Kota Uenishi]


Is there an easy way to integrate a title for each photo?

Hi Michael,
Is there an easy way to integrate a title for each photo, like in the facebook app? or is it something you are planning to add?
Another thing is that, when I added MWPhotoBrowser to my project, there were a few irregularities (I can't see those in your project but I couldn't figure out why they are in my app)

After single tap, the navigationBar disappears (as it should). but when it appears again, it appears with a gap of 20 px between the status bar the navBar
The status bar disappears but the space just becomes plain white.

Thanks and regards,
Ashish.

I'm also really interested on the photo caption feature.
Thanks for sharing this great piece of code!

That's awesome, Michael. Thanks.

That's awesome, Michael. Thanks.


Chore - rename 'incrementUjNavigation' to 'trackUJNavigation'

Remove enabledNavigationTracking as an init option and always use navigation tracking.
Renamed the incrementUjNavigation to trackUJNavigation for clarity.
Update readme to reflect the changes

Why advancedUJStep is more clear? What's the thinking behind.
My personal bias when I read advanced is more some sort of hard challenge, like an Advance Class.
Even when I google the word, I am not quite sure how it maps with the functionality we have.

Mistyped, it should be advanceUJStep, not advancedUJStep
advance meaning 'move forward'.
I think 'increment' relates more to mathematics adding, so I think advance fits more here.

Niiice! I love how we are thinking around making the API simpler to use.


Bedrock user login failure with Geyser MOTD passthrough

Players on bedrock clients are unable to login when Geyser's MOTD/protocol/playercount passthrough is enabled.
The login error can be viewed by clicking this.

I believe I'm having this same issue, logs.

This is a known issue, we currently don't support org.geysermc.geyser.platform.velocity.GeyserVelocityPingPassthrough$GeyserInboundConnection connections, thus the error occurs. The workaround at the moment is to simply not use passthrough pings or passthrough MOTDs.


Open next with SST on AWS - Missing Cloudfront Behaviors

Hi everyone, Greg here founder of https://cloudbuddies.io/, nice to meet you.
We were running for quite some months an SST stack on AWS with opennext hosting our Application (cloud self-service platform). All was fine until 21st of June. Yesterday I wanted to update the stack like I usually do and noticed after that, that my login function (nextauth api) is not working anymore and throws a 500 error.
I reviewed the cloudfront configuration and notice some changes in the behaviors compared to previous deployments.
For example it seems that the api/* route is not created anymore. Maybe others as well, still investigating. Maybe the lambda function running the node server has issues as well.
I also analysed the CF template from the stack created and don't find the behavior (path pattern) for the /api route neither.
Any support possible ?
You can try the login on the application here : https://archie-sandbox.cloudbuddies.io/
Available if further details are needed. Thanks for the support.
Also if we refer to the architecture diagram visible in the open next doc, we  see that the API route is sent to the lambda server function.

The api/* behavior is not needed, all the route will go to the default function. That is not your issue here.
@Archie-Cloudbuddies What version of Next, OpenNext and SST are you using ?
Have you looked at the cloudwatch logs ?

Hey @conico974 thanks for taking time to reply.
Oh interesting I thought it was needed as visible on the diagram here (https://open-next.js.org/inner_workings/architecture), but fine.
Here are the answers, this is shown when I do npx sst deploy

SST v2.43.3
Next.js version : 14.2.1
OpenNext v3.0.2

Yes I had some look in cloudwatch logs, I found this in the sitedefaultserverfunction log, but not sure if it's related
2024-07-04T13:25:04.393Z	98945a3a-7e27-43e1-88b2-95e8f9cea134	ERROR	[next-auth][error][NO_SECRET]
https://next-auth.js.org/errors#no_secret Please define a `secret` in production. r [MissingSecretError]: Please define a `secret` in production.

Also I have a nextauth secret in my env. file.

There was an issue in this version of OpenNext that didn't copy .env file. Updating OpenNext to the latest (i.e. 3.0.6 ) should be enough https://docs.sst.dev/constructs/NextjsSite#opennextversion.
You can close the issue if it did solve it

OK, will try and let you know or close the issue if fixed.

Looks like I have the latest open next install 3.06 (when i run a local build with yarn build)

npx open-next@latest build
Next.js version : 14.2.1
OpenNext v3.0.6

but when I do the sst deploy it seems that it is not using the latest version (as it also does a build before creating the infra)

npx sst deploy
Please enter a name you’d like to use for your personal stage. Or hit enter to use: archie-sandbox
SST v2.43.3
➜  App:     nextjs-home
Stage:   archie-sandbox
Region:  us-east-1
Account: 7xxxxxxxxx
Next.js version : 14.2.1
OpenNext v3.0.2

...any additional help u can provide ?

You need to set openNextVersion in the NextJsSite construct. Just like in the link i provided above
SST doesn't pick the version of OpenNext you installed ( i assume you installed it with npm/pnpm/yarn ), you need to provide it as i showed or change the buildCommand ( same thing it's in the NextJsSite construct )

Fixed by conico974 recommendation (opennextversion in construct)


Privacy error when publishing a public map

When they share a public map with the "Get link" option they get the next message: Looks like this map no longer exists or is set as private.
In the console log there is this message: Failed to load resource: the server responded with a status of 403 (Forbidden).
If they share the map with the "Embed it" option, the map is loaded correctly.
Steps:
1- Create map (public map)
2- Click Publish button.
3- Copy the url from "Get link" option and paste the url in a new tab
4- Instead of a map they have this message:

This is caused by the table name starting by an underscore: #5843


Audit how API routes work when users don't explicitly do anything

@Api
fun someRoute(ctx) {
   // Do nothing
}

right now, this sends an empty "200" response, the idea being that the user defined the endpoint, so that's enough. But I'm not sure if that's intuitive, or if instead we should require the user do something:
@Api
fun someRoute(ctx) {
   ctx.res.status = 200 // body automatically set to "" if not already set
}

or
@Api
fun someRoute(ctx) {
   ctx.res.body = "..." // status automatically set to 200 if not already set
}

This way, we can write code like:
@Api
fun someRoute(ctx) {
   if (ctx.req.method == GET) {
      ctx.res.body = ...
   }
}

and other methods will return a 400

One other option is that the @Api annotation could specify what routes it is expecting. This would be optional but encouraged behavior:
@Api
fun someRoute(ctx) {
   // Do nothing, returns 200 for all request types
}

@Api(methods = listOf(GET))
fun someRoute(ctx) {
   assert(cxtx.req.method == GET)
   // Do nothing, returns 200 on get, 400 on other request methods
}

Note that next.js behavior is to hang (and log a warning) if you do something like this:
export default function handler(req, res) {
//  do nothing
}

Output: API resolved without sending a response for /api/..., this may result in stalled requests.

And given their expertise, it would probably make sense to follow their lead.


Latest vulkan-sdk version (<IP_ADDRESS>)

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

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@TorrentialFire please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@microsoft-github-policy-service agree

And from the installer schema docs:

Note: The Windows Package Manager does not support any behavior related to dependencies.

I don't think the pipeline will pass unless the VC++Redist can be installed ahead of this package.

I have confirmed that it does correctly install when the dependencies are installed ahead of this manifest.

@stephengillie or @JohnMcPMS any ideas on a path forward for this?

@stephengillie or @JohnMcPMS any ideas on a path forward for this? (edit): Is there a way to tell the pipeline to run another package install before this one, or do we avoid adding manifests that would unexpectedly force user interaction during an otherwise silent install (if they are missing the required redistributable)?

For an installer to be included in Winget, it must have an unattended mode that requires no user interaction (beyond elevation and similar prompts). My understanding of Winget's operation is that it doesn't necessarily install dependencies before running the package's installer. I'm working to add an experimental feature to my testing VM to see if a future version might help mitigate this.

@stephengillie or @JohnMcPMS any ideas on a path forward for this? (edit): Is there a way to tell the pipeline to run another package install before this one, or do we avoid adding manifests that would unexpectedly force user interaction during an otherwise silent install (if they are missing the required redistributable)?

There is an experimental feature in winget currently to install package dependencies. However, because it is experimental, it has not been enabled on the validation pipelines. Until dependency support is in the stable release of winget, or the VCRedist is bundled as part of the application, this likely won't pass the pipelines
@msftbot dependencies

@msftbot dependencies

The new version installs dependencies before running the package installer, I believe. In my testing VM, this does occur, and the installer runs fully unattended. It should pass validation once the new version is in the pipelines.

https://github.com/microsoft/winget-cli/releases/tag/v1.4.3132-preview

@wingetbot waivers Add Validation-Unattended-Failed

I look forward to the dependency installation feature in future versions of winget. Thanks!


Use Wiredep

Remember my request at #75? I found a better solution to do this: grunt-wiredep!
Easy as a pie.
We can cleanup and automatize all Bower dependencies imports.
For example, in Jade:
templates/layouts/base.jade
// build:js(client) scripts/global.js
script(src='/bower_components/jquery/dist/jquery.js')
script(src='/bower_components/foundation/js/vendor/fastclick.js')
script(src='/bower_components/foundation/js/vendor/jquery.cookie.js')
script(src='/bower_components/foundation/js/vendor/placeholder.js')
script(src='/bower_components/foundation/js/foundation/foundation.js')
// endbuild

becomes
// build:js(client) scripts/global.js
// bower:js
// endbower
// endbuild

And after wiredep task it fills with all installed Bower components:
// bower:js
script(src='../../bower_components/modernizr/modernizr.js')
script(src='../../bower_components/jquery/dist/jquery.js')
script(src='../../bower_components/fastclick/lib/fastclick.js')
script(src='../../bower_components/jquery.cookie/jquery.cookie.js')
script(src='../../bower_components/jquery-placeholder/jquery.placeholder.js')
script(src='../../bower_components/foundation/js/foundation.js')
// endbower

using main section of every bower.json into bower_components.
We can specify cwd and other cool options for do it properly. For example, we can provide exclude list for things like foundation.css. It's complex but very smart.
And don't worry, the order of injecting uses package dependencies.
I am going to know it closer and will use right now.

This had now been added. Check it out in the new release


[Service Bus] Review Quick Start doc for Track 2 Service Bus

This issue is to track the review of the quick start docs that @spelluru has written for the latest v7 of the servicebus library.
Use either this issue or email to share feedback with @spelluru and close this issue once we are satisfied with the content.

Quick start for Queues
Quick start for Topics & Subscriptions

Hi @chradek - Please wait until tomorrow morning to review the quick start. I am going to test with the latest preview packages and update the quick starts today. Thanks!

Hi @chradek - The above two articles are ready for your review now. I tried to keep the quick start as simple as possible with the following in mind:

Use Async
Use batching
Use receiver handlers

And, provide a link to all the samples on GitHub.
I appreciate if you could review and provide your feedback as soon as you can.
Thanks,
Sreedhar

Quick note, it looks like the 1st link is pointing to Java docs, but I assume you want me to review the JavaScript docs located at https://review.docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-nodejs-how-to-use-queues?branch=release-service-bus-track2-sdk-ga

@chradek - yes. sorry about that. Please see the links below.
Queues quick start:
https://review.docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-nodejs-how-to-use-queues?branch=release-service-bus-track2-sdk-ga
Topics/subscriptions quick start:
https://review.docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-nodejs-how-to-use-topics-subscriptions?branch=release-service-bus-track2-sdk-ga

I created a PR for some minor edits to the docs and tagged @spelluru. Thanks for putting the changes together!


Add winmanifest import

I am admittedly not that familiar with golang, but building using the most recent MinGW64 + Go tools gave the following error:
The procedure entry point TaskDialog could not be located in the dynamic link library.

This appears to be fixed by adding an additional import, see: https://github.com/andlabs/ui/blob/master/README.md#windows-manifests

Hmm, that's interesting. I wonder if something has changed since the last time I've made a Windows build (admittedly, a while ago). Seems like a sensible workaround though :+1:


Is there a way to set the direction of the touch drag event?

Hello
I'm making a carousel that displays a blog list.
Currently, the problem with this is that the carousel moves along with the touch drag (scroll) while pressing the carousel component.
I found a way to block the touch slide through the Carousel's plug-in, and I'm going to use it when a scroll event is called.
I think I can't use scroll events and wheel events on mobile and I have to use touch move events.
So can I call the touch drag event only when I move it up/down (touch scroll)?
For example, can I use it like 'v-touch:drag.up' or 'v-touch:drag.down'?
Thank you.

+1


Show size of text next to the TextEdit

This would be a pre-requisite for future work on minifying inputs and optimizing SVGs. But long before work on that begins, it's going to just be a nice quality of life thing.

Already done in a WIP pull request that will take a while longer to finish.


Revisit need for spherical shell domains

Land surface models have a very large aspect ratio, so there is no real need for a spherical shell domain.
Currently, it may be nice to have for coupled simulations if the lower atmos boundary space and upper land boundary space need to match. In the long run, they do not need to match, so we may be able to remove the spherical shell domain for LSMs.
@LenkaNovak do the boundary spaces for land and atmos need to match for AMIP? either way is fine! we have the spherical shell domain ready to merge if it is needed.

@kmdeck We can do both, but if the boundary spaces are not the same, we would have to use remapping and (at the moment with serial regridding) that would make the high-res AMIP sim significantly slower, especially since we're coupling at every timestep. If the boundary spaces are the same, we can use the same MPI distribution for atmos and land. It's good to start thinking about different spaces though for experiments beyond this first AMIP version. :)

ok! That's good to know. It makes sense then for us to have this capability! I will merge the PR adding it in. then we should be ready to try the enhanced bucket model coupled with atmos!


[CARBONDATA-3562] Fix for SDK filter queries not working when schema is given explicitly while Add Segment

Problem : Queries will not return correct result from added segment when the schema is given explicitly in case of SDK.
Solution : Handled it by validating based on both column name and column id if it matches for the SDK column.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@ravipesala Please review.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/717/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/725/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/726/

@manishnalla1994 I think this fix might induce new issues in alter table , like drop column and add back the column with the same name might not work as expected.

@ravipesala when we add the same column then the column id will be different, so it wont match in our case.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/745/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/751/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/753/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/754/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/762/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/760/

LGTM

LGTM

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/784/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/790/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/792/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/789/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/797/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/795/

LGTM


"Truncated frame" when insersing data using Batch

I am creating a platform to upload data from files into Scylla tables using scylla-rust-driver. The data is loaded as serde_json::Values, which can't be uploaded directly, so it's encapsulated in a custom struct named DataValue that implements scylla::_macro_internal::Value.
This code is from this src/entities/data_value.rs file
use serde_json::Value as SerdeValue;
use scylla::_macro_internal::Value as ScylaValue;
...

pub struct DataValue(SerdeValue);

impl ScylaValue for DataValue {
    fn serialize(&self, buf: &mut Vec<u8>) -> Result<(), ValueTooBig> {
        let json_value = self.into();

        match json_value {
            SerdeValue::String(string_value) => {
                string_value.serialize(buf)?;
            }
            SerdeValue::Null => {
                let unset = Unset;
                unset.serialize(buf)?;
            },
            SerdeValue::Bool(bool_value) => {
                bool_value.serialize(buf)?;
            },
            SerdeValue::Number(value_number) => {
                if value_number.is_u64() {
                    let value = value_number.as_u64().unwrap() as i64;
                    value.serialize(buf)?;
                } else if value_number.is_i64() {
                    let value = value_number.as_i64().unwrap();
                    value.serialize(buf)?;
                } else if value_number.is_f64() {
                    let value = value_number.as_f64().unwrap();
                    value.serialize(buf)?;
                }
            },
            SerdeValue::Array(value_array) => {
                let data_value_array: Vec<_> = value_array.iter().map(|item| DataValue(item.clone())).collect();
                data_value_array.serialize(buf)?;
            },
            SerdeValue::Object(value_map) => {
                let hash_map: HashMap<String, DataValue> = value_map.iter().map(|(key, value)| {
                    let data_value = DataValue(value.to_owned());
                    (key.to_owned(), data_value)
                }).collect();

                hash_map.serialize(buf)?;
            },
        };

        Ok(())
    }
}

I used a prepared statement to insert data into the database, when inserting line by line, everything works fine (but it takes really a lot of time to upload the whole data).
pub async fn insert_batch(&self, batch: Vec<serde_json::Value>) -> anyhow::Result<()> {
        if self.prepared_statement.borrow().is_none() {
            self.update_prepared_statement_and_field_names(&batch).await?;
        }

        let preapared_statement = self.prepared_statement.borrow().clone().unwrap();

        for serde_values in batch.iter() {
            let values: HashMap<String, DataValue> = DataValue::new(serde_values.to_owned()).into();
            self.session.execute(&preapared_statement, &values).await?;
        }

        Ok(())
    }

To improve performance, I prefer using a Batch, repeating the prepared statement all for each value then running all at once:
pub async fn insert_batch(&self, batch: Vec<serde_json::Value>) -> anyhow::Result<()> {
        if self.prepared_statement.borrow().is_none() {
            self.update_prepared_statement_and_field_names(&batch).await?;
        }

        let mut command_batch = Batch::default();

        let preapared_statement = self.prepared_statement.borrow().clone().unwrap();
        let mut values_batch: Vec<HashMap<String, DataValue>> = Vec::new();

        for serde_values in batch.iter() {
            command_batch.append_statement(preapared_statement.clone());
            let values: HashMap<String, DataValue> = DataValue::new(serde_values.to_owned()).into();
            values_batch.push(values);
        }

        self.session.batch(&command_batch, &values_batch[..]).await?;

        Ok(())
    }

That results on the following error:
Error: Database returned an error: Invalid protocol message received from the driver, Error message: truncated frame: expected 553057 bytes, length is 1115

Is that a mistake on my side or a bug in the code? Is there a solution or an alternative?
Thanks in advance.

Could you describe the problem you are trying to solve with your custom implementation of Value? Scylla expects that the value is serialized in the format corresponding to the column value, e.g. you should not serialize a map into a text column or a string into a map<_, _> column. This might accidentally work but it won't produce correct results because the driver does not validate the types. We started working on it in https://github.com/scylladb/scylla-rust-driver/issues/463 which will improve the situation.

Recently the serialization API was refactored and is now more type-safe, such issues should be harder to produce.
Due to new API (which made code provided in the issue obsolete) and the lack of response closing for now - feel free to open new issue (or reopen that one, not sure if you have permissions) if the problem persists.


Add Owners as Maintainers for SCITT-API-Emulator repo

https://github.com/scitt-community/scitt-api-emulator/blob/main/OWNERS to be added as maintenance members for the repo. I have a PR out for traceability
https://github.com/kkarunakaran89/scitt-api-emulator/pull/3

This is a tracing item to complete the configuration of the OWNERS added to the GitHub Team: scitt-api-emulator-maintainers
This is now complete